Handle symlinks to directories

When cargo encounters a link, check if it's a directory when considering it for further processing.
I'm not sure how this interacts with things such as submodules – it allowed me to publish the crate pqcrypto-kyber768 which uses a link to a submodule.
Fixes #2748.
This PR:

Add new tests that demonstrate that links to dirs can't be packaged
Fixes those tests
Enabled symlink tests to run on Windows
Fix a bug in the handling of symlinks
Add new test runner behaviour on Windows (it will ignore symlink-related tests and instead run them if you specify --ignored.)

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

Thanks for the PR! I sort of forget the context around this code, but do you remember why COMMIT means directory? Additionally would it be possible to add a test for this as well?
I wonder if we could possibly query git instead of the filesystem to attempt to learn if a symlink points to a directory?

I have no clues how any of those things work.

COMMIT is a git submodule.  The is_dir code below will then walk the submodule (or if it is not a submodule, just walk the filesystem).
Querying git looks like it would be more trouble than it's worth.  It would need to read the link's contents, do an Index::get_path, and check the result of that, and still handle the case where get_path returns another link or returns None (and it would wind up needed to get the metadata anyways).  It would be a nontrivial amount of code for an edge case for an optimization.
I would suggest maybe something like this would be simpler.
diff --git a/src/cargo/sources/path.rs b/src/cargo/sources/path.rs
index fd2e4bb5..d35e75ca 100644
--- a/src/cargo/sources/path.rs
+++ b/src/cargo/sources/path.rs
@@ -323,9 +323,17 @@ impl<'cfg> PathSource<'cfg> {
         // the untracked files are often part of a build and may become relevant
         // as part of a future commit.
         let index_files = index.iter().map(|entry| {
-            use libgit2_sys::GIT_FILEMODE_COMMIT;
-            let is_dir = entry.mode == GIT_FILEMODE_COMMIT as u32;
-            (join(root, &entry.path), Some(is_dir))
+            use libgit2_sys::{GIT_FILEMODE_COMMIT, GIT_FILEMODE_LINK};
+            // `is_dir` is an optimization to avoid calling `fs::metadata` on
+            // every file.
+            let is_dir = if entry.mode == GIT_FILEMODE_LINK as u32 {
+                // Let the code below figure out if this symbolic link points
+                // to a directory or not.
+                None
+            } else {
+                Some(entry.mode == GIT_FILEMODE_COMMIT as u32)
+            };
+            (join(root, &entry.path), is_dir)
         });
         let mut opts = git2::StatusOptions::new();
         opts.include_untracked(true);

@thomwiggers The tests are not generated.  There is an introduction to writing tests at testsuite/support/mod.rs. There are a variety of git-helpers in support/git.rs.  I would stick the test in testsuite/package.rs.  There is a similar test in there (package_git_submodule) which you can use for inspiration, though just creating a symlink should be simpler.

What is the desired behaviour for these symlinks anyway?
Say we have the folowing project layout:
target/cit/t0/
├── foo
│   ├── Cargo.lock
│   ├── Cargo.toml
│   ├── src
│   │   └── lib.rs
│   ├── submodule
│   │   └── Makefile
│   ├── submodule-link -> /home/thom/git/cargo/target/cit/t0/foo/submodule
│   └── target
│       └── package
│           └── foo-0.0.1.crate
├── home
└── submodule
    └── Makefile

8 directories, 6 files

Do we then want to handle the symlink like a copy of the submodule folder (how it's currently handled) or do we want Cargo to properly package a symlink.
What about the following scenario?
/home/thom/git/phd/tls/pqcrypto/
├── pqclean
│   └── Makefile, etc
├── pqcrypto-kyber
│   ├── pqclean -> ../pqclean
│   └── src

There just archiving as a symlink would probably not be sufficient, and you do want cargo to copy the files in .../pqclean over to pqcrypto-kyber.crate.

There appears to be a compile issue in unrelated code that I can't reproduce locally.
error: cannot borrow `*p` as mutable because it is also borrowed as immutable
   --> tests\testsuite\support\mod.rs:794:17
    |
793 |             if let Some(cwd) = p.get_cwd() {
    |                                - immutable borrow occurs here
794 |                 p.cwd(cwd.join(path.as_ref()));
    |                 ^     --- immutable borrow later used here
    |                 |
    |                 mutable borrow occurs here
    |
note: lint level defined here
   --> tests\testsuite\main.rs:2:45
    |
2   | #![cfg_attr(feature = "deny-warnings", deny(warnings))]
    |                                             ^^^^^^^^
    = note: #[deny(mutable_borrow_reservation_conflict)] implied by #[deny(warnings)]
    = warning: this borrowing pattern was not meant to be accepted, and may become a hard error in the future
    = note: for more information, see issue #59159 <https://github.com/rust-lang/rust/issues/59159>
error: aborting due to previous error
error: Could not compile `cargo`.
warning: build failed, waiting for other jobs to finish...
error: build failed

Making it a strict error would seriously complicate maintenance of packages such as the one I'm building here: https://github.com/rustpq/pqcrypto. This project will at some point have:

pqcrypto-kyber512
pqcrypto-kyber768
pqcrypto-kyber1024
pqcrypto-frodokem-....
...

This project will be importing a whole bunch of different C implementations of cryptographic libraries and generating Rust bindings to use them. The neat place to put the shared C code would be in the parent directory; having to mount the submodule or copy and commit the files in each separate subpackage would be a huge pain.

I think if we want to support this we'll want to avoid symlinks in archives and just copy all the contents in, and if that works for your use case it seems like a reasonable way to implement this perhaps

avoid symlinks in archives and just copy all the contents in

I believe that is how it works now (or with this patch when using git).

avoid symlinks in archives and just copy all the contents in

I believe that is how it works now (or with this patch when using git).

Correct. In very weird edge cases where there are a lot of symlinks to the same files this might inflate packages, but on the other hand the compression algorithm should take care of that I guess.

Rebased on master to re-trigger tests.

Error on Windows:
failures:
---- package::package_symlink_to_submodule stdout ----
running `C:\projects\cargo\target\debug\cargo.exe package --no-verify -v`
thread 'package::package_symlink_to_submodule' panicked at '
Expected: execs
    but: exited with exit code: 101
--- stdout
--- stderr
warning: manifest has no description, license, license-file, documentation, homepage or repository.
See https://doc.rust-lang.org/cargo/reference/manifest.html#package-metadata for more info.
error: The directory name is invalid. (os error 267)
', tests\testsuite\support\mod.rs:842:13
note: Run with `RUST_BACKTRACE=1` environment variable to display a backtrace.
failures:
    package::package_symlink_to_submodule
test result: FAILED. 1558 passed; 1 failed; 4 ignored; 0 measured; 0 filtered out

How did this not occur on Travis?

I needed to put in some extra effort to fix some things in how the test supports treat symlinks, but I think this should be reasonable now.

:umbrella: The latest upstream changes (presumably #6900) made this pull request unmergeable. Please resolve the merge conflicts.

I'm getting privilege errors for the below tests on Windows 10.  I think it would be good to avoid that if the system isn't configured to handle symlinks on Windows.
    package::broken_symlink
    package::package_symlink_to_dir
    package::package_symlink_to_submodule

Perhaps add some kind of check if symlinks are supported on Windows, and skip the test if they are not?
Also, can you squash all the commits?

Perhaps add some kind of check if symlinks are supported on Windows, and skip the test if they are not?

Unfortunately you need admin privileges on Windows to create them in the first place. It's not possible to really skip tests in Rust... so I can't really do detection (marking them 'passed' seems inaccurate).
I could perhaps mark them as #[ignored] on Windows.

I have rebased and cleaned up the commit history. I did not fully squash it, as I wanted the history to reflect the different things this PR addresses:

Add new tests that demonstrate that links to dirs can't be packaged
Fixes those tests
Enabled symlink tests to run on Windows
Fix a bug in the handling of symlinks
Add new test runner behaviour on Windows (it will ignore symlink-related tests and instead run them if you specify --ignored.)

Appveyor appears to fail for completely unrelated reasons.

:umbrella: The latest upstream changes (presumably #7139) made this pull request unmergeable. Please resolve the merge conflicts.

:umbrella: The latest upstream changes (presumably #7137) made this pull request unmergeable. Please resolve the merge conflicts.

Rebased, removed running --ignored tests, ran cargo fmt

It appears rustfmt nightly and stable disagree...

Yea, I was trying to figure out how to deal with the rustfmt changes, but I think I'm just gonna leave it until 1.38 hits stable. It is annoying.
Thanks for seeing this through!
@bors r+

:pushpin: Commit d0f7c0ee31c81cd405dc2c163e4ca7638895610c has been approved by ehuss

:hourglass: Testing commit d0f7c0ee31c81cd405dc2c163e4ca7638895610c with merge 023795336d605511af9a5b35559b380081044668...

:sunny: Test successful - checks-azure
Approved by: ehuss
Pushing 023795336d605511af9a5b35559b380081044668 to master...


Add time to tracewriter timeout test to reduce flake

@ofrobots @cristiancavalli PTAL.

@matthewloring This LGTM and the test passes on my machine with the proposed change. Not sure if the travis-ci is flaking though

The failure is pending the fix in #273. It is unrelated to this change.

LGTM given test dependencies


Revert "feature: move vim from active to base"

Reverts pyqr-dev/pyqr#71

this should actually be done in the wsl image


Getting SignerValidateException

Hello 👋 ,
when I run the script I get this exception:
php /home/ubuntu/oci-arm-host-capacity/index.php
PHP Fatal error:  Uncaught Hitrov\OCI\Exception\SignerValidateException: URL is invalid: https://iaas..oraclecloud.com/20160918/instances/?availabilityDomain=&compartmentId= in /home/ubuntu/oci-arm-host-capacity/vendor/hitrov/oci-api-php-request-sign/src/Hitrov/OCI/Signer.php:329
Stack trace:
#0 /home/ubuntu/oci-arm-host-capacity/vendor/hitrov/oci-api-php-request-sign/src/Hitrov/OCI/Signer.php(67): Hitrov\OCI\Signer->validateParameters()
#1 /home/ubuntu/oci-arm-host-capacity/src/OciApi.php(111): Hitrov\OCI\Signer->getHeaders()
#2 /home/ubuntu/oci-arm-host-capacity/index.php(34): Hitrov\OciApi->getInstances()
#3 {main}
  thrown in /home/ubuntu/oci-arm-host-capacity/vendor/hitrov/oci-api-php-request-sign/src/Hitrov/OCI/Signer.php on line 329

I guess that I made some mistake in declaring the key file path (aka OCI_PRIVATE_KEY_FILENAME), but I can't figure out what exactly I did wrong.
(Also sorry for posting this issue initially empty, I clicked publish before being finished by accident)

hi,
I've just cloned the repo in temp dir, run composer install.
adjusted values in constructor (only there).
and set SSH key with env var before running the script.
it's working.
I see that for some reason you have URL with two dots, why's that?

hi :),
Thanks for the quick answer.
I guess I found out why it didn't work:
I was following the README and was confused by the new introduced env vars for the test (intruduced here).
After I filled out the data like only with strings (so without the getenv(...) ??) I get now:
Already have an instance(s) [] in state(s) (respectively) []. User: 'ocid1.user.oc1..aaaaaaaa***'

Okay. After removing the getenv(...) ?? everywhere (and filling in the data like discribed in the README I get now:
{
    "code": "InternalError",
    "message": "Out of host capacity."
}

That sound's already better 👍

btw I fixed one issue because of your attention, so thanks as well=)
https://github.com/hitrov/oci-arm-host-capacity/commit/5c7cb7a371bb8fa0066168558f45831fa8ce4900
it was related to edge case when you have falsy value as env var value of max instances.

@dpschen I see that it also caused an issue with $shape var, because now I don't have empty names for instances which I already have :)

Thanks for the help :)
Seems to be running now. Let's hope I get lucky 🎉

It worked! :) Thanks so much!!


[TextField] Underlined inputs border breaks when transitioning of it in Safari

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯

The outside borders remain when transitioning in Safari.
Expected behavior 🤔

No outside borders should remain when transitioning.
Steps to reproduce 🕹
You can reproduce it on official docs site using Safari
https://mui.com/components/text-fields/#sizes
Steps:

Focus on underlined TextField
Click outside of underlined TextField

Context 🔦
No response
Your environment 🌎

Browser : Safari 15.3(176<IP_ADDRESS>.8)
OS : macOS Monterey 12.2.1(21D62)

I can reproduce but can't figure out what's causing this issue.

It seems to be Safari transform issue.
So I'm created pull request with solution. But I don't know my pull request is the best way.
Please check my pull request

Nice work!. However, your PR did not work with my issue, in iOS, it is still broken @KimBiYam

Nice work!. However, your PR did not work with my issue, in iOS, it is still broken @KimBiYam

In my case, adding the will-change property solved the problem in iOS Safari likewise macOS Safari.
Before
https://user-images.githubusercontent.com/59679962/162203087-d2be7787-d250-4b22-ab5d-914e161f8d2b.MP4
After adding will-change attribute
https://user-images.githubusercontent.com/59679962/162203294-9a4c6c5a-590e-44c8-8ce3-ddb4b6285e3d.MP4

@siriwatknp I'm already submitted a PR
https://github.com/mui/material-ui/pull/31912

@siriwatknp,
I came up with an alternative workaround that doesn't require the will-change:
[`&.${inputClasses.focused}:after`]: {
  transform: 'translateZ(0) scaleX(1)',
},

I think the additional no-op transform forces Safari to re-render the layer, which also eliminates the visual glitch in question.
This looks like a much safer bet to me. Happy to submit a PR if you prefer this approach over the will-change.

@siriwatknp,
I came up with an alternative workaround that doesn't require the will-change:
[`&.${inputClasses.focused}:after`]: {
  transform: 'translateX(0) scaleX(1)',
},

I think the additional no-op transform (translateX(0)) forces Safari to re-render the underline layer, which also eliminates the visual glitch in question.
This looks like a much safer bet to me. Happy to submit a PR if you prefer this approach over the will-change.

Definitely better than will-change if it fixes the glitch. Feel free to submit a PR.

@siriwatknp,
I came up with an alternative workaround that doesn't require the will-change:
[`&.${inputClasses.focused}:after`]: {
  transform: 'translateX(0) scaleX(1)',
},

I think the additional no-op transform (translateX(0)) forces Safari to re-render the underline layer, which also eliminates the visual glitch in question.
This looks like a much safer bet to me. Happy to submit a PR if you prefer this approach over the will-change.

I'm agree too if it fixes

The PR #32188 is ready
Preview: https://deploy-preview-32188--material-ui.netlify.app/components/text-fields/#main-content
Before:
https://user-images.githubusercontent.com/64441155/162430442-2523b3ba-e6d0-4796-ab97-20e56778bda2.mov
After:
https://user-images.githubusercontent.com/64441155/162430480-20d92787-9654-4066-aa48-435a5f041841.mov


Minor docs enhancements

This PR supercedes #85.

Thank you for all this! Are you happy for me to merge this and #92? I think you would normally be able to but because it's not run the tests you can't right?

Thank you for all this! Are you happy for me to merge this and #92? I think you would normally be able to but because it's not run the tests you can't right?

Sure. Merge!
Why didn’t the tests run though?

Thank you for all this! Are you happy for me to merge this and #92? I think you would normally be able to but because it's not run the tests you can't right?

Sure. Merge! Why didn’t the tests run though?

They're set to only run when src or test change. Its from when the repo was private so we had limited action minutes but I guess I may as well change it to run for all changes now


[android][AsyncStorage]crashed on HUAWEI MT7-CL00 , android 4.4.2

log:
E/logserver( 2518): handle_notify_event, send msg [submit:trigger=0,bugtype=2,modulename=com.constructionquality,level=1,testtype=NORMAL,path=/data/log/unzip/MT7-CL00_V100R001C92B135_G2W0215904000624_20160527095114_crash;]
E/SubmitMessage( 3373): Submit string: submit:trigger=0,bugtype=2,modulename=com.constructionquality,level=1,testtype=NORMAL,path=/data/log/unzip/MT7-CL00_V100R001C92B135_G2W0215904000624_20160527095114_crash
W/ContextImpl(  949): Failed to ensure directory: /storage/sdcard1/Android/data/com.constructionquality/cache
i'm using react-native:0.26.1,
<uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE"/>  has been added.
My app crashed when i click login, it maybe caused by AsyncStorage.setItem() in my login logic to remember user's account.

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out its new home: https://productpains.com/post/react-native/androidasyncstoragecrashed-on-huawei-mt7-cl00-android-442
Product Pains has been very useful in highlighting the top bugs and feature requests:
https://productpains.com/product/react-native?tab=top
Also, if this issue is a bug, please consider sending a pull request with a fix.


Update opensearch version to 1.3.6.

Signed-off-by: sreepuramsudheer<EMAIL_ADDRESS>Description
Update Opensearch version to 1.3.6 in chef-server softwares/Omnibus softwares
[Please describe what this change achieves]
Issues Resolved
[List any existing issues this PR resolves, or any Discourse or
StackOverflow discussions that are relevant]
Check List

[ ] New functionality includes tests
[ ] All buildkite tests pass
[ ] Full omnibus build and tests in buildkite pass
[ ] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/main/CONTRIBUTING.md#developer-certification-of-origin-dco
[ ] PR title is a worthy inclusion in the CHANGELOG

Adhoc build: https://buildkite.com/chef/chef-chef-server-main-omnibus-adhoc/builds/5050

https://buildkite.com/chef/chef-umbrella-main-chef-server/builds/1601


Make BankID session time configurable

We should have a Duration telling how long a BankID session is allowed to be.

And also display a "Retry"-button when we timeout. This will start a new BankID auth/sign.


Add the ability to pause message processing.

Thi smight be helpful when you want to analyse bugs in a high throughput/high load environment.

This could either be done by shutting down the AMQPX object or by some amqp/RabbitMQ specific mechanism that prevents new messages from being received by our application. Filling up the corresponding queue until we start processing the messages again.

https://goplay.tools/snippet/Sjhm_DrWG60


Add Shell Interface

Once version 2.0 of https://github.com/daniele77/cli gets pushed on conan, consider adding a shell interface much like the Crash shell was added on the lower half of https://maxdemarzi.com/2017/01/18/our-own-multi-model-database-part-5/

Nah, let's keep the interface http only.


icedtea7-native_2.1.3.bb Compilation issue SEGFAULT - in OPENSUSUE

Any updates?

This is affecting any Linux distribution with gcc-11 or greater. This probably means the very ancient icedtea7-native version needs to be bumped to one which fixes whatever is causing the segfault.
There is no mythical pool of developers waiting to fix issues here. This is an open source community layer. If you are willing to dig into an update to a newer version of icedtea7-native, please send a pull request.

@tanginik is this possibly resolved by the suggestion in https://github.com/meta-java/meta-java/issues/18#issuecomment-1914620702 ?

@tanginik @buonan which version of OpenSUSE are you using? latest https://get.opensuse.org/leap/15.5/ ?

I tried the fix mentioned in https://github.com/meta-java/meta-java/issues/18#issuecomment-1914620702 and it has fixed my error.  I am using Yocto v3.3.6 with meta-raspberrypi (hardknott) and meta-java (dunfell-next).

My host is WSL2 Debian and my gcc is Debian (12.2.0-14) hope this helps!
Okay got it, thanks for the tips!

Hi @moto-timo'
I observe the same icedtea7-native_2.1.3.bb build failure in kirkstone.
kirkstone-next builds fine. Could you please merge kirkstone-next into kirkstone to fix the problem?

Everyone is ignoring that the specific point of this Issue was that it affected OpenSUSE (at a non-reported version). Closing as I have pushed to kirkstone.

https://github.com/meta-java/meta-java/discussions/25
OpenSUSE 15.3


Extremely Outdated Docs & Example Plugin

Howdy. I'm trying to use this library in my plugin, but the docs are extraordinarily outdated, and even the example plugin - DeluxeMediaPlugin - makes use of classes and methods that have been renamed or even removed entirely.
I appreciate the work you're doing, and I'm very impressed with how far it has come, but it's nearly impossible to use this without spending days or weeks trying to read through this big of a project and figure out how it all works, since the docs and example are so outdated.

Pulse (the dev) has been very busy these last few months, and has not been able to work on the project. He says he will be back when he has time, and I know the plugin has a future because I saw it when it was working and it was perfect. In the Discord server you have more information.

Closing as I am working on the project. Documentation comes after the project itself functioning (I am rewriting some parts of it and I really recommend people to not use it at the moment).


[Event Request] Table 7317 - "Warehouse Receipt Line" - new event "OnDeleteQtyToReceiveOnBeforeModify"

Dear ALAppExtensions Team:
We would like to request a new event publisher in table 7317 "Warehouse Receipt Line". This event is needed in order to modify values of fields of our solution.
NEW PUBLISHER
[IntegrationEvent(false, false)]
local procedure OnDeleteQtyToReceiveOnBeforeModify(var WhseRcptLn: Record "Warehouse Receipt Line")
begin
end;
The parameter WhseRcptLn is referenced as VAR.
The code in procedure "DeleteQtyToReceive" in table 7317 "Warehouse Receipt Line" would be modified as follows:
OLD CODE:
procedure DeleteQtyToReceive(var WhseReceiptLine: Record "Warehouse Receipt Line")
begin
    with WhseReceiptLine do begin
        if Find('-') then
            repeat
                Validate("Qty. to Receive", 0);
                Modify;
            until Next = 0;
    end;
end;

NEW CODE:
procedure DeleteQtyToReceive(var WhseReceiptLine: Record "Warehouse Receipt Line")
begin
    with WhseReceiptLine do begin
        if Find('-') then
            repeat
                Validate("Qty. to Receive", 0);

                //start new code
            OnDeleteQtyToReceiveOnBeforeModify(WhseReceiptLine);
                //stop new code
                Modify;
            until Next = 0;
    end;
end;

Thanks!

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


COOK-4553: Removed Apache configuration files are restored on system upgrade on RHEL

Migration of  https://tickets.opscode.com/browse/COOK-4553 by Andy Thompson
Since the apache2 cookbook deletes Apache configuration files on Redhat-based repos, when security updates are installed these files are restored. If the server is then restarted, the configuration files are then loaded.
This causes a big problem when using mod_ssl, as the ssl.conf configuration then makes the SSL port double listen on 443, causing the web server to fail to start.
This in addition to other configuration not expected to be re-added potentially causing issues as well.
I know that a Chef converge re-running will resolve this, but it would be great if security updates could be installed without putting the system in an unstable state.
I'd like the cookbook to truncate the files (perhaps putting in a comment why they are there), rather than remove them, making them harmless. A security update would then at most create .rpmnew versions of the files, which wont be included in Apache.
I could potentially do some of these changes, however I want to see how receptive the idea is before I do the work.

Tracked in #279


Build IML GUI when running the local provisioner

It will be helpful to build the iml-gui when provisioning with
install-iml-local
Signed-off-by: johnsonw<EMAIL_ADDRESS>

This change is

Might need to update imlteam/copr-wasm:latest to include nodejs & yarn

Bumping to try and get a passing Jenkins run


Desktop: Add support for file associations

Feature Request
The current desktop packaging tasks do not include options for setting up file associations, a particularly valuable feature for desktop apps.
Possible Workaround
The AbstractJPackageTask allows us to add custom args with the freeArgs property, which get passed-through directly to jpackage. So to add our file associations, we should be able to follow the documentation linked above and manually add the args we need.
// build.gradle.kts
plugins {
    kotlin("multiplatform")
    id("org.jetbrains.compose")
}
compose.desktop {
    application {
		...
    }
}

tasks.withType<AbstractJPackageTask>().all {
    // add custom args to jpackage to set up file associations
    freeArgs.add("--file-associations")
    freeArgs.add(project.projectDir.resolve("ext.properties"))
}

macOS Problems
The above workaround does not seem to work on macOS (I have not tried it on other OSs). macOS handles file associations differently than Windows or Linux, and has a particularly verbose "registration" process that involves adding a bunch of additional information to the package's Info.plist file. Normally, jpackage would add all this info to the Info.plist file itself in accordance with the --file-associations property, but the Compose for Desktop Gradle plugin is writing that file itself. This means that even if we add those flags manually using the freeArgs property, the application will still not be registered to open custom file types, because the Compose Gradle plugin is not adding the necessary properties to the plist it writes.
In addition, the CfD plugin adds freeArgs before it configures itself, and it specifies --resource-dir for its own use when writing the Info.plist, so we cannot provide our own resource dir to write the plist by hand ourselves either.
Solution
Rather than writing the Info.plist file within the CfD Gradle plugin, it should let jpackage do it.
Potential Issues
There may already be info CfD is manually adding to Info.plist that jpackage is not adding (which may have been the reason for manually writing it in the first place). Removing the step to manually write the plist may cause issues with some of the existing Gradle plugin properties, and deeper research into customizing jpackage usage or other workarounds may need to be added to fully support them.

@cjbrooks12 it has been implemented for MacOs in https://github.com/JetBrains/compose-jb/issues/795
Task for linux and windows is here - https://github.com/JetBrains/compose-jb/issues/855
Also there is another task about it - https://github.com/JetBrains/compose-jb/issues/380
Considering above do you need to keep this task open?

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.

@okushnikov The integration didn't import comments.


Plugin: Fix Gradle configuration cache by not keeping classloader as a nested task field

Fixes https://github.com/detekt/detekt/issues/3563
I tried to reproduce the issue in tests, however I'm unable to trigger the cannot serialize object of type 'java.net.URLClassLoader', a subtype of 'java.lang.ClassLoader' error there :/ I noticed tests are passing the dry-run flag which calls simpler DryRunInvoker, but even after adding a test case with the DefaultCliInvoker the test still doesn't fail.
I'd love to hear some ideas how to cover the linked scenario in tests 👀
I verified manually on a failing project changes I'm suggesting work 🤷

We would need to use DslGradleRunner which leverages the TestKit to pass in the additional --configuration-cache.
The change looks good to me though.

The test you mentioned is already there: https://github.com/detekt/detekt/blob/4a89a25c82d35874571270fa1ff32751f2f268a1/detekt-gradle-plugin/src/test/kotlin/io/gitlab/arturbosch/detekt/ConfigurationCacheTest.kt#L8
I expected that removing the dryRun() call there, it'd invoke DefaultCliInvoker which would reveal the serialization issue, but that doesn't happen. Basically, I can't make the test to fail with linked issue on current main branch (I just pushed an updated test which I expected to fail)

When inspecting the task output in the aforementioned test, I can even observe Stored cache entry for task ':detekt' with cache key beef5daf9ca25f7b5c981f92fb0b68d0, which makes me think if the configuration cache works for write. And test passes mean configuration cache works for read.
On second thought, we could use a real detekt task instead of dry run to verify the configuration cache behavior. We could create another issue for that.

Wouldn't the

Stored cache entry for task

be a log for regular Gradle Build Cache? For some reason detekt tests use build cache.
AFAIU configuration cache prints different log messages, like "Calculating task graph as no configuration cache is available for tasks" or "Configuration cache entry stored" or "Reusing configuration cache" (and actually that's what the tests makes assertions on)

we could use a real detekt task instead of dry run to verify the configuration cache behavior. We could create another issue for that.

That's right, the real detekt task, aka DefaultCliInvoker breaks Configuration Cache. That's what gradle report points at in the linked issue. And that's why I added a real detekt task call to the test.
I expected it to fail without the fix I applied, but it doesn't - and that's my challenge here :/


Adjust warning thresholds for each products

Now that the threshold for changing the background color of a cycle's date to yellow (to indicate that date is approaching) is configurable on a per-product / per-column, look at all the products and adjust the value (which is 121 days for all dates currently).
We should also documenting some rules to make things easier and "normalized".

From @BiNZGi () :

On the gitlab page we can shorten the threshold also because they are releasing new minor versions each month and the support is yellow right from the beginning.

gitlab done -> #3147

Thinking about this, we could code the 1/3 rule (e.g. warn when there is only 1/3 of the time left before end of support / eol...) in https://github.com/endoflife-date/endoflife.date/blob/master/_plugins/product-data-enricher.rb and calculate this date by release cycle. This way:

no need to manually set things,
the date will be always correct (no more warning when released).

For example:

for a release cycle with 3 weeks between release and eol, warn when there is only 1 week left,
for a release cycle with 3 months between release and eol, warn when there is only 1 month left,
for a release cycle with 3 years between release and eol, warn when there is only 1 year left (we could put a hard limit that cannot be exceeded, such as 121 days like today).

WDYT @BiNZGi ?

This is a really good idea. For the team it is simpler to maintain and the approach fits better for the various scenarios. Goos to go from my side :+1:


⚠ DDoS-Relay US-E-ash-1 has degraded performance

In 8968d3e, DDoS-Relay US-E-ash-1 (http://<IP_ADDRESS>:42069/;csv) experienced degraded performance:

HTTP code: 200
Response time: 13 ms

Resolved: DDoS-Relay US-E-ash-1 performance has improved in 1a4b014 after 5 minutes.


Fill out Thermodynamics section

Just needs to be done.  Any takers?  If I don't hear anything by Friday afternoon, I'll work on this while traveling.

Dear All,
Due to unforeseen administrative responsibilities and grant writing I could
not contribute to the paper. Please remove my name from the author list, if
you have not already done so.
Thank you for taking this forward and sorry for the inconveniences caused
by mine irregular response.
Thanks,
Avisek
On Thu, Jul 26, 2018 at 11:11 AM, Jacob Monroe<EMAIL_ADDRESS>wrote:

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/MobleyLab/basic_simulation_training/issues/35, or mute
the thread
https://github.com/notifications/unsubscribe-auth/Ad5Xu7Q8cZRNlaabvJ8vUGR40GsM2s2Zks5uKVZ4gaJpZM4VhNYP
.

@JIMonroe got a draft of this in already.


Unable to connect to HANA DB having Multi-tenant container DB Configuration

I am unable to connect to HANA DB using PyHDB driver. I have a Multi-tenant Container DB (MCD) configuration.
I can connect to HDB using the following command:
hdbsql -u SYSTEM -p  -d SystemDB
However, I am unable to pass the parameterr for database in pyhdb.connect.
Please advise.

I was able to resolve the issue. For Multi-tenant DB Container configuration, we need to use the following ports.
xx --> Instance Number
Port 3xx13 connects to the SYSTEMDB SQL port
Port 3xx41 connects to the 1st tenant database SQL port
Port 3xx44 connects to the 2nd tenant database SQL port


Broken by upstream changes by stable horde

Read the title. When git cloning the repo, there is no cli. So we need to fork their repo and use our own version.

Fixed


Granted did not prompt to configure alias on ubuntu 2024 with Bash

An issue with the first time setup led to granted not prompting to configure and alias.
The effect was that no environment variables were being expected to the shell.
Adding alias assume=". assume" manually to ~/.profile and restarting fixed the issue with env vars not being exported.

24.04LTS just to be certain.

Can confirm on another ubuntu 24.04, fresh install - same issue, no entry added in profile (assumego hadnt been activated), and despite manual entry in this file it didn't work oddly. had to add it to users' bashrc before it began to work correctly.
By didnt work I mean you assume a profile but the credentials aren't available to the session.

I can also confirm that I ran into this issue as well. I needed to manually add the alias for it to work properly.


Accessing allure-docker-service-ui using ip address in  different host create "Network error"

Starting docker UI container in different host and then try to access the UI with IP address in other host pc ,
throws below issue. Please let know which configuration needed to make UI also access into different host
"Network Error"
Check if you have access to http://localhost:5050/allure-docker-service
Or pull the latest "frankescobar/allure-docker-service" API image (remove your current latest version)
{"message":"Network Error","name":"Error","stack":"Error: Network Error\n at e.exports (http://<IP_ADDRESS>:5252/static/js/2.fc83d11e.chunk.js:2:70398)\n at p.onerror (http://<IP_ADDRESS>:5252/static/js/2.fc83d11e.chunk.js:2:69367)","config":{"url":"/config","method":"get","headers":{"Accept":"application/json, text/plain, /"},"baseURL":"http://localhost:5050/allure-docker-service","transformRequest":[null],"transformResponse":[null],"timeout":0,"xsrfCookieName":"XSRF-TOKEN","xsrfHeaderName":"X-XSRF-TOKEN","maxContentLength":-1,"maxBodyLength":-1,"withCredentials":true

API and UI containers must share the same domain. You need to deploy the containers using Kubernetes with a proper domain configured.


luci-app-nft-qos: support mac address based speed limit

This patch adds MAC address based speed limit to luci.
This patch is used together with nft-qos-mac.
See https://github.com/openwrt/packages/pull/13097
Signed-off-by: Tong Zhang<EMAIL_ADDRESS>Screenshot below

Hi,
Can anyone help me to get this PR merged as well?
since it is already been about two weeks and the depend PR is already merged.
Or what additional steps do I need to take to get this PR merged,
Thank you!

Tong

@lzto thanks for your work


2.Grants step: "Add Grants" select box does not react to already listed awardNumbers

I log in as a person with one grant. Go to Grants page. Pick Create Submission for that Grant. On step 2. Grants. Award number is already added, but Add Grants shows that one Award Number. I select it, nothing happens. I remove the one Award Number from the list, Add Grants still has that Grant selected. I cannot re-add that Award Number without leaving the page and coming back to it so that the Add Grants box clears and i can re-select the Award Number.
One suggestion for fixing this is to make it so that selecting something in the Add Grants list blanks out the box afterwards so values do not get stuck in there. It might also be useful to have a "Grant already added" message so that the user doesn't think it isn't working. Another option is to not display grants that are already added.

Need to replicate disbehavior for clarification

@karenhanson could you tell me which test user has 1 grant so i could test this?


cssnano + @unocss/transform-directives causes an 'Cannot split a chunk that has already been edited' error

UnoCSS version
0.51.12
Describe the bug
Using:

Vite 4.3.2
SASS 1.62.1
UnoCSS 0.51.12
cssnano: 6.0.1

Using nested at-rules inside sass file causes error.
.example {
  // Doesn't matter which one is used
  @apply text-red-500;
  // --at-apply: text-red-500;

  @screen lt-md {
    // This causes an error
    @apply text-green-500;

    // Adding this instead doesn't work at all
    // --at-apply: text-blue-500;
  }
}
// Using this example https://github.com/unocss/unocss/blob/main/packages/transformer-directives/README.md#screen
// But it also doesn't work if cssnano is enabled
.example-2 {
  .grid {
    --uno: grid grid-cols-1;
  }
  @screen md {
    .grid {
      --uno: grid-cols-1;
    }
  }
  @screen lg {
    .grid {
      --uno: grid-cols-3;
    }
  }
}

[plugin:unocss:transformers:default] Cannot split a chunk that has already been edited (0:52 – "@screen lt-md{.example{@apply ")
![image](https://user-images.githubusercontent.com/16374473/236627648-2ada371a-7480-48d9-bcfc-9949759c3149.png)

Error appears when using postcss plugin: **cssnano**.
Using `--at-apply` or `--uno` does nothing.

I know it might be a duplicate of https://github.com/unocss/unocss/issues/2022 but seems like different things caused the same error.

### Reproduction

Reproduction:
https://stackblitz.com/edit/vitejs-vite-5anp17?file=src/example.scss

Comment out cssnano plugin in `postcss.config.cjs` to remove this error.

### System Info

Windows 10 64-bit
Chrome 113.0.5672.63
Package manager: npm & pnpm

### Validations

- [X] Read the [Contributing Guidelines](https://github.com/unocss/unocss/blob/main/CONTRIBUTING.md).
- [X] Check that there isn't [already an issue](https://github.com/unocss/unocss/issues) that reports the same bug to avoid creating a duplicate.
- [X] Check that this is a concrete bug. For Q&A open a [GitHub Discussion](https://github.com/unocss/unocss/discussions) or join our [Discord Chat Server](https://chat.antfu.me).
- [X] The provided reproduction is a [minimal reproducible example](https://stackoverflow.com/help/minimal-reproducible-example) of the bug.

I also encountered the same problem.
I looked at it and it seems that the vue file that uses the @apply directive will trigger this error.
Using:
Vite v3.1.2
UnoCSS v0.45.21
UnoCSS vscode plugin v0.53.1

⚠ Error on annotation
Error: Cannot split a chunk that has already been edited
// ...（my code）
// ...
// ...
at MagicString._splitChunk (c:\Users\hi\.vscode\extensions\antfu.unocss-0.53.1\dist\index.js:68217:13)
    at MagicString._split (c:\Users\hi\.vscode\extensions\antfu.unocss-0.53.1\dist\index.js:68210:21)
    at MagicString.appendLeft (c:\Users\hi\.vscode\extensions\antfu.unocss-0.53.1\dist\index.js:67780:10)
    at parseApply (E:\project\test\node_modules\.pnpm\@unocss+transformer-directives@0.45.30\node_modules\@unocss\transformer-directives\dist\index.cjs:159:12)
    at List.<anonymous> (E:\project\test\node_modules\.pnpm\@unocss+transformer-directives@0.45.30\node_modules\@unocss\transformer-directives\dist\index.cjs:106:7)
    at async Promise.all (index 0)
    at handleApply (E:\project\test\node_modules\.pnpm\@unocss+transformer-directives@0.45.30\node_modules\@unocss\transformer-directives\dist\index.cjs:102:3)
    at processNode (E:\project\test\node_modules\.pnpm\@unocss+transformer-directives@0.45.30\node_modules\@unocss\transformer-directives\dist\index.cjs:202:7)
    at async Promise.all (index 52)
    at transformDirectives (E:\project\test\node_modules\.pnpm\@unocss+transformer-directives@0.45.30\node_modules\@unocss\transformer-directives\dist\index.cjs:205:3)
    at getMatchedPositionsFromCode (c:\Users\hi\.vscode\extensions\antfu.unocss-0.53.1\dist\index.js:68557:5)
    at updateAnnotation (c:\Users\hi\.vscode\extensions\antfu.unocss-0.53.1\dist\index.js:68734:10)

normalizeWhitespace: false, - solves the issue.
Looks like regex for @screen works not as expected when nanocss removes the whitespaces.
temporary solution, but it works.
const options = {
  preset: [
    'default',
    {
      normalizeWhitespace: false,
    },
  ],
};

module.exports = () => ({
  plugins: [
    require('autoprefixer')(),
    require('cssnano')(options),
  ],
});

I have same error using transformerAttributifyJsx


add basic healthcheck feature #105

Summary of the changes (in less than 80 chars)

Added the most basic healthz endpoint

Addresses https://github.com/bagetter/BaGet/issues/105
I tried adding the database provider health checks at the same time, however the way the registration of those via the IProvider interface made it a lot more difficult than I have time for right now.

@Hechamon issue 123 does not exist. Which do you refer to?

@Hechamon issue 123 does not exist. Which do you refer to?

105, sorry :-)

Can you make the slug/path of the health check overwriteable via a config HealthCheckPath?
Additionally a mention somewhere in the docs that we now have a health check and its default value would be good.
Thank you!

Call me oldschool, but I prefer a more common name for routes. So I prefer the more common /health route name.
In rare cases I also came across /hc.
If I find some time the next days and if you would like, I could try to extend/adjust this PR. Because you said you are short of time.

I have pushed a new commit to make the endpoint configurable with a default of /health


pageInfo.startCursor & endCursor are NonNull in the relay spec

According to
https://relay.dev/graphql/connections.htm#sec-undefined.PageInfo.Fields
startCursor and endCursor should be Non Null. We have those fields as nullable to handle the case where a page has no values. There is no mention in the spec how to handle that case
This task is to investigate how it's handled in similar systems and decide if we want to change. Note, moving for Nullable to NonNull is not a breaking change so no version bump would be required.

This was a documentation error that has been fixed: https://github.com/facebook/relay/pull/2655

awesome, thanks for letting me know


Add pagination feature to the web page for sample list

FEATURE REQUEST
Inspiration
The sample list page gets less user friendly, as more samples continue to be loaded into the database.
Description
Add pagination feature to web page for Sample list.   Add Bootstrap-Table client-side pagination feature as the first step.
Alternatives
May switch to Bootstrap-Table server-side pagination if it could work nicely with data source from Pandas data frames.
Dependencies
none
Comment
Plan to implement the change to Sample List page first. The strategy could also apply to Animal list and the list for all studies/animals/samples (from study_summary view) in the future.
To use Bootstrap-Table server-side pagination, the data source needs to be accessed from data-url in required format, and data filtering/slicing needs to use query parameters defined by Bootstrap-Table.

ISSUE OWNER SECTION
Assumptions
none
Requirements

[ ] 1. Sample list is paginated

Limitations
none
Affected Components

change: sample.py and sample_list.html
add: need to add a view to output json data required by Bootstrap-Table data-url attribute.

DESIGN
Interface Change description
The Sample list page will be paginated.
Code Change Description
Need to evaluate if it's feasible to display filtered data from a Pandas data frame based on selected values for page size and page number for a table in Bootstrap-Table format on web page. If it worked, it would allow data filtering based on JSON data from a url with GET request without introducing a new form.
Tests

[ ] 1. Test that sample list page is paginated

I'll be curious to see if you're able to get a view to generate the content for the data url.  When I first tried out server side pagination, it had to be a json file and I could not coerce it to be generated dynamically.  I tried quite hard to get it to work and ultimately decided it was easier to use their paging form utilities to do my own pagination because there was no way to only load a portion of the content.  It all had to be loaded up front because it would not query again for each subsequent page.  I'd have to dig through my notes to get the specifics of what I learned about how it works and why it didn't do what I needed.

I went through a part of documentation and tried a small size of samples, and it seemed to me it could work. But I need to dig into more details to build a case for evaluation. The data source has to be from data-url with the following format:
e.g.
{
    "total": 200,
    "rows": [
        {
            "id": 0,
            "name": "Item 0",
            "price": "$0"
        },
        {
            "id": 1,
            "name": "Item 1",
            "price": "$1"
        },
...
}

I suppose it depends on your intent.  My goal was to only process and return the number of records required for each individual page, because generating all data was slow, but when I evaluated bootstrap-table's server-side pagination, it only ever queried the server once upon initial load and it saved all the json and used that to paginate client-side without further server queries, which seemed to me to defeat the purpose.  The whole point of server side pagination was to query the server for each new page.  And there was no way at the time, to do that. But maybe it has improved or else, I was missing something.  I encountered the issue when I tried to figure out how to tell it what page to load next, i.e. how to tell the server I wanted the next page...

I think just figured out how to do this.  Totally had to read between the lines because it doesn't say anywhere how to implement the view underlying the data url.  For the life of me, when I'd tried out the server side pagination, I couldn't figure out how to obtain the page number and rows per page (let alone the sort column, sort direction, filter, or any custom parameters), but the lowest voted answer on this stack post had the critical clue I needed the URL parameters:

Offset
limit

I tried it out using the data URL in this demo:
https://examples.wenzhixin.net.cn/examples/bootstrap_table/data
If you go to that URL, you see all 200 rows in the table.  But, if you append the URL parameters:
https://examples.wenzhixin.net.cn/examples/bootstrap_table/data?Offset=10&limit=10
...you get a subset.  So if you create a view that outputs json and takes those parameters, you can do it!
There's probably similar URL parameters for server-sort as well, but I have no idea where/if those parameters are documented.  I never found them before.  I can't believe they're not the first thing in the documentation.

I agree the documentation is not clear enough. Late last week, I figured out the parameters used by Bootstrap-Table server-side parameters based on the following document/examples:
https://bootstrap-table.com/docs/api/table-options/#queryparams
https://live.bootstrap-table.com/example/options/query-params-type.htm
https://live.bootstrap-table.com/example/options/query-params.html
I got "search" parameter working in addition to offset and limit, but ran out of time to fix a syntax error with searching a whole Pandas dataframe before my vacation. I'll try to make some progress today.
Another issue I faced was formatting column values (e.g. adding hyperlink) when using Bootstrap-Table server-side pagination. I was unable to use Django's customtag and had to use JavaScript to add function to data-formatter attribute for some columns.


Running nose tests reports the wrong number of tests passing or failing.

Environment data

VS Code version: 1.31.1
Extension version (available under the Extensions sidebar): 2019.2-RC (Also tested in 2019.1 where the behaviour is also exhibited)
OS and version: Windows 10
Python version (& distribution if applicable, e.g. Anaconda): Python 3.7.2
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): venv
Relevant/affected Python packages and their versions: nose==1.3.7

Expected behaviour
Test results report 1 pass and 1 fail.
Actual behaviour
Test results report 2 passes and no failures when the passing test is run.
Test results report 2 fails and no passes when the failing test is run.

Gif showing that running the test from code lens duplicates the test results

Steps to reproduce:

Create a workspace
Set up venv for the workspace in a Powershell window:

cd <workspace>
py -3.7 -m venv .venv
.venv/Scripts/Activate.ps1
python -m pip install -U pip
python -m pip install nose

Open VS Code and open the workspace folder you created.
Add a single file called test_nose.py to the workspace.
Add the following code to test_nose.py:

def test_passing():
    assert 42 == 42

def test_failure():
    assert 42 == -13

Click the Run Tests button in the status bar.
Click on the Run All Unit Tests in the command bar selection that pops up.
Note that the two test methods are marked with code-lens appropriated (test_passing is marked as passing, test_failure is marked as not passing).
Click on the Run Test of the test_passing code lens.

Note that 2 tests are reported as passing.

Click on the Run Test of the test_failure code lens.

Note that 2 tests are reported as failing.

Logs from Python::Python Test Log
Logging from Python Run All Unit Tests
.F
======================================================================
FAIL: test_nose_tests.test_failure
----------------------------------------------------------------------
Traceback (most recent call last):
  File "c:\dev\github\d3r3kk\test\bugbash_feb_2019\.venv\lib\site-packages\nose\case.py", line 198, in runTest
    self.test(*self.arg)
  File "c:\dev\github\d3r3kk\test\bugbash_feb_2019\test_nose_tests.py", line 6, in test_failure
    assert 42 == -13
AssertionError

----------------------------------------------------------------------
Ran 2 tests in 0.005s

FAILED (failures=1)

Logging from Clicking the Run Test Code Lens of the passing test:
..
----------------------------------------------------------------------
Ran 2 tests in 0.001s

OK

Logging from Clicking the Run Test Code Lens of the failing test:
FF
======================================================================
FAIL: test_nose_tests.test_failure
----------------------------------------------------------------------
Traceback (most recent call last):
  File "c:\dev\github\d3r3kk\test\bugbash_feb_2019\.venv\lib\site-packages\nose\case.py", line 198, in runTest
    self.test(*self.arg)
  File "c:\dev\github\d3r3kk\test\bugbash_feb_2019\test_nose_tests.py", line 6, in test_failure
    assert 42 == -13
AssertionError

======================================================================
FAIL: test_nose_tests.test_failure
----------------------------------------------------------------------
Traceback (most recent call last):
  File "c:\dev\github\d3r3kk\test\bugbash_feb_2019\.venv\lib\site-packages\nose\case.py", line 198, in runTest
    self.test(*self.arg)
  File "c:\dev\github\d3r3kk\test\bugbash_feb_2019\test_nose_tests.py", line 6, in test_failure
    assert 42 == -13
AssertionError

----------------------------------------------------------------------
Ran 2 tests in 0.002s

FAILED (failures=2)

Output from Console under the Developer Tools panel
Not very interesting
console.ts:134 [Extension Host] Python Extension: Cached data exists ActivatedEnvironmentVariables, c:\dev\github\d3r3kk\test\bugbash_feb_2019
console.ts:134 [Extension Host] Python Extension: getActivatedEnvironmentVariables, Class name = b, Arg 1: <Uri:c:\dev\github\d3r3kk\test\bugbash_feb_2019>, Arg 2: undefined
console.ts:134 [Extension Host] Python Extension: Cached data exists ActivatedEnvironmentVariables, c:\dev\github\d3r3kk\test\bugbash_feb_2019
console.ts:134 [Extension Host] Python Extension: getActivatedEnvironmentVariables, Class name = b, Arg 1: <Uri:c:\dev\github\d3r3kk\test\bugbash_feb_2019>, Arg 2: undefined
console.ts:134 [Extension Host] Python Extension: Cached data exists ActivatedEnvironmentVariables, c:\dev\github\d3r3kk\test\bugbash_feb_2019
console.ts:134 [Extension Host] Python Extension: getActivatedEnvironmentVariables, Class name = b, Arg 1: <Uri:c:\dev\github\d3r3kk\test\bugbash_feb_2019>, Arg 2: undefined

Closing via #16371


Refactored tests using test_helpers library

This PR gives us the rewritten ppx-expect-tests. The new tests have been written using the test_helpers library, which has been applied to generate signatures for files automatically for comparison of interfaces. @NathanReb Please review.

@NathanReb I have made the suggested changes to the string literals. They do look better with string delimiters!
Also, I have squashed the commits into a single meaningful commit. Not running dune fmt before making a commit was a mistake on my part. I will keep note of that in my further contributions. Hope this PR is okay now!

@NathanReb I ran dune build @fmt on my terminal and it updated some of my code. I used 'git add` on the code to incorporate those changes, but now I see the changes were not added. This may be because I use ssh to connect my virtual machine where this project is located and VSCode which is in my local machine.
I really apologise for this inconvenience.

Running dune build @fmt does not reformat, it simply checks whether the file are properly formatted and shows the diff otherwise. You need to run dune promote if it showed you a diff to "accept" those changes and promote them to the source files.
You can also run dune build @fmt --auto-promote if you want to include the promotion step automatically.


Feature request: Side-by-side preview

Hi, it would be nice if we could get a side-by-side preview of the dashboard while editing it. I've made a mockup using the browser's inspector and putting the dashboard in an iframe:

It seems it would be easy to implement, but I'm not sure about the details, for example:

When you edit a dashboard from the launcher, should it always open the preview to the side?
If not, should there be a menu item to open the preview for the current file? Can we detect that the current file belongs to a dashboard?

and so on. I'm curious if this is something others would be interested in as well.

It's a good idea, and matches the way Voilà works. To be honest, I think this is something that Streamlit would be best placed to work on, independently of any of the ContainDS work. I guess Streamlit is less focused on the Jupyter ecosystem precisely because there isn't an immediately deployment system built in (although nor is there for Voilà really!).


pin noteable-origami

Current dependencies on/for this PR:

main

PR #217   👈

This comment was auto-generated by Graphite.


spacy not imported by default in quick_start.ipynb

Hi,
Just a little thing, spacy isn't imported by default in quick_start.ipynb, so running the cells top to bottom gives an error.
Also, FileNotFoundError: [Errno 2] No such file or directory: 'data/crunchbase_companies.json' occurs in the gazetteers cell, as crunchbase_companies.json is not actually present in the data directory yet.

Fixed!


How to wear glasses to the face

Nice work. If i want to extend the work for the glasses to the face. How could I do it? Thanks

Nice work. If i want to extend the work for the glasses to the face. How could I do it? Thanks

I am also looking for wearing glasses tool. I found some virtual try-ons but exactly what I am looking for. Please share if you found one. Thanks.


Release 0.4.0

Composition postgres 142 #147
Retry on specific exception type 84 #135
Add name attribute to workers and print it in the logs #132
New main contributors #138
First monitor and corresponding cli command #129
Custom JSON encoders and decoders #131
Typo fix and a missing sentance in the doc. #136
Set enable_uuid to False when calling aiopg.connect #134
Fix bug in get_connection #130
Make two tests in test_testing async #128
Use asyncio.TimeoutError #125
Quick improvement of logic flow #121
Remove the Task name/full_path warning #123
Fix/improve quickstart #120
Use real command names, not my personnal shell alias #122
Add @sophie-ulti to the codeowners #116
add *.pyc to gitignore #115
Release 0.3.0 #114

Codecov Report

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

@@          Coverage Diff          @@
##           master   #154   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          21     21
  Lines         817    817
  Branches       81     81
=====================================
  Hits          817    817

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 acf002d...64a17f0. Read the comment docs.


not worked with vue-cli-plugin-svg-sprite

Not worked with vue-cli-plugin-svg-sprite.
Demo - https://github.com/acopalipsis/svg-bug-demo
I really hope For your help.
UPDATE REPO

WARNING in ./src/assets/icons/baseline-error_outline-24px.svg
Module build failed (from ./node_modules/svg-sprite-loader/lib/loader.js):
ExtractPluginMissingException: svg-sprite-loader exception. svg-sprite-loader in extract mode requires the corresponding plugin

@ multi<EMAIL_ADDRESS><EMAIL_ADDRESS>./config/storybook/config.js (webpack)-hot-middleware/client.js?reload=true

Hello!
This warning happens due to vue-cli-plugin-storybook filtering all the plugins according to the internal list of allowed plugins. There is functionality to manually extend this list through pluginOptions described in README.md, but it isn't published to npm yet.
@pksunkara can you publish it please?

This was done long ago. Should work now.


⚠ Bofrost.fr has degraded performance

In 45ef331, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1097 ms

Resolved: Bofrost.fr performance has improved in c0e0ceb after 6 minutes.


Flag class file collision as error

Fixes #19248

Nicolas's Github status says On vacation...
@smarter Would you like to take a really quick look at this PR? This PR is small and should be light to review.


[Feature]: Why do you delete @Tailor?

Problem
@Tailor was good decision for my project. I delegate to it many boilerplate. Now I need to write this boilerplate by hand with @TailorMixin.
Desired Solution
Generate theme constructing from static field definitions, like it was with @Tailor
Alternatives Considered
No response
On which platorm do you expect this solution?
 It is error-prone work

Thanks for sharing your thoughts. I understand the frustration with the extra work required by @TailorMixin compared to the old @Tailor approach.
The switch was made because the previous method had some issues, especially with static properties. Working with those was not type-safe and often led to errors when assigning values to the theme. The need for a dummy class for code generation was also problematic—it wasn't really aligned with best practices, like avoiding unnecessary code.
The new approach with @TailorMixin might involve more manual setup, but it's more reliable (you're also able to freely add your own methods and work with other generators without them being handled by the package) and closer to what we envision for the package. There’s a chance things will improve in future Dart releases, especially when macros get fully implemented.


About Dataset License

Dear Authors:
Thank you very much for providing the useful benchmark dataset.
It seems to be made by integrating and refining other original benchmark datasets. I have two questions as follows:

Does the TSB-AD dataset have any specific usage terms or conditions?
If yes, how do they comply with the terms of the original datasets?

Hi! Thanks for bringing up the discussion. The preprocessing and curation steps are provided under the Apache 2.0 license. If you use any of these underlying datasets in your research, please refer to the original data source. License information for each dataset included in TSB-AD is provided at [Link] for your reference.
Let us know if you have any questions regarding the license issues.


Connect Redirection change local IP address

I have tested and found out I can change local IP address for TCP protocol in connect-redirect layer FWPM_LAYER_ALE_BIND_REDIRECT_V4 on Windows 10 (proof by TCPView). But with UDP protocol I can't.
Can you confirm that this feature work on Windows 10 ? Because the document said it's not supported in connect-redirect layer https://docs.microsoft.com/en-us/windows-hardware/drivers/network/using-bind-or-connect-redirection

Hi @zeltrax00,
Can you please clarify your question? The documentation says that UDP packets are supported for redirection.
The layer at which redirection is performed determines the effect of the change. Changes at connect layers affect only the flow being connected. Changes at bind layers affect all connections that are using that socket.
The redirect layers are only available for Windows 7 and later versions of Windows.

Hi @aviviano , the documentation says Changing the local address and port of a flow is only supported in the bind-redirect layer. This is not supported in the connect-redirect layer.. But from my experiment, I can successfully change local address and port of a TCP connect in connect-redirect layer, but not UDP. So I want to know if the documentation is out of date or it's just a bug.

Hi @zeltrax00,
I spoke with the product team, and they confirmed that the documentation is correct. They are aware that changing the local address and port of a flow can be done in TCP, but it is still not supported.
Thanks for your question.

I have few more question:

So what do I need to do if I want to change the interface for UDP packet after bind phase ?
Does the example code in the above documentation supports UDP or it is an TCP example.

Hi @zeltrax00, I spoke with the product team, and they confirmed that the documentation is correct. They are aware that changing the local address and port of a flow can be done in TCP, but it is still not supported. Thanks for your question.

I have few more questions:

What am I supposed to do if I want to change the interface for UDP packet after bind phase ?
Does the example in the above documentation support UDP or it is a TCP example ?


Adding a home button at the submition of the quiz at the analysis of quiz page.

There should be home button at the submittion page of the quiz which take you back to the home page. As there is no way back option after you reach your analysis of the quiz.

Hello @AbhinavBattu Thanks for creating this issue but unfortunately this has been already done. You can add some other issue (if it has). Thanks once again.


powershell 7.4 sort cmdlet sort différenlty from PS 5.1 cmdlet

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
I am experiencing  problems with the result of sort cmdlet.
Here a string array 'file.txt','file_h.txt'
I apply sort  in it : 'file.txt','file_h.txt' | sort
with powershell 5.1 the result is  :
'file.txt','file_h.txt' | sort
file.txt
file_h.txt

with powershell 7.4.1 the result is :
'file.txt','file_h.txt' | sort
file_h.txt
file.txt

wich is not lexical order
Expected behavior
PS>'file.txt','file_h.txt' | sort
file.txt
file_h.txt

Actual behavior
PS>'file.txt','file_h.txt' | sort
file_h.txt
file.txt

Error details
No response
Environment data
Name                           Value
----                           -----
PSVersion                      7.4.1
PSEdition                      Core
GitCommitId                    7.4.1
OS                             Microsoft Windows 10.0.19045
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

I think this is for cross platform compatibility:
$ 'file.txt','file_h.txt' | /usr/bin/sort
file_h.txt
file.txt

bash:
$ echo -e 'file.txt\nfile_h.txt' | sort
file_h.txt
file.txt

I think this is for cross platform compatibility:

The OP is using windows and your solution requires POSIX /usr/bin/sort.
This post says the sorting rules are not just based on ASCII code or code-point.

your solution

Did you mean to say "assumption"? If I wrote a solution, it could be like this:
'file.txt','file_h.txt' | sort { [int[]][char[]] $_ }

Curious is when you add . to end of strings that result change
PS C:\Users\dom> 'file','file_h' | Sort-Object
file
file_h
PS C:\Users\dom> 'file','file_h.' | Sort-Object
file
file_h.
PS C:\Users\dom> 'file.','file_h.' | Sort-Object
file_h.
file.
PS C:\Users\dom> 'file.','file_h' | Sort-Object
file_h
file.

the example can be simplified
PowerShell 7.4.1
'.','_' | Sort-Object -Culture en-US
_
.

Powershell 5.1
'.','_' | Sort-Object -Culture en-US
.
_

note the -culture  same result with fr-FR
to hex
'._' | format-hex

   Label: String (System.String) <7E9D5112>

          Offset Bytes                                           Ascii
                 00 01 02 03 04 05 06 07 08 09 0A 0B 0C 0D 0E 0F
          ------ ----------------------------------------------- -----
0000000000000000 2E 5F                                           ._

How to explain this change of result ?
I try linux on https://www.tutorialspoint.com/linux_terminal_online.php
$ touch h.
$ touch h_
$ ls | sort
h.
h_
$ uname -a
Linux c07a703aeb3b 5.15.0-89-generic #99-Ubuntu SMP Mon Oct 30 20:42:41 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Interestingly, if I add criteria for sorting, the sorting will work as expected
$ 'h_','h.' | sort
h_
h.

$ 'h_','h.' | sort fake
h.
h_

PowerShell is only the messenger here:
.NET's sorting behavior on Windows changed with .NET 5's move to cross-platform, ICU-based internationalization, as described in .NET globalization and ICU.
PowerShell v7.1+ is based on .NET 5+
As a side effect, the sort order of _ vs. . is now different relative to Windows PowerShell (based on .NET Framework, which uses Windows-specific NLS sorting rules): for instance, in the en-US culture _ sorts before . in PowerShell 7.1+, on all platforms, whereas in Windows PowerShell . sorts before _.
This not only affects Sort-Object, but also implicit sorting contexts such as the output order of Get-ChildItem and tab-completion order.
On Windows (only), an opt-in to the old behavior is available in .NET 5+ (see the linked documentation), but note that that would take effect session-globally in PowerShell and also change other culture-related behaviors.
Curiously, like Windows PowerShell, the sort utility on Unix-like platforms places . before _ e.g. in the en-US culture
('_', '.' | /usr/bin/sort), even though you'd expect its sorting to be equally ICU-based; ditto for ls - it's unclear to me what rules are used, but it is not ordinal sorting (e.g., e and é sort together).
For more information, see:

https://github.com/PowerShell/PowerShell/issues/14757#issuecomment-1664221189
The following issues are effectively also duplicates:

#19736
#20730

As for ordinal sorting in PowerShell (sorting strictly based on the Unicode code points of the characters that make up the input strings):

You must currently use .NET APIs directly to get reasonable performance, e.g., with [string[]] $arr = 'z'..'a' as input, and case-insensitive ordinal comparison:

[Array]::Sort($arr, [System.StringComparer]::OrdinalIgnoreCase) (sorts in place)

Sort-Object-based workarounds are possible, but perform poorly, e.g., the following is about two orders of magnitude slower (and is memory-inefficient):

$sortedArr = $arr | Sort-Object { -join ([int[]] [char[]] $_.ToLower()).ForEach({ $_.ToString('x4') }) }

https://github.com/PowerShell/PowerShell/issues/9348 asks for a number of cmdlets, including Sort-Object to be enhanced so as to support ordinal sorting on demand, along with an associated RFC, https://github.com/PowerShell/PowerShell-RFC/pull/167. Unfortunately, activity has stalled have stalled quite a while ago.

@237dmitry:
For me, printf '_\n.' | /usr/bin/sort always outputs . first on Unix-like platforms, so I'm curious what your environment / locale / culture settings are that produce a different result.
... | Sort-Object { [int[]][char[]] $_ } doesn't actually work as intended, because it is treated like ... | Sort-Object { [string] [int[]][char[]] $_ } - try with 'z'..'a' -replace '^', 'a' | Sort-Object { [int[]] [char[]] $_ }
As for 'h_','h.' | Sort-Object fake: this produces pointless output, because you're comparing by nonexistent property .fake, so it is all $null values that are being compared.

I'm curious what your environment / locale / culture settings are that produce a different result.

Diiffrent where? In bash or in pwsh? bash sorting h. and h_ properly, but not file.txt and file_h.txt. If bash launched with --posix switch then result also proper.
$ locale
LANG=en_US.UTF-8
LC_CTYPE="en_US.UTF-8"
LC_NUMERIC=en_US.UTF-8
LC_TIME=en_US.UTF-8
LC_COLLATE="en_US.UTF-8"
LC_MONETARY=en_US.UTF-8
LC_MESSAGES="en_US.UTF-8"
LC_PAPER=en_US.UTF-8
LC_NAME=en_US.UTF-8
LC_ADDRESS=en_US.UTF-8
LC_TELEPHONE=en_US.UTF-8
LC_MEASUREMENT=en_US.UTF-8
LC_IDENTIFICATION=en_US.UTF-8
LC_ALL=

$ sls '^[^#]' /etc/locale.gen -raw
en_US.UTF-8 UTF-8

$ Get-Culture

LCID             Name             DisplayName
----             ----             -----------
1033             en-US            English (United States)

For me, the following places . first, in both PowerShell and POSIX-compatible shells, whether or not you use bash --posix, because the behavior is specific to the /usr/bin/sort utility, not to the shell it is being called from (assuming the same locale, such as en_US.UTF-8):
printf 'h_\nh.' | sort
printf 'file.txt\nfile_h.txt\n' | sort

Addendum: I've observed the above on macOS 14.3.1 and Ubuntu 22.04.
However, on CBL/Mariner 2.0.20231106, I see a modified version of this behavior - and this may apply to other Linux distros as well, but I don't know what that depends on.
# On CBL/Mariner 2.0.20231106 - both macOS 14 and Ubuntu 22.04 behave differently with the 2nd command.
printf '_b\n.a\n' | sort  # . first
printf '_a\n.b\n' | sort  # _ first

Seemingly, the logic there is (I'm speculating):

. and _ have the same weight in sorting if other parts of the string determine the sort order; if it comes down to . vs. _, . comes first; e.g., printf '_a\n.b\n' | sort prints _a before .b, because a sorts before b, whereas printf '_a\n.a\n' | sort sorts .a before _a, because the difference comes down to . vs. _.

That said, returning to the issue at hand: on Windows (where built-in alias sort refers to the Sort-Object cmdlet), https://github.com/PowerShell/PowerShell/issues/21322#issuecomment-1986961818 (hopefully) fully explains the issue at hand.

@237dmitry: Your screenshot provides no additional information beyond my previous observation.
The question is (moot in the context of the issue at hand, but, given that there's nothing actionable here, I believe it's fine to continue to explore it):
For two given Linux distros that use the same locale (locale) as well as the same version of the GNU implementation of the /usr/bin/sort utility (/usr/bin/sort --version), what explains the different outputs from the following, assuming that locale is en_US.UTF-8 and that /usr/bin/sort --version reoprts 8.32?
printf '%s\n' _ . _a .b | /usr/bin/sort

Output on Ubuntu 22.04 (via WSL) - . vs. _ drives the sorting, with . coming first:
.
.b
_
_a

Output on CBL/Mariner 2.0.20231106, which is different - a vs. b drives the sorting, and only if it comes down to . vs _ does . come first:
.
_
_a
.b

Additionally:

Why do both results differ from what PowerShell outputs, via printf '%s\n' _ . _a .b | Sort-Object, on both these distros (presumably based on the host platform's ICU library)? . vs, _ drives the sorting, with _ coming first:
_
_a
.
.b

How does ICU factor into this?

it seems that order can be view here https://www.unicode.org/charts/collation/

@237dmitry:
After further research, I've concluded:

Standard Unix utilities such as /usr/bin/sort as well as system calls are unrelated to ICU - they use distro-specific locale definitions typically stored in /usr/lib/locale

The differences observed in the output from printf '%s\n' _ . _a .b | /usr/bin/sort come down to differences in these locale definitions - it sounds like more recent versions of Linux distros exhibit the behavior where placing . before _ only applies if the input strings differ only by these characters.

I've updated the previous comment with a succinct summary, and I've hidden my subsequent comments - likewise I encourage you to hide yours.

Thanks, @DcaCodes, https://www.unicode.org/charts/collation/chart_Punctuation.html indeed shows that _ sorts before . based on the Unicode collation order, as reflected in '.', '_' | Sort-Object printing _ first in PowerShell Core (across all platforms).

@DcaCodes, collation has culture-specific elements, and I don't actually know how the linked Unicode chart accounts for that.
Overall, collation is a complex matter.
In the realm of ICU, there's https://icu4c-demos.unicode.org/icu-bin/collation.html, where you can experiment with sorting, and which has a dropdown for culture selection, as well as numerous other parameters - I presume these demos are based on the latest ICU version.
As the Unicode standard itself, ICU continues to evolve, and on a given platform you can determine which specific version of the ICU library is being used as follows:
(Add-Type -PassThru -NameSpace net.same2u -Name Aux -UsingNamespace System.Reflection -MemberDefinition @'
  public static Version GetIcuVersion() {
    Type interopGlobalization = Type.GetType("Interop+Globalization, System.Private.CoreLib");
    if (interopGlobalization != null)
    {
        MethodInfo methodInfo = interopGlobalization.GetMethod("GetICUVersion", BindingFlags.NonPublic | BindingFlags.Static);
        if (methodInfo != null)
        {
            int version = (int)methodInfo.Invoke(null, null)!;
            return new Version(version >> 24, (version >> 16) & 0xFF, (version >> 8) & 0xFF, version & 0xFF);
        }
    }
    return new Version(0, 0, 0, 0);
  }
'@)::GetIcuVersion()

Using the default settings seems to match the de facto behavior of Sort-Object with en-US,
but ICU and therefore .NET 5+ does exhibit culture-specific variations.
For instance, the infamous "Turkish i problem" results in different sorting of the various variants of the letter "i":
i I ı İ:
& {
  $prevCulture = [cultureinfo]::CurrentCulture
  'en-US', 'tr-TR' | % {
    [cultureinfo]::CurrentCulture = $_
    (Write-Output i I ı İ | Sort-Object -CaseSensitive) -join ' '
  }
  [cultureinfo]::CurrentCulture = $prevCulture
}

Output:
i I İ ı  # en-US
ı I i İ  # tr-TR

You can replicate the above at https://icu4c-demos.unicode.org/icu-bin/collation.html by pasting the following as input:
i
I
ı
İ

and, with the culture dropdown list item at the default, und (type=standard) root (Standard Order)), vs. tr (type=standard): Turkish (Standard Sort Order), clicking sort

@rhubarb-geek-nz, re the Stack Overflow post you mentioned:
While it indeed implies that Sort-Object does not perform ordinal sorting - instead, it performs linguistic sorting based on the current culture, case-insensitively by default ([StringComparer]::CurrentCultureIgnoreCase) - the other issues discussed there, while interesting, are incidental to the discussion here.
I've posted an answer there that clarifies the fundamentals and also explains the incidental issues.

Let me try to provide a succinct summary here, which hopefully brings closure to the issue at hand as well as the aforementioned near/effective duplicates:

Sort-Object uses  linguistic sorting based on the current culture, case-insensitively by default ([StringComparer]::CurrentCultureIgnoreCase); case-sensitivity is opt-on, via -CaseSensitive ([StringComparer]::CurrentCulture), and the culture context can be altered ad hoc with a -Culture argument (use 127 to request the invariant culture).

As noted, Sort-Object currently unfortunately does not support ordinal sorting (see the stalled issue #9348).

In v7.1+ / .NET 5+, it is the ICU library that provides the collation (sorting) rules - PowerShell merely surfaces the underlying .NET behavior.

These ICU-based rules differ both from earlier versions on Windows and Windows PowerShell, and notably also from the native sorting rules on Unix-like platforms, such as exhibited by /usr/bin/sort

This discrepancy is unfortunate, especially in the context of sorting of file-system items, where the now-frustrated expectation is that . sorts before _, but unlikely to change.

On the plus side, it results in consistent cross-platform behavior, assuming the same ICU library version (see next point); that said, new library versions often bring only additive and minor changes.

A major caveat, however, is that on Windows you still get the NLS defaults for the regional settings, in combination with ICU collation rules, for instance - see https://github.com/dotnet/runtime/issues/81853#issuecomment-1424974267

As all cultural rules, ICU's rules are subject to change over time.

The above comment shows how to determine the specific ICU library version that is used on a given machine.
The ICU Locale Explorer allows interactive experimentation, albeit seemingly only based on the most recent ICU version.


Interfacing Blood Pressure Sensor with BeagleBone Black

What issues did you or others might face when connection of a Heart rate Sensor to the BBB in terms of hardware and software?
How did you solve them?

Ans1:- First and most important issue we faced to find out the Blood Pressure sensor which will send the data to the beaglebone. The second problem was the blood pressure we used is the finale product of one MQ company. which we modified to get data: - but the problem was to understand the data. Because It sends the data in packages and ASCCI char. which will first need to convert in Decimal numbers and we have to Verifiyies which byte in a package of 10 bytes contain the data for our required output.
Ans2:- The first problem was solved by choosing a sensor that will send the data to the LED screen. Which we thought will be used to connect the beaglebone using an LED connecting port but finally, we Find the serial pin which sends the data UART 8N1. which we understand because of the logic analyzer. Logic analyzer shows the data in channel of pulse view software. which gave us an understanding of 10 bytes of the package and which are our need bytes. So a logic analyzer is the best option to understand the unknow output of device. In this way, we solve the above mention problems.


Investigate Build issue while upgrading 3.9.X to 3.11.X

Steps taken -

Fixed the code to enable start build command.
Initiated a fresh build through CI/CD pipeline

The new build seems to be failing because the MOODLE_BRANCH_VERSION Variable in docker file is referencing to an upstream branch that doesn't exist. The upstream branch is controlled by the environment variable in Docker file as follows
ENV MOODLE_BRANCH_VERSION MOODLE_39_STABLE(for 3.9)
We are using MOODLE_BRANCH_VERSION = MOODLE_11_STABLE(this doesnt't exist upstream)
Instead we might want to use MOODLE_311_STABLE for 3.11, perhaps.
@warrenchristian1telus  please confirm if MOODLE_311_STABLE should be good to go.
@freshjwarren @will-freshworks

I fixed the Moodle branch version typo and the build completed successfully. I ran the required DB updates and verified that the site and GUID updates are working as expected.


Bump moc to 0.7.6

@ggrief, it looks like something is up with the repository permissions. I could open a JIRA ticket if that would be helpful in regards to time zone.

(it's @ggreif :-)
Which permissions? I am not an admin here.

Whoops; sorry about the typo! Thanks.
Since you're part of dfinity/dx, you should be able to approve PRs in this repository, right?

My vote is not good enough here :-(


Security 5.2: Well Edit - Sample field updates

The sample field is no longer required in Well Edit.
Allow for a blank value to be selected
Show 'Unspecified' and 'Multiple values selected' in the sample field as appropriate
HTML updates

application_main.js: Update sample section of fillWellEditForm function. Use jquery approach to setting selectpicker value.

Test plan

Can add a sample to a single well or to a group of wells - the assignment is committed to the db and is persistent in UI
When blank, the Well Edit form loads with 'Unspecified' in the sample field
'Multiple values selected' shows the Well Edit form loads for multiple wells representing more than one sample

Test Plan - OK!


Update template-typescript-w-decorator.vue to use nuxt

We will need to update .vscode/vetur/snippetstemplate-typescript-w-decorator.vue to

import { Vue } from 'nuxt-property-decorator'

Is juts to update those import lines? If so I would like to work on it

@arcadioramos sure let's do it


added unit tests for following endpoints:

Work dashboard
work resources
Work staff roles

Get
Create
Details
Update
Validate / Exists

Download workplan
Work phase template upload status
Work first nations

Get
Create
Details
Update
Validate / Exists
Download
Import

Work notes

first nation notes
issue notes
status notes

Get work types

Codecov Report
Attention: Patch coverage is 80.00000% with 1 lines in your changes are missing coverage. Please review.

:exclamation: No coverage uploaded for pull request base (develop@417ef41). Click here to learn what that means.

Files
Patch %
Lines

epictrack-api/src/api/services/work.py
75.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #1883   +/-   ##
==========================================
  Coverage           ?   76.11%
==========================================
  Files              ?      267
  Lines              ?     8873
  Branches           ?        0
==========================================
  Hits               ?     6754
  Misses             ?     2119
  Partials           ?        0

Flag
Coverage Δ

epictrack-api
76.11% <80.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


⚠ Spryr has degraded performance

In a5c7052, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 4892 ms

Resolved: Spryr performance has improved in 6ab97eb after 11 minutes.


Style refactoring - regulation nav

Create generic mixin based font integration points for regulation nav (drawer).
Create custom-mixins.less to give the ability to overwrite or extend mixins.less

Going thru the existing css and implementing Jen's changes, it's clear that to implement any new theme we will need a granular way to modify font setting for a lot of different content.  Doing this thru variables can easily become unwieldy since the design can call for changes in size, line height, color, family, and style across any of these integration points.  This can lead to a variable explosion. Mixins give the flexibility to specify all or none of these neatly in one place.
The general approach I intended to take was apply this concept across all integration points.

This has been reviewed by a few folks now. The expected flow:

define core variables/mixins
allow a hook for agencies to override
build meaningful modules on top of those variables/mixins
continue to allow agencies to completely override whole less files if need be


It features server Cache?

Hello! The router is also a cache server like Nginx? If not, there is some functionality that is similar? Thank you! Using Google Translate.

 You could run your own cache service (nginx, varnish, etc.) behind the router if you wanted to.

Got it. Thank you!


Propagate "-O" to Clang (as "-Os")

It affects Clang IRGen, among other things.

@swift-ci Please smoke test and merge.

This broke a number of tests that use precompiled bridging headers. I'm not sure why that didn't show up in your smoke test, but it's pretty clearly this commit since it refers to a change in the __OPTIMIZE__ macro.

Okay.  @graydon, what did I do wrong here?  Do I need to include this in PCHHashComponents despite it presumably being accounted for in Clang's module hashing?

It looks like the O group of options isn't part of the driver's addCommonFrontendOptions, probably because it might trigger SIL passes during module merging or extra optimization in the REPL. So the PCH generation job isn't getting this at all.

It's probably not blocking CI because CI probably doesn't run optimized tests.

Please see rdar://problem/33541306

Fixing in #11216.


Conditional check for templates isn't working correctly

Hello,
I've tried to set up a conditional check for sidebars to be hidden of several template files, but it doesn't work as it should.
This is my code and these templates don't show the sidebar anymore:
[ 'is_page_template', ['template-contact.php','template-flexible.php', 'template-custom.php']],
But it hides the sidebar for other page templates as well besides the default one.
Marton

Looks like you're doing it wrong. Pass those three template files as your second parameter.
    $sidebar_config = new Roots_Sidebar(
      /**
       * Conditional tag checks (http://codex.wordpress.org/Conditional_Tags)
       * Any of these conditional tags that return true won't show the sidebar
       *
       * To use a function that accepts arguments, use the following format:
       *
       * array('function_name', array('arg1', 'arg2'))
       *
       * The second element must be an array even if there's only 1 argument.
       */
      array(
        'is_404',
        'is_front_page'
      ),
      /**
       * Page template checks (via is_page_template())
       * Any of these page templates that return true won't show the sidebar
       */
      array(
        'template-contact.php',
        'template-flexible.php',
        'template-custom.php'
      )
    );

You technically could do it by using is_page_template as part of the array in the first parameter, but you'd have to repeat it for each template, which I'm not going to demonstrate here because it's pointless.
You should take issues like this to http://discourse.roots.io/


Parse -0 as float -0.0 instead of integer 0

Fixes #799.

This seems to break a "round trip" of a 0 value.  We have a test that does a conversion from JSON -> typed structure then back to JSON, when we compare the reference JSON to the generated JSON we get a breakage because 0 is not equal to -0.


Issue with bench.cpp in the new version 1.4x of ospray

Hi,
I am working with bench.cpp the 1.4x version of the ospray .  The new version build fine . But the code bench.cpp and ospBenchmark.exe  crashes when we give the command arguments  as in the image below. Please advise.

From the cmd we tried running the command
ospBenchmark.exe G:/projects/Newfolder/wooddoll/wooddoll_3.obj .
But we are getting the error as below

Thanks for reporting. Fix is in release-1.4.x branch.

Hi .I was able to fix it.
On Tue, Oct 24, 2017 at 6:10 AM, Johannes Günther<EMAIL_ADDRESS>wrote:

Thanks for reporting. Fix is in release-1.4.x branch.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ospray/ospray/issues/188#issuecomment-338984568, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AWbURIlA8uBP3iyZVn1jaK04uiqbzqxGks5sveHjgaJpZM4QEAKh
.

Thanks a lot


Has something changed since the Jenkins tutorial?

I'm following these instructions: https://jenkins.io/doc/tutorials/build-a-python-app-with-pyinstaller/ with the difference being that I am using the python3 instead of python2 and, obviously that the name of my python file is different.
You can see at: https://github.com/djotaku/ELDonationTracker/blob/devel/JenkinsFile
The error I'm getting is:
+ pyinstaller --onefile extralifedonations.py

<EMAIL_ADDRESS>1<EMAIL_ADDRESS>pyinstaller: not found

script returned exit code 127

Has something changed since they wrote that tutorial? Or is something different with Py3 or something?
Thanks!

So if I change "steps" to
sh ''
Jenkins stops complaining, but nothing runs and I don't get a binary.

Here's what the pipeline.log contains:

[Pipeline] withEnv
[Pipeline] {
[Pipeline] sh
+ docker inspect -f . cdrx/pyinstaller-linux:python3
.
[Pipeline] withDockerContainer
Jenkins seems to be running inside container 7425a4d1ee8a9dbf06ad189d97bc1914aa5e8187246ba901d88dba8d4ea1b7a2
$ docker run -t -d -u 0:0 -w /var/jenkins_home/workspace/extralifedonationtracker --volumes-from 7425a4d1ee8a9dbf06ad189d97bc1914aa5e8187246ba901d88dba8d4ea1b7a2 -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** cdrx/pyinstaller-linux:python3 cat
$ docker top d7f1e992b70d59cb4fee891e933d8228f93c4cfaf65d6435e6d24437c4f3618b -eo pid,comm
ERROR: The container started but didn't run the expected command. Please double check your ENTRYPOINT does execute the command passed as docker run argument, as required by official docker images (see https://github.com/docker-library/official-images#consistency for entrypoint consistency requirements).
Alternatively you can force image entrypoint to be disabled by adding option `--entrypoint=''`.
[Pipeline] {
[Pipeline] sh
Post stage
[Pipeline] archiveArtifacts
Archiving artifacts
[Pipeline] }
$ docker stop --time=1 d7f1e992b70d59cb4fee891e933d8228f93c4cfaf65d6435e6d24437c4f3618b
$ docker rm -f d7f1e992b70d59cb4fee891e933d8228f93c4cfaf65d6435e6d24437c4f3618b
[Pipeline] // withDockerContainer
[Pipeline] }
[Pipeline] // withEnv
[Pipeline] }
[Pipeline] // node
[Pipeline] }
[Pipeline] // stage
[Pipeline] End of Pipeline
Finished: SUCCESS

Possibly related to #38
I think this may be something to do with the entry point being overwritten, but I've never used Jenkins so not sure how it works.
Will leave this open for now incase any jenkins experts spot this issue...

@djotaku what Jenkins is doing here is running container in background mode with cat command. Then before any command is run within the container it does sanity check, whether cat ps is actually running within the container namespace.
When you have container with entrypoint set, it is a bit tricky and behaves differently between docker-pipeline-plugin versions. Since version 1.9 of that plugin, entrypoint is not respected - JENKINS-41316 - so your attempt would work. In your case, since there is already entrypoint in pyinstaller-linux docker image, it will fail, because /entrypoint.sh script is what really gets run.
Ok, so what can you do to solve that:

upgrade docker pipeline plugin to >=1.9 - simple but may break stuff for other people using same Jenkins server
run this docker image as it was intended by @cdrx  - instead of docker inside use shell as in sh "docker run -v $(pwd -P):/src cdxr/pyinstaller...
most proper way in my opinion - derive your own docker image starting with FROM cdxr/pyinstaller-linux:python3 and then apply whatever customizations you need to entrypoint / command

Thanks for the info. It looks like there are still complaints that it's not fixed even though Jenkins folks consider it fixed. I'm going to go with your third option. If there's a way to close this ticket while still having it be searchable by others who are following the Jenkins Python Pipeline tutorial, that would be best.

Have you solved this problem?


Data augmentation layers generating multiple images

@lakshmanok, i have recently read you article on writing a custom augmentation layer (this tutorial) and wondered if you have any ideas on how to handle augmentation layers that generates multiple images. I have implemented one model to do so but i don't know how to associate the original labels to the corresponding generated images which leads to mismatches between logits and labels when training the model.
Code in the link below:
https://stackoverflow.com/questions/68136779/logits-and-labels-must-be-broadcastable-data-augmentation-layers-makes-logits-a
Any help and suggestions are welcome.

You would have to do this in the ingest pipeline.
Any Keras layer well have to keep the batch size the same as the input, so it's not possible as a Keras layer.

I understand, thank you very much for your time.


Changed noirjs_app.md for v.30.0

Description
Simple md updates.
Problem*
Resolves
Summary*
Additional Context
Documentation*
Check one:

[ ] No documentation needed.
[ ] Documentation included in this PR.
[ ] [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist*

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Thank you for the PR! However, we have updated to the latest versions here so I am going to close this.

Thank you for the PR! However, we have updated to the latest versions https://github.com/noir-lang/noir/pull/5652 so I am going to close this.


fixed the responsive example

spriteMiddle not defined and uses game variable game instead of this.game was breaking the entire example

Nice :+1:


Failed to download Valheim server from Steam

Hi,
when running docker image sha256:e51af3ae784d4976978347df77028aefbe5624292dbc97676c782e66b61fd4f8 following error occurs during startup.
May 24 17:45:49 /supervisord: valheim-updater ERROR - Failed to download Valheim server from Steam - retrying later - check your networking and volume access permissions

Mounted my configs and savegames, even disabled my firewall, and started using:
docker run -d \
    --name valheim-server \
    --cap-add=sys_nice \
    --stop-timeout 120 \
    -p 2456-2457:2456-2457/udp \
    -v $HOME/valheim-server/config:/config \
    -e SERVER_NAME="foo" \
    -e WORLD_NAME="bar" \
    -e SERVER_PASS="secret" \
    ghcr.io/lloesche/valheim-server

Set permissions for /config to a=rwx recursively, still does not work.
/config owner is my local root user.
I'm on CentOS 7.
Edit:
Mounted server with -v $HOME/valheim-server/data:/opt/valheim, error still occurs.

Was caused by my local docker setup, container wasn't able to reach the internet.


Update gems

What this PR does / why we need it:
Updated gems, in particular, the nokogiri gem.
Release note:
NONE

Build for netlify is failing on the jekyll build command:
9:53:02 AM: Activating bundler (2.0.2) failed:
9:53:02 AM: Could not find 'bundler' (2.0.2) required by your /opt/build/repo/site/Gemfile.lock.
9:53:02 AM: failed during stage 'building site': Build script returned non-zero exit code: 42
9:53:02 AM: To update to the latest version installed on your system, run `bundle update --bundler`.
9:53:02 AM: To install the missing version, run `gem install bundler:2.0.2`
9:53:02 AM: Checked in 'GEM_PATH=/opt/buildhome/.rvm/gems/ruby-2.6.2:/opt/buildhome/.rvm/rubies/ruby-2.6.2/lib/ruby/gems/2.6.0', execute `gem env` for more information
9:53:02 AM: To install the version of bundler this project requires, run `gem install bundler -v '2.0.2'`

This command jekyll build is specified in the netlify.toml file. @jonasrosland do you know if we need to add extra steps there or is there some other route we should pursue to get netlify working again? I can locally run this just fine.

@jonasrosland found this is a known issue: https://github.com/netlify/build-image/issues/296


Correct the link for a paper

The old url links to paper NFVnice: Dynamic Backpressure and Scheduling for NFV Service Chains not for A seamless virtualized network functions migration mechanism in mobile edge networks.

Thanks for your correction!


Add proper Home Manager module output (Updated #113)

Hey, I took care of the requested change in #113 but don't had permission to update the MR so why I created this 🙏🏻 Let me know if you would like something different - rebased on latest master.

@talyz Any chance we can get this merged? 🙏🏻

Looks good to me! Can you please rebase on the latest master? I think merging now would create duplicate entries.

Looks good to me! Can you please rebase on the latest master?
Done 🙏🏻

@talyz Can we get this merged? ✌🏻


Added missing ports declaration.

Debug.xml has more ports than others XML files.

Thanks. I usually update production configs just before an official release.