need some info regarding some #define

it is obvious whats the usage of "ANTI_DEBUG", but what about "NOP_FLOOD"?
another thing is with "__CRASH", would you recommend it over abort, exit, or exitprocess? or it is for future use as it is currently not used at all.
what about "ANTITAMPER", how is it going to prevent tampering with code?
last thing, do you recommend:
#define NO_CFLOW       0
#define NO_ANTIDEBUG   0
or
completely remove NO_*?
currently it is same whether it is defined as 0 or completely removed, but the question is really for future consistency if for example other switches in the future may accept other values.
thank you.

Hello, @zmn28hgbn59kcmlpio8unfh7fdre523esd28q9a! Thank you for your questions; they are quite relevant. I’ll address them in the same order.

it is obvious whats the usage of "ANTI_DEBUG", but what about "NOP_FLOOD"?

The NOP_FLOOD macro is designed exclusively for use within the obfus.h library. It acts as auxiliary obfuscation functionality, similar to the BREAK_STACK_* macros.

another thing is with "__CRASH", would you recommend it over abort, exit, or exitprocess? or it is for future use as it is currently not used at all.

__CRASH deliberately causes a program crash through intentional failure, rather than a proper exit. This is meant to protect critical sections from hooking functions like ExitProcess or exit(). Whether to use it depends on your goals, but if it’s undocumented, consider whether it’s necessary for your case.

what about "ANTITAMPER", how is it going to prevent tampering with code?

This feature is currently in the conceptual stage and might even be removed. The original idea was to assign an N-flag within control flow obfuscation and validate it during execution. If any condition was skipped, the program would crash. The implementation, however, remains undecided.

last thing, do you recommend:
#define NO_CFLOW 0
#define NO_ANTIDEBUG 0

or

completely remove NO_*?

If you don’t plan to disable any obfuscation features, it’s cleaner to omit the NO_* macros entirely. Initially, I considered general macros like CFLOW_LVL with values from 0 to 2 (0 - disabled, 1 - default, 2 - enhanced). However, this approach could confuse users, who might think arbitrary values are allowed or that removing a declaration disables the feature.
To avoid this confusion, I opted for two clear options: disable the feature or enable a stronger mode. The default medium level doesn’t require explicit settings or declarations.
It’s worth noting that the example code provided in the documentation intentionally includes all settings to demonstrate the full range of configurations available. However, in real-world use, the code can be significantly simplified by omitting unused options or relying on defaults.
For future consistency, if additional switches are introduced, you can always expand or adapt the NO_* macros. That said, for now, whether to declare them as 0 or omit them entirely is up to you and depends on your coding preferences.

Hello, @zmn28hgbn59kcmlpio8unfh7fdre523esd28q9a! Thank you for your questions; they are quite relevant. I’ll address them in the same order.

it is obvious whats the usage of "ANTI_DEBUG", but what about "NOP_FLOOD"?

The NOP_FLOOD macro is designed exclusively for use within the obfus.h library. It acts as auxiliary obfuscation functionality, similar to the BREAK_STACK_* macros.

another thing is with "__CRASH", would you recommend it over abort, exit, or exitprocess? or it is for future use as it is currently not used at all.

__CRASH deliberately causes a program crash through intentional failure, rather than a proper exit. This is meant to protect critical sections from hooking functions like ExitProcess or exit(). Whether to use it depends on your goals, but if it’s undocumented, consider whether it’s necessary for your case.

what about "ANTITAMPER", how is it going to prevent tampering with code?

This feature is currently in the conceptual stage and might even be removed. The original idea was to assign an N-flag within control flow obfuscation and validate it during execution. If any condition was skipped, the program would crash. The implementation, however, remains undecided.

last thing, do you recommend:
#define NO_CFLOW 0
#define NO_ANTIDEBUG 0
or
completely remove NO_*?

If you don’t plan to disable any obfuscation features, it’s cleaner to omit the NO_* macros entirely. Initially, I considered general macros like CFLOW_LVL with values from 0 to 2 (0 - disabled, 1 - default, 2 - enhanced). However, this approach could confuse users, who might think arbitrary values are allowed or that removing a declaration disables the feature.
To avoid this confusion, I opted for two clear options: disable the feature or enable a stronger mode. The default medium level doesn’t require explicit settings or declarations.
It’s worth noting that the example code provided in the documentation intentionally includes all settings to demonstrate the full range of configurations available. However, in real-world use, the code can be significantly simplified by omitting unused options or relying on defaults.
For future consistency, if additional switches are introduced, you can always expand or adapt the NO_* macros. That said, for now, whether to declare them as 0 or omit them entirely is up to you and depends on your coding preferences.

Hello, @zmn28hgbn59kcmlpio8unfh7fdre523esd28q9a! Thank you for your questions; they are quite relevant. I’ll address them in the same order.

it is obvious whats the usage of "ANTI_DEBUG", but what about "NOP_FLOOD"?

The NOP_FLOOD macro is designed exclusively for use within the obfus.h library. It acts as auxiliary obfuscation functionality, similar to the BREAK_STACK_* macros.

another thing is with "__CRASH", would you recommend it over abort, exit, or exitprocess? or it is for future use as it is currently not used at all.

__CRASH deliberately causes a program crash through intentional failure, rather than a proper exit. This is meant to protect critical sections from hooking functions like ExitProcess() or exit(). Whether to use it depends on your goals, but if it’s undocumented, consider whether it’s necessary for your case.

what about "ANTITAMPER", how is it going to prevent tampering with code?

This feature is currently in the conceptual stage and might even be removed. The original idea was to assign an N-flag within control flow obfuscation and validate it during execution. If any condition was skipped, the program would crash. The implementation, however, remains undecided.

last thing, do you recommend:
#define NO_CFLOW 0
#define NO_ANTIDEBUG 0
or
completely remove NO_*?

If you don’t plan to disable any obfuscation features, it’s cleaner to omit the NO_* macros entirely. Initially, I considered general macros like CFLOW_LVL with values from 0 to 2 (0 - disabled, 1 - default, 2 - enhanced). However, this approach could confuse users, who might think arbitrary values are allowed or that removing a declaration disables the feature.
To avoid this confusion, I opted for two clear options: disable the feature or enable a stronger mode. The default medium level doesn’t require explicit settings or declarations.
It’s worth noting that the example code provided in the documentation intentionally includes all settings to demonstrate the full range of configurations available. However, in real-world use, the code can be significantly simplified by omitting unused options or relying on defaults.
For future consistency, if additional switches are introduced, you can always expand or adapt the NO_* macros. That said, for now, whether to declare them as 0 or omit them entirely is up to you and depends on your coding preferences.

thank you for your detailed answer, i like the feature_lvl [0..2], but i agree that it can confuse some users.
i have 1 more question:
https://github.com/DosX-dev/obfus.h/blob/main/include/obfus.h#L88
"#if defined(FAKE_SIGNS) && (FAKE_SIGNS != 0)"
why did you use this combination, why not straight to #if FAKE_SIGNS != 0?
this could also open 1 more question for future use, like:
the usage of #define without any values, i.e. #define FAKE_SIGNS without the need to provide any specific value.
so far this is the only inconsistency i found in macro switches.
thank you again, and you may close this issue after explaining the statement above.


Fetch password from credential store before connecting to knox endpoint

The password isn't always populated in the connection profile yet so make sure we fetch it before trying to connect. We were already doing this below for setting the Spark context - this is just moving it up so the Knox query also has it.
Also optimized it a bit to not doing the query if we're integrated auth since we don't need to use the username in that case anyways.

Thanks, @Charles-Gagnon. Is this why we see some failures when using a spark kernel initially?

What kind of errors were you seeing? This shouldn't be fixing anything with that since it was already getting the credentials.
There is another possible issue though I wasn't able to consistently repro where the credential wasn't returned so it'd fail then. Maybe that's what you're talking about?

@Charles-Gagnon from time to time, using one of the spark kernels would fail due to no password being passed into the do_not_call_change_endpoint, and we'd get stuck in that 401 loop. i can help test this when it's in to see if there would be any (positive) behavior difference 😄


bug fix: default values in get_manta_sv

There was a discrepancy in how get_manta_sv() worked when no metadata was provided since the approach in id_ease has changed. This small PR resolves the issue.

This resolves the #6

There is an error when I rum devtools:check() and I don't think it is related to this PR. The error also happen in my main branch.
Error in loadNamespace(j <- i[[1L]], c(lib.loc, .libPaths()), versionCheck = vI[[j]]) :
  there is no package called ‘GAMBLR’
Calls: <Anonymous> ... loadNamespace -> withRestarts -> withOneRestart -> doWithOneRestart
Execution halted
ERROR: lazy loading failed for package ‘GAMBLR.results’

Do you get this error as well?

This repo is not related to GAMBLR, and my GAMBLR is working fine!

0 errors ✔ | 0 warnings ✔ | 6 notes ✖

I think this means that the documentation in one of the functions is not up-to-date since it refers somewhere to GAMBLR that your check picked up. I run the check with no vignettes, no examples arguments

Got the very same error message with devtools::check(vignettes = FALSE, args = '--no-examples')

Interesting! Have you already uninstalled GAMBLR (the main repo)?

I can't understand yet why this error but I realized that I had used git checkout to a old commit in GAMBLR. then I git checkouted back to master and now devtools::check(vignettes = FALSE, args = '--no-examples') gives this
0 errors ✔ | 2 warnings ✖ | 2 notes ✖

To have this tested properly and avoid any confusion, we need to properly set up the github actions just like we had it for the GAMBLR main repo. I imagine everyone's R state is different because of different packages installed from different commits etc but having it configured in actions will ensure the reproducibility. This is a topic for a separate and more wide discussion


Test against pyspark3.1

Description
Please describe your pull request!
Checklist

[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[ ] I have updated the CHANGELOG.md

@ekstromchristian good job 👍 now re-add support for pyspark 3.0 (in conftest.py)


Added example cpp20_streams, which reproduces an assertion.

I've created both a PR and an issue as I wasn't sure what would be the preferred way to deal with this. Feel free to close either one.
Please see this PR for a description:
https://github.com/boostorg/redis/pull/84

Hi, thanks for reporting, I will have a look. Does the problem persist if you use only one thread.

Indeed, I also can't reproduce using only 1 thread.

I've just added a commit to clean it up a bit.

The current workaround I'm using is by giving the redis_client its own ioc (instead of sharing it with other classes that use it), and that ioc runs on a single thread. However, this shouldn't be a requirement for using boost::redis, correct? In other words, redis_client and boost::redis should work with an ioc that runs on multiple threads, even though the strand ensures that calls to async_exec and its completion handler aren't executed concurrently.

Correct, it shoud be possible to use boost.redis with multi-threaded io_contexts as well. There is some concurrent issue there that I couldn't spot yet, probably in the use of strand.

I have rewritten your example using some facility functions I am developing in another branch and couldn't also see any crash: https://gist.github.com/mzimbres/b4e25f4b95a1c4f5c8ce8aa29dde6352.

I've added a new branch with some changes to the examples, see here:
https://github.com/bveldhoen/redis/tree/83-fix-reconnect-loop-in-the-subscriber-example-with-streams-example
Points of attention:

I've had to add a call to req_.clear() in cpp17_streams. Without it, the example didn't function properly on my machine (it took a long time before the writes to the console became visible).
The problem seemed to go away in cpp20_streams when using the new redis::async_run method. Other changes (i.e. members for req and resp) didn't make any difference. See the changes in the top commit for the fix for cpp20_streams.
Going forward, I think the name cpp17_streams and the changed implementation are better suitable. cpp20_streams can be removed.

Yes, that would be helpful indeed, thanks!

Note to self: Health checks may be a bad thing to use with XREAD on blocking mode.


"Desctructive Application" of Saved Gardens

What's New?

Add "destructive" and "non-destructive" application of saved gardens (plus dep. tracking checks to make sure data does not go bad)
Refactor the Transport module to not be a singleton (simplifies testing)

Coverage remained the same at 90.862% when pulling 7169ee78a4fae814269239281254d25aeeae5468 on RickCarlino:garden_snapshot_ii into ca7453e4ae60fd219ba53808129c9f3a6f30f71d on FarmBot:staging.


launch_fluent start_instance documentation

The description could be more straightforward, starting with the True case.
"Whether to start a local Fluent instance. The default is None, which indicates True. Otherwise, use ... < rest as current doc >

The description could be more straightforward, starting with the True case.

Why can not I connect  to a existed fluent, by
solver=pyfluent.launch_fluent(
    # product_version='22.2.0',
    start_instance=False,
    ip='<IP_ADDRESS>',
    port=49818)


Automated Package Upgrade

:tada: This PR is included in version 2.2.254 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


(chore) - track size of bundles

Track bundle size of changes compared to the main branch

Closing for now :v: I think the main problem right now is that the action writes a comment and not to the checks output, which we could set up as a custom action; but mainly I've also checked and the action double-builds and runs checkouts of the main branch, which I find a little unexpected. My hope is that #1568 can supersede this and that we can compare actual artifact outputs between the target branch and PR branch. But for now, I also planned that PR to double as a CSB CI replacement which it can't be right now, because GitHub's artifacts are always zipped on download.


Since latest update release 9.1 all my devices deleted and wont come up anymore

What issue do you have? Please be as thorough and explicit as possible.
Since latest update release 9.1 all my devices deleted and wont come up anymore.
After downgrade to 9.01 still all devices deleted and wont come up anymore.
I am using this plugin for about 3 years without trouble until 9.01.
This issues comes up yesterday by updating to 9.1
Details of your setup.

Do you use (1) Homebridge UI-X (2) Homebridge CLI or (3) HOOBS?

1

Which version of Homebridge/HOOBS do you have?

1.5.0

Which platform do you run Homebridge/HOOBS on (e.g. Raspberry Pi/Windows/HOOBS Box)? Please also mention your version of Node.js/NPM if known.

macOS
node.js=16.16.0
npm=8.13.2

Which version of this plugin (homebridge-ewelink) do you have? Has the issue started since upgrading from a previous version?

9.1.0

Which eWeLink devices do you have that are causing issues? Please include product models if applicable.

King-Q4
Please paste any relevant logs below.

Thanks for you quick response. That only confirm my issue, but why these devices wont get initialized again by the version before?

Bummer, my Eachen GD-DC5 WI-FI Garage Door Openers have been booted out of the plugin!
They both work in the eWeLink app.
Mo Sonoff Basic still works.
First time I've ever had HomeKit issues.
I hope there is a fix soon and we haven't permanently lost a good thing.
I'd hate to have to switch to a HomeKit Only device.

Yes I can confirm, the beta 9.11 works well with the blind switch and yes I needed to create some new scenes on HomeKit
Thanks for the fast fix

Ja, ich kann bestätigen, die Beta 9.11 funktioniert gut mit dem Blindschalter und ja, ich musste einige neue Szenen auf HomeKit erstellen. Danke für die schnelle Lösung

You have to create your plugins as a subbridge, then not all devices should be thrown out of the Homekit if there are problems. Homebridge makes a backup every evening, you could then import it too.


inline all the functions in the asm module

@homunkulus r+

:pushpin: Commit 2e534ea has been approved by japaric

:hourglass: Testing commit 2e534ea with merge 2e534ea...

:sunny: Test successful - status-travis
Approved by: japaric
Pushing 2e534ea4d85f321a4175a58fc77455380360ca9d to master...


ARM binary 32bit or 64bit?

Hi, I can see you've produced an array of cross platform binaries, but I'm wondering if the arm binary is 32bit or 64bit?
With the advent of the RPi4 and the new 64bit Raspberry OS I think there's a case for both 32bit arm binaries for the older raspberry pi models and a 64bit arm binary for the newer architecture, although I understand that would be more work for yourself, at the least I think it might be helpful to clarify which arm architecture it's for like you currently do with the Linux version.
Thanks for reading and for all your work.

The arm binary is 32bit (note that 64bit is aarch64). I can easily add aarch64 binaries to the release script, and I'll do that for the next version.

Awesome, I was 98% sure they'd be 32 bit.  Forgive me for the terminology slip up.

I can easily add aarch64 binaries to the release script, and I'll do that for the next version

Appreciate that, I really do.  Thanks mate.
I'll leave the decision on whether to close this (non)issue to yourself, clearly my question has been answered, just not sure if you like to leave things open to track them, or whether you prefer to clear the decks.

This has been released in v3.1.3.


Enable testing with sqlite

Describe the maintenance task
Set sqlite as the default database.
Describe why this task is necessary
Testing requires a Postgres database, which adds unnecessary complications.
Additional context
CI should continue to test against postgres (possibly also sqlite and multiple postgres versions)

We use Postgres 13 in CE production


Add note on extra result on DataDriven Tests

Summary
Data Driven tests give an extra result that is the sum of all the data driven tests. So if all are passing then the summary test passes, if one fails then the summary test also fails. We got some feedback that this extra result was confusing and does not provide them much value.

If the test fails, I am only about which set of input data caused the failure. A comment on this docs would also help eliminate the confusion.

Adding a note to docs would help reduce confusion while we take this design feedback into account.

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

ID: e1351cc8-f703-27a2-32c7-904256cb9e82
Version Independent ID: 7a6912ef-da8d-6690-6da9-cc741fd5acde
Content: How To: Create a Data-Driven Unit Test in Visual Studio - Visual Studio
Content Source: docs/test/how-to-create-a-data-driven-unit-test.md
Product: visual-studio-dev15
GitHub Login: @gewarren
Microsoft Alias: gewarren

CRUNCH!


Introduce the concept of a DialogService

The current ProgressService has started to be used for multiple things, and the intention of this Service is becoming unclear.
Lets introduce a new IDialogService, which will control the creation and showing of the various dialog that are currently used within Chocolatey GUI, for example when showing a Message, asking for confirmation on an action, etc.
This should make it much clearer about which service should be used/required when necessary.

:tada: This issue has been resolved in version 0.20.0 :tada:
The release is available on:

GitHub Release
Chocolatey Package

Your GitReleaseManager bot :package::rocket:


Crash when playing HLS on Android Emulator level 29

Version
Media3 1.2.0
More version details
No response
Devices that reproduce the issue

Emulator running API 29 on mac with M series processor

Devices that do not reproduce the issue

Xiaomi Redmi 9c running Android 10 (API 29 also)

Reproducible in the demo app?
Yes
Reproduction steps

Run demo app
Go to HLS
Choose the first one
App will stuck in there and not showing anything and emulator will be frozen
Emulator will crash after long period of frozen

Expected result
The media should plays successfully
Actual result
Player frozen and emulator also frozen. Here is the log
error.log
Media
Media is the same as in the demo and HLS
Bug Report

[X] You will email the zip file produced by adb bugreport to<EMAIL_ADDRESS>after filing this issue.

Hi @albilaga,
Happy new year and thanks for reporting! From your log, I saw the playback was trying to start and the last piece of log from the demo app is
{
      "header": {
        "logLevel": "DEBUG",
        "pid": 3895,
        "tid": 3895,
        "applicationId": "androidx.media3.demo.main",
        "processName": "androidx.media3.demo.main",
        "tag": "EventLogger",
        "timestamp": {
          "seconds":<PHONE_NUMBER>,
          "nanos": 433000000
        }
      },
      "message": "audioTrackInit [eventTime\u003d5.64, mediaPos\u003d0.00, window\u003d0, period\u003d0, 2,12,22050,false,false,22052]"
    },

I haven't got a solid idea on what was going on after it though.
Did you see this issue on a emulator with the specific definition? I'm using M1 mac and couldn't reproduce the issue with Pixel 5, Pixel 6 and Nexus 4. And all of them are with API 29.

@tianyif do you test on devices or in emulator? emulator always fail for me but devices all ok. emulator always like this

Yep, I tested on the emulators of Pixel 5, Pixel 6 and Nexus 4.

@tianyif Is it working properly for you in the emulator?

hmm.. there is something wrong with my emulator then. But not really sure which one is wrong also

@thisisthekap yes, they were working properly on the emulators when I ran. Did you see the same issue?

@tianyif Yes, same as @albilaga

@tianyif Have you gained any further insight on that one?


Panic if you specify localhost:port in allowed_http_hosts

If you write allowed_http_hosts = ["localhost:3001"] (forgetting the protocol), then make an outbound call, the Spin worker thread panics:
thread 'tokio-runtime-worker' panicked at 'called `Option::unwrap()` on a `None` value', crates/outbound-http/src/lib.rs:52:47

This occurs because it tries to parse localhost:3001 as { scheme: "localhost", host: None, port: None, path: "3001" }.  This passes load-time validation as a syntactically legitimate (a la urn:blah-blah-blah scheme) though absurd URI, but crashes when we try to unwrap the host part.

Fixed via #727.


Difficulties with saved model format

Hi all,
Currently I'm running into some difficulties using the TCAV framework for models based on the saved model format. When I use models stored and loaded using the h5 or frozen graph format all experiments work fine. When I then try to conduct experiments based on models stored and loaded with the saved model format there arise some issues.
First problem is that the loaded graph seems to be missing some operations of specific types such as "MaxPool" or "Reshape" which you could use as bottleneck layer with h5 or frozen graph models. After adapting the code to the loaded graph by changing the layer types/names I get an exception "tensorflow.python.framework.errors_impl.FailedPreconditionError [...]". Based on that I tried to implement some workarounds which unfortunately didn't solve the problem so far.
Did anyone had similar problems and managed to use models based on the saved model format for the TCAV framework?
Thanks!

Hi again,
are there any updates regarding this problem? Could anyone use models stored in the saved model format for the TCAV framework?
Thank you in advance!

@BeenKim any input?

@lkreiskoether I haven't seen this issue before, but believe that this is problem with tensorflow rather than TCAV. Are you able to load up the graph without TCAV and do a inference? If so, please feel free reopen and share a reproducible code (if you can) and we'll look int it. If not, I recommend posting the issue to stackoverflow for (more prompt) help!


SB-839: Cleanup the left-overs from the JAAS LDAP implementation

Removed the JwtEnabledCondition (SB-785).

@fuss86 ,
With your permission, I would now like to remove this, as it will be obsolete. Would you mind reviewing the changes in 83f80e61b827e469c1ada2b7c8c30b022b20c741?

Correct, everything here is unused now.

Thanks!


Fix Broken Tests

Closes #445
Mosts of the tests in test_broken don't need to be revived since the code was was already tested elsewhere.
One thing that I revived was the test_basic_star which might be useful since this is the easiest possible test of detection/flux estimation i.e. if this one is broken we are in trouble. I also added some assertions on precision/recall for the a sdss location encoder test in my case_studies folder. Most of the work was updating the star_basic.yaml.

Tests are running in the GPU, I will merge once that is done.

After running in GPU there are some issues see #485 , will resolve a couple in a separate PR.


[Feature] add rate limiter to login api to enhance security

Description
i want to add a express rate limiter to the login api
it will:

prevent bots attack
reduce load on api
please assign this issue to me @swaraj-das

Screenshots
No response
Checklist

[X] I have checked the existing issues.
[X] I have read the Code Of Conduct
[X] The changes don't break the code
[X] Please attach the screenshots and video of the development work once it is done

Hi there! This issue is still open. We are looking forward to your response. Assignees: @AyushSharma72

Yes I will do it you can add level


Below the screenshot using safemode.
https://pasteboard.co/JTwMYbO.png

looks like you have a corrupted .resume file, perhaps just delete the entire c:\Users\GTX1160\.frostwire\libtorrent folder inside your FrostWire settings folder before starting FrostWire

Problems solved. Thank you so much mate!
Btw, for the resuming half way transfers, is there ways to start multiple instead of clicking one by one?
Thank you in advance!

any time!


cfg attribute when running cargo-mutants?

This is a really nice crate and works pretty great!
I have one issue: I really don't want mutants to be a non-dev dependency.  Do you think it's possible to define an attribute which can be checked, so for example the following example would work? This probably won't need the mutants crate at all.
#[cfg_attr(mutants, mutants::skip)]
fn function_to_skip(&mut self);

Another possibility would be to set the test-attribute when creating mutations, so this would work (this would only need a dev-dependency):
#[cfg_attr(test, mutants::skip)]
fn function_to_skip(&mut self);

Thanks!
I think something like this is possible.
Due to the way cargo-mutants works, we currently look for the marker attributes by looking at the AST rather than running the real compiler's attribute/cfg evaluation system. (It would be more flexible to do that but I don't currently see a way to do it.) I could teach cargo-mutants to recognize either of those patterns.
In the first case it's kind of "grabbing" the feature flag "mutants" in this crate's namespace but perhaps that's OK...

Great to hear!
If you only parsing the AST it's most probably better to prefer the test-attribute pattern as (for me) it feels intuitive (as in: "I want to skip mutation this function when testing this create").
Is it possible to provide some help on implementing this without knowing exactly how this crate is working?

I played around a little bit and this seems to work (enhancing attr_is_mutants_skip):
/// True if the attribute is `#[mutants::skip]`.
fn attr_is_mutants_skip(attr: &Attribute) -> bool {
    if attr.path
        .segments
        .iter()
        .map(|ps| &ps.ident)
        .eq(["mutants", "skip"].iter()) {
        return true
    }

    if !attr.path.is_ident("cfg_attr") {
        return false;
    }

    if let syn::Meta::List(meta_list) = attr.parse_meta().unwrap() {
        let mut nested_metas = meta_list.nested.iter().filter_map(|meta| {
            if let syn::NestedMeta::Meta(syn::Meta::Path(path)) = meta {
                Some(path.segments.iter().map(|ps| &ps.ident).take(2).collect::<Vec<_>>())
            } else {
                None
            }
        }).take(2).collect::<Vec<_>>();
        if (nested_metas[0] == ["test"] || nested_metas[0] == ["mutants"]) && nested_metas[1] == ["mutants", "skip"] {
            return true;
        }
    }

    false
}


Yara 3.7.1 issue with Androguard

Hi,
I followed all the steps mentioned in Readme.txt for adding androguard module and recompiled my yara(version 3.7.1)
After that, I used the example mentioned in the Readme.txt.
python download_androguard_report.py -s d8adb784d08a951ebacf2491442cf90d21c20192085e44d1cd22e2b6bdd4ef5f
And downloaded the report and also downloaded the respective sample and pointed to the test rule file given in the repo,
yara -x androguard=d8adb784d08a951ebacf2491442cf90d21c20192085e44d1cd22e2b6bdd4ef5f-report.json test/rule.yar d8adb784d08a951ebacf2491442cf90d21c20192085e44d1cd22e2b6bdd4ef5f
I got the following error:
test/rule.yar(1): error: unknown module "androguard"
test/rule.yar(1): error: invalid field name "package_name"
Kindly help me with this issue

Same issue
please help

Did you compile yara with cuckoo module enabled? (

Yes, compiled with cuckoo module enabled. Also tried in two different systems and compiled yara several number of times. The mentioned issue happened every time.

I am also facing and have tried with cuckoo enabled


Table remarks overwritten by column remarks when new column is added to a table

Environment
Liquibase Version: 3.8.5, 4.0.0-beta1
Liquibase Integration & Version: CLI
Liquibase Extension(s) & Version:
Database Vendor & Version: MySQL 5.7
Operating System Type & Version: Win10
Description
The column remarks from an addColumn change overwrite the table remarks.
Steps To Reproduce
Create a change and add a changeSet createTable, add remarks for table and columns.
Add another changeSet addColumn, add remarks for column.
Run the changelogs against a database with updateSql. See the attached files to the next comment.
Examine the output sql.
Actual Behavior
Liquibase Community 3.8.5 by Datical
--  *********************************************************************
--  Update Database Script
--  *********************************************************************
--  Change Log: ./ChangeLog.yaml
--  Ran at: 6/12/20 8:54 PM
--  Against: user01@localhost@jdbc:mysql://localhost:3307/msrv_t15
--  Liquibase version: 3.8.5
--  *********************************************************************
--  Create Database Lock Table
CREATE TABLE msrv_t15.DATABASECHANGELOGLOCK (ID INT NOT NULL, LOCKED BIT(1) NOT NULL, LOCKGRANTED datetime NULL, LOCKEDBY VARCHAR(255) NULL, CONSTRAINT PK_DATABASECHANGELOGLOCK PRIMARY KEY (ID));
--  Initialize Database Lock Table
DELETE FROM msrv_t15.DATABASECHANGELOGLOCK;
INSERT INTO msrv_t15.DATABASECHANGELOGLOCK (ID, LOCKED) VALUES (1, 0);
--  Lock Database
UPDATE msrv_t15.DATABASECHANGELOGLOCK SET LOCKED = 1, LOCKEDBY = 'test (<IP_ADDRESS>)', LOCKGRANTED = '2020-06-12 20:54:53.868' WHERE ID = 1 AND LOCKED = 0;
--  Create Database Change Log Table
CREATE TABLE msrv_t15.DATABASECHANGELOG (ID VARCHAR(255) NOT NULL, AUTHOR VARCHAR(255) NOT NULL, FILENAME VARCHAR(255) NOT NULL, DATEEXECUTED datetime NOT NULL, ORDEREXECUTED INT NOT NULL, EXECTYPE VARCHAR(10) NOT NULL, MD5SUM VARCHAR(35) NULL, DESCRIPTION VARCHAR(255) NULL, COMMENTS VARCHAR(255) NULL, TAG VARCHAR(255) NULL, LIQUIBASE VARCHAR(20) NULL, CONTEXTS VARCHAR(255) NULL, LABELS VARCHAR(255) NULL, DEPLOYMENT_ID VARCHAR(10) NULL);
--  Changeset ./ChangeLog.yaml::1::me
CREATE TABLE msrv_t15.table01 (id BIGINT NOT NULL, column01 SMALLINT NOT NULL COMMENT 'column01 description', column02 SMALLINT NOT NULL COMMENT 'column02 description', CONSTRAINT PK_TABLE01 PRIMARY KEY (id)) COMMENT='table01 description';
ALTER TABLE msrv_t15.table01 COMMENT = 'table01 description';
INSERT INTO msrv_t15.DATABASECHANGELOG (ID, AUTHOR, FILENAME, DATEEXECUTED, ORDEREXECUTED, MD5SUM, DESCRIPTION, COMMENTS, EXECTYPE, CONTEXTS, LABELS, LIQUIBASE, DEPLOYMENT_ID) VALUES ('1', 'me', './ChangeLog.yaml', NOW(), 1, '8:15f66f4dd66c63a0560aa2114602a00a', 'createTable tableName=table01', '', 'EXECUTED', NULL, NULL, '3.8.5', '1984495461');
--  Changeset ./ChangeLog.yaml::2::me
ALTER TABLE msrv_t15.table01 ADD column03 BIGINT NULL COMMENT 'column03 description';
ALTER TABLE msrv_t15.table01 COMMENT = 'column03 description';
INSERT INTO msrv_t15.DATABASECHANGELOG (ID, AUTHOR, FILENAME, DATEEXECUTED, ORDEREXECUTED, MD5SUM, DESCRIPTION, COMMENTS, EXECTYPE, CONTEXTS, LABELS, LIQUIBASE, DEPLOYMENT_ID) VALUES ('2', 'me', './ChangeLog.yaml', NOW(), 2, '8:adfd8cf083db547ed1e7454727367879', 'addColumn tableName=table01', '', 'EXECUTED', NULL, NULL, '3.8.5', '1984495461');
--  Release Database Lock
UPDATE msrv_t15.DATABASECHANGELOGLOCK SET LOCKED = 0, LOCKEDBY = NULL, LOCKGRANTED = NULL WHERE ID = 1;
Liquibase command 'updateSql' was executed successfully.
You will find this in the output sql:
ALTER TABLE msrv_t15.table01 COMMENT = 'column03 description';
It shouldn't be there.
Expected/Desired Behavior
The generated sql shouldn't update the table description after adding a new column.
Screenshots (if appropriate)

ChangeLogOutput.txt
ChangeLog.yaml.txt

ChangeLogOutput.txt
ChangeLog.yaml.txt

ChangeLogOutput.txt
ChangeLog.yaml.txt

Thanks @nelucon for the detailed description and screenshots.  It looks like there is an extra alter table statement being added in addition to the correct column description.
This would overwrite any table description that may have been there.
We will add it to the queue of changes needed for the community.

Workaround
After addColumn you have to add setTableRemarks to put back the table remarks.

Oooh. This is a good one. Thank you!!!

This is a duplicate of #2301 and fixed by #2752.
I'll close this version to centralize tracking


"Enter" Key for Search

Add "enter" key response for searchable results.

I solved this issue on pr #54


python310Packages.aiocurrencylayer: 1.0.3 -> 1.0.4

Description of changes
https://github.com/home-assistant-ecosystem/aiocurrencylayer/releases/tag/1.0.4

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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/)
22.11 Release Notes (or backporting 22.05 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 192866 run on x86_64-linux 1

2 packages built:

python310Packages.aiocurrencylayer
python39Packages.aiocurrencylayer


Add github IDs to maintainers

Signed-off-by: Richard Scothern<EMAIL_ADDRESS>

Your itchy finger was a bit too quick for the last PR, @dmp42

LGTM


Clarify if orphan

to address #4717
Please check the accuracy of this when you get a chance, @msullivan. I'm particularly concerned if I properly understand the implications I've described in the second paragraph of my note.
If my description is accurate, I wonder if we can and should consider another way to handle beyond just documenting it as-is. I feel like the result I've described would be fairly surprising.

We definitely could implement the other behavior.
I believe that Elvis was concerned about the performance implications, though it might not be that bad: we have to do the exact same checks when we actually do delete an object as we do to check if it is an orphan...


b4: 0.8.0 -> 0.10.1

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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/)
22.11 Release Notes (or backporting 22.05 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 195263 run on x86_64-linux 1

2 packages built:

b4
patatt

Last interaction from maintainer was in #167192.
Waiting for maintainer response.
LGTM.

On Sun, Oct 09, 2022 at 01:51:20PM -0700, 0x4A6F wrote:

Last interaction from maintainer was in #167192.
Waiting for maintainer response.

I'm still here, you don't need a maintainer response. lgtm.


Add Juicebox

Name/Description
Juicebox
Type

New addition

Compatible Networks
- Mainnet
- Rinkeby

Code for review
Web Interface
Contracts

You can find contract ABIs and Addresses under ./deployments.
The only Rinkeby ABIs that should be needed are the ones that have corresponding ABIs in the Mainnet folder.

IPFS hash/App URL
https://juicebox.money
Desired date
ASAP. We are getting tons of activity from the ConstitutionDAO project.

I apologize that we have chosen not to add the project to the default apps list because it is not audited, and our users trust us when we list apps by default.
You can ask your users to add the app as a custom app in the meantime.


⚠ Samsung has degraded performance

In a527843, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2866 ms

Resolved: Samsung performance has improved in 00734dd.


[📚] firebase_messaging example does not work on web

The firebase messaging example should be changed to work on web. The first issue is that it will crash because of local notifications. This can be fixed with something like
AndroidNotificationChannel channel;
FlutterLocalNotificationsPlugin flutterLocalNotificationsPlugin;

Future<void> main() async {
  WidgetsFlutterBinding.ensureInitialized();
  await Firebase.initializeApp();

  // Set the background messaging handler early on, as a named top-level function
  FirebaseMessaging.onBackgroundMessage(_firebaseMessagingBackgroundHandler);

  if (!kIsWeb) {
    /// Create a [AndroidNotificationChannel] for heads up notifications
    channel = AndroidNotificationChannel(
      'high_importance_channel', // id
      'High Importance Notifications', // title
      'This channel is used for important notifications.', // description
      importance: Importance.high,
    );

    /// Initialize the [FlutterLocalNotificationsPlugin] package.
    flutterLocalNotificationsPlugin = FlutterLocalNotificationsPlugin();

    /// Create an Android Notification Channel.
    ///
    /// We use this channel in the `AndroidManifest.xml` file to override the
    /// default FCM channel to enable heads up notifications.
    await flutterLocalNotificationsPlugin.resolvePlatformSpecificImplementation<AndroidFlutterLocalNotificationsPlugin>()?.createNotificationChannel(channel);

    /// Update the iOS foreground notification presentation options to allow
    /// heads up notifications.
    await FirebaseMessaging.instance.setForegroundNotificationPresentationOptions(
      alert: true,
      badge: true,
      sound: true,
    );
  }

  runApp(MessagingExampleApp());
}

and in onMessage.listen
      if (notification != null && android != null && !kIsWeb) {

Then there is what was mentioned in https://github.com/FirebaseExtended/flutterfire/issues/5515
PS the example will also crash whenever a notification is successfully received because of local notification on macos.

Also, the current example implies that the flutter service worker can be registered alongside another service worker dedicated for FCM. Following the example will cause this error:
Uncaught (in promise) TypeError: Failed to register a ServiceWorker for scope ('http://localhost:4200/') with script ('http://localhost:4200/firebase-messaging-sw.js'): ServiceWorker script evaluation failed
Commenting out the line linked above (as suggested here and here) fixes the issue, but the app will lose the functionality that the generated Flutter service worker provides.

Also the documentation for background notifications says that you should put the js file load for the service worker after the main.js but Flutter 2.2 which was just released doesn't load just the script it has a whole ceremony that also loads the service worker.
The result is that I can't find anywhere to load the firebase stuff and get "Cannot read property 'messaging' of undefined.
I can confirm that it does load firebase-messaging-sw.js and const messaging = firebase.messaging() is hit in the javascript when I put the service worker right before the setTimeout in the page but then main() from web_entrypoint throws the error about messaging being undefined thus blocking the use of firebase messaging.


Solve Hotbackup blockage [BTS-1694]

This addresses https://arangodb.atlassian.net/browse/BTS-1694
This is the backport for 3.10.
Long story short:

Hotbackup blocks commits on coordinators to get a consistent state.
This should not block read-only transactions.
Furthermore, the user authentication in the IO thread occasionally
reloads the user data from the _users collection. This involves
an AQL query with a read-only transaction. We should not stop all
authentications from happening whilst this reload takes place.

Commits:

Use old user state whilst reloading users.
Do not block read-only transactions from commit when performing hotbackup.

Scope & Purpose
(Please describe the changes in this PR for reviewers, motivation, rationale - mandatory)

[*] :hankey: Bugfix
[*] :fire: Performance improvement

Checklist

[*] Tests

[*] Regression tests
[*] integration tests

[*] :book: CHANGELOG entry made
[*] Backports

[*] Backport for 3.10: this is it

Related Information
(Please reference tickets / specification / other PRs etc)

[*] Enterprise PR: none
[*] GitHub issue / Jira ticket: https://arangodb.atlassian.net/browse/BTS-1694
[*] Original PR: https://github.com/arangodb/arangodb/pull/20203

[ ] https://jenkins.arangodb.biz/job/arangodb-ANY-jslint/31248/


Replace num_cpus with std::thread::available_parallelism.

This PR removes the dependency on the num_cpus crate by using the standard library’s std::thread::available_parallelism instead.
Fixes #902.

LGTM, the fns called by both the old and the new code seem to query for some number of "logical" CPUs so that's consistent. One minor point: I did wonder whether it would make sense to extract this repeated code fragment into a helper fn:
available_parallelism().map(|x| x.get()).unwrap_or(1)

Sounds like a good idea, indeed. Done!


Cannot save 3D Scenes due to Alert: Requested file format unknown. Both Collada and Wavefront meshes.

Operating system or device - Godot version:
Godot 3.0 latest master build: 246b574
Manjaro Linux 17.0.2 64-bit
Issue description:
Upon saving a 3D Scene with emmision materials, An alert is displayed, and the scene cannot be saved.
Steps to reproduce:
Simply create a 3D scene and try to save it. Also happens when running the game.

To be clear, the scene saves if you remove emission materials? Does the command line/terminal display anything?

There are 2 (but 3) error reports. These errors appear when the game is run, but they do not popup when saving. When running the game the same Alert dialog is shown, and nothing is displayed this time. Even when there is a camera in the scene.
If this helps, my GPU is NVIDIA Geforce 660 TI 3GB VRAM.

A little update: These errors above are most likely not related to the actual issue experienced with 3D scenes. Even running a 2D project gives me the same error reports.

BTW, how are you trying to save the scene? Godot uses an internal format (.tscn or .scn) for saving scenes, and can't save to .dae (thus your error). Are you trying to overwrite an imported scene?

It looks like it was the problem. Using "Saving As..." to save it as .tscn and .scn works. I have the habit of doing Ctrl+S. So it could mean that Godot is indeed trying to overwrite the .dae/.obj scene. Yet i always expect for my imported scenes to be saved or, attempt to be saved, as a Godot .tscn/.scn/.res file.
If you think this is normal, feel free to close the issue.
My bad :/

Could do with it defaulting to .tscn/.tres then, or some notification that it cannot be saved as .dae.

This is not a bug, when you opened the scene to edit it you were warned that you would not be able to save it.

If this was not the case, please reopen the issue.. but .dae scenes can not be saved, as they are automatically reimported.


Support for nullsafety

With the null safety of dart landing in beta. Is there a timeline when this package will be migrated to nullsafety?

Any update on this?

I've followed the dependency chain down and a lot of them are not nullsafe. If anyone is interested in starting a group to start migrating the libraries I would be down, I'll be starting on my own.

This was one of the things I was most excited about bringing to Aqueduct!
Unfortunately, as you can tell, we have no ability to invest resources in this project right now. We have tried a few different avenues in looking for help from Google, but the Dart team doesn't have the resources & budget to provide, either. With each major change to Dart and our inability to keep up, the likelihood of Aqueduct catching up is getting smaller and smaller. I am hanging on the off-chance there is a significant opportunity that changes Aqueduct's fortunes - but I don't realistically see that happening outside of funding specifically for Aqueduct, and that funding is greater than small donors can provide.
You can try the migration and forking Aqueduct for null safety, but I think it's much larger & robust framework than you might be expecting, and that migration will take considerable effort. Additionally, I am not sure Dart is really making headway as a language - it seems like it is easier to improve JS and other popular languages than it is for programmers to adopt Dart.
TLDR: Aqueduct was an amazing framework that I personally sunk thousands of hours into. The cost-to-benefit ratio of continuing the development of Aqueduct isn't attractive for Stable Kernel. Other languages and ecosystems are far better supported for this type of framework.

As someone who has become js-phobic, I think dart still has a good chance at overtaking nodejs and doubly so if Google actually focuses their efforts. Dart's tooling and flutter's native-centered approach make for cleaner development than React imo. Even despite Google's weak community support, the tech is still going strong.

@Reductions it doesn't really need to be transferred, but someone needs to be on top of reviews to get the ball rolling.

@Reductions it doesn't really need to be transferred, but someone needs to be on top of reviews to get the ball rolling.

A transfer is not required but I think it is in the best interest of the community to move it to another organization.

@j4qfrost @vanlooverenkoen A transfer is required as stablekernel is privatly owned company.

@Reductions The license is still open source. It's not really required.  This is a digression though.
Some of the dependencies down the line are actually somewhat circular e.g. https://github.com/j4qfrost/pubspec_parse/blob/master/pubspec.yaml depending on a package that is dependent on an older version of it, which makes it exceedingly hard to upgrade to nullsafe. Trying to find a suitable substitute.

Yes, you can fork the project - there are 188 forks already. The license is open source.
The primary challenge is that the "supporting cast" - documentation, website, CI/CD, etc. - would need to be replaced.
The other challenge is that Dart is a moving target - non-nullability will impact all of the packages we have in the ecosystem (Postgres, Aqueduct, safe_config, Codable, open_api, and password_hash) and the changes are going to be significant. It is potentially hundreds more hours of work - and that's for someone who has already been onboarded and trained on the codebase.
There is currently no plan to support or update any of Stable Kernel's Dart packages, but there are no plans to revoke licenses or delist packages. As unfortunate as it is, we are simply unable to find a way where continued investment in the Dart ecosystem is sustainable for us. I believe we have an internal meeting late next week to evaluate our options.

Dart is a moving target.

hehehe

For lack of a better place https://github.com/stablekernel/dart-isolate-executor/pull/10

I've made PRs for all the dependencies except for password_hash which already appears to have a PR for nullsafety. Going to check in again sometime next week.

https://github.com/stablekernel/open-api-dart/pull/7 with this all aqueduct dependencies now have nullsafety PRs in review. Going to be a while until all the dev-dependencies get migrated. Crossing my fingers that there's enough inertia to keep maintenance going in the coming weeks.  The porting itself might take a week, but even after that validating that nothing broke is an entirely different issue. Once this does get migrated it'll probably be unstable for a while. Do you guys want this enough to help contribute?

https://github.com/j4qfrost/aqueduct Whelp, I got all the tests to pass. Looks like the project is being sunsetted, so all the dependencies will be maintained on my end.

@j4qfrost will you publish this to pubdev. Under a aqueduct_community?
It should be better if @joeconwaystk passes the admin rights to the repo and pubdev dependencies

@j4qfrost will you publish this to pubdev. Under a aqueduct_community?
It should be better if @joeconwaystk passes the admin rights to the repo and pubdev dependencies

I want to take the conversation with him to an email first. The sunset message alludes to some of the reasons why Stable Kernel didn't opt immediately for a community transition, and I want to write a personal thank you.

Alright perfect. Keep us posted :)


call stack location of bottom-most function are incorrect

The line number for the bottom-most function is always -1 which is wrong.

In fact all stack locations are wrong because they weren't being updated by _step after the change to using loc instead of line.


Get all parent workitems of work items in pipeline artifacts

Azure DevOps Extensions
Generate Release Notes (Node Cross Platform)
Is your feature request related to a problem? Please describe.
When I generate release notes for a pipeline that consumes pipeline artifacts, I would like to have the fields of parent work items available for consumed artifacts, not just references to them. I would like something with the same data structure as "relatedWorkItems".
This would allow me to display parents and filter them by their fields. For example, if a task work item is linked to a commit, I would like to show its user story parent or feature grandparent in the release notes like it's currently possible for the primary artifact.
With relatedWorkItems of the primary artifact this is possible:
{{#forEach relatedWorkItems}}
{{#with fields}}
{{#if (eq (get 'System.WorkItemType' this) 'Feature')}}
#{{../id}}
{{/if}}
{{/with}}
{{/forEach}}
I could not find a way to do the same for other consumed pipeline artifacts. The parameter getAllParents  does not seem to have any effect on consumed artifacts.
Describe the solution you'd like
I would like additional data added to consumedArtifacts if getAllParents is enabled.
I have tried this approach locally on my machine, I was able to add a "parentexperiment" object by calling getAllParentWorkitems as it is called to create "relatedWorkItems" for the primary artifact. See example below.
This would work for me.
Example:
globalConsumedArtifacts.push({
                                        "artifactCategory": (currentBuildArtifact as any).artifactCategory,
                                        "artifactType": (currentBuildArtifact as any).artifactType,
                                        "alias": (currentBuildArtifact as any).alias,
                                        "properties": {
                                            "projectId": (currentBuildArtifact as any).properties.projectId
                                        },
                                        "versionName": `${(lastGoodBuildArtifact as any).versionName} - ${(currentBuildArtifact as any).versionName}`,
                                        "commits":	await enrichChangesWithFileDetails(
                                            gitApi,
                                            tfvcApi,
                                            await buildApi.getChangesBetweenBuilds((currentBuildArtifact as any).properties.projectId, (lastGoodBuildArtifact as any).versionId, (currentBuildArtifact as any).versionId),
                                            gitHubPat),
                                        "workitems": await getFullWorkItemDetails(workItemTrackingApi, wi),
                                        "parentexperiment": await getAllParentWorkitems(workItemTrackingApi, artifactworkitems),
                                    });

Describe alternatives you've considered
Generating release notes of consumed artifacts separately and join markdown files, but comparing the correct pipeline IDs would be challenging.
Additional context
No response

Happy to consider PR if you have a working prototype


provide a custom error message for hosts validation at /mon/install/

Signed-off-by: Andrew Schoen<EMAIL_ADDRESS>

This will probably need the error handler for schema errors updated since it doesn't use .reason currently but rather the whole __str__ for the exception


Trying the new build server

Hey @hlidotbe - how can I contact you privately? I want to send you instructions to add your plugin to the new build server before it launches in a bit.

Hi, you can drop me a mail at<EMAIL_ADDRESS>if you will.


Add test for LinearSVC

Add test for LinearSVC
In using Go1.6, please use extern $GODEBUG=cgocheck=0

Patch looks good, the CI failure looks spurious. I'll retest it against all the versions I currently have available.

Works on Go 1.5.


"Couldn't build backend. No module named 'gpt4all' "

Expected Behavior
Please describe the behavior you are expecting.
I expected the UI to open and work after running webui.bat .
Current Behavior
Please describe the behavior you are currently experiencing.
Open downloaded webui.bat batch file and wait few seconds for the error message to pop out telling me that "Couldn't build backend. No module named 'gpt4all' ". Can't open the UI.
Steps to Reproduce
Please provide detailed steps to reproduce the issue.

Step Open downloaded webui.bat batch file and wait few seconds for the error message to pop out

Possible Solution
If you have any suggestions on how to fix the issue, please describe them here.
Context
Please provide any additional context about the issue.
Im using Python 3.7 on Windows 7 with Gpu GTX 1080.
Screenshots
If applicable, add screenshots to help explain the issue.

open the UI in web browser http://localhost:9600 then go to settings
then Models Zoo then choose backend and select model that you have installed, then apply changes then press the save button on other side (diskette icon), reboot..

 If not then check configs/local.yam file for newUI flag and set it to true. then fist go to root of the page then go to settings.


Provide just the path for resource.image

Currently resource.image returns an image.
eg:
![](/z_Compendium/deities/img/symbol-of-tempus.jpg#symbol)
While this is logical, it makes it impossible to do any formatting on the image via a custom template.
If it just returned the path: /z_Compendium/deities/img/symbol-of-tempus.jpg, formatting can be done in the template, like this:
{#if resource.image}![symbol|200]({resource.image}){/if}

good plan. ;)


IEvent is not serializable

Describe the bug
Handlers can't react to Marten.IEvent<T> messages when the endpoint is configured to be durable since Marten.IEvent<T> is not serializable.
To Reproduce

Integrate Wolverine with Marten; it doesn't matter whether you use .EventForwardingToWolverine or .PublishEventsToWolverine.
Declare a Wolverine handler that reacts to a Marten.IEvent<T>, e.g.:

public static class SomeHandler
{
    public static object Handle(IEvent<SomeEvent> martenEvent)
    {
        Console.WriteLine(martenEvent.StreamKey);
    }
}

Configure the relevant endpoint to be durable, e.g. options.Policies.UseDurableLocalQueues()
Append the Marten event, T, to an event stream.

Expected behavior
SomeHandler executes and outputs the stream key.
Actual behavior
Serialization of Marten.IEvent<T> fails with the following exception:
System.NotSupportedException: Serialization and deserialization of 'System.Type' instances is not supported. Path: $.EventType.

@anhtin I think I'm going to cheat and make this public w/ a no arg ctor in Marten, then close this w/ a test.
Didn't make the 2.7 cut, but will the next release this week


NoUnusedVariables validation rule

Pull Request Test Coverage Report for Build 481

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 80.293%

Totals

Change from base Build 458:
0.2%

Covered Lines:
2249

Relevant Lines:
2801

💛  - Coveralls


"completed"-Attribute on ud-stepper doesn't notify and doesn't work correctly

Hello!
I noticed that the property completed on a ud-step itself works as expected, but sadly that isn't the case for ud-stepper. Neither does it notify other components of a value change, nor does it work as expected.
What I would expect from this property: If all child-steps of a stepper are completed, it is true, else it's false.
Kind regards
Fabian

The issue has been fixed in release v0.3.2.


Unable to inject the custom JavaScript in .Net Core 2.2

I have written a JavaScript which I would like to inject in swagger with .Net Core 2.2. I would like to highlight that wwwroot folder wasn't created by default when I created web project with .Net Core 2.2. I then created wwwroot folder manually and added my JavaScript file inside that. When I am launching swagger, I am getting an error that file couldn't be loaded. I tried to create a folder inside wwwroot and place a file there but that too didn't work.
Here is my sample code:
SwaggerUIOptions uiConfig;
uiConfig.InjectJavascript("mycustomfile.js");

Also I would like to highlight what I am trying to achieve here. I got a drop-down in my header for a database provided i.e. Oracle/MSSQL etc. Now based on the selection, I wan to change value of another header i.e. connection string. I have written a JavaScript for this purpose but let me know if that can be achieved in some other and simplistic was as well with .Net Core along with Swashbuckle.

I did try to create swagger folder inside wwwroot and placed my JavaScript file there but still no helps.

I have written a JavaScript which I would like to inject in swagger with .Net Core 2.2. I would like to highlight that wwwroot folder wasn't created by default when I created web project with .Net Core 2.2. I then created wwwroot folder manually and added my JavaScript file inside that. When I am launching swagger, I am getting an error that file couldn't be loaded. I tried to create a folder inside wwwroot and place a file there but that too didn't work.
Here is my sample code:
SwaggerUIOptions uiConfig;
uiConfig.InjectJavascript("mycustomfile.js");

Also I would like to highlight what I am trying to achieve here. I got a drop-down in my header for a database provided i.e. Oracle/MSSQL etc. Now based on the selection, I wan to change value of another header i.e. connection string. I have written a JavaScript for this purpose but let me know if that can be achieved in some other and simplistic was as well with .Net Core along with Swashbuckle.

Obviously your path is wrong.
If it is in wwwroot, it should be written like this:
c.InjectJavascript("/swagger-lang.js");

Hi @HanJunJun ,
I tried that too but still getting an error that failed to load resource. In .Net Framework, we used to provide the assembly and resource file. Something similar is possible here?

Are you getting an exception or a 404 on the swagger page?
Have you enabled static files?
public void Configure(IApplicationBuilder app) {     app.UseStaticFiles(); }

You're using a configuration that gives you full control over the index.html file - so I'm not really sure how your jquery issue can be resolved on the SB side. But, if you have an idea, feel free to submit a PR.

You're using a configuration that gives you full control over the index.html file - so I'm not really sure how your jquery issue can be resolved on the SB side. But, if you have an idea, feel free to submit a PR.


undefined local variable or method `conn'

When using --push-results I get an error
$ spin serve -Itest --test-unit --push-results
Preloaded Rails env in 2.5115768909454346s...
Pushing test results back to push processes
/Users/ben/.rvm/gems/ruby-1.9.2-p290@reader2/gems/spin-0.4.1/bin/spin:153:in `block in fork_and_run': undefined local variable or method `conn' for main:Object (NameError)
        from /Users/ben/.rvm/gems/ruby-1.9.2-p290@reader2/gems/spin-0.4.1/bin/spin:148:in `fork'
        from /Users/ben/.rvm/gems/ruby-1.9.2-p290@reader2/gems/spin-0.4.1/bin/spin:148:in `fork_and_run'
        from /Users/ben/.rvm/gems/ruby-1.9.2-p290@reader2/gems/spin-0.4.1/bin/spin:122:in `block in serve'
        from /Users/ben/.rvm/gems/ruby-1.9.2-p290@reader2/gems/spin-0.4.1/bin/spin:98:in `loop'
        from /Users/ben/.rvm/gems/ruby-1.9.2-p290@reader2/gems/spin-0.4.1/bin/spin:98:in `serve'
        from /Users/ben/.rvm/gems/ruby-1.9.2-p290@reader2/gems/spin-0.4.1/bin/spin:245:in `<top (required)>'
        from /Users/ben/.rvm/gems/ruby-1.9.2-p290@reader2/bin/spin:19:in `load'
        from /Users/ben/.rvm/gems/ruby-1.9.2-p290@reader2/bin/spin:19:in `<main>

Fixed in v0.4.2.

Cheers


[SYSTEMML-848] Create Release Creation Process doc

Thank you @asurve for updating the release document.  I viewed new content as web page under bundle exec jekyll serve.  Please make some minor formatting corrections to the new markdown file.  For example, <br> tokens can be removed.

thanks @gweidner Updated changes.

LGTM - I will merge PR and push gh-pages.

Thanks @gweidner


JNI and HaxeObject.callX crash

Hello,
I'm trying to move on the last lime and openfl haxelib versions and I have some trouble with extensions.
I made a basic sample that's produce the crash here : https://github.com/TBaudon/HaxeObject-test
Apparently, using HaxeObject.callX from java crash the app.
i'm using :
openfl 3.3.0
lime 2.5.3
hxcpp 3.2.102
I also use java jdk 1.7 .
Thanks.

I believe this is an issue with threading, the Java code and Haxe code do not execute on the same thread, so it is important to only call Haxe from Java within a callback, such as:
Extension.callbackHandler.post (new Runnable ()
{
	@Override public void run ()
	{
		// call Haxe from here
	}
}


Crash in _deep_copy_contours when a glyph includes a non-existent component

I had this problem while building a Glyphs font with fontmake -g XXXX.glyphs -i. The test font happens to have glyphs that point to non existent components. This crash happens now that https://github.com/googlei18n/glyphsLib/issues/317 has been fixed in https://github.com/googlei18n/glyphsLib/pull/329. glyphsLib warns about the problem as such:
INFO:fontmake.font_project:Building master UFOs and designspace from Glyphs source
INFO:glyphsLib.classes:Parsing "../../../Downloads/XXXX.glyphs" file into <GSFont>
WARNING:glyphsLib.builder.builders.UFOBuilder:Anchors not propagated for inexistent component circumflexcomb_acutecomb in glyph acircumflexacute

Now that glyphsLib does not crash anymore on references to undefined components, fontmake does, here:
https://github.com/googlei18n/fontmake/blob/aa412a101e4258c4e6da21c95556607ae03df8ec/Lib/fontmake/font_project.py#L132
I'm not actually sure we want to "fix" this by not crashing, because it means the font is at least partially broken, but maybe the error message from fontmake could be a bit better. Currently for example:
INFO:fontmake.font_project:Decomposing glyphs for XXXX-Regular
Traceback (most recent call last):
  File "c:\python27\Lib\runpy.py", line 174, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "c:\python27\Lib\runpy.py", line 72, in _run_code
    exec code in run_globals
  File "C:\UsersLocal\jany.belluz\Documents\venvs\glyphs\Scripts\fontmake.exe\__main__.py", line 9, in <module>
  File "c:\userslocal\jany.belluz\documents\venvs\glyphs\lib\site-packages\fontmake\__main__.py", line 186, in main
    project.run_from_glyphs(glyphs_path, **args)
  File "c:\userslocal\jany.belluz\documents\venvs\glyphs\lib\site-packages\fontmake\font_project.py", line 369, in run_from_glyphs
    designspace_path, instance_data=instance_data, **kwargs)
  File "c:\userslocal\jany.belluz\documents\venvs\glyphs\lib\site-packages\fontmake\font_project.py", line 437, in run_from_designspace
    interpolate_layout_from=interpolate_layout_from, **kwargs)
  File "c:\userslocal\jany.belluz\documents\venvs\glyphs\lib\site-packages\fontmake\font_project.py", line 476, in run_from_ufos
    ufos, remove_overlaps, **kwargs)
  File "c:\userslocal\jany.belluz\documents\venvs\glyphs\lib\site-packages\fontmake\font_project.py", line 166, in build_otfs
    self.decompose_glyphs(ufos)
  File "c:\userslocal\jany.belluz\documents\venvs\glyphs\lib\site-packages\fontTools\misc\loggingTools.py", line 372, in wrapper
    return func(*args, **kwds)
  File "c:\userslocal\jany.belluz\documents\venvs\glyphs\lib\site-packages\fontmake\font_project.py", line 124, in decompose_glyphs
    self._deep_copy_contours(ufo, glyph, glyph, Transform())
  File "c:\userslocal\jany.belluz\documents\venvs\glyphs\lib\site-packages\fontmake\font_project.py", line 132, in _deep_copy_contours
    ufo, parent, ufo[nested.baseGlyph],
  File "build\bdist.win32\egg\defcon\objects\font.py", line 219, in __getitem__
  File "build\bdist.win32\egg\defcon\objects\layer.py", line 261, in __getitem__
  File "build\bdist.win32\egg\defcon\objects\layer.py", line 178, in loadGlyph
KeyError: 'circumflexcomb_acutecomb not in layer'

Or does it make sense to build such a font anyway? Like, skip decomposing those glyphs?

skip and warn should be fine I think. Does Glyphs.app complain?

just tried to export the file Marc sent from within Glyphs.app. I can see the missing components are simply (silently) dropped.


Sort of bug or at least unexpected behaviour

Hi,
I downloaded the ga from usegalaxy.org and I got:

I guess "label": null means that it is not a workflow output right.

The IWC profile requires defining a label for each workflow output. Since the label value is null is considered as not really defined.
The workflow is valid itself, if you go back to the basic validation profile it will not show this as an error. This only means that galaxy doesn't enforce IWC best practices workflows right now :)

But this is not a workflow output... This is a workflow input parameter...

And it seems that sometimes it is defined and sometimes not... I don't know why... Maybe if the type is data_input or parameter_input or data_collection_input the workflow_output should not be checked...

Or this needs to be directly in galaxy code... I don't know.

But this is not a workflow output... This is a workflow input parameter...

I'm a bit confused and out of context here, also I'm no expert in workflows myself so please excuse me :sweat_smile:
In your screenshot, at line 66 looks like this output is defined in the workflow_outputs section, that's what makes the extension consider this a workflow output without a label.
Is your concern that Galaxy is not exporting your workflow as you expect? or that the rule of "all workflow outputs must define a label" is not right?

Sorry, my explainations are unclear.

I will try to better explain what is happening:
When you create a input box in the workflow editor (data_input or parameter_input or data_collection_input), then sometimes when you download the ga file you have:

            "workflow_outputs": [
                {
                    "label": null,
                    "output_name": "output",
                    "uuid": "d4a5d221-29f4-469d-ad39-63c56bab4e8c"
                }
            ]

And sometimes you have:
            "workflow_outputs": []

(See https://github.com/galaxyproject/iwc/pull/130/commits/d7fa5bc623fe6a90a1da0dfbb3c187b7038934df).
Both works and give the same results.

What is my concern:

Galaxy should never write non empty workflow_outputs for inputs.
Because it does, maybe the linter could adapt and do not consider as error the fact that an input has a non labelled workflow_outputs.

Is it clearer?

Thank you so much for the clarification!
So If now I understand it correctly:

Galaxy should never write non empty workflow_outputs for inputs.

Yep, this sounds like a bug in Galaxy, might be worth reporting it there if it is not already.

Because it does, maybe the linter could adapt and do not consider as error the fact that an input has a non labelled workflow_outputs.

I'm not so sure about this part... I think it is better to fix the underlying issue in Galaxy than kind of hide it on the linter side. What do you think @mvdbeek

AFAIK, a label: null in the workflow_outputs of an input param does not get reported as a Best Practice violation.
@mvdbeek has repeatedly argued that inputs are outputs though so having the workflow_outputs section present seems justified.
otoh, then the labels within should probably not be null by default?

So @davelopez, imho, it's a feature not a bug if the extension complains about the null value here.

@wm75 to be sure I understand, you would say:

having a workflow_outputs section in input is an expected behaviour from galaxy. However the choice of label: null is maybe not ideal.

Is the fact that this section disappear when you upload a workflow is a bug from galaxy?
Should I close the issue I just opened?

The galaxy workflow iwc linter should not mark workflow_outputs with label: null as errors.

Is it correct?


how can this publish topics represent the bounding box?

Hi
Thanks for your work on this package,
how can the bounding box be published as a topic after the segmentation?
after catkin build, it seems that they is no rosnode related to  the following two file:
``

save_clusters_node.cpp
2.show_objects_node.cpp
``
Best
Welson

Yes, this will need to be implemented as I did not focus on publishing anything through ros. I only use ros as a method to deliver me data.

Hi, @niosus
Do you have any ideas on this issue?  how can we get the bounding box information and then publish them?
Best
Welson

You can look where the visualizer gets the clusters (e.g. like here and write a similar component that would receive the clusters and create bounding boxes and send them through ros.

Hi, @niosus
I tried this and got the value, but once the main function run to the following code:
auto exit_code = application.exec();
the code blocked in  application.exec() and I can not jump out of application.exec() to publish the topic.
Do you have any advice on this ?
Best
Welson

Please make sure you understand everything that is going on in the file, especially this line
You have to be comfortable with understanding where and why a thread is spawned. I will not be able to explain it here fully, but the main idea is that there is a ros thread and a qt thread. The main thread of the application is owned by qt, while ros gets a new thread to own. You must implement a component that connects to other by using AddClient(...) functions. This way, the whole pipeline will be triggered by ros receiving data. This data will follow the path through all the components and must end up with your new components that will publish the data through ros again.

Thanks for your kind advice, it is clear to me, I will implement this!
Best
Welson

Hi!
Did you ever implement this @weisongwen  ? I would be very interested in using the published bounding boxes myself!

Hi, @didriksg
I have not implement this yet, as i am busy with some other things.
I may conduct this next month,
Best
Welson

Hi @weisongwen
That's alright! If I get the time, I might just look into implementing it myself.
Thank you for your quick answer! :)

@weisongwen Hello, I have the same confusion with you. Have you found a good way to publish the bounding boxes or clusters as a ROS topic? Thanks for your help :)

hi, @PanZhichen
This function have not been implement yet,
Best
Welson

Hi, @PanZhichen and @weisongwen , Did you publish this bounding boxes ? I am trying to publish but i cannot find the way. Thanks for help !
Best
Elif

Hi @PanZhichen  and @weisongwen ,
Did you publish the bounding box? I cannot find the way. Could you please share your method? Thanks for help!
Best,
Elif


[NIFI-1762] Remove bower and npm from convenience binaries

Thanks @scottyaslan ... Just a heads up I fixed a small typo and updated the IDs in your PR prior to merging to master. Thanks!


missing quotes in curl command

if I add brew 'https://raw.githubusercontent.com/MoritzFago/homebrew-core/airport-bssid/Formula/airport-bssid.rb' to my Brewfile it's unable to download the file due a problem with the curl command:
curl: (22) The requested URL returned error: 404 Not Found
Error: Failure while executing: /usr/bin/curl --fail --progress-bar --remote-time --location --user-agent Homebrew/1.0.0\ (Macintosh;\ Intel\ macOS\ 10.12)\ curl/7.49.1 https://raw.githubusercontent.com/moritzfago/homebrew-core/airport-bssid/formula/airport-bssid.rb -o /Users/moritz/Library/Caches/Homebrew/Formula/airport-bssid.rb

It's because that URL 404s. We don't support using URLS like this in a Brewfile; create a tap and use that instead.


React  npm start failing on out of the box code.

Describe the bug
(Write your answer here.)
create-react-app app name builds the app. However. However, npm start fails out of the box. By this, I mean I haven't touched any of the codebase. I've been using Vs code. I have tried uninstalling and reinstalling node_modules, creating completely new react apps, and using different browsers. I tried updating my nodejs and my npm, and have tried npm install.
I always get the following error. Note, There aren't 43 columns in line one of index.Js, so i'm not even sure what its referring to.
Failed to compile.
./src/index.js 1:43
Module parse failed: Unexpected token (1:43)
File was processed with these loaders:

<EMAIL_ADDRESS>./node_modules/babel-loader/lib/index.js
You may need an additional loader to handle the result of these loaders.

$RefreshRuntime$ = require('C:/Users/Ethan's PC/Desktop/react Test/try-this/node_modules/react-refresh/runtime.js');
| $RefreshSetup$(module.id);
|

Did you try recovering your dependencies?
yes, multiple times.

(Write your answer here.)
npm 7.5.6 I have followed all of the steps.
Which terms did you search for in User Guide?

(Write your answer here if relevant.)
pasted the error. Also tried "react fails to compile on innitial load" and "react module parse error on initial load"
Environment

Environment Info:
current version of create-react-app: 4.0.3
running from C:\Users\Ethan's PC\AppData\Roaming\npm\node_modules\create-react-app
System:
OS: Windows 10 10.0.19042
CPU: (4) x64 Intel(R) Core(TM) i5-7200U CPU @ 2.50GHz
Binaries:
Node: 14.15.5 - C:\Program Files\nodejs\node.EXE
Yarn: Not Found
npm: 7.5.6 - ~\AppData\Roaming\npm\npm.CMD
Browsers:
Chrome: 85.0.4183.102
Edge: Spartan (44.19041.423.0), Chromium (88.0.705.74)
Internet Explorer: 11.0.19041.1
npmPackages:
react: ^17.0.1 => 17.0.1
react-dom: ^17.0.1 => 17.0.1
react-scripts: 4.0.3 => 4.0.3
npmGlobalPackages:
create-react-app: Not Found
Steps to reproduce

(Write your steps here:)

npx create-react-app appName
npm start
and it fails! : )

Expected behavior
 It should have opened React in a browser.
Actual behavior

A browser DOES open, but it is just a blank page with a copy of the error.

Reproducible demo

I would but there is nothing to send. It's a basic level issue. As a student in school for programming, I'm having to build all of my
React projects in codeSandbox and it's been challenging. Please let me know if you have any insight!

Experienced the same issue; tried reinstalling npm, npx and removed any global installations of create-react-app. Nothing worked.
Solution 1:
I suggest you to try running:
npx create-react-app@latest my-app
Solution 2:
If the above command doesn't resolve your issue, do the following steps:

Navigate to your current project directory
Open package.json
Change the version of react-scripts from 4.0.3 to 4.0.1. Save the file.
Now, run npm install in your current project directory.

If these workarounds didn't fix this, you might want to check the workarounds suggested in #10601.

Experiencing the exact same issue.
System:
OS: MacOS 10.15.7
CPU: 2.8 GHz Quad-Core Intel Core i7
Binaries:
Node: 14.15.5
npm: 6.14.11
Browsers:
Chrome: Version 88.0.4324.192 (Official Build) (x86_64)
npmPackages:
"react": "^17.0.1",
"react-dom": "^17.0.1",
"react-scripts": "4.0.3",
global create-react-app has been uninstalled.
Just trying to get the initial create-react-app to start - no modifications made. No matter what I do I get this error every time:
./src/index.js 1:59
Module parse failed: Unexpected token (1:59)
File was processed with these loaders:
 *<EMAIL_ADDRESS> * ./node_modules/babel-loader/lib/index.js
You may need an additional loader to handle the result of these loaders.
> $RefreshRuntime$ = require('/Users/<username>/Documents/Programming/react-tictactoe-tutorial/node_modules/react-refresh/runtime.js');
| $RefreshSetup$(module.id);
|

Experienced the same issue; tried reinstalling npm, npx and removed any global installations of create-react-app. Nothing worked.
Solution 1:
I suggest you to try running:
npx create-react-app@latest my-app
Solution 2:
If the above command doesn't resolve your issue, do the following steps:

Navigate to your current project directory
Open package.json
Change the version of react-scripts from 4.0.3 to 4.0.1. Save the file.
Now, run npm install in your current project directory.

If these workarounds didn't fix this, you might want to check the workarounds suggested in #10601.

Reverting react-scripts to version 4.0.1 and rerunning npm-install  (Solution 2 above) resolved the issue for me.

Going back from 4.03 to  4.01 in scripts worked. Thank you @iastanley


test_stats: stats4 failed

In debian Sid environment, the stats4 in test_stats in tests/test_osmnx.py fails with the message:
E           networkx.exception.NetworkXNotImplemented: not implemented for multigraph type

This test failure is resulting from networkx no longer implementing the betweenness_centrality algorithm on Multi(Di)Graph types as of v2.4: https://github.com/networkx/networkx/pull/3454
I need to change the OSMnx stats module to accommodate that.

See also: https://github.com/networkx/networkx/issues/3432


Failed to execute: [adb], [start-server]

Hello, thanks for this amazing project.
When I use scrcpy as usual, but suddenly face an error.
Below is an error that appears when executing scrcpy.
scrcpy 1.23 https://github.com/Genymobile/scrcpy
exec: Operation not permitted
2022-04-05 00:13:49.918 scrcpy[98301:1385587] ERROR: Failed to execute: [adb], [start-server]
2022-04-05 00:13:49.918 scrcpy[98301:1385587] ERROR: Could not execute "adb start-server"
2022-04-05 00:13:49.918 scrcpy[98301:1385587] ERROR: Could not start adb daemon
2022-04-05 00:13:50.318 scrcpy[98301:1385578] ERROR: Server connection failed
And provide additional information:
adb devices -l
List of devices attached
R5CR41CFS3B            device usb:338690048X product:o1qzcx model:SM_G9910 device:o1q transport_id:1

Which platform? How did you install it?

macOS version 12.2.1
I use Homebrew install it.

I have same issue ,
macOS version 12.3.1
brew install it

What if you execute manually adb start-server?

What if you execute manually adb start-server?

Its OK,  all adb command can be execute mannually

So it looks like a system restriction.
Maybe related to:

https://appletoolbox.com/seeing-error-operation-not-permitted-in-macos-mojave/
https://osxdaily.com/2018/10/09/fix-operation-not-permitted-terminal-error-macos/

?

In my case , just config WRONG ADB  variable in  PATH  'xxxx/platform-tools/' .
I change to 'xxxx/platform-tools/adb' , its work

which file that you change the WRONG ADB path?

In my case , just config WRONG ADB variable in PATH 'xxxx/platform-tools/' .
I change to 'xxxx/platform-tools/adb' , its work

which file that you change the WRONG ADB path?

@iknow4 It seems that @Chaoyang0201 issue was not the same as yours, it assigned a wrong value to the ADB environment variable. You can check with echo $ADB.

Failed to execute: [adb], [start-server]

In you case, it is probably empty (adb without explicit path is the default value).
What is the result of adb start-server for you?

@iknow4 It seems that @Chaoyang0201 issue was not the same as yours, it assigned a wrong value to the ADB environment variable. You can check with echo $ADB.

Failed to execute: [adb], [start-server]

In you case, it is probably empty (adb without explicit path is the default value).
What is the result of adb start-server for you?

From my side:
➜  ~ adb $ADB
Android Debug Bridge version 1.0.41
Version 29.0.5-5949299
Installed as /Users/user/Library/Android/sdk/platform-tools//adb

command: adb start-server   output nothing.

From my side:
➜ ~ adb $ADB

echo $ADB?

Sorry , when i execute  "echo $ADB"   also output nothing.

OK, as expected, so go back to https://github.com/Genymobile/scrcpy/issues/3167#issuecomment-1090031152.

In my case , just config WRONG ADB variable in PATH 'xxxx/platform-tools/' .
I change to 'xxxx/platform-tools/adb' , its work

which file that you change the WRONG ADB path?

I use zsh ，   ~/.zshrc

I also use zsh, change the config, but unfortunately, it's not work.

Maybe you can try add "adb" after platform-tools/

yes, i did it like you say, but it does not work.

Wow, it's work, thank you for you help!!!

@iknow4 I'm surprised that adb (without an explicit path) works alone (which means that it's found in your path), but not via scrcpy.
What is the result of which adb?

here you go.

Does it work:
ADB=/usr/local/bin/adb scrcpy

?

have try, and seem it not work.

If /usr/local/bin/adb devices works, then the command above should work too. It not, it's probably a restriction of your system.

In my case , just config WRONG ADB variable in PATH 'xxxx/platform-tools/' . I change to 'xxxx/platform-tools/adb' , its work

yes, it works.

In my case , just config WRONG ADB variable in PATH 'xxxx/platform-tools/' . I change to 'xxxx/platform-tools/adb' , its work

I have the same issue in Windows 10,and I fixed the ADB PATH variable by following this,now it works,thanks~

On Windows, unless you want a very specific adb, you should not set the ADB environment variable at all.

How to solve this problem on windows?
C:\scrcpy-win64-v2.4>scrcpy
scrcpy 2.4 https://github.com/Genymobile/scrcpy
ERROR: CreateProcessW() error 5
ERROR: Failed to execute: [C:\ADB], [start-server]
ERROR: Could not execute "adb start-server"
ERROR: Could not start adb server
ERROR: Server connection failed

ERROR: Failed to execute: [C:\ADB], [start-server]

Remove your ADB environment variable. You don't need to set it, and you set it to a wrong value (if t is set, it must point to the adb executable, not to the directory).


Add-ModulesToBlobStorage WARNING: Access to the path is denied

Environment
PS C:\Support> $PSVersionTable
Name                           Value

PSVersion                      5.1.19041.5007
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.19041.5007
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
The error happens when I try to run "Prepare Dependencies" pipeline but can be reproduced easily.
#Preparing all variables
$m365Module = Get-Module -Name Microsoft365DSC -ListAvailable | Sort-Object -Property Version -Descending | Select-Object -First 1
$destination = Join-Path -Path $env:TEMP -ChildPath 'M365DSCModules'
$savePath = Join-Path -Path $destination -ChildPath $m365Module.Version.ToString()
New-Item -Path $savePath -ItemType 'Directory'

Save-Module -Name "Microsoft365DSC" -RequiredVersion '1.24.1016.1' -Path **$savePath**
#At this step I receive an error

PS C:\Support> Save-Module -Name "Microsoft365DSC" -RequiredVersion '1.24.1016.1' -Path $savePath                      WARNING: Access to the path                                                                                'C:\Users\USER~1\AppData\Local\Temp\M365DSCModules\1.24.1016.1\Microsoft365DSC\1.24.1016.1' is denied.               PackageManagement\Save-Package : Unable to save the module 'Microsoft365DSC'.                                          At C:\Program Files\WindowsPowerShell\Modules\PowerShellGet\2.2.5\PSModule.psm1:11794 char:21                          +             $null = PackageManagement\Save-Package @PSBoundParameters
+                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (Microsoft.Power...ets.SavePackage:SavePackage) [Save-Package], Except
   ion
    + FullyQualifiedErrorId : ProviderFailToDownloadFile,Microsoft.PowerShell.PackageManagement.Cmdlets.SavePackage

If I use Save-Module -Name "Microsoft365DSC" -RequiredVersion '1.24.1016.1' -Path $destination
Command finish without error

Found the same error while running the prepare pipeline (prepare.yaml) from a private windows agent (which is a member of the administrators group).
[INFO] Add-ModulesToBlobStorage: Saving module Microsoft365DSC (v1.24.1106.3)
WARNING: Access to the path
'C:\Users\MYADOA~1\AppData\Local\Temp\M365DSCModules\1.24.1106.3\Microsoft365DSC\1.24.1106.3' is denied.
##[error]Unable to save the module 'Microsoft365DSC'.
##[error]PowerShell exited with code '1'.
Added TLS 1.2 in session.

In the current version of the whitepaper we are storing all prerequisites on an Azure Blob Storage and during deployment downloading that package instead of install all modules from the PSGallery. That pre-staging method is much faster (~2 minutes) than installing from the PSGallery (~15 minutes).
This method however requires creating and configuring an Azure Blob Storage.
Recently we discovered a new and even faster method to download all prerequites, which we have already implemented. I am not working on updating the whitepaper and code on the repositories. Expect these updates to be released before the end of 2024.
So I won't be investigating this issue any further. I rather spent that time in updating the documentation.

I have the same problem, but the module is saved correctly in the folder.
Error:
Saved Module:
To make the prepare.yaml pipeline work, you can modify it, by editing errorActionPreference to SilentlyContinue and FailOnStandardError to false, to just continue and do not fail on error.
The problem is, that the M365DSCTools Module will be installed new everytime the pipeline runs. So changes in the .psm1 File will do not help.
Here you get the error in the pipeline:
But the pipeline continues, after the changes.
When the pipeline is finished and you set the variables correct, you will get the zip File in the Azure Blob Storage:

@ykuijs  Thanks for that!
For sure it will be better to invest time in new development.


[occm] Set node selector and tolerations by default in OCCM helm chart

What this PR does / why we need it:
This defines suitable default values for nodeSelector and tolerations to ensure a nicer out-of-the-box experience for users.
Without this change all users would have to maintain these in their respective values file.
See:

https://kubernetes.io/docs/reference/labels-annotations-taints/

Special notes for reviewers:
Please note that this (somewhat) matches the default settings used in cloud-provider-aws, with the notable difference of me having removed the toleration for the following deprecated taint:
tolerations:
- key: node-role.kubernetes.io/master
  effect: NoSchedule

Release note:
NONE

/ok-to-test

The lint error has been addressed in https://github.com/kubernetes/cloud-provider-openstack/pull/2347
I'll rebase this once that PR has been merged.

I'll rebase this once that PR has been merged.

I've rebased this now and the linter is happy.

What are your thoughts on adding node-role.kubernetes.io/master:NoSchedule? The taint is currently deprecated, but we could add it for a better default experience on clusters that used an older version that still sets it.
Documentation states:

Taint that kubeadm previously applied on control plane nodes to allow only critical workloads to schedule on them. Replaced by the node-role.kubernetes.io/control-plane taint. kubeadm no longer sets or uses this deprecated taint.

What are your thoughts on adding node-role.kubernetes.io/master:NoSchedule? The taint is currently deprecated, but we could add it for a better default experience on clusters that used an older version that still sets it.
Documentation states:

Taint that kubeadm previously applied on control plane nodes to allow only critical workloads to schedule on them. Replaced by the node-role.kubernetes.io/control-plane taint. kubeadm no longer sets or uses this deprecated taint.

What are your thoughts on adding node-role.kubernetes.io/master:NoSchedule? The taint is currently deprecated, but we could add it for a better default experience on clusters that used an older version that still sets it.
Documentation states:

Taint that kubeadm previously applied on control plane nodes to allow only critical workloads to schedule on them. Replaced by the node-role.kubernetes.io/control-plane taint. kubeadm no longer sets or uses this deprecated taint.

To me an additional toleration doesn't hurt.

To me an additional toleration doesn't hurt.

That was my thinking too. I'll add it back in, and it can be removed eventually.
/hold

Done.
/hold cancel

/lgtm


timeout

$ node ngoldman-himawari-bg-example.js
Downloading satellite images [==                  ] 12% 797.2sstream.js:74
      throw er; // Unhandled stream error in pipe.
      ^

Error: ETIMEDOUT
    at null._onTimeout (/Users/Kiven/node_modules/himawari-bg/node_modules/himawari/node_modules/request/request.js:762:15)
    at Timer.listOnTimeout (timers.js:92:15)

This appears to be an unhandled timeout error in himawari.js. I'll report this issue in that repo and keep this open for reference until it's resolved.

This appears to be resolved in the latest release of himawari.js (see https://github.com/jakiestfu/himawari.js/issues/5). If you reinstall himawari-bg it should us the latest version of himawari.js. I'll bump the dependency to ensure this doesn't happen in future releases. Please let me know if you have any further issues.


odoo: 15.0.20230317->15.020230816 fix broken fetcher

Description of changes
Upstream seems to publish and retain .zip more consistently than .tar.gz, which caused the previous expression to break. This modifies the odoo package and updater script to use the zip sources.
Things done
Switch from fetchurl to fetchzip.  Change update switch to use nix-prefetch fetchzip.

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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/)
23.11 Release Notes (or backporting 23.05 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.

build succeeds on my machine, but this is still draft since I haven't run the tests yet.
Fixes #248610

Tests pass, ready for review @mkg20001

I'm not sure why @mkg20001 had to force-push over my history since these commits pretty much identical to mine except the updater now points to v16 of the package without any of the associated attributes being updated to reflect version 15...

there were other changes, like odoo15 getting introduced, so i had to account for that. the changes in 15 and 16 are the same.


Duckduckgo,Brave,Startpage都失效

这三个搜索引擎页面可能改变了.
pipenv install Search-Engines-Scraper-Tasos
duckduckgo报错如下:

from search_engines import Google,Bing,Duckduckgo,Brave,Startpage

>>> proxy="http://<IP_ADDRESS>:1080"
>>> engine=Duckduckgo(proxy)
>>> results=engine.search("test")
Searching Duckduckgo
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
  File "/home/user/.local/share/virtualenvs/scraper/lib/python3.11/site-packages/search_engines/engine.py", line 162, in search
    response = self._get_page(request['url'], request['data'])
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/user/.local/share/virtualenvs/scraper/lib/python3.11/site-packages/search_engines/engines/duckduckgo.py", line 44, in _get_page
    response = self._http_client.get(page)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/user/.local/share/virtualenvs/scraper/lib/python3.11/site-packages/search_engines/http_client.py", line 21, in get
    page = self._quote(page)
           ^^^^^^^^^^^^^^^^^
  File "/home/user/.local/share/virtualenvs/scraper/lib/python3.11/site-packages/search_engines/http_client.py", line 41, in _quote
    if utl.decode_bytes(utl.unquote_url(url)) == utl.decode_bytes(url):
                        ^^^^^^^^^^^^^^^^^^^^
  File "/home/user/.local/share/virtualenvs/scraper/lib/python3.11/site-packages/search_engines/utils.py", line 15, in unquote_url
    return decode_bytes(requests.utils.unquote(url))
                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/urllib/parse.py", line 657, in unquote
    if '%' not in string:
       ^^^^^^^^^^^^^^^^^
TypeError: argument of type 'NoneType' is not iterable

brave报错::
results=engine.search("test")
Searching Brave
ERROR Failed to decode response content.

Startpage无论搜索什么都返回空结果:
>>> engine3 = Startpage(proxy)
>>> results3 = engine3.search("test")
Searching Startpage
>>> links3 = results3.links()
>>> print(links3)
[]

Thanks for informing me about those issues. Actually about half of the engines don't work, and I'll have to update them when I have some time


[Question] Why not just wildcard the domains?

Why not just wildcard-block *.adobe.io, *.prod.cloud.adobe.io and *.adobestats.io? Is there functionality being served over subdomains as well, or is there a lack of wildcard functionality on the blocking side of that we need to be aware of?

I wonder that, too.

I feel that answer falls short of what's trying to be figured out here, especially with the postscript. If I can wildcard on PiHole, then what's the rationale behind that if you don't recommend it anyway, even after you've said that if I can then "this repository is too simple for you"? Additionally, why is it that adobestats.io could be wildcarded (an assumption), but not adobe.io?
What I'm trying to get at here is, are some of these domains purely telemetry? Or, are some of them providing core services that we need to be wary of?

It was a simplistic reply for a simple repository. As of now, only *.adobe.io domains are not recommended to be wildcarded because there's more services running on them. The thing is, we don't know whether Adobe feels like migrating them to *.adobestats.io ever, so I don't encourage wildcarding in this repository. Again, this is a historical list of subdomains. If you have the ability of using wildcards, do not use this repository and do the wildcard by yourself
Issue locked


JavaScript: Replace Custom* with *::Range.

The old names are kept as deprecated aliases.

Comments addressed.


No Output

I tried to generate a map but it shows a broken image with 0.

Seems to be resolved now, I think after the traffic spike its again running on just one instance. Closing.