get FME info activity returns 415 Unsupported Media Type error

Hi team,
I'm getting a 415 Unsupported Media Type error when using the get FME info activity. I've dig into the code and found a "Content-Type: application/x-www-form-urlencoded" header has been added in the request (line 50 at https://github.com/vertigis/workflow-activities-fme/commit/90fef6ac9e0a6c857bb3ae05ea0e1f4be3ac5098#diff-f294c7c8541dceeadc1682d3423488918beee90117e33328e977eec7a33e330d).
I've tested it against a FME server with Postman, and can replicate the issue if I add "Content-Type: application/x-www-form-urlencoded" to the header.
I suggest to remove that Content-Type from the header as this shouldn't be required for this GET request anyway.
Thanks,
Johnny

Thanks for the detailed report. That makes it really easy. The fix is now in a PR and should be available in a day or two.
If you need an immediate workaround you can use the Use FME Service activity and set the path to info.


Failed to load \bin\x86\SNI.dll

FYI only, You can close the issue after reading
After Upgrade from 5.7.2. to 6.2.0
Most of the developer was getting an issue
Failed to load ...l\bin\x86\SNI.dll
I upgraded Microsoft.Data.SqlClient 1.1.0 to 2.0.1.
That fixed the issue. :)

I can bump up to 2.x in provider 7.x version. But usually we saw this issue when sni.dll is not actually deployed to target server (due to weird packaging logic that copies files to output dir after packaging script has ran).
Does any of other projects pull in 2.x data client library?

Nope, there is no other project within our solution, having reliance on 2.x
/K
On Tue, Sep 15, 2020 at 6:50 PM valdis iljuconoks<EMAIL_ADDRESS>wrote:

I can bump up to 2.x in provider 7.x version. But usually we saw this
issue when sni.dll is not actually deployed to target server (due to weird
packaging logic that copies files to output dir after packaging script has
ran).
Does any of other projects pull in 2.x data client library?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/valdisiljuconoks/localization-provider-epi/issues/77#issuecomment-692840486,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACYFYCS6BW5KPC2PHKMTTU3SF6LNJANCNFSM4RNIA2DQ
.

Does sni.dll exist in output directory?
Is app pool set to 32bit?
Get Outlook for Androidhttps://aka.ms/ghei36

From: khurramkhang<EMAIL_ADDRESS>Sent: Wednesday, September 16, 2020 12:01:55 PM
To: valdisiljuconoks/localization-provider-epi<EMAIL_ADDRESS>Cc: valdis iljuconoks<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [valdisiljuconoks/localization-provider-epi] Failed to load \bin\x86\SNI.dll (#77)
Nope, there is no other project within our solution, having reliance on 2.x
/K
On Tue, Sep 15, 2020 at 6:50 PM valdis iljuconoks<EMAIL_ADDRESS>wrote:

I can bump up to 2.x in provider 7.x version. But usually we saw this
issue when sni.dll is not actually deployed to target server (due to weird
packaging logic that copies files to output dir after packaging script has
ran).
Does any of other projects pull in 2.x data client library?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/valdisiljuconoks/localization-provider-epi/issues/77#issuecomment-692840486,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACYFYCS6BW5KPC2PHKMTTU3SF6LNJANCNFSM4RNIA2DQ
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fvaldisiljuconoks%2Flocalization-provider-epi%2Fissues%2F77%23issuecomment-693274267&data=02|01||8388397f39d74ab76fac08d85a1f2a01|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637358437171797707&sdata=EO%2FOz7g8kue2fnLnADTgQumYgAh%2FzgHvDMM2RreVjAg%3D&reserved=0, or unsubscribehttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAANPOC3LB2276G7VNPNWIILSGB5IHANCNFSM4RNIA2DQ&data=02|01||8388397f39d74ab76fac08d85a1f2a01|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637358437171807703&sdata=Nmf7IWt74P0Rd0t7hqzqu32zhaqLY01Tl5BHQW952lc%3D&reserved=0.

For a developer when it worked dll was in x86 folder, but in case of
cleaning solution and removing bin folder, the Package wasn't restoring. i
i think all run app in iis express
/K
On Wed, Sep 16, 2020 at 11:13 AM valdis iljuconoks<EMAIL_ADDRESS>wrote:

Does sni.dll exist in output directory?
Is app pool set to 32bit?
Get Outlook for Androidhttps://aka.ms/ghei36

From: khurramkhang<EMAIL_ADDRESS>Sent: Wednesday, September 16, 2020 12:01:55 PM
To: valdisiljuconoks/localization-provider-epi <
<EMAIL_ADDRESS>Cc: valdis iljuconoks<EMAIL_ADDRESS>Comment <
<EMAIL_ADDRESS>Subject: Re: [valdisiljuconoks/localization-provider-epi] Failed to load
\bin\x86\SNI.dll (#77)
Nope, there is no other project within our solution, having reliance on 2.x
/K
On Tue, Sep 15, 2020 at 6:50 PM valdis iljuconoks <
<EMAIL_ADDRESS>wrote:

I can bump up to 2.x in provider 7.x version. But usually we saw this
issue when sni.dll is not actually deployed to target server (due to
weird
packaging logic that copies files to output dir after packaging script
has
ran).
Does any of other projects pull in 2.x data client library?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<
https://github.com/valdisiljuconoks/localization-provider-epi/issues/77#issuecomment-692840486
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ACYFYCS6BW5KPC2PHKMTTU3SF6LNJANCNFSM4RNIA2DQ
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub<
https://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fvaldisiljuconoks%2Flocalization-provider-epi%2Fissues%2F77%23issuecomment-693274267&data=02|01||8388397f39d74ab76fac08d85a1f2a01|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637358437171797707&sdata=EO%2FOz7g8kue2fnLnADTgQumYgAh%2FzgHvDMM2RreVjAg%3D&reserved=0>,
or unsubscribe<
https://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAANPOC3LB2276G7VNPNWIILSGB5IHANCNFSM4RNIA2DQ&data=02|01||8388397f39d74ab76fac08d85a1f2a01|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637358437171807703&sdata=Nmf7IWt74P0Rd0t7hqzqu32zhaqLY01Tl5BHQW952lc%3D&reserved=0

.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/valdisiljuconoks/localization-provider-epi/issues/77#issuecomment-693280135,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACYFYCTE3NTIYLF2BJSKY7LSGB6SZANCNFSM4RNIA2DQ
.

As said - we also noticed some weird behaviour of that package when it comes to copying files. Ok - will bump to 2.x version then.
Get Outlook for Androidhttps://aka.ms/ghei36

From: khurramkhang<EMAIL_ADDRESS>Sent: Wednesday, September 16, 2020 12:27:12 PM
To: valdisiljuconoks/localization-provider-epi<EMAIL_ADDRESS>Cc: valdis iljuconoks<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [valdisiljuconoks/localization-provider-epi] Failed to load \bin\x86\SNI.dll (#77)
For a developer when it worked dll was in x86 folder, but in case of
cleaning solution and removing bin folder, the Package wasn't restoring. i
i think all run app in iis express
/K
On Wed, Sep 16, 2020 at 11:13 AM valdis iljuconoks<EMAIL_ADDRESS>wrote:

Does sni.dll exist in output directory?
Is app pool set to 32bit?
Get Outlook for Androidhttps://aka.ms/ghei36

From: khurramkhang<EMAIL_ADDRESS>Sent: Wednesday, September 16, 2020 12:01:55 PM
To: valdisiljuconoks/localization-provider-epi <
<EMAIL_ADDRESS>Cc: valdis iljuconoks<EMAIL_ADDRESS>Comment <
<EMAIL_ADDRESS>Subject: Re: [valdisiljuconoks/localization-provider-epi] Failed to load
\bin\x86\SNI.dll (#77)
Nope, there is no other project within our solution, having reliance on 2.x
/K
On Tue, Sep 15, 2020 at 6:50 PM valdis iljuconoks <
<EMAIL_ADDRESS>wrote:

I can bump up to 2.x in provider 7.x version. But usually we saw this
issue when sni.dll is not actually deployed to target server (due to
weird
packaging logic that copies files to output dir after packaging script
has
ran).
Does any of other projects pull in 2.x data client library?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<
https://github.com/valdisiljuconoks/localization-provider-epi/issues/77#issuecomment-692840486
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ACYFYCS6BW5KPC2PHKMTTU3SF6LNJANCNFSM4RNIA2DQ
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub<
https://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fvaldisiljuconoks%2Flocalization-provider-epi%2Fissues%2F77%23issuecomment-693274267&data=02|01||8388397f39d74ab76fac08d85a1f2a01|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637358437171797707&sdata=EO%2FOz7g8kue2fnLnADTgQumYgAh%2FzgHvDMM2RreVjAg%3D&reserved=0>,
or unsubscribe<
https://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAANPOC3LB2276G7VNPNWIILSGB5IHANCNFSM4RNIA2DQ&data=02|01||8388397f39d74ab76fac08d85a1f2a01|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637358437171807703&sdata=Nmf7IWt74P0Rd0t7hqzqu32zhaqLY01Tl5BHQW952lc%3D&reserved=0

.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/valdisiljuconoks/localization-provider-epi/issues/77#issuecomment-693280135,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACYFYCTE3NTIYLF2BJSKY7LSGB6SZANCNFSM4RNIA2DQ
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fvaldisiljuconoks%2Flocalization-provider-epi%2Fissues%2F77%23issuecomment-693287320&data=02|01||261efaa9429841be018c08d85a22b189|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637358452332559588&sdata=ZtTIpXmiGOiyNKcM3NrVyLT7SKOZRQGCiVvsZAqLtIE%3D&reserved=0, or unsubscribehttps://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAANPOC6TV7OTW3BD7TQIY6LSGCAHBANCNFSM4RNIA2DQ&data=02|01||261efaa9429841be018c08d85a22b189|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637358452332569583&sdata=s8OXU8KIWWIqv8KVZOJtyQbNIiUp4TiOSuqjX4QFU%2BI%3D&reserved=0.

Thanks :)
On Wed, Sep 16, 2020 at 11:30 AM valdis iljuconoks<EMAIL_ADDRESS>wrote:

As said - we also noticed some weird behaviour of that package when it
comes to copying files. Ok - will bump to 2.x version then.
Get Outlook for Androidhttps://aka.ms/ghei36

From: khurramkhang<EMAIL_ADDRESS>Sent: Wednesday, September 16, 2020 12:27:12 PM
To: valdisiljuconoks/localization-provider-epi <
<EMAIL_ADDRESS>Cc: valdis iljuconoks<EMAIL_ADDRESS>Comment <
<EMAIL_ADDRESS>Subject: Re: [valdisiljuconoks/localization-provider-epi] Failed to load
\bin\x86\SNI.dll (#77)
For a developer when it worked dll was in x86 folder, but in case of
cleaning solution and removing bin folder, the Package wasn't restoring. i
i think all run app in iis express
/K
On Wed, Sep 16, 2020 at 11:13 AM valdis iljuconoks <
<EMAIL_ADDRESS>wrote:

Does sni.dll exist in output directory?
Is app pool set to 32bit?
Get Outlook for Androidhttps://aka.ms/ghei36

From: khurramkhang<EMAIL_ADDRESS>Sent: Wednesday, September 16, 2020 12:01:55 PM
To: valdisiljuconoks/localization-provider-epi <
<EMAIL_ADDRESS>Cc: valdis iljuconoks<EMAIL_ADDRESS>Comment <
<EMAIL_ADDRESS>Subject: Re: [valdisiljuconoks/localization-provider-epi] Failed to load
\bin\x86\SNI.dll (#77)
Nope, there is no other project within our solution, having reliance on
2.x
/K
On Tue, Sep 15, 2020 at 6:50 PM valdis iljuconoks <
<EMAIL_ADDRESS>wrote:

I can bump up to 2.x in provider 7.x version. But usually we saw this
issue when sni.dll is not actually deployed to target server (due to
weird
packaging logic that copies files to output dir after packaging script
has
ran).
Does any of other projects pull in 2.x data client library?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<

https://github.com/valdisiljuconoks/localization-provider-epi/issues/77#issuecomment-692840486

,
or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/ACYFYCS6BW5KPC2PHKMTTU3SF6LNJANCNFSM4RNIA2DQ

.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub<

https://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fvaldisiljuconoks%2Flocalization-provider-epi%2Fissues%2F77%23issuecomment-693274267&data=02|01||8388397f39d74ab76fac08d85a1f2a01|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637358437171797707&sdata=EO%2FOz7g8kue2fnLnADTgQumYgAh%2FzgHvDMM2RreVjAg%3D&reserved=0

,
or unsubscribe<

https://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAANPOC3LB2276G7VNPNWIILSGB5IHANCNFSM4RNIA2DQ&data=02|01||8388397f39d74ab76fac08d85a1f2a01|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637358437171807703&sdata=Nmf7IWt74P0Rd0t7hqzqu32zhaqLY01Tl5BHQW952lc%3D&reserved=0

.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<
https://github.com/valdisiljuconoks/localization-provider-epi/issues/77#issuecomment-693280135
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ACYFYCTE3NTIYLF2BJSKY7LSGB6SZANCNFSM4RNIA2DQ
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub<
https://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fvaldisiljuconoks%2Flocalization-provider-epi%2Fissues%2F77%23issuecomment-693287320&data=02|01||261efaa9429841be018c08d85a22b189|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637358452332559588&sdata=ZtTIpXmiGOiyNKcM3NrVyLT7SKOZRQGCiVvsZAqLtIE%3D&reserved=0>,
or unsubscribe<
https://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAANPOC6TV7OTW3BD7TQIY6LSGCAHBANCNFSM4RNIA2DQ&data=02|01||261efaa9429841be018c08d85a22b189|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637358452332569583&sdata=s8OXU8KIWWIqv8KVZOJtyQbNIiUp4TiOSuqjX4QFU%2BI%3D&reserved=0

.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/valdisiljuconoks/localization-provider-epi/issues/77#issuecomment-693288969,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACYFYCREYX77NPLYK2HD45DSGCATBANCNFSM4RNIA2DQ
.

We are having another weird issue during automatic release (Azure Cloud web Aps)
Error Code: ERROR_FILE_IN_USE More Information: Web Deploy cannot modify the file 'Microsoft.Data.SqlClient.SNI.x64.
Have to Stop WebApp/Release/Start App

Welcome to SqlClient package world ;)
Get Outlook for Androidhttps://aka.ms/ghei36

From: khurramkhang<EMAIL_ADDRESS>Sent: Wednesday, September 16, 2020 5:59:50 PM
To: valdisiljuconoks/localization-provider-epi<EMAIL_ADDRESS>Cc: valdis iljuconoks<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [valdisiljuconoks/localization-provider-epi] Failed to load \bin\x86\SNI.dll (#77)
We are having another weird issue during automatic release (Azure Cloud web Aps)
Error Code: ERROR_FILE_IN_USE More Information: Web Deploy cannot modify the file 'Microsoft.Data.SqlClient.SNI.x64.
Have to Stop WebApp/Release/Start App
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fvaldisiljuconoks%2Flocalization-provider-epi%2Fissues%2F77%23issuecomment-693463945&data=02|01||0fc038478ab946801cac08d85a512aa0|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637358651932096463&sdata=99z5m%2BsLx69A4J%2FaTmvJfoGvDRQvn2L07jbuku%2BDPTg%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAANPOC2DN6SLL2GDBLRXVX3SGDHGNANCNFSM4RNIA2DQ&data=02|01||0fc038478ab946801cac08d85a512aa0|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637358651932096463&sdata=Q2OkWNwjVqVYERU1kUaepaWD0FAIAWLJadW75tKf4Yc%3D&reserved=0.


Update to Go v1.22

ref: https://issues.redhat.com/browse/ACM-14251

/override ci/prow/test-e2e-grc-framework
/override security/snyk (Red Hat Advanced Cluster Management for Kubernetes)

/override ci/prow/test-e2e-grc-framework
/override "security/snyk (Red Hat Advanced Cluster Management for Kubernetes)"


use PhantomJS for integration testing

Adds capybara, capybara-screenshot, and poltergeist as development dependencies.

@ollietulloch - looks like the 3.2 builds have failed; from memory, the screenshotting library is configured for @TestUnit@ vs. @MiniTest@ separately, so we may need a bit of conditional logic.


Fix doulbe shift to capslock trick

Double shift to capslock was behaving in an inconsistent way.
Also only one shift should be required to exit (you can still use shift normally during capslock (as long as you don't expect it to be sticky))

Thanks!


Can't find Logs and Milestones Tabs

Command '100 Days of Code: View Milestones' resulted in an error (command 'DoC.viewMilestones' not found)
Got the error when i manually tried running the command
And 100days of code Section is empty

@AugustBemsen a fix was just pushed in 1.0.7 - can you try this version? Thank you!


client: Add basic support for projects

This allows passing the project name as a string to Client and have it
automatically added to all URLs.
Blackened by @d0ugal for easier merging.
Signed-off-by: Stéphane Graber<EMAIL_ADDRESS>Signed-off-by: Dougal Matthews<EMAIL_ADDRESS>

This is #414 but blackened and rebased.

Codecov Report

Merging #430 (53cf367) into master (bd30853) will decrease coverage by 0.39%.
The diff coverage is 42.85%.

@@            Coverage Diff             @@
##           master     #430      +/-   ##
==========================================
- Coverage   97.29%   96.90%   -0.40%
==========================================
  Files          53       53
  Lines        3887     3911      +24
==========================================
+ Hits         3782     3790       +8
- Misses        105      121      +16

Impacted Files
Coverage Δ

pylxd/client.py
91.78% <42.85%> (-7.67%)
:arrow_down:

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 bd30853...53cf367. Read the comment docs.


Add license file

I noticed this repository is public but does not contain a license, which makes it hard for others to use and contribute to it. I have added our standard MIT license.
Review @wvanbergen @burke

:+1: from me. cc @byroot

:+1: cc @burke

Whoops, LGTM

Does anybody understand the CircleCI failure? I'm fairly confident it is not caused by my change but I hate pushing with a red CI. I am guessing it is the "data race(s)" message that causes the failure.

It's failing because of the data races, which is a concurrency issue somewere. Maybe it's only seeing those now, with a more recent go version?

Just ignore the failure; whatever the problem it's not the LICENSE file :)

Ok I have merged, and opened Issue #11 for the data race in tests.


Image output

Hello, Is there any possibility to publish sensor_msgs/Image and sensor_msgs/CameraInfo from ROS/ROS2 driver?

Currently, it's not possible to retrieve the images via the ROS driver.
However, we're making the image output available over UDP in our next customer release (~2 weeks). Please keep an eye out for the new release.
We're also evaluating making it available through the ROS driver and will keep you updated on this thread.

Hi @ShepelIlya, we would greatly appreciate a PR.

@AngeLoGa @JJRedmond, in our latest stable release (2.85.3), you can access the camera stream via Real-time Transport Protocol (RTP). You can find more information here: https://docs.fixposition.com/fd/camera-image-streaming.
Let me know if you have any questions ;)


Add support for ppc64le builds

Description

Fixes https://github.com/tigera/operator/issues/1842
For PR author

[ ] Tests for change.
[ ] If changing pkg/apis/, run make gen-files
[ ] If changing versions, run make gen-versions

For PR reviewers
A note for code reviewers - all pull requests must have the following:

[ ] Milestone set according to targeted release.
[ ] Appropriate labels:

kind/bug if this is a bugfix.
kind/enhancement if this is a a new feature.
enterprise if this PR applies to Calico Enterprise only.

@caseydavenport any blockers here? We are looking for this operator image on ppc64le.
@tmjd could you please help with the review?

@yussufsh I'll let @caseydavenport get the build working and then he can ping me if he needs my review (he knows how to reach me :grinning: ).

Fixed here: https://github.com/tigera/operator/pull/2048


Custom SQL files for different database systems

while debugging issue #144, I found several sql-schema-files for MariaDB, MySQL and Postgres.

Coverage remained the same at 24.566% when pulling 02dfc31235abd0068fb51212736df56c0d8e2ffb on sebastianblum:patch-1 into cef92e90bdd4846faa97fcfb7b4161b91888eb40 on prooph:master.


Feature Request & upstream bug?: Update CoreDNS and the CoreDNS Addon's Default's that eks blueprints uses

Describe the feature
Problem Statement: AWS's EKS Upstream has poor default values for CoreDNS
Feature Request: Have future version of EKS Blueprints default to better defaults, specifically:

have a hard requirement to spread replicas of coredns across nodes
have CoreDNS autoscaling on by default

Summary of Possible Bug:

Note the feature request is cdk-eks-blueprints specific.
The bug is EKS Upstream, but I'm not sure where to report it? I'll include it in the other information section. The upstream bug would need to be fixed to complete this feature request.

Use Case
EKS Blueprints is supposed to implement best practices by default.
Best practice is

hard requirement to spread coredns replicas across nodes, vs a soft preference.
coredns autoscaling enabled by default.

Just because EKS upstream has poor default values doesn't mean this project should.
Proposed Solution
Have a future version of EKS Blueprints use the following default values for CoreDNS
Note: "auto" would have to map to a supported version of the addon.
new blueprints.addons.CoreDnsAddOn( "auto", {
    configurationValues: {
            "autoscaling": {
              "enabled": true,
              "minReplicas": 2,
              "maxReplicas": 100
            },
            "affinity": {
              "nodeAffinity": {
                "requiredDuringSchedulingIgnoredDuringExecution": {
                  "nodeSelectorTerms": [
                    {
                      "matchExpressions": [
                        {
                          "key": "kubernetes.io/os",
                          "operator": "In",
                          "values": [
                            "linux"
                          ]
                        },
                        {
                          "key": "kubernetes.io/arch",
                          "operator": "In",
                          "values": [
                            "amd64",
                            "arm64"
                          ]
                        }
                      ]
                    }
                  ]
                }
              },
              "podAntiAffinity": {
                "requiredDuringSchedulingIgnoredDuringExecution": [
                  {
                    "labelSelector": {
                      "matchExpressions": [
                        {
                          "key": "k8s-app",
                          "operator": "In",
                          "values": [
                            "kube-dns"
                          ]
                        }
                      ]
                    },
                    "topologyKey": "kubernetes.io/hostname"
                  }
                ]
              }
            }
    } //end CoreDNS configurationValues override
  }) //end CoreDNS AddOn

Other Information
I think the bug is with AWS/EKS Upstream, but I'm not sure where to report?
https://aws.amazon.com/about-aws/whats-new/2024/05/amazon-eks-native-support-autoscaling-coredns-pods/
mentions coredns autoscaling has been supported since May, 2024.
aws eks describe-addon-configuration --addon-name coredns --addon-version v1.11.1-eksbuild.8 --query configurationSchema --output text | jq .
^-- shows autoscaling is not available on the older version
(the above shows a shema, with autoscaling missing)
aws eks describe-addon-configuration --addon-name coredns --addon-version v1.11.1-eksbuild.11 --query configurationSchema --output text | jq .
^-- shows autoscaling should be available in the newer version
aws eks describe-addon-configuration --addon-name coredns --addon-version v1.11.1-eksbuild.11 --query configurationSchema --output text | jq . | grep auto
        "autoScaling": {
          "description": "autoScaling configurations",

Useful Note about cdk-eks-blueprints v1.15.1:

"auto" results in DEBUG Core add-on coredns has autoselected version v1.11.1-eksbuild.8
"v1.11.1-eksbuild.11" results in DEBUG Core add-on coredns is at version v1.11.1-eksbuild.11

Oddity:

Per the above queries about configurationSchema, CoreDNS "v1.11.1-eksbuild.11", should support autoscaling config.
I swear I got the following to working the ESK Addon Web Console configuration, once before on  CoreDNS "v1.11.1-eksbuild.11", but haven't been able to reproduce. And it consistently fails with cdk.
Here's what worked in the Web GUI once (coredns edited configuration value):

{
            "autoscaling": {
              "enabled": true,
              "minReplicas": 3,
              "maxReplicas": 100
            },
            "affinity": {
              "nodeAffinity": {
                "requiredDuringSchedulingIgnoredDuringExecution": {
                  "nodeSelectorTerms": [
                    {
                      "matchExpressions": [
                        {
                          "key": "kubernetes.io/os",
                          "operator": "In",
                          "values": [
                            "linux"
                          ]
                        },
                        {
                          "key": "kubernetes.io/arch",
                          "operator": "In",
                          "values": [
                            "amd64",
                            "arm64"
                          ]
                        }
                      ]
                    }
                  ]
                }
              },
              "podAntiAffinity": {
                "requiredDuringSchedulingIgnoredDuringExecution": [
                  {
                    "labelSelector": {
                      "matchExpressions": [
                        {
                          "key": "k8s-app",
                          "operator": "In",
                          "values": [
                            "kube-dns"
                          ]
                        }
                      ]
                    },
                    "topologyKey": "kubernetes.io/hostname"
                  }
                ]
              }
            }
}

The following currently works for me on cdk-eks-blueprint v1.15.1 & EKS v1.30 cluster, as a more optimized default, again "auto" maps to "v1.11.1-eksbuild.8"
(where the replicas have a hard requirement to spread across nodes.)
new blueprints.addons.CoreDnsAddOn( "auto", {
    configurationValues: {
            "affinity": {
              "nodeAffinity": {
                "requiredDuringSchedulingIgnoredDuringExecution": {
                  "nodeSelectorTerms": [
                    {
                      "matchExpressions": [
                        {
                          "key": "kubernetes.io/os",
                          "operator": "In",
                          "values": [
                            "linux"
                          ]
                        },
                        {
                          "key": "kubernetes.io/arch",
                          "operator": "In",
                          "values": [
                            "amd64",
                            "arm64"
                          ]
                        }
                      ]
                    }
                  ]
                }
              },
              "podAntiAffinity": {
                "requiredDuringSchedulingIgnoredDuringExecution": [
                  {
                    "labelSelector": {
                      "matchExpressions": [
                        {
                          "key": "k8s-app",
                          "operator": "In",
                          "values": [
                            "kube-dns"
                          ]
                        }
                      ]
                    },
                    "topologyKey": "kubernetes.io/hostname"
                  }
                ]
              }
            }
    } //end CoreDNS configurationValues override
  }) //end CoreDNS AddOn

Bug? The following fails for me when implemented with cdk:
new blueprints.addons.CoreDnsAddOn( "v1.11.1-eksbuild.11", {
    configurationValues: {
            "autoscaling": {
              "enabled": true,
              "minReplicas": 3,
              "maxReplicas": 100
            },
            "affinity": {
              "nodeAffinity": {
                "requiredDuringSchedulingIgnoredDuringExecution": {
                  "nodeSelectorTerms": [
                    {
                      "matchExpressions": [
                        {
                          "key": "kubernetes.io/os",
                          "operator": "In",
                          "values": [
                            "linux"
                          ]
                        },
                        {
                          "key": "kubernetes.io/arch",
                          "operator": "In",
                          "values": [
                            "amd64",
                            "arm64"
                          ]
                        }
                      ]
                    }
                  ]
                }
              },
              "podAntiAffinity": {
                "requiredDuringSchedulingIgnoredDuringExecution": [
                  {
                    "labelSelector": {
                      "matchExpressions": [
                        {
                          "key": "k8s-app",
                          "operator": "In",
                          "values": [
                            "kube-dns"
                          ]
                        }
                      ]
                    },
                    "topologyKey": "kubernetes.io/hostname"
                  }
                ]
              }
            }
    } //end CoreDNS configurationValues override
  }) //end CoreDNS AddOn

Acknowledgements

[ ] I may be able to implement this feature request

CDK version used
2.133.0
EKS Blueprints Version
1.15.1
Node.js Version
v20.15.0
Environment details (OS name and version, etc.)
Mac OS Sonoma 14.5

ugh ok, nvm I'll edit this, there's no upstream bug. I just typo'd the auto scaling part
"autoscaling" --> "autoScaling", that said the feature request is still valid.


How to define enumeration with int value in IDL?

Is it possible to define an enumeration as the following in IDL?
enum Enumeration : int32
{
    RED = 1,
    GREEN = 2,
    BLUE = 3
};

If yes, what's the correct syntax for such a definition? The code snippet above isn't valid in IDL.

@Octavian-Zhang
The IDL to C++11 language mapping specification states in clause 6.9 that an IDL enumeration should map to an uint32_t C++11 enumeration. The DDS XTypes specification allows to modify the size of the enumerated type using the @bit_bound builtin annotation (Fast DDS-Gen does not support yet this specific annotation for enumerations, as stated in Fast DDS documentation). It does not seem to be a manner to define negative values within an IDL enumeration.


Added repo on gradient boosting papers.

A curated list of gradient and adaptive boosting papers with implementations from the following conferences:
Machine learning
    NeurIPS
    ICML
    ICLR
Computer vision
    CVPR
    ICCV
    ECCV
Natural language processing
    ACL
    NAACL
    EMNLP
Data Mining
    KDD
    ICDM
    WWW
Artificial intelligence
    AAAI
    IJCAI
    UAI

And thanks again!


[Improvements] Add a loader after the prompt has been sent to the server.

Either create a loader to indicate to the user that the request is processing, or stream the responses.

loader added


power-profiles-daemon: 0.13 -> 0.20

Description of changes
commit 9998f40bb36ba6fc3b8252cfd7a3ea494b146056 (HEAD -> pic/ppd020, fork/pic/ppd020, master)
Date:   Sun Feb 18 13:46:56 2024 +0100

    nixosTests.power-profiles-daemon: test profilectl CLI

    We also take advantage of that change to point to the new dbus
    canonical names.

commit cc557e1055de6e913e1aa0aa312001d206b864c5
Date:   Sun Feb 18 13:43:46 2024 +0100

    power-profiles-daemon: 0.13 -> 0.20

    The upstream original maintainer is not paid anymore to maintain the
    project and decided to step down. The project has been taken over by
    the upower team.

    This is the first release part of this upower team.

    The integration tests are now relying on the powerprofilectl command:
    we need to patch the python path of this dependency before running the
    integration tests instead of doing that during in the fixup phase.

    Taking advantage of this to remove the PostFixup phase. Not 100% about
    this move: it's a bit less future proof.

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

[x] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Skimming upstream changes, they look okay: https://gitlab.freedesktop.org/upower/power-profiles-daemon/-/compare/0.13...0.20

ping

Can we possibly backport this onto 23.11? This is a boon for new AMD laptops.

Hello, is there any news of the back porting for this PR on 23.11 ? Is there any we can help for the backporting ?

It's a breaking change, so it's not really supposed to be backported.

I believe this has been mentioned in the framework forum and this is a nice battery efficiency upgrade for the new Framework AMD laptop.
On March 24, 2024 1:37:13 PM PDT, K900 @.***> wrote:

It's a breaking change, so it's not really supposed to be backported.
--
Reply to this email directly or view it on GitHub:
https://github.com/NixOS/nixpkgs/pull/289748#issuecomment-2016935798
You are receiving this because you commented.
Message ID: @.***>

It's a breaking change, so it's not really supposed to be backported.

The DBus API stayed stable and backward compatible (by exporting the legacy name on top of the new one). I don't think this is a breaking change.
Only the driver names changed to take into account the new multiple driver setup.
I'm unfamiliar with the assumptions made in the KDE/Gnome parsers, though. Maybe they're making extra assumptions wrt. driver name values on top of the DBus API.

I actually want to report I may have encountered very weird issues, but it may be my hardware, I've reverted to 0.13, due to the laptop shutting down randomly while being plugged to an AC source.
I haven't checked if there is an issue upstream open for this weird behavior. But my hardware may have just overheated.

I did not see any related issue upstream. Would you mind opening one describing your problem in more details?

I had overheating issues due to the fabric under the laptop.
It triggered full shutdown for safety, nothing related to this PR it seems.
Reverting the PR or using this unstable pkg, the behavior is stable now.


Extremely odd behaviour - ReflectionException depending on number of unit tests.

Laravel Version: 5.7.16
PHP Version: 7.1.23
Database Driver & Version: SQlite (:inmemory:), 3.20.1

Description:

With a single test in my unit test, the tests run as expected.
With two or more tests, a ReflectionException is thrown.

This seems to only happen if I call $this->seed() or $this->artisan('db:seed') in setUp.
So, for example, with this code the tests run and I get an OK.
class GuildTest extends TestCase
{
    public function setUp()
    {
        parent::setUp();

        // $this->artisan('migrate:fresh');
        // $this->artisan('db:seed');

        $this->seed();
    }

    public function test_that_all_guilds_are_returned()
    {
        $this
            ->getJson('/api/guild')
            ->assertJson([
                [
                    'id' => 1,
                    'discord_id' => '185729604968495029',
                    'owner_id' => '106940394960393296'
                ],
                [
                    'id' => 2,
                    'discord_id' => '858396818693074861',
                    'owner_id' => '306940782386976078'
                ],
                [
                    'discord_id' => '682960486781665869',
                    'owner_id' => '106849682076849369'
                ]
            ]);
    }

    // public function test_my_sanity()
    // {
    //     $this->assertEquals(true, true);
    // }
}

But if I uncomment test_my_sanity (so now there are two tests), I get:
1) Tests\Unit\GuildTest::test_my_sanity
  ReflectionException: Class env does not exist

I'm absolutely tearing my hair out here. Any ideas?
Steps To Reproduce:
I'm not sure how I can give you steps to reproduce since this is happening on a private/closed-source project but I can give you absolutely any information you need.

My base TestCase -
abstract class TestCase extends BaseTestCase
{
    use CreatesApplication;
    use RefreshDatabase;
    // use DatabaseMigrations;
}

NOTE: Use statements removed for clarity.

I think I've semi-tracked down what is causing it but I still have no idea what is going on.
If I change one of the values I'm trying to insert in my seeder from "682960486781665869" to anything else then it goes away.
Which implies that perhaps the db isn't being reset properly. Even though I use RefreshDatabase.
And this is happening even if I do $this->artisan('migrate:fresh'); in setUp nor is it failing with a logical error like `"unique constraint failed".
What on earth is happening here?

Finally tracked it down to this.


In AbstractJobLauncher, open the job lock earlier and close it later.

@sahilTakiar Can you review this?

@chavdar Can you get this to the right reviewer?

@abti Can you please review this?


Haskell Dev Container definition wrong

VSCode Version: 1.72.2
Local OS Version: Windows 11 22H2
Local chip architecture: x64
Reproduces in: Remote - Containers
Name of Dev Container Definition with Issue: Haskell
Docker version output:

Steps to Reproduce:

Press F1: Add Dev Container configuration file > Haskell (Community)
Press F1: Reopen in Container
remoteContainers-2022-10-28T10-33-48.150Z.log

You can close this issue. There is a bug in Dev Container extension


Fixed basemaps modal UI

Fixes #8956

CR @matallo

noticed but not mentioned it, thanks!
btw, what's that lib/assets/javascripts/cdb change?

OMG I forgot to ignore that, hold on.

✨


how to replace kubeshare executables with the local build?

tried to build kubeshare-config-client docker image. It failed to build with the error:  "0 added, 0 removed; done.
Running hooks in /etc/ca-certificates/update.d...
done.
2022-01-26 18:13:36 URL:https://storage.googleapis.com/golang/go1.17.6.linux-amd64.tar.gz [134830580/134830580] -> "-" [1]
make: *** No rule to make target 'kubeshare-config-client'.  Stop.
The command '/bin/sh -c apt update &&     apt install -y g++ wget make &&     wget -nv -O - https://storage.googleapis.com/golang/go${GOLANG_VERSION}.linux-amd64.tar.gz | tar -C /usr/local -xz &&     make kubeshare-config-client' returned a non-zero code: 2"
Does it require the mount drive?

copy Dockerfile to /KubeShare, build from there.


WIP: Refactor Elasticsearch controller to use a semi-state machine design

What this PR does / why we need it:
This change refactors the Elasticsearch controller to use a state-machine 'style' of operation.
We introduce an 'Action' interface with a simple spec:
type Action interface {
	Name() string
	// Execute should attempt to execute the action. If it is not possible to
	// apply the specified changes (e.g. due to the cluster not being in a
	// 'ready state', or some transient error) then an error will be returned
	// so the action can be requeued. This allows for non-blocking blocking of
	// actions, with retries. The workqueues default scheduling and rate limit
	// will thus handle fairness within Navigator, and handle backing off on
	// retries.
	Execute(state *State) error
}

The cluster_control.go contains the 'state machine' like logic. For now, this is implemented as a simple function that uses if/else branches to determine which 'Action' to return depending on the current state of the cluster.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Fixes #215
Special notes for your reviewer:
I'm currently working on a general purpose unit testing fixture, StateFixture that can be used to mock the State structure. This should allow for easy testing of all of our Action implementations, between Elasticsearch and in future Cassandra.
Right now, this PR does not properly wait for document count to be zero during scale down, however it does manage a rolling upgrade across multiple statefulsets by careful management of the partition field on statefulset.spec.updateStrategy.rollingUpdate`.
Things still to do:

managed scale down
support changing node pool resources
support changing node pool roles
support changing sysctls
support changing pilot image
support changing securityContext

Release note:
Introduce a managed node pool upgrade process

/cc @mattbates @simonswine @wallrj

/test verify

/test e2e v1.9

@wallrj this PR has grown large, so I'm not going to continue any more work here. Our e2e tests are passing and stand up an ElasticsearchCluster (both single and multi-node), plus wait for the cluster to transition into a Yellow/Green state respectively. I'll be following up with more PRs to address the remaining items:

managed scale down
support changing node pool resources
support changing node pool roles
support changing sysctls
support changing pilot image
support changing securityContext

I'll also be expanding the test suite further, to:

test performing an upgrade
test writing data to one node and reading from another
test reading & writing data whilst the cluster is in a degraded state
test updating resources/other fields in the cluster

This PR also introduces two new test frameworks:

the StateFixture, which is useful for unit/integration testing controller.Actions. I've also added e2e tests for the upgrade and scale actions, that ensure the controller behaves as expected under various 'initial states'

the ginkgo e2e test framework. This is based off the kubernetes/kubernetes test framework.

Once merged, we can start to migrate the remaining cassandra e2e tests to this new format too. It appears to be running stable, and cleans up after itself nice and quickly 😄
Please reach out if you need a hand with review/if there's anything I can do to help.


Fix annotation definition not captured issues

Purpose
Fixes #29213
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[x] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Seems like this fixes #29131 too


Update to latest dependencies

Codecov Report

Merging #68 (178a0c0) into master (f6b3b73) will increase coverage by 0.28%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #68      +/-   ##
==========================================
+ Coverage   93.63%   93.92%   +0.28%
==========================================
  Files          56       56
  Lines        2735     2848     +113
==========================================
+ Hits         2561     2675     +114
+ Misses        174      173       -1

Impacted Files
Coverage Δ

src/hcrystalball/model_selection/_configuration.py
96.42% <ø> (ø)

tests/unit/model_selection/test_model_selection.py
100.00% <ø> (ø)

tests/unit/wrappers/test_prophet_wrapper.py
100.00% <ø> (ø)

...ests/unit/model_selection/test_data_preparation.py
97.64% <100.00%> (+0.02%)
:arrow_up:

...sts/unit/model_selection/test_prefect_executors.py
100.00% <100.00%> (+4.76%)
:arrow_up:

tests/unit/wrappers/test_base.py
100.00% <0.00%> (ø)

tests/integration/test_frequency.py
100.00% <0.00%> (ø)

tests/unit/wrappers/test_tbats_wrapper.py
100.00% <0.00%> (ø)

tests/unit/wrappers/test_sarimax_wrapper.py
100.00% <0.00%> (ø)

... and 33 more

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 713636e...178a0c0. Read the comment docs.


Add Node for publishing events.

Thanks for the PR @techlemur, good idea to add the ParticlePublish node to the package.
processSSE() shouldn't need to be called within your proposed new ParticlePublish node: https://github.com/chuank/node-red-contrib-particle/blob/dcacad928c8d225db1a730757ec8e20644b5a4d0/particle/particle.js#L267-L269
While you're at it, take out your older test code here:
https://github.com/chuank/node-red-contrib-particle/blob/dcacad928c8d225db1a730757ec8e20644b5a4d0/particle/particle.js#L294-L302
Will merge and bump up the npm package when this is done.

Thanks for catching those two issues.  Should be fixed now.  I plan to add privacy and ttl options soon, do you want those added before the merge?

I'll merge this first, thanks!
Public/private event types are a good addition, so feel free to add that in. I'd suggest simple radios or a dropdown menu UI element (GUI-wise, checkboxes seem ambiguous for a public/private option).
I have considered TTL implementation when I started work on this node, but left it out as the official docs don't seem to support a user-defined TTL interval and I didn't want to confuse users:
https://docs.particle.io/reference/firmware/photon/#particle-publish-
Then again the Particle documentation hasn't been updated in a long while; have you tested TTL timeouts? In terms of your ParticlePublish node, my suggestion is that you add support for TTL, but note this in the html docs.

@techlemur please help to test this commit, appreciate it. I've also added the private/public and TTL option.


Text clean-up

Description & motivation
This PR removes whitespace from an inline code snippet and removes an "Options" header that had no contents.

thanks again @deanna-minnick - one small tweak!

good catch, @mirnawong1! I've committed that change and merged the most recent changes from current into this branch.


Fix memory leak when listening to 'message' events from child iframes

Related to #184. fromIframe message endpoint has two bugs:

It will listen to message events even after the promise is resolved or the iframe is removed. This will continuously compound as iframes are added/removed from the DOM.
It's also possible that the remote-ui::ready message is never received (e.g. the iframe fails to lead), so the event handler is never cleaned up.

This PR addresses both issues to avoid memory leaks.
I didn't address testing in this PR, but happy to add as directed 😄

@lemonmade done!

Congratulations, you are the proud new owner of the very first Changesets-based release of remote-ui, with<EMAIL_ADDRESS>😛 Thank you again for the great fix!

Awesome - glad to help! 💅


Tongue Visualizations

Hi @salmedina & @salvadom
Thanks for this great paper,
I read your paper and saw the tongue visualizations you made there and was wondering if you can share the code you used to visualize the tongue using the provided 3d landmarks in the dataset?
Thanks,
Tal

Hi Tal,
Thank you for your interest. Yes, I am working on releasing a nice piece of code for everyone. I've been busy with my dissertation, but expect a release soon.

Bump…just wondering if you’re planning to release the tongue visualization code?

Would you consider making the palate_mesh file public on GitHub? It would be a valuable addition to the repository.


Secondary buttons look exactly like disabled buttons

Describe the Bug

The "preview" and "save as draft" buttons are grey to separate them from the "publish/update" buttons.
However, this has the disadvantage that they look exactly like disabled buttons.
Expected Behavior

It's ok to separate them from the "main" button by a separate color, but grey is a bad choice
Actual Behavior

They look disabled
Additional Information

For the "preview" button I think it  could make sense to use the same color as for filters (since the meaning of the light blue could be described as "something that has no immediate influence on the state of the system"):

For "draft" this isn't ideal however, since it changes the database and the click itself cannot be reversed.

As an option - make the disabled button lighter + gray text?

@digitalfabrik/ui-ux-team do you have input on this?

I know my opinion isn't really asked here, but I like outline and ghost-buttons as secondary buttons
. I think if we only have two levels of hierarchy I would go for outline buttons, if we have more than two I would go for ghost buttons

@JoeyStk Cool idea! If we don't get a response by the UI/UX team, I think we can go ahead and implement this guideline:
        &.btn-ghost {
            @apply bg-blue-100 text-blue-500;
            &:hover {
                @apply bg-blue-200;
            }
        }
        &.btn-outline {
            @apply border-2 border-blue-500 text-blue-500 bg-white;
            &:hover {
                @apply bg-blue-100;
            }
        }

@JoeyStk @timoludwig from an ui/ux perspective the outlined button is the best choice here, as it complies with best practice standards, see Google Material components ("Use an outlined button for actions that need attention but aren’t the primary action[ ...]"
If you tag @digitalfabrik/ui-ux-team , you can also tag @hauf-toni , this will also point me to the thread via mail. I appreciate that, thank you :)

@hauf-toni ah sorry, I thought mentioning the team had the same effect, but ok I will use the individual username in the future.
Would be cool though if we could somehow fix the notification settings, since the team name is easier to remember...

We discussed this issue in our bi-weekly-meeting between service team, UI/UX team and our team. We came up with the following solution:

Secondary buttons should become outline buttons (as seen in the screenshot above)
Buttons that fulfill a different purpose than the rest of the buttons on the page should become ghost buttons (as seen for draft in the screenshot above)
Disabled buttons should remain grey, but if possible a less saturated grey tone (for example #A1A1AA)
Add styleguide / documentation for this so future developers have a guidance when making the choice

With this information the issue should be ready for implementation

@JoeyStk Sounds good, thanks for clarification!

Disabled buttons should remain grey, but if possible a less saturated grey tone (for example #A1A1AA)

I would suggest to stick with our existing color palette, so ideally we could choose a lighter grey from https://tailwindcss.com/docs/customizing-colors
(At the moment, it is gray-500/#6b7280, so we could move to gray-400/#9ca3af)

Yes, sounds reasonable. @hauf-toni was referring to the Integreat color scheme, but also said we were free in making the final decision of what works best for us. So let's try gray-400 then :+1:


Automate version updating process

See this PR - https://github.com/frictionlessdata/datapackage/pull/975 - creating new profiles version required to change version numbers in many places manually. This process needs to be automated (e.g. using a version variable)

Agreed. In addition:
We should have a mechanism to separate minor corrections to the website (live immediately) from changes that are part of a new release. The easiest solution I can think of:

main branch = website. PRs that make quick corrections (e.g. #978) should be merged with this branch.
upcoming-version branch = all changes for a next version. PRs that affect the standard (e.g. #961) should be merged with this branch.

upcoming-version can be brought in sync by merging in main, but the reverse should only be done when we plan to release a new version. That could kick off a bunch of build scripts (preferably a python script triggered by a GitHub Action) that:

Create generated files
Update version numbers
Make a GitHub release


Core shut down already

Hi there - thanks for this library.
I'm experiencing a problem where there should be no workers running, having had some running, and then the pool goes out of scope with the following panic:
thread 'thread 'thread 'test tests::test_greeting ... thread 'cb-ws-pool-worker-0cb-ws-pool-worker-2cb-ws-pool-worker-1' panicked at '' panicked at '' panicked at 'Core shut down already!Core shut down already!Core shut down already!cb-ws-pool-worker-3', ', ' panicked at '', /Users/myuser/.cargo/registry/src/github.com-1ecc6299db9ec823/executors-0.8.0/src/crossbeam_workstealing_pool.rs/Users/myuser/.cargo/registry/src/github.com-1ecc6299db9ec823/executors-0.8.0/src/crossbeam_workstealing_pool.rs/Users/myuser/.cargo/registry/src/github.com-1ecc6299db9ec823/executors-0.8.0/src/crossbeam_workstealing_pool.rs:ok::818818
:818::44Core shut down already!4444

Any pointers? Thanks.
(Using 0.8.0)

Hmm, can you show me how you are shutting it down? Usually this happens if you don't shut it down properly before dropping the last pool reference.
It also isn't necessarily a problem. It just means a worker was trying to run its run loop after the core was deallocated already. So if you aren't losing any work that should have been done in the process, you could theoretically ignore it, unless you have to run with panic=abort. It's just a very brutal way of making sure no pool threads end up orphaned.

Thanks for replying so quickly. I’m not in front of the code at the mo, but I’m not doing anything specific to shut the pool down. Should I be, other than letting the pool go out of scope? I didn’t see a specific method to call...

Yes, if you want to clean things up nicely, you should call either shutdown() or shutdown_async() on the pool before letting it go out of scope. This is so you can control where you incur the cost of an orderly shutdown, instead of just happening wherever you happen to drop the last reference to the pool.

So sorry that I missed that. I’ll close this. Thanks again.


[DRAFT] change file name format

relates to #23

@pjfanning Is this ready or does it need more work?

@pjfanning Is this ready or does it need more work?

I was going to get this looked at after merging the sha512 change (they will conflict a bit) - but you're welcome to comment on the changes as they are.


Wrong NodeInternalIP

I run a Kubernetes without any cloud providers, and then start  the heapster container:
docker run kubernetes/heapster:v0.17.0 -v=5 --source=kubernetes:http://api-server?inClusterConfig=false&useServiceAccount=false

And the heapster get a wrong NodeInternalIP <IP_ADDRESS>
I0805 07:49:03.013146       1 kube.go:117] kube nodes found: &{Items:map[node2:{PublicIP:<IP_ADDRESS> InternalIP:<IP_ADDRESS> ExternalID:node2}

the node info:
{
  "kind": "Node",
  "apiVersion": "v1",
  "metadata": {
    "name": "node2",
    "selfLink": "/api/v1/nodes/node2",
    "uid": "2640f0a2-3b3a-11e5-973f-005056817c3e",
    "resourceVersion": "2431",
    "creationTimestamp": "2015-08-05T06:21:04Z",
    "labels": {
      "kubernetes.io/hostname": "node2"
    }
  },
  "spec": {
    "externalID": "node2"
  },
  "status": {
    "capacity": {
      "cpu": "1",
      "memory": "1884812Ki",
      "pods": "100"
    },
    "conditions": [
      {
        "type": "Ready",
        "status": "True",
        "lastHeartbeatTime": "2015-08-05T08:25:44Z",
        "lastTransitionTime": "2015-08-05T06:35:05Z",
        "reason": "kubelet is posting ready status"
      }
    ],
    "addresses": [
      {
        "type": "LegacyHostIP",
        "address": "<IP_ADDRESS>"
      }
    ],
    "nodeInfo": {
      "machineID": "e3448a36021a4ac494c01a7fb65a9ffa",
      "systemUUID": "4201C1A8-F3A0-EE67-AEDF-833EDEF665B1",
      "bootID": "aae79eff-e3d7-475d-91ef-5bead8231d97",
      "kernelVersion": "3.10.0-229.el7.x86_64",
      "osImage": "CentOS Linux 7 (Core)",
      "containerRuntimeVersion": "docker://1.6.0",
      "kubeletVersion": "v0.21.4",
      "kubeProxyVersion": "v0.21.4"
    }
  }
}

I run the heapster on host directly, and it get the right NodeInternalIP
I0805 18:17:02.318569   23839 kube.go:117] kube nodes found: &{Items:map[node2:{PublicIP:<IP_ADDRESS> InternalIP:<IP_ADDRESS> ExternalID:node2}

Is this code[1] gets some problem inside the conatiner?
addrs, err := net.LookupIP(hostname)

[1]https://github.com/GoogleCloudPlatform/heapster/blob/v0.17.0/sources/nodes/kube.go#L82

That's correct. I wonder why kubernetes is not populating the node internal
IP for your cluster.
On Wed, Aug 5, 2015 at 3:19 AM, Tim Wu<EMAIL_ADDRESS>wrote:

I run the heapster on host directly, and it get the right NodeInternalIP
I0805 18:17:02.318569   23839 kube.go:117] kube nodes found: &{Items:map[node2:{PublicIP:<IP_ADDRESS> InternalIP:<IP_ADDRESS> ExternalID:node2}
Is this code[1] gets some problem inside the conatiner?
addrs, err := net.LookupIP(hostname)
[1]
https://github.com/GoogleCloudPlatform/heapster/blob/v0.17.0/sources/nodes/kube.go#L82
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/heapster/issues/455#issuecomment-127948973
.

Because my Kubernetes's cloud providers is nil, the code[1] only set NodeLegacyHostIP.
Is that ok to add NodeInternalIP?
[1]https://github.com/GoogleCloudPlatform/kubernetes/blob/master/pkg/kubelet/kubelet.go#L1974

I think yes. Can you post a PR to fix that?
On Wed, Aug 5, 2015 at 6:13 PM, Tim Wu<EMAIL_ADDRESS>wrote:

Because my Kubernetes's cloud providers is nil, the code[1] only set
NodeLegacyHostIP.
Is that ok to add NodeInternalIP?
[1]
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/pkg/kubelet/kubelet.go#L1974
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/heapster/issues/455#issuecomment-128199488
.

@vishh OK, I will commit it soon

@wulonghui: Closing this issue, since its fixed in Kubernetes


Fix crash when no plugins are specified

If no plugins are specified, attempting to iterate over arg.plugins
(which happens as a result of the membership test) results in a crash.

Sorry, I didn't realise this was from @goldmann's plugins branch; I've raised the PR there.


Update livereload.js

Fix for Chrome bug: https://github.com/livereload/livereload-extensions/issues/26#issuecomment-54250594

I can merge this if it helps others - please ask people to add +1s here to confirm.

I've made it so you can change those values in the Guardfile: https://github.com/guard/guard-livereload/pull/147
If there are any issues there, please open a new PR.


Out of memory protection feature.

Adds possibility to adjust OOM killer points of running process. Just to be sure nobody hurts except us when memory ends.

Thank you kind officer!


It is fix batchsize in training and evaluation.

When batchsize is not chosen appropriately, accuracy may exceed 1.

LGTM :+1:

Thank you for the patch!


No module named 'triton'

大佬们!请问这是什么问题?

pip install triton

谢谢大佬,爱你~

我也遇到了 但是pip install triton也不行

pip install triton

我也遇到了这个问题，但是pip install triton不行

俺也一样
pip install triton
ERROR: Could not find a version that satisfies the requirement triton (from versions: none)
ERROR: No matching distribution found for triton

Running pip install https://huggingface.co/r4ziel/xformers_pre_built/resolve/main/triton-2.0.0-cp310-cp310-win_amd64.whl
Installed Triton on my Windows PC and I am able to use the module in python now, working perfectly

Running pip install https://huggingface.co/r4ziel/xformers_pre_built/resolve/main/triton-2.0.0-cp310-cp310-win_amd64.whl
Installed Triton on my Windows PC and I am able to use the module in python now, working perfectly

Thx！！！！！！U re my god！

请问在arm架构的mac上或者arm架构的ubuntu上怎么解决呀？
I wonder how to solve this problem in Mac or Ubuntu ,whose arch is ARM?


Too many queues to execute ??

In mongod.log file, I find a lot of hangfire executions every 15 seconds. It seems to many.

I don't know if this holds the mongodb. Because sometimes other applications couldn't query database .
And you can see that almost 99% query are all related to hangfire .

@one0410,
thanks for posting. You can configure hangfire poll times in the options you give. "QueuePollInterval" will determine when the db is polled per worker.
default value is 15 seconds. eg. if you have 20 workers they will check the db for jobs each 15s.
The jobs are defined in its own collections which doesnt have influence on other collections. also they are reads which does not lock documents. looking at the logs provided, mongo doesnt seem overwhelmed though.
Things to try:

You should define the jobs in its own db. just so they are completely separate from you business logic. this doesnt effect performance.
You can set "QueuePollInterval" to 1 minute.
You can disable notifications by setting "UseNotificationsCollection" to false. this will disable the tailable cursor listening for queued jobs.

hope this helps. :)

@gottscj Thank you for your quickly response.
I have another question.
I only have 2 jobs defined. But why there are almost 20 concurrent queries running ???
Do I do something wrong ??

@one0410,
You can specify the worker counter in the backgroundworker options provided by Hangfire.Core:
the default value is set to: Math.Min(Environment.ProcessorCount * 5, 20);
hope this helps :)


deps warning when run npx @acala-network/eth-rpc-adapter

it's back...
@polkadot/util-crypto has multiple versions, ensure that there is only one installed.
Either remove and explicitly install matching versions or dedupe using your package manager.
The following conflicting packages were found:
	cjs 9.4.1	node_modules/@acala-network/eth-transactions/node_modules/@polkadot/util-crypto/cjs
	cjs 9.4.1	node_modules/@polkadot/api-derive/node_modules/@polkadot/util-crypto/cjs
@polkadot/wasm-crypto has multiple versions, ensure that there is only one installed.
Either remove and explicitly install matching versions or dedupe using your package manager.
The following conflicting packages were found:
	cjs 6.1.1	node_modules/@acala-network/eth-transactions/node_modules/@polkadot/wasm-crypto/cjs
	cjs 6.1.1	node_modules/@polkadot/api-derive/node_modules/@polkadot/wasm-crypto/cjs
......

https://github.com/AcalaNetwork/acala.js/pull/371
Acala.js upgraded peerDependencies to cause this error. now fixed.
Delete the npx cache, then run npx @acala-network/eth-rpc-adapter and will no longer see this message.


Create radio test for system check script

System check script should not contain radio stuff, requires another board to test.


Observer-Concepts - predecessor description is incorrect?

On page:
https://github.com/YaccConstructor/QuickGraph/wiki/Observer-Concepts
It says:
VertexPredecessorRecorderObserver,
creates a dictionary that links vertices to their parent edge,
and
EdgePredecessorRecorderObserver,
create a dictionary that links vertices to their parent edge,
They are worded essentially the same. Is this intentional?
I think one of them probably should say "...that links edges to their parent vertex" but I'm not sure which just based on the class names.

Hello,
On my fork  I re-created the wiki for QuikGraph that I fully read and adjusted to fit evolutions of the library.
Concerning this point, the correct thing is: creates a dictionary that links edges to their parent edge, since it keeps a links between edges and not vertices.
See wiki.


add(Encord blog): Guide to Reinforcement Learning from Human Feedback (RLHF) for CV

Hi there,
Thank you for curating the repo. We'd love to contribute one of our most-referenced blogs, "Guide to Reinforcement Learning from Human Feedback (RLHF) for Computer Vision."
We think it could be a good value-add 🙌🏽.

Thanks for your contribution! We will merge this PR soon.


Contrib to Jenkins Chart for Pod Naming

Right now the chart names pod with default. Contribute upstream to make that better.
https://github.com/kubernetes/charts/blob/master/stable/jenkins/templates/config.yaml#L35

Fixed with agent.podName from the upstream chart.


Add return type will change attributes for php 8.1

From PHP 8.1 onwards some internal interfaces have been changed by adding return types and argument types. This will currently emit warnings and/or exceptions depending on the configuration. For backward compatibility, PHP has added the ReturnTypeWillChange attribute. This will signal to PHP that the method declaration will change from the internal interface thus preserving backward compatibility. These attributes can be removed once the library exclusively runs on PHP 8.1+.
Related Issue or Design Document

Did not create an issue, opted to create a PR instead.
Some external reading material about this: https://php.watch/versions/8.1/ReturnTypeWillChange

Checklist

[x] I have read the contributing guidelines
and signed the CLA.
[ ] I have referenced an issue containing the design document if my change
introduces a new feature.
[x] I have read the security policy.
[x] I confirm that this pull request does not address a security
vulnerability. If this pull request addresses a security vulnerability, I
confirm that I got green light (please contact
<EMAIL_ADDRESS>from the maintainers to push
the changes.
[ ] I have added tests that prove my fix is effective or that my feature
works.
[ ] I have added necessary documentation within the code base (if
appropriate).

Further comments

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

Hi @aeneasr can you review this PR?
The contributing guidelines state to request a review from a maintainer, linking to the GitHub docs where you are able to select a reviewer. Unfortunately this is not possible for me (and possibly other people not in the organization). Might want to address this, or publish a list of maintainers to mention in a comment 👍

Thank you so much for your hard work on this! Unfortunately this repository is fully auto-generated (see README: https://github.com/ory/hydra-client-php#generation )
Any changes you make will be overwritten in a few days by the pipeline. The changes you've made however can still be done in the OpenAPI Generator project which we use to generate these files!
Sorry for the hassle...

I am closing this one to declutter our review log, your contribution and ideas are welcomed though any time :)


KeyError with non-str key

import inspect

from friendly_traceback.info_specific import get_likely_cause

try:
    {}[0]
except KeyError as e:
    get_likely_cause(type(e), e, {}, inspect.currentframe())

Traceback (most recent call last):
  File "/home/alex/.config/JetBrains/PyCharm2020.2/scratches/scratch_874.py", line 8, in <module>
    get_likely_cause(type(e), e, {}, inspect.currentframe())
  File "/home/alex/.cache/pypoetry/virtualenvs/book-E12DIahb-py3.8/src/friendly-traceback/friendly_traceback/info_specific.py", line 20, in get_likely_cause
    cause = get_cause[etype.__name__](value, info, frame)
  File "/home/alex/.cache/pypoetry/virtualenvs/book-E12DIahb-py3.8/src/friendly-traceback/friendly_traceback/info_specific.py", line 109, in key_error
    ).format(key_name=str(value).split("'")[1])
IndexError: list index out of range

I suggest:

Not referring to it as 'the name of the key'
Showing the repr without stripping quotes. Not only will this not break, it will distinguish between 1 and "1".

    return _(
        "In your program, the key\n"
        "that cannot be found is `{key_name!r}`.\n"
    ).format(key_name=value.args[0])

Good suggestions, thanks.


[Question] how to get dense textured mesh

Hi! I'm comparing OpenSfM results to OpenMVS. Since I'd prefer using OpenSfM due to it's webplayer and compability with machines that are used here, I'm trying to get the same result I'm getting from OpenMVS.
With the standard config of OpenSfM whoever I get a much faster calculation (aprox. 5 min compared to about 30 mins with Visual FSM for sparse point cloud and OpenMVS for dense, resconstruction, refinement and texturing accoring to https://groups.google.com/forum/#!topic/vsfm/VcXGvP1HuMo on an i7 4,2ghz 6 core, GForce 1070) but it will get me a way less accurate result when running bin/opensfm_run_all. Since I'm using default settings for both of these configurations I wonder if anybody could help me getting the correct settings to construct a similar result with OpenFsm
EDIT: just realized, is OpenSfM doing a textured mesh calculation at all?

@pierotofy thx for your reply! Is there any way I can import a dense(+textured) model into the web viewer/ openSfm?

With some coding, probably yes. The viewer is built on top of three.js, so check out the documentation for threejs on loading OBJ files (+ MTL). https://threejs.org/docs/#examples/loaders/OBJLoader

@chris-aeviator Are you only looking at the online reconstruction? This is a sparse mesh just for demonstration purposes as far as I can see. You actually want to look at "merged.ply" in the depthmaps folder within your project folder.

No answer in a while, closing.


Revert "Rename Schlick GGX to GGX."

Reverts godotengine/godot#12296 as per https://github.com/godotengine/godot/pull/12296#issuecomment-338517506

@kubecz3k FYI :)

@akien-mga hah thanks :)


(re)adding C++ support

Hey👋
Last week I tried to upgrade a WPILIB project to the 2023 Beta. I quickly realized all our dependencies broke, but rather than waiting for an update, I decided to upgrade any dependencies I could manually as I didn't see any updates yet. Today I noticed your project and thought I'd drop in my solution since you haven't had a chance to get to C++ support yet. I still need to test it on an actual robot (which hopefully will happen this Thursday) and I need to go through and fix all the warnings before re-enabling treating warnings as errors, but for now it compiles and passes tests.

Hi, just saw this PR. I saw the comment on the discussion board.
I am curious where you pulled the sources- it's missing a decent amount of stuff for the newer versions of the navX firmware.
I grabbed the 2022 sources and headers from here.
The CI builds are not failing in relation to this pull request- wpilib as a whole is being hit by this.

I pulled straight from https://github.com/kauailabs/navxmxp, so I'm not sure why it would be outdated, unless there was another source last year.

That repository hasn't been updated since April 2021- they stopped doing it open source for (from what I've heard) potential IP infringement. They still published sources to Maven, however, which is where I pulled the source from.

Ah, thank you. My team ran 40.0.441 this year; I did not realize there were any updates until today. I'll try to update my code later this week.

If it's alright with you I'm gonna pull in the 2022 sources and add the changes you made to make it build. I verified that the 2022 sources with those changes at least run on the roborio and get some data back. I want to get a c++ release out sooner than later so teams can use it for beta.

Should now be ready for merge. I updated to the latest version, made it compile, and then fixed the warnings and reenabled treating warnings as errors.


Setting upload speed disables autoerase for flash memory

I'm using PlatformIO to upload to standalone ATmega328p using Adruino Nano as programmer.
My env config:
[env:ATMega328p_8MHz]
# Standalone ATMega328p running on 8MHz
platform = atmelavr
board = pro8MHzatmega328
board_mcu = atmega328p
board_f_cpu = 8000000L
framework = arduino
# Upload using programmer
upload_protocol = stk500v1
upload_speed =19200

build_flags = -Wall
# Adafruit-DHT, LowPower, RF24, RF24Network
lib_install = 19,38,433,435

With this config, platformio run -t upload fails with following error:
avrdude: verifying ...
avrdude: verification error, first mismatch at byte 0x0000
0x08 != 0x6a
avrdude: verification error; content mismatch

The avrdude command is this:
"/home/radek/.platformio/packages/tool-avrdude/avrdude" -v -p atmega328p -C "/home/radek/.platformio/packages/tool-avrdude/avrdude.conf" -c stk500v1 -b 19200 -D -P /dev/ttyUSB1 -U flash:w:.pioenvs/ATMega328p_8MHz/firmware.hex:i

I've figured out that if I remove '-D' command line option from avrdude command, it works fine.
When upload_speed is defined, the '-D' is automatically inserted:
https://github.com/platformio/platformio/blob/release/v2.3.4/platformio/builder/scripts/atmelavr.py#L35

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/27587043-setting-upload-speed-disables-autoerase-for-flash-memory?utm_campaign=plugin&utm_content=tracker%2F2014790&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F2014790&utm_medium=issues&utm_source=github).

See related issue where -D option is required (Arduino IDE also uses it).
If you don't need this option, you can remove it using extra-script.
from SCons.Script import DefaultEnvironment

env = DefaultEnvironment()

env.Replace(UPLOADERFLAGS=env['UPLOADERFLAGS'].remove("-D"))

P.S: I've not tested it. It just a hint.

See https://github.com/platformio/platformio/blob/release/v2.3.4/platformio/builder/scripts/atmelavr.py#L90

I can't get the extra-script to working, your example fails with ValueError: list.remove(x): x not in list. It seems that the BeforeUpload function is called after the extra-script, so that env var is not defined completely. The UPLOADERFLAGS env contains ['-v', '-p', '$BOARD_MCU', '-C', '"$PIOPACKAGES_DIR/tool-avrdude/avrdude.conf"', '-c', '$UPLOAD_PROTOCOL'] in the extra-script.
https://github.com/platformio/platformio/blob/develop/platformio/builder/scripts/atmelavr.py#L17
I'm not familiar with SCons, so I haven't yet figured out how to work it around.

Are you sure that Arduino IDE needs -D option? It didn't used it when I tried - that's how I found that it's causing troubles for me. The difference might be that I used Sketch -> Upload Using Programmer option from the menu instead of normal Upload button (I need to program ATmega using Arduino as ISP).

I've just tested it, sorry. Please see updated comment https://github.com/platformio/platformio/issues/311#issuecomment-149967444

Now it works (I just had to add "-P", "$UPLOAD_PORT" to the list of options). Thank you very much.
Do you consider this use case as something that PlatformIO should supports? It would be great to do it without custom scripts.

Which is default behaviour of Arduino IDE?

It seems it depends if you program Arduino itself (upload) or program something using Arduino as ISP (program):
(from platform.txt):
tools.avrdude.upload.pattern="{cmd.path}" "-C{config.path}" {upload.verbose} -p{build.mcu} -c{upload.protocol} -P{serial.port} -b{upload.speed} -D "-Uflash:w:{build.path}/{build.project_name}.hex:i"

tools.avrdude.program.pattern="{cmd.path}" "-C{config.path}" {program.verbose} -p{build.mcu} -c{protocol} {program.extra_params} "-Uflash:w:{build.path}/{build.project_name}.hex:i"

I hope I understand it correctly, I'm not very familiar with this.

Thanks. I will implement it via additional target named program. You should use platformio run -t program instead of platformio run -t upload

That makes sense. Thank you.

Please re-test it using the latest development version http://docs.platformio.org/en/latest/installation.html#development-version

Sorry for the delay. It works perfectly now. Thank you.

Thanks for the report and that using @PlatformIO


SonarQube

Test the project with SonarQube and find bugs.

Added SonarQube config file.
af07d27

I have already fixed bugs and vulnerabilities, but I forget to pull request it.
969bec7

Duplicated issue: #3

I checked the bugfixes, they seem okay.

OK


Issues with Bloodhound + %QUERY

Previous versions of Bloodhound used to let me do the following:
var bloodhound = new Bloodhound({
remote: '/search/?term=%QUERY'
});
However %QUERY now doesn't seem to work - whats the new method of doing this? I've trawled the docs to no avail...

var bloodhound = new Bloodhound({
remote: '/search/?term=%QUERY',
wildcard: '%QUERY'
});

This doesn't seem to work for me, yet if i edit typeahead.js directly, and change line 712 from
wildcard: null,
to
wildcard: '%QUERY',
then it does work. Is this a possible bug?

can you post your full initialization object?

I ran into this problem too and the solution you recommended worked for me.  I have a pull request clarifying this documentation and I'd also like to know why the solution didn't work here.

I changed
remote: '/location_typeahead/%QUERY'
to
remote:
url: '/location_typeahead/%QUERY'
wildcard: '%QUERY'
All is good...

@atstockland Thanks, that works!

thanx it works

yes!  Thank you!


Use organization member identity for member to remove from org

Hi @jackpgriffin,
heroku-go is generated from a Ruby script based on the Heroku API's JSON schema. While this PR fixes the generated code, this code would be overwritten if somebody runs the generator again.
The generator, though, is pretty tough to maintain, and I don't have the time right now to try and fix it. If you're not inclined to fix the code in the generator (I wouldn't blame you), I'd recommend you instead check out @cyberdelia's heroku-go. It uses a much more robust generator.

Sorry! Didnt read the big red warning at the top... Gave the ruby generator a shot but couldn't get it to work.


net::ERR_CLEARTEXT_NOT_PERMITTED error

Reproduce:

open a website with a typical 大外宣 audience
e.g.

http://chinaplus.cri.cn/recommended/1661/436968
http://en.people.cn/n3/2020/0317/c90000-9669139.html

see the message net::ERR_CLEARTEXT_NOT_PERMITTED

Cause:
Since Android 9, by default web widget disallow clear-text (non-https)
Why is it a problem:
Many websites in China opt to not to use https since the Party can't sensor what's in the https and might shut you down. (Big sites like Baidu have special backdoors so they are exempted.) Website especially those based in China for Chinese audience, demonstrates such bug (including Chinese blockchain-related sites). Even People's Daily English Version, which is oriented towards an international audience, failed to have the entire page covered in https example  Given China's mark up of global crypto users we must respond to the user's need.
Suggestion:
Display a warning message that the user can click through to open such websites.

@colourful-land I am new to this Org. Can I take up this issue ??

@colourful-land I am contributing first time to this Organization. Can I take up this issue ??

Hi @justdvnsh many apologies for the very late reply; yes please check if it is still an issue (may have been fixed in the meantime). If not, the solution may lie in AndroidManifest.
Also - if you wish to try to clear an old issue please go ahead and submit PR. We always like getting PR's from outside AW.


test and warn when rasterizing empty things

Codecov Report

Merging #525 (ba78903) into main (a0f6184) will increase coverage by 0.21%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #525      +/-   ##
==========================================
+ Coverage   80.26%   80.48%   +0.21%
==========================================
  Files          49       50       +1
  Lines        4165     4176      +11
==========================================
+ Hits         3343     3361      +18
+ Misses        822      815       -7

Files
Coverage Δ

src/methods/extract.jl
78.94% <100.00%> (ø)

src/methods/rasterize.jl
87.92% <100.00%> (+0.13%)
:arrow_up:

src/polygon_ops.jl
84.91% <100.00%> (+0.07%)
:arrow_up:

... and 5 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Update discord stable to v2.5.2

v2.5.2 is the same as 2.5.1 but with the latest fixes for sizes in jsonConfig.
Ref crycode-de/ioBroker.discord#76

RE-CHECK!


Websockets won't work with special chars in key

I think this could be clear to some, and not even considered a bug, but I think it should be mentioned that you can't place special characters in PUSHER_APP_KEY because it goes through the URL.

Thank you so much I didn't even think about this and was banging my head against the wall staring at 404 errors I couldn't figure out.


zenith: set subpath and path when parent module is git

The logic around Git refs having both Path and Subpath fields that each include the actual subpath is quite convoluted and this code all needs general refactoring/cleanup, but this fix is needed in the short term to fix problems elsewhere, so just making the smallest possible fix at this time.

Scan failure is unrelated, everything is failing w/ it now and it's being handled elsewhere.
The test that is still running is fine but GHA somehow got stuck on Post Run actions/checkout (that's a new one)

So merging now since neither of those matter.


Use venv instead of virtualenv if available

I'm wondering if it's worth using venv if available instead of virtualenv. That would also skip having to install virtualenv in the first place.

i don't think so, i prefer using the virtualenv command personally

Maybe it’s possible to make both options co-exist though? Casually browsing through the source I don’t see anything specifically coupled with Virtualenv except the command used to create the env. Virtualenv can always be the default, but the user can (for example) pass --venv to creates .venv with python3 -m venv instead of virtualenv.

I'd honestly be relying solely on pipenv and wouldn't care if virtualenv or venv was been used, but figured we could leverage builtins when available.

I urge @kennethreitz to reconsider his comment here. Using python3 -m venv instead of virtualenv is recommended by the standard documentation. I am personally put off using pipenv because it uses virtualenv. I only use Python 3 and only use venv  nowadays.

@bulletmark Now that Pipenv delegates almost all Virtualenv management to Pew it is even more difficult to switch. There is plan in pew to support venv however (berdario/pew#67) so there is still chance. If Pew can get it working it would be much easier (and make more sense to me at least) to implement venv support in Pipenv.

@uranusjr sounds great!

I've personally never used venv, hence my apprehension towards it. I'm sure it's good, though.
Things are working well, right now. No need to change things that aren't broken.

venv is simply not good enough to make this happen. If you really care about this, head over to the Python bug tracker and lend a hand.

It is not a problem of venv, but virtualenv. I have post some explanation under the issue tracker.

There is a wordaround ->
Use venv to create a virtualenv first, and then run pipenv install 🙈 .

Just a note:
The workaround above mostly works, but pipenv run ... fails for me because there is no activate_this.py under .venv/bin. If issue 21496 is to be believed, there is no plan on adding this file to venv. So even if pew fixes this, pipenv would likely still have a problem.

Using virtualenv, using system tk module in macOS is impossible. But using venv, it is possible.
I strongly want to use venv with pipenv (may be optionally).

Not to sound cynical, but I strongly want someone to actually write the code to support this if they really care so strongly. I’ve met numerous people saying they feel “strongly” about this, but as far as I know I am the only one to actually do anything. And I failed. This is not as simple as you think. If you think you can do better, do something. If you don’t want to, please take what you can get.

(kenneth and myself and probably erin and nate also tried to do this with venv much earlier in the project and also failed due to compatibility reasons)
This would be nice, everyone agrees. I'd love to see it working across platforms and python versions.

I would like to point out an other reason why venv would probably be a better choice than virtualenv:

one of the uglier aspects of virtualenv's implementation is that it has to have its own copy of the site module, which is used for all virtualenvs regardless of which version of Python they are created with.
-- from https://github.com/pypa/virtualenv/issues/228#issuecomment-4165148

This means, for example, that rlcompleter is disabled by default and that is a huge usability loss in my opinion.
Moreover since this is an issue that impacts multiple of your users, I would suggest keeping it open until the problem is fixed, even if you are waiting for some changes upstream to happen.

Again. We know it would be better. It’s not actually possible. What do you suppose has a more negative impact, completely breaking functionality for a large segment of users, or having some users have to turn on readline features?
I’m sure this is inconvenient, it’s also inconvenient that there only seem to be people demanding this and yet nobody who will offer an implementation that works. We have tried. Until something changes drastically we have no plans to support venv and therefore no plans to open this issue.

Seems right, I saw tox has a workaround, so maybe that would work if someone ported it over

pew! what a discussion.. if this pew thing is not that crucial, would it be possible to drop them altogether along with virtualenv and let pipenv, venv, and python go hand in hand, as they all recommend one another?
Packaging/virtual environment solutions for python is too much of a wild terrain right now, and some sacrifications need to be made, imho.

@doganmeh We can abandon virtualenv altogether, building an equivalent product to only support Python 3.3 or above, but I guess it would be easier to make that another project.

Just a note:
The workaround above mostly works, but pipenv run ... fails for me because there is no activate_this.py under .venv/bin. If issue 21496 is to be believed, there is no plan on adding this file to venv. So even if pew fixes this, pipenv would likely still have a problem.

Is that really necessary?  Once you've created a virtual environment with venv, you do not need to activate it to use it.  You simply run using the python binary from the venv/bin directory.

Dropping Python 2.7/virtualenv support to me doesn’t sound like a particularly terrible idea, if it does allow pipenv to rely on the stdlib of a Python version whose support is not about to expire. As long as it would not corrupt any other part of the pipenv toolchain that’s not easily fixable for the maintainers.
I would like to be able to contribute actual code, but I don’t think I have the experience necessary to do so or the time :frown: The best I can do is add to the discussion at the moment.

adding my voice here; i just hit this because of the same issue as @dmtucker .
I am currently resorting to ignoring pipenv completely for this project when on my mac, and making the environment by hand with venv for now.

+1 to use venv.
virtualenv have many wontfix bugs.
pipenv completely usefull on Windows 10 with Python 3.7 from Windows Store (virtualenv bug#1362)

virtualenv is moving to venv model - see https://github.com/pypa/virtualenv/issues/1366, that will generate pyvenv.cfg and all the good things, so this should be solved via that.

---------- Forwarded message ---------
From: Dan Ryan<EMAIL_ADDRESS>Date: Mon, Nov 26, 2018, 2:39 PM
Subject: Re: [pypa/pipenv] Use venv instead of virtualenv if available (#15)
To: pypa/pipenv<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Seems right, I saw tox has a workaround, so maybe that would work if
someone ported it over
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pypa/pipenv/issues/15#issuecomment-441533872, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AqyQvrs-RNNTb1YhxaDCWDLpRxrZLKoxks5uy4ydgaJpZM4Lqk2f
.


Jupyterlab bundler extension

I'm unable to find an information how to install a bundler extension into jupyter lab. Maybe someone can help?

As shown in https://github.com/jupyterlab/jupyterlab/pull/5217 the bundlers of nbconvert art automatically used. Thanks to the registration in https://github.com/m-rossi/jupyter-docx-bundler/pull/12 this bundler now also works for JupyterLab!


[FEATURE] Support retries for StreamingChatLanguageModel implementations

Sometimes it fails with
2024-04-09T06:14:47.7736852Z java.util.concurrent.TimeoutException
2024-04-09T06:14:47.7739451Z 	at java.util.concurrent.CompletableFuture.timedGet(CompletableFuture.java:1784)
2024-04-09T06:14:47.7740897Z 	at java.util.concurrent.CompletableFuture.get(CompletableFuture.java:1928)

Because of this, ITs are often failing.

Duplicate of https://github.com/langchain4j/langchain4j/issues/681


ModuleNotFoundError: No module named 'typing_extensions' when using pipenv and python 3.8

Hi, I have installed fastapi thru pipenv which indirectly require pydantic, and since I have asked pipenv to use python 3.8 with the setting
[requires]
python_version = "3.8"

I have started to get an exception on pydantic, which is the following:
.........virtualenvs\app-backend-ombd6ijq\lib\site-packages\fastapi\params.py:4: in
from pydantic.fields import FieldInfo, Undefined
pydantic_init.py:2: in init pydantic.init
???
pydantic\dataclasses.py:3: in init pydantic.dataclasses
???
pydantic\class_validators.py:8: in init pydantic.class_validators
???
pydantic\errors.py:5: in init pydantic.errors
???
pydantic\typing.py:23: in init pydantic.typing
???
E   ModuleNotFoundError: No module named 'typing_extensions'_
The pydantic code leading to this error is:
https://github.com/samuelcolvin/pydantic/blob/5ccbdcb5904f35834300b01432a665c75dc02296/pydantic/typing.py#L23
Pydantic manual says that typing extensions is not needed if python version is newer enough, but this line still requires it.
Is it possible for you to either:

remove this lib from the dependencies, or
make a clear statement on your install instructions that this dependency has to be installed  (for instance by telling it to be installed with pip install pydantic[typing-extensions]), or similar, or
make typing_extensions lib be automatically downloaded when pydantic is installed
somehow tell people from fastapi that they should add this dep?
tell people from typing_extensions that they should stop blocking python 3.8 on it?
any other?

I dont know what is the best way forward here, Im just proposing some ideas.
Thanks,

Hi @namoscagnm
typing-extensions IS required.

Pydantic manual says that typing extensions is not needed if python version is newer enough

Can you please share the link to this statement in the doc ?
Thanks

I am not sure why but pipenv is not installing typing-extensions, I think this is due to the configuration that pipenv is using to install dependent packages, because when You use pip install pydantic it will install typing-extensions for python 3.8 version, even though it is not required.

Can't reproduce

@namoscagnm @PrettyWood
I have this problem when compiling a FastApi app using Nuitka
pydantic==1.9.0
fastapi==0.7.20
Nuitka==<IP_ADDRESS>
pipenv, version 2021.11.23
python is 3.9.6 installed with pyenv; the problem is not present when I downgrade to python 3.7.11
How did you fix this @namoscagnm ?


Rename Tram preset

Is "Tram" a correct name for railway=tram?
I think it should be renamed becasue "tram" is a vehicle...
Although there is a meaning for "tram" as "One of the rails of a tramway"
https://en.wiktionary.org/wiki/tram#Noun
but it's marked as "obsolete".

Maybe tramway would be better?

@maro-21 Good point. A bunch of the railway presets looked unclear so I added "Track" to them, like Tram Track, Train Track, etc.

Great! Thanks!
What about Railway Under Construction preset and similar? Check please if they're ok, or should also be renamed.
Railway Under Construction: railway=construction
Abandoned Railway: railway=abandoned
Disused Railway: railway=disused
Disused Railway Feature: disused:railway=* (this one is ok I think, becasue it applies not only to tracks)


DRAFT: add verbose logs to E2E

@sadym-chromium As requested, here's the PR that adds verbose logging to E2E to help debug the flakiness in E2E (ubuntu-latest, headless).
Sample failure: https://github.com/GoogleChromeLabs/chromium-bidi/actions/runs/4861063256/jobs/8665646389
tests/test_browsing_context.py ..node:internal/errors:705
  const ex = new Error(msg);
             ^

Error: socket hang up
    at connResetException (node:internal/errors:705:14)
    at Socket.socketOnEnd (node:_http_client:[51](https://github.com/GoogleChromeLabs/chromium-bidi/actions/runs/4861063256/jobs/8665646389#step:10:52)8:23)
    at Socket.emit (node:events:[52](https://github.com/GoogleChromeLabs/chromium-bidi/actions/runs/4861063256/jobs/8665646389#step:10:53)5:35)
    at endReadableNT (node:internal/streams/readable:1358:12)
    at processTicksAndRejections (node:internal/process/task_queues:83:21) {
  code: 'ECONNRESET'
}
❌ [server-headless] Service exited unexpectedly

!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! KeyboardInterrupt !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
/opt/hostedtoolcache/Python/3.10.11/x64/lib/python3.10/pathlib.py:240: KeyboardInterrupt
(to show a full traceback on KeyboardInterrupt use --full-trace)

Ironically, the first run passed. Will re-trigger to try to make it fail.

Great, now it's failing: https://github.com/GoogleChromeLabs/chromium-bidi/actions/runs/4862921009/jobs/8674522969?pr=669

I added the full logs. Maybe it can give us extra hints on what is happening

It looks like a race condition during closing/launching browser. In the example above the process 12005 should not be killed:
.  bidiServer:internal Tue May 02 2023 19:40:26 GMT+0000 (Coordinated Universal Time) Peer ::1 disconnected. +222ms
  puppeteer:browsers:launcher Trying to kill 11940 +207ms
  puppeteer:browsers:launcher Browser process 11940 exists +1ms
  bidiServer:internal new WS request received: / +11ms
  bidiServer:SEND ▸ {"id":568,"result":{"type":"success","result":{"type":"undefined"},"realm":"-808600436653280652.5308073926180371153"}} +13ms
  puppeteer:browsers:launcher Launching /home/runner/work/chromium-bidi/chromium-bidi/chrome/linux-1138546/chrome-linux/chrome --headless --hide-scrollbars --mute-audio --disable-component-update --disable-popup-blocking --enable-automation --no-default-browser-check --no-first-run --password-store=basic --remote-debugging-port=9222 --use-mock-keychain --user-data-dir=/tmp/web-driver-bidi-server-Ph159W about:blank { detached: true, env: undefined, stdio: [ 'pipe', 'ignore', 'pipe' ] } +11ms
  puppeteer:browsers:launcher Launched 12005 +3ms
  puppeteer:browsers:launcher Browser process 11940 onExit +6ms
  bidiMapper:internal Establishing session with cdpUrl:  ws://<IP_ADDRESS>:9222/devtools/browser/18db609e-32b5-4feb-8975-16cde9ce9dd8 +148ms
  puppeteer:browsers:launcher Trying to kill 12005 +594ms
  puppeteer:browsers:launcher Browser process 12005 exists +2ms

@OrKoN and I were trying to debug this today. Please see https://github.com/GoogleChromeLabs/chromium-bidi/pull/675


MICRO-KERNEL

  Description: The initial kernel build cut many interfaces at a very high level.
               The resulting monolithic kernel is then rather large.  It would
               not be a prohibitively large task to reorganize the interfaces so
               that NuttX is built as a micro-kernel, i.e., with only the core
               OS services within the kernel and with other OS facilities, such
               as the file system, message queues, etc., residing in user-space
               and to interfacing with those core OS facilities through traps.
  Status:      Open
  Priority:    Low.  This is a good idea and certainly an architectural
               improvement.  However, there is no strong motivation now do
               do that partitioning work.

Maybe this can be closed? Any real desire in going this direction?


Deprecate support for Apache Spark 3.1.2

We have trouble to get Rikai work with Databricks 9.1 LTS (based on Apache Spark 3.1.2). See https://github.com/eto-ai/rikai/issues/522
And Rikai works well with Databricks 10.4 LTS (based on Apache 3.2.1).

closed by #705