Offer option to influence stories or rules based on custom condition

What problem are you trying to solve?
Hi folks,
based on a discussion with @dakshvar22 I would like to discuss an issue that has been bothering me for some time. If you currently want to influence the course of a storyor a rule, you can achieve this in the case of stories for example with intents, slot events and custom actions - based on which defined follow-up actions are then executed, in the case of rules you still have the possibility to specify conditions in the case of which a rule may become active.
I have now tried to influence story progression based on detected emotions, as I believe this will play a role in conversation progressions in the future.
I'd like to be able to define e.g. a story like this:
- story: joy path
  steps:
    - intent: mention_purpose
    - slot_was_set:
      - emotion:
        - joy
        - shame
    - action: utter_something_politely

I have evaluated the following options:

creating an entity and a slot with the same name
conditional response variants based on a slot value
an NLU GraphComponent that classifies the tracker's last message for emotions at each iteration
a Policy GraphComponent, which classifies the last message of the tracker for emotions at each iteration

The extraction of emotions is performed with a multi-label approach, i.e. a user input can contain up to n emotions, where n was fixed by me to 30.
Variant 1 was implemented in such a way that the NLU GraphComponent fills the entity in the process method at each iteration and by the deliberately used slot-auto-filling option the slot was also filled.
The problem here is that both the entity and the slot actually contain a list of emotions, and a story/rule where the slot value is to be considered per influence_conversation only checks whether the list is filled or not in comparison - thus one would have to fundamentally modify this approach to make it work as desired. In addition, there is the fact that an emotion is not strictly speaking an entity and I wanted to avoid this approach anyway.
After consulting with @dakshvar22 , I created a slot with a custom slot mapping whose featurization can take n emotions into account. In combination with variant 1 the whole scenario works now - but I am not satisfied with it, because you need:

An entity
A custom action or NLU GraphComponent that populates the entity
A slot with the same name
An active auto-slot-filling, so that the slot takes the value of the entity
The CustomMapping definition of the slot

In my mind, I wanted to be able to include a GraphComponent that does most of the above directly, so there are fewer dependencies.
When I implemented variant 2 with the policy, I later realized that I also need a slot here, but unlike in the NLU GraphComponent, I can set it via optional_events directly from the policy. This works for each iteration, but the prediction of a follow-up-action is based on a point in time of the storyline where this slot has not been set yet. Since there is no way to write a policy whose "result" is taken into account, e.g. via optional_events, even if another policy wins the race and predicts a follow-up-action, I've discarded this way for now.
What's your suggested solution?
In my opinion, it would be a good thing if you were able to define conditions that you want to use in a story or rule yourself. I'm not sure yet how to best implement this, but I would like to be able to write the following story:
- story: frustration and anger route
  steps:
  - intent: request_status_update
  - conditions:
      - path_to_my_custom_condition.emotion:
        - anger
        - frustration
  - action: utter_appointment
  - action: action_listen

Where the requirement here would be, similar to slots, that a condition must return a feature vector that the TEDPolicy can use to make its decision, for example. The custom condition could be passed e.g. the tracker, so that you can get all the contextual information you need in the implementation.
With this suggestion - which please be sure to check for anything I might have forgotten - one would solve the boundaries that currently lie on conditions regarding the slot mechanics. I wouldn't need anything from the list above - just the custom condition.
Examples (if relevant)
No response
Is anything blocking this from being implemented? (if relevant)
No response
Definition of Done
No response

Thanks for submitting this feature request 🚀 @pythenry will get back to you about it soon! ✨

@JulianGerhard21 putting this in the research team's inbox to look at, but we probably won't be able to look at this until January.

@JulianGerhard21 I think one could resolve it with a custom emotion-classifier NLU component that adds the emotions as metadata to the message, a custom (global) slot mapping that moves this metadata into a slot, and a custom slot that featurizes the emotion labels for the policy.


Avoid switching back to a version already on

Don't do a switch back to the default version when already on it. See Example Output in README.md.

Closed with 32c777119a441eaaeb4e32d1f7967496e984bfe7.


added joke feature

Added joke functionality..

Closing this PR as we are going with a new file structure.


Only one worker node is provisioned

Three worker starts provisioning after executing the below command. However, only one worker node actually gets provisioned (worker-0).

I have followed the guide on Quotas to change my API queries limit per day.

Resolved with the help of Google Cloud Platform Support.


Add support for bulk appends

It would be nice to support bulk appends for SuccinctRDD and SuccinctKVRDD[K] as follows:
// For SuccinctRDD; the preservePartitioning flag dictates whether the
// partitioning scheme for the data RDD should be preserved
def bulkAppend(data: RDD[Array[Byte]], preservePartitioning: Boolean = false)

// For SuccinctKVRDD
def bulkAppend[K](data: RDD[(K, Array[Byte]], preservePartitioning: Boolean = false)

Added in v0.1.6


Users should be able to configure test logging using Kotlin DSL

@eriwen commented on Wed Apr 25 2018
It looks like we don't have Action accepting overloads for AbstractTestTask beforeSuite, afterSuite, beforeTest, afterTest, and onOutput methods.
Expected Behavior
I should be able to configure test logging using the Kotlin DSL without creating extension functions.
Current Behavior
There is no easy equivalent to something like this in Kotlin:
afterSuite { desc, result ->
    if (!desc.parent) {
        println "\nTest result: ${result.resultType}"
        println "Test summary: ${result.testCount} tests, " +
        "${result.successfulTestCount} succeeded, " +
            "${result.failedTestCount} failed, " +
            "${result.skippedTestCount} skipped"
    }
}

The only Kotlin solution I could find is creating something like this in buildSrc.
Context
Looks like over 1000 GitHub projects call afterSuite in their test config. These projects will be harder to migrate to Kotlin DSL.
Your Environment
Kotlin DSL 0.16.3 in Gradle 4.7

@eskatos commented on Wed Apr 25 2018
This also applies to plugins implemented in Java, they would have to import groovy.lang.Closure and fiddle with the untyped Closure parameters.

@eskatos commented on Wed Apr 25 2018
One workaround is to use a typed TestListener instead, it is more verbose because you need to implement its 4 functions though, an adapter would help:
tasks.withType<Test> {
    addTestListener(object : TestListener {
        override fun beforeSuite(suite: TestDescriptor) {}
        override fun beforeTest(testDescriptor: TestDescriptor) {}
        override fun afterTest(testDescriptor: TestDescriptor, result: TestResult) {}
        override fun afterSuite(suite: TestDescriptor, result: TestResult) {}
    })
}

For test output it is simpler given the listener type is a SAM an can be expressed as a lambda:
tasks.withType<Test> {
    addTestOutputListener { testDescriptor, outputEvent ->
        // ...
    }
}

---

@eskatos commented on [Wed Apr 25 2018](https://github.com/gradle/kotlin-dsl/issues/836#issuecomment-384212274)

For three of these methods (`afterTest()`, `afterSuite()` and `onOutput()`) the passed "closure" takes two parameters. `Action` is out of question in those cases. Note that `BiAction` is currently internal.

---

@eskatos commented on [Wed Apr 25 2018](https://github.com/gradle/kotlin-dsl/issues/836#issuecomment-384333766)

Another potential approach without `BiAction`:
```java
interface TestDescriptorEvent<T> {
    TestDescriptor getDescriptor();
    T getEvent();
}

class AbstractTestTask {
    void onOutput(Action<TestDescriptorEvent<TestOutputEvent>> action);
}

tasks.withType<Test> {
    onOutput {
        println("$descriptor -> $event")
    }
}

@eriwen commented on Wed Apr 25 2018
This last solution came to my mind as well, and seems more forward-compatible as it guards against the TestListener interface getting more methods.

@eriwen commented on Wed Apr 25 2018
One last idea that came to my mind is Action<Pair<TestDescriptor, TestResult>> action — is that possible?

@bamboo commented on Thu Apr 26 2018
@eriwen Yes. It would imply usage similar to the following though:
tasks.withType<Test> {
    onOutput {
        println("$first -> $second")
    }
}

@eskatos commented on Thu Apr 26 2018
Moreover, the changes need to happen in the Gradle Java API where we don't have a public Pair type. So far my preference goes to https://github.com/gradle/kotlin-dsl/issues/836#issuecomment-384333766

@eriwen commented on Fri May 04 2018
Agreed @eskatos. Should we solicit community help to address this?

any update on this since last year?

Just run into this issue, I would like to use a typed api for afterSuite. Is there an official way to do it by now? I am using the following workaround now: https://github.com/gradle/kotlin-dsl-samples/issues/836#issuecomment-384206237

Is there another option that should be used instead of the old way?

@eskatos commented on https://github.com/gradle/kotlin-dsl/issues/836#issuecomment-384206140
This also applies to plugins implemented in Java, they would have to import groovy.lang.Closure and fiddle with the untyped Closure parameters.

It's not so bad, there is a helper KotlinClosure2 in Kotlin DSL to produce the Groovy closure:
    afterTest(KotlinClosure2({ descriptor: TestDescriptor, result: TestResult ->
        println("[${descriptor.className}] > ${descriptor.displayName}: ${result.resultType}")
    })

Case with KotlinClosure2 doesn't translate 1:1 to KotlinClosure1 for beforeTest (taking unary function), so instead you can use:
beforeTest(closureOf<TestDescriptor> { logger.lifecycle("Test: $this") })

For reference I was trying to dynamically log stdout on CI using the mentioned APIs. And indeed Kotlin API would be nice, is there a recommandation to contribute a patch there ?
onOutput(KotlinClosure2({ descriptor: TestDescriptor, output: TestOutputEvent ->
    outputPerTest.computeIfAbsent(descriptor.name) { BoundedEvictingQueue(1000) }
        .add(output.message)
}))
afterTest(KotlinClosure2({ descriptor: TestDescriptor, result: TestResult ->
    val output = outputPerTest.remove(descriptor.name)!!
    if (result.resultType == TestResult.ResultType.FAILURE && output.size > 0) {
        output.forEach { logger.lifecycle(" > $it") }
    }
}))
afterSuite(KotlinClosure2({ _: TestDescriptor, _: TestResult ->
    outputPerTest.clear()
}))


LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20210705093347295 to main

LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20210705093347295 to main with localized lcls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 42.754%

Totals

Change from base Build 995411089:
0.0%

Covered Lines:
26168

Relevant Lines:
55576

💛  - Coveralls


JSON file cannot be resolved without explicit ".json" extension when using require

Environment details

OS: Windows 10
Node.js version: 9.11.1
npm version: 6.1.0
@google-cloud/datastore version: 2.0.0

Steps to reproduce

compile code with webpack

When compiling the code, the next error occurs:
ERROR in<EMAIL_ADDRESS>Module not found: Error: Can't resolve './datastore_client_config' in '...\node_modules\@google-cloud\datastore\src\v1'

Due to require not having explicit '.json' extension, compiler fails to find the file
// datastore_client.js
const gapicConfig = require('./datastore_client_config');

Fixed by adding .json at webpack resolve extensions
module.exports = {
  ...
  resolve: {
    extensions: [..., '.json'],
  },
}

This should be fixed by using an explicit extension in the gapic generator.


update taxonomy values

Purpose

update taxonomy slug to new value
Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[X] Other... Please describe: metadata

/azp run


[INLONG-1447] Fix Group Control API logic bug

Fixes #1447

Codecov Report

Merging #1450 (91b792c) into INLONG-570 (9e66f54) will decrease coverage by 0.05%.
The diff coverage is 5.85%.

@@               Coverage Diff                @@
##             INLONG-570    #1450      +/-   ##
================================================
- Coverage         12.25%   12.19%   -0.06%
  Complexity         1046     1046
================================================
  Files               392      392
  Lines             32563    32686     +123
  Branches           5130     5145      +15
================================================
- Hits               3990     3986       -4
- Misses            27819    27942     +123
- Partials            754      758       +4

Impacted Files
Coverage Δ

...che/inlong/tubemq/corebase/utils/TStringUtils.java
3.79% <0.00%> (-0.10%)
:arrow_down:

...er/bdbstore/bdbentitys/BdbGroupFlowCtrlEntity.java
40.52% <ø> (ø)

...bemq/server/master/metamanage/MetaDataManager.java
0.00% <0.00%> (ø)

...e/metastore/dao/entity/GroupConsumeCtrlEntity.java
38.68% <0.00%> (+0.28%)
:arrow_up:

...tamanage/metastore/dao/entity/TopicCtrlEntity.java
47.20% <0.00%> (-0.77%)
:arrow_down:

...astore/impl/bdbimpl/BdbBrokerConfigMapperImpl.java
0.00% <0.00%> (ø)

...re/impl/bdbimpl/BdbGroupConsumeCtrlMapperImpl.java
0.00% <ø> (ø)

...r/master/web/handler/WebAdminGroupCtrlHandler.java
0.00% <0.00%> (ø)

...master/web/handler/WebGroupConsumeCtrlHandler.java
0.00% <ø> (ø)

...ver/master/web/handler/WebGroupResCtrlHandler.java
0.00% <0.00%> (ø)

... and 8 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9e66f54...91b792c. Read the comment docs.


Bug : Reinitialization of platform doesn't remove old platform info.

platformio init --ide eclipse --board esp01_1m

and I see this info. inside platformio.ini
[env:esp01_1m]
platform = espressif
framework = arduino
board = esp01_1m

Now again I tried the same initialization command with different ESP type
platformio init --ide eclipse --board esp12e

and now I see both platform's listed inside platformio.ini
[env:esp01_1m]
platform = espressif
framework = arduino
board = esp01_1m

[env:esp12e]
platform = espressif
framework = arduino
board = esp12e

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/29476867-bug-reinitialization-of-platform-doesn-t-remove-old-platform-info?utm_campaign=plugin&utm_content=tracker%2F2014790&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F2014790&utm_medium=issues&utm_source=github).

This is not a bug, this a feature. The idea of platformio.ini is to have more that one environment. See doc http://docs.platformio.org/en/latest/userguide/cmd_init.html

Initialize new PlatformIO based project or update existing with new data.


Fix script to run on Arch Linux and Fedora

Although PiHole isn't officially supported on distros other than Ubuntu/Debian, some people (like myself), will run it on other distributions.  This patch will use which to locate gawk in $PATH instead of relying on dpkg; if not found, it'll attempt to either use, apt-get, pacman or dnf to install it, whichever it finds first.

@lpereira Thanks for the great PR! Merging...


New feature: Add support to destroy/recover volumes

Description

cloudstack provides options to destroy and recover a vm. For volumes, we do not have similar operations. When we delete a volume, it will be removed from primary storage permanently.
With this change,
(1) we have option to destroy a volume (expunge=false) or delete a volume permanently (expunge=true)
(2) we are be able to recover a Destroy volume to Ready state.
(3) Destroy volume will be removed in storage clean up.
(4) when reinstall a vm, the old root volume will be Destroy state. It can be recovered to Ready state and attached to other vm so we can restore some data.
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[X] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Screenshots (if appropriate):
How Has This Been Tested?

destroy vm (expunge=false)
recover a Destroy vm
delete vm (same as destroy vm with expunge=True)
More test cases can be found in the integration tests.

seems like a useful feature @ustcweizhou . I had a global look and saw some style issues but no real ones. I'll do a thourough review when I have time. @PaulAngus @andrijapanicsb any thoughts?

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✖centos6 ✖centos7 ✖debian.  I'll keep you posted as I make progress.

Packaging result: ✖centos6 ✔centos7 ✔debian. JID-532

@blueorangutan test

@andrijapanicsb a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-695)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 39191 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3688-t695-kvm-centos7.zip
Smoke tests completed. 77 look OK, 0 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

code looks good, two minor remarks @weizhouapache

@weizhouapache I've been committing the translation messages directly to your branch - hope that is ok? (wouldn't date to touch the code)

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✖centos6 ✔centos7 ✔debian. JID-739

@rhtyd this passed all tests last week, do we need a retest?

2 approving changes,
The last commit just edits the version in the comment/message - i.e. no need to retest again, previous Trillian tests are all fine.
Jenkins happy.
Merging.

@rhtyd are you referring to this with https://github.com/apache/cloudstack/pull/3601#issuecomment-584604429 ?

Yes @DaanHoogland maybe Wei can comment?

@rhtyd @DaanHoogland
I have run the integration test component/test_volumes.py on kvm, all passed.
ok
Detach a Volume attached to a VM ... === TestName: test_02_detach_volume | Status : SUCCESS ===
ok
Delete a Volume unattached to an VM ... === TestName: test_03_delete_detached_volume | Status : SUCCESS ===
ok
Create a volume under a non-root domain as non-root-domain user ... === TestName: test_create_volume_under_domain | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 4 tests in 277.694s

OK

and smoke/test_volumes.py on kvm
Test destroy Virtual Machine and it's volumes ... === TestName: test_11_destroy_vm_and_volumes | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 1 test in 678.315s

OK

and smoke/test_vm_life_cycle.py
Delete a Volume unattached to an VM ... === TestName: test_09_delete_detached_volume | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 1 test in 364.770s

OK

I have create a pr for travis test:  https://github.com/apache/cloudstack/pull/3880

Yes smoketests indeed pass on kvm but Travis runs tests against simulator. I'll review the PR. Thanks.


feat: Allow cloning of running executions

lyft/flyte#468
This allows "relaunch" functionality for an execution which hasn't yet finished. This is helpful for some users who would like to launch multiple copies of an execution with slightly different input parameters.
Also fixed a warning from a test using a deprecated jest-dom method.

Codecov Report

Merging #93 into master will increase coverage by 0.80%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #93      +/-   ##
==========================================
+ Coverage   66.70%   67.50%   +0.80%
==========================================
  Files         370      371       +1
  Lines        5962     5992      +30
  Branches      930      932       +2
==========================================
+ Hits         3977     4045      +68
+ Misses       1985     1947      -38

Impacted Files
Coverage Δ

src/test/utils.ts
93.33% <ø> (ø)

...ExecutionDetails/ExecutionDetailsAppBarContent.tsx
95.65% <100.00%> (+48.15%)
:arrow_up:

...omponents/Executions/ExecutionDetails/constants.ts
100.00% <100.00%> (ø)

src/components/common/MoreOptionsMenu.tsx
100.00% <100.00%> (ø)

src/components/common/constants.ts
100.00% <100.00%> (ø)

src/components/Executions/utils.ts
76.69% <0.00%> (+2.91%)
:arrow_up:

src/components/Executions/ExecutionStatusBadge.tsx
90.90% <0.00%> (+4.54%)
:arrow_up:

...ns/TerminateExecution/TerminateExecutionButton.tsx
100.00% <0.00%> (+42.85%)
:arrow_up:

... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e0b1f5c...969006b. Read the comment docs.

:tada: This PR is included in version 0.11.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Logout

Hi !
This app is great ! really easy to deploy and customise thanks to his creator.
Got an issue regarding authentication, once logged, you can't logout except by flushing your cookies.
Is there any way to create a link or button to do that in the go file ?

Find a solution by adding a logout button with a link pointed to<EMAIL_ADDRESS>


auto-instrumentation for Java Spring Kafka Streams - is NOT generating the Jaeger Deep Dependency graph with Producer and Cosnumer

Hi,
I have written 1 Producer and 1 Consumer using Spring Kafka streams/Cloud streams and able to see the traces in JaegerUI but when I click on the Deep Dependency graph its not showing the graph like
Producer --> Kafka TopicName --> Consumer.
Is it possible to generate Information above like Producer ->TopicName -> Consumer informations like traces and metrics?
Thanks
Dharma

This repo is for the OpenTelemetry website. You should direct your question to a more appropriate forum. For example, see the links to discussions or mailing lists under the Community > Communication.
Since you aren't reporting a problem with the website, I'll close this issue.


impellerc should be codesigned for macOS

impellerc is being executed on the host machine, so should be codesigned/notarized on macOS to avoid the "Developer unknown" popup when the SDK is downloaded from the website (because it is quarantined).
https://github.com/flutter/flutter/blob/25e88cb1542d858f144ea10ffe372719e877ea15/packages/flutter_tools/lib/src/build_system/targets/shader_compiler.dart#L171-L173
Instead of being in
https://github.com/flutter/flutter/blob/25e88cb1542d858f144ea10ffe372719e877ea15/dev/conductor/core/lib/src/codesign.dart#L183
it should be in
https://github.com/flutter/flutter/blob/25e88cb1542d858f144ea10ffe372719e877ea15/dev/conductor/core/lib/src/codesign.dart#L140
Introduced in https://github.com/flutter/flutter/pull/102551

cc @itsjustkevin @CaseyHillers

Technically both binariesWithEntitlements and binariesWithoutEntitlements are codesigned, the difference is whether we codesign them with this set of entitlements: https://github.com/christopherfujino/codesign.py/blob/master/Entitlements.plist:

<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple Computer//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
    <dict>
        <key>com.apple.security.cs.allow-jit</key>
        <true/>
        <key>com.apple.security.cs.allow-unsigned-executable-memory</key>
        <true/>
        <key>com.apple.security.cs.allow-dyld-environment-variables</key>
        <true/>
        <key>com.apple.security.network.client</key>
        <true/>
        <key>com.apple.security.network.server</key>
        <true/>
        <key>com.apple.security.cs.disable-library-validation</key>
        <true/>
    </dict>
</plist>

Which impellerc may not even need. Also note, this is a very crude system, ideally we would track per binary which entitlements are necessary.

However, we are not verifying that impellerc is codesigned because of https://github.com/flutter/flutter/issues/95297

Okay, never mind, it is codesigned and notarized, and doesn't need the entitlements:
$ spctl -a -vvv -t install  bin/cache/artifacts/engine/darwin-x64/impellerc
bin/cache/artifacts/engine/darwin-x64/impellerc: accepted
source=Notarized Developer ID

Ran without the pop-up.  Apologies for the noise.

Maybe the action item here then is to update: https://github.com/christopherfujino/codesign.py/blob/master/codesign.py. I'm assuming this was codesigned by local changes to this script, but we should push these back to github @XilaiZhang

yep so it was signed based on the pull request to https://github.com/christopherfujino/codesign.py/blob/master/codesign.py

https://github.com/christopherfujino/codesign.py/pulls

sync'd with @XilaiZhang offline; it turns out what is being used in production is neither codesign.py master branch nor the PR, but local changes on the bot. Xilai will open a PR against christopherfujino/codesign.py to merge these changes upstream, which will resolve this issue.

https://github.com/christopherfujino/codesign.py/pull/3 wasn't correct, it needs to codesign the arm64 iOS framework.  @XilaiZhang
I'll reopen this to track that the codesign script should reflect master's file list.

@jmagman is there a solution for this issue?  On macOS  the "Developer unknown" popup shows when I try to run the sample app

I think the issue still exist on 3.3.4. @jmagman
Reproduction steps:
Install flutter SDK macOS(intel) 3.3.4
Run the sample counterr app
Macos shows the popup "Developer unknown" for impellerc.

If this helps anyone, issue still on 3.3.7

Hm, I'm still not seeing it, I downloaded 3.3.7, ran /path/to/downloadedflutter/bin/flutter create then /path/to/downloadedflutter/bin/flutter run --enable-impeller on a physical phone, and I didn't see any developer unknown pop up, and the counter app ran fine.
Binary is codesigned and notarized.  otool -l shows reasonable LC_LOAD_DYLIB paths (which was a cause of this issue for other binaries).  Could you open Console, start streaming logs, and reproduce the issue? I would expect logs from the subsystems XprotectService, trustd, and syspolicyd right when the pop up appears.  If you see those logs can you paste them here?

@Asquare17 or @diver2 were you able to try this?  I wasn't able to reproduce this.

Could you open Console, start streaming logs, and reproduce the issue? I would expect logs from the subsystems XprotectService, trustd, and syspolicyd right when the pop up appears. If you see those logs can you paste them here?

Hi guys,
I have installed a fresh MacOS ventura 13.2 and then installed flutter 3.7.0 on it.
Followed the instruction of flutter official page to do that. And even when I run "flutter doctor" command, everything passed fine without any issue.
But when I try to run the app on any Simulator either ios or android, I am getting the error message regarding "impellerc" file. This is the error message I am getting:

I appreciate any hence that can help. Thanks a lot

Hi guys, I have installed a fresh MacOS ventura 13.2 and then installed flutter 3.7.0 on it. Followed the instruction of flutter official page to do that. And even when I run "flutter doctor" command, everything passed fine without any issue. But when I try to run the app on any Simulator either ios or android, I am getting the error message regarding "impellerc" file. This is the error message I am getting:

I appreciate any hence that can help. Thanks a lot

Awesome, thanks for sharing this! I believe this is the same as https://github.com/flutter/flutter/issues/116703

I wasn't able to reproduce this before because the x64 stable artifacts are codesigned, but the arm ones are not.
https://storage.googleapis.com/flutter_infra_release/releases/stable/macos/flutter_macos_3.7.1-stable.zip
$ codesign -d -vv ./artifacts/engine/darwin-x64/impellerc
...
Signature size=8979
Authority=Developer ID Application: FLUTTER.IO LLC (S8QB4VV633)
Authority=Developer ID Certification Authority
Authority=Apple Root CA
Timestamp=Jan 31, 2023 at 3:52:49 PM
Info.plist=not bound
TeamIdentifier=S8QB4VV633

https://storage.googleapis.com/flutter_infra_release/releases/stable/macos/flutter_macos_arm64_3.7.1-stable.zip
Signature=adhoc
Info.plist=not bound
TeamIdentifier=not set

Created https://github.com/flutter/flutter/pull/119971 to run codesigning verification from on both architectures.

Next steps:

Codesign and reupload the stable artifacts.
Update process at go/flutter-manual-codesign (sorry for the non-public link)
Validate engine v2 scripts will codesign both.

Validate engine v2 scripts will codesign both.

Engine v2 will be much more reliable, as gn will output the list of targets which will then be fed as an input into the new codesign script.

bin/cache/artifacts/engine/darwin-x64/impellerc: accepted
source=Notarized Developer ID
origin=Developer ID Application: FLUTTER.IO LLC (S8QB4VV633)

Stable version 3.7.2 arm64 SDK zipped artifacts are codesigned.  The codesigning script has been updated, and there are now verification tests that run on x64 and arm64 bots that should keep this from happening again.  Closing.


[CT-2527] Adapters allow for optionally installing MetricFlow

The packaging of the individual adapters need to allow for the optional installing of MetricFlow, e.g. pip install dbt-snowflake[metricflow]. This issue is likely just for tracking, as part of this project, issues on the adapter specific repositories for ensuring this functionality.

We're going to take a different approach here, so I'm going to close this issue for now (cc @tlento)


Update http4s-blaze-client, ... to 0.21.17

Updates

org.http4s:http4s-blaze-client
org.http4s:http4s-blaze-server
org.http4s:http4s-circe
org.http4s:http4s-dsl

from 0.21.16 to 0.21.17.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, semver-patch

Superseded by #27.


Split out some of the longer-running tasks

An additional workflow can run in parallel.  This increases overall work, but it might allow us to reduce the overall time to getting a positive answer.

Concurrency tests seem to be the long pole here.  I want to get them running alongside the regular tests.

I could tune them a bit to make them run faster. Every single improvement in IPA makes concurrency tests better too :)


refactor store wizard

adds a saveToAPI module that centralizes saving data and pulling missing data from the appropriate model
switches store wizard view to the new module so name is not required to be in the form
closes #118 return showing when it is empty on item screen
adds terms and conditions to item screen
fixes settings and store wizard bug where selecting zero moderators did not clear the moderator list

@hoffmabc this is ready for you to review

Ok this one LGTM.


Authorization header sent to upstream unconditionally

Currently have oauth2_proxy running off the latest tag with an nginx as an upstream that is configured as a proxy to ab AWS-managed ElasticSearch that authenticates via IP.
When trying to access Kibana through this setup, we get an error stating: not a valid key=value pair (missing equal-sign) in Authorization header: 'Basic REDACTED'. .
Current Behavior
After some debugging, it would appear that Oauth2_Proxy is passing through an Authorization to nginx in the request.
Documentation states that this should not be the case unless we use the option -pass-authorization-header=true. Unfortunately, that is not the case and ES can only be accessed after adding one of the following lines to the nginx configuration:
proxy_pass_request_headers off;
proxy_set_header  Authorization "";

I have also tried adding -pass-authorization-header=false to the command and the results are the same. Checking the developer console I can see that no Authorization header is sent from the browser nor there is any present in the response of HOST/oauth2/auth
Expected Behavior
Authorization header should not be passed through to upstream as documented
Steps to Reproduce (for bugs)
Running latest executing the following command:
oauth2_proxy \
   --upstream=http://nginx-container:8080/ \
   --http-address=<IP_ADDRESS>:8081 \
   -cookie-secret=64-char-long-string \
   -cookie-expire=24h0m0s \
   -footer=- \
   -provider=oidc \
   -email-domain=REDACTED \
   -client-id=REDACTED \
   -client-secret=REDACTED \
   -login-url=REDACTED \
   -profile-url=REDACTED \
   -redeem-url=REDACTED \
   -oidc-issuer-url=REDACTED

The Nginx container is setup quite simply with the following proxy configuration:
        location / {
            proxy_pass AWS-ELASTICSEARCH-DNS;
            proxy_set_header  X-Real-IP  $remote_addr;
            proxy_set_header  Host AWS-ELASTICSEARCH-DNS;
            proxy_set_header  X-Forwarded-For $proxy_add_x_forwarded_for;
            proxy_set_header  X-Forwarded-Proto $scheme;
       }

Version used: latest

--pass-basic-auth defaults to True, you'll want to use --pass-basic-auth=false

(This --pass-basic-auth option sends just username, but no password, or a static password you configure separately, just as a way to pass username to some applications which may expect it this way. The --pass-authorization-header option sends Bearer with real token.)

@ploxiln I will try that and let you know

you were right... my bad for not reading the header correctly


TypeError: buffer is too small for requested array

Hello,
I have trained a custom model with 9 classes based on yolov4-416. I have tested it with darknet and it works, but when I try to pass it to onix I get the following error:
$ python3 yolo_to_onnx.py -m yolov4-416 -c 9
Parsing DarkNet cfg file...
Building ONNX graph...
Traceback (most recent call last):
File "yolo_to_onnx.py", line 973, in
main()
File "yolo_to_onnx.py", line 956, in main
verbose=True)
File "yolo_to_onnx.py", line 447, in build_onnx_graph
weight_loader = WeightLoader(weights_file_path)
File "yolo_to_onnx.py", line 278, in init
self.weights_file = self._open_weights_file(weights_file_path)
File "yolo_to_onnx.py", line 343, in _open_weights_file
buffer=weights_file.read(length_header * 4))
TypeError: buffer is too small for requested array
I have attached my cfg file for more information.
can any one help?
Thanks!
yolov4-416.cfg.txt

This usually happens when you don't mark the correct layers as outputs in your custom model.  Please refer to https://github.com/jkjung-avt/tensorrt_demos/issues/334#issuecomment-765809470.

Hi @jkjung-avt,
Thanks for answering.
I saw that changes needed on the code in other issues, but my cfg file has not any change on the layers. The only thing that I have changed  is the classes number and the number of filters in the convolutional layers before the yolo layers to adapt to my number of classes and y thought that 'c = (args.category_num + 5) * 3' covered yet this change. Am I wrong? What exactly do I have to check to do the changes? I do not really get the logic in them.
Regards,
Antonio.

The only thing that I have changed is the classes number in the yolo layers and the number of filters in the convolutional layers before them

This is correct.
I've just checked your cfg file.  I didn't see anything wrong with it.  I don't have other suggestions than making sure you have named the weights exactly the same as the cfg file: "yolov4-416.weights" and "yolov4-416.cfg".

Hello,
The name of the weights file was the same and I have added some traces to the code, and it was entering the correct places.  However, I realized that the problem appeared when setting the weight file to an array, so the problem surely was in the weights file. I uploaded again the weights file to the Jetson and it worked! So finally the problem was that the weights file was corrupted.
Thanks for your help!


document attributes on CRS,Geod,Proj classes

Addresses #253

Thanks for the better descriptions for the Geod attributes @jswhit! I updated the docs with your suggestion.


fixed #247 build error on context.Idp

@BrettBailey,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla2.dotnetfoundation.org.
It will cover your contributions to all .NET Foundation-managed open source projects.
Thanks,
.NET Foundation Pull Request Bot

@BrettBailey, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, .NET Foundation Pull Request Bot

thanks. i guess we should get some CI for the sample repo... :)

not a bad idea :-)  happy to help


test/test.js simply require()s other test suites

It's not obvious from the tests below but I simply did a $ git mv test/test.js test/linter-test.js and then added a test/test.js that requires the other two *-test.js files in that directory.
Prior to this, $ npm test was not running test/jsconf-policy-test.js`.

Merged as 1712a41be658e6530afc1c28a9722555d7f70f89


Capture the ColumnResize event

Whenever user resizes the column of a table, I would like to capture the columnResize event and store the new width in the browser's local storage, so that, on reopening the page, user can restore his column width applied.
However, I don't see any event exposed by this Table component to capture the columnResize event.
Can you help me understand how to achieve this?

You would need to add a prop to be called on the resize event form within the code I think.

@NageshAndani please let me know if you need some help with this! It looks like you'll just need to pass that event up the chain and expose it. I will reopen this if needed. Thanks for your contribution!

I've been looking into this and other aspects of column resizing.  I've got changes working locally that:

allow table to expand as column are resized
added onColumnResize callback
supress resize icon when table header cells have borders

I've got some tidying up to do then I'll submit a pull for consideration.
I'd already made one local patch to material-table and then found another bug.

That's awesome. I PR would be most welcome. Link me once you have it.

I've got my changes ready to push to a branch - extended up a bit more extensive then I thought originally.


Clean up unnecessary get_tag_info juggling

I'm not sure about the overall view what's happening. Why are we removing the caching? :+1: On the code, looks good to me.

We're only getting it through the cache in one place (during routing), so the extra layer isn't buying us anything. The driving force behind the change is cleaning up unnecessary complexity around the message store so we can switch to the new implementation more easily.

:+1:

:+1: :computer:


Permission denied

I cannot use tiktoken in python:
Input encoding = tiktoken.get_encoding('cl100k_base'),
it gives the error PermissionError: [Errno 13] Permission denied: '/tmp/data-gym-cache/9b5ad71b2ce5302211f9c61530b329a4922fc6a4.2749b823-646b-45d7-9fcf-11414469d900.tmp'
But I can use it in my colab.

Try setting the environment variable TIKTOKEN_CACHE_DIR=""

Is there another way to fix the same issue? I do not have access to set an environment variable.

getting the same issue when deployed a django application with langchain package in IIS using HttpPlatformModule, can anyone let me know how to resolve this?

If you are using this in a kubernetes environment, you can do the following:

make an environment variable for tiktoken
TIKTOKEN_CACHE_DIR = "desired_path_to_tmp"

Add the directory to your dockerfile and make the user owner of that directory
RUN mkdir /desired_path_to_tmp && chown -R 2:2 /desired_path_to_tmp

You will get this error in a multi-user system. When some other users have already created the cache dir /tmp/data-gym-cache, they own the directory, and you don't have the permission. It's a bad idea to hard-code the temporary directory.

I've pushed a change that will swallow the PermissionError in the default case. This will be released in tiktoken 0.5.2


Removes needless conversion.

Removes no-op type checks from the ncbiTaxonId code path. The type of the value in the DB is integer, so the value within python must be either an integer or None. Also removes the call to setNcbiTaxonId as none of the other attributes did this, and there is no real difference between between _ncbiTaxonId and the rest.  Calling setters is good practise, but this should be done uniformly within the method on all attributes,  not piecemeal on one.

+1
This passes compliance just fine. I assume the removed code was there to catch the case where a backend incorrectly records the NCBI Taxon ID as a string. I believe I've seen these kinds of conversions in other places as well, I take this as a call to remove such code when we see it, and  assume the database provides the correct type?

I believe I've seen these kinds of conversions in other places as well, I take this as a call to remove such code when we see it, and assume the database provides the correct type?

Yes, I think we should. As a general principle, types should be checked on the way in to an object, so that an error can be propagated back to the source of the problem. Silently 'fixing' bad types on the way out (i.e. in getters) is a bad idea because this will cause the problems to be propagated further down stream and lead to difficult to trace problems. Having the wrong type inside an instance variable is a programming error, and it should lead to an exception.

@dcolligan, can you take a look and merge if you're happy please?

@macieksmuga, @david4096, can one of you merge this please (if you're happy with it)?


Auto-faucet upon landing with zero balance

If a user doesn't have a balance, SwaySwap should auto facuet me coin and present a message nicely saying "Fauceting you test Ether".
I shouldn't need to manually hit the faucet button.
Also, if you want the faucet button to be the primary action on the wallet page, then it should be in an action color like green to signal I need to press it.

what if I added liquidity used all my ETH and got to zero balance, should it faucet me more?


unable to assemble ship

I placed the helm and went to assemble the ship and it told me to sneak to do so. sneaking+right click picks up the helm to move it instead of opening the menu. i looked in the "key binding" tab under the "controls" setting and could not fix this as there was no category for the mod's key bindings. i am using Minecraft 1.18.2 Fabric edition.

then change the keybind for picking up blocks

then change the keybind for picking up blocks

again. there is no option to change the key binding.

remove optifine and add eureka helm to carry-on blacklist


how can i get my wallet address memo or tag

const { TonClient, WalletContractV4, internal } = require("@ton/ton");
const { mnemonicNew, mnemonicToPrivateKey } = require("@ton/crypto");
    // Create Client
    const client = new TonClient({
        endpoint: 'https://toncenter.com/api/v2/jsonRPC',
        /* apiKey: "jsjs" */
    });

    const mnemonics = [
        'camera', 'hunt', 'hunt',
        'race', 'private', 'category',
        'virtual', 'escape', 'awkward',
        'update', 'mammal', 'length',
        'safe', 'pool', 'laptop',
        'red', 'crowd', 'steak',
        'gorilla', 'daughter', 'just',
        'autumn', 'disease', 'perfect'
    ];

    // Generate new key
    //let mnemonics = await mnemonicNew();

    let keyPair = await mnemonicToPrivateKey(mnemonics);

    // Create wallet contract
    let workchain = 0; // Usually you need a workchain 0
    let wallet = WalletContractV4.create({ workchain, publicKey: keyPair.publicKey });
    let contract = client.open(wallet);

You can get your wallet contract address using contract.address.toString()
Memo is a comment on transactions coming to the wallet, the wallet does not have such a parameter. It is necessary to identify the recipient of the transfer to exchanges, for example

You can get your wallet contract address using contract.address.toString() Memo is a comment on transactions coming to the wallet, the wallet does not have such a parameter. It is necessary to identify the recipient of the transfer to exchanges, for example

I try to send Ton from my wallet I created to another wallet on Bybit, they required me to send input Memo while sending, but since there is no place to input memo In my code, I just send it like that without putting the memo, till today I didn't see the Ton inside my Bybit account


logical error vs functionality in 1099

hi everyone,
today i was working with 1099 report. I found some wired code which looks like that it is an issues.
while creating any report, for each vendor we are calling following function

in this function, we are fetching entries from vendor ledger enteries and from that function, for each of entry we are calling another function to calculate 1099 amount

after calculating 1099 amount, we are updating array of amount in the following function

and in this function, we are adding the adjustment amount

so, which means for every vendor ledger entry line we are adding adjustment amount and means in future if end user has many lines then it will going to add adjustment(s) for each line which doesn't make any sense.
but still i want to confirm with you guys?

Thank you for reaching out. Your request could be reporting a product bug not related to the AL extensibility.
Please try to provide a business scenario that shows the problem you described from AL code point of view.
Please reach out to the relevant support channels as specified in Product Support
On premises - https://support.microsoft.com/en-us/supportforbusiness/productselection?sapId=93d37907-ad94-d591-22e9-593cfa09dd3f
On line – read more here https://docs.microsoft.com/en-ca/dynamics365/business-central/dev-itpro/administration/manage-technical-support or contact your partner
If you are a partner open a support incident through Partner Center - https://partner.microsoft.com/en-us/pcv/dashboard/overview

Thank you for reaching out. Your request could be reporting a product bug not related to the AL extensibility.
Please try to provide a business scenario that shows the problem you described from AL code point of view.
Please reach out to the relevant support channels as specified in Product Support
On premises - https://support.microsoft.com/en-us/supportforbusiness/productselection?sapId=93d37907-ad94-d591-22e9-593cfa09dd3f
On line – read more here https://docs.microsoft.com/en-ca/dynamics365/business-central/dev-itpro/administration/manage-technical-support or contact your partner
If you are a partner open a support incident through Partner Center - https://partner.microsoft.com/en-us/pcv/dashboard/overview


Enable background job tracking in Skylight

https://www.skylight.io/support/background-jobs#oss

Looks like we need to add “SKYLIGHT_ENABLE_SIDEKIQ=true” in the procfile for the worker?

donezo


Fragments must be restorable

Right now, if a user leaves the app on, e.g. GalleryFragment, and switches to another app, and OS kills our app — after Android attempts to restore our app, it will crash. The reason is that Android can restore only fragments with empty constructors.
Fragments should be reworked to gain this capability, particularly GalleryFragment and ResourcesFragment.

Related #4

Not sure how to reproduce this. I tried this:

opened a picture in the app
minimised the app and opened a few different apps
returned to the app after a few minutes

The OS didn't reallocate the resources assigned to the app and it opened instantly without a hitch, so not sure how to reproduce this.

Okay, that was my assumption too. Thanks for confirming! I will try this again

Checked with steps form PR #135  and fine. Used buld 170 and device was Xiaomi A2
logL.txt

Closed by #135


Added a cooking website

I have added a cooking website with the name of simply recipes.
resolved issue #191

Please create a demo folder inside your project and add a working demo of your project named demo.gif

Can you suggest a good website for screen recording in PC? @irshadmd

yeah, all done !!

great


DockerSwarmSuite.TestSwarmContainerAutoStart is flaky

Noticed in https://github.com/docker/docker/pull/26584 on experimental CI.
01:14:43 ----------------------------------------------------------------------
01:14:43 FAIL: docker_cli_swarm_test.go:246: DockerSwarmSuite.TestSwarmContainerAutoStart
01:14:43
01:14:43 [d20f68f1ba9c8] waiting for daemon to start
01:14:43 [d20f68f1ba9c8] daemon started
01:14:43 [d20f68f1ba9c8] exiting daemon
01:14:43 [d20f68f1ba9c8] waiting for daemon to start
01:14:43 [d20f68f1ba9c8] daemon started
01:14:43 docker_cli_swarm_test.go:265:
01:14:43     c.Assert(strings.TrimSpace(out), checker.Not(checker.Equals), "")
01:14:43 ... obtained string = ""
01:14:43 ... expected string = ""
01:14:43
01:14:43 [d20f68f1ba9c8] exiting daemon
01:14:45
01:14:45 ----------------------------------------------------------------------

----------------------------------------------------------------------
FAIL: docker_cli_swarm_test.go:246: DockerSwarmSuite.TestSwarmContainerAutoStart

[d4f8197a8140c] waiting for daemon to start
[d4f8197a8140c] daemon started
docker_cli_swarm_test.go:254:
    c.Assert(err, checker.IsNil)
... value *exec.ExitError = &exec.ExitError{ProcessState:(*os.ProcessState)(0xc420250c40), Stderr:[]uint8(nil)} ("exit status 125")

[d4f8197a8140c] exiting daemon
OOPS: 0 passed, 1 FAILED
--- FAIL: Test (17.16s)
FAIL
---> Making bundle: .integration-daemon-stop (in bundles/1.13.0-dev/test-integration-cli)
+++++ cat bundles/1.13.0-dev/test-integration-cli/docker.pid
++++ kill 8058
make: *** [test-integration-cli] Error 1

testing in macos

@mrjana Some suspicious lines from logs: https://gist.github.com/tonistiigi/00de73e9a25bf7f404ec8cc563a03429#file-docker-log-L517-L565

It exposes a iptable rule plumbing race in overlay driver in kernels < 3.16. Will fix it libnetwork.

@mrjana failed again in https://jenkins.dockerproject.org/job/Docker-PRs-experimental/24208/console

@tonistiigi How do I get the daemon logs for the above run?

@mrjana Copied them out https://gist.github.com/tonistiigi/979c41d5977766ddf5a5cb9167abb49f It seems to go to a pretty bad loop there. Logs can be pulled by clicking on build number in menu and then there is a link to download artifacts.

Ok. But this one looks like a very different problem. After restart the agent is not able to setup a session successfully with manager.

Can this be closed now that #26879 was merged? Not sure why it didn't get closed automatically. Perhaps GitHub doesn't let a reopened issue get closed automatically by a PR.

Cool

@aaronlehmann #26879 was against v1.12.x. I think same fix was already posted against master but didn't fix it for the ci.
@mrjana Fresh logs from this morning https://jenkins.dockerproject.org/job/Docker Master/label=fedora23 && devicemapper/6961/console same loop in the logs https://gist.github.com/tonistiigi/48e1be9075865ba85d50ede015c6718f after the restart.

@tonistiigi This flakiness is due to a different reason. So even though the previous fix, fixed the previous issue it still looks like there are some more issues. But this one doesn't reveal too much information. The agent session handling code goes wild. I am not sure if manager/dispatcher is stuck or not. @LK4D4 Do you have any clues looking at the logs?

@mrjana Looks like agent somehow can't properly register and forever uses a wrong session. However, the code is quite mysterious for me. Maybe @stevvooe can help us.
My theory is that probably UpdateTaskStatus in agent/agent.go shouldn't close a session, but instead send error to session.errs, then backoff will work and sessionq will be set to nil. But I'm not sure.

@stevvooe Can you take a look at the logs?

I believe this was fixed with recent swarmkit update.

@mrjana @LK4D4 Failed again today https://jenkins.dockerproject.org/job/Docker Master/label=ubuntu-1404 && zfs/7051/ network foo not found in logs https://gist.github.com/tonistiigi/871e626e36044fb297a5e6b5b39914d3#file-docker-log-L588

:/

@LK4D4 @tonistiigi Do we know what's the root cause?

I think I know the root cause. It is because the Agent cleanup of old tasks is happening concurrently with the container getting restarted when daemon starts and sometimes the Agent might cleanup the network that was just created locally for the container starting phase. Working on a fix for this.

@tonistiigi @LK4D4 @mrjana @aluzzardi Looks like we fixed one issue and introduced another. Any way we can more reliably propagate removal?


v1.3 Checkup 2,3,5 Unable to be identified by UE4

I downloaded the TAG package of V1.3, But the documents of the levels 2, 3, 5 cannot be identified by UE4,Then I downloaded V1.1 is normal.

Are you using Cesium for Unreal v1.3.1 or later?

all right,I update and it's working,thanks


Introduce methods for checking type of symbol on SymbolRef

Introduce methods for checking type of symbol on SymbolRef. Puts the kind bits in SymbolRef to work! Avoids a .data call in the common case.
Motivation

Prerequisite for introducing different classes for different types of symbols, which will save memory and make code safer to write.
Here's the path I'm envisioning:

Introducing ClassOrModuleRef/MethodRef, and change code paths that expect one specific type of symbol to accept them instead of the general purpose SymbolRef.

SymbolRef will have asMethod() / etc methods to convert to a symbol-specific ref.

Remove .data() on SymbolRef, and have symbol type-specific methods (.klass, .method, etc.).

Methods common to all symbol types will be moved up to SymbolRef for convenience.
We cannot have .data() when/if different symbol types do not share a parent class.
Naturally, the symbol type-specific ref classes (e.g., ClassOrModuleRef) will continue to have a .data() method. I expect most call sites to be using these.

Break up Symbol class into separate classes for ClassOrModule, Method, etc.

By this point, most of the changes will be in GlobalState, which has a some symbol-type-agnostic code dealing with flags.

Test plan

See included automated tests.

With respect to long term plan, I am slightly worried that this will make it easier for features to be implemented on a really specific thing when they mean to work for many kinds of symbols. For example, Usually if you're doing something for classes and modules, you also want to handle static fields (non-module constants) and type members ("generic" static fields). For example, this is how constant completion works.
A scenario I'd like to avoid is:

someone thinks they only need to handle a ClassOrModuleRef case, and threads that through a ton of methods.
later, someone realizes that they want to handle StaticFieldRef and TypeMemberRef too.
the code is already passing these specific references along, and now it's a huge rewrite to thread the potential for "it could be one of three things" through everywhere it needs need to be threaded, so they don't do the refactor.

it's a huge rewrite to thread the potential for "it could be one of three things" through everywhere it needs need to be threaded, so they don't do the refactor.

This is the current situation though; we have tons of enforces that check that something is the proper symbol type. I don't know if going from ClassOrModuleRef to SymbolRef would be a huge rewrite; just change it to SymbolRef, fight the compiler, and insert ENFORCEs as needed.
I'm hoping that this change makes people more conscious of the types of symbols being passed around. SymbolRef is still going to be useful for a wide variety of use cases (especially in-editor). Any methods that are useful for ~all symbol types can be lifted to SymbolRef for convenience (e.g., show, toString).
fwiw, when I first started working on sorbet I tripped on what a symbolref can and cannot be in various parts of the codebase. I still trip on it (e.g., can owner be a non-class? yes it can). I hope that the more-specific refs can permeate the codebase to self-document places where the symbol type is truly constrained.


Deprecating picks in beamformers

Remove the picks parameter from beamformer functions (LCMV and RAP-MUSIC) to make input consistent with other inverse functions: channel selection will be handled via info in the future.
@agramfort and @dengemann

Codecov Report

Merging #4455 into master will decrease coverage by 4.09%.
The diff coverage is 42.85%.

@@           Coverage Diff            @@
##           master   #4455     +/-   ##
========================================
- Coverage    82.7%   78.6%   -4.1%
========================================
  Files         349     349
  Lines       64808   64823     +15
  Branches    10023   10029      +6
========================================
- Hits        53597   50956   -2641
- Misses       8430   11005   +2575
- Partials     2781    2862     +81

that's if for me.
so 1 conceptual question and a quick design fix to do.

No strong feeling

see why CIs complain
+1 for MRG on my side if CIs are green

The failures are on 3.6 which have been fixed in master yesterday, so I restarted them
The CircleCI has also been fixed on master but can only be tested via rebase, as CircleCI doesn't build merged versions (perhaps we should change this)

Okay, I'll rebase, thanks!

Good for me. Let's see if Travis is happy, or at least happy for unrelated reasons.

Thanks a lot @britta-wstnr for your patience. Merged by rebase after dropping you last commit which as caused due to my bad.


IPV6 Routing problem on LAN if re0

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[ x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x ] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
The simplest description is that i cannot pint the router using ipv6.  needless to say I cannot contact the internet via ipv6. 23.1.4 worked, other prior 23.1.v versions also had the same problem.  22.7.x versions worked.
Ping for client:
to router LAN re0 2603:: address              No
to router LAN re0 fe80:: address               Yes
to any other client Lan 2603:: Address         Yes
to any internet ipv6 address                        No
Ping from router:
to router LAN re0 2603:: address             No
to router LAN re0 fe80:: address              Yes
to any client LAN 2603::                              No   ping: UDP connect: Network is unreachable
to any internet ipv6 address                       Yes   with DNS name resolution
The router WAN re1 is fully functional.
The router LAN re0 ISP assigned address is not honored.
To Reproduce
Steps to reproduce the behavior:

Install opnsense v23.1.5_4
restore config
no ipv6 routing

Expected behavior
IPV6 should route.
Describe alternatives you considered
firewall rules look good and worked on 22.7.x
Screenshots

Relevant log files
glog.txt
Environment
Software version used and hardware type if relevant, e.g.:
OPNsense 23.1.5_4 (amd64, OpenSSL).
Zotac n5105
Network realtek

I notice this report https://github.com/opnsense/core/issues/6435 in Wich the user is having the same trouble on the wan interface. in the other case the interface is vmn0 and my trouble is on the lan re(0).  Could there be and ipv6 initialization problem on any 0(zero) interface or the first interface.

If 23.1.4 did work, I would try to revert the base package to that version first and restart, the report doesn't contain a lot of relevant details, the amount of changes between 23.1.4 and  23.1.5_4 are quite limited.
opnsense-revert -r  23.1.4 opnsense

(https://docs.opnsense.org/manual/opnsense_tools.html#opnsense-revert)

Thanks, I reverted and ipv6 was fully function immediately.  I also rebooted and ipv6 is functional.  You mention lack of info.  is the attached general log(glog.txt) not helpful?  It contains I believe relevant errors.

Immediately as in without reconfiguration right after revert. Usually it’s easy to find these things if reports are clear but here it isn’t.
The most likely change is https://github.com/opnsense/core/commit/eec08e055028674114b7d2e763b35d87fe7bd64e but I fail to see what could be going wrong unless radvd is still buggy.

I am not sure how radvd affects the LAN port.  Ifconfig shows the 2006:: address assigned by my ISP on the LAN interface yet, a ping from the LAN has no response and a ping from the router shows “ping: UDP connect: Network is unreachable”.  Why is the LAN interface ignoring the address?  Could this be a kernel problem?

I fear the questions here are not representative of the issue at hand. Debugging IPv6 starts with posting your WAN config and diagnosing if WAN is connected to the upstream router and if it has a GUA to talk to the Internet. After that we can switch to the LAN side. ;)

You might want to reread my problem description.  My WAN functions correctly.  from the opnsense console I can ping any address on the internet by IPV6 address or by DNS name.  From the opnsense console a ping to the LAN address(or any LAN address) yields “ping: UDP connect: Network is unreachable”.  It seems like the LAN interface is ignoring its own address.

Sure, WAN info is still completely missing. I’m writing this for you and anyone else later jumping into this with „I have the same problem“ and „why was this not fixed?“

I've been having this same issue (router has IPv6 connectivity but LAN hosts don't) on and off for several of the last releases and posted about it in the community forums but got little interest.  I'm not entirely convinced that it's version-specific rather than just a sporadic issue.  But packet traces on the LAN interface clearly show that the local clients have valid IPv6 addresses but every SYN packet they deliver to the router goes unanswered.
I will be more than happy to share whatever "WAN info" you need to track this down and fix it.  Specifically what details do you need?

Thanks @agh1701 and @Sot3 for bringing this issue also here on the GitHub repo.
IPv6 has been quite unstable for me too, and this time an upgrade to 23.1.6 broke things for good.
I would normally be able to make IPv6 work again after a reboot by running:
/usr/local/etc/rc.configure_interface wan
from the FreeBSD shell as root, but this time it did not work out.
I reverted back to 23.1.4 as hinted by @AdSchellevis on this thread by using the command he suggested:
opnsense-revert -r  23.1.4 opnsense
and without rebooting, IPv6 came back to life after giving the reconfigure_interface wan command listed above.
I will NOT update my OPNsense box any further, as IPv6 is really unstable at this time, and I'd rather have it than not.
Many thanks,
Luca

The only thing I see in 23.1.5/6 is the radvd SIGHUP change, which fixes intermittent connectivity issues during reconfiguration. There are no plans to change something there at the moment.
If that is indeed problematic I first need to know how it is problematic because I'm not seeing it (like most IPv6 instability issues reported) while having worked on IPv6 improvements since 2016 at least and running it here in production.
Thanks,
Franco

Hi Franco, appreciate you looking into this.
Thing is, I have IPv6 issues right on the OPNsense box after reboot, regardless of radvd or DHCPv6 having issues, I believe. In fact, when the box reboots, IPv6 does not work even by doing a ping6 from the OPNsense box CLI, so it's got nothing to do with radvd nor DHCPv6, which matter for machines on the local LAN, I believe.
After I force the WAN link reconfiguration from the CLI with:
/usr/local/etc/rc.configure_interface wan
then IPv6 comes up fine, and I can ping6 from the CLI too.
I have noticed that sometimes after boot the DHCPv6 daemon appears stopped on the dashboard, with the red stopped icon; after I force the WAN interface reconfiguration with the CLI command, it comes up fine and goes green.
All I can say is that moving back to 23.1.4 at least put me back in a situation where forcing a command from the CLI after a reboot makes IPv6 work fine. With the 23.1.6 update not even the CLI trick made things work.
Of course I would love to help by providing troubleshooting information and logs as required, as long as there is value in doing so. Let me know if you think I can help in any way.
Vielen Dank,
Luca

I have tried both patches from https://github.com/opnsense/core/issues/6522 and this did not work.  can you tell me what info i need to provied?

Dear all,
I would like to have a perfect radvd 2.20, it is possible to talk about your radvd problems upstream?
Several Issues/PRs are always opened:

https://github.com/radvd-project/radvd/issues
https://github.com/radvd-project/radvd/pulls

Thanks in advance.


Testing

Created to test , dont merge
https://github.com/TypeFox/gitpod-dwave/issues/110

@hemantdwave Is this test done? May I close this issue?

yes please delete it , i created this while testing ide functionality, thanks


Grafana Plugin oci-grafana-plugin did not support GOV tenancy

We are setting up oci-grafana-plugin for the Grafana on the UK Gov
The region name of UK Gov tenancy is uk-gov-london-1. However, when I created the datasource by oci-grafana-plugin. I am not able to find the the uk-gov-london-1 in the region dropmenu and there is only ashburn, phoenix and so on. The UK Gov tenancy didn't contain the ashburn, phoenix or the other region mentioned in the menu list.  The UK Gov tenancy only contains the region uk-gov-london-1.

When I created the datasource on the Grafana by oci-grafana-plugin, if I selected the region not existed in the tenancy, it will reported the error. So when I created the datasource for UK Gov tenancy, it will report error because the menu list didn't contian any region for the UK Gov tenancy .
This request is to support Grafana Plugin oci-grafana-plugin on the GOV tenancy. Thank you !

closing since gov regions are supported in recent version of the plugin


rPackages.igraph: fix Darwin build

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build rPackages.igraph

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: rPackages.igraph
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnfree = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnfree = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: rPackages.igraph
Partial log (click to expand)
** building package indices
** testing if installed package can be loaded
* DONE (igraph)
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/fiaqq2hyha39w50dmrjqmayqiwnwd6rp-r-igraph-1.2.1
shrinking /nix/store/fiaqq2hyha39w50dmrjqmayqiwnwd6rp-r-igraph-1.2.1/library/igraph/libs/igraph.so
strip is /nix/store/4mf2xm9p32lzrim927yk92xhx35yaz62-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/fiaqq2hyha39w50dmrjqmayqiwnwd6rp-r-igraph-1.2.1
checking for references to /build in /nix/store/fiaqq2hyha39w50dmrjqmayqiwnwd6rp-r-igraph-1.2.1...
/nix/store/fiaqq2hyha39w50dmrjqmayqiwnwd6rp-r-igraph-1.2.1

Success on x86_64-darwin (full log)
Attempted: rPackages.igraph
Partial log (click to expand)
** byte-compile and prepare package for lazy loading
** help
*** installing help indices
** building package indices
** testing if installed package can be loaded
* DONE (igraph)
post-installation fixup
strip is /nix/store/8q05v40n0f7amgw70pk6bkb23r7hrzgn-cctools-binutils-darwin/bin/strip
patching script interpreter paths in /nix/store/jns9nscb3m2hmaq0h9bbm2nx9wd2rmab-r-igraph-1.2.1
/nix/store/jns9nscb3m2hmaq0h9bbm2nx9wd2rmab-r-igraph-1.2.1


Timeseries extensions

Would it be possible to support timeseries extensions like pg_timeseries or timescale?
I have a use case for building a local-first charting app and having good time series support is key.
Thank you!

We'd love to support some time series extensions.
Our current focus is to make building extensions easer for the community. We intent to maintain a few core extensions ourselves but hope that the community can help to with others. Docs are coming.

Hey @samwillis I'd like to work on adding support for timescale. If you have any WIP docs or anything, I could try them out, and help reviewing them :)


Text is not retina ready

The text on the SVG badge is blurry and pixelated on high resolution screens.

Where are you seeing this behavior?

Sorry... should have given you that info... On Safari latest on Mac. Generally that happens when the SVG file is actually coded as smaller than the final size it is displayed at.

Cool, would you be interested in opening a PR to patch it?

Sure thing! Will do as soon as I can :)

Great, thanks =D

Hi @twolfson. I have made quite a number of changes to clean things up and make them more cross-browser compatible. I used outlined SVG contents instead of the basic format (which doesn't yet have full browser support for fonts) previously used.
I haven't submitted a PR yet, because there are a number of things that would affect other areas of the repo, and I wanted to first get your thoughts on my approach.
I added aria and accessibility to the SVG version too.
You can check it out here https://github.com/Cam/gratipay-badge

Cool, thanks for taking the time to work on this. I think the 2x PNG is a good idea (although a 2x SVG seems silly).
With respect to edits on the SVG itself, the badge's current size and style are based off of what GitHub uses. This goes to the extent that the CSS is ripped directly from GitHub. As a result, I would like to request that we don't define colors that aren't derived from the CSS somehow. I am open to other options like relocating the SVG template outside of HTML and dynamically loading it.
Outside of those changes, can you explain things like removing the text itself (I am guessing this is browser support as you mentioned but I would prefer articles/issues) and the text shadow?

Hey @twolfson. The SVG shouldn't be 2x. Where is that? I can fix it.
It may prove hard to get it to work across all current browsers without using more defined SVG, but I can look into options. Where is the GitHub badge you replicated? I'd love to take a look at what they have done, and how compatible it is.

Ah, sorry. I must have looked at the non-master version while comparing. It looks like you patched the 2x SVG name: https://github.com/Cam/gratipay-badge/commit/b7b42ade2cfd0aafb6681755dfc51a1ddd5f3930#diff-04c6e90faac2675aa89e2176d2eec7d8
The source is a GitHub button, not a badge. For example, the "Comment" on an issue page.

Ahhh... yep. The GitHub buttons use text inside the button, rather than a full SVG button. I can do something like that for you if you like, but the markup would be a little more complex. It could use an iframe, like the unofficial GitHub badges, but that doesn't seem perfect to me. Although, it does allow for updates to be pushed globally without people updating their code.

I think there was a context loss. The existing code on gratipay/master is already based off of it. We are generating PNGs and SVGs without issues except for our rasterization on retina displays.
The current SVG template was generated via Sketch which might be causing those issues. Maybe we could rewrite the retina incompatible parts and keep the CSS color extractions?

To get around it, the text element of the SVG should be "flattened" rather than actual text. That alone would remove the high res screen issues. I think adding the aria accessibility stuff too would be a good idea, as the text would no longer actually be machine readable text.
Could I have the sketch file, and the export method used, so I can create a compatible version?

The .sketch file was created by @lvillani. I only know about it due to a line inside of #9. @lvillani Do you still have the .sketch file used for gratipay-badge?

@Cam Can you explain why or link me to an article explaining why we need to rasterize a font for SVG? That seems counter-intuitive since fonts are vectors themselves.

@twolfson It's just an issue with where SVG is at right now. It affects Safari and Chrome on high resolution displays. Hopefully the SVG standards will be updated soon to fix the issue.

This is starting to sound more like a problem with the browsers than our SVG. Upon researching what other sites do, it looks like their SVGs still use text (which is much more easy to maintain).
https://api.travis-ci.org/gratipay/inside.gratipay.com.svg
http://viewsource.in/https://api.travis-ci.org/gratipay/inside.gratipay.com.svg
<text x="62.5" y="14">passing</text>

https://img.shields.io/github/downloads/atom/atom/latest/total.svg
http://viewsource.in/https://img.shields.io/github/downloads/atom/atom/latest/total.svg
<text x="35" y="14">downloads</text>

It is... in a way. But it's best to have cross-browser compatibility. I'm having a look through your SVG code now to see if I can find the problem...

Interestingly, if I remove the shadow, the problem goes away. I think the problem must be filter="url(#filter-2)"

Hm, any idea if there's another way to generate a text shadow in SVG that's compatible (e.g. 2 sets of text)?

Just testing that now :)

Yep! I managed to make an almost identical facsimile using two layers and opacity :)
Check out the lib/index.html on this fork to see it in action https://github.com/Cam/gratipay-badge

The code seems good but the SVG doesn't seem to be generated from the HTML yet =/
https://github.com/Cam/gratipay-badge/blob/0f4f58a1b60f515a0be7c24f37d18c35d8ee390c/dist/gratipay.svg
https://raw.githubusercontent.com/Cam/gratipay-badge/0f4f58a1b60f515a0be7c24f37d18c35d8ee390c/dist/gratipay.svg

Yeah, I wanted to see if you were happy with that method before spending any time on it ;)

!m @Cam
!m @twolfson

@rohitpaulk :)

@twolfson What do you think? Is the new SVG and PNGs to your liking?

The rendering doesn't looks so good on Firefox in Linux (Linux Min 17.1 to be specific) =/
Current:

New:

Ha! Damn. At least that is the least used platform. But still, the share is substantial enough for it to still be important. Are you open to using more suitable methods for the current state of SVG support? Something with real text should work for everything.

What about modernising it a bit and removing the shadow? That would fix all of the issues. Or do you really want it to mimic the GitHub buttons? Unfortunately I don't think you can have the best of both worlds and keep it effective and properly formatted for popular platforms.

I am trying to find issues filed for Chrome and Safari about our issue but am failing to. It would be nice to get an ETA on their solve times before making a decision =/
Also, I have a feeling it isn't the shadow itself but the upscaling of the shadow itself. Maybe we could render the text as 2x size with a 1x shadow and downsize it?

Hmm, searching again for "retina svg text shadow" found some possible options:
http://stackoverflow.com/questions/14664407/svg-drop-shadow-filter-pixelates-svg-on-mobile-safari
http://creativeandcode.com/fix-css-filters-creating-blurry-images-on-retina/

Hmmm... the "-webkit-transform: translateZ(0);" thing looks promising.

@twolfson let me know what you think will suit your preferences best.

I have a strong preference to keeping the SVG close to what it currently is. As a result, we should prob try out the workarounds to see if they work in GitHub READMEs =/


[Task]: Check on how to retrieve trace data for Ballerina/MI unified control plane

Description
Currently Trace data is being published via span exporters in opentelemetry. The published data is collected by the tracer providers such as Jaeger, Zipkin & New Relic. We can get the trace data via APIs exposed by the tracer providers.
But we need to collect trace data for Ballerina/MI unified control plane without a tracer provider. Therefore we need to find out how to collect trace data from an exporter in a json format.
Describe your task(s)
No response
Related area
-> Observability
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

We can create custom span exporters using SpanExporter interface in opentelemetry. We can write our own process to execute under export() method.

Our current metrics implementation uses pull mechanism for the metric providers to retrieve metric data.
As mentioned in the Jaeger documentation, Jaeger uses push mechanism to retrieve trace data. In our ballerina jaeger extension, we pushes trace data as a batch of spans to jaeger collector endpoint (http://localhost:4317) through GRPC Span Exporter.
We also work on migrating metrics  implementation into opentelemtry format. Therefore it will use a Metric Exporter which is very similar to Span Exporters with push mechanism.
The common architecture of push mechanism is that there is a trace data/metric data collector endpoint in each server which will store the data. Therefore, in the Ballerina/MI control plane, we need to have a data collector for opentelemetry.
We can use opentelemetry-collector which will collect all the data (traces, metrics & logs) and expose them through APIs.

Following diagrams show how to store observability data in Ballerina/MI control plane.

We have multiple users and users may have multiple instances. Each instance (microservice) exports observability data via exporters to opentelemetry collectors which will store the data (traces, metrics & logs). The data traffic is handled by a Data Collector Handler which will distribute the data among multiple opentelemetry collectors. We can use user_id of each instance to categorize which collector should be used to collect data.
When the dashboard needs data, it will request the data from the Data Retriever Handler and the handler will identify the collector which has stored the data for the given user_id. Then it will request the data from relevant opentelemetry collector and the data will be accessed via the provided APIs by the collector. Then the data will be forwarded to dashboard to represent in graphs or trace UI.

Following diagram explains how a ballerina application publishes observability data (traces, metrics & logs) to the Ballerina/MI Control Plane and how we retrieve the data from the OTLP collector to visualize in the dashboard.


Always delete the latest project

I'm attempting to delete a project, but it's always that the latest project was deleted.

Thanks for your feedback, I confirmed this situation.


butein, grandstanded

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[b,g]

/trunk merge


[BUG] CPM_ARGS GIT_TAG <something> Does Not Override versions.json

Describe the bug
If you have a git_tag field in versions.json, then rapids-cmake will insert a call to FetchContent_Declare(... GIT_TAG <something>). Internally, FetchContent will cache the results of this call in __FetchContent_declareDetails whose docstring states:
#=======================================================================
# Recording and retrieving content details for later population
#=======================================================================

# Internal use, projects must not call this directly. It is
# intended for use by FetchContent_Declare() only.
#
# Sets a content-specific global property (not meant for use
# outside of functions defined here in this file) which can later
# be retrieved using __FetchContent_getSavedDetails() with just the
# same content name. If there is already a value stored in the
# property, it is left unchanged and this call has no effect.
# This allows parent projects to define the content details,
# overriding anything a child project may try to set (properties
# are not cached between runs, so the first thing to set it in a
# build will be in control).

Note (emphasis mine)

Sets a content-specific global property [...] If there is already a value stored in the property, it is left unchanged and this call has no effect

This means that next time CPM goes to try and override these (i.e. in rapids_cpm_find()), FetchContent_Declare() they seemingly have no effect.
Steps/Code to reproduce bug
versions.json:
{
  "packages" : {
    "Legion": {
      "version" : "24.1.0",
      "git_url" : "https://gitlab.com/StanfordLegion/legion.git",
      "git_tag" : "1a5e1d237b4f07a39ddfde0bd43f30018045cd10",
      "git_shallow" : false
    }
}

CMakeLists.txt:
cmake_minimum_required(VERSION 3.22.1 FATAL_ERROR)

# <install rapids somehow>

project(Example VERSION 1.0 LANGUAGES C CXX)

rapids_cpm_init(OVERRIDE /path/to/versions.json)

rapids_cpm_find(Legion 24.1.0
  GLOBAL_TARGETS Legion::Realm Legion::Regent Legion::Legion Legion::RealmRuntime Legion::LegionRuntime
  BUILD_EXPORT_SET legate-core-exports
  INSTALL_EXPORT_SET legate-core-exports
  CPM_ARGS
    GIT_REPOSITORY https://gitlab.com/StanfordLegion/legion.git
    GIT_TAG master # NOTE: we want this to override versions.json!
    GIT_SHALLOW FALSE
    URL https://gitlab.com/StanfordLegion/legion/-/archive/master//legion-master.zip
)

$ cmake -S . -B build

Expected behavior
The above should download the master branch of Legion, not 1a5e1d237b4f07a39ddfde0bd43f30018045cd10.
Environment details (please complete the following information):

Environment location: Bare-metal
CMake version: 3.29.1

This unforunately can't be supported. The behavior you are seeing is the explicit use case that rapids_cpm_package_override was added to solve ( https://docs.rapids.ai/api/rapids-cmake/nightly/command/rapids_cpm_package_override/ ) . To be functional, it needs to override the behavior of rapids_cpm_find calls.
You have a couple of solutions:

Not have Legion in your override file at all. Use the rapids_cpm_find as you currently have it and it will use the master tag. If you want to force no local searches via find_package before building from source, you can add set(CPM_DOWNLOAD_Legion ON) before the rapids_cpm_find.
Not have Legion in your override file at all. Have N separate  legion_${tag}_versions.json files that you load after with rapids_cpm_package_override. Those files would have the different tags/sha1's/urls of legion that need to support and you would load only one of them based on the requested build information.

Hmm the use-case we are trying to support is: have a default git hash/commit for Legion then allow the user to (possibly) override at runtime. Do you have any recommendations on how to support this? Or is this just not feasible?

I opened https://github.com/rapidsai/rapids-cmake/pull/578 so that users better understand what exactly the OVERRIDE option is doing.

Hmm the use-case we are trying to support is: have a default git hash/commit for Legion then allow the user to (possibly) override at runtime. Do you have any recommendations on how to support this? Or is this just not feasible?

@Jacobfaib  I expect you want something like this:
if(NOT LEGION_GIT_TAG)
  set(LEGION_GIT_TAG 1a5e1d237b4f07a39ddfde0bd43f30018045cd10)
endif()

if(NOT LEGION_GIT_REPO)
  set(LEGION_GIT_REPO https://gitlab.com/StanfordLegion/legion.git)
endif()

rapids_cpm_find(Legion 24.1.0
  GLOBAL_TARGETS Legion::Realm Legion::Regent Legion::Legion Legion::RealmRuntime Legion::LegionRuntime
  BUILD_EXPORT_SET legate-core-exports
  INSTALL_EXPORT_SET legate-core-exports
  CPM_ARGS
    GIT_REPOSITORY ${LEGION_GIT_REPO}
    GIT_TAG ${LEGION_GIT_TAG}
    GIT_SHALLOW FALSE
    URL https://gitlab.com/StanfordLegion/legion/-/archive/master//legion-master.zip
)

This kind of pattern is used by projects like raft ( https://github.com/rapidsai/raft/blob/branch-24.06/cpp/cmake/thirdparty/get_hnswlib.cmake ). So that users can specify cmake -DLEGION_GIT_TAG=master to try out a new branch

so couldn't e.g. do the legion_${tag}_versions.json

Well... actually maybe we could. We could generate this at runtime if needs be. Are you allowed to call rapids_cpm_override() multiple times? I.e. would the following work:
rapids_cpm_package_override(version_except_legion.json)
if(Use_Default_Legion_Version)
  rapids_cpm_package_override(version_legion_only.json)
else()
  generate_version_json(User_Legion_Version)
  rapids_cpm_package_override(runtime_generated_legion_version.json)
endif()

Are you allowed to call rapids_cpm_override() multiple times? I.e. would the following work

Yes, that will absolutely work.


[🐛] launchImageLibrary doesn't return the data file when it is called

Description
When I call the launchImageLibrary I am successfully shown the images from my library and an empty array {} is returned to my function call.
When I click on an image I can see the details of the image in console.log no data is returned from my function. Later on the console log displays the image data.
How to repeat issue and example
selectFile = () => {
console.log("at the selectFile");
const options = {
title: 'Select Image',
customButtons: [
{
name: 'customOptionKey',
title: 'Choose file from Custom Option'
},
],
storageOptions: {
skipBackup: true,
path: 'images'
}
}
launchImageLibrary(options, res => {
  console.log('Response = ', res);

  if (res.didCancel) {
    console.log('User cancelled image picker');
  } else if (res.error) {
    console.log('ImagePicker Error: ', res.error);
  } else if (res.customButton) {
    console.log('User tapped custom button: ', res.customButton);
    alert(res.customButton);
  } else {
    console.log("got to the else launchImageLibrary");
    //sendURI(res.uri);
    return res;
  }
})

}

step to reproduce the issue
...

Solution
When launchImageLibrary runs it should wait and return the image from the launchImageLibrary call.
Additional Information

Image Picker version: [FILL THIS OUT: ^3.2.1]
React Native version: [FILL THIS OUT: 0.63.4]
Platform: [iOS]
Development Operating System: [FILL THIS OUT:  MacOS]
Dev tools: [FILL THIS OUT: Xcode and atom]

The callback shouldn't be called until an image is selected.  If it is in your instance, I recommend you create a reproducible example and share that for investigation.


Add db connection commands and create a non-root user for the web container

Fixes

Related to #952 by @zackkrida in the sense that, if I remember correctly, it was driven by a difficulty presented to @stacimc to connect to the API DB.
Description

This PR:

installs pgcli as dev dependency
adds a just dbshell and just pgcli commands, taking inspiration from the one in the catalog repository and using the provided django command as well
adds a non-root opener user to the web container, removing the tricky pass of the host user, it wasn't working well in macOS, so I believe creating a proper user is the best route we can follow
adds GITHUB_TOKEN to steps so setup just, it's necessary to prevent rate limiting errors (as said in docs)
bumps action dependencies to prevent warnings on deprecated commands

Testing Instructions

just up & just db-shell
Enter the development password: deploy
You should be able to make queries in the pgcli console

If you try these commands at the commit 2c1c7246dbf019c2cf1747211273923fa2e50826, without the user created internally, you will notice it fails (at least on macOS).
Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

I'm glad this ended up working out! Thanks a lot, @AetherUnbound, for your suggestion (in slack)! It was crucial for this. And also, @dhruvkb for suggesting adding dbshell by Django that doesn't require writing the password 😃


Support for Google Adsense?

Is there a plan to support google adsense to this theme?

@archeranimesh It is not a priority, but it shouldn’t be hard to implement. If you look into it and produce a pull request, we would be happy to merge it.


download_urls doesn't return any result

When use pornhub.page function, it doesn't return any download_urls. I've tried many links and none of them work

Hello urielexis64,
Thank for your finding,
Maybe Pornhub has updated their website again.
I will look at it after my work. I will let you know.

Thank you so much!

Hello urielexis64,
I have looked into it with more attention,
It works on some url but does not on some others.
PH apparently added a new layer of complexity in their algo that I need to reverse and understand.
I will let you know here as soon as I push an update.

Hello urielexis64,
I have looked into it with more attention,
It works on some url but does not on some others.
PH apparently added a new layer of complexity in their algo that I need to reverse and understand.
I will let you know here as soon as I push an update.

I have created a PR regarding this issue. Can you check ?


Run on ARM Android emulators on Apple Silicon

Android emulators will not be able to use Rosetta, which does not support whole system virtualization, so will be running in native ARM and not Rosetta translated x86.
Track down places in the tool where x86 is used as a proxy for "Android emulator".
https://github.com/flutter/flutter/blob/e0ec9a06dc8d244a9abb623d0dc4c5001c53b899/packages/flutter_tools/lib/src/android/android_device.dart#L229-L234
Blocked on b/162290052

The tool only makes best efforts guesses on what is an android emulator, because they are surfaced the same due to adb. The only restriction applied is that we prevent release mode from running on x86 because there is no release x86 slice.

I'm hoping there's almost no changes that need to be made here, but we can't know until the emulators start working.  This is a note-to-future-Jenn to test more than anything.

Hi, we have a preview out for ARM64 on ARM64 for Android Emulator on Apple Silicon:
https://github.com/741g/android-emulator-m1-preview

The underlying bug is closed, should it be?


bower.json "main" add  "src/material-design-lite.scss"

any objection to adding src/material-design-lite.scss to the "main" key in the bower.json file?
Currently must use a bower override in order to use MDL scss source in my build.

I am not sure what the semantic meaning of the main field is, but material.js feels just as main to me as the CSS.

I don't think we should worry about this right now. We should re-assess the package.json data with 2.x.
Also, ideally we would only include the raw sources over the compiled files if it is something we are really putting support  behind. Sass is currently more of a means-to-an-end that then got twisted into some wonky level of support.

So you are saying that eventually MDL will not use/provide access to the sass source?  Good to know because in which case I'll opt out of using MDL.

The SASS sources will still be in the repo, just not in the files provided via bower.

Whatever we use for a preprocessing setup, the raw files will still be provided in the NPM package (bower, no idea on that one.) I'm simply saying we should not add them to the package information without having an actual level of support internally for using the source files. Right now, the sass pipeline isn't great and quite a bit of it needs to be refactored or removed.
Until we put a focus onto supporting preprocessors, it shouldn't be a main thing. Provided for those that know how to use it, otherwise stick to the compiled files.


New RSE specialization: Open Source Developer

Open source developer RSEs work on large pieces of software relevant for the wider research community. Oftentimes, it will be general purpose statistical or data analysis library, such as Stan or SciPy.
They are probably closer to industry roles than other RSE specializations currently in the paper but I still believe it makes sense to consider them as RSEs because:

they need to understand the research lifecycle to inform the software lifecycle and their processes regarding breaking changes, new features rollout, etc.
they need to be able to communicate efficiently with researchers who are the majority of their userbase and contribute most bug reports & feature requests
it's more common for them to have a research background (or even to currently be affiliated to a research structure) than on average in industry

The name may sound similar to Open Science RSE but I believe they are very different: the Open Science RSE will mostly focus on making research artifacts open & FAIR, while the Open Source Developer RSE will play a role earlier in the research process, by providing low-level libraries used to produce these research artifacts.
In some way, the closest specialization is probably the Web-Development RSE but they focus on very different product and use very different languages & frameworks.

Hi,
discussion seems a bit cooled down.
Anyways... my take on this is the OpenScience-RSE, as currently described in the master version seems to be more of a domain RSE, that works in an open project. Therefore artifacts might be simulation data (and similar) and to make things reproducible a container or "reproducibility" with scripts.
An OpenSource-Dev-RSE will have no (or little) simulation output but his/her artifacts will be the software as packages, like those packages Bisaloo gave. E.g. interoperability, speed, efficiency is much more important than for a domain RSE. Exaggerated: OSource-Developer-RSE run unit tests, OpenScience-RSEs run simulations.
I see overlap with the HPC-RSE, but not so focused on hardware and administration, legal, environmental, scheduling and proposals.

I think this is now covered with a combination of "open science RSE" and "project/community manager RSE".

(don't hesitate to re-open - I am just in the process of cleaning up a little)


ignore the derps

recheck

Starting gate_j2sol/pyendeavor jobs.

Starting gate_j2sol/pyendeavor jobs.

Starting gate_j2sol/pyendeavor jobs.

recheck

recheck

Starting gate_j2sol/pyendeavor jobs.
https://bonnyci.io/status/

Starting gate_j2sol/pyendeavor jobs.
https://bonnyci.io/status/


⚠ ClamAV has degraded performance

In 7304447, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 226 ms

Resolved: ClamAV performance has improved in cdce9b4.