Smoke tests are broken in cf-d v12.33.0

Upgrading to v12.33.0 has caused open /dev/null: too many open files to be thrown when the tests try to log out. This is a blocking issue preventing the cloud.gov team from rolling out cf-d v12.33.0 to production.
Using deployment 'cf-staging'

Task 12178406

Task 12178406 | 16:31:27 | Preparing deployment: Preparing deployment
Task 12178406 | 16:31:50 | Preparing package compilation: Finding packages to compile (00:00:01)
Task 12178406 | 16:31:51 | Preparing deployment: Preparing deployment (00:00:24)
Task 12178406 | 16:31:51 | Creating missing vms: smoke-tests/181818d3-71a0-4027-a7e3-e46b83af3529 (0) (00:00:53)
Task 12178406 | 16:32:44 | Updating instance smoke-tests: smoke-tests/181818d3-71a0-4027-a7e3-e46b83af3529 (0) (canary) (00:01:13)
Task 12178406 | 16:33:57 | Running errand: smoke-tests/181818d3-71a0-4027-a7e3-e46b83af3529 (0) (00:01:14)
Task 12178406 | 16:35:11 | Fetching logs for smoke-tests/181818d3-71a0-4027-a7e3-e46b83af3529 (0): Finding and packing log files (00:00:01)

Task 12178406 Started  Thu Feb 20 16:31:27 UTC 2020
Task 12178406 Finished Thu Feb 20 16:35:12 UTC 2020
Task 12178406 Duration 00:03:45
Task 12178406 done

Instance   smoke-tests/181818d3-71a0-4027-a7e3-e46b83af3529
Exit Code  1
Stdout     Running smoke tests...
           Running binaries smoke/isolation_segments/isolation_segments.test
           smoke/logging/logging.test
           smoke/runtime/runtime.test
           [1582216438] CF-Isolation-Segment-Smoke-Tests - 4 specs - 2 nodes SSSS SUCCESS! 6.115259258s
           [1582216438] CF-Logging-Smoke-Tests - 2 specs - 2 nodes S• SUCCESS! 17.602344764s
           [1582216438] CF-Runtime-Smoke-Tests - 2 specs - 2 nodes S
           ------------------------------

           [2020-02-20 16:34:23.99 (UTC)]> cf push -b binary_buildpack -m 16M -k 16M SMOKES-2-APP-caaf7a7dc3468898 -p ../../assets/binary -d fr-stage.cloud.gov
           Deprecation warning: Use of the '-d' command-line flag option is deprecated in favor of the 'routes' property in the manifest. Please see https://docs.cloudfoundry.org/devguide/deploy-apps/manifest-attributes.html#routes for usage information. The '-d' command-line flag option will be removed in the future.

           Pushing app SMOKES-2-APP-caaf7a7dc3468898 to org cf_smoke_tests_org / space cf_smoke_tests_space as cf_smoke_tests...
           Getting app info...
           Creating app with these attributes...
           + name:         SMOKES-2-APP-caaf7a7dc3468898
             path:         /var/vcap/data/packages/smoke_tests/377cbbd917fe93953237530ffcc63bfd67d99cab/src/github.com/cloudfoundry/cf-smoke-tests/assets/binary
             buildpacks:
           +   binary_buildpack
           + disk quota:   16M
           + memory:       16M
             routes:
           +   smokes-2-app-caaf7a7dc3468898.fr-stage.cloud.gov

           Creating app SMOKES-2-APP-caaf7a7dc3468898...
           Mapping routes...
           Comparing local files to remote cache...
           Packaging files to upload...
           Uploading files...
            0 B / 594 B    0.00% 594 B / 594 B  100.00% 594 B / 594 B  100.00% 594 B / 594 B  100.00% 594 B / 594 B  100.00% 594 B / 594 B  100.00% 594 B / 594 B  100.00% 1s

           Waiting for API to complete processing files...

           Staging app and tracing logs...
              Downloading binary_buildpack...
              Downloaded binary_buildpack
              Cell f3b3f020-4811-45df-8498-d23d5871e3d7 creating container for instance 7c8f6a43-fcdf-49dd-ad5a-dc54292962d0
              Cell f3b3f020-4811-45df-8498-d23d5871e3d7 successfully created container for instance 7c8f6a43-fcdf-49dd-ad5a-dc54292962d0
              Downloading app package...
              Downloaded app package (594B)
              -----> Binary Buildpack version 1.0.36
              Exit status 0
              Uploading droplet, build artifacts cache...
              Uploading droplet...
              Uploading build artifacts cache...
              Uploaded build artifacts cache (215B)
              Uploaded droplet (747B)
              Uploading complete
              Cell f3b3f020-4811-45df-8498-d23d5871e3d7 stopping instance 7c8f6a43-fcdf-49dd-ad5a-dc54292962d0
              Cell f3b3f020-4811-45df-8498-d23d5871e3d7 destroying container for instance 7c8f6a43-fcdf-49dd-ad5a-dc54292962d0
              Cell f3b3f020-4811-45df-8498-d23d5871e3d7 successfully destroyed container for instance 7c8f6a43-fcdf-49dd-ad5a-dc54292962d0

           Waiting for app to start...

           name:              SMOKES-2-APP-caaf7a7dc3468898
           requested state:   started
           routes:            smokes-2-app-caaf7a7dc3468898.fr-stage.cloud.gov
           last uploaded:     Thu 20 Feb 16:34:33 UTC 2020
           stack:             cflinuxfs3
           buildpacks:        binary

           type:            web
           instances:       1/1
           memory usage:    16M
           start command:   ./app
                state     since                  cpu    memory     disk       details
           #0   running   2020-02-20T16:34:40Z   0.0%   0 of 16M   0 of 16M

           [2020-02-20 16:35:10.95 (UTC)]> cf app SMOKES-2-APP-caaf7a7dc3468898 --guid

           [2020-02-20 16:35:10.95 (UTC)]> cf delete SMOKES-2-APP-caaf7a7dc3468898 -f -r

           • Failure [47.166 seconds]
           Runtime: linux apps [It] can be pushed, scaled and deleted
           /var/vcap/packages/smoke_tests/src/github.com/cloudfoundry/cf-smoke-tests/smoke/runtime/runtime_test.go:52

             Timed out after 30.001s.
             Error: Unexpected non-nil/non-zero extra argument at index 1:
             	<*url.Error>: &url.Error{Op:"Get", URL:"https://SMOKES-2-APP-caaf7a7dc3468898.fr-stage.cloud.gov", Err:(*net.OpError)(0xc00117b4f0)}

             /var/vcap/packages/smoke_tests/src/github.com/cloudfoundry/cf-smoke-tests/smoke/runtime/runtime_test.go:73
           ------------------------------

           [2020-02-20 16:35:10.95 (UTC)]> cf logout

           Panic [0.000 seconds]
           [AfterSuite] AfterSuite
           /var/vcap/packages/smoke_tests/src/github.com/cloudfoundry/cf-smoke-tests/smoke/runtime/init_test.go:29

             Test Panicked
             open /dev/null: too many open files
             /var/vcap/packages/smoke_tests/src/github.com/cloudfoundry/cf-smoke-tests/vendor/github.com/cloudfoundry-incubator/cf-test-helpers/internal/cf.go:15

             Full Stack Trace
             	/var/vcap/data/packages/golang-1.13-linux/85a6ca409faf1c1a19614ff7006f5d5e43e5c7c5/src/runtime/panic.go:679 +0x1b2
             github.com/cloudfoundry/cf-smoke-tests/vendor/github.com/cloudfoundry-incubator/cf-test-helpers/internal.CfWithCustomReporter(...)
             	/var/vcap/packages/smoke_tests/src/github.com/cloudfoundry/cf-smoke-tests/vendor/github.com/cloudfoundry-incubator/cf-test-helpers/internal/cf.go:15
             github.com/cloudfoundry/cf-smoke-tests/vendor/github.com/cloudfoundry-incubator/cf-test-helpers/internal.Cf(0x988a20, 0xc00008b700, 0xc0034cd300, 0x1, 0x1, 0x34fc0)
             	/var/vcap/packages/smoke_tests/src/github.com/cloudfoundry/cf-smoke-tests/vendor/github.com/cloudfoundry-incubator/cf-test-helpers/internal/cf.go:9 +0x125
             github.com/cloudfoundry/cf-smoke-tests/vendor/github.com/cloudfoundry-incubator/cf-test-helpers/workflowhelpers.UserContext.Logout(0xc0000c28a0, 0x1e, 0x7fbba7ddc1d8, 0xc00008c0c0, 0x98c160, 0xc0000cf700, 0x0, 0x988a20, 0xc00008b700, 0xdf8475800, ...)
             	/var/vcap/packages/smoke_tests/src/github.com/cloudfoundry/cf-smoke-tests/vendor/github.com/cloudfoundry-incubator/cf-test-helpers/workflowhelpers/user_context.go:154 +0x9b
             github.com/cloudfoundry/cf-smoke-tests/vendor/github.com/cloudfoundry-incubator/cf-test-helpers/workflowhelpers.(*ReproducibleTestSuiteSetup).Teardown(0xc00013c1c0)
             	/var/vcap/packages/smoke_tests/src/github.com/cloudfoundry/cf-smoke-tests/vendor/github.com/cloudfoundry-incubator/cf-test-helpers/workflowhelpers/test_suite_setup.go:146 +0xa0
             github.com/cloudfoundry/cf-smoke-tests/smoke/runtime.TestSmokeTests.func3()
             	/var/vcap/packages/smoke_tests/src/github.com/cloudfoundry/cf-smoke-tests/smoke/runtime/init_test.go:30 +0x2a
             github.com/cloudfoundry/cf-smoke-tests/vendor/github.com/onsi/ginkgo/internal/leafnodes.(*runner).runSync(0xc00009ccc0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
             	/var/vcap/packages/smoke_tests/src/github.com/cloudfoundry/cf-smoke-tests/smoke/runtime/init_test.go:44 +0x20e
             testing.tRunner(0xc0000f6900, 0x90da68)
             	/var/vcap/data/packages/golang-1.13-linux/85a6ca409faf1c1a19614ff7006f5d5e43e5c7c5/src/testing/testing.go:909 +0xc9
             created by testing.(*T).Run
             	/var/vcap/data/packages/golang-1.13-linux/85a6ca409faf1c1a19614ff7006f5d5e43e5c7c5/src/testing/testing.go:960 +0x350

           ------------------------------

           Summarizing 1 Failure:

           [Fail] Runtime: linux apps [It] can be pushed, scaled and deleted
           /var/vcap/packages/smoke_tests/src/github.com/cloudfoundry/cf-smoke-tests/smoke/runtime/runtime_test.go:73

           Ran 1 of 2 Specs in 48.895 seconds
           FAIL! -- 0 Passed | 1 Failed | 0 Pending | 1 Skipped

           Ginkgo ran 3 suites in 1m12.6477299s
           Test Suite Failed

Stderr     Error: failed to run job-process: exit status 1 (exit status 1)

1 errand(s)

Errand 'smoke-tests' completed with error (exit code 1)

Exit code 1

//cc @bengerman13

Hi @mxplusb ,
Looking in our pipelines, we haven't encountered any errors of this type with the latest update to smoke tests. Based on the error message, we wonder if there is a relevant difference in our environments that explains the issue. What does the smoke tests job look like in your cf-staging deployment and what does the cloud config for that bosh director look like? And beyond that config, is there a reasonable file descriptor limit on the vm running the smoke tests errand? If you go bosh ssh onto the vm while it's running, what does it say the file descriptor limit is?
We're currently running smoke tests against standard bosh directors with task counts in the 10,000 range if the general age of the bosh director happens to be part of it (I notice the task number in these logs is up into some high values). My only other thought here is that maybe there's some other leak and this is coincidentally the process that breached the threshold. Is this problem reproducible with the previous cf-d release on your staging environment? For instance, if you try to run the new version of the smoke tests job in the previously stable deployment, does it work? An opsfile for this might just be a simple update on the smoke tests release version.
Thanks,
Andrew and John @pivotaljohn

The BOSH director we run this against as 12 million tasks under it's belt, which also means there's a lot of leftovers. There was an expired certificate that was missed, it had expired in-between v12.32 and v12.33, but the log message gave us zero indication that was the problem, it was just a panic. I'm closing this out.


Vagrant and Docker script changes for consul cluster integration

This set of changes brings in necessary scripting and configuration to bring up a consul cluster in our test/demo environments.
In subsequent PRs I shall bring in consul integration with the netplugin.
Note: As part of docker-in-docker script changes, I have moved to using the customized ubuntu image (viz. contiv/ubuntu) that is published and pulled from docker-hub. That image get's updated everytime a change is pushed to our Dockerfile in the repo (by using the automated-build feature of docker hub). This relieves us of managing the image every time Dockerfile changes. But this requires us to push Dockerfile to github first, which implies it requires some manual testing of image on Dockerfile editor's end. I need to see if I can eliminate this requirement while keeping goodness of automated-build.

@mapuri - I understand you already merged the changes; one small question - do we bring up both clusters (etcd and consul) with these changes and per the choice of runtime use one or the other?

do we bring up both clusters (etcd and consul) with these changes and per the choice of runtime use one or the other?

Yes, that is correct. For the test/demo environment I bring-up both the data stores. The idea is to be able to run systemtests for both the data-stores in one spin-up of the test environment. The data store will be chosen based on the configuration settings (for netplugin, netdcli etc). A little bit refactoring of code is needed there, which I am planning to push in my next PR.

I’m seeing some heisenbugs in the unit tests surrounding consul testing. Is it related to this patch?

On Jun 11, 2015, at 5:30 PM, Madhav Puri<EMAIL_ADDRESS>wrote:
This set of changes brings in necessary scripting and configuration to bring up a consul cluster in our test/demo environments.
In subsequent PRs I shall bring in consul integration with the netplugin.
Note: As part of docker-in-docker script changes, I have moved to using the customized ubuntu image (viz. contiv/ubuntu) that is published and pulled from docker-hub. That image get's updated everytime a change is pushed to our Dockerfile in the repo (by using the automated-build feature of docker hub). This relieves us of managing the image every time Dockerfile changes. But this requires us to push Dockerfile to github first, which implies it requires some manual testing of image on Dockerfile editor's end. I need to see if I can eliminate this requirement while keeping goodness of automated-build.
You can view, comment on, or merge this pull request online at:
https://github.com/contiv/netplugin/pull/96 https://github.com/contiv/netplugin/pull/96
Commit Summary
Add necessary configuration for consul cluster in Vagrantfile
Add necessary configuration for consul cluster in dind script
File Changes
M Vagrantfile https://github.com/contiv/netplugin/pull/96/files#diff-0 (15)
M scripts/dockerhost/Dockerfile https://github.com/contiv/netplugin/pull/96/files#diff-1 (19)
M scripts/dockerhost/start-dockerhosts https://github.com/contiv/netplugin/pull/96/files#diff-2 (40)
Patch Links:
https://github.com/contiv/netplugin/pull/96.patch https://github.com/contiv/netplugin/pull/96.patch
https://github.com/contiv/netplugin/pull/96.diff https://github.com/contiv/netplugin/pull/96.diff
—
Reply to this email directly or view it on GitHub https://github.com/contiv/netplugin/pull/96.

@erikh do you have failed test logs? I can check. There were some timing related conditions around updating state and starting watch back to back in the tests for which I had added a workaround by adding a sleep. Are you seeing failures in Watch related tests?


Add missing prop resource to SelectionDialog

Fix #6
FieldTitle handles i18n internally. It does so by, when field label is set, using it as key, and when only source is set, using key 'resources.${resource}.fields.${source}'. Currently, with undefined resource being passed to SelectionDialog, and so to FieldTitle, when no label is set, it uses key 'resources.undefined.fields.${source}'. Using that key on i18n files leads to correctly translated label with current version.

There's already a similar pull request.


iOS push notifications failing after several days

I'm facing an issue where a new iOS user will initially be able to receive FCM push notifications, but after a few days, stops being able to receive them. However, the server which is sending the push notifications still receives a success message from FCM:
{ multicast_id:<PHONE_NUMBER>438816000,
  success: 1,
  failure: 0,
  canonical_ids: 0,
  results: [ { message_id: '0:1471123826177313%13c7f1c613c7f1c6' } ] }

After I tried manually sending a notification to the relevant FCM token through the firebase console (which failed), the server then starts getting a failed message from FCM:
{ multicast_id:<PHONE_NUMBER>517732000,
  success: 0,
  failure: 1,
  canonical_ids: 0,
  results: [ { error: 'NotRegistered' } ] }

This led me to believe that the FCM token my server had was somehow out of date; however, I double checked, and the FCM token my server is using is consistent with the token given by calling FCM.on('refreshToken', (token) => ...  in the app.
Is there some other reason why this might be occurring?

The token may change after couple days. I assume it will be through either refreshToken or getFCMToken. Are you listening on both callback?

I am listening to both. The output of refreshToken still matches the FCM Token I stored on my server, but the messages still fail to send.

do you get refreshToken called all the time?

Yes, at every app start. Often it will first receive null, but it will be called a second time with the correct token value.

sorry for long response time. My firebase token doesn't get refreshed often, usually on when app is reinstalled.

@abc123s my guess is that there is something wrong in your code
could you post a snippet here please?

I didn't know that the registration token will get refreshed. If so, it'll be quite a problem. The problem can be solved when the user logs in and refreshes the token. But what if the user is inactive for a few months and the token is refreshed in fcm side, the user can't receive any notifications then.

gcm token refreshes too and I think they have similar behavior. the token should remain unchanged until user log in again, but that is just my guess

@evollu I think u can close this

Hello everyone, i have exactly the same abc123s's issue.
I used the PushController component that is showed on sample code and I call it  on main scene of my project.
I call FCM.getFCMToken() every time that a i want retrieve the Fcm token. This method call returns the refreshed token, isn't it?
What did i may get wrong?

@carlosfilipe what issue are you having?

A new iOS user will initially be able to receive FCM push notifications, but after a few days, stops being able to receive them.

do you check FCM.getFCMToken() on every app launch?

Yes!
The app starts on main scene and a I have the follow code:

		<View style={{ flex: 1 }}>
                    <PushController
                      firebase={this.state.configFirebase}
                      navigator={navigator}
                      onChangeToken={token => {
                        // console.log(`onChangeToken  ${token} `);
                        this.onChangeFCMToken(token);
                      }}
                    />
                    <ListScene
                      navigator={navigator}
                      firebase={this.state.configFirebase}
                      data={route.data}
                    />
                  </View>

PushController component is exactly like https://github.com/evollu/react-native-fcm/blob/master/Examples/simple-fcm-client/app/PushController.js
I have a component that is responsible for user configuration. On willmount, I call :
FCM.getFCMToken().then(token => {
      this.fcmToken = token;
    });

Then i use this.fcmToken object to save user config and the token on my server.
This only happen with one of our test iOS device ( we have two).
We observed that firebase user in this device, associated with this fcm token, had been excluded. But there is some relationship between firebase users and fcm? I think not...
(sorry about my english xD)

might be. If that user is no long associated with firebase, he won't get firebase token (it is the same as fcm token)

He keeps the same token… I think that is a firebase bug. We works with anonymous users. When we remove a firebase user by firebase console, he keeps signed in on app, with the previous token. So, if we only deactivate the user, the app verify that he is no signed, and create a new anonymous  user, with a new firebase token. And more, if we deactivate a user, and then remove it, when we start the app, firebase auth verify that users keeps signed in... a bit weird...
But a think that we don’t need use the firebase auth api to use fcm api, do we?

try listen on token refresh event and see if it returns anything.
otherwise I don't have any solution because it seems to be related to Firebase SDK itself

i have same problem :/

In case this is helpful for others - I experienced a similar issue, where the FCM token had not changed, but at some point the user was no longer able to receive remote push notifications. This was because we failed to regularly keep the APNs device token up to date: we did not call registerForRemoteNotifications appropriately on every app launch (i.e. in AppDelegate), as Apple requires. This method then triggers the Firebase-swizzled didRegisterForRemoteNotificationsWithDeviceToken to map the new APNs device token to the current FCM token.
This issue only seemed to occur on devices where we were regularly installing new app builds, so I suspect the APNs device token changes under certain circumstances, even when the FCM token is still the same.


replace sklearn by numpy in sdba.measures

Pull Request Checklist:

[ ] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #xyz

[ ] Tests for the changes have been added (for bug fixes / features)

[ ] (If applicable) Documentation has been added / updated (for bug fixes / features)

[x] HISTORY.rst has been updated (with summary of main changes)

[ ] Link to issue (:issue:number) and pull request (:pull:number) has been added

[ ] bumpversion patch has been called on this branch
[x] The relevant author information has been added to .zenodo.json

What kind of change does this PR introduce?
I replace th sklearn functions in sdba.measures.rmse and sdba.measures.mae by their numpy equivalent. This improves the performance with dask.
Does this PR introduce a breaking change?
No
Other information:

Coverage increased (+0.01%) to 91.595% when pulling f834a67cb8ed6c4832130183235e86b4726a1184 on fix-sklearn_measures into e6f803c7bcfa101e1aeef14036c337dcd6fac000 on master.


Bugfix/axiom curry

For a curried App of an Axiom, you only want to trigger normalization at specific spots. For this reason, Thorin maintains a Def::curry_ counter that each App decrements. The Axiom::normalizer() will be triggered when Axiom::curry() becomes 0. These are also the spots that you can match/force. After that, the counter will be set to Axiom::trip(). E.g., let's say an Axiom has this type:
A -> B -> C -> D -> E
          ^         |
          |         |
          +---------+

Using an initial value as 5 for Axiom::curry and 3 as Axiom::trip has the effect that here
x a b c1 d1 e1 c2 d2 e2 c3 d3 e3

the Axiom::normalizer will be triggered after App'ing e1, e2, and e3
In the above case you can use Axiom::infer_curry_and_trip to automatically retrieve these values (happens as default in the frontend). You can specify other values, as you like in the frontend after the normalizer name:
.ax foo: A -> B -> C -> D -> E, normalize_foo, 3;    // curry 3, trip 0
.ax foo: A -> B -> C -> D -> E, normalize_foo, 3, 1; // curry 3, trip 1

See also the unit test Axiom::curry for details.

@NeuralCoder3 I re-introduced the raw_app (now receiving the type of the App) because we don't need to recompute the dependent type twice. I also changed a couple of raw_apps in autodiff with proper apps. Seems to work.
I've also merged your latest work with inline optimizations into master and now in to this branch. Can you have a look whether this magic here actually fixes your problems?

I've also merged your latest work with inline optimizations into master and now in to this branch. Can you have a look whether this magic here actually fixes your problems?

Yes, the normalization for recursive types and higher-order returns are now working.


Rewards keep switching back to 0

There are rewards in the account. And sometimes the UI would jump back to 0 DSM and then show the correct value again.

@kwunyeung related to BDJuno Hasura

@kanniec826 @calvinkei @kwunyeung This is not related only to BDJuno and/or Hasura. It's related to the fact that Forbole X uses a subscription to keep watching the rewards value and update them. What happens is the following:

Forbole X subscribes to Hasura
When a new reward value is fetched from BDJuno, the existing  value is replaced inside the database
During the replace, Hasura fires two events: the first one with an empty value and the second one with the updated value
Since Forbole X is subscribing to events, it shows the updates every time

This is due to the fact that in order to implement live queries, Hasura keeps constantly querying the database for new data and emits the result of the latest query (more details here). What we could try to do is increasing the value of HASURA_GRAPHQL_LIVE_QUERIES_MULTIPLEXED_REFETCH_INTERVAL to above 1 second, but I'm not sure it will work.
Anyway, I think there is another problem inside Forbole X which is: it's using subscriptions even inside the withdraw rewards module. This makes the module continuously flickering due to the fact that rewards are constantly updating, which honestly gives me headache. I think it would be much better to use a query there instead @calvinkei

@RiccardoM when you say "which honestly gives me headache" do you mean it's affecting the server or you think the flickering number is an UX issue?

@RiccardoM when you say "which honestly gives me headache" do you mean it's affecting the server or you think the flickering number is an UX issue?

It's a UX issue: when I open the popup and it continuously update numbers I keep looking at it, which is distracting. I think that when you withdraw rewards you want to see fixed amount of DSM, not something that continues updating

After 5 days, I'm back again on this issue.
Unfortunately, there seems not to be an easy way of solving this from the side of BDJuno. The problem is that PostgreSQL does not allow concurrent INSERT statements on tables that rely on a CONSTRAINT (which is the case of all our tables actually).
I've also tried incrementing the HASURA_GRAPHQL_LIVE_QUERIES_MULTIPLEXED_REFETCH_INTERVAL time to 2 seconds (from the current 1) but that does not solve it either. And we can't make it smaller since it only accepts non-zero integers.
So, what I propose we do to fix this @calvinkei is simply remove the subscription and make it a query inside the account details. This will grantee that the values do not change over time causing all sorts of problems.

@calvinkei would it create too many https call if it's being changed to query with pollInterval?

if i query at a 3s interval i think it will be fine


currently flag -i and -d cannot be specified together

Hello Team
I am trying to execute below command
nerdctl run -t -d -i
-e GREENMAIL_OPTS='-Dgreenmail.setup.test.all -Dgreenmail.hostname=<IP_ADDRESS> -Dgreenmail.auth.disabled -Dgreenmail.verbose'
-e JAVA_OPTS='-Djava.net.preferIPv4Stack=true -Xmx512m'
-p 3025:3025 -p 3110:3110 -p 3143:3143
-p 3465:3465 -p 3993:3993 -p 3996:3996
ARTIFACTORY/ORG/greemail:1.5.4
It is giving below error
FATA[0043] currently flag -i and -d cannot be specified together (FIXME)
Please help me with solution . I am using nerdctl-full-0.11.0-linux-amd64.tar.gz

For most apps nerdctl run -t -d -i can be substituted with nerdctl run -d

How I can keep both interactive and detach mode together in  nerdctl  greemail container need both options to be enabled

Unimplemented


Random nations convert strangely

"I played EU4 with the feature that comes with the new patch in which nations are created randomly and they are named after their EU4 capitals. The converter made these custom nations into traditional nations seemingly at random. Some examples in the screenshot:
Glamorganshire (in Wales) --> Annam
Burgos (in Spain) --> Venice
Anjou (in France) --> Cologne
Bessarabia (in Romania) --> Vijayanagar
at the same time some were converted accurately, e.g.:
Varasd
Artois
Nyland"
https://forum.paradoxplaza.com/forum/index.php?threads/the-eu4-to-v2-converter-project.707362/page-152#post-19940140
Sample save linked here:
https://forum.paradoxplaza.com/forum/index.php?threads/the-eu4-to-v2-converter-project.707362/page-153#post-19940687

The issue is that random/custom nations will reuse existing tags. Fortunately, they seem to have custom_name and custom_national_ideas as items in their country data, so the country mapping code should be able to distinguish them from ordinary nations.


Add experimental macos support

@SFAntti I've merged your branch into this one.  This does build a binary, but I show a 'tk' error when running it, which IIRC was what was happening previously.
The GUI and stuff works fine, but when running this from source on my x86 macbook I do get an error mounting dirs, so not sure if that's something I need to tweak on my system or is an issue with this.

@kubilus1 , I just tried this branch and I do not encounter any errors with tk. What is the error you see?
If I don't eject the macFuse volumes between starting up autoortho I get this error:
mount_macfuse: mount point /Users/apiira/X-Plane 12/Custom Scenery/z_ao_na is itself on a macFUSE volume
mount_macfuse: mount point /Users/apiira/X-Plane 12/Custom Scenery/z_ao_eur is itself on a macFUSE volume
ERROR:autoortho:Exception detected when running FUSE mount: 1.  Exiting...
ERROR:autoortho:Exception detected when running FUSE mount: 1.  Exiting...

However, that does not prevent me from running AO ok.
I am on a M1 MacBook Pro running Sonoma 14.0.

@kubilus1 , I just tried this branch and I do not encounter any errors with tk. What is the error you see?
If I don't eject the macFuse volumes between starting up autoortho I get this error:
mount_macfuse: mount point /Users/apiira/X-Plane 12/Custom Scenery/z_ao_na is itself on a macFUSE volume
mount_macfuse: mount point /Users/apiira/X-Plane 12/Custom Scenery/z_ao_eur is itself on a macFUSE volume
ERROR:autoortho:Exception detected when running FUSE mount: 1.  Exiting...
ERROR:autoortho:Exception detected when running FUSE mount: 1.  Exiting...

However, that does not prevent me from running AO ok.
I am on a M1 MacBook Pro running Sonoma 14.0.

The tk error was from the built binary.  See: https://github.com/kubilus1/autoortho/actions/runs/6563296310
Yeah, I'll look more into why I'm having trouble mounting.  I don't think it's an issue with pending unmounts like you are showing.  Odd that is happening on your end.  On shutdown do you see the program attempt to unmount?  That should be logged.

On shutdown do you see the program attempt to unmount?

Yes, if I quit via the config UI everything is unmounted cleanly.


[CARBONDATA-1661] Fixed bug related to display of timestamp values in presto

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/788/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1421/

verified. LGTM


Bugs in NFW profile

Milky Way 1e12 mass gives vmax and vcir values that are probably too low. For c=5, I get 150km/s.
Also, when I try to compute vmax nfw.vmax(1e12, 15), I get a divide by zero error and a NaN result.

Actually, this seems to agree with Colossus and so is likely not a bug after all. No current reason to think NFWPhaseSpace has bugs, though we will see what the soon-to-follow unit-tests with Elise Jennings reveal.


Question about the implementation of Double-Head RCNN

First of all, thanks for this amazing project.
I was wondering why the implementation of Double-Head RCNN rescales the ROI before extracting the features for the bbox regression head. I read the paper and I have not noticed this implementation detail, and it appears to me this is the main reason why we can't use DoubleConvFCBBoxHead with other detectors (e.g. Cascade RCNN). While I can imagine giving a bigger ROI to the conv. head improves result, I am curious to hear if any study was done to measure the impact of this change when compared to using the same features for the fc head and the conv. head.

We got this implementation detail from the authors. It is not in the paper but used in the original implementation, which will bring 0.4~0.5 mAP improvement.

Perfect! Thanks for the quick answer!

I combine cascade with double and the performance is poor. Do you combine the two? How is the performance? Can you tell me the key details of their combination?

I combine Cascade RCNN with Double-Head RCNN and the performance is poor. Do you combine the two? How is the performance? Can you tell me the key details of their combination?

I combine Cascade RCNN with Double-Head RCNN and the performance is poor. Do you combine the two? How is the performance? Can you tell me the key details of their combination?

请问如何将Cascade RCNN加入Double-Head RCNN，我尝试失败了。
dear yhs95, can you hepl me how to add cascade RCNN to double head RCNN. My attempt failed.

I have tried to combine the cascade rcnn with double head and trained the model on my own dataset,the result is poor too.

我将Cascade RCNN与Double-Head RCNN结合使用，性能很差。您将两者结合起来吗？表现如何？您能告诉我他们组合的关键细节吗？

I combine Cascade RCNN with Double-Head RCNN and the performance is poor. Do you combine the two? How is the performance? Can you tell me the key details of their combination?

请问怎么在cascadercnn中用doublehead呀？能分享一下吗

I have tried to combine the cascade rcnn with double head and trained the model on my own dataset,the result is poor too.

dear zhaoxin111, can you hepl me how to add cascade RCNN to double head RCNN. My attempt failed.

I have tried to combine the cascade rcnn with double head and trained the model on my own dataset,the result is poor too.

dear zhaoxin111, can you hepl me how to add cascade RCNN to double head RCNN. My attempt failed.

我记得直接组合配置吧，好像没什么特别设置的

我尝试将级联 rcnn 与双头结合起来，并在我自己的数据集上训练模型，结果也很差。

Can you tell me how to add double_head to cascadeRcnn, I changed the configuration file, but it failed

I have tried to combine the cascade rcnn with double head and trained the model on my own dataset,the result is poor too.

dear zhaoxin111, can you hepl me how to add cascade RCNN to double head RCNN. My attempt failed.

我记得直接组合配置吧，好像没什么特别设置的

我配置失败了，其中roi head 的类型是设置为cascaderoihead 还是doubleheadroihead呢，然后在cascade rcnn中添加reg_roi_scale_factor=1.3这个配置不成功呢。请问你是怎么配置的，能分享一下你的配置吗？谢谢

I have tried to combine the cascade rcnn with double head and trained the model on my own dataset,the result is poor too.

dear zhaoxin111, can you hepl me how to add cascade RCNN to double head RCNN. My attempt failed.

我记得直接组合配置吧，好像没什么特别设置的

我配置失败了，其中roi head 的类型是设置为cascaderoihead 还是doubleheadroihead呢，然后在cascade rcnn中添加reg_roi_scale_factor=1.3这个配置不成功呢。请问你是怎么配置的，能分享一下你的配置吗？谢谢

请问你解决了吗

我之前成功了，不仅需要修改配置文件，你需要debug到股东的位置去查看那里的输出，我当时经过少量的代码修改成功实现了，但是，我可以很负责任的告诉你们，这样的效果提升是有限的，而带来的模型的重量确是罕见的，你可以仔细查看双头模型，你会发现他很重，在cacade起来，那么它将是一个巨无霸，喔经过一系列实验放弃了，因为我的认知一直是简约才优美，如果你希望实现这样的效果，就好好查看二阶段的代码吧，少量修改应该就可以达到了
发自我的iPhone

在 2022年6月25日，17:25，wzr0108 @.***> 写道：

I have tried to combine the cascade rcnn with double head and trained the model on my own dataset,the result is poor too.
dear zhaoxin111, can you hepl me how to add cascade RCNN to double head RCNN. My attempt failed.
我记得直接组合配置吧，好像没什么特别设置的
我配置失败了，其中roi head 的类型是设置为cascaderoihead 还是doubleheadroihead呢，然后在cascade rcnn中添加reg_roi_scale_factor=1.3这个配置不成功呢。请问你是怎么配置的，能分享一下你的配置吗？谢谢
请问你解决了吗
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.


Feature request: More customizable output

Thanks for this app, which is super simple and useful and has no equivalent on the play store!
My use case is the following: I often want to capture a webpage title and url, and some selected content of the page, to share it with a markdown-based note taking app.
The ideal format would be like: [{TITLE}]({URL}) {SOME SELECTED CONTENT}
This is just my personal use case, but I guess other people would like to format the output of markdownr in other different ways.
I think there are different ways to achieve this and that it could really increase the usability of the app. From the simplest solution to more complex (but also more powerful) ones :
Simpler solutions:

Similarly to allow to include or exclude front matter, url in content and excerpt as in the current version of the app, allowing to exclude the markdown translation of the page content would already make it much more flexible for my use case. (I could just include url in content and excerpt, the page title being included by default. With litlle post-editing of the output it will do the job.)
Even better, if the app output could be directly edited within the app, one could just reformat it as needed before sharing it with another app. Of course this could be done in the other app instead, but in my case it does not allow it (or at least not in a usable way).

More complex, but killer solution:

Provide a way for the user to specify its own template of desired output. In my use case explained above, the template would be [{TITLE}]({URL}) {SOME SELECTED CONTENT}. This means that markdownr would provide a set of keywords, like those within {.} in my example, that could be arranged in various ways as a template specification. For {SOME SELECTED CONTENT}, one could take advantage on the Android feature to share selected text of a web page, encoded in a url (at least in chrome, not sure if it works in other android browsers). E.g. if I select some text in this web page and share it, I obtain a string like https://github.com/sanzoghenzo/markdownr/issues/new#:~:text=Remember%2C%20contributions%20to%20this%20repository%20should%20follow%20our. (I.e. with the selected text encoded after #:~:text=.)

The complex solution probably requires quite a lot of work, but what is your opinion on the two simpler ones?
Thanks!

Hi @clement-moulin-frier, I'm glad you find this app helpful!
If you don't mind sharing, what is your target app?
To your points: solution 1 seems feasible, it would be another option to add to the menu and "just another if" (as my colleagues say when they want me to change something in my software 😅). I can give it a try as soon as I have some free time.
As for the point 2: I wanted to keep this app as simple as possible, mainly because android is not my usual/preferred development platform, but also because the apps to which I wanted to send the info already have a text editor in them, so I didn't want to reinvent the wheel.
Regarding the complex version: I like the template idea, that can also replace (and simplify to some extent, but complicate things on another level) the current settings;
the problem I see with the "some selected text" is that it is passed as plain text in the share intent, and that defeats the whole purpose of this tool (to convert html text to markdown).
To get a good result, the tool should find the (unformatted) text inside the downloaded html page and extract only that part. This is not impossible, but at the moment I can't appreciate the time I would spend implementing this feature  just to avoid some editing post conversion.

Hi @sanzoghenzo , thank you for the rapid reply :)

If you don't mind sharing, what is your target app?

Sure, my target app is the mobile version of Roam Research. It is a great (although proprietary) app, but the quick capture mode (using Android sharing) is quite minimal and does not allow for easy editing within the app.

To your points: solution 1 seems feasible, it would be another option to add to the menu and "just another if" (as my colleagues say when they want me to change something in my software 😅). I can give it a try as soon as I have some free time.

This one will already be a huge game changer!

As for the point 2: I wanted to keep this app as simple as possible, mainly because android is not my usual/preferred development platform, but also because the apps to which I wanted to send the info already have a text editor in them, so I didn't want to reinvent the wheel.

I totally understand that, editing should be the responsibility of the target app, not of markdownr (unless it is trivial to add that feature in markdownr, but I understand it is not the case). I should better make a feature request to the target app for this ;)

Regarding the complex version: I like the template idea, that can also replace (and simplify to some extent, but complicate things on another level) the current settings; the problem I see with the "some selected text" is that it is passed as plain text in the share intent, and that defeats the whole purpose of this tool (to convert html text to markdown). To get a good result, the tool should find the (unformatted) text inside the downloaded html page and extract only that part. This is not impossible, but at the moment I can't appreciate the time I would spend implementing this feature just to avoid some editing post conversion.

Indeed, my proposed solution would defeat the original purpose of html to markdown conversion, I didn't think about that. Yet, allowing template specification would still be super useful even without the feature of including selected text (could just be {PAGE CONTENT}instead). In particular, it will allow to format markdownr output as a standard markdown hyperlink with [{TITLE}]({URL}).
I'm sorry I can't help with a PR as I have no knowledge of neither JS or Android development :( (I'm just a humble Python developer). In case you find some time to implement something, I guess the first solution would the easiest and quickest, and would already be a super useful feature. On the other hand, the last solution is obviously the more general and flexible one, but probably require much more work.
Thanks again!

Hi @sanzoghenzo ,
Thanks a lot for the PR! As I'm new in using open-source apps on Android, I'm not sur me how to update to new version though. In F-Droid the last version is still 1.4.1.
Do you know if I should just wait that the update becomes available on F-Droid, or is there something else to be done?
Thank you

I'm sorry, I didn't prepare the release yet (I thought I could add some other refactorings, but it is taking more time than I expected.
I'll do a proper release in a minute, then it will take some time (I still don't know how much) for it to be available in F-Droid...

Thanks a lot, but no worries if you need more time to include more changes of  course

Hi @clement-moulin-frier,
I just released a fix for this feature, there was a bug (#47) hitting fresh installs.
I'm keeping this issue open to remind me and discuss more about the template solution.
I don't know if it's overkill, but using something like the jinja library allows more flexibility other than a simple replace of placeholders. The question here is: it is needed?
For the UX part, it is imperative to add a "settings" page in which configure the template.
I was thinking of a textbox in which to edit the template, and a list of ready-made templates, mostly to showcase some examples.
Another thing I am dubious about is the ability to save a custom template; I personally use only one style/template to send the article to my note-taking app, so I don't need this feature, but I'm open to suggestions.
(by the way, I'm also mostly a python programmer, but I had to learn other languages in order to survive at work and to create android apps :sweat_smile: )

waves hey guys, I thought I'd comment on this vs opening a new feature request since my needs are very similar to this one.. My target app is Obsidian, and I'd love the ability to set the number of hashtags at the beginning of the text content, or set the format to be title of page.. Right now since there's no way to remove the hash tag from the  slug created from the conversion, it's added to any existing note as an H1 heading, aka massive as heck. More hashtags (usually followed by a space before the text) = smaller header, but allowing for the traditional markdown format of square brackets for the link name and round brackets for the url itself would be amazing.
I found a fork for obsidian, but it seems to refer to building the app in flutter and no information on how this would create an apk for installation on Android https://github.com/IAmStoxe/obsidian-markdownr

Hi @sh0nuff, thanks for sharing your use case.
I found that we don't need templating engines, this little piece of code will suffice! As long as we don't want to perform transformation on the text returned by readability and html2md.
As of now, the available tokens to create the templates are:

url
title
author
creationDate: date formattted as yyyy-MM-ddThh:mm:ss
excerpt
content: the article translated to markdown

I kindly ask you (all of you) to provide me with the templates you need to have to be compatible with your destination applications, so that I can include them as pre-made templates along with the ability to specify a custom template.
I'll start with my preferred output (something similar to what Dendron wants):
---
title: {title}
desc: {excerpt}
created: {creationDate}
source: {url}
author: {author}
stub: false
isDir: false
---
{content}

As per the H1 heading being huge: doesn't Obsidian have the ability to set styles? the size of the texts is CSS responsibility, and starting with something different than a H1 title makes markdown linters angry. This is the equivalent to use Heading3 to style a microsoft word title instead of changing the Heading1 style font in its settings.

Thanks for replying so quickly! The other fork I posted has a format much more similar to what I would want, but I think I would honestly be happy with something as simple as
[Title](URL)
But if it were possible (I am very green to all things programming language/formatting) I would also appreciate
`Title
Excerpt'
(I am not sure what markdown code would be used to insert a tab before the content, but this would auto nest the excerpt under the previous line.
In regards to the Hashtag, I would prefer to not have one at all, and since I am not savvy with the issue caused by linters (I also just learned that term and went researching!) .. I did find this post that seems to discuss the issue to some degree, at least with github links, but I am not sure if it's applicable here..
Are you saying that there's no way to escape having a single hash at the beginning? It's true that I can use CSS snippets to adjust what various hashes  produce, the built in format for all things Obsidian is #(space)H1, ##(space)H2 and so on, so reassigning a single hash to a smaller heading varietal would break a million other plugins and existing notes. I only use a single H1 header in each note, at the top, and have no need for any sort of H format for each URL I share to each note, as I embed them under an H3 "Links" section.
Sorry for my lack of understanding, I am happy to learn!

Sorry, I understand what you (both @sh0nuff  and @clement-moulin-frier) are after, I never thought about using markdownr output to add to an existing note, I only create new notes and then edit them in my note-taking app, so I just assumed that a title should be inside a H1 as the heading of a page!
That having said, doesn't obsidian already support that? I read from reddit that you can share an url to Obsidian, and it asks to add it to an existing note... Or am I totally wrong?
In any case, once I got the time to implement the templating, what you're asking will be totally possible; you'll just need to use a custom template in the form of [{title}]({url}).

Sharing a URL to Obsidian .. just does that. A bare url, so I'm still forced to go back into my daily note at the end of the day and go through the laborious task of opening each one and creating a title/text slug for each.
Thanks! Basically my workflow is open link, share link to Markdownr, then share from there to Obsidian in existing note :)

Hello developer,
For me unchecking Include Body option is not useful because I get this:
`# Title
`
while I need just this:
[Title](URL)
As I build lots of lists (articles, programs, web resources) getting just the Title/URL will facilitate my workflow very much.
So my suggesttion is a new option:

Title/URL only (Title)

which will just extract the title of  the webpage plus the URL in Markdown format (no spaces, no new lines, no headers, just Title).
As a workaround when I use Obsidian I have installed Obsidian URL Namer plugin. But as main notes app is not Obsidian  I will greatly appreciate if this simple (in my opinion) feature would be added to Markdownr.
Regards

Hi there,
unfortunately I don't use this tool anymore, so I don't have the need and will to keep maintaining it and adding features.
That may change in the future, but for the time being you would have more luck by forking the project and adding the feature yourself (or ask somebody you know to do it).
Pull requests are still welcome!


password not accepted

Issue description
firefly will not accept my password. every letter has been checked and rechecked.
Firefly version
1.3.0
OS details
windows
Can the issue be reliably reproduced?
Yes
Steps to reproduce the issue
1.enter password
2.enter password
3.enter password
...
Expected behaviour
accept password
Actual behaviour
incorrect password entered
Errors
No response

Please join the IOTA Discord if you need assistance with this issue (visit the #help channel). You can also contact me via direct message if you want (rck#3810).
Please never share your seed, passphrase or any kind of sensitive information with anyone.

@waterloochris Were you able to solve your issue?

Hello thanks for reaching out. no the problem is still there. when i try to stake my coins my password is not accepted. im sure its the correct password. so then i tried to create a new profile in firefly but one of my recovery phrases is not accepted. thats where im at

You mean one of the words in your recovery phrase is not accepted?

thank you

thank you


GRANDPA consensus

Please have a look at https://github.com/w3f/consensus/blob/master/pdf/grandpa.pdf, and give your feedback vs. FFG and CBC. Note that this is pre-emptive opening this as I haven't finished reading the paper yet myself. Section 1.4 compares with Casper: https://raw.githubusercontent.com/w3f/consensus/master/pdf/grandpa.pdf#subsubsection.1.4.1. Use a Chromium browser to view this link, it doesn't work for me in Firefox, even with Preview PDF in Firefox set (by default) and the No PDF Download extension.

Care to respond?

Parity is currently building a 2.0 client in substrate. Although the promise of using substrate is high, they have had to make a few modifications to the underlying framework to accommodate for the 2.0 design (for example, being able to evaluate fork choice at slot boundaries rather than just when receiving a block).
I have not done a deep dive on GRANDPA but FFG and GRANDPA satisfy many of the same design goals -- finality gadget on top of a chain, safety under asynchrony, and plausible liveness. FFG is well studied and understood by the protocol researchers designing 2.0 and is already integrated into the spec. I don't expect the consensus algorithm at this point to change without a powerful argument.
It is far more likely that we switch the consensus algorithm to a different family with more desirable properties in the long term rather than switching from FFG to another finality gadget. For example, CBC for the beacon chain is under R&D, and looks promising in resolving issues that arise between the finality gadget and the fork choice.

I don't expect the consensus algorithm at this point to change

I'll second that. No actionable spec changes in this direction in the short term. Nudging the discussion towards ethresear.ch :)


Overdue outstanding integration

Notion card
https://www.notion.so/saeloun/Frontend-Integration-Overdue-and-outstanding-amount-in-client-page-19a7ae6fef9a4bdf85865cee6b387a81
Summary

Added overdue and outstanding amount integration with backend API.
Added helper functions which will give output of numbers in number + words

Preview
https://user-images.githubusercontent.com/104751221/167789112-9e91d5b8-df55-432a-8194-f7e2403f6818.mp4
Type of change

[ ] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Checklist:

[ ] I have manually tested all workflows
[ ] I have performed a self-review of my own code

@onkar-saeloun Please fix lint failing issues.

@onkar-saeloun Can you please take the latest changes from develop and re-run the job again?


@conda-forge-admin,please add bot automerge

Additional comment:
No response

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/gctree-feedstock#4.


fix(CustomViewportOverlay): pass accurate data to Custom Viewport Functions

Context
The current CustomViewportOverlay does not provide very useful data to any custom overlay function defined in the config.
This was discussed in a previous OHIF Office Hours: https://github.com/OHIF/OHIF-Office-Hours/blob/main/notes/2024-06-07.md

Changes & Results
This change is to provide the current displayed instance(s) if it is in the acquisition plane and a general reference instance for all other viewports. The current instance allows the user to write custom functions which might display particular instance level tags in the acquisition plane viewport such as SliceLocation and the reference instance allows the user to write custom functions which will make use of general tags such as SeriesDescription or ImagePositionPatient.

Testing
Here is an example of a custom overlay function you can define in the config to display SliceLocation:
cornerstoneOverlayTopRight: {
      id: 'cornerstoneOverlayTopRight',
      items: [
        {
          id: 'InstanceNmber',
          customizationType: 'ohif.overlayItem.instanceNumber',
        },
        {
          id: 'SliceLocation',
          customizationType: 'ohif.overlayItem',
          label: 'Loc:',
          title: 'Slice Location',
          instanceIndex: 0,
          contentF: ({ referenceInstance,
            currentInstance,
            viewportData,
            imageSliceData,
            viewportId,
            instanceNumber, formatters: { formatNumberPrecision } }) =>
            {
            if (!referenceInstance) {
                              return;
                           }
              var instanceNum;
              if (referenceInstance.ImagePositionPatient == null) {
                return;
              }
              if (viewportId.toLowerCase().includes("fusion")) {
                return;
              }
              if (viewportId.toLowerCase().includes("sagittal")) {
                instanceNum = imageSliceData.imageIndex;
                var res = formatNumberPrecision(referenceInstance.ImagePositionPatient[0] + (referenceInstance.PixelSpacing[0] * instanceNum), 2) + ' mm';
                return res;
              } else if (viewportId.toLowerCase().includes("coronal")) {
                instanceNum = imageSliceData.imageIndex;
                var res = formatNumberPrecision(referenceInstance.ImagePositionPatient[1] + (referenceInstance.PixelSpacing[1] * instanceNum), 2) + ' mm';
                return res;
              } else {
              instanceNum = currentInstance.ImageIndex - 1;
              var res = formatNumberPrecision(currentInstance.SliceLocation, 2) + ' mm';
              return res;
            }},
            }

Checklist
PR

[] My Pull Request title is descriptive, accurate and follows the
semantic-release format and guidelines.

Code

[] My code has been well-documented (function documentation, inline comments,
etc.)

Public Documentation Updates

[] The documentation page has been updated as necessary for any public API
additions or removals.

Tested Environment

[] OS: Windows 10
[] Node version: 16.17.0
[] Browser: Chrome 125.0.6422.142 (Official Build) (64-bit)

@wayfarer3130  As discussed last Thursday in Office Hours

I pushed an update to your branch to merge master.  Going to add a couple of changes and then let you look at them.

@wayfarer3130 Can you please check if these two issues are resolved? if not we fix it in this PR?
https://github.com/OHIF/Viewers/issues/4175
https://github.com/OHIF/Viewers/issues/4168

This has broken the sorting of the studies https://github.com/OHIF/Viewers/pull/4214


Change for mdn

@kaycebasques This is to follow what MDN does. Please look at it before I merge.

@jpmedley I wondered about that... whether to use a consistent constructor convention or to use the accurate signature


Add missing c query

I noticed that the c queries were missing enumerator_list, so I added it.

Merged, thank you.


p(blank symbol) >> p(non-blank symbol) during NN-CTC training

Hi all
I want to discuss some issue regarding training DNN/CNN-CTC for speech recognition. (Wall Street Journal Corpus). I modeled output unit as characters.
I observed that CTC objective function was increasing and finally converged during training.

But I also observed that final NN outputs have clear tendency : p(blank symbol) >> p(non-blank symbol) for all speech time frame as following figure

In Alex Graves' paper, trained RNN should have high p(non-blank) at some point like following figure

Do you have same situation when you train NN-CTC for sequence labeling problem?  I am suspecting that the reason is I use MLP/CNN instead of RNN, but I can't clearly explain why this can be a reason.
Any idea about this result?
Thank you for reading my question.

Hi,
I have quite the same experience with handwriting recognition.
I did some exploration of CTC training with different NNs during my PhD and the results are the following:

CTC training works especially well with RNNs
CTC training make NNs first learn to predict only blanks. It might take some time for relevant predictions to appear -> adaptive LR methods like RMSProp work very well to circumvent this issue
Maybe training 1 epoch with HMM Viterbi alignments before switching to CTC would help.. from scratch it might be hard to learn to align and transcribe.


Assassin's Creed Origins broken graphics (with apitrace)

As title, graphics are broken with DXVK under native Windows, as was discussed in #535. Game doesn't work with Wine at all, likely due to the game's convoluted DRM scheme.
Software information
Assassin's Creed Origins, default settings
System information

GPU: Nvidia 980ti
Driver: 398.82 (Windows)
Wine version: N/A
DXVK version: Git master

Apitrace file(s)

Windows apitrace

Log files

d3d11.log:  d3d11 log
dxgi.log: dxgi log

Thanks for recording a trace, but can you please upload it somewhere else (such as Mega, Google Drive, or Dropbox)? The file hoster you picked won't let me download more than half of it.

My bad, I fixed the link.

Thanks. Sadly, replaying the trace on my system causes a GPU hang, which means that I won't be able to do anything at all until the RADV/LLVM issues get fixed.

Something in the logs bothers me:
  Offset: 0
  Range:  461373440
  Format: VK_FORMAT_R32_UINT```

This is the same issue as in #537, #366. It looks like Nvidia reverted the fix for the driver bug which is causing these issues.

Good call, the Windows vulkan beta (398.91) has less broken graphical output, so presumably has that fix.

Game should render fine with latest master if you use a driver that includes the buffer view fix (see #278).


TVR-13872 Update v4.request-events.md

Add a comment "Whenever a Request is approved" and replace report to request a the title

:shipit:


Improve download scripts

I slightly changed the download scripts to be more verbose while decreasing tar verbosity (esp. in download_wick.sh). Also added support for calling download_all.sh from any location without breaking calls to the three scripts.
If you have any objections against any of these changes let me know and I'll amend.

Thanks, this looks cleaner!


Sub-menu items not reachable with the collapsed main navigation menu

Bug description
With the main navigation menu collapsed, sub-menu items are not reachable
Expected behavior
With the main navigation menu collapsed, the sub-menu items should be reachable through the same accordion style behaviour as when the main navigation menu is not collapsed
Steps to reproduce the issue:

Click on the 'Double Arrow' icon to collapse the main navigation menu
Click on any navigation tile with sub-menu options (excluding Host, this seems to work as expected)
No sub-menu options appear (Except 'Host', though some icons for submenus would be nice when the menu is collapsed)

Technical details:

Portainer version: 2.6.3
Docker version (managed by Portainer): Standalone on WSL - 20.10.8, Docker Swarm on Rocky Linux 8.4 - 20.10.8
Kubernetes version (managed by Portainer): N/A
Platform (windows/linux): Windows Subsystem for Linux 2 (WSL2)
Command used to start Portainer: docker-compose up -d

version: '3.9'

services:

  portainer:
    container_name: portainer
    image: portainer/portainer-ce:latest
    restart: unless-stopped
    ports:
      - '9000:9000/tcp'
      - '8000:8000/tcp'
    networks:
      - <Docker Network to join>
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
      - v-portainer-data:/data
    # command:
      # - --admin-password-file: admin-password.txt #create before deploy
      # - --edge-compute
      # - --host, -H: Docker daemon endpoint
      # - --logo: URL to an image to be displayed as a logo in the UI. Uses the Portainer logo if not specified
      # - --snapshot-interval: Time interval between two endpoint snapshot jobs expressed as a string, e.g. 30s, 5m, 1h… as supported by the time.ParseDuration method (default: 5m)
      # - --ssl true
      # - --sslcert /certs/portainer.crt
      # - --sslkey: /certs/portainer.key
      # - --templates, -t: URL to templates (apps) definitions
      # - --tlscacert: Path to the CA (default: /certs/ca.pem on Linux, C:\certs\ca.pem on Windows)
      # - --tlscert: Path to the TLS certificate file (default: /certs/cert.pem, C:\certs\cert.pem on Windows)
      # - --tlskey: Path to the TLS key (default: /certs/key.pem, C:\certs\key.pem on Windows)
      # - --tlsverify: TLS support (default: false)

volumes:
  v-portainer-docker.sock:
    external: true
  v-portainer-data:
    external: true

networks:
  <Docker Network>:
    external: true

Browser: Microsoft Edge Version 93.0.961.52 (Official build) (64-bit)
Use Case (delete as appropriate): Using Portainer at Home, Using Portainer in a Commerical setup.
Have you reviewed our technical documentation and knowledge base? Yes

@JamieMcH
Thanks for reporting this. Confirmed.


Update Invoice.php

Checking whether an invoice has a discount is inaccurate when an invoice has tax applied as the sub total and total do not match. This will resolve the issue, or a check for if the subtotal + tax equals the total would work too.

Has this failed just because of missing env files?

How about leaving the other check in there as well as adding the null discount check? Also you would probably want to send this to the default branch instead of master.

Done at #155 - thanks for all the great work on this, I've learnt quite a bit just from the source!


Many Problems with Feed Submission

Hello,
I am trying to use the amazon feed class to upload a single line of inventory. I know the feed works because I have submitted it through their interface and it works just fine.
I keep running into two issues that I cannot for the life of me solve.
The first issue is that I keep getting an error about the keys. The response says "Keys cannot contain" and then it has a "Tab" character. There is indeed tabs in my feed as it is a text tab delimited file. I was wondering why the tab would still be showing up, and it's showing because even though the content gets encoded into MD5, the code still puts the original content in the POST parameter. I don't know how or why to fix the issue. I tried to circumvent it by just excluding the post parameter, by making the following change.
$response = $this->sendRequest("$url?$query",array('Header'=>$headers));//,'Post'=>$this->feedContent));
I just commented out that second part. That got me through that error, but then I run into a second error.
It keeps telling me the signature calculated doesn't match the signature sent. I wasn't sure if there was something wrong with the signature creation, so I actually took the methods and calculations from a different API call that I had that was working, and pasted them into your code. There were only a few differences in how it calculated the signature, but even with this code, which I know works and tested several times before and after, I get the same error. I am at my wits end, I just can't figure out what's wrong with the signature. Can anyone help?
below is my code, its essentially the same as the sample. I filled out the configuration page correctly, it's the exact same values as the other API call I have that works. Though that one uses a completely different architecture, it does not have a good simple feed submission function like this one. I really hope I can get this to work.
$amz=new AmazonFeed(); //if there is only one store in config, it can be omitted
$amz->setFeedType('_POST_FLAT_FILE_INVLOADER_DATA_'); //feed types listed in documentation
$amz->setFeedContent($feed); //can be either XML or CSV data; a file upload method is available as well
$amz->submitFeed(); //this is what actually sends the request

$response =  $amz->getResponse();

I would appreciate any advise on the issue, thank you.

First of all, you need to specify store name while creating AmazonFeed, just like this:
$amz = new AmazonFeed("myStore");
Then, be sure that your $feed var contains valid string. If you are not sure, try to load content from file. Here is my working example:
    $amz = new AmazonFeed("myStore");
    $amz->setFeedType("_POST_FLAT_FILE_LISTINGS_DATA_");
    $filename = public_path() . '/' . $file; // your file url
    $amz->loadFeedFile($filename);
    $amz->submitFeed();
    $response = $amz->getResponse();

That sounds odd. Unfortunately, I mainly have experience with XML feeds, so I do not know the cause of the error message off-hand. What is the error code for the message? Knowing that code might help us look for answers.
I can already say that disabling the post body is not the way to solve the problem, because that is where the actual feed contents are sent. The contents are only converted to MD5 for the sake of creating a header, as Amazon cannot decode the MD5 hash. Since the signature is generated from the feed contents, changing the contents (by omitting it) will cause Amazon's generated signature to be different from yours, which is where the signature error comes from.
Normally, I would say that something like this would be outside the scope of this library, but the fact that you were able to upload the same feed on Amazon's site directly means the library could be doing something different.
I am not able to look into it myself at this time, so searching with the error code number might get some better answers in the meantime.

Thank you both for the quick response.
saikaze: I only have the one store, but I will try loading the file tonight and let you know what happens. Do you really think that has something to do with the signature?
Peardian: I am away from that workstation so I don't recall the error code but I do remember it saying it meant the signature didn't match. Yes, I eventually used rawurlencode to put the post back into the code, and that did get me past that part but again I was stuck on signature not matching. I checked the usual issues, it's definitely using the right secret key, and I see that it's putting the parameters in alphabetical order before generating the signature. The only other thing I can think to do is log the params one by one and see if they are all being included and if they are indeed in alphabetical order.

Don't worry about the signature mismatch, that likely isn't the problem. The mismatch came from disabling the post body.
You could also try using getOptions to see if any of the parameters or their keys have a tab in them for some reason.

The mismatch was happening even after I re-enabled the post body using rawurlencode on the feedcontent. At first it thought maybe it was because it just wouldn't take tab delimited files through the API, so  tried changing it into a csv. However when I do that, it then gives me the error   "Keys cannot contain ," . I'm curious, at this point, might it be easier to figure out how to write an XML for it than to keep pounding away at making a flat file work. Has anyone ever successfully used a flat file upload with this system?

Learning to write XML might possibly solve your problem, but it would not help catch any bugs in flat files if there is one. I don't know for sure whether or not anybody has done flat files before, but I haven't had any bug reports about it before, either.
I recommend trying a dump of getOptions, just in case the parameter keys are somehow getting messed up. You don't have to show me the output or anything, just let me know if all of the keys are tab-free.

Thanx, I will try both loading a file like saikaze said and dumping getOptions. I can probably show them here, though it won't be until tonight.

OK, I used a file and still got the signature error below:
[Urgent][2016/09/07 18:46:14<EMAIL_ADDRESS>AmazonFeed.php:263 checkResponse] Bad Response! 403 Forbidden: SignatureDoesNotMatch - The request signature we calculated does not match the signature you provided. Check your AWS Secret Access Key and signing method. Consult the service documentation for details.
And here I iterated through the options array and printed them.
SellerId => AT
AWSAccessKeyId => AQ
SignatureVersion => 2
SignatureMethod => HmacSHA256
Version => 2009-01-01
Action => SubmitFeed
FeedType => POST_FLAT_FILE_INVLOADER_DATA
I'm going to work on a different part but I'll back later tonight to give it another shot. If I can't get it to work, does someone know where I could get a good template?

Make sure you undo any of the code changes you did previously. Your original message makes it sound like you didn't start getting the signature mismatch error until after you started changing things in the code.
Also, I feel I should clarify, in terms of the progress from sending the message to Amazon and Amazon processing it, the "keys" error is farther along the process than the signature mismatch error.  So, the change in error responses was actually a step backward rather than a step forward.

Yes, I reverted all changes back to what they were, that last attempt was with no changes in the code.
Yes it's true, at first I was getting the "keys" error. I then changed the code to try to circumvent it but I changed that back to how it was a while ago. As of right now I'm back to the original code, only difference is now I'm loading the file instead of a string, and getting the signature error.

I've checked few things for you, and I can say, that Amazon could throw HTTP 403 (about wrong signature) just because can't understand your request. You can reproduce and confirm that error easy, just by comment feed content as you said at the beginning:
$response = $this->sendRequest("$url?$query",array('Header'=>$headers,'Post'=>$this->feedContent));  // That will work
$response = $this->sendRequest("$url?$query",array('Header'=>$headers)); // That won't
Because Amazon expects both, Header and Post, request has to contain them, if not, Amazon should throw missing Post error, but returning wrong signature error. I've checked it and can confirm that happened to me when I delete Post var from request, as you at the beggining.
Try to make clean installation, as I do, and everything should work.  Just do no more than here:
https://gist.github.com/saikaze/7dc4725a277c75461089081666b5c97f
After you do that, you will be sure that communication with Amazon is ok, and you can start to trying send valid feed content.

Uploading the feed from a string should work exactly the same way as loading from a file if the string's contents match the file contents perfectly. Are you using file_get_contents to make your string? If not, I would check to see if using it gives the same result as loading the file, or if it gives the same result as the string you were using before.

saikaze: I will give it another try with a clean install, but it won't be for another week, I won't be back in that office until the 19th.
Peardian: I will give that a try too. I'll post my results when I get back there.

@gsobero Have you had any luck with getting the feeds to work?

@Peardian Unfortunately no, we have limited time so I have moved to a completely different part of our project and will come back to this maybe next month. I hate to leave the issue hanging but I had to move on for the time being, I will come back to it though.

I completely understand. I will leave this open for a now, but I may have to close it if too much time passes unless someone else has the same problem.

It has been quite a while now, so I'm going to close this now.

Hi I have same issue, how you solved this?

A definite solution for this problem was not found, as indicated by the last few comments. I can reopen this issue if the advice here does not solve the problem.


SamplerV2 does not have optimization_level option though it is documented

Describe the bug
SamplerV2 does not have optimization_level option though it is documented as

V2 primitives support optimization levels 0 and 1 only.

https://docs.quantum.ibm.com/run/configure-runtime-compilation
Steps to reproduce
from qiskit_ibm_runtime import QiskitRuntimeService, SamplerV2, EstimatorV2

service = QiskitRuntimeService()
backend = service.get_backend('ibm_osaka')
estimator = EstimatorV2(backend)
print('EstimatorV2')
print(estimator.options.optimization_level)
sampler = SamplerV2(backend)
print('SamplerV2')
print(sampler.options.optimization_level)

output
EstimatorV2
Unset
SamplerV2
Traceback (most recent call last):
  File "/Users/ima/tasks/4_2024/qiskit/runtime/tmp/optlevel.py", line 10, in <module>
    print(sampler.options.optimization_level)
AttributeError: 'SamplerOptions' object has no attribute 'optimization_level'

Expected behavior
SamplerV2 supports optimization_level or the documentation is updated.
If SamplerV2 does not support optimization_level, what is the default optimization level used in SamplerV2 inside?
Suggested solutions
Additional Information

qiskit-ibm-runtime version: 0.21.1
Python version: 3.10.13
Operating system: macOS 14.4

fyi @beckykd


Speeds up and improves startup logic

With the introduction of ECC provisioning inside the hm-diag repo (see https://github.com/NebraLtd/hm-diag/pull/162), we can need to rework the logic for starting the miner.
This PR includes the following:

(Changes hard tabs to soft tabs in dbus-wait.sh)
Forces the miner to wait for hm-diag to start properly, which in turn is blocked by the key provisioning

Before merging, I need to review and make sure we're actually not using /var/data/public_keys anymore.

hm-config appears to be the only repo referencing it, but it's not actually used as far as I can tell:
$ grep 'ONBOARDING_KEY_FILEPATH' ./* -R                                                           [±master ✓]
./Dockerfile:# ENV ONBOARDING_KEY_FILEPATH=/opt/example/device_keys.txt
./gatewayconfig/constants.py:ONBOARDING_KEY_FILEPATH = "/var/data/public_keys"

@marvinmarnold can you confirm that this isn't used anymore?

Yes, hm-config is still using that filepath here.

./Dockerfile:# ENV ONBOARDING_KEY_FILEPATH=/opt/example/device_keys.txt

This is still a useful comment for debugging but the name is stale and should be MINER_KEYS_FILEPATH

./gatewayconfig/constants.py:ONBOARDING_KEY_FILEPATH = "/var/data/public_keys"

This variable is no longer being used and can be removed.


Contributor Signup

If you would like to help contribute to this project, please comment on this issue so that I can tag you in the future on other issues and PRs.

I'm up for contributing to this project - youugotssponged


Device related issue when using GV with --gv-state-grid-model-type=cnn

Hi,
Again, just trying to reproduce things with the GV envs, I'm trying to run the following:
python main_a2c.py ../gym-gridverse/gym_gridverse/registered_envs/gv_memory_four_rooms.7x7.yaml a2c --gv-state-grid-model-type cnn
But I get some error because I have a gpu and cpu on my machine and the computations are not being done on the same device. I'm not sure exactly how to fix this.
Loading using gym.make
Environment with id ../gym-gridverse/gym_gridverse/registered_envs/gv_memory_four_rooms.7x7.yaml not found. Trying as a GV YAML environment.
Loading using YAML
/gv_memory_four_rooms.7x7.yaml a2c

Traceback (most recent call last):
  File "/home/fishy/python_ws/aais-baisero3/code/asym-rlpo/main_a2c.py", line 684, in <module>
    raise SystemExit(main())
  File "/home/fishy/python_ws/aais-baisero3/code/asym-rlpo/main_a2c.py", line 648, in main
    done = run(runstate)
  File "/home/fishy/python_ws/aais-baisero3/code/asym-rlpo/main_a2c.py", line 471, in run
    episodes = sample_episodes(
  File "/home/fishy/python_ws/aais-baisero3/code/asym-rlpo/asym_rlpo/sampling.py", line 61, in sample_episodes
    return [
  File "/home/fishy/python_ws/aais-baisero3/code/asym-rlpo/asym_rlpo/sampling.py", line 62, in <listcomp>
    sample_episode(env, policy, render=render) for _ in range(num_episodes)
  File "/home/fishy/python_ws/aais-baisero3/code/asym-rlpo/asym_rlpo/sampling.py", line 23, in sample_episode
    policy.reset(numpy2torch(observation))
  File "/home/fishy/python_ws/aais-baisero3/code/asym-rlpo/asym_rlpo/policies.py", line 46, in reset
    self.history_integrator.reset(observation)
  File "/home/fishy/python_ws/aais-baisero3/code/asym-rlpo/asym_rlpo/features.py", line 164, in reset
    input_features = self.compute_input_features(
  File "/home/fishy/python_ws/aais-baisero3/code/asym-rlpo/asym_rlpo/features.py", line 126, in compute_input_features
    return compute_input_features(
  File "/home/fishy/python_ws/aais-baisero3/code/asym-rlpo/asym_rlpo/features.py", line 21, in compute_input_features
    observation_features = observation_model(gtorch.to(observation, device))
  File "/home/fishy/python_ws/aais-baisero3/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/fishy/python_ws/aais-baisero3/code/asym-rlpo/asym_rlpo/representations/gv.py", line 97, in forward
    return self.fc_model(self.cat_representation(inputs))
  File "/home/fishy/python_ws/aais-baisero3/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/fishy/python_ws/aais-baisero3/code/asym-rlpo/asym_rlpo/representations/cat.py", line 21, in forward
    [representation(inputs) for representation in self.representations],
  File "/home/fishy/python_ws/aais-baisero3/code/asym-rlpo/asym_rlpo/representations/cat.py", line 21, in <listcomp>
    [representation(inputs) for representation in self.representations],
  File "/home/fishy/python_ws/aais-baisero3/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/fishy/python_ws/aais-baisero3/code/asym-rlpo/asym_rlpo/representations/gv.py", line 256, in forward
    cnn_output = self.cnn(cnn_input)
  File "/home/fishy/python_ws/aais-baisero3/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/fishy/python_ws/aais-baisero3/lib/python3.10/site-packages/torch/nn/modules/container.py", line 139, in forward
    input = module(input)
  File "/home/fishy/python_ws/aais-baisero3/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/fishy/python_ws/aais-baisero3/lib/python3.10/site-packages/torch/nn/modules/conv.py", line 457, in forward
    return self._conv_forward(input, self.weight, self.bias)
  File "/home/fishy/python_ws/aais-baisero3/lib/python3.10/site-packages/torch/nn/modules/conv.py", line 453, in _conv_forward
    return F.conv2d(input, weight, bias, self.stride,
RuntimeError: Input type (torch.cuda.FloatTensor) and weight type (torch.FloatTensor) should be the same

Mmmh, full device support might be a bit wonky, because for reasons I won't get into, I've always run my experiments on CPU-only clusters.  I wonder where the torch.cuda.FloatTensor is coming from, maybe there's some environment variable that defaults the creation of some tensors to GPU?  I'll have to look into this.

I think some of the parameters of layers for the cnn case gets stored in gpu for some reason.
Looking at this part
File "/home/fishy/python_ws/aais-baisero3/code/asym-rlpo/asym_rlpo/representations/gv.py", line 256, in forward
    cnn_output = self.cnn(cnn_input)

and adding the following to that function
print('Device0: ', next(self.embedding.parameters()).device)

gives the result cuda at some point which then causes the issue while trying to process it with cpu based tensors.

Maybe what is happening is that you're running the code in an environment which is set such that only GPU devices are available..?  sadly, I don't have much experience with the mesh between CPU and GPU devices..

Could you try to set the environment variable export CUDA_VISIBLE_DEVICES = -1 before running the script?  Just to see if there is a way to force everything to go through CPU?

Yes, that works! I suppose just submitting jobs with only cpu devices should also work right?

I was running everything on CPU-only machines, so yes, I would assume so.
Apparently if GPU devices are available, some of the models are automatically created assigned to the GPU, while the rest of the code is not worrying about devices.. If you can run on CPU for the time being, that would be great.  I'll add a note that I'm not sure GPU saves all that much time compared to CPU, since the RNN computations are sequential in nature anyway.  But that's just a hunch.
I can come back to try and fix this at some point, it shouldn't be too hard overall, but I'll need to investigate a few things, and won't have the time in the next 1-2 weeks for sure.

Yes, I also don't think that running on GPU will be better by much. This is good enough for me, thank you!


Tooltip documentation needs improvements regarding custom tooltips

We should provide more docs on how to use custom tooltips and what problem they are intended to fix.
We should also do a better job documenting what the tooltipItem struct contains

Until this can be done here are some quick helpers for people looking for answers to common situations:
Here is how to put together a basic label:
 options: {
      animation: false,
           tooltips: {
              callbacks: {
                  label: function (tooltipItem, data) { return data.datasets[tooltipItem.datasetIndex].label + ' - ' + tooltipItem.yLabel}
                    }
                },

And here is how to return a label and then format the number to a currency:
options: {
  animation: false,
  tooltips: {
        callbacks: {
            label: function (tooltipItem, data) { return data.datasets[tooltipItem.datasetIndex].label + ' - ' + '$' + Number(tooltipItem.yLabel).toFixed(0).toString().replace(/\B(?=(\d{3})+(?!\d))/g, ",");}
                    }
                },

Hi guyz,
I've been runing around in circles trying to make this work, but your examples and docs never do so.
I have just tested the example Jaded-Design posted, the first one about "how to put together a basic label" and although i was able to add text to the end of the tooltip, the first part of it was left as "undefined", followed by empy space after the fist dash, and then a dash followed by the text i added.
Here's my code:
`
var ctx = document.getElementById("myChart").getContext("2d");
var config = {
type: 'pie',
data: {
                datasets: [{
                    data: [
                        100,
                        58,
                        85,
                        23,
                        65,
                        85,
                        23,
                        65,
                        65
                    ],
                    backgroundColor: [
                        "#F7464A",
                        "#46BFBD",
                        "#FDB45C",
                        "#949FB1",
                        "#4D5360",
                        "#dd921b",
                        "#1bdd7c",
                        "#4533b2",
                        "#e25ca1"
                    ]
                }],
                labels: [
                    "Depósitos à Ordem",
                    "Depósitos a Prazo",
                    "Contas Poupança",
                    "Seguros Capitalização",
                    "Carteira Investimento",
                    "Fundos Investimento",
                    "Títulos Investimento",
                    "Títulos Capital",
                    "Bolsa"
                ]
            },
            options: {
                responsive: true,
                tooltips: {
                    callbacks: {
                        label: function (tooltipItem, data) { return data.datasets[tooltipItem.datasetIndex].label + ' - ' + tooltipItem.yLabel + ' - my text'}
                    }

                },
                legend: {
                    display: true,
                }
            }
        };

        var myChart = new Chart(ctx, config);`

I know this is a free solution and you guyz do amazing work developping it, but realy, the documentation leaves a lot of empty holes... it would be nice to have full working examples that actually work using the most recent build, instead of its and bits scatered around gihub that when we try to put them togheter rarely work leaving us mere mortals desperate to make it work

Got it working here:
https://jsfiddle.net/vLw2zfs2/4/
Needed to have
return data.labels[tooltipItem.index]

Thx Jaded. With the corrections to your initial example i was able to get the tooltips showing... now this brings me to the issue i started on , which is how to change the text color of the tooptip text... i can change the background color, but none of the documented properties seems to take any effect on the text color. Can you help out on this?

Yeah, it doesn't look like there's an easy way to do it but this may help you find an answer as it looks like it's "possible":
https://github.com/chartjs/Chart.js/blob/master/samples/line-customTooltips.html

At the moment it seems like the following doesn't work:
Chart.defaults.global.tooltips.callbacks.label = function(tooltipItem, data) { return <blah> } .
I was able to get my custom function to work only if I reassigned the .label property after declaring the object. I'm using the CDN with v2.2.1.

I am having this same problem, this doesn't work:
Chart.defaults.global.tooltips.callbacks = {     label: function(items, data){       return items.yLabel + ' €';     }   }

Fixed in #3751


Node 6.2.0 - Error

Hi,
With Node 4.4.4 works very well. But in 6.2.0 happens this issue:
Installing in /home/ec2-user/Test forder...
path.js:7
throw new TypeError('Path must be a string. Received ' + inspect(path));
^
TypeError: Path must be a string. Received undefined
at assertPath (path.js:7:11)
at Object.dirname (path.js:1324:5)
at new daemon (/home/ec2-user/Test/system/daemons/node-linux/lib/daemon.js:208:30)
at Object. (/home/ec2-user/Test/install.js:10:13)
at Module._compile (module.js:541:32)
at Object.Module._extensions..js (module.js:550:10)
at Module.load (module.js:458:32)
at tryModuleLoad (module.js:417:12)
at Function.Module._load (module.js:409:3)
at Function.Module.runMain (module.js:575:10)

Hi!
Looking at the source code from daemon.js it is looking for a cwd property.
If you provide this in the configuration document you can get away with node 6.x
var svc = new Service({
  name: "servicename",
  description: "Service Description",
  script: "server.js",
  cwd: "/path/to/your/app/"
});

cheers!

is there any way to pass the cwd to the node process?

@kekkokk - there is now. I just merged a fix that allows the solution from @myAlterX to work properly. It's in version 0.1.11.

maaany thanks


Support CASE WHEN and BETWEEN

Closes: #487

Better to review this after PR is merged

The diff in GitHub haven't updated because of the force push. Would it be possible to merge your branch with the datafusion-sql-planner once again to ensure GitHub recalculates the diff properly?

Yep, sorry force pushed on purpose because the history was getting long and this is just a dev branch.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Ayush Dattagupta @.>
Sent: Wednesday, May 4, 2022 3:17:53 AM
To: dask-contrib/dask-sql @.>
Cc: Jeremy Dyer @.>; Author @.>
Subject: Re: [dask-contrib/dask-sql] Support CASE WHEN and BETWEEN (PR #502)
The diff in GitHub haven't updated because of the force push. Would it be possible to merge your branch with the datafusion-sql-planner once again to ensure GitHub recalculates the diff properly?
—
Reply to this email directly, view it on GitHubhttps://github.com/dask-contrib/dask-sql/pull/502#issuecomment-1116998864, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAQHLA27BZJRSZOCJRJ5KTTVIIQCDANCNFSM5UWNT6JA.
You are receiving this because you authored the thread.Message ID: @.***>

FYI - I'm waiting on https://github.com/dask-contrib/dask-sql/pull/509 to merge before continuing on and fixing the failing issues here.

Codecov Report

:exclamation: No coverage uploaded for pull request base (datafusion-sql-planner@11b2db4). Click here to learn what that means.
The diff coverage is n/a.

@@                    Coverage Diff                    @@
##             datafusion-sql-planner     #502   +/-   ##
=========================================================
  Coverage                          ?   53.01%
=========================================================
  Files                             ?       69
  Lines                             ?     3467
  Branches                          ?      699
=========================================================
  Hits                              ?     1838
  Misses                            ?     1507
  Partials                          ?      122

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Unable to use Craft mote spell

Every time I try to use the Craft spell, it posts in chat:
Craft threw an exception
(java.lang.ArrayIndexOutOfBoundsException: Index 9 out of bounds for length 9). This is a bug in the mod.
I don't think I'm using it wrong, I'm just trying to craft 8 cobblestone stairs as a test

Playing on Forge 1.19.2


random port for full static generate

Version
v2.13.1
Reproduction link
https://github.com/nuxt/nuxtjs.org/tree/dev
Steps to reproduce
Open any nuxt project and run dev. This will then use port of 3000. Then open a project that uses v2.13 and run generate which will use nuxt build and nuxt export. As port 3000 is already taken you cannot generate your site unless you close the other port.
What is expected ?
should generate a random port if port is in use
What is actually happening?
Nuxt Fatal error

This bug report is available on Nuxt community (#c10840)

cc @hecktarzuli

Fixed with v2.13.2

How can the random port be retrieved?

Hi @andre-brdoch You may use both nuxt.server.listeners or use nuxt.hook('listen', (server, listener) => {})
If wanted more details would be appreciated making a new issue with usage example :)


Bug: yarn.lock: could not unmarshal lockfile: yaml: unmarshal errors:

What version of Turborepo are you using?
1.1.1
What package manager are you using / does the bug impact?
Yarn v2/v3 (node_modules linker only)
What operating system are you using?
Mac
Describe the Bug
Running any turbo command, like yarn turbo run build:
 ERROR  yarn.lock: could not unmarshal lockfile: yaml: unmarshal errors:
  line 5: cannot unmarshal !!str<EMAIL_ADDRESS>into fs.YarnLockfile

error Command failed with exit code 1.

Expected Behavior
Should read yarn lockfile.
To Reproduce
Bug happens on this branch: https://github.com/tamagui/tamagui/pull/33
Just yarn install and yarn turbo run build.

https://github.com/tamagui/tamagui/blob/ac9ae69d3c4232ca06591c1af7cf9f61bff2ffc1/package.json#L55
Make sure to set this line to Yarn V1, as that's what your lockfile version is.


0.1.5

Addressed issues #18 and #19 as well as adding yet more functions! whee!

All checks have passed, let's publish!


AudioLink Prefab Camera Disables Post Processing in Scene View

When using the AudioLink prefab with its camera component (so I can have a basic preview of my lighting without uploading to VRChat), the post-processing in the scene view is disabled, but still works in Game View... Is there a workaround for this?
Many thanks

I'm not sure I understand how to recreate this issue, but it seems like your scene is missing a default Camera where the post-processing should live. You should have to view the camera on the audio link component to get audio link to work.


Cleanup AKS autoscaler help documentation

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:
docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/seanmck/azure-cli-extensions.git@aks-autoscale-help#subdirectory=src/$EXT&egg=$EXT"


x/tools/gopls: consider using &S form instead of *S for pointer type struct field autocompletion.

Currently autocompletion for a struct field uses the field's type as the placeholder text. For example,
type F struct {...}
type S {
   Field *F
}

The autocompletion for the Field will return "newText":"Field: ${1:*F},".
gopls uses the type info for placeholder text, so I don't think the choice is incorrect nor invalid.
But if the suggestion used &F instead, it would be more convenient when users want to use  composite literals. Especially, if the type name is long to type, reusing what's already in the placeholder will save many keystrokes.
(Based on https://github.com/golang/vscode-go/issues/85)

/cc @muirdm

This was discussed briefly in #38416, but we should consider getting rid of the struct field placeholder text altogether. It is weird to insert invalid, informational Go code when most editors already present the type information to the user.
Composite literals prepend the "&" for you automatically as needed, and sometimes you don't want a composite literal, so I don't think having "&" placeholder text is a net win.

@muirdm Interestingly I found the struct field placeholder is useful when dealing with badly named long type names with common prefixes, even though I dislike the current way of formulating function parameter place holders.
I don't know about other editors, but at least, in VS Code the suggested text syntax is followed. So if I don't want a composite literal, just typing the other value at the cursor completely replaces the entire suggested text with my input. Actually, that's easier than using the suggested text.
VS Code presents the type info as a hover message, which means I have to hover over the field to remember the type name to trigger meaningful autocompletion results. Additional cognitive overhead. It would be convenient if &someproto.VeryLongMessage{} was suggested. (And, syntax-wise, it's a valid syntax)
As a workaround, I hoped just typing & triggered completion and suggested the suitable composite literal snippet for the field, but it doesn't seem like VS Code ever triggers autocompletion with this symbol (&) for a reason I don't know. :-(
BTW I didn't encounter automatically prepended "&". Is there any configuration I need to set to enable it? Or, do you mean the fact that the completion response includes &F{} as one of the candidates?

It would be convenient if &someproto.VeryLongMessage{} was suggested.

Yes, that would be convenient in many cases. And of course it also contains the type information of the current placeholder.

As a workaround, I hoped just typing & triggered completion

You can also try triggering completion without typing anything at all (ctrl-space in VSCode I think). That will often pop up the literal candidate. I think you can use ctrl-space to trigger completion after "&" as well. Perhaps you can configure VSCode's "trigger" characters to include "&".

I didn't encounter automatically prepended "&" ... do you mean the fact that the completion response includes &F{}

Yes, the candidate will include "&" if it needs to be a pointer. In your example you could type "Very" and probably get a candidate for "&someproto.VeryLongMessage{}".
@stamblerre how do you complete a long composite literal for a struct field value when you don't remember the name? Do you rely on hover popup, trigger completion manually, or something else?

@stamblerre how do you complete a long composite literal for a struct field value when you don't remember the name? Do you rely on hover popup, trigger completion manually, or something else?

Ah, so I was confused by this and now I see why this has never been an issue for me. I don't use placeholders, so when I complete a struct literal, it looks like this:

(This image shows a completion triggered at Field: <> inside of the S struct literal, with the first option being &F{}.)
I also trigger completion manually a lot with Ctrl + Space. This flow works well for me, but I could see it being hard with placeholders. I would support having the placeholder be an &veryLongName{<>} with the <> marking where the user's cursor ends up. I'll turn on placeholders for a few weeks to learn more, and obviously I defer to the opinions of anyone who does use placeholders.


more datasets

not sure if some of these are already on the list
https://aws.amazon.com/datasets/
https://cloud.google.com/bigquery/public-data/#public-datasets-hosted-by-bigquery

Yes, they are.