Python cannot be found on unsupported architectures

On unsupported architectures (e.g. riscv64), uv cannot find the local Python because the machine is not in a list of known items:
uv output
# uv python find --verbose
DEBUG uv 0.4.19
DEBUG Searching for default Python interpreter in managed installations or system path
DEBUG Searching for managed installations at `/root/.local/share/uv/python`
DEBUG Failed to inspect Python interpreter from search path at `/usr/bin/python3`
DEBUG Skipping bad interpreter at /usr/bin/python3 from search path: Querying Python at `/usr/bin/python3` did not return the expected data
unknown variant `riscv64`, expected one of `aarch64`, `armv6l`, `armv7l`, `powerpc64le`, `powerpc64`, `x86`, `x86_64`, `s390x`
--- stdout:
{"result": "success", "markers": {"implementation_name": "cpython", "implementation_version": "3.12.3", "os_name": "posix","platform_machine": "riscv64", "platform_python_implementation": "CPython", "platform_release": "6.11.0-8-generic", "platform_system": "Linux", "platform_version": "#8.1-Ubuntu SMP PREEMPT_DYNAMIC Tue Oct  1 11:40:56 UTC 2024", "python_full_version": "3.12.3", "python_version": "3.12", "sys_platform": "linux"}, "sys_base_prefix": "/usr", "sys_base_exec_prefix": "/usr", "sys_prefix": "/usr", "sys_base_executable": "/usr/bin/python3", "sys_executable": "/usr/bin/python3", "sys_path": ["/root/.cache/uv/.tmpvWackB", "/usr/lib/python312.zip", "/usr/lib/python3.12", "/usr/lib/python3.12/lib-dynload", "/usr/local/lib/python3.12/dist-packages", "/usr/lib/python3/dist-packages"], "stdlib": "/usr/lib/python3.12", "scheme": {"platlib": "/usr/local/lib/python3.12/dist-packages", "purelib": "/usr/local/lib/python3.12/dist-packages", "include": "/usr/include/python3.12", "scripts": "/usr/local/bin", "data": "/usr/local"}, "virtualenv": {"purelib": "lib/python3.12/site-packages", "platlib": "lib/python3.12/site-packages", "include": "include/site/python3.12", "scripts": "bin", "data": ""}, "platform": {"os": {"name": "manylinux", "major": 2, "minor": 39}, "arch": "riscv64"}, "manylinux_compatible": true, "gil_disabled": false, "pointer_size": "64"}
--- stderr:

---
DEBUG Failed to inspect Python interpreter from search path at `/usr/bin/python3.12`
DEBUG Skipping bad interpreter at /usr/bin/python3.12 from search path: Querying Python at `/usr/bin/python3.12` did not return the expected data
unknown variant `riscv64`, expected one of `aarch64`, `armv6l`, `armv7l`, `powerpc64le`, `powerpc64`, `x86`, `x86_64`, `s390x`
--- stdout:
{"result": "success", "markers": {"implementation_name": "cpython", "implementation_version": "3.12.3", "os_name": "posix","platform_machine": "riscv64", "platform_python_implementation": "CPython", "platform_release": "6.11.0-8-generic", "platform_system": "Linux", "platform_version": "#8.1-Ubuntu SMP PREEMPT_DYNAMIC Tue Oct  1 11:40:56 UTC 2024", "python_full_version": "3.12.3", "python_version": "3.12", "sys_platform": "linux"}, "sys_base_prefix": "/usr", "sys_base_exec_prefix": "/usr", "sys_prefix": "/usr", "sys_base_executable": "/usr/bin/python3.12", "sys_executable": "/usr/bin/python3.12", "sys_path": ["/root/.cache/uv/.tmp84TuZT", "/usr/lib/python312.zip", "/usr/lib/python3.12", "/usr/lib/python3.12/lib-dynload", "/usr/local/lib/python3.12/dist-packages", "/usr/lib/python3/dist-packages"], "stdlib": "/usr/lib/python3.12", "scheme": {"platlib": "/usr/local/lib/python3.12/dist-packages", "purelib": "/usr/local/lib/python3.12/dist-packages", "include": "/usr/include/python3.12", "scripts": "/usr/local/bin", "data": "/usr/local"}, "virtualenv": {"purelib": "lib/python3.12/site-packages", "platlib": "lib/python3.12/site-packages", "include": "include/site/python3.12", "scripts": "bin", "data": ""}, "platform": {"os": {"name": "manylinux", "major": 2, "minor": 39}, "arch": "riscv64"}, "manylinux_compatible": true, "gil_disabled": false, "pointer_size": "64"}
--- stderr:

---
DEBUG Failed to inspect Python interpreter from search path at `/bin/python3`
DEBUG Skipping bad interpreter at /bin/python3 from search path: Querying Python at `/bin/python3` did not return the expected data
unknown variant `riscv64`, expected one of `aarch64`, `armv6l`, `armv7l`, `powerpc64le`, `powerpc64`, `x86`, `x86_64`, `s390x`
--- stdout:
{"result": "success", "markers": {"implementation_name": "cpython", "implementation_version": "3.12.3", "os_name": "posix","platform_machine": "riscv64", "platform_python_implementation": "CPython", "platform_release": "6.11.0-8-generic", "platform_system": "Linux", "platform_version": "#8.1-Ubuntu SMP PREEMPT_DYNAMIC Tue Oct  1 11:40:56 UTC 2024", "python_full_version": "3.12.3", "python_version": "3.12", "sys_platform": "linux"}, "sys_base_prefix": "/usr", "sys_base_exec_prefix": "/usr", "sys_prefix": "/usr", "sys_base_executable": "/bin/python3", "sys_executable": "/bin/python3", "sys_path": ["/root/.cache/uv/.tmpxChyuH", "/usr/lib/python312.zip", "/usr/lib/python3.12", "/usr/lib/python3.12/lib-dynload", "/usr/local/lib/python3.12/dist-packages", "/usr/lib/python3/dist-packages"], "stdlib": "/usr/lib/python3.12", "scheme": {"platlib": "/usr/local/lib/python3.12/dist-packages", "purelib": "/usr/local/lib/python3.12/dist-packages", "include": "/usr/include/python3.12", "scripts": "/usr/local/bin", "data": "/usr/local"}, "virtualenv": {"purelib": "lib/python3.12/site-packages", "platlib": "lib/python3.12/site-packages", "include": "include/site/python3.12", "scripts": "bin", "data": ""}, "platform": {"os": {"name": "manylinux", "major": 2, "minor": 39}, "arch": "riscv64"}, "manylinux_compatible": true, "gil_disabled": false, "pointer_size": "64"}
--- stderr:

---
DEBUG Failed to inspect Python interpreter from search path at `/bin/python3.12`
DEBUG Skipping bad interpreter at /bin/python3.12 from search path: Querying Python at `/bin/python3.12` did not return theexpected data
unknown variant `riscv64`, expected one of `aarch64`, `armv6l`, `armv7l`, `powerpc64le`, `powerpc64`, `x86`, `x86_64`, `s390x`
--- stdout:
{"result": "success", "markers": {"implementation_name": "cpython", "implementation_version": "3.12.3", "os_name": "posix","platform_machine": "riscv64", "platform_python_implementation": "CPython", "platform_release": "6.11.0-8-generic", "platform_system": "Linux", "platform_version": "#8.1-Ubuntu SMP PREEMPT_DYNAMIC Tue Oct  1 11:40:56 UTC 2024", "python_full_version": "3.12.3", "python_version": "3.12", "sys_platform": "linux"}, "sys_base_prefix": "/usr", "sys_base_exec_prefix": "/usr", "sys_prefix": "/usr", "sys_base_executable": "/bin/python3.12", "sys_executable": "/bin/python3.12", "sys_path": ["/root/.cache/uv/.tmpzRiSA8", "/usr/lib/python312.zip", "/usr/lib/python3.12", "/usr/lib/python3.12/lib-dynload", "/usr/local/lib/python3.12/dist-packages", "/usr/lib/python3/dist-packages"], "stdlib": "/usr/lib/python3.12", "scheme": {"platlib": "/usr/local/lib/python3.12/dist-packages", "purelib": "/usr/local/lib/python3.12/dist-packages", "include": "/usr/include/python3.12", "scripts": "/usr/local/bin", "data": "/usr/local"}, "virtualenv": {"purelib": "lib/python3.12/site-packages", "platlib": "lib/python3.12/site-packages", "include": "include/site/python3.12", "scripts": "bin", "data": ""}, "platform": {"os": {"name": "manylinux", "major": 2, "minor": 39}, "arch": "riscv64"}, "manylinux_compatible": true, "gil_disabled": false,"pointer_size": "64"}
--- stderr:

---
error: No interpreter found in virtual environments, managed installations, or system path

If the platform_machine of a local Python matches the machine type from uname, uv should also accept it.

Thanks for the report! It seems fine to just add this to the list.

Added in https://github.com/astral-sh/uv/pull/8660

Thank you so much!


Windows Explorer crashes after interacting with the tray icon

when you click the icon twice and then try to click on a window, explorer restarts. probably because the task view was already open or something.
Originally posted by @ayunami2000 in https://github.com/reserfodium/VirtualDesktopIndicator/issues/1#issuecomment-540755066

@ayunami2000 I could not reproduce this bug, it does what it is intended to do when I press on the traybar icon. I am on the latest Windows Version 1903.
What are your Computer Sepcs?

Windows 10 Home Version 1903
NVIDIA & Intel (50% of the time every issue with this laptop is because of the two graphics cards)
Also I think I had clicked on the window almost immediately after clicking on the icon in the taskbar. May have had something to do with it

Hi everyone! I am the maintainer of this repository - reserfodium. But I lost access to this account. Thank you very much for your interest. Development continues in this repository. Check out latest release, I fixed some bugs and add new features.


Linear interpolation doesn't preserve gradients

Demo Slope.txt
Hi,
This is an amazing bit of work, thank you for releasing it here. Can you please help with this problem I found when interpolating using it?
I have attached a file (please rename .txt. to .gpx). This is a smooth gradient of 7.2%
When I interpolate using option 1 (linear), it does not linearly interpolate the elevation - so the elevation becomes sawtooth with slopes of 7.2, 7.2, 7.1, 7.2, 7.2, 7.1, 7.2, 7.2, 7.1 and so on.
Is there a way to retain the slope (in this case 7.2%) throughout please?
This is an artificial scenario, but the real scenario is a hill with multiple points with uneven distance between points where I want to interpolate between points without messing up the gradients, because riding the route on a turbo trainer needs smooth gradients without changing (the particular software I use, requires close to evenly spaced points and doesn't allow smoothing in a manner that would help, so it has to be perfect to begin with).

Thank you, but sadly, that is effectively the same result. Yes, it happens with any number of points, I chose two to illustrate the effect.

Uploading Test Hill 8 points-fixed.txt…
I've attached a better example file. 8 points with 7 linear gradients.

The code in my last answer does fix your problem. Here is what it looks like on your first example:

How Strange! What am I doing wrong?
GPX_interpolate.txt

Ah for me:
Commenting out all these solves it:
# remove insignificant digits
# lat_interp = np.round(lat_interp1e6)/1e6
# lon_interp = np.round(lon_interp1e6)/1e6
# ele_interp = np.round(ele_interp1e1)/1e1
OR
# remove insignificant digits
# lat_interp = np.round(lat_interp1e6)/1e6
# lon_interp = np.round(lon_interp1e6)/1e6
ele_interp = np.round(ele_interp1e6)/1e6
Thanks!

This problem is  back with the new release! This was originally 2 points interpolated -d 1 -r 1. If linear interpolation, it should be a straight line between the two.

Can you share you GPX file? I can't reproduce the issue with the last version of the code and your Demo_Slope.gpx file.

I'm very sorry, I think it was due to rounding in the application I used
afterwards. I'm so sorry for the trouble.
Jonathan
Sent from Mail https://go.microsoft.com/fwlink/?LinkId=550986 for Windows
10
*From: *Remi Salmon<EMAIL_ADDRESS>*Sent: *04 July 2020 16:44
*To: *remisalmon/gpx_interpolate<EMAIL_ADDRESS>*Cc: *jonathancolledge<EMAIL_ADDRESS>Author
<EMAIL_ADDRESS>*Subject: *Re: [remisalmon/gpx_interpolate] Linear interpolation doesn't
preserve gradients (#3)
Can you share you GPX file? I can't reproduce the issue with the last
version of the code and your Demo_Slope.gpx file.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/remisalmon/gpx_interpolate/issues/3#issuecomment-653781110,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBEFAEQMLYVUL73Z7Y6WCTRZ5E7VANCNFSM4KNLYOOQ
.

No problem, thanks for following up.


change fraud translation

Coverage remained the same at 90.296% when pulling 4df6451554b2b83417ba7a57a0315e2958a5d37f on dictionaries_fraud into 8b6ce2d6e4a2c4e9ae6293ee989a890405aa2adf on develop.


The pictures on the official website are not displayed.

Screenshots

Thanks for the feedback, the official website images will be uniformly migrated to oss storage in the near future.

The problem still persists, when can it be fixed?


@conda-forge-admin, please add user @rudolfix

Additional comment:
No response

I thihnk this might work @sultaniman

Thank you Wolf!


Unable to fetch by key on Android

Please specify your environment :
Plugin version:

[x] released version: 0.99.0
[x] repository master

Toolchain:

[ ] Cordova cli
[ ] Phonegap cli
[ ] Phonegap cloud
[x] Ionic
[ ] Other: _____

Platforms affected:

[x] Android
[ ] iOS/macOS
[ ] LocalStorage fallback for browser and blackberry
[ ] Windows and Windows Phone 8.1 and later
[ ] Windows Phone 8 and earlier (deprecated)

What the scope of your problem:

[x] General functionality (store/fetch/remove/clearAll)
[ ] Suites
[ ] Cloud synchronization and events

Please describe the issue:
fetch is always returning null in android. It never makes it inside of this if statement(line:204 AppPreferences.java):
if (sharedPrefs.contains(key)) {
Even though the keys do exist, it is unable to find them in sharedPrefs.
Tested with both cordova 5.4.0 and 6.1.1, and Android 4.4.2 and 5.1.1.

Same as #100


pythonPackages.mpld3: init at 0.3

Motivation for this change
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p 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.

Tests are currently disabled as they require access to an X display.

@worldofpeace my bad. Fixed now.

It's actually possible to get the tests running by encouraging matplotlib to use e.g. the ps backend something like:
  checkInputs = [ nose ];
  checkPhase = ''
    export HOME=$TMPDIR
    mkdir -p $HOME/.config/matplotlib
    echo "backend: ps" > $HOME/.config/matplotlib/matplotlibrc
    ln -s $HOME/.config/matplotlib $HOME/.matplotlib

    nosetests mpld3
  '';

However a number of these tests genuinely fail and I'm not totally sure about the compatibility of this release with our current matplotlib and numpy. Do you find this package works well enough for you?

I mostly got to this package as a dependency of https://github.com/NixOS/nixpkgs/pull/70458 .
On my system the package works but is quirky. Hard to say if due to NixOS or the package itself as there are several open issues on the official issue tracker. Version 0.3 was released in 2016 so, a bit old. The repository is still being somewhat maintained so hopefully a new release will happen at some point in the near future.
Tests are also failing in the official repository so these might be upstream issues.

If we're referring to the same thing, the "tests failing in the main repository" look like they just haven't set up travis properly. But yeah, it seems they haven't really kept up with dependencies and with nixpkgs insistence on "one version of everything" and very recent versions, this causes trouble. If you're a competent python programmer, you can always fork the project & fix it of course...

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/pytest-bdd-on-shell-nix/4616/4

@grahamcofborg build pythonPackages.mpld3

Closing due to inactivity from author.


make_unique used without including

I'm getting this compiler error when using Menu:
In file included from /home/runner/work/calchart/calchart/build/_deps/wxui-src/include/wxUI/wxUI.h:43,
                 from /home/runner/work/calchart/calchart/src/AnimationPanel.h:26,
                 from /home/runner/work/calchart/calchart/src/AnimationPanel.cpp:23:
/home/runner/work/calchart/calchart/build/_deps/wxui-src/include/wxUI/Menu.h:280:44: error: expected primary-expression before ‘>’ token
  280 |         auto menu = std::make_unique<wxMenu>();
      |                                            ^
/home/runner/work/calchart/calchart/build/_deps/wxui-src/include/wxUI/Menu.h:280:46: error: expected primary-expression before ‘)’ token
  280 |         auto menu = std::make_unique<wxMenu>();
      |                                              ^

we need to include  before we use make_unique.

Fixed in #119


feat: add Writer::abort method

This PR:

adds oio::Write::abort API to allow cancellation of an on-going append writer
for those writers do not support append, adds a Write::abort implementation that simply return Ok(())
adds delegated implementations of oio::Write::abort for all layers
adds oio::Write::abort implementation for S3Writer as an example

This PR does not add an equivalent API in oio::BlockingWrite, since we can always add a Drop implementation for blocking writers and delete partial objects in Drop::drop.
Related issue
#1607

Hi, @v0y4g3r, is this PR ready for review? Do you want to add abort API first and then add support for different services?

Maybe I can add S3 implementation along with the Writer::abort definition in this PR so that any one who is interested in that can involve in.

LGTM!

I'm wondering if we can add a service test for Write::abort?

I'm wondering if we can add a service test for Write::abort?

Yes, we can add abort in behavior test (by ignoring not support errors)! This is PR good to get merged. How about adding a new PR for that?


⚠ Cioos National CKAN has degraded performance

In 89e8223, Cioos National CKAN (https://catalogue.cioos.ca/) experienced degraded performance:

HTTP code: 200
Response time: 5304 ms

Resolved: Cioos National CKAN performance has improved in 579f86c after 29 minutes.


Hypervisor_type option required in bosh_openstack_cpi

Dear Team,
I am using qemu as a hypervisor in Openstack. I am unable to launch stemcell because for stemcell hypervisor_type is kvm. I am getting not enough host available. While I am able to launch instance manually. Please do needful for this. If possible try to put this option in cpi.yml, So we can change it.
Thanks & Regards:
Arpit Sharma

Hi @arpitvipulsharma
The hypervisor_type attribute is taken from the stemcell and simply passed to OpenStack by the CPI without any processing. One thing you could try is to repack the stemcell using the bosh-cli, keep the same name and os version and just modify the cloud properties so that hypervisor_type is set to qemu you can follow the documentation here: https://bosh.io/docs/repack-stemcell.html. An alternative if you are not using the bosh-cli v2 then you could simply extract the stemcell and modify the property and then re-package the stemcell.
A question from me to you would be, do you have more than one hypervisor on that machine? And also could you provide the OpenStack logs where you see the error, so that we can get a clearer picture of what is going on.
Thanks!

Hi mauromorales! Thanks for your response. Issue solved. I have done by repacking of stemcell. I followed link which you shared.
I was using only one hypervisor with QEMU. In nova schedular logs, I was getting image filter errors.
Thanks to all for your support.

@arpitvipulsharma thank you for reaching out and glad I was able to help!
Closing fixed


Define closure criteria for issues and discussions

We know we have a backlog problem.  Right now, the focus is on closing very old and stale issues, particularly ones that are out of date or never got any sort of traction.
We also often close things that are out of scope, or are clearly not going to be addressed because of some design principle of OpenAPI.  It would be good to define and document as many of these as we can so we can reference the criteria, and close more things with confidence.
Ideally, if we are going to continue with both issues and discussions, this should include criteria for migrating from one to the other.  Loosely speaking, issues are good for things that can be clearly mapped to actions, while discussions are good for exploring options when the topic is complex, or when it's not obvious which of several possible actions we might want to take, if any.
Issue #3508 regarding transferring issues/discussions to Moonwalk is somewhat related.
Some things I think are good reasons to close (just the first few that came to mind):

JSON Schema concerns, including new proposed extensions, are out of scope for OpenAPI 3.1+
Proposals that limit what sort of HTTP APIs can be described go against the "big tent" principle
Proposals that are too oriented towards either strictly or loosely typed languages, at the expense of the other category, also go against the "big tent" principle

These would join other obvious categories like "Questions about tools do not belong in the spec repo."

Sounds, good @handrews.  @darrelmiller , @lornajane , @earth2marsh  - kinda feel powerless, since I can't even label any issues.  Would be nice to be able to label issues that could be (should be) closed.

@miqui once the new TSC members are in place, they'll look at expanding @OAI/triage which is the group that can do those things (and close issues, actually - just can't merge PRs).

Specific questions to consider:

What JSON Schema-related requests, if any, remain in-scope for the project?  And can we close all the rest as out-of-scope?
Resolve other OAI scope questions and document the decisions

Copying @lornajane 's comment from https://github.com/OAI/OpenAPI-Specification/issues/3614#issuecomment-1984137500:

We should start new ideas off as discussions on this repository, once general agreement is reached, those ideas can become a proposal in a pull request. The proposal has a formalised format, which means we can then use it to make changes to the specification, so it's a good outcome. The idea is to have something that just needs polishing once it's in proposal format.

Should we now move issues that feel more open-ended to discussions?  Should we move more focused discussions to issues?  And what are we doing with all the community support stuff?

I think we need to put out guidance before we start moving things, in case that makes someone feel they are "doing it wrong" because we rewrote the unwritten processes.
I've opened #3673 to try to formalise the proposal part of the process which should help to get ideas going to a new place. If usage questions should be discussions on the community repo (or issues, or somewhere else), we need to make that very clear both here and wherever they are going instead, and direct answerers as well as questioners to mingle in that space.

Added to the CONTRIBUTING file https://github.com/OAI/OpenAPI-Specification/blob/main/CONTRIBUTING.md#issues


PHP INTL extension

Hi,
Because some packages requires INTL extension, this PR installs it to PHP images.
Best regards :)

Thanks :)


Trailing space option

Closes #4

@ianwalter Any plans to fix the linting error? Also, moving husky should be a separate PR.

Pull Request Test Coverage Report for Build 54

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.7%) to 98.921%

Totals

Change from base Build 37:
-0.7%

Covered Lines:
187

Relevant Lines:
187

💛  - Coveralls


[Event Request] Function InsertAsmHeader in Codeunit 99000813 "Carry Out Action"

Dear Team, we would like to have the following Publishers added to Function InsertAsmHeader in Codeunit 99000813 "Carry Out Action":
procedure InsertAsmHeader(ReqLine: Record "Requisition Line"; var AsmHeader: Record "Assembly Header")
var
    [...]
begin
    Item.Get(ReqLine."No.");
    AsmHeader.Init();
    AsmHeader."Document Type" := AsmHeader."Document Type"::Order;
    // START NEW EVENT 1
    OnBeforeAsmHeaderInsert(AsmHeader, ReqLine);
    // END NEW EVENT 1
    AsmHeader.Insert(true);
    // START NEW EVENT 2
    OnAfterAsmHeaderInsert(AsmHeader, ReqLine);
    // END NEW EVENT 2
    AsmHeader.SetWarningsOff;
    AsmHeader.Validate("Item No.", ReqLine."No.");
    AsmHeader.Validate("Unit of Measure Code", ReqLine."Unit of Measure Code");
    [...]
end;

[IntegrationEvent(false, false)]
local procedure OnBeforeAsmHeaderInsert(var AsmHeader: Record "Assembly Header"; ReqLine: Record "Requisition Line")
begin
end;

[IntegrationEvent(false, false)]
local procedure OnAfterAsmHeaderInsert(var AsmHeader: Record "Assembly Header"; ReqLine: Record "Requisition Line")
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


GH-44054: [C++] Implement PostgreSQL dedicated date extracting function

Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?
Add functions in temporal component extraction for processing PG's DATA/TIMESTAMP data type.

@ZhangHuiGui would you mind take a look?

Could you fix CI failures?
You can check CI results on your fork too: https://github.com/marsno1/arrow/actions


Redirect to pages like YouTube with ?=

Please add a function to redirect to pages with an URL like. youtube.com/watch?v=IiO...
(Question Mark and Equal sign)

you can do a HTML/PHP redirect else you can use .opts-slash. on youtu.be like youtu.be.opts-slash.[insert video id].redirect.center . Sorry for bad english but i'm italian

@timhaettich You can now use URL parameters, thanks to @mike1808


feature request: make canOnlySwipeFromEdge non-final

Can you also ensure that canOnlySwipeFromEdge is not final? So that it can be changed dynamically as well?
It would create a better UX when using in combination with TabBarView, so that when you swipe from edge, it goes back, otherwise it swipes inside the TabBarView.
Related: https://github.com/JonasWanke/swipeable_page_route/issues/8

Sorry, I missed your other comment. canOnlySwipeFromEdge is now mutable as part of v0.4.0, as are backGestureDetectionWidth and backGestureDetectionStartOffset.

Super sweet!


Switch Netlify to use Ubuntu Focal 20.04 build image, redux

This is a Feature Request

What would you like to be added
Adjust Netlify to build https://kubernetes.io/ using a worker running Ubuntu 20.04 (currently: Ubuntu 16.04)
Why is this needed
The march of progress.
Comments
This change happens in Netlify, not in Git.
Eventually Ubuntu 16.04 will go fully out of support.
Previously attempted in #27948

Insta-freezing this as we do need to do it eventually:
/lifecycle frozen

We can update https://app.netlify.com/sites/kubernetes-io-vnext-staging/ as a way to test this (for a short period, then either revert or also update the main site).

/priority important-longterm

/triage accepted

Testing kubernetes-io-vnext-staging:
https://app.netlify.com/sites/kubernetes-io-vnext-staging/deploys/61db3b7af43444884a1ee03d

LGTM.
Applying to live site.

Also LGTM.
Changes made.
/close


Commonize special polynomials

We currently prioritize each model being independent and self-contained. Some orthogonal polynomials can be made common. Reconsider whether the code should be common before updating to v1.0.0. This issue is related to https://github.com/ohno/Antique.jl/issues/49.

I think the same holds true also for the spherical harmonics. All (spherical symmetric) 3D problems they can use the same common function.

I gave priority to that all member functions are self-contained in each file.
Also, Commonization of spherical harmonic functions, etc. does not change　the Public API. Therefore, commonization can be done after the release of a major version 1.0.0 if we need.


Replace hyper client with reqwest client

Usage related changes

Close #424

Development related changes

Currently not working, explanation in https://github.com/0xSpaceShard/starknet-devnet-rs/issues/424#issuecomment-2056783440

Checklist:

[ ] Checked the relevant parts of development docs
[ ] Applied formatting - ./scripts/format.sh
[ ] No linter errors - ./scripts/clippy_check.sh
[ ] No unused dependencies - ./scripts/check_unused_deps.sh
[ ] Performed code self-review
[ ] Rebased to the latest commit of the target branch (or merged it into my branch)
[ ] Updated the docs if needed, including the TODO section
[ ] Linked the issues which this PR resolves
[ ] Updated the tests if needed; all passing (execution info)

Superseded by #469


exception in WorkGiver Train

Describe the bug
This bug is being spammed in log till it gets limited
To Reproduce
not reproducible
Error Log
https://gist.github.com/thefriendlycacti/1cf6015636740cf6e7f612b881be4dfc
Mod List
https://gist.github.com/thefriendlycacti/100a1eff20f09a80a4a5988b407fabf9

Test with newest version of RT and re open this issue if needed


Mark Clipboard::write's metadata argument as optional

Clipboard::write()'s metadata argument is optional, but it's currently marked as required - this updates the docs to mark it as optional.
/cc @atom/feedback

:balloon:


Report diagnostics for unused "using" and "import" so that they will be shown properly in ide

When "using" and "import" are not needed, dim them and provide quickfix in IDE.
related issues:
#3373 , #4861

All changed packages have been documented.

:white_check_mark: @typespec/compiler

Show changes
@typespec/compiler - feature ✏

Support diagnostics for unused import and using statements

You can try these changes here

🛝 Playground
🌐 Website
📚 Next docs

Question that might need input from others:
Do we want the unused import/using to show up as a trace diagnostic when running the compiler? It's a little surprising to see it as a trace without enabling tracing. The TypeScript compiler has a few flags for enabling some "unused" checks which then treats them as errors - wonder if we want to do something similar here.

Question that might need input from others:
Do we want the unused import/using to show up as a trace diagnostic when running the compiler? It's a little surprising to see it as a trace without enabling tracing. The TypeScript compiler has a few flags for enabling some "unused" checks which then treats them as errors - wonder if we want to do something similar here.

They are logged as "trace" because I can't find a better loglevel to log them. I think they are definitely not "error" or "warning". Please feel free to suggest if there is a better way to log them.
I like the "flags" idea, I think these hint diagnostics are mainly for IDE user experience, then how about just including the check when "designTimeBuild" option is true, but skip it in normal compile?

Talked offline and we are not able to 100% determine whether an import is used because user may modify the tree in their lib or emitters, so I will

extract the unused-using part and file another PR for it
do some investigation to see whether we can express these unused-import information in a more accurate way which is still very useful to end user in most case (Or if we have type-only lib supported, then we should not have this problem)

close this PR and the issues are still open to track the 2 actions above.


Extend the e2e tests with append a partitioned file

As a follow-up for https://github.com/apache/iceberg-rust/pull/349.
We should add another test for adding a partitioned file.

@feniljain Thanks, I've merged the PR. I always want to give others the chance to chime in as well.


Terraform modules: generic vs release

Should we separate those conceptually?

terraform.tfvars (final values) ← live module (with variable interpolation) ← generic module

Both generic and live modules should be of one of the two kinds: infra or kube.


Format fails with "expecting an argument"

I have "green build" projects in Kotlin using the excellent ktlint Maven plugin -- thank you!
I started a new project today, copied over the pom.xml recipes from others, but in this single-file project, I receive from ./mvnw clean verify:
[ERROR] Failed to execute goal com.github.gantsign.maven:ktlint-maven-plugin:1.4
.2:format (default) on project kotlin-dice: Execution default of goal com.github
.gantsign.maven:ktlint-maven-plugin:1.4.2:format failed: Expecting an argument -
> [Help 1]

Looking at the stacktrace from -X:
Caused by: com.pinterest.ktlint.core.ParseException: Expecting an argument
    at com.pinterest.ktlint.core.KtLint.format (KtLint.kt:370)

I feel I must have made a silly mistake, but various diffs are not turning up anything.  Is this something I can address myself?
https://github.com/binkley/kotlin-dice/blob/76ceb23f0963b570cc89823fc4b230af95b759f8/pom.xml
(Build is presently broken due to this issue; pushing to GitHub so that pom.xml is visible.)

@boxleytw, thanks for reporting this issue. Unfortunately, this plugin doesn't support Kotlin 1.4 yet. I'll look into improving the error message.

@freemanjp Aha!  Thank you.  I'll go back to 1.3.72.  There are no 1.4 features needed in the project.

I moved back to Kotlin 1.3.72, simplified the POM and removed 1.4 features, and still get the same complaint.
Is there anything I can do to provide more/better information on this?

I may have found the problem.  I was using the "trailing comma" feature of 1.4.  After going back to 1.3-style on this (no trailing comma), ktlint ran fine.


Software Steering Council Representative

Let's get started on selecting a Software Steering Council Representative.
Here's my proposal for a process.

We have an open nomination on this thread for ~1 week—until the next Jupyter Server meeting—starting today.
Any active team member can nominate another active team member by posting their name in this thread.
If nominated, you can accept/reject the nomination by giving a :+1: or :-1: on the comment where you were nominated.
At the end of the week, we'll open a voting period in separate thread with the list of nominee names.

@jupyter-server/active-team-members

I would like to nominate @Zsailer for the role

I have created a Google Form to conduct this vote. I propose we create a Google Group for correspondence on issues of nominations and voting. Let's discuss that during today's call and I will send out the form to the group once we've created it. I don't have everybody's email addresses so I can't send it out right away.
cc: @Zsailer @blink1073 @kevin-bates @jess-x @damianavila @SylvainCorlay @davidbrochart @echarles @vidartf @ellisonbg @ivanov @lresende @mwakaba2

Fixed by #40


Allow psr/cache v2

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
2.3%

Covered Lines:
337

Relevant Lines:
384

💛  - Coveralls


403 Forbidden: Resource not accessible by integration

⨯ 403 Forbidden
"method: post url: https://api.github.com/repos/supersu-man/bumper/releases\n\n          Data:\n          {\"message\":\"Resource not accessible by integration\",\"documentation_url\":\"https://docs.github.com/rest/reference/repos#create-a-release\"}\n          "
Headers: {
  "server": "GitHub.com",
  "date": "Wed, 11 Jan 2023 12:53:27 GMT",
  "content-type": "application/json; charset=utf-8",
/home/runner/work/_actions/samuelmeuli/action-electron-builder/v1/index.js:144
				throw err;
				^

Error: Command failed: npx --no-install electron-builder --linux --publish always -c.snap.publish=github
    at checkExecSyncError (child_process.js:635:11)
    at execSync (child_process.js:671:15)
    at run (/home/runner/work/_actions/samuelmeuli/action-electron-builder/v1/index.js:21:27)
    at runAction (/home/runner/work/_actions/samuelmeuli/action-electron-builder/v1/index.js:132:4)
    at Object.<anonymous> (/home/runner/work/_actions/samuelmeuli/action-electron-builder/v1/index.js:1[50](https://github.com/supersu-man/bumper/actions/runs/3892963803/jobs/6644999556#step:5:51):1)
    at Module._compile (internal/modules/cjs/loader.js:999:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)
    at Module.load (internal/modules/cjs/loader.js:863:32)
    at Function.Module._load (internal/modules/cjs/loader.js:708:14)
    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:60:12) {
  status: 1,
  signal: null,
  output: [ null, null, null ],
  pid: 1760,
  stdout: null,
  stderr: null
}
  "content-length": "136",
  "x-github-media-type": "github.v3; format=json",
  "x-github-api-version-selected": "2022-11-28",
  "x-ratelimit-limit": "1000",
  "x-ratelimit-remaining": "970",
  "x-ratelimit-reset": "1673444667",
  "x-ratelimit-used": "30",
  "x-ratelimit-resource": "core",
  "access-control-expose-headers": "ETag, Link, Location, Retry-After, X-GitHub-OTP, X-RateLimit-Limit, X-RateLimit-Remaining, X-RateLimit-Used, X-RateLimit-Resource, X-RateLimit-Reset, X-OAuth-Scopes, X-Accepted-OAuth-Scopes, X-Poll-Interval, X-GitHub-Media-Type, X-GitHub-SSO, X-GitHub-Request-Id, Deprecation, Sunset",
  "access-control-allow-origin": "*",
  "strict-transport-security": "max-age=31536000; includeSubdomains; preload",
  "x-frame-options": "deny",
  "x-content-type-options": "nosniff",
  "x-xss-protection": "0",
  "referrer-policy": "origin-when-cross-origin, strict-origin-when-cross-origin",
  "content-security-policy": "default-src 'none'",
  "vary": "Accept-Encoding, Accept, X-Requested-With",
  "x-github-request-id": "9441:1381:3F1F00B:812956B:63BEB147",
  "connection": "close"
}  failedTask=build stackTrace=HttpError: 403 Forbidden
"method: post url: [https://api.github.com/repos/supersu-man/bumper/releases\n\n](https://api.github.com/repos/supersu-man/bumper/releases/n/n)          Data:\n          {\"message\":\"Resource not accessible by integration\",\"documentation_url\":\"https://docs.github.com/rest/reference/repos#create-a-release\"}\n          "
Headers: {
  "server": "GitHub.com",
  "date": "Wed, 11 Jan 2023 12:[53](https://github.com/supersu-man/bumper/actions/runs/3892963803/jobs/6644999556#step:5:54):27 GMT",
  "content-type": "application/json; charset=utf-8",
  "content-length": "136",
  "x-github-media-type": "github.v3; format=json",
  "x-github-api-version-selected": "2022-11-28",
  "x-ratelimit-limit": "1000",
  "x-ratelimit-remaining": "970",
  "x-ratelimit-reset": "1673444667",
  "x-ratelimit-used": "30",
  "x-ratelimit-resource": "core",
  "access-control-expose-headers": "ETag, Link, Location, Retry-After, X-GitHub-OTP, X-RateLimit-Limit, X-RateLimit-Remaining, X-RateLimit-Used, X-RateLimit-Resource, X-RateLimit-Reset, X-OAuth-Scopes, X-Accepted-OAuth-Scopes, X-Poll-Interval, X-GitHub-Media-Type, X-GitHub-SSO, X-GitHub-Request-Id, Deprecation, Sunset",
  "access-control-allow-origin": "*",
  "strict-transport-security": "max-age=31536000; includeSubdomains; preload",
  "x-frame-options": "deny",
  "x-content-type-options": "nosniff",
  "x-xss-protection": "0",
  "referrer-policy": "origin-when-cross-origin, strict-origin-when-cross-origin",
  "content-security-policy": "default-src 'none'",
  "vary": "Accept-Encoding, Accept, X-Requested-With",
  "x-github-request-id": "9441:1381:3F1F00B:8129[56](https://github.com/supersu-man/bumper/actions/runs/3892963803/jobs/6644999556#step:5:57)B:[63](https://github.com/supersu-man/bumper/actions/runs/3892963803/jobs/6644999556#step:5:64)BEB147",
  "connection": "close"
}
    at createHttpError (/home/runner/work/bumper/bumper/node_modules/builder-util-runtime/src/httpExecutor.ts:30:10)
    at IncomingMessage.<anonymous> (/home/runner/work/bumper/bumper/node_modules/builder-util-runtime/src/httpExecutor.ts:197:13)
    at IncomingMessage.emit (events.js:412:35)
    at endReadableNT (internal/streams/readable.js:1333:12)
    at processTicksAndRejections (internal/process/task_queues.js:[82](https://github.com/supersu-man/bumper/actions/runs/3892963803/jobs/6644999556#step:5:83):21)

I'm alao having this issue. Gonna leave the comment in the hopes someone fix it.

I'm also having this issue. Gonna leave the comment in the hopes someone fixes it.

Did you try creating a release before using this action ? or maybe tried for a workaround?

I'm also having this issue. Gonna leave the comment in the hopes someone fixes it.

Did you try creating a release before using this action ? or maybe tried for a workaround?

I tried creating a release, as it seemed the only workaround, but it didn't work. I didn't try any other workaround, though.

The action is not working anymore, I've made a test repo and been testing by making changes to .yml file but no luck.

The GITHUB_TOKEN that's automatically generated for the workflowy doesn't have permission to create the release, hence the 403.
The easiest solution is to allow Read and write permissions in the repo's Actions settings:

Go to: https://github.com/REPO_OWNER/REPO_NAME/settings/actions
(Your Repo -> Settings -> Actions -> General)

Update the permission under Workflow permissions, then save.

More info:
https://docs.github.com/en/actions/security-guides/automatic-token-authentication

The GITHUB_TOKEN that's automatically generated for the workflow doesn't have permission to create the release, hence the 403.
The easiest solution is to allow Read and write permissions in the repo's Actions settings:

Go to: https://github.com/REPO_OWNER/REPO_NAME/settings/actions
(Your Repo -> Settings -> Actions -> General)
Update the permission under Workflow permissions, then save.

More info: https://docs.github.com/en/actions/security-guides/automatic-token-authentication

I can attest that this method is currently functional, and may just be the workaround for those who are waiting for a change in the action itself.

I can attest that this method is currently functional, and may just be the workaround for those who are waiting for a change in the action itself.

@Gilvan-Araujo I wouldn't call it a "workaround". This isn't an issue with the the action itself. Actions shouldn't be able to set what permissions they have, so there's nothing to fix here.
Maybe it should be better documented now though, because for new repos, the permissions for the auto-generated GITHUB_TOKEN are all read-only now.
So people can either use the solution I posted above, use a Personal Access Token, or set the permissions key in their workflow file.

Please, add documentation about this in the README.


lz77 decompression

So the code on the branch lz77 is able to produce readable output.
I'm very close to getting it right so I thought maybe someone else's pov will yield some helpful tips.
Example record
good friends now, good friends and very strong. They would help him. Baggins would pay for it.
That was his chief thought. He hated Bilbo and cursed his name. What is more, he knew where he
came from.â

âBut how did he find that out?â asked Frodo.

âWell, as for the name, Bilbo very foolishly told Gollum himself;
and after that it would not be difficult to discover his country, once Gollum came out. Oh yes,
he came out. His longing for the Ring proved stronger than his fear of the Orcs, or even of the
light. After a year or two he left the mountains. You see, though still bound by desire of it,
the Ring was no longer devouring him; he began to revive a little. He felt old, terribly old,
yet less timid, and he was mortally hungry.

âLight, light of Sun
and Moon, he still feared and hated, and he always will, I think; but he was cunning. He found
he could hide from daylight and moonshine, and make his way swiftly and softly by dead of night
with his pale cold eyes, and catch small frightened or unwary things. He grew stronger and bolder
with new food and new air. He found his way into Mirkwood, as one would expect.â

âIs that where you found him?â asked Frodo.

âI saw
him there,â answered Gandalf, âbut before that he had wandered far, following Bilboâs trail. It was
difficult to learn anything from him for certain, for his talk was constantly interrupted by curses
and threats. âWhat had it got in its pocketses?â he said. âIt wouldnât say, no precious. Little
cheat. Not a fair question. It cheated first, it did. It broke the rules. We ought to have squeezed
it, yes precious. And we will, precious!â

â

But I also get some like this:
Rivcurlartsyighketopplas swed te,dmplungarto Rivnk tnk aoMfalRauroskseeinaunkamas spit;
ke glimmeras sr plbow ss yartitting utfumg. A,dmEthir A,duinard sawHen,utmonlyy deltaEye
of tat Rikse,dmmyriadoMfalsea-birdoMwhirlas ssyighkewhite dustkse of tsu Hee,dmben Grh of
mke greoine, usil Rivsea,eripplas sse endamelelasng i
<pardonly="0pt"nmedth="1em">But
everywmote.rd he lookrd saw of tsigla falrth imageMisty Mas t pla wote.crawlas ssyighanthi
ao: orca wote.issuing out falkethouse, uuoame. UndRivnk tbougha falMirkwoohenk te.
Thedeadundstrif Eye El Rsne, us. Ms, une asbe. Eg imageseelEye of tBeorninga wTheafl,
ke cloud wTheo RivMoria;e m lo ros Eye of tbordRia falLÃ³rien i
<pardonly="0pt"nmedth="1em">
Horsorln wote.gk aoping oe of tgraelefalRohan;kwol Rsnpou a unrom Isengthw. From of tmed la
falHarad shipa falrth put fut einsea;ke, uout falof ts. Eus. Mwote.moight endamelly: ss. dsrln,
spearrln, bowrln ittinhorsoain uncioresfalchieft pla e, ulad. Mw pla. A asn,utpowot falof tDark
L. d wThese  yeion image ourning wasrh ag plkrd beheTheMinTheTirith. Fth awrelite worldHee,
dmbeautiful:ewhite-wk andHemany-towotndHeproud shrunkir ittiniresmas t pl-seat;kiresbattlo
rlnresglittotndnmerh sgeelHee,dmiresourrereswote., only merh manysbannRia siopgeseapookse his
hearn sBut ag pl EusinTheTirithent. Hetuanonk tunk t aoaingreatotse,dmmxpldstrong imaithRikse.
Erthw,stswi aing his e. E Thedrawn iIt p. sld srd ruinld , oding falOsgiliGrhHen,utgrasning gatoa
falMinTheMorgulHee,dmof tmes tld Mas t plaHee,dmir he lookittinGorgoronkHen,utvk any faloerrorkse
of tLeelEye MordmenoDarksnglelrelnk te.undRivnk tSun iFite.glowookemidmof t m lo. Mas t Doom

use this as main.rs:
use mobi::Mobi;
use std::{thread, time};

fn main() {
    let m = Mobi::init("/home/wojtek/Downloads/lotr.mobi").unwrap();
    let ten_millis = time::Duration::from_millis(70);
    let mut count = 0;
    for record in m.records {
        if count == 50 {
            break;
        }
        println!("{}", record.record_data);
        thread::sleep(ten_millis);
        count += 1;
    }
}

This is the type of output it's able to produce.
You can check it out by switching to lz77 branch.

Managed to solve it :)


Backup limit issue, not deleting oldest versions, overwriting newest backup

Hi!
Yesterday I had reached the max 10 backups (I changed it to 10, the default is 25).  However, when I make a new backup manually or via the scheduler, the newest backup is removed, not the oldest.  Which means I cannot restore a slightly older version anymore.  I have to manually delete the oldest backup to make a new backup that doesn’t overwrite the last one.
This happens with the scheduled backup as well.  I lost the Jan 2 backup, which was overwritten by the Jan 3 backup.

Found the cause of the bug, and patched it. Fix will be in new release shortly!

Manual backup deleted the oldest instead of newest, so this bug is verified fixed.


[FEATURE] Allow writing and reading a sketch file

Summary
This PR changes the following:

[x] It removes the anyway unused function read_hll_files_into
[x] It changes the semantic of the cmd option --output-sketches-to *1
[x] It add to the semantic of the cmd option --input *2
[ ] Adds/changes test for the new behavior

*1 --output-sketches-to
Before this PR --output-sketches-to DIR would write one sketch file for each input file.
This had several disadvantages: many files, not scalable to millions of user bins, with additional minhashes there are even more files, no direct information about the input filenames as well as the meta information of how the sketches were generated (k,w,sketch-bits).
Instead, now the option --output-sketches-to FILENAME, writes a sketch file (chopper::sketch::sketch_file) that contains the chopper config, filenames and the sketches, the latter two in the same order. This way, only a single file is written will all important information contained. Slight disadvantage: You can (currently) not read a fraction of the input, e.g. "just the first 1000 sketches of the data set". You would need to create a separate sketch file for this.
*2 --input
Before this PR, only a list of filenames would be allowed as input to choppper (layout).
After this PR, a sketch file is also a valid source of input as it contains the filenames and the sketches.

Documentation preview available at https://docs.seqan.de/preview/seqan/chopper/254

One more thing:
Do you use the stuff in https://github.com/seqan/chopper/blob/main/src%2Fsketch%2Foutput.cpp ?
If not, it could be removed.


Landing manifest bits broke the build

https://github.com/breadboard-ai/breadboard/commit/1af3ef03ff5f35042c96457e5e1f3659831f19f8 broke the build:
As seen locally and on CI/CD
❌ [packages/breadboard-web:build:tsc] exited with exit code 1. Output:

src/make-graphs.ts:13:8 - error TS2307: Cannot find module '@google-labs/breadboard-manifest' or its corresponding type declarations.

13 } from "@google-labs/breadboard-manifest";
          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

src/providers/examples.ts:13:36 - error TS2307: Cannot find module '@google-labs/breadboard-manifest' or its corresponding type declarations.

13 import { BreadboardManifest } from "@google-labs/breadboard-manifest";
                                      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

src/providers/examples.ts:24:32 - error TS7006: Parameter 'board' implicitly has an 'any' type.

24     const blank = boards.find((board) => {
                                  ~~~~~

src/providers/examples.ts:40:15 - error TS[70](https://github.com/breadboard-ai/breadboard/actions/runs/9635725496/job/26572822244#step:8:71)06: Parameter 'board' implicitly has an 'any' type.

40         .map((board) => ({
                 ~~~~~

src/providers/examples.ts:44:16 - error TS7006: Parameter 'a' implicitly has an 'any' type.

44         .sort((a, b) => a.title!.localeCompare(b.title!))
                  ~

src/providers/examples.ts:44:19 - error TS7006: Parameter 'b' implicitly has an 'any' type.

44         .sort((a, b) => a.title!.localeCompare(b.title!))
                     ~

src/providers/examples.ts:45:15 - error TS7006: Parameter 'board' implicitly has an 'any' type.

45         .map((board) => [
                 ~~~~~

Found 7 errors.

❌ 2 scripts failed.

@Mearman ^^^

Also -- I missed that the package name was @google-labs/breadboard-manifest. Let's instead use the new namespace and change it to @breadboard-ai/manifest.

ah I did wonder about the new namespace


multiple fixes for comparisons and loops in security/test_bmc_connections.robot and minor enhancements

In security/test_bmc_connections.robot the testcase Flood Patch Without Auth Token And Check Stability Of BMC is failing because Should Be Equal is used to compare what it finds are types <class 'int'> and <class 'str'>.
I've fixed that by instead using Should Be Equal As Integers.
Secondly, it does not complete the desired Login And Create User after the last set of posts because the loop count is not correctly set. As a result, when using ${iterations} = 100 (as I typically do for a smoke test) it will always pass.
Additionally, I've slightly enhanced the output report to indicate where in the loop the errors occur, to facilitate debugging.
Finally, I also applied the changes above to Flood Post Without Auth Token And Check Stability Of BMC and similarly fixed the iteration value in Test Stability On Large Number Of Wrong Login Attempts To GUI
Patch file:
diff --git a/security/test_bmc_connections.robot b/security/test_bmc_connections.robot
index 07b5364c..d877edd8 100644
--- a/security/test_bmc_connections.robot
+++ b/security/test_bmc_connections.robot
@@ -48,12 +48,12 @@ Test Patch Without Auth Token Fails
 Flood Patch Without Auth Token And Check Stability Of BMC
     [Documentation]  Flood patch method without auth token and check BMC stability.
     [Tags]  Flood_Patch_Without_Auth_Token_And_Check_Stability_Of_BMC
-    @{status_list}=  Create List

+    @{fail_list}=  Create List
     ${active_channel_config}=  Get Active Channel Config
     ${ethernet_interface}=  Set Variable  ${active_channel_config['${CHANNEL_NUMBER}']['name']}

-    FOR  ${i}  IN RANGE  ${1}  ${iterations}
+    FOR  ${i}  IN RANGE  ${1}  ${iterations} + 1
         Log To Console  ${i}th iteration
         Run Keyword And Ignore Error
         ...  Redfish.Patch  ${REDFISH_NW_ETH_IFACE}${ethernet_interface}  body={'HostName': '${hostname}'}
@@ -61,13 +61,14 @@ Flood Patch Without Auth Token And Check Stability Of BMC
         # Every 100th iteration, check BMC allows patch with auth token.
         ${status}=  Run Keyword If  ${i} % 100 == 0  Run Keyword And Return Status
         ...  Login And Configure Hostname
-        Run Keyword If  ${status} == False  Append To List  ${status_list}  ${status}
+
+        Run Keyword If  ${status} == False  Append To List  ${fail_list}  ${i}
     END
     ${verify_count}=  Evaluate  ${iterations}/100
-    ${fail_count}=  Get Length  ${status_list}
+    ${fail_count}=  Get Length  ${fail_list}

-    Should Be Equal  ${fail_count}  0
-    ...  msg=Patch operation failed ${fail_count} times in ${verify_count} attempts
+    Should Be Equal As Integers  ${fail_count}  ${0}
+    ...  msg=Patch operation failed ${fail_count} times in ${verify_count} attempts; fails at iterations ${fail_list}

 Verify User Cannot Login After 5 Non-Logged In Sessions
@@ -100,11 +101,11 @@ Flood Post Without Auth Token And Check Stability Of BMC
     [Documentation]  Flood post method without auth token and check BMC stability.
     [Tags]  Flood_Post_Without_Auth_Token_And_Check_Stability_Of_BMC

-    @{status_list}=  Create List
+    @{fail_list}=  Create List
     ${user_info}=  Create Dictionary
     ...  UserName=test_user  Password=TestPwd123  RoleId=Operator  Enabled=${True}

-    FOR  ${i}  IN RANGE  ${1}  ${iterations}
+    FOR  ${i}  IN RANGE  ${1}  ${iterations} + 1
         Log To Console  ${i}th iteration
         Run Keyword And Ignore Error
         ...  Redfish.Post   /redfish/v1/AccountService/Accounts/  body=&{user_info}
@@ -112,13 +113,13 @@ Flood Post Without Auth Token And Check Stability Of BMC
         # Every 100th iteration, check BMC allows post with auth token.
         ${status}=  Run Keyword If  ${i} % 100 == 0  Run Keyword And Return Status
         ...  Login And Create User
-        Run Keyword If  ${status} == False  Append To List  ${status_list}  ${status}
+        Run Keyword If  ${status} == False  Append To List  ${fail_list}  ${i}
     END
     ${verify_count}=  Evaluate  ${iterations}/100
-    ${fail_count}=  Get Length  ${status_list}
+    ${fail_count}=  Get Length  ${fail_list}

-    Should Be Equal  ${fail_count}  0
-    ...  msg=Post operation failed ${fail_count} times in ${verify_count} attempts
+    Should Be Equal As Integers  ${fail_count}  ${0}
+    ...  msg=Post operation failed ${fail_count} times in ${verify_count} attempts; fails at iterations ${fail_list}

 Make Large Number Of Wrong SSH Login Attempts And Check Stability
@@ -161,7 +162,8 @@ Test Stability On Large Number Of Wrong Login Attempts To GUI

     Go To  ${bmc_url}

-    FOR  ${i}  IN RANGE  ${1}  ${iterations}
+    Printn
+    FOR  ${i}  IN RANGE  ${1}  ${iterations} + 1
         Log To Console  ${i}th login
         Run Keyword And Ignore Error  Login to GUI With Incorrect Credentials

@prkatti1  can u take a look

Thank you @generatz for the feedback.. will look into it..

Changes :  https://gerrit.openbmc-project.xyz/c/openbmc/openbmc-test-automation/+/52701
update extra improvement along with it


python3Packages.pydicom: 2.1.1 -> 2.1.2

Motivation for this change
Update Pydicom to 2.1.2 and fix https://github.com/NixOS/nixpkgs/issues/106262
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

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

The only problem is with nitime but it's not because of Pydicom.
❯ nix-shell -p nixpkgs-review --run "nixpkgs-review pr 107550"
$ git -c fetch.prune=false fetch --force https://github.com/NixOS/nixpkgs master:refs/nixpkgs-review/0 pull/107550/head:refs/nixpkgs-review/1
remote: Enumerating objects: 26, done.
remote: Counting objects: 100% (26/26), done.
remote: Compressing objects: 100% (10/10), done.
remote: Total 31 (delta 17), reused 18 (delta 16), pack-reused 5
Unpacking objects: 100% (31/31), 52.75 KiB | 412.00 KiB/s, done.
From https://github.com/NixOS/nixpkgs
   a4d86d5425b..ae46445e624  master                -> refs/nixpkgs-review/0
   e78a2d39f0e..d27486656c3  refs/pull/107550/head -> refs/nixpkgs-review/1
$ git worktree add /home/thiago/.cache/nixpkgs-review/pr-107550/nixpkgs ae46445e62484b06e8e61acea54fda3e43a08252
Preparing worktree (detached HEAD ae46445e624)
HEAD is now at ae46445e624 Merge pull request #107540 from flokli/websocketpp-fix-cross
$ nix-env -f /home/thiago/.cache/nixpkgs-review/pr-107550/nixpkgs -qaP --xml --out-path --show-trace
$ git merge --no-commit d27486656c3ac589a68492d78d933e2b51fc86e8
Automatic merge went well; stopped before committing as requested
$ nix-env -f /home/thiago/.cache/nixpkgs-review/pr-107550/nixpkgs -qaP --xml --out-path --show-trace --meta
24 packages updated:
python3.7-dcmstack python3.7-dicom2nifti python3.7-dipy python3.7-heudiconv python3.7-nibabel python3.7-nilearn python3.7-nipy python3.7-nipype python3.7-nitime python3.7-pybids python37Packages.pydicom (2.1.1 → 2.1.2) python3.8-dicom2nifti python3.8-dipy python3.8-nibabel python3.8-nilearn python3.8-nipy python3.8-nipype python3.8-pybids python38Packages.pydicom (2.1.1 → 2.1.2) python3.9-dicom2nifti python3.9-dipy python3.9-nibabel python3.9-nipy python39Packages.pydicom (2.1.1 → 2.1.2)

$ nix --experimental-features nix-command build --no-link --keep-going --option build-use-sandbox relaxed -f /home/thiago/.cache/nixpkgs-review/pr-107550/build.nix
error: --- Error --- nix-daemon
builder for '/nix/store/s1fly98c55djaa0ynifdmsxm19xkzm5m-python3.7-nitime-0.8.1.drv' failed with exit code 1; last 10 log lines:
      format='%.2f')

  nitime/tests/test_viz.py::test_drawmatrix_channels
    /build/nitime-0.8.1/nitime/viz.py:351: MatplotlibDeprecationWarning: The 'norm' parameter to Colorbar has no effect because it is overridden by the mappable; it is deprecated since 3.3 and will be removed two minor releases later.
      format='%.2f')

  -- Docs: https://docs.pytest.org/en/stable/warnings.html
  =========================== short test summary info ============================
  FAILED nitime/tests/test_utils.py::test_detect_lines - AssertionError: Harmon...
  ================== 1 failed, 67 passed, 7 warnings in 24.54s ===================
error: --- Error ----------------------------------------------------------------------------------------------------------------------------------------- nix
1 dependencies of derivation '/nix/store/q6y6ka8imn6lqvxsa26xsxsym7mc3vwn-env.drv' failed to build

Link to currently reviewing PR:
https://github.com/NixOS/nixpkgs/pull/107550

1 package failed to build:
python37Packages.nitime

23 packages built:
python37Packages.dcmstack python37Packages.dicom2nifti python37Packages.dipy python37Packages.heudiconv python37Packages.nibabel python37Packages.nilearn python37Packages.nipy python37Packages.nipype python37Packages.pybids python37Packages.pydicom python38Packages.dicom2nifti python38Packages.dipy python38Packages.nibabel python38Packages.nilearn python38Packages.nipy python38Packages.nipype python38Packages.pybids python38Packages.pydicom python39Packages.dicom2nifti python39Packages.dipy python39Packages.nibabel python39Packages.nipy python39Packages.pydicom

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

24 packages built:

python37Packages.dcmstack
python37Packages.dicom2nifti
python37Packages.dipy
python37Packages.heudiconv
python37Packages.nibabel
python37Packages.nilearn
python37Packages.nipy
python37Packages.nipype
python37Packages.nitime
python37Packages.pybids
python37Packages.pydicom
python38Packages.dicom2nifti
python38Packages.dipy
python38Packages.nibabel
python38Packages.nilearn
python38Packages.nipy
python38Packages.nipype
python38Packages.pybids
python38Packages.pydicom
python39Packages.dicom2nifti
python39Packages.dipy
python39Packages.nibabel
python39Packages.nipy
python39Packages.pydicom

Result of nixpkgs-review pr 107550 run on x86_64-darwin 1

3 packages failed to build:

python37Packages.nitime
python38Packages.nipype
python39Packages.nipype

22 packages built:

python37Packages.dcmstack
python37Packages.dicom2nifti
python37Packages.dipy
python37Packages.heudiconv
python37Packages.nibabel
python37Packages.nilearn
python37Packages.nipy
python37Packages.nipype
python37Packages.pybids
python37Packages.pydicom
python38Packages.dicom2nifti
python38Packages.dipy
python38Packages.nibabel
python38Packages.nilearn
python38Packages.nipy
python38Packages.pybids
python38Packages.pydicom
python39Packages.dicom2nifti
python39Packages.dipy
python39Packages.nibabel
python39Packages.nipy
python39Packages.pydicom

@SuperSandro2000 thanks! Please, can you try to build nitime and nipype in Mac with the previous version of Pydicom? I think it'll not work because Pydicom wasn't building before.

@SuperSandro2000 thanks! Please, can you try to build nitime and nipype in Mac with the previous version of Pydicom? I think it'll not work because Pydicom wasn't building before.

They are also broken on master.

It looks like I made a near-dupe of this: https://github.com/NixOS/nixpkgs/pull/107864

Before pydicom-data in my derivation was pointing to  master. After seeing @rmcgibbo now the my derivation is pointing to explicitaly to a commit revision like in the @rmcgibbo derivation.

This is a semi-automatic executed nixpkgs-review. If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 107550 run on x86_64-linux 1

1 packages failed to build and are new build failures:

python37Packages.heudiconv
python37Packages.nilearn
python37Packages.nipype
python37Packages.pybids
python38Packages.nilearn
python38Packages.nipype
python38Packages.pybids

17 packages built:

python37Packages.dcmstack
python37Packages.dicom2nifti
python37Packages.dipy
python37Packages.nibabel
python37Packages.nipy
python37Packages.nitime
python37Packages.pydicom
python38Packages.dicom2nifti
python38Packages.dipy
python38Packages.nibabel
python38Packages.nipy
python38Packages.pydicom
python39Packages.dicom2nifti
python39Packages.dipy
python39Packages.nibabel
python39Packages.nipy
python39Packages.pydicom

cfn-lint
/build/source/test/unit/rules/functions/test_sub_needed.py:30: DeprecationWarning: invalid escape sequence \$

@SuperSandro2000 cfn-lint is not building because of a bug. This bug was already post and is being fixed by @rmcgibbo in https://github.com/NixOS/nixpkgs/pull/108079

I rebased to the master after the merge of the PR that fixes cfn-lint, now everything is building correctly:
❯ nix-shell -p nixpkgs-review --run "nixpkgs-review pr 107550"
$ git -c fetch.prune=false fetch --force https://github.com/NixOS/nixpkgs master:refs/nixpkgs-review/0 pull/107550/head:refs/nixpkgs-review/1
From https://github.com/NixOS/nixpkgs
   5a214b6b687..f0e774407c3  master                -> refs/nixpkgs-review/0
 + bf3a52ae0a7...4553efaedd2 refs/pull/107550/head -> refs/nixpkgs-review/1  (forced update)
$ git worktree add /home/thiago/.cache/nixpkgs-review/pr-107550-3/nixpkgs f0e774407c3ce6ac41d79a5a564a9265b6efc600
Preparing worktree (detached HEAD f0e774407c3)
HEAD is now at f0e774407c3 Merge pull request #108141 from AndersonTorres/update-wio
$ nix-env -f /home/thiago/.cache/nixpkgs-review/pr-107550-3/nixpkgs -qaP --xml --out-path --show-trace
$ git merge --no-commit 4553efaedd2e0f209daa75eca77559eca8b2099d
Updating f0e774407c3..4553efaedd2
Fast-forward
 pkgs/development/python-modules/pydicom/default.nix | 48 +++++++++++++++++++++++++++++++++++++-----------
 1 file changed, 37 insertions(+), 11 deletions(-)
$ nix-env -f /home/thiago/.cache/nixpkgs-review/pr-107550-3/nixpkgs -qaP --xml --out-path --show-trace --meta
24 packages updated:
python3.7-dcmstack python3.7-dicom2nifti python3.7-dipy python3.7-heudiconv python3.7-nibabel python3.7-nilearn python3.7-nipy python3.7-nipype python3.7-nitime python3.7-pybids python37Packages.pydicom (2.1.1 → 2.1.2) python3.8-dicom2nifti python3.8-dipy python3.8-nibabel python3.8-nilearn python3.8-nipy python3.8-nipype python3.8-pybids python38Packages.pydicom (2.1.1 → 2.1.2) python3.9-dicom2nifti python3.9-dipy python3.9-nibabel python3.9-nipy python39Packages.pydicom (2.1.1 → 2.1.2)

$ nix --experimental-features nix-command build --no-link --keep-going --option build-use-sandbox relaxed -f /home/thiago/.cache/nixpkgs-review/pr-107550-3/build.nix

Link to currently reviewing PR:
https://github.com/NixOS/nixpkgs/pull/107550

24 packages built:
python37Packages.dcmstack python37Packages.dicom2nifti python37Packages.dipy python37Packages.heudiconv python37Packages.nibabel python37Packages.nilearn python37Packages.nipy python37Packages.nipype python37Packages.nitime python37Packages.pybids python37Packages.pydicom python38Packages.dicom2nifti python38Packages.dipy python38Packages.nibabel python38Packages.nilearn python38Packages.nipy python38Packages.nipype python38Packages.pybids python38Packages.pydicom python39Packages.dicom2nifti python39Packages.dipy python39Packages.nibabel python39Packages.nipy python39Packages.pydicom

$ nix-shell /home/thiago/.cache/nixpkgs-review/pr-107550-3/shell.nix

I rebased to the master after the merge of the PR that fixes cfn-lint, now everything is building correctly:
❯ nix-shell -p nixpkgs-review --run "nixpkgs-review pr 107550"
$ git -c fetch.prune=false fetch --force https://github.com/NixOS/nixpkgs master:refs/nixpkgs-review/0 pull/107550/head:refs/nixpkgs-review/1
From https://github.com/NixOS/nixpkgs
   5a214b6b687..f0e774407c3  master                -> refs/nixpkgs-review/0
 + bf3a52ae0a7...4553efaedd2 refs/pull/107550/head -> refs/nixpkgs-review/1  (forced update)
$ git worktree add /home/thiago/.cache/nixpkgs-review/pr-107550-3/nixpkgs f0e774407c3ce6ac41d79a5a564a9265b6efc600
Preparing worktree (detached HEAD f0e774407c3)
HEAD is now at f0e774407c3 Merge pull request #108141 from AndersonTorres/update-wio
$ nix-env -f /home/thiago/.cache/nixpkgs-review/pr-107550-3/nixpkgs -qaP --xml --out-path --show-trace
$ git merge --no-commit 4553efaedd2e0f209daa75eca77559eca8b2099d
Updating f0e774407c3..4553efaedd2
Fast-forward
 pkgs/development/python-modules/pydicom/default.nix | 48 +++++++++++++++++++++++++++++++++++++-----------
 1 file changed, 37 insertions(+), 11 deletions(-)
$ nix-env -f /home/thiago/.cache/nixpkgs-review/pr-107550-3/nixpkgs -qaP --xml --out-path --show-trace --meta
24 packages updated:
python3.7-dcmstack python3.7-dicom2nifti python3.7-dipy python3.7-heudiconv python3.7-nibabel python3.7-nilearn python3.7-nipy python3.7-nipype python3.7-nitime python3.7-pybids python37Packages.pydicom (2.1.1 → 2.1.2) python3.8-dicom2nifti python3.8-dipy python3.8-nibabel python3.8-nilearn python3.8-nipy python3.8-nipype python3.8-pybids python38Packages.pydicom (2.1.1 → 2.1.2) python3.9-dicom2nifti python3.9-dipy python3.9-nibabel python3.9-nipy python39Packages.pydicom (2.1.1 → 2.1.2)

$ nix --experimental-features nix-command build --no-link --keep-going --option build-use-sandbox relaxed -f /home/thiago/.cache/nixpkgs-review/pr-107550-3/build.nix

Link to currently reviewing PR:
https://github.com/NixOS/nixpkgs/pull/107550

24 packages built:
python37Packages.dcmstack python37Packages.dicom2nifti python37Packages.dipy python37Packages.heudiconv python37Packages.nibabel python37Packages.nilearn python37Packages.nipy python37Packages.nipype python37Packages.nitime python37Packages.pybids python37Packages.pydicom python38Packages.dicom2nifti python38Packages.dipy python38Packages.nibabel python38Packages.nilearn python38Packages.nipy python38Packages.nipype python38Packages.pybids python38Packages.pydicom python39Packages.dicom2nifti python39Packages.dipy python39Packages.nibabel python39Packages.nipy python39Packages.pydicom

$ nix-shell /home/thiago/.cache/nixpkgs-review/pr-107550-3/shell.nix

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or torch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 107550 run on x86_64-linux 1

24 packages built:

python37Packages.dcmstack
python37Packages.dicom2nifti
python37Packages.dipy
python37Packages.heudiconv
python37Packages.nibabel
python37Packages.nilearn
python37Packages.nipy
python37Packages.nipype
python37Packages.nitime
python37Packages.pybids
python37Packages.pydicom
python38Packages.dicom2nifti
python38Packages.dipy
python38Packages.nibabel
python38Packages.nilearn
python38Packages.nipy
python38Packages.nipype
python38Packages.pybids
python38Packages.pydicom
python39Packages.dicom2nifti
python39Packages.dipy
python39Packages.nibabel
python39Packages.nipy
python39Packages.pydicom

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or torch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 107550 run on x86_64-linux 1

24 packages built:

python37Packages.dcmstack
python37Packages.dicom2nifti
python37Packages.dipy
python37Packages.heudiconv
python37Packages.nibabel
python37Packages.nilearn
python37Packages.nipy
python37Packages.nipype
python37Packages.nitime
python37Packages.pybids
python37Packages.pydicom
python38Packages.dicom2nifti
python38Packages.dipy
python38Packages.nibabel
python38Packages.nilearn
python38Packages.nipy
python38Packages.nipype
python38Packages.pybids
python38Packages.pydicom
python39Packages.dicom2nifti
python39Packages.dipy
python39Packages.nibabel
python39Packages.nipy
python39Packages.pydicom

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 107550 run on x86_64-darwin 1

1 package failed to build and are new build failure:

python37Packages.nitime

24 packages built:

python37Packages.dcmstack
python37Packages.dicom2nifti
python37Packages.dipy
python37Packages.heudiconv
python37Packages.nibabel
python37Packages.nilearn
python37Packages.nipy
python37Packages.nipype
python37Packages.pybids
python37Packages.pydicom
python38Packages.dicom2nifti
python38Packages.dipy
python38Packages.nibabel
python38Packages.nilearn
python38Packages.nipy
python38Packages.nipype
python38Packages.pybids
python38Packages.pydicom
python39Packages.dicom2nifti
python39Packages.dipy
python39Packages.nibabel
python39Packages.nipy
python39Packages.nipype
python39Packages.pydicom

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 107550 run on x86_64-darwin 1

1 package failed to build and are new build failure:

python37Packages.nitime

24 packages built:

python37Packages.dcmstack
python37Packages.dicom2nifti
python37Packages.dipy
python37Packages.heudiconv
python37Packages.nibabel
python37Packages.nilearn
python37Packages.nipy
python37Packages.nipype
python37Packages.pybids
python37Packages.pydicom
python38Packages.dicom2nifti
python38Packages.dipy
python38Packages.nibabel
python38Packages.nilearn
python38Packages.nipy
python38Packages.nipype
python38Packages.pybids
python38Packages.pydicom
python39Packages.dicom2nifti
python39Packages.dipy
python39Packages.nibabel
python39Packages.nipy
python39Packages.nipype
python39Packages.pydicom

It looks like perhaps this did not build on x86_64-darwin on hydra, despite building for Sandro: https://hydra.nixos.org/build/134332798

It looks like perhaps this did not build on x86_64-darwin on hydra, despite building for Sandro: https://hydra.nixos.org/build/134332798

The only failing test looks like it might be something about the filesystem and mtimes?
=================================== FAILURES ===================================
_______________________ TestDeferredRead.test_time_check _______________________

self = <pydicom.tests.test_filereader.TestDeferredRead object at 0x1222b7c10>

    def test_time_check(self):
        """Deferred read warns if file has been modified"""
        ds = dcmread(self.testfile_name, defer_size="2 kB")
        from time import sleep

        sleep(0.1)
        with open(self.testfile_name, "r+") as f:
            f.write("\0")  # "touch" the file

        msg = r"Deferred read warning -- file modification time has changed"
        with pytest.warns(UserWarning, match=msg):
>           ds.PixelData
E           Failed: DID NOT WARN. No warnings of type (<class 'UserWarning'>,) was emitted. The list of emitted warnings is: [].

Maybe that test could just be disabled.

The only failing test looks like it might be something about the filesystem and mtimes?
=================================== FAILURES ===================================
_______________________ TestDeferredRead.test_time_check _______________________

self = <pydicom.tests.test_filereader.TestDeferredRead object at 0x1222b7c10>

    def test_time_check(self):
        """Deferred read warns if file has been modified"""
        ds = dcmread(self.testfile_name, defer_size="2 kB")
        from time import sleep

        sleep(0.1)
        with open(self.testfile_name, "r+") as f:
            f.write("\0")  # "touch" the file

        msg = r"Deferred read warning -- file modification time has changed"
        with pytest.warns(UserWarning, match=msg):
>           ds.PixelData
E           Failed: DID NOT WARN. No warnings of type (<class 'UserWarning'>,) was emitted. The list of emitted warnings is: [].

Maybe that test could just be disabled.

@rmcgibbo I have PR to update nitime to its last version. Please, can you try if this error happens yet? https://github.com/NixOS/nixpkgs/pull/108407

@rmcgibbo I have PR to update nitime to its last version. Please, can you try if this error happens yet? https://github.com/NixOS/nixpkgs/pull/108407

@tfmoraes: I don't think I understand. It looks like the error I posted above is not related to nitime.

@tfmoraes: I don't think I understand. It looks like the error I posted above is not related to nitime.

sorry @rmcgibbo, I confused the word mtime with nitime :)

sorry @rmcgibbo, I confused the word mtime with nitime :)


Confluent Ansible 7.2+ require python 3.6+,  but Ansible yum module only works on Python2

Describe the issue
https://docs.confluent.io/ansible/current/ansible-requirements.html#general-requirements
https://docs.ansible.com/ansible/latest/collections/ansible/builtin/yum_module.html
Confluent Ansible 7.2+ require python 3.6+,  but Ansible yum module only works on Python2，
this will have conflict and get error like log show
Logs
TASK [common : Install OpenSSL and Unzip] ******************************************************************************************************************************************************************
Wednesday 22 February 2023  19:12:56 +0800 (0:00:00.165)       0:00:15.531 ****
fatal: [<IP_ADDRESS>]: FAILED! => {"changed": false, "msg": "The Python 2 bindings for rpm are needed for this module. If you require Python 3 support use the dnf Ansible module instead.. The Python 2 yum module is needed for this module. If you require Python 3 support use the dnf Ansible module instead."}
fatal: [<IP_ADDRESS>]: FAILED! => {"changed": false, "msg": "The Python 2 bindings for rpm are needed for this module. If you require Python 3 support use the dnf Ansible module instead.. The Python 2 yum module is needed for this module. If you require Python 3 support use the dnf Ansible module instead."}
fatal: [<IP_ADDRESS>]: FAILED! => {"changed": false, "msg": "The Python 2 bindings for rpm are needed for this module. If you require Python 3 support use the dnf Ansible module instead.. The Python 2 yum module is needed for this module. If you require Python 3 support use the dnf Ansible module instead."}
Environment (please complete the following information):

OS: [e.g. RHEL 7.9]
CP-Ansible Branch: [7.3.2-post]
Ansible Version [e.g. ansible 2.14]

Additional context
Add any other context about the problem here.

Leave python2 as is for yum to work; but add this var in your host_vars or group_vars:
ansible_python_interpreter: /usr/bin/python3


Fixed issue #41

Fixes #41 and Improved Formatting of Contributing.md also

@shivanigithub have a look !

Thanks for the PR but the contributing.md file is a standard file that shouldn't be changed. Likewise the readme.md changes seems unnecessary.


Stacked Potions Deleting Items in Shulker Boxes

This bug was reported to me by one of my players, it seems that when you place stacked potions inside of a Shulker Box, then break the Shulker Box, the potions become unstacked and delete items inside of it to make space. For instance, in the attached screenshots, there are four potions stacked together, 3 beetroot seeds, and 25 stone bricks.
When the Shulker Box is broken, and then placed again, the four potions are unstacked, and there are only 22 Stone Bricks in the Shulker Box.

There are no visible errors in the console at all, only that the items simply get deleted to make room for the stacked potions to be unstacked.

Thanks for the bug report! I've been able to fix this bug and a bug fix release will be out soon including this fix.
If you want a technical explanation of the bug:
When breaking a shulker box, items in shulker boxes are automatically stacked back down to their maximum stack amounts for vanilla by default. A fix for Simple Stack had been implemented a while back that had fixed this issue. However, there was an issue with this fix where it was first checking to make sure that the shulker box was whitelisted in the config before it attempted to perform this fix. The issue with that is if shulker boxes aren't added in the config and whitelist mode is on that means that the fix will not be performed and items will unstack and overflow in the shulker box, resulting in the item loss that you noticed in the screenshots above.


Make utilities directory

Contents
Shell scripts to do the following:

Clue export
Clue import
Word list preparation

and others.

Also refactored clue_import.py and clue_export.py to sort puzzle names by date last saved.
Common list_puzzles function is in __init__.py


[5.8] Add message value assertion to assertJsonValidationErrors

Resubmit of #27655.
This PR adds the assertion for JSON validation error messages in assertJsonValidationErrors.
Currently, we can only check json validation error keys:
$this->assertJsonValidationErrors('key');
$this->assertJsonValidationErrors(['foo', 'bar']);
With this addition, you will be able to check the key and the message:
$this->assertJsonValidationErrors(['key' => 'validation message']);
You can still continue to use the normal functionality by just passing a key, array of keys, or a mix of each.

@ahinkle I shipped an augmentation to this method so that it handles arrays of error messages for a single key:


[Models] Edge Network Relationship Between Services and Statefulsets

Current Situation

Currently, Edge Network relationships between Statefulsets and services are disabled.
Proposed Change
This should be enabled because Statefulsets are controllers responsible for managing the lifecycle of Pods and Services route traffic to these pods.

General Contributor Guides and Handbook

🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

Understanding Meshery Models

📚 Models

📚 Components
📚 Relationships

👨💻Models Repository

Contributing to Meshery Models

Contributing Models

Contributing Components
Contributing Relationships

I would like to work on this.

Hi @Alero-Awani @MUzairS15 I am working on this issue; please find a draft of the Edge Network Relationship I was able to come up with below:
json {   "schemaVersion": "relationships.meshery.io/v1alpha2",   "version": "v1.0.0",   "kind": "Edge",   "metadata": {     "description": "A relationship that defines network edges between a StatefulSet and a Service",     "animation": "marching-ants",     "capabilities": {       "designer": {         "edit": {           "style": true,           "config": true,           "label": false         }       }     }   },   "model": {     "schemaVersion": "models.meshery.io/v1beta1",     "version": "v1.0.0",     "name": "kubernetes",     "model": {       "version": "v1.25.2"     },     "displayName": "Kubernetes",     "category": {       "name": "Orchestration \u0026 Management",       "metadata": null     },     "metadata": {}   },   "type": "Network",   "subType": "Network",   "evaluationQuery": "edge_network_relationship",   "selectors": [     {       "allow": {         "from": [           {             "kind": "Service",             "model": "kubernetes"           },           {             "kind": "Ingress",             "model": "kubernetes"           }         ],         "to": [           {             "kind": "Endpoints",             "model": "kubernetes"           },           {             "kind": "Pod",             "model": "kubernetes"           },           {             "kind": "StatefulSet",             "model": "kubernetes"           }         ]       },       "deny": {         "from": [           {             "kind": "Ingress",             "model": "kubernetes"           }         ],         "to": [           {             "kind": "Endpoints",             "model": "kubernetes"           },           {             "kind": "Pod",             "model": "kubernetes"           },           {             "kind": "StatefulSet",             "model": "kubernetes"           }         ]       }     }   ] }


How to make accordion in  table row?

How to make accordion in  table row?

Can you please provide some more information and examples on what you are trying to achieve. If you can also include a JSFiddle indicating exactly what you are wanting.


Screenshot fullpage

Cypress support full page screenshot now. (Implement it by scrolling to the top, then taking a screenshot, scrolling, taking a screenshot, and so on, then stitching them together.)
Is there anyway we are able to compare full page screenshot for regression?
Refer: https://github.com/cypress-io/cypress/issues/1424

that feature seems more more of code snapshot? @CassieRyu  not an image shot... right? I just read through the post a bit more.

I just changed capture to 'fullPage' and it's working fine.

As @kylemh mentioned using capture should do the trick:
cy.matchImageSnapshot({ capture: 'fullPage' }).

The options section of our README mentions how you can pass any Cypress screenshot options to matchImageSnapshot. You can read more about all the options they support in the Cypress docs: https://docs.cypress.io/api/commands/screenshot.html#Arguments.


[docs] Docs are not being generated for Hidden

Imported PropTypes not working with react-docgen
I am encountering an error when running the npm script to build docs:
Error generating markdown for Hidden
TypeError: Cannot read property 'name' of undefined
    at generatePropDescription (/material-ui/scripts/generate-docs-markdown.js:32:7)
    at /material-ui/scripts/generate-docs-markdown.js:144:25
    at Array.reduce (native)
    at generateProps (/material-ui/scripts/generate-docs-markdown.js:142:29)
    at generateMarkdown (/material-ui/scripts/generate-docs-markdown.js:198:8)
    at /material-ui/scripts/build-api-docs.js:60:18
    at tryToString (fs.js:456:3)
    at FSReqWrap.readFileAfterClose [as oncomplete] (fs.js:443:12)

This is happening because react-docgen is not providing required, description, flowType, or type for any of the props defined in src/Hidden/types.js:

xsUp: { defaultValue: { value: 'false', computed: false } }
smUp: { defaultValue: { value: 'false', computed: false } }
mdUp: { defaultValue: { value: 'false', computed: false } }
lgUp: { defaultValue: { value: 'false', computed: false } }
xlUp: { defaultValue: { value: 'false', computed: false } }
xsDown: { defaultValue: { value: 'false', computed: false } }
smDown: { defaultValue: { value: 'false', computed: false } }
mdDown: { defaultValue: { value: 'false', computed: false } }
lgDown: { defaultValue: { value: 'false', computed: false } }
xlDown: { defaultValue: { value: 'false', computed: false } }

It looks like the last time Hidden.md was built was with this commit, but it should have been built when @oliviertassinari updated the docs with PR #6892.
Also, the type listed on Grid for the Hidden property is HiddenProps, which may benefit from more of an explanation.  Maybe a link to the Grid/Hidden demos would be sufficient.
Versions

Material-UI: 1.0.0-alpha.15

react-docgen has an open issue for imported propTypes (I see you in the comments, @oliviertassinari! 👋)

Closing for #6818.

Also, the type listed on Grid for the Hidden property is HiddenProps, which may benefit from more of an explanation. Maybe a link to the Grid/Hidden demos would be sufficient.

Regarding that point. I agree, to you want to submit a PR? :)


Cancel running CI when a new commit is pushed to the branch

Taken from https://stackoverflow.com/a/70972844

:books: Documentation preview :books:: https://equistore--268.org.readthedocs.build/en/268/

@PicoCentauri should we do this on all lab-cosmo repos?

Yes sounds like a very useful addition.


Before performing an RPC service lookup, normalize the URL path

Fixes #650

@tizoc
I just forced push rpc-cache-tweaks here, so closing this


Generic LabeledMatrix type

Hi all,
A project I maintain uses pairwise similarity matricies that are essentially distance matricies pre-normalisation and conversion to distances. I would love to be able to manipulate these matricies in a way that is consistent with DistanceMatrix objects with regards to IO and metadata  storage, but I can not find a type in skbio that allows storing non-hollow labeled square matrices.
Is a generic LabeledMatrix of interest? This would make the inheritance go from
SkbioObject -> DissimilarityMatrix -> DistanceMatrix

to:
SkbioObject -> LabeledMatrix -> DissimilarityMatrix -> DistanceMatrix

If there is I will have a go at implementing one.
Cheers,
Kevin

@kdmurray91 how about a SimilarityMatrix class that supports non-hollow, asymmetric square matrices? If we were to add it, I'd imagine a hierarchy like this:
SkbioObject -> LabeledMatrix -> DissimilarityMatrix -> DistanceMatrix
SkbioObject -> LabeledMatrix -> SimilarityMatrix

where LabeledMatrix is a package-private class that exists purely for code reuse. SubstitutionMatrix (#161) could then inherit from LabeledMatrix in the future.
Does this factoring make sense? DissimilarityMatrix and SimilarityMatrix would have very similar APIs, at least initially, but it seems valuable to make a semantic distinction between similarity and dissimilarity matrices in case there is functionality that expects one over the other.

Sounds great. This would also allow conversion and normalisation functions to turn a SimilarityMatrix into a DistanceMatrix.

Awesome!

One thing though, shouldn't the similarly matrix be symmetric? It's just the non normalised inverse of the distance matrix.

I found some asymmetric similarity measures (e.g., Tversky index, asymmetric text similarity measures) and wikipedia seems a bit fuzzy on the issue (taken from here):
"""
Although no single definition of a similarity measure exists, usually such measures are in some sense the inverse of distance metrics...
"""
From this, I inferred that a similarity matrix is roughly the inverse of a dissimilarity matrix, where asymmetry is possible. Does that make sense? I don't have much experience with similarity measures so could be missing something important here.

That makes sense, yes. You can always store a symmetric similarity matrix,  and then have a helper or member function to convert to a DistanceMatrix that only works if the matrix is symmetric.

Which of those would be preferred, by the way? similarity_to_distance(), SimilarityMatrix.to_distance(), or (a class method) DistanceMatrix.from_similarity()

similarity_matrix.to_distance_matrix()

@kdmurray91 cool, thanks for verifying. We could also have a method to convert a DissimilarityMatrix into a DistanceMatrix, where it would only work if the matrix is symmetric. I think the DistanceMatrix constructor already works this way but having an explicit method is probably a good idea.

similarity_matrix.to_distance_matrix()

:+1:

According to this page, dissimilarity matrices are often hollow (zeros on the diagonal) but don't have to be. I think the symmetry and hollowness checks should only happen in DistanceMatrix, and SimilarityMatrix and DissimilarityMatrix will allow asymmetric, non-hollow square matrices.

Out of curiosity, what's the difference between a DissimilarityMatrix and a SimilarityMatrix? Can it just be a flag in the object? Perhaps it will make more sense to simply have symmetric and asymmetric objects, not sure.

That's a great question! The current DissimilarityMatrix API makes sense for SimilarityMatrix. Are there operations we'd add in the future that only make sense for one type and not the other? Totally unsure about this.
It seems valuable to make a type distinction between similarities and dissimilarities so that, for example, a function expecting a similarity matrix can state that succinctly in numpydoc:
Parameters
----------
sm : SimilarityMatrix

Having different types also allows their APIs to diverge if there are operations that make sense for one but not the other.
If we used a flag, it'd be harder to distinguish between types (at least in numpydoc). Also, if the flag wasn't a required parameter to the constructor, it could be error-prone to suggest a default in case users don't realize the flag exists (e.g., accidentally saying a matrix contains dissimilarities where it really has similarities).

Perhaps it will make more sense to simply have symmetric and asymmetric objects, not sure.

That's an interesting idea. One concern is storage: we'd want to store symmetric matrices in scipy's "condensed form" (docs), which assumes the matrix is hollow. That seems like a reasonable requirement for a distance matrix but not for a similarity matrix.

In biology/ecology we have [similarity and dissimilarity matrices (1 - similarity)] (http://www.davidzeleny.net/anadat-r/doku.php/en:similarity), which are given by the method used. Thus, shouldn't be easier to simply have one object to represent either one and have a property with the name of the metric and a boolean to see if it's similarity or dissimilarity? Also, this could give the opportunity to store all of them as hollow condensed matrixes and show the true values based on the boolean.

@antgonza this is more or less what I will do. Essentially all DissimilarityMatrix functionality will be moved to LabelledMatrix, and both SimilarityMatrix and DissimilarityMatrix will be thin subclasses of this object. I would like to keep them a separate objects, for clarity in user code and in the documentation as @jairideout explains above. It would also allow divergence in the API if we decide that's a good thing in the future (e.g. with similarity_mat.to_distance_matrix() discussed above).

@antgonza:

In biology/ecology we have similarity and dissimilarity matrices (1 - similarity)

(1 - similarity) is one way to convert similarities to dissimilarities, but there are many other ways to do this (e.g., 1 / similarity) and the "correct" conversion depends on your data and the metric used (e.g., see this R mailing list discussion).

Thus, shouldn't be easier to simply have one object to represent either one and have a property with the name of the metric and a boolean to see if it's similarity or dissimilarity?

See my previous comment for reasons why I think having separate types are better than a flag.

Also, this could give the opportunity to store all of them as hollow condensed matrixes and show the true values based on the boolean.

We cannot store similarities or dissimilarities in condensed form because the matrices must be hollow and symmetric, which isn't always true (e.g., UniFrac Gain or Tversky index). Thus, I think it makes sense to have SimilarityMatrix and DissimilarityMatrix store the full matrices, and provide a method to convert to DistanceMatrix, which will only allow matrices that can be stored in condensed form.

@kdmurray91, thanks, I think that separation makes sense.
@jairideout thanks for the explanation. I'm just worry that we are over designing something that we are not sure about, as you mentioned before, we are not sure what we will need. Anyway, I see the benefits and more with @kdmurray91 plan.


DSO-15449 improve IR stream footer text.

Remove the equal mark from IR stream footer text on mouse picking.

There were conflicts which I fixed... waiting on Travis


Question: what is the purpose of spi->could_be_busy?

I'm trying to understand in what situation the spi->busy_pre_check routine would be executed.`
This flag seems to be set in only one condition:
  if (spi->could_be_busy) {
    spi->busy_pre_check = 1;
  }

The  spi->could_be_busy flag does not seems to get set anywhere.
In what case would the busy_pre_check would be triggered?
Am I missing something?
Thanks for this great library.

Hi, it was some time since I wrote this, but I have a vague recollection of that the user can set could_be_busy if there are other drivers acting on the spi flash also. If not, and this driver is the lone and only actor, it is always false and we do not need to waste time checking the busy flag.
I'll add a comment in near future. Thanks for pointing this out, and thanks for the kind words. 🙏

Thanks for your answer.
I did not understand this use case.
It was a pleasure to read your code. I like the way you decoupled the sync and async.


Fix spelling mistake

Thanks, @srpomeroy!
@AjayTripathy are you able to push changes to prod?

Looks like it already has.


FIX: gracfully handle missing seaborn dependency

xref: https://github.com/pydata/xarray/pull/9561#discussion_r1861273284
Fixes  a regression I introduced in #9561
MWE:
On current main, this code will fail if seaborn is not installed even if matplotlib is installed:
conda create -n xr-test python=3.10 xarray pooch scipy matplotlib

import xarray as xr
air = xr.tutorial.open_dataset("air_temperature")
air.air.isel(time=0).plot.contourf(colors="k")

As mentioned in https://github.com/pydata/xarray/pull/9561#discussion_r1861273284 , it seems that this wasn't caught by a CI because there isnt' a CI that has matplotlib but not seaborn installed.

Thanks a lot for checking & following up!

Thanks a lot for checking & following up!

No problem, figured I should fix what I broke! 😅

Thanks!


multiple adapter model inference

hey folks in the adapter community,
I am looking for a way to inference multiple NLP services eg NER, POS, QA, summarization, chatbot using a single given base model like LLaMa-2 where concurrent users use multiple services but the base model is shared across other adapters.
hoping such an implementation would reduce GPU utilization significantly as instead of using parallel finetuned models, one base model with mutliple adapters could be used.
requesting anyone with such info reach out, any help woiuld be highly appreciated.

@StephennFernandes I am also interested in this use case. Did you find a solution?

Hi,
I'm not sure if I understand you correctly, but this sounds like a case for the Parallel composition block.
This block can be used to load and use multiple adapters in parallel, each with its own prediction head.
I took the example from the docs:
from adapters import AutoAdapterModel
from transformers import AutoTokenizer
import adapters.composition as ac

model = AutoAdapterModel.from_pretrained("distilbert-base-uncased")
tokenizer = AutoTokenizer.from_pretrained("distilbert-base-uncased")

adapter1 = model.load_adapter("sts/sts-b@ukp")
adapter2 = model.load_adapter("sts/mrpc@ukp")

model.active_adapters = ac.Parallel(adapter1, adapter2)

input_ids = tokenizer("Adapters are great!", "Adapters are awesome!", return_tensors="pt")

output1, output2 = model(**input_ids)

Each adapter generates an output. Depending on which service is required, you can then process the specific output further.
The section in the docs can be found here if you want some more information about possible composition blocks.
Hope this helps!
Best,
Timo


Replace __invoke() to toNode()

Description
In order to improve readability I renamed the __invoke() functions to use toNode() instead.
Changes

Use toNode() instead of __invoke() for the Analyzer objects and TupleSymbol.
Introduce an interface (TupleSymbolToNode) for all TupleSymbol so they are easier to build.

Look the refactoring done in AnalyzeTuple.

Instead of toNode I would call it analyze. The interface is a good idea, but isn't it bad that the return type of the methods is different? (For example DefNode instead of Node). Seem like PHP isn't that strict on this currently.

Well, actually, DefNode is a Node because it extends from it. The interface ensures that the returning type is a Node, so any of its children are, by definition, a Node as well.