Consider using well-known pbkdf in crypto/entropy

From the source:
// Increase entropy by multiple hashing.
func (p *sEntropyBooster) BoostEntropy(pData []byte) []byte {

The comment is misleading. This function does not and cannot increase entropy.
IIUC, this function is a password-based key derivation. That is it derives key-material from a password and the salt.
It is important to only feed passwords as a key. Feeding a proper key material to this function might reduce security. So it might be beneficial to rename the parameter to password.
And last but not least, instead of using a variation PBKDF1, it might be better to use a modern algorithm, like argon, scrypt, or at least bcrypt. What is your position on this?

The comment is misleading. This function does not and cannot increase entropy.

It can increase. When designing the entropy booster, I was guided by Chapter 22 (Secret Storage), section 2.1 (Salt and Stretch) from the book "Practical Cryptography" (N. Ferguson, B. Schneier):

What kind of winnings did we get? If r = 2^20 (this is a little more than a million), then the attacker will have to perform 2^20 hashing operations to verify each password option. Iterating over 2^60 passwords requires 2^80 hashing operations, so using r = 220 is equivalent to increasing the password size by 20 bits. The greater the value of r, the greater the gain in the size of the password we get.

Feeding a proper key material to this function might reduce security.

I don't quite understand what exactly can reduce security when transferring a non-password? If we consider a certain randomness with an entropy equal to N, then IEntropyBooster will increase this parameter by +X (of course, taking into account that N will not exceed the size of the hash function itself).

And last but not least, instead of using a variation PBKDF1, it might be better to use a modern algorithm, like argon, scrypt, or at least bcrypt. What is your position on this?

Indeed, it would be possible to use one of these algorithms. The reason why I didn't do it was:

simplicity of implementation, where the vulnerability can be hidden in only 10 lines of code;
Scrypt similarity algorithms make calculations more complex when using FPGAs and their like, equating them to more or less one computing resource. But at the same time, scrypt does not change in any way and does not make the algorithmic complexity of the search itself less.

However, there really are a number of problems in my choice that the same scrypt can solve. For example, the size of the resulting block. Now it is equal to the SHA256 block size. Also, protection from FPGA will be quite a plus. Therefore, I think I will really rewrite the crypto/entropy implementation soon.

It can increase. When designing the entropy booster, I was guided by Chapter 22 (Secret Storage), section 2.1 (Salt and Stretch) from the book "Practical Cryptography" (N. Ferguson, B. Schneier):

The entropy is a measure of true randomness, not computational complexity. Entropy can be coarsely defined as "what is the expected number of tries one needs to guess the password" (specifially a log2 of it).
So a non-random function H(k) cannot increase entropy. Moreover, if H(k) is not injective (has collisions), it will reduce the entropy (we have fewer choices). I assume that salt is public and cannot be accounted as a source of randomness.

I don't quite understand what exactly can reduce security when transferring a non-password?

Password-based kdf are desinged to be fed with low-entropy keys. Due to collisions pb-kdf might reduce entropy. This is not a concern for passwords as the possibility is neglible.
High-entropy key material has pretty high possibility of hitting collisions, and thus pb-kdf will substantially reduce entropy of KM.

Scrypt similarity algorithms make calculations more complex when using FPGAs and their like, equating them to more or less one computing resource. But at the same time, scrypt does not change in any way and does not make the algorithmic complexity of the search itself less.

Neither pbkdf change algorithmic complexity. Pbkdfs are specifically designed as a band-aid to slow down bruteforce attack. Algorithmically all pbkdfs have constant complexity (assuming fixed output size and cost hyperparam). They only differ in performance by a constant factor.

.... The reason why I didn't do it was:

simplicity of implementation, where the vulnerability can be hidden in only 10 lines of code;

If implementation simplicity is a concern, then you can use pbkdf2. It is similar to your impl, but uses a proper HMAC and has well-studied security.
And to state the common truth: do not roll your own crypto.

Translated the implementation to pbkdf2. Also changed the naming. The amendments are in the commit: https://github.com/number571/go-peer/commit/533c45d99e5dc829c4d13d815360614c505c428d. Thanks for pointing out the problem


Simple Event Replaying mechanism

Do NOT USE yet.

Deviated too much from develop, shared functionality was developed in another branch, so making a fresh PR, closing this one.


Skipping DASH manifest: ExtractorError(u'Cannot decrypt signature without player_url; please report this issue on https://yt-dl.org/bug

see also #4430 and #4398 for similar results
BUT
for those two the error message wasn't generated  when using "the --verbose flag"
and
I'm using youtube-dl  2<IP_ADDRESS>
and
I have --write-annotations switched in from ~/youtube-dl.conf
Error message on download from youtube . . . .
WARNING: [youtube] dR1_YrC4f6Y: Skipping DASH manifest: ExtractorError(u'Cannot decrypt signature without player_url; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.',)
Error message  requests "please report this issue on https://yt-dl.org/bug"
A search on github.com shows two similar with previous youtube-dl versions . .
https://github.com/rg3/youtube-dl/search?q=Skipping+DASH+manifest&type=Issues&utf8=✓
NOTE
1/ the .mp4 and the annotations .xml files both arrive intact despite the error message
2/ if a .part file (or the full mp4) exists then I don't see the Error message
3/ when I made a test directory and tried a few downloads of this same file, deleting previous downloads before each run the error message stopped ocurring after a few runs see last paragraph
So . . . .
python youtube-dl.py -U
youtube-dl is up-to-date (2<IP_ADDRESS>)

(Python version 2.7.6 - Linux-3.11.10-25-desktop-x86_64-with-SuSE-13.1-x86_64)
And running  --verbose  as requested . . .
python youtube-dl.py --verbose https://www.youtube.com/watch?v=dR1_YrC4f6Y
[debug] System config: []
[debug] User config: ['--write-sub', '--no-overwrites', '--write-annotations', '--all-subs', '--title']
[debug] Command-line args: ['--verbose', 'https://www.youtube.com/watch?v=dR1_YrC4f6Y']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2<IP_ADDRESS>
[debug] Python version 2.7.6 - Linux-3.11.10-25-desktop-x86_64-with-SuSE-13.1-x86_64
[debug] exe versions: ffmpeg 2.1.4, ffprobe 2.1.4, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] dR1_YrC4f6Y: Downloading webpage
[youtube] dR1_YrC4f6Y: Extracting video information
WARNING: video doesn't have subtitles
[youtube] dR1_YrC4f6Y: Searching for annotations.
WARNING: [youtube] dR1_YrC4f6Y: Skipping DASH manifest: ExtractorError(u'Cannot decrypt signature without player_url; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.',)
[info] Writing video annotations to: MFJ CUB - 40m QRP Kit (ARRL Package)-dR1_YrC4f6Y.mp4.annotations.xml
[debug] Invoking downloader on u'https://r11---sn-ab5l6nld.googlevideo.com/videoplayback?signature=74B6CCB1972672F3B054C46A53D362503A85872F.2508F6CDE2DAB351408E3FADDF293371B0EF758B&ip=my.ip.address.here&mm=35&id=o-AB_puRRIsM1pbtOYzHOrTp7u_WrP-nj92LoRB0kSnG3f&fexp=900718%2C912134%2C927622%2C931342%2C932404%2C9405735%2C9406065%2C943917%2C947209%2C947218%2C948124%2C948528%2C952302%2C952605%2C952901%2C955301%2C957103%2C957105%2C957201&sparams=dur%2Cid%2Cip%2Cipbits%2Citag%2Cmime%2Cmm%2Cms%2Cmv%2Cratebypass%2Crequiressl%2Csource%2Cupn%2Cexpire&upn=bnmIs89BbTQ&itag=18&ms=pm&mv=s&dur=502.177&source=youtube&mime=video%2Fmp4&requiressl=yes&ipbits=0&sver=3&key=yt5&expire=1419789225&mt=1419765566&ratebypass=yes'
[download] Destination: MFJ CUB - 40m QRP Kit (ARRL Package)-dR1_YrC4f6Y.mp4
[download] 100% of 28.09MiB in 00:47
BUT
after removing the downloaded files and re-running a few times I eventually see a different googlevideo url and no error message . . .
python youtube-dl.py https://www.youtube.com/watch?v=dR1_YrC4f6Y --verbose
[debug] System config: []
[debug] User config: ['--write-sub', '--no-overwrites', '--write-annotations', '--all-subs', '--title']
[debug] Command-line args: ['https://www.youtube.com/watch?v=dR1_YrC4f6Y', '--verbose']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2<IP_ADDRESS>
[debug] Python version 2.7.6 - Linux-3.11.10-25-desktop-x86_64-with-SuSE-13.1-x86_64
[debug] exe versions: ffmpeg 2.1.4, ffprobe 2.1.4, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] dR1_YrC4f6Y: Downloading webpage
[youtube] dR1_YrC4f6Y: Extracting video information
WARNING: video doesn't have subtitles
[youtube] dR1_YrC4f6Y: Searching for annotations.
[youtube] dR1_YrC4f6Y: Downloading DASH manifest
[info] Writing video annotations to: MFJ CUB - 40m QRP Kit (ARRL Package)-dR1_YrC4f6Y.mp4.annotations.xml
[debug] Invoking downloader on u'https://r1---sn-3jvh-ajte.googlevideo.com/videoplayback?id=o-AMTc__Oeu27Sa2wbL_yj0OqR2nI_6BsCmwevgwN5Gc8_&upn=Nor6wcGVxEE&sparams=dur%2Cid%2Cinitcwndbps%2Cip%2Cipbits%2Citag%2Cmime%2Cmm%2Cms%2Cmv%2Cratebypass%2Crequiressl%2Csource%2Cupn%2Cexpire&fexp=900234%2C900718%2C901803%2C912820%2C916644%2C927622%2C932404%2C939975%2C943917%2C947209%2C947218%2C948124%2C952302%2C952605%2C952901%2C955301%2C957103%2C957105%2C957201&expire=1419794668&itag=18&ipbits=0&ip=<IP_ADDRESS>&mime=video%2Fmp4&source=youtube&ms=au&mt=1419773036&mv=m&requiressl=yes&sver=3&signature=3A63B61BA982400CCDAF5912BA3D924E7829A71E.8507D2E6373E084307EB936F43BFCA0FCDF5BDCC&ratebypass=yes&initcwndbps=750000&dur=502.177&mm=31&key=yt5'
[download] Destination: MFJ CUB - 40m QRP Kit (ARRL Package)-dR1_YrC4f6Y.mp4
[download] 100% of 28.09MiB in 00:34
Any use ???
I don't know anything about python (or googlevideo) so I can't offer any insight . .
cb

Works fine with latest version (2016.05.16).


Travis-CI: update caching stuff for sbt 1.3

I verified at https://travis-ci.org/lightbend/scala-sculpt/jobs/584687173 that when the job ran a second time, the cache remained unchanged


Blurry output

I have installed the project on windows and its working, but I noticed while my input images are both high resolution images, the swapped output face is quite blurry. It doesn't look similar to the example images. Is this a known issue, or is it is a matter of configuration? Any thoughts would be helpful.

There are also 2 errors shown, maybe this gives a clue:
sberswap\lib\site-packages\torch\nn\functional.py:3000: UserWarning: The default behavior for interpolate/upsample with float scale_factor changed in 1.6.0 to align with other frameworks/libraries, and uses scale_factor directly, instead of relying on the computed output size. If you wish to keep the old behavior, please set recompute_scale_factor=True. See the documentation of nn.Upsample for details.   warnings.warn("The default behavior for interpolate/upsample with float scale_factor changed "
and
'sberswap\lib\site-packages\kornia\augmentation\augmentation.py:1833: DeprecationWarning: GaussianBlur is no longer maintained and will be removed from the future versions. Please use RandomGaussianBlur instead.
category=DeprecationWarning,'

These warnings are irrelevant. To improve the image quality, you can enable 'use_sr' option which will call face_enhancement. However, the pix2pix super resolution is not good, I'm working on an alternative model.

I tried use_sr but it didn't make and huge difference. Still somewhat blurry and suboptimal.
What alternative model are you looking at, something like GPEN/GFPGAN?

Hi, @epicstar7! The output face may look blurry because our model only generates a 256x256 resolution. To solve this problem, you can look at the face enhancement function. If you need a stronger super resolution model, check out sber-swap implementation with the GFPGAN sr model SberSwapInferenceGFPGAN.ipynb. It scales the output images up to 512x512 and the faces look more detailed. However, in this case, the inference time is longer.

@epicstar7 check inference_GFPGAN.py:
python inference_GFPGAN.py --source_paths examples/images/beckham.jpg --target_video examples/videos/nggyup.mp4 --use_sr True

You need to clone GFPGAN repository, download weights GFPGANCleanv1-NoCE-C2.pth and place to GFPGAN/experiments/pretrained_models/ folder

Thanks I tried but I get the following error:
from utils.inference.image_processing import crop_face, get_final_image, get_final_image_gfpgan, show_images ImportError: cannot import name 'get_final_image_gfpgan'
I suspect you need to commit an updated image_processing.py

This works mate, amazing job, this is a great improvement to this repo. Thanks!

I do get an error CUDA out of memory when trying to swap on video. I'm curious to know why this happens for videos while for single images there is no issue. This shouldn't happen right?

@epicstar7, try to reduce batch size, it might help.

This works, changed it with -batch_size 16 and now no CUDA memory error any more. But the face is not swapped in the output video!

Managed to get it working using a targetimage of the face. I noticed though swapping is sometimes failing after just a few frames in the video. You see the swapped face for a few frames and then it just returns to the original pictures. Any thoughts why this happens?


Pages Language support and tools page has broken tabs

Example URL(s)
https://developers.cloudflare.com/pages/platform/language-support-and-tools/
Actual Behavior
When you first load the page, both v1 and v2 tabs are active on the tools,  frameworks, and environment tables:

Further, the v2 tab does not work when you click the "beta" tag - only when you click the v2 text does it activate

Expected Behavior

Only the v2 tab should be highlighted on page load.
When you click any part of the v2 tab button, it should switch to the v2 tab.

Additional information
No response

Howdy @GregBrimble, I believe this is b/c both tabs are in the active state on page load.
Appears like the Build Environments tabs further down the page works correctly.


nuBASIC

https://github.com/eantcal/nubasic

Prebuilt binaries on sourceforge nubasic-ide-1.51-setup_x64.zip extracted as setup-vs64.exe (so it seems require Visual Studio?), virustotal said setup-vs64.exe is safe.

Why it's 64 bit but install to Program Files (x86)?
Default path to install by setup: C:\Program Files (x86)\nuBASIC (x64)

Generic link to awesome-basic curated list added.


WIP: buildGoPackage: Dont copy sources to $out

Motivation for this change
I don't see the reason to copy sources that are potentially quite large, especially with the go community practices of vendoring dependencies.
Do these sources serve any purpose?
Things done

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

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

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

CCing maintainers & others who have touched this recently @pbogdan @Mic92 @yegortimoshenko @kamilchm @lethalman @cstrahan @orivej @velovix

I believe it was meant for Go packages to depend on each other, but this was never realized. I think it is good to remove sources from $out.
This should be checked with nox-review.

@orivej As I suspected then. I'm running nox-review on this now.

nox-review passes except for a few packages (gopherclient and goa) which were already broken since before.

Failure on aarch64-linux (full log)
Attempted: go-ethereum
Partial log (click to expand)
github.com/ethereum/go-ethereum/eth/downloader
github.com/ethereum/go-ethereum/internal/ethapi
github.com/ethereum/go-ethereum/miner
github.com/ethereum/go-ethereum/eth/tracers
github.com/ethereum/go-ethereum/eth
github.com/ethereum/go-ethereum/les
github.com/ethereum/go-ethereum/ethstats
github.com/ethereum/go-ethereum/cmd/faucet
builder for '/nix/store/kcvg0gjwzfj32ihbxqyw7bkix31s4j1k-go-ethereum-1.8.11.drv' failed with exit code 45
error: build of '/nix/store/kcvg0gjwzfj32ihbxqyw7bkix31s4j1k-go-ethereum-1.8.11.drv' failed

Success on x86_64-linux (full log)
Attempted: go-ethereum
Partial log (click to expand)
shrinking /nix/store/lzrir76gyihkax7hgncd86h9nk0baxid-go-ethereum-1.8.11-bin/bin/wnode
shrinking /nix/store/lzrir76gyihkax7hgncd86h9nk0baxid-go-ethereum-1.8.11-bin/bin/p2psim
shrinking /nix/store/lzrir76gyihkax7hgncd86h9nk0baxid-go-ethereum-1.8.11-bin/bin/bootnode
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/lzrir76gyihkax7hgncd86h9nk0baxid-go-ethereum-1.8.11-bin/bin
patching script interpreter paths in /nix/store/lzrir76gyihkax7hgncd86h9nk0baxid-go-ethereum-1.8.11-bin
checking for references to /build in /nix/store/lzrir76gyihkax7hgncd86h9nk0baxid-go-ethereum-1.8.11-bin...
cannot find section .dynamic
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
/nix/store/lzrir76gyihkax7hgncd86h9nk0baxid-go-ethereum-1.8.11-bin

Success on x86_64-darwin (full log)
Attempted: go-ethereum
Partial log (click to expand)
find: '/nix/store/1k89f7k4g8grk2qrlyfh756c3485hqxf-go-ethereum-1.8.11': No such file or directory
find: '/nix/store/1k89f7k4g8grk2qrlyfh756c3485hqxf-go-ethereum-1.8.11': No such file or directory
find: '/nix/store/1k89f7k4g8grk2qrlyfh756c3485hqxf-go-ethereum-1.8.11': No such file or directory
find: '/nix/store/1k89f7k4g8grk2qrlyfh756c3485hqxf-go-ethereum-1.8.11': No such file or directory
find: '/nix/store/1k89f7k4g8grk2qrlyfh756c3485hqxf-go-ethereum-1.8.11': No such file or directory
find: '/nix/store/1k89f7k4g8grk2qrlyfh756c3485hqxf-go-ethereum-1.8.11': No such file or directory
find: '/nix/store/1k89f7k4g8grk2qrlyfh756c3485hqxf-go-ethereum-1.8.11': No such file or directory
find: '/nix/store/1k89f7k4g8grk2qrlyfh756c3485hqxf-go-ethereum-1.8.11': No such file or directory
find: '/nix/store/1k89f7k4g8grk2qrlyfh756c3485hqxf-go-ethereum-1.8.11': No such file or directory
/nix/store/jyl5f3j8q7cckq4d1xnf4jdg9196dyk3-go-ethereum-1.8.11-bin


[Crash] Server Environment not supported

Legacy4J Version
Legacy4J-1.21-<IP_ADDRESS>4.3 | commit 3f3bda4c0096338fe7852909cec476abf84219a9
Mod Loader Version
NeoForge - 1.21.1 | 21.1.73
Bug/Glitch Occurrence Context
Unable to launch a NeoForge server.
Install NeoForge server environment:
https://projects.neoforged.net/neoforged/neoforge

Install NeoForge Legacy 4J 1.21-<IP_ADDRESS>4.3
Crash Report
N/A - not provided.
What happened wrong?
Server won't launch.
Latest.log:
latest.log

Additional context from Console:
https://gist.github.com/Cixon/d120819b1adeb18b4cd82fbedc006726

Fixed in <IP_ADDRESS>4.4, thanks for reporting.


/claimname issue

When typing /claimname  in a subdivision, it sets the name of the parent claim rather than the claim you are standing in.

The same issue persists when using /deleteclaim


Dynamic Configuration static navigationOptions dont work!!!

Current Behavior

as the document say, i use the dynamic config  to set navigationOtions ,but it dont work! I found it add in 1.0.0-beta.9 , so i change my react-navigation version to it ! the same questions and answerr in stackoverflow

when i  use the object set navigationOptions, it works! but when i use funtion dynamic to add it , it dont not work ! even the static funtion have never run !
Expected Behavior

Your Environment

software
version

react-navigation
1.0.0-beta.9  or 1.0.0-beta.11

react-native
0.44.0

node
4.7.3

npm or yarn
npm

it is my fault ! i set the child of PageView  navigationOptions, so it dont work!


Helpers for page navigation

In the course of writing tests for https://github.com/GoogleChrome/sw-helpers/pull/1, I found it necessary to script page navigations, running a set of tests before and after each navigation. I did this by "dropping down a level" and using the WebDriver directly to perform .get(url1), then later .get(url2), etc. This worked, but I wonder what folks think about providing a generic helper that could automate it a bit.

I imagine in most cases you'd want to perform some kind of action / assertions between each navigation, if that's the case, I'm not sure you can slim down the webdriver code any more than it currently is.
Did you have something in mind?

@jeffposnick please reopen this on selenium-assistant if you think it's still worth exploring.


Convert services tests to NNBD

Description
This converts the packages/flutter/test/services directory to NNBD, now that the services package is converted.
I changed the signature of checkMessageHandler and checkMockMessageHandler on BinaryMessenger to take a nullable handler, since the tests wanted to check to make sure a handler wasn't set, and that functionality no longer works if the handler is non-nullable.
Related Issues

dart-lang/sdk#40146

I think this PR is paused until we can convert the web code to NNBD.

Marked this as WIP for now since it is blocked on https://github.com/flutter/flutter/issues/62886.


Unused/obsolete code in XtermEngine

In the XtermEngine::StartPaint method, there's some code that calculates the dirty area here:
https://github.com/microsoft/terminal/blob/183a8956f699e1967d4cb4712184d0dbc395a0e5/src/renderer/vt/XtermEngine.cpp#L85-L97
However, the result of that calculation is not actually used. There was a time when there was more code in that branch that compared the dirty area against the last viewport, but that was removed in PR #4741.

And in the XtermEngine::_MoveCursor method there's a local variable performedSoftWrap which is initially set to false at the start of the method here:
https://github.com/microsoft/terminal/blob/183a8956f699e1967d4cb4712184d0dbc395a0e5/src/renderer/vt/XtermEngine.cpp#L249-L249
And it's potentially changed to true later in the code:
https://github.com/microsoft/terminal/blob/183a8956f699e1967d4cb4712184d0dbc395a0e5/src/renderer/vt/XtermEngine.cpp#L273-L278
However, the variable is otherwise never read. It looks like it was added as part of PR #5181, but even then it doesn't appear to have been used.
Unless there is something I'm missing, I'd suggest we should remove the unused code.

I'm alright removing it!


Which model should I use for a cell typing task

I have a question regarding the use of your tools, and it would be more convenient to discuss it here. I am currently analyzing a single cell dataset obtained from a mouse neuronal region, with three different treatment conditions and several replicates per condition. I already used scvi which has been helpful in my experience. However, we faced difficulties in identifying neuronal subtypes. Despite the availability of a rich reference for this region (see here), I am considering utilizing your tools to integrate our data with the existing dataset to facilitate cell typing. Could you please advise me on which model to select? Thank you in advance for your assistance.

SCANVI, right?
https://scarchest.readthedocs.io/en/latest/scanvi_surgery_pipeline.html


add comment on insecure mode and link to documentation

registry can run in insecure mode without certificates but it requires some configuration in the nodes. I've added it to the tutorial. This changes the documentation to reflect that:
https://github.com/dcos/examples/pull/16

cc: @mhausenblas on the related Tutorials PR https://github.com/dcos/examples/pull/16


Not receiving notifications on iphone

Hi guys,
I have been trying for several hours to make this working but without success.
Cordova
9.0.0<EMAIL_ADDRESS>IOS Platform
<EMAIL_ADDRESS>I installed cordova-plugin-firebase, added ios platform and then inside platforms/ios I ran pod install it seems it installed all necessary dependencies:
Analyzing dependencies
Downloading dependencies
Using Crashlytics (3.12.0)
Using Fabric (1.9.0)
Using Firebase (6.3.0)
Using FirebaseABTesting (3.0.0)
Using FirebaseAnalytics (6.0.2)
Using FirebaseAnalyticsInterop (1.4.0)
Using FirebaseAuth (6.1.2)
Using FirebaseAuthInterop (1.0.0)
Using FirebaseCore (6.0.3)
Using FirebaseInstanceID (4.2.4)
Using FirebaseMessaging (4.1.1)
Using FirebasePerformance (3.1.0)
Using FirebaseRemoteConfig (4.1.0)
Using GTMSessionFetcher (1.2.2)
Using GoogleAppMeasurement (6.0.2)
Using GoogleToolboxForMac (2.2.1)
Using GoogleUtilities (6.2.5)
Using Protobuf (3.9.0)
Using nanopb (0.3.901)
Generating Pods project
Integrating client project
Sending stats
Pod installation complete! There are 7 dependencies from the Podfile and 19 total pods installed.

Then I opened platforms/ios/*.xcworkspace file and the project opened correctly. Then I ran the build on my iphone and I opened the application and allowed the notifications but then when I tried to send notification to my token or to that ios application using firebase I never received any.
I don't know why it doesn't work nor how to debug it, because I don't know where the error might be.
Also when adding a platform this was in the console as a output and might mean something:
`Installing "cordova-plugin-androidx-adapter" for ios
Running command: pod install --verbose
[!] The T and T kids [Debug] target overrides the LD_RUNPATH_SEARCH_PATHS build setting defined in Pods/Target Support Files/Pods-T and T kids/Pods-T and T kids.debug.xcconfig'. This can lead to problems with the CocoaPods installation
[!] The T and T kids [Release] target overrides the LD_RUNPATH_SEARCH_PATHS build setting defined in Pods/Target Support Files/Pods-T and T kids/Pods-T and T kids.release.xcconfig'. This can lead to problems with the CocoaPods installation
I uploaded the cert to firebase ios project, it's not the first time I am making an app that uses firebase and cordova, but right now I don't know what to do here.

Try building and running the example project and set its package ID to the one for your own app so the provisioning profile is correct and use the GoogleService-Info.plist for your own project. Then use the built-in FCM API client in the test project or the Firebase Console to send test messages. That should help validate your APNS configuration independently of your codebase.

Thanks to the error message I received in terminal when I tried to send a notification using the example project I figure out that I had the wrong cert/Key ID uploaded to my firebase project. Great.


[Bug]: dashboard selectively displays tasks

Before proceeding, is there an existing issue or discussion for this?

[X] I have done a search for similar issues and discussions.

OS and version
Ubuntu 22.04
Open-RMF installation type
Source build
Other Open-RMF installation methods
No response
Open-RMF version or commit hash
main, deploy/hammer
ROS distribution
Humble
ROS installation type
Docker
Other ROS installation methods
No response
Package or library, if applicable
No response
Description of the bug
Automatic charging tasks are not displayed on the dashboard, while charging tasks in the form of task cancellation are shown
Steps to reproduce the bug

Have robots perform normal charging tasks, these do not get displayed on the dashboard
Perform tasks that are going to timeout, with cancellation behavior kicking in
The cancellation task of going back to charging, issued by the fleet adapters are shown on the dashboard

Expected behavior
No response
Actual behavior
No response
Additional information or screenshots
No response

Closed as this was due to whether charging tasks were inserted via automatic retreat. Those will show up on the UI, as opposed to finishing tasks.


Error "cannot use import statement outside a module" with React

Getting an error when importing the module into a React script. Same works fine in a Node app or plain HTML. Does anyone have a fix?

This is because this module ships as native modules and the React ecoystem is some what legacy in this respect,
I've got a PR to try to address this but i havent had time to test it properly, and would need to be a breaking change.
https://github.com/Georgegriff/query-selector-shadow-dom/pull/53/files
If you want to experiment with this you should be able to change your package.json entry for this package to:
"query-selector-shadow-dom": "git://Georgegriff/query-selector-shadow-dom.git#consider_essm"
If you have any time to help contribute/test t would be appreciated

Cheers, will look into it and definitely contribute if I come up with a solution :)

There isn't I would be open to contributions I started a pr but didn't get around to following it up, not sure when I will next get a chance to investigate https://github.com/Georgegriff/query-selector-shadow-dom/pull/53

Thanks for your time following up with me - I really appreciate it. Sorry, I thought the PR above was fully finished.

This is because this module ships as native modules and the React ecoystem is some what legacy in this respect,
I've got a PR to try to address this but i havent had time to test it properly, and would need to be a breaking change.
https://github.com/Georgegriff/query-selector-shadow-dom/pull/53/files
If you want to experiment with this you should be able to change your package.json entry for this package to:
"query-selector-shadow-dom": "git://Georgegriff/query-selector-shadow-dom.git#consider_essm"
If you have any time to help contribute/test t would be appreciated

Could you tell me the mean of "change your package.json entry"?

Bump on this -- also running into this issue!


Add ends_with filter and test filter statements

Add support the 'ends_with' filter, and add tests for it.
I found that assertSameEntity ignores the 'id' value if the type of the entities matches. I fixed this and added tests for assertSameEntity and assertNotSameEntity. This includes a fix for my previous test that was incorrect.
I added basic tests for all the existing filters.

Spectacular. Thank you for pushing these my way!


Add xml support for getDestinations

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: petar-iv:x: alexpenev-sYou have signed the CLA already but the status is still pending? Let us recheck it.

LGTM ✅
Don’t forget that you need to open a PR in compass repo to bump the PR number of ord-service in the chart’s values.yaml with the number of this PR.
Otherwise, the change won’t be part of the next release.


Deduplicate-stub-game-arena-elements-in-tests

Issue 286 related. for start, limiting the imports

@Mikolaj the purpose of this draft PR is to see if your OK with limiting the imports.  it might be helpful to follow the dependencies, and perhaps additional self documentation.

Yes, this is useful. Some remarks:
Although https://github.com/LambdaHack/LambdaHack/blob/master/LambdaHack.cabal#L32-L33 convention does not hold for tests, I'd still rather import Game.LambdaHack.Client and then some Game.LambdaHack.Client.* for whatever is missing, rather than only Game.LambdaHack.Client.*.
I think it's an overkill to both import qualified and list imported entities, as in
import qualified Game.LambdaHack.Core.Dice as Dice (d, AbsDepth (..))

The few modules imported qualified are usually not designed to be imported unqualified (e.g., they have 1-letter identifiers or dozens of short identifiers that benefit from the module name prefix), so let them remain qualified.
BTW, if you refer to the issue as #283, it becomes a link and also shows up as a mention in the issue to which your refer.

The CI runs with build with cabal build -f-release are failing, because some functions are apparently used that are in inside the
#ifdef EXPOSE_INTERNAL
CPP.

Perhaps we should set -f-release in cabal.project.development so that we get the errors early.

Let me have a look.


Using a lazy_static in its own initializer doesn't result in an error

I refactored some code I had that was creating a HashSet from a const slice to use lazy_static instead. I renamed the const and used its original name for the static, and then copy/pasted the initialization and forgot to rename the variable it was using, so the static wound up trying to initialize by iterating over itself. Surprisingly (to me), the code compiled anyway!
Here's a simple reduced testcase:
https://play.rust-lang.org/?gist=d77058ab6da1a4e7815fab94305f2092&version=stable
Running this will deadlock and time out on the Playground.
I realize that this is probably hard to fix within the constraints of what Rust macros can do, but it's pretty unfortunate!

Yeah, that's a unfortunate error scenario - I'm not sure if we can do anything about it.
Maybe an extra boolean flag for a recursion check during initialization, but even for debug mode I'm not sure if thats to heavy-handed.

Is this sound? Is Once reentrance-safe and blocking in this case, guaranteeing a deadlock?

Hello, I think I ran into the same problem with this example:
lazy_static::lazy_static! {
    static ref FOO: [usize; 2] = [0, FOO[0]];
}

fn main() {
    println!("{}", FOO[0]);
}

The code compiles fine but hangs.

The macro could shadow the name with another definition, but I can't think of any way to cause a compiler error only on use. At most it could do something like this:
struct RecursiveUseWithinInitialierOfFOO;
const FOO: RecursiveUseWithinInitialierOfFOO = RecursiveUseWithinInitialierOfFOO;

And then the type errors might be a decent enough hint.
However, there's not really any way I can think of to stop code outside of the initializer expression, from using it, and getting called from the initializer, other than some kind of attribute that makes the compiler warn if the <typeof(FOO) as Deref>::deref method can ever call itself (that looks through the entire callgraph involving Once::call_once and whatnot - a bit like the "unconditional recursion" lint, but more powerful).


Adds a host-config for the rocm+hip compiler on LLNL's rznevada cluster

Partial fulfillment of #534
For this host-config, I used the rocm compilers for C and C++, but the CCE compiler for Fortran.
I tested this using<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

Looks fine by me.
Thoughts on using cmake-3.19+ presets for these things at some point?

Major revamp of HIP and CUDA support coming very soon.


removing duplicate YAML keys

@PatMyron : Thanks for your contribution! The author, @, has been notified to review your proposed change.

Verified editorial corrections in YML content.

@PatMyron  Hi Pat,
Would you follow the instructions in the Contributor Guide to Link your GitHub and Microsoft accounts? Doing so will help us to verify you as a Microsoft employee when you make contributions in the public repo.
Also, keep in mind that Microsoft authors should make updates to content through the private repository so the updates can be staged and validated by the validation rules. See this article for more information.
Thanks


WSO2 service lb for APIM

This change is

recomputing cla status...

[APPROVALNOTIFIER] The Following OWNERS Files Need Approval:

service-loadbalancer

We suggest the following people:
cc @bprashanth
You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancelin a comment


[Bug] No --fields option for temporal workflow list

What are you really trying to do?
Trying to list workflows customizing displayed fields, specifically showing the HistorySizeBytes field
Describe the bug
I’m using Temporal CLI 1.1.1 (the latest version) on MacOS (I also got the same on Linux)  and when I try to use the --fields option for temporal workflow list as documented here, it appears it is not a valid option and I get the following output. What’s going on? On an older version of the CLI (0.11.0) this option works fine
Minimal Reproduction

Run temporal workflow list --fields 'Type.Name,HistorySizeBytes'
Output will be: Error: unknown flag: --fields  and then the command help which does not mention --fields

Environment/Versions

OS and processor: M1 Mac as well as Linux Intel
Temporal Version: CLI 1.1.1
MacOS installed via brew, Linux downloaded binary

Hi @nathanielobrown, unfortunately, this is intentional—the --fields option was removed in 0.12 as part of a major house-cleaning and rewrite we did across the entire CLI.
We recommend people use -o json and use a tool like jq to parse the JSON output instead. The reason we went this route is the JSON output format is guaranteed to be stable, while the text output format (including field names and such) may change without notice. Also, maintaining --fields across a whole bunch of commands added a lot of complexity, and with JSON available, we didn't feel there was a strong enough reason to keep --fields.
That said, if there's a compelling reason why JSON won't work for you and you need the --fields option brought back, let us know, we're always willing to consider it!
As for the docs site still showing --fields, apologies for that—this is a known issue we're working on. We still have some internal tooling work to do to make sure the docs site is kept in sync with the latest CLI release. For now, the --help output is authoritative.
Hope this helps!

Ah, extracting the history_size_bytes field from JSON was the missing link for me! I guess I mistakenly assumed that only the fields specified would be included in the JSON option. Thank you for explaining

Sure thing! I'll go ahead and close this but don't hesitate to reach out again if you need anything else!


⚠ Turnos 24x7 has degraded performance

In 2fe4db7, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 954 ms

Resolved: Turnos 24x7 performance has improved in 87f35ce.


allow customization of CoverletOutput output name as to avoid overwrites when redirecting output to another path

https://github.com/tonerdo/coverlet/blob/c04fc782af3a77426e449f2b33c4c36fa738dd28/src/coverlet.msbuild/coverlet.msbuild.props#L7
^^ should have a condition as to not overwrite user provided values.

@tonerdo : I put in a PR for this already, just in case you want to accept it ;-)

New release here: https://www.nuget.org/packages/coverlet.msbuild/2.0.0


compose-bom 2024.02.00 breaks hold and drag functionality of the slider

Summary:
Upgrading to latest compose-bom (2024.02.00) seem to partly break the slider and no longer allows holding and dragging the progress bar.
Steps to Reproduce:

Swap the compose-bom version here with latest.
Build and run the project.
Try to hold and drag the progress bar to seek.

Expected:
Seek bar should function as expected when holding and dragging.
Actual:
Holding and dragging the seek bar doesn't work on the latest compose-bom. Short touching a certain position on the slider to seek still functions as expected though.
Reproduction Rate:
10/10.
Device:
Tested on Pixel 5 (Android 14)
https://github.com/THEOplayer/android-ui/assets/74964451/2fa8dee5-47a2-4aaa-86a6-8d1d50880e32
https://github.com/THEOplayer/android-ui/assets/74964451/2e8b47e7-1b7f-4096-b92a-e203042630b8

Landed in 1.5.0.


Updated Documention: Task examples for Browser.Dom.focus were pointing to the wrong location

The Task examples that used Browser.Dom.focus pointed to the wrong location. They also used the wrong Error type so I updated all of these to point in the correct location and use the correct modules, functions, and types.

It looks like this got fixed by https://github.com/elm/core/commit/e2b4e514cc3eab1bfeafa75b75acc60398076819 in response to https://github.com/elm/core/issues/997
So if it's not already good on the website, it will be when 1.0.3 comes out with some miscellaneous bug fixes soon.


Chrisdats eflowgpudocs update 2

Add support for Intel iGPUs
Add support for Windows 10 21H2, and updated Windows support wording to match the main page
Clarified that user must specify correct GPU passthrough method for deployment to be successful

@chrisdats : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Looks good. Thanks @chrisdats
#sign-off

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi @kgremban I made some edits for Microsoft style. Before I merge, could you verify that this versioning/branding edit is correct?

Hi @chrisdats - This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. We'll merge this PR, but would you make future content updates in the private repo? Let me know if you need more information about this. Thank you!

Hi @ktoliver In terms of the Windows Server version, that's not exactly correct. I suggest we break it up into two bullet points then:

Windows Server 2019, build 17763 with all cumulative updates installed
Windows Server 2022

Apologies for not using the correct repo. Will do so next time.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks @chrisdats. Fixed; I'll merge now.


Track Runme runs (logged into Stateful-only)

Track runme cell run events and save it to the Cloud database only when the user is logged in.
Some type narrowing clean up to avoid casting

Depends on https://github.com/stateful/platform/pull/1213

I'll look at it over the weekend.


Hack/ script needs to get the right kubeconfig

Looks like the docker ip is returned by: (update --internal is the solution)
EOF
    kind get kubeconfig --name ${E2E_CLUSTER_NAME} > "${E2E_ARTIFACTS}/kubeconfig.conf"
    export KUBECONFIG="${E2E_ARTIFACTS}/kubeconfig.conf"
    echo "cluster is up"
    echo -e "Let's move on.\n"
}

KPNG cant access this bc obviously its not an internal IP.
Whats the idiomatic way to program an extenral APIServer endpoint into a kind CI script ?

Kpng core logs:
E1205 10:37:47.086447    1 reflector.go:138] k8s.io/client-go@v0.22.2/tools/cache/reflector.go:167: Failed to watch *v1.Service: failed to list *v1.Service: Get "https://<IP_ADDRESS>:58003/api/v1/services?labelSelector=%21service.kubernetes.io%2Fheadless%2C%21service.kubernetes.io%2Fservice-proxy-name&limit=500&resourceVersion=0": dial tcp <IP_ADDRESS>:58003: connect: connection refused
E1205 10:37:58.618212    1 reflector.go:138] k8s.io/client-go@v0.22.2/tools/cache/reflector.go:167: Failed to watch *v1beta1.EndpointSlice: failed to list *v1beta1.EndpointSlice: Get "https://<IP_ADDRESS>:58003/apis/discovery.k8s.io/v1beta1/endpointslices?limit=500&resourceVersion=0": dial tcp <IP_ADDRESS>:58003: connect: connection refused
E1205 10:38:02.434729    1 reflector.go:138] k8s.io/client-go@v0.22.2/tools/cache/reflector.go:167: Failed to watch *v1.Node: failed to list *v1.Node: Get "https://<IP_ADDRESS>:58003/api/v1/nodes?limit=500&resourceVersion=0": dial tcp <IP_ADDRESS>:58003: connect: connection refused
E1205 10:38:03.977163    1 reflector.go:138] k8s.io/client-go@v0.22.2/tools/cache/reflector.go:167: Failed to watch *v1.Service: failed to list *v1.Service: Get "https://<IP_ADDRESS>:58003/api/v1/services?labelSelector=%21service.kubernetes.io%2Fheadless%2C%21service.kubernetes.io%2Fservice-proxy-name&limit=500&resourceVersion=0": dial tcp <IP_ADDRESS>:58003: connect: connection refused

https://github.com/kubernetes-sigs/kpng/blob/master/hack/kubeconfig.conf <-- what we used to use
/assign @nehaLohia27
/assign @FriedrichWilken

https://github.com/kubernetes-sigs/kpng/pull/148 is addressing this issue.
(can you please allow me to link PRs to issue?)

@jayunit100 Can you also allow me to link PRs to issues ?

Issue is fixed. closing it.


WIP: Allow artifact names in bst shell

Now that we cache the full tree when builds fail, it becomes more interesting to have a buildshell on a failed artifact without needing to reproduce the project data.
Fixes #1711

I've added some more testing to this branch (locally), as I was trying to see what happens when you try bst shell <artifact-name>, instead of using a build tree.
But I encountered:
Error launching shell: Error: '--deps run' is not supported for artifact names

This is correct because we don't encode runtime dependencies into artifacts (which necessary by design), so I conclude that we cannot support shelling into artifact names except for when running build shells.
I will add a more convenient user facing error in the case the user has attempted to launch a shell on an artifact name without specifying --build and --use-buildtree.

Updated, added a new test to ensure the correct error messaging when the user tries to use bst shell with artifact names but not using build trees.
$ bst shell test/autotools-amhello/847d23cd0e61c54a6beca9071b64643388ad2ab783191358b2a499fe77e86563 -- hello

Error launching shell: Only buildtrees are supported with artifact names

    Use the --build and --use-buildtree options to shell into a cached build tree


Satyampsoni link formatting

Fixes
Fixes #47 by @satyampsoni
Description

This pull request sets the link in the format of another links in the repository.
Technical details

The link  contains custom google search legal path tools.
Tests

Screenshots

Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main or master).
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[x] I added or updated tests for the changes I made (if applicable).
[x] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin

For the purposes of this DCO, "license" is equivalent to "license or public domain dedication," and "open source license" is equivalent to "open content license or public domain dedication."

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Apologies!
I thought that might look the file good.


Crash on server load

Information

Minecraft Version: 1.16.2
Fabric API Version: fabric-api-0.18.0+build.397-1.16
OS: Ubuntu
Other installed mods:
appliedenergistics2-fabric-8.1.0-alpha.1.
astromine-1.6.1+fabric-1.16.2
dynmap-fabric-0.2.12+1.16.2
fabric-chunkpregen-0.2.2
facility-mc116-0.11.184
gunpowder base 0.2.9
all the other gunpowder modules
currency
market
teleport
utilities

** what happened **
server boot up.. no go brrrrrr
logs
https://controlc.com/8250e624
To Reproduce
run a server
Expected behavior
server boot up... go brrrrrr

Install PlayerAbilityLib


About the dataset centroid initialization

About two hours passed，the ‘Initialize source-domain class centroids in the hybrid memory’ is still there.
Is this normal?
the nvidia-smi screen shot（it seem like normal）：

target-domin question：

It's abnormal. Did your GPUs work normally? Try to check it by nvidia-smi.

It's abnormal. Did your GPUs work normally? Try to check it by nvidia-smi.

today the ‘source-domain class centroids in the hybrid memory initialzation’ is very fast，but the  ‘Initialize target-domain class centroids in the hybrid memory’ . please look my new screen shot..

It seems your GPUs did not run since the usage is 0% and the third GPU only occupied 983M. It's abnormal, but I did not meet such an issue in my machines. Try to use one GPU first and check if the problem occurs when communicating between multiple GPUs.

It seems your GPUs did not run since the usage is 0% and the third GPU only occupied 983M. It's abnormal, but I did not meet such an issue in my machines. Try to use one GPU first and check if the problem occurs when communicating between multiple GPUs.

thank u，one GPU did working，i will check the comunicating between multiple GPUs.
i'm sorry to disturb u

It seems your GPUs did not run since the usage is 0% and the third GPU only occupied 983M. It's abnormal, but I did not meet such an issue in my machines. Try to use one GPU first and check if the problem occurs when communicating between multiple GPUs.

emmmm，i trained 1 epoch，then a memory error has occurred， it is normal？ why after one epoch instead of the starting of trainning ..  the screenshot i have updated..

use search_option=3 for compute_jaccard_distance

use search_option=3 for compute_jaccard_distance
ok i will try it ，thank u

use search_option=3 for compute_jaccard_distance
ok i will try it ，thank u

hi， i met the same problem ,have you fix it ?


feat: run saber server on 3001 port if 3000 is unavailable

Summary
Closes #182
What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[x] Feature
[ ] Code style update
[ ] Refactor
[ ] Docs
[ ] Build-related changes
[ ] Other, please describe:

If changing the UI/CSS related code, please provide the before/after screenshot:
Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)

You have tested in the following browsers: (Providing a detailed version will be better.)

[ ] Chrome
[ ] Firefox
[ ] Safari
[ ] Edge
[ ] IE

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature
[ ] Related documents have been updated
[ ] Related tests have been updated

To avoid wasting your time, it's best to open a feature request issue first and wait for approval before working on it.
Other information:

the failling checks doesn't have anything to do with the PR

Nice

In a rare case, I run saber and it automatically "switched" to a new port (I didn't run any other server). But actually it still serves the pages on port 3000. It doesn't bother a lot so I don't open an issue for this.
~\..\..\the-story: master ≣ +1 ~6 -1 | ~5 -0 ! > saber
info Using config file: saber-config.js
info Using theme: ./theme
info Using 20 plugins
success Compiled client successfully in 14s!
info Available at http://localhost:3000
info 165 MB memory used
warning Found a change in your Saber config file, restart server to see the effect.
info (21:3:37) Rebuilding client due to changes in ~\Documents\Projects\the-story\.saber\config.json
success Compiled client successfully in 282ms!
warning Port 3000 is in use, switched to a new port
info Available at http://localhost:3001
info 93.7 MB memory used
info (21:5:4) Rebuilding client due to changes in ~\Documents\Projects\the-story\.saber\config.json
success Compiled client successfully in 257ms!
warning Port 3000 is in use, switched to a new port
info Available at http://localhost:3001
info 98.8 MB memory used

@kidonng nice catch. found the problem will make a PR to fix it


Example code doesn't work but this will

Hi so your example code doesn't work for pulling JWT claim. You need to do this instead:
    <set-header name="x-someheadernametobepassedtobackend" exists-action="append">
        <value>@{
        string subject = "NoSub";
        string authHeader = context.Request.Headers.GetValueOrDefault("Authorization", "");
        if (authHeader?.Length > 0){
            string[] authHeaderParts;
            authHeaderParts = authHeader.Split(' ');
            if (authHeaderParts?.Length == 2 && authHeaderParts[0].Equals("Bearer", StringComparison.InvariantCultureIgnoreCase)){
                Jwt jwt;
                if (authHeaderParts[1].TryParseJwt(out jwt)){
                    subject = jwt.Claims.GetValueOrDefault("sub", "NOSUBJECT");
                }
            }
        }
        return subject;
    }</value>
    </set-header>

This of course pulls the Authorization header, strips out the Bearer token and decodes it and in this example looks at the sub claim, then passes that to the backend as a header. Cheers.

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

ID: 248a0753-0fa7-1cc0-d13a-d378ab182e95
Version Independent ID: 7efa247a-7184-6e7f-c1f5-3ff2531df357
Content: Azure API Management access restriction policies
Content Source: articles/api-management/api-management-access-restriction-policies.md
Service: api-management
GitHub Login: @vladvino
Microsoft Alias: apimpm

@cplunkett Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

Thanks for raising this issue in Azure documentation. Unfortunately, at this time we have been unable to review or resolve your issue in a timely manner and we sincerely apologize for the delayed response. We have captured your feedback for consideration in future documentation updates and are now closing this issue. To discuss further, you can also get answers from community experts in Microsoft Q&A. If you have a support plan and you need technical help, create an Azure support request.
#please-close


[BIOMAGE-1442] Add redux selectors for cellSets

Background
Link to issue
https://biomage.atlassian.net/secure/RapidBoard.jspa?rapidView=1&projectKey=BIOMAGE&modal=detail&selectedIssue=BIOMAGE-1442
Link to staging deployment URL
Links to any Pull Requests related to this
Anything else the reviewers should know about the changes here
Changes
Code changes
Definition of DONE
Your changes will be ready for merging after each of the steps below have been completed:
Testing

[ ] Unit tests written
[ ] Tested locally with Inframock (with latest production data downloaded with biomage experiment pull)
[ ] Deployed to staging
[ ] Passed integration tests

To set up easy local testing with inframock, follow the instructions here: https://github.com/biomage-ltd/inframock
To deploy to the staging environment, follow the instructions here: https://github.com/biomage-ltd/biomage-utils
Documentation updates
Is all relevant documentation updated to reflect the proposed changes in this PR?

[ ] Relevant Github READMEs updated
[ ] Relevant wiki pages created/updated

Approvers

[ ] Approved by a member of the core engineering team
[ ] (UX changes) Approved by vickymorrison (this is her username, tag her if you need approval)

Just before merging:

[ ] After the PR is approved, the unstage script in here: https://github.com/biomage-ltd/biomage-utils is executed. This script cleans up your deployment to staging

Optional

[ ] Photo of a cute animal attached to this PR

there is some issue here with the commits in the PR, it's showing up as 46 commits for some reason, most of them by agi and iva


4 more tests passing test-doc-7 and test-doc-8.

@bsharpataz : Request you to take a look some of my doubts as Todos in the comments

Fixing the comments by @bsharpataz. Looks like this can be merged..

Not so fast... 😆

@kwalcock : had left checked-in a compilation error. Fixed that now

Is @bsharpataz happy with this now?

@bsharpataz : Could you please approve this PR whenever you get a chance next ?

@bsharpataz, what should be done with this now?  Has it expired?

@kwalcock yeah prob, sorry, but this reminds me that one day we should see if we can make some of the failing tests pass... some may already


Do sharpness and auto_contrast do much?

Hi,
I'm currently trying to find the perfect trivial augmentation policy for my problem. When I apply the auto_contrast or sharpness at 2 strength levels and i display a few augmentations, i can't tell if there happens anything compared to the original image. I've already tried to rise the minVal but nothing changed.
Here's a minimal working example
from PIL import Image
import aug_lib

orig_img = Image.open('pizza.jpeg')
aug_lib.set_augmentation_space(augmentation_space='wide_standard_custom',
                                           num_strengths=2,
                                           custom_augmentation_space_augs=["auto_contrast" ]) # or sharpness
augmenter = aug_lib.TrivialAugment()
for _ in range(4):
    display(augmenter(orig_img))

Thanks in advance for your commitment!
Kind regards,
Johannes

Hi Johannes,
Sorry for the late answer!
I am not sure tbh. I think, I also saw some augmentations that have little impact.
You should check the exact pixel values. To me it seems logical that a little amount of auto contrast is (almost) invisible to the human eye.
In general: It might very well be that I did not tweak out the most perfect setting for all the augmentation bounds. I did not tune them a lot. Especially if you just care about performance on one dataset and not across CIFAR10/100/ImageNet/SVHN, you might be able to squeeze out more by changing these.

Hi Samuel,
Thanks for your reply. I will consider your recommendations!
FYI: I did some tuning of the "ordinary" contrast for my problem. We could raise the accuracy by reducing the range to: MinMax(min=0.5, max=2.0). The previous max value completely greyed the image out (to the human eye).


Update using-an-extension-inf-file.md

Removing filter driver reference as it's not extension specific

@NabilFates,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Docker image - Static binaries shouldn't need host libraries

I noticed that the Docker image seems to include the libraries necessary to compile shapeit5, but does not compile them inside the docker container. Instead, static binaries from outside the container get copied in.
Because you are copying in already-compiled static binaries, I believe that the docker image could be dramatically simplified and much smaller, by including only the static binaries? I could send a PR if you're interested and test it myself.
It looks like you also want bcftools in the docker container as well, because you're using those at the same step of your process as you want the shapeit5 tools? If you want to include those, the generally accepted practice would be a multi-stage build, which I'd also be glad to help with. That lets the docker image that you ship not include compilers or development libraries, which helps it be smaller and more secure.
You could also use a distro-provided bcftools, but it seems like you want to pin to htslib and bcftools version 1.15 right now, so the newer versions in Debian or Ubuntu may not be appropriate.

Hi, @pettyalex.
Were you able to run an analysis using the Docker image? May I please know the command used? I tried this:
sudo docker run lindonkambule/shapeit5_2023-05-05_d6ce1e2:latest --help
and got this:
docker: Error response from daemon: failed to create task for container: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: exec: "--help": executable file not found in $PATH: unknown. ERRO[0000] error waiting for container:
Which executable file is needed? Sorry, I'm new to this and still have so much to learn. Thank you in advance for your help.

+1 to this.
The static binaries should be compiled inside a controlled docker environment, as opposed to copied in from a local environment. The final images can be stripped of the build time dependencies, and only packaged with the necessary binaries. Ideally, this comes in the form of one binary per image, where the entrypoint can be specified as that binary.
The compiled binaries are very small, so the resulting images can be kept under 100M.
I have a build set up in the UM org and I'd be happy to share that as well.

@kscott-1 that would be extremely helpful! I fail to see how the "Dockerfile" could possible build the application cleanly - it required a great deal of modification to get mostly done.
In addition for downstream development, we want to make sure we don't introduce regressions, therefore a full container build is essential.


[5/x][lamport] Remove SequenceNumber::decrement and ::increment

SequenceNumber::decrement is used primarily to revert state
updates (e.g. when an update crosses an epoch or configuration change
and then needs to be rolled back).
With the introduction of lamport timestamps, this will no longer be
feasible with decrement, because a transaction can increment an
object's version by an arbitrary amount (not always one), depending on
the versions of other input objects.
This commit re-implements revert_state_update to rely on a list of
initial versions for modified inputs that gets stored in
TransactionEffects and gets rid of the decrement API altogether.
increment is replaced by two pieces:

SequenceNumber::lamport_increment to calculate a lamport timestamp.
SequenceNumber::increment_to to increment to a number that should
be strictly greater than the current version.

And now, rather than objects being incremented when they are modified,
versions are updated as follows:

Shared object scheduling pre-computes their new versions based on
the transaction's inputs.
The TemporaryStore is aware of the lamport timestamp for the
transaction it's serving, and assigns all modified objects (this
includes inputs and child objects) this version when committing
modifications to effects.
We also remember the versions modified and deleted objects had,
in case we need to revert -- we can't rely solely on the information
from the certificate for this, because we could have dynamically
loaded child objects which need to be reverted as well.

Gateway also needs some special support in order to replay
certificates: Previously it took the objects that were indicated as
modified in effects and replayed those modifications into a store, but
new protections that this commit introduces to ensure that all objects
get incremented would complain (because the objects in the effects
come with thei post-modified version), so before replaying the
modifications, the gateway will decrement an object's version to their
pre-modified state, as specified by modified_at_versions in effects,
using a new SequenceNumber::decrement_to API.
Test Plan
cargo simtest
cargo nextest run

Thanks for the reviews both!  Agreed that with us dropping support for owned to shared upgrades, all the logic can now be moved out of the adapter, which is nice!  I'll rebase and make those changes.


How to add click event for points?

How to add click event for points?For example , when I click one point , it may pop up a dialog that shows more info about this point.The key point is click event . So how to do that ? Thanks!

chart.setOnValueTouchListener(new ValueTouchListener());
and than in your code
private class ValueTouchListener implements LineChartOnValueSelectListener {
@Override
public void onValueSelected(int lineIndex, int pointIndex, PointValue value) {  }
    @Override
    public void onValueDeselected() { }
}


Add check-for-update endpoint

the is_updated flag is checking for updates in nuimo connection status and sonos configuration

As the frontend polls this endpoint, if the backend sees something has changed it should send the response data of /-/confnuimos

When I'm in the office next, I'll test this and then merge it.


Example-Forum latest build, VulcanJS 1.12.8, new way  of handling localization ?

Hi,
In the previous version of Example-Forum/Vulcan (based on meteor 1.6), I have 2 files for the text used in my app:
i18n-en.js = english
i18n-fr.js = french
then I import these files in my index.js as below:
import './i18n-en.js';
import './i18n-fr.js';
then in my settings.json
for English, I use
"locale": "en",
for French, I use
"locale": "fr",
Using above seems not working anymore in the latest version of Example-Forum latest build + VulcanJS 1.12.8, my App doesn't show text in French even if my settings.json use "fr" (still show English).
Seeing this, I check the package vulcan-i18n-fr-fr then I see that the localization file is add on the package.js, like below:
api.addFiles([
'lib/fr_FR.js'
], ['client', 'server']);
Could you confirm that this is the new way of handling localize file in VulcanJS ?
Thank You
Adalidda

Hi @adalidda ,
The I18n in Vulcan was updated at v1.11. You can see the docs here : http://docs.vulcanjs.org/internationalization.html

Seeing this, I check the package vulcan-i18n-fr-fr then I see that the localization file is add on the package.js, like below:

'lib/fr_FR.js'
], ['client', 'server']);```

This won't help, it's just importing the files in the module, which your other files are already (through a longer pipeline though)

I don't know myself how to deal with this. It seems that the language is based on the client's locale now, so basically if your browser is not set to French the language will fallback to English by default. The docs probably provide something to handle this, but I have not had time to look into that yet

Thank You @Apollinaire I will look to the doc in detail.

After I set the cookies as following:
cookies.set('locale', 'fr');
Everything is working fine and my App is now displaying the text in French.
Thank You @Apollinaire

@adalidda could you describe more in detail what you did ?


Property IsProject does not exist. Class IPPCustomer

When i am trying to add cutomer to Quickbooks using following code.
$dataService->Add($customerObj);
I am getting the following exception in sendRequestParseResponseBodyAndHandleHttpError internal function of sdk.

Property IsProject does not exist. Class IPPCustomer

@Abhilash-k-p Can you please update your SDK to use the latest vesion?

I am receiving this error on the 5.3.4 branch.

That is super odd, can you please share the request? I am on the latest branch and I don't see the error. Can you please check your code in vendor directory to see if you have this line of code: https://github.com/intuit/QuickBooks-V3-PHP-SDK/blob/master/src/Data/IPPCustomer.php#L525 ?

Yes, I had found that the library does in fact have the IsProject property on the IPPCustomer class. I'm thinking it must be conflicting with a completely different ancient forked version of the SDK that is on the server which also has an IPPCustomer class. Although that's strange because it is namespaced totally differently. I ran out of time this afternoon, but I'll debug some more tomorrow and let you know if that is the problem.

Removing the other SDK did fix the problem.
This is a problem because the XML Serializer prioritizes the non-namespaced IPP class rather than the namespaced class. I'm not sure if there is a reason for that, but it seems to me it shouldn't ever be looking for a class outside of the namespace:
The issue is in src/Core/Http/Serialization/XmlObjectSerializer.php in the PhpObjFromXml method starting on line 86:
    if (class_exists($className, CoreConstants::USE_AUTOLOADER)) {
        $phpObj = new $className;
    } elseif (class_exists(CoreConstants::NAMEPSACE_DATA_PREFIX . $className, CoreConstants::USE_AUTOLOADER)) {
        $className = CoreConstants::NAMEPSACE_DATA_PREFIX . $className;
        $phpObj = new $className;
    } else {
        throw new \Exception("Can't find corresponding CLASS for className" . $className . "during unmarshall XML into POPO Object");
    }

I propose that the first 'if' block should be completely removed. At the very least, it should come after the first elseif.

@KelseySheely calling the issue closed, you can send a PR with the changes, we can evaluate and merge. Thanks!


[model] Merge model finalize to destructor

Merge model finalize to destructor itself
User need not call finalize explicitly
Other minor updates:

Merge all train methods into 1
isInitializable() is made private. Removed some of the name checking from this function
to when the layer itself was created.

Self evaluation:

Build test: [x]Passed [ ]Failed [ ]Skipped
Run test: [x]Passed [ ]Failed [ ]Skipped

Signed-off-by: Parichay Kapoor<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #603. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.


Added visibility toggle for symlinks

… of symlinks.
Description of Changes
Closes #1352
Added a new setting g:NERDTreeShowSymlink, For filtering symlinks, This setting can be dynamically toggled, per tree, with the  mapping.

New Version Info
Author's Instructions

[ ] Derive a new MAJOR.MINOR.PATCH version number. Increment the:

MAJOR version when you make incompatible API changes
MINOR version when you add functionality in a backwards-compatible manner
PATCH version when you make backwards-compatible bug fixes

[ ] Update CHANGELOG.md, following the established pattern.

Collaborator's Instructions

[ ] Review CHANGELOG.md, suggesting a different version number if necessary.
[ ] After merging, tag the commit using these (Mac-compatible) bash commands:git checkout master
git pull
sed -n "$(grep -n -m2 '####' CHANGELOG.md | cut -f1 -d: | sed 'N;s/\n/,/')p" CHANGELOG.md | sed '$d'
git tag -a $(read -p "Tag Name: " tag;echo $tag) -m"$(git show --quiet --pretty=%s)";git push origin --tags

https://github.com/preservim/nerdtree/assets/3788964/b39e9868-2dd2-409e-b859-ca9a5d2899a4
https://github.com/preservim/nerdtree/assets/3788964/8e075b28-e4e1-4c20-8bbf-5fbb63dbc040

I'm not convinced this addresses the feature request. I think the issue opener just wanted to hide the target of the links, not the fact that symlinks even existed. To be honest I'd hesitate to include this. I've seen tons of file manager interfaces that hide hidden files, but can't think of any that completely hide the existence of symlinks. At the very least I'd expect first to have the functionality to hide the targets (which look a little noisy) and only then to consider what the use case is for completely ignoring them.

I'm up for closing this PR, at first it seemed natural to add the ability to hide symlinks, But now that I'm thinking again about it, I think you are right maybe only hiding the linked path is enough

I'll close this for now. We can reconsider if anybody has a use case for it that they want to pitch. Personally I don't have one and can't imagine one, but that doesn't mean one doesn't exist. If we do include something like this it would be nice to know it actually met a need and didn't just add clutter to the docs.
In the mean time an option to hide symlink targets does seem to make sense to me.


gromacs: update application description

Description of changes
Update the license and website for GROMACS:

The website was updated ~1 year ago; can be verified by checking the URLs.
The license was changed many years ago from GPL2 to LGPL 2.1; see the same "about" page or the COPYING file.

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[X] Fits CONTRIBUTING.md.

This is a bit complicated, because the licensing info is not retroactive.
Have you tried to update the package?

I'm not a NixOS user; I initially wanted to open an issue, but a PR seemed to be an easier way to describe the requested change :) I can close this and open an issue instead if that's preferred.
If I understand you correctly, the problem is that updating an already installed package won't change its license metadata. GROMACS switched to LGPL >10 years ago (https://github.com/gromacs/gromacs/commit/e6cd064af2032c3e2c9b05dc95dd4651700d5691#diff-400d38e5cfee181230373a8b02d38f50271a5bf62c62410382efddb8e8b19e22); presumably, few enough people are troubled by the wrong license data, so I see little reason to overcomplicate things.

I can close this and open an issue instead if that's preferred.

No need to open an issue.
The purpose of an issue is distinct from a PR.

If I understand you correctly, the problem is that updating an already installed package won't change its license metadata.

The meta field must be accurate.
If I package a software whose 1.0 is GPL but the 1.1 is BSD, it makes no sense to change the license of 1.0.
On the other hand, if you are merely fixing up a wrong metainfo, say the 1.1 is BSD but we marked GPL, then the change is most welcome.

presumably, few enough people are troubled by the wrong license data

Well, nix provides access to each and every past commit of nixpkgs. This is useful for data hoarding and software preservation

, so I see little reason to overcomplicate things.

Fair. I want only to know if the info was accurate.

If I package a software whose 1.0 is GPL but the 1.1 is BSD, it makes no sense to change the license of 1.0.
On the other hand, if you are merely fixing up a wrong metainfo, say the 1.1 is BSD but we marked GPL, then the change is most welcome.

I see, thanks for the explanation. I'm just fixing the metainfo here which has been incorrect since 24c79d66261d0d4a71feb14bc57a59e97f071ab8.


[WFLY-14968] Collapse if-statements (Undertow)

Fixes https://issues.redhat.com/browse/WFLY-14968

@bstansberry This one has passed tests and is approved, too. Please, could you check and merge?

Just be clear regarding approval, the last comment was that the component lead may want an opportunity to review the code for the component they maintain.

@fl4via FYI

My apologies getting this wrong. I don't want to break any formal or informal rules of review and approval. @darranl Thanks for the clarification. @bstansberry  Please ignore my previous merge request.

Good Morning @fl4via , could you review and approve this PR, please? It's one of two missing to finish the Jira and I hope to get it into the release.

@rhusar Hi Radoslav, thanks for your feedback. It's fixed. Could you review again, please?
@fl4via fyi

@boris-unckel thanks for reopening! @bstansberry  this one is good to go


implementation of HHL algorithm

Summary
This adds a general-purpose implementation of the HHL algorithm, which can be used to solve linear systems of equations based on QPE.
Details and comments
A new input is added LinearSystemInput, which takes a matrix and a vector defining the system of equations as input.
It adds necessary component Eigenvalues with variant EigsQPE for eigenvalue estimation based on QPE with special handling of negative eigenvalues, and component Reciprocal with variants LookupRotation and LongDivision for calculating the reciprocal of the eigenvalues, which is needed for controlled rotation of the HHL ancillary qubit.
Unittests are provided for testing various HHL algorithm configurations and different simulators in test_hhl.py as well as for testing the circuit generation of the LookupRotation in test_lookup_rotation.py.

@chunfuchen thanks for your feedback, added the changes accordingly!

@woodsp-ibm had to increase the number of shots in qasm test as it was failing previously


[8.x] Add new Stringable::matches method

This pull request introduces a new Stringable::matches method that checks whether the string matches a particular pattern, but doesn't return the match(es).
This is essentially just a shortcut for $stringable->match('//')->isNotEmpty() but something that I do often enough, and also macro, that it warranted a pull request to the core.

I realise now that matches might get confused with matchAll, perhaps another good method name would be test, just like JavaScript's method.

Yeah, maybe test would be good since that is what JavaScript calls it.

@taylorotwell All done. 🤟

Can we have it on Illuminate\Support\Str too?


Pip-compile ignores setup_dependencies when parsing setup.py

Similar to #625 it seems that pip-compile ignores setup_dependencies when resolving dependencies. This causes a problem for us as our workflow uses pip-compile to freeze dependencies w/ hashes. Then we use the requirements files generated to pip download all dependencies prior to installing in a network isolated environment.
Environment Versions

OS Type - Windows
Python version: Python 3.7.1
pip version:  pip 18.1
pip-tools version: pip-compile, version 3.7.0

Steps to replicate
pip-compile --generate-hashes --allow-unsafe --output-file test-windows.txt test.in
Where test.in contains a dependency which has setup_dependencies.
Expected result
Setup dependency to be populated in windows.txt
Actual result
Not included.

Hello @bgklein,
Thanks for the info! Could you paste here a content of the test.in which causes the bug?

One example is flake8==3.50 (setup_dependency of pytest-runner).
test.in.txt
test-windows.txt

I'm little bit confused, what do you mean by setup_dependency?

Sorry my bad. put the wrong terminology. setup_requires as in https://github.com/PyCQA/flake8/blob/8acf55e0f85233c51c291816d73d828cc62d30d1/setup.py#L146

Thanks! Looks like that's a correct behavior since packages in setup_requires are not supposed to be installed, see a note in docs:

(Note: projects listed in setup_requires will NOT be automatically installed on the system where the setup script is being run. They are simply downloaded to the ./.eggs directory if they’re not locally available already. If you want them to be installed, as well as being available when the setup script is run, you should add them to install_requires and setup_requires.)

Hmm that makes sense. Do you know if it would be possible to make this an optional flag for pip-compile? Otherwise my team will need to add them to our requirements file/maintain them separately as our actual install happens without internet connectivity causing 'They are simply downloaded to the ./.eggs directory if they’re not locally available already' to fail.

You may simply add pytest-runner to the test.in. Wouldn't it be enough?

Yea it would be enough just mean more maintenance. Since we do hash checking we specify the versions of  our dependencies. This means when upgrading flake8  and/or any other dependencies we will need to dive into the dependency tree to upgrade their respective setup_requires.

FYI there is no need to specify version in *.in file, that's not how it works. Let the pip-compile pin the packages in requirements.txt.

While pip-compile does not require pinned versions, we do pin our versioning for our core dependencies. Looking over the setup_requires for most of our dependencies it looks like none of them pin their setup_requires to a specific version so we should be able to leave them unpinned, but it does defeat some of the purpose of pinning our dependencies.
You can close the issue if you'd like, as the solution should work. I still think it'd be helpful to have a flag which would automatically get and add setup_requires as currently we either have to try to build and see failures to determine what's missing or crawl the dependency tree to realize what is in the field, but I will leave teh decision on that to you guys.

FYI, refs #492.


Network isolation validation

Validate network isolation configuration in a single location rather than performing validation in multiple places

@vknabel Do want to support empty RegistryMirrors or not?
	if p.NetworkIsolation == nil || len(p.NetworkIsolation.RegistryMirrors) == 0 {
		m.logger.Info("no registry mirrors specified in this shoot, nothing to do here", "shoot", shootName)
		return nil
	}

Source: https://github.com/metal-stack/gardener-extension-provider-metal/blob/b38f4e809a4fd20ae5d0b48022f20784a1e32521/pkg/webhook/shoot/mutator.go#L170

@robertvolkmann if the network access type is not baseline, the NetworkIsolation and its RegistryMirrors must be set and contain at least one mirror.

rebase and run in test

Worked fine in test


read_until_image_data doesn't seem to read until an IDAT

I basically copied all the bytes of a png into another file (.pnle which is just smth i made up for a temp file). So technically up until the first IDAT chunk, all the metadata before should 100% be right.
I open the .pnle using png::Decoder to get a decoder object, then i call decoder.read_info(), which according to the spec and the src code, should read until the first IDAT. However, i truncate my .pnle a little bit towards the end, but all the metadata before the first IDAT is untouched. When the read_info func is called, it errors out giving me a MissingImageData inner error. Only, when i have the full entire png in the pnle does the read_info func complete.
In the Decoder src, read_info() calls read_until_image_data() and everytime it crashes saying that it's reached Decoded::ImageEnd already, which is only returned in stream.rs if it's found IEND.

Are you sure you didn't entirely remove the IDAT chunks? The decoder can't know it has reached the image data until it sees a chunk with type=IDAT. If it hits the end of the input first it'll return a MissingImageData error.

@fintelia Im doing this (basically the example usage for this crate):
let encoder = png::Encoder::with_info(w, img_info)?;
let mut writer = encoder.write_header().unwrap();
writer.write_image_data(data);

But im not fully writing down all the data. If i do &data[..data.len()-1], it doesn't seem to work but there should at least be a single IDAT? Or am i misunderstanding an IDAT or something?
I was just looking for a function that could parse all the ihdr chunks and pallete, everything not including the data.

I'd have to look into the implementation of write_image_data to be sure, but I think the writer might wait to write an IDAT until it has enough data buffered and/or a full frame worth of data has been written.

ok interesting thanks!


53 zoom with scroll

on annotations and groundtruth page:

update zoom factor from 1.1 to 1.41;
add feature zoom with scroll wheel
display zoom level between zoom in and zoom out icon

PR fixes #53

looks good overall; can you change the calculation to start at 0% zoom instead of 100%? since we don't zoom out, it seems the right place to start for clarity

just make sure there is no misunderstandings, do we minus 100% from the current result, or divide this result by 100? suppose the images has been zoomed in to 2000px(height), original height is 650px, what number do we display?

Okay, met with @simbamangu and we came to a (mostly) agreement. He has until you start work tomorrow to contradict what I'm writing.
Whole picture, all the way zoomed out = 0%
maximum zoom in = 100%

No contradictions from this end … got feedback from one annotator who
likes the idea.
On 27 Nov 2024 at 07:14:28, Tanya @.***> wrote:

Okay, met with @simbamangu https://github.com/simbamangu and we came to
a (mostly) agreement. He has until you start work tomorrow to contradict
what I'm writing.
Whole picture, all the way zoomed out = 0%
maximum zoom in = 100%
—
Reply to this email directly, view it on GitHub
https://github.com/WildMeOrg/scout/pull/122#issuecomment-2502687504, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAI3ERHUX4XIHNN5HW6347D2CVBKJAVCNFSM6AAAAABSOW4OJWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBSGY4DONJQGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

gotcha, will update accordingly


Compliance history not working?

Reproduce:

create some custom compliance policy that always fails agains vms, I used[1]
attach to policy profile and attach that profile to a vm
run compliance on that vm
go to vm screen;
result is [2] while policy log shows:

[----] I, [2016-05-23T17:41:47.341701 #26326:123598c]  INFO -- : MIQ(policy-enforce_policy): Event: [vm_compliance_check], To: [afri-F23]
[----] I, [2016-05-23T17:41:47.380700 #26326:123598c]  INFO -- : MIQ(policy-enforce_policy): Resolving policy [puku]...
[----] I, [2016-05-23T17:41:47.409390 #26326:123598c]  INFO -- : MIQ(condition-eval): Name: pukuku, Expression evaluation result: [false]
[----] I, [2016-05-23T17:41:47.412677 #26326:123598c]  INFO -- : MIQ(action-invoke) Invoking action [Mark as Non-Compliant] for failed policy [puku], event: [VM Compliance Check], entity name: [afri-F23], entity type: [Virtual Machine (Red Hat)], sequence: [1], synchronous? [true]
[----] I, [2016-05-23T17:41:47.412728 #26326:123598c]  INFO -- : MIQ(action_compliance_failed): Now executing [Mark as Non-Compliant] of ManageIQ::Providers::Redhat::InfraManager::Vm [afri-F23]

In evm.log I was able to get (should that not be in policy log as well?)
[----] I, [2016-05-23T18:26:05.647154 #26321:123598c]  INFO -- : MIQ(Compliance.check_compliance) Checking compliance...
[----] I, [2016-05-23T18:26:05.686281 #26321:123598c]  INFO -- : MIQ(Compliance.set_compliancy) Marking as Compliant Object with Class: [ManageIQ::Providers::Redhat::InfraManager::Vm], Id: [1], Name: [afri-F23]
[----] I, [2016-05-23T18:26:05.693549 #26321:123598c]  INFO -- : MIQ(Compliance.check_compliance) Raising EVM Event: vm_compliance_failed

and
[----] I, [2016-05-23T18:24:35.178797 #26321:123598c]  INFO -- : MIQ(Compliance.check_compliance) Checking compliance...
[----] I, [2016-05-23T18:24:35.242683 #26321:123598c]  INFO -- : MIQ(Compliance.set_compliancy) Marking as Compliant Object with Class: [ManageIQ::Providers::Redhat::InfraManager::Vm], Id: [1], Name: [afri-F23]
[----] I, [2016-05-23T18:24:35.267998 #26321:123598c]  INFO -- : MIQ(Compliance.check_compliance) Raising EVM Event: vm_compliance_passed

But UI always shows vm as compliant
[1]

[2]

closed through #8933


App quits if user has exactly one friend

If a steam user has only one friend, the app correctly creates menu with single menu item for that user. However, when 'Send invite' menu item for that user is clicked, then app cleanly quits instead of creating the invite.

Bug was cause by incorrect event binding. Thought i could get away with it... https://github.com/m4dEngi/RemotePlayWhatever/commit/355269526c214d9fe671d10dc4fbf2ef576917c5
Latest release should fix the problem.

Bug was cause by incorrect event binding. Thought i could get away with it... https://github.com/m4dEngi/RemotePlayWhatever/commit/355269526c214d9fe671d10dc4fbf2ef576917c5
Latest release should fix the problem.


define struct types instead of using anonymous structs

Some fields were defined inline as anonymous structs.
This made it tedious to use these structs in testcases, because
the anonymous type needed to be redefined (incl the exact same JSON
tags) whenever it was assigned.
Using anonymous struct also does not scale well when the struct changes.
For example when a field is added to an anonymous struct, every place
where the anonymous struct is assigned must also be changed.
Another advantage of using regular types is that accessors  will be
generated automatically and can be used instead of having to manually
check for nil values when accessing the struct fields.
Define the types as regular structs instead.

PR Status: PR is finished but I noticed that my company hasn't signed the CA yet. As soon as it is done, I'll transition the PR to "ready for review".

@gmlewis I also replaced the other anonymous struct types.

Also, @fho - could you please remove the "Draft" setting when you are ready?

@gmlewis I'll remove the draft status as soon as my employer signed the CA, probably beginning of next week. I only noticed that it was missing after I created it. :-)

@gmlewis I'll remove the draft status as soon as my employer signed the CA, probably beginning of next week. I only noticed that it was missing after I created it. :-)

Interesting - the googlebot marked the PR as having passed the CLA test.  OK, sounds good.

@gmlewis I'll remove the draft status as soon as my employer signed the CA, probably beginning of next week. I only noticed that it was missing after I created it. :-)

Interesting - the googlebot marked the PR as having passed the CLA test. OK, sounds good.

It's because I signed the PR for myself already in the past.

@gmlewis my employer signed the CLA now also.
I can't see it on https://cla.developers.google.com/clas for my account but I'm in the contributors google-group of the CLA.

Friendly ping, @wesleimp

If we are expecting to replace all anonymous structs with defined struct types, then I guess a few are still missing in this PR.
I am listing them below :

File: github.go / ErrorResponse
File: activity.go / Feeds

@gmlewis

@sagar23sj thanks, I'll update the PR and also replace those anonymous structs.
It was not my initial intent of the PR to replace all anonymous structs but it makes sense to have a consistent solution

done

@fho  - would you like to resolve the merge conflicts, or would you like me to do that?  I'm fine either way.

@gmlewis also both fine with me, feel free to do changes :-)
Otherwise I can solve the conflict tomorrow


org.http4s.InvalidBodyException: Received premature EOF

Hi, we are getting this issue in our application. I saw a few links related to the same issue. But it does not look like the same thing in our case. we are getting it 1-2 percent of requests.
Basically, we are doing performance testing of our application but it's not related to performance because we don't have this issue in prod or we are getting this at 1 rps also.
I have created a StackOverflow question with an explanation of how are doing this performance testing link. I am not able to understand what we are doing wrong, If someone can help us it would be great.

Hi @vimitdhawan could you provide more information like what version of http4s you are using, what backend, the nature of the application and the performance test, and any other relevant context ?

Hi @RaasAhsan, we rare using http4s = "0.21.7" . we are using org.http4s.server.middleware. Those requests even not coming to this middleware. It looks like they are garbage in and out. For client we are using sttp = "3.2.3".
These all are prod requests and we did not face this issue in prod. You can check this link to see the architecture how we are requesting the application.
If you want more detail, please let me know. If you want me to try something else, I can do that.

Sorry, which server middlewares are you using?  And which server backend?  Blaze, ember...?  This error is appearing on the server?

@rossabaker We have found that we have some half build requests. when a client sends those requests we are getting eof on the server and timeout on the client side. Not sure why we are not getting bad requests. we have blaze at server side.


Submit submittable WorkflowTemplate through the rest api

Summary
Workflow are re-submitable from the rest api, I think it would be reasonable to be able to submit  submittable WorkflowTemplate with something like PUT /api/v1/workflow-templates/{namespace}/{name}/submit.
Motivation
I want to be able to submit WorkflowTemplates from a web service running in the cluster with a simple PUT request.

Message from the maintainers:
If you wish to see this enhancement implemented please add a 👍 reaction to this issue! We often sort issues this way to know what to prioritize.

It is good enhancement. I would enhance the workflow API CreateFromResource API will have resourcekind and resource name rather enhancing in workflow-template , cron-workflow and clusterworkflowtemplate APIs
CreateFromResource API can  submit the workflow from workflowtemplate,cronworkflow and ClustetWorkflowTemplate

I have same problem.I'm looking forward you can resolve this issue

I am working on it

I am working on it

[thumb up]👍👍

Hey guys, I can no longer find "/api/v1/workflows/{namespace}/submit-from" in the swagger documentation. Has it been removed? I am using argo-workflow-server v2.12.9 and the endpoint returns 501 for me.

It is renamed to submit #2778.
try /api/v1/workflows/{namespace}/submit

Ah, thank you, found it. If I may ask, what is the format for submitOptions.parameters? So lets imagine I have a parameter named "message" that I would like to set to the value "hello".

Ah, thank you, found it. If I may ask, what is the format for submitOptions.parameters? So lets imagine I have a parameter named "message" that I would like to set to the value "hello".

submitOptions.parameters="param1=val1, param2-val2"

Perfect, is works 👍 thanks a lot!


Third party buckets

🗣 Description
This PR adds:

A new S3 bucket meant to be used to store third-party files
A new role and policy that can be used to read objects in the bucket above

Resolves #45 and #46.

💭 Motivation and Context
We have a new project (nessus-packer) whose build user requires access to files in a bucket.  A related PR adds the ability to include additional policies (for example, the new ThirdPartyBucketRead policy in this PR) in the AMI build user role.

🧪 Testing
This terraform code was successfully applied in the Images (Staging) account.

📷 Screenshots (if appropriate)
🚥 Types of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (causes existing functionality to change)

✅ Checklist

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

@jsf9k @felddy @mcdonnnj - Please hold off on reviewing this PR until you hear from me.


Create links or buttons for the discover section to lead into a detailed steakMenu.js file and a wineMenu.js file

Done!


"Unknown command" when attempting glob matching with exclude

When filing an issue, please include the following information if
possible as well as a description of the problem.

What is your rclone version (eg output from rclone -V)
rclone v1.29

Which OS you are using and how many bits (eg Windows 7, 64 bit)
Mac OS X 10.11.5 64 bit

Which cloud storage system are you using? (eg Google Drive)
S3

The command you were trying to run (eg rclone copy /tmp remote:tmp)
rclone -n --exclude {one,two}_potato sync ~/test s3:test

Response:
2016/06/05 12:38:10 Unknown command "two_potato"

ugh, needed single quotes around the exclude value ...
rclone -n --exclude '{one,two}_potato' sync ~/test s3:test

Easy mistake to make.  +1 for use of potato as a nonce word ;-)


Ghatotkach.md

hey i would like to contribute in the project . thank you :)

@Ghat0tkach Go for it!


Rendering post items elements

Hello,
I need to render extra elements at the end of the list (just before the loader element).
I can't add them in the loader div since they have to by at the same dom level as the elements.
I am not very familiar with React and I hoped I could do something like this, but this doesn't work:
<InfiniteScroll
  pageStart={0}
  loadMore={loadFunc}
  hasMore={true || false}
  loader={
My post content 1

My post content 2

Loading ...
}
  useWindow={false}
>
    {items}
</InfiniteScroll>

Is it possible to achieve this with your component?
Maybe this is not possible and this require to add a new feature like a postContent option?

The loader prop only accepts one element to render and you are passing in 3 separate divs. You can't do this anywhere in React, so you need to learn/remember to wrap multiple children in divs.
<InfiniteScroll
  pageStart={0}
  loadMore={loadFunc}
  hasMore={true || false}
  loader={

My post content 1

My post content 2

Loading ...

}
  useWindow={false}
>
    {items}
</InfiniteScroll>

Solution was easy, I just had to add them inside the <InfiniteScroll>:
<InfiniteScroll
  pageStart={0}
  loadMore={loadFunc}
  hasMore={true || false}
  loader={
Loading ...
}
  useWindow={false}
>
    {items}

My post content 1

My post content 2

</InfiniteScroll>