Tal

Issue Addressed

Proposed Changes and Benefits

I am adding Tal Zigelnik to Before I Die with images and text.
Screenshots

Notes to Reviewers

Checklist

[X] I have thoroughly read and understand The Contribution Guidelines.
[X] The title of this PR follows the keyword: brief description of change format, using one of the following keywords:

Feature - adds new or amends existing user-facing behavior.
Chore - changes that have no user-facing value, refactors, dependency bumps, etc.
Fix - bug fixes.
Docs - fixing typos and adding new content.

[x] I have listed my proposed changes and their benefits.
[X] I have linked the corresponding issue in the Issues section.

@zigelnik   Your pull request has been successfully merged!!! Congratulations
If you haven't already, please go ahead and star this repo and you are more than welcome to continually contribute to this project by assisting with our open issues, or adding your own new open issues, and additionally you can get an idea of ideas we have to continually grow this project further as technology and tech stack continually evolve by looking at the ROADMAP for the project. Have a great day, and thank you for taking the time to contribute, Tal! You can view your contribution now on the live link. Have a great weekend and will be sending you an invite to be a member of the Before I Die Code now.
Xander

Hi @zigelnik,
I wanted to let you know that that preview deployment looks fantastic! Your image and text are both excellent, and all of your file uploads were correct and made a valuable contribution. It's amazing to hear that you plan on going scuba diving with sharks. Have you thought about where in the world you might like to try scuba diving with whale sharks?
Thank you for taking the time to contribute, Tal! Your work will be merged into the main branch. It was also great to see that the text Before I Die was correctly translated to Hebrew where you are based.

Thanks for merging!
So I already have a diving license and seens several sharks worldwide, but my dream is to see this one beautiful creature.
They do gather occasionally in my country in a city called Eilat which is close to the Red Sea but I want to travel to the far East and see them there as well.
I also surprised to see the Hebrew translation although it needs a Right-to-left fix according to the language properties.

Hi @zigelnik,
I wanted to let you know that that preview deployment looks fantastic! Your image and text are both excellent, and all of your file uploads were correct and made a valuable contribution. It's amazing to hear that you plan on going scuba diving with sharks. Have you thought about where in the world you might like to try scuba diving with whale sharks?
Thank you for taking the time to contribute, Tal! Your work will be merged into the main branch. It was also great to see that the text Before I Die was correctly translated to Hebrew where you are based.

Thanks for merging! So I already have a diving license and seens several sharks worldwide, but my dream is to see this one beautiful creature. They do gather occasionally in my country in a city called Eilat which is close to the Red Sea but I want to travel to the far East and see them there as well. I also surprised to see the Hebrew translation although it needs a Right-to-left fix according to the language properties.

I see. So the whale shark is pretty rare to see in only certain parts of the world? I have only swam with small sharks in the French Polynesia islands, but those were harmless, and you could touch them and weren't in danger😅 Very impressive.
Great to hear the comment on the right-to left fix as we had another contributor translate the Before I Die to Arabic and mentioned to me how it needed to be from right to left, and he went ahead and fixed this. If you are up for it can you open an issue about fixing the Hebrew translation to be displayed from Right-to-left? I can assist if you need help identifying where in the code to look. Check out pull request #101, where Sherif made the adjustment for translating Arabic. Let me know, and have a nice day!


Settings Menu

When clicking the settings from the menu, It asks me to login (I already have upon entering the app) and after signing in, it continues to ask me to sign in. Like when I click the sing in button after inputting details, it just goes back to the start.

Setting are for Apple account settings not for the app settings. IMO it should be removed completely

We kept the normal Settings option to allow users to still access the full functionality of the Apple Music webapp. To make it clear, App Settings is the option to open the Settings GUI.


Vanilla players are unable to join the server due to CCA sync requirement

Hello,
CCA is now enforcing this mod to be installed on all clients. Vanilla players are unable to join due to https://ladysnake.org/wiki/cardinal-components-api/upgrade-instructions/CCA-6-changes#client-optional-components
Here is an example of what happens(with a different mod, but the error message is the same) with a solution to enable vanilla clients to join again.
https://github.com/Ladysnake/Cardinal-Components-API/issues/174
Thank you

Ah, lovely, I completely missed that they changed that. I'll find a workaround soon™.

Thank you.
My friends and I really like this mod, thank you for developing it and sharing it with all of us. ❤

Should be fixed as of version 1.10.2 of the mod, let me know if there are any more issues.


Duplicate entry in accessibility settings

Kiwi Browser 112.0.5615.137. Android 11(oppo coloros). This is strange because i checked this on another phone(android 13 Samsung )but there is no duplicate. Is this android 11 thing or my phone is somewhere broken?

Fixed on Kiwi v116


Events on slotted elements behave unpredictable

I'm working on a select component with a button that opens the "dropdown", made with a dialog[popover], using popovertarget.
In the dialog i have options that are clickable so you can select one of them, and when you do the dropdown closes.
When testing this i ran into some inconsistencies in how it works across browsers.
I've made a simplified version of my component available here: https://main--symphonious-croissant-e608ab.netlify.app/
The code can be seen here: https://stackblitz.com/edit/vitejs-vite-yu5jkd?file=src%2Fmy-element.ts
The little counter in the option should increase with every click (and is persisted also when closing the dialog)
The difference between the options is that one is placed in the dialog via a slot and the other one directly.

In chrome (114) you can click both options and none of them close the dialog. The counter is increased in both options.
In Firefox and Safari both the options increase the counter when clicked, but the slotted option closes the dialog.
In Safari (and chrome) on iPhone the "direct" option works the same; counter increases and dialog stays open, but the slotted one doesn't seem to register the click on the element; the counter is not increased and the dialog is closed.

I couldn't find an existing issue regarding slotted elements and was wondering is this is at all supported or whether this is a bug?

Hey @Diaan! Were you able to find a workaround for this? Or was it specific to a certain framework?

Hey @Diaan! Were you able to find a workaround for this? Or was it specific to a certain framework?

No, we haven't found a work-around yet, it still doesn't work.
We are using Lit, I haven't tried it in vanilla JS yet, but from what I understand Lit doesn't add/change that much functionality in runtime.

Not sure which version (or iPhone update) fixed it, but it seems to be working now!


Add an async disruptor/ring-buffer queue abstraction for buffered communication between producer/consumer coroutines

We have async_generator<T> that can be used for producer/consumer coroutines to communicate, however it has no buffering capability which means that the elements can only be generated and processed one at a time.
We should look at something similar to https://github.com/lewissbaker/disruptorplus but that uses co_await and coroutine suspension as a wait-strategy for handling the full/empty buffer cases.

Add support for a single-threaded producer as well as a multi-threaded producer.
Add a sequence_barrier abstraction.
Needs integration with schedulers to allow producer to resume consumer asynchronously.

I have some code kicking around in a side-project that I can port to cppcoro.

I've added some initial abstractions for this to the 'disruptor' branch:
https://github.com/lewissbaker/cppcoro/tree/disruptor
This has the sequence_barrier, single_producer_sequencer and multi_producer_sequencer abstractions for dealing with sequence numbers.
Some simple multi_producer_sequencer tests on my (slow) dev machine show it can achieve a throughput of around 20M items (64-bit integers) per second for two producer coroutines enqueuing one item at a time and a single consumer consuming the items in batches and simply summing the values. A higher throughput can be achieved if producers are able to batch-enqueue items.
Still need to add a ring_buffer helper class to simplify indexing into the buffer.
Also need to add some documentation.

Interesting. I ported the disruptor to C++ about a year ago and found it to be pretty damned good; so long as you use the correct wait_strategy's for your requirements. Can't wait to try this out as we will need a messaging system at some point and if it can work with cppcoro and coroutines, all the better!

The performance of the coroutine-based wait strategy is not quite going to be able to match that of a spin-wait strategy that avoids any synchronisation other than acquire-reads and release-stores.
The coroutine wait-strategy requires a seq_cst memory barrier whenever a coroutine needs to suspend in a wait_until_published() operation. It also needs a seq_cst memory barrier for every publish() operation.
This should still be more efficent than a wait-strategy that uses std::mutex and std::condition_variable and blocks the thread, though.


Coordinate marker discrepancy between Old and New Unity viewer in hospitalupper mission 5 story composite

In discord I've posted about some trouble I've had spawning an android in the misssion story script for mission 5.
There's an offset which I've accounted for, but still the character is not being spawned in expected location or at all.
For completeness sake here's the message link: https://discord.com/channels/1116018062630592593/1116021579005038713/1189479038499172423
In talking with Darkstar he says to use Unity to locate the target entity and send those to the commands editor.
I did, but the character spawned in a different location than expected still.
He showed me a screenshot of Unity with coordinates in the Day Room. I used those numbers which work fine.
He says he's using the old version of the Unity viewer before it connected with commands editor.
There seems to be a visual marker discrepency between his version of the level viewer and mine.
When I open Unity I'm seeing the same coordinates, but the marker is not the same as his;

This first screenshot is what Darkstar sent in discord to help me locate the droid position.  The numbers on the right work fine.
The coordinate maker in the screenshot though is at issue;

This is what I get in Unity when I show the same coordinates.  The marker is in the air.
I've used the viewer to location some things like smoke bombs  in the level composites without issue.
Seems to be something quirky in this mission story composite.
So is this just a setting I've got setup wrong in Unity?  Or what is going on?
The two markers are at odds given same coordinates for both.   Only difference is the different viewer version.

The coordinates you see when you select something are relative to the parent they're contained within in the hierarchy on the left - make sure you're positioning things relative to the right parent.
I'm gonna close this issue as I suspect that's the cause, but if you still can't figure it out feel free to reopen and I'll take a look when I get chance.


ProjectedFSLib: do not copy into GVFS.Platform.Windows output dir

Currently, ProjectedFSLib.dll is expected to be copied into the GVFS.Platform.Windows
output directory during a build, and from there is copied into the installer. Due to
how this is included in the project, it is not consistently copied into
the GVFS.Platform.Windows output directory, and this can cause a build
failure simiar to:
Error on line in C:\Sources\Vfs4Git\src\GVFS\GVFS.Installer.Windows\Setup.iss: Source file "C:\Sources\Vfs4Git\src\..\BuildOutput\GVFS.Windows\bin\x64\Debug\ProjectedFSLib.dll" does not exist.

To fix this, we could fix the project setup to have the file copied over
consistently. However, this file is not needed by the
GVFS.Platform.Windows project. Post RS4, ProjectedFSLib.dll should be
provided by the operating system and not copied over the product.
Previously, the product had to bring over its own copy of
ProjectedFSLib.dll, as it was not providided by the OS. By now, we
expect developers to have moved to a version of Windows that includes
this the ProjectedFSLib.dll component in box.
This also means we can remove logic for deleting this file as part of a
post build step on systems where this file should not be provided by the
product.

I think an alternative change would be to change this to a "Content" item.

I would vote for this option as it's more 'correct' IMO. This file does not play part in the build process but is a file that is required in the final product.

I would vote for this option as it's more 'correct' IMO.

I am not against this - the main reason I didn't choose this was because of the precedence with the libgit2 native binaries. If we go with this approach, I guess we should update the libgit2 native binary props to be consistent.

Changing to Content means the item will show up in the solution tree.

Changing to Content means the item will show up in the solution tree.

Probably a good thing. Make your weak/runtime dependency on this file more visible?

Looks like our build machines are pre RS4

OS Version: Microsoft Windows NT 10.0.17134

RS4 is 1803
/cc @wilbaker @kewillford

The actual failure is:

Invalid : C:/agent/_work/11/BuildOutput/GVFS.UnitTests.Windows/bin/x64/debug/GVFS.UnitTests.Windows.exe
Unable to load one or more of the requested types. Retrieve the LoaderExceptions property for more information.
----> Could not load file or assembly 'ProjectedFSLib.Managed.dll' or one of its dependencies. The specified module could not be found.
----> Could not load file or assembly 'ProjectedFSLib.Managed.dll' or one of its dependencies. The specified module could not be found.

I am still investigating to see if this could be caused by running my changes on a Pre-RS4 machine...

Looks like our build machines are pre RS4 - from the failing tests:

Our build machine pools is RS4+, but it appears that the Hosted Agents pool is not (or it does not have the ProjFS optional feature enabled).
As we discussed offline, it's probably easiest to copy the native DLL to the GVFS.UnitTests.Windows output folder.

As we discussed offline, it's probably easiest to copy the native DLL to the GVFS.UnitTests.Windows output folder.

My latest commit does this. I used @kewillford's code to conditionally delete the ProjectedFSLib.dll if the dll exists in the system folder already... I am still not sure this is last step is necessary, as we are not actually mounting repositories in the unit test code (I don't think...).
If we don't like this approach, or if it has problems, it might just be easier to go with the original approach (but not explicitly include the ProjectedFSLib.dll package dependency in GVFS.Platform.Windows)

I used @kewillford's code to conditionally delete the ProjectedFSLib.dll if the dll exists in the system folder already... I am still not sure this is last step is necessary, as we are not actually mounting repositories in the unit test code (I don't think...).

I don't think this step should be necessary.  Assuming the tests pass with your latest changes I think it'd be worth taking the conditional delete out and see if everything still passes.


What's new in 0.24.0

Main feature is support of the custom documents
Backend Updates and Extended Schemata

[x] Switch to the AREkit==0.24.0 (https://github.com/nicolay-r/arekit-ss/commit/be14ffc3868db039f13e56d8b4f492816c528e7e)

Quality of the sampled data
According to the #52  experiments:

[x] #70
[x] #71

Dynamic prompting support
When analysing NEREL-bio outputs, it was found that some labels and relations are better to describe more precisely with prompts

[x] #67
[ ] #66

Simplify new sources registration
There were few feedbacks on bad API with labels and that whole project might be crashed because of the incompletely registered source

[x] #62
[ ] #58
[x] #59

Other / Minor updates

[x] #68
[x] #69
[x] #42
[x] doc_ids argument support which allows to select the specific doc_ids for processing (https://github.com/nicolay-r/arekit-ss/commit/90488c225bba7845af1270f3d102901ab4583b0b)
[x] [output_dir] customization removed (https://github.com/nicolay-r/arekit-ss/commit/234ae212eaf097b469ebb7eb5c3dbc514fbb490d)
[x] #72
[x] dest_lang is optional parameter (by default is src_lang) (https://github.com/nicolay-r/arekit-ss/commit/234ae212eaf097b469ebb7eb5c3dbc514fbb490d)
[x] #74
[x] :wrench:  #75
[x] #76

:partying_face:


shortest path look up from allneighborsdistances?

add a shortest path tree look up object from between snapped PointPattern objects in allneighborsdistances.

[ ] create tree lookup object

resolved in #116


Possible Redundancy?

Isn't magic method __call is called whenever there is no existing method ?
Meaning __call is never called for method_exists($this, $method)? so it is just redundant right?
https://github.com/nWidart/laravel-modules/blob/d56e518737fcc411ef197c59135a1990a95f91c5/src/Json.php#L224

In Repository.php as getCached() is already in array  what is point of using formatCached($this->getCached())?

Consistency

in update($module) method we did with(new Updater($this))->update($module); cannot we do similar in install method like this
with(new Installer($name, $version, $type, $subtree))->run();

The __call magic method intercepts ALL method calls.
At first it checks if the methods exists within the Json class instance. If not, it assumes you're calling a collection method and passes the call to the 'attributes' property which is of type \Illuminate\Support\Collection.
I hope it clarifies.
Regards!

@renanBritz hi bro.
Suppose i asked for $class->hello();
If hello is in class then it will be used before __call so why do we need to check if $class has method hello or not?

This can also check for other classes that extend this for instance, iirc.
I don't remember, check the code :D
?

I'll close this as there hasn't been a reply in a while, feel free to comment if needed.


Fix std.range.Zip.save

It's funny how the pseudo-code is actually correct.

Is there a bugzilla issue filed for this problem?

This problem doesn't show up when searching the bugzilla for zip related issues and I haven't filed anything myself (seemed unnecessary).

Auto-merge toggled on


Add Swift frameworks

Hope you don't mind I just threw all three together into a single PR. Vapor, Perfect and Kitura. All build and run fine.
I also added make and neph build targets and threw everything into the benchmarker as well.

Hope you don't mind I just threw all three together into a single PR

NP :smile:
I'll review this, thanks!!

It works very fine!!! Perfect!!!

Thanks 😊


Prevent colons and slashes in environment names

Describe your changes

Provide Linear issue reference (e.g. MOD-1234) if available.

 Backward/forward compatibility checks

Check these boxes or delete any item (or this section) if not relevant for this PR.

[ ] Client+Server: this change is compatible with old servers
[ ] Client forward compatibility: this change ensures client can accept data intended for later versions of itself

Note on protobuf: protobuf message changes in one place may have impact to
multiple entities (client, server, worker, database). See points above.

Changelog

I would suggest that we copy what we do with check_object_name and run the environment name check locally, providing a more detailed error message. (i.e., it's very annoying to get an error that says you've used illegal characters but doesn't tell you which ones they are).
Also, is this going to prevent people from renaming or deleting environments with newly-illegal names? Something to consider, as that would also be annoying...

I added a check similar to check_object_name called check_environment_name that validates the environment locally first and gives a friendlier message.
Also added some tests to make sure you cannot create/update environments with invalid names but you can update an environment with an invalid name to a new valid name. Also didn't add the checks for other environment actions, so you can still delete/list environments if they have invalid names


Misunderstanding of the estimated computing time

I am not sure whether I misunderstand something or there is an error, but when building my index with autofaiss is written Train: 16.7 minutes but takes ~11 secs Finished "Launching the whole pipeline" in 11.1440 secs?
Using 16 omp threads (processes), consider increasing --nb_cores if you have more
Launching the whole pipeline 01/28/2022, 08:15:47
There are 4269 embeddings of dim 1024
	Compute estimated construction time of the index 01/28/2022, 08:15:47
		-> Train: 16.7 minutes
		-> Add: 0.0 seconds
		Total: 16.7 minutes
	>>> Finished "Compute estimated construction time of the index" in 0.0000 secs
	Checking that your have enough memory available to create the index 01/28/2022, 08:15:47
20.6MB of memory will be needed to build the index (more might be used if you have more)
	>>> Finished "Checking that your have enough memory available to create the index" in 0.0009 secs
	Selecting most promising index types given data characteristics 01/28/2022, 08:15:47
	>>> Finished "Selecting most promising index types given data characteristics" in 0.0000 secs
	Creating the index 01/28/2022, 08:15:47
		-> Instanciate the index HNSW15 01/28/2022, 08:15:47
		>>> Finished "-> Instanciate the index HNSW15" in 0.0036 secs
The index size will be approximately 17.2MB
The memory available for adding the vectors is 7.0GB(total available - used by the index)
Will be using at most 1GB of ram for adding
		-> Adding the vectors to the index 01/28/2022, 08:15:47
Using a batch size of 244140 (memory overhead 953.7MB)
100%|██████████| 1/1 [00:00<00:00, 74.53it/s]		>>> Finished "-> Adding the vectors to the index" in 0.1602 secs
	>>> Finished "Creating the index" in 0.1647 secs
	Computing best hyperparameters 01/28/2022, 08:15:47

	>>> Finished "Computing best hyperparameters" in 3.3091 secs
The best hyperparameters are: efSearch=21
	Compute fast metrics 01/28/2022, 08:15:50
2000
	>>> Finished "Compute fast metrics" in 7.6499 secs
	Saving the index on local disk 01/28/2022, 08:15:58
	>>> Finished "Saving the index on local disk" in 0.0091 secs
Recap:
{'99p_search_speed_ms': 30.39110283832997,
 'avg_search_speed_ms': 3.7983315605670214,
 'compression ratio': 0.9678652870286923,
 'index_key': 'HNSW15',
 'index_param': 'efSearch=21',
 'nb vectors': 4269,
 'reconstruction error %': 0.0,
 'size in bytes': 18066382,
 'vectors dimension': 1024}
>>> Finished "Launching the whole pipeline" in 11.1440 secs

Yes the estimated time is not very good indeed.
We could improve it


Support scikit-build-core

Description

This PR adds a new cython-core module of rapids-cmake that can be used with scikit-build-core. cython-core is designed as a drop-in replacement for cython, allowing consumers to simply change the include(cython) to include(cython-core) and have things generally work as expected when switching Python build backends.
Checklist

[x] I am familiar with the Contributing Guidelines.
[x] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.
[x] The cmake-format.json is up to date with these changes.
[ ] I have added new files under rapids-cmake/

[ ] I have added include guards (include_guard(GLOBAL))
[ ] I have added the associated docs/ rst file and update the api.rst

Missing documentation and cmake-format updates

I added docs. The new functions are identical to the old ones, just included differently (include(rapids-cython-core) instead of include(rapids-cython)), so there are no new additions required for cmake-format at this stage.

/merge


.swipeenabled does not work.

carousel.swipeEndabled = true does not let the user swipe on the carousel to move it. one must still press the next item for the carousel to move

this works if you turn panenabled = false

Please take a look at the demo project there are two variants presented


Element not found on page when opacity is 0 and visible set to false

I know that I can use the workaround Capybara.ignore_hidden_elements = false, but it seems to me that the visible flag set to false should account for opacity 0 objects on the page.

Please provide an example that shows this issue

Example:
HTML:

Hello world

Hello world

Ruby:
@session.find('.foo', visible: false).text # returns ""
@session.find('.bar', visible: false).text # returns "Hello world"

Shouldn't the foo line also return "Hello world"?  Or is there another solution?

ok, so neither of those raises an "element not found" so the title of this issue is incorrect.  #text takes a parameter of whether or not to return invisible text as documented here - https://github.com/jnicklas/capybara/blob/1a9417061c08194d9c4332cbbac186798cd5978a/lib/capybara/node/element.rb#L55 .  Because of this your .bar example should also return an empty string as written, what driver are you using?

Ok

Thanks for your help>


[Backport release-23.11] linuxPackages.nvidiaPackages: add common patch for new kernel releases

Description of changes
Manual backport of #286058
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/cant-update-nvidia-driver-on-stable-branch/39246/3

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/cant-update-nvidia-driver-on-stable-branch/39246/25


Added state_uncertain attribute to the dictionary for the Keylock device

Description:
The Homematic Keylock Device holds an attribute to show if the status is defined or the lock was opened manually. This flag is called - STATE_UNCERTAIN
To make this attribute working, it is necassary to add the flag to the dictionary, which i have done.
Checklist:

[x] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
[ ] There is no commented out code in this PR.

Hi @trilu2000,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Oh, and of course the tests have to succeed.
./homeassistant/components/homematic/light.py:5:80: E501 line too long (85 > 79 characters)
./homeassistant/components/homematic/light.py:82:1: W293 blank line contains whitespace

Those are the two errors.

There hasn't been any movement here for more than a month. I'll close now. Please open a new PR when ready to finish.


2864 - Bump common entity model version and fix failing tests

Motivation and Context

ssdc-rm-common-entity-model was recently modified. Need to update the pom and get the tests passing.
What has changed

Bump POM version, tweak codeql config and fix test
How to test?

Run mvn clean install
Links

https://trello.com/c/21FNrXpB/2864-bump-common-entity-versions-1

 Bump common entity versions (1?)

 Bump common entity versions (1?)


Like Feature for Services

Implement like feature for services which users can like the services without any contribution or application to it.

Like feature is implemented and details are given under #80.


update deps to match velero v1.0.0

Signed-off-by: Steve Kriss<EMAIL_ADDRESS>NOTE: this PR is for the (newly created) v1.0.0 branch, and pins the dependencies to match the velero v1.0.0 ones.
cc @nrb @carlisia @prydonius

Does heptio/velero@9a62d88 need to be included?

No, but it'll get bumped for v1.1.


OnlyOffice App not configured

I installed only office on a separate server using docker and am able to access onlyoffice through that server.  I enabled the app in Owncloud and cloned the dir  as instructed, I then entered the ip address of the server that hosts my OnlyOffice on the admin page of owncloud. however when I try to open a doc from within owncloud it gives me a blank page with the message "ONLYOFFICE app not configured. Please contact admin"

Hello

I then entered the ip address of the server that hosts my OnlyOffice on the admin page of owncloud

Are you using http or https protocol? Please secify the address.
Wrong prefix may be a reason of this issue, we will add the validation in the next version
Looking forward to hearing from you
Best regards

Both my owncloud server and my onlyoffice server are using https and i am accessing both on a local network <IP_ADDRESS>(owncloud) <IP_ADDRESS> (onlyoffice)

Hello
Please make sure that the link is available from the client pc
https://<IP_ADDRESS>/web-apps/apps/api/documents/api.js

Ok, that's giving me a runtime error.

Ok, that's giving me a runtime error.

It means that the issue isn't in an integration application, make sure that Document Server is running and available.

i am accessing both on a local network <IP_ADDRESS>(owncloud) <IP_ADDRESS> (onlyoffice)

Could you please describe what is happening when you open the address of the Document Server in your local network.


Pulumi 18 create eks nodegroups

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


Skip crowdin download

Master issue: #14824
Motivation
The crowdin translations do not get updated often enough to warrant 3 downloads a day, especially given that the download and upload are currently timing out and preventing the english version of the website to get published.
Modifications

Make the crowdin-download step only run once a day.
Add license to the yarn package.json.

Verifying this change
This is a trivial change.

FYI @urfreespace

@Anonymitaet @urfreespace - I think we need to look at upgrading the crowdin cli version, so I created this issue: https://github.com/apache/pulsar/issues/14837. Also, is there a way committers can get access to the crowdin account?


Question about using embedding table

In my understanding, language models and embedding models are bound together during training. If I use a different embedding model, will the language model still work effectively? Or is it the case that the backend actually maintains different versions of the GPT model that have been fine-tuned with different embedding models, and will route to different GPT versions based on my configuration?

You can use any embedding model you like. Doesn't even need to be an OpenAI model.


fix get_objects_request passing of user_data and callback

13535 is the only really changed line - the others are just moved into the normal ordering of the API contract instead of all special cased requests being at the beginning.

error = _internal_request_dispatcher(client, request, user_data, callback, NULL, NULL, NULL);

updated with simple get_object test, and re-enabled checksum tests


ratatui Version issue

Because this crate returns ratatui::text::Text, as ratatui evolves the types here don't line-up with future version. For example at this point you can do something like
fn render_text(s: &str) -> Text {
    md_to_tui::MarkdownParsable::parse_markdown(&some, None).unwrap_or_else(|_| {
        Text::raw(s)
    })
}

will fail if the crate consuming this crate us using a different version of ratatui, right now this would provide an error like the following
error[E0308]: mismatched types
   --> src/work.rs:199:13
    |
3 | /             Text::raw(s)
    | |______________^ expected `ratatui::text::Text<'_>`, found `Text<'_>`
    |
    = note: `Text<'_>` and `ratatui::text::Text<'_>` have similar names, but are actually distinct types
note: `Text<'_>` is defined in crate `ratatui`

I am not sure what the right way to fix this but I wanted to report it as I ran into it today.

thanks for telling me about it. I kinda forgot about this crate. I don't know if i will be working on it anymore.  But for now if you want make any changes to fix it feel free and, then I can merge it.


Fixed link to Finalize a Recovered Database

Fixed a link to Finalize a Recovered Database in the Geo-Restore/Azure Portal section

Hi @tlmii, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;


Improved errors handling

I'm new to Clojure and started to use it for personal project connected with Aritable. For this reason I improved a library a little to handle 429 errors (too many requests per minute) and pagination (retrieving more than 100 records with one request). Please check the code before merging: as I said I'm newbie.

Thank you for a review. Unfortunately I'm new Github and open source contributions. How do I revert lines that you've commented? I can't find that function anywhere in Github web interface. Should I just make changes in given files on my local machine and push them to the branch of pull request?

Also please take a look at that part https://github.com/EvanHahn/airtable-clj/pull/1#discussion_r387509695 I've commented.

You'll do those reversions as part of your local Git branch, and then you'll push those up to GitHub.
You'll do something like this:
# Make sure you're on the right branch
git checkout improved-errors-handling

# Revert things
git checkout master project.clj .gitignore

# Commit those changes
git commit

# Push them up to your branch as normal, which will update this pull request
git push

Shall I take another look? Or are you still working?

Yes. Take a look please.

There's been no movement on this PR for over 2 years, so I'm going to close. Let me know if that's wrong!


Fix all mixins according to pint

Before adding a new alerting rule into this repository you should consider creating an SLO rules instead.
SLO helps you both increase the quality of your monitoring and reduce the alert noise.

How to create a SLO rule: https://github.com/giantswarm/sloth-rules#how-to-create-a-slo
Documentation: https://intranet.giantswarm.io/docs/monitoring/slo-alerting/

Towards: https://github.com/giantswarm/...
This PR fixes mixins according to basic pint recommendations
Checklist

[x] Update CHANGELOG.md
[ ] Add Unit tests
[ ] Follow Alert structure
[ ] Consider creating a dashboard (guidelines) (if it does not exist already) to help oncallers monitor the status of the issue.
[ ] Request review from oncall area, as well as team (e.g: oncall-kaas-cloud GitHub group).

Have upstream mixins been updated accordingly? So next time we upgrade we don't have to re-apply all these changes?

Which ones ? The Kubernetes ones most likely not. Thé Loki and Mimir ones nope but that's s in other PRs and it's sadly not ready yet anyway

The kubernetes mixin script was updated but for loki and mimir it's currently not merged

See https://github.com/giantswarm/prometheus-rules/pull/1131

@hervenicol I fixed the upgrade script to add the labels using sed https://github.com/giantswarm/giantswarm/issues/30310


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 185 ms

Resolved: ClamAV performance has improved in 3f0add2 after 5 minutes.


Add vscode workspace

/assign @liamfallon @johnbelamaric @henderiw

/approve

@efiacor  I'm not sure if this should be in the in the checked in code, it's a vscode thing. Maybe add an entry in the .gitignore so that it gets ignored on checkins?


merge function is crashing

when running the following command line
$ibdtools merge -i ${chr}.sibd -m ${chr}.meta -o ${chr}.mibd -M 10
ibdtools merge options received:
--ibd_in:  19.sibd
--meta_in: 19.meta
--ibd_out: 19.mibd
--max_snp: 1
--max_cm:  0.6
--mem:     10
I am getting this error:
Error from ../src/../include/ibdmerger.hpp:168:
I am not sure what it means by this error but when. I increased the memory to 500I received:
terminate called after throwing an instance of 'std::bad_alloc'
what():  std::bad_alloc
Aborted (core dumped)
any thought?

Thank you for reporting this. I will first check whether the latest commit has already fixed this issue. If not, I will need to use your files to debug. Could you share the path of the files used in this command?

Thanks!!!
I report it today to not forget it
Here are the paths:
IBD compressed files: /local/chib/toconnor_grp/TOPMed_analyses/IBD_analyses/IBDprocessing
HAPIBD results: /local/chib/toconnor_grp/TOPMed_analyses/IBD_analyses/hap-ibd_outputs
VCF files:           /local/chib/toconnor_grp/TOPMed_analyses/MAF_filtered/freeze.10b.chr${chr}.phased.mac5.vcf.gz
Genetic Maps:  /local/chib/toconnor_grp/victor/Public_data/INS_LDGH_data/hg38//IBD_analyses/genetic_maps/genmap_chr${chr}_space.txt
I did not see any flag for the vcf file
Thanks!

Oh, I meant the VCF file you used for "ibdtools encode: encode the IBD file, VCF file, and PLINK map file into binary format for better/quicker IO."
In the current implementation, ibdtools assumes all sites are phased and biallelic to achieve high memory compaction for storing genotype information. Do you know if the VCF file used in ibdtools encode contains unphased or multiallelic sites? I am checking /local/chib/toconnor_grp/TOPMed_analyses/MAF_filtered/freeze.10b.chr19.phased.mac5.vcf.gz, but it is a bit large and takes some time to finish. Could you confirm that this was the VCF file you used for ibdtools encode?

Oh! :(
I included multiallelic since hap-ibd can handle them. So maybe that is the problem; I will run a test to check if this is the problem.
By the way, I moved from the sort function to the matrix function, assuming that just the merge was problematic, and I had a similar error.
Thank you so much!

Got it. Yes, please try to use only biallelic sites if possible. The human genome has plenty of biallelic sites, which should suffice for IBD calling.

If multiallelic sites are necessary, we can consider relaxing the biallelic VCF restriction (note that refactoring the code may take some time).
If multiallelic sites are not necessary, I will add code in the ibdtools encode step to print an error message when encountering multiallelic VCF/bcf records.

Let me know your thoughts :)

matrix function, assuming

Could you also share the error message and the files used for the matrix function?


NEURON compatible morphologies from ArrayMorphology.

Allows to do something like
fname = 'Morphologies/pyr/MouseLight_Final/CNG version/AA0995.CNG.swc'
SWCLoader.load_swc_single(fname).to_neuroml_morphology("pyr_morph")

and use successfully this morphology in NEURON simulations. Tested with https://neuromorpho.org/neuron_info.jsp?neuron_name=AA0995 . If you like it you can merge it; if you'd prefer not to merge right away, you can wait. I'll continue to use this code with other morphologies for a project of mine so I'll push fixes if this does not work with other morphologies.
I attached the generated morphology, for reference. The biophys is rubbish (they are the same as for the tutorial on the OLM  cell), but the morphology seems reasonable to me.
pyr.cell.nml.zip

Hi @christian-oreilly , thanks for that. Could you rebase that against the development branch please? The master branch is "stable, released code", so all new features/fixes go into development. We then merge development into master when we make a release.
The development branch has a number of new cell builder methods and so on which could perhaps make this easier for you too:
https://libneuroml.readthedocs.io/en/development/userdocs/coreclasses.html#cell

For example, this may help:
https://libneuroml.readthedocs.io/en/development/userdocs/coreclasses.html#neuroml.nml.nml.Cell.add_unbranched_segments

Very nice to know, @sanjayankur31. I am used to the master being the "development version" and releases being flagged as such, so I did not think of looking for a bleeding-edge branch. This is very good to know! I'll look at the development code and edit the PR accordingly.

@sanjayankur31 Sorry for taking so long to get around to do that but this PR is now rebased onto development.

Well, @sanjayankur31, that is somewhat counterproductive guys (and a failure in communication) if you ask me to rebased this PR and the next day that I rebase, you put a runtime error to deprecate the method that this PR is relying on ( https://github.com/NeuralEnsemble/libNeuroML/commit/c2f829f7f29ae388162d836a19f31f5e5b82a793 ). Anyway, I'll just fork this converter out of libneuroml and provide it as a standalone if you are not interested in maintaining this as part of libneuroml as your doc https://docs.neuroml.org/Userdocs/ImportingMorphologyFiles.html do not point to any viable Python option for import SWC, as far as I can tell.

Ah, sorry about that @christian-oreilly. I hadn't looked at this PR in a while and I missed that the code was being used here. I'll revert that change to mark it for deprecation in a future release.
This rather naive coverter is not really complete/tested. For example, it runs none of the checks listed in our documentation to ensure that the generated NeuroML file is viable for modelling. A user will need to do all of these checks themselves. A full SWC converter is being worked on here:
https://github.com/NeuroML/pyNeuroML/issues/89
https://github.com/WardDPeeters/Morphologies
The plan is to include this in pyneuroml to create an interactive converter, hopefully in the new year.
Until then, the suggested method is to use neuroConstruct, which runs all the necessary checks already provides an interactive UI for the user to make tweaks where required. If your files are NEURON files, this is the pythonic way of doing it:
https://github.com/sanjayankur31/Human-L2-3-Cortical-Microcircuit/blob/feat/neuroml/NeuroML2/cellmorph2nml.py

Error softened to warning: https://github.com/NeuralEnsemble/libNeuroML/commit/f4415dcb9fde4afbf8ed3467b777be1f6c57ec88

Thanks for the info, @sanjayankur31. It is much appreciated and useful. I'll look at these different options. Your conversion from NEURON seems interesting. Do you have anything going on about converting mod files? I started a while back to work on that (based on the work of others using the domain-specific language converter textx). At the time, I could not find anything public doing a satisfying job, but I'd be very interested to know if you are working on something for that under the radar. I don't want to lose time on that if there is a more "official" solution to that already being developped.

There's no completely automated way to convert mod files that I'm aware of, primarily because they can contain equations and so on in them that need a human to go over and convert into a declarative form that NeuroML requires. We have helper tools though. They're noted here:
https://docs.neuroml.org/Userdocs/CreatingNeuroMLModels.html?highlight=chananalysis#b-convert-channels-to-neuroml
In short:

pyneuroml includes the export_mod_to_neuroml2 method which will try to fill in as much information as it can manage into a NeuroML channel file: https://pyneuroml.readthedocs.io/en/development/pyneuroml.neuron.html#pyneuroml.neuron.export_mod_to_neuroml2

Then, a human must go over the mod file and fill in the rest. To help with this there are:

pynml-modchananlyis: generates info plots about the activation variables listed in the mod files
pynml-channelanalysis: generates info plots about the activation variables used in the NeuroML

So the idea is to complete the neuroml channel file iteratively until the plots from pynml-channelanalysis match the ones that pynml-modchananalysis.
I think neuroConstruct does have lots of these utilities built in too, but I haven't used it much so i can't be sure. (I find it easiest to find existing NeuroML channel files and tweak them. The ones in the L5bPyrCellHayetal repo tend to cover quite a few and there should be more on NeuroML-DB. channelpedia also provides NeuroMLv1 exports of channels, and these can be converted to NeuroML using the pynml-channelml2nml tool.)


Feature/taskflowconfig

Implements the missing BaseTaskFlowConfig.list_flows() method, adds unit tests for that class, and splits test_config.py into two files, one that does YAML parsing and one thats fast.

Is splitting the tests out just intended to make it easier to run the fast tests without the slow ones? It would also be possible to use two test classes in the same module and tell the test runner to run one of them. I kind of like having the organization of tests mirror the organization of code (i.e. one test module per module of actual code).

Yes, that was the purpose for splitting out the tests. There are multiple TestCase classes in each file. I don't know a ton about setting up testrunners... I do think that we want the granularity to run subsets of cumulusci.core.config, since its really quite big. Thoughts on how you'd split up the existing TestCases? Part of me wants to recombine some of the smaller classes in cumulusci.core.config back into a larger module... oy...
Yes, I do want to refactor cci.py to use some of these now... I can rescind this PR if you want to wait for that, or if you are ok with multiple parts, your comments have been addressed.

You can run e.g nosetests cumulusci/core/tests/test_config.py:NameOfTestCase to run just the tests in one TestCase class.
I'm fine with cci adjustments happening later.


Editor shows error when all properties are commented

I created an .editorconfig file with the following properties:
root = true

[*]
# charset = utf-8
# indent_style = space
# indent_size = 2
# end_of_line = lf
# insert_final_newline = true
# trim_trailing_whitespace = true

As you can see all properties are commented so nothing should happen. But the EditorConfig editor shows this error in the IDE editor:

mismatched input '' expecting {'root', 'charset', 'end_of_line', 'indent_size', 'indent_style', 'insert_final_newline', 'max_line_length', 'tab_width', 'trim_trailing_whitespace', PROPERTY_KEY_UNKNOWN}

I'll fix it. (I thought that user may forget to add properties.)
Thanks.

Cool! :smiley:  You can show a warning if you want and if possible but showing an error is a bit to harsh.


FSAC runtime not found on netcore on Windows in Spanish

When initializing language services, it gives the following error:
[10:47:19 INFO ] finding FSAC for 	runtime: NetcoreFdd 	mono: mono 	dotnet: C:\Program Files\dotnet\dotnet.exe [10:47:19 ERROR] Failed to start language services. It was not
The path does exists (Program Files should translated automatically to Archivos de Programa) and it contains the executable. I've tried this in a Windows in English language and this works.
SDK de .NET Core (reflejando cualquier global.json):
Version:   3.0.100-preview-009812
Commit:    e3abf6e935
Entorno de tiempo de ejecución:
OS Name:     Windows
OS Version:  10.0.17134
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\3.0.100-preview-009812\
Host (useful for support):
Version: 3.0.0-preview-27122-01
Commit:  00c5c8bc40
.NET Core SDKs installed:
3.0.100-preview-009812 [C:\Program Files\dotnet\sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.App 3.0.0-preview-18579-0056 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 3.0.0-preview-27122-01 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.0.0-alpha-27128-4 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

This should be solvable by setting ProgramFiles environmental variable to the appropriate path (C:\Archivos de Programa\ in this case)


Security issue: User Data of instance contains secrets

After terraform creates couchbase server instances, credentials and other secrets included in user data bash script

Did you read the warnings in the comments?
https://github.com/gruntwork-io/terraform-aws-couchbase/blob/master/examples/couchbase-cluster-simple/user-data/user-data.sh#L123-L125

This repo is being archived, feel free to use a fork if necessary.


Put plugin in debug-mode for certain scenarios

When debugging a problem it can often be helpful to be able to put the plugin into a certain state, be it for testing something locally, in the browser (with #2709) or to just not be forced to do n number of steps before.
We should be able to put the plugin into a certain state, by picking from a list of possible states, and then have the plugin be put into a state where we can continue.
Those states could include

certain token sets with a couple of tokens inside them
certain themes with token sets
certain storage providers that are setup

Effectively this could also act like a good way to get seeds for local development, instead of always starting from scratch. This should only be possible in local development mode, not the live version of the plugin.
Definition of done

[ ] We can put the plugin into a certain state by choosing that state
[ ] 2-3 (or more) states that are useful are available to users
[ ] We can trigger this when environment is development
[ ] Either triggerable via console or via UI

[WIP]
Depends on:

#2709
#2934

To Discuss
Suggested Actions

Developer/debug UI hidden in settings when developer mode is enabled

Need some design/UI for this

Estimate:

To be defined: Would be awesome to have sync providers here as well so we can pick from existing storage providers without being forced to set those up


[Doc-Issue][App-manifest-link]

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/TOC.yml
98
6
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 4a5cf0b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


vector.transmissions() should take a failed argument

same for info.transmissions

This issue was moved to Dallinger/Dallinger#130


Hotfix/chobk 693

Fixed cardToken functions exports.
Old behavior:

If the seller imported createCardToken or updateCardToken from the root of the project, the method was from Secure Fields ( expected)
If the seller imported createCardToken or updateCardToken from the root of the coreMethods folder, the method was from Secure Fields ( unexpected)

New behavior:

If the seller imports createCardToken or updateCardToken from the root of the project, the method is from Secure Fields ( expected)
If the seller imported createCardToken or updateCardToken from the root of the coreMethods folder, the method is from Core Methods ( expected).

This change was made to make the SDK according to our documentation.
While developing, a more serious error was found: It was impossible to generate a card token with Fields:

This is now fixed:

Gui, acredito que a importação do createCardToken e do updateCardToken sem Secure Fields deveria ser feita apenas com o full path do arquivo, não estamos facilitando essa importação de propósito.


TT multithreading

 Issue by Trimatix
Monday Aug 03, 2020 at 17:23 GMT
Originally opened as https://github.com/Trimatix/GOF2BountyBot/issues/62

Move timedtask heaps into their own threads with their own scheduling, instead of handling them in bountybot

fixed on port to BASED


libcamera-raw: Fix video capture with full res raw images

The number of raw buffers must be set up to match the number of output
buffers, unconditionally. libcamera_app.hpp requires all streams that
will feature in the same requests to have the same number of buffers
available.
Signed-off-by: David Plowman<EMAIL_ADDRESS>

@naushir Hi Naush, another reasonably trivial one for you, I'm hoping...!


A Flux application example, tested on both Jest + Karma

Hi everybody,
this is my attempt to provide a Flux example application with tests.
Since I haven't found much examples regarding that (most of them are without tests) I decided to provide one in 2 versions by using different test environments: Jest and Karma.
Afaics both have their advantages and disadvantages and some stuff were a bit a pain in the ***, not to mention weird errors (see my FIXME comments).
Nevertheless, I hope this would be helpful for anyone starting to prototype with React + Flux with a test environment.
I would also be thankful if someone can suggest improvements or point out mistakes :)
Here a short summary of this PR:

flux-jest, flux-karma are the same example of flux application (simply renders a friends list), tested with 2 different environments
ES6 syntax
usage of Immutable data

Looking forward for any feedback, thanks! :)

Hello @emmenko,
thank you so much for building this example. I'd prefer to keep this repository lightweight and not opinionated. Would you mind putting this example into a separate repo (like jest-react-flux-example)? We can then link to it from the README.
Thank you!


Link to Archived-At value

Group mailing lists often include a URL to a location on the Web for (typically publicly) viewing a mailing list post.
The header is defined in https://tools.ietf.org/html/rfc5064
The header (and its value) show up in the "Other" tab in MHA, but getting to it, copying it out, and putting it to use in a browser (or email to someone else) is super tedious.
Would it be possible to surface this "actionable" bit of info either sooner in the UX in MHA or in it's own button/context menu item?
I've poked around in the code a bit, but haven't seen a clear route to providing a PR for this...would be happy to with some pointers! 😃
Thanks!
🎩

In Summary.js you'd want to add a row to summaryRows, with a corresponding update to the string table


Touchscreen swipes crash hyprland

Hey there,
Yesterday I installed Hyprland On my PPP, but I've been running into some issues. The issue I want to highlight here is the weird touch behavior.
If I just tap on the screen, the cursor hides and seemingly moves to my tap position and clicks, as expected. Now, if I swipe on my screen, hyprland crashes, but there is nothing in the logs, not even a crash log (So I suspect it just closes? Why though?)
I would also like to configure touch gestures, but I haven't seen an option for that in the wiki. I would use lisgd, but not if any swipes kill hyprland.
Any idea what's happening?

checked on my laptop, no crashing occurs.

I would also like to configure touch gestures

Not supported atm

checked on my laptop

On the touchscreen?

ye

Did you try swiping on a window? I tried this someme more and I can only get it to crash when I swipe on windows, and even then it takes a few seconds to do.

of it crashes, send a crash report.

bruh. There is none. It's as if it closed normally. I don't know what to do...

I got an output:
Thread 1 "Hyprland" received signal SIGSEGV, Segmentation fault.
0x0000aaaacaaf6fe4 in CInputManager::onTouchMove(wlr_touch_motion_event*) ()
(gdb) backtrace
#0  0x0000aaaacaaf6fe4 in CInputManager::onTouchMove(wlr_touch_motion_event*) ()
#1  0x0000ffffb44e9544 in wl_signal_emit_mutable () at /usr/lib/libwayland-server.so.0
#2  0x0000ffffb44e9544 in wl_signal_emit_mutable () at /usr/lib/libwayland-server.so.0
#3  0x0000ffffb469c9b4 in handle_touch_motion ()
    at /home/raccoon/.cache/yay/hyprland-git/src/hyprland/subprojects/wlroots/build/libwlroots.so.12032
#4  0x0000ffffb4698b70 in handle_libinput_readable ()
    at /home/raccoon/.cache/yay/hyprland-git/src/hyprland/subprojects/wlroots/build/libwlroots.so.12032
#5  0x0000ffffb44ebd40 in wl_event_loop_dispatch () at /usr/lib/libwayland-server.so.0
#6  0x0000ffffb44e8df4 in wl_display_run () at /usr/lib/libwayland-server.so.0
#7  0x0000aaaaca9fde68 in main ()

do you have a mouse in that? as in, a touchpad, pointer, trackball, whatever, apart from the touchscreen?

so that's why

should be fixed in 19809532df61b47750485ec7d84bedd083c809f1

Oh thank you so much that works like a charm!


[TreeView] Fix TreeItem label overflow

[x] I have followed (at least) the PR section of the contributing guide.

Applied the suggested changes by @eps1lon
Closes #27372

@LorenzHenk Awesome, thanks


Uncaught Error: spawn /Users/tvoet/.rbenv/shims/rubocop ENOENT

Uncaught Error: spawn /Users/tvoet/.rbenv/shims/rubocop ENOENT
Atom Version: 0.156.0
System: Mac OS X 10.10.1
Thrown From: Atom Core
Steps To Reproduce

...
...

Stack Trace
At events.js:85

Error: spawn /Users/tvoet/.rbenv/shims/rubocop ENOENT
  at exports._errnoException (util.js:742:11)
  at Process.ChildProcess._handle.onexit (child_process.js:1051:32)
  at child_process.js:1142:20
  at process._tickCallback (node.js:378:11)

Please install rubocop and make sure it is on your path

@benogle are you kidding me?  I've been using Rubocop on this same computer for months now, Atom crashed when i opened it up, and asked if i wanted to submit the error.  Nothing has changed.  It was perhaps a fluke, but telling ym to install rubocop is kind of ridiculous without finding out more details.  Very helpful.

Sorry about that. So far every time we have had this error reported (and we get it a lot), it is because some package is trying to spawn a binary that does not exist, or is not on the path. Please make sure the plugin spawning rubocop is able to.

I think i have diagnosed the error ( or at least the cause ).  If you happen to close atom with a blank file, when restarting atom, it re-opens said file and rubocop then fails to validate the file.  But then again, this appears to be a best guess type of situation

@benogle I've updated the issue with the steps i just sent you.

Can you open an issue on the related package?

yes.

Thanks


Cannot send alerts to multiple receivers simultaneously using matchers field.

The following example configuration using deprecated match_re field can send alerts to multiple receivers simultaneously. The alerts show up on both slack receiver and opsgenie receiver simultaneously.
But, it is not working using the recommended matchers field. The alerts only appear in the slack receiver, not opsgenie receiver.

Alertmanager configuration file:

         routes:
           - receiver: 'slack'
             continue: true
             group_wait: 10s
-            matchers:
-              - alertname="*"
+            match_re: # The 'matchers' field is using a deprecated syntax which will be removed in future versions.
+              alertname: ".*"
           - receiver: 'opsgenie'
             continue: true
             group_wait: 10s
-            matchers:
-              - alertname="*"
+            match_re: # The 'matchers' field is using a deprecated syntax which will be removed in future versions.
+              alertname: ".*"

Is the wildcard matches on alertname wrongly defined?

You need to use continue: true if you want an alert to match multiple routes.
# Whether an alert should continue matching subsequent sibling nodes.
[ continue: <boolean> | default = false ]

https://prometheus.io/docs/alerting/latest/configuration/#route

Is the mistake not * when it should be +*:
matchers:
-  alertname="*"
+  alertname="+*"

Be careful with .* as it also matches missing labels.

Hi! 👋 You are using the equal operator instead of the regex operator, and should also use +* instead of *:
matchers:
-  alertname=*"
+  alertname=~"+*`

One of =, !=, =~, or !~. = means equals, != means not equal, =~ means matches the regular expression and !~ means doesn't match the regular expression.

Be careful using .* as it also matches when the alertname label is missing.
https://prometheus.io/docs/alerting/latest/configuration/#matcher

Thanks for the tip, @grobinson-grafana!

matchers:
-  alertname=*"
+  alertname=~"+*`

There was a typographical error, I guess. Did you mean the following?
-              - alertname="*"
+              - alertname=~"+*"

Be careful using .* as it also matches when the alertname label is missing.

Is it possible to exclude missing alertname label?

Be careful using .* as it also matches when the alertname label is missing.

Is it possible to exclude missing alertname label?

Yes use +* instead of .* 🙂

            matchers:
              - alertname=~"+*"

gives me the following warning.
alertmanager ts=2024-06-13T19:53:32.537Z caller=opsgenie.go:178 level=warn integration=opsgenie msg="Truncated message" alert="{}/{alertname=~\"^(?:.*)$\"}:{alertname=\"PrometheusOperatorSyncFailed\", namespace=\"prometheus\"}" max_runes=130

How to fix this warning?

            matchers:
              - alertname=~"+*"

gives me the following warning.
alertmanager ts=2024-06-13T19:53:32.537Z caller=opsgenie.go:178 level=warn integration=opsgenie msg="Truncated message" alert="{}/{alertname=~\"^(?:.*)$\"}:{alertname=\"PrometheusOperatorSyncFailed\", namespace=\"prometheus\"}" max_runes=130

How to fix this warning?

That error message is unrelated to the matchers in the configuration file. Opsgenie has truncated the message in the notification because it's too long. Please share your Opsgenie configuration in Alertmanager (with secrets redacted).

That error message is unrelated to the matchers in the configuration file. Opsgenie has truncated the message in the notification because it's too long. Please share your Opsgenie configuration in Alertmanager (with secrets redacted).

Thank you for the information. Let's ignore this warning message as it is unrelated to the matchers in the configuration file.
Do you know if the following error message related to matchers or something else?
failed: provision alertmanager configuration: failed to initialize from secret: error parsing regexp: missing argument to repetition operator: `+`"

That error message is unrelated to the matchers in the configuration file. Opsgenie has truncated the message in the notification because it's too long. Please share your Opsgenie configuration in Alertmanager (with secrets redacted).

Thank you for the information. Let's ignore this warning message as it is unrelated to the matchers in the configuration file.
Do you know if the following error message related to matchers or something else?
failed: provision alertmanager configuration: failed to initialize from secret: error parsing regexp: missing argument to repetition operator: `+`"

Sorry I made a mistake in my previous comments, it should be .+ not +*.

@grobinson-grafana, thank you for the correction!


Update rini.h (https://github.com/raysan5/rini/issues/3)

Made line 498 const to fix const conversion error in C++.
Since const char* buffer is const in the parameter list, making a mutable pointer to buffer is illegal; however, with buffer_ptr not evidently being modified it should be able to be made const and forgotten about.

@tyqualters thanks for the review!


fix: address failure caused by method signature change in SPARK-48791

Which issue does this PR close?
Closes #692
Rationale for this change
A private method in TaskMetrics that Comet accesses changed its signature
What changes are included in this PR?
Use reflection to call the method
How are these changes tested?
Local build of Spark with the breaking change and then running HiveParquetSuite

@andygrove removed the shims and moved the code into BatchReader.

Codecov Report
Attention: Patch coverage is 0% with 12 lines in your changes missing coverage. Please review.

Project coverage is 33.69%. Comparing base (b558063) to head (3aa57ec).
Report is 6 commits behind head on main.

Files
Patch %
Lines

...ain/java/org/apache/comet/parquet/BatchReader.java
0.00%
12 Missing :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #693      +/-   ##
============================================
+ Coverage     33.63%   33.69%   +0.06%
- Complexity      821      827       +6
============================================
  Files           109      109
  Lines         42529    42567      +38
  Branches       9343     9360      +17
============================================
+ Hits          14304    14343      +39
- Misses        25262    25268       +6
+ Partials       2963     2956       -7

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


🧪 Integration test POC 🚀

This commit introduced a POC CI testing framework; this is not a
complete solution; it is a POC to show if we want to follow this way.
The main goal here is to use the kind cluster that GH actions are
running and use the Bats-core testing framework[0] to create edge devices
using an HTTP client(curl). Using yggradsil and device-worker will be
hard due to the nature of the workers, but we can try.
Each test file can contain a small test; the idea is to have a
lightweight CI on each PR, like:

Creating an edgedevice and checking that all new MTLS workflow will
work.
Trigger a small Jmetter scenario that we can reuse, but with fewer
clients.
Run operator with race conditions and deadlocks checks
Testing a deployment is correctly triggered, etc..
Be able to test getting starting guides.

I know it's bash code, but it is easy to understand and keep up to date.
(I hope I do not regret about this)
[0] https://bats-core.readthedocs.io/en/stable/tutorial.html#your-first-test
Signed-off-by: Eloy Coto<EMAIL_ADDRESS>

I'm a bit afraid of using bash for testing; been there with CNV Chaos testing (not with bats, though) and it's been pain to debug. WDYT about implementing it in Go - CNV uses go for upstream e2e tests (and we used it in the vm-import-operator).

This is a great idea and I like it.
I also think that it would be safer to test the code using golang vs bash. It will allow us using the API and generated code based on our swagger file and avoid mismatch between the API and the tests.

I agree with @jakub-dzon and @masayag let's use go instead of bash.

Closing this then!


[Add] Benchmark: NaturalBench (NeurIPS24)

Add NaturalBench(NeruIPS24):
paper
dataset

Hey, I have fixed the code style according to pre-commit requirements. Please review and check the codes. Thank you !

Hey, I have fixed the code style according to pre-commit requirements. Please review and check the codes. Thank you !

Hi,
We evaluated InternVL2-8B on NaturalBench, and the results are slightly lower than those reported in the paper, as shown below. Could you please let us know if this discrepancy is significant?

@PhoenixZ810  @linzhiqiu
The score is only about 1.% lower than what's reported in the article, which I think is reasonable.
We ensured that the dataset we uploaded is exactly the same as the one used in the paper. Possible reasons for the discrepancy include:

Differences in hyperparameter settings as we used our own codebase for evaluating.
The inherent randomness in the model's output.

We will conduct further tests across multiple open-source platforms and consider updating the results in the paper and on the leaderboard.
Thank you very much !


Godot debugging is very slow without Internet

Tested on Godot 2.0.4, Windows 10 64 bits
For a long amount of time today I had no internet connection (I had WiFi but there was no internet).
When I launch a game with Godot in that case, it takes a lot longer to start. Sometimes I even thought it was frozen, but after waiting between 8 to 30 seconds, my little test game starts. But in the meantime it looks like Godot is stuck.
I don't see a reason why Internet access should slow down the editor. Debugging uses the network for sure but since it's local I didn't expected that slowdown Oo

Try using a tool like Process Hacker which can tell you which ports are used for listening. Godot uses the port 6007 by default, I believe you can change this in the Editor Settings. If another application uses it, it can cause a significant delay every time you start your project.

Cannot replicate with Windows 8.1 64bits and <IP_ADDRESS>.
It might be totally unrelated but, i remember once my game was like frozen because Windows was showing a dialog box in the background asking for me to authorize my game to access to local or public network.
It also happened to me with fullscreen games from steam, where the dialog box is not even visible, and freeze the game till i press [Return] or [Ctrl+Tab] to see the dialog box.

@SuperUserNameMan I didn't had this dialog.
The last time I had such an issue was with MinGW or Git: everytime I launched their executable, there was a huge freeze before they start doing something, and the isse was that it tried to access a network drive that didn't exist for some reason. But Godot is built with MSVC on Windows so it could be unrelated to the compiler.

If the debugger is trying to connect to "localhost" instead of "<IP_ADDRESS>",
maybe your hosts file is broken and the host name "localhost" is not being
resolved locally, you should look into that. What happens if you do "ping
localhost" in the command line?
On 21 July 2016 at 09:42, Marc<EMAIL_ADDRESS>wrote:

@SuperUserNameMan https://github.com/SuperUserNameMan I didn't had this
dialog.
The last time I had such an issue was with MinGW or Git: everytime I
launched their executable, there was a huge freeze before they start doing
something, and the isse was that it tried to access a network drive that
didn't exist for some reason. But Godot is built with MSVC on Windows so it
could be unrelated to the compiler.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/5828#issuecomment-234241729,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGVmPeFy1O_6-2aNKmOR70NKomjrlUSoks5qX2lMgaJpZM4JRM-q
.

I turned off my WiFi then try to ping localhost: no delay. Godot is also fine. But I guess a real test would be to actually have no connection beyond the router, which I cannot easily do...

Also maybe "localhost" is now on your DNS cache, that's why the editor also
works with no problem.. I'm not sure how to clear it, maybe rebooting?
On 21 July 2016 at 17:07, Marc<EMAIL_ADDRESS>wrote:

I turned off my WiFi then try to ping localhost: no delay. Godot is also
fine. But I guess a real test would be to actually have no connection
beyond the router, which I cannot easily do...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/5828#issuecomment-234367953,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGVmPYQWc-f8_YbXl9P_0pvKSdBv_jE5ks5qX9FlgaJpZM4JRM-q
.

The command to clear the DNS cache is: ipconfig /flushdns

And I couldn't reproduce this, BTW.

It seems the only times I get a repro is when the WiFi is available but there is no internet. I'm not sure what I should do if it happens again :s

Do you have access to the router? Try to disconnect the internet cable from it and see what happens. I'll try to do that once I have the opportunity (the people on my house won't be happy if I turn off the internet now...).

@vnen I don't own the router, that's the point :'D

It happened again, and flushing the DNS cache did not solve the problem.
I shot a video of it https://www.youtube.com/watch?v=Vd_DRA0whUs&feature=youtu.be

I cannot reproduce this at all :disappointed: I tried using my phone as an access point without internet, but I had no delay to start debugging.
Have you looked at the console to see if it show anything strange?

The console doesn't shows anything useful. It seems to me that a function call is hanging forever in Godot. Next time I should eventually launch Godot with a debugger and see where it is blocked...

flushing the dns cache won't help, it's just to diagnose it.. try
disconnectibg from the wifi, flush dns, then ping localhost on the
console.. the point is you os is unable to resolve localhost for some
reason when there's no internet.. maybe your hosts file is broken?
On 23 July 2016 at 12:58, Marc<EMAIL_ADDRESS>wrote:

The console doesn't shows anything useful. It seems to me that a function
call is hanging forever in Godot. Next time I should eventually launch
Godot with a debugger and see where it is blocked...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/5828#issuecomment-234725600,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGVmPciQfNyMh4LeoBJo5Lh-6NY1mfvOks5qYjpDgaJpZM4JRM-q
.

Bump? Getting stale here...

I didn' got to repro this recently since my internet is much more stable at the moment.

This is supposed to have been fixed by #8699.


Fixing counter caches not supported when :delta_magnitude is a Proc

I've hit this when trying to fix counter_culture counts:
Fixing counter caches is not supported when :delta_magnitude is a Proc; you may skip this relation with :skip_unsupported => true

Is there no way to tell it to just do it anyway? Do I have iterate over every record and manually set the cache column myself?

Yes. That's what this message says, right?
Or feel free to open a pull request that adds support for this to counter_culture.

Perhaps you could help me with some syntax. I'm not sure how to provide :column_names to the counter_culture_fix_counts method. Looking at the code hasn't helped so far and it's not in the examples in the readme.
In my app, a Section has many Comments. I have this on my Comment model:
counter_culture :section, column_name: proc { |comment| comment.open_status? ? :open_count : :closed_count }

How can I fix the count for both open_count and closed_count? The method looks to accept a hash but I'm not sure what the key and value are meant to be. All my experimentation keeps getting Must provide :column_names option for relation [:section] when :column_name is a Proc; you may skip this relation with :skip_unsupported => true).


Eid 1062

I was going to approve this, with a suggestion to add a tech debt ticket, or something in the backlog, about finding a solution to this issue of using the ConcurrentSkipListSet.
But I can't, as I opened the PR...

We could think about using some kind of (temporary) user session to remember the AuthnRequest ID to validate against, rather than storing a list of all the IDs seen thus far but agreed this could be in a later ticket.

@vixus0 @CharlesIC Have added a test for the unhappy path if you could take a look.

Wait are we not removing the requestId from the list after a successful validation?


Invalid proposal key [Error Code: 1006] on 0.3.0-alpha.14

Problem
As 0.3.0-alpha.15 has issues with getAccountAddress and apparently with @onflow/flow-cadut as well, we had to downgrade to 0.3.0-alpha.14 to get the getAccountAddress working.
Sadly this leads to another error though:
execution error code 1006: [Error Code: 1006] invalid proposal key: public key 0 on account
f8d6e0586b0a20c7 does not have a valid signature: [Error Code: 1009] invalid envelope key:
public key 0 on account f8d6e0586b0a20c7 does not have a valid signature: signature is not valid

Next, we downgraded it to 0.3.0-alpha.13, and finally, all our tests passed.
Have recreated flow.json with flow init:
{
  "networks": {
    "emulator": "<IP_ADDRESS>:3569",
    "mainnet": "access.mainnet.nodes.onflow.org:9000",
    "testnet": "access.devnet.nodes.onflow.org:9000"
  },
  "accounts": {
    "emulator-account": {
      "address": "f8d6e0586b0a20c7",
      "key": "acd1bdd97463e4ef135c06e2f3552a4bdd9111efa481abc4cece4d5286b91e8a"
    }
  }
}

Steps to Reproduce
Config for failing 0.3.0-alpha.14 test
Flow version: v0.39.3
package.json
{
  "name": "test",
  "version": "1.0.0",
  "main": "index.js",
  "author": "",
  "license": "ISC",
  "dependencies": {
    "@babel/core": "^7.19.1",
    "@babel/preset-env": "^7.19.1",
    "@onflow/flow-js-testing": "0.3.0-alpha.14",
    "babel-jest": "^29.0.3",
    "jest": "^29.0.3"
  }
}

babel.config.json
{
  "presets": [
    [
      "@babel/preset-env",
      {
        "targets": {
          "node": "current"
        }
      }
    ]
  ]
}

jest.config.json
{
  "testEnvironment": "node",
  "verbose": true,
  "coveragePathIgnorePatterns": ["/node_modules/"],
  "testTimeout": 50000
}

deploy-contract.test.js
import path from "path"
import {
  init,
  emulator,
  getAccountAddress,
  deployContract,
  executeScript,
  shallResolve,
  shallPass,
} from "@onflow/flow-js-testing"

beforeEach(async () => {
  const basePath = path.resolve(__dirname, "../cadence")

  await init(basePath)
  await emulator.start()
})

test("deploy contract", async () => {
  const to = await getAccountAddress("Alice")
  const name = "Wallet"
  const code = `
        pub contract Wallet{
            pub let balance: UInt
            init(balance: UInt){
              self.balance = balance
            }
        }
    `
  const args = [1337]

  await shallPass(deployContract({to, name, code, args}))

  const [balance] = await shallResolve(
    executeScript({
      code: `
      import Wallet from 0x01
      pub fun main(): UInt{
        return Wallet.balance
      }
    `,
    })
  )
  expect(balance).toBe("1337")
})

afterEach(async () => {
  await emulator.stop()
})

Acceptance Criteria
Tests run without errors.
Context
Upgrading to the latest testing library is currently blocked.

@adbario Does this issue still exist with alpha 16?

@adbario Does this issue still exist with alpha 16?

@justinbarry This seems to be solved now as well, thanks!


Fix Image/Tabel/Section links in asciidoc

Currently all references in Asciidoc for images, tables and sections are just a static text.
We need to make these references interactive, such that on click the forwarding to the position with an according Image/Table/Section will be executed.
We need to define for each image, table referenced in text link names, which will be used for referencing in the text as links.

This fix should be done in the stage of automatic generation of asciidoc from word in https://github.com/admin-shell-io/word2asciidoc

Closed as opened in https://github.com/rwth-iat/word2asciidoc/issues/1


Convert JSON to a table and update readme ...

... or find an API to convert the JSON data as GUI

Issue completed, uses Readme to output the table


Feature request: Apply name filter for files in the stat command

Enhancement's description:
Should provide the statistics only for the files matching the filter.
Expected behavior / output:
./qt-ts-tools stat my_file.ts --filter=*.cpp --verbose
./qt-ts-tools stat my_file.ts --filter=*sub_directory/* --verbose

The summary only counts for the files matching the filter.
Verbose mode also only print files for the matching filter.

Sample translation file
n/a
Sample expected / correct output file
n/a

Brainstorm
What should the filter support (syntax) ?

Asterisk * only (any number of char)
Asterisk + underscore _ (single char)
Full regex

The full regex option is a bit overkill and would considerably increase the size of the executable (by at least 500KB if not 1MB), for a very specific use case.
Asterisk covers most use cases, including the underscore.
Perfect match is always included.
The filter is always case insensitive.

After thinking, not sure about the use case for this.


[lua] Fix find_package warning

Describe the pull request

What does your PR fix? Fixes #13185
Due to find_package(Lua REQUIRED) use FindLua.cmake, modifying find_package(lua REQUIRED) to find_package(Lua REQUIRED) to fix find_package(Lua REQUIRED) warning:

The package name passed to `find_package_handle_standard_args` (Lua) does
  not match the name of the calling package (lua).  This can lead to problems
  in calling code that expects `find_package` result variables (e.g.,
  `_FOUND`) to follow a certain pattern.

/azp run

Thanks for your contribution!


Relayer fee error (RPC tho maybe?)

err: insufficient funds for gas * price + value: address 0x6d91e35d502fb6F654992283e6E4f397fC23DC2c have<PHONE_NUMBER>048922631 want<PHONE_NUMBER>0710925147 (supplied gas 14893604)

same thing happened on Gnosis but then it went away

new error

disregard user errror haha


python app - missing pip shim

There is not pip/pip3 shim reference in python app

The scripts directory is added to your PATH on installation, so pip/pip3 should be available.
https://github.com/lukesampson/scoop/blob/2ea5330a5c3064d3bacb303110d7cd212914e773/bucket/python.json#L42-L44

Yes. I am not sure what happend, I was not able to execute pip3, but I am not able to reproduce it now :-(
Thank you.

currently have python 2.7 installed and there's no pip shim and running python -m pip gives me "there's no module named pip" error, and the Scripts folder has no mention of pip


Initial Work For Acceleration Wizard UI

Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] 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.

Initial work for #748

where are the user workflow ?


Have two sections for Actions and Workflows

I think it could be a good decision to possibly separate the "Community Resources" into an "actions" and "workflows" as some people may share solo-actions while others may share workflows that use a large series of actions.
What are everyone's thoughts on this?

I think it's a good proposal but I don't think we should head in that direction because it doesn't leave a lot of room for projects to scale. Besides, every action has a workflow even if it's single, so people could easily confused here. I'm open to other ways of splitting things up though.


Migrate to MSBuild 15 csproj

@domaindrivendev, can we migrate to MSBuild 15 csproj that works in VS 2017?
Will you accept such pull request?

This is a dup of #335.
Which I'm currently working on


escapeShellArg breaks using paths as template

The following error message is written to file instead of the output.
Error: open /nix/store/syncbcpk5c9s0cl7cq203dhh2wab9kkz-source/stylix/palette.json.mustache: no such file or directory

It seems like running the path through escapeShellArg stops it being listed as an input to the derivation, so presumably the sandbox is hiding it. It definitely exists from my perspective.
The build succeeds when escapeShellArg is removed, however I can see why that might be necessary if the name contains spaces.

Hi danth, can you please try out the latest release?

Seems to be working, thanks!


Node filter for LocationIndex

I would like to have a variant of LocationTree.findClosest where I can pass a node filter instead of an EdgeFilter. Like
public QueryResult findClosest(final double queryLat, final double queryLon, final Predicate<Integer> /* or something equivalent */ nodeFilter);

so that always
nodeFilter.test(findClosest(lat, lon, nodeFilter).getClosestNode()) == true

I tried to emulate this functionality with an EdgeFilter, but to no avail, because edges are undirected.

Yes, the filter should be more generic. Highly related to #729 ... but I'm currently unsure how. Also there is a use case regarding map matching (findNClosest). Maybe a more generic filtering interface and returning a weighting list of QueryResults or something

@michaz could you provide a use case here? Right now nodes don't have a lot of information stored, basically Lat, Lon, Ele (if I am not mistaken).

@boldtrn .. and an additionalNodeField, which in my case it is. The use case is public transit (branch pt), where one may want to find the nearest station.
Right now, I am using special loop edges to mark nodes and filter for them. Possible, but not as such elegant.

On the other hand, I could create several indices: One for stations, one for road network nodes. But that is also not easy, because the LocationIndex implementations are very tightly coupled to the Graph, so it does not seem easy to create a LocationIndex and add some nodes and links to it.

I have millions of inner nodes which should not be part of any index at all and just waste resources there

We could implement a filtering here, also necessary for road routing #322

But that is also not easy, because the LocationIndex implementations are very tightly coupled to the Graph

It stores just node IDs and uses the Graph interface to fetch more information, we could decouple this with another interface

@michaz Thanks, makes sense now. I was just wondering since in GH we usually don't have a lot of information per node.

I was just wondering since in GH we usually don't have a lot of information per node

But we should definitely add something like this, e.g. making it easier to store OSM node IDs would be very helpful.

Another use case was mentioned in the forum: https://discuss.graphhopper.com/t/how-to-build-a-busflagencoder/1680


Clearing statements from native store with inference does not work as expected

Add [] a rdfs:Resource in a transaction. Call .clear(). Statement is still there.

		long originalNumberOfStatements;
		try (RepositoryConnection connection = testRepository.getConnection()) {
			originalNumberOfStatements = connection.getStatements(null, null, null, true).stream().count();
		}

		try (RepositoryConnection connection = testRepository.getConnection()) {
			connection.begin(level);
			connection.add(vf.createBNode(), RDF.TYPE, RDFS.RESOURCE);
			connection.commit();
			connection.begin(level);
			connection.clear();
			connection.commit();
		}

		try (RepositoryConnection connection = testRepository.getConnection()) {
			List<Statement> collect = connection.getStatements(null, null, null, true)
					.stream()
					.collect(Collectors.toList());
			assertEquals(originalNumberOfStatements, collect.size());
		}


Fix #581, Propagate the OSAL compile definitions to CFE build

Describe the contribution
Use the INTERFACE_COMPILE_DEFINITIONS and INTERFACE_INCLUDE_DIRECTORIES properties from the osal target and apply them to the entire CFE build as a directory-scope property.
At this time, the OSAL library build does not use/export these properties so this is effectively a no-op for the CFE build and can be merged with no effect.  However, in a future version (specifically after nasa/osal#312 gets fixed), the OSAL library will export these interface properties and this will become important.
Fixes #581
Testing performed
Built for all three test platforms (ppc-vxworks6.9, i686-rtems4.11, native/x86-64 Linux).  Sanity Check CFE build boots and runs, unit tests run.
Expected behavior changes
No impact to behavior - build is identical because these properties are currently empty/not set in the current OSAL build.
System(s) tested on
Ubuntu 18.04 LTS 64 bit.
Additional context
Also locally tested against a version of OSAL+PSP that includes all the latest proposed changes and confirm that this still works.
Contributor Info - All information REQUIRED for consideration of pull request
Joseph Hickey, Vantage Systems, Inc.

CCB 20200408 - APPROVED

Rebased on master so it can be tested with the bundle


Remove workflow update

Description of your changes
As we change or CI/CD process to use reusable workflows, we need to remove updating of old workflow from renovate-bot
configuration

I have:

[x] Run make reviewable to ensure this PR is ready for review.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Test is broken

I go to privacytests.org/me.html
I run the test
Everything goes seemingly as normal
The test ends, it says "done", then says "not found"

Please help me fix this, i don't know what to do
Browser: chromium

im seeing this too on librewolf


Update DG: Command History Navigation and Undo/Redo

Updated remaining instances of "se-edu" to point to our repository instead.
Updated remaining instances of "address book" -> "task book", "EditCommand" -> "ContactEditCommand", etc in sequence diagrams.
Updated and added to proposed implementation for original undo/redo.
Added proposed implementation for command history navigation.

LGTM, though hopefully we figure out how to resolve the lifeline issue

Unfortunately, I think we cannot do it in PlantUML, unless we use another software!
Reference: https://plantuml.com/sequence-diagram


Added cross provider relationship for oVirt

@abonas @simon3z Please review.

@miq-bot add_label providers/containers
@miq-bot add_label enhancement

UI checks out as well:

LGTM
@simon3z , please also review.

@zeari @matthewd @Fryguy if there's nothing to improve in the query then LGTM :+1:

@Fryguy @blomquisg this patch is ready to be merged.
cc @simon3z


@NonNull not added when refactor classes

Short description
When refactoring a class it doesn't check if the get for a field is never null and then add the @NonNull tag to the appropriate field.
Expected behavior
Add @NonNull to fields that are not allowed to be null
Version information

IDEA Version:
IntelliJ IDEA 2017.2 EAP
Build #IU-172.3198.6, built on June 27, 2017
IntelliJ IDEA EAP User
Expiration date: July 27, 2017
JRE: 1.8.0_152-release-915-b4 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Linux 4.4.0-81-generic

JDK Version:
java version "1.8.0_131"
Java(TM) SE Runtime Environment (build 1.8.0_131-b11)
Java HotSpot(TM) 64-Bit Server VM (build 25.131-b11, mixed mode)

OS Type & Version:
RELEASE=18
CODENAME=sarah
EDITION="Cinnamon 64-bit"
DESCRIPTION="Linux Mint 18 Sarah"
DESKTOP=Gnome
TOOLKIT=GTK
NEW_FEATURES_URL=http://www.linuxmint.com/rel_sarah_cinnamon_whatsnew.php
RELEASE_NOTES_URL=http://www.linuxmint.com/rel_sarah_cinnamon.php
USER_GUIDE_URL=help:linuxmint
GRUB_TITLE=Linux Mint 18 Cinnamon 64-bit

Lombok Plugin Version:
0.14.16

Lombok Dependency Version: What version of lombok.jar are you using?
1.16.16

I didn't know what I was talking about


[Docs]The document of python function example is incorrect

Describe the bug
There was an error in the example of using python function that causes the following exception to be thrown:
2019-11-11 09:48:43 +0000] [ERROR] log.py: Traceback (most recent call last):
[2019-11-11 09:48:43 +0000] [ERROR] log.py:   File "/pulsar/instances/python-instance/python_instance_main.py", line 211, in <module>
[2019-11-11 09:48:43 +0000] [ERROR] log.py: main()
[2019-11-11 09:48:43 +0000] [ERROR] log.py:   File "/pulsar/instances/python-[instance/python_instance_main.py", line 192, in main
[2019-11-11 09:48:43 +0000] [ERROR] log.py: pyinstance.run()
[2019-11-11 09:48:43 +0000] [ERROR] log.py:   File "/pulsar/instances/python-instance/python_instance.py", line 186, in run
[2019-11-11 09:48:43 +0000] [ERROR] log.py: self.input_serdes[topic] = serde_kclass()
[2019-11-11 09:48:43 +0000] [ERROR] log.py: TypeError
[2019-11-11 09:48:43 +0000] [ERROR] log.py: :
[2019-11-11 09:48:43 +0000] [ERROR] log.py: __init__() takes exactly 2 arguments (1 given)

Parameters should not appear in the initialization function of class TweetSerDe, messages should come from the topic and should not be assigned during initialization. the document example is wrong, and the correct example should be:
class Tweet(object):
    def __init__(self, username, tweet_content):
        self.username = username
        self.tweet_content = tweet_content

from pulsar import SerDe

class TweetSerDe(SerDe):
    def __init__(self):
        pass

    def serialize(self, input):
        return bytes("{0}|{1}".format(input.username, input.tweet_content))

    def deserialize(self, input_bytes):
        tweet_components = str(input_bytes).split('|')
        return Tweet(tweet_components[0], tweet_componentsp[1])

Reference: https://github.com/apache/pulsar/blob/master/pulsar-functions/python-examples/custom_object_function.py
Expected behavior
Fix document
Additional context
Use the correct example on the document

Also,
    def __init__(self):
        pass

is redundant and can be omitted entirely.


Remove redundant apps constants files; move all settings to AppConfigs

Replaces redundant AppConfig classes with Django AppConfig classes. Removes the constant.py files from the addons.

The failing tests are unrelated to this PR and are failing on develop, so merging this in.


Does not reply on users other than me

I followed the sample and got it working. Whenever I message the bot it echoes it. However when I asked other people to message the fb page there is no reply. Is your sample supposed to work with other users other than the one who created the fb page? If yes then I did something wrong which I'm thinking is because the fb page access token I put in Heroku config vars is only applicable for my user, is that correct?

I think they need to be added as admin to the app/page that the bot is messaging from as the app/page is not public.
If you log the Heroku output it will show the authorization error (does with me anyway)
type heroku logs --tailin the terminal once you have pushed the project to server and you should get the live logging. Then message the bot from your account, see what it says (I have loads of console logs set up to see what's happening). Then message from another account - the error handler of the standard webhook post should show the error.
Having said all this - I couldn't get the system working with a dummy token so skipped that for the moment to just get up and running.
If someone could explain the purpose of that step I would appreciate it!


Autocompletion for mapped type with inferred keys and values all the same.

TypeScript Version:  nightly 2.3.0-dev.20170324
Code
declare function f<T extends { [P in keyof T]: { x: number } }>(t: T): void;

f({
    arbitraryKey: {
        /*Here*/
    }
});

Expected behavior:
Autocomplete gives x.
Actual behavior:
Don't get useful autocomplete.

This is also an issue over at typestyle/typestyle#144
I mocked out another example where this occurs on the typescript playground using mapped types. Try typing on line 17 and see the autocomplete has no idea what is happening. I've noticed this doesn't happen if you type the variable as the return instead of relying on the function return.
Example


add okhttp and okhttp2 modules to the bom

Description
I realized after the latest release that neither okhttp nor okhtttp2 were in the bom.
Motivation and Context
Developers will expect the bom to be complete.
Types of changes

[X ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.

Good catch! :+1:

Released as 1.12.1. Thanks for the contribution! :tada:


Feature/gift entry  ken populate lookup sibling fields

Critical Changes
Changes
Issues Closed
Community Ideas Delivered
New Metadata
Deleted Metadata

**lurch: attach W-039031

Tracking W-039031

@kenlewis-sfdo I think we should close this PR, or else re-base it to the feature/gift-entry-fast-follow branch (and rename accordingly). What do you think?

Closing due to replacement PR created, parented to the fast follow branch here: https://github.com/SalesforceFoundation/NPSP/pull/5151/files