Referencing secrets in environment variables must use "secretref:" not "secretRef:"

The example of referencing a secret in an environment variable has the following:
az containerapp create `
  --resource-group "my-resource-group" `
  --name myQueueApp `
  --environment "my-environment-name" `
  --image demos/myQueueApp:v1 `
  --secrets "queue-connection-string=$CONNECTIONSTRING" `
  --env-vars "QueueName=myqueue" "ConnectionString=secretRef:queue-connection-string"

which doesn't work - you have to use secretref - all lowercase.
[Enter feedback here]

Document details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 735a0e03-0632-feb7-8787-e62118809072
Version Independent ID: 938a7233-56e5-dd1b-05c0-2a15b8920f12
Content: Manage secrets in Azure Container Apps
Content Source: articles/container-apps/manage-secrets.md
Service: container-apps
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

@chrisseed Thanks for reporting this item! I have updated the doc and it will be published soon. We appreciate the feedback!


webserver: don't refetch xlog files if we happen to have them already

Previously during recovery if we reached a recovery target and restarted
PostgreSQL, pghoard would refetch all xlog files since the last checkpoint
again. Now we check if we already have them and just use them as-is.

There's a problem with this code with one of the unittests when run concurrently. Namely test_get_encrypted_archived_file() which when run concurrently also has a suitable XLOG file ready for early returning instead of the data that the testcase creates and expects to be there. Any ideas which way we should fix it?

Fixed comments, travis passing.


Checking actuals data

In the comments at the top of RegionEstimator it says that it checks all the sensor IDs against those in the sensors data - but it doesn't.
We should either add this functionality and test it, or remove the comment.

I've added the checking of sensor ids.
Have (manually) tested, + has run/passed automated tests set up so far.
I'll add some more automated tests to test this specifically.

Closing issue as is dealt with. Will test.


Could you export boolean_point_in_polygon function

We need to use this specific function but isn't export, please.

Hi @favazHF please create a pull request for that. It would be highly appreciated

@jsiedentop I think this is one of the implemented functions that is not exported yet?
https://github.com/dartclub/turf_dart/blob/main/lib/src/booleans/boolean_point_in_polygon.dart

Exactly, so far none of the boolean functions have been exported at all. I have exported all boolean functions as part of booleanWithin #167 in package:turf/boolean.dart. But we need to take a closer look:

package:turf/turf.dart should export all functionality, but a lot is missing (boolean functions among others)
package:turf/intersection.dart is missing
package:turf/invariant.dart is missing
package:turf/line_intersect.dart is missing
package:turf/line_slice.dart is defined as library turf_along. Seems to be an copy paste error.
package:turf/extensions.dart and package:turf/cluster.dart are the only meta functionality directly exported.
package:turf/meta.dart has all meta files except extensions.dart. Interesting, that meta also has export 'src/line_segment.dart' hide lineSegment;

Seems to have grown historically. Maybe we should take another look at it together?

closing this in favor of #169, which includes the requested improvement from this issue


Add OrchestrationTemplateRunner to queue up orchestration stack deployment

Provider specific provisions executed as part of a service provision are supposed to be queued to the provider zone.
Blocks https://github.com/ManageIQ/manageiq-content/pull/500.
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1656351
@miq-bot assign @gmcculloug
@miq-bot add_label bug, gaprindashvili/yes, hammer/yes, services, orchestration

@bzwei @mkanoor Please review.

The goal of this refactoring is that every promise ServiceOrchestration has made does not compromise so that the automate can continue to work.
@lfu Please rework ServiceOrchestration to fulfill the goal.

@bzwei Updated.


Skip invalid signs

Kotlinc was reporting some warnings about the classpath it was being called with. These didn't have line numbers associated with them so a line number of -1 was used when attempting to add the sign which caused an error. I've added a handler around the code that adds the sign so invalid signs will no longer prevent valid signs from being added. The loclist is still populated with errors/ warnings that cannot be added as signs. There might be a more efficient way to do this, I'm not overly familiar with vimscript.

koitlinc shouldn't be reporting errors at line -1, they should be at line 1.


Deprecate across the board max_satisfaction_weight

Adds a since=10.0.0 to all max_satisfaction_weight deprecations. Adds a note telling users to check #476 for more details.
Closes #637.

Thanks for following through man.

CI fails need you to use the new, non-deprecated versions, and there is some clippy stuff, could be unrelated to this PR but I didn't look.

I'm on it...

OK CI should pass now. I had to add some dead_code that were not being touched with deprecations anymore in some Psbt traits.

CI fail is unrelated to this PR.
miniscript has MSRV of 1.48 still, that won't work when its dependencies are using 1.56.1 - that should have been changed before last release.

FWIW the most thorough way to go about this is, IMO, to fix the clippy as a separate PR, fix the MSRV as a separate PR, then rebase this. I understand if you don't want to do all that, I can try get to it, likely won't be till next week though.

FWIW the most thorough way to go about this is, IMO, to fix the clippy as a separate PR, fix the MSRV as a separate PR, then rebase this. I understand if you don't want to do all that, I can try get to it, likely won't be till next week though.

I was tinkering with trying to get things to work with Rust 1.48.0.
So I am free to make other PRs changing MSRV to 1.56.1?
Good!
About the clippy warnings it seems also very easy.
Don't worry we can hold this until I finish these tasks and rebase.

So I am free to make other PRs changing MSRV to 1.56.1?

Yep, go for it. Sorry that this hasn't been done already. I'm surprised we managed to update our bitcoin dependency without doing it.

Ok PRs are up.
Suggested order of merging:

#639
#638 (this)
#640

Of course I'll need to rebase once an ancestor PR is merged...

Honestly because these are independent (in terms of diff) and only affect the Github Actions CI (which we can't possibly bisect on) I'd be fine just merging them without them being rebased.
But see my comment on your first one.


New to Node

Hello, I'm a .NET developer trying to dive into the node universe. I'm having issues trying to setup nuts locally for testing. I can see that in web.js it checks env variables for github info. I've hardcoded these and when debugging I get the following errors.
debug.txt
I know this is a setup issue, but am not clear on how to go about this, a quick tutorial on a simple setup would be helpful.
Thank you

Nevermind, I was able to figure out how to deploy to Heroku.


./gradlew assembleRelease throws "Direct local .aar file dependencies are not supported when building an AAR"

When trying to build release:
> Task :react-native-tor:bundleReleaseAar FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':react-native-tor:bundleReleaseAar'.
> Direct local .aar file dependencies are not supported when building an AAR. The resulting AAR would be broken because the classes and Android resources from any local .aar file dependencies would not be packaged in the resulting AAR. Previous versions of the Android Gradle Plugin produce broken AARs in this case too (despite not throwing this error). The following direct local .aar file dependencies of the :react-native-tor project caused this error: /home/overtorment/Documents/BlueWallet/node_modules/react-native-tor/android/libs/sifir_android.aar

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

Deprecated Gradle features were used in this build, making it incompatible with Gradle 7.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/6.3/userguide/command_line_interface.html#sec:command_line_warnings

BUILD FAILED in 31s
1117 actionable tasks: 272 executed, 845 up-to-date

Related PR: BlueWallet/BlueWallet/pull/2295

Thanks for this, Good catch.

Will publish AAR to a public maven repo, and update the module deps to use the repo instead.

thanks!

@Overtorment  I just tried building Sifir (which use the package also as a depedency) as an AAB
[gus@homebase android]$ ./gradlew bundleRelease
Configuration on demand is an incubating feature.
....
info Writing bundle output to:, /home/gus/Projects/sifir-app/android/app/build/generated/assets/react/release/index.android.bundle
info Writing sourcemap output to:, /home/gus/Projects/sifir-app/android/app/build/generated/sourcemaps/react/release/index.android.bundle.map
info Done writing bundle output
info Done writing sourcemap output
info Copying 136 asset files
info Done copying assets
....
BUILD SUCCESSFUL in 1m 21s
454 actionable tasks: 8 executed, 446 up-to-date

as an APK:
$ cd android && ./gradlew assembleRelease
....
BUILD SUCCESSFUL in 3m 9s
695 actionable tasks: 695 executed

and as a test release APK:
./node_modules/react-native/cli.js run-android --variant=release
BUILD SUCCESSFUL in 1m 25s
450 actionable tasks: 423 executed, 27 up-to-date
info Connecting to the development server...
info Starting the app on "emulator-5554"...
Starting: Intent { cmp=com.sifir/.MainActivity }

and I tried  the build having both github and npm as the source of react-native-tor.
What's the command you guys use to build your APKs/AAB ? Or are you trying to build react-native-tor into another AAR ?
Let me know thanks!

Im not very good with andoid build scripts (IMO its a nightmare).
the steps to reproduce are:
git clone git@github.com:BlueWallet/BlueWallet.git
cd BlueWallet/
git checkout --track origin/tor4lndhub
npm i
cd android/
./gradlew assembleRelease

result:
> Task :react-native-tor:bundleReleaseAar FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':react-native-tor:bundleReleaseAar'.
> Direct local .aar file dependencies are not supported when building an AAR. The resulting AAR would be broken because the classes and Android resources from any local .aar file dependencies would not be packaged in the resulting AAR. Previous versions of the Android Gradle Plugin produce broken AARs in this case too (despite not throwing this error). The following direct local .aar file dependencies of the :react-native-tor project caused this error: /tmp/BlueWallet/node_modules/react-native-tor/android/libs/sifir_android.aar

afaik this is roughly the same appcenter.ms does (its our CI/build server) to make us a production apk

any progress?

Nope, haven't really looked into this as it seems more of a build/CI issue when trying to build Bluewallet as an AAR. Maybe someone or another project can explain why that is being done for an Application ?
I'll keep this issue open so others can comment on it should they be facing the same issue.

I did some googling, and looks like its gradle4 to blame (gradle3 works fine), so any RN project that is built with gradle4 will experience this.
the solution might be to upload AAR to maven and link it from there.
OR to manually include AAR in my build.gradle file (that's what Im gonna try to do next)
https://stackoverflow.com/questions/60878599/error-building-android-library-direct-local-aar-file-dependencies-are-not-supp/63665094#63665094
https://github.com/transistorsoft/react-native-background-geolocation/issues/1077
https://stackoverflow.com/questions/16682847/how-to-manually-include-external-aar-package-using-new-gradle-android-build-syst/23326397#23326397

If uploading the AAR solves it for you then I think that is doable from the point of view of this repo. I also think it's probably gonna be easier for you track and stay up to date considering this repo is new and not mature yet.
I'll do it this week and let you know, so you can give it a try.

lets try uploading to maven, yes. then react-native-tor will have to reference it from maven.
Ill try to solve it manually, meanwhile

in a change that I did here https://github.com/Sifir-io/react-native-tor/pull/10  maven upload is not necessary

left a comment on your PR. I dont think compileOnly is the way to go

Commented on #10 , let me know what you think.

Commented on #10 , let me know what you think.

Handled by #10

Handled by #10


Add a form in docservice that will post requests to services mapped f...

/cc @delegacy
…or debugging using text protocol.

Forks and adds Twitter's TTextProtocol to armeria. Converts it to use jackson (already used in armeria) and add support for rpcs. Main edits are in commit https://github.com/anuraaga/armeria/commit/38f8c3552d635229f651cc4f2bf52a454d6e045c
Update doc service specification to detect any services that have an additional mapping with SerializationFormat set to the text protocol
Add a form on the docservice page of a method to allow posting requests in text format

Ideas:

We could improve ThriftService and its codec accept more than one ThriftProtocolFactory.

It could determine the SerializationFormat from the HTTP Content-Type and Accept-Encoding headers automatically for the requests and responses. A legacy client that does not send the ; protocol=T... part in its header will get the default serialization format.
Then, we don't need 'debugPath' and PathMapping.exactPath() at all.

Could we implement TTextProtocol.writeMessageBegin() and more so that an Armeria client can speak TTEXT?

Sniffing headers to determine serialization format on a single path sounds great, will followup in another PR a discussed offline.
Added support for writing messages.


Downgrade angular-bootstrap to 0.14

1.0.0 removes the deprecated directive names, eg. modal, that were
replaced by the uib prefix equivalents, eg. uib-modal.

hawtio modules still use version 0.13 and retain the original directive
names so upgrading causes errors in the browser. Cannot upgrade without
hawtio upgrading first.

Looks good Paul


What means a d:IsOptimized="True" in the samples?

<DoubleAnimation Duration="0:0:2.2" To="0.35"
    Storyboard.TargetProperty="(UIElement.RenderTransform).(CompositeTransform.ScaleY)"
    Storyboard.TargetName="StoryboardRectangle"
    d:IsOptimized="True"/>

Google and Bing not helped.

My understanding is that it's a tooling artifact from Expression Blend.
When Blend decides it can optimise an animation from say a Key Frame Animation to a simpler one it marks with this attribute.
It has no impact on the samples or the app at execution time.


add proxy support with got and node-tunnel

I replaced ky with got and using tunnel package to create an agent as proxy.
It should resolve #7

@ziofat
The build fails when runing npm install then npm run build. Can you check on your end? Am I missing something?

Maybe you should check https://github.com/configcat/common-js/pull/3 first, as it introduces the proxy attribute in ClientOptions where this PR is relying on.
You should run npm link on common-js and then run npm link configcat-common in this PR. Then it should work.

I have tested, merged and released a new version.
Thank you very much for the contribution and the help!

Great! I was worried about the SSR thing.


Blank cursor when used with folke/noice.nvim

When using noice.nvim, the cursor seems to disappear, making it really hard to type commands.

fix in 99d09bc


[Feature]

What is a problem?
OpenApi specification describes the ContentType as:
The Content-Type for encoding a specific property. Default value depends on the property type: for string with format being binary – application/octet-stream; for other primitive types – text/plain; for object - application/json; for array – the default is defined based on the inner type. The value can be a specific media type (e.g. application/json), a wildcard media type (e.g. image/*), or a comma-separated list of the two types.
precisely a wildcard media type (e.g. image/*)
Openapi3-parser support only one image type:
image/png
Describe the solution you'd like
Openapi3-parser is able to parse Content type with various types of images, for example image/jpeg and others
Describe alternatives you've considered
Additional context
https://spec.openapis.org/oas/v3.0.3#fixed-fields-12

Hi @rtyping, there is a possibility to disable strict content-type checks with parse() function calls by passing strict_enum=False as an argument. It will allow you to use any Content-Type you want, even if it is not officially registered as a media type. However, I will extend the current enum with a few more values for images.


 I want to play a game to make a feature request 😸
It would be great to have support for booru tag autocompletion from tag list CSV files a la a1111-sd-webui-tagcomplete; from the looks of things most of the infrastructure needed to make that work is already in place (for the various <segment:> etc autocomplete popups) so I suspect it wouldn't be too hard?
The tag list files are in tags/ inside the auto1111 extension repo and updated semi-regularly, so (especially given some of the content) I wouldn't expect you to rehost them, but it would probably be a good idea to support the CSV format they use.
thanku mcmonkey/team ❤

To add to this I also enjoy the https://github.com/Physton/sd-webui-prompt-all-in-one addon because of the Keyword Grouping / One-Click Keyword Addition, you can go through it and build a prompt.

Added! See docs here: https://github.com/Stability-AI/StableSwarmUI/blob/master/docs/Autocompletions.md


Restore best_vloss in finetune

This pull request is related to issue best_vloss isn't restored even if restart_filename arg is specified #6.

I'm closing this pull request because of conflicting files and to reflect the updates. I'll create a new one.


Disable Gateway API by default

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
What type of PR is this?

/kind chore
Special notes for your reviewer:
Does this PR introduce a user-facing change? Then add your Release Note here:

Gateway API is disabled by default for CCM and manager

Documentation:

NONE

/lgtm


Update logging behavior to a daily rotating one

Make sure to update the log viewer in the System Admin area to display each log and allow the user to select a log to review


cascadia-code: 2111.01 -> 2404.23

Description of changes

https://github.com/microsoft/cascadia-code/releases/tag/v2404.23
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[x] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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


BUG: optimize: rename private functions to include leading underscore

Reference issue

Closes gh-13350
What does this implement/fix?

renames the following HiGHS linprog extensions to avoid collision with local user-defined modules:

highs_wrapper -> _highs_wrapper
constants -> _highs_constants
mpswriter -> _mpswriter

adds __init__.py files to avoid import side-effects
remove executable bit from _linprog_highs.py

Additional information

Running python -c "from scipy import signal; import _highs_constants; print(_highs_constants)" now reports the error you'd expect.
The solution to this issue could be a good candidate for backport into 1.6.0.

This fixes the problem reported with the name constants, but there is still something wrong with how the names are being exposed.  These private modules should not be exposed in the Python import path.  That is,  this should generate a ModuleNotFoundError, but it doesn't:
$ python3 -c "import scipy.optimize; import _highs_wrapper; print(_highs_wrapper)"
<module 'scipy.optimize._highs._highs_wrapper' from '/Users/warren/mc37allpip/lib/python3.7/site-packages/scipy-1.7.0.dev0+0a04ab7-py3.7-macosx-10.9-x86_64.egg/scipy/optimize/_highs/_highs_wrapper.cpython-37m-darwin.so'>

This fixes the problem reported with the name constants, but there is still something wrong with how the names are being exposed.  These private modules should not be exposed in the Python import path.  That is,  this should generate a ModuleNotFoundError, but it doesn't:
$ python3 -c "import scipy.optimize; import _highs_wrapper; print(_highs_wrapper)"
<module 'scipy.optimize._highs._highs_wrapper' from '/Users/warren/mc37allpip/lib/python3.7/site-packages/scipy-1.7.0.dev0+0a04ab7-py3.7-macosx-10.9-x86_64.egg/scipy/optimize/_highs/_highs_wrapper.cpython-37m-darwin.so'>

@WarrenWeckesser Yes, all this is doing is obfuscating the hidden module.  I'm still not sure why it behaves in this way, but even refactoring _highs_constants.pyx to put all the constants in a data class and importing that class into _linprog_highs.py doesn't seem to hide it

@WarrenWeckesser Yes, all this is doing is obfuscating the hidden module.  I'm still not sure why it behaves in this way, but even refactoring _highs_constants.pyx to put all the constants in a data class and importing that class into _linprog_highs.py doesn't seem to hide it

I think the underlying issue is that the _highs cython code is written and cythonized as if it were 3 top-level modules (constants, highs_wrapper, and mpswriter) rather than its own _highs package or part of scipy.  The following may be overzealous with adding __init__.pys etc. but it solves the issue:

diff --git a/scipy/optimize/_highs/cython/src/Highs.pxd b/scipy/optimize/_highs/cython/src/Highs.pxd
index ef7bf0462..891871cd7 100644
--- a/scipy/optimize/_highs/cython/src/Highs.pxd
+++ b/scipy/optimize/_highs/cython/src/Highs.pxd
@@ -6,11 +6,11 @@ from libc.stdio cimport FILE
 from libcpp cimport bool
 from libcpp.string cimport string

-from HighsStatus cimport HighsStatus
-from HighsOptions cimport HighsOptions
-from HighsLp cimport HighsLp, HighsModelStatus
-from HighsInfo cimport HighsInfo
-from HighsLp cimport HighsSolution, HighsBasis, ObjSense
+from .HighsStatus cimport HighsStatus
+from .HighsOptions cimport HighsOptions
+from .HighsLp cimport HighsLp, HighsModelStatus
+from .HighsInfo cimport HighsInfo
+from .HighsLp cimport HighsSolution, HighsBasis, ObjSense

 cdef extern from "Highs.h":
     # From HiGHS/src/Highs.h
diff --git a/scipy/optimize/_highs/cython/src/HighsLp.pxd b/scipy/optimize/_highs/cython/src/HighsLp.pxd
index 956d73f9e..7ecb8bc91 100644
--- a/scipy/optimize/_highs/cython/src/HighsLp.pxd
+++ b/scipy/optimize/_highs/cython/src/HighsLp.pxd
@@ -5,7 +5,7 @@ from libcpp cimport bool
 from libcpp.string cimport string
 from libcpp.vector cimport vector

-from HConst cimport HighsBasisStatus
+from .HConst cimport HighsBasisStatus

 cdef extern from "HighsLp.h" nogil:
     # From HiGHS/src/lp_data/HighsLp.h
diff --git a/scipy/optimize/_highs/cython/src/HighsOptions.pxd b/scipy/optimize/_highs/cython/src/HighsOptions.pxd
index f1e7d382f..cb7798581 100644
--- a/scipy/optimize/_highs/cython/src/HighsOptions.pxd
+++ b/scipy/optimize/_highs/cython/src/HighsOptions.pxd
@@ -7,7 +7,7 @@ from libcpp cimport bool
 from libcpp.string cimport string
 from libcpp.vector cimport vector

-from HConst cimport HighsOptionType
+from .HConst cimport HighsOptionType

 cdef extern from "HighsOptions.h" nogil:

diff --git a/scipy/optimize/_highs/cython/src/constants.pyx b/scipy/optimize/_highs/cython/src/constants.pyx
index 2fe784f61..a064b7db0 100644
--- a/scipy/optimize/_highs/cython/src/constants.pyx
+++ b/scipy/optimize/_highs/cython/src/constants.pyx
@@ -3,18 +3,18 @@

 '''Export enum values and constants from HiGHS.'''

-from HConst cimport (
+from .HConst cimport (
     HIGHS_CONST_I_INF,
     HIGHS_CONST_INF,
 )
-from HighsIO cimport (
+from .HighsIO cimport (
     ML_NONE,
     ML_VERBOSE,
     ML_DETAILED,
     ML_MINIMAL,
     ML_ALWAYS,
 )
-from HighsLp cimport (
+from .HighsLp cimport (
     HighsModelStatusNOTSET,
     HighsModelStatusLOAD_ERROR,
     HighsModelStatusMODEL_ERROR,
@@ -29,7 +29,7 @@ from HighsLp cimport (
     HighsModelStatusREACHED_TIME_LIMIT,
     HighsModelStatusREACHED_ITERATION_LIMIT,
 )
-from SimplexConst cimport (
+from .SimplexConst cimport (
     # Simplex strategy
     SIMPLEX_STRATEGY_CHOOSE,
     SIMPLEX_STRATEGY_DUAL,
diff --git a/scipy/optimize/_highs/cython/src/highs_wrapper.pyx b/scipy/optimize/_highs/cython/src/highs_wrapper.pyx
index 683ea473c..12584a104 100644
--- a/scipy/optimize/_highs/cython/src/highs_wrapper.pyx
+++ b/scipy/optimize/_highs/cython/src/highs_wrapper.pyx
@@ -10,24 +10,24 @@ from libcpp.string cimport string
 from libcpp.memory cimport unique_ptr
 from libcpp.map cimport map as cppmap

-from HighsIO cimport (
+from .HighsIO cimport (
     ML_NONE,
 )
-from HConst cimport (
+from .HConst cimport (
     PrimalDualStatusSTATUS_FEASIBLE_POINT,
     HighsOptionTypeBOOL,
     HighsOptionTypeINT,
     HighsOptionTypeDOUBLE,
     HighsOptionTypeSTRING,
 )
-from Highs cimport Highs
-from HighsStatus cimport (
+from .Highs cimport Highs
+from .HighsStatus cimport (
     HighsStatus,
     HighsStatusError,
     HighsStatusWarning,
     HighsStatusOK,
 )
-from HighsLp cimport (
+from .HighsLp cimport (
     HighsLp,
     HighsSolution,
     HighsModelStatus,
@@ -45,8 +45,8 @@ from HighsLp cimport (
     HighsModelStatusREACHED_TIME_LIMIT,
     HighsModelStatusREACHED_ITERATION_LIMIT,
 )
-from HighsInfo cimport HighsInfo
-from HighsOptions cimport (
+from .HighsInfo cimport HighsInfo
+from .HighsOptions cimport (
     HighsOptions,
     OptionRecord,
     OptionRecordBool,
diff --git a/scipy/optimize/_highs/cython/src/mpswriter.pyx b/scipy/optimize/_highs/cython/src/mpswriter.pyx
index 81a298834..041232eb0 100644
--- a/scipy/optimize/_highs/cython/src/mpswriter.pyx
+++ b/scipy/optimize/_highs/cython/src/mpswriter.pyx
@@ -6,8 +6,8 @@ from libcpp cimport bool
 from libcpp.vector cimport vector
 from libcpp.string cimport string

-from HighsStatus cimport HighsStatus
-from HighsLp cimport (
+from .HighsStatus cimport HighsStatus
+from .HighsLp cimport (
     ObjSense,
     ObjSenseMINIMIZE,
 )
diff --git a/scipy/optimize/_highs/setup.py b/scipy/optimize/_highs/setup.py
index 2354e02cb..e5953378a 100644
--- a/scipy/optimize/_highs/setup.py
+++ b/scipy/optimize/_highs/setup.py
@@ -49,6 +49,7 @@ def _get_version(CMakeLists, start_token, end_token=')'):

 def configuration(parent_package='', top_path=None):
+    print("******** _highs  ***********\n"* 5 + parent_package)

     from numpy.distutils.misc_util import Configuration
     config = Configuration('_highs', parent_package, top_path)
diff --git a/scipy/optimize/setup.py b/scipy/optimize/setup.py
index 7a4fc8ad8..b702a7b22 100644
--- a/scipy/optimize/setup.py
+++ b/scipy/optimize/setup.py
@@ -13,6 +13,8 @@ def configuration(parent_package='', top_path=None):
     from scipy._build_utils.compiler_helper import (
         set_cxx_flags_hook, set_cxx_flags_clib_hook, set_c_flags_hook)

+    print("******** optimize  ***********\n"* 5 + parent_package)
+
     config = Configuration('optimize', parent_package, top_path)

     include_dirs = [join(os.path.dirname(__file__), '..', '_lib', 'src')]

I think the underlying issue is that the _highs cython code is written and cythonized as if it were 3 top-level modules (constants, highs_wrapper, and mpswriter) rather than its own _highs package or part of scipy.  The following may be overzealous with adding __init__.pys etc. but it solves the issue:

diff --git a/scipy/optimize/_highs/cython/src/Highs.pxd b/scipy/optimize/_highs/cython/src/Highs.pxd
index ef7bf0462..891871cd7 100644
--- a/scipy/optimize/_highs/cython/src/Highs.pxd
+++ b/scipy/optimize/_highs/cython/src/Highs.pxd
@@ -6,11 +6,11 @@ from libc.stdio cimport FILE
 from libcpp cimport bool
 from libcpp.string cimport string

-from HighsStatus cimport HighsStatus
-from HighsOptions cimport HighsOptions
-from HighsLp cimport HighsLp, HighsModelStatus
-from HighsInfo cimport HighsInfo
-from HighsLp cimport HighsSolution, HighsBasis, ObjSense
+from .HighsStatus cimport HighsStatus
+from .HighsOptions cimport HighsOptions
+from .HighsLp cimport HighsLp, HighsModelStatus
+from .HighsInfo cimport HighsInfo
+from .HighsLp cimport HighsSolution, HighsBasis, ObjSense

 cdef extern from "Highs.h":
     # From HiGHS/src/Highs.h
diff --git a/scipy/optimize/_highs/cython/src/HighsLp.pxd b/scipy/optimize/_highs/cython/src/HighsLp.pxd
index 956d73f9e..7ecb8bc91 100644
--- a/scipy/optimize/_highs/cython/src/HighsLp.pxd
+++ b/scipy/optimize/_highs/cython/src/HighsLp.pxd
@@ -5,7 +5,7 @@ from libcpp cimport bool
 from libcpp.string cimport string
 from libcpp.vector cimport vector

-from HConst cimport HighsBasisStatus
+from .HConst cimport HighsBasisStatus

 cdef extern from "HighsLp.h" nogil:
     # From HiGHS/src/lp_data/HighsLp.h
diff --git a/scipy/optimize/_highs/cython/src/HighsOptions.pxd b/scipy/optimize/_highs/cython/src/HighsOptions.pxd
index f1e7d382f..cb7798581 100644
--- a/scipy/optimize/_highs/cython/src/HighsOptions.pxd
+++ b/scipy/optimize/_highs/cython/src/HighsOptions.pxd
@@ -7,7 +7,7 @@ from libcpp cimport bool
 from libcpp.string cimport string
 from libcpp.vector cimport vector

-from HConst cimport HighsOptionType
+from .HConst cimport HighsOptionType

 cdef extern from "HighsOptions.h" nogil:

diff --git a/scipy/optimize/_highs/cython/src/constants.pyx b/scipy/optimize/_highs/cython/src/constants.pyx
index 2fe784f61..a064b7db0 100644
--- a/scipy/optimize/_highs/cython/src/constants.pyx
+++ b/scipy/optimize/_highs/cython/src/constants.pyx
@@ -3,18 +3,18 @@

 '''Export enum values and constants from HiGHS.'''

-from HConst cimport (
+from .HConst cimport (
     HIGHS_CONST_I_INF,
     HIGHS_CONST_INF,
 )
-from HighsIO cimport (
+from .HighsIO cimport (
     ML_NONE,
     ML_VERBOSE,
     ML_DETAILED,
     ML_MINIMAL,
     ML_ALWAYS,
 )
-from HighsLp cimport (
+from .HighsLp cimport (
     HighsModelStatusNOTSET,
     HighsModelStatusLOAD_ERROR,
     HighsModelStatusMODEL_ERROR,
@@ -29,7 +29,7 @@ from HighsLp cimport (
     HighsModelStatusREACHED_TIME_LIMIT,
     HighsModelStatusREACHED_ITERATION_LIMIT,
 )
-from SimplexConst cimport (
+from .SimplexConst cimport (
     # Simplex strategy
     SIMPLEX_STRATEGY_CHOOSE,
     SIMPLEX_STRATEGY_DUAL,
diff --git a/scipy/optimize/_highs/cython/src/highs_wrapper.pyx b/scipy/optimize/_highs/cython/src/highs_wrapper.pyx
index 683ea473c..12584a104 100644
--- a/scipy/optimize/_highs/cython/src/highs_wrapper.pyx
+++ b/scipy/optimize/_highs/cython/src/highs_wrapper.pyx
@@ -10,24 +10,24 @@ from libcpp.string cimport string
 from libcpp.memory cimport unique_ptr
 from libcpp.map cimport map as cppmap

-from HighsIO cimport (
+from .HighsIO cimport (
     ML_NONE,
 )
-from HConst cimport (
+from .HConst cimport (
     PrimalDualStatusSTATUS_FEASIBLE_POINT,
     HighsOptionTypeBOOL,
     HighsOptionTypeINT,
     HighsOptionTypeDOUBLE,
     HighsOptionTypeSTRING,
 )
-from Highs cimport Highs
-from HighsStatus cimport (
+from .Highs cimport Highs
+from .HighsStatus cimport (
     HighsStatus,
     HighsStatusError,
     HighsStatusWarning,
     HighsStatusOK,
 )
-from HighsLp cimport (
+from .HighsLp cimport (
     HighsLp,
     HighsSolution,
     HighsModelStatus,
@@ -45,8 +45,8 @@ from HighsLp cimport (
     HighsModelStatusREACHED_TIME_LIMIT,
     HighsModelStatusREACHED_ITERATION_LIMIT,
 )
-from HighsInfo cimport HighsInfo
-from HighsOptions cimport (
+from .HighsInfo cimport HighsInfo
+from .HighsOptions cimport (
     HighsOptions,
     OptionRecord,
     OptionRecordBool,
diff --git a/scipy/optimize/_highs/cython/src/mpswriter.pyx b/scipy/optimize/_highs/cython/src/mpswriter.pyx
index 81a298834..041232eb0 100644
--- a/scipy/optimize/_highs/cython/src/mpswriter.pyx
+++ b/scipy/optimize/_highs/cython/src/mpswriter.pyx
@@ -6,8 +6,8 @@ from libcpp cimport bool
 from libcpp.vector cimport vector
 from libcpp.string cimport string

-from HighsStatus cimport HighsStatus
-from HighsLp cimport (
+from .HighsStatus cimport HighsStatus
+from .HighsLp cimport (
     ObjSense,
     ObjSenseMINIMIZE,
 )
diff --git a/scipy/optimize/_highs/setup.py b/scipy/optimize/_highs/setup.py
index 2354e02cb..e5953378a 100644
--- a/scipy/optimize/_highs/setup.py
+++ b/scipy/optimize/_highs/setup.py
@@ -49,6 +49,7 @@ def _get_version(CMakeLists, start_token, end_token=')'):

 def configuration(parent_package='', top_path=None):
+    print("******** _highs  ***********\n"* 5 + parent_package)

     from numpy.distutils.misc_util import Configuration
     config = Configuration('_highs', parent_package, top_path)
diff --git a/scipy/optimize/setup.py b/scipy/optimize/setup.py
index 7a4fc8ad8..b702a7b22 100644
--- a/scipy/optimize/setup.py
+++ b/scipy/optimize/setup.py
@@ -13,6 +13,8 @@ def configuration(parent_package='', top_path=None):
     from scipy._build_utils.compiler_helper import (
         set_cxx_flags_hook, set_cxx_flags_clib_hook, set_c_flags_hook)

+    print("******** optimize  ***********\n"* 5 + parent_package)
+
     config = Configuration('optimize', parent_package, top_path)

     include_dirs = [join(os.path.dirname(__file__), '..', '_lib', 'src')]

@seberg Yes, the _highs directory has a bit of an odd structure because it is modeled after a fork of HiGHS project.  It will probably remain this way as that makes it easy to pull in upstream changes.
I was unable to compile with your diff due to the cimports being changed to relative imports.  Did you change anything else about the setup.py that is not in the diff?

@seberg Yes, the _highs directory has a bit of an odd structure because it is modeled after a fork of HiGHS project.  It will probably remain this way as that makes it easy to pull in upstream changes.
I was unable to compile with your diff due to the cimports being changed to relative imports.  Did you change anything else about the setup.py that is not in the diff?

No, I hadn't changed anything in the setup.py (only moved that one line, since it seemed odd, but it probably doesn't matter), or a different cython version?  Maybe, all you need is to change everything to relative imports and add the __init__.py?  But I honestly don't know now if such changes should even be necessary.

No, I hadn't changed anything in the setup.py (only moved that one line, since it seemed odd, but it probably doesn't matter), or a different cython version?  Maybe, all you need is to change everything to relative imports and add the __init__.py?  But I honestly don't know now if such changes should even be necessary.

@WarrenWeckesser @seberg Does the current state of this PR look good to you all?

@WarrenWeckesser @seberg Does the current state of this PR look good to you all?

It probably is a decent stop gap, I am generally not a fan of 95% solutions but underscored names are rare enough outside modules that it probably won't be noticed much.  (I expect there will be more issues, i.e. it might be problematic to try and pickle something from the _highs module).
I don't understand why changing all imports to relative imports and adding an __init__.py did not work for you, or why it is necessary to begin with.
Aside from this, the only idea I have is asking on cython, maybe there is another simple fix.

It probably is a decent stop gap, I am generally not a fan of 95% solutions but underscored names are rare enough outside modules that it probably won't be noticed much.  (I expect there will be more issues, i.e. it might be problematic to try and pickle something from the _highs module).
I don't understand why changing all imports to relative imports and adding an __init__.py did not work for you, or why it is necessary to begin with.
Aside from this, the only idea I have is asking on cython, maybe there is another simple fix.

@seberg I tried your diff again, got the same result as last time, did a build from clean, and now it's working.  I think it is indeed a better solution than I had, so I'm happy to go with that.

@seberg I tried your diff again, got the same result as last time, did a build from clean, and now it's working.  I think it is indeed a better solution than I had, so I'm happy to go with that.

squash-merged to simplify the backport task; thanks @mckib2 @mdhaber @seberg @WarrenWeckesser

Tested. Looks like it fixes the issue without any controversial changes. One thought - and I'd merge either way - is that _mpswriter could be renamed to _highs_mpswriter for consistency (and the very, very low chance of a name conflict with _mpswriter. Then again, maybe even if there is a name conflict now there's no problem?)

We can take care of this suggestion in a future HiGHS sync update

Sure, if you agree. It's purely cosmetic, I think, because as @seberg mentioned it seems that even name conflicts pose no problem now. Thanks for fixing this!


Add support for multi-level wildcard paths in composer

A customer reported that nested wildcard resolution fails in Dependabot.  Given a composer.json repository like:
"repositories": [
   {
     "type": "path",
     "url": "components/*/*",
     "options": {"symlink": false}
   }
 ],
 "require": {
   "monolog/monolog": "1.0.*",
   "symfony/polyfill-mbstring": "1.0.*"
 }
}

We would fail to properly handle both * and would attempt to retrieve components/* from the source repository.  When this attempt returns an error we fail the update job.
This PR adds support to traverse the path and expand the directory list at each level of the path which is represented by a wildcard.  It also adds a test for this added path traversal support.

Thank you for explaining the changes during our 1-1.

It would be good if you attach the test cases result here as well and thank you for working on this issue!


Waf Exclusion rule difference between "Request args key" and "Request args name"

Hi,
what is the difference in a exclusion rule between "Request args key" and "Request args name"?

@ciroskat
It would be great if you could add a link to the documentation you are following for these steps? This would help us redirect the issue to the appropriate team. Thanks!!

This picture show my misunderstanding, i dont know what is the difference between key and name, and this selection exclude the value too? Or only the key/name?

@ciroskat
Thank you for your feedback!
Since this issue isn't directly related to improving our docs, and to gain a better understanding of your issue, I'd recommend working closer with our support team via an Azure support request. Or you can leverage our Q&A forum by posting your issue there so our community, and MVPs can further assist you in troubleshooting this issue or finding potential workarounds.
Teams Q&A forum for technical questions about the configuration and administration of Microsoft Teams on Windows.
Microsoft Teams Community forum
Thank you for your time and patience throughout this issue.


Pipe found list files to single cmake-format call

Fixes running with arg --check and other reasons why cmake-format would fail such as with a bad config.
Using \; invokes multiple calls to cmake-format with each file as an arguments, while + pipes all files into the arguments.

With \;:

cmake-format $* ./test/CMakeLists.txt
cmake-format $* ./CMakeLists.txt
cmake-format $* ./CMakeModules/FindSDL2.cmake
cmake-format $* ./CMakeModules/Findspdlog.cmake
cmake-format $* ./CMakeModules/vcpkg_android.cmake
cmake-format $* ./src/CMakeLists.txt

exit codes of cmake-format are disposed of

With +:

cmake-format $* ./test/CMakeLists.txt ./CMakeLists.txt ./CMakeModules/FindSDL2.cmake ./CMakeModules/Findspdlog.cmake ./CMakeModules/vcpkg_android.cmake ./src/CMakeLists.txt

exit code is maintained

Nice catch! Thanks for the PR


⚠ PrivateHD has degraded performance

In 58e3e14, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 5479 ms

Resolved: PrivateHD performance has improved in d10fc97.


HBASE-26310 Repro balancer behavior during iterations

TestStochasticLoadBalancerRegionReplicaWithRacks is failing by design  until we fix https://issues.apache.org/jira/browse/HBASE-26311. Please hold off on merging.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 30s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+1 :green_heart:
mvninstall
4m 49s
master passed

+1 :green_heart:
compile
0m 24s
master passed

+1 :green_heart:
shadedjars
8m 37s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 19s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 20s
the patch passed

+1 :green_heart:
compile
0m 22s
the patch passed

+1 :green_heart:
javac
0m 22s
the patch passed

+1 :green_heart:
shadedjars
8m 20s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 17s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
6m 58s
hbase-balancer in the patch passed.

36m  7s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3711/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3711

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 37e7a01159f6 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 121bdea230

Default Java
AdoptOpenJDK-11.0.10+9

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3711/1/testReport/

Max. process+thread count
313 (vs. ulimit of 30000)

modules
C: hbase-balancer U: hbase-balancer

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3711/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 29s
Docker mode activated.


test data

test data

Codecov Report

Merging #108 (97fd265) into main (23b182f) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 97fd265 differs from pull request most recent head 2486cfe. Consider uploading reports for the commit 2486cfe to get more accurate results

@@           Coverage Diff           @@
##             main     #108   +/-   ##
=======================================
  Coverage   22.09%   22.09%
=======================================
  Files           2        2
  Lines         172      172
=======================================
  Hits           38       38
  Misses        133      133
  Partials        1        1

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 23b182f...2486cfe. Read the comment docs.

Refer the example files from charts repo directly to avoid repeatence.


Avoid using the patch for the Makefile

This feedstock currently uses a patch to inject the PREFIX and GCC variables into the Makefile, but something like this should work:
make -j${CPU_COUNT} prefix=$PREFIX CC=$CC

For some reason, this still failed due to the prefix being set to /usr/bin. It would be good to move away from using the patch file.

Fixed in #12


Bugfix/failed test returns

This bugfix solves https://github.com/LLNL/ATS/issues/90 by adding return values to interrupts, ATS errors, and various test failures.

Looks good and passes my test.


Offshore wind production for H2 does not work because of error in curves

I tried to put 10 GW of dedicated hydrogen offshore in the NL and it returned:

The same is true for dedicated solar for hydrogen

Also notifying @antw because I think it is something in ETengine, because the bold route exists:
No such file or directory @ rb_sysopen - /var/www/etengine/releases/20200225153723/tmp**/etso
urce/datasets/nl/curves/wind_offshore_baseline.csv**
Although the folder structure is a bit different now in ETsource /etso
urce/datasets/nl/curves/**weather/default/**wind_offshore_baseline.csv

In this commit the curves were deleted from the curves folder
https://github.com/quintel/etsource/commit/4333023a05c91243c6f2b98bb4527f30b4772802#diff-873de58cc67aad55c5aebf36e841abb5
I think the fastest fix is to put the baseline curve for

solar_pv.csv
wind_coastal_baseline.csv
wind_inland_baseline.csv
wind_offshore_baseline.csv
back in the curves folder.

Another fix would be to make sure nodes that currently use a curve from the curves-folder link to curves/weather_years/default.

In this commit the baseline curves for solar_pv and wind were deleted from the curves folder
This was intended, since we wanted these curves to be stored in the curves/weather/... sub directory.

Another fix would be to make sure nodes that currently use a curve from the curves-folder link to curves/weather_years/default.
This seems like a better fix. I think the fix is similar to https://github.com/quintel/etsource/pull/2177/commits/78d4c1b8d32d5aab43c22c1e54895efc1cd46d53

Oof. This is because the hydrogen and network gas don't support "dynamic: ..." curves. I will see what I can do about this.


Rewrite serial

This PR:

gets rid off the old chapters explaining how registers etc. work since a person starting out with embedded rust doesn't have to know that anymore
moves the UART and serial communication chapters a little forward since they seemed like the most simple of hardware peripherals to me (after LEDs that is).
changes the UART code from register level code to nrf51-hal code

Everything is rewritten now.

:warning: Warning :warning:

Pull requests are usually filed against the master branch for this repo, but this one is against rewrite. Please double check that you specified the right target!

That CI error is coming from a chapter in the later pats of the book referencing back to a chapter I now moved, any ideas what I should do regarding that? I guess I could just comment out the link or something?

Since this is just a WIP branch anyways I'll go ahead and comment it out and remove it during a later chapter rewrite.

bors r+


Index error when parsing message from some user.

During testing, I found this error.
Here are a few details.

I was able to get the message string from a previous error from the same type as message as the one in the screenshot above.
"@badge-info=founder/102;badges=moderator/1,founder/0;color=#8A2BE2;display-name=SheepFarmer;emotes=;first-msg=0;flags=;id=f9912e9d-474a-4cba-8897-0eb7bb9391e7;mod=1;reply-parent-display-name=hughmyron17;reply-parent-msg-body=!giveaway;reply-parent-msg-id=500ea2a8-f58f-407c-97a4-4c0e3274f374;reply-parent-user-id=133248066;reply-parent-user-login=hughmyron17;reply-thread-parent-msg-id=500ea2a8-f58f-407c-97a4-4c0e3274f374;reply-thread-parent-user-login=hughmyron17;returning-chatter=0;room-id=56649026;subscriber=1;tmi-sent-ts=1696780419533;turbo=0;user-id=69861108;user-type=mod<EMAIL_ADDRESS>PRIVMSG #quin69 :@hughmyron17 For Quins $1,000 giveaway & more, enter here: http://bit.ly/quin69giveaway"

testable example:
>>> message = "@badge-info=founder/102;badges=moderator/1,founder/0;color=#8A2BE2;display-name=SheepFarmer;emotes=;firs
... t-msg=0;flags=;id=f9912e9d-474a-4cba-8897-0eb7bb9391e7;mod=1;reply-parent-display-name=hughmyron17;reply-parent-msg
... -body=!giveaway;reply-parent-msg-id=500ea2a8-f58f-407c-97a4-4c0e3274f374;reply-parent-user-id=133248066;reply-paren
... t-user-login=hughmyron17;reply-thread-parent-msg-id=500ea2a8-f58f-407c-97a4-4c0e3274f374;reply-thread-parent-user-l
... ogin=hughmyron17;returning-chatter=0;room-id=56649026;subscriber=1;tmi-sent-ts=1696780419533;turbo=0;user-id=698611
... 08;user-type=mod<EMAIL_ADDRESS>PRIVMSG #quin69 :@hughmyron17 For Quins $1,000
... giveaway & more, enter here: http://bit.ly/quin69giveaway"
>>>
>>> name = message.split("PRIVMSG")[0].split("!")[0].split(":")[1]
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
IndexError: list index out of range

>>>

I don't know enough to talk about a solution, maybe indexing and splitting is not the right tool when the data is not predictable.
and many selecting negative indexes would work. name = message.split("PRIVMSG")[0].split("!")[-2].split(":")[1] as an example for discussion.
Since the results are unclear, regex may be the right tool for this.
eivl

Thanks for reporting the error and contributing to the project, much appreciated. I'll release a fix as soon as possible.

My pleasure.

    badges = {badge.rstrip("/1") for badge in message.split("badges=")[1].split(";")[0].split(",")}
                                              ~~~~~~~~~~~~~~~~~~~~~~~~^^^
IndexError: list index out of range

another split issue.
i dont have the message that broke this. Adding it here.
I could try to rewrite the parser based on the RFC 1459 spec.

I'll be adjusting this as soon as possible. Apologies for the issues you're encountering.

no worries, im just glad i did not have to write my own socket and data parser from scratch :D
here is a message that failed in the parser.
m-id=56649026;subscriber=0;tmi-sent-ts=1696789151006;turbo=0;user-id=400074398;user-type=<EMAIL_ADDRESS>PRIVMSG #quin69 :guys stop saying shit about this being quins audio, this is much better

Im doing a full test to see if it breaks more places.
 @classmethod
    def parse(cls, message: str) -> "TwitchPrivateMessage":
        """
        Parse a Twitch private message.
        :param message: The message to parse.
        """
        try:
            badges = {badge.rstrip("/1") for badge in message.split("badges=")[1].split(";")[0].split(",")}
            color = message.split("color=")[1].split(";")[0]
            display_name = message.split("display-name=")[1].split(";")[0]
            first_message = message.split("emotes=")[1].split(";")[0] == "-1"
            message_id = message.split("id=")[1].split(";")[0]
            mod = message.split("mod=")[1].split(";")[0] == "1"
            broadcaster = "broadcaster" in badges
            returning_chatter = message.split("room-id=")[1].split(";")[0] == "1"
            room_id = int(message.split("room-id=")[1].split(";")[0])
            subscriber = message.split("subscriber=")[1].split(";")[0] == "1"
            turbo = message.split("turbo=")[1].split(";")[0] == "1"
            user_id = int(message.split("user-id=")[1].split(";")[0])
            name = message.split("PRIVMSG")[0].split("!")[-2].split(":")[1]
            channel = message.split("PRIVMSG")[1].split(":")[0].strip().lstrip("#")
            message = ":".join(message.split("PRIVMSG")[1].split(":")[1:]).strip()
        except IndexError:
            print()
            print(message)
            print()
            return cls(
                set(),
                "",
                "",
                False,
                "",
                False,
                False,
                False,
                0,
                False,
                False,
                0,
                "",
                "",
                "",
            )
        return cls(
            badges,
            color,
            display_name,
            first_message,
            message_id,
            mod,
            broadcaster,
            returning_chatter,
            room_id,
            subscriber,
            turbo,
            user_id,
            name,
            channel,
            message,
        )

I just wrap it in a try except and return an empty object back.
And here is another one that failed.
82a1-4d44-afc7-7acd1cbe1baa;mod=0;returning-chatter=0;room-id=56649026;subscriber=0;tmi-sent-ts=1696789889991;turbo=0;user-id=30278702;user-type=<EMAIL_ADDRESS>PRIVMSG #quin69 :Aware 2015

I ran this messages through a RFC 1459 tokenizer, and these lines still fails as it does not have the source. makes me think that twitch does not follow the IRC spec like I expect it to.

a97eba093e39423f9c4c2;color=#B22222;display-name=Khazerox;emotes=;first-msg=0;flags=0-3:P.3;id=9e48a829-3f4a-4bae-b9e2-7b30e9fc5acb;mod=0;returning-chatter=0;room-id=56649026;subscriber=1;tmi-sent-ts=1696790043418;turbo=0;user-id=58797167;user-type=<EMAIL_ADDRESS>PRIVMSG #quin69 :C***

and another one.
Should give you a few ideas how to troubleshoot I hope.

Solved with #5.


Fix SetRequest shadow path unmarshaling

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

15 of 15 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 88.518%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
1565

Relevant Lines:
1768

💛  - Coveralls

Worth adding a test here? else looks good.

I think not yet, since the generates code will never create a ConfigQuery with IsState is true. (only our lemming hack does). If/when we add custom/dynamic/schemaless queries to ygnmi I'll have tests for this behavior.


[WIP] Use sphinx_automodapi to generate the API docs

This is a backup for PR #152, because I messed up @barentsen's branch.
@barentsen said:

We should consider switching to AstroPy's sphinx-automodapi module to auto-generate the API docs, because it will show class properties (implemented using the @property decorator) in a nice table called Attributes Summary alongside other attributes. I can't figure out how to achieve this using automodule, and this is important because we use @property a lot.
...however automodapi is a bit funny in other ways. @mirca what do you think?

@mirca I thought you fixed the failing travis test (caused by lightkurve.version not being available pre-install) but the commit seems gone?

Codecov Report

Merging #153 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #153   +/-   ##
=======================================
  Coverage   92.27%   92.27%
=======================================
  Files          17       17
  Lines        2292     2292
=======================================
  Hits         2115     2115
  Misses        177      177

Impacted Files
Coverage Δ

lightkurve/lightcurve.py
91.5% <ø> (ø)
:arrow_up:

lightkurve/prf.py
89.77% <ø> (ø)
:arrow_up:

lightkurve/correctors.py
79.27% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@barentsen what do you think of this workaround?


Security concerns to automate chrome browser using puppeteer's wsEndpoint to reconnect to opened window[Bug]: #7551

We have concerns from security team to automate chrome browser using puppeteer and it's web socket endpoint.Their concern is "When webSocket is enabled at the given port. This WebSocket provides access to the remote debugging protocol, which offers control over the whole browser. This results in scripting access to arbitrary websites, the ability to read local files, write to local files, and launch external programs". Can anyone please help me how to fix this issue.
//disconnecting from and reconnecting to a Browser
(async () => {
  const browser = await puppeteer.launch("chrome path");
  // Store the endpoint to be able to reconnect to Chrome
  const browserWSEndpoint = browser.wsEndpoint();
  // Disconnect puppeteer from Chrome
  browser.disconnect();

  // Saving WSEndpoint somewhere in user tem folder and Using the endpoint to reestablish a connection. This is to connect to pages which already opened by user manually
  const browser2 = await puppeteer.connect({browserWSEndpoint});
  // Close Chromium
  await browser2.close();
})();

That's a valid concern @siddusri01. Puppeteer supports also a pipe connection (inter-process communication). But we don't have (yet) a way to implement that in .NET.
I think that the only way to solve it at the moment is by implementing some kind of security at the OS level.

We are currently mitigating it with validating the HTTP header of wsEndpoint either localhost or <IP_ADDRESS>

@kblok, I saw, you have mentioned, Pipe feature will be implemented in v1.1.0,  Is it in progress? or Is it not possible with .Net?
"#223, Hey @JipingWang. Pipe connection will be implemented when we get to v1.1.0"

Yeah, that was a long time ago. I wasn't able to implement that feature.

Thanks @kblok, Is there any work around to solve this issue, you have mentioned, need to implement some kind of security at the OS level. Actually we are almost end of the project and then got stuck with this issue,now we are in painful situation, And this is major road block for us and  I think it is the major bug in puppeteer-sharp as well as per security concerns. Please let me know in any way around i can contribute to this feature

The only thing that someone with access to that WebSocket will be able to do is to automate the browser. I don't see how it can access a local file.

I got this link, they explained how it can be hacked
https://bugs.chromium.org/p/chromium/issues/detail?id=813540

I took a look at the link. According to that bug report, it was an issue with Chromium 63 and has since been fixed back in 2018. Is there any documentation regarding this security concern for the latest Chromium versions (90, 91, 92, etc.)?

Closed due to inactivity. Feel free to reopen it if needed.


Fatal error: editor placeholder in source file

Run swift-format on this source:
switch <#value#> {
case <#pattern#>:
	<#code#>
default:
	<#code#>
}

I get the following errors:
1:8: editor placeholder in source file
2:6: editor placeholder in source file
3:5: editor placeholder in source file
5:5: editor placeholder in source file
Note: These errors should just be warnings and shouldn't affect the formatting of code (like Xcode does).

cc @ahoppen

Having an editor placeholder in a source file should generally be considered an error because the code won’t compile. swift-format could downgrade these errors to warnings as in https://github.com/apple/swift/pull/64382.


Add happiness

We should add happiness to this.
Right now,  the formatting is inconsistent.

+1

+1

Thanks @007lva


KUBOS-360  First iteration of the kubos-cli integration test

This is the first version of a kubos-cli integration test. This test aims to exercise the "happy path" update, project initialization, and build procedures for both kubos-linux and kubos-rt projects.
Edit: This is dependent on #9  and #12  Being merged in prior to review.

@plauche could you take a look at this at your convenience?

@plauche Friendly bump for 👀


Switch library to ES6

Hi there,
I am a fellow watcher of your stream from time to time. I thought I might contribute something. I switched the code in this repo to ES6. Notable changes:

Installed JavaScript Standard Style as a linter to have consistent code rules
Removed the old prototype way and switched to a proper ES6 class
Made changes to conform to the new linting rules
Made some additional syntax changes like destructuring for arrays (const [ var1, var2 ] = array)

I ensured that your tests are still running. Unfortunately I am not set up to test the changes on real devices right now so I would be super glad if you could do that. In theory nothing should be broken, but better check twice.
I know this is kind of a major change to this repo, but it uses a modern linter and the newest stable language features node has to offer. For newcomers this will be easier to read in the future since most people getting into Node.js now are only learning this new style.
Thanks a ton!

This is great! Such a step forward, I hope @noopkat can review this

According to the tests, node 4 and 5 will break with these changes. I'm not sure how far back @noopkat was planning to provide support for older node, but it's worth noting that Node 8 has been out for a bit and Node 10 as well.

Thanks so much @Algram and for your patience with how long it took to merge this ✨


A number of fixes - please do rebase and merge

To keep all of the nice commit info 😄

Awesome work!


Refusal to open a scene with a syntax error in the embedded script

Godot version
4.0.3.stable
System information
Win 10, i5-11400H, 16GB RAM, RTX3050M
Issue description

The scene does not load due to the fact that there is a syntax error in the embedded script. Of course, I can't fix the script because it is built-in (except through an external editor). This behavior is abnormal, the scene should still load even if there is an error in its script.
Breaking the scene code example:
var dirSAV = DirAccess.new()
var path = \"user://SaveGame/Worlds/\" + str(name_save)
if !dirSAV.dir_exists(path):
	dirSAV.make_dir_recursive(path)

Steps to reproduce
Create a project in 3 godot create a scene there with a built-in script, for example, create a folder, then transfer the project to 4 godot and try to open the scene.
Minimal reproduction project
Link to my repository - https://github.com/Miskler/kHUTER
Branch godot4

It isn't a syntax error it's a formatting error in the file, seems to be a failure to escape quotes correctly when converting projects

@AThousandShips so the transition module from the 3rd generation to the 4th generation cannot process the built-in scripts correctly?

For large projects this is a very big problem as fixing these errors takes a very long time

I want to draw attention to the fact that this error is not found everywhere

It is in your example?

yes

Where is it not? Note that the other "\"" are from the original 3.x code

Working on a fix for this


Added new command sender filter based on target connection icons

Added new filter for "Commands" block which allows user to efficiently filter targets by type of their session's icon. This adds capability to group hosts in a easy manner.
There are new icon file(none.png) with 'X' inside to allow user to choose "Unselected/Disabled" state.
I draw the icon myself in Paint(Thats, why it's ugly :D), so it's public domain and doesn't need licenses/approvals.

Updated to use stop icon from famfamfam instead of custom X.


java.lang.IllegalStateException: Could not find any HAPI-FHIR structure JARs on the classpath

James,
this is quite a confusing error.
I'm getting this when loading the OSGi bundles I've created (both hapi-fhir-base and hapi-fhir-jpaserver-base are now bundles).
This error is being thrown on the start of hapi-fhir-base. Here is the relevant part:
Caused by: java.lang.IllegalStateException: Could not find any HAPI-FHIR structure JARs on the classpath. Note that as of HAPI-FHIR v0.8, a separate FHIR strcture JAR must be added to your classpath (or project pom.xml if you are using Maven)
at ca.uhn.fhir.context.FhirContext.(FhirContext.java:130)
at ca.uhn.fhir.context.FhirContext.(FhirContext.java:108)
at ca.uhn.fhir.context.FhirContext.(FhirContext.java:96)
at ca.uhn.fhir.rest.server.RestfulServer.(RestfulServer.java:109)
Where RestfulServer.java:109 is:
/**

Constructor
*/
public RestfulServer() {
this(new FhirContext());
}

Where the constructor for FhirContext is:
private FhirContext(FhirVersionEnum theVersion, Collection<Class<? extends IBaseResource>> theResourceTypes) {
if (theVersion != null) {
if (!theVersion.isPresentOnClasspath()) {
throw new IllegalStateException(getLocalizer().getMessage(FhirContext.class, "noStructuresForSpecifiedVersion", theVersion.name()));
}
myVersion = theVersion.getVersionImplementation();
} else if (FhirVersionEnum.DSTU1.isPresentOnClasspath()) {
myVersion = FhirVersionEnum.DSTU1.getVersionImplementation();
} else if (FhirVersionEnum.DSTU2.isPresentOnClasspath()) {
myVersion = FhirVersionEnum.DSTU2.getVersionImplementation();
} else if (FhirVersionEnum.DSTU2_HL7ORG.isPresentOnClasspath()) {
myVersion = FhirVersionEnum.DSTU2_HL7ORG.getVersionImplementation();
} else if (FhirVersionEnum.DEV.isPresentOnClasspath()) {
myVersion = FhirVersionEnum.DEV.getVersionImplementation();
} else {
throw new IllegalStateException(getLocalizer().getMessage(FhirContext.class, "noStructures"));
}
ourLog.info("Creating new FHIR context for FHIR version [{}]", myVersion.getVersion().name());
scanResourceTypes(toElementList(theResourceTypes));
}
This matches the exception being thrown.
But the question is, why is something in hapi-fhir-base have a dependency of these packages, given that these packages have a dependency on hapi-fhir-base? I've noticed in the latest hapi-fhir-base POM the dependencies for the dstu and dstu2 jars have been removed - I suspect for this reason.
So, how would this work normally if this was just in a normal App Server? Would it just rely on a shared class loader?
PP

Hi Paul,
This is basically done by design: The hapi-fhir-base JAR has the main HAPI library code, and the hapi-fhir-structures-XXX JARs have different versions of the model. The idea is that applications using HAPI need to depend on both of those things, but only need to pick one of the various model versions in order to save space. They can also include multiple model versions if they need their app to suport multiple versions of FHIR.
This does seem to cause a challenge in OSGi environments, since you have two JARs depending on each other. FWIW we use the same setup in the HAPI-HL7v2 library and the dev who put together the OSGi bundle for that library did it by creating a specific OSGi JAR with all versions of the model. I don't know if there is a better way than that (possibly the model JARs need something in their manifest?) but it would be cool if there was...

So is there an OSGi guide for the HL7 library? If so, can you point me to it, please?
I'm thinking that the only way to go is to split the base jar into two bundles. One that relies on the structure-xxx bundles, and the other that the structure-xxx bundles rely on.
I'll look more deeply into that and the HL7 OSGi guide if I can.
PP
Please excuse my fat fingers, as I'm using my iPhone
On Tue, Jun 9, 2015 at 5:05 AM -0700, "James Agnew"<EMAIL_ADDRESS>wrote:
Hi Paul,
This is basically done by design: The hapi-fhir-base JAR has the main HAPI library code, and the hapi-fhir-structures-XXX JARs have different versions of the model. The idea is that applications using HAPI need to depend on both of those things, but only need to pick one of the various model versions in order to save space. They can also include multiple model versions if they need their app to suport multiple versions of FHIR.
This does seem to cause a challenge in OSGi environments, since you have two JARs depending on each other. FWIW we use the same setup in the HAPI-HL7v2 library and the dev who put together the OSGi bundle for that library did it by creating a specific OSGi JAR with all versions of the model. I don't know if there is a better way than that (possibly the model JARs need something in their manifest?) but it would be cool if there was...
—
Reply to this email directly or view it on GitHub.

Unfortunately there isn't. You could have a look though at how the hapi-osgi-base project gets built (this is the non-fhir version of HAPI, and this project bundles the base as well as all of the structures into a single JAR for OSGi)
http://sourceforge.net/p/hl7api/code/HEAD/tree/trunk/hapi-mvn/hapi-osgi-base/

Ok. Thanks James.
 You could have a look though at how the hapi-osgi-base project gets built (this is the non-fhir version of HAPI, and this project bundles the base as well as all of the structures into a single JAR for OSGi)
http://sourceforge.net/p/hl7api/code/HEAD/tree/trunk/hapi-mvn/hapi-osgi-base/
—
Reply to this email directly or view it on GitHub.


New release?

@sokra I see you merged a pile of PRs last night (including mine, #64, thanks).  Do you have an ETA for a new release of the package?  NPM tells me 0.8.5 was released seven months ago.

done

Thanks, much appreciated.


⚠ Bofrost.at has degraded performance

In e8a3681, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 928 ms

Resolved: Bofrost.at performance has improved in 3edadfc.


System.Data.SqlClient.SqlConnection is now Microsoft.Data.SqlClient.SqlConnection

as of .net core 3.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 31b0d33c-f88e-d50c-2926-7933940f3b92
Version Independent ID: 4aaef906-f501-6947-5d23-1b704470f09d
Content: Tutorial: Access data with managed identity - Azure App Service
Content Source: articles/app-service/app-service-web-tutorial-connect-msi.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

Sorry for the delay. It's fixed in https://github.com/MicrosoftDocs/azure-docs-pr/pull/112640 and should be published sometime today. #please-close


library not found for -lPods

Hi,
I followed the iOS tutorial to setup the plugin but I have this error ld: library not found for -lPods-test.
Another strange things is, I have libPods-test.a in red in the Frameworks folder. I don't really know pods, I changed the Framework Search Paths to use $(inherited) for warning reason.
Also the Header Search Path is correctly set to : $(SRCROOT)/../node_modules/react-native-onesignal/ios/RCTOneSignal without recursion.
Thanks for your help.

Can you please provide a repo with this issue we can take a look at?

Hi @avishayil
Thanks for your reactivity, I tried on clean project and it works properly !
There must be a configuration problem on the other project.
Note: I will update this issue when I found the solution.
Thanks !

A little update, I'm getting this error now :

After some research, I found issue about the same things but all are correctly set in Link Binary with Libraries and OneSignal.framework is also imported in the Frameworks folder.
Am I missing something ?

@DCKT I can't really tell without looking on the repo closely. Can you provide a repo here or privately with the reproducible issue?

@avishayil I'm sorry, I'm unable to share the repo. And when I create a new project, I don't have any problem so it's obvioulsy a bug in my project configuration with other plugins.

Do you mind if i'll TeamViewer your computer? I would really like to help you on this.


Weights file not being read properly during training

Hi,
Thanks for this python implementation. I just tried it out on my custom data files and the initial weights file provided by you. I also tried it out on the weights file yolov4.conv.137 which is shipped by darknet.
On your file I am getting the below error:
File "train.py", line 10, in <module>
    pre_trained_weights="yolov4.weights",
  File "/data/mtare/darknet-py/venv/lib/python3.6/site-packages/yolov4/tf/__init__.py", line 152, in train
    self.load_weights(pre_trained_weights)
  File "/data/mtare/darknet-py/venv/lib/python3.6/site-packages/yolov4/tf/__init__.py", line 322, in load_weights
    utils.load_weights(self.model, weights_path)
  File "/data/mtare/darknet-py/venv/lib/python3.6/site-packages/yolov4/core/utils.py", line 135, in load_weights
    assert len(wf.read()) == 0, "failed to read all data"
AssertionError: failed to read all data

On yolov4.conv.137 I get the following error:
File "train.py", line 10, in <module>
    pre_trained_weights="yolov4.conv.137",
  File "/data/mtare/darknet-py/venv/lib/python3.6/site-packages/yolov4/tf/__init__.py", line 152, in train
    self.load_weights(pre_trained_weights)
  File "/data/mtare/darknet-py/venv/lib/python3.6/site-packages/yolov4/tf/__init__.py", line 324, in load_weights
    self.model.load_weights(weights_path).expect_partial()
  File "/data/mtare/darknet-py/venv/lib/python3.6/site-packages/tensorflow/python/keras/engine/training.py", line 250, in load_weights
    return super(Model, self).load_weights(filepath, by_name, skip_mismatch)
  File "/data/mtare/darknet-py/venv/lib/python3.6/site-packages/tensorflow/python/keras/engine/network.py", line 1259, in load_weights
    with h5py.File(filepath, 'r') as f:
  File "/data/mtare/darknet-py/venv/lib/python3.6/site-packages/h5py/_hl/files.py", line 408, in __init__
    swmr=swmr)
  File "/data/mtare/darknet-py/venv/lib/python3.6/site-packages/h5py/_hl/files.py", line 173, in make_fid
    fid = h5f.open(name, flags, fapl=fapl)
  File "h5py/_objects.pyx", line 54, in h5py._objects.with_phil.wrapper
  File "h5py/_objects.pyx", line 55, in h5py._objects.with_phil.wrapper
  File "h5py/h5f.pyx", line 88, in h5py.h5f.open
OSError: Unable to open file (file signature not found)

Technically both should work as they do with the C implementation of darknet yolov4.
Please help

Hi @mon28
yolov4.weights is trained weights using data/classes/coco.names
Maybe, the number of classes is different from yours. So remove pre_trained_weights or set to None.
this module recognize if xxx.weights or not, so, I renamed yolov4.conv.137 to yolov4.conv.137.weights but failed. perhaps, yolov4.conv.137's format is not .weights

PS.
This module is not yet fully tested. So there may be a bug.

yolov4.weights is trained weights using data/classes/coco.names
Maybe, the number of classes is different from yours. So remove pre_trained_weights or set to None.

Oh I see! It makes sense in prediction, but training should facilitate a different number of classes too. That's how the transfer learning could work.
I hope this could get fixed soon.
Thanks for the quick response and a temp solution. This does work.
The training finished very quickly with no stats on console. Any way to make it print something?
What about adjusting hyperparameters? epochs/learning rate/ input width and height etc.

Actually my training existed right after creating the very first checkpoint with no stats or anything in the logs. @hhk7734 could you help with this?

hahah... there are concepts that I don't know because I only studied machine learning for a month.
When training, it seems that Yolo can get the weight of the rest of the network except for the part that changes the network structure based on the number of classes. but I am not sure.
https://github.com/hhk7734/tensorflow-yolov4/blob/19ae0babc461b1a607bd79a4cc9fd9fb50d27046/py_src/yolov4/tf/__init__.py#L119-L148
In the above part, you can set the parameters. maybe ~/.local/lib/python3.6/site-packages/yolov4/tf/__init__.py
It will be changed later so that it can be assigned as a parameter.

Alright i can change here but the training is not starting apparently. It wont print a single epoch.

Can you share your training data?

@hhk7734 not the image files as they are confidential but other files I can. Where would it be convenient to share personally?

<EMAIL_ADDRESS>attach files or add a download link.

Have sent them to you. Please help resolve this.

yolov4.core.dataset.Dataset class use format liki:
/home/hhk7734/Desktop/coco/images/val2017/000000356094.jpg 185,35,342,384,0 405,117,471,335,0 304,84,459,124,34 402,228,424,253,35

but yolo dataset is
/home/hhk7734/Desktop/coco/images/val2017/000000356094.jpg

/home/hhk7734/Desktop/coco/images/val2017/000000356094.jpg
/home/hhk7734/Desktop/coco/images/val2017/000000356094.txt
0 0.502445652174 0.51884057971 0.0244565217391 0.00579710144928

To fix this, Dataset class should be modified to recognize the yolo data format.

Could you explain the yolov4 data structure again? I don't get where the label id for each bounding box should be written. Also what is the overall structure?
Mine is:
./data
|
| ------ obj.names (names of all classes)
| ------ obj.data
| ------ train.txt
| ------ test.txt
| ------- /obj
| ----- all jpg files
| ----- all txt files in the format      for each image where all of these are relative to width and height of the image
I can change it to yolov4 format if you could clarify it like above.

Oh.. sorry, I accidentally deleted a part of your comment.
Anyway
./data
    | ------ obj.names (names of all classes)
    | ------ train.txt
    | ------ /obj
        | ----- all jpg files

train.txt
<file path> <#class> <x pixel> <y pixel> <w pixel> <h pixel>

train.txt + obj/xx.txt == coco_train.txt

here x, y, width, and height, are they relative to image width/height or absolute?

https://github.com/hhk7734/tensorflow-yolov4/blob/19ae0babc461b1a607bd79a4cc9fd9fb50d27046/scripts/coco_annotation.py#L73-L81
using script/coco_convert.py and script/coco_annotation.py can covert coco_data.json to coco_train.txt

example
640X480 image
left top is 0, 0
right bottom is 640,480
person is LRTB = (400, 450, 140, 250)
person class name number is 0
person.jpg 400,140,450,250,0

Alright, got it. Thanks :)

Add Yolo train dataset format support: 7bf3527

core: utils: implement a way to partially load weights: c4f092a


Fix URL on failure for OpenIdConnect Sample

I used this sample to set up our Auth0 with a new MVC 6 app, worked great!!  Only issue I had was when navigating to a protected page when not logged in, it would redirect to a non-existent route.  Just need to correct line 85 of Startup.cs to use the "/Account/Login" route instead of just "/Login".
Thanks!

Thanks for reporting this, we fixed this in our updated sample.


Slash Commands

Description
Add Slash Commands To The Discord Bot With The Most Used Commands
Reasoning
This Would Make Using Common Commands Easy And Accessible

[ ] I'd like to implement this feature myself
[ x] Anyone can implement this feature

Hello @NeilShah2026, thanks for your interest in our project.
As Discord.py doesn’t provide any slash command support at the moment and your issue doesn’t suggest any other way to achieve this, I am going to close this for now.
Please reach out in the #dev-contrib in our Discord server before opening such major issues.
Thank you,

Akarys


add flags to control stages of dependency build

Description
For building the Rust bindings we need to provide crates.io with a complete source tree that does not depend on external downloads. So during the process of generating the bindings we want to download all the dependencies, then when building the crate we do no further network communication and jsut build the already downloaded dependencies.
To do this I've added support for two environment variables to the relevant build scripts: OIIO_DEP_BUILD_ONLY and OIIO_DEP_DOWNLOAD_ONLY that do what you would expect.
Tests
N/A
Checklist:

[x] I have read the contribution guidelines.
[x] If this is more extensive than a small change to existing code, I
have previously submitted a Contributor License Agreement
(individual, and if there is any way my
employers might think my programming belongs to them, then also
corporate).
[x] I have updated the documentation, if applicable.
[ ] I have ensured that the change is tested somewhere in the testsuite
(adding new test cases if necessary).
[x] My code follows the prevailing code style of this project.

Umm yeah. Duh. I removed the BUILD_ONLY stuff and added the missing guard to build_gif.bash

Looks good, I'll merge it.
There are two missing pieces that you'll need, though. The cmake build system itself still downloads fmt and robin-map (see the end of src/cmake/externalpackages.cmake). They only download if the packages are not found installed. But still, I think what we want is to make build_fmt.bash and build_robinmap.bash scripts so that they can be easily pre-installed to avoid any possible download. (And maybe afterwards, we should also simplify the cmake to assume that they are present instead of doing downloads as part of the build.)

Yep currently I just run a configure to get robin-map and fmt. Which works,
but is ugly because of the error messages. I can look into adding specific
download scripts for them too.
On Sun, 15 Aug 2021 at 04:58, Larry Gritz @.***> wrote:

Merged #3058 https://github.com/OpenImageIO/oiio/pull/3058 into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OpenImageIO/oiio/pull/3058#event-5158919884, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOYQXM24NCJNRGNREYMQLTT42ODXANCNFSM5CEQKYOA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

I can cobble them together also if you don't have the time. I mainly just wanted to be sure you were aware that the current set of build_*.bash scripts don't 100% get everything you need to avoid network access during any part of the cmake process.


train.py line35 has error

when I run train.py, it shows the error message: "TypeError: Expected int32, got list containing Tensors of type '_Message' instead."
I'm wonder what's the problem is?
I didn't modify any code, just run the downloaded package.

The code only works for early version of tensorflow. Maybe you can try 0.10


nixos/tests: unbreak the tested job

I expect it suffices that the channel only blocks on one firefox ESR
test - the one for the default ESR.  I didn't want to have the
information about the default in two places, so either of the tests will
be evaluated twice (but to the same *.drv I hope).
Motivation for this change
Hydra evaluations are blocked!
Things done
N/A... this doesn't change hashes.

There are multiple ways of resolving the problem... comment fast if you prefer another, or we switch later.

Tough. Could this be a reference somehow instead of a redefinition? I think we are talking about a let binding then.
Meh. Many ways to do this, as you said. Let's go with that.


Deadlock during dynamic cluster bringup

Erlang/OTP 20
Elixir 1.5.1
Swarm 3.1.0

I may have observed a deadlock with simultaneous bringup of 3 nodes. In this case all 3 trackers are stuck in :syncing or :cluster_wait states, and they do not recover.
16:48:35.537 [info] [swarm on<EMAIL_ADDRESS>[tracker:init] started
16:48:35.978 [info] [swarm on<EMAIL_ADDRESS>[tracker:ensure_swarm_started_on_remote_node] nodeup<EMAIL_ADDRESS>16:48:36.186 [info] [swarm on<EMAIL_ADDRESS>[tracker:ensure_swarm_started_on_remote_node] nodeup<EMAIL_ADDRESS>16:48:40.538 [info] [swarm on<EMAIL_ADDRESS>[tracker:cluster_wait] joining cluster..
16:48:40.538 [info] [swarm on<EMAIL_ADDRESS>[tracker:cluster_wait] found connected nodes<EMAIL_ADDRESS><EMAIL_ADDRESS>16:48:40.538 [info] [swarm on<EMAIL_ADDRESS>[tracker:cluster_wait] selected sync node<EMAIL_ADDRESS>16:48:40.636 [info] [swarm on<EMAIL_ADDRESS>[tracker:syncing] pending sync request from<EMAIL_ADDRESS>16:51:14.556 [info] [swarm on<EMAIL_ADDRESS>[tracker:syncing] pending sync request from<EMAIL_ADDRESS>
16:48:35.633 [info] [swarm on<EMAIL_ADDRESS>[tracker:init] started
16:48:35.978 [info] [swarm on<EMAIL_ADDRESS>[tracker:ensure_swarm_started_on_remote_node] nodeup<EMAIL_ADDRESS>16:48:36.192 [info] [swarm on<EMAIL_ADDRESS>[tracker:ensure_swarm_started_on_remote_node] nodeup<EMAIL_ADDRESS>16:48:40.634 [info] [swarm on<EMAIL_ADDRESS>[tracker:cluster_wait] joining cluster..
16:48:40.635 [info] [swarm on<EMAIL_ADDRESS>[tracker:cluster_wait] found connected nodes<EMAIL_ADDRESS><EMAIL_ADDRESS>16:48:40.635 [info] [swarm on<EMAIL_ADDRESS>[tracker:cluster_wait] selected sync node<EMAIL_ADDRESS>16:48:40.904 [info] [swarm on<EMAIL_ADDRESS>[tracker:syncing] pending sync request from<EMAIL_ADDRESS>16:49:38.217 [info] [swarm on<EMAIL_ADDRESS>[tracker:syncing] pending sync request from<EMAIL_ADDRESS>
16:48:35.899 [info] [swarm on<EMAIL_ADDRESS>[tracker:init] started
16:48:36.182 [info] [swarm on<EMAIL_ADDRESS>[tracker:ensure_swarm_started_on_remote_node] nodeup<EMAIL_ADDRESS>16:48:36.189 [info] [swarm on<EMAIL_ADDRESS>[tracker:ensure_swarm_started_on_remote_node] nodeup<EMAIL_ADDRESS>16:48:40.535 [info] [swarm on<EMAIL_ADDRESS>[tracker:cluster_wait] pending sync request from<EMAIL_ADDRESS>16:48:40.900 [info] [swarm on<EMAIL_ADDRESS>[tracker:cluster_wait] joining cluster..
16:48:40.901 [info] [swarm on<EMAIL_ADDRESS>[tracker:cluster_wait] found connected nodes<EMAIL_ADDRESS><EMAIL_ADDRESS>16:48:40.901 [info] [swarm on<EMAIL_ADDRESS>[tracker:cluster_wait] selected sync node<EMAIL_ADDRESS>

This should be addressed in the latest release, and there are some additional fixes/improvements in master.

@bitwalker I'm having the same issue reproducible very often when I bring up containers with the app at roughly the same time. Here are the :sys.get_status(Swarm.Tracer) results from my 5 nodes: https://pastebin.com/EYLg6YNE . Looks like each node is waiting for another one.


Add 'save and open' command

This PR adds a command that saves the current workspace (without having to select which workspace they're saving, it works it out) and then lets the user switch to a different workspace.
If the current project isn't recognised as a workspace, the user will be asked to save the workspace before opening a new one.
There's definitely room for improvement on this, but any thoughts are appreciated. 😄

@jakewtaylor hey :) should we continue with this PR?

I no longer use atom so I don’t really mind what happens to this PR! 😀

@jakewtaylor me neither :) ok, so will close it, thanks :)


Resolved Alertmanager alerts are fired against the sink

Describe the bug
Alerts in resolved status are sent again to the sinks, giving the wrong impression that the problem hasn't been resolved.
To Reproduce
Steps to reproduce the behavior:

Trigger a Prometheus alert
Resolve that alert
Check against the main sink

Expected behavior
Either the alert shouldn't be fired again, or it should be labeled in a way that makes clear that the problem has been resolved
Screenshots

A duplicate alert, where the 2nd message is from the resolved alert in Prometheus

Yeah, that's a bug.
My preference is to not send a message the second time at all. @arikalon1 what are your thoughts?
@michelesr This is slightly off-topic, but we're planning to do some work around alert lifecycle in the UI. If the same alert fires multiple times, we'd like to show those firings as one continously firing alert, not several distinct alerts. When the alert is finally resolved we'd like to also show that and mark the alert as closed. Do you have any thoughts about this? Other than fixing this bug, are there other features you'd like to see regarding alert lifecycle in either the UI or Slack?

Maybe implement a repeat interval like in Alertmanager?

Yeah, perhaps we could do it with better rate limiting (as part of the trigger) or some form of rate limiting on the sink.

I think we should add the alert status (firing / resolved) and send it to sinks, similarly to what alert manager does.
I think repeat interval/rate limiting applies on the same alert, but if the alert status has changed from firing to resolved the rate limiting shouldn't silence it.

@arikalon1 if we do that, how do we address the original issue here? By filtering out resolved alerts in the Slack sink?

Got it. This sounds good to me.

@michelesr hey, we're going to fix this in the current sprint.

@michelesr we added a fix tha labels the alert as resolved.
You're more then welcome to share more feedback with us 🙂