Locality Load Balancing is not working for multi-cluster mesh

Bug description
I setup a multi-clusers mesh, the 2 clusters are in different regions.
follow: https://archive.istio.io/v1.3/docs/setup/install/multicluster/shared-gateways/
I want test 「Locality Load Balancing」 in this mesh.

why 「distributed between instances on cluster1 and cluster2 」?
[ x]Networking
Expected behavior
if  「Locality Load Balancing」 is working ,  pod sleep in cluster1 should always access helloworld-v1, because helloworld-v1 is in cluster1
pod sleep in cluster2 should always access helloworld-v2
is there something Im missing? how can I make 「Locality Load Balancing」 working?
this is the eds info from pilot debug port:

the configmap istio in primary cluster:
localityLbSetting:
  enabled: true

Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
k8s: 1.14
istio: 1.3.6
How was Istio installed?
https://archive.istio.io/v1.3/docs/setup/install/multicluster/shared-gateways/

you need to configure outlier detection
https://preliminary.istio.io/docs/ops/configuration/traffic-management/locality-load-balancing/#requirements

hi @howardjohn
then I add this dr:
apiVersion: networking.istio.io/v1alpha3
kind: DestinationRule
metadata:
  name: helloworld
  namespace: sample
spec:
  host: helloworld.sample.svc.cluster.local
  trafficPolicy:
    tls:
      mode: ISTIO_MUTUAL
    outlierDetection:
      consecutiveErrors: 3
      interval: 30s
      baseEjectionTime: 3m

after this,  the cluster1 seems ok, sleep in cluster1 will always access helloworld v1, which is in cluster1
but, sleep in cluster2 still can access helloworld v1 and helloworld v2
% kubectl exec --context=$CTX_CLUSTER2 -it -n sample -c sleep $(kubectl get pod --context=$CTX_CLUSTER2 -n sample -l app=sleep -o jsonpath='{.items[0].metadata.name}') -- curl helloworld.sample:5000/hello
Hello version: v1, instance: helloworld-v1-7bb88866c4-6bfp5
% kubectl exec --context=$CTX_CLUSTER2 -it -n sample -c sleep $(kubectl get pod --context=$CTX_CLUSTER2 -n sample -l app=sleep -o jsonpath='{.items[0].metadata.name}') -- curl helloworld.sample:5000/hello
Hello version: v2, instance: helloworld-v2-6cd449dff4-qvwq6

figured out：

when remote pod connect to pilot, pilot will try to find the pod info, from the aggregate registries, pilot will travel all the registries(primary cluster registry, service entry registry, remote registry ...)

https://github.com/istio/istio/blob/1.3.6/pilot/pkg/serviceregistry/aggregate/controller.go#L234

in version 1.3, some patch logic was added to GetProxyServiceInstances: pilot will try to find pod info from metadata:

https://github.com/istio/istio/blob/48148f5be3c6d84fe1fce8a0491d3c89593145a2/pilot/pkg/serviceregistry/kube/controller/controller.go#L532
ISTIO_META_CLUSTER_ID is important meta.
but in the demo: https://archive.istio.io/v1.3/docs/setup/install/multicluster/shared-gateways/
the pods of remote cluster don't set ISTIO_META_CLUSTER_ID properly. in both primary cluster and remote cluster, this meta are Kubernete
so: when pilot try to find remote pod info from the aggregate registries,  it first search in primary cluster,  no result by using ip,  and then try GetProxyServiceInstances, now the pod ISTIO_META_CLUSTER_ID is the same as primary cluster name, so pilot treat this remote pod is from primary cluster. result to the pof locality missing.


The slider sometimes does not show up

The issue is like in the image. I have a lot of products in one page and each one has a slider. Link: http://www.ouluntyostokeskus.com/fi/miekat
I notice that the last slider in the first row (and also some other random places) always not show up unless i resize the browser to a breakpoint. i have no idea why this happens, can you help? Thanks!

It has something to do with the width not being calculable by the JS untill all are visible. I'm not 100% sure this will apply for you, but I found something that worked for me without changing the js.
I have added the following css:
#parent-id-or-class > .tab-content > .tab-pane {
	display: block;
	height: 0;
	opacity: 0;
}

#parent-id-or-class> .tab-content > .tab-pane.active {
	height: auto;
	opacity: 1;
}

This ensures the width is calculable while the other tabs are invisible.


styx: 0.4.0 -> 0.5.0

Motivation for this change
Update styx and themes to the latest version.
Things done

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

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Thanks for merging!


Improve configure script, header includes - [merged]

In GitLab by @perarnau on Jun 27, 2019, 10:32
Merges build-improvements -> master
Trying to compile AML with other compilers or extra compiler flags shows that the build infra could be improved. Starting with configure and config.h...
This is just small stuff, we'll register more job steps soon to enforce more rules.

In GitLab by @perarnau on Jun 27, 2019, 10:33
merged

In GitLab by @perarnau on Jun 27, 2019, 10:33
mentioned in commit 506010453d113c88e0565186b7a6ccce7434940c


package layout

Before the first release, we need to improve the package layout.

In general, I still find our package structure quite confusing.
Worse, it's not very clear which packages are public APIs and which aren't.

(Note that impl packages are fine when you have a package containing APIs, and then a package containing implementations of those APIs. That's not the function they're performing here.)
We currently only expose three APIs to clients: Stage, Mutiny, and ReactiveConnectionPoolProvider/ReactiveConnection. But it's almost impossible to see that by looking at our source code layout.
Instead of using lots of impl packages to indicate non-public APIs, I would propose the following top-level layout:

org.hibernate.reactive.stage for the CompletionStage API
org.hibernate.reactive.mutiny for the Mutiny API
org.hibernate.reactive.connection for the abstraction over reactive connections
org.hibernate.reactive.internal package for everything else

Alright, so I'm now much happier with the package structure, and I can finally start to see that codebase coalescing into a set of logical subsystems.
I don't think we need to introduce the internal package after all because we would wind up with ReactiveIdentifierGenerator in the (non-internal) id package exposingReactiveSession from the internal session package. And I think we'll likely add other APIs which expose ReactiveSession.
I still want to get rid of some impls, but I don't want to cause further disruption by doing it right now.
So I think I'm going to close this for now.


⚠ Samsung has degraded performance

In d55a767, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 1799 ms

Resolved: Samsung performance has improved in 8cda57d.


create a logger for quality control

Currently the QualityControl class uses puts to do logging. Upgrade this to a proper logger so you can get feedback while using the tool but it doesn't show up when running the test suite.

I would call this a 3 point story but I've never made a logger before. Shouldn't be hard? Probably?

Are you making a logger from scratch or just injecting the rails (or
another existing) logger into it?
On Tue, Dec 15, 2015 at 4:32 PM William Jeffries<EMAIL_ADDRESS>wrote:

I would call this a 3 point story but I've never made a logger before.
Shouldn't be hard? Probably?
—
Reply to this email directly or view it on GitHub
https://github.com/heatseeknyc/heatseeknyc/issues/31#issuecomment-164848661
.

Good question. I'm actually not familiar with the rails logger. Given that Rails already has a logger, I would say we should probably just use that. We need it to log to the console when using rails console but not log to STDOUT when running the test suite. Can the Rails logger do that?

It outputs to a log/(test|development|production).log file and to STDOUT as the server output (rails s). Under rspec it outputs only to the test.log file.
On the console, you'll still need puts to see anything.
You can always leave a terminal open tailing the log to see it in real time, no matter which environment you're in:
$ tail -f log/development.log
$ tail -f log/development.log

Hmm, perhaps the best solution then is to pass an optional parameter that suppresses the puts during testing instead of messing with loggers then. The idea here is to provide some console tools in case we need to do some quality control on the data. That's why the feedback in the console is important. This issue got created when someone noticed that the QualityControl class was filling the test suite up with noise from the all the puts statements.

sure, you could have a debug option and a conditional checking for it. Or you could just have a check for Rails.env.test?.
I'm not familiar with the purpose of the QualityControl class (the tests don't seem have much info about it), so I wouldn't know if having a historical log to be checked afterwards would be useful.

Using Rails.env.test? would work, but making the method return different results depending on the environment could create surprises.
The idea of the QualityControl class is to allow us to update the outdoor temperatures for readings that are missing that information, as well as to remove duplicate readings. Because of that, I think historical logs to be checked afterwards would be nice to have but not necessary.

I put in an optional parameter to make it silent, which is working well, so I'm closing this issue.


Feature Request: PyTorch Dataloader Support

Hi,
really excited to see this repo and the idea of model registries.
I just wanted to say that we are very much looking forward to the PyTorch Dataloader integration which will allow us to introduce TileDB within our organization for our ML projects.

Hi,
really excited to see this repo and the idea of model registries.
I just wanted to say that we are very much looking forward to the PyTorch Dataloader integration which will allow us to introduce TileDB within our organization for our ML projects.

Hi @RAbraham ! Thank you very much for you interest in TileDB! We are already working on the PyTorch Dataloader integration for dense TileDB arrays, and this will hopefully be out by the end of next week. Support for sparse TileDB arrays is also coming very soon for both Tensorfow and PyTorch data APIs.

That's wonderful! Thank you so much. Closing issue.


feat: allow providing multiple apolloClients outside of setup/vue context in vue-apollo-composable

What?

adds a provideApolloClients that mirrors the api of app.provide(ApolloClients, { <clients> }) to allow using different apollo clients outside of setup/vue contexts (for "global queries")
should also fix the issue in #1338

Why?

It was impossible to use global queries/mutations outside of setup

How?

useApolloClients.ts now maintains a currentApolloClients client dictionary instead of just a single client

I added an extra e2e test and a short section in the docs.

@Akryum Could we get some 👀  on this? This would be VERY helpful for those of us using multiple clients.

@Akryum patiently wait for this 🙏🏻

@Akryum 🙏


Lot of MB's

Hi Guys,
I just have iPokeGo installed for 2 days, connected to my pogom map. It used 550 MB over 2 days. I think this is a lot, I run it in background and notifications for special pokemons.
Is there any way to solve this issue?

I think enabling filters helps with recent commits since it requests less from server and uses slightly less data. I made another request in a diff ticket about same issue that perhaps it can poll server less often by user setting. right now it downloads data every 5 seconds but to be honest you could do 10 or 15 use half the or one third the data and not really "miss out" on anything.

You can disable Pokestops because all of them are always present on the JSON file.

Maybe they can implement downloadable cache of map tiles for entire zones like Google Maps app does?  I think the larger part of the data may be from map tiles reload vs getting data from the server?  Does enabling filters actually cut down on the requests made too? I would figure that mostly everything would be handled client side with the app version and only pull new scan updates from the server and the data consumption would be / should be low... Any devs able to enlighten us on this?

@istornz  Does this app download all pokemons' data searched by server when it refresh? I think Pokemongo map support only catching raw_data of pokemons in field of view. (using sw and ne position information).

@istornz  Maybe they could add a feature : set range of showing pokemons. As a result, the App could only fetch data within a specific range around user.


MM-17549: support upgrading plugins

Summary
This change introduces the ability to upgrade plugins to the version listed in the marketplace. It does not yet include the changes to confirm the upgrade, and instead simply acts like an "Install" button in a different place.
As part of evaluating the next set of changes necessary to prompt for upgrade, I ran into some unit testing and state modelling issues, and decided to dive down a rabbit hole and promote most of the state of this component into Redux at views.marketplace. As such, this change includes a proposed refactoring along with exhaustive unit tests. The actual upgrade changes are relatively tiny. Feedback welcome on this approach! :D
I then extended the e2e tests to include both the ability to upgrade, and cover additional base functionality. Note that the existing and new tests are currently failing due to an unrelated change: https://community-daily.mattermost.com/core/pl/u15mhfthktgeu8dxtnc14isoza, since the test to examine the main menu containing "Plugin Marketplace" is spuriously failing.
@asaadmahmood, I took my best stab at implementing the required CSS, but I'm not familiar with our coding style and not up to speed on best practices. I also wasn't sure how to handle the hover effect or what the transition should be when installing. I'd love your advice on all this!
Ticket Link
https://mattermost.atlassian.net/browse/MM-17549
Related Pull Requests

Has server changes: https://github.com/mattermost/mattermost-server/pull/12983
Has redux changes: https://github.com/mattermost/mattermost-redux/pull/962

Screenshots

Creating a new SpinWick test server using Mattermost Cloud.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-4108.test.mattermost.cloud

Account Type
Username
Password

Admin
sysadmin
Sys@dmin123

User
user-1
User-1@123

New commit detected. SpinWick will upgrade if the updated docker image is available.

Test server creation failed. See the logs for more information.

Test server destroyed

Creating a new SpinWick test server using Mattermost Cloud.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-4108.test.mattermost.cloud

Account Type
Username
Password

Admin
sysadmin
Sys@dmin123

User
user-1
User-1@123

@lieut-data The sub row needs to sit inside the more-modal__details.
Currently, it appears like this when you hover over it, since it's not inside the plugin, and looks like a separate item on hover.

It should be like this:

That would also simplify the css.

@asaadmahmood, thanks for that! The challenge I had with that is the display: flex styling of the current row — there’s no provision for a vertical layout and then a horizontal layout without rewriting the common row component used in this modal. Can you suggest a strategy here for how to resolve? Should we break out this CSS for this page so as to not be shared?

I'll get a local copy and see the code.

New commit detected. SpinWick will upgrade if the updated docker image is available.

Mattermost test server updated with git commit 5649e55f840b07cef6f933312de372d64e29edad.
Access here: https://mattermost-webapp-pr-4108.test.mattermost.cloud

New commit detected. SpinWick will upgrade if the updated docker image is available.

As discussed with @asaadmahmood, I've simplified the DOM/CSS here, in part to help make it more obvious that the Update section belongs to the plugin:

This section now completely disappears once the installation begins -- the plan is to prompt first, after which the disappearing won't seem to "abrupt".

Mattermost test server updated with git commit 746317dc686d10e59d1e7986784603433ac3a6e9.
Access here: https://mattermost-webapp-pr-4108.test.mattermost.cloud

@lieut-data Looks good in the screenshot, however on the PR, there's no update available for any plugin. Can you update that so I can also have a look at it on the PR as well?

Heads up that I'm rebasing against mm-18517-feature-plugin-upgrade to isolate this to a feature branch.

@asaadmahmood, I've downgraded Webex to expose the upgrade API. Let me know :)

New commit detected. SpinWick will upgrade if the updated docker image is available.

New commit detected. SpinWick will upgrade if the updated docker image is available.

New commit detected. SpinWick will upgrade if the updated docker image is available.

Test server creation failed. See the logs for more information.

Mattermost test server updated with git commit 6cc123d32b8c1284c102db82bdcad732d01ef01f.
Access here: https://mattermost-webapp-pr-4108.test.mattermost.cloud

Test server creation failed. See the logs for more information.

New commit detected. SpinWick will upgrade if the updated docker image is available.

Removing 2: QA Review, since we're merging into a feature branch, and will test holistically.

Mattermost test server updated with git commit 2d8ac4f0456837954d20e572728d601863c48b89.
Access here: https://mattermost-webapp-pr-4108.test.mattermost.cloud

New commit detected. SpinWick will upgrade if the updated docker image is available.

Mattermost test server updated with git commit 9f159d40663fa665c784aca605fd98548ed81a82.
Access here: https://mattermost-webapp-pr-4108.test.mattermost.cloud

@asaadmahmood, any concerns with updated (and simplified) implementation?

New commit detected. SpinWick will upgrade if the updated docker image is available.

Mattermost test server updated with git commit 602239cbcfb3b91a651f6388011ca6c6d3368909.
Access here: https://mattermost-webapp-pr-4108.test.mattermost.cloud

Test server destroyed

Oops, didn't see that you had suggestions @ali-farooq0 -- will address with other changes I'm about to submit. Thanks!


Docs dev

This is now working as planned on new-raku eg 101-basics
Also try about where the information in the page is about the repo, with separate information about the Raku/doc-website files, and the Raku/docs files, as well as information about the 'about.rakudoc' file itself (not its contents) in the Commit footer and edit button

"About the new-raku.finanalyst.org websitr"

Mousing over commit is slow. If you hadn't told me something was there, I would have just kept moving.

We're missing a plugin on the CI system

1 bin/ script [collection-refresh-plugins] installed to:
--
  | /home/builder/MoarVM/install/share/perl6/site/bin
  | <git clone https://github.com/Raku/doc.git local_raku_docs/> was successful
  | Error occurred when processing source file ｢local_raku_docs/doc/language/101-basics.rakudoc｣.
  | No such method 'last-edited' for invocant of type 'PodFile'  in block <unit> at ./bin_files/build-site line 3
  | in block MAIN at ./bin_files/build-site line 5
  | in block collect at /home/builder/MoarVM/install/share/perl6/site/sources/8FF8035D035FB265D24BF86A9F6B0B5BD78CA772 (Collection) line 21
  | in block collect at /home/builder/MoarVM/install/share/perl6/site/sources/8FF8035D035FB265D24BF86A9F6B0B5BD78CA772 (Collection) line 260
  | in block collect at /home/builder/MoarVM/install/share/perl6/site/sources/8FF8035D035FB265D24BF86A9F6B0B5BD78CA772 (Collection) line 21
  | in block collect at /home/builder/MoarVM/install/share/perl6/site/sources/8FF8035D035FB265D24BF86A9F6B0B5BD78CA772 (Collection) line 305
  | in block report-method-not-found at src/vm/moar/dispatchers.nqp line 676
  | in block throw_or_die at src/Perl6/Metamodel/Configuration.nqp line 60

@finanalyst do you intend to check that into this repo like the others? If not, I will need to update the image that the CI environment runs in.

@dontlaugh  I will need to look at this.
You said:

If not, I will need to update the image that the CI environment runs in.
These changes need the latest versions of Pod-Render and Collection. Does that require a change of the image?

@coke

Mousing over commit is slow. If you hadn't told me something was there, I would have just kept moving.

It is slow. I'm not sure whether it is a deliberate browser delay or something that can be fixed in CSS.
A popup is generated magically when a tag has a title attribute. But I have not seen any information about the delay on mouse-over.

@coke open the browser tools memory profile tab while you mouse over. Just to make sure it's not loading something big on the fly

@dontlaugh I have made sure that the distribution versions working for me locally are now uploaded to fez (I think they were, but maybe not :) ) . The two distributions that needed changing were Raku-Pod-Render and Collection.
I  have just bumped the version requirements in META6.json file in this branch.
I tried hard to ensure that the new versions are backwards compatible.
The error seen above was because I added another attribute to the PodFile class to hold the commit data. This will not affect older versions of Collection, which dont look for it, but will affect the new version of Collection, which does.

@coke  I just googled the title delay, and found a stackoverflow thread. Apparently not possible to dictate using 'title', which leaves it to the browser. (Presumably some browsers more customisable than others).
This mod has already taken up FAR FAR more time than I thought it would for a simple addition of data. So I am loath to add extra divs and CSS to get a faster popup.
For those for whom the information will be useful, they will soon find out or be told to wait.

@coke @dontlaugh The last commit worked properly. The changes can be seen on docs-dev. The pop-up seems to work quite fast. It may be a server or cache issue as well.
Also the commit id on about has three different commit strings, one for the Raku/doc repo, one for the Raku/web-docsite repo, and one for the About page itself.
Just noticed an error on About. Will fix now.

"This is now working" - what's this PR about? Should I be looking at your site (which still has the typo I called out at the top) or docs-dev?

@coke several things

@dontlaugh +++ has done some magic so that any patch to the docs-dev branch automatically deploys to docs-dev.raku.org. This means that we can experiment a bit inside the doc-website environment without it affection the main branch or being pushed to production. It also means that there is not the problem of desynchronisation between my new-raku site.
The PR on the docs-dev branch concerns adding commit information to each documentation file. The commit information is not immediately obvious, but is there so that someone who has edited the documentation sources can know whether the edits have yet been included onto the production site, or when a source was last edited.
My new-raku  site does have synchronisation problems - the spelling mistake is an example (thanks, I hadn't noticed it, even when you pointed it out). You will see that the about page on docs.raku.org does not have the error. I'll fix it soon.
My IRC message to you was intended to ask you to look at 'docs-dev', decide whether the commit information is any good (I am strongly hoping you will like it), and to let me know what to do.
I want to start using 'docs-dev' for the ebook patch. I have an ebook that builds, but it needs some wider testing and review, eg. Kindle requires some slightly out of standard tweaks to get a TOC, but I dont have a Kindle to test the ebook with.
I can't upload the EBook patch until we decide what to do with the commit info idea.

Hope this is clear now.

@coke I have merged. If you hover over the edit button, you will get the commit id & the date relating to when that  source was last edited - if the source can be edited


Add __radd__ function

Codecov Report
Base: 93.18% // Head: 93.25% // Increases project coverage by +0.07% :tada:

Coverage data is based on head (95310ca) compared to base (bd9f8fc).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev      #15      +/-   ##
==========================================
+ Coverage   93.18%   93.25%   +0.07%
==========================================
  Files           5        5
  Lines         176      178       +2
==========================================
+ Hits          164      166       +2
  Misses         12       12

Impacted Files
Coverage Δ

telegram_text/bases.py
87.14% <100.00%> (+0.37%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


JSDoc documentation for interface members does not show in tooltips for properties on anonymous objects that are typed to those interfaces

TypeScript Version:  3.1.0-dev.201xxxxx

Search Terms:
jsdoc interface tooltip
Code
interface IMyInterface
{
    /**
     * my property a
     */
    a: string;
}

let o: IMyInterface = { a: "xyz" };

let oo: { [index: string]: IMyInterface } = {
    'test': {
        a: 'abc'
    }
};

Expected behavior:
The tooltip when hovering on "a" should show JSDoc
Actual behavior:
The tooltip when hovering on "a" does not show JSDoc
Playground Link:
Playground
Related Issues:
Seems like a regression of #18358

This works for me with typescript@next.

@andy-ms Sorry I mistakenly thought the playground is the @next version. with vscode and typescript 3.1.0-dev.20180823. I found following example doesn't work:
interface ITest {
  /**
   * Test JSDoc
   */
  test: () => void;
}
export interface ITest2 {
  t: ITest;
}
const testObj: { [index: string]: ITest2 } = {};
testObj['a'].t.test();

Thanks, repros with:
interface ITest {
    /**
     * Test JSDoc
     */
    test: () => void;
}
const testObj: ITest = <any>null;
testObj.test();


Unable to deploy war file or restart tomcat instance due to Neo4j store_lock file issue

Multiple instances of Neo4j graph db instantiated for every request which throws the following error for every request:
04:59:25.301  14247 [http-nio-8080-exec-2] ERROR i.o.r.dao.impl.RegistryDaoImpl - Exception occurred when adding an entity:
java.lang.RuntimeException: Error starting org.neo4j.kernel.impl.factory.GraphDatabaseFacadeFactory, /data/databases/graph.db
	at org.neo4j.kernel.impl.factory.GraphDatabaseFacadeFactory.initFacade(GraphDatabaseFacadeFactory.java:209)
	at org.neo4j.kernel.impl.factory.GraphDatabaseFacadeFactory.newFacade(GraphDatabaseFacadeFactory.java:124)
	at org.neo4j.graphdb.factory.GraphDatabaseFactory.newDatabase(GraphDatabaseFactory.java:140)
	at org.neo4j.graphdb.factory.GraphDatabaseFactory.newEmbeddedDatabase(GraphDatabaseFactory.java:133)
	at org.neo4j.graphdb.factory.GraphDatabaseFactory$1.newDatabase(GraphDatabaseFactory.java:108)
	at org.neo4j.graphdb.factory.GraphDatabaseBuilder.newGraphDatabase(GraphDatabaseBuilder.java:193)
	at org.neo4j.tinkerpop.api.impl.Neo4jFactoryImpl.newGraphDatabase(Neo4jFactoryImpl.java:46)
	at org.neo4j.tinkerpop.api.Neo4jFactory$Builder.open(Neo4jFactory.java:32)
	at org.apache.tinkerpop.gremlin.neo4j.structure.Neo4jGraph.<init>(Neo4jGraph.java:125)

Caused by: org.neo4j.kernel.lifecycle.LifecycleException: Component 'org.neo4j.kernel.internal.StoreLockerLifecycleAdapter@3431a467' was successfully initialized, but failed to start. Please see the attached cause exception "null".
	at org.neo4j.kernel.lifecycle.LifeSupport$LifecycleInstance.start(LifeSupport.java:444)
	at org.neo4j.kernel.lifecycle.LifeSupport.start(LifeSupport.java:107)
	at org.neo4j.kernel.impl.factory.GraphDatabaseFacadeFactory.initFacade(GraphDatabaseFacadeFactory.java:205)
	... 76 common frames omitted
Caused by: org.neo4j.kernel.StoreLockException: Unable to obtain lock on store lock file: /data/databases/graph.db/store_lock. Please ensure no other process is using this database, and that the directory is writable (required even for read-only access)
	at org.neo4j.kernel.internal.StoreLocker.storeLockException(StoreLocker.java:94)
	at org.neo4j.kernel.internal.StoreLocker.checkLock(StoreLocker.java:86)
	at org.neo4j.kernel.internal.StoreLockerLifecycleAdapter.start(StoreLockerLifecycleAdapter.java:40)
	at org.neo4j.kernel.lifecycle.LifeSupport$LifecycleInstance.start(LifeSupport.java:434)
	... 78 common frames omitted

This issue has been fixed by injecting the GraphDBFactory in the RegistryDaoImpl and gracefully shutting down tomcat by removing by shutting down both the Neo4JGraph and GraphDatabaseService objects.


feat(adapt): ios/android disctinction

I find it very useful to have the ability to distinguish between iOS and Android separately.
Technically it can be achieved by isIos and isAndroid but JSX declarative way is way much more readable and consistent.

cc @natew @TheEhsanSarshar

Looks good, I'll wait to merge until we're ready to do a minor release.


How to contribute?

Hi,
Would love to contribute to the module (specifically for iOS part first) and start with small stuff.
I see that the iOS folder doesn't have an .xcworkspace - how do you open it in xCode to work on it?
Thanks

open the example ios there you will find plugin side

Could you do a screenshot of the tab where you see the code of the plugin in the example ?
Sorry for that, I've never really code in iOS but would be nice to start somewhere :)

like this in pods file 👇

ok got it :)
I had many issues with the example project -_-
I'll try to do a PR for this

There was no version in the pubspec.yaml
There was the runnerTest but no runnerTest
Version of platform should be  '12.0'

( I wrote it so I don't forget what to do)

Mmmm after looking at the project I'm a bit perplex about the use of Tangram-es.
I see that the library hasn't been updated for 2 years -_-.
Do you think it's a good idea to still use it?
I saw that you import mapkit sometimes (but didn't use it (if I understand correctly)) Where you thinking of moving out of Tangram ?

i will migrate to another sdk this weekend and next week
it was nice sdk

the issue that the new sdk use swift package that not supported by flutter plugins yet
that maybe will add some manual integration by developer but for now i think its ok to do it
i will try to find other solution but the new sdk will contain the same feature as in android almost

That would be great, what is the new SDK name ? Is it still Tangram ?

this is : https://github.com/openmobilemaps/maps-core


How to fix url contain Chinese words?

it keeps showing "'latin-1' codec can't encode characters in position 30-31: ordinal not in range(256)" while testing, how can i fix it?

Thanks for your reply. Site like "www.hkood.com" would cause this error.

Should be fixed now, let me know if you still have this issue

It is fixed now. Thanks for your help.


Dev/crutkas/ci use win UI build agent

Summary of the Pull Request
What is this about?
PR Checklist

[ ] Applies to #xxx
[ ] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request
What does this include?
Validation Steps Performed
How does someone test & validate?

@enricogior, this is in a state where we can leverage the new instance.  now we should be able to start ramping up the parallelism.
Your call if you want to do work here or on a different thread

as is, shaves 50+ minutes off compile time.  I'm keeping in some of the PS files since they could help in the future unblock if the agent doesn't get updated when we need something on it.


sw-schedule-check-in not working for Companion Pass

Hi there, I'm relatively new to AWS Lambda so appreciate the help here. The setup process went smoothly but now I am stuck because it seems like the sw-receive-email function is successful in parsing my forwarded email but sw-schedule-check-in. As the result, I am not getting the confirmation email.
In Cloudwatch, I'm seeing logs for sw-receive-email with "Found reservation", but getting checkin-schedule-errors: State changed to OK at 2019/01/08. Reason: Threshold Crossed: no datapoints were received for 1 period and 1 missing datapoint was treated as [NonBreaching].
Anywhere I should look to diagnose where it went wrong?

If you're reading this and you have an upcoming companion pass flight, please email me (see Github profile) so I can try to figure out how to make this work.

I have an upcoming flight, next week, with a Minor Reservation linked.
Your link gives:
This reservation is linked to a minor reservation. In order to change one of the linked reservations, the other must be cancelled. If you have already cancelled one of the reservations and are still unable to make a change, we may be experiencing a delay. If you need assistance, please contact a Southwest Airlines Customer Representative at 1-800-I-FLY-SWA (1-800-435-9792).

But the link @anyangzhao provided works.
How can I help?


[Bug]: vite+vue-router alias work error

What happened?
I make a repo to reproduce. https://github.com/hunshcn/vitets_alias_broken
It use vite + vue + vue-router.
In main.ts when I use @/ alias instead of ./, the build artifact will be broken.
It looks like related to vue-router. If I delete all router usage, errors will be disappear.
The above problems only appear when compiling with rules_js. pnpm build or use vite_binary and chdir to "$$BUILD_WORKSPACE_DIRECTORY/" + package_name() to use local node_modules, and there is no problem.
Version
Development (host) and target OS/architectures:
Output of bazel --version: 7.1.1
Version of the Aspect rules, or other relevant rules from your
WORKSPACE or MODULE.bazel file: 1.41.0
Language(s) and/or frameworks involved: js
How to reproduce
https://github.com/hunshcn/vitets_alias_broken

Any other information?
No response

Fixed
the reason of the problem is the usage of fileURLToPath.


Pre train link is invalid

https://hanlab.mit.edu/projects/bevfusion/files/pretrained_updated/bevfusion-det.pth
This link is invalid makes the bevfusion-det.pth undowloadble.

Please use hanlab18.mit.edu in place of hanlab.mit.edu in the link as a temporary solution.

Thanks for your timely reply!


LinqKit fails with Entity Framework Core 5

LinqKit's query expansion functionality does not work with Entity Framework Core 5, well, since the 5.0.0-preview.2.20159.4 to be exact. It throws an exception: 'Unhandled expression type: 'Extension''
I don't know what could be that Extension expression, but it's been there since .NET Framework 4.0.
Of course, you could handle it in your code, but I suggest you to use the official ExpressionVisitor class, at least for the newer target frameworks, so it would be future-proof.

@btihanyi
Can you try https://www.nuget.org/packages/LinqKit.Microsoft.EntityFrameworkCore/5.0.0-preview.1 ?

@btihanyi
Did you have time to test this preview version?

@StefH
Sorry, I'm currently on my vacation. I will check it next week.

I've checked it and now it works fine. But I still wonder why don't you just use the official ExpressionVisitor class if the target framework is not .NET 3.5? They are doing basically the same. Like I said, it would be future-proof and I haven't found any performance difference between the two.

@btihanyi
Thanks for testing.
Actually I don't know why a custom ExpressionVisitor is used in LinqKit, but for now I keep it like it is.
Maybe add a new issue/PR to just use this custom ExpressionVisitor for .NET 3.5 and use the normal one for all other frameworks.
Other question:
What version should I give to LinqKit.Microsoft.EntityFrameworkCore?
Just 5.0.0?
Or keep in line with Microsoft.EntityFrameworkCore, so version 5.0.0-preview.7?

From the comment of the class, the custom ExpressionVisitor class comes from the this old blog post. It's from 2007, even before .NET 3.5 was released, when there was no built-in ExpressionVisitor class. I'm going to write a PR and check if there is any real difference in their implementations, but don't think so.
On the second question: I think while you use a preview (alpha, beta, etc.) package, yours should be too. But you don't have to follow their release numbers (apart from the major part), so 5.0.0-preview1, 5.0.0-preview2 is the proper way to go I believe. Then, when it's officially released, try to target that 5.0.0 version as long as possible.


Crashes on node delete

using the cyberether-git aur package
JETSTREAM [DEBUG] | [BENCHMARK] Adding benchmark: FFT | CPU | CF32
JETSTREAM [DEBUG] | [BENCHMARK] Adding benchmark: Multiply | CPU | CF32
JETSTREAM [DEBUG] | [BENCHMARK] Adding benchmark: Multiply | CPU | F32
JETSTREAM [DEBUG] | [BENCHMARK] Adding benchmark: Invert | CPU | CF32
JETSTREAM [DEBUG] | [BENCHMARK] Adding benchmark: MultiplyConstant | CPU | CF32
JETSTREAM [DEBUG] | [BENCHMARK] Adding benchmark: MultiplyConstant | CPU | F32
JETSTREAM [DEBUG] | [BENCHMARK] Adding benchmark: Lineplot | CPU | F32
JETSTREAM [DEBUG] | [COMPOSITOR] Refreshing interface state.
JETSTREAM [DEBUG] | [INSTANCE] Started.
JETSTREAM [DEBUG] | [INSTANCE] Building default viewport and render.
JETSTREAM [DEBUG] | Initializing Vulkan backend.
JETSTREAM [WARN]  | [VULKAN] Couldn't find validation layers. Disabling Vulkan debug.
JETSTREAM [DEBUG] | [VULKAN] Supported required instance extensions: ["VK_KHR_surface", "VK_KHR_xcb_surface"]
JETSTREAM [DEBUG] | [VULKAN] Supported required device extensions: ["VK_KHR_swapchain"]
JETSTREAM [DEBUG] | [VULKAN] Supported optional device extensions: {"VK_EXT_external_memory_host", "VK_KHR_external_memory_fd"}
JETSTREAM [INFO]  | -----------------------------------------------------
JETSTREAM [INFO]  | Jetstream Heterogeneous Backend [VULKAN]
JETSTREAM [INFO]  | -----------------------------------------------------
JETSTREAM [INFO]  | Device Name:     AMD Radeon RX 7600 (RADV NAVI33)
JETSTREAM [INFO]  | Device Type:     DISCRETE
JETSTREAM [INFO]  | API Version:     1.3.267
JETSTREAM [INFO]  | Unified Memory:  NO
JETSTREAM [INFO]  | Processor Count: 20
JETSTREAM [INFO]  | Device Memory:   8.00 GB
JETSTREAM [INFO]  | Staging Buffer:  64.00 MB
JETSTREAM [INFO]  | -----------------------------------------------------
JETSTREAM [DEBUG] | [VULKAN] Creating GLFW viewport.
JETSTREAM [DEBUG] | [VULKAN] Swap mailbox presentation mode is available.
JETSTREAM [DEBUG] | [VULKAN] Creating window.
JETSTREAM [DEBUG] | [VULKAN] Creating ImGui.
JETSTREAM [DEBUG] | [COMPOSITOR] Loading assets.
JETSTREAM [DEBUG] | [VULKAN] Creating texture.
JETSTREAM [DEBUG] | [VULKAN] Creating texture.
JETSTREAM [DEBUG] | [VULKAN] Swap mailbox presentation mode is available.
JETSTREAM [INFO]  | [FLOWGRAPH] Creating flowgraph in-memory.
JETSTREAM [DEBUG] | [COMPOSITOR] Running graph auto-route.
JETSTREAM [DEBUG] | [INSTANCE] Adding new block 'cas0'.
JETSTREAM [DEBUG] | [INSTANCE] Adding new module 'cas0-cast'.
JETSTREAM [DEBUG] | Initializing Cast module.
JETSTREAM [ERROR] | Input is empty during initialization.
JETSTREAM [DEBUG] | [INSTANCE] Module 'cas0-cast' is incomplete.
JETSTREAM [DEBUG] | [COMPOSITOR] Adding block 'cas0'.
JETSTREAM [DEBUG] | [COMPOSITOR] Refreshing interface state.
/usr/include/c++/13.2.1/optional:477: constexpr _Tp& std::_Optional_base_impl<_Tp, _Dp>::_M_get() [with _Tp = Jetstream::Locale; _Dp = std::_Optional_base<Jetstream::Locale, false, false>]: Assertion 'this->_M_is_engaged()' failed.
Aborted (core dumped)

output of cyberether -v
JETSTREAM [DEBUG] | [BENCHMARK] Adding benchmark: FFT | CPU | CF32
JETSTREAM [DEBUG] | [BENCHMARK] Adding benchmark: Multiply | CPU | CF32
JETSTREAM [DEBUG] | [BENCHMARK] Adding benchmark: Multiply | CPU | F32
JETSTREAM [DEBUG] | [BENCHMARK] Adding benchmark: Invert | CPU | CF32
JETSTREAM [DEBUG] | [BENCHMARK] Adding benchmark: MultiplyConstant | CPU | CF32
JETSTREAM [DEBUG] | [BENCHMARK] Adding benchmark: MultiplyConstant | CPU | F32
JETSTREAM [DEBUG] | [BENCHMARK] Adding benchmark: Lineplot | CPU | F32
CyberEther v1.0.0-release

That's new for me! I'll take a look.

 Just to make sure I properly patched the bug.
$ gdb ./cyberether
> r
# wait for it to crash
> bt

yeah 1 second

had to recompile it as the aur package uses the release compilation, i was unable to reproduce it by compiling it manually
https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=cyberether-git
also adding @armv8-a as they made the pkgbuild
  510  git clone https://github.com/luigifcruz/CyberEther.git
  511  cd CyberEther/
  512  paru glslang
  513  paru glfw
  514  meson setup -Dbuildtype=debugoptimized build && cd build
  515  ninja -C build # didnt work
  516  ninja build # didnt work
  517  ninja # was the command i used
  518  ls
  519  gdb ./cyberether

running gdb on aur package built with debug mode
[electron271@saturn cyberether-git]$ gdb cyberether
GNU gdb (GDB) 13.2
Copyright (C) 2023 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
Type "show copying" and "show warranty" for details.
This GDB was configured as "x86_64-pc-linux-gnu".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
<https://www.gnu.org/software/gdb/bugs/>.
Find the GDB manual and other documentation resources online at:
    <http://www.gnu.org/software/gdb/documentation/>.

For help, type "help".
Type "apropos word" to search for commands related to "word"...
Reading symbols from cyberether...

This GDB supports auto-downloading debuginfo from the following URLs:
  <https://debuginfod.archlinux.org>
Enable debuginfod for this session? (y or [n]) n
Debuginfod has been disabled.
To make this setting permanent, add 'set debuginfod enabled off' to .gdbinit.
(No debugging symbols found in cyberether)
(gdb) r
Starting program: /usr/bin/cyberether
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/usr/lib/libthread_db.so.1".
[New Thread 0x7fffe92006c0 (LWP 1555747)]
JETSTREAM [INFO]  | -----------------------------------------------------
JETSTREAM [INFO]  | Jetstream Heterogeneous Backend [VULKAN]
JETSTREAM [INFO]  | -----------------------------------------------------
JETSTREAM [INFO]  | Device Name:     AMD Radeon RX 7600 (RADV NAVI33)
JETSTREAM [INFO]  | Device Type:     DISCRETE
JETSTREAM [INFO]  | API Version:     1.3.267
JETSTREAM [INFO]  | Unified Memory:  NO
JETSTREAM [INFO]  | Processor Count: 20
JETSTREAM [INFO]  | Device Memory:   8.00 GB
JETSTREAM [INFO]  | Staging Buffer:  64.00 MB
JETSTREAM [INFO]  | -----------------------------------------------------
[New Thread 0x7fffdfe006c0 (LWP 1555748)]
[New Thread 0x7fffda0006c0 (LWP 1555749)]
[New Thread 0x7fffd96006c0 (LWP 1555750)]
[Thread 0x7fffdfe006c0 (LWP 1555748) exited]
[New Thread 0x7fffdfe006c0 (LWP 1555751)]
[New Thread 0x7fffd8c006c0 (LWP 1555752)]
JETSTREAM [INFO]  | [FLOWGRAPH] Creating flowgraph in-memory.
[Thread 0x7fffd8c006c0 (LWP 1555752) exited]
[New Thread 0x7fffd8c006c0 (LWP 1555753)]
JETSTREAM [ERROR] | Input is empty during initialization.
[Thread 0x7fffd8c006c0 (LWP 1555753) exited]
[New Thread 0x7fffd8c006c0 (LWP 1555754)]
/usr/include/c++/13.2.1/optional:477: constexpr _Tp& std::_Optional_base_impl<_Tp, _Dp>::_M_get() [with _Tp = Jetstream::Locale; _Dp = std::_Optional_base<Jetstream::Locale, false, false>]: Assertion 'this->_M_is_engaged()' failed.

Thread 9 "cyberether" received signal SIGABRT, Aborted.
[Switching to Thread 0x7fffd8c006c0 (LWP 1555754)]
0x00007ffff70ac83c in ?? () from /usr/lib/libc.so.6
(gdb) bt
#0  0x00007ffff70ac83c in  () at /usr/lib/libc.so.6
#1  0x00007ffff705c668 in raise () at /usr/lib/libc.so.6
#2  0x00007ffff70444b8 in abort () at /usr/lib/libc.so.6
#3  0x00007ffff72dd3b2 in std::__glibcxx_assert_fail(char const*, int, char const*, char const*) (file=<optimized out>, line=<optimized out>, function=<optimized out>, condition=<optimized out>)
    at /usr/src/debug/gcc/gcc/libstdc++-v3/src/c++11/debug.cc:61
#4  0x00007ffff7768d30 in  () at /usr/lib/libjetstream.so
#5  0x00007ffff72e1943 in std::execute_native_thread_routine(void*) (__p=0x7fffd4019330) at /usr/src/debug/gcc/gcc/libstdc++-v3/src/c++11/thread.cc:104
#6  0x00007ffff70aa9eb in  () at /usr/lib/libc.so.6
#7  0x00007ffff712e7cc in  () at /usr/lib/libc.so.6
(gdb)

That's interesting. The GDB didn't help too much because all the symbols were optimized away. This is probably because the AUR pkg-config configured it that way. This is also likely why it's not crashing when you compile manually.
Does this happen with every block or just some in particular?

every it seems
something to note is that it says stripped /usr/bin/cyberether: ELF 64-bit LSB pie executable, x86-64, version 1 (GNU/Linux), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, BuildID[sha1]=1aa7383a2c5ad0f212a1ebc72a81669b1feaa120, for GNU/Linux 4.4.0, stripped, going to rebuild with debug instead of debug optimized

I am unable to reproduce this with the AUR PKGBUILD, but it's likely something to do with my building/packaging since they can't reproduce it with the manual build

There is a tiny chance of this being caused by some peculiarity of the RADEON driver and CyberEther's Vulkan render. Unfortunately, I don't have a similar hardware to verify. But I'm writing a patch with stronger assertions to at least identify the root cause of this problem.

I just searched the GitHub global issue tracker, and there are a lot of recent bug reports with the same crash signature. I'm pending towards something wrong with libstdc++ or libc++.

 I was using a std::optional value in an asynchronous call that was subsequently cleared by the main thread. Anyway, it should be fixed in the v1.0.0-alpha2 branch.

testing a new aur package using the new branch

confirmed to work, some slight issues but i am unsure if they are a thing with the new thing
will investigate further tomorrow


Fix bug where brakes are set to 0

Fixed the bug where brakes are set to 0 on load until down button is pressed.

i think the owner is no longer active

His profile shows that he's been active very recently. Eventually he'll see this (hopefully).


Changing the 'on' value in the query doesn't get affected when using the rest API

following two queries return the same result when used with the rest api
from ApiUserPerAppAgg on gatewayType=='SYNAPSE' within '2018-08-** **:**:**' per 'months' SELECT sum(totalRequestCount) as sum
from ApiUserPerAppAgg on gatewayType=='MICRO' within '2018-08-** **:**:**' per 'months' SELECT sum(totalRequestCount) as sum
Restarting the server would give the correct result

This is a Siddhi level bug, the following siddhi test case is also failing
https://github.com/Niveathika92/siddhi/commit/bee6bdcd77fa991cf645bb8a8af6d901cea7f947


Remove taxon confirmation feature from Report Table.

Also made last column narrower.

You could git reset schema.rb from master or db rollback and try generating a migration again. Or that could be a separate PR

Okay I'll try to get it working. If I can't, then I'll remove the migration from this PR.

Okay I'll try to get it working. If I can't, then I'll remove the migration from this PR.

It's possible staging db actually has some relics in it. Since you pulled staging db into your development, you would have those too.
I think dropping the taxon_confirmations table is not too urgent

Okay, I've removed the db migration.


Log "batch verification receiver lagged" at info level

Motivation
This error-level log is alarming to users, but there's nothing the user can do about it:

2022-06-30T17:51:02.928651ZERROR{net="Main"}:sync:try_to_sync:extend_tips:zebra_consensus::primitives::redpallas: batch verification receiver lagged and lost verification results

https://github.com/ZcashFoundation/zebra/runs/7135923149?check_suite_focus=true#step:6:644
Each batch verifier has a separate instance of this log, they all need to be fixed.

This is fixed in #4750 by removing the channel that lags.


customer paymet

[x] customer payment record from customer detail screen

[x] total due amount not updated after payment record while doing transaction from customer detail screen

issue closed


Help with reading from unbounded channel in listener thread -- receive operation blocks at exactly 50%

Hi there!  Can anybody help me understand what I'm doing wrong with reading messages from a crossbeam unbounded() channel, or have suggestions on where I should be asking?
I have a listener thread created using std::thread::spawn() that's meant to do some heavy processing on each item before insertion to a non-atomic/non-concurrent data structure, but for right now just counts how many items were in the channel receiver.  The data is sent to the channel using a rayon threadpool that reads individual lines from the file, where the test file is 1_000_000 lines.  For whatever reason, the rx.recv() call in the listener thread halts at exactly 50% // 500_000, without an error message or panic printing at all.  I know that it is because of how I am handling the recv() call, because when I use the rayon tools for just collecting everything into a Vec, that has the right count.  I suspect that this is because there is an unmatched case where the receiving channel is empty, but have no idea based on the crossbeam docs how to just check if the receiving channel is empty or disconnected.  Screenshots of output and the main.rs file are attached.
Here are screenshots of the differing outputs, and a copy/paste of the main.rs file follows:

use std::fs::File;
use std::path::PathBuf;
use std::io::{Error, BufRead, BufReader};

//use actix::prelude::*;

use rayon::prelude::*;

use crossbeam::channel::{ unbounded, bounded, RecvError} ;

//use serde::{ Serialize, Deserialize };

use indicatif::{ProgressBar, ProgressStyle};

mod lidarpoint;
use crate::lidarpoint::LiDARPoint;

/**
 *
 */
fn main() {
    let filename = PathBuf::from("../basic-laser-scan-viewer/assets/pointclouds/TEST/cuboid.pts");
    //let filename = PathBuf::from("../basic-laser-scan-viewer/assets/pointclouds/TEST/FERNS-Chapel-Unified-100percent.pts");
    let fd = File::open(&filename).unwrap();

    let mut reader = BufReader::new(fd);

    let mut linetotal: String = String::new();
    let mut _total: u64 = 0;

    // parse total count from first line of .pts file
    reader.read_line(&mut linetotal).unwrap();
    match linetotal.trim().parse::<u64>() {
        Ok(value) => { _total = value; },
        _ => { _total = 0; }
    }

    //
    // process values
    //

    let pb = ProgressBar::new(_total);
    pb.set_style(
            ProgressStyle::default_bar()
                .template("{spinner:.green} [{elapsed_precise}] [{wide_bar:.cyan/blue}] {percent}% ({eta})")
                //.with_key("eta", |state| format!("{:.1}s", state.eta().as_secs_f64()))
                .progress_chars("#>-")
    );

    let (tx, rx) = unbounded();
    let (valsend, retval) = bounded(0);

    println!("Starting listener thread...");

    let handle = std::thread::spawn(move || {
        let mut counter: usize = 0;

        for _ in &rx {
            match rx.recv() {
                Ok(val) => {
                    counter = counter + 1;
                    pb.inc(1);
                },
                Err(e) => { panic!("{}", e); }
            }
        }

        valsend.send(counter).unwrap();
    });

    println!("Starting file read...");

    reader.lines()
        .enumerate()
        .par_bridge()
        .for_each(|(nr, line)| {
            // unwrap line result first and check for errors
            let l = line.unwrap();

            // process a line
            let point: Vec<&str> = l.split_whitespace().collect();

            if point.len() < 7 {
                panic!("Line with less than 7 items detected.");
            }

            tx.send(
                Box::new(
                    LiDARPoint::new(
                        [
                            f32::from(point[0].parse::<f32>().unwrap()),
                            f32::from(point[1].parse::<f32>().unwrap()),
                            f32::from(point[2].parse::<f32>().unwrap())
                        ],
                        point[3].parse::<i32>().unwrap(),
                        Some([
                            point[4].parse::<u8>().unwrap(),
                            point[5].parse::<u8>().unwrap(),
                            point[6].parse::<u8>().unwrap()
                        ]),
                        None,
                        nr
                    )
                )
            ).unwrap();
        });

    println!("Finished reading from channel, processing in receiver...");

    //let mut points: Vec<Box<LiDARPoint<f32>>> = rx.try_iter().collect();
    //println!("{}", points.len());

    let _x = handle.join().unwrap();
    let res = retval.recv().unwrap();
    println!("{:?}", res);

}

StackOverflow post with additional files needed for reproducing problem included (couldn't upload to GitHub): https://stackoverflow.com/questions/71878179/sending-messages-to-unbounded-channel-from-rayon-with-listener-thread-handling

Closing.  Root problem is related to #814 .


Dynamically adjust the Exclude/Priority dropdown lists

Issue ported from issue in original repo

Thanks for this app!
I'd love if the list of games in the exclusion and priority dropdown lists automatically get removed once you've added them to the active list of exclusions/priorities. This is to save on having to scroll through all of the same games every time you want to add a new game.
Example:
I want to exclude Fortnite.
I scroll down the list of available games in the Exclude dropdown, and select Fortnite.
Once Fortnite has been added it still appears in the dropdown list among all the other games you can select - this is redundant, and is the part I'd like to see changed.

this is the only thing im missing a bit too ^^


Add ObjectId casting

Now casts an ObjectId to keep it consistent with allowing any form of id input.

Coverage remained the same at 100.0% when pulling fae8058722d015afdfa65340277527130e99b553 on Jtupiter:objectid-casting into 63c2ad5116db7fae5832c1442eb114e9483521b8 on pebble:master.

thanks!


Not able to install bitcore-node-dash globally.

Below is the error i am getting while installing bitcore-node-dash
sudo npm install -g bitcore-node-dash
\
><EMAIL_ADDRESS>preinstall /usr/local/lib/node_modules/bitcore-node-dash
> ./scripts/download

sh: 1: ./scripts/download: Permission denied
npm ERR! Linux 3.16.0-30-generic
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install" "-g" "bitcore-node-dash"
npm ERR! node v4.0.0
npm ERR! npm  v2.14.2
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>preinstall: `./scripts/download`
npm ERR! Exit status 126
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>preinstall script './scripts/download'.
npm ERR! This is most likely a problem with the bitcore-node-dash package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     ./scripts/download
npm ERR! You can get their info via:
npm ERR!     npm owner ls bitcore-node-dash
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /home/afzal/npm-debug.log

Discussed about that on other issues, see cross-posted :

https://github.com/dashpay/bitcore-node-dash/issues/17
https://github.com/dashpay/insight-ui-dash/issues/10

Should be fixed. This issue can be closed now :)

@afzydev  have you got the solution??
As i tried to follow the steps in https://gist.github.com/Alex-Werner/6de16b72a9070d945e3e74691d318f24  but it gives again error in npm install


PySide missing from AppImage of version 1.6.5

Downloading and running the AppImage for the latest version (1.6.5) on fedora 32 results in an error:

Could not import GUI libraries. If you do not have PySide installed then you will need to install it for the GUI to work.

This is unexpected as it's an AppImage distribution which I expected to have all dependencies included.
Installing the library (PySide2) to my system python does not help.
I also extracted the AppImage and ran a python shell using the "embedded python" in ./usr/bin/python and could successfully import PySide2

I also found the module itself within the extract AppImage files under: conda/lib/python3.7/site-packages/PySide2

I've added a print of the exception just before this line
it printed:

cannot import name 'QtWidgets' from 'syncplay.vendor.Qt'

I cannot replicate the issue (Debian 11) which is rather strange. I tried both official release of 1.6.5 and latest master for upcoming 1.6.6.
QtWidgets is part of PySide2, so I'm wondering what might be missing here. For the record I do have all the Qt libraries which Syncplay needs on my OS anyway as I normally use .deb installation.
That said - it seems like something that's specific either to Fedora 32 or even your system in particular. Otherwise I'd expect this to be reported way sooner. I'm not intimately familiar with our AppImage packaging process so I'm not quite certain how exactly it ensures all dependencies are packaged in. I know it uses requirements.txt and requirements_gui.txt, but those seem fine.

I am also not able to reproduce on Ubuntu 18.04. Nevertheless, I think you are having an issue with Qt.py, the shim that we use to support both PySide and PySide2 at the same time.
I would try to get some logs from this shim by using an environment variable designed for that. @Sioma112233, could you please open a terminal, go where you downloaded the AppImage file, and run
chmod +x Syncplay-1.6.5-x86_64.AppImage
export QT_VERBOSE=True
./Syncplay-1.6.5-x86_64.AppImage

and report here all the terminal output?
Thanks.

and report here all the terminal output?

yeah, this new output appeared:
Order: 'PySide2', 'PySide', 'PyQt5', 'PyQt4'
Trying PySide2
Misplaced member has no source: QtGui.QStringListModel
Misplaced member has no source: QtCore.Property
Misplaced member has no source: QtCore.Signal
Misplaced member has no source: QtCore.Slot
Misplaced member has no source: QtCore.QAbstractProxyModel
Misplaced member has no source: QtCore.QSortFilterProxyModel
Misplaced member has no source: QtCore.QItemSelection
Misplaced member has no source: QtCore.QItemSelectionModel
Misplaced member has no source: QtCore.QItemSelectionRange
QtCompat: AttributeError: module 'syncplay.vendor.Qt' has no attribute '_QtWidgets'
QtCompat: AttributeError: module 'syncplay.vendor.Qt' has no attribute '_QtWidgets'
QtCompat: AttributeError: module 'syncplay.vendor.Qt' has no attribute '_QtWidgets'
QtCompat: AttributeError: module 'syncplay.vendor.Qt' has no attribute '_QtWidgets'
QtCompat: AttributeError: module 'syncplay.vendor.Qt' has no attribute '_QtWidgets'
QtCompat: AttributeError: module 'syncplay.vendor.Qt' has no attribute '_QtWidgets'
QtCompat: AttributeError: module 'syncplay.vendor.Qt' has no attribute '_QtWidgets'
QtCompat: AttributeError: module 'syncplay.vendor.Qt' has no attribute '_QtWidgets'
QtCompat: AttributeError: module 'syncplay.vendor.Qt' has no attribute '_QtWidgets'

This issue seems to be related

The issue you linked does not match your log, which rather indicates that the AppImage was unable to import QtCore.
I installed a Fedora 32 VM to debug this issue. On Fedora 32, I can indeed reproduce. After mounting the AppImage and running the embedded Python interpreter, I got:
Python 3.7.7 (default, May  7 2020, 21:25:33)
[GCC 7.3.0] :: Anaconda, Inc. on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from PySide2 import QtCore
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ImportError: /lib64/libk5crypto.so.3: undefined symbol: EVP_KDF_ctrl, version OPENSSL_1_1_1b

A quick search revealed this is a known issue with PySide2 and conda on Fedora: https://bugreports.qt.io/browse/PYSIDE-1357?gerritReviewStatus=All
Since we depend on conda for packaging the AppImage and since the PySide2 team already stated they cannot/would not fix this bug, I would advise you to open a new issue in the corresponding conda-forge repository: https://github.com/conda-forge/pyside2-feedstock , describing your problem and, perhaps, linking to this comment.
As a workaround, I would suggest to install Syncplay via snap, which is possible with some care. After that, though, I encountered issues in running both mpv and VLC, so, I stopped playing with this setup. If you are still interested in this route, I can try to provide more support, but I would advise to report encountered problems to the player's issue trackers.

At the meantime I use an older version (1.6.4a) which works fine (didn't encounter any problems yet when connecting with friends that are running 1.6.5).

ImportError: /lib64/libk5crypto.so.3: undefined symbol: EVP_KDF_ctrl, version OPENSSL_1_1_1b

Yeah I had this error after trying to manually update Qt.py with the latest release
Thanks for your time, I'll open an issue the the relevant party

@albertosottile If I understand correctly the issue is with the conda packaging of PySide2 right? They should include libk5crypto.so binary and not depended on the system-installed version

Yes the issue is in how conda is packaging PySide2, but I am not sure about the solution you propose, given that the issue affects only Fedora.

@Sioma112233: another option is to just use the make install after downloading the zip file. Assuming you have all the required libraries on your system it should "just work".

Closing this as we determined that the issue was outside our codebase.


four programs added

@Samarthasbhat add solution links for all these problems in readme file

@Samarthasbhat add solution links for all these problems in readme file

I have merged your MR but please add solution links for above problem in next commit

Ok


updated promote artifacts lib s3 upload

Signed-off-by: Abhinav Gupta<EMAIL_ADDRESS>Description
Remove s3 upload to builds and dist directly from promoteArtifacts.groovy
Issues Resolved
Remove stray artifacts from release folder
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@abhinavGupta16 Care to explain why adding this was a mistake, please?

@abhinavGupta16 Care to explain in the PR description why adding this was a bad idea, please?

Thanks @abhinavGupta16 for the quick fix.
A bit context: I added the logic to upload builds/dist under releases url during the tar-promotion job as part of the following PR
https://github.com/opensearch-project/opensearch-build/pull/1490 with the intention to let Dashboards integ test run against a static OpenSearch url.
However, the tar-promotion job https://github.com/opensearch-project/opensearch-build/blob/main/jenkins/promotion/tar-promotion.jenkinsfile only runs once manually during release meeting. Thus the url won't be ready (which also means that the Dashboards integ test won't get the OpenSearch url) until the last minute.
A better option would be uploading such builds/dist in a periodic automatic job, e.g the distribution build job. Also since this build will be ongoing and is not actual release artifacts, we can consider uploading to release-candidates url.

A better option would be uploading such builds/dist in a periodic automatic job, e.g the distribution build job. Also since this build will be ongoing and is not actual release artifacts, we can consider uploading to release-candidates url.

I think https://github.com/opensearch-project/opensearch-build/issues/1492 can solve this in a way where "latest" = latest released or latest build.


improve detection of "perfect" QR codes

Hey, thanks for this great library.
We found that it had some issues with "perfect" QR codes which are not captured by a camera but e.g. read from the clipboard. This PR adds some improvements and tests for those QR codes.
It's not 100% polished (there are some TODO's left) but it works well enough in real-life use-cases.

Great contribution! Thank you!


Proposal Page - Time Misalignment between Toaster execution alert and the execution button

See video at 5.23 and 3.23
https://www.loom.com/share/3af51acc15ae4f3282c3287eebcc44d8
When the toaster pops up showing a 'executed successfully' for a brief second the button on the proposal page (currently greyed out as it should be) will light up orange again. If the user clicks it, it will error (as expected) ... but why is it lighting up at all (i.e. prompting a user action)
There's clearly a lag when the UI hasn't fully caught up with the toaster on the current proposal status.
This lag maybe impossible to fix, ...but lets explore removing the 'light up' of the button. The button should remain grey until an actual transaction possibility is on the cards

@adamgall this one I'd like to bring to Ready column.
Is there more detail needed here?

I don't think there's any more detail needed. My only question is, does this still occur? Is it consistent? @tomstuart123

I'll try and replicate by end of day

I'm assuming this is Multisig Safe?

Sorry for the delay on this @adamgall / @Da-Colon. I finally, got to this and was able to replicate this . It happens on

timelock execution  - very briefly  (like 1 second)
execute tx - much longer (I was able to click between screens and return to see the button still open to clicking. If i clicked i got the same error messaging

See video for the number 2) - https://decentdao.slack.com/archives/C06QZTUTKRC/p1717496070661879?thread_ts=1717419826.480139&cid=C06QZTUTKRC
See link to Safe it happened on

Would like to get eyes on this so moving to P1

@DarksightKellar is #1511 related to this issue too?

@adamgall - it sounds similar.... but I'm still not entirely sure what 1511 is meant to do.
@Da-Colon if you can confirm what you meant in the above issue?


typescript support

This package doesn't have any typescript definitions so typescript projects will struggle to work with it.

Thank you for your report! Starting from version 4.1.0 package includes typescript definitions :relaxed:
npm install -SE<EMAIL_ADDRESS>


Update deps 1.11

update deps to main

Codecov Report

Merging #534 (ed12f17) into main (7973b80) will decrease coverage by 0.15%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #534      +/-   ##
==========================================
- Coverage   66.23%   66.08%   -0.15%
==========================================
  Files          38       38
  Lines        2079     2079
==========================================
- Hits         1377     1374       -3
- Misses        657      660       +3
  Partials       45       45

see 1 file with indirect coverage changes


Add code search functionality

This PR adds code search functionality
@robbecker-wf

Closing and rebuilding this PR to make additional fixes in https://github.com/DirectMyFile/github.dart/pull/120
I kept Claire's original commits though.


[ MB-16199 ] Updating test harness scenarios

Summary
@michaelvaldes-truss brought this up in a previous review of #10806 and instead
of holding up that review, I went ahead and added the suggested changes over in
this PR.
See the conversation here: https://github.com/transcom/mymove/pull/10806#pullrequestreview-1475510265

Messages

:book:
:link: MB-16199

  Generated by :no_entry_sign: dangerJS against 76806b5a0127d6f3f67d29c02e745c33b0f14261


"decoupleChildFromParent=true" I need to select the parent Node by default. please provide me a solution

Please help me to support and maybe need a pull request, I am very busy now, and can't support this immediately. Sorry :(


Replace executable jar with GeneratedStrategyVerifier utility class

This reverts https://github.com/RoboZonky/robozonky/commit/742cac81b643cc66b93e5d64c6d154d93bc36150 and adds separate helper class instead. My idea is to use the new helper class like this.
I don't like that parsing failure is signalled by exception (I'd prefer seeing list of errors or something when parsing fails) but looking at code generated by antlr I didn't find a way to extract those errors in any other way.

FYI @jhrcek faee3d6f80584a96e0eb8a62a46e3e732189682a


Vendor CLI11 library inside ignition-utils

Depends on: https://github.com/ignitionrobotics/ign-cmake/pull/143
Signed-off-by: Michael Carroll<EMAIL_ADDRESS>

From @scpeters homebrew has a package: https://github.com/Homebrew/homebrew-core/blob/master/Formula/cli11.rb

From @scpeters homebrew has a package: https://github.com/Homebrew/homebrew-core/blob/master/Formula/cli11.rb

looks like it's in vcpkg as well: https://repology.org/project/cli11/versions

looks like it's in vcpkg as well: https://repology.org/project/cli11/versions

@scepters and I discussed this with @j-rivero offline. Since this will only be used in executables, is header only, and has no existing Debian package, it makes sense to include the CLI11 headers as part of the ign-utils install.
Additionally, rather than using the packaged versions for vcpkg and homebrew via cmake logic, we can use the same set of headers across all platforms for consistency.
This is still pending me developing a prototype for using this with ign-tools

@scepters and I discussed this with @j-rivero offline. Since this will only be used in executables, is header only, and has no existing Debian package, it makes sense to include the CLI11 headers as part of the ign-utils install.
Additionally, rather than using the packaged versions for vcpkg and homebrew via cmake logic, we can use the same set of headers across all platforms for consistency.
This is still pending me developing a prototype for using this with ign-tools

Closing and reopening to see if DCO takes effect

Closing and reopening to see if DCO takes effect

How do we deal with side-by-side installs in the future? We have used inline namespaces for this in our other packages, but CLI will have its own namespace. This could potentially be a problem down the road if we want to update the version of CLI we vendor.

How do we deal with side-by-side installs in the future? We have used inline namespaces for this in our other packages, but CLI will have its own namespace. This could potentially be a problem down the road if we want to update the version of CLI we vendor.

How do we deal with side-by-side installs in the future? We have used inline namespaces for this in our other packages, but CLI will have its own namespace. This could potentially be a problem down the road if we want to update the version of CLI we vendor.

are you concerned about conflicting header file locations or conflicting symbols? I would expect CLI11 to only be used in executables, so I wouldn't expect problems with symbols

How do we deal with side-by-side installs in the future? We have used inline namespaces for this in our other packages, but CLI will have its own namespace. This could potentially be a problem down the road if we want to update the version of CLI we vendor.

are you concerned about conflicting header file locations or conflicting symbols? I would expect CLI11 to only be used in executables, so I wouldn't expect problems with symbols

Yeah, I think you're right. If we limit its usage only to executables, we shouldn't have a problem.

Yeah, I think you're right. If we limit its usage only to executables, we shouldn't have a problem.

let's go ahead and merge and make a prerelease

We could also use a nightly

we don't have to wait a day if we make a prerelease now


Add to vote continued

Sometimes, playing on a good map, everyone has a desire to continue it. And they do not want to change the map to any other.
To do this, you can make the option "Continue playing on actual map" in the selection vote menu.

Added two new cvars to allow this


Merging PR: Speedup placement hosts update

Description
Merging #4359 because contributor's fork does not allow updates from maintainer.
Issue reference

Please reference the issue this PR will close: #4359
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

Co-reviewed with @artursouza, merging.
This PR changes some core aspects of the actor runtime and needs to be examined throughly during release.


HTMLRenderer | rendering of empty paragraphs

There is an issue if you are using empty paragraphs as spaceholder
Steps to reproduce:

Add a textbox
Enter some text + Press enter multiple times to add space
Render the content with the HTMLRenderer
the space is gone

Version:

0.2.25

Problem:
Ory adds something like this:

After the renderer processed the content only empty p-tags are left:

Possible Workaround:

Use shift+enter to add linebreaks instead of empty paragraphs

I think the problem is that the p tags are considered empty, so nothing is included. The span/br thing is included during editing so it's possible to put the cursor in that line. In rendering mode however, this hack is not included as it's not necessary, leading to an empty p tag and thus the browser not displaying any space for it as it's empty.
I think one possibility is maybe setting min-height or using the work around you mentioned. I'm not sure if newer slate versions have solved this.

Yeah, but See the workaround is okay for me AS the developer. As an enduser
i would expect, that everything in rendered mode looks exactly like in
editmode. Thats the thing of something like a wysiwyg editor. And this
should work out of the box. Without stripping the p Tag content.
Am 08.12.2017 1:30 nachm. schrieb "Aeneas"<EMAIL_ADDRESS>I think the problem is that the p tags are considered empty, so nothing is
included. The span/br thing is included during editing so it's possible to
put the cursor in that line. In rendering mode however, this hack is not
included as it's not necessary, leading to an empty p tag and thus the
browser not displaying any space for it as it's empty.
I think one possibility is maybe setting min-height or using the work
around you mentioned. I'm not sure if newer slate versions have solved this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ory/editor/issues/496#issuecomment-350252166, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACKOYKiBOnN8KnhzX2Xgr-_LNKEwyeZJks5s-SvagaJpZM4Q62k2
.

As I mentioned above you could solve this easily with a CSS rule. Personally, I think using empty lines to layout is malpractice and leads to shitty readability on mobile which is why I think it is ok to strip them in the standard version, since it is possible to just write one CSS rule changing this.

I know and i solved it with a CSS rule, but i thought ory is there to give
the user something like wysiwyg for a static site. So i expected it should
be the standard behavior to transform the look of the editor view to the
rendered result. ;)
Am 08.12.2017 3:51 nachm. schrieb "Aeneas"<EMAIL_ADDRESS>As I mentioned above you could solve this easily with a CSS rule.
Personally, I think using empty lines to layout is malpractice and leads to
shitty readability on mobile which is why I think it is ok to strip them in
the standard version, since it is possible to just write one CSS rule
changing this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ory/editor/issues/496#issuecomment-350281756, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACKOYCuGnGNwgfpJzVbYrhsfsV_gnopnks5s-UzwgaJpZM4Q62k2
.

The editor is specifically designed to not make the same mistakes that we had for the last 10 years with things like TinyMCE or CKEditor. We wanted to give freedoms, but also keep restraints. Writing digital content requires some experience and understanding of how things work on mobile. Using tables for layout (looking at you TinyMCE) or having 10 newlines so things align is not how we create content that works everywhere. Compromises have to be made, this is definitely one of them.

Goooooood ckeditor... Memories and flashbacks :D. So i am using quill for
those reasons.
I und erstand you, but i think it is missleading , if even the preview in
the editor mode shows the space and rendered result not.
My Problem is not to add this CSS rule. And i am totally with you and the
mobile first approach. But the editor should show somewhere some how the
real rendered result. Without making changes in my own.
Am 08.12.2017 4:06 nachm. schrieb "Aeneas"<EMAIL_ADDRESS>
The editor is specifically designed to not make the same mistakes that we
had for the last 10 years with things like TinyMCE or CKEditor. We wanted
to give freedoms, but also keep restraints. Writing digital content
requires some experience and understanding of how things work on mobile.
Using tables for layout (looking at you TinyMCE) or having 10 newlines so
things align is not how we create content that works everywhere.
Compromises have to be made, this is definitely one of them.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ory/editor/issues/496#issuecomment-350285511, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACKOYKi5lNgABBOq5nvf5eYMnc6Yv3B2ks5s-VBxgaJpZM4Q62k2
.

That's a good point. Newlines are particularly tricky because without the br tag in edit mode, it will be impossible to actually click on the line. With an empty editor it will not even be possible to select the text area. One possibility would be to have a rule that adds a br tag to empty paragraphs, but I don't think I'd like to see it as default.

close due to inactivity. Please reopen if you still experience problems with it!


new voting sheets, generated in Python (notebook) not Makefile

needs better documentation/organization/refresh plans for the files currently in notebooks/environmental_context_value_sets/
mam-env-package-overrides.tsv: can't be recreated from within this repo. do not delete.
common.py: nobody uses this yet. there are some functions that use global dicts for very crude caching
pulled on demand, not committed:

ncbi_biosamples_2024-09-23.duckdb
ncbi_biosamples_2024-09-23.duckdb.gz
goldterms.db
goldterms.db.gz

ncbi_biosamples_2024-09-23_package_usage.tsv
nmdc_env_package_predictions.tsv
environmental_context_value_sets.Makefile
environmental_context_value_sets/README.md

Here are the PO terms that were used by any of our sources:
PO:0000003,whole plant
PO:0009007,portion of plant tissue
PO:0009008,plant organ
PO:0009011,plant structure
PO:0025025,root system
PO:0025034,leaf

We could add more of these plant structure descendants without needing to consult any additional ontologies

closing in favor of

https://github.com/microbiomedata/submission-schema/pull/280
https://github.com/microbiomedata/external-metadata-awareness/pull/35


Setup CloudWatch alerts and integrate with Slack

@mukul-mehta this was done. can close it now.

Done. We now have AWS CloudWatch alerts delivered right to Slack!


debian: dont strip packages.json also inject shebang

Hi i noticed a couple of issues with the current Debian packaging, in that i was stripping packages.json resulting in node not being able to find any dependencies and also the resulting bgptalert bin had no shebang.  This PR should fix both issues

Thanks!


About the training hyper parameters.

Thank you for the great work. I am really interested in your holomorphic network. However, I ran into some issues when trying to run your code. I ran wildlife_ppo.py with the default hyperparameters, but the training failed to converge and the average return remained around -5. Can you please provide some insight into why this might be happening?

Hi, thank you for your interest in our work and for letting me know about the missing files! The missing scripts are now included in examples/scripts/, if this does not resolve the issue please feel free t reopen the issue.


Auditing use of unsafe rust

Hi - I am new to the logos ecosystem.
I don't want to cast any doubt on the correctness of logos or that anyone should choose to use it - it looks like its a remarkable project that is clearly a great choice for many users.
I am upstreaming a crate which uses logos to generate a lexer ( https://github.com/bytecodealliance/wasmtime/pull/8872 ) into wasmtime. As part of accepting logos to be used as a transitive dependency in the wasmtime project, I need to certify that it meets cargo-vet's safe-to-deploy criteria: https://mozilla.github.io/cargo-vet/built-in-criteria.html#safe-to-deploy .
The lexer will be handling untrusted input, and after spending some time examining the way logos codegen works, I don't feel that I can certify that the use of unsafe rust is sound: while I don't have any evidence that it is unsound, the code generator, and the code it generates, is too complex for me to reasonably declare that any use of the logos derive macro is fully sound, as would be implied by marking it as safe-to-deploy.
My rough understanding, from reading the code generator and the author's blog, is that performance is a huge goal of logos, and it has achieved very high performance. In my use case, I don't really care about performance: the inputs to the lexer are small and infrequent, and if the lexer was one or two orders of magnitude slower, that would be fine. However, I do care about correctness, to the point of being very conservative in what dependencies I can accept.
One path forward might be to have an alternative code generator for the logos macros that uses entirely safe rust. Have the logos authors ever considered this approach? Otherwise, I will have to rewrite my lexer by hand, and in doing so I will lose composability with other logos lexers in our ecosystem.

Hello @pchickey! Thanks for your comment on using unsafe Rust.
I am not the author of the code itself, so I cannot really guarantee that the use of unsafe code is done safely, nor that it is actually important at every place it is used.
But I guess that @maciejhirsz knows that better than me, and could maybe give his opinion.
I think an important first step would be to investigate the use of the unsafe keyword across all crates, and maybe list them here. I know some Rust tools have been developed to this end. Second, we could analyze each use-case separately, and see if we can simply go back to a safe alternative without loss of performances.
The solution to have a opt-in for safe (or unsafe) Rust could be obtained via a feature, #[cfg(feature = safe-only)] for example, but I wonder if that is that easy to implement, or if the compile-time might increase a lot.
This question needs further investigation, and help from anyone is more than welcome!

I also have an application for logos handling untrusted input, so I took some time today to audit the usage of unsafe code in logos.
The logos crate itself does not pull in any external dependencies (yay!), so the only source of unsafety is the crate itself. Of these, there are four areas using unsafe: read_byte_unchecked, slice_unchecked, Chunk::from_ptr, and ManuallyDrop.

read_byte_unchecked - these are defined in LexerInternal and Source traits, but not actually used anywhere. Disposition: remove entirely
slice_unchecked - defined by the Source trait, and that trait already defines a safe alternative slice. Disposition: feature flag
from_ptr - defined by the Chunk trait. Disposition: add a from_slice function and feature flag the usage
ManuallyDrop - wraps the temporary storage of token in the parser. Disposition: use Option::take instead and feature flag the usage

Beyond that, there is an additional safety issue with the current implementation. The Source trait may be implemented by the user as a safe trait. However the data returned by the functions of this trait are ultimately used for pointer manipulation in unsafe code. In particular if the source trait implements the len, find_boundary, or is_boundary functions incorrectly, then this can lead to out-of-bounds access in the unsafe code.
If the unsafe code is enabled, then the entire Source trait should be marked as an unsafe trait to indicate to the implementer that incorrect implementation can cause UB.
I've implemented all of the above, except for marking the Source trait unsafe.
There is a decision to be made around including allow_unsafe in the default features (unsafe opt-out), or not and allow for it to be opt-in. Or even, less conservatively, removing unsafe code entirely. I have some opinions but there are pros/cons to each option which warrant a discussion.
Regarding benchmarks, I hoped there would be a clear difference which would help make that decision. But compilers are smart and there are surprises! I'll include benchmark results on the PR.

@pchickey The forbid_unsafe feature in https://github.com/maciejhirsz/logos/pull/413 is now ready for review. I think this should address the unsafe concern that you raised in this issue. If you have a moment to check it against your PR I'm happy to make any changes needed before that gets merged.
Should just need to add a forbid_unsafe feature to your logos dependency and both the unsafe code internal to the logos crate, and the unsafe code generated by the proc-macro will be disabled.
Some benchmarking showed that there wasn't a massive throughput difference, but the performance outcome will depend greatly on your specific grammar.

@davidkern Thanks so much David for your work on this feature! @jeertmans Can you ping me when a new release is made, so I can move https://github.com/bytecodealliance/wasmtime/pull/8872 forward?

Done with v0.14.2 @pchickey ;-) Note that you can also subscribe to new release notification if you want!


Fail to deserialize when charset is provided

Hello,
Following the refacto https://github.com/MarketSquare/robotframework-openapitools/pull/18, when the content type returned by the server contains the charset, e.g. application/json;charset=utf-8, the response validator always fails because the deserializer tries to pass as input argument the charset:
Traceback (most recent call last):
  File "/Users/Shared/robocorp/ht/***/lib/python3.9/site-packages/openapi_core/deserializing/media_types/deserializers.py", line 48, in deserialize
    return deserializer_callable(value, **parameters)
  File "/Users/Shared/robocorp/ht/***/lib/python3.9/json/__init__.py", line 359, in loads
    return cls(**kw).decode(s)
TypeError: __init__() got an unexpected keyword argument 'charset'

I'll see about addressing this one somewhere this week, should be a simple fix.


Tags crash the app

Description
Whenever I set a tag on a file, the file never shows up on the tag filter, and a pop-up notification shows up: "something went wrong: files ran into a problem that the developers didn't prepare for yet".
Steps To Reproduce

Open the Files app
Set a color tag on a file
Choose the tag color filter on the sidebar
No files show up, and the app crashes.

Steps to fix

eg. Fix crash that occurs when...
eg. Correct action happens when clicking...

Files Version
Files (Preview) version <IP_ADDRESS>
Windows Version
Windows 11 Pro 22H2 10.0.22621 Build 22621
Relevant Assets/Logs
debug.log

Thanks for reporting. I have sent a quick fix to this issue.


Retry 'service unavailable' failures in proxy test

Ref #10792
cc @davidopp @quinton-hoole

LGTM
@quinton-hoole is out for a month.
We won't be able to get a shippable result until we fix #11136 and #11106

shippable is fixed - @davidopp mark this OK if you're ready to merge it

@thockin - ready for merge

I don't know about this-- we already try N times, the point of which was to make sure it always works. If the network is eating our requests that's a bug that needs to be fixed. Adding this retry logic kinda defeats the purpose.

@lavalamp - I would agree if this was because of our network (e.g. kube-proxy or sth like that eating our requests). However, in this case apiserver was sending a request: "GET http://"kubelet-ip":10250/logs" and it's not reaching Kubelet at all. IIUC it's not going through any other components - it should just reach Kubelet. Or am I missing something?

To give you a bit more context:

an example failure is: kubernetes-e2e-gce/7561

at 22:26:06 apiserver gets the request to proxy logs from node: e2e-gce-minion-4qfv:1025:
  I0712 22:26:06.333028       8 handlers.go:137] GET /api/v1/proxy/nodes/e2e-gce-minion-4qfv:10250/logs/: (2.052688ms) 503

it sends the request directly to Kubelet using its IP address: "https://<IP_ADDRESS>:10250/logs/'

Kubelet doesn't even get the request

Kubelet is up at that time, it's ready, it correctly responded to all requests before and after it - it just didn't respond to that one particular request

Unfortunately we can't get more information out of the logs that we have, but from Kubernetes perspective everything seems to be correct. My feeling is that it is some problem with GCE networking, that's why I added this retry mechanism - but if you have any other suggestions, I'm happy to revert this change if you know what else can be wrong.

I agree with you that it seems mysterious, and I don't have an explanation. But I'd rather have it count up how many requests succeeded/failed (like ping) so we have records in our test logs and can keep track of this mysterious request loss. We can set a threshold that keeps our tests green. We can get GCE networking folks involved. We should not be papering over this, IMO.

It would be nice to log failures, that's a great point.  I just can't repro a lot of these things readily.

I agree with you that it seems mysterious, and I don't have an explanation. But I'd rather have it count up how many requests succeeded/failed (like ping) so we have records in our test logs and can keep track of this mysterious request loss. We can set a threshold that keeps our tests green. We can get GCE networking folks involved. We should not be papering over this, IMO.
@lavalamp
SGTM - I will leave the threshold as I made it in this PR (10% requests can fail), but I will add logging (log error if there will be non-zero number of failures). I will prepare a PR tomorrow.
Does it sound ok for you?

Yes, thanks.
On Tue, Jul 14, 2015 at 10:54 AM, Wojciech Tyczynski <
<EMAIL_ADDRESS>wrote:

I agree with you that it seems mysterious, and I don't have an
explanation. But I'd rather have it count up how many requests
succeeded/failed (like ping) so we have records in our test logs and can
keep track of this mysterious request loss. We can set a threshold that
keeps our tests green. We can get GCE networking folks involved. We should
not be papering over this, IMO.
@lavalamp https://github.com/lavalamp
SGTM - I will leave the threshold as I made it in this PR (10% requests
can fail), but I will add logging (log error if there will be non-zero
number of failures). I will prepare a PR tomorrow.
Does it sound ok for you?
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/pull/11135#issuecomment-121322669
.

@thockin is unable to reproduce. I think we should not assume this is a GCE network problem. Also see #11119 which sounds an awful lot like this problem.

@lavalamp - I also tried to reproduce it many times and didn't manage to do it. My hypothesis (but this is just a hypothesis and I don't know how to confirm it) is that it depends on the load on GCE and it fails if the load in GCE is high.
Also #11119 talks only about Vagrant - these failures happens on GCE so this doesn't need to be the same thing.


Remove Redundant StartMempoolSyncRoutine Starts Before Mempool Is Initialized

#1133  👈
feature/proof-of-stake

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tholonious and the rest of your teammates on  Graphite


⚠ Fetchers has degraded performance

In ffbd873, Fetchers ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 317 ms

Resolved: Fetchers performance has improved in 379a18c.


Can anyone have reproduce the paper result?

you can download dataset from pretrained model

you can download dataset from pretrained model


exception occur handle?

how to handle error perfect?
currently in effect loop only success result handle correctly. but if an error occur, the rx effect loop will exit

expect that default effects catch the error and rethrow it.

Maybe related to https://github.com/ngrx/effects/issues/112#issuecomment-299869758?

@xsilen Even if you do catch the error and generate a new Observable from it, effects will stop running.

Hi ,
When i receive any error from http (e.g. 404, 415 etc ) into effects catch then my UI gets freeze .
Unable to do any action on UI (e.g. click on menu , click on button etc).
When i am calling http service from components rather than effects and handling error into the error block of subscribe inside component then everything is working fine.

@vkvarma currently @nfantone 's answer will be usefull

This must be a defect !. The catch should not cause a permanent freeze. The workaround is to catch any exceptions inside the observable mergeMap, switchMap.. and not let the exception get to the outer catch Handler. Very unfriendly..


tscTask() does not support mutually exclusive build option

For better or worse, TypeScript chose to make the --build CLI parameter mutually exclusive from all other parameters, including the project parameter. Further, TypeScript requires the use of the --build parameter in order to respect the composite: true and/or references options in the tsconfig.json.
This conflicts with Just's current implementation of tscTask(), which always sets the project if it can successfully find the tsconfig.json.
Proposal: If build is true in the options argument to tscTask(), then skip setting the options.project value itself and instead provide the path to the discovered tsConfigFile as documented in https://www.typescriptlang.org/docs/handbook/project-references.html and party on.

Good catch!

Congrats on making the bug #123... It's being addressed in #128

This has been released on just-scripts 0.21!


AuthHttpClient.java:46-49: Based on the input from...

The puzzle 182-2f110c5f from #182 has to be resolved:
https://github.com/amihaiemil/docker-java-api/blob/49e9cb18f0a8d2da9e9e0bc5604dd8047d689ba5/src/main/java/com/amihaiemil/docker/AuthHttpClient.java#L46-L49
The puzzle was created by Boris Kuzmic on 06-Dec-18.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@amihaiemil please pay attention to this new issue.

@amihaiemil/z please, pay attention to this issue

Job #208 is now in scope, role is DEV

The puzzle 182-2f110c5f has disappeared from the source code, that's why I closed this issue. //cc @amihaiemil

The job #208 is now out of scope

Order was finished: +30 point(s) just awarded to @bkuzmic/z


Is there any way we can find the id of a playlist without searching for it manually?

Seeing as there are a lot of functions requiring the id of a playlist, the only way I can see how is using the user_playlists function, loop through the results and match it by the desired name, and get the id there. I think there would be a benefit with adding this functionality with the get go, unless there is some easier way I could not find.
Additionally, I am creating the playlist and immediately adding tracks to it. It would be nice to have the playlist id as a returned value to the user_playlist_create function. Unless this is already a thing, and it's not in the documentation.

I haven't used the spotipy user_playlist_create function but I would think it returns the playlist object from Spotify's playlist endpoint. Create a playlist documentation shows the playlist id is returned as part of the playlist object so it should be an easy value to get.
Hope this helps

Thanks so much!


JDK8 Solaris/SPARC builds disabled

I believe this was because they were taking over six hours to complete:
https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk8u/job/jdk8u-solaris-sparcv9-hotspot/

I'm working on getting access to other solaris hardware. If we are able to get access to other hardware then we could re install a fresh version of Solaris on our sparc server and see if that fixes things.

The question is what's causing it to be slow? And can we re-enable it (despite it being slow) and have it complete before te other platforms have finished testing?

Re-enabled at https://github.com/AdoptOpenJDK/openjdk-build/pull/2222
Slowness raised at https://github.com/AdoptOpenJDK/openjdk-build/issues/2206
Fix to upstream build problem (which we backported into 8u271 and 8u275) has now been merged into the main JDK8 stream has now gone in