When core

Description
Added org.opensearch.dataprepper.expression to public context
Added DI support for DataPrepperExpressionParser
Check List

[x] New functionality includes testing.
[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed with a real name per the DCO

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.

Codecov Report

Merging #1157 (f1aa1ec) into main (5ecec92) will decrease coverage by 0.07%.
The diff coverage is 77.77%.

@@             Coverage Diff              @@
##               main    #1157      +/-   ##
============================================
- Coverage     91.89%   91.82%   -0.08%
- Complexity      703      704       +1
============================================
  Files            87       87
  Lines          2037     2042       +5
  Branches        171      170       -1
============================================
+ Hits           1872     1875       +3
- Misses          122      124       +2
  Partials         43       43

Impacted Files
Coverage Δ

...zon/dataprepper/plugin/PluginArgumentsContext.java
94.73% <71.42%> (-5.27%)
:arrow_down:

...in/java/com/amazon/dataprepper/ContextManager.java
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


[HUDI-676] Address issues towards removing use of WIP Disclaimer

What is the purpose of the pull request
This pull request addresses issues towards removing use of WIP Disclaimer
Brief change log

Address issues towards removing use of WIP Disclaimer

Verify this pull request
This pull request is a trivial rework / code cleanup without any test coverage.
Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@vinothchandar @smarthi Have changed the disclaimer to the standard. Please help to review. More details please see here: https://incubator.apache.org/guides/releasemanagement.html#choice_of_disclaimers

Codecov Report

Merging #1386 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1386      +/-   ##
============================================
- Coverage     67.19%   67.19%   -0.01%
  Complexity      223      223
============================================
  Files           335      335
  Lines         16279    16279
  Branches       1661     1661
============================================
- Hits          10939    10938       -1
  Misses         4604     4604
- Partials        736      737       +1

Impacted Files
Coverage Δ
Complexity Δ

...n/java/org/apache/hudi/common/model/HoodieKey.java
88.88% <0%> (-5.56%)
0% <0%> (ø)

Continue to review full report at Codecov.

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


User authenticators refactoring

Currently user auth view should be refactored so adding new ways of authentication will be easy

Working on it in https://github.com/DarkKnightCZ/LXC-Web-Panel/tree/58-authenticators

closed merging #63


Round Robin Geocoders with DB Cache  loading

RRCacheGeocoder: Round Robin Geocoders  with DB Cache loading from the database.rrcache query.
database.rrcache: it must to return the follow fields -> regs,latitude,longitude,address
Configuration Example:
    <entry key='geocoder.enable'>true</entry>
    <entry key='geocoder.cacheSize'>3000</entry>
    <entry key='geocoder.rrcache'>2</entry>
    <entry key='geocoder.type'>google</entry>
    <entry key='geocoder.limit'>2500</entry>
    <entry key='geocoder.type.1'>nominatim</entry>
    <entry key='geocoder.url.1'>https://locationiq.org/v1/reverse.php</entry>
    <entry key='geocoder.limit.1'>10000</entry>
    <entry key='geocoder.key.1'>xxxxx_your_key_xxxxx</entry>
    <entry key='geocoder.type.2'>opencage</entry>
    <entry key='geocoder.url.2'>http://api.opencagedata.com/geocode/v1</entry>
    <entry key='geocoder.limit.2'>2500</entry>
    <entry key='geocoder.key.2'>xxxxx_your_key_xxxxx</entry>

    <entry key='database.rrcache'>
	SELECT
		COUNT(*) as regs,
		CAST(p.latitude AS NUMERIC(8, 5)) AS latitude,
		CAST(p.longitude AS NUMERIC(8, 5)) AS longitude,
		p.address
	FROM
		positions p
	INNER JOIN
	(
		SELECT
			COUNT(*) AS ad_regs,
			address
		FROM
			positions
		WHERE
			address IS NOT NULL
		GROUP BY
			address
		HAVING
			COUNT(*) > 100
		ORDER BY
			ad_regs DESC
		LIMIT 1000
	) as ad
	ON
		p.address = ad.address
	GROUP BY
		latitude,longitude,p.address
	HAVING
		COUNT(*) > 5
	ORDER BY
		regs desc
	LIMIT 2000
    </entry>

Thank for pull request, but I don't think we can include it in Traccar. Three main reasons:

Cache SQL query is not universal, so it means it won't work on some databases
Cache loading will be very very slow if positions table is big
I am not sure that it's very ethical to encourage people to abuse free geocoding quota

I feel like I should explain last point a bit more. Most services that provide free quota do it for testing purposes, not for production use. We already have problems with Google, Nominatim and Gisgraphy where they seem to completely block Traccar requests because people abused their free quota in the past. It means that now I can't even use free services to test Traccar code.

I'm agree with your reasons.
Most important the last reason. Because the first and second could be fixed.
What about a Round Robin email sender?
I'll get my email account with servermx.com. But, I need to improve the limits on sending email and I was thinking get 2-4 email account and implement a  Round Robin email sender.
Regards,

Email services are really cheap comparing to reverse geocoding, so I don't see much point. For example, you can use Amazon SES for $0.10 per 1000 emails. Or even configure your own SMTP server.

What I thought that could be useful under this would be a way so that if the location only moves by a couple of meters it does not geocode and uses just last geocode. When stopped trackers can wonder a little and it then gives different addresses from next door etc, this would be a geo lookup each time unless there is an internal location table which prevents this based on distance.

@tsmgeek, such option is already available. Not sure how it's related to this pull request though.

I misunderstood the cache part on this PR.
The round Robin code though could be useful if altered to be used as backup, ie primary coder goes down or fails it will switch to backup for a while.

That version has reload issue.
I could provide the fixed version.
By the way, my version use locations with 5 decimal digits
On Oct 20, 2017 5:42 PM, "tsmgeek"<EMAIL_ADDRESS>wrote:

I misunderstood the cache part on this PR.
The round Robin code though could be useful if altered to be used as
backup, ie primary coder goes down or fails it will switch to backup for a
while.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tananaev/traccar/pull/3401#issuecomment-338346093,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdKb95SqCMbCn6sPdvMPGmr6zxZrVEN6ks5suS_fgaJpZM4OnxYU
.


Possibility to run an event when timer runs out?

(Required) What is your Quests version / build number (do not say "latest")?
3.2.5
(Required) Please describe the problem and how to replicate it. Any console errors?
When you create a timed event, it fails the quest when the timer runs out like expected. But there doesn't seem to be a way to run an event when that happens (like you can have death event and logout event). Have I overlooked something or is this really the case?

This is not possible because you'd be running an event for a Quest which is no longer being taken.

Well you can run a death or a logout event even if they have fail-quest set
to true.
That way when someone for example dies during the quest, they fail the
quest and also a bunch of commands can be run through the death event to
"reset" everything back to how it was at the start.
Now the timer timeout does fail the quest automatically but there is no way
to run an event
sunnuntai 25. helmikuuta 2018 FlyingPikachu<EMAIL_ADDRESS>kirjoitti:

This is not possible because you'd be running an event for a Quest which
is no longer being taken.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/FlyingPikachu/Quests/issues/313#issuecomment-368265931,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AiBOTFjHiRekF77a988mRESBT5IfoMVJks5tYI-pgaJpZM4SR7Ec
.


Add recorded tests for Storage

Switch from MSTest to NUnit 3
Azure.Core testing framework tweaks (recording related)
Add Storage recorded tests
Replay test execution is ~20s across all Blobs, Files, and Queues tests

Most of the changes are tweaking the Storage tests to be less stateful.  They used a lot of random values and DateTimeOffset.Now which needed to be isolated to individual test methods so replaying works correctly.  The static TestHelper class, in particular, had to be turned into base classes that can access the current test Recording member variable.
You can set the AZURE_TEST_MODE  environment variable to Live for running tests against the live service, Record for updating recordings, Playback to run the recordings, or leave it empty to default to Playback.  Additionally, all of the Storage test classes allow you to pass the RecordedTestMode enum into their ctor if you want to override the environment variable (only intended for use while developing).
/cc @kfarmer-msft and @seanmcc-msft

/azp run net - storage - tests

/azp run net - storage - tests

/azp run net - storage - tests

Changes to Azure.Core testing framework look good!

/azp run net - storage - tests

/azp run net - storage - tests


ViewPager Null Skip

🐛 Bug Report
In ViewPager, if a section returns null, it still gets a page.
I'm hoping to do some conditional pages (in a wizard), and when I do a conditional section like
{ false &&
<Layout>

</Layout>
}

It still renders the entire child as an empty spot.   Ideally, this empty child would be skipped in a pager, thus giving me the ability to optionally show certain steps in onboarding in a pager as long as the index size doesn't shrink.

To Reproduce
Return null for any child in viewpager
Expected behavior
Null children should not be rendered, just like they aren't rendered in react.
Link to runnable example or repository (highly encouraged)
https://snack.expo.io/6NRfntmPi
Scroll right on the example.   The empty page is the issue.

Hi 👋
This is most likely a feature request rather than a but report, since currently you may filter optional children by your own
However, I agree this should be handled out of the box. Thanks for pointing on this 👍


Performance issues in /MRC (by P3)

Hello! I've found a performance issue in /MRC: batch() should be called before map(), which could make your program more efficient. Here is the tensorflow document to support it.
Detailed description is listed below:

/BiDAF/util.py: dataset.batch(config.batch_size)(here) should be called before .map(parser, num_parallel_calls=num_threads)(here).
/MRC/BiDAF/util.py: .batch(config.batch_size)(here) should be called before .map(parser, num_parallel_calls=num_threads)(here).
/MRC/Hybrid/util.py: dataset.batch(config.batch_size)(here) should be called before .map(parser, num_parallel_calls=num_threads)(here).
/MRC/Hybrid/util.py: .batch(config.batch_size)(here) should be called before .map(parser, num_parallel_calls=num_threads)(here).
/MRC/QANet/util.py: dataset.batch(config.batch_size)(here) should be called before .map(parser, num_parallel_calls=num_threads)(here).
/MRC/QANet/util.py: .batch(config.batch_size)(here) should be called before .map(parser, num_parallel_calls=num_threads)(here).
/MRC/RNet/util.py: dataset.batch(config.batch_size)(here) should be called before .map(parser, num_parallel_calls=num_threads)(here).
/MRC/RNet/util.py: .batch(config.batch_size)(here) should be called before .map(parser, num_parallel_calls=num_threads)(here).

Besides, you need to check the function called in map()(e.g., parser called in .map(parser, num_parallel_calls=num_threads)) whether to be affected or not to make the changed code work properly. For example, if parser needs data with shape (x, y, z) as its input before fix, it would require data with shape (batch_size, x, y, z).
Looking forward to your reply. Btw, I am very glad to create a PR to fix it if you are too busy.

Hello, I'm looking forward to your reply~


Can I omit the etag in updateCalendarObject?

I'm implementing a one-way sync, so I don't care what the current state of the remote object is, I just want to overwrite it. Omitting the etag field seems to work, but it fails typechecking. Could that be changed in the function signature or is there some other issue I'm overlooking?

you can omit the etag, I will make the type optional


Git Sync improvement

请描述该需求尝试解决的问题
Hello,
I'd like to suggest to improve git sync functionality in order to make it possible for scenarios where there are dozens (or even hundreds) of spiders. Currently the functionality requires that each spiders has its own repository. In such scenarios mentioned before, I would have too much repositories to make it doable.
Currently I'm using as workaround bash script + crontab job to pull data from one single repository on github, where each spider has its own folder, same structure that is found on /app/spiders path.
Best,
Eliton

Hello.
It is also doable. No idea how complex is following this or that way to make it work, actually. The way I suggested was the way "I did" with bash scripting.
Thanks,
Eliton

Perhaps I didn't convey my idea clearly. You can follow this issue and a proposed model/process on file management in Crawlab will be implemented later this or the month after.

There is also a common use case in Scrapy where multiple spiders are hosted in a single folder. This allows spiders to reuse common pipelines and middlewares in a shared code base:
├── my_scraper
│   ├── items.py
│   ├── middlewares.py
│   ├── pipelines
│   │   ├── pipeline1.py
│   │   ├── pipeline2.py
│   │   └── ....
│   ├── settings.py
│   └── spiders
│       ├── spider1.py
│       ├── spider2.py
│       ├── ...

If we have a single codebase with 100 spiders in them, at the moment we need to clone the same repository 100 times, and if there is an update to the repository, we need to pull changes 100x.
It would be really nice if we could point spiders to the same code base, but this would mean to decouple the concept of files / git repos from spiders.


Fail to compile cmdstan 2.26.0

I compiled and have used the release candidate. Now when I tried to compile the latest release it bailed on me (latest OS X):
* Building CmdStan binaries...
cp bin/mac-stanc bin/stanc
clang++ -std=c++1y -Wno-unknown-warning-option -Wno-tautological-compare -Wno-sign-compare -D_REENTRANT -Wno-ignored-attributes      -I stan/lib/stan_math/lib/tbb_2019_U8/include   -O3 -I src -I stan/src -I lib/rapidjson_1.1.0/ -I lib/CLI11-1.9.1/ -I stan/lib/stan_math/ -I stan/lib/stan_math/lib/eigen_3.3.9 -I stan/lib/stan_math/lib/boost_1.72.0 -I stan/lib/stan_math/lib/sundials_5.6.1/include    -DBOOST_DISABLE_ASSERTS         -c -fvisibility=hidden -o bin/cmdstan/stansummary.o src/cmdstan/stansummary.cpp
clang++ -std=c++1y -Wno-unknown-warning-option -Wno-tautological-compare -Wno-sign-compare -D_REENTRANT -Wno-ignored-attributes      -I stan/lib/stan_math/lib/tbb_2019_U8/include   -O3 -I src -I stan/src -I lib/rapidjson_1.1.0/ -I lib/CLI11-1.9.1/ -I stan/lib/stan_math/ -I stan/lib/stan_math/lib/eigen_3.3.9 -I stan/lib/stan_math/lib/boost_1.72.0 -I stan/lib/stan_math/lib/sundials_5.6.1/include    -DBOOST_DISABLE_ASSERTS         -c -fvisibility=hidden -o bin/cmdstan/diagnose.o src/cmdstan/diagnose.cpp
clang++ -std=c++1y -Wno-unknown-warning-option -Wno-tautological-compare -Wno-sign-compare -D_REENTRANT -Wno-ignored-attributes      -I stan/lib/stan_math/lib/tbb_2019_U8/include   -O3 -I src -I stan/src -I lib/rapidjson_1.1.0/ -I lib/CLI11-1.9.1/ -I stan/lib/stan_math/ -I stan/lib/stan_math/lib/eigen_3.3.9 -I stan/lib/stan_math/lib/boost_1.72.0 -I stan/lib/stan_math/lib/sundials_5.6.1/include    -DBOOST_DISABLE_ASSERTS         -c -fvisibility=hidden -o bin/cmdstan/print.o src/cmdstan/print.cpp
chmod +x bin/stanc
clang++ -pipe   -Wno-unknown-warning-option -Wno-tautological-compare -Wno-sign-compare -D_REENTRANT  -O3 -I stan/lib/stan_math/lib/sundials_5.6.1/include -DNO_FPRINTF_OUTPUT     -O3  -c -x c -include stan/lib/stan_math/lib/sundials_5.6.1/include/stan_sundials_printf_override.hpp stan/lib/stan_math/lib/sundials_5.6.1/src/nvector/serial/nvector_serial.c -o stan/lib/stan_math/lib/sundials_5.6.1/src/nvector/serial/nvector_serial.o
In file included from src/cmdstan/diagnose.cpp:1:
In file included from src/cmdstan/stansummary_helper.hpp:4:
In file included from stan/src/stan/mcmc/chains.hpp:4:
In file included from In file included from stan/src/stan/io/stan_csv_reader.hpp:src/cmdstan/stansummary.cpp4:1:
In file included from :
stan/lib/stan_math/lib/boost_1.72.0/boost/algorithm/string.hppIn file included from :src/cmdstan/stansummary_helper.hpp:19:
4:
In file included from In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/algorithm/string/trim.hppstan/src/stan/mcmc/chains.hpp::164:
:
In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/range/begin.hpp:In file included from 24:
In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/range/iterator.hppstan/src/stan/io/stan_csv_reader.hpp::420:
In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/range/mutable_iterator.hpp:
In file included from :21stan/lib/stan_math/lib/boost_1.72.0/boost/algorithm/string.hpp::
19In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/range/detail/extract_optional_type.hpp:
In file included from :stan/lib/stan_math/lib/boost_1.72.0/boost/algorithm/string/trim.hpp19:
In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/mpl/has_xxx.hpp:16:
In file included from :20:
In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/mpl/aux_/na_spec.hpp:28:
In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/mpl/aux_/preprocessor/def_params_tail.hpp:stan/lib/stan_math/lib/boost_1.72.0/boost/range/begin.hpp:24:
In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/range/iterator.hpp66:
:20In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/preprocessor/arithmetic/add.hpp:
In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/range/mutable_iterator.hpp::21:
In file included from 21stan/lib/stan_math/lib/boost_1.72.0/boost/preprocessor/tuple/elem.hpp:22:
:
stan/lib/stan_math/lib/boost_1.72.0/boost/preprocessor/tuple/rem.hpp:20:11:In file included from  stan/lib/stan_math/lib/boost_1.72.0/boost/range/detail/extract_optional_type.hppfatal error: 'boost/preprocessor/tuple/detail/is_single_return.hpp' file not found
:19:
In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/mpl/has_xxx.hpp:20:
In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/mpl/aux_/na_spec.hpp:28:
In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/mpl/aux_/preprocessor/def_params_tail.hpp:66:
In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/preprocessor/arithmetic/add.hpp:21:
In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/preprocessor/tuple/elem.hpp:22:
stan/lib/stan_math/lib/boost_1.72.0/boost/preprocessor/tuple/rem.hpp:20:11: fatal error: 'boost/preprocessor/tuple/detail/is_single_return.hpp' file not found
# include <boost/preprocessor/tuple/detail/is_single_return.hpp>
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
# include <boost/preprocessor/tuple/detail/is_single_return.hpp>
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from src/cmdstan/print.cpp:2:
In file included from src/cmdstan/stansummary_helper.hpp:4:
In file included from stan/src/stan/mcmc/chains.hpp:4:
In file included from stan/src/stan/io/stan_csv_reader.hpp:4:
In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/algorithm/string.hpp:19:
In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/algorithm/string/trim.hpp:16:
In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/range/begin.hpp:24:
In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/range/iterator.hpp:20:
In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/range/mutable_iterator.hpp:21:
In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/range/detail/extract_optional_type.hpp:19:
In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/mpl/has_xxx.hpp:20:
In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/mpl/aux_/na_spec.hpp:28:
In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/mpl/aux_/preprocessor/def_params_tail.hpp:66:
In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/preprocessor/arithmetic/add.hpp:21:
In file included from stan/lib/stan_math/lib/boost_1.72.0/boost/preprocessor/tuple/elem.hpp:22:
stan/lib/stan_math/lib/boost_1.72.0/boost/preprocessor/tuple/rem.hpp:20:11: fatal error: 'boost/preprocessor/tuple/detail/is_single_return.hpp' file not found
# include <boost/preprocessor/tuple/detail/is_single_return.hpp>
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

I might add that I used cmdstanr.

I might add that I used cmdstanr.

@torkar Thanks! I was just about to start checking whether the built tarball is fine.
@serban-nicusor-toptal it seems that the release tarball is missing some files and we need to rebuild it
A few things I noticed:

the tarball contains a folder cmdstan and all the contents in it
the seems to be some additional files in bin/ like mac-stanc:Zone.Identifier
there are missing files: stan/lib/stan_math/lib/boost_1.72.0 contains 21065 files, while the rc contains 22588 files in the same folder

Do you have time to look at this?

@torkar Thanks! I was just about to start checking whether the built tarball is fine.
@serban-nicusor-toptal it seems that the release tarball is missing some files and we need to rebuild it
A few things I noticed:

the tarball contains a folder cmdstan and all the contents in it
the seems to be some additional files in bin/ like mac-stanc:Zone.Identifier
there are missing files: stan/lib/stan_math/lib/boost_1.72.0 contains 21065 files, while the rc contains 22588 files in the same folder

Do you have time to look at this?

Hey @rok-cesnovar I'm on it.
It's made out of the 2.26.0 release, not sure why it has less files and the strange identifier ones ... must be tar :)

Hey @rok-cesnovar I'm on it.
It's made out of the 2.26.0 release, not sure why it has less files and the strange identifier ones ... must be tar :)

Hey @torkar can you try again with the tarball from release I've re-created it and double-checked to make sure it's all good.

Hey @torkar can you try again with the tarball from release I've re-created it and double-checked to make sure it's all good.

The boost subfolder still does not contain enough files.
And it unzips to a folder cmdstan-2.26.0 that has a folder cmdstan-2.26.0 but it should unzip directly in the first cmdstan-2.26.0. No need for the second folder.

The boost subfolder still does not contain enough files.
And it unzips to a folder cmdstan-2.26.0 that has a folder cmdstan-2.26.0 but it should unzip directly in the first cmdstan-2.26.0. No need for the second folder.

I'm extracting with tar -xvzf cmdstan-2.26.0.tar.gz and could only see cmdstan-2.26.0 then the root, what are you using to extract the tarball ?
I'm not sure to be honest why there are less files ? Maybe because how git-archive-all does it ? I've cloned a fresh cmdstan 2.26.0 release and installed all submodules.

I'm extracting with tar -xvzf cmdstan-2.26.0.tar.gz and could only see cmdstan-2.26.0 then the root, what are you using to extract the tarball ?
I'm not sure to be honest why there are less files ? Maybe because how git-archive-all does it ? I've cloned a fresh cmdstan 2.26.0 release and installed all submodules.

Weird, I've checked to what commit hash /stan and /stan/lib/stan_math point and they both point to the correct commit hash corresponding to the today's release.
Math shows 06b495e77f and the commit hash for v4.0.0 is 06b495e77f6d6518f85376ea1d8ec25e17fb6d41.
Stan shows 3ef672314 and the commit hash for v2.6.0 is 3ef6723145a75b2122b5f6290d9f379d518dec95.
Inside cmdstan after a fresh clone of 2.26.0, checking out submodules I see 20442 inside cmdstan/stan/lib/stan_math/lib/boost_1.72.0.
Inside math after a fresh clone of v4.0.0 I see 20442 files inside `lib/boost_1.72.0

Weird, I've checked to what commit hash /stan and /stan/lib/stan_math point and they both point to the correct commit hash corresponding to the today's release.
Math shows 06b495e77f and the commit hash for v4.0.0 is 06b495e77f6d6518f85376ea1d8ec25e17fb6d41.
Stan shows 3ef672314 and the commit hash for v2.6.0 is 3ef6723145a75b2122b5f6290d9f379d518dec95.
Inside cmdstan after a fresh clone of 2.26.0, checking out submodules I see 20442 inside cmdstan/stan/lib/stan_math/lib/boost_1.72.0.
Inside math after a fresh clone of v4.0.0 I see 20442 files inside `lib/boost_1.72.0

Set all submodules to master and cmdstan to master and re-ran your commands. It all looks good on my end, not sure what is the issue for you locally. Uploaded the tarball and I think we should be good now. Will check again but I think its fine now.

Set all submodules to master and cmdstan to master and re-ran your commands. It all looks good on my end, not sure what is the issue for you locally. Uploaded the tarball and I think we should be good now. Will check again but I think its fine now.

Cmdstanr tests are passing now (https://github.com/stan-dev/cmdstanr/pull/437/checks?check_run_id=1771401111) so we should be good. Closing this.
Thanks again @torkar!

Cmdstanr tests are passing now (https://github.com/stan-dev/cmdstanr/pull/437/checks?check_run_id=1771401111) so we should be good. Closing this.
Thanks again @torkar!


Determine data to include in Census Block and Tract Dimension

Include selected population, demographic and socioeconomic data.
Does it make sense to include selected block group data in each block record? Typically in data warehouses you will denormalize data to make the queries more efficient.

Done by Steve Ediger. Thanks!


Commits to monitored repo do not show

We are doing an audit and noticed that one repo is missing all commits in Jira.   This repo is configured to be monitored.   I did a full sync yesterday (successfully) and still nothing.
A few examples of what is not showing on the associated linked JIRA ticket.
https://github.com/Retail-Success/Luci.Android/pull/191
https://github.com/Retail-Success/Luci.Android/pull/243
https://github.com/Retail-Success/Luci.Android/pull/295
https://github.com/Retail-Success/Luci.Android/pull/406
https://github.com/Retail-Success/Luci.Android/pull/405
We're wondering if it's due to rebasing and if this issue can be fixed?
Thank you

Hey @celinakRS. I'm definitely seeing logs coming through for the Luci.Android repo which confirms that it is configured correctly to trigger webhook events. However, for push events I'm seeing the message Halting further execution for push since no commits were found for the payload which is logged when no issue keys are found in a commit. Can you confirm that you're adding issues keys to your commits (for example a jira issue with key TEST-80 your commit message would need need to have TEST-80 somewhere in the commit message e.g. 'made a change for TEST-80 to fix bug') and provide an example?

Duh, you can close this.  Developer confirmed they were in the PR comment and not in the commit message.


Create a middleware to populate rendering variables

What needs to be done?
Most routes, at least for website, need to retrieve rendering variables from settingsService.
We should make a middleware that does that.
Details that might help to better understand the problem
The critical part is to handle exceptions that occur in this middleware, e.g. if db is not reachable or there's any other issue with settingsService.

See also publicController.getError, publicController.getStar introduced in #52.


[BUG]Unable to set environment variable: MODEL_LIST_CONFIG_JSON

I followed the instructions above to set the environment variables to no avail, I cannot find the file for the environment variables I set from the startup command. After startup, it will still be read from the network.
Here's how I set it up:
npx flowise start
--MODEL_LIST_CONFIG_JSON=C:\Users\msrobot\AppData\Roaming\npm\node_modules\flowise\node_modules\flowise-components\models.json

it has to be an online path like https://s3-your-json-file.json

How can I use a local file?

@musuiot made a PR to allow user specify local file path


Try to restore CI

Just a quick attempt at getting green CI, after short conversation with @bf4 here: https://github.com/zipmark/rspec_api_documentation/pull/458#issuecomment-582043015
Cleanup Gemfile stuff

Untrack Gemfile.lock
Move inch gem from Gemfile to development dependency in gemspec

Details:
https://yehudakatz.com/2010/12/16/clarifying-the-roles-of-the-gemspec-and-gemfile/
Freeze development dependencies
Now Gemfile.lock is gone, the only thing governing which version of a gem to use in CI is the gemspec. First step was to lock the version of enough gems enough that bundle exec rake passes.
Next steps
Update dependencies
From here, anyone could look a the dependencies one by one and do the following work:

confirm whether that dependency is needed or not
if it is, try updating it
fix any code that breaks because of the update

For indirect dependencies, once their version constraint has been cleared, they should be removed from the gemspec file.
Depending on the versions of Ruby rspec_api_documentation wants to support, there might be more work (some gems drop support for older versions of Ruby, which means that depending on the local version of Ruby, one or another version of a gem could be needed).
Test in newer versions of Ruby
Then someone might want to introduce newer versions of Ruby on CI (in .travis.yml).

Yass! 🎉

@oestrich let's merge this!

@bf4 @davidstosik merging this.. sorry for the long delay.  Open to thoughts around the future of project maintenance, see: #478


Adding vendor Deviations

"This code is a Contribution to the OpenConfig Feature Profiles project ("Work") made under the Google Software Grant and Corporate Contributor License Agreement ("CLA") and governed by the Apache License 2.0. No other rights or licenses in or to any of Nokia's intellectual property are granted for any other purpose. This code is provided on an "as is" basis without any warranties of any kind."

Pull Request Functional Test Report for #2576 / 4854859dbc2303a022ba1fc25758a065239f8191
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

RT-5.8: IPv6 link localRT-5.8: IPv6 Link Local

Cisco 8000E

RT-5.8: IPv6 link localRT-5.8: IPv6 Link Local

Cisco XRd

RT-5.8: IPv6 link localRT-5.8: IPv6 Link Local

Juniper cPTX

RT-5.8: IPv6 link localRT-5.8: IPv6 Link Local

Juniper ncPTX

RT-5.8: IPv6 link localRT-5.8: IPv6 Link Local

Nokia SR Linux

RT-5.8: IPv6 link localRT-5.8: IPv6 Link Local

Openconfig Lemming

RT-5.8: IPv6 link localRT-5.8: IPv6 Link Local

Hardware Devices

Device
Test
Test Documentation
Raw Log

Arista 7808

RT-5.8: IPv6 link localRT-5.8: IPv6 Link Local

Cisco 8808

RT-5.8: IPv6 link localRT-5.8: IPv6 Link Local

Juniper PTX10008

RT-5.8: IPv6 link localRT-5.8: IPv6 Link Local

Nokia 7250 IXR-10e

RT-5.8: IPv6 link localRT-5.8: IPv6 Link Local

Help

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1668

Relevant Lines:
3199

💛  - Coveralls


Add unique_id to mqtt camera

Description:
Add unique_id to mqtt camera
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#6262
Checklist:

[X] The code change is tested and works locally.
[X] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[X] Documentation added/updated in home-assistant.github.io

Is the "unique id" supposed to be specified by the user? Shouldn't it always be generated automatically?

@abmantis The same way of specifying a unique_id exists for MQTT switch or MQTT light. unique_id can be provided by the device via MQTT discovery or via configuration (by the user).

@bieniu  Right, I've missed those.


Add quotes to table string where missing

Fixes syntax errors when creating tables/views or performing ALTER TABLE statements in projects with a name that requires quoting (e.g. ends in a number).

LGTM!


Bug in new Resource Manager

Description of bug
There might be a small glitch in the implementation of the new resource manager. If I click on "Manage > Site Assets", I get exactly the list of folders and files that I expect. If I then click on "Manage > Global Assets", I get exactly the same list of folders and files (which is not correct).
Selecting "Host Root" from the dropdown gives me the expected list of folders and files. So it seems as if when I click on "Manage > Global Assets" that the Resource Manager is not been told to look at the Global Assets folder (until I directly prod it to do so). Has anyone else run into this problem? (DNN v9.8.1 with Telerik removed)
Steps to reproduce

Create test DNN v9.8.1 website
Remove Telerik components and install new Resource Manager according to instructions in the notes for DNN v9.8.0 in the Github Releases page.
Click on Manage > Site Assets (you get a list of files and folders)
Click on Manage > Global Assets (you get exactly the same list of files and folders which is not right)
Selecting "Host Root" from the dropdown refreshes the list of files and folders to the correct view

Current behavior
Manage > Global Assets not refreshing as expected. Requires prodding as described above.
Expected behavior
Manage > Global Assets should show correct list of files and folders after invocation of Manage > Site Assets
Screenshots
None
Error information
None
Additional context
Provide any additional context that may be helpful in understanding and/or resolving the bug.
Affected version

[ ] 10.00.00 alpha build
[X] 09.08.01 latest supported release

Affected browser

[x] Chrome
[x] Firefox
[ ] Safari
[ ] Internet Explorer 11
[ ] Microsoft Edge (Classic)
[x] Microsoft Edge Chromium

I can confirm this behavior on a clean install of DNN 9.8.1, however, the following resolved this issue for me.

Go to Manage > Global Assets.
Enter into Edit Mode.
Go to Settings for the Resource Manager module, and select the "ResourceManager Settings" tab.
With the "Home Folder" property still set to "Host Root", click on the Update button.
Global Assets now defaults to the Host Root folder.

I can confirm this behavior on a clean install of DNN 9.8.1, however, the following resolved this issue for me.

Go to Manage > Global Assets.
Enter into Edit Mode.
Go to Settings for the Resource Manager module, and select the "ResourceManager Settings" tab.
With the "Home Folder" property still set to "Host Root", click on the Update button.
Global Assets now defaults to the Host Root folder.

The new update persists after clearing cache and restarting the application.

The workaround doesn't seem to work for me. If I apply your solution and then go to Manage > Site Assets and then back to Manage > Global Assets, I am still pointing at the Site Assets. Maybe I'm missing something. It seems that going to Manage > Site Assets messes up what Manage > Global Assets points to.

@skarpik I received an email of your comment on Saturday, but when I came here to reply the comment was not here.  Did the workaround work for your?  I can't remember, but I may have had to cleared the DNN cache between step 4 and 5.

@skarpik I received an email of your comment on Saturday, but when I came here to reply the comment was not here. Did the workaround work for your? I can't remember, but I may have had to clear the DNN cache between step 4 and 5.

I tested your workaround and it didn't seem to work. So I posted a reply to you. But after posting my reply, I wasn't 100% sure that I'd followed your steps properly so I tried it again and it worked perfectly. So I deleted my comment.  Sorry for confusing you.


"Note Stepping" being used as a way to suppress good quality Notes. Requires simple fix.

"Note Stepping" ( as it is called within the Community) is when someone puts up a bogus note as a way to suppress votes on another note that they disagree with, regardless of it's validity.
The problem arises from the fact that notes are arranged by time of submission, with the newest note showing at the top. And raters are most likely to see the note at the top, and less likely to scroll down and see those below it, especially when the top note is a bogus one.
So all someone has to do, if they see a pending note they don't like is to submit another note, and the time it takes for the original note to become visible will get much longer ( if not impossible ) since it will stop receiving as many ratings.
Fix:
Just change the way pending notes are sorted in the pending notes area. Instead of sorting by new, either sort by random or sort by top. Best is to sort by random at first and then sort by top once a certain threshold has been exceeded.

Agree. Random spread, or a more algorithmic choice on what's being displayed first.
There's statistics to be looked re: how much of a problem this really is though. The amount of votes total may smooth out any noise introduced by stepping

Strongly disagree. By ordering newest note first or random the system is vulnerable to gaming by bots.
The only correct way is to display the notes in chronological order with the OLDEST ONE ON TOP. This because most raters start with the first one they see, the top one.
It is absurd that  now one takes time to write a good note with links etc, only to see it buried at the bottom under 5-10 other fake notes which are just copying its content or links. So the impersonators are scoring points and improve their ratings, and are later able to do real damage. Not good.
In addition, when rating notes or reasons why a note is not needed, if the oldest is seen first it will be easy to spot and reject duplicates. This will reduce the incentives for 10 people to post 10 almost identical NNN replies, if the 9 most recent will be rated themselves as not needed.
In other words
1-real people need to be discouraged to add redundant notes
2-bots and scammers which post duplicate info to increase their impact will have their impact actually reduced


Fast path for modular image decoding

libjxl has several optimized fast paths for specific MA tree types. Implementing some of them would help speed up modular decoding.

#82

Now there are special routines for decoding libjxl effort 1 "fast-lossless" and effort 2 "gradient-only fixed tree".


Add NetBSD support

Looks like the way to get the number of cpus is different on netbsd...

If you rebase now, and install gmake, it should just work.


onBeforeUnmount hook in useQuery is undesirable when used in pinia store

Describe the bug
I was recently investigating a bug where various refetch calls for queries where not fetching data anymore after some time on the page. What I found is that queries are being stopped internally by Apollo through the onBeforeUnmount hook. While this might make sense on components, it can get troublesome on pinia stores where they are currently defined in my project. The hooks defined in a store are directed to the component the store is initialized on, and once that component is unmounted it also stops the query, even if it is still in use through the store. This might pose no problem if the store was initialized e.g. on the root component which never is unmounted, but is undesirable if initialized on any component that will unmount during the page's lifetime.
To Reproduce
Steps to reproduce the behavior:

Create a pinia store and use useQuery.
Initialize the pinia store (= call the use[..]Store for the first time) on a component that you can unmount through any means.
Unmount the component.
Call refetch from the useQuery defined in the store - it will return undefined, the query object from useQuery will be null, and the data - if being changed - will not update.

Expected behavior
In pinia stores, queries should not be stopped internally at all, regardless of the component they were first initialized on.
Versions
vue: 3.4.20
vue-apollo: 4.0.1
@apollo/client: 3.9.5
Additional context
A workaround for this issue would be to initialize every store that uses useQuery on the root component to ensure that the onBeforeUnmount hook will never be called before the lifetime of the page ends.

I encountered the same problem，After reading your explanation，And I also checked the source code，You're right，Your solution is ok，I initially wanted to package <keep-alive><keep-alive /> outside the <router-view><router-view />，But this is not a good solution


Debug examples do not work post-Rollup

The debug examples do not work post-Rollup release.
Some problems:

They are looking for ../../build/deps.js but this file is no longer present.
They are looking for ../leaflet-include.js, which is a symlink to

leaflet-include.js -> leaflet-rollup-src.js

where leaflet-rollup-src.js is not present.

There is an example map/rollup.html, which I thought demonstrates the usage, but is also linking to the non-existent leaflet-rollup-src.js.

I have the same problem, deps.js doesn't exist, so examples in /debug folder are not working

OK, then somebody please grep those files and remove the references to deps.js.

@IvanSanchez I'm happy to do all the fixes. Can you make one example (like map.html) the way it's supposed to be? It's not clear to me how it should work, with leaflet-rollup-src.js not present.

@hyperknot See https://github.com/Leaflet/Leaflet/blob/master/CONTRIBUTING.md#using-rollupjs - if you run npm run-script watch, you'll find leaflet-rollup-src.js in the right place.

@IvanSanchez thanks, then the problem is that rollup or build does not create that file, only watch does. The docs mention "rollup or watch", and I totally believe we should create that file upon rollup. I think watch shouldn't be needed for simply opening an html file.

As I see the watch config is almost the same as the build one. Can we just use dist/leaflet-src.js in examples instead?

One note is I think we should rename debug/leaflet-rollup-src to dist/leaflet-src.js to be consistent with the build script. update @hyperknot beat me to it.

@hyperknot Please feel free to tinker with the build scripts. The one thing I wanted to care about in the watch builds is to add a timestamp, to let the developer know that the bundle might be old.
Maybe you want to experiment with something more refined, like loading a small JS file that appends a <script> pointing to dist/leaflet-src.js and managing the timestamps. I'm very open to suggestions or new ways to do the debug builds.

@IvanSanchez I don't think we should overcomplicate this. Never cared about build timestamps in all my projects and it all worked fine.


Handle synchronize events for pull requests in github

Signed-off-by: Juan Antonio Osorio<EMAIL_ADDRESS>Summary
We should just try to re-evaluate the PR instead of creating a new
entry.
Fixes #5139
Change Type
Mark the type of change your PR introduces:

[x] Bug fix (resolves an issue without affecting existing features)
[ ] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Outline how the changes were tested, including steps to reproduce and any relevant configurations.
Attach screenshots if helpful.
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[ ] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

Perhaps the tests need to be updated 👍


Iconography has meaning

Chris had a great idea to drive some meaning with the iconography, like a box = React, or a moon is angular or something and we keep that consistent with the courses and provide a key! This could be fun to think through.

this seems related to #441 if we could have this theme go across automated images and iconography

this seems related to #441 if we could have this theme go across automated images and iconography


D3D11 minimal tess shader app used when Wine 2.9 added tess shader support..

Hi,
I know I know.. you said tess shaders were coming not soon(tm).. :-)
anyway I searched that Wine added support for tess shaders in version 2.9 and mentioned as bug fixed:
https://bugs.winehq.org/show_bug.cgi?id=41184
(DX11 Tesselation tutorial doesn't work correctly)
I mention because this is a very minimal D3D11 test app using tess shaders complete with executable and source code too:
http://www.rastertek.com/dx11tut38.html
http://www.rastertek.com/dx11exe38.zip
http://www.rastertek.com/dx11src38.zip
Hope it can help with your eventual efforts adding tess shader support..

Works as of 88c4e363e5f079bb54de44c92b6867073812707f.


<IP_ADDRESS>

Are there any plan for the new release (<IP_ADDRESS>) ?

Thanks for the ping -- I hadn't upgraded our systems yet so hadn't addressed it.  But knowing somebody cares is a motivator ;)   Going through CI/CD now

Released as tag <IP_ADDRESS> so images named like neomantra/onload:<IP_ADDRESS>-focal


test harness should handle missing etcd and kube-api-server

What happened:
Currently, the test harness panics if it cannot find etcd and the kube-api-server and start-control-plane is set. We should handle this more nicely. Either we download them for the user or bail out with a nice error.

Priority := low since this is a documented requirement


ChromaDB score goes the wrong way

System Info

langchain==0.0.227
langchainplus-sdk==0.0.20
chromadb==0.3.26

Who can help?
No response
Information

[ ] The official example notebooks/scripts
[X] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[X] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
I'm sorry, but I don't have the time to carve out a MRE right now. My take is that it's still better to report it than not to.
Expected behavior
similarity_search_with_score returns the distance as expected, but similarity_search_with_relevance_scores gives the same values, so that the closest distances return the smallest values, even though the output of the latter function is supposed to be higher for vectors that are closer:

similarity_search_with_relevance_scores
Return docs and relevance scores in the range [0, 1].
0 is dissimilar, 1 is most similar.

I just noticed the same, the code should do a 1 - score

Can't check right now, but maybe this has been resolved by the merge of #6570?


Migrate pickle usage of static multi-agent simulation data to protobuf

The pickled ExperimentData is too memory intensive, so instead of storing one object with both Sim.SimData and Sim.Stats, break them out into 2 serialized protobufs.

I don't have good ideas on how to migrate to, since I found out that the protobufs have a 2 GB limit. Typically the ExperimentData and its intended final form VisualizationDataGroup is much larger than that.
Sticking with solutions that revolve around (re-)structuring data collection for now, e.g., run experiments with reduced number of varying parameters.


Test: added feature for manual test AB#19154

@girtsfre
A couple of remarks:

The name of the file should always end in .feature
The name of the file says 'for_espocrm', it should say 'from_espocrm'
Not sure if its a preview bug but the indentation and new lines should be cleaned up
On the contents of the test, the body properties depend on the chosen FSP. So I think it's not 100% correct that everything should always be provided.


[Enhancement] Method encodes String bytes without specifying the character encoding [RedisProducerTest]

Search before asking

[X] I had searched in the issues and found no similar issues.

Enhancement Request

located at:
eventmesh-connector-plugin/eventmesh-connector-redis/src/test/java/org/apache/eventmesh/connector/redis/producer/RedisProducerTest.java line 68 104
analysis and explanation:
The behavior of the String(byte[] bytes) and String.getBytes() is undefined if the string cannot be encoded in the platform's default charset.
Describe the solution you'd like
reference:

Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

i want to fix it


Create custom README

This task involves creating a comprehensive README file for the repository profile. The README will serve as a guide for users, providing essential information such as project overview, installation instructions, usage guidelines, and contribution guidelines.

Closing this issue as the comprehensive README file has been successfully created and added to the repository profile. The README serves as a thorough guide for users, offering essential information including project overview, installation instructions, usage guidelines, and contribution guidelines. If there are any further adjustments or updates needed, feel free to reopen the issue or create a new one. Thank you for your attention to detail and collaboration!


[utils] Python 3 support + tidies in check-incremental

What's in this pull request?

Use print as a function, not as a statement
Commonise the code for compiling and getting binary statistics into
a reusable function
Pull out VERBOSE and NUM_ITERATIONS as top-level constants, to
make it clear where/how to change them.
Use a docstring rather than a block comment.
In the original version, "Iteration {}:".format(iteration) would
index from 0; in human-readable output, index 1 is more sensible.

Resolved bug number: (SR-)
n/a

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

cc @eeckstein, who's the author of this utility.

@alexwlchan did you test your changes by compiling with the SWIFT_CHECK_INCREMENTAL_COMPILATION option?

@swift-ci please Python lint.

@swift-ci please smoke test.

Thanks @alexwlchan!


Mac App hang at CFGetRetainCount with Chinese String in data?

I have a Mac App using j2objc lib, it runs normal when there's only English characters ( from files ), but when there are Chinese Character it hang at CFGetRetainCount.
The code actually finish running, but App was hanging, it seems ARC was running try to release memory. Any solutions ?

Cannot reproduce. Please reopen with an attached test if it continues to be an issue.


(windows) Application window never appears

The Anchor application never shows up for me except  for signing requests. Even when I close and re-open the application it's just...invisible.

There's a few possible reasons for this.
The first one is that Anchor lives in the system tray near the clock on the menu bar. If you close the main window, it closes down to that position and you can click that icon to make it reappear.
The second being related to the monitor setup of the computer. Any chance you use more than one monitor? Anchor tries to remember the last position and open in the same place - so if monitor configurations change, sometimes it ends up in a place that doesn't exist. Maybe we should remove this "feature" 😅

There's a few possible reasons for this.
The first one is that Anchor lives in the system tray near the clock on the menu bar. If you close the main window, it closes down to that position and you can click that icon to make it reappear.
The second being related to the monitor setup of the computer. Any chance you use more than one monitor? Anchor tries to remember the last position and open in the same place - so if monitor configurations change, sometimes it ends up in a place that doesn't exist. Maybe we should remove this "feature" 😅

My monitor wasn't connected at the time but I do use multiple monitors. I connected it and it's not showing up still. I also use an application that kind of adds multiple screens like it's a tablet pc? I can see there's a window but it's nowhere to be seen

It could have either been that the monitor wasn't connected, or the multiple screen application. Clicking that system icon is going to put it right back where it thinks it should be.
We have a new release coming up pretty soon - I'll open an issue to remove that bit of code so the window just opens on whatever screen is the default. Sorry about the hassle!


Rotate keys in database [T671]

Fetch AcraStructs from database, decrypt, encrypt with new keys, put back to the database.

lgtm


make clean should not error if no coverage files

Update Makefile to ensure make clean doesn't fail when there are no coverage files to remove.
Verification
$ rm -f coverage*.out
$ make clean

Should not report an error.

Helps to be on the PR branch! ;)  Thanks! 👍


problems with stdout

we have a plugin that is already communicating on stdout. for some reason, when terminal-size tries to open stdout again and call getconsolescreenbufferinfo, it fails. It would be nice to do something kind of similar to term-size where they check all 3 like this, instead of just silently failing.
    if unsafe { GetConsoleScreenBufferInfo(GetStdHandle(STD_OUTPUT_HANDLE), &mut console_data) } != 0 ||
        unsafe { GetConsoleScreenBufferInfo(GetStdHandle(STD_INPUT_HANDLE), &mut console_data) } != 0 ||
        unsafe { GetConsoleScreenBufferInfo(GetStdHandle(STD_ERROR_HANDLE), &mut console_data) } != 0 {
        Some(((console_data.srWindow.Right - console_data.srWindow.Left + 1) as usize,
                (console_data.srWindow.Bottom - console_data.srWindow.Top + 1) as usize))
    } else {
        None
    }

Thanks for the report.  I think you have the right idea here.   We should be checking all 3 streams, and using the size from the first one that doesn't return an error.


fix: rust-analyzer spamming overly error message when workspace not being loaded

Fixes #10120

Were you able to test this? I don't see those messages on my PC.

After using this version, these issues didn't happen anymore for me.

Eh, I think there are some real bugs, but we shouldn't annoy users like this.
bors r+


apiembed.com down

It looks like the hosted website is down. Is this a temporary situation or should we be looking at alternatives?

Sorry about that @robvk it should be back now 👍

I think api.apiembed.com has the same problem:

fixed now


Add Estonian translations to Crowdin

Description
Were you to enable Estonian in Crowdin, I would be happy to get started on implementing translations (it's my native language). Opening an issue for this was suggested in the #translations Discord channel.
Please tick the boxes

[X] You've read the docs
[X] You've checked for duplicate issues

Sure! Could you tell me your email on crowdin so that I can make you a proofreader for the language ?

@ajnart Thanks, took the liberty of pinging you on Discord with the e-mail. :)

Access granted, closing issue.


[ALLUXIO-2974] Replace $ALLUXIO_LOG_DIR with $ALLUXIO_LOGS_DIR.

https://alluxio.atlassian.net/browse/ALLUXIO-2974
In alluxio-masters.sh and alluxio-workers.sh, we should not use
$ALLUXIO_LOG_DIR since we already have an enviroment variable called
$ALLUXIO_LOGS_DIR that we should always respect.

@apc999 @aaudiber Here is the newly created PR. Could you take a look and comment? Thanks!

https://alluxio.atlassian.net/browse/ALLUXIO-2974
In alluxio-masters.sh and alluxio-workers.sh, we should not use
$ALLUXIO_LOG_DIR since we already have an enviroment variable called
$ALLUXIO_LOGS_DIR that we should always respect.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/16204/
Test PASSed.


RC.2 Spot Check: Mean: Share and Balance

RC Spot-Check Test
Mentions: @kathy-phet, @KatieWoe, @samreid
Simulation links

github repository for issues
phet top level directory
sim: all_phet.html

Test Matricies
General Test:

[x] Tester = KW, Platform = Win 11 Chrome, Time = 1hr
[x] Tester = NS, Platform = Mac + Safari, Time = 1hr

Additional Features Test:

[ ] Tester = , Platform = , Time =
[ ] Tester = , Platform = , Time =

Features included

PhET-iO
[x] Alternative Input
UI Sound
Sonification
Description
Voicing

Focus and Special Instructions

The issues that came up for RC.1 were simple fixes. A spot check to verify those issues have been resolved is all that is necessary. https://github.com/phetsims/mean-share-and-balance/issues/124 was specific to Safari, so make sure to double check and verify there. The other two issues can be spot checked on a single randomly selected platform. https://github.com/phetsims/mean-share-and-balance/issues/102 is an issue that was brought up by @amanda-phet. Please verify that there are 8 rings and the paper color is not stark white.
Issues to Verify
These issues should have the "status:ready-for-review" label. Unless an issue says to close after verifying, assign the
issue back to the developer.

[x] https://github.com/phetsims/mean-share-and-balance/issues/102

[x] https://github.com/phetsims/mean-share-and-balance/issues/123

[x] https://github.com/phetsims/mean-share-and-balance/issues/124

The artifacts from https://github.com/phetsims/mean-share-and-balance/issues/122 will be fixed in future common code commit.

https://github.com/phetsims/mean-share-and-balance/issues/125 will be fixed in future common code commit, and team has decided it does not block prototype publication.

For QA...

General features

What to Test

Click every single button.
Test all possible forms of input.

Test all mouse/trackpad inputs.
Test all touchscreen inputs.

If there is sound, make sure it works.
Make sure you can't lose anything.
Play with the sim normally.
Try to break the sim.
Test some query parameters. (See QA Book for a list of query parameters.)
When making an issue, check to see if it was in a previously published version.
Try to include version numbers for browsers.
If there is a console available, check for errors and include them in the Problem Description.
As an RC begins and ends, check the sim repo. If there is a maintenance issue, check it and notify developers if there is a problem.
As the RC ends, notify the developer of any new QA credits that need to be added.

PhET-iO features

What to Test

Make sure that public files do not have password protection. Use a private browser for this.
Make sure that private files do have password protection. Use a private browser for this.
Make sure standalone sim is working properly.
Make sure the wrapper index is working properly.
Make sure each wrapper is working properly.
Launch the simulation in Studio with ?stringTest=xss and make sure the sim doesn't navigate to youtube
For newer PhET-iO wrapper indices, save the "basic example of a functional wrapper" as a .html file and open it. Make
sure the simulation loads without crashing or throwing errors.
Load the login wrapper just to make sure it works. Do so by adding this link from the sim deployed root:/wrappers/login/?wrapper=record&validationRule=validateDigits&&numberOfDigits=5&promptText=ENTER_A_5_DIGIT_NUMBER

Further instructions in QA Book
Conduct a recording test to Metacog, further instructions in the QA Book. Do this for iPadOS + Safari and one other
random platform.
Conduct a memory test on the stand alone sim wrapper (rc.1).
Test one platform combination with ?phetioDebug=true on the Studio and State wrapper.
If Pan/Zoom is supported, make sure that it works when set with PhET-iO State.
Test that the sim works offline:

Click the link to the phet-io zip file (at top of issue) to download the zip file.
Unzip it to a spot locally.
Open index.html by double clicking it on your desktop or in a Finder-view.
It should look like the standalone version of the sim in PhET-iO brand.

Accessibility features

What to Test

Specific instructions can be found above.

Make sure the accessibility (a11y) feature that is being tested doesn't negatively affect the sim in any way. Here is
a list of features that may be supported in this test:

Alternative Input
Interactive Description
Sound and Sonification
Pan and Zoom
Mobile Description
Voicing

Test all possible forms of input.

Test all mouse/trackpad inputs.
Test all touchscreen inputs.
Test all keyboard navigation inputs (if applicable).
Test all forms of input with a screen reader (if applicable).

If this sim is not in this list or up to date there, make an
issue in website to ask if PhET research page links need updating. Please
assign to @terracoda and @emily-phet.

Screen Readers
This sim may support screen readers. If you are unfamiliar with screen readers, please ask Katie to introduce you to
screen readers. If you simply need a refresher on screen readers, please consult the
QA Book, which should have all of the information
you need as well as a link to a screen reader tutorial made by Jesse. Otherwise, look over the a11y view before opening
the simulation. Once you've done that, open the simulation and make sure alerts and descriptions work as intended.

Platforms and Screen Readers to Be Tested

Windows 10 + Latest Chrome + Latest JAWS
Windows 10 + Latest Firefox + Latest NVDA
macOS + Safari + VoiceOver
iOS + Safari + VoiceOver (only if specified in testing issue)

Critical Screen Reader Information
We are tracking known screen reader bugs in
here. If you find a
screen reader bug, please check it against this list.

Keyboard Navigation
This sim supports keyboard navigation. Please make sure it works as intended on all platforms by itself and with a
screen reader.
Magnification
This sim supports magnification with pinch and drag gestures on touch screens, keyboard shortcuts, and mouse/wheel controls. Please test magnfication and make sure
it is working as intended and well with the use cases of the simulation. Due to the way screen readers handle user input, magnification is NOT expected to
work while using a screen reader so there is no need to test this case.

FAQs for QA Members

There are multiple tests in this issue... Which test should I do first?
Test in order! Test the first thing first, the second thing second, and so on.

How should I format my issue?

We typically assign the developer who opened the issue in the QA repository.

My question isn't in here... What should I do?
You should:

Consult the QA Book.
Google it.
Ask Katie.
Ask a developer.
Google it again.
Cry.

QA is done


Implicit Function Adjoint Rules for Enzyme

I am trying to use Enzyme to differentiate a function which builds and solves a NonlinearProblem. I believ the issue is with the enzyme extension which is why I'm opening the issue here.
Expected behavior
I expect the SciMLSensitivity overload for NonlinearProblem to be called by Enzyme when differentiating.
Minimal Reproducible Example 👇
using SciMLSensitivity
using NonlinearSolve
using Enzyme
using Zygote
using NonlinearSolve
using Test

function nonlinearf(du, u, p)
    du[1] = u[1] .- p[1]
    nothing
end
p = [1.0]
function f(p)
    prob_nl = NonlinearProblem{true}(nonlinearf, [0.0], p)
    sol_nl = solve(prob_nl, TrustRegion(); abstol = 1e-9, reltol=1e-9)
    return sol_nl.u[1] * 2
end

dp = zeros(1)
dp_zygote = Zygote.gradient(f, p)[1]    #Zygote Works
Enzyme.autodiff(Reverse, f, Active, Duplicated(p, dp))  #Enzyme Fails

Error & Stacktrace ⚠
I believe this is the problematic line: https://github.com/SciML/DiffEqBase.jl/blob/1658ed0c18e5e8e6776e7da67c52a56af5076fa7/ext/DiffEqBaseEnzymeExt.jl#L33
ERROR: MethodError: no method matching copyto!(::Float64, ::Base.Broadcast.Broadcasted{…})

Closest candidates are:
  copyto!(::AbstractArray, ::Base.Broadcast.Broadcasted{<:Base.Broadcast.AbstractArrayStyle{0}})
   @ Base broadcast.jl:959
  copyto!(::AbstractArray, ::Base.Broadcast.Broadcasted)
   @ Base broadcast.jl:956
  copyto!(::DiffEqArrayOperator, ::Any)
   @ SciMLBase C:\Users\Matt Bossart\.julia\packages\SciMLBase\SDjaO\src\operators\basic_operators.jl:127
  ...

 Only LinearSolve and ODEProblems have gotten that treatment so far.

@ChrisRackauckas I can help with the EnzymeRules infra, is there an adjoint function lying around for this. Similarly what would be the right function to hook into with a rule

It's just like the ODE infrastructure in that it just lowers to call SciMLSensitivity for all of the real work. The current dispatch is in DiffEqBase.jl right next to the ODE one.

Sorry you're going to have to give me some more concrete pointers to functions which should have a rule.

I believe this is the dispatch Chris is referring to:
(https://github.com/SciML/DiffEqBase.jl/blob/08fbaf0561d797359027849bac7797ce10948846/src/solve.jl#L1066C1-L1083C4)
For the ODE case, the enzyme rule is for the solve_up  function, so I'm guessing the same should be true for the case of nonlinear problems:
https://github.com/SciML/DiffEqBase.jl/blob/08fbaf0561d797359027849bac7797ce10948846/ext/DiffEqBaseEnzymeExt.jl#L9C1-L37C4
I'm not sure though, @ChrisRackauckas can you confirm?

Yup and then the adjoint just passes it on: https://github.com/SciML/DiffEqBase.jl/blob/08fbaf0561d797359027849bac7797ce10948846/ext/DiffEqBaseChainRulesCoreExt.jl#L22C28-L29 . Looking at the ODE one, the NonlinearProblem one might just work if NonlinearProblem is made a mutable struct.

I tried making NonlinearProblem a mutable struct but am still getting an error:
Closest candidates are:
  copyto!(::OrdinaryDiffEq.ArrayFuse, ::Base.Broadcast.Broadcasted)
   @ OrdinaryDiffEq C:\Users\Matt Bossart\.julia\packages\OrdinaryDiffEq\tAI61\src\wrappers.jl:26
  copyto!(::AbstractArray, ::Base.Broadcast.Broadcasted{<:Base.Broadcast.AbstractArrayStyle{0}})
   @ Base broadcast.jl:959
  copyto!(::AbstractArray, ::Base.Broadcast.Broadcasted)
   @ Base broadcast.jl:956
  ...

It is failing at this line:
https://github.com/SciML/DiffEqBase.jl/blob/1658ed0c18e5e8e6776e7da67c52a56af5076fa7/ext/DiffEqBaseEnzymeExt.jl#L33

What is the type of the variable being zero's there
Enzyme semantics require it be zeroed but it looks like it may not be defined for that type?

The type of the variable being zero'd is Float64. The error I hit above as the same error you get if you do (trying to broadcast a value to a float):
x = 1.0
x .= 0.0

The part I'm unclear on is why this works for an ODEProblem and not a NonlinearProblem. @ChrisRackauckas is it expected that the return from _solve_adjoint would be different for those two?

Oh that's definitely it.
Here's a solution. Replace the deepcopy with an Enzyme.make_zero and get rid of the for loop

Will https://github.com/SciML/SimpleNonlinearSolve.jl/blob/6b682af1e2155298064d2a035939e6aa373edbed/test/gpu/cuda_tests.jl#L40-L72 these still work if we make the struct mutable?

No, which makes this a major PITA. With DiffEqGPU we had to create a separate immutable problem that gets used internally to keep CUDA kernel compatibility. It converts under the hood so the user doesn't know, but it's definitely code smell.

I can confirm that by using Enzyme.make_zero instead of the for loop a simple case runs. It seems to work as well even without making NonlinearProblem mutable. What is the underlying reason for needing it to be mutable?

Limitation of Enzyme mixed activities.

At one point it’s on the roadmap to remove the mixed activity limitation,
but some other GPU, type stability and blas stuff is currently higher
priority
On Wed, Jun 5, 2024 at 7:20 PM Christopher Rackauckas <
@.***> wrote:

Limitation of Enzyme mixed activities.
—
Reply to this email directly, view it on GitHub
https://github.com/SciML/NonlinearSolve.jl/issues/439#issuecomment-2150575969,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJTUXCT4AV2UTFAW4BR5FLZF5CGZAVCNFSM6AAAAABIJ3YE3WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJQGU3TKOJWHE
.
You are receiving this because you commented.Message ID:
@.***>

Interesting, I think I am hitting this same mixed activity limitation for another non-mutable struct in my application. @wsmoses would you expect passing a reference to the struct as in the docs here: https://enzyme.mit.edu/julia/stable/faq/#Mixed-activity would be a suitable workaround for the mixed activity limitation?

I ask because I've tried it and still get Mixed activity errors when passing the reference instead

I started two draft PRs above. @ChrisRackauckas I just added the code to SimpleNonlinearSolve because that is where the tests are. Should it be moved somewhere else? I'm hitting a method error when trying to convert which I don't understand; I mentioned it in the PR. Can you see what I'm doing wrong there?

@wsmoses let me know when the new make_zero! is available. Then I will fix the enzyme rule in DiffEqBase.jl and add a test in SciMLSensitivity.jl and we should be all set.

@m-bossart see here: https://github.com/EnzymeAD/Enzyme.jl/pull/1518 maybe test it out and see if it works [and in interim getting things ready for to release]

It is released
On Sat, Jun 8, 2024 at 11:49 AM Matt Bossart @.***>
wrote:

@wsmoses https://github.com/wsmoses let me know when the new make_zero!
is available. Then I will fix the enzyme rule in DiffEqBase.jl and add a
test in SciMLSensitivity.jl and we should be all set.
—
Reply to this email directly, view it on GitHub
https://github.com/SciML/NonlinearSolve.jl/issues/439#issuecomment-2156083306,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJTUXC5ZVA4KEWQI4MHP33ZGMR2DAVCNFSM6AAAAABIJ3YE3WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJWGA4DGMZQGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

Ok I now have 4 PRs open that should address this. @ChrisRackauckas can you review these collectively? @wsmoses is the use of make_zero and make_zero correct?

SciML/SciMLBase.jl#708
SciML/SimpleNonlinearSolve.jl#151
SciML/DiffEqBase.jl#1057
SciML/SciMLSensitivity.jl#1066

Reviewed the ones I can offer comments on


Difference between two dates reversed?

In arithmetic, if a and b are numbers, a - b < 0 iff a < b
I would think that when subtracting dates, this relationship would hold true since they can be represented as numbers. The reverse is however true:
let now = Date()
let futureDate = now + 1.days
futureDate - now        // this is negative

This is very counterintuitive, since the result is the opposite of regular arithmetic. And is especially confusing when the result of this is positive:
futureDate.timeIntervalSinceReferenceDate - now.timeIntervalSinceReferenceDate

Is there a reason for this behaviour?

I encountered this issue when running on iOS 9, but not when I ran it on the simulator which runs iOS 10. Can you please confirm which iOS version you are testing this on?
I have had to use .absoluteDate.timeIntervalSince(....) to maintain compatibility with iOS 9 so far...

Yeah that's right; I've fixed it in 4.1.0.
Reference commit is https://github.com/malcommac/SwiftDate/commit/0c23cb8adba2a3c2ef3716b9df8e75533be365de


Confusion about feature maps/ heatmaps

Hi,
first of all thanks for providing the code. While reading the paper and looking through your code, I got a bit confused with the following: In your paper you state that you compute the difference between the two heatmaps, feed them though some conv layers to receive offsets, and use these to warp heatmap B. This seems to happen in the code as well. But figure 2 implies that you are computing the difference between two feature encodings (from the last layer of the downsampling path), then compute the offsets, warp the encoding and then decode it back into heatmaps. Would be great to get some clarification here.

Figure 2 in the paper is made for illustrative purposes. In practice, we can do the warping either on top of pose heatmaps or on top of any of the selected features. In our experiments, we tried both schemes, and observed that warping on top of heatmaps worked slightly better in the context of video pose estimation. Therefore, in the end we used the scheme where the difference is computed between two heatmaps, which is then used to predict the offsets. Lastly, these offsets are used to warp pose heatmap from one frame to another. Hope this makes it clearer, and sorry for the confusion!

Great, thanks for the fast response!


[cocoon] Angular Dart project fails to build on Flutter dev channel

[SEVERE] build_web_compilers:entrypoint on web/repository.dart:
Dart2Js finished with:

packages/flutter_web/src/material/page.dart:52:15:
Error: Getter not found: 'opaque'.
       assert(opaque),
              ^^^^^^
packages/flutter_web/src/widgets/framework.dart:1693:15:
Error: Getter not found: 'runtimeType'.
            "$runtimeType's children must not contain any null values, "
              ^^^^^^^^^^^
packages/flutter_web/src/cupertino/route.dart:110:15:
Error: Getter not found: 'opaque'.
       assert(opaque),
              ^^^^^^
Error: Compilation failed.

When the dev channel rolls into beta, this will cause issues with new deployments of Cocoon.

Dup of https://github.com/flutter/flutter/issues/69595


[Feature Request]: Add the emoji menu to Shortcut Guide

Summary of the new feature/enhancement
Add the Windows logo key + . (period) keyboard shortcut to the Shortcut Guide on Windows 10. As described in the Windows 10 keyboard tips and tricks article, it will display the emoji keyboard.
Note that Windows logo key + ; (semicolon) is mapped to the same action in Windows 10.

being tracked in #179


gen: middleware support

Description
At the moment the project uses a custom static router which doesn’t support middleware.

It would be great if it was possible to add support for different types of middleware:

middleware which runs for all requests
middleware that runs per endpoint

References

We have ogen/middleware (that is 1) and (2) looks like #566

Thanks for the quick response.

Would be great to have more information exposed (entire Route struct) inside the middleware context.
I came across a situation where I have a external module implementing the middleware that need the Route.pathPattern field. In that situation I can use (1) because the Request struct doesn't have the needed field and (2) does not work because i can't/doesn't want to import generated code to be able to call FindRoute
Any ideas ?


Reset TestTransport on ensureKernelShutdown

An alternative approach to #82, requires https://github.com/symfony/symfony/pull/58240

requires symfony/symfony#58240

why don't we implement ResetInterface? So we won't need Symfony 7.2 to benefit from this feature.

@nikophil Hi, the reason is your comment about introducing a BC break. With this approach, the existing method's behavior remains unchanged.

Would you mind adding a test which confirms that everything is working? We should test this in both a test which uses InteractsWithMessenger trait, and a test which doesn't, I guess

Sure, no problem, but I don't think I can actually cover the original scenario as it depends on the execution order and a specific test being run first:

So if we have this scenario :

Test1 add messages on transport but doesn't care about messenger assertions so doesn't implement InteractsWithMessenger
messages remains in transports
Test2 implements InteractsWithMessenger and should reset collected messages before tests.

the reason is https://github.com/zenstruck/messenger-test/pull/82#issuecomment-2096198752. With this approach, the existing method's behavior remains unchanged.

oups yes, you're right :hand_over_mouth:  let's do your way then!

Also, https://github.com/symfony/symfony/pull/58240 is a bugfix, so it won't require 7.2.

perfect

depends on the execution order and a specific test being run first

yeah... we're doing strange things to test behavior between tests in this library :sweat_smile: ... basically, not using --order-by nor any executionOrder  in the xml will play the tests in the natural order (alphabetically). I know this is not ideal, but the only way to test such behaviors. No problem if you don't want to bother with this, I might do it at some point. Maybe I'll upgrade first to PHPUnit 10, in order to leverage #[DependsExternal] attribute...

depends on the execution order and a specific test being run first

yeah... we're doing strange things to test behavior between tests in this library 😅 ... basically, not using --order-by nor any executionOrder  in the xml will play the tests in the natural order (alphabetically). I know this is not ideal, but it is the only way to test such behaviors. No problem if you don't want to bother with this, I might do it at some point. Maybe I'll upgrade first to PHPUnit 10, in order to leverage #[DependsExternal] attribute...

@nikophil Actually, I just noticed the testsuite name="zenstruck/messenger-test transports are reset correctly" which is exactly what I need, so I think the test i possible.

@nikophil Funny thing, the test was kind of already there, it just didn't work when all tests were run cause it requires TestTransport::$enableMessagesCollection to be set to its initial value of true for the bug to manifest. The tests are now failing cause they require https://github.com/symfony/symfony/pull/58240. See https://github.com/zenstruck/messenger-test/pull/90/commits/ab2123d74dbb586b92c6bfb35095deea08c44eea

Don't hesitate to ping me once the fix is released in SF, and I'll merge and release this PR
It seems there is a bunch of issues and PR that can be closed as well. Thanks!

@nikophil The fix has been released by Symfony, however, I don't think it's possible to get the tests for 6.3 and 7.0 to pass since those versions are no longer supported. I'm not sure what to do about this, the fix is valid for all supported versions.

hey @HypeMC
I just discovered that some tests are failing, maybe because of a new version of Symfony. I'll fix them asap, before merging your PR

@HypeMC would you mind to rebase your PR?

@HypeMC would you mind to rebase your PR?

@nikophil Done

released in 1.10.0


cp of a single file hangs on a prompt

Describe the bug
Currently, a copy of a single file, e.g. cp linker-scripts/bootstrap.ld link.T will generate the command xcopy linker-scripts\bootstrap.ld link.T. This command will prompt the user, staying stuck there forever:
Does link.T specify a file name
or directory name on the target
(F = file, D = directory)?

A simple way to avoid this would be to use copy for cp without -r. cp without -r can only have files in source (directory sources are ignored), which is exactly what the copy command on windows is for. This way:
cp -r a b
xcopy a b

cp a b
copy a b

With this solution, cp -r file1 file2 will stay broken, but I think that's an okay trade-off to make. Alternatively, cp a b could be turned into echo f | xcopy a b.
Here's a stackoverflow link about the whole xcopy on single files issue: https://stackoverflow.com/questions/33752732/xcopy-still-asking-f-file-d-directory-confirmation
To Reproduce
From cargo-make: Make a folder with a file1 file, then run the following Makefile.toml:
[tasks.default]
workspace = false
script_runner = "@shell"
script = ["cp file1 file2"]

It will hang until the user press F or D.

i liked your echo f |  solution better then have the copy directories broken.
this way we support both file copy and recursive dir copy. right?

I think so. I'm not super good at command-line windows stuff though, so don't take my word for it ^^'.
There are four scenarios off the top of my head that we need to support:

cp file file will work
cp -r directory directory will work
cp -r file file should work (needs testing)
cp file directory should work (needs testing)

ya, i'll check it out unless you want to check it out yourself and publish a PR?
up to you

I already have a fix coded up using the first solution, since I needed a quick fix to make progress on my project. I'll investigate the above cases when I'm around a windows machine sometimes tonight.

Turns out, microsoft provides windows VM so I managed to test it right now. And the result is really sad:
C:\Users\IEUser\Desktop>xcopy /E desktop.ini test.ini
Cannot perform a cyclic copy
0 File(s) copied

So cp -r file file would not work under the echo f | xcopy solution.

More fun stuff:
C:\Users\IEUser\Desktop>mkdir test

C:\Users\IEUser\Desktop>xcopy desktop.ini test
0 File(s) copied

it seems cp file directory

copy has its own share of problem. It won't allow copying multiple files:
C:\Users\IEUser\Desktop>copy eula.lnk test.txt test
The syntax of the command is incorrect.

A stackoverflow solution:
for %I in (eula.lnk test.txt) do copy %I test

so.... if there is no good solution, maybe completely different approach.
shell2batch tries to convert commands, but sometimes you might want to help it out.
so what if we had something like the following
some command
another command
problematic command //shell2batch: dos command

meaning, if a line ends with some comment and a prefix, we take the rest of the line as the replacement command.
so users could define a specific shell command line replacement but only for specific problematic lines.
mkdir -p somedir
cd somedir
cp a b //shell2batch: copy a b
echo whatever

also i wonder if /i solves the xcopy issues?

/i doesn't solve the xcopy issue:
C:\Users\IEUser\Desktop>xcopy /i eula.lnk test2.txt
Does test2.txt specify a file name
or directory name on the target
(F = file, D = directory)?

Current discoveries, I'll edit this comment if I find any new ones:

cmd
echo f
xcopy/copy

cp file file
✅
✅

cp -r directory directory
✅
✅

cp -r file file
🔴
🔴

cp file directory
🔴
✅

cp file1 file2 directory
🔴
🔴

cp -r file1 file2 directory
🔴
🔴

if a line ends with some comment and a prefix, we take the rest of the line as the replacement command.

Independently of this issue, this might be a good idea. It would help big scripts that need to differenciate only for a couple commands. So that's a big :+1: for me.
However, I still think we should try to find something that works for cp. It's such a simple command, it'd be a shame if we couldn't find a good translation for it.
I'm currently thinking this: If it's a cp -r, treat it as an xcopy. If it's a non-recursive cp, use copy. And in both case, apply the for %I in (files) do copy %I dest loop if there's more than one non-flag argument. I think this would make all the common cases of cp work correctly.

i'm for doing the cp changes and in addition doing the hint (as separate issue).
as for powershell.... i'm not sure since if i remember correctly, you can't run ps files as is. you need to first enable powershell running files via some policy command.


Improve user's experience to report bugs

Waiting on vscode people to get back to me on the following questions:

Should I store log files on the user's computer? If so, is there a place where I should be storing those log files?
Is it possible to automatically send the log files to me when the users ran into an error?

https://github.com/microsoft/vscode/issues/144638#issuecomment-1062126478


Fix use of cheerio objects when not using URL strings

Hey, i played around with your module and had some encoding issues. Saw in your source that i can use cheerio objects (and load html and decode on my own) as input on top of URL strings. Tried it, crashed with "False input data".
The cheerio object is indeed of type function.
Changed that in your code and it works like a charm :-)

thx :) I'm wondering why I didn't notice this myself... Well as this is really old, here be dragons I guess

Whooohooo. Thanks for that. One more thing, - can you please do a npm publish so that we can use it


ScaleAnimation with param causes the error

First of all, my development environment as like below
"react": "16.2.0",  "react-native": "0.53.3"
I tried to create ScaleAnimation with toValue prameter as you described in document
const scaleAnimation = new ScaleAnimation({ toValue: 10 })
but, it never works with below error message

Invariant Violation: Cannot interpolate an input which is not a number

am I missing something?

not accept object argument.
Usage:
new ScaleAnimation(10)


support MIDI 2.0

It's not clear how this will affect the standard MIDI file format, but it would be good to support any such changes.  Since MIDI 2.0 is still being developed, this may be ongoing for a while.

Currently MIDI 2.0 will need a new SMF structure. This will be sometime before it is released I would think :)


Problem with jquery 3.5.1

Hi everyone,
I've noticed an issue when you use jquery.steps in a context with jquery 3.5.1.
More exactly, there is a rendering problem and functional problem when you try to press on next button, in this case the system throws the following error:
Uncaught TypeError: Cannot read property 'currentIndex' of undefined
at increaseCurrentIndexBy (jquery.steps.js:528)
at goToNextStep (jquery.steps.js:459)
at HTMLAnchorElement.paginationClickHandler (jquery.steps.js:801)
at HTMLAnchorElement.dispatch (jquery.min.js:2)
at HTMLAnchorElement.v.handle (jquery.min.js:2)
otherwise if you use jquery 3.3.1 everything work very well
About rendering problem...

Thank you very much, anyway

I have fixed it
The problem was when I set the structure
$(".wizard").steps({
...
labels: {
next: "<i class="icon-arrow-right14" />",
previous: "<i class="icon-arrow-left13" />",
...
in this case in jquery 3.3.1 worked, whereas in jquery 3.5.1 didn't work, but if I change the tag  with  it's ok and works also with jquery 3.5.1
just to share, I hope it helps you


Test very large witnesses with gnarkd

Tested with witness size = 20M (640Mb) on 2 separate machine all good.


question(1.x): should service names begin with $?

In Angular 1.x, angular-provided services typically begin with a $.

I think it's the right way to go to name the 1.x router $router. Even if it's clear that the router is actually a back port from 2.x to 1.x, where no $ prefixes are used anymore, I don't think that people will be thankful if it follows yet another convention in 1.x.
My 0.02£

:+1:

We should then also change the name of componentLoaderProvider to $componentLoaderProvider accordingly.

done, and available in v0.3.0, which I just published.

:+1:


"Access Denied" message during running prism validate on mock server

I created a mock server as shown below:
C:\Users\ajayara\Downloads\prism>prism_windows_386.exe  mock --spec C:\Project\bitbucket_new\mypath\MyService.oas2.yml
Prism version is 2.0.12
Prism mock server started on port 4010
Now when I run prism validate to perform contract test, I get the error "access denied"
C:\windows\system32>C:\Users\ajayara\Downloads\prism>prism_windows_386.exe validate --spec C:\Project\bitbucket_new\mypath\MyService.oas2.yml --upstream http://localhost:4010
Access is denied.
My Prism version is 2.0.12 and I am running this on a windows 10 machine.
Am I missing anything in the way I execute the commands?

@amalsgit are you running the mock server at the same time you are running the validate server? If you are it looks like you want to send requests through the validate server to the mock server? If that is the case, you actually don't need to run the validate server, when you mock your specification with the mock command, your request/response headers/body will be contract tested.
The response will have up to 4 headers on it Sl-Request-Valid, Sl-Request-Validation-Messages, Sl-Response-Valid, Sl-Response-Validation-Messages.

Sl-Request-Valid: can 'true' or 'false'. If it is false, then Sl-Request-Validation-Messages will be a stringified array of objects and each object is a validation error. If Sl-Request-Valid isn't set, it means validation wasn't run because nothing was defined in your specification.

Sl-Response-Valid: samething as the request headers, but just for the response.

Just to clarify what is happening when you run the mock server, and then the validate server, prism by default starts a server on 4010, so in if you update your validate command with -p 4011, then when you send a request through localhost:4011, if will be forward to your mock server.


Add backups restored progress when migrating from Azure Blob Storage

This PR addresses customer confusion on the backup restore progress for blob storage migrations.

Update the logic to enable "Complete cutover" button for blob storage migrations. Should be only enabled if "Last restored file" exists, meaning there is at least one fully restored file.
Add "Currently restoring file" field to Migration Cutover page for Azure Blob Storage migrations. If "Last restored file" and "Currently restoring file" have the same name, should display "All backups restored"; otherwise, display the value from the API response.

Migration is in progress (migrationState: WaitRestoreStart)

There are no completely restored backup file yet; Currently restoring a file

Migration is ready for cutover - "All backups restored"

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
27232

Relevant Lines:
59278

💛  - Coveralls


instalation

plese explain how to install.

You can use pip to install the library directly from github, and then if you run leapcast --help it'll tell you how to use it.


SAK-43273 - Group Manager - Manually added users are not a selectable option when auto-forming groups

https://jira.sakaiproject.org/browse/SAK-43273

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- site-manage/site-group-manager/src/main/java/org/sakaiproject/groupmanager/controller/AutoGroupsController.java  3

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- site-manage/site-group-manager/src/main/java/org/sakaiproject/groupmanager/controller/AutoGroupsController.java  1

See the complete overview on Codacy

Hi @bjones86, many thanks for your review, I've addressed the requested changes. Let me know where I can help.


Revert "Update description on docs"

Reverts flutter/flutter#14660

Closing in favour of https://github.com/flutter/flutter/pull/15355


Filters are not applied to pagination

Fixes Issue: Filters are not applied to pagination #269
Added filter params along with developer URL, so that filters are considered
for next page request.
Pull request checklist

[x] Your code contains tests relevant for code you modified
[x] You have linted and tested the project with bin/check

Thanks for the PR! Along with my comment, can you also add a test that ensures the "next" page links render correctly?

Okay

@kaushikhande, thanks again for this PR. I had a business call out that pagination is broken when filtering. So I wanted to get this out ASAP and pushed a few commits.

@joemasilotti Thank you for merging my PR. I can totally understand your urgency. I was on vacation for last 2 days. I look into it last night but was having some difficulty in completing the pagination tests. Thank you once again.


324 replace the llvm legacy pass manager with the llvm new pass manager clean

Functional testing timing:
with new PassManager
with legacy PassManager

Functional testing timing: with new PassManager with legacy PassManager

Would be even more important to check how does CPU perform (cuda as well).

Compare full sanity check from CI directly after rebase
Compare timing using omniscript benchmarks


RawGit phasing out

It seems we serve this through RawGit which will be closed in October 2019.

A similar ticket was solved by switching to https://www.jsdelivr.com/