Building image with CpusetCpus set. Running image shows blank CpusetCpus

Hi,
Quick question:
Just noticed that after building an image with --cpuset-cpus="0,1", and running a container from that image, the "inspected" CpusetCpus is "". A complete example can be seen below
Is this behavior expected -> all platform dependent stuff has to be explicitly defined in the docker run command?
Thanks!
Stefan
--
root@e097e49d1bb1:/go/src/github.com/docker/docker# cat my_dockerfile
FROM fedora:latest
RUN echo hi
root@e097e49d1bb1:/go/src/github.com/docker/docker# docker build -t my_image --cpuset-cpus="0,1" - < my_dockerfile
Sending build context to Docker daemon 2.048 kB
Step 0 : FROM fedora:latest
---> ded7cd95e059
Step 1 : RUN echo hi
---> Running in 22c6de7093fd
hi
---> 1364b51c13d3
Removing intermediate container 22c6de7093fd
Successfully built 1364b51c13d3
root@e097e49d1bb1:/go/src/github.com/docker/docker# docker run --name=my_container my_image sleep 100 &
[1] 31161
root@e097e49d1bb1:/go/src/github.com/docker/docker# docker ps -a | grep my_container
a82c0ae15191        my_image            "sleep 100"         14 seconds ago      Up 12 seconds                                   my_container
root@e097e49d1bb1:/go/src/github.com/docker/docker# docker inspect a82c0ae15191 | grep CpusetCpus
"CpusetCpus": "",
root@e097e49d1bb1:/go/src/github.com/docker/docker#

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

I haven't found a duplicate for this issue
I will post relevant information upon request

This is expected behavior.
Resource constraints (among other things) are not saved as part of an image.


Update scripts.js

Closes #
📑 Description

✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[x] I have updated the documentation as required
[x] All the tests have passed

☢ Does this introduce a breaking change?

[ ] Yes
[x] No

ℹ Additional Information

Description by Korbit AI
What change is being made?
Introduce and utilize an updated gaugeOptionsUsage object with specific configurations for rendering gauge charts in showCPanel.
Why are these changes being made?
The new gaugeOptionsUsage object introduces more detailed and specific visual configuration options for gauge charts, such as setting color ranges (green, yellow, red) to enhance data visualization clarity. This setup makes the UI component more intuitive for end-users by highlighting different data thresholds.

Is this description stale? Ask me to generate a new description by commenting /korbit-generate-pr-description

Summary by CodeRabbit

New Features

Enhanced visual representation of usage data with updated gauge chart configurations.
Improved clarity in color coding for data interpretation through new thresholds.

Bug Fixes

Removed unnecessary console logging for usage data to streamline performance.

@gstraccini codacy bypass


Dynamic Nav Bar Color

Navbar starts with a black background.
When user scrolls down after the 'SOAR' logo, navbar turns to red

Currently, the navbar would start with the grey colour and transitions to red when reaching the about section. It will also revert when going back up.
However, buttons on the navbar are currently not configured to follow the same behavior and thus, they will remain red throughout.


Eslint

Convert JSHint && JSCS rule to eslint with polyjuice.
And manually do some correction.
JSHint rules are totally compatible, JSCS ones are not that lucky.
The rules isn't compatible had been commented with TODO label.

Nice tool, I did not know about it!

Ok I checked and I have still some issue with eslint. The major one is related to spaces, and how it makes no differences for spaces inside function signature and outside. And issue is opened about that since a while now :( https://github.com/eslint/eslint/issues/3587
Maybe you'll got a solution about this.
In the mean time could you update your PR to add eslint into the package.json, remote jscs & jshint, and update the lint scripts entry?

In the mean time could you update your PR to add eslint into the package.json, remote jscs & jshint, and update the lint scripts >

Sure, give me some time i will finish it.

And issue is opened about that since a while now :( eslint/eslint#3587

Hahaha, such a long time, sigh.Maybe we need a PR :).

As your wish, done.:laughing:

I could not take your PR, too much changes on client.js + some tab / space weirdness elsewhere so I cherry pick some and re-write some part. thank you @yeyu456 this project is now eslint!


Add Response to DownloadTarget or to the callback?

We currently don't have access in the DownloadTarget to the Response object of the CURLHandle.
I am wondering if we should expose it. It would be useful to be able to check e.g. the http_status in user code.

We do have the Response now as part of the end_callback.


Orientation/Rotate Issues and Warnings

I am getting a few warnings when running an IOS build with NativeScript. The main issue is that changes in orientation are not being handled correctly. When rotating the phone, I can see that the orientation change event is registering and correctly recognizes the new orientation but the view does not adjust appropriately.
My guess is that this behavior is related to the warnings generated below.
I have gone through the latest update process and should be up to date on everything. I have not been able to find any issues related to these warnings and NativeScript anywhere. Does anyone know anything about how to solve this issue.
`[19-03-14 10:27:30.059] (CLI) 3 warnings generated.
[19-03-14 10:27:30.833] (CLI) ****/platforms/ios/Pods/TOCropViewController/Objective-C/TOCropViewController/TOCropViewController.m:458:1: warning: implementing deprecated method [-Wdeprecated-implementations]
[19-03-14 10:27:30.833] (CLI) - (void)willRotateToInterfaceOrientation:(UIInterfaceOrientation)toInterfaceOrientation duration:(NSTimeInterval)duration
[19-03-14 10:27:30.833] (CLI) ^ [19-03-14 10:27:30.833] (CLI) In module 'UIKit' imported from /Users/benjaminb/Desktop/projects/HelloWorld/platforms/ios/Pods/Target Support Files/TOCropViewController/TOCropViewController-prefix.pch:2: [19-03-14 10:27:30.833] (CLI)
/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS12.1.sdk/System/Library/Frameworks/UIKit.framework/Headers/UIViewController.h:328:1: note: method 'willRotateToInterfaceOrientation:duration:' declared here
[19-03-14 10:27:30.833] (CLI) - (void)willRotateToInterfaceOrientation:(UIInterfaceOrientation)toInterfaceOrientation duration:(NSTimeInterval)duration NS_DEPRECATED_IOS(2_0,8_0, "Implement viewWillTransitionToSize:withTransitionCoordinator: instead") __TVOS_PROHIBITED; [19-03-14 10:27:30.833] (CLI) ^ [19-03-14 10:27:30.833] (CLI) ***/platforms/ios/Pods/TOCropViewController/Objective-C/TOCropViewController/TOCropViewController.m:490:1: warning: implementing deprecated method [-Wdeprecated-implementations]
[19-03-14 10:27:30.833] (CLI) - (void)willAnimateRotationToInterfaceOrientation:(UIInterfaceOrientation)toInterfaceOrientation duration:(NSTimeInterval)duration
[19-03-14 10:27:30.833] (CLI) ^ [19-03-14 10:27:30.833] (CLI) In module 'UIKit' imported from ****/platforms/ios/Pods/Target Support Files/TOCropViewController/TOCropViewController-prefix.pch:2: [19-03-14 10:27:30.833] (CLI)
/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS12.1.sdk/System/Library/Frameworks/UIKit.framework/Headers/UIViewController.h:331:1: note: method 'willAnimateRotationToInterfaceOrientation:duration:' declared here
[19-03-14 10:27:30.833] (CLI) - (void)willAnimateRotationToInterfaceOrientation:(UIInterfaceOrientation)toInterfaceOrientation duration:(NSTimeInterval)duration NS_DEPRECATED_IOS(3_0,8_0, "Implement viewWillTransitionToSize:withTransitionCoordinator: instead") __TVOS_PROHIBITED;
[19-03-14 10:27:30.833] (CLI) ^ [19-03-14 10:27:30.833] (CLI) ****/platforms/ios/Pods/TOCropViewController/Objective-C/TOCropViewController/TOCropViewController.m:515:1: warning: implementing deprecated method [-Wdeprecated-implementations]
[19-03-14 10:27:30.833] (CLI) - (void)didRotateFromInterfaceOrientation:(UIInterfaceOrientation)fromInterfaceOrientation [19-03-14 10:27:30.833] (CLI) ^ [19-03-14 10:27:30.833] (CLI) In module 'UIKit' imported from ***/platforms/ios/Pods/Target Support Files/TOCropViewController/TOCropViewController-prefix.pch:2:
[19-03-14 10:27:30.833] (CLI) /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS12.1.sdk/System/Library/Frameworks/UIKit.framework/Headers/UIViewController.h:329:1: note: method 'didRotateFromInterfaceOrientation:' declared here
[19-03-14 10:27:30.833] (CLI) - (void)didRotateFromInterfaceOrientation:(UIInterfaceOrientation)fromInterfaceOrientation NS_DEPRECATED_IOS(2_0,8_0) __TVOS_PROHIBITED; [19-03-14 10:27:30.833] (CLI) ^`

Hi @bbarne8, These warnings are coming from the TOCropViewController CocoaPod library that you are using. I suggest that you make sure you're using the latest version and if the problem persists with it I suggest that you create an issue in TOCropViewController's repository.


Can this plugin work if attachments are in a subfolder like this? ... Vault / Folder / attachments / {notename} / ....  here comes attachment

I am using Attachment MANAGER plugin because I couldn't figure out the Attachment MANAGEMENT plugin here to work the way I need. I want to save attachments to this folder:
Vault / Folder / attachments / {notename} / ....  here comes attachment
So it's one level deeper into a subfolder in attachments folder. The subfolder is named after the note name.
The issue is I can't figure out how to make it work with this plugin here.
So instead I am using Attachment MANAGER, another plugin which works. But.. I also need to rename the attachments. For that I was hoping to use the Attachment Name Formatting plugin, but renaming only works if the attachments are in the attachments folder but doesn't work if attachments are one level deeper in a subfolder, like it's my case.
So I am wondering whether this plugin here Attachment MANAGEMENT can be configured the way I need.
I tried this but didn't work (and number of variations of this ... .. and of course I have other plugs disabled when testing to make sure there are no conflicts):

Here is my Obsidian setting:

Hey, is this the layout you're looking for?

My settings for "Files and links"

My settings for "Attachment Management"


add config to choose pretty/ndjson

pub fn with_level(level: log::LevelFilter) {
    #[cfg(target_arch = "wasm32")]
    wasm::start(level);

    #[cfg(not(target_arch = "wasm32"))]
    {
        // Use ndjson in release mode, pretty logging while debugging.
       // here add some cfg! to choose pretty/ndjson
        if cfg!(debug_assertions) {
            pretty::start(level);
        } else {
            ndjson::start(level);
        }
    }
}

hey @lagudomeze ! To make sure I understand this correctly, you're asking for an explicit feature attribute to set for pretty logging? Meaning you'd want pretty logging in production? Would be curious to hear your use case for this.
When thinking about this, I expected parseable logs (for this lib it's ndjson) in production.

https://github.com/lrlna/femme/pull/23 should address this

Hi @lrlna , it would be great if it will be possible to specify timestamp in pretty print too, and to change timestamp unit (seconds, milli, micro or nano)


[material-ui][Transitions] External ownerState is incorrectly forwarded to inner components

Steps to reproduce
Link to live example: https://codesandbox.io/p/sandbox/cold-firefly-dpmqj7?file=%2Fsrc%2FDemo.tsx%3A29%2C5
Steps:

Provide an ownerState prop to Collapse transition
Set transition as in={true} initially

Current behavior
The transition is collapsed initially, even though it shouldn't.
Expected behavior
The transition should be expanded initially.
Context
This is a very mild symptom of a bigger underlying problem: If an external ownerState is provided to a transition component, then that ownerState is forwarded to the inner components and treated as the component's own ownerState. Here's a trace of this using Collapse as an example:

The Collapse component receives an ownerState prop
The ownerState property goes into ...other
The other is provided to Transition
Transition does not recognize the ownerState prop, so it forwards it as childProps
childProps.ownerState overrides Collapse's own ownerState, and the animation breaks.

The symptom is only perceivable on the initial render as for subsequent enter calls, the size in handled on the onEntered event.
The impact of this is that the transition components are unsuitable for usage with the slots pattern: We would like to be able to provide the ownerState for custom transitions used in components' slots, but this could have unintended consequences because of the behavior explained above.
Another symptom this has is that the Collapse transition does not work for the Tooltip component, as it provides ownerState to the transition component.
It is not very noticeable at the moment as the Fade, Grow, Slide, and Zoom transitions do not rely on an internal ownerState.
Discovered while working on #40418. When fixing this issue, we should remove this workaround.
Your environment

npx @mui/envinfo
  System:
    OS: macOS 13.4.1
    CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 28.03 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.19.0 - ~/.nvm/versions/node/v18.19.0/bin/node
    npm: 10.2.3 - ~/.nvm/versions/node/v18.19.0/bin/npm
    pnpm: 8.13.1 - ~/.nvm/versions/node/v18.19.0/bin/pnpm
  Managers:
    Homebrew: 4.2.0 - /usr/local/bin/brew
    pip3: 21.2.4 - /usr/bin/pip3
    RubyGems: <IP_ADDRESS> - /usr/bin/gem
  Utilities:
    Make: 3.81 - /usr/bin/make
    GCC: 14.0.3 - /usr/bin/gcc
    Git: 2.39.2 - /usr/bin/git
    Clang: 14.0.3 - /usr/bin/clang
    Curl: 7.88.1 - /usr/bin/curl
  Servers:
    Apache: 2.4.56 - /usr/sbin/apachectl
  IDEs:
    VSCode: 1.85.1 - /usr/local/bin/code
    Vim: 9.0 - /usr/bin/vim
    Xcode: /undefined - /usr/bin/xcodebuild
  Languages:
    Bash: 3.2.57 - /bin/bash
    Perl: 5.30.3 - /usr/bin/perl
    Python3: 3.9.6 - /usr/bin/python3
    Ruby: 2.6.10 - /usr/bin/ruby
  Databases:
    SQLite: 3.39.5 - /usr/bin/sqlite3
  Browsers:
    Chrome: 120.0.6099.216
    Safari: 16.5.1
  Monorepos:
    Lerna: 8.0.2

Search keywords: transitions collapse ownerstate

@DiegoAndai
What are your thoughts about setting this prop after the spread operation of childProp? This way, the forwarded ownerState from the Transition component won't be able to override the internal one. By doing so, we can revert the delete operation here, and ownerState can then be forwarded to the custom transition component if needed.

@gitstart

What are your thoughts about us setting this prop after the spread operation of childProp?

I think it would be better to avoid destructuring ownerState to ..other here, so it never goes through childProps. That's more explicit for future maintainers, so it would be harder to overlook. What do you think?

@DiegoAndai, for the benefit of future maintainers, we can add comments that explicitly explain the rationale behind relocating the ownerState prop after the childProp destructuring.
Even though the ownerState is abstracted from ...other , it will eventually be passed to the transition component because we want to allow users access to the ownerState in custom transition components. While we could attempt to check if users provide a custom transition component, the issue may still persist if the custom transition component provided by the user is the same as the one used by MUI that doesn't handle ownerState.

Even though the ownerState is abstracted from ...other , it will eventually be passed to the transition component because we want to allow users access to the ownerState in custom transition components.

Good point. Let's try moving the ownerState prop after the childProps spread as you suggested.


Hybrid Identity Administrator role does not work for kerberos key roll over

[Enter feedback here]
Hybrid Identity Administrator role causes error:
Uploading computer account information failed. Error message: Internal error: An error occurred during the 'DesktopSsoNumOfDomains' process.
Global Entra ID Administrator role is working fine.
Please update the docs.

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

ID: 20f01328-ebfa-31af-4e67-6de92c8f924e
Version Independent ID: a990551f-e6c9-dede-308a-b5578fb7ee1e
Content: Microsoft Entra Connect - Microsoft Entra ID
Content Source: docs/identity/hybrid/connect/how-to-connect-sso-faq.yml
Service: entra-id
Sub-service: hybrid-connect
GitHub Login: @billmath
Microsoft Alias: billmath

@palito1980
Thanks for your feedback! We will investigate and update as appropriate.

Following up with engineering

@palito1980
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly.

Thank for submitting this.  I have not heard back from engineering yet so I have created a work item to investigate this further. I am not sure what the timeline will be, so I am going to close this for now. But once I have an update I will drop it in here and let you know.
Thank you!
Bill
#please-close


[Mat-Table & Mat-Checkbox] When user click more than one checkbox given the functions are not working in @Angular6

Bug, feature request, or proposal:
HTML File:-

<ng-container matColumnDef="position">
   Organisations
   {{element.position}}
</ng-container>

<ng-container matColumnDef="server">
   Server Name
   {{element.server}}
</ng-container>

<ng-container matColumnDef="company">
   Company Code
   {{element.company}}
</ng-container>

TS File:-
formedArray: any = [];
serverArray: any = [];
numberArray: any = [];
yes: Boolean = true;
showNumber() {
this.yes = !this.yes
if (this.yes === false) {
let newArray = this.displayedColumns.splice(0, 1)
console.log(this.yes)
this.formedArray = newArray
console.log(this.formedArray.length)
} if (this.yes === true && this.formedArray.length > 0) {
this.displayedColumns.splice(0, 0, this.formedArray[0])
console.log(this.displayedColumns)
}
}
showServer(){
}
showName(){
}
What is the expected behavior?
Need when user unchecked the checkbox field more than one unchecked also it should work.
What is the current behavior?
Now If user click more than one check box the functionality & behaviour totally change.It push two time same column or it doesn't shown what user click.
What are the steps to reproduce?
I'm doing inside the burger menu, I have a 3 mat-checkbox,When user click the burger menu icon.It shows available 3 mat-checkbox and one mat-table .inside the mat-table available column name what i given the checkbox.If user uncheck the checkbox .It will affect the mat-table full column should be hide.If i uncheck more than one checkbox the function not working.
What is the use-case or motivation for changing an existing behavior?
It's should work more than one checkbox also.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Latest Version Angular & Material 6+
Is there anything else we should know?

Please keep GitHub issues for bug reports / feature requests. Better avenues for troubleshooting / questions are stack overflow, gitter, mailing list, etc.


String without delimiter should not be considered a valid hash

Description
Fixes another bug that prevents bug from #80 from being fixed

Closing this for now as there is no appetite to touch coercion logic.


Handle the situation when our desired sandbox is empty

Neat! One more thing I can cross of the TODO list! :)
Can you add a test for this change to test/Tinc/PackageSpec.hs?
And congrats, you are probably one of the first tinc users outside of Zalora!
BTW, @soenkehahn and I have been working hard on support for "git dependencies".  That will hit master soon ;)

@sol today tinc has spoilt me, I don’t even want to work on a cabal idea I had :P

I applied this and did some refactoring (see notes bellow).

@llelf thanks a lot for tracking this down :)

In addition, we print an error message on unexpected output from cabal install --dry-run now https://github.com/zalora/tinc/commit/aec54c1b1639dcf280c156291d8e2b20bdc3f850


Not loading tweets

The normal twitter always have another tweet at the bottom of feed so the users wouldn't stop scrolling.
Is this a bug?

I can scroll down just like without the plugin. Repro process would help.

This sometimes happens here too, but I think this is a Twitter issue

I would suggest you try reproducing this issue in incognito mode, i.e. with all other extensions off.

I thought I've closed this issue, the extension works as it should. There is no bug.


New Crowdin updates

:tada: This PR is included in version 1.10.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add ABIs for CoboSafe on mainnet

Cobo Safe Module has launched on mainnet: https://etherscan.io/address/0x09bd0ad84c44abd7b36c7f7bfff90776b9c42ae6
Please decode all the methods in the contract. Thank you!

If it's verified on Sourcify and etherscan it will be picked by the service 😉

Thanks!


Move docs deployment from Netlify to Vercel

[x] Create project on Vercel
[x] Make sure redirects are set properly in website-v8 repo
[x] Page change Github action for Vercel links
[x] AssetPrefix

#3545


Are masks needed if we use classification models like dfm ?

I tried to run dfm to classify my images as anomalous/non anomalous. I made my dataset in MVTec hierarchy but I get an error when i run the train script with the error being that the trainer can't access segmentation masks. Are they mandatory when the task is to classify? Below is a the error I get.
File "C:\Users\hp\anomalib\env\lib\site-packages\torch\utils\data_utils\fetch.py", line 44, in
data = [self.dataset[idx] for idx in possibly_batched_index]
File "c:\users\hp\anomalib\anomalib\data\mvtec.py", line 256, in getitem
mask = cv2.imread(mask_path, flags=0) / 255.0
TypeError: unsupported operand type(s) for /: 'NoneType' and 'float'

@MhdKAT, it's the same issue that was found in folder dataset. Need to propose the same fixes to address the issue.
@LukasBommes, yes that's on our todo list, but didn't have a chance to work on it this week unfortunately.

I see ! Thanks for the clarification @LukasBommes,  @samet-akcay.


🐛 Bug Report: OwnedEntityPicker generates invalid catalogFilter when original catalogFilter is array

📜 Description
OwnedEntityPicker intends to add owner relationship criteria to an EntityPicker-compatible catalogFilter, but EntityPicker accepts an ORed array of filters while the OEP code assumes that catalogFilter is not an array. The resulting generated filter query yields no results at all.
👍 Expected behavior
OwnedEntityPicker should accept the same catalogFilter structure as EntityPicker and display the discovered catalog entities, filtered by ownership.
👎 Actual Behavior with Screenshots
This is a screenshot of the generated catalog API request.
👟 Reproduction steps
Create a template that uses OwnedEntityPicker and specify the catalogFilter as an array, e.g.:
ui:field: OwnedEntityPicker
ui:options:
  catalogFilter:
    - kind: Resource
      spec.type: foo

Where your test user owns some such resource. The template will not generate selections.
📃 Provide the context for the Bug.
No response
🖥 Your Environment
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
Yes I am willing to submit a PR!

Closed by https://github.com/backstage/backstage/pull/25359


Master

Update V3 Thirdweb

Pull request successfully merged and closed
You’re all set—the master branch can be safely deleted.


Rewrite of libratbag in Python

I've now spent enough time on this to gather some feedback or general opinions, so I'd appreciate exactly that or those.
I have a github repo here that is a rewrite of libratbag in Python: https://github.com/whot/ratbag-python, documentation at https://whot.github.io/ratbag-python/ratbag.html#ratbag.Ratbag
The code layout is hopefully fairly straightforward:

The ratbag package has all the Device, Profile, Button etc. classes. That is the "public API".
ratbag.drivers has the helpers to write a driver but it comes down to: a driver is ratbag/drivers/drivername.py and the driver has a single probe() function which is expected to set up a ratbag.Device
the rest is handled by signals between the objects, so user tools signal back to the driver (e.g. commit), the driver signals forward (e.g. resync).

It's woefully incomplete, there is part of the hidpp20 driver (ratbag/drivers/hidpp20.py) and some of the roccat driver (ratbag/drivers/roccat.py), both of which can dump parts of the device state. So it's not a replacement but it's at the state where it's sufficient to get the point across.
The eventual goal is to have a drop-in replacement.  ratbagd will be an executable that hooks the DBus server in front of that. But unlike libratbag there is no strict library separation/hard API between the two because we've learned that lesson already, ratbagd will simply use the subset of ratbag that we can support over DBus.
Tools with root access can use the module directly which gives us better extensibility. We can have libratbag be more flexible and featureful and ratbagd be more limited (if necessary). So ratbagd+Piper might only configure some parts of the device, but there may be a ratbag-roccat tool that can access the full functionality of roccat devices like, e.g. report rates per profile.
I have some recording and replaying functionality implemented (ratbag.recorder and ratbag.emulator) so it's possible to record the device interaction and then use that recording to replay the interactions. The idea here is to have recorders/emulators that do whatever we need at the time, making it easier to analyze/debug/reproduce. Right now, they record/replay hidraw level data.
It also makes it easier to reproduce but also to test things in the CI - we can emulate devices and thus test the drivers without needing physical devices.
It uses the same data files to dynamically load the driver (ratbag.drivers.roccat for example), to test just symlink to the data dir in libratbag.git.
cc @FFY00, @stephanlachnit, @perigoso, @bentiss, @amfern, @phomes (did I forget anyone? please cc)

What's the idea on porting drivers? Rewrite them or have a bridge using e.g. ctypes for the meantime?

When I had a look at the drivers, most of them are relatively short (excepting hid++). So porting should be possible, alas it's obviously hard to test them without access to the specific hardware. Porting would also give us the recording/replay facility, better logging and maybe even dump some stuff we don't keep (e.g. I'm planning on dumping the button types because afaict nothing uses them).
Writing a ctypes bridge requires emulating significant parts of the ratbag API which is likely more effort than just porting the driver.

I love it.
Do you want PRs/issues to your repo?
hidtools claims to not have a stable API. Will that change?
https://gitlab.freedesktop.org/libevdev/hid-tools#hidtools-python-module

Do you want PRs/issues to your repo?

I think it's too early, I'm still heavily rebasing and messing around to get the API right. If you find an obvious bug please go for it but I'll likely have to squash it in. I'm hoping to have something stable enough for contributors by end of next week.

hidtools claims to not have a stable API. Will that change?

I'm using hidtools right now to make it easy to extract the report ids we need. Once we have two-three drivers we'll have a good overview on what we actually need from hidtools. If it's just the report IDs we may be better off just writing a mini parser (like libratbag has). Otherwise, I'll have to talk to @bentiss to see if API can be made stable - but right now the hidtools API is clunky, it's grown organically for its tools and is not a great API to actually work with.

FWIW, I have a couple projects that provide a stable API for most of what hidtools supports.
https://github.com/FFY00/python-ioctl
https://github.com/FFY00/python-uhid
https://github.com/usb-tools/python-hid-parser

@FFY00 - thanks, python-hid-parser looks like it's worth switching to, there are a few teething problems touch that I've filed issues for

I'm hoping to have something stable enough for contributors by end of next week.

So, I got the roccat driver working, I can change resolutions and button assignments. It's not fully tested but at least that driver shouldn't need any huge restructurings. In the process a few things got cleaned up in the API and some extra helpers added that should make writing hid drivers a breeze.

I like it, will try to contribute when it's ready, right now I get type mismatch errors.
I think I understand the reasons to rewrite libratbag in python.

It's easier to find developers and build community(this comes up often in the issues)
Configuring mouse doesn't need the speed of C, so trading speed for ease of development is fine.
looking around, similar projects, i see people prefer to use Python.  git@github.com:pwr-Solaar/Solaar.git, https://github.com/flozz/rivalcfg and many more

But I still have to ask: why departure from C?

good question. libratbag started in 2015 with the intention of producing a generic library that could be used by projects to control the mouse. This made C a natural fit since it's only real advantage is that it can be used by anything. Rust wasn't really an option for us back then for multiple reasons. Fast forward a few years and we realised that ratbagd is the only useful way to interact with it, so we merged that into libratbag and stopped bothering too much about the library being perfect. IOW, we no longer have the same focus that we had originally.
IMO even ratbagd is a wrong split - it's aimed at being able to easily write applications but we only have one full consumer (Piper) and while that is available as a flatpak it cannot work without ratbagd of the same version running. A polkit check would be a lot more useful than the current "you need one system package" + flatpak split.
So right now, we effectively have a standalone application (ratbagd) in what is possibly the worst language to write standalone applications in. The things we care about are all incredibly complicated to do in C and hard to get right from the callers. Just the simple "here's a list of dpis" required wrappers in C and then unwrappers on the other side - so we hardcode a (slightly incorrect) smaller list of DPIs to make it less awful (ratbag_resolution_set_dpi_list_from_range). In python, the same thing is a dpis = list(range(200, 8200, 50)) and caller filtering is equally simple with list comprehensions.
For a project like libratbag, the killer feature is how many devices we support and how well we do that feature-wise. Speed doesn't come into it, even if it takes 10s to write a profile to a device you'd still prefer that over not having that device supported at all. So making it simple to add devices (and tools to help doing that) is the most important thing and C, quite frankly, gets in the way of that.

I've now spent enough time on this to gather some feedback or general opinions, so I'd appreciate exactly that or those.
I have a github repo here that is a rewrite of libratbag in Python: https://github.com/whot/ratbag-python, documentation at https://whot.github.io/ratbag-python/ratbag.html#ratbag.Ratbag

I like the idea.

The eventual goal is to have a drop-in replacement. ratbagd will be an executable that hooks the DBus server in front of that. But unlike libratbag there is no strict library separation/hard API between the two because we've learned that lesson already, ratbagd will simply use the subset of ratbag that we can support over DBus.

IMO even ratbagd is a wrong split - it's aimed at being able to easily write applications but we only have one full consumer (Piper) and while that is available as a flatpak it cannot work without ratbagd of the same version running. A polkit check would be a lot more useful than the current "you need one system package" + flatpak split.

How do you image DBus-API changes going forward? I feel if there is an entire rewrite anyway, maybe we can also rethink this. Features like #953, #931 and #870 newer went through, iirc mostly due to DBus concerns. If instead we import a python ratbagd module, adding new features which a changing DBus interface would be less of a hassle, as long as we can keep the Python interface somewhat stable (deprecating old commands after a couple of version still should be no problem as I don't see that many other applications than Piper coming).
Regarding Piper, since I want to spend some time to learn GTK4 with Python anyway, I could also imagine a "rewrite" of Piper (too much of the current code is not understood by anyone). But since time is of course limited as always, no promises.

cc [...] (did I forget anyone? please cc)

Maybe @skitt

How do you image DBus-API changes going forward

Simply said - I think we should get rid of it. There are two use-cases for ratbag, one is Piper, one is ratbagctl. Piper can work with polkit, ratbagctl can run as root. There's no real case for DBus imo, it just adds another API for less benefit that we expected.
My handwavy plan/idea is that ratbag itself will expose anything that's reasonable to support, even more device-specific things that we traditionally used to ignore unless they were common enough. ratbagctl can access those [1] (much easier than now), Piper will have to be more limited out of necessity. So, the interface should be reasonably stable but not the same set-in-stone that we'd need if we had more consumers. It may be worth merging Piper into the same repo too but I haven't thought that through yet.
[1] the current ratbagcli in the above repo takes a yaml file with the configuration and applies that which makes the commandline interface less crazy than ratbagctl is.

@whot Your answer really clarified things for me, I agree with you on the point that C is gets in the way.

Rust wasn't really an option for us back then for multiple reasons
I had a lot of frustration working with dynamically typed languages because bugs with type mismatch are detectable only during runtime and I think Rust is great but overkill for configuring mouse, I think GO Lang can be a perfect fit as it pretty easy to write and enforces type checking, but I may be wrong on that.

That said, I really like the direction and would like to contribute to https://github.com/whot/ratbag-python

I agree, Go would probably work well for the reasons you mention, but at least I don't have any familiarity with it beyond some tutorials. I don't think Go is as easy to learn/add as Python is either, but that may just be my bubble here.
I don't disagree on the frustrations with type mismatches, I've run into those as well. Type annotation in Python is a thing but I haven't used them yet, @FFY00 may have some comments on their usefulness.
All that aside, the repository is now in a decent-enough state (still Alpha) that I won't be rebasing master anymore, so I'll be happily accepting PRs :) Right now only the roccat driver works and you need to symlink the data/ directory from libratbag.git (it falls back to /usr/share/libratbag too). The obvious immediate targets are:

more drivers ported over
polkit integration to be able to open devices without privs

And a better name than ratbag-python would be good too :) While I don't want to drop the ratbag name given we have a logo and what not, "ratbag-python" seems lazy and the language should not be at the core of a project.

libratbag is 0.xx which makes ratbag2 not 100% logical but no one has to know.

we can always retroactively rename the current libratbag, and have the new one br libratbag, but that can lead to complications i suppose, and maybe confusion, still worth considering

How about pyratbag?

Hey everyone, thank you for sharing your thoughts. I am in favor of this, there isn't really any need for libratbag to be written in a low level language like C, and moving to a language like Python has many benefits (lower maintenance/development burden, better testing, etc).

I don't disagree on the frustrations with type mismatches, I've run into those as well. Type annotation in Python is a thing but I haven't used them yet, @FFY00 may have some comments on their usefulness.

I find maintaining untyped Python projects very anxiety inducing, there isn't a strong contract in signatures, and it's hard to be sure changing some code won't break things, even with tests. All my projects use type hints, solving this issues described, while still allowing me to take advantage of the flexibility the dynamic typing in Python provides. Because the type system is not hard tied to the language, we can always override things or tell the type checker to ignore something if we need an escape hatch.

Simply said - I think we should get rid of it. There are two use-cases for ratbag, one is Piper, one is ratbagctl. Piper can work with polkit, ratbagctl can run as root. There's no real case for DBus imo, it just adds another API for less benefit that we expected.

I don't really agree with this. While I acknowledge that there is a complexity introduced by DBus that can be arguably not worth, I think there are a couple reasons why keeping DBus would make sense:

Requiring the configuration client to be run with, or acquire, root privileges is a bad, or at the very least suboptimal, UX
It essentially locks the clients to Python. While I think this kind of limitation is mostly fine for vendor/hardware specific projects, as they will have a limited number of clients, I always thought that one of libratbag's strong points was that it provides a vendor-agnostic API, making it the obvious choice for integrators like GNOME to use in settings apps, etc. Moving the API to Python would have a huge impact here.
Having the clients responsible for IO makes creates the possibility of collisions where two different apps might be trying to access the hardware at the same time. This might now seem that bad, but there are protocols like Logitech HID++ that might take quite some time to perform a transaction (usually because it requires enumeration, and/or rewriting many different settings). Using a daemon architecture allows us to perform each of the client transactions at a time.

Now, like I hinted earlier, choosing DBus will add some technical overhead, making maintenance harder. This is not helped by Python's poor history regarding DBus APIs. I attempted to solve this in https://github.com/FFY00/dbus-objects, which is a project that should limit the overhead of writing DBus APIs to a minimum and make it mostly seamless with modern Python development.

And a better name than ratbag-python would be good too :) While I don't want to drop the ratbag name given we have a logo and what not, "ratbag-python" seems lazy and the language should not be at the core of a project.

Why do we need a new name?

I've added type hints now to the main parts (driver itself is still missing) and ... oh boy, did that find a bunch of bugs. So type hints is going to be a requirement for this from now on, I've hooked mypy into the CI to enforce this.

Requiring the configuration client to be run with, or acquire, root privileges is a bad, or at the very least suboptimal, UX

This should be possible with polkit by installing the necessary rules files, then forking to pkexec and passing the file descriptor back. This would pop up a "ratbag wants to access /dev/hidraw0" but we can't really work around this (except obviously by running a trusted system daemon that is always root :)

I always thought that one of libratbag's strong points was that it provides a vendor-agnostic API

Yes, but in 7 years of libratbag, no-one has written anything in another API, at least not to my knowledge. Exclusing Piper and we wrote that in Python anyway. Adding a DBus frontend is simple enough but I'd rather we do this when there's explicit demand for it. Right now it's a "build it and they will come", but the last years have shown it'd be better to approach this as "let them come and we build it".
That same argument goes for the daemon architecture too - that theory only works as long as everyone is using the daemon. For logitech device it's already not the case. For example, Solaar is unaware of ratbagd and vice-versa.

Why do we need a new name?

If it's a new repo, I'd rather it not be named ratbag-python. If we merge it into libratbag itself (and effectively replace the current repo), we keep the libratbag name. Also, oh boy, I just googled ratbag and that has quite a few meanings I didn't intend...

Fun stuff: I originally wrote ASUS driver in python, then rewritten it in C and ported to libratbag and now I have rewritten it again in python and ported it to ratbag-python.
I tried to keep it as close as possible to libratbag version, because it will be easier to maintain this way.
I wasn't able to completely abandon and remove my rogdrv project (but I had this idea in mind), because some people still prefer it over libratbag for some reason. It's also easier to debug mice with this driver. But I didn't liked the python-hid(api) backend, which breaks from time to time, and replaced it with ratbag-python. I also wanted to migrate to libratbag-compatible device configuration files. So now rogdrv is a native (without D-Bus) frontend to ratbag-python.

IMO even ratbagd is a wrong split - it's aimed at being able to easily write applications but we only have one full consumer (Piper)

Not anymore, ratbag-python got another consumer.
I also think it would be easy to integrate the Razer driver into the ratbag-python because it's already have some python API.


Fix RPMS missing codes being skipped on REDCap imports

We noticed subject JE94766 had chrscid_a27 and chrscid_a28 as -3 on RPMS CSVs, but had '3' on REDCap.
Here is what we believe happened:

JE94766 had 3 filled in for chrscid_a27 and chrscid_a28 originally, but was later changed to -3
rpms_to_redcap.py script skips/ignores missing codes on fields marked @NOMISSING when uploading to REDCap.
So, this updated value is not reflected on REDCap.

We had discussed 2 potential solutions to this issue:

Scheduled wiping on add 'PRESCIENT' REDCap records to clean such values
Switching to a more restrictive overwriteBehavior on REDCap API:

This should overwrite overwrite blank/empty fields as well

https://redcap.partners.org/redcap/api/help/?content=imp_records

Observation:

if -3 is pushed up against a radio variable with @NOMISSING tag, the current REDCap (v14.5.17) did not reject that. However, on the front-end, -3 did not have any effect. In other words, the radio options in front-end remained unchecked.

However, when that JSON was downloaded, -3s showed up in radio variables' field.

The above change was possible by replacing continue with value='' in these lines:
https://github.com/AMP-SCZ/utility/blob/5d2777c4fa97515541f87f8b39c60671dee3c0c7/rpms_to_redcap.py#L238-L253

New issues:

why did -3 get pushed up when blank value was populated in value? Notably, -3 is present in the original data.

TODO:

delete rpms-records.txt before 6 pm.


Technical Service XML retrieval + formatting + code cleaning

Changes in this Pull Request:

Reimplemented GET endpoints to retrieve technical services templates as XML (embedded in JSON)

Mandatory checks:

[X] latest changes were fetched from the upstream branch before creating this PR (branch is up to date)
[X] changes were tested manually
[X] unit tests were properly implemented
[ ] communication about interdependent changes has been sent to the team (if applicable)
[ ] respective Jenkins jobs were referenced in additional context section of this PR

OSCM Build References:
Tbd

This change is

oscm-rest-api-service/src/main/java/org/oscm/rest/service/TechnicalServiceBackend.java line 72 at r1 (raw file):

  public RestBackend.Get<TechnicalServiceXMLRepresentation, ServiceParameters> getXML() {
    return params -> {
      List<VOTechnicalService> ts =

Where is the exception handling? (e.g. OperationNotPermittedException)


[ask] extracting lips point from iBug-300W dataset?

Hi @yinguobing ,
in this code https://github.com/yinguobing/image_utility/blob/master/extract_face_from_ibug.py
can we extract only some point (not exact 68 point). like I want to take the lips point (facelandmark keypoint) only

I think so. But you need to know the indices of the lips marks first.
And I would recommend using this repo instead:
https://github.com/yinguobing/facial-landmark-dataset

Thank for replying 👍
as far as I know, from this picture we can the index for lips start from point 49-68
https://ibug.doc.ic.ac.uk/media/uploads/images/annotpics/figure_68_markup.jpg
any idea how I can
[ ] extract (get only lips landmark), then
[ ] convert the annotation for csv
[ ] convert csv to tfrecord
[ ] finally can train model for lips detection
IF any suggestion, really appreciate it @yinguobing


After Upgrade Re-Installation raises an error

Hi,
have upgraded to version 1.2.3 running on HA 10.5 with 2023.8.3.
I've Nilan integrated via Serial, which was working now for a while.
Now after the HACS Upgrade, restart of HA and unplug/plug of Nilan USB, I've clicked on "Add Integration" and have selected the   Interface type "Serial".
Normally next step is the selection of the usb path but now the system is throwing a popup with title "Fehler" without any further information.

Enabling log, restart HA and trying same to find this in the system log.
Any idea how to solve it?
`Logger: aiohttp.server
Source: data_entry_flow.py:398
First occurred: 19:24:21 (1 occurrences)
Last logged: 19:24:21
Error handling request
Traceback (most recent call last):
File "/usr/local/lib/python3.11/site-packages/aiohttp/web_protocol.py", line 433, in _handle_request
resp = await request_handler(request)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/aiohttp/web_app.py", line 504, in _handle
resp = await handler(request)
^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/aiohttp/web_middlewares.py", line 117, in impl
return await handler(request)
^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/http/security_filter.py", line 85, in security_filter_middleware
return await handler(request)
^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/http/forwarded.py", line 100, in forwarded_middleware
return await handler(request)
^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/http/request_context.py", line 28, in request_context_middleware
return await handler(request)
^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/http/auth.py", line 236, in auth_middleware
return await handler(request)
^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/http/view.py", line 148, in handle
result = await handler(request, **request.match_info)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/config/config_entries.py", line 181, in post
return await super().post(request, flow_id)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/http/data_validator.py", line 72, in wrapper
result = await method(view, request, data, *args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/helpers/data_entry_flow.py", line 110, in post
result = await self._flow_mgr.async_configure(flow_id, data)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 292, in async_configure
result = await self._async_handle_step(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 398, in _async_handle_step
if not isinstance(result["type"], FlowResultType):
~~~~~~^^^^^^^^
TypeError: 'NoneType' object is not subscriptable
`

Hi, that is odd. Can you test if version 1.1.x works?

Hi, that is odd. Can you test if version 1.1.x works?

Hi, thanks for your quick response. Going back to version 1.1.19 works perfectly again.

I encountered same issue. Revert to 1.1.19 last working version.

This issue should now be resolved in 1.2.4. I don't know how but an indentation got slipped into the serial part breaking it. It was a simple fix.


FIELDLINES error when running tutorial case

@lazersos I tried to run the tutorial case for FIELDLINES by running the precompiled versions on PPPL portal. I got the following error prompt:
$ xfieldlines -vmec ncsx_c09r00_free -mgrid mgrid_c09r00.nc
FIELDLINES Version  1.50
-----  HDF5 Parameters  -----
HDF5_version:   1.12 release: 00
-----  MPI Parameters  -----
MPI_version:   3.01
Open MPI v4.0.3, package: Open MPI<EMAIL_ADDRESS>Distribution, ident: 4.0.3, repo rev: v4.0.3, Mar 03, 2020
Nproc_total:         1
Nproc_shared:        1
----- Input Parameters -----
FILE: input.ncsx_c09r00_free
R   = [ 1.00000, 1.80000];  NR:    201
PHI = [ 0.00000, 2.09440];  NPHI:   60
Z   = [-0.70000, 0.70000];  NZ:    201
of Fieldlines:     27
----- MGRID Information -----
FILE:mgrid_c09r00.nc
R   = [ 0.43600, 2.43600];  NR   =  201
PHI = [ 0.00000, 2.09440];  NPHI =   37
Z   = [-1.00000, 1.00000];  NZ   =  201
----- VMEC Information -----
FILE: ncsx_c09r00_free
R       = [  1.02288,  1.77845]
Z       = [-0.64057, 0.64057]
BETA    =   0.041;  I  =  -0.179 [MA]
AMINOR  =   0.323 [m]
PHIEDGE =   0.497 [Wb]
VOLUME  =   2.962 [m^3]
----- Virtual Casing Information -----
INTEGRAL TYPE: Surface Current (DCUHRE)
MIN_GRID_DISTANCE =  6.0473E-02
NORMAL_AREA =  2.4587E+01
NR =    1;   NU =  128;  NV =  128;  NFP =   3
NUVP =  49152
ABS_TOL =  0.0000E+00;   REL_TOL =  1.0000E-04
MIN_CLS =      0   (16777216)
Plasma Field Calculation [000]% ERROR in GetBcyl Detected
R,PHI,Z   1.0000000000000000        0.0000000000000000      -0.69999999999999996
br,bphi,bz,myworkid   1.1445121261107514        2.1219957909702130E-312 -0.18279070659749758                0
Note: The following floating-point exceptions are signalling: IEEE_INVALID_FLAG IEEE_UNDERFLOW_FLAG IEEE_DENORMAL
STOP ERROR in GetBcyl
I also tried some other free-boundary VMEC outputs and got the same error. How do I fix this? Thanks!

This is an error originating from the GetBCyl routine in the vmec_utils module. Try running the command using the proper MPI wrapper and see if you get the same error. Sometimes reducing the mesh slightly helps, try changing NR and NZ to 200.

This is an error originating from the GetBCyl routine in the vmec_utils module. Try running the command using the proper MPI wrapper and see if you get the same error. Sometimes reducing the mesh slightly helps, try changing NR and NZ to 200.

@lazersos Thanks. I tried running with mpirun/mpiexec but did not help. I reduced NR and NZ to 200, then 160, then 100, which did not help either. Any other tips I could try?

@lazersos Thanks. I tried running with mpirun/mpiexec but did not help. I reduced NR and NZ to 200, then 160, then 100, which did not help either. Any other tips I could try?

@yaozhou1989 You can try to use the Intel compiler version by
module load mod_stellopt
module load stellopt/develop_intel

There is even a "debug" version, but I am not quite sure if I successfully compiled it because sometimes I was debugging STELLOPT using that version.

@yaozhou1989 You can try to use the Intel compiler version by
module load mod_stellopt
module load stellopt/develop_intel

There is even a "debug" version, but I am not quite sure if I successfully compiled it because sometimes I was debugging STELLOPT using that version.

@lazersos I have seen the free-boundary VMEC crashes because of the MPI sharing feature several times (both on the PPPL portal cluster and my MacBook). But I didn't have a good record on my hand right now. It happens with some cases on some machines. Once I changed either the input or the machine, it disappeared. I will try to write it down next time.

@lazersos I have seen the free-boundary VMEC crashes because of the MPI sharing feature several times (both on the PPPL portal cluster and my MacBook). But I didn't have a good record on my hand right now. It happens with some cases on some machines. Once I changed either the input or the machine, it disappeared. I will try to write it down next time.

@yaozhou1989 You can try to use the Intel compiler version by
module load mod_stellopt
module load stellopt/develop_intel

There is even a "debug" version, but I am not quite sure if I successfully compiled it because sometimes I was debugging STELLOPT using that version.

Just tried, still getting "ERROR in GetBcyl".

@yaozhou1989 You can try to use the Intel compiler version by
module load mod_stellopt
module load stellopt/develop_intel

There is even a "debug" version, but I am not quite sure if I successfully compiled it because sometimes I was debugging STELLOPT using that version.

Just tried, still getting "ERROR in GetBcyl".

@lazersos I now have FIELDLINES working for some cases (still not for the tutorial case though). I am trying to read in the field data, but it seems that the output contains only the field data but not the grid data. I also could not find the domain boundary info so could not infer the grid either. Therefore I do not know at which points the magnetic fields are evaluated. Am I missing something? Thanks.

@lazersos I now have FIELDLINES working for some cases (still not for the tutorial case though). I am trying to read in the field data, but it seems that the output contains only the field data but not the grid data. I also could not find the domain boundary info so could not infer the grid either. Therefore I do not know at which points the magnetic fields are evaluated. Am I missing something? Thanks.

@lazersos I now have FIELDLINES working for some cases (still not for the tutorial case though). I am trying to read in the field data, but it seems that the output contains only the field data but not the grid data. I also could not find the domain boundary info so could not infer the grid either. Therefore I do not know at which points the magnetic fields are evaluated. Am I missing something? Thanks.

Never mind, I see that 'raxis', 'zaxis', and 'phiaxis' actually refer to grid point locations. Thanks.

@lazersos I now have FIELDLINES working for some cases (still not for the tutorial case though). I am trying to read in the field data, but it seems that the output contains only the field data but not the grid data. I also could not find the domain boundary info so could not infer the grid either. Therefore I do not know at which points the magnetic fields are evaluated. Am I missing something? Thanks.

Never mind, I see that 'raxis', 'zaxis', and 'phiaxis' actually refer to grid point locations. Thanks.

@yaozhou1989 Can we close this issue now?


cannot share full image in chat-window.

I tried to upload an image. Instead of uploading entire image, it ask for selecting a square region of image. Why full image cannot be uploaded?
But when I tried to taking picture (instead of uploading image), I can share full image captured by camera (any size).
Is there a option to allow full image size instead of partial image?

Hi @goldiaxy ,
I believe cropping should only be enabled for profile pictures. This may indeed be a bug. @bensmiley are you cropping all images in the iOS version of the chat SDK?

Hi @kykrueger
This issue is observed in Android version of the chat SDK. I tried HEAD branch.

@kykrueger,
Currently in the latest iOS version we deal with images and profile images slightly differently. Both can be cropped and use the TOCropViewController for this functionality.
Profile images: This image is cropped into a circle to show the user what their image will look like. This then crops the image to the square size which fits into the shown circle.

Images: These have a crop functionality enabled on them so the users can make the images specific to the exact part of the image they want.
I don't know if Android has any similar libraries like this but you can use this as a guide for how the functionality would like to be.
In the scenario mentioned above the user should be able to use the entire image as well as crop the image to be smaller if they wish.
Simon

Hey @simonsmiley64 .
Issue is not with cropping. My point is that why cropping is needed when user sharing pictures in chat window.
chat-Widnow --> Choose Picture --> Browse the Picture and select one.
Here you will see it will ask for Cropping ( a square size cropping).
IMO, cropping is not needed here. It should share entire pictures in chat. No cropping is needed.
This issue I observed in Android  device.
Whereas if I select "Take Picture" and take picture using camera. It does not do cropping. It just share entire picutures.


Removing not required deps from install; Bumping SKL version

Working on Issue #68

This is a little tricky because we haven't really tested much without having all three (skl, xgb,lgbm) installed.  It "APPEARS" that the tests are passing, but that is because we are only testing the "all installed" case.
I've been testing this manually in a clean container, but I'm going to pause for a second to setup some tests like:
    - name: Install basic dependencies
      run: |
        python -m pip install --upgrade pip
        pip install  --no-cache-dir .
    - name: Run basic tests without extra
        # SOMETHING HERE
    - name: Install extra dependencies
      run: |
        pip install  --no-cache-dir .[extra,tests]

Yes now that we have separate dependencies it would be really good if we can run some test without all dependencies.

BTW, it seems XGB has a dependency on SKL.  I started a clean container, only installed XGB (not SKL, not LGBM), and I get:
>>> model = xgb.XGBRegressor()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.6/dist-packages/xgboost/sklearn.py", line 146, in __init__
    raise XGBoostError('sklearn needs to be installed in order to use this module')
xgboost.core.XGBoostError: sklearn needs to be installed in order to use this module

And LBGM installs scikit-learn as a dep.
root@2c5fb8629da4:~/hummingbird# pip3 freeze | grep sci
scipy==1.4.1
root@2c5fb8629da4:~/hummingbird# pip3 install lightgbm>=2.2
root@2c5fb8629da4:~/hummingbird# pip3 freeze | grep sci
scikit-learn==0.23.1
scipy==1.4.1`

But, this new installer will at least allow JUST SKL without the requirement on XGB and LGBM.
@interesaaat - Do you think we should make SKL a required dependency since all 3 of our targets require it? (I believe also the onnx work will require it at least for now?)

Yes if lgbm requires skl, than also the lgbm converter for onnx will require it, so let's have it! We can remove it later on in case we have some library that does not need skl.

On python 3.8, we're hitting https://github.com/scikit-learn/scikit-learn/issues/15328 because we're using SKL 0.21.3
With the new installer, for some stupid reason even though I have Cython listed before Sklearn, it's installing sklearn first and failing on
      File "sklearn/utils/setup.py", line 8, in configuration
        from Cython import Tempita
    ModuleNotFoundError: No module named 'Cython'

So i think we should either upgrade to a newer version of SKL (see https://github.com/microsoft/hummingbird/issues/3), or not support python 3.8.  WDYT @interesaaat ?   I believe the newer SKL versions was only a problem with our Imputer code (yet to be added)

Ready for review @interesaaat


add source column

thanks @elmodeer can you resolve the conflicts please? There was another PR merged (#9) that removed a duplicate line

I think we should refer to it as "Venture Capital Statement of Support for Israel" instead of og pro-israel list. Apologies for the confusion, I'll remember to be more precise in my issues in the future


refactor: use thread pool for sync parallelism

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#394
#393  👈
#392
#391
#382
#381
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rphmeier and the rest of your teammates on  Graphite


Vital Beacon spell effect healing

When using the inline healing roll from spell effect: vital beacon, it always rolls 0 when done by hovering over the effect.

I was able to replicate -
As a workaround, you can send the effect to chat and it will let you roll without any issues.


feat: add codesandboxes to search results

Leaving as draft due to intermittent rate-limiting from CSB.

Still need to find a better solution with Codesandbox, but ready to ship for now.


Integration tests

this is an initial skeleton of integration tests using Spectron, Mocha, and Chai. For the included test to work, you need to either replace the path of the app with your app's path, or better yet, copy your app to live in your /Applications/ folder.
Also - I looked into stubbing out the USB stuff, but I didn't make any real progress on actually testing it, so I left my experiments with that out of this PR.

LGTM


SST state and assets S3 buckets fail AWS Security hub

Currently the buckets used for SST 'state' and 'assets' (Lambda zip files) fail the AWS Security Hub rule: 'S3 general purpose buckets should require requests to use SSL' (AWS Foundational Security Best Practices v1.0.0).
Is there a known solution for this? I don't believe these buckets can be accessed within the SST config file for modification.
All other S3 buckets managed by SST (for Nextjs in particular) have bucket policies in place, requiring SSL. So I'm wondering if theres a reason as to why these two buckets do not have a similar rule?
Thanks! 😄

Thanks @woodsjd-cr! Fixed in v3.0.53.


complex properties removed each time some other property is updated

We have been using ADT Explorer for 6 months using RealEstateCore Ontology (https://github.com/Azure/opendigitaltwins-building/tree/master/Ontology).
We are using externalIds property that is a complex property of type 'Map'
    {       "@type": "Property",       "displayName": {         "en": "External IDs"       },       "name": "externalIds",       "dtmi:dtdl:property:schema;2": {         "@type": "Map",         "dtmi:dtdl:property:mapKey;2": {           "name": "externalIdName",           "schema": "string"         },         "dtmi:dtdl:property:mapValue;2": {           "name": "externalIdValue",           "schema": "string"         }       },       "writable": true     }
Since the 18th of October, using the ADT Explorer of the Azure Portal (https://explorer.digitaltwins.azure.net/) we have experienced this warning:

If we use this ADT Explorer version, it deletes the property. I understand it is comparing the twin with the model,  but until this other feature https://github.com/Azure-Samples/digital-twins-explorer/issues/164 about editing complex properties, is completed, it is a problem to work with complex properties.

In the new property inspector, un-modelled 'floating' properties are removed on the next patch event.  This is because the ADT API doesn't allow patching properties when the twin doesn't match the shape of the model.
It looks to me like the model for the twin you shared is missing the externalIds map property in it's interface.  If that's the case, this doesn't look like a bug to me, but rather the intended experience.  If externalIds is on the model interface for that twin, can you share the models being used?
The updated property inspector adds support for the asks mentioned in #164 - with support for maps, objects, and enum editing.


2.3: Import csv file with --current-user option fails

The ConsoleToken is created without roles in ConsoleContextListener
The issue in is fixed in OroPlatform 2.4, is it planned to fix this in 2.3 ?

Hi,
maintenance versions receive bugfixes only in Enterprise versions.

What about OroCommerce 1.3 which is an LTS version is based on OroPlatform 2.3 ?

LTS receives only security related fixes, not regular bug fixes, so this fix is included in 2.3 only for enterprise level customers.

Hey @zeykzso,
could you please explain your roadmap on orocrm.com.

The key difference between regular and LTS version is the duration of maintenance (bugfix) period. LTS versions are also supported for security fixes.

Maintenance for 2.3 is announced until end of 2018 and support (security fixes) until mid of 2020. EE Version is not mentioned in that post.
Thx.

@RocKordier looks like we need to update our website to state it more clearly. Maintenance and support are provided for Enterprise versions only.

This bug also exists in platform-er 2.3.13

@dimonixx as platform-er involves EE code, please use EE Customers Support -
http://support.orocommerce.com or http://support.orocrm.com

I am now closing this issue as it has been inactive for a while. Please, feel free to reopen it if you have any questions.


chore: Make label prop optional in Dropdown component

[x] I have followed the Your First Code Contribution section of the Contributing guide

Summarize the changes made and the motivation behind them.
Reference related issues using # followed by the issue number.
If there are breaking API changes - like adding or removing props, or changing the structure of the theme - describe them, and provide steps to update existing code.

https://github.com/themesberg/flowbite-react/issues/1466

Summary by CodeRabbit

New Features

The label property in the Dropdown component is now optional, enhancing flexibility for developers when using the component.

Bug Fixes

Adjustments to the Dropdown interface improve compatibility for various use cases, ensuring components function correctly without a specified label.

I'm not sure why vercel is failing, but it is happening in other PRs as well. The code looks good to me, thanks @raky291

silly cache.


Avoid usage of center_on_hmd()

Instead, move the ARVROrigin appropriately. The rationale behind this
is to keep the ARVROrigin's local coordinate system intact, so that it
refers to the actual physical space.
Closes GodotVR/godot-vr-common#4

Looks good to me, I don't have time to try it out right now but I'll trust you tested this properly :)


nz-upload     i want to add preview and delete button for nzListType="picture"

What problem does this feature solve?
nzListType="picture" and also to add delete button for nz-upload
What does the proposed API look like?
<nz-upload class="upload-plugins-width"
    nzAction=""
    nzListType="picture"
    [(nzFileList)]="fileList1" [nzRemove]="remove">
    <button nz-button>
      删除
    </button>
  </nz-upload>

Keepping react antd styles.


Refactor/hocs 2275 separate integration tests

Some of the tests in the project have the dependency  of a running postgres database. This change marks those as integration tests, by moving them to an integration test directory and applying standard gradle config.
This allows unit tests to be run much quicker, and without the external dependency.
This change is a precursor to introducing more unit tests to the workflow project.

Your commit message is a bit strange.
The first line should be a summary that stands alone without further context.
If you're going to use (the optional) extra lines to add more context, the second line should always be blank, then you continue typing on line three and beyond.
Here you've put the Jira ticket on line 1, and then the actual message on line 2.
You can change it with
git commit --amend -m "HOCS-2275 Separate integration tests" &&
git push origin +refactor/HOCS-2275-Separate_integration_tests # the plus means "force push but only to this branch"

These might sound like particularly persnickety comments, and to an extent they are -- but adhering to these Git commit guidelines generally makes Git easier to work with. Squash these commits prior to merge for bonus points.
(I'm too bad at java to actually review this PR properly.)


Horusec show error on create log file in windows

What happened:
when i runed horusec in windows using command in start analysis show this error when create log file

What you expected to happen:
not show error when start analysis
How to reproduce it (as minimally and precisely as possible):
horusec start -p=.

Anything else we need to know?:
Environment:

Horusec version (use horusec version): 2.5.0
Operating System: Windows

I think that this already fixed this issue. Could you try again using main branch?

that's great, I think the only thing left is to actually release a oficial version with this improvement, correct ?


Add TRIE Data Structure

Algorithm:
TRIE Data Structure
DS:
Trees
Languages Supported:

C
C++
C#
Java
Javascript
Python

I have coded lots of tries in c++. So, I can write code for trie data structure in c++. I maybe able to convert it to java and python as well since I have a little experience with these languages.

I have assigned you the Issues in C++, Java and Python @16priyesh
You can start working on it. Please follow the general guidelines:

Make a Branch off your Master and work on that.
Make a proper README of how your Script works and what all libraries and modules you have used.
Add Proper Comments and focus on Clean Code, in practice.
Also reference this Issue in your PR so that it becomes easy for us to review.

remote: Permission to TesseractCoding/NeoAlgo.git denied to 16priyesh.
fatal: unable to access 'https://github.com/TesseractCoding/NeoAlgo.git/': The requested URL returned error: 403
I'm haven't been able to sort it out since yesterday. I followed the What is Pull Request and how to do it guide on the README of this repo.

Hello, @16priyesh
Can you please let us know what was the Issue. Probably you tried to push directly to the Master which caused the error. We are here to help you sort this out and make your Contribution smooth.

Hello @HarshCasper wanna to work on this issue in C++. Is anyone is working on that.

Closing by #1278


The websocket messages are not received from the client when using Custom Domain

Steps to reproduce

Install the template in this repo
Create a Custom Domain in the API Gateway
Attach created websocket api into to API mappings in the domain
Setup the CNAME entry in your domain DNS
RUN following commandwscat -c wss://<your.custom.domain>
> {"action": "sendmessage", "data":"hello"}

This message does not return 'hello' back
The Cloudwatch logs for the SendMessage logs show that the message is received by the server and there is no error in sending the reply back

the issue was in sendMessage lambda , the lambda function was unable to construct the right connections endpoint when used with custom domain


Listen socket keeps opened indefinitely when stopped via StopChan

When using StopChan to stop the discovery the listen() function never returns and therefore keeps the socket opened indefinitely.
I was able to quickly hack it in a way so that the local exit variable was part of the peerDiscovery type and listen() would check if that variable is set and quits.
Not sure if that's nice though. Also, in case no more peer discoveries are happening that function would still sit there forever with its socket listening for valid data.

Closed via #29


[Das Schwarze Auge (TDE) 5 OFFICIAL by Ulisses] DSA5 v1.0.3 FW Modifier Fix

Changes / Comments

Fixing a major bug with wrongly calculated skill value modifiers

Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[x] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


Fix #120

The JIT is emitting lots of temporary spills when we do a branch
with a value on the stack.  This leads to a huge frame size.  Most
of these branches are happening on our branches for error checks.
This stores the value in a temp and reloads it instead of branching
with the value on the stack.  This greatly reduces the number of
these, but once we get something like:
a() + b()
Where we do an error check on the result of b() we still have the
result of a() on the stack, which causes us to still spill.  This
at least gets us passing on this test, but we'll need to see if
we can be more efficient in the future, or if we just need to store
all of the values around an error check.

Assuming you ran the test suite for this, LGTM.

Our tests pass, but apparently I failed to even build it after a last minute tweak!


ERROR Cannot use import statement outside a module

this happen with my nuxt project at version ≥ 2,  how i fix this?

What happened? Can you send your error?

Would you mention the exact nuxt and vue-bottom-navigation versions?


fix SNAT on pod startup

Fix #732 by updating ipset local-pod-ip-nat when adding/deleting nic.

please squash to one commit


'New Update is Available' and then 'You will be notified when it is ready'

My Setup
Windows 10

Description
Current Behavior
Usually, I get the message as per the screen image...I get asked to upgrade to the latest version, and then get a message letting me know it will be updated when it is ready?
Expected Behavior
should just update

Same as Ticket 1300 I Think:
https://github.com/RocketChat/Rocket.Chat.Electron/issues/1300

Look at the top of the notification box. “Downloading Update”.
This message tells you that the update will now be downloaded in the background and you will be notified when “it is ready to be installed”.
Shortly after that the installer opens.
Also this notification box isn’t new and has been part of the update process since forever.

Please test if the latest release did solve this. If not please reopen the issue or create a new one and tell us. I will be closing this one for now.
Thank you!


Refactor moderator selection

This is an extensive refactor of the store tab in settings. It's updated to use the same code between the purchase and settings/store views.
The store view will now support showing invalid moderators if a saved moderator has stopped moderating or can't be loaded, and will give the user the option to remove that moderator.

Do we really want to have that text here when it's always going to be 1 of 1 in this case?

FWIW, I'm also in favor of simplifying that text across the board and only showing a generic loading... and/or spinner. I remember the counts got us into trouble before with edge cases.

When purchasing this listing, I get an exception when loading the mods:
ob://QmfCtcdJkoH6YMQCKpxS6Wi1zn3Gj3LQvb2kkxjKJGWk8C/store/letter-height-measure-tool

Do we really want to have that text here when it's always going to be 1 of 1 in this case?

We want to have some kind of loading indicator. It could be something else.
In the verified moderators PR I make it more flexible, so it can either be loading x of y or just X loaded.
The X loaded is important in the verified mod PR, because it needs to show how many non-verified have been loaded when it's only showing the verified mods (ie: it's loading mods for a long time but no verified mods are showing up, the user should know 30 non-verified mods have loaded or it looks like nothing is happening).

@rmisio good catch on that bug, I forgot to move a function that changes the DOM when I moved where the render to the add listener. This is ready for another look.


Bug when creating many tabs

Bug when filling tab bar with many tabs. It seems like after a point, tabs are just created out of view.

+1

What alternative behavior would you want? I can think of quite a few, but none stand out to me as clear winners:

Allow the tab bar to scroll horizontally
Enforce a maximum number of tabs and minimum tab bar width necessary to display this max
Show multiple rows of tabs
Show a dropdown to access extra/hidden tabs, similar to how the Chrome extension button works

Tab stacking of some sort

It should be noted that the latest Chrome (42.0.2311.90) on Mac OS X hides overflowing tabs roughly the same way we do.

This window has 16 tabs and yet only 10 are shown. Even selected 11th tab is not even shown. I vaguely remember a time when Chrome used to show arrows to let you scroll the tab bar, but I guess that’s gone.

Closing for now. Please provide a specific suggestion for improvement and we’ll reopen. Thx

I just tested and when you reach a limit it starts creating them out of view (which is the correct behavior) but the last tab gets cut in half.

I think the dropdown is a great idea, although it isn't chrome's "natural" behavior.


Remove push to GitHub Packages

I suggest we remove GitHub packages for Spectre.Console since it literally had 0 downloads since we added it.

Yes, one needs to be signed in and have access to the feed to use it.
I have the feeling GPR is rather unusable for most cases.

Back when the GitHub Support Community was still a thing, a very long thread named Download from Github Package Registry without authentication was full of people complaining about the authentication requirement for public packages. This URL now redirects to the GitHub Community Discussions homepage. There, you'll find another round of issues with people looking for a solution without authentication but ultimately that's still not possible.

How to make Github packages to the public
How to allow unauthorised read access to GitHub packages maven repository?
NPM public packages can't be accessed without a token
Why Do We Need to Authenticate to Download/Read Ruby Gems?

So yes, more than 3 years after its introduction, the GitHub Package Registry is still pretty much useless for open source projects. ¯\_(ツ)_/¯
(I also tried to publish a NuGet package of mine on GitHub Package Registry last year and deleted everything when I realised that requiring a complicated authentication setup would mean nobody would use it, not even myself.)

This was resolved in https://github.com/spectreconsole/spectre.console/commit/383bee0e3e0ffb9bcadc2071018a40811600a02e


Add solutions to deal with failure in service call back

closes  https://github.com/ros2/ros2_documentation/issues/2341
Regarding the discussion in #2341, there are three solutions to handle failure in service callback as mentioned in this PR

I'm going to close this PR.


add-profile: Aryan Gaikwad

Description
I contributed to a devFind repository . This is my first open-source contribution as a Software Engineer.
Changes Proposed
I have added my profile to a devFind repo. Please review and merge the request.
Checklist

[x] I have read and followed the Contribution Guidelines.
[x] All new and existing tests passed.
[x] I have updated the documentation to reflect the changes I've made.
[x] My code follows the code style of this project.
[x] The title of my pull request is a short description of the requested changes.

Please merge my pull request !!!

@shyamtawli Can you please merge my pr ?


Alternative screenshot naming not working correctly

Hey so due to variable name missing a letter the alternative screenshot naming does not work to maintain compatibility with Jasmine 2 screenshot naming convention.
Please review PR: https://github.com/abhishekkyd/protractor-html-reporter-2/pull/3
Thank You @abhishekkyd

fixed in: https://github.com/abhishekkyd/protractor-html-reporter-2/pull/3


cigien: Watch snapseedforpcguide.co

cigien requests the watch of the watch_keyword snapseedforpcguide\.co. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

snapseedforpcguide\.co has been seen in 1 true positive, 0 false positives, and 0 NAAs.

snapseedforpcguide\.co has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Spevacus in Charcoal HQ

Approved by Spevacus in Charcoal HQ


Mark "options" param as optional in jsdoc

Without it some IDE/editors show message "Invalid number of arguments, expected 2" when using
ms('1 day')

Thanks!


drivers: timer: nrf_rtc_timer: user channels count dependency inv

This PR is related to the https://github.com/zephyrproject-rtos/zephyr/issues/67117
and is inspired by https://github.com/zephyrproject-rtos/zephyr/pull/65908 .
For calculation of number of user channels of nrf_rtc_timer a new mechanism allowing dependency inversion is introduced.
The CONFIG_NRF_RTC_TIMER_USER_CHAN_COUNT is no longer used directly in the C code. Instead it is replaced by the
K_NRF_RTC_TIMER_USER_CHAN_COUNT macro which is calculated automatically during the build.
Users can add own requirements on the K_NRF_RTC_TIMER_USER_CHAN_COUNT macro by:

setting the value of the Kconfig option NRF_RTC_TIMER_USER_CHAN_COUNT (legacy appoach).
providing Kconfig options named according to pattern NRF_RTC_TIMER_USER_CHAN_COUNT_ADD_<custom_name>, each of which increases the value of K_NRF_RTC_TIMER_USER_CHAN_COUNT by given value.
providing Kconfig options named according to pattern NRF_RTC_TIMER_USER_CHAN_COUNT_MIN_<custom_name>, each of which imposes limit on the minimum value of the K_NRF_RTC_TIMER_USER_CHAN_COUNT macro.

Mentioned new possible Kconfig options can be added in multiple modules, also outside of Zephyr tree or in downstream repositories. They express distributed requirements on value of K_NRF_RTC_TIMER_USER_CHAN_COUNT.
The definition of the Kconfig option NRF_RTC_TIMER_USER_CHAN_COUNT does not need to know now every possible user (including known downstream repositories) to set up reasonable default value. It is the nrf_rtc_timer module users' responsibility to express requirements in their own modules. Because of that default values 2 and 3 depending on NRF_802154_RADIO_DRIVER have been removed. In the pure Zephyr at the current state the nRF 802.15.4 does not require any user channel of nrf_rtc_timer. If user channels are required by projects based on Zephyr (including more sophisticated implementation of platform support code for nRF 802.15.4 Radio Driver), individual requirements can be easily expressed in separate repositories.

I think the implementation follows your concept nicely. I do have two comments about the concept itself though.
You decided to provide two separate mechanisms to for expressing requirements on the value of K_NRF_RTC_TIMER_USER_CHAN_COUNT: one for requesting/reserving a specified number of a resource instances, and the other for specifying a total minimum for the number of all requested instances. The intention behind the latter is unclear to me. It seems that such a pattern could be used to allow a single resource instance to be reused instead of reserving a separate instance. However, there's no policy on how such reuse is expected to work. As a result, any module that would like to reuse a single resource instance and express it using the syntax you provided would need to know and understand other clients in the system that it might want to reuse the instance with to determine if that's at all possible.
Let's consider an example from the current implementation of the PR where you used the pattern in question to specify the minimum required number of additional channels for sync_rtc module as 1. This module only needs the channel at initialization, which allows the channel (i.e. the resource) to be reused later, when the main program function starts. However, what would happen if there was a different module, let's temporarily name it B, that uses the resource in the same way? B's developer would follow the same reasoning as you did in the PR, and they would only specify the minimum required number of additional channels as 1, so that the resource can be reused by other modules later. As a result, there would be an implicit coupling between sync_rtc and B, which could cause any one of them to be unable to acquire the resource despite correctly specifying their requirements. That could happen always, never, or non-deterministically, depending on the details of their implementations.
The purpose of this PR is to remove coupling between clients of the same resource and it seems to me that this pattern breaks this promise.
The second comment I have regarding the proposed concept is that I don't see any mechanism for controlling the upper bound of the resource instances that can be requested. In case of finite resources such as peripheral channels/instances etc. such an upper bound should be possible to specify at build-time. In my opinion, if it's possible to detect a wrong configuration at build time and throw a build error, that's preferable over delegating the responsibility for detecting and acting on such a wrong configuration to runtime.
I understand that the current implementation also doesn't provide such a mechanism. However, currently the application developer is responsible for setting it to a correct value. With the proposed approach, it happens behind the scenes using values distributed throughout the entire system, which makes it more difficult to investigate potential problems.
Let me know what you think on these two potential issues.

@jciupis to your https://github.com/zephyrproject-rtos/zephyr/pull/67854#issuecomment-1903664189:
About 1st issue:
generally agree that there is the risk you mentioned, but it existed before already, and I don't attempt to fix it.
Please have a look here (code before my change):
https://github.com/zephyrproject-rtos/zephyr/blob/db429004fdea646289bdb778e40a171cacfd16cf/soc/arm/nordic_nrf/nrf53/Kconfig.sync_rtc#L23-L26
If CONFIG_NRF_802154_RADIO_DRIVER=y the nrf_rtc_channel was still "borrowed" from the channels pool required by the NRF_802154_RADIO_DRIVER. The difference is that sync_rtc uses the rtc channel during the system initialization and the nrf_802154 uses the rtc channels later.
I'm open for suggestions about fixing this issue.
About 2nd issue: there is mechanism for nrf_rtc_timer to check upper bound, it is BUILD_ASSERT here:
https://github.com/ankuns/zephyr/blob/9709255774d08a4e862b8792d788279e6246ef56/drivers/timer/nrf_rtc_timer.c#L31

@nordic-krch @anangl @tejlmand What do you think about proposed approach? (PR is a draft because I don't want it to be merged yet, I wanted to discuss it with you first).
The commit is cherry-picked to downstream sdk-zephyr to see how much does it simplify, you can follow it.
I know I could make the code I put into drivers/timer/CMakeLists.txt into a cmake function which would be called like:
zephyr_compile_definition_kconfig_based(
    K_NRF_RTC_TIMER_USER_CHAN_COUNT  /* macro that is being provided to zephyr, calation based on Kconfigs */
        KCONFIG_INIT CONFIG_NRF_RTC_TIMER_USER_CHAN_COUNT      /* Kconfig being initial value of the macro */
        KCONFIG_ADD CONFIG_NRF_RTC_TIMER_USER_CHAN_COUNT_ADD_  /* optional Kconfig prefix for expressing increase of the macro value */
        KCONFIG_MIN CONFIG_NRF_RTC_TIMER_USER_CHAN_COUNT_MIN_  /* optional Kconfig prefixfor expressing minimal value of the macro */
)

So that the pattern could be easily re-used.
I will do that if you generally agree for such change.The mentioned cmake function could be later be used to express in a similar way requirements on stack sizes or "sizes"/"capacities" of other resources that could be fine-trimmed build-time, but it's hard do do it properly taking into account dependencies.


Clippy

depends on #18

Merged #18, rebase to check Travis? It actually failed logically (expected_string != actual_string) but I can't imagine why.


[BUG] I can't export issues to github repository

What's the pb ?
I tried to export "Website" issues to a private repo on github, and it seems there is something wrong with TRPC

(Prod only)

OK, so the cookie used in production didn't used the same name as the one in localhost (the production one was prefixed with some __Secure text). So, for better user accessToken management, I deleted the code that was using the cookie thing and used the getSession({ ctx }) method that I supercharged with data using the async session callback. Of course, it needed some TypeScript adaptation that I did.


Is this project dead?

This project has not been updated for a long time despite the fact that it is not up-to-date with recent JVM versions. Is the project dead?

The project itself is not dead, it has merely been superseded by the Gluon Substrate project, which works on Java 11 and later and has a better overall approach for supporting multiple platforms.
The javafxmobile project can only be used with projects that have not yet migrated away from Java 8.


chore: remove pytest-watch - DIA-46202

Description

pytest-watch is not maintained anymore and is not compatible with toml config;
We were using a branch of the git repo which caused publishing the package on pypi to fail:

HTTP Error 400: Invalid value for requires_dist. Error: Can't have direct dependency: 'pytest-watch @ git+https://github.com/aldencolerain/pytest-watch.git@fix-toml-config ; extra == "tests"'

Related issues

#66
DIA-46202

Related PRs

Is there any alternative to ptw?

pytest-watch uses watchdog which can be used directly.
I made myself a script:
#!/bin/sh -x

/bin/sh -c "$*"

watchmedo shell-command \
  --wait \
  --patterns '*.py;*.txt,*.toml,*.md,*.yml' \
  --recursive \
  --command "$*" \
  $PWD

Only drawback: it does not interrupt when a filesystem event occurs -> https://github.com/gorakhargosh/watchdog/pull/319


check boot of vm with ssh connection

Proposed Changes
Fixes #217

i did over 300 runs and every run was successful. i couldn't simulate problem with getting private IP. It looks as env problem.

PR can be merge after PR https://github.com/ironcore-dev/libvirt-provider/pull/245. Some small changes for improve stability from this PR will be implemented there.


Section headings not aligned properly

The section headings namely Books, Organizations, Talks and Conferences are not aligned properly in the center of the screen. As a newbie to open source this issue can be resolved by me. I would request the project admins to please assign me the issue

hello sir @TeeWrath . i have visited your website and i found that all the sections like books , organizations etc are not aligned in a proper way and due to this it is not looking good  . as i am a contributor of gssoc'24 i want to work on these issues . so its a request to you please assign this issue to me ...


Refactor and fix inconsistent block times when staking

This resulted in contracts seeing a certain timestamp at staking time
but later in validation saw a different timestmap, making the block invalid
Various refactorings were done while fixing the problem that cleans up the code some

@qtum-neil The modification looks OK. Details: The example time 0x599FD028 will be reduced to 0x599FD020 applying the mask (last 4 bits to 0), that is like the difficultly but the zeros are now at the least important bytes, in order to increase the difficulty into an intrinsic random system. Using the counter i+=STAKE_TIMESTAMP_MASK+1 i.e. (i+=16) will produce valid time stamp like 0x599FD030.


chore: add data related docs

It's draft status without a final decision, so I'll merge and we can still make changes as needed.


What to test before a big release so first time use is seamless

As a new user (or a recurring one about to enjoy a new version) I would like to have a seamless experience in the first minutes using Monokle. From version to version we have detected that some of the dialogues and procedures "break", so UX is not optimal. We must set up some testing around first use experience (manual or automatic) that checks out the following aspects - prior to any big release:
Bottom-right restart dialogue (in the old version)

[ ] Make sure the Restart CTA works
[ ] Inform the user visually that something is happening (spinner...)
[ ] New version needs to actually launch itself without further user interaction

Telemetry

[x] seamless, whereas it is accepting or rejecting

Tooltip guide

[ ] We add new ones for new feagures and we deprecate previous version ones
[ ] New users see the whole bunch (permanent + new)
[ ] Recurring users see only the new ones

Order of dialogues

[ ] Dialogues and popups happen in the correct order

OS testing

[ ] Test all of this for iOS
[ ] Test all of this for Windows

Hey. Can we do this as a recurrent element for all versions?
Also, it did not work as expected. What can we do to improve it for next release


fix(doc): transform use linkedom now

fix #236
Haven't tested code I was wrote in README.

Coverage remained the same at 97.549% when pulling 307ec92fda637219bc17a91ca3f3d5e7beae4aef on SettingDust:patch-1 into c59cdd28f48a18da4382e9d35442d3c0e8b08286 on ndaidong:main.


Update PeerID Regex

Once the go server's IPFS refactor is complete for 2.3.1, we will need to update the client code to recognize an additional format for peerIDs. They will start with "12D3Koo" instead of "qm".
Example 12D3KooWDUcbMF23kLEVAV3ES7ysWiD2GBh87DHDx3buRNDLFpo8
Existing peerIDs will still be valid, so we'll need to check for both.
*this needs some more validation by the server team.

Closed in #1698


User completed preferences

Fixed some issues pointed out by Phuc. Tested the functionality, might need some more rigorous testing with filled in values (not null) but that can wait until we have the actual storing of answers completed. I tested the syntax and it works fine

Looks good to me.


Filter Button in Explore tab.

I want to add Filter button in Explore tab to filter the techs. Can you assign it to me under gssoc 22?

Sir can you change the label to GSSoC22 because this is the correct label so that score will be given to me.


Add FAQ

Question 1: Why not "just" use HDF5?
I could probably think of a few others too, such as "Why don't you have parallel I/O?" and things like that.
Should probably link to it from pyasdf as well.  I don't think pyasdf needs a FAQ yet because (and you can correct me on this) I don't think pyasdf has enough users for there to be actual FAQs yet :)

@embray, i've pulled together the wiki for this repo to support some of this; and i've put a quick FAQ page as well.  I figure that can serve to collect/distribute info until there's enough sections to maybe include in the actual documentation.
Thoughts?

An FAQ page is on the wiki.