EOPNOTSUPP using raspberry pi I2C interface

I am playing around with an PN532 NFC HAT on a RaspberryPi 3B using I2C. I got everything to compile, but always received EOPNOTSUPP when trying to do anything with the device. The linux example code worked fine.
After some debugging I narrowed the problem down to this line: https://github.com/WMT-GmbH/pn532/blob/25a0e4dc7f21dd59786a96436b5becdc95568223/src/i2c.rs#L64
For some reason, the raspberry i2c (or the implementation from linux-embedded-hal?) does not like the two Read Commands. I changed the code to this:
    fn read(&mut self, buf: &mut [u8]) -> Result<(), Self::Error> {
        let mut kbuf = vec![0; buf.len() + 1];
        let res = self.i2c.exec(
            I2C_ADDRESS,
            &mut [Operation::Read(&mut kbuf)],
        );
        buf.copy_from_slice(&kbuf[1..]);
        res
    }

And with this, everything works wonderfully. The implementation is somewhat ugly because of the temporary buffer and the copying over, but I did not find another way. That said, I am not very experienced with I2C, so maybe I am doing it wrong?
I am happy to provide a merge request if the above solution is acceptable to you.

Hi, thanks for writing and taking the time to analyze this.
From looking at the linux embedded hal crate this could be caused by https://github.com/rust-embedded/linux-embedded-hal/issues/82
Unfortunately we can't use your fix, since vec! won't be available on no_std targets.
Maybe we could look into resolving the linux driver issue instead

While I in principal agree with you that the fix in the linked repo would be right place to fix this. However, this is above my technical abilities and I still need a working solution for my project. So my suggestion is to do a workaround for a moment in this repo. I see two paths:

Extend the static buffer by one byte and read into this, then drop the first byte when doing anything with the data. This is a bit cumbersome in the code right now, but I think could be made to work.
Use conditional compilation (i.e. if no-std is on, use the current code, otherwise use my suggested implementation). I think this is the easiest workaround for now and should not cause further issues, because linux-embedded kinda implies that std is available.

What is your preferred solution? I can try to provide a patch.

Looking further into it there seems to be a fix ready: https://github.com/rust-embedded/rust-i2cdev/pull/85
Maybe you could ask the devs of rust-i2cdev and linux-embedded-hal to publish a new release.
That being said, if you'd like I would accept a PR with your fix gated behind a linux-i2c-compat feature. (that feature would also enable the std feature)
Note that we'd probably remove that fix in a future release, when the upstream crates published new releases

Using linux-embedded-hal = "0.4.0" will pull in i2cdev = "0.6.1" which contains the fix you need.
I'll close this issue but if you encounter further problems, please reach out


Ensure trailing newline in IDL files

In https://github.com/web-platform-tests/wpt/pull/9837 one can see that there's a missing newline at the end of the file, and this seems to be the case for all of https://github.com/tidoust/reffy-reports/tree/master/whatwg/idl.
Including that missing newline will:

stop git from complaining on command line and GitHub on web UI
look right if you cat a file on the command line (which I do a lot)

@foolip Reffy now adds a trailing newline and all IDL files in the report have been updated accordingly. Closing the issue as a result. Just reopen if I missed something.


"View SLOs" fails to load after updating a particular SLO

Description
SLO view fails to load (both grid and table). Combine SLOs is able to load all SLOs except one. The SLO in question (AWD Online) was modified to add additional transactions and this appears to have caused the issue for multiple (probably all) users in the account.
I believe this SLO failing to load is causing a rendering failure for all SLOs on the page, but I don't think we have access to remove it from the NerdStore.
Steps to Reproduce
Navigate to Table view. Table headers will load, but with no content.
Expected Behaviour
Combined/Grid/List view to load properly and display SLOs.
Relevant Logs / Console output

one.newrelic.com-1599744613684.log
Your Environment

NR1 CLI version used: @datanerd/nr1/1.22.2 darwin-x64 node-v10.16.3
Browser name and version: Chrome 85.0.4183.83
Operating System and version: macOS 10.15.6

Additional context

I opened request #426123 with support as well, to see if they can copy the definition of the offending SLO and remove it.

Hi - I'm new to SLO/R but am experiencing near-identical issues.  See image below:

This is after defining new SLO's for any type except "Errors".  I'm new to this but have double and triple checked that I've followed the correct setup instructions.  I'd be happy to provide more info if it would help, and/or open an additional Github Issue.  If you could let me know what the best way to proceed is, I'd appreciate it!

@forestb What does your table view look like? What errors are appearing in your console?

@forestb What does your table view look like?

The table view is empty.

What errors are appearing in your console?

Is this helpful?

@andrewseling , regarding:

I opened request #426123 with support as well, to see if they can copy the definition of the offending SLO and remove it.

Were you able to get someone to remove the definition?  I've tried removing the SLO/R App "altogether" but I suspect the data is persisting in New Relic - probably within Insights somewhere.

There is a NerdGraph API that should allow access to remove the definition, but we weren't able to get it to work. The case is still open.
Have you checked the webhook setup for your alert-based SLOs. I haven't used them, but the lack of anything on the table page makes me think there is an issue when some SLOs fail to load, regardless of cause, that prevents the component from loading in properly.

I haven't used them, but...

This is helpful; if you're not using them, then I should stop hijacking your issue and reach out to support.

Have you checked the webhook setup for your alert-based SLOs. I haven't used them, but the lack of anything on the table page makes me think there is an issue when some SLOs fail to load, regardless of cause, that prevents the component from loading in properly.

I've tripled checked the configuration.  What's interesting is that when I press, "Send a test notification", my SLO configuration picks up on that payload and I am able to properly display an SLO based on that.  (The data is meaningless, of course...)

So there must be something about the payload from my "actual alert" that results in something which breaks the displaying/reporting of an SLO (e.g. my dashboard is blank with javascript errors in my console)...

Ugh.  Update: I've figured out the cause of my problem is that my policy name had an apostrophe in it, and that appears to completely breaks the SLO/R plugin.  (My policy was named Forest's test policy.)
I still have the same problem where now I have orphaned SLO/R definitions that I can't access, so I'll poke around the API a bit and see if I can figure something out.  I'll keep an eye out here in case you figure that bit out, too.

Keagan Peet from NR removed the offending SLO definition from our Nerdstore and everything is working normally again. I'll add "reproduce bug on my local development copy" to my TODO list so we can understand what about that particular SLO definition causes issues - my guess is length.


Add ‘Overall Equipment Effectiveness’ to menu

 CPG Next: Adjusting Article schedules

 CPG Next: Adjusting Article schedules


Using the BPF Task iterator to recover all user stacks for a process

Is your feature request related to a problem? Please describe.
I would like to use the BPF task iterator to retrieve the user stack for a task (and all tasks/threads for a PID). This feature would use the iter:task iterator along with the bpf_get_task_stack helper. Like so: https://github.com/torvalds/linux/blob/master/tools/testing/selftests/bpf/progs/bpf_iter_task_stack.c
Additional info here : https://developers.facebook.com/blog/post/2022/03/31/bpf-iterator-retrieving-kernel-data-with-flexibility-and-efficiency/
Describe the solution you'd like
bpftrace -e 'iter:task /pid == 123123/ { printf("%s\n", ustack()) }'
Describe alternative solutions or features you've considered
There is'nt an alternative feature available.

Can you elaborate a bit more? Maybe add some motivation example?

Can you elaborate a bit more? Maybe add some motivation example?

Sorry, i pasted all my text in the comment sections. Have updated the request

While it would be great to re-use ustack and kstack, I'm wondering if that's going to prove difficult as we have to use a different method for getting task stacks e.g. bpf_get_task_stack which doesn't return an id but rather just populates a buffer.

Just to note: bpf_get_task_stack does not allow retrieving user stacks for any task that isn't the current one. There is a way to get the user stacks from non current tasks but it requires the bpf prog to be sleepable and it might not even be accurate if the task is not interrupted (e.g. RUNNING).


Add recursive delete support

Could you please add recursive deletes? My children objects are being orphaned :(

Recursive deletes should work. Would have to see some code to comment further.

Closing for now. Feel free to re-open if you still see this issue and can provide example code.


Warning in console: Upgrade Loadable

@nodeEnthu Thanks for raising the issue, we have solved this issue with latest implementation of @pawjs/pawjs.
Please check the outdated packages via npm outdated and upgrade accordingly or you can simply upgrade to npm i @pawjs/pawjs@next


Add support to view formatted traceback in resource log

The web-console shows the kwargs, part of the resource log entry as a dictionary. This way the traceback of a deployment failure becomes hard to read. This issue proposes to add support to view the traceback in a formatted way, i.e. with the interpretation of all special characters.

Hey team! Please add your planning poker estimate with ZenHub @andraskvr @Stijnkool


unmap rbd volume if nodestage fails

Describe what this PR does
if the mapping of rbd device is passed and the mounting device to staging path fails, we need to do the unmapping the device which may leave up stale mapping if unstage is called
Signed-off-by: Madhu Rajanna<EMAIL_ADDRESS>Is there anything that requires special attention

Thanks @Madhu-1 , this should solve some issues with "reference counting" ..etc on mapping and unmapping.

@ShyamsundarR Can you please review this in priority?

added DNM adding more checks

updated PR


WOFF2 Support

It looks like Fontello now supports WOFF2, so running fontello convert results in a .scss file that looks something like:
@font-face {
  font-family: 'fontname';
  src: font-url('fontname.eot?36722241');
  src: font-url('fontname.eot?36722241#iefix') format('embedded-opentype'),
       font-url('fontname.woff2?36722241') format('woff2'),
       font-url('fontname.woff?36722241') format('woff'),
       font-url('fontname.ttf?36722241') format('truetype'),
       font-url('fontname.svg?36722241#fontname') format('svg');
  font-weight: normal;
  font-style: normal;
}

The issue is the actual .woff2 font file isn't downloading with it. I've got a (very simple) fix here, but I'm not sure how to provide test coverage for this (or if it even needs it).
If somebody can walk me through that I'd be happy to submit a PR

thanks for reporting this.  I would say definitely make a PR for this and we will see if there are any failing tests or if additional tests should be written.  But it looks simple enough to me.


radarr: <IP_ADDRESS>8 -> <IP_ADDRESS>0

Motivation for this change
The radarr test is passing
Things done

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

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

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

Notify maintainers
cc @edwtjo

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/98


How To Screenshot

How do we use the screenshot feature, please? Thank you.

Hello
If not make error (Armel confirm if correct)
1.Must compile with ENABLE_FEAT_F4HWN_SCREENSHOT ?= 1
2. Flash with "custom-custom" firmware
3. Remove function on M-Long (set to NONE)
4. When you navigate, you can take screenshoot with long press on [M]
You must have a serial monitoring and take screenshoot with plug  progamation  cable.
Serial setting : 38400,8,N,1
73's
Boris

OKay, I think in v2.8, Armel said he'd enabled this. M long press still works for me, though. I'll hook u the serial line and try. Do you know the format of the serial data, please?

Hello,
I confirm Boris @f1ixx answer.
This function is not activated by default and requires recompiling the firmware as Boris explains.
About format, it's PBM P1.
Of course, after, it's possible to convert image in others formats with Imagick.
convert Screenshot.ppm +level-colors Grey20,Orange1 -resize 256x192! target.png
Regards,
Armel F4HWN

I've write a  dirty code python to display. You can read here
It write for linux system. I can't test on window.
In code change "/dev/ttyUSB0" by "comx:" where x is serial com. number of you prog. cable.
Soft use : python 3.xx, tkinter, pyserial
No save function, but you can make screenshot.
73's

Ah. I misread "not enabled by default" as the dormant code was already there, but in v2.8 you had enabled it. My mistake. Understood. Thank you.

You're welcome ;)
Armel F4HWN.


No colon names referenced

It was frustrating having to search/remember the colon name for each of these, so I'll do a PR adding it to the table, hopefully it helps!

PR created https://github.com/dannyfritz/commit-message-emoji/pull/25

Long ago, the colon names lived together in harmony.
But then everything changed when the PR #15 attacked.
Only the @dannyfritz master of all unicodes could stop them.
But when the repo need him the most, he vanished

lol
Definitely still here. We've been conversing in #25


Script to run full set of backwards compatibility tests

Description

This PR introduces a script which reads in a json configuration of tests to run and create a Github Workflow with the specified values.
The script outputs markdown, this makes it easy to just copy paste the output into a github issue to track test results. And also ensures it is easily extendable if we want to automate the creation of github issues using the contents.
Each time we want to run tests against a different set of versions, we just need to make one of these configurations. They will likely be largely similar configurations, with different versions specified. They live under scripts/test-matricies
The script assumes you have both jq and gh installed.
The below is output from running make compatibility-tests
Raw:
## Backwards compatibility tests
### Matrix Version: main
#### TestTransferTestSuite
  - [ ] [chain A (main) -> chain B (v4.0.0)](https://github.com/cosmos/ibc-go/actions/runs/2888267079)
  - [ ] [chain A (main) -> chain B (v3.0.0)](https://github.com/cosmos/ibc-go/actions/runs/2888267530)
  - [ ] [chain A (main) -> chain B (v2.0.0)](https://github.com/cosmos/ibc-go/actions/runs/2888268007)
  - [ ] [chain A (v4.0.0) -> chain B (main)](https://github.com/cosmos/ibc-go/actions/runs/2888268466)
  - [ ] [chain A (v3.0.0) -> chain B (main)](https://github.com/cosmos/ibc-go/actions/runs/2888268658)
  - [ ] [chain A (v2.0.0) -> chain B (main)](https://github.com/cosmos/ibc-go/actions/runs/2888268900)
#### TestFeeMiddlewareTestSuite
  - [ ] [chain A (main) -> chain B (v4.0.0)](https://github.com/cosmos/ibc-go/actions/runs/2888269188)
  - [ ] [chain A (v4.0.0) -> chain B (main)](https://github.com/cosmos/ibc-go/actions/runs/2888269651)
#### TestInterchainAccountsTestSuite
  - [ ] [chain A (v0.3.2) -> chain B (v0.1.3)](https://github.com/cosmos/ibc-go/actions/runs/2888270083)
  - [ ] [chain A (v0.1.3) -> chain B (v0.3.2)](https://github.com/cosmos/ibc-go/actions/runs/2888270515)
  - [ ] [chain A (v0.2.2) -> chain B (v0.3.2)](https://github.com/cosmos/ibc-go/actions/runs/2888271011)
  - [ ] [chain A (v0.3.2) -> chain B (v0.2.2)](https://github.com/cosmos/ibc-go/actions/runs/2888271288)

Rendered:
Backwards compatibility tests
Matrix Version: main
TestTransferTestSuite

[ ] chain A (main) -> chain B (v4.0.0)
[ ] chain A (main) -> chain B (v3.0.0)
[ ] chain A (main) -> chain B (v2.0.0)
[ ] chain A (v4.0.0) -> chain B (main)
[ ] chain A (v3.0.0) -> chain B (main)
[ ] chain A (v2.0.0) -> chain B (main)

TestFeeMiddlewareTestSuite

[ ] chain A (main) -> chain B (v4.0.0)
[ ] chain A (v4.0.0) -> chain B (main)

TestInterchainAccountsTestSuite

[ ] chain A (v0.3.2) -> chain B (v0.1.3)
[ ] chain A (v0.1.3) -> chain B (v0.3.2)
[ ] chain A (v0.2.2) -> chain B (v0.3.2)
[ ] chain A (v0.3.2) -> chain B (v0.2.2)

closes: #2029

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (see CONTRIBUTING.md)
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[x] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

@crodriguezvega I'm not actually sure if it's possible to trigger a workflow from inside a workflow 🤔 , but I don't see why it shouldn't work,  I can look into it this!
I left it relatively simple for now, the markdown format output let's us do whatever we want with it, and we can always include the output in other workflows or processes.


[Feature] better way to disable show_progress

Is this your first time submitting a feature request?

[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing client functionality

Describe the feature
When using Pinecone from langchain's Pinecone.from_existing_index there is no way to disable the progressbar. That it is by default True is odd, because this is like verbose output.
Please make this False by default and add an option to turn it on if needed
Describe alternatives you've considered
Patch the code
Who will this benefit?
No response
Are you interested in contributing this feature?
No response
Anything else?
No response

It looks like this is something that could be taken up in LangChain itself, as its used in their usage of upsert queries. https://github.com/hwchase17/langchain/blob/57d8a3d1e8fd376d8db8772977870d01ee5e444e/langchain/vectorstores/pinecone.py#L95

Looks like it's been fully merged, you can now use kwargs to specify the way you want to interact with the pinecone client.


UI failure

Description
UI failure, the "check mark" icon in the lower right corner of the database name, it seems that it is not a vector icon, my computer resolution is enlarged to 175%, other icons are displayed normally, only this one appears a lot of aliasing, please replace this icon, I uploaded a screenshot to illustrate

DBeaver Version
22.3.0
Operating System
No response
Database and driver
No response
Steps to reproduce
No response
Additional context
No response

And one more place

The startup page seems to be the same

DBeaver uses raster icons rather than vector icons.
We can't just replace them with bigger ones, we need to start migrating to vector icons.


Search clubs by location broken

Results don't show when you type in a place you want to search for. I'm guessing the API changed a little. Will need to pinpoint this problem ASAP.

Confirmed. I got this message in the console: Uncaught TypeError: Cannot destructure property lat of 'undefined' or 'null'.


Merge release-5.0.1 to master

Thanks @ryanwilson. Until we get better version management scripts, each release manager will need to update the comments in push-pods.sh for what is not releasing in that particular release.


combineLatest without project returns mutating array.

We have been using combineLatest in combination with distinctUntilChanged and ran into an issue where the array returned from combineLatst was mutation and reused, resulting in distinctUntilChanged comparing the array against itself.
Unsafe:
console.clear();

let first$ =  Rx.Observable.interval(500).take(3).map(i => `A${i}`);
let second$ =  Rx.Observable.interval(500).take(3).map(i => `B${i}`);
let third$ =  Rx.Observable.interval(500).take(3).map(i => `C${i}`);

let combined$ = Rx.Observable
          .combineLatest(first$, second$, third$)
          .debounceTime(1)
          .flatMap(state => Rx.Observable.interval(10).take(2).map(i => state))
          .distinctUntilChanged(x=>x, (a, b) => {
            console.log("A:" + a);
            console.log("B:" + b);
            return a !== b;
          });

combined$.subscribe(console.log);

Safe:
console.clear();

let first$ =  Rx.Observable.interval(500).take(3).map(i => `A${i}`);
let second$ =  Rx.Observable.interval(500).take(3).map(i => `B${i}`);
let third$ =  Rx.Observable.interval(500).take(3).map(i => `C${i}`);

let combined$ = Rx.Observable
          .combineLatest(first$, second$, third$, (a, b, c) => [a, b, c]))
          .debounceTime(1)
          .flatMap(state => Rx.Observable.interval(10).take(2).map(i => state))
          .distinctUntilChanged(x=>x, (a, b) => {
            console.log("A:" + a);
            console.log("B:" + b);
            return a !== b;
          });

combined$.subscribe(console.log);

I would expect both snippets to give the same result, as the project function from combineLatest is optional.
Ps. @babeal brought up this snipped on gitter on 28 juli 2016 19:07.

I assume this is dupe of issue https://github.com/ReactiveX/rxjs/issues/1686 with PR https://github.com/ReactiveX/rxjs/pull/1841?

Looks very similar.

Closed by #1841


fix: set fips in manifest according to our customizations

addresses #708

Thanks, this looks good, I did a small followup in https://github.com/osbuild/bootc-image-builder/pull/715 that adds a (manifest level) test to ensure this does not regress.

I also did a small followup in https://github.com/osbuild/images/pull/1035 for non-iso bootc containers but there is an open questions if this should come via customization or the bootc container itself would set it.

I guess we also want to support this customization for non-iso images(?)

Only ISO use non-bootc to deploy image. bootc supports kargs. So for non-ISO, FIPS can be enabled in Containerfile. Just in case fips=1 configured twice.


[WIP] Change postgresql92 SCL references to rh-postgresql94.

https://trello.com/c/aLn9eAj1
Related PRs:
https://github.com/ManageIQ/manageiq/pull/3676
https://github.com/ManageIQ/manageiq-appliance/pull/9 [this branch]
https://github.com/ManageIQ/manageiq-appliance-build/pull/19
Depends on:

[ ] Depends on https://github.com/ManageIQ/manageiq-appliance-build/pull/19

This is ready to be un-WIP'd and merged after ManageIQ/manageiq-appliance-build#19.

I think selinux will probably be using equivalence
can you ensure that the right selinux tag is on the data directory?
ls -Z /opt/rh/rh-postgresql94/root/var/lib/pgsql/data/pg_log/

UGH, they changed the default data directory, < I think >
ls -Z /opt/rh/rh-postgresql94/root/var/lib/pgsql/data/pg_log/ doesn't exist.  :cry:
/var/opt/rh/rh-postgresql94/lib/pgsql/data looks to be the new location.
So, choices are, set PGDATA environment variable and pass -D #{PostgresAdmin.data_directory} to initdb or use the new default location everywhere.
Thoughts @kbrock?

In other words, it worked, we created the database disk in my test vm, but initdb setup the database in /var/opt/rh/rh-postgresql94/lib/pgsql/data and that's what used when the database starts.  It basically ignored our customization.

cc @gtanzillo @abellotti Thoughts?

My take is leverage APPLIANCE_PG_DATA everywhere for the appliance, the only place you may not be able to use are the couple of places in the kickstart itself, which is ok for now.

I went with the route of specifying the PGDATA env variable in the hopes that initdb and all postgres functions will use it to find the data directory.  The alternative is to be victim to the default data directory changing within the packaged postgresql.
Running a build with this to see how it goes.

We are going to use the same version of postgres for a while. Let not start adding all sorts of environment variables.
(Same as with Apache. We're not changing that directory name for the next year, adding an env variable is just adding complexity.)

Yes @kbrock, it didn't work anyway, it didn't honor PGDATA environment variable.  Instead, I'll move all the hardcoded paths to the new /var/opt/rh/rh-postgresql94/lib/pgsql/data path.

This is ready to go now.  I did a full end-to-end build with the other PRs and it works.  :sunglasses:


Scoreboards to display data (Money, Job Info, etc.)

Player Job
Job Level

I would love this on my server. This is VERY important for me, please implement soon.

I looked into this today and it appears that displaying custom data is rather difficult... Maybe it's a good idea to delay this (again),


Move repo to ansible github organization

As we formalize the support of tox-ansible with the ansible devtools team, we need to move the repo to the ansible org as we have processes in place to ensure the repo is configured correctly and has the appropriate controls in place to ensure a secure supply chain for ansible platform componenets.
This will als allow the repo and it's PR, issues to be added to ansible project boards.
The move should take place on or shortly after September 22nd.
All existing premissions, issues, and PR will be preserved. Github will take care of redirects.
Changes to pypi will happen after the move for secure publishing.
Please comment here with question or concerns related to the move.
Thanks- Brad

This is complete.


empty temp files when downloading NCBI

Hello,
Many of the temporary files generated by the following NCBI download code (using Linux) have 0 bytes, and it happens differently each time I repeat it. The result is that the final fasta file varies in size (44 to 300+ MB). It also happens for other large insect orders (e.g., Diptera). I'm guessing this may be from NCBI/entrez connection errors(?), but no indication of an error is provided. So, it is hard to know which downloads to trust. I'm using CRABS v. 0.1.8 installed via cloning from GitHub. Thanks,
Mike
crabs db_download \
--source ncbi \
--database nucleotide \
--query "((Coleoptera[Organism] OR Coleoptera[All Fields]) AND mitochondrion[filter])" \
--keep_original no \
--email<EMAIL_ADDRESS>\
--batchsize 5000 \
--output "ncbi_Coleoptera.fasta"

Hello @mikeallen-eco,
Thank you for using CRABS.
The file sizes can differ based on sequence length. However, the fact you're observing 0 byte intermediate files does indeed point to an unstable network connection, an overload of the NCBI servers, or restricted access on your account based on too much usage.
To get an idea of which download to trust, I will need to write a script that checks the number of sequences after each download to verify the number of sequences matches the batch size. In case the numbers don't match, the script could redownload these sequences. In case multiple times the download was unsuccessful, I could make CRABS error out or continue based on the last download via user input. Would this be what you're after?
Thanks,
Gert-Jan

Yes, I think that would be a great feature to add. The thought of possible hidden problems with the downloads was troubling me. Thanks for the response (and the software)!
Mike

Perfect, I will add it to the list of items to add in the next version of CRABS.
Thanks,
Gert-Jan

Hello @mikeallen-eco,
crabs --version 1.0.0 is now available and can be downloaded by cloning the GitHub repo (we hope to update the Docker container and conda package shortly). This version should resolve your issue, but please reopen this thread if the issue persists.
Best wishes,
Gert-Jan


FastRPC clarification - HexRouter vs KraitRouter

It seems that KraitRouterImpl.cpp runs on the Hexagon/aDSP and that HexRouterImpl.cpp runs on the Krait/CPU. Is that correct?
Also, I found your QDN forum question about FastRPC and UART. I am hoping to stream high-rate sensor data (MAVLink messages from ROSflight) from the autopilot on the aDSP to the MAVLink<-->ROS parser (think mavros) on the CPU. Have you had any practical (i.e., bandwidth, latency) limitations with running HEXREF and using FastRPC?
My understanding is that by using rpcmem_* in combination with the standard FastRPC/idl method of calling methods on the DSP allows for zero-copy transfer.

Yes, that's correct, except one nomenclature thing: I don't run code on the aDSP (no DSPAL support at this time), but rather the SLPI, on the 8096 (new Snapdragon flight board). The SLPI is where Qualcomm run their flight stack on the 8096.
Are you trying to run ROSflight on the DSP? Or are you trying to use the DSP I/O to talk to a microcontroller running ROSflight? In that Qualcomm ticket that you reference, we were doing something more similar to the latter.
I haven't quantified latency in either direction (Hexagon to Linux, or Linux to Hexagon) using my implementation in Hex/Krait Router. I run both attitude and position control loops on the Hexagon, so any latency wouldn't affect my phase margins. I haven't tried to run a controller on the Linux side - you could always send attitude and thrust setpoints in to my controller from a position controller in ROS (I created a custom message, mav_msgs::AttitudeRateThrust), running only my attitude loop on the Hexagon, because latency in position commands would be less of an issue.
The simplest way for me to get a latency estimate is to use rostopic delay to get the delay of the IMU messages I'm sending - I'll try to do that tonight or tomorrow.
As for bandwidth, I am sending IMU data (acquired at 1kHz, shipping 2 IMU samples per 500Hz control cycle) and odometry (propagated at 500Hz) from Hexagon to Krait without any problems. State and IMU biases come back from Krait to Hexagon at 30 Hz (camera rate, so measurement update rate).
I compensate for latency in either direction by calculating offset between Hexagon and Krait clocks (I wrote a kernel module to read a special register with that value and map it to a file), and correcting timestamps as appropriate. The IMU propagator can handle delayed state updates by buffering IMU samples.
This post has some good info from the Qualcomm folks about cache flushing - I don't know how FastRPC cache synchronization is implemented / differs when you use rpcmem_alloc, but my working assumption is that performance will only get better when using those specialized alloc functions.
Here are some more Qualcomm forum posts that don't have full answers, but that at least to me, paint a consistent story:
https://developer.qualcomm.com/forum/qdn-forums/software/hexagon-dsp-sdk/hlos/29823
https://developer.qualcomm.com/forum/qdn-forums/software/hexagon-dsp-sdk/hlos/35087
https://developer.qualcomm.com/forum/qdn-forums/maximize-hardware/multimedia-optimization-hexagon-sdk/computer-vision/27324

These are immensely useful insights---thanks.
My plan is to run ROSflight (attitude control/estimation; low-level io) on the DSP and communicate with the CPU/ROS via FastRPC, where the position loop would live. That way if we are using Snapdragon or a companion computer/microcontroller the software architecture is essentially the same.
For hexagon<-->krait clock compensation, I'm curious why you decided to go with the kernel module. It's my understanding that each time you read the sysfs device the walltime_dsp_diff_show method is called and the offset is recalculated. I had assumed that a one time calibration would suffice, which is what the Pixhawk does. Are there any pros / cons that stick out to you between the two methods? I like the idea of having the offset calculation code in one place which can be used by any code base without having to link to a library, as is the case for the ko.

your ko which reads from a magic memory location and creates a sysfs device on the cpu side
Pixhawk, which reads /sys/kernel/boot_adsp/qdsp_qtimer on the cpu side once and then uses that value to compensate

And thanks for the links!

The kernel module is standalone - it is just in the repo for convenience. I've always built it using the Yocto / bitbake tooling from CodeAurora; I found that much easier than trying to set up the kernel headers for cross-compiling the kernel module myself. You can use my kernel module for what you want to do without having to link against anything - just install the kernel module with insmod and then read the file.
I did the calculation in the kernel module so that I could lock out interrupts to avoid context switching in the (admittedly small) time / # of instructions between reading the file and getting the time. This method also eliminates any effect on the calculated offset from latency from that file read. I was planning to also add a file with the offset CLOCK_MONOTONIC <-> CLOCK_REALTIME, as I use that in the camera components, and they are still vulnerable to such context switching at the moment.
I repeatedly get the offset in case something (like chrony / ntp) is slewing or stepping the Linux clock. That slew / step would invalidate an offset determined only once.
I haven't used ROSFlight myself, but issues like the following make me wonder why you want to use it for low-level estimation/control/io on the Snapdragon if you have to implement support yourself? Is it so you can keep tuning parameters and device support the same? That seems like a stronger argument than software architecture, except in the most general sense (high-level computer connected to low-level one), with split-level estimation, control, etc.
https://github.com/rosflight/firmware/issues/289
The attitude filter / position propagator in this flightstack is modified from:
https://www.mdpi.com/1424-8220/15/8/19302/pdf
And the controller is the standard nonlinear backstepping one for multirotors:
https://deepblue.lib.umich.edu/handle/2027.42/97267
What interface does your position controller expect on the Linux side? I'm happy to take a look at what it would take to interface my flight stack to your position controller... It'll just be a custom ROS message type with attitude, angular velocity, and thrust setpoint. Side-note - we're supporting the TI TMS570 microcontroller with all the same code as above, just interfacing over UART:
http://www.ti.com/tool/LAUNCHXL2-570LC43
I'd need to implement more precise PWM on the nHET peripheral, but that's the only piece missing to fly a multirotor with it.

Here are some latency stats with a non-RT priority subscriber:
In this case, I'm reading the IMU data at 50 Hz from Linux, using BlspSpiDriver to call the dsp_relay_ functions over FastRPC.
/data/eng/load # rostopic delay /imu_0
...
average delay: 0.002
	min: 0.001s max: 0.010s std dev: 0.00107s window: 1012
average delay: 0.002
	min: 0.001s max: 0.010s std dev: 0.00107s window: 1062
average delay: 0.002
	min: 0.001s max: 0.010s std dev: 0.00109s window: 1106

/data/eng/load # rostopic hz /imu_0
subscribed to [/imu_0]
average rate: 50.600
	min: 0.012s max: 0.027s std dev: 0.00158s window: 50
average rate: 50.595
	min: 0.012s max: 0.027s std dev: 0.00147s window: 101
average rate: 50.594
	min: 0.012s max: 0.027s std dev: 0.00143s window: 151
average rate: 50.581
	min: 0.012s max: 0.027s std dev: 0.00134s window: 174

This is from the scenario I described above, where I run attitude (and position) control on the DSP, and pipe the IMU data across to Linux and to a filter.
/data/eng/load # rostopic delay /imu_0
...
average delay: 0.020
	min: 0.005s max: 0.073s std dev: 0.00969s window: 24695
average delay: 0.020
	min: 0.004s max: 0.073s std dev: 0.00965s window: 25745
average delay: 0.020
	min: 0.004s max: 0.073s std dev: 0.00976s window: 26787

/data/eng/load # rostopic hz /imu_0
subscribed to [/imu_0]
average rate: 1031.606
	min: 0.000s max: 0.038s std dev: 0.00399s window: 976
average rate: 1023.100
	min: 0.000s max: 0.038s std dev: 0.00386s window: 2009
average rate: 1020.751
	min: 0.000s max: 0.039s std dev: 0.00406s window: 3024
average rate: 1018.731
	min: 0.000s max: 0.039s std dev: 0.00417s window: 3702

I took another measurement to get more detailed stats - I buffer everything I send from DSP to Linux, in order to reduce the number of FastRPC calls, and avoid spinning by blocking in my retrieve call until the transfer buffer is full. This could easily be reconfigured if you would rather return immediately if some lesser threshold for transfer is met (but the buffer still isn't full). Anyway, here are IMU latency measurements in microseconds.


Clicking the 'scan' button does not automatically rescan the  CIS Benchmarks

Environment
Platform:Rancher
Kubernetes/Platform Version(s): 1.20.4
Describe the bug
Usually, after clicking the scan button, it should automatically scan for host vulnerabilities and CIS baselines. However, the current situation is that it does not rescan the baseline during scanning. However, calling the API interface can trigger the rescan mechanism, which requires us to call the API interface to verify whether the configuration is effective. I think this is a bug, and the interface should also rescan the baseline to make sense
To Reproduce
Steps to reproduce the behavior:

Go to 'Assets'
Click on 'Nodes'
Click on 'scan'
Click on 'Compliance'
See the exclamation mark next to Docker CIS Version: 1.2.0

Expected behavior
It should display the time of the last click to scan
Screenshots
N.A
UI Issue
Desktop (please complete the following information):**

OS: Windows 10
Browser  chrome
Version 116.0.1938.62

Additional context
NA

Same issue here on the UI we don't the last scanned status.

What's the version of NeuVector? (indicated on the left bottom corner of UI)

We are using the latest one 5.2.1 The cluster has around 60 Nodes so not sure if Neuvector is taking time to complete the scans.
Another thing we noticed is under system components we are not able to see the scanner pods, though am able to see the controllers and enforcers.
Attaching a screenshot for reference.

I do have it installed on another small POC cluster and I do see the scanner pods available in it and am able to see the scanned status. One difference the Neuvector Installation in that is 5.2.0

Which API interface do you call that can trigger the rescan mechanism?
On that 5.2.1 env, could you open browser developer tool panel(F12), reproduce the issue and see what's the output in the dev tool panel after you click Scan?

Reinstalled and the issue is resolved,

我们正在使用最新的 5.2.1 集群大约有 60 个节点，因此不确定 Neuvector 是否需要时间来完成扫描。
我们注意到的另一件事是在系统组件下，我们无法看到扫描器吊舱，尽管我能够看到控制器和执行器。
附上屏幕截图以供参考。

我确实将它安装在另一个小型 POC 集群上，并且我确实看到了其中可用的扫描程序 pod，并且能够看到扫描状态。Neuvector 安装的一个区别是 5.2.0

The same bug occurred when I executed the updater pod update and the scanner pod log kept repeating. The external status was not running, but it actually couldn't work properly. Therefore, I also saw that it was empty on this interface

Which API interface do you call that can trigger the rescan mechanism? On that 5.2.1 env, could you open browser developer tool panel(F12), reproduce the issue and see what's the output in the dev tool panel after you click Scan?

5.2.0，I called the scanning API in Restapi, specifically https://ip:port/v1/bench/host/host -ID/Docker


ament_cmake_auto problem

hello,
when I ran the "colcon build --symlink-install" I got cmake version error and upgrade the cmake to 3.23.2.
Then I face error as below:
Starting >>> isaac_ros_common
Starting >>> isaac_ros_test
Starting >>> FortranCInterface
Starting >>> IntelFortranImplicit
--- stderr: isaac_ros_common
CMake Error at CMakeLists.txt:16 (find_package):
By not providing "Findament_cmake_auto.cmake" in CMAKE_MODULE_PATH this
project has asked CMake to find a package configuration file provided by
"ament_cmake_auto", but CMake did not find one.
Could not find a package configuration file provided by "ament_cmake_auto"
with any of the following names:
ament_cmake_autoConfig.cmake
ament_cmake_auto-config.cmake

Add the installation prefix of "ament_cmake_auto" to CMAKE_PREFIX_PATH or
set "ament_cmake_auto_DIR" to a directory containing one of the above
files.  If "ament_cmake_auto" provides a separate development package or
SDK, be sure it has been installed.

Failed   <<< isaac_ros_common [0.96s, exited with code 1]
Aborted  <<< IntelFortranImplicit [0.67s]
Aborted  <<< FortranCInterface [0.98s]
Aborted  <<< isaac_ros_test [1.31s]
my jetson is as follows:

Did you happen to source /opt/ros/humble/setup.bash first? Not finding ament_cmake_auto is usually a sign that ROS 2 core libraries are not on the path yet.


DRAFT: Support for IS 6.1.0

Purpose
The purpose of the PR is to support the operator to the latest version of IS ; IS 6.1.0
Goals
Being able to run the operator with IS 6.1.0
Approach

First make sure the operator runs with IS 5.11.0 (already supported version)
Create and host docker image of IS 6.1.0
Integrate 6.1.0 to the operator

Learning

https://faun.pub/part-01-deploying-wso2-identity-server-5-11-0-on-kubernetes-with-all-new-k8s-operator-e6d9e76d7e7

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


Purple Models?

Is there a setting to remove the models from being the unity missing texture?

If you hit purple models then your installation probably failed. We use this as a deprecated mode when our outline file is not found.
Could you re-run the installer and give the ouptut here?

Well, well I fixed it by re-installing SPT/Tarkov it was actually some of my files messed up. Thanks for telling me in case it does happen that way!

Great if it is working now


Manage elm version by npm

npm run test
  shell: /usr/bin/bash -e {0}

><EMAIL_ADDRESS>test
> elm-test

/bin/sh: 1: /home/runner/work/emobu/emobu/node_modules/elm/bin/elm: Text file busy
Compiling
`elm make` failed with exit code 12[6](https://github.com/mobu-of-the-world/emobu/actions/runs/3776209443/jobs/6419381649#step:6:7).
Error: Process completed with exit code 1.

🤔

Passed! 🎉

Time performance is not many affected
Unfortunately needed some hack, however made less dependencies


[BUG] pip's dependency resolver

Local Installation of the trainer, following original video (https://www.youtube.com/watch?v=DFWVvJx1Cfs).
I already installed anaconda and trainer on other desktop pcs with no issues.
This time I got this errore message when installing NAM:
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
anaconda-cloud-auth 0.1.4 requires pydantic<2.0, but you have pydantic 2.7.3 which is incompatible.
OS: Windows 11
Does this affects the training or is it something ignorable?

You can ignore it--anaconda-cloud-authisn't used by NAM--it's just there in the base environment that Anaconda ships with (which is getting bloated, it seems. I'd like to find a better solution and stop depending on it.)
Lmk if something does go wrong; otherwise, feel free to close this Issue if you're happy.


Can we have a java library for this please ?

if you want to port it yourself, feel free to do so. I don't have any Java projects that will have use for this so I can't put time into it.


Prevent floating tab to move outside of parent window

Since the floating tab "belongs" to the parent window, the users should not be able to move it outside of the parent window's bounds.
This should probably include the toolbar too.

This is not possible, as there's no "window moved" event in the Chrome API. It's probably also not the right thing to do because it unnecessarily restricts the user. Closing this.


[ZEPPELIN-1489] Remove jdbc-like interpreter example properties and improve JDBC docs

What is this PR for?
Currently we can use %jdbc(prefix) for multi database connections(e.g. configuring both postgresql and hive in one JDBC interpreter). But after #1360 merged, Zeppelin doesn't support %jdbc(prefix) to %prefix anymore. So auto-prefix feature is not working for %jdbc(prefix). The auto-prefix only works for %prefix now. Considering this status, it would be better we remove the JDBC connection examples in jdbc/interpreter-setting.json (this is come from #1096) so that users can create several JDBC interpreters instead of using multiple connections with one JDBC interpreter.  I removed the related contents in jdbc.md as well.
Also, the contents of current JDBC documentation page is quite confusing and unorganized. So I updated the contents with some screenshot images to guide "How to create JDBC interpreter", "How to edit the interpreter properties for the connection", "How to use %prefix with the interpreter", and so on.
What type of PR is it?
Improvement & Documentation
What is the Jira issue?
ZEPPELIN-1489
How should this be tested?

Removing example properties in interpreter-setting.json

after applying this patch and build with mvn clean package -DskipTests -Pspark-1.6 -pl 'jdbc,zeppelin-interpreter,zeppelin-web,zeppelin-server,zeppelin-zengine,zeppelin-display'
create JDBC interpreter and check whether the example settings are gone or not

JDBC docs

Build only docs/ dir as described in here
Go to interpreter -> JDBC and just compare this locally builded page with the original JDBC page

Screenshots (if appropriate)

Before

After

Some parts of updated JDBC docs
Since many contentst are changed, it would be better to build docs/ locally to review all of the updated contents.

Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? yes

@AhyoungRyu Great work! It definitely helps using JDBC datasource in Zeppelin.
CI failure is not related.
LGTM and merge if there're no further comments.

@Leemoonsoo Thanks!


Add Default ViewWidth and ViewHeight to Actor

In ActorLoader._ReadShot, there's a block for grabbing the view width and height. Stepping through the loading process in Chrome seems that these values are never set.
actor._ViewWidth = block.reader.readFloat32();
actor._ViewHeight = block.reader.readFloat32();

Ideally I'd love these values to be the base width and height of the initial frame of the animation. Currently in the Construct 2 Plugin, I have it set that the user has to hand-type the width and height so that scaling in the game engine works correctly to match what you've set in the editor. Having this based in the runtime would be pretty great.

That View block only gets exported when you post a shot to the 2D site. It's used to calculate the bounds of the camera you align when exporting the shot. This library gets used for that too.
We are going to do a pretty serious "maturity" upgrade to this library in the near future. That will introduce support for more features like calculating bounding boxes dynamically. We could export that information for the animation too but I think having it be dynamically calculated will be relatively inexpensive, more accurate, keep download sizes down, and it'll be future-proof as our animations start becoming more dynamic (more on that later...).


attribute "mode" ignored in the select component

Ionic version:  (check one with "x")
[ ] 1.x
[x] 2.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
[ ] support request
Current behavior:
The popup modal stay on the default mode of the application
Expected behavior:
mode must be applied regardless the default of the system, as what described here: https://ionicframework.com/docs/v2/api/components/select/Select/
Steps to reproduce:
<ion-select [(ngModel)]="selectedOption" mode="md">
  <ion-option *ngFor="let opt of options" [value]="opt.value">{{opt.label}}</ion-option>
</ion-select>

This is the app config:
export const IonicAppConfig = {
  pageTransition: 'ios-transition',
  iconMode: 'ios',
  mode: 'ios'
};

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
Cordova CLI: 6.5.0
Ionic Framework Version: 2.0.1
Ionic CLI Version: 2.2.1
Ionic App Lib Version: 2.2.0
Ionic App Scripts Version: 1.1.3
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 10
Node Version: v7.2.0
Xcode version: Not installed

P.S: do you want me to close this issue after 1 or 2 days? since you love close issues even if they are not solved !? or maybe you need to rename this framework to: ionic-close-issues

Thanks for opening an issue with us! We will look into this.

So the mode is actually working properly on the Select component; the select component being the input box with the arrow.

I think what you're looking for is changing the mode of the Alert that pops up, which could be achieved with the following:
<ion-item>
  <ion-label>Select</ion-label>
  <ion-select [selectOptions]="{ mode: 'md' }">
    <ion-option value="1">1</ion-option>
    <ion-option value="2">2</ion-option>
    <ion-option value="3">3</ion-option>
    <ion-option value="4">4</ion-option>
  </ion-select>
</ion-item>

We could change it so that if you set the mode on the Select it passes it to the alert automatically, but I wonder if anyone is using this to where they want the input changed but not the alert? Including @manucorporat for discussion.

The look must be like this:

It is clear like this, nothing else, I tried to use alert and was not the desired look.
I noticed that you change the mode like [selectOptions]="{ mode: 'md' }" not like what in documentation and what I used  mode="md" .
We gave up on ionic for desktop, so I can't test this in an actual project, today we completely finished refactor to get rid of Ionic2.

Yes, the documentation on select options are here: http://ionicframework.com/docs/v2/api/components/select/Select/#select-options
This is how you style the alert or action sheet that you call from the select since it is just using the alert/action-sheet API under the hood. 🙂


prompt is unsupported in electron

I tried to open simpypresent in electron, but it failed because 'prompt is not and will never be supported' in electron. This means the selection of the solid provider fails to start up which blocks the rest of the environment.
Can we replace the prompt with something else that will be compatible with electron?

I'd vote for using <dialog> as an alternative to take user inputs.


(multipart-fetch) - Upgrade extract-files

Resolve #1791
Summary
The multipart fetch exchange no longer works correctly with ReactNative. This is due to the extract-files package being somewhat out of date.
When performing a file upload the files list is not correctly extracted from the operation variables, this results in the operation being sent to the server as a normal fetch request with application/json as the content type.
The server doesn't recognise the upload with [CombinedError: [GraphQL] map[string]interface {} is not an Upload].
A simple version bump of this package fixes uploads.
Set of changes

Upgrade extract-files to the latest release 11.0.0

Hiya 👋 Thanks for taking the time to try out the upgrade and figure this out.
Can you please take a quick glance at the PR section of the contributing guide? We have a section on adding changesets for the changelog, in case of package releases. That's mentioned and described there two: https://github.com/FormidableLabs/urql/blob/main/CONTRIBUTING.md#what-are-the-pr-conventions
Otherwise please make sure that the checkbox for "Allowing maintainers to make edits" (in the right sidebar) is ticked so I can add that. Cheers! 🙌

@kitten i've added changeset.

@haswalt just chiming in, you haven't added a changeset, the contributing guidelines detail how to do so

@JoviDeCroock correct, missed the commit. Have pushed it up now

@kitten this package upgrade broke my react build process:
yarn run v1.22.11
$ craco build
Creating an optimized production build...
Failed to compile.

<EMAIL_ADDRESS>"export 'default' (reexported as 'ReactNativeFile') was not found in './ReactNativeFile.js'

This issue should be re-opened. Version 0.1.12 works fine.

@PhilippMolitor At this point please direct build issues to extract-files' repo. It's their file structure and something they'll need to address, which is also what previously affected our block for upgrades. Now that we've upgraded we cannot simply downgrade and break React Native again.


Input file is missing or of an unsupported image format

I'm getting the above error when using the response from a request call for constructing the sharp object.
const url = "https://padletuploads.blob.core.windows.net/dev/2/1d5f5e1d281a2145aae25595d050e77b/JPG_Image.jpg"
request(url, function(error, response, body) {
  if(!error) {
    sharp(body).rotate().resize(300,300).toBuffer().then(function(data) {
      console.log("Done")
    })
  }
})

I've checked the URL and it returns a valid image.
In fact, when using streaming, things work. E.g.
//THIS WORKS
const url = "https://padletuploads.blob.core.windows.net/dev/2/1d5f5e1d281a2145aae25595d050e77b/JPG_Image.jpg"
const pipeline = sharp().rotate().resize(300,300)
request(url).pipe(pipeline)

I tried to wrap the body variable in a new Buffer(body, "binary") call but that did not help either. It just gave me a different error message: Input buffer contains unsupported image format
Wondering what I am doing wrong.

Hello, the request module expects encoding to be set to receive body as a Buffer.
- request(url, function(error, response, body) {
+ request({ url, encoding: null }, function(error, response, body) {

https://github.com/request/request#requestoptions-callback

I'm getting the same error, but I try to use the input stream returned by the Google Cloud Storage File object: File.createReadStream, which returns a readable stream. Any tips? https://stackoverflow.com/questions/48960700/how-to-run-sharp-transform-on-readable-http-stream-and-write-tofile


PT Award Eligibility - BCAG

User Story
As a part-time student I am eligible for BCAG if attending a BC Public institution, have assessed need, a family income below the threshold and am in an eligible program credential type.

Current Behaviour
The award eligibility check correct checks for institution type, assessed need and family income thresholds. It is checking for the incorrect program credential types.
if institutionType = "BC Public"
  and (calculatedDataTotalAssessedNeed >= 1)
  and (calculatedDataTotalFamilyIncome < dmnPartTimeAwardFamilySizeVariables.limitAwardBCAGThresholdIncome)
  and (programCredentialType = "undergraduateDegree"
       or programCredentialType = "graduateCertificate"
       or programCredentialType = "graduateDiploma"
       or programCredentialType = "qualifyingStudies")
then true
else false

Expected Behaviour
The valid credential types to be eligible for PT BCAG are undergraduate degree, undergraduate diploma and undergraduate certificate.
if institutionType = "BC Public"
  and (calculatedDataTotalAssessedNeed >= 1)
  and (calculatedDataTotalFamilyIncome < dmnPartTimeAwardFamilySizeVariables.limitAwardBCAGThresholdIncome)
  and (programCredentialType = "undergraduateCertificate"
       or programCredentialType = "undergraduateDiploma"
       or programCredentialType = "undergraduateDegree"
)
then true
else false

Acceptance Criteria

[ ] Update awardEligibilityBCAG in parttime-assessment-2023-2024.bpmn to the following:

if institutionType = "BC Public"
  and (calculatedDataTotalAssessedNeed >= 1)
  and (calculatedDataTotalFamilyIncome < dmnPartTimeAwardFamilySizeVariables.limitAwardBCAGThresholdIncome)
  and (programCredentialType = "undergraduateCertificate"
       or programCredentialType = "undergraduateDiploma"
       or programCredentialType = "undergraduateDegree")
then true
else false

@HRAGANBC - I've confirmed this with policy via email.

@CarlyCotton thanks - moving to grooming pipeline

PY limit deductions(dmnPartTimeAwardAllowableLimits) are not part of this ticket as per grooming discussion. As PY limit deductions are going to be handled in separate ticket.


Do not mess with scl repos if package is already available

Sometimes packages and yum repos are managed outside the "standard" mechanisms (e.g. RHSM). The existing SCL code (at least for RHEL) always insists on checking for and enabling a specific repo vis RHSM.
This patch and supports alternate repo strategies by checking to see if the package we want is already available (in Chef's YUM cache) before messing with repos. This also happens to avoid excess YUM cache flushes. So as long as a prior cookbook/recipe or existing config makes the appropriate RPM available, now poise won't mess with (and possibly break) yum repos/subscriptions.

cc @jmercier-ndp

There is a small assumption here, that if the primary scl package is available, then so is the devel package. But a similar assumption already exists in the install_scl_repo method that a single repo has both packages.


Update LDAP connection initialization API

Purpose
This PR updates the LDAP connection initialization API with a return type of ldap:Error. But the issue https://github.com/ballerina-platform/ballerina-lang/issues/17774 is applicable for here also.
Fixes https://github.com/ballerina-platform/ballerina-lang/issues/17353
Check List

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

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

[ ] Increased Test Coverage
[ ] Added necessary documentation

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

Codecov Report

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

@@           Coverage Diff           @@
##           master   #17901   +/-   ##
=======================================
  Coverage   15.17%   15.17%
=======================================
  Files          46       46
  Lines        1206     1206
  Branches      188      188
=======================================
  Hits          183      183
  Misses       1009     1009
  Partials       14       14

Continue to review full report at Codecov.

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


How to use the xAxisTransform property.

Since I want to use the xAxisTransform function,but the document does't show how it work.
Cloud someone tell me how to  use the xAxisTransform property.

xAxisTransform expects a function which takes the x-Value and returns the transformed value (int, string, null, ...), see here.
For example to invert the labels, use:
let xLabels = (i) => -i
<Chart xAxisTransform={xLabels}>

@msom Thanks a lot!Sorry for late to reply you.


Research how to implement processing for ?? and .?

https://github.com/DevExpress/testcafe-hammerhead/commit/0461f98d7598e00db0b8e44c28bbd3a571300ce3


Forward request headers on redirects

What
Catch request headers in the original loadUrl request, and forward them to handleUrl so request headers are not dropped.
Why
On API31+ devices, OneAuth's MSA interactive SSO tests started failing consistently. Upon further investigation, this is because on API31+ (loaded with WebView 88+), Webview shipped a security fix that prevents request headers from being forwarded if the domain doesn't match the original URL. OneAuth-MSAL appends the refresh token in the request header, but in Fiddler traces, this is dropped when browser redirects from login.microsoftonline.com to login.live.com. This results in other telemetry information like correlation ID being dropped.
Reference: https://infosecwriteups.com/leakage-of-sensitive-data-through-android-webviews-3b0b86486a28
How
Capture request header in original loadURL request, and then in handleUrl, override the current request and append original request headers there.
Testing
Confirmed OneAuth's MSA interactive SSO tests are working again with this fix.

Curious, why in your testing was this specific to MSA? The same should be applicable to AAD too, right?

Curious, why in your testing was this specific to MSA? The same should be applicable to AAD too, right?

AcquireCredentialInteractively MSA is the only thing where SSO relies on the RT in a header being carried to a cross-host redirect.  This general problem does show up everywhere (e.g. correlationid getting dropped).


Sign in with Apple Button

Are there any plans to add a Sign in with Apple button? Seems especially useful for web based iOS applications that are now required to include Sign in with Apple. I'd be happy to submit a PR.

You can implement it with the createButton() function.
Here is my approach:
import React from "react";
import { createButton } from "react-social-login-buttons";
import AppleIcon from "./apple.svg";

const config = {
  text: "Sign in with Apple",
  icon: (props) => <AppleIcon {...props} style={{ fill: "white", width: "26px", height: "26px" }} />,
  style: { background: "black" },
  activeStyle: { background: "#293e69" }
};

const SignInWithAppleButton = createButton(config);

export default SignInWithAppleButton;

In this case, you need to find the Apple icon.

Thank you for the request. This should be now available in the version 3.3.0.


Investigate if migration is needed for tft price pallet

Storage changed from U64F64 to u32, might cause problems during upgrade if there are contracts that need to be billed.

https://github.com/threefoldtech/tfchain/commit/543176fa60d7307ee69bf8ff7c01cf38068f4230


Code generation computes pointer size for host platform, not target

rs_client.py contains this line:
    ptr_size = 8 if sys.maxsize > 2**32 else 4

This is probably incorrect when cross-compiling for example from a 64-bit platform to a 32-bit platform.

TARGET is available from Cargo, and I can use this table: https://github.com/llvm-mirror/llvm/blob/master/lib/Support/Triple.cpp#L1167

I’ve only noticed this line in passing and went “Uh, this looks weird”. I haven’t seen any actual bug around this.
That said, Servo only (indirectly) use rust-xcb on desktop Linux, which if I remember correctly we only test on 64-bit. So we wouldn’t notice that bug.

v1.0 do not have pointer size dependent code generation.

v1.0.0 is released.


Possible error in scipy.stats.linregress

I have used this routine on a simple dataset to create a linear regression.  The answer I get is different from the one obtained using MATLAB and SPSS.  Is there an error in the code?
The dataset is attached with the first variable is IBI (Y) and the second is TSS_MAX (x).  Also attached is the SPSS and linregress results as PDF files.
scipyCodeoutput.pdf
spss_output_paint.pdf
paintcr2.pdf

Numpy arrays (just like python lists) use zero-based indexing.  This code:
IBI=paintcr2csv[1:10,0]
TSS_MAX=paintcr2csv[1:10,1]

discards the first row of paintcr2csv.  You could change that to
IBI = paintcr2csv[:10,0]
TSS_MAX = paintcr2csv[:10,1]

if you want just the first 10 rows of the array.
Here's the result of linregress using all 10 samples:
In [10]: x = np.array([15, 13, 24, 43, 55, 52, 62, 60, 78])

In [11]: y = np.array([51, 56, 52, 47, 37, 44, 44, 36, 35])

In [12]: linregress(x, y)
Out[12]: LinregressResult(slope=-0.30412621359223302, intercept=58.250970873786407, rvalue=-0.91415542657021986, pvalue=0.00056068311341998637, stderr=0.050971781908560367

That matches the SPSS output.


K8SPXC-1446: use kubectl in wait_for_delete function

CHANGE DESCRIPTION
Problem:
In pitr test a check whether all the proxy pods were removed during restore was added. until in wait_for_delete function "hangs" for 2+ mins while checking pod/pitr-proxysql-0 deletion.  During this time the restore finishes and  when pod/pitr-proxysql-1 is checked, the restore is finished and all the pods are up and running again.
Cause:
kubectl_bin usage instead of kubectl
Solution:
Use kubectl in  wait_for_delete
CHECKLIST
Jira

[ ] Is the Jira ticket created and referenced properly?
[ ] Does the Jira ticket have the proper statuses for documentation (Needs Doc) and QA (Needs QA)?
[ ] Does the Jira ticket link to the proper milestone (Fix Version field)?

Tests

[ ] Is an E2E test/test case added for the new feature/change?
[ ] Are unit tests added where appropriate?
[ ] Are OpenShift compare files changed for E2E tests (compare/*-oc.yml)?

Config/Logging/Testability

[ ] Are all needed new/changed options added to default YAML files?
[ ] Are all needed new/changed options added to the Helm Chart?
[ ] Did we add proper logging messages for operator actions?
[ ] Did we ensure compatibility with the previous version or cluster upgrade process?
[ ] Does the change support oldest and newest supported PXC version?
[ ] Does the change support oldest and newest supported Kubernetes version?

Test name
Status

affinity-8-0
passed

auto-tuning-8-0
passed

cross-site-8-0
passed

custom-users-8-0
passed

demand-backup-cloud-8-0
failure

demand-backup-encrypted-with-tls-8-0
passed

demand-backup-8-0
passed

haproxy-5-7
passed

haproxy-8-0
passed

init-deploy-5-7
passed

init-deploy-8-0
passed

limits-8-0
passed

monitoring-2-0-8-0
failure

one-pod-5-7
passed

one-pod-8-0
passed

pitr-8-0
passed

pitr-gap-errors-8-0
passed

proxy-protocol-8-0
passed

proxysql-sidecar-res-limits-8-0
passed

pvc-resize-5-7
passed

pvc-resize-8-0
passed

recreate-8-0
passed

restore-to-encrypted-cluster-8-0
failure

scaling-proxysql-8-0
passed

scaling-8-0
passed

scheduled-backup-5-7
passed

scheduled-backup-8-0
passed

security-context-8-0
passed

smart-update1-8-0
passed

smart-update2-8-0
passed

storage-8-0
passed

tls-issue-cert-manager-ref-8-0
passed

tls-issue-cert-manager-8-0
passed

tls-issue-self-8-0
passed

upgrade-consistency-8-0
passed

upgrade-haproxy-5-7
passed

upgrade-haproxy-8-0
passed

upgrade-proxysql-5-7
passed

upgrade-proxysql-8-0
failure

users-5-7
passed

users-8-0
failure

validation-hook-8-0
passed

We run 42 out of 42

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1893/commits/892c3a5243d6a48a5ebeafb964cea168072e59db
image: perconalab/percona-xtradb-cluster-operator:PR-1893-892c3a52

Test name
Status

affinity-8-0
passed

auto-tuning-8-0
passed

cross-site-8-0
passed

custom-users-8-0
passed

demand-backup-cloud-8-0
failure

demand-backup-encrypted-with-tls-8-0
passed

demand-backup-8-0
passed

haproxy-5-7
passed

haproxy-8-0
passed

init-deploy-5-7
passed

init-deploy-8-0
passed

limits-8-0
passed

monitoring-2-0-8-0
passed

one-pod-5-7
passed

one-pod-8-0
passed

pitr-8-0
passed

pitr-gap-errors-8-0
passed

proxy-protocol-8-0
passed

proxysql-sidecar-res-limits-8-0
passed

pvc-resize-5-7
passed

pvc-resize-8-0
passed

recreate-8-0
passed

restore-to-encrypted-cluster-8-0
passed

scaling-proxysql-8-0
passed

scaling-8-0
passed

scheduled-backup-5-7
passed

scheduled-backup-8-0
passed

security-context-8-0
passed

smart-update1-8-0
passed

smart-update2-8-0
passed

storage-8-0
passed

tls-issue-cert-manager-ref-8-0
passed

tls-issue-cert-manager-8-0
passed

tls-issue-self-8-0
passed

upgrade-consistency-8-0
passed

upgrade-haproxy-5-7
passed

upgrade-haproxy-8-0
passed

upgrade-proxysql-5-7
passed

upgrade-proxysql-8-0
passed

users-5-7
passed

users-8-0
passed

validation-hook-8-0
passed

We run 42 out of 42

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1893/commits/892c3a5243d6a48a5ebeafb964cea168072e59db
image: perconalab/percona-xtradb-cluster-operator:PR-1893-892c3a52


Add copyright headers in all files

Add copyright headers in all files. See docTR repo as an example.

Moved to product board


deserialization to std::bitset doesn't follow the endianness

Issue
The deserialization of std::bitset seems always to follow the little endian (the least significant digit comes first).
Reproduce
Godbolt link: https://godbolt.org/z/4MeaYcYTq
Code:
#include "zpp_bits.h"
#include <fmt/format.h>
#include <fmt/ranges.h>
#include <bitset>

void test()
{
    auto data = std::vector<uint8_t>{0x19, 0x5f, 0x1c, 0x7c, 0xb8, 0x7c, 0x43, 0x33};
    auto deserialize_to = zpp::bits::in{ data, zpp::bits::endian::network{} };

    auto my_struct = std::bitset<64>{};
    deserialize_to(my_struct).or_throw();
    fmt::print("deserialized bitset: {:x}\n", my_struct.to_ullong());

    deserialize_to.reset();
    auto my_struct2 = uint64_t{};
    deserialize_to(my_struct2).or_throw();
    fmt::print("deserialized int64: {:x}\n", my_struct2);
}

auto main() -> int
{
    test();
    return 0;
}

Output from console:
deserialized bitset: 33437cb87c1c5f19
deserialized int64: 195f1c7cb87c4333

Version info:
zpp_bits: latest version
compiler: gcc 14.2

Bitset underlying data is just an array of bytes, there should not be a byte order / endianness here, what am I missing?

Hi,
Thanks for the response.

Bitset underlying data is just an array of bytes, there should not be a byte order / endianness here

Did you actually "an array of bits" instead of "an array of bytes"? If we have an array of bytes, the byte order should matter.
But from the perspective of consistency, I think it would better if the deserialized std::bit_set<64>::to_ullong() has the same value as the unsigned long long.

No I literally meant that while being an array of bits, it stores the bits in some bytes (the underlying data) - which is why I referred to it an as array of bytes - and an array of bytes is a construct that does not have a byte order.

because it is an array of bytes there is never a byte order, how could there be - maybe a bitset of 64 bits is actually meant to store 8 individual bytes and never represent a multibyte integer, maybe it represents four 2-byte integers - there is no telling

I think std::bit_set is normally treated as a consecutive bit sequence instead of array of integers. Therefore, in the case of std::bit_set<64>, it should be treated as a multibyte integer instead of an array. Also from the standard, std::bit_set only has interfaces like to_ullong() or to_ulong(), which could mean that std::bit_set represents a value instead of an array of values. So like other literal types (size of >1 byte), std::bit_set de/serialization should be depending on an endianness specified by the user. For now, the convention is quite unintuitive, from the byte arrays of std::vector<char>{"0x10", "0x20", "0x30"}, the deserialized bitset is actually std::bitset<16>{"0x302010"}.

The same could be said about an std::array of bytes of course you can bitcast to a multibyte integer, but how could you assume the array of bytes is an integer of some size as opposed to just individual bytes

Sorry, I don't quite understand the ambiguity here. If I want to deserialize to a multibyte integer, I would do
uint64_t a{};
in(a); // With certain endian

If I want to deserialize to an array of bytes, I would do
auto byte_array = std::array<uint8_t, N>{}
in(byte_array); // No endianness

I don't quite know whether it's normal to represent a 4-byte integer using 4 chars instead of just uint32_t.

The same could be said about https://en.cppreference.com/w/cpp/utility/bitset/to_string
Would we want the string representation to change across the network when byte order is swapped?


[BUG]: CSM docs is having dead links

Bug Description
Some of the links in CSM docs are throwing 404 error
Logs
Screenshot attached
Screenshots

Additional Environment Information
No response
Steps to Reproduce

Expected Behavior

CSM Driver(s)

Installation Type
No response
Container Storage Modules Enabled
No response
Container Orchestrator
NA
Operating System
NA

/sync


fix issues with linting new test files

Which issue this PR addresses:

Fixes linting issues
What this PR does / why we need it:

This pr resolves CI build failures due to the new linter
Test plan for issue:

CI builds pass on the PR
Is there any documentation that needs to be updated for this PR?

no

Merging as changes are all in unit test files and will not impact e2e.


No tilemap can be created in multiple threads

Tested versions
v4.2.1.stable.official [b09f793f5]
System information
win
Issue description
I created a sub-thread in the main thread, created a scene in the sub-thread, and added a new tilemap by the scene. This is not allowed in godot. Although this will be rendered and loaded in the main scene, the console will Continuously reporting errors.
我在主线程创建了一个子线程，在子线程中创建了一个场景，由场景添加 新的tilemap ，这在godot中不被允许，虽然这在主场景中会被渲染和加载，但是控制台会不断报错。
Steps to reproduce

Minimal reproduction project (MRP)
tilemap_test.zip

Please try 4.2.2, this bug should be fixed there

I have tried using the latest version of godot, and it has no effect. This is the latest error picture.

I have tried using the latest version of godot, and it has no effect. This is the latest error picture.我已经尝试了使用最新版本的godot ，并且没有效果，这是目前最新的报错图

我尝试使用最新版本的 godot，但没有效果。This is the latest error picture.我已经尝试了使用最新版本的godot ，并且没有效果，这是目前最新的报错图

Oh my bad I couldn't read the error message since it wasn't English, that's something to handle yes, unsure how it should be handled

You can use Thread.set_thread_safety_checks_enabled(false) (it's unsafe though, so only use it when you know what you are doing).

You can use Thread.set_thread_safety_checks_enabled(false) (it's unsafe though, so only use it when you know what you are doing).
也就是说，这只是一个警告，并不是报错，我可以正常的使用他，那么我希望这个提示能够修改为警告，而不一个错误等级。
That is to say, this is just a warning, not an error. I can use it normally, so I hope this prompt can be changed to a warning instead of an error level.


Replace the users crate with sysinfo

The users crate that we currently use in our integration tests is unmaintained and security patches are not being applied. Ditch that crate in favour of sysinfo.
There is a direct fork of users called uzers, but that crate is not packaged in Fedora, and sysinfo is, so the change seems easier this way.

FWIW, the CoreOS folks are using uzers in their projects - https://github.com/search?q=org%3Acoreos uzers&type=code
Not sure how they are handling the lack of a Fedora package (probably vendoring?), but wanted to mention it.

They are vendoring, we can add extra packages, we just need to be cognizant of the extra work, I would sooner go to something that had a healthy upstream and have to package it rather than move to something that doesn't and have other issues.

...I would sooner go to something that had a healthy upstream...

Based on the "Insights" from GH on uzers and sysinfo, it does look like sysinfo is more active...so feels like it is the better choice on that metric.


ReadAll assumes string data

We will remove the readAll because it is not relevant for a file storage plugin


Ignore / Suppress specific magic number warning in detekt baseline.xml for a Compose function

Discussed in https://github.com/detekt/detekt/discussions/7274

Originally posted by talhahasanzia May  8, 2024
To ignore magic number in a class, we can do this:
    <ID>MagicNumber:FeatureManager.kt$FeatureManager$30</ID>

Where, FeatureManager is class, and 30 is magic number.
What would happen if this is in compose lets say:
@Composable
fun CustomComposable(){
    LaunchedEffect(Unit) {
        delay(3000) // this needs to be added to baseline
        ...
    }
}

I have tried multiple approaches, but still I am unable to suppress this.
Referencing direct function from file reference:
    <ID>MagicNumber:CustomComposable.kt$CustomComposable$3000</ID>

Referencing direct magic number from file reference:
    <ID>MagicNumber:CustomComposable.kt$3000</ID>

Referencing direct magic number from element (function) reference:
    <ID>MagicNumber:CustomComposable$3000</ID>

This is hit and trial. I know that already, that this is not the way to do it (not mentioned in the docs what to do either)
Only reference to read for Baseline I found is this detekt website

Answered on the Discussion


Red and Black Theme (Dark/Light mode toggle)

Hey, would it be possible to request a red and black theme and a red and white theme toggle? I've noticed there is green/black. Example: https://not-matthias.github.io/

I figured it out. I had to add the CSS file to the config.toml file. I was able to do this by adding the following to the config.toml file:
[params]
    customcss = ["css/main.css"]

The main.css content is: https://github.com/athul/archie/blob/master/assets/css/main.css and the file is located in assets/css/main.css at the root of the project.
Thanks.


⚠ Bofrost.lu has degraded performance

In d473895, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1813 ms

Resolved: Bofrost.lu performance has improved in f292351 after 49 minutes.


Support for multiple TXs when testing

Would be nice to have the ability to have tests perform more than 1 tx if needed. Some testing scenarios require this, and using vm.warp or vm.roll/rollFork haven't really worked in this way. Currently depending on hardhat for these things.
function testSomething() public {
  vm.sendTX(
    token.transfer(xxxx)
    market.borrow()
  )
  vm.sendTX(market.repayBorrow)
}

ping @Maurechi
have a feeling this is a misconception of what is available to the programmer - if i dont get a response in a couple more days going to close

Hey @brockelmore . I'm sory for the delay, currently under deadlines and haven't been able to come to this again.
I'll try to test thoroughly and see if I can find a foundry native way to fix my issue. If not i'll try and bring an easy to reproduce example and reopen the issue.


known security issue in dropwizard-jetty version 2.0.21 (sub-)dependency

Hi there,
please bump jetty to 2.34 or 3.0.2 or above as mentioned under Solution in the following security notice of nist:
ID: 3392a333f9525d590a865cc11b9b2fdbdb29714e128b9265cc1629b338783876
Severity: Medium
Name: Incorrect Permission Assignment for Critical Resource
Message: Incorrect Permission Assignment for Critical Resource in org.glassfish.jersey.core/jersey-common
Solution: Upgrade to versions 2.34, 3.0.2 or above.
In: code/build.gradle (org.glassfish.jersey.core/jersey-common:2.33)
Link: https://nvd.nist.gov/vuln/detail/CVE-2021-28168
thank you.

@mschwenke-up Thanks for reporting this! Unfortunately, this is not a straight forward dependency update.
Jersey 3.x is using the Jakarta EE namespace jakarta.* instead of javax.*, so we won't upgrade to Jersey 3.x in Dropwizard 2.x because we'd also need to upgrade every single dependency or integration with Jersey as well. This includes at least Jetty, Hibernate Validator, and Jackson.
As for Jersey 2.34, it introduced its own provider for Optional<T> parameters which has a different behavior than the one used in Dropwizard since Dropwizard 1.0.0 (https://github.com/dropwizard/dropwizard/pull/1365).
This has been outlined in https://github.com/eclipse-ee4j/jersey/issues/4651#issuecomment-826080312 (and following comments) and https://github.com/eclipse-ee4j/jersey/issues/4790 and it would break existing users of Dropwizard who are using Optional<T> parameters in query parameters, headers, or cookies.
The advisory at https://github.com/eclipse-ee4j/jersey/security/advisories/GHSA-c43q-5hpj-4crv describes a workaround by defining an application-specific temporary directory via the java.io.tmpdir system property, which I would recommend doing for the time being.
@dropwizard/committers Any opinions on breaking existing consumers (for Optional<T> parameters) with Jersey 2.34+?

Unless we can define the tmpdir system property within the framework somehow, I think security issues take precedence over backward compatibility breaks. Assuming we can document it well enough.

In my opinion the Optional<T> parameter converter in Jersey 2.34 (and Jersey 3.0.1) is seriously broken while this security issue can be easily mitigated for existing users.
Next issue: The Optional<T> parameter converter doesn't support custom types (https://github.com/eclipse-ee4j/jersey/issues/4798).

Resolving vulnerabilities and we're getting flagged for jersey-commons:2.33 as well via Dropwizard, any plans to upgrade to 2.34 by any chance?

@fliden The comments from https://github.com/dropwizard/dropwizard/issues/3969#issuecomment-841816169 still apply and Jersey 2.35 hasn't been released yet.

I noticed that Jersey 2.35 has been released.


it is very slow to scan samples of detection dataset

Passing the class function to the multi-process pool causes slow operation. The class function should be extract from class into independent function.


Crashing while Using OneDrive Folder

Explain the bug
Write a clear description of what the bug is.
Files crashes randomly while using the OneDrive Folder.
To reproduce

Go to OneDrive Folder
Use OneDrive as you would. Copy, cut, paste, move, go forwards/backwards, open files, anything. After a couple of minutes it might crash!

Expected behavior
Write a clear description of what you expected.
I expected it not to crash.
debug_fulltrust.txt
debug.txt
Logfile
Upload the log file here for us to understand your issue better.

You can access it from Settings->About->Open log location.
or
Copy the debug.txt and debug_fulltrust.txt from the following directory, and upload the files here.
%localappdata%\Packages\49306atecsolution.FilesUWP_et10x9a9vyk8t\LocalState

Screenshots
Add screenshots to help explain your problem if needed.
System Information

OS Version: [e.g. Windows 10 20H2 19042.804] (Press WIN + PAUSE/BREAK, use the copy button at the bottom)
App version: [e.g. v1.0] (In the app, go to Settings -> About, the version is shown at the top)

Additional comment
Add any other comment about the problem here. Does this problem happen again after restarting the app?

@oOII-J4HN-IIOo is this still an issue in Files v2.0?

@oOII-J4HN-IIOo is this still an issue in Files v2.0?

I don't think so.

@oOII-J4HN-IIOo thanks for the update.


start on docs update for labels

fixes #7550
This is not done - but a start to discuss and iterate on
/do-not-merge
NOTE!
The updated labels actually have mutable values - won't this be a problem when used with deployment api v1? As discussed in the meeting?

/do-not-merge

/assign @mattfarina - care to comment?

/hold
use hold to apply the do not merge label

Should we reopen this and finish it?

This LGTM,
@davidkarlsen have you had a chance to test the mutating label issue?


Re-use asset building script in Dockerfile

Let's try to reuse bootstrap.sh in the Dockerfile to avoid multiple asset generation code. This gets confusing and out of sync quickly has been my experience.

I think the idea is to not have any scripts, and perform all actions from the CLI. In the next release we should move all those to invenio-cli. Some reasons I can think about are:

Easier upgrade, its quite difficult to upgrade scripts, people might have their custom code there...
If we do all programatically we have more control over what happens and code will be cleaner (e.g. this issue would not exists, since we probably would only have one function for such functionality)
See also: https://github.com/inveniosoftware/invenio-cli/issues/24

I'd vote to open an issue in invenio-cli such as move all scripts content to cli python code and close this one. What do you think?

Moving all scripts to one location (invenio-cli) sounds good to me. The process would then be for the developer:
1- install invenio-cli
2- use that to setup  the assets
3- use that to build the images which would copy the above generated assets
We will just have to be clear on the order of commands.

Perfect. AFAIK Invenio in general wants to get away of bash scripts cuz they are messy to keep up to date without forcing users to "pull" again a repo (which is not the idea).
Closing this ticket? I can create a new one to "migrate scripts to code"

Good for closing and creating a new one.


vmware adapter on vSphere 6.5: Accessinfo IP address not returned

[Version]
V17.0.2
[Date]
2017/06/08
[Test category]
Creative
[Description]
When deploying an instance on vSphere server 6.5 the access info for the IP address is not returned from vSphere:

[Details]
See above;
settings in subscription:

[Messages and logs]
app-vmware.log entry:
[2017-06-22 17:32:26] [__ejb-thread-pool14] DEBUG vcenter: estvcsadev.intern.est.fujitsu.com
[2017-06-22 17:32:26] [__ejb-thread-pool14] DEBUG locale: en
[2017-06-22 17:32:26] [__ejb-thread-pool14] DEBUG Generated access information for service instance 'Blubber3':
Blubber3,
[2017-06-22 17:32:26] [__ejb-thread-pool14] DEBUG Transition from current state 'END' with event 'success' into state 'SUCCESS'

Test with 1 network adapter with DHCP option. The adapter name is mandatory in this case ("VM Network").
The IP address is then available.

Test with 1 network adapter with Database option. The adapter name and all necessary settings are read from database (data service of VMWare controller is corrected).


com.sun.proxy.$Proxy548 cannot be cast

I ran into this error:
com.sun.proxy.$Proxy548 cannot be cast to org.springframework.cloud.bootstrap.config.PropertySourceLocator
I already added this class into reflect config but not sure why I got this error. I attached everything in the zip.
profile.zip

Cannot be cast doesn't sound like a configuration issue.  The startup log in the zip file just seems to show the problem from earlier in the week:
org.springframework.beans.factory.BeanDefinitionStoreException: Failed to parse configuration class [com.inspira.hub.profile.ProfileServer]; nested exception is org.springframework.core.annotation.AnnotationConfigurationException: A
ttribute 'value' in annotation [org.springframework.web.bind.annotation.RestController] is declared as an @AliasFor nonexistent attribute 'value' in annotation [org.springframework.stereotype.Controller].

I don't see an error about casting. I won't be able to make much progress without a sample app that shows something broken to debug. We don't have feign/sleuth/eureka/spring-social-facebook/mapstruct/springfox/firebase samples, so the failures can be due to anything in any of those.
For something we don't have samples for it is worth trying with the agent to collect configuration. I can help you with that if you don't know how to do it.

@aclement I am sorry, the startup log in the zip is wrong.
I did use agent to generate the configs that are included in the zip. The casting error popup both when I am using agent or not.
Here is the log for when I was runnning it with the agent.
log.txt

I see the new log, thanks for that. This is really hard to diagnose when I can't recreate though. Given it is occurring in startup, maybe we could have a test app which is just that pom plus an empty SpringApplication, plus the config you are using that is exercising the property source infrastructure (if any).

ok I'll create an empty app to see if I can reproduce it and send it to you. But I am not sure what property source infrastructure is. Can you give any names as an example?

Hi @aclement, here is a demo spring cloud app. All I do is run java -jar -Dspring.profiles.active=native target/config-server.jar to start config server on JVM. Make sure you fix the config storage path. Although, demoApp won't build due to the same issue as the log I attached above.
demo.zip

Thanks for the app, I'll try it out. In working on something else today I think I know what it might be - I think it'll be that we can't create CGLIB proxies under native-image, only JDK proxies and in some cases those are not sufficient, I think this might be one of those cases.

do you know if there is anything i can do to fix this?

Haven't had time to dig into the app yet - it is possible JDK proxies could work if interfaces are added to a class but without knowing the cause I can't say where that pattern needs applying. We will fix the overall issue here in time, I just don't have a quick option at the moment.

ok thanks I will look forward to it.

@VuDo98 Please make a try with our latest 0.8.3-SNAPSHOT and GraalVM 20.3, and if you still see the issue please provide a minimal repro, the current one has too much dependency for making it useful for us to help you.

Hi @sdeleuze I am still seeing this error with the same previous one with spring graalvm 0.8.3, graalvm 20.3, and spring cloud deps 2020.0.0-SNAPSHOT follow the example in samples. Log (1) is attached. I am not sure if I can reproduce this on a minimal because I tried to run a minimal config on the discovery client sample but it gave me error like log (2).
log 1.txt
log 2.txt


Alternatives for long callback URLs

I understand that it is a common problem that some browsers cap the URL length (e.g. IE Edge). We have now shaved client-ids, scope names and moved all claims to come through the userinfo endpoint. Our callback URL is about 2090 chars right now (I have read that 2048 might be a good ballpark number to try to stay below).
We looked at changing our access_token to a reference token in IdentityServer3, but we are worried that the reference token store is not distributed between IdSrv nodes in our cluster. We do not want to move away from using a round robin algorithm in our load balancer. We are also unsure about the performance loss by using a reference token.
It would seem like making the callback use POST instead of GET, as described in the optional part of the Openid Connect spec could be a good alternative (http://openid.net/specs/oauth-v2-form-post-response-mode-1_0.html).
Is this something you would consider implementing in this library?
Are there other alternatives that we can do in the IdentityServer part?
I am sorry this got a bit mixed between IdSrv and openid-client-js. Yell at me if you want me to split it up.

POST is designed for server-side code, so if you want that approach then use a server side framework for OIDC, not this library.

Brainfart right there. The SPA can't read the POST body, obviously.
Any initial thoughts around the reference token before closing this, or should I take that to IdentityServer?

Yea, ask about ref tokens over there.


Rb-E2F complex

A new term request:
transcription factor complex ; GO:0005667
--%Rb-E2F complex ; GO:NEW ; synonym:retinoblastoma-E2F
complex
Rb-E2F complex ; GO:NEW
exact synonym: retinoblastoma-E2F complex
A mammalian multiprotein complex containing a
heterodimeric E2F transcription factor and a
Retinoblastoma (Rb) family member. This complex is
capable of repressing transcription of E2F-regulated
genes in order to regulate cell cycle progression.
PMID:14616073
will add in on Fri (June 11th) if there's no objections.
thanks,
Becky
Reported by: beckyfoulger
Original Ticket: "geneontology/ontology-requests/1785":https://sourceforge.net/p/geneontology/ontology-requests/1785

thanks www.movied.org


Add gas price oracle values to CLI arguments

Describe the feature
gas price orcale introduced here https://github.com/paradigmxyz/reth/pull/2600
we need to make this configurable via CLI
TODO:

check geth cli for compatibility, see (gpo.*) values
add new Args struct for GPO
integrate with rpc-builder in rpc:

need new gpo config values here:
https://github.com/paradigmxyz/reth/blob/92b557d84e6b39a695ca7e205e51dfa61ffd429d/crates/rpc/rpc-builder/src/eth.rs#L23-L25
and integrate in CLI here:
https://github.com/paradigmxyz/reth/blob/92b557d84e6b39a695ca7e205e51dfa61ffd429d/bin/reth/src/args/rpc_server_args.rs#L296-L297
Additional context
No response

I will fix this