chore: Remove en-us from msdn.microsoft.com links

Not sure if the locale needs to be in the metadata, so this can be closed if it does. Otherwise, this matches the link localization guidelines

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

technical-reference/10-n-functoid.md
:white_check_mark:Succeeded
View

technical-reference/absolute-value-functoid.md
:white_check_mark:Succeeded
View

technical-reference/actions-editor.md
:white_check_mark:Succeeded
View

technical-reference/activity-status-portal-help.md
:white_check_mark:Succeeded
View

technical-reference/adapter-handler-properties-dialog-box.md
:white_check_mark:Succeeded
View

technical-reference/adapter-handlers-node-details-pane.md
:white_check_mark:Succeeded
View

technical-reference/adapter-properties-dialog-box.md
:white_check_mark:Succeeded
View

technical-reference/adapter-type-node.md
:white_check_mark:Succeeded
View

technical-reference/adapter-ui-help.md
:white_check_mark:Succeeded
View

technical-reference/adapters-node-details-pane.md
:white_check_mark:Succeeded
View

technical-reference/adapters-node.md
:white_check_mark:Succeeded
View

technical-reference/add-adapter-metadata-wizard-dialog-box.md
:white_check_mark:Succeeded
View

technical-reference/add-application-reference-dialog-box.md
:white_check_mark:Succeeded
View

technical-reference/add-days-functoid.md
:white_check_mark:Succeeded
View

technical-reference/add-policies-dialog-box.md
:white_check_mark:Succeeded
View

technical-reference/add-policies-page.md
:white_check_mark:Succeeded
View

technical-reference/add-resources-dialog-box.md
:white_check_mark:Succeeded
View

technical-reference/addition-functoid.md
:white_check_mark:Succeeded
View

technical-reference/administration-ui-help.md
:white_check_mark:Succeeded
View

technical-reference/advanced-functoids-reference.md
:white_check_mark:Succeeded
View

technical-reference/affiliate-applications-properties-accounts.md
:white_check_mark:Succeeded
View

technical-reference/affiliate-applications-properties-fields.md
:white_check_mark:Succeeded
View

technical-reference/affiliate-applications-properties-general.md
:white_check_mark:Succeeded
View

technical-reference/affiliate-applications-properties-options.md
:white_check_mark:Succeeded
View

technical-reference/affiliate-applications-properties.md
:white_check_mark:Succeeded
View

technical-reference/affiliate-applications.md
:white_check_mark:Succeeded
View

technical-reference/aggregation-viewer.md
:white_check_mark:Succeeded
View

technical-reference/alert-manager.md
:white_check_mark:Succeeded
View

technical-reference/all-artifacts-node.md
:white_check_mark:Succeeded
View

technical-reference/all-group-node-properties.md
:white_check_mark:Succeeded
View

technical-reference/any-attribute-node-properties.md
:white_check_mark:Succeeded
View

technical-reference/any-element-node-properties.md
:white_check_mark:Succeeded
View

technical-reference/application-deployment-ui-help.md
:white_check_mark:Succeeded
View

technical-reference/application-name-node.md
:white_check_mark:Succeeded
View

technical-reference/application-properties-dialog-box.md
:white_check_mark:Succeeded
View

technical-reference/applications-node.md
:white_check_mark:Succeeded
View

technical-reference/arc-tangent-functoid.md
:white_check_mark:Succeeded
View

technical-reference/as2-message-and-correlated-mdn-status-page-ui.md
:white_check_mark:Succeeded
View

technical-reference/ascii-to-character-functoid.md
:white_check_mark:Succeeded
View

technical-reference/assembly-viewer-ui-help.md
:white_check_mark:Succeeded
View

technical-reference/assert-functoid-reference.md
:white_check_mark:Succeeded
View

technical-reference/attribute-formdefault-node-property-of-all-schemas.md
:white_check_mark:Succeeded
View

technical-reference/attribute-group-node-properties.md
:white_check_mark:Succeeded
View

technical-reference/bam-add-in-for-excel.md
:white_check_mark:Succeeded
View

technical-reference/base-data-type-node-property-of-all-schemas.md
:white_check_mark:Succeeded
View

technical-reference/base-specified-logarithm-functoid.md
:white_check_mark:Succeeded
View

technical-reference/base-type-node-property-of-all-schemas.md
:white_check_mark:Succeeded
View

technical-reference/batch-filter-page.md
:white_check_mark:Succeeded
View

technical-reference/batch-schedule-page.md
:white_check_mark:Succeeded
View

technical-reference/batch-status-page-ui.md
:white_check_mark:Succeeded
View

This comment lists only the first 50 files in the pull request.
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@nschonni Why are you removing the en-us from the URLs? Are they causing build warnings in OPS?

Links to other docsets are supposed to omit the locale https://docs.microsoft.com/en-ca/contribute/how-to-write-links#links-to-pages-that-are-not-part-of-the-technical-documentation-set
I'm not sure about ms:mtpsurl because I haven't seen that in other docs metadata before though. If it's needed I can try and update this so it doesn't touch those lines.

Yeah - I know. I'm not sure who put those links in the ms:mtpsurl metadata. The publishing team is doing some redirects, and I suspect they put those links in there for some reason. To confirm, did these links cause build warnings? Or, are you removing the locale to follow guidelines?
For the links to other docSets, it'd be great if we could filter those out, remove the locale, and then I'll merge.

Rebased and changed the find/replace to ignore the ms:mtpsurl links

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

technical-reference/10-n-functoid.md
:white_check_mark:Succeeded
View

technical-reference/absolute-value-functoid.md
:white_check_mark:Succeeded
View

technical-reference/actions-editor.md
:white_check_mark:Succeeded
View

technical-reference/adapter-handler-properties-dialog-box.md
:white_check_mark:Succeeded
View

technical-reference/adapter-handlers-node-details-pane.md
:white_check_mark:Succeeded
View

technical-reference/adapter-properties-dialog-box.md
:white_check_mark:Succeeded
View

technical-reference/adapter-type-node.md
:white_check_mark:Succeeded
View

technical-reference/adapter-ui-help.md
:white_check_mark:Succeeded
View

technical-reference/adapters-node-details-pane.md
:white_check_mark:Succeeded
View

technical-reference/adapters-node.md
:white_check_mark:Succeeded
View

technical-reference/add-adapter-metadata-wizard-dialog-box.md
:white_check_mark:Succeeded
View

technical-reference/add-application-reference-dialog-box.md
:white_check_mark:Succeeded
View

technical-reference/add-days-functoid.md
:white_check_mark:Succeeded
View

technical-reference/add-policies-dialog-box.md
:white_check_mark:Succeeded
View

technical-reference/add-policies-page.md
:white_check_mark:Succeeded
View

technical-reference/add-resources-dialog-box.md
:white_check_mark:Succeeded
View

technical-reference/addition-functoid.md
:white_check_mark:Succeeded
View

technical-reference/advanced-functoids-reference.md
:white_check_mark:Succeeded
View

technical-reference/affiliate-applications-properties-accounts.md
:white_check_mark:Succeeded
View

technical-reference/affiliate-applications-properties-fields.md
:white_check_mark:Succeeded
View

technical-reference/affiliate-applications-properties-general.md
:white_check_mark:Succeeded
View

technical-reference/affiliate-applications-properties-options.md
:white_check_mark:Succeeded
View

technical-reference/affiliate-applications.md
:white_check_mark:Succeeded
View

technical-reference/all-artifacts-node.md
:white_check_mark:Succeeded
View

technical-reference/all-group-node-properties.md
:white_check_mark:Succeeded
View

technical-reference/any-attribute-node-properties.md
:white_check_mark:Succeeded
View

technical-reference/any-element-node-properties.md
:white_check_mark:Succeeded
View

technical-reference/application-name-node.md
:white_check_mark:Succeeded
View

technical-reference/application-properties-dialog-box.md
:white_check_mark:Succeeded
View

technical-reference/applications-node.md
:white_check_mark:Succeeded
View

technical-reference/arc-tangent-functoid.md
:white_check_mark:Succeeded
View

technical-reference/ascii-to-character-functoid.md
:white_check_mark:Succeeded
View

technical-reference/assembly-viewer-ui-help.md
:white_check_mark:Succeeded
View

technical-reference/assert-functoid-reference.md
:white_check_mark:Succeeded
View

technical-reference/attribute-formdefault-node-property-of-all-schemas.md
:white_check_mark:Succeeded
View

technical-reference/attribute-group-node-properties.md
:white_check_mark:Succeeded
View

technical-reference/bam-add-in-for-excel.md
:white_check_mark:Succeeded
View

technical-reference/base-specified-logarithm-functoid.md
:white_check_mark:Succeeded
View

technical-reference/bind-party-to-role-dialog-box.md
:white_check_mark:Succeeded
View

technical-reference/biztalk-editor-ui-help.md
:white_check_mark:Succeeded
View

technical-reference/biztalk-flat-file-schema-wizard-child-elements-page.md
:white_check_mark:Succeeded
View

technical-reference/biztalk-flat-file-schema-wizard-delimited-record-page.md
:white_check_mark:Succeeded
View

technical-reference/biztalk-flat-file-schema-wizard-overview-page.md
:white_check_mark:Succeeded
View

technical-reference/biztalk-flat-file-schema-wizard-positional-record-page.md
:white_check_mark:Succeeded
View

technical-reference/biztalk-flat-file-schema-wizard-schema-information-page.md
:white_check_mark:Succeeded
View

technical-reference/biztalk-flat-file-schema-wizard-schema-view-page.md
:white_check_mark:Succeeded
View

technical-reference/biztalk-flat-file-schema-wizard-select-document-data-page.md
:white_check_mark:Succeeded
View

technical-reference/biztalk-flat-file-schema-wizard-select-record-format-page.md
:white_check_mark:Succeeded
View

technical-reference/biztalk-flat-file-schema-wizard-summary-page.md
:white_check_mark:Succeeded
View

technical-reference/biztalk-framework-assembler-pipeline-component-properties.md
:white_check_mark:Succeeded
View

This comment lists only the first 50 files in the pull request.
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Scrivner Windows

I'm a Scrivner Windows user and was wondering if the Cite as you Write feature will be made available for Windows in the future.

CAYW already works on windows (clicking http://localhost:23119/better-bibtex/cayw?format=mmd&clipboard=yes will call up the picker and leave the results in both the browser and the clipboard), but pasting the value into Scrivener would have to be done separately.
I know Scrivener has a configurable shortcut key for calling up a reference manager, but it's pretty feeble. It's best to ask scrivener support how to integrate BBT CAYW -- BBTs CAYW returns the picks, but it's up to the editors that want to use it to integrate those picks.

Thanks for the quick reply, Emiliano.
I'll do just that, but one more question - the link you sent appears dead as it states, 'No endpoint found'. What exactly is it supposed to link to?
Many thanks

Do you have BBT installed and zotero running when you call that url?

closing for inactivity -- commenting will automatically re-open the issue.


Add now function to MysqlSqlModule

Add support for function now described in the MySQL docs.
Add function to MysqlFunctionDef in MysqlSqlModule
Add test in FunctionDefSpec

I would like to take this one.


refactor: migrate interpretation unit test using annotated_eval

coverage: 80.321% (+5.3%) from 75.07%
when pulling be511453cf90cd279d6d675c1dbf3ad358ad06cc on migrate-unittest
into ba5029537763e1727b849663847b89313dc504c1 on master.


feat: support kotlin, jetpack compose, TxRxChecker

Summary
Euphony will be used easily if it supports android compose.
So, I added kotlin and jetpack compose dependencies and implemented TxRxChecker.
TxRxChecker has 3 components

input text field (you can input text that you want to send using euphony, if it is empty, it will generate random string; a-Z,A-Z,0-9)
play button (if you click this button, euphony Tx/Rx will work)
result icon (it shows euphony Tx/Rx result)

Due to android compose, I updated version of gradle and JDK of github action.
And, executing UI test at local was worked on API 32, so I also updated compileSdkVersion and targetSdkVersion.
Usage
@Composable
fun TestTxRxChecker() {
    // it will auto-generate text with length=5
    TxRxChecker(randomLength = 5)

    // users can input text with their own keyboard
    TxRxChecker(randomLength = 0)
}

Initial State

Success Result

Fail Result

Build & Test Result

@designe I updated jvmargs and compose version :smile: Including this commit, please review PR more :)

@designe
As you suggested, I updated build.gradle and moved constants :smile:
Thank you for detailed reviews!

@yeonns Great :) Thanks for reflecting my suggests.
Finally, we need to resolve the below issue.
 Process: com.example.myapplication, PID: 22298
    java.lang.NoClassDefFoundError: Failed resolution of: Landroidx/compose/runtime/livedata/LiveDataAdapterKt;
        at co.euphony.ui.TxRxCheckerKt.TxRxChecker(TxRxChecker.kt:37)

After porting to sample app, above issue occurred.
I'm gonna check why this issue happened :)

@designe Now, I changed from LiveData to StateFlow. So, user can only import euphony dependencies and just call TxRxChecker()!


More documentation

cc @abarth

I'm really worried about all this sample code going stale.

I'm really worried about all this sample code going stale.

My plan is to analyze it soon, using logic similar to what we have on the Web site.

I'm going to land this now then rename it to MyImage in a separate PR momentarily.

Work in progress for analysis: https://github.com/flutter/flutter/compare/master...Hixie:analyze-sample-code


add "exit" Cargo feature

When this feature is enabled the panic handler performs an exit semihosting call
after logging the panic message. This is useful when emulating the program on
QEMU as it causes the QEMU process to exit with a non-zero exit code; thus it
can be used to implement Cortex-M tests that run on the host.
this also prepares the crate for a new release

bors r+

Published: https://crates.io/crates/panic-semihosting/0.5.1


Possible Unintended Side Effects of: always look at .config.{yml,yaml,json,js,cjs}

8.1.0

Added: always look at .config.{yml,yaml,json,js,cjs} file to configure cosmiconfig itself, and look for tool configuration in it using packageProp (similar to package.json)
For more info on this, look at the end user configuration section of the README

Is there a way to turn this off? I'm hunting down a crashing issue https://github.com/streetsidesoftware/vscode-spell-checker/issues/2583 that is most likely caused by a user's .config.* changing the .json parser.

You can explicitly not overwrite loaders in that config file (as of now): https://github.com/cosmiconfig/cosmiconfig/blob/dacce131ed0d8e121d3155e5922bfb4998822187/src/index.ts#L125-L127
This might be a bug relating to the loading though. Will investigate as soon as your user has answered your question in your issue so I have a proper lead on this.

@d-fischer,
He has confirmed that there is a .config.js: https://github.com/streetsidesoftware/vscode-spell-checker/issues/2583#issuecomment-1473284396
The issue is right there in the code, options is lost if there is a metaConfig:
https://github.com/cosmiconfig/cosmiconfig/blob/dacce131ed0d8e121d3155e5922bfb4998822187/src/index.ts#L119-L138

@d-fischer,
Does it make sense to "merger" the searchPlaces instead of replacing them? I can understand if a User wants to add their preferred places, but replacing the ones specified by the cli-tool can easily cause things to break.
For example, cspell has multiple places for a configuration file. It has a defined search pattern. Sub folders and parent folders might both have config files. If the .config.js at the root is there to clean up the root, it would mess up the sub folders.
Case in point, if someone adds a .config.js to adjust the location of prettier, they could break cspell or eslint because a "single" config is applied to ALL tools that use Cosmiconfig.

Fixed in 8.1.1. Again, thanks for your efforts!


In android, when click a href with img returns img src on onCreateWindow

[O] I have read the Get Started - Installation section
[O] I have read and done the Get Started - Setup Android section
[O] I have read and done the Get Started - Setup iOS section
[O] I have already searched for the same problem

Environment

Technology
Version

Flutter version
2.2.3

Plugin version
5.3.2

Android version
10

iOS version

Xcode version

Device information: LG G7, Samsung Galaxy Z flip
Description
In android, when click a-tag with img-tag,
createWindowRequest.request.url returns img-src on onCreateWindow method.

Open link in a new window or tab: Visit W3Schools!

When I modified the following in the onCreateWindow function of the InAppWebViewChromeClient.java class, I was able to get the correct result.
`
public boolean onCreateWindow(WebView view, boolean isDialog, boolean isUserGesture, final Message resultMsg) {
WebView.HitTestResult result = view.getHitTestResult();
String url = result.getExtra();
Log.i(LOG_TAG, "onCreateWindow url = "+ result.getExtra() + " , type = " + result.getType());
//Link with Image
if(result.getType()==HitTestResult.SRC_IMAGE_ANCHOR_TYPE) {
  Message href = view.getHandler().obtainMessage();
  view.requestFocusNodeHref(href);
  String url2 = href.getData().getString("url");
  Log.i(LOG_TAG, "onCreateWindow url2 = "+ url2);
  if(url2.isEmpty() == false)
    url = url2;
}

}
`
Hopefully it will be improved in the next version.

Thank you, this is a key fix for Android.

Merged 6b7e4b856f8c6ec56e8522853a27c84b2d0704b3


Installation on OSX 10.11

default machine can't boot-up in OSX 10.11
Error as shown in Kinematics:
/Applications/Docker/Kitematic (Beta).app/Contents/Resources/resources/docker-machine -D create -d virtualbox --virtualbox-memory 2048 default returned non zero exit code. Stderr: Converting from raw image file="stdin" to file="/Users/<redacted>/.docker/machine/machines/default/disk.vmdk"... Creating dynamic image with size<PHONE_NUMBER>0 bytes (20000MB)... Error creating machine: exit status 1 You will want to check the provider to make sure the machine and associated resources were properly removed.

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

docker version:
Client:
 Version:      1.8.1
 API version:  1.20
 Go version:   go1.4.2
 Git commit:   d12ea79
 Built:        Thu Aug 13 02:49:29 UTC 2015
 OS/Arch:      darwin/amd64
Get http:///var/run/docker.sock/v1.20/version: dial unix /var/run/docker.sock: no such file or directory.
* Are you trying to connect to a TLS-enabled daemon without TLS?
* Is your docker daemon up and running?

docker info:
Get http:///var/run/docker.sock/v1.20/info: dial unix /var/run/docker.sock: no such file or directory.
* Are you trying to connect to a TLS-enabled daemon without TLS?
* Is your docker daemon up and running?

uname -a
Darwin Vikrams-Air.local 15.0.0 Darwin Kernel Version 15.0.0: Wed Aug 26 19:41:34 PDT 2015; root:xnu-3247.1.106~5/RELEASE_X86_64 x86_64

Steps to reproduce:
Install docker on OSX 10.11 and do initial setup.
Result received: unable to launch default docker vm with error:
/Applications/Docker/Kitematic (Beta).app/Contents/Resources/resources/docker-machine -D create -d virtualbox --virtualbox-memory 2048 default returned non zero exit code. Stderr: Converting from raw image file="stdin" to file="/Users/<redacted>/.docker/machine/machines/default/disk.vmdk"... Creating dynamic image with size<PHONE_NUMBER>0 bytes (20000MB)... Error creating machine: exit status 1 You will want to check the provider to make sure the machine and associated resources were properly removed.
Expected result: A docker vm to spin up
I am using preview release of OSX El Captain which might be the primary reason for the bug.

This could actually be an issue in VirtualBox, and looks like a dup of https://github.com/docker/machine/issues/1725#issuecomment-133660200 and https://github.com/docker/toolbox/issues/119
Can you check if updating VirtualBox resolves this for you?

Updating VirtualBox with the test builds worked. Thanks!

Great!


548 navbar accessibility

🔗 Issue
#548
✍ Description
Note: Enthusiastic to hear cleaner, alternative approaches. I am not a fan of bootstrap and avoid it in my personal projects, so I may not be familiar with some better ways to overwrite some of this behavior.
This PR aims to make the navigation bar more accessibility friendly. Currently the right corner navbar on desktop view only shows if you hover the icon. This is terrible for keyboard users as it makes it potentially impossible for them to open it, and that's the only place where some of those links live!
This behavior is coming from media queries from the theme we are using. Some of the culprit rules are here:
@media (min-width: 1200px) {
  .navbar .dropdown-menu {
    display: block;
    opacity: 0;
    transform: translateY(20px);
    transition: all 0.3s ease-in-out;
    visibility: hidden;
  }
  .navbar .dropdown-menu-end {
    left: auto;
    right: 0;
  }
  .navbar .dropdown-menu-start {
    left: 0;
    right: auto;
  }
  .navbar .dropdown-submenu:hover > .dropdown-menu,
  .navbar .dropdown:hover > .dropdown-menu {
    opacity: 1;
    transform: scaleY(1);
    visibility: visible;
  }
}

The first approach I tried was to just overwrite the CSS rules I found from the theme. However, that approach did not work because I don't think there is a way to overwrite the JavaScript that the theme is utilizing as well based off of the viewport widths.
What I opted for instead was to create our own custom class for the menu and also use a simple Stimulus controller to toggle/collapse the menu.
What I do not like about this approach is the custom CSS. If we change the theme, we would have to update this CSS as well. I think the Stimulus controller is nice for our own control, but it may not fully mimic the behavior from BS.
Alternatives:

Instead of removing the theme classes and adding our own to the elements, we could keep the old theme classes, overwrite all the rules causing this behavior, and then also use our own JavaScript to enforce the toggling for the desktop view. Advantage to this approach is that we no longer need the custom css for our own class, but now we have custom css to overwrite all related rules. It may also be confusing having BS handling the JS for all the smaller viewports and then our own JS for the larger desktop one. We may have to enforce our own JS to only target the related elements for the desktop view too because the BS Javascript is going to still try to target them in the mobile view. That may cause some conflict.
Yuri on slack suggested reorganizing the HTML to try to avoid BS applying the theme's navbar behavior by taking the navbar menu button out of the "navbar":

 Navbar Link etc

Icon Stuff

I think I might like this approach better, but I also am not fully satisfied with it too. Potential issues with it I see are:

We may have to fight the navbar to get it to look the same as it currently does, as the BS theme isn't expecting the menu button to be outside of it. For example, the second mobile menu will need some adjustment.
If we end up wanting more dropdowns in the navbar for any reason, not placed at the edge, this could potentially get even more convoluted.
Having a BS theme dictate how our elements are semantically arranged feels bad.

Fun fact, I noticed this hover behavior of the navbar has a test using a 5 second hover wait built into the test/system/user.rb test. Once @kasugaijin refactors the navbars so we use this one in both places, this new, no-hover behavior means we can shave 5 seconds off our test suite! lol
📷 Screenshots/Demos
In this clip, the left side is the live site (main) and the right side is this branch.
https://github.com/rubyforgood/pet-rescue/assets/81536479/23e4683b-872e-47e9-b104-eff73a45e23d

@Developer3027 I just uploaded this PR for the issue, if you have a chance to look it over and share your thoughts as well.

@mononoken this is a good thing to change up, though I do have some thoughts on it:

I prefer we don't modify the default theme much, to keep things simple. However, in this case, this is improved accessibility for an important feature. So worth some investment in my opinion (thank you!). Some googling on UX best practices yields 'The hover approach is popular but click-to-open is the best way to ensure the menu works reliably and intuitively across all devices.' So, yes, let's go with the click.
I am fine with your solution. I think adding some CSS and JS in this case is justifiable. Though, I would request we add comments to both so that it is obvious what they are doing i.e. overriding the theme functionality, and why.
In your implementation, I need to click on the div element (circle) to open the menu, which is fine. I was expecting to be able to click anywhere else on the window to close the menu. Currently, I have to click the div element to close the menu.

@mononoken this is a good thing to change up, though I do have some thoughts on it:

I prefer we don't modify the default theme much, to keep things simple. However, in this case, this is improved accessibility for an important feature. So worth some investment in my opinion (thank you!). Some googling on UX best practices yields 'The hover approach is popular but click-to-open is the best way to ensure the menu works reliably and intuitively across all devices.' So, yes, let's go with the click.
I am fine with your solution. I think adding some CSS and JS in this case is justifiable. Though, I would request we add comments to both so that it is obvious what they are doing i.e. overriding the theme functionality, and why.
In your implementation, I need to click on the div element (circle) to open the menu, which is fine. I was expecting to be able to click anywhere else on the window to close the menu. Currently, I have to click the div element to close the menu.

@kasugaijin Good suggestion! I added clicking outside to close the menu. I also added the comments.

I agree with above. Will review this afternoon. Thanks!

After review, I like this approach. This does use a new css style set, however it follows convention, works like intended, and utilizes current theme attributes. The javascript in the Stimulus controller is straight forward and easy to read. Comments allow a clear understanding of what it does and the why is
intuitive.
I agree it needs to be understood why this was done and to take note of both the Stimulus controller and the styles set. Maybe consider a note in the read me? Is documentation being considered, maybe there?
@mononoken @kasugaijin

After review, I like this approach. This does use a new css style set, however it follows convention, works like intended, and utilizes current theme attributes. The javascript in the Stimulus controller is straight forward and easy to read. Comments allow a clear understanding of what it does and the why is intuitive.
I agree it needs to be understood why this was done and to take note of both the Stimulus controller and the styles set. Maybe consider a note in the read me? Is documentation being considered, maybe there? @mononoken @kasugaijin

Documentation is an ongoing challenge in codebases. I think the further the documentation gets away from the code, the harder it can be to find and update, and the README might not be the most practical location for code decisions. Ideally, code speaks for itself, and comments add context. I think in this case, I am happy with the inline comments, as added by @mononoken.

Thank you both!
Got the comment updated to ERB per Slack @kasugaijin . Think this should be good for final review.


Add wslbridge-backend path from command

I've added the -p option to add wslbridge-backend path. With that option, the backend find can be put in anywhere.

Please reply if I've to edit something small.

You already closed https://github.com/rprichard/wslbridge/issues/23?
I don't think I understand why this option is needed. You've run into a situation where the two files can't be in the same directory, I guess?
I'd prefer something more descriptive than -p. Maybe --backend <PATH>.
It looks like your -p option accepts a Windows-style path rather than a WSL-style path. The Windows-to-WSL path conversion has had some issues, so if there's an option to specify the backend path, maybe it ought to be a WSL-style path to sidestep those issues.

With this option, users have no restriction to place both file in same directory, they can place those anywhere. For the -p option, I follow your previous one alphabetic options.  To convert the path you've use that function, so do I.
I tried with both 1. Place backend and  frontend in same path. 2. Use -p option with only the directory of backend and both works.

I've tried to compile the backend as statically links. But some errors occur. See this https://github.com/mintty/mintty/issues/500#comment-368812621

Ok wait sometime, I'll fix that.

@rprichard Sorry for the delay. I've exams. Try the fix.

Any feedback?

Is there any problem?

May you please accept these changes? This helps in wsltty.appx project.

Why does the wsltty.appx project need the flag?

Because in UWP environment the backend cannot be executed. So it need to copied in LocalAppData. And with that option I can execute backend from that path.

🙇

Ok, that makes sense. I'll try to look at the change soon. Maybe I'll just add a --backend option.

I had added that option I second commit.

Not necessary. I already have this behavior written. I'll push it within a day:
  --backend BACKEND
                Overrides the default path to wslbridge-backend. BACKEND is a
                Cygwin-style path (not a WSL path).

It takes a path to the file rather than a path to a directory containing the file. I assume that's not an issue.

Is it then take cygdrive/c not C:\? But in UWP API that although will be in Windows style.

It's just the normal Cygwin rules. /cygdrive/c/foo is OK, as is C:\foo. /foo and \foo, however, will refer to C:\path\to\cygwin\foo, whereas the ordinary Windows rules would interpret it as drive-relative, e.g. C:\foo.

Should I close this PR?

Yes, please close it.

May you list all the options of frontend and backend in readme file?

When do you release this current feature?


where is the config file?

anyone know?
I opened a new connection and now it keeps trying to connect to my old session. wtf

in the home folder. use this if it breaks:
{"sidebarWidth":"250","locked":"false","CLIHeight":"420","CLIOpen":"true","default_connections":[{"label":"local","host":"<IP_ADDRESS>","port":"6379","password":"","dbIndex":"0"}]}

Cheers.

For the people searching for the config file with unwanted connections, try cat ~/.redis-commander.

As this ticket refers to the old config file not used anymore it will be closed to not distract new users.
New configuration files can be found inside config folder of the app. To fully understand the new possiblities of the config folder please refer to https://github.com/lorenwest/node-config


UI for landing page

Is your feature request related to a problem? Please describe.
When I clicked on try now button, I landed on the home page for login where the color contrast of the page made things difficult to see.
https://github.com/amanjaiman1/Product_3D/assets/140234197/108e471e-ff56-4f0c-8d33-51786cdd24ff
Describe the solution you'd like
I would like to proceed with the changes in the UI for this page.
Describe alternatives you've considered
No response
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[ ] I'm a GSSoC'23 contributor
[X] I want to work on this issue

Hey @07tAnYa , drop PR
I'll approve it


Graceful reloading/reauthentication on credentials rotation

Ensure that both the kafka and pubsublite code gracefully handles credentials changes.
Related: https://github.com/elastic/apm-queue/issues/234 https://github.com/elastic/apm-queue/issues/235

At least in theory, the seed brokers list could change over time too. e.g. if migrating to a different Azure Event Hubs namespace. This can be achieved by calling kgo.Client.UpdateSeedBrokers.

For Kafka, I'm thinking of writing a kgo hook that will periodically load a file from disk, extracting select properties defined at https://github.com/confluentinc/librdkafka/blob/master/CONFIGURATION.md (namely bootstrap.servers and sasl.{username,password}), compare them to the last loaded values, and reconfigure the client if they have changed.

@axw is the kafka part done with https://github.com/elastic/apm-queue/pull/241?
I think it would be best to re-focus this issue on aws and create a dedicated one for the work in azure EventHubs.

@simitt that PR handles both Kafka and Event Hubs. I don't think I tested Pub/Sub Lite reloading yet, so that would need a new issue.

The part that is left to do here is updating each of the services to load the config from a file. @lahsivjar pointed out that https://github.com/knadh/koanf#reading-config-from-files might be a good choice.

Still need to verify GCP, will come back to that later.


Rebasing appears to lose non-greenkeeper commits

Hello greenkeeper team,
Your recent change to automatically rebase GK pull requests when a prior GK PR is merged is great for nice clean git histories, so thank you for that.
However it seems like it's not really doing a rebase, and instead is re-creating the commit and force-pushing, which has the side effect of wiping out any other commits in the branch. Since the PR body itself says to "use this branch to work on adaptions and fixes", this has the potential to lose some very important work.
Hopefully you can fix it soon. Cheers!

Hey @orangejulius,
thanks for this report. As a quick fix, to not lose data, we're now leaving branches with additional commits alone. We might work on actually rebasing those as well in the future.
This change is effective since April 25th, but I failed to communicate it. Sorry for that.
Thanks,
Stephan


Bug: xstate swallows exceptions if there are subscribers

Description
When an exception is thrown in an action, the error is visible and reported, unless there are subscribers to the state machine.
If there are subscribers, then the machine is simply stopped without any output.
Expected result
I would expect the error to be raised independently of subscribers.
Actual result
Exceptions just disappear
Reproduction
https://codesandbox.io/s/ancient-violet-2jp4fz
Additional context
Tested with v5.0.0-beta.10 to beta.17

The error story will be revamped soon. The intention is to never swallow things and that will be the primary design goal of the refactor in this area.

Until then, if anybody runs into this: this is how you still get errors:
  machine.subscribe({
    next: (state) => {
      console.log('State:', state.value)
    },
    complete: () => console.log(`Game machine ${id} completed`),
    error: (error) => console.error(‘ERROR’, error),
  })

I confirmed manually that this is no longer the problem with https://github.com/statelyai/xstate/pull/4145


RETURN returning wrong amount of values

RETURN is returning the wrong amount of values. RETURN seems to be returning the correct number + the proto param count.

Must've missed this, could you add a test to check for this in the future?

On the topic of tests I should probably add some automatic ci to test, I haven't had time to properly port Luau's conformance tests so a few bugs have been missed. Once Luau's conformance tests pass when they're implemented I'll make a proper release for Fiu.

Must've missed this, could you add a test to check for this in the future?

How would I do that? The values it tries to return are usually nil, since it goes past the stack.

How would I do that? The values it tries to return are usually nil, since it goes past the stack.

Make a file called "Returns" in Sources and run CreateTests

I've added the conformance tests and split up my original tests into staging. I'll take a look at making conformance tests work properly next month hopefully. I have also added tests for your fix under a staging test called "Returns". Once all tests pass I'll make a proper release for Fiu and then implement github actions for testing Fiu every time Luau updates and for pull requests. Thank you for the fix.


Generate changelog.md file

Fixes #27

The documentation is missing at this point. It would be nice to check that the approach makes sense before I do so.
Overall, this how it's supposed to work, it will:

Check if a new version should be released, if not stop here, otherwise continue.
Update CHANGELOG.md and replace the placeholder <!--next-version-placeholder--> with changelog from the version being released
Do git add for the CHANGELOG.md file
Proceed as currently with updating version in source control, git add, commit, push, etc...

What are the expected behaviour when the CHANGELOG.md file doesn't exist or when it doesn't have the placeholder? Should we just skip this step and log a warning?

Actually, I've update the docs.
I realised while writing it that if the changelog file doesn't exist, we can have a better default and create it, so I've changed the code accordingly. However if the placeholder isn't in the file, probably safer to skip this step.
I'm open to suggestions to alternative solution that work without placeholders (or give it a better name).

Nearly there: https://github.com/browniebroke/django-codemod/blob/master/CHANGELOG.md
The commit hash aren't linked automatically like in the GH release... I guess we could generate the links but would need to be working with gitlab as well.

You can see the results of the following commits in releases v0.17.3, v0.17.4 and v0.17.5 here: https://github.com/browniebroke/django-codemod/blob/master/CHANGELOG.md


EOS tokenizer support.

This is really cool stuff.
I was using vicuna api server so far, it worked great for me. I would like to explore this now, and I was wondering how you were taking care of the eos token problem, as different models have different tokens as eos. if you are, can you point me to the list of models you are supporting?

Hi @iRanadheer, thanks for your interest! We use HuggingFace's tokenizer. If I understand correctly, the tokenizer = AutoTokenizer.from_pretrained(model_name) associates the tokenizer with tokenizer.eos_id. We use this for identifying the EOS token. Users can also specify their stopping strings in SamplingParams.
Here's the list of supported models: https://vllm.readthedocs.io/en/latest/models/supported_models.html
You can use the open source models in HF model hub, or your own model that follows one of our supported HF model architectures.


Allied Vision Vimba-X Support

Allied Vision has released a new SDK called "Vimba X". It uses vmbpy as opposed to the earlier "Vimba" SDK which had the vimba module.
https://www.alliedvision.com/en/products/software/vimba-x-sdk/

I think we probably won't pursue this. The site you linked hints at support for Windows 10+, and I would like to maintain support for older versions of Windows. The new features that Vimba-X do not seem applicable to our needs. I would suggest that we only move to Vimba-X if 1) Vimba loses support, or 2) it is clear that Vimba-X is more stable than Vimba.

I changed my mind. We should upgrade next release.

#81 and #83 implement this.


how to get values (MQTT subscribe?) and store into DB

Hello, first, big thanks for this project, working great. I'd like to store data from a BMS to a DB (already used for inverter data), but struggling how to do that. I did a scan of MQTT topics with MQTT.fx tool, and if I subscribe to f.i. homeassistant/sensor/jbd-bms-uart/jbd-bms-uart_discharging_power/config, payload data are like below, but no value. How can I get them? Thanks!
{"dev_cla":"power","unit_of_meas":"W","stat_cla":"measurement","name":"jbd-bms-uart discharging power","stat_t":"jbd-bms-uart/sensor/jbd-bms-uart_discharging_power/state","avty_t":"jbd-bms-uart/status","uniq_id":"ESPsensorjbd-bms-uart_discharging_power","dev":{"ids":"c8c9a30bf3fd","name":"jbd-bms-uart","sw":"esphome v2022.12.4 Jan 22 2023, 18:37:42","mdl":"nodemcu","mf":"espressif"}}

If your BMS is attached the component should request measurements periodically and publish these values as sensor state. You have to subscribe to all state topics you are interested in. You could use a tool telegraf to pass the data into a influxdb.

I had to enable mqtt in the yaml.


prohibit R reg from using CoreRegs::get

Closing since in the new v0.12 ISA there is no R registers (they will become ISA extension and will be re-implemented from scratch)


Event Type checking

In the future, we should be able to allow developers to enforce a specific type when declaring an event:
let event = new ObjectEvent<funcType>() // Is this valid?

So that functions being connected to said event must to take the argument types defined within the event:
let event = new ObjectEvent<(a: number, b: number) => void>()

// ERROR!
event.Connect(function(a: string) {
    print(a)
})

So the tackling goes as following:

Verify whether this is possible;
Investigate the syntax for this;
Implement!

I think it might be possible. Opened a dev branch for this.

We might be able to accomplish this by exposing two versions of the interface:
ObjectEvent - behavior will be the one we're used to (pass anything, receive any)
ObjectEvent<T1, T2, T3, ...> - this generic will restrict whatever you'll be able to pass in, and what kind of functions you can connect to (but we ensure type safety at compile time)

This would be hard to do without writing the entire module in TS.

I'm not sure - I hadn't the time to check this one out so far... 😅

Not as "beautiful" as I wanted, but this does the trick


Anyway to turn the service worker off for development?

I keep getting service worker errors, like all the time...any way to disable the service worker until i'm testing on the server?

thanks :-)

Which version of template pwa do you use? 2.0.1 version has this patch https://github.com/vuejs-templates/pwa/pull/21

let me check...where is the version of the template listed for me to check?

doesn't look like i do...

Then you probably need to upgrade manually. You can check the diff of #21 https://github.com/vuejs-templates/pwa/pull/21/files
If you didn't touch these files before, maybe git patch will work.
GitHub provides patch file for every commit, for #21: https://patch-diff.githubusercontent.com/raw/vuejs-templates/pwa/pull/21.patch
Another solution is to generate a new project and move your code to the new project.

I'll go through these options and see what works then, thanks :-)

k, updated my vue-cli and made a new project through the cli using the steps outlined in the instructions, aaaaaand...

I'm till getting the service worker errors, using everything up to date...
?

@rchrdnsh Would you check out that /build/service-worker-dev.js files is in the project and let me see the code of index.html file

here is my src of index.html as get out of box

<!DOCTYPE html>

    <meta charset="utf-8">
    <meta http-equiv="X-UA-Compatible" content="IE=edge">
    <meta name="viewport" content="width=device-width, initial-scale=1">
    <title>vpwa2</title>
    <link rel="icon" type="image/png" sizes="32x32" href="/static/img/icons/favicon-32x32.png">
    <link rel="icon" type="image/png" sizes="16x16" href="/static/img/icons/favicon-16x16.png">

    <link rel="manifest" href="/static/manifest.json">
    <meta name="theme-color" content="#4DBA87">

    <meta name="apple-mobile-web-app-capable" content="yes">
    <meta name="apple-mobile-web-app-status-bar-style" content="black">
    <meta name="apple-mobile-web-app-title" content="vpwa2">
    <link rel="apple-touch-icon" href="/static/img/icons/apple-touch-icon-152x152.png">

    <meta name="msapplication-TileImage" content="/static/img/icons/msapplication-icon-144x144.png">
    <meta name="msapplication-TileColor" content="#000000">

    <link rel="preload" href="/app.js" as="script">

This is your fallback content in case JavaScript fails to load.

Here is all the code in the index.html document. I added a style reset, but otherwise everything is as I got it form the vue-pwa template:
<!DOCTYPE html>

    <meta charset="utf-8">
    <meta http-equiv="X-UA-Compatible" content="IE=edge">
    <meta name="viewport" content="width=device-width, initial-scale=1">
    <title>vue-pwa</title>
    <link rel="icon" type="image/png" sizes="32x32" href="/static/img/icons/favicon-32x32.png">
    <link rel="icon" type="image/png" sizes="16x16" href="/static/img/icons/favicon-16x16.png">

    <link rel="manifest" href="/static/manifest.json">
    <meta name="theme-color" content="#4DBA87">

    <meta name=”viewport” content=”minimal-ui”>
    <meta name="apple-mobile-web-app-capable" content="yes">
    <meta name="apple-mobile-web-app-status-bar-style" content="black">
    <meta name="apple-mobile-web-app-title" content="vue-pwa">
    <link rel="apple-touch-icon" href="/static/img/icons/apple-touch-icon-152x152.png">

    <meta name="msapplication-TileImage" content="/static/img/icons/msapplication-icon-144x144.png">
    <meta name="msapplication-TileColor" content="#000000">
    <% for (var chunk of webpack.chunks) {
        for (var file of chunk.files) {
          if (file.match(/\.(js|css)$/)) { %>
    <link rel="<%= chunk.initial?'preload':'prefetch' %>" href="<%= htmlWebpackPlugin.files.publicPath + file %>" as="<%= file.match(/\.css$/)?'style':'script' %>"><% }}} %>

This is your fallback content in case JavaScript fails to load.

I don't really understand what's going on in here, so I thank you in advance for your help :-)

It seems that your project was not updated yet. You should init with newer version of template or patch the server-worker code by hand.

just made a brand new  vue-pwa project, same issue...

Here is the full index.html file, nothing changed at all...
<!DOCTYPE html>

    <meta charset="utf-8">
    <meta http-equiv="X-UA-Compatible" content="IE=edge">
    <meta name="viewport" content="width=device-width, initial-scale=1">
    <title>test-project</title>
    <link rel="icon" type="image/png" sizes="32x32" href="/static/img/icons/favicon-32x32.png">
    <link rel="icon" type="image/png" sizes="16x16" href="/static/img/icons/favicon-16x16.png">

    <link rel="manifest" href="/static/manifest.json">
    <meta name="theme-color" content="#4DBA87">

    <meta name="apple-mobile-web-app-capable" content="yes">
    <meta name="apple-mobile-web-app-status-bar-style" content="black">
    <meta name="apple-mobile-web-app-title" content="test-project">
    <link rel="apple-touch-icon" href="/static/img/icons/apple-touch-icon-152x152.png">

    <meta name="msapplication-TileImage" content="/static/img/icons/msapplication-icon-144x144.png">
    <meta name="msapplication-TileColor" content="#000000">
    <% for (var chunk of webpack.chunks) {
        for (var file of chunk.files) {
          if (file.match(/\.(js|css)$/)) { %>
    <link rel="<%= chunk.initial?'preload':'prefetch' %>" href="<%= htmlWebpackPlugin.files.publicPath + file %>" as="<%= file.match(/\.css$/)?'style':'script' %>"><% }}} %>

This is your fallback content in case JavaScript fails to load.

    <%= htmlWebpackPlugin.options.serviceWorkerLoader %>

I dunno....¯_(ツ)_/¯

This index.html is in the ./build before rendered. Please post the index.html from View Page Source

hmmmm...the messages seem to have disappeared...not sure what I did, though...the only thing i can think of is that i updated to chrome 60...otherwise i have not changed anything else in the code of the app...gonna look into it more, though...

Should I post the page source of the local version or the hosted version? Or both?

I can't judge what happened in your environment. I never face the same issue before. Please feel free to close the issue as you can't face the issue

Yeah, I'll keep an eye on it, but things seem to have fixed themselves ¯_(ツ)_/¯


docs: better navigation and notebooks

Summary of Changes

Show "Back to Top" button
Allow to attach documents to section headers (good for overview pages)
Add notebook support

🦙 MegaLinter status: ✅ SUCCESS

Descriptor
Linter
Files
Fixed
Errors
Elapsed time

✅ REPOSITORY
git_diff
yes

no
0.01s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff
MegaLinter is graciously provided by


Add container build

Closes #307

Question: did you push the image to a docker registry (ML Commons' for instance)?

not yet, I wanted to get this merged first, before doing that, it also seems like a better idea to wait until the editor is more finalized before we publish images to anywhere.


version information on FreeBSD

I noticed there's a couple of very minor things that don't work in the version information on my system (FreeBSD 10.3 jail on FreeNAS 9.10 host):
$ ./dehydrated --version
#
# !! WARNING !! No main config file found, using default config!
#
Dehydrated by Lukas Schauer
https://dehydrated.de

Dehydrated version: 0.4.0
GIT-Revision: 0be0ab083f290afbc757b8388a80df458ddfd33c

cat: /etc/issue: No such file or directory
OS:
Used software:
 bash: 4.4.12(2)-release
 sed: sed: illegal option -- -
 curl: curl 7.55.1

At this point the program is hanging waiting for Ctrl+D, after that it resumes:
 awk: awk: unknown option -W ignored
 grep: grep (GNU grep) 2.5.1-FreeBSD
 mktemp: mktemp: illegal option -- -
 diff: diff (GNU diffutils) 2.8.7
 openssl: OpenSSL 1.0.2l  25 May 2017

Not super important in my opinion, although I was momentarily confused by the hang while awk was waiting for input.

This should be fixed with ce9b42d8ad5bc225b7eeeaf47df0690b5d6b54e3 and bc20ec79f3a0edc30b53ed532cdf3e17169f12a6.


fix(notifications): add name to filter

Add to AiNotificationsDestinationFilter the Name attribute.

Codecov Report

Merging #1014 (ea52c39) into main (f2ea03b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1014   +/-   ##
=======================================
  Coverage   40.85%   40.85%
=======================================
  Files          79       79
  Lines        4935     4935
=======================================
  Hits         2016     2016
  Misses       2773     2773
  Partials      146      146

Flag
Coverage Δ

unit
40.85% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Looks good to me 👍


Include country and state fields in sample metadata

Could we include country and state fields to make it easy to compare with the prevalence data?

Just realized that "country" is there. Just missing "state"


Important Notice: Canvas must be visible for Croppie to work

Here's an important notice I think you should put promptly into the page's documentation.
I struggled 2 days to resolve why the .croppie("result") function only turned up blank images with 0 width & height.
The answer was, that in my implementation, I hid the canvas the moment the result is produced. Actually, I hid the canvas before the result was asked for, and this seemingly prevents the canvas from extracting an image.
So, please add to the docs that the canvas needs to be visible to produce results, and also that the place where to hid the canvas after the result operation would be in the following .then( $(element).hide() ).
Cheers

Good point.
The beauty of open source is that you can modify the documentation yourself, and submit a PR.  Especially if you'd like it promptly.
I'd gladly merge in any decent documentation improvement.

Haha, indeed. Kinda forgot that the documentation is in the repo as well. :)


bug: Some of the Edit actions are not possible to rollback using Ctrl+Z

Cody Version
6.0.3
IDE Information
PyCharm 2024.1.2
Windows
Describe the bug

Create unittests using Generate Unit Tests
Select all content and click Document Code
Run Edit Code... and ask it to "remove all comments"

Now try to bring back comments using Ctrl + Z. It does not work.
Expected behavior
Ctrl + Z should always undo most recent text edit.
Additional context
No response

I think that each undo action affects only the most recent edit, which is usually just a few bytes long. If the Cody request has generated a lot of code, the total number of edit actions might exceed the IntelliJ's undo stack.

This does undo-redo as one big chunk now.
The one case where we don't: Generate unit tests, as we stream the test content into the new file. That produces incremental edits and you have to undo-undo-undo through them. But the specific issue here is fixed.


docs: add examples on new methods

Description
Added example notebooks for scale, map, and mirror functions as well as sweep sketch (also known as sweep profile) and sweep chain.
Issue linked
1049
Checklist

[x] I have tested my changes locally.
[x] I have added necessary documentation or updated existing documentation.
[x] I have followed the coding style guidelines of this project.
[x] I have added appropriate unit tests.
[x] I have reviewed my changes before submitting this pull request.
[x] I have linked the issue or issues that are solved to the PR if any.
[x] I have assigned this PR to myself.
[x] I have added the minimum version decorator to any new backend method implemented.
[x] I have made sure that the title of my PR follows Conventional commits style (e.g. feat: extrude circle to cylinder)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 90.22%. Comparing base (c41a222) to head (0932383).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1089   +/-   ##
=======================================
  Coverage   90.22%   90.22%
=======================================
  Files          93       93
  Lines        6835     6835
=======================================
  Hits         6167     6167
  Misses        668      668

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Looks like the examples are missing some basic objects - see https://github.com/ansys/pyansys-geometry/actions/runs/8507282654/job/23298884789?pr=1089#step:11:2509

Working on this to get it running and merged

Thanks a lot for the examples @dastan-ansys!


Allow Chr in chrom

https://github.com/InformaticsGenomicMedicine/DraftCoreDataModel/blob/79450932a44ee677816f9c02e6f5660ea03e5ec1/src/core_variant.py#L179C19-L180
If we're going to check the value at this level of granularity, we might as well support formats like "Chr 2" as well. Please add a regex to first match the pattern ^chr\s*([0-9a-z]+)$ (case insensitive) and then check the captured group against the list of allowed values.

Add support to CHR format:
https://github.com/InformaticsGenomicMedicine/DraftCoreDataModel/blob/66f815799c6c614970c4caed71740a6bd41ce2e6/src/core_variant.py#L305-L336

@rrfreimuth review via zoom. Looks good!


Fix: Support partial initial values for card form

Why
The react native card component supports building a partial form, but the types require a complete card object for initial values.
Also, the 3ds example implementation required a backend to send authenticated requests to the 3ds API.
How

Update the Card component's types to support a partial object being passed in.
Update example RN app to have an example backend which calls the Evervault API with an API key for 3ds sessions

The structure of this isn't ideal, but the alternative of sending authenticated requests from the client isn't something that should ever happen.

Format the example RN app

SocketSecurity ignore-all

@SocketSecurity ignore-all


WatchConfig() support

It’d be great if Piper supported a feature like WatchConfig() from viper across all the source files.

@abh have you tried piper.V().WatchConfig()?


dont show the editor for archives and directories

while viewing files at
project/<uuid:uuid>/resources/<int:pk>/

don't show the ace editor if the file type is a directory or an archive or which cannot be read

I was thinking of doing like this
remove the url for the project/<uuid:uuid>/resources/ item which has type directory

or let the url for directory be unchanged and instead show this message in project/<uuid:uuid>/resources/<int:pk>/


Specify mongo database name in the configuration file.

Since multiple separate Slivka instances may use the same database, it's necessary to specify the database name for each project individually instead of hard-coded "slivkadb".
Options include:

Full %-encoded mongodb URI: mongodb://[username:password@]host[:port][/database]
Very strict and unambiguous, but hard to use.
Mongodb URI but without %-encoding mongodb:\/\/(?:([%\w]+):(.*)@)?([\w./]+(?::\d+)?)\/(\w+)
Easier to use, but may produce ambiguous results if uname or password contains special characters.
Passing connection data object e.g.:host: <IP_ADDRESS>:8000
socket: /tmp/mongo.sock
username: warownia1
password: P@44w0rd
database: slivkadb

If passed a string, strip database name and pass the rest directly to the pymongo.MongoClient
host, dbname = hostname.rstrip('/', 1)
database = MongoClient(host)[dbname]

Closed by 2fca2b198354a5de66f3cfddafe41247054d712f

Closed by e0ff034758b55642a371b5c20e60059c7cec1ed3


NTR: Regulation terms for cGAS/STING signaling pathway

Could I have regulation terms for the GO:0140896  cGAS/STING signaling pathway?
PMID:29875158 https://pubmed.ncbi.nlm.nih.gov/29875158/ shows that Bombyx mori Casp8L can negatively regulate the cGAS/STING signaling pathway

If there is only evidence for the negative regulation, we should only create that one term.

Thanks


Fix sidebar style

It should be of the same height of the page

Fix Merged: https://github.com/BeaconCMS/beacon_live_admin/pull/159/


Example VPN config uses  MachineMethod Eap which other page says isn't valid value

The page here: https://docs.microsoft.com/en-us/windows/security/identity-protection/vpn/vpn-profile-options
Has the sample authentication:
<MachineMethod>Eap</MachineMethod>
However, on the document describing the valid parameters here: https://docs.microsoft.com/en-us/windows/client-management/mdm/vpnv2-csp?redirectedfrom=MSDN
It describes the MachineMethod parameter as only accepting the value Certificate:

VPNv2/ProfileName/NativeProfile/Authentication/MachineMethod
This is only supported in IKEv2.
This value can be one of the following:

Certificate

Value type is chr. Supported operations include Get, Add, Replace, and Delete.

Not sure I misunderstand or if this is inconsistent?
Thanks

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

ID: 79add4e0-6997-5b15-542c-be3f1c2b1baf
Version Independent ID: 47bd5f53-cd36-3205-42b1-6d99c68e9b39
Content: VPN profile options (Windows 10) - Microsoft 365 Security
Content Source: windows/security/identity-protection/vpn/vpn-profile-options.md
Product: w10
Technology: windows
GitHub Login: @DulceMontemayor
Microsoft Alias: dansimp

@corruptmem You are correct, the value for ..........  should be Certificate.
Keeping in mind the referenced article describes a Sample EAP profile, i think it should look like (i'll try to amend that):

<Authentication>
  <UserMethod>Eap</UserMethod>
  <Eap>
   <Configuration>

@corruptmem
With the help of your feedback, the content has been updated with relevant changes accordingly via #7167. Thanks.


Cannot access blog data to create custom templates.

I have the following config. There are two posts in the database but nothing shows when I try to create a custom template.
It works when i used the  {{> blogIndex}} template but I cant fetch the template context.
Configuration:
Blog.config({
    blogLayoutTemplate: 'blogLayout',
    blogIndexTemplate: 'myBlogIndex',
});

Template:
<template name="myBlogIndex">
    <div class="container">
        {{#each posts}}
            Title: {{title}}
        {{/posts}}
    </div>
</template>

Yeah, I think I am going to take this out.
So this was an earlier feature that was possible with Iron Router, since IR allows you to set the data context from the route. But Flow Router eschews this pattern (and I agree) and makes it very hard to do this (though still technically possible). I had to move all data logic to template-level subscriptions in order to support both routers.
I think I have to take this out, unless someone can think of a way to do this AND still support both routers. I agree it was a nice feature, and removing it greatly reduces the flexibility of meteor blog.

I am thinking the best way to provide blog post data is a mechanism similar to how Flow Router helps with subscriptions, i.e. a helper that helps you with blog post subscriptions. It would work something like this:
Template.onCreated ->
  Blog.subscribe()
  Tracker.autorun ->
    if Blog.ready()
      # do things that depend on blog posts

<template name="myBlogIndexTemplate">
  {{#each posts}}
    Title: {{title}}
  {{else}}
    Loading...
  {{/each}}
</template>

Or
<template name="myBlogIndexTemplate">
  {{#if blogReady}}
    {{#each posts}}
      Title: {{title}}
    {{/each}}
  {{else}}
    Loading...
  {{/if}}
</template>

WDYT?

Actually, I think I have a way to do this...


How to pass a params from ajax query?

I want to display different result list via def choices_for_request.
But how to define params from HTML?

Did you read that one ? http://django-autocomplete-light.readthedocs.org/en/master/dependant.html

The above reply should answer your question, if not, please reopen.


Refactor modules

This PR implements the suggested changes from PR https://github.com/Geek-1001/Clue/pull/20.

Great!
Thank you @andrea-prearo !
I'm merging this PR.


0.11.2 Internal version not updated

Describe the bug
Hi, Homebrew maintainer here 👋🏻
When building version 0.11.2 our regression test is failing because the internal version is still returning 0.11.1
Command: kitex --version
Error: kitex:
failed ::error::kitex: failed An exception occurred within a child process: Minitest::Assertion:
Expected /v0\.11\.2/ to match "v0.11.1\n".

https://github.com/Homebrew/homebrew-core/pull/185999

Sorry, today there was a tag that didn't meet expectations, and we create a new tag v0.11.2 but didn't update version info, which caused this mismatch issue.
We will create a new tag to fix the mismatch issue

v0.11.3 has been done


Nuget restore on agent without connection to internet

Environment

Server - TFS 2017.3
Agent - Private Win 2012 R2 agent version 2.122.1
Task - Nuget restore 1.0.1

Issue Description
I run Nuget restore on agent computer without internet connection (corporate security decision). Task want to download nuget.exe from dist.nuget.org and fail. I found that before download task try to find nuget.exe in some caches. Than I copy several versions to agent _work_tool\Nuget folder (3.3.0, 3.5.0, 4.0.0, 4.1.0, 4.4.1) but no one match. In documentation I found that Nuget restore use version 4.1.0 if I dont say to use diferent version in task Nuget Tools Instaler. But if I use task Nuget Tools Instaler with version 4.4.1 or 4.0.0 it fail nearly same log output. User used for agent run has perrmissions to read and execute files in _work folder
Is it possible to use this task without net connection and If yes what I do wrong?
Thank you
Error logs
2017-11-23T16:10:07.8393018Z ##[debug]packagesDirectorypath supplied :false
2017-11-23T16:10:07.8393018Z ##[debug]Getting NuGet
2017-11-23T16:10:07.8393018Z ##[debug]isExplicit: 4.0.0
2017-11-23T16:10:07.8393018Z ##[debug]explicit? true
2017-11-23T16:10:07.8393018Z ##[debug]Exact match expected on version: 4.0.0
2017-11-23T16:10:07.8393018Z ##[debug]Trying to get tool from local cache
2017-11-23T16:10:07.8393018Z ##[debug]isExplicit: 4.0.0
2017-11-23T16:10:07.8393018Z ##[debug]explicit? true
2017-11-23T16:10:07.8393018Z ##[debug]Agent.Version=2.122.1
2017-11-23T16:10:07.8393018Z ##[debug]Agent.ToolsDirectory=D:\Agents\Agent1_work_tool
2017-11-23T16:10:07.8393018Z ##[debug]checking cache: D:\Agents\Agent1_work_tool\NuGet\4.0.0\x64
2017-11-23T16:10:07.8393018Z ##[debug]not found
2017-11-23T16:10:07.8393018Z ##[debug]Agent.Version=2.122.1
2017-11-23T16:10:07.8393018Z ##[debug]Agent.ToolsDirectory=D:\Agents\Agent1_work_tool
2017-11-23T16:10:07.8393018Z ##[debug]isExplicit: 3.3.0
2017-11-23T16:10:07.8393018Z ##[debug]explicit? true
2017-11-23T16:10:07.8393018Z ##[debug]isExplicit: 3.5.0
2017-11-23T16:10:07.8393018Z ##[debug]explicit? true
2017-11-23T16:10:07.8393018Z ##[debug]isExplicit: 4.0.0
2017-11-23T16:10:07.8393018Z ##[debug]explicit? true
2017-11-23T16:10:07.8393018Z ##[debug]isExplicit: 4.1.0
2017-11-23T16:10:07.8393018Z ##[debug]explicit? true
2017-11-23T16:10:07.8393018Z ##[debug]isExplicit: 4.4.1
2017-11-23T16:10:07.8393018Z ##[debug]explicit? true
2017-11-23T16:10:07.8393018Z ##[debug]evaluating 0 versions
2017-11-23T16:10:07.8393018Z ##[debug]match not found
2017-11-23T16:10:07.8393018Z ##[debug]Querying versions list
2017-11-23T16:10:07.8393018Z ##[debug]task result: Failed
2017-11-23T16:10:07.8393018Z ##[error]getaddrinfo EAI_AGAIN dist.nuget.org:443
2017-11-23T16:10:07.8393018Z ##[debug]Processed: ##vso[task.issue type=error;]getaddrinfo EAI_AGAIN dist.nuget.org:443
2017-11-23T16:10:07.8549264Z ##[debug]Processed: ##vso[task.complete result=Failed;]getaddrinfo EAI_AGAIN dist.nuget.org:443

Where do you want to get the package from?  Your on prem server or the internet?  If your feed is the on prem server on the local network it should work.  Obviously, getting a package from an internet feed won't work without an internet connection.

Im on premise and we have on local network sonatype Nexus which act as proxy for nuget.org. Nexus has connection to internet. We are using version 0.* of this task several years and everything works. I have problem with version 1.*

@McMlok,
If you are using v1 of the NuGet task then create and set a variable called NuGetExeToolPath the path to your nuget.exe (e.g. "C:\Users\you\Downloads\nuget.exe").

Keith

@keithrob thanks it works.
Yes. We have defined several internal feeds for our nugets and one feed for nuget.org which is proxy + cache. And all of this feeds are joined in one meta feed. Than we use this feed in VS and on global configs on build agent servers.

@McMlok,
2 quick points.  I asked about the proxy because TFS2017U3 has a NuGet Tool Installer which will respect your corporate proxy if one is configured.  IOW, it will fetch the version of nuget you want through your corporate proxy.  Place the tool installer ahead of a NuGet Command build step so that your NuGet command step will use the version you want.
As for your "meta feed", you should know that we just released the concept of "nuget upstreams" for hosted VSTS.  It will land in TFS2018 soon.  A feed with nuget.org as an "upstream" allows you to have only one source in your nuget.config (i.e. your feed).  The TFS server will fetch packages from nuget.org on the client's behalf when they are not found in your feed and save them permanently in your feed just as if you had manually pushed the package.
Finally, we are working on intra-account feed upstreaming (i.e. Feed A upstreams Feed B).  This would allow you to have a single source in your config file and eliminate the need to manually push/duplicate packages in various feeds.
Keith


Highlight parent nodes

Parents of the active step will be highlighted if the active node is hidden.

fixes #230


Group OPT_ARGUMENTS into mods

Group OPT_SOME_CLI_OPTION into options::SOME_CLI_OPTION
This pull request does not intend to end all re-ordering of options, but rather establish a norm to follow in all tools/commands.
I'm grouping several related options into sub-mods. They can be improved by pointing them out here, in this PR, or later in the future

wahou, I love it :)
bravo
Let me know if you want me to merge now or not ?

@sylvestre let me do some more files… will @ you when it's ready

oh, I missed the draft, sorry :)


Add more tests

Codecov Report

Merging #27 into master will increase coverage by 4.64%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #27      +/-   ##
==========================================
+ Coverage   55.46%   60.11%   +4.64%
==========================================
  Files          10       10
  Lines         357      361       +4
==========================================
+ Hits          198      217      +19
+ Misses        159      144      -15

Impacted Files
Coverage Δ

src/utils.py
73.68% <100%> (+3.09%)
:arrow_up:

src/combiner/combiner.py
12% <0%> (+12%)
:arrow_up:

Continue to review full report at Codecov.

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


Error after installing page-blocks

When i try to navigate to Pages after installing page-blocks, i get the following error:

It looks like it really is missing the route. When i add the following to the /vendor/pvtl/voyager-page-blocks/routes/web.php file at line 14, it works:
Route::post('edit', ['uses' => "PageBlockController@edit", 'as' => 'edit']);

Hey @adrienne, this can be caused if BREAD isn't created in Voyager, running php artisan voyager-page-blocks:install after installing via composer should prevent this from happening. But let us know if this wasn't the case.


Revert transmission to "acquisition-based" from "modulo"

Description
Reverting transmission. Performance looks VERY BAD in the benchmark, 10x more time, but a practical application in TBsim changed from 107s in main to 110s with this branch. Not sure what's going on with the benchmark (or TBsim), but otherwise happy with the changes. Because this part of the model is heavily used, we might consider number or even cython for further performance optimization.
Checklist

[ ] Code commented & docstrings added
[ ] New tests were needed and have been added
[ ] A new version number was needed & changelog has been updated
[ ] A new PyPI version needs to be released

I'm curious how much performance difference there really is in practice. On TBsim, the difference between modulo and acquisition-based (roulette) is negligible. But benchmark.json shows an order of magnitude difference. Did you get to the bottom of this difference with any of your follow-up testing?

Superseded by #546


ChainedHashTable test sometimes fail

Describe the bug
During development some other feature, and run frequently "dotnet test" i notice that some test connected to ChainedHashTable sometimes pass sometimes not. Problem also occur on clear master branch.
To Reproduce
Run dotnet test command many times, that test doesn't crash every time.
Expected behavior
Test should always pass.
Additional context
X UnitTest.DataStructuresTests.HashTableSeparateChainingTest.DoTest [61ms]
Error Message:
System.NullReferenceException : Object reference not set to an instance of an object.
Stack Trace:
at DataStructures.Dictionaries.ChainedHashTable2.CopyTo(KeyValuePair2[] array, Int32 arrayIndex) in D:\repositories\public\C-Sharp-Algorithms\DataStructures\Dictionaries\ChainedHashTable.cs:line 609
at UnitTest.DataStructuresTests.HashTableSeparateChainingTest.DoTest() in D:\repositories\public\C-Sharp-Algorithms\UnitTest\DataStructuresTests\HashTableSeparateChainingTest.cs:line 107

I create pull request for fixing this bug. And i added description why this bug occured to pull request.


LogEvent autosave fix

LogEvent method was putting the autosave flag in error instead, therefore calling the logging method with autosave on true never actually saved the log. Now calling logEvent from logInfo & logDebug with 'null' for 'error' parameter.

Thank you! 🙌


Change to Menu Broke View Menu Items

Toolbar buttons still work for changing view, but the recent change to Professional Menu broke the View Menu Items.

I was wrong. It was ampersands.


mobile: Fix page height

https://developers.google.com/web/updates/2016/12/url-bar-resizing
https://www.456bereastreet.com/archive/201306/height_in_percent_when_parent_has_min-height_and_no_height/

Before
After

probably closes #415


fix: proper form data decoding

Checklist

[x] Tests have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce?
Bugfix
What is the current behavior? What is the new behavior?
Currently, body encoded with form-data is improperly decoded. We are using qs package which is not capable of decoding all styles, especially those described in encoding > style.
The new behavior is that we support all encoding styles through our deserializers.
Does this PR introduce a breaking change?
No.

Now that I think about it, would it make sense to extract this logic into a different/external library? I am quite sure this could be used somewhere else in Stoplight @philsturgeon @marbemac

Now that I think about it, would it make sense to extract this logic into a different/external library? I am quite sure this could be used somewhere else in Stoplight @philsturgeon @marbemac

I think about two library candidates: request-validation, request-deserialize.

At least request-deserialization is a unique code. At the time of writing (1yr ago), there was no similar tool.

Now that I think about it, would it make sense to extract this logic into a different/external library? I am quite sure this could be used somewhere else in Stoplight @philsturgeon @marbemac

I think we could hold off for a while on doing this. We have talked a bit recently about how the validation logic in general needs to be refactored, about how middlewares could be useful, and about how it could become its own Lerna package.
I think all of these things could be done together in the future while we refactor things, and maybe it could become its own package outside of the repo. Either way we can kick that can down the road whilst we sort out Prism more IMO.

Thanks a lot for this, I hope it was not that painful to make it happen. I'm merging when tests will pass.

I learned lots while working on this + thank you for guiding while doing so!


Changed displaying modifiers

show more modifiers if available for method,
for example "!_ @" for final public static;
configuration between modifier and its sign was separated;

What do you think about this change made by the way another debugging?

Using the getModifiers method and bitwise operation is fantastic :+1:

@przepompownia :+1:


libx52: Saitek X52 Joystick driver

Project description
With the rise of gaming on linux. It would be great to add this joystick driver.
I am not sure where to start from or how hard would it be to packaage this? Would it work something similar to this: Packaging out-of-tree kernel modules

Metadata

homepage URL: https://nirenjan.github.io/libx52/
source URL: https://github.com/nirenjan/libx52
license: gpl2
platforms: linux

I tried to get a mkDerivation working (it doesn't, and I'm very new to this). This is my work so far:
{
  lib,
  stdenv,
  fetchFromGitHub,
  automake,
  autoconf,
  gettext,
  hidapi,
  libtool,
  libusb1,
  libevdev,
  pkg-config,
  python3,
  git,
}:
stdenv.mkDerivation rec {
  pname = "libx52";
  version = "0.3.2"; # You can set the appropriate version here

  src = fetchFromGitHub {
    owner = "nirenjan";
    repo = "libx52";
    rev = "v${version}";
    sha256 = "0lmf73cnv6gn86s9xk6brlnsf5zvjsyks8hkdjynlvqx8mjzfbf5"; # Update with the correct sha256
  };

  nativeBuildInputs = [automake autoconf libtool pkg-config python3 gettext git];
  buildInputs = [libusb1 hidapi libevdev];

  configurePhase = ''
    ./autogen.sh
    ./configure --prefix=$out --localstatedir=/var --sysconfdir=/etc
  '';

  buildPhase = ''
    make
  '';

  installPhase = ''
    make install
  '';

  meta = with lib; {
    description = "Library for interfacing with the Saitek X52 Flight Control System";
    homepage = "https://github.com/nirenjan/libx52";
    license = pkgs.lib.licenses.gpl3;
    maintainers = with pkgs.lib.maintainers; [maintainers.yourself];
  };
}

I'm getting
error: function 'anonymous lambda' called with unexpected argument 'system'
I tried a nix-shell
{ pkgs ? import <nixpkgs> {} }:

pkgs.mkShell {
  buildInputs = [
    pkgs.automake
    pkgs.autoconf
    pkgs.gettext
    pkgs.hidapi
    pkgs.libtool
    pkgs.libusb1
    pkgs.libevdev
    pkgs.pkg-config
    pkgs.python3
    pkgs.git
  ];

  shellHook = ''
    git clone https://github.com/nirenjan/libx52.git
    cd libx52

    install_dir=$(mktemp -d)
    export PATH=$install_dir/bin:$PATH
    export LD_LIBRARY_PATH=$install_dir/lib:$LD_LIBRARY_PATH
    epxort PKG_CONFIG_PATH=$install_dir/lib/pkgconfig:$PKG_CONFIG_PATH

    ./autogen.sh
    ./configure --prefix=$install_dir --localstatedir=/var --sysconfdir=/etc
    make
    make install

    echo "Installation completed. The binaries and libraries are installed in $install_dir"
  '';
}

And it goes well until this:
 /nix/store/i7qhgc0bs725qw3wdanznfkdna4z2ns2-coreutils-9.5/bin/mkdir -p '/nix/store/ncpzg5dklh1qvhj8lrdwx36hwlxl0ry1-systemd-minimal-libs-255.6/lib/systemd/system'
/nix/store/i7qhgc0bs725qw3wdanznfkdna4z2ns2-coreutils-9.5/bin/mkdir: cannot create directory ‘/nix/store/ncpzg5dklh1qvhj8lrdwx36hwlxl0ry1-systemd-minimal-libs-255.6/lib/systemd’: Read-only file system
make[3]: *** [Makefile:2878: install-systemdsystemunitDATA] Error 1
make[3]: Leaving directory '/home/zombiefleischer/libx52/libx52'
make[2]: *** [Makefile:3495: install-am] Error 2
make[2]: Leaving directory '/home/zombiefleischer/libx52/libx52'
make[1]: *** [Makefile:2968: install-recursive] Error 1
make[1]: Leaving directory '/home/zombiefleischer/libx52/libx52'
make: *** [Makefile:3488: install] Error 2

So I'm a bit out of my breath. Some help would be very appreciated.


[stable/external-dns] Upgrade to v0.5.8

Signed-off-by: Sébastien Prud'homme<EMAIL_ADDRESS>What this PR does / why we need it:
Use new version of external-dns, which contains especially a new RFC2136 provider.
Special notes for your reviewer:
No breaking changes in this new version.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

Lint failed because chart doesn't have maintainers, i don't know what to do about that as it is not the fault of this PR.

/assign @mgoodness


openapi-python-client: init at 0.21.4

Description of changes
A python application to generate modern Python clients from OpenAPI 3.0 and 3.1 documents.
Homepage: https://github.com/openapi-generators/openapi-python-client
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4516

nixpkgs-review result
Generated using nixpkgs-review.

Command: nixpkgs-review pr 337741
Systems: x86_64-linux,aarch64-linux,x86_64-darwin,aarch64-darwin

x86_64-linux

:white_check_mark: 2 packages built:

openapi-python-client
openapi-python-client.dist

aarch64-linux

:white_check_mark: 2 packages built:

openapi-python-client
openapi-python-client.dist

x86_64-darwin

:white_check_mark: 2 packages built:

openapi-python-client
openapi-python-client.dist

aarch64-darwin

:white_check_mark: 2 packages built:

openapi-python-client
openapi-python-client.dist

offtopic: @GaetanLepage mind sharing how did you generate such a nice nixpkgs-review report for all architectures? It's extremely nice

offtopic: @GaetanLepage mind sharing how did you generate such a nice nixpkgs-review report for all architectures? It's extremely nice

This is an addition currently under review: https://github.com/Mic92/nixpkgs-review/pull/419


reset password not working

Expected Behavior
To be able to reset my password and log in
Actual Behavior
I can't reset my password - i get an error "all device users must agree to tos"
but there's no where i can aggree to this TOS, and i can't log in to do so.....

Steps to Reproduce
Reset password
follow link from email
type password twice
press "reset"
Results of git log -1
Run git log -1 and paste the results here.

Thanks for the heads up @Br3nda. Investigating now.

@Br3nda I think I see what's happening. Is this happening with the account listed on your Github profile, or a different email? If it's a different email, please send the email address  to my_first_name at farmbot.io.
Also, how old is this account? Was it created more than a year ago?

It's more than a year old, yes. I managed yo guess my old password, and accept the terms.
I think the password reset form should let you continue, and only on login ask the user to accept the terms.