Is there support for wi-fi only uploads

I know that Android WorkManager supports scheduling uploads for when there is a wi-fi connection. From what I gathered, the NSURLSessionConfiguration also supports restricting the uploads to wi-fi.
Will support for this be added to the plugin? Would this be more involved than adding a flag to the UploadJob and passing it all the way through to the native side?

Yes this should be a flag in dart which is picked up by the native side.

The challenge with this approach is that the configuration is built immediately when the plugin is registered:
let sessionConfiguration = URLSessionConfiguration.background(withIdentifier: Keys.backgroundSessionIdentifier)
        sessionConfiguration.httpMaximumConnectionsPerHost = maxConcurrentTasks.intValue
        sessionConfiguration.timeoutIntervalForRequest = URLSessionUploader.determineTimeout()
        self.session = URLSession(configuration: sessionConfiguration, delegate: self, delegateQueue: queue)

This is the place, where the flag would need to be respected and added to the configuration: sessionConfiguration.allowsCellularAccess = someFlag.
Since this configuration is only available in the initializer and the session is supposed to be longlived, I don't see a way to schedule individual uploads with this flag.
My idea here was that the user has a setting in the App to restrict uploads to wi-fi networks. I assumed that it would make sense to put this flag into each and every upload so that you could change this setting on the fly so to speak.
For this to work, the session would need to be rebuilt every time an upload is scheduled. That seems wasteful and unnecessary.
Some Apps require a restart in case a setting like this is changed. So maybe it would be possible to pass this flag along when constructing the FlutterUploader itself.
Or put the flag in SharedPreferences and read it from there when initializing the plugin on the Swift side.

@lyio could we have 2 sessions (wifi, wifi+cellular) and the uploader picks the right one depending on the flag?

Configuring the session for wi-fi only uploads could be achieved similarly to how the background handler is attached.
    await _platform.invokeMethod<void>('setBackgroundHandler', {
      'callbackHandle': handle,
    });

We could add a method on the channel configureSession and invoke this method after the channels have been registered with the appropriate flag.
This only applies to iOS, of course, because on Android the WorkManger takes the wi-fi restriction on a per job basis, iirc.
So this way we could delay constructing the session (or even the whole UrlSessionUploader) to when

@lyio could we have 2 sessions (wifi, wifi+cellular) and the uploader picks the right one depending on the flag?

@lyio could we have 2 sessions (wifi, wifi+cellular) and the uploader picks the right one depending on the flag?

I could give this a try. Would this impact the backchannel to the Dart side of things? We would also have two session identifiers. But the taskId is not generated based on this identifier anymore, so that would not be a problem

@lyio Start by having 2 session objects in the swift plugin and choose between them according to the upload object (which needs a new constraint parameter, or boolean flag) passed.
You may not need additional methods on Dart side & only extend the Upload object.

@lyio Start by having 2 session objects in the swift plugin and choose between them according to the upload object (which needs a new constraint parameter, or boolean flag) passed.
You may not need additional methods on Dart side & only extend the Upload object.

I have started to do just that. I will get a PR to you today. The only problem atm is that I do not have an iPhone that has cellular data. :)
My testing capabilities are limited right now.

Send something over, I'm happy to give it a spin later. Please do extend the example project with a checkbox or similar.
Thank you for looking at this issue! :-)

You have fixed it yourself. Thanks again.


Help differentiating between custom ui and hosted fields

AFAIK, hosted field is the same as custom UI or is there any difference?
Confusion stem from
https://www.braintreepayments.com/products-and-features/custom-ui
https://www.braintreepayments.com/products-and-features/custom-ui/hosted-fields

Our main two integration options are dropin and custom. Hosted Fields is a sub-feature for custom integrations.
Within custom, you can accept credit cards one of two ways. I've added links to our developer documentation, as it may be clearer than our marketing site.

Use your own form fields for the credit card inputs (not SAQ-A)

Traditional custom integration documentation

Use Hosted Fields for the credit card inputs (SAQ-A)

Hosted Fields documentation

Thank you.


Python builds using Poetry break with 20201129.1

Description
Builds in github actions have broken and the only change I can find is that the version has changed from Version: 20201115.1 to 20201129.1
Area for Triage:

Python
Question, Bug, or Feature?:

Bug
Virtual environments affected

[ ] Ubuntu 16.04
[x] Ubuntu 18.04
[x] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11.0
[ ] Windows Server 2016 R2
[ ] Windows Server 2019

I have not tried on 16.04
Expected behavior
Since the build did not break locally and python, poetry and nox all have their version pinned, I expected builds to continue to work.
Actual behavior
Builds fail with a pip and or poetry issue.
nox > poetry export --dev --format=requirements.txt --output=/tmp/tmp2k2e6e06
nox > pip install --constraint=/tmp/tmp2k2e6e06 flake8 flake8-black flake8-bugbear flake8-import-order
nox > Command pip install --constraint=/tmp/tmp2k2e6e06 flake8 flake8-black flake8-bugbear flake8-import-order failed with exit code 1:
Collecting flake8
ERROR: In --require-hashes mode, all requirements must have their versions pinned with ==. These do not:
    flake8 from https://files.pythonhosted.org/packages/d4/ca/3971802ee6251da1abead1a22831d7f4743781e2f743bd266bdd2f46c19b/flake8-3.8.4-py2.py3-none-any.whl#sha256=749dbbd6bfd0cf1318af27bf97a14e28e5ff548ef8e5b1566ccfb25a11e7c839

Repro steps
I have an issue open with poetry. My PR is here and from my PR you can of course get the details for the passing and failing builds using the Test or Coverage workflows. I have also created a Diff of a working and failing Test builds up to the point where the one fails.
The only thing I find strange about the diff is that in the build that works, it doesn't install some of the packages. For example, if you search for Collecting flake8 you would expect to find it since poetry and nox should make sure it's installed from scratch every build. Even though it doesn't seem to install flake8, flake8 runs just fine. If you go to the failing build and search you see that installing those packages is what is failing the build.
nox > Command pip install --constraint=/tmp/tmp2k2e6e06 flake8 flake8-black flake8-bugbear flake8-import-order failed with exit code 1:
Collecting flake8
ERROR: In --require-hashes mode, all requirements must have their versions pinned with ==. These do not:
    flake8 from https://files.pythonhosted.org/packages/d4/ca/3971802ee6251da1abead1a22831d7f4743781e2f743bd266bdd2f46c19b/flake8-3.8.4-py2.py3-none-any.whl#sha256=749dbbd6bfd0cf1318af27bf97a14e28e5ff548ef8e5b1566ccfb25a11e7c839

Edit:
Also of note from the dff that the failing builds on 20201129.1 complain about pip not being latest:
WARNING: You are using pip version 20.2.4; however, version 20.3.1 is available.
You should consider upgrading via the '/opt/hostedtoolcache/Python/3.8.6/x64/bin/python -m pip install --upgrade pip' command.

This warning about pip not being fully updated is not seen on the passing builds from the earlier version. Edit: I just checked and updating pip did not fix the builds. Successfully installed pip-20.3.1
Edit: Here is the diff between the working image and the newer one that is not working. https://github.com/actions/virtual-environments/compare/ubuntu18/20201115.1...ubuntu18/20201129.1
If you ctrl-f and search for unbuntu1804 you can quickly locate the readme and json files for this image version. According to the readme, the python or pip version should not have changed at all. The only python related change I could see in the readme was a change to PyPY version and it was only a patch bump.
The other change from the json shows that this script is not being run https://github.com/actions/virtual-environments/blob/ubuntu18/20201115.1/images/linux/scripts/installers/test-toolcache.sh on the newer image.
The only thing strange I found while reviewing the diffs is that the image version in the readme does not match the tag.

It should be version 20201115.1 not 20201108.1, I'm guessing this is a bug in the diff viewer? Which makes me question my findings so far, or maybe it's using a cached version of the tag and this tag was forced push to fix the version info? Now sure.

@shadycuz while i am investigating the problem, can you please provide a reference to the build that actually fails and provide a minimal repro with a bit less than 14 files PR?

@dsame I have opened a new PR https://github.com/DontShaveTheYak/cloud-radar/pull/6 and the only change is the README.
All builds for workflow "Tests"  >= 19 are failing because of version 20201129.1.
All previous builds passed but on version 20201115.1. The last build to pass was for a merge into master https://github.com/DontShaveTheYak/cloud-radar/actions/runs/388656939 .
Even though only the readme changed its build has failed https://github.com/DontShaveTheYak/cloud-radar/actions/runs/420040846 .
Please let me know if I can provide any other info or assistance.
Thanks

@shadycuz thanks a lot, not the problem seems to be clear

@dsame
Someone has tracked it down to an issue with newer pip versions.  I tried forcing pip to a pre 20.3 version but that did not work. What did work for pinning virtualenv to a different version.
See this working build. Also here is the pip issue.

I feel good enough about closing this now. The issue is with the change to how pip handles the constraints flag. pip maintainers have confirmed its not a bug but a design choice based on the new pip resolver.
Workarounds:

Stop passing in packages with pinned hashes using the pip --contstraints flag.
If you are using pip directly then pin it to a version before 20.3
python -m pip install --upgrade pip==20.2.4
If you are using something else like virtualenv that depends on pip, make sure you pin it.
python -m pip install --upgrade virtualenv==20.0.26
Or use a env var VIRTUALENV_PIP=20.2.4


GUARD_INTERVAL_1_128 not supported in tsduck, but used in DVB-T2 in the UK

I'm unsure of the significance of this, but it seems that the tsduck guard intervals are missing a few options.

Thanks for the report, this has been added to the latest commit.


Add readme-generator to release workflow

Should we make this template private? It's pretty chainguard specific now e.g. posting to our slack


Impossible to Upload to App Store on Xcode 8 GM

We are using the 4.0.0-beta2 version to develop our app.
When we try to upload it to the app store for test flight internal testing purpose, iTunes Connect tells us that - character is not allowed in version number.
We achieved to do the upload to iTunes Connect for TestFlight internal testing purpose by removing the - character.
Is it possible to change the version number to remove the - character in the version number ?
Thanks :)

beta version

It will be removed once we officially release Alamofire 4.0.0 @CedricEugeni. We're working tirelessly to wrap up everything to make AF 4 as awesome as possible! We're almost 100% code complete and just need to update the docs.
Your patience is greatly appreciated. 🍻


Prefer JDK ThreadLocalRandom implementation over ours.

Motivation:
We have our own ThreadLocalRandom implementation to support older JDKs . That said we should prefer the JDK provided when running on JDK >= 7
Modification:
Using ThreadLocalRandom implementation of the JDK when possible.
Result:
Make use of JDK implementations when possible.

4.1 (fbf0e5f) 4.0 ( see https://github.com/netty/netty/pull/6402 )


[10.x] Validate request data keys with dots and asterisks

This PR attempts to fix validation for values that have dots or asterisks in their key at the top level.
Recently I faced with a situation, where I had to validate data with dots (.) in their keys at the top level. The validator failed every time. First, I tought it's a bug, but then after taking a look at the tests (https://github.com/laravel/framework/blob/master/tests/Validation/ValidationValidatorTest.php#L5159-L5177) I realized, this is intentional.
However, in my opinion having dots or asterisks in an array key is valid and a very possible use-case, even if it's not very common. For example:
$data = ['price.eur' => 20];

 $v = new Validator($translator, $data, ['price.eur => ['required']]);
 $v->passes(); // fails, while it is present and filled

This is because the Valiadtor transforms the top level data keys that contain . or *.

So what this PR does, is just an extra check in the getValue method if there is a transformed key matching with the given attribute.
This should not be a BC, since nested arrays are still have the priority for dot notation rules (as the original tests are still passing), and handling top level keys with special characters are just a "fallback" for a not very common but a very possible scenario. However, I still submit this into master to make sure nothing breaks.

No plans to make changes to this behavior.


Update scala-collection-compat to 2.8.0

Updates org.scala-lang.modules:scala-collection-compat from 2.7.0 to 2.8.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang.modules", artifactId = "scala-collection-compat" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "org.scala-lang.modules", artifactId = "scala-collection-compat" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Codecov Report

Merging #267 (f6edc57) into main (71de420) will increase coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #267      +/-   ##
==========================================
+ Coverage   59.92%   60.01%   +0.09%
==========================================
  Files         153      153
  Lines        3244     3244
  Branches       91       91
==========================================
+ Hits         1944     1947       +3
+ Misses       1300     1297       -3

Impacted Files
Coverage Δ

...health/windows/actor/HealthSignalWindowActor.scala
93.23% <0.00%> (-0.76%)
:arrow_down:

.../surge/internal/kafka/KafkaProducerActorImpl.scala
77.87% <0.00%> (+0.60%)
:arrow_up:

.../surge/kafka/streams/KafkaStreamManagerActor.scala
55.69% <0.00%> (+2.53%)
:arrow_up:

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 b8de2a5...f6edc57. Read the comment docs.


issues of  Preparing dataset living17, mode: fine, partition: val..

because the subclass of imageNet\ILSVRC\Data\CLS-LOC\val is not dir ,so it is failed

thanke you very much!


BUG: No separate button for canteen owners to access their canteen details

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Hello Project Admin, the bug i encountered is, that after the login as canteen user the route is directly redirected to section/canteenId, there is no separate button in the Home Page Navbar to access this page.
The problem here is the canteen owner finds it inconvenient to navigate between other routes and his canteen section.
What I would like to include is, after successful login all the users are redirected to home page only and here if the user is canteen owner a separate button "My Canteen" will be shown. By clicking this he can access the route section/canteenId to find  all their canteen details.
Please assign this issue to me.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Great, Idea start working on this


Fixes most of #862

Replaces two images with one animated image
Cleans up and clarifies some text

Hi @ardalis, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

aspnetcore/fundamentals/dependency-injection.md
:white_check_mark:Succeeded

Details

aspnetcore/fundamentals/dependency-injection/_static/lifetimes.gif
:white_check_mark:Succeeded

Details

aspnetcore/fundamentals/dependency-injection/_static/lifetimes_request1.png
:white_check_mark:Succeeded

Details

aspnetcore/fundamentals/dependency-injection/_static/lifetimes_request2.png
:white_check_mark:Succeeded

Details

aspnetcore/fundamentals/startup.md
:white_check_mark:Succeeded

Details

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@Rick-Anderson quick review please? I'll hit 2068 separately.

It's recommended that you follow this convention, placing extension methods in the Microsoft.Extensions.DependencyInjection namespace, to encapsulate groups of service registrations.

Is that from the PU?
LGTM  :shipit:

PU? Product Unit?
Yes it was from one of the product team members in a related issue, I
think. Might've been Fowler.
Steve
On Fri, Nov 11, 2016 at 1:29 PM, Rick Anderson<EMAIL_ADDRESS>wrote:

It's recommended that you follow this convention, placing extension
methods in the Microsoft.Extensions.DependencyInjection namespace, to
encapsulate groups of service registrations.
Is that from the PU?
LGTM [image: :shipit:]
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aspnet/Docs/pull/2145#issuecomment-260023022, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAvvL0T81U9B6mXoJDIbxjYmn8fMrN_-ks5q9LQVgaJpZM4KvJ6T
.

--
Steve Smith
http://ardalis.com | http://twitter.com/ardalis

@ardalis  thanks, yes PU is Product Unit.  I thought so.


The following signatures were invalid: EXPKEYSIG 1BFC33752912B99C

Add stable repositories to ubuntu 18.04
# wget -O - http://nuitka.net/deb/archive.key.gpg | apt-key add -
...
# echo >/etc/apt/sources.list.d/nuitka.list "deb http://nuitka.net/deb/stable/bionic bionic main"
# apt-get update
...
Err:7 https://nuitka.net/deb/stable/bionic bionic InRelease
  The following signatures were invalid: EXPKEYSIG 1BFC33752912B99C Kay Hayen<EMAIL_ADDRESS>...
W: An error occurred during the signature verification. The repository is not updated and the previous index files will be used. GPG error: https://nuitka.net/deb/stable/bionic bionic InRelease: The following signatures were invalid: EXPKEYSIG 1BFC33752912B99C Kay Hayen<EMAIL_ADDRESS>W: Failed to fetch http://nuitka.net/deb/stable/bionic/dists/bionic/InRelease  The following signatures were invalid: EXPKEYSIG 1BFC33752912B99C Kay Hayen<EMAIL_ADDRESS>

I have updated the key for the new release, sorry for the long time this took.

Hi. Happy new year!
I'm encountering the same issue.
On https://nuitka.net/deb/ , I see the archive.key.gpg file was last modified on 2017-11-12 10:57.
Maybe you recently restored the repo server from backups and the old key reappeared? Or some sort of cache / mirror server issue?
Can you check it please? (I use apt-get install --allow-unauthenticated nuitka as workaround, though risky). Thanks.

Ah, I just noticed that it's global thing, so I was assuming the repo rebuild would contain the key, but it does not. I will update my ansible to do this automatically then, may take a bit though.

I changed the key, now need to make sure it's actually the one used to create the repo.

Ok, so the signing key id didn't really change, unsure if the updated will work with new repos, but it ought to be all good after next release now.


[flash_ctrl/sw] Remove legacy code

Also add test utility functions for bank erase and waiting for init
completion, and use those in the test ROM and other appropriate places.
Signed-off-by: Alexander Williams<EMAIL_ADDRESS>

thanks for removing the old flash code!
On Thu, Apr 14, 2022 at 9:49 PM Alex Williams @.***>
wrote:

Also add test utility functions for bank erase and waiting for init
completion, and use those in the test ROM and other appropriate places.
Signed-off-by: Alexander Williams @.***
You can view, comment on, or merge this pull request online at:
https://github.com/lowRISC/opentitan/pull/12147
Commit Summary

7278f1c
https://github.com/lowRISC/opentitan/pull/12147/commits/7278f1c5a3390e0b663a913aab1900f9fec5db90
[flash_ctrl/sw] Remove legacy code

File Changes
(29 files https://github.com/lowRISC/opentitan/pull/12147/files)

D sw/device/lib/flash_ctrl.c
https://github.com/lowRISC/opentitan/pull/12147/files#diff-fe3a49c38f70dd818755b5e5f3b97952db5a5d4c5132ad19fbdaed6904846f18
(262)
D sw/device/lib/flash_ctrl.h
https://github.com/lowRISC/opentitan/pull/12147/files#diff-a182512a4071bc94125c6838183dbfaa2912ced22e7989500e792f99b73a7534
(148)
M sw/device/lib/meson.build
https://github.com/lowRISC/opentitan/pull/12147/files#diff-6523745247acad18e03aae2949f9f8ebc9b99c05af150eaf2763117ac35bfe8d
(14)
M sw/device/lib/testing/BUILD
https://github.com/lowRISC/opentitan/pull/12147/files#diff-5d59c3aa44806ec736a592a91019c91d9dde92e54549fb333a7839cf3cb4ad9b
(1)
M sw/device/lib/testing/flash_ctrl_testutils.c
https://github.com/lowRISC/opentitan/pull/12147/files#diff-03d14c1432bf5dc148b905df7163ef31d05ae55f3bdc6d6c51590d9d95c49714
(86)
M sw/device/lib/testing/flash_ctrl_testutils.h
https://github.com/lowRISC/opentitan/pull/12147/files#diff-dbf6b6f24daee4dce927e21641d7412b44798c811bf0be6a3b6d48cf6fb7f74c
(50)
M sw/device/lib/testing/meson.build
https://github.com/lowRISC/opentitan/pull/12147/files#diff-03814366a3c14e2e4b357e9c59aa0dbef22ab438151eac6aa5aa923918c72d4d
(3)
M sw/device/lib/testing/test_rom/BUILD
https://github.com/lowRISC/opentitan/pull/12147/files#diff-17d1e8116044cdb4dba1ea43488a9973236b9d78792cd0bcd8492edaa4845095
(4)
M sw/device/lib/testing/test_rom/bootstrap.c
https://github.com/lowRISC/opentitan/pull/12147/files#diff-7a53750abff747f48fad7536e8ed48d21a38262708bacb8aac8a936841f2ebc8
(66)
M sw/device/lib/testing/test_rom/bootstrap.h
https://github.com/lowRISC/opentitan/pull/12147/files#diff-685b6885e77177dfb8fe686d2b43419d4026f82ba2448c906ca7c0b2e33582da
(5)
M sw/device/lib/testing/test_rom/meson.build
https://github.com/lowRISC/opentitan/pull/12147/files#diff-814811385acf9b4f3d3e32c66e2c6faec582a43cd13bfe1c2ff99ba987b1b581
(4)
M sw/device/lib/testing/test_rom/test_rom.c
https://github.com/lowRISC/opentitan/pull/12147/files#diff-35fb40c43ec7ca6030d6c139780fa6596ce17ef8b319ad556a424cc6e1e29326
(19)
M sw/device/silicon_creator/lib/drivers/BUILD
https://github.com/lowRISC/opentitan/pull/12147/files#diff-10d39876ea8174f79e5e764e2555d16c0f2c19d7f3d4e425d57154c90d1e87ef
(2)
M sw/device/silicon_creator/lib/drivers/keymgr_functest.c
https://github.com/lowRISC/opentitan/pull/12147/files#diff-7fcade5eb0cbaefdaf8a8b2ef9c5e4c3a8baaf2e9af156130f5b03878f7d373d
(38)
M sw/device/silicon_creator/lib/drivers/meson.build
https://github.com/lowRISC/opentitan/pull/12147/files#diff-c27a0080621eee10289aaa0d7e43266bcd804496021faada205294f2ef9b97e9
(2)
M sw/device/silicon_creator/mask_rom/BUILD
https://github.com/lowRISC/opentitan/pull/12147/files#diff-6204027cbd26714f315d3b9f9d0826a9fb128c397aafc0e5845acfef570efe3d
(4)
M sw/device/silicon_creator/mask_rom/meson.build
https://github.com/lowRISC/opentitan/pull/12147/files#diff-e9458d8a5a37b3caee3abd9173b97d8a4a45906948289ce89a1df78f1dba8343
(3)
M sw/device/silicon_creator/mask_rom/primitive_bootstrap.c
https://github.com/lowRISC/opentitan/pull/12147/files#diff-f373e89aeb0d2a1f6064b0475d49639efc14b1179accae1d17de97754bafabc1
(71)
M sw/device/tests/BUILD
https://github.com/lowRISC/opentitan/pull/12147/files#diff-923e565356e32daee8f8bbe454109866398ebb25664edfa81a9f6aede76eb297
(5)
M sw/device/tests/flash_ctrl_idle_low_power_test.c
https://github.com/lowRISC/opentitan/pull/12147/files#diff-f6e44441f582bdf3855ce54214e918c0d149bce3c2be27fa92f700a379e7bb25
(4)
M sw/device/tests/flash_ctrl_ops_test.c
https://github.com/lowRISC/opentitan/pull/12147/files#diff-8f3b43a5bdacc11a3415b271bed20ed3a6be929a7fb961e477e2f747edea077b
(28)
M sw/device/tests/flash_ctrl_test.c
https://github.com/lowRISC/opentitan/pull/12147/files#diff-b805d63b160f72bbf5694a45c9022e80d6016bc3cf99f36ccf784605657e00d0
(135)
M sw/device/tests/hmac_smoketest.c
https://github.com/lowRISC/opentitan/pull/12147/files#diff-0916327e34288aab677b64e3d519367b2e979da60fed53beb0d813ce7c9c9eb3
(1)
M sw/device/tests/kmac_mode_cshake_test.c
https://github.com/lowRISC/opentitan/pull/12147/files#diff-3bc8e1d560fc06eeed567c7c8be3f3fd51e60cf112afe18f30e56de841820633
(1)
M sw/device/tests/kmac_mode_kmac_test.c
https://github.com/lowRISC/opentitan/pull/12147/files#diff-316bc63fc8a0f0cee89c0de5500e4fe9f7566a89d1db1dc8c4a3ed27b2953ea8
(1)
M sw/device/tests/meson.build
https://github.com/lowRISC/opentitan/pull/12147/files#diff-cf4f6736c512baad9ccfdcc6eed0c519dd1bdc1068ed37656fc959e9c703d6fb
(11)
M sw/device/tests/sim_dv/flash_ctrl_lc_rw_en_test.c
https://github.com/lowRISC/opentitan/pull/12147/files#diff-03c1ad7f42987f047ee8aaaa522de9b6d2ed1f9ffee12c047e6bfce7934a6792
(6)
M sw/device/tests/sim_dv/flash_rma_unlocked_test.c
https://github.com/lowRISC/opentitan/pull/12147/files#diff-b896d363d895e9c59699c763976edc74fa685c7beced60dcc145f9cb895a581e
(4)
M sw/device/tests/sim_dv/keymgr_key_derivation.c
https://github.com/lowRISC/opentitan/pull/12147/files#diff-0b2d8dbceeeae4234d46ba27dedb9d2c7b66610f520c7f55abad230dddfef732
(6)

Patch Links:

https://github.com/lowRISC/opentitan/pull/12147.patch
https://github.com/lowRISC/opentitan/pull/12147.diff

—
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/12147, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSXP2J5LVXRY6UTCVXLVFDYOFANCNFSM5TPSBKWA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Oops, looks like I need to debug what I did for the bootstrap code.

Oops, looks like I need to debug what I did for the bootstrap code.

This is done--Had a mistake in handling the address for bank erase. The FPGA now bootstraps and runs tests successfully.

Thanks @timothytrippel. LGTM!


docs: improve translation

Fix typos & propose translation changes & translate a sentence

@asantarissy Thank you!


Thanks!

Sorry, not an issue. Just wanted to say thank you for making an awesome CLI watcher. =)

Thanks for the feedback! I appreciate it :)

:100:


Support OpenSimplex

Will you consider adding this implementation to the group ?
Thank you very much for this package !

@Solido I will implement open simplex noise and create a package for it 👍

Oh very good !
I tinkered myself a basic version of V1 but having the full V2 would be the goto solution


[MM-27213] Cypress/E2E: Automate backlogs - Auth - Part 01 (10 test cases)

Summary

Cypress/E2E: Automate backlogs - Auth - Part 01 (10 test cases)
Ticket Link

https://mattermost.atlassian.net/browse/MM-27213

@hahmadia By the way, I wouldn't worry too much about the MFA failures as those are a bigger issue in the code base generally. I would just focus on items 1 and 2. Sorry for the confusion if ever.

@josephbaylon
Thanks @josephbaylon for super duper help! Cleaned up as per suggestions.

/update-branch

@furqanmlk I've updated it to fit the standard you posted. I did my best at converting as much as I could.


Include fn skeleton of traits in impl

Would it be possible to enhance impl so that it includes the fn's from the trait your implementing?  For instance consider this:
pub trait FooPrimitive {
	fn read<R: Read>(cursor: &mut R) -> Self;
	fn write(&self) -> Vec<u8>;
}
impl FooPrimitive for String{
	fn read<R: Read>(cursor: &mut R) -> Self{

    }
	fn write(&self) -> Vec<u8>{

    }
}

Now if i type impl and fill in FooPrimitive it would be nice if it copied these fn's automatically for me so I don't have to keep copy pasting them.  :)

That's a great idea and would make writing impls much less work. But I don't think this is possible with normal snippets in Atom. Maybe it's possible to write plugin code for it, but unfortunately I don't know enough about Atom plugins to do that.
Any help from JS or Atom experts appreciated


Is there a way to disable some/all the shortcuts ?

In all cases, can you explain where they come from ? I spend some time in the code but only found an eventListener for the 'F' shortcut 🤔

I finally found it, there is a boolean prop that you can pass to disable accessibility listeners :
<Carousel frameProps={{ accessibility: false }}  />


Enabling Reverse Input Button Breaks SOCD

Prerequisites
Please check the following before posting an issue / bug report.

[x] I am running the latest version of the firmware found HERE
[x] I have checked the documentation HERE and found no answer
[x] I checked to make sure that this issue has not already been filed HERE
[x] I am reporting the issue to the correct repository

Context
Please provide all relevant information about your setup. This is important in case the issue is not reproducible except for under certain conditions.

Firmware Version: GP2040-CE : v0.7.7
Name of device: SallyBox, Flatbox R1, etc.
Link to where you purchased the device: https://fightboxarcade.com/collections/fightbox-r-game-controllers / https://www.aliexpress.us/item/3256805359350735.html
Is this a custom build?: No
Operating System: Windows 11
Browser: Firefox / Chrome

Expected Behavior
When turning on the addon for Input Reverse, pressing the button should reverse the input as defined in the reverse input rules. Additionally, existing functionality should not be impacted when this button/mode is active.
Current Behavior
When the addon for Input Reverse is enabled, all SOCD modes break. All buttons can be held down at any time without SOCD cleaning. This is irrespective of the Input Reverse button being pressed.
Steps to Reproduce
Please provide detailed steps for reproducing the issue.

Set SOCD cleaning mode to any mode other than "Off"
Reboot device and see that SOCD presses are properly handled
Enable a button for Input Reverse
Reboot device and see that SOCD presses are not properly handled

Thank you for the report @tomatosoupcan ,
We are aware of this issue.


Breaking the pyqg dependency

The utils.py currently depends on the pyqg repository, but Laure says she would like this equation discovery repository to be completely independent of pyqg .

Is dropping the pyqg  dependency definitely what we want to achieve?
How can we refactor/redesign this repository such that it is standalone with no pyqg  dependency?

Yes, it would be great to removepyqg dependency so that users can feed in their data without having to install pyqg which is a numerical solver.

My take on this is that we should support pyqg and require it for certain things (e.g. computing similarity metrics that are specifically designed for pyqg.Models), but we shouldn't require it for the entire library, since much of the code (e.g. the actual hybrid symbolic procedure) can operate just as easily on an xarray.Dataset.
To accomplish that, I think we'll need to do something like:

try importing pyqg and store a boolean indicating whether the import succeeded
replace existing type checks for pyqg.Model with something like is_pyqg_model that returns isinstance(pyqg.Model) if the import succeeded and False if the import failed

Does that make sense? I think that's a better approach than removing all references to pyqg altogether, since then it will become very difficult to run the library with pyqg models.

Thanks @asross; i like the idea. I feel as long as one can use the hybrid symbolic procedure without pyqg  that would be really great (including calculating the derivatives).


Library saving dates ignoring timezone application

Laravel-mongodb Version: #.#.#
PHP Version: 7.4.27
Database Driver & Version: 1.12.0

Description:
Steps to reproduce

set date_default_timezone_set("America/Sao_Paulo")
record in database one register with datetime
check record registred timezone america +3 not America/Sao_Paulo

Expected behaviour
Record registred with timezone configured from system
Actual behaviour
Record registred with timezone diferente from timezone configured from system
Logs:
Insert log.txt here (if necessary)

Hi
thanks for response.
Atenciosamente,
Em ter., 10 de mai. de 2022 às 16:33, joaovitoralvares <
@.***> escreveu:

@ceresaconsultoria https://github.com/ceresaconsultoria Dates in MongDB
are always stored in UTC.
When you set the timezone to America/Sao_Paulo, it will be converted to
UTC before the record is saved.
For instance, if you store a record with date 2022-05-10 16:28:00 (
America/Sao_Paulo -03:00), the date stored in database will be 2022-05-10
19:28:00 (UTC +00:00)
—
Reply to this email directly, view it on GitHub
https://github.com/jenssegers/laravel-mongodb/issues/2338#issuecomment-1122784318,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANMHJ2FAQUZKGIKDNFJIA23VJK2X5ANCNFSM5MYHUFFQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Track session states

Summary

track session states
allow user to provide custom function to handle session state transfers

Description

Additional Reviewers
@karenc-bq
@davecramer
@hsuamz
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

thanks for this -- we will be using ConnectionProviderManager.setConnectionInitFunc() from #705 to initialize new connections that were not able to go through PoolBase.setupConnection()
Regarding SessionStateTransferCallable, RestoreSessionStateCallable -
The default behavior for SessionStateTransferCallable is to propagate all properties from one connection to the other, including transaction isolation -- so this new default behavior may actually be sufficient.
But the default behavior for RestoreSessionStateCallable is only to restore a locally-saved readonly and autocommit property.
What is the reason for the limited number of properties restored by RestoreSessionStateCallable compared to SessionStateTransferCallable?  Should we consider a RestoreSessionStateCallable event similar to ConnectionProviderManager.setConnectionInitFunc() and just reset all properties from the ground up?
thanks for any insight


chore: Fill out Cohorts page data (WEB-52)

Task
Add appropriate data and links to Cohorts page.

[ ] Fill out data for the various cohorts (by group)

p.s. this might be a scavenger hunt.

Steps to contribute. NEVER COMMIT TO MAIN. NEVER COMMIT TO MAIN. It should always be on your own branch and that branch name should match the issue. Ensure that you are addressing the correct issue. Each issue will be listed as something like a chore, fix, bug, etc. Your commit message MUST start with one of these tags that correspond with your issue. Followed by your commit message and your commit message must end with the branch name, for example (WEB-1)
An example commit message would look like this: chore: create contributing guidelines (WEB-8)
Commit messages should not be written in an action or present tense. It also should not mention tasks unrelated to the issue "Creating Contributing Guidelines and will work on next task" is not what we would want to see as a commit message.
After a pull request is approved, the branch should be deleted once merged.

Can we open this back up? I did not realize that tagging it in a PR would close. This is a task that still needs to be done.


Add animation + focus on input bar when hitting "new conversation"

needs to be attarcting the eye when you click

We already focus the input bar. What animation?

After investigating we loose the focus if the user is already in new conversation and click again new conversation

I believe this is done. Closing


Building Kubernetes page is not beginner friendly

The Building Kubernetes page is not beginner friendly.
I have been trying to make a containerised build work on my Local Mac laptop for 2 days without success.
The following items need better explaining in my opinion

Pre-requisites:
For the Docker Desktop, it is not sufficient to mention a requirement of 8 GB memory for the Docker VM.. There should be a mention for minimum CPU and Storage allocation to the Docker VM.
Example: My Docker VM had 60 GB of storage allocated to it and the unit test executions failed with No space left on device errors.
I increased it to 120 GB and I was able to overcome the space issue, but I have no idea if 120 GB is recommended or not.
This should be specified in the document.

Key scripts:
This section describes the scripts available and what each script does.
However, for a reader who is a beginner, it does not mention the order in which the scripts should be executed or tried.
Example:
If my goal is to run all the unit tests on my local machine, it is enough to directly run build/run.sh make test?
Is there any pre-requisite to maybe run build/run.sh make first?

There should be a clear step-by-step instruction describing the order in which these scripts should be run for a beginner who has no prior experience in this.
The reader should not have to guess this.

There is no troubleshooting section:
Some frequently occurring errors may be documented as a troubleshooting section.
When the unit tests fail, a beginner has no idea where to look for the execution logs or how to troubleshoot the issue.

These points are hampering the experience for a developer who is trying to get his/her local build up and running.

The points for 1) are a moving target as the project evolves and many active contributors are building on linux instead, PRs welcome!

If my goal is to run all the unit tests on my local machine, it is enough to directly run build/run.sh make test?
Is there any pre-requisite to maybe run build/run.sh make first?

No, makefiles can encode ordering requirements (one of the main uses!) it would be unusual to require a manual ordering of make targets.

When the unit tests fail, a beginner has no idea where to look for the execution logs or how to troubleshoot the issue.

Debugging unit tests isn't exactly in scope for this particular document.


More precise varnames config

List of related issues or pull requests
Refs: -
Describe the changes made in this pull request
In this PR, I updated the variable names in Repo to be more precise. In particular,

I changed *_file to *_filename to indicate these variables are strings, not File objects or something.
I prepended any *_config_filename and *_config variables with prefixes [default|repo|user] indicating where each configuration originates.

Instructions to review the pull request

woohoo making progress


v3.0.0 - previewData and path returning undefined

Hey,
I've just upgraded to 3.0.0 and I'm trying to implement Previews on my site following the guide here: https://github.com/angeloashmore/gatsby-source-prismic/blob/master/docs/previews-guide.md
I've copy and pasted the example code into my site, and the only thing I've changed in my preview.js file is the repositoryName. When I then try to preview a post in Prismic, my localhost preview opens as expected, but redirects to http://localhost:8000. I tried looking into the data returned and when I console.log(isPreview) it first returns undefined and then returns true.
But both console.log(previewData) and console.log(path)  return undefined.
I don't have any linkResolvers or pathResolvers set up on my site, but I've tried implementing these to see if it would fix the issue but my previewData and path still return undefined.
Let me know if I can provide anymore information.
Thanks!

I had the issue, try to test the data in useEffect before navigate
if (isPreview === false        || isPreview === undefined       || previewData === undefined        || path === undefined) return

Hi @kieranstartup, sorry for the late reply. @ahmedghazi is correct, you will only want to navigate if isPreview is true and isLoading is false (I just realized we need to update that example in the guide to reflect this). You don't necessarily need to check that previewData and path have values, however.
It's likely that path is undefined in your case since you aren't providing a link resolver. You should be giving the same link resolver to both gatsby-node.js and usePrismicPreview.

Closing this out to clean up issues but feel free to continue this discussion if you're still having trouble.

Hey @angeloashmore, sorry for my delay in getting back to you. I've just tested out @ahmedghazi fix above, and implemented a linkResolver in my Preview page and everything is working correctly now.
Thanks for your help!

Awesome!


Upstreaming plugin repository (2023 Q2)

Summary
(Please add a short summary why your pull request is useful)
Target Platform For Release Notes (Mandatory)
Aids the generation of release notes

[ ] VPUX3011
[ ] VPUX3011
[ ] VPUX3110
[ ] VPUX37XX
[ ] NONE (Not included in release notes)

Classification of this Pull Request

[ ] Maintenance
[ ] BUG
[ ] Feature

Related PRs
(Please add links to related PRs (if you have such PRs) and a small note why you depend on it)

 ()

Related tickets
(Please list tickets which the PR closes if you have any)

E#####

Code Review Survey (Copy and Complete in your code review)

number_minutes_spent_on_review[0]
number_p1_defects_found[0]
number_p2_defects_found[0]
number_p3_defects_found[0]
number_p4_defects_found[0]

@Maxim-Doronin hi!
Could you please take a look?


made pydantic change

[x] I understand that this repository is auto-generated and my pull request may not be merged

Changes being requested
In the model_dump_json method of the BaseModel class on line 365 the if statement was essentially set to always throw the ValueError if you have done what you are supposed to. I changed the "if warnings != True: -> raise ValueError" to "if warnings == True: -> raise ValueError" so now the code will properly handle when you have pydantic v1 and set your warnings to False.
Additional context & links

Thanks for the PR but I'm sorry I don't quite understand the change here, this means that when you call .model_dump_json() you'll get an error and you can't set warnings to False in Pydantic v1 in the first place?

Thanks for the PR but I'm sorry I don't quite understand the change here, this means that when you call .model_dump_json() you'll get an error and you can't set warnings to False in Pydantic v1 in the first place?

Thanks for the reply!
Sorry I was not as clear as I thought, this might be a bigger problem than I had originally looked at. So here is what was happening:
The Repo I am working on is not able to upgrade to Pydantic-V2.
When I am using the AsyncAssistantEventHelper it tries to call the "accumulate_event()" function which if it hits the IndexError tries to call model_dump with warnings set to False.
except IndexError:                 current_message_snapshot.content.insert(                     content_delta.index,                     cast(                         MessageContent,                         construct_type(                             # mypy doesn't allow Content for some reason                             type_=cast(Any, MessageContent),                             value=model_dump(content_delta, exclude_unset=True, warnings=False),                         ),                     ),                 )
Since "warnings is only supported in Pydantic V2" This would be correct I would want warnings to be set to False. The problem is the "if warnings != True:", since warning is False, raises the ValueError telling me that warnings cannot be true when I have Pydantic V1. So I changed that line so that if Pydantic is not V2 and warnings is True then raise the error about not being able to set warnings to True with Pydantic V1.
After I made that change my code worked as intended and I no longer had issues. I just think that the statement is backwards because that if statement only runs if Pydantic is not V2 then only raises an error if warnings is set to not set True. Shouldn't it raise the error when the user is trying to use warnings in Pydantic V1?

What openai version were you using? If I remember correctly that was a bug that has since been fixed, can you try again with the latest version?

What openai version were you using? If I remember correctly that was a bug that has since been fixed, can you try again with the latest version?

I just updated to version 1.55.0 then ran the functions, and I got the same errors as before. So then I went back into _models.py and changed that if statement back to == True instead of != True. Which resolved the error again

oh, can you open an issue with a minimal reproduction?

oh, can you open an issue with a minimal reproduction?

I created that issue, here is the link: https://github.com/openai/openai-python/issues/1882#issue-2680436051

thanks, closing this in favour of a different fix!


Could not add key because queue is full

i'm replicating data from Elasticache to Redis Enterprise cloud using riot-redis live mode and got this error
Could not add key because queue is full
what does this mean? i still have enough memory and ops on my RE database and see no latency issue too.

This could be because updates on the source database are happening faster than RIOT can write them to the target. This situation is described in the documentation: https://developer.redis.com/riot/riot-redis/index.html#live-replication
What data structures are in use in the source database? Are any collection data structures (list, set, sorted set, ...) updated repeatedly at a high frequency?


[2.2.0rc1] bootstrap modal dialog

When I use autocomplete input with bootstrap modal dialog, choices are displayed behind the dialog.
However, it is just fine in previous release (2.1.1).

With a PR it's always better
Thx for the report we want this fixed in next release however even a PR to
reproduce in the test project would help A LOT
Le 26 mai 2015 20:08, "Tatiana"<EMAIL_ADDRESS>a écrit :

When I use autocomplete input with bootstrap modal dialog, choices are
displayed behind the dialog.
However, it is just fine in previous release (2.1.1).
—
Reply to this email directly or view it on GitHub
https://github.com/yourlabs/django-autocomplete-light/issues/426.

Ok so when I see an issue with only one line of description like this, 0 detail, 0 reserch, I want to say: I don't care about bootstrap modal, it's not supported by Django like the Admin is.
If you want this to be supported, then you'll have to help with PRs.

Try changing z-index.

I've traced this issue to the 61e54ee. The cause is new version of fixPosition function in autocomplete.js. The old one appended span with choices just after the input. The new function appends this span at the end of the body section. It looks ok in most cases but it's not when you use autocomplete in z-ordered elements like dialogs.

Oh, and yes, for modal dialog it is rendered ok when z-index > 1050, but I think this is not the thing to be hardcoded in css because we don't know what maximum z-index could be enough.

Ok so what if we set z-index of the autocomplete to be equal to the one calculated for the input, would that work ?

The z-index may be set not for the input but for one of its parents. If you get input's z-index, it will get you "auto" so this won't help.

Could you try with fix/zindex branch ?
I don't have a bootstrap project with an autocomplete in a modal window, there is none in test_project, I don't have time to make one for the moment, maybe after Djangocon. So your help is welcome on this.

Ok, with this fix it works in Firefox. But I've discovered that Chrome and Opera adds some random 0 z-indexes in addition to "auto" values. So this code will do the trick:
var zIndex = this.input.parents().filter(function() {
    return $(this).css('z-index') != 'auto' && $(this).css('z-index') != 0;
}).first().css('z-index');

Didn't see your reply and was about to ping you about this heh
Pretty awesome, I'm packing that with a couple of bugfixes in rc3 ASAP.

Please could you try with 2.2.0rc4 ? I've included your fix and credited you in CHANGELOG.
Thanks

The problem with z-index is fixed with 2.2.0rc4. Thank you for your time.
I added bootstrap_modal example in my fork https://github.com/lucky-user/django-autocomplete-light
During the tests I found another issue #431 related to the absolute positioning.

Yes, thanks for the example !

I'm having this problem with jquery ui dialog. I can't see autocomplete options when writing things on the autocomplete input. Nevertheless, if i hit "enter" i get the expected result (one option selected and correctly displayed instead of the text input). I'm using django-autocomplete-light==2.2.3, jquery-ui==1.9.2, and jquery==1.10.2.
I tried using django-autocomplete-light==2.1.1 and it works fine. Do you know what the problem could be?

The problem with the way 2.1.1 works is that it's not supported in some cases in django admin.
Ideally, we'd support both methods, using the overflow: hidden method for django admin and using the clearfix method for modern css integration.
Pull requests would be welcome.

I don't understand: is this still an issue in versions <= 2.2? If it is, is there any open issue for this bug?

Details in the CHANGELOG would help ?

There is a comment in CHANGELOG that says:

2.2.0rc3

#429: handle required=True in form fields again, regression introduced
in rc1's python BC break.
#426: handle z-index since we're absolutely-positioning in 2.2.x

What i understand is that this bug was fixed in version 2.2.0rc3. But with version 2.2.3 i'm still getting this bug. So i don't understand if it is this bug that wasn't fixed or if it is a problem with my code.

I was talking about:
2.2.0rc1
CSS BREAK

We've moved back to pre-1.1.10 CSS positioning. This means appending the
autocomplete box to an arbitrary DOM element (body by default) and using
calculating the ``top`` and ``bottom`` attribute in javascript with
``yourlabs.Autocomplete.fixPosition()`` pretty much like Django admin's
calendar widget does. While blunt, this change should help the widget

being
more compatible across Django admin themes.
While this positioning system has been used since around 2005 in Django
when Adrian Holovaty open sourced admin media in commit dd5320d, it has
never been documented that's it's a good system that works well and

there's
no reason to break backward compatibility in Django admin for that
- note to Django admin template customizers.

Note, there might be more info, please consider reading the whole
2.2.0rc1 CHANGELOG :)


Memory improvements in routes.rb

The primary change is to consolidate common routes with constraints.  Rails' route loader is very expensive as you add individual match elements.  By consolidating all of the actions for a controller into a single match entry, and using a regex to validate the action, we can still whitelist, but eliminate an enormous number of entries in Rails' route loader.
With a simpler rails runner process

Memory
Time
Route entries

Before
187_430_229 MB
6.838s
3798

After
166_028_629 MB
6.142s
656

Savings
~21 MB (11.4%)
0.7s (10.2%)
3142 (82.7%)

Additionally

I moved the routes file into a YAML file.  Although this doesn't reflect much in memory savings, it eliminates a great number of Ruby VM entries for the strings, which impacts GC times.
I cleaned up some of the API constants
Added routing to an optional id for the index route.  See https://github.com/ManageIQ/manageiq/pull/12895#issuecomment-263620362

@jrafanie Please review.

cc @gtanzillo

cc @martinpovolny @dclarizio I wanted to just ping you because I am changing the routes file significantly.  It is still just as easy to add routes as you did before though because I tried very hard to make the YAML file nearly match the Ruby code.

cc @abellotti for the API changes.  Additionally, in looking at rake routes, I feel like there's a way to massively consolidate that section as well.  I think if we use the conventional Rails actions instead of custom ones, then we can use the regular resources call, and significantly eliminate a number of entries.  I wasn't sure if it was well tested or not, so I didn't attempt it, but I'd like to sit with you and understand some of the routes there.

@martinpovolny @dclarizio Added one more change to fix the failing spec that ends up being n enhancement.
The test was failing because it was expecting host/index/1, but was getting host?id=1.  Technically those are equivalent, so a simple fix would have been to just update the spec.  This occurs because there is no longer an explicit route to index, so Rails chose the default / route instead.   However, neither is a "standard" restful routes way of accessing the record.  So I changed the default / route to have an optional id.  This allows the url_for method to generate host/1, which is better than both of the above.

Wow, great work @Fryguy !

@jrafanie Wait on merge...I'm seeing the following deprecation warning which I didn't see previously...
DEPRECATION WARNING: Using a dynamic :action segment in a route is deprecated and will be removed in Rails 5.1. (called from block (2 levels) in <top (required)> at /Users/jfrey/dev/manageiq/config/routes.rb:86)

Opened https://github.com/rails/rails/issues/27231 to see what Rails core thinks.

@Fryguy should this be moved to https://github.com/ManageIQ/manageiq-ui-classic

@chessbyte Yes, it should, though there may be things that have to change for the API routes as well.

Closing, we can tackle this later if it's still something we want to try.


Stable for production?

Hi,
I'm wondering if this library is considered stable for productive use for simple image scaling / cropping tasks? I need some image processing on azure web apps and this seems the only choice I could find right now...
Greetings

I have been working with lwip and azure websites (iisnode) for awhile. I can't get<EMAIL_ADDRESS>to work, I do have<EMAIL_ADDRESS>working however. You need to build the 32-bit version, since azure only runs Node 32-bit (unless you are packaging your own Node versions). As far as getting<EMAIL_ADDRESS>to work, make sure you're building with VS >= 2013, npm i<EMAIL_ADDRESS>--msvs_version=2013. Pray that the install command works, otherwise you're in for a rough time.

We're using it at Sportradar in production to generate cards: https://twitter.com/sportradarcards


Update price for iBTC

Please update price for iBTC

Version test: https://file.cdmteck.com/s/4oimk9zDJ8R5Aai/download?path=%2F&files=./922-merge-build-0.7.4-1670929845322.zip
Validation passed!


Release a new version

camaleon-mandrill-plugin just released 0.0.4 (https://github.com/raulanatol/camaleon-mandrill-plugin/issues/1) which requires the change in #488 to work. It would be good to have a new camaleon-cms release as a result.

Released 2.3.4!


The silent query parameter is not documented for some endpoints

In the Document HTTP API, there are a few endpoints that accept the silent query parameter but it's not documented.

PUT /_api/document/{collection} (replaces multiple documents)
PATCH /_api/document/{collection} (updates multiple documents)
DELETE /_api/document/{collection} (removes multiple documents)

Found in ArangoDB docs stable 3.6.
https://www.arangodb.com/docs/stable/http/document-working-with-documents.html
I would expect the "Query Parameters" section of those endpoints to include the silent parameter, as it is done for other endpoints, e.g. POST /_api/document/{collection}#multiple (creates multiple documents).

silent (optional): If set to true, an empty object will be returned as response. No meta-data will be returned for the created document. This option can be used to save some network traffic.

Thanks, we are looking into this!


fix: trying to get headers after page is closed

This change helps to fix https://github.com/valendres/playwright-msw/issues/26 in my case. But I don't really know why it fixes it so feel free to close this PR if it actually doesn't fix anything.

Thanks for making a PR to address this @lesha1201. My apologies on the delay for reviewing it, it's been a super busy week. Between preparing to move countries and finishing up some things at work before my last day, I haven't found time to actually look at open source stuff 😞
It makes sense this would fix the issue. From what I understand, when the playwright test finishes executing, the browser context is immediately closed. I suspect if there was a network call right at the end of the test, it might be possible for the promise to not have had a chance to resolve in time. Adding the await here makes sense. I'm actually not sure why I chose to not await it in the past, seems like a bad decision now 😅

@valendres No problems. Thank you for your work.
The interesting part is that, from Playwright source code, it should wait callback for page.route even without returning Promise because it internally wraps it into a promise which is resolved once route.fullfill or others are called. So that's why I don't really understand why it fixes in my case since route.fullfill and route.continue are properly called in playwright-msw (both for onUnhandledRequest and onMockedResponse though not for onPassthroughResponse but even handling that doesn't fix it). Maybe there are some other async code which is executed after the context is closed.


Resolve out-of-space issue in azure pipelines CI

Summary

Attempt to resolve 'out-of-space' issue in azure pipelines CI (#214)
Sets 'clean' step to 'true' in azure-pipelines

@white238 resolved this in #212 by disabling the gcc-6 build.
We might have found (part of) the problem in #216.


fossil: 2.14 -> 2.15.1

Motivation for this change
New version available
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 120700 at fdcb1279 run on x86_64-linux 1

1 package built successfully:

fossil

2 suggestions:

warning: build-tools-in-build-inputs
which is a build tool so it likely goes to nativeBuildInputs, not buildInputs.
Near pkgs/applications/version-management/fossil/default.nix:31:3:
   |
31 |   buildInputs = [ zlib openssl readline sqlite which ed ]
   |   ^

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/applications/version-management/fossil/default.nix:47:3:
   |
47 |   installPhase = ''
   |   ^


How to run dat inside kubernetes

Is anyone successfully running dat from inside kubernetes? The following Deployment is currently working for me, but it would be nice to avoid requiring hostNetwork: true. Also wondering how to best specify the .dat keys. I am currently sharing them from the host which works all right in this scenario, but isn't a very general solution. Also, the node this container is running on essentially has a public ip which might be making this easier. I haven't yet tested it from inside kubernetes on a node without a public ip.
apiVersion: apps/v1
kind: Deployment
metadata:
  name: dat-server
  labels:
    app: dat-server
spec:
  replicas: 1
  selector:
    matchLabels:
      app: dat-server
  template:
    metadata:
      labels:
        app: dat-server
    spec:
      volumes:
        - name: nfs
          hostPath:
            path: /mnt/nfs
            type: Directory
        - name: dat-key
          hostPath:
            path: /root/.dat
            type: Directory
        # kubernetes secrets can't have subdirectories so this doesn't work
        # - name: dat-key
        #   secret:
        #     secretName: dat-key
      containers:
      - name: dat-server
        image: registry/dat:latest
        command:
          - "dat"
          - "share"
          - "/mnt/nfs/"
          # - "sleep"
          # - "300"
        volumeMounts:
          - name: nfs
            mountPath: /mnt/nfs
          - name: dat-key
            mountPath: /root/.dat
      # this gets dat network access, otherwise i get symmetric nat errors from dat doctor
      hostNetwork: true
      # sort of a hack, run this only on the node running the nfs server
      nodeSelector:
        nfs: server
# ---
# # I was still unable to access dat with this service
# kind: Service
# apiVersion: v1
# metadata:
#   name: dat-server
# spec:
#   type: NodePort
#   selector:
#     app: dat-server
#   ports:
#   - name: dat-tcp
#     protocol: TCP
#     nodePort: 30282
#     port: 30282
#     targetPort: 30282
#   - name: dat-udp
#     protocol: UDP
#     nodePort: 30282
#     port: 30282
#     targetPort: 30282

I think @ninabreznik and @jam10o were messing around with getting it to work in docker. I'd assume that it'll be hard to get incoming connections to work (or the DHT?), but I don't have hands on experience with it myself.

Thanks, btw I've tested the above solution on nodes that dont have public ips with success. Would still like avoid hostNetwork: true, but it is running.

@dwiel that's great. Can you share your dockerfile to see your configuration and hopefully make it run too?
Thanks @RangerMauve for a ping!

The dockerfile is very simple:
FROM node:11

# https://stackoverflow.com/a/45505787/2093984
RUN npm -g config set user root

RUN npm cache clean --force

RUN npm install -g dat

From another thread here i got the flags for running inside of docker (not kubernetes):  -p 3282:3282/udp -p 3282:3282/tcp

It looks like you are missing udp port 3282. You have to open udp ports independently from tcp ports.

It looks like you are missing udp port 3282. You have to open udp ports independently from tcp ports.

Like this? https://github.com/ninabreznik/ethplay/blob/master/Dockerfile
I tried this and then ran it on Digital Ocean with a command -p 3282:3282/tcp -p 3282:3282/udp
but sadly still nothing :/

Hmm, that looks right. Have you tried running dat doctor from inside the container? That helped me debug my setup.

The problem is I am running a hypertrie and afaik dat doctor doesn't work
with the hypertrie, so I am not sure how to debug this...
On Thu, Oct 31, 2019 at 11:29 AM Zach Dwiel<EMAIL_ADDRESS>wrote:

Hmm, that looks right. Have you tried running dat doctor from inside the
container? That helped me debug my setup.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/datproject/dat/issues/1158?email_source=notifications&email_token=AAIGZSY6XXG4CQDA646UCYDQRJNJ5A5CNFSM4JEWDLFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECWQ4UY#issuecomment-548212307,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIGZS6FHIOTSIQCA5HEDD3QRJNJ5ANCNFSM4JEWDLFA
.

--

Nina Breznik, Vision Baker
play.ethereum.org
wizardamigos.com
Twitter: @ninabreznik
Telegram: +1 510 747 84 39
Gitter: gitter.im/ninabreznik

This is interesting. @ninabreznik you can try add DEBUG=discovery-swarm-web-server to your npm discovery script, so it would be something like this:
"scripts": {
  "start": "node app.js",
  "discovery": "DEBUG=discovery-swarm-web-server discovery-swarm-web"
}

If everything is starting OK, the debug should output any connection error. Maybe that could help to debug.

@dpaez That's cool, thanks. Now, I see that there is another problem. And that's the fact that discovery-swarm-web doesn't support hypertrie... Any idea how to fix this? Or how to use hyperswarm in the browser?

@ninabreznik I haven't used hypertrie directly nor in the browser yet. But I give it a look to the discovery-swarm-web dependencies and seems like hyperdiscovery needs an update to be compatible with the latest hypercore-protocol used by hypertrie. Withouth that discovery-swarm-web wont work with hypertries directly. That's my guess.

Thanks @dpaez, looks like something @RangerMauve mentioned too. He'll try to help out and we'll see what needs to be done and how to fix it :)
Thanks a lot for helping out. Will share my findings once I make it work :dancing_women:


test_api_retrying started failing with requests==2.30.0

Tests were passing with 2.29.0. I haven't looked into this yet but I wanted to get it written down.
Fails are TypeError: 'Mock' object is not iterable and TypeError: expected string or bytes-like object, got 'Mock' so it's probably going to be solvable within the test suite (rather than requiring a bug fix within the library).

Looks like they started supporting urllib3 2.0, so the simplest solution is probably pinning urllib<2 (and I confirmed the test suite passes with urllib3==1.26.15). I don't yet understand why that fixes the problem, so I'm not sure it's correct.


Swift 4

Any updates on the progress to update this for Swift 4?
ActionKit.swift:118:30: Instance method 'append' expects a single parameter of type '(String, ActionKitClosure)'
gestureArr.append(name, closure) should be gestureArr.append((name, closure))

I'm not sure about the second part of your issue.
As far as Swift 4, I'm happy to accept a PR to support Swift 4, but since Xcode9 fully supports Swift 3, I don't see a need for it right away.

Turns out @brandons opened a PR, and I happily just merged it for ActionKit version 2.1
Swift 3.2 branch incoming, for backwards compatibility should that be desired. I hope this answers your issue, and I will be closing this now, please feel free to open another one if you have any issues!
Thanks again for reporting this @Gerst20051


Need video on this.

Your gonna need a video on this as i have no idea what this plugin does.

I think you're right. I'll do it tomorrow.

video was added ✌😅

video was added ✌😅


Publish to crates.io

TODO:

[x] Merge #10 (v0.3.0) into main
[x] Update documentation (README.md) on how to use this crate
[x] Check tutorial
[ ] Mark as new release: v0.3.1

Prepare release v03.1 in #26


docs(page-header): use ellipsis icon in template instead of text '...'

Sorry for that but text is expected since the icon is too strong for the button compare with text stroke.

Sorry for that but text is expected since the icon is too strong for the button compare with text stroke.

I agree text looks better but I figured the icon was expected since the EllipsisHorizontal is imported but not used in the template. Maybe remove the import statement?


ArcedPolyline is filling at zoom level more than 16

In the US Airports and Flight Data example if we zoom beyond zoom level 16 the Arced Polyline are filling the color in the entire Map.
Attached is the snapshot of the Map behavior. This is not happening in IE Browser only happening in Chrome.

@Mahendershah apologies for the delayed response.  You're right.  That's weird.  I'll investigate, but I'm not certain this will be an easy fix.

Any updates on this issue?

@Mahendershah not yet.  I've looked into it a little bit, but no solution is jumping out to me at the moment.


[WIP][SPARK-34544][PYTHON] Convert PandasDataFrameLike and PandasSeriesLike to aliases of Pandas types

What changes were proposed in this pull request?

This PR proposes replacing currently used Protocols:

PandasDataFrameLike
PandasSeriesLike

with simple aliases of upstream types.
This exposed a number of typing issues, pirmairly around pyspark.pandas API, which will be resolved in this PR.
Additionally it adds VirtusLab/pandas-stubs to CI dependencies.
Why are the changes needed?

Currently used Protocols where a workaround, included to improve typing coverage until Pandas exposes their type hints.
In a meantime, relatively stable stubs sources emerged (with a lot of ongoing discussions around) and pandas-stubs package (available on PyPI and conda-forge) provides better coverage without adding maintenance overhead on our side.
Does this PR introduce any user-facing change?

Better typing experience around Pandas UDFs.
How was this patch tested?

Existing typecheck pipeline and unit tests.

FYI @HyukjinKwon
There is still a lot of work to done here (reduced typing errors from ~150  to  <50 so far).

Test build #146302 has started for PR 34927 at commit 5958791.

Test build #146302 has finished for PR 34927 at commit 5958791.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

I like this!!

cc @ueshin @itholic @xinrong-databricks FYI

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50774/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50774/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/146302/

A brief review of the work so far looks good to me!!
Let me revisit and take a closer look when it's ready to review.

Test build #146332 has started for PR 34927 at commit 3c5c894.

Test build #146332 has finished for PR 34927 at commit 3c5c894.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50807/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/146332/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50807/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50807/

Test build #146339 has started for PR 34927 at commit e5043a5.

Test build #146339 has finished for PR 34927 at commit e5043a5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50813/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/146339/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50813/

I've brief reviewed of the work, and so far looks pretty nice to me!!
Let me revisit and take a closer look when it's ready to review.

I'll ping you once I am closer to completion ‒ I resolved most of the minor problems, but hit a bigger issue with pandas_udf annotations ‒ seems like we'll have to rethink how this stuff is done, which is something I am not very happy about.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50813/

Test build #146341 has started for PR 34927 at commit d81bfee.

Test build #146341 has finished for PR 34927 at commit d81bfee.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

We can remove the pandas entry from mypy.ini?
https://github.com/apache/spark/blob/d81bfee6b56ffffa3deac2ab14e8a88821998e77/python/mypy.ini#L119-L120

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50815/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/146341/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50815/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50815/

Test build #146356 has started for PR 34927 at commit 045d587.

Test build #146357 has started for PR 34927 at commit 111b6cc.

Test build #146358 has started for PR 34927 at commit 535e3fb.

Test build #146356 has finished for PR 34927 at commit 045d587.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #146357 has finished for PR 34927 at commit 111b6cc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50830/

Test build #146358 has finished for PR 34927 at commit 535e3fb.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50831/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50832/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/146358/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/146356/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/146357/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50830/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50831/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50832/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50830/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50832/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50831/

Test build #146362 has started for PR 34927 at commit 28684cd.

Test build #146362 has finished for PR 34927 at commit 28684cd.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50836/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/146362/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50836/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50836/

Test build #146370 has started for PR 34927 at commit 47a727d.

Test build #146370 has finished for PR 34927 at commit 47a727d.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/146370/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50844/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50844/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50844/

Test build #146445 has started for PR 34927 at commit 1224d1e.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50920/

Test build #146445 has finished for PR 34927 at commit 1224d1e.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
trait ExposesMetadataColumns extends LogicalPlan

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/146445/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50920/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50920/

Merged into master.
Thanks all!


[SCB-607]Service registry failed when service before Service Center start complete

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [SCB-XXX] Fixes bug in ApproximateQuantiles, where you replace SCB-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean install to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage decreased (-0.005%) to 87.366% when pulling 7853f4c9fe9235e52c26a3d8fa84817b4bc0f4e5 on laijianbin:patch-1 into 61cda009b7ac8539364bd0f1d1855d804ea4dd27 on apache:master.

wujimin helps me refactoring some demo . But I modify demo through git hub front page and don't wanna to create a more jira issue, so I close this PR and then commit another PR.


Add {valid, same} option for Pooling layer

Mxnet's pooling layer's output doesn't match Caffe's pooling's output, although Mxnet's convolution layer's output shape is same as Caffe. This issue will cause Googlenet from caffe convertor failing.
The following links shows pooling output size formula.
for caffe,
https://github.com/BVLC/caffe/blob/master/src/caffe/layers/pooling_layer.cpp#L90
for mxnet,
https://github.com/dmlc/mxnet/blob/master/src/operator/pooling-inl.h#L202
And please see #2999. @winstywang @ChenFengAndy
So I add two options for pooling layer {valid, same}. valid is for current mxnet's pooling output and for caffe, it uses same option

default pooling option is valid, same as current mxnet.
if model is from caffe, same option for pooling layer will be on.

Please give some comments. Thanks a lot.

Why call it same?

@sxjscience
Maybe "full" is one better name? I will change it to "full". That makes more sense.

@Darwin2011 Could we manually adjust the padding to achieve the similar result?

@sxjscience Could you please explain a little bit more? Thanks.

@Darwin2011 I'm not sure whether this will break the C++ implementations.

@antinucleon @piiswrong @winstywang What do you think about this? Some old caffemodels like VGG_CNN_M requires this type of compatible pooling. Another solution is to fine-tune these networks using the revised structure (pad 1 in the pooling layer) and add them to our own model zoo.

@sxjscience
Caffe model converter also cannot translate bvlc googlenet on my side.

@Darwin2011 Yes, this is a problem. One way to deal with the problem is to add pad=(1,1) for lower level pooling layers like here (https://github.com/dmlc/mxnet/blob/master/example/image-classification/symbol_inception-bn-28-small.py#L19-L20). However, we may need to fine-tune the network again since the pooling function has been changed.
BTW, have you tested if the pooling layer works well for the SAME mode?

@sxjscience
I haven't tested it. What I only test is in "same" mode is on, Caffe convertor model can work well and mxnet can get same accuracy as Caffe. Let me test it with cifar10 training.
Thanks.

I think this should be added. Although the name "same" sounds cryptic. How about "compatible"?

@piiswrong @Darwin2011 @sxjscience: These names are really bad. In the literature, 'same', 'valid' and 'full' have precise meanings for pooling and convolutions in the literature:

same: the tensor is padded so that the input and output tensors are the same width and height.
valid: no padding

This is used by TensorFlow or Numpy, for example. Using these names for this will cause confusion (esp. if MXnet decided to add these very useful options for convolution + pooling).
How about "caffe_compatibibility" which is a bool?
Here is a question that we should answer first: what is the actual computational difference in the pooling layer? Is the MXNet pooling layer equivalent to Caffe with some extra padding for certain input dimensions? If so, we can fix this at the Caffe importer level rather than hacking it into the MXNet pooling definition.

the difference is basically round down vs up in division. So valid makes
sense. We just need to name the other one better.
On Sep 30, 2016 4:16 AM, "Sebastian Bodenstein"<EMAIL_ADDRESS>wrote:

@piiswrong https://github.com/piiswrong @Darwin2011
https://github.com/Darwin2011 @sxjscience
https://github.com/sxjscience: These names are really bad. In the
literature, 'same', 'valid' and 'full' have precise meanings for pooling
and convolutions in the literature:

same: the tensor is padded so that the input and output tensors are
the same width and height.
valid: no padding

This is used by TensorFlow
https://www.tensorflow.org/versions/r0.10/api_docs/python/nn.html#convolution
or Numpy
http://docs.scipy.org/doc/numpy/reference/generated/numpy.convolve.html,
for example. Using these names for this will cause confusion (esp. if MXnet
decided to add these very useful options for convolution + pooling).
How about "caffe_compatibibility" which is a bool?
Here is a question that we should answer first: what is the actual
computational difference in the pooling layer? Is the MXNet pooling layer
equivalent to Caffe with some extra padding for certain input dimensions?
If so, we can fix this at the Caffe importer level rather than hacking it
into the MXNet pooling definition.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dmlc/mxnet/pull/3392#issuecomment-250720020, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAiudGFZW_pavp69Y6cygNwmCsBW_WUnks5qvO-pgaJpZM4KIiS7
.

@sbodenstein Thanks for your advice.
Please see https://github.com/soumith/convnet-benchmarks/issues/82.
Googlenet symbols fails because of different pooling output size conventions. That's why I add one more options to pooling layer.

Thanks a lot for your review, @piiswrong
renaming done and add some descriptions.

@Darwin2011 I think we can merge this in if the test passes.

please update


Implement update passes

Implement update_anim pass.
Implement update_focus pass.

I'm closing this PR. Will reopen as two separate PRs.


feat(list slider): add new component and remove swiper :pray:

Infos
Task
What is being delivered?

Add ListSlider component
Create debounce as utils
Remove DeprecatedCarrousel

What impacts?
It removed completely the DeprecatedCarousel and the lib "Swiper" from the project, so we have a great difference in the final size of the project, it also removed some bugs and dependencies that are not necessary.
Bundle Sizes
{
  "key": "atom-deprecated-carousel.entry",
  "components": [
    "atom-deprecated-carousel"
  ],
  "originalByteSize": 417651
},
{
  "key": "atom-list-slider.entry",
  "components": [
     "atom-list-slider"
  ],
  "originalByteSize": 16491
},

DeprecatedCarousel (using Swiper): 417647 bytes 😱
ListSlider: 16491 bytes 😎

So if count Carousel (#434) and List Slider together have just 33168 bytes, more than 70% less than the DeprecatedCarousel
closes #445
Evidences

I love that! :rocket:  The slider works so well


[request] search feature

If possible a search feature would be really nice that allows users to search a string and directly open the index in which it is situated.

This or a reference list like in Kaihelios asset editor that tells you what export uses what imports, I'm aware you can get a pseudo version of this using the export information with dependencies column but it's harder to look through that.

Implemented in commit 2278ec1


Empty dump

I've tried to use the script, uploads fine. But, returns me a empty dump .sql file, i miss some configuration? And I was thinking there is no need info the database server?

Yes there's no need for the database server because the script is running on the same server as the mysql database (at least as the script is working right now).
Did you check if it's allowed to use PHP exec on your server to execute shell commands?
Does it work when you try the mysqldump command on the command line without the PHP script?
mysqldump -u youruser -pyourpassword --allow-keywords --add-drop-table --complete-insert --hex-blob --quote-names yourdbname > backupfile.sql
(make sure there's no space between argument -p and your password)
If this will work out for you and you're using the same account and db info in the PHP script and execution of shell commands is allowed it should be working.

Oh, great!
I made the question because I was looking for something different in general. But maybe is my hosting server who are blocking the commands via exec on PHP.
Anyway I wrote a code, compiling some others code to do the job 100% via PHP script instead doing exec on PHP or cron commands.
I'll try your code in a dedicated server instead a shared for tests, thanks anyway.
Best regards,


[Bug] worker-server/conf/olphinscheduler_env.sh read permission problem

Search before asking

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

What happened
dolphinscheduler在每次重启后都会做一些属性重置：
1、worker-server/conf目录属性为drwx------.
2、worker-server/conf/dolphinscheduler_env.sh属性为rwx------.
这样当使用hadoop或其他用户运行flinksql-shell脚本时，海豚会先执行worker-server/conf/dolphinscheduler_env.sh，此时发生权限拒绝的错误。
临时解决方法：将flink目录的权限设置为dolphinscheduler:root，然后使用dolphinscheduler用户来执行flinksql-shell脚本任务，避免执行dolphinscheduler_env.sh时出错。
What you expected to happen
在每次重启后dolphinscheduler保持现有worker-server的conf目录和dolphinscheduler_env.sh的读取属性。.
How to reproduce
使用其他非root系统用户执行一个flinksql脚本即可。
例如：sql-client.sh -f /app/software/flink-1.16.0/test-all.sql
Anything else
No response
Version
3.1.x
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

as i know, DS has remove the chmod -R 700

as i know, DS has remove the chmod -R 700

what's i did is changing the top level env.sh

as i know, DS has remove the chmod -R 700

on which version ? i use 3.0.2 and the problem is not fixed

as i know , chmod -R 700  is removed in 3.1.4, I don't know why there is such a odd command.


s6-overlay: init at <IP_ADDRESS>

Description of changes
I want to use s6 in a docker image, and figured s6-overlay is probably an easy solution... until I figured out it's not in nixpkgs :sweat_smile:
So I tried to package it:
Attempt 1 - the "nix way"
trying to link dependencies from nixpkgs and build from source - but that has proven to be difficult, because there is a lot, and it needs specific versions... I didn't manage
Attempt 2 - fetch deps from skaware & build from source
Trying to give the makefile everything it needs and linking it so that the makefile would think it fetched it.... felt like a good idea in the beginning, but it wasn't :sweat_smile:
(made an interim commit before starting with:)
Attempt 3 - just fetch release tarball
So that's what I ended up with as a "fair enough" situation, but I'm also not sure with symlinkJoin & srcs and installPhase - if that's a good way to do it.
Feedback appreciated :tipping_hand_man:
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Pinging skaware packages maintainers - maybe you could give some feedback:
@pmahoney @Profpatsch

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3885


Added preprocessor support.

Closes #67
This time it should be better than last time :smile:.

@Hardmath123 ping?

Yeah, I saw this. Looks good, but I can't review/merge until the week-end. :-)

:smile: Ok, I can wait a few days ...

Note: This PR is a bit broken, as some ebnf functions get serialized as JS not CoffeeScript

Ok, no longer broken, fixed it (and made a small merge conflict in the process :smiley: )...

Looking over this right now… could you explain what tabulateString does?
Also, I believe CoffeeScript allows you to include literal JavaScript in backticks or something. Then you don't need to rewrite built-in postprocessors in CoffeeScript. The whole {"special": x} seems a little icky and best avoided if possible (minimizing the number of moving parts and all that).
Thanks, and sorry again for the wait. :-)

Oh, one last thing: after @jcorbin's refactoring, this PR now has some merge conflicts. Shouldn't be too hard to fix, though.

@Hardmath123 About the backticks, it isn't modular enough, e.g. TypeScript won't allow it (who know for ClosureScript, etc.)

tabulateString will add a string to every line of the an input string (or, if it is an array, it will first join it with "\n")
I decided to go with {"special": x}, because it allows for every generator function pick it's own. (The other option was to use some special string, e.g. "INTERNAL__DONT_USE___x", but it seemed to much of a hack). Maybe we should change special to something else though...

@Hardmath123 Ok, merged, now it is ready.

I left some thoughts as line notes.

@Hardmath123 Ok, I'll make another commit it seems....

@Hardmath123 Ok, fixed all comments, I think. Anything else?

Sorry, one last thing that bothers me: rather than a _default, an error should be thrown if no such preprocessor exists. So a grammar with @preprocessor jkhldsfghjklfsg should not compile. Just throw new Error("No such preprocessor: "+…); should suffice.
Everything else looks great. Thanks for including a nice test case. :-)

@Hardmath123 Ok, now I think everything is ready!

Thanks; merged! :-)


Fix center logo in LoginDialog

auto size login logo div
Description
horizontal align custom logos centered independent of their size.
Motivation
logo <200px
Fixes:
Please provide a list of the fixes implemented in this PR

horizontal align center custom logos in login dialog

Changes:
Please provide a list of the changes implemented by this PR

removed flex from div to auto size and align centered

Summary by Sourcery
Bug Fixes:

Fix horizontal alignment of custom logos in the login dialog by removing fixed flex size.

@cfelder Could you please post a couple of image with before and after of your fix? Thanks

w/o this PR, Logo is offcentered:

w/ this PR, centered:


j: 701_b -> 806

Motivation for this change
j was way behind, at least 3 years. This fixes that and updates the build script.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip" (Forgot this step, but I don't believe that there are any packages that depend on j)
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

/usr/bin/env is not available in the sandbox (https://hydra.nixos.org/build/65198033), fixed in cdef334423a4d37b538d457bbea0d453e1b5eb97. Are you sure you have tested using sandboxing?

@orivej: I did nix-shell -I nixpkgs=/home/synthetica/devel/nixpkgs/ -I "nix.useSandbox=true" -p j, which in didn't work (20/20 hindsight), but also didn't give an error. What is the correct way to do this, for future reference?

The -I option just extends Nix search path, so -I nix.useSandbox=true will not enable the sandbox. If nix-info --sandbox says sandbox: yes, you can build in the sandbox with nix-build /home/synthetica/devel/nixpkgs -A j.


bump: 0.15.3rc0 -> 0.15.3rc1

hopefully without Codecov, the release action will upload to PyPI successfully.
This also reverts the change to what is labeled "stable" in the version switcher JSON, which probably shouldn't have been done at the RC stage.

@trallard already gave approval to remove codecov in https://github.com/pydata/pydata-sphinx-theme/issues/1797#issuecomment-2093260408 so I'll self-merge when this is green.


[Suggestion] Basic usage instruction

Hi luminoso, nobody is holding me hostage in his basement and is forcing me to make a client for your internet protocol. Could you maybe add to the README a basic example usage for the protocol? Thanks!

The protocol specification lays out how to send and recieve messages. Any implementation would simply follow the specification.

nobody is holding me hostage in his basement

Sounds legit.

The protocol specification lays out how to send and recieve messages. Any implementation would simply follow the specification.

But like do I compile it or something to use??? Nobody said he's gonna start harvesting my organs and selling them on the dark web to buy supplies for a Taser Keyboard v2 if I don't finish in 24 hours.

I thought I told you to be quiet

Which part?
piper-srv is a rust project, and can be compiled with cargo.
a reference client implementation can be found at another repository of mine.
Both of these being rust projects, you simply install rustup/cargo, and run cargo build inside the cloned project dir.

thank you, now i shall pray my organs are still intact inside my body after the next 24 hours


Google Fallback

Currently we fallback to Google's public DNS servers if we don't have a config. This affected probably only Windows, as we don't have /etc/resolv.conf there. On other systems, that could only happen if there is an error reading /etc/resolv.conf.
I don't like sending all DNS queries silently as a fallback to Google. If the system config can't be read, it's either a weird system not yet supported (a bug then), or something other that has to be fixed on the system. @DaveRandom mentioned he'd like to keep the fallback, as we can still continue resolving domain names, but would like to issue a warning when the default config is used.
#40 plans to deprecate the default config and I plan to remove default nameservers when we release a new major version based on Amp v2.
Please add a :+1: if you want to drop the fallback, add a :-1: if you want to keep the fallback.

If someone explicitly sets the config, then it shouldn't do the fallback; it should behave exactly as they set the config.
If someone doesn't explicitly set the config, then people just want it to work and so the code should do the best attempt to work.

I don't like sending all DNS queries silently as a fallback to Google.

I trust with my DNS lookups (aka think they are less likely to care about them) than I do my ISP, that is far more likely to be in cahoots with GCHQ or other industrial espionage.

We could also factor it out to a WindowsConfigReader, BindConfigReader and fallback to a EnvConfigReader that reads from a environment variable. This ensures you can always set the DNS nameserver in any application in case your system is weird.
We could also prioritize the environment variable / use it as fallback without the above.

There's no point in factoring that out...
Also env variables, if we support them, should take priority in any case.

There's no point in factoring that out...

Accept if you want to fetch the information from another source, but that's not required I guess.

The man page mentions environment variables for options and local domain, but not for nameservers. http://man7.org/linux/man-pages/man5/resolv.conf.5.html

Related: https://twitter.com/fudoreaper/status/873926204046405632

The Google fallback has been removed in the Amp v2 version.


tls/server: move server.go in its proper location

Signed-off-by: Nagy Károly Gábriel<EMAIL_ADDRESS>

merged. thanks.