gnucash: 4.2 -> 4.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/gnucash/versions.
meta.description for gnucash is: "Personal and small-business financial-accounting application"
meta.homepage for gnucash is: "http://www.gnucash.org/"
meta.changelog for gnucash is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 4.4 with grep in /nix/store/paf7z9p0g7gyaqhf2vpssd5cdifg75h3-gnucash-4.4

directory tree listing: https://gist.github.com/3212b36d73c6fa944d8afc3fab0a4781

du listing: https://gist.github.com/1f5eaaf48df98731987e358936146468

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
gnucash

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/paf7z9p0g7gyaqhf2vpssd5cdifg75h3-gnucash-4.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A gnucash https://github.com/r-ryantm/nixpkgs/archive/c9282e52ae41884801038a0fee21ddeb29bea545.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/paf7z9p0g7gyaqhf2vpssd5cdifg75h3-gnucash-4.4
ls -la /nix/store/paf7z9p0g7gyaqhf2vpssd5cdifg75h3-gnucash-4.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

gnucash

Maintainer pings
cc @peti @domenkozar for testing.


Installing tensorflow-text triggers a rollback of current's tensorflow version

Issue type
Support
Have you reproduced the bug with TensorFlow Nightly?
No
Source
source
TensorFlow version
2.10.1
Custom code
Yes
OS platform and distribution
Windows 10
Mobile device
No response
Python version
3.10.13
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?

pip install --upgrade tensorflow
Successfully installed flatbuffers-24.3.25 h5py-3.12.1 ml-dtypes-0.4.1 numpy-2.0.2 protobuf-5.28.3 tensorboard-2.18.0 tensorboard-data-server-0.7.2 tensorflow-2.18.0 tensorflow-intel-2.18.0

pip install --upgrade tensorflow-text
Successfully installed keras-2.10.0 protobuf-3.19.6 tensorboard-2.10.1 tensorboard-data-server-0.6.1 tensorflow-2.10.1

Standalone code to reproduce the issue
Just type in sequence the 2 command lines written above.

Relevant log output
No response

Hi @Mark531
This issue occurs because tensorflow-text has restrictive version dependencies, which leads to a downgrade of tensorflow to version 2.10.1 when it is installed. The documentation says that when installing, note the version of tensorflow you are running, as you must specify the corresponding minor version of TF Text.

@Mark531 I searched and saw that the library is active and in use, especially for cases where text processing is essential for machine learning models with tensorflow., but it is delayed to maintain alignment with newer versions of tensorflow. That is, updates to the tensorflow-text library are not updated immediately.

So, so it is currently usable with tensorflow 2.10.1.

@Venkat6871 need help?

Hi @Mark531 ,
Apologies for the delay, and thank you for raising your concern here. Could you please confirm where exactly you are facing the issue—with TensorFlow or with TensorFlow-Text triggers? If it is with TensorFlow-Text triggers, please raise the issue in that particular repository.
Thank you!


SUCCESSFULLY EHANCE THE USER INTERFACE OF THE PASSWORD GENERATOR APPL…

fixed: issue #81
@iamrahulmahato
### Enhancement of the User Interface for Password Generator Application
preview of the deployed one: https://poetic-biscochitos-eaaaa2.netlify.app/projects/password generator/
### VISUALISATION OF OVER ALL UPDATION AND THE CHANGES MADE IN THE PAGE:
### BEFORE UPDATION:
https://github.com/user-attachments/assets/a448286a-efc6-41c8-90c0-bff14cc4f6d7
### AFTER UPDATION
https://github.com/user-attachments/assets/85dd8093-56b8-4d05-bbcb-eb8eb933bc73
Introduction
Initially, the user interface (UI) of the password generator application was suboptimal in terms of design, usability, and user engagement. The landing page lacked clarity and failed to communicate the purpose of the application effectively. Additionally, the colors, fonts, and overall layout were poorly arranged, making the user experience (UX) dull and unappealing. To address these issues, significant improvements were made to enhance both the functionality and visual appeal of the site, resulting in a more interactive and user-friendly interface.
### Enhancements Implemented
### Improved Heading and Introduction
**A prominent and visually appealing heading was introduced with engaging frontend effects. This instantly communicates the purpose of the application, guiding users effectively from the outset. The enhanced heading creates a professional and attractive entry point for users, ensuring clarity and focus on the application’s purpose.
### BEFORE

### AFTER

### Incorporation of Visual Elements**
A meaningful image was added to the main page to visually represent the concept of password generation. This helps users quickly grasp the purpose of the application and adds an appealing aesthetic to the page. The inclusion of visual content complements the textual information, enhancing overall user engagement.
### BEFORE

### AFTER

Revamped Form and Input/Output Section
The form, which is the core of the password generator, was completely redesigned for improved usability. The input and output sections were optimized to make the process clearer and more intuitive for users. By adding interactive frontend elements, the user experience was made more engaging, helping users better understand the password generation process.
### User Awareness Section on Password Security
### BEFORE

### AFTER

A dedicated section was added at the bottom of the page to educate users on the importance of strong passwords in the digital world. This informative area not only provides guidance but also enhances the overall value of the application by promoting cybersecurity awareness.
### AFTER

### Custom Scrollbar for Enhanced Focus
### To maintain user focus and further improve the UI, a customized blue-colored scrollbar was implemented. This subtle yet effective enhancement adds visual harmony to the page while ensuring the site remains visually engaging.
### AFTER

### Optimized Font Usage
Font choices were carefully revised to reflect the professional nature of the application. The new fonts are clean, readable, and well-suited to the purpose of the site, ensuring the text is easy to follow while maintaining an attractive visual flow.
### Refined Color Palette
A modern and appealing color scheme was applied, aligning with the site's purpose and making the UI more visually cohesive. The chosen shades are not only aesthetically pleasing but also serve to guide the user’s attention effectively.
### Conclusion
The comprehensive improvements made to the user interface of the password generator application have significantly enhanced its functionality, usability, and visual appeal. These changes have resulted in a 10x improvement in user engagement, making the application far more intuitive and user-friendly than before. The new design effectively balances aesthetics with functionality, delivering an optimal user experience

RESPECTED SIR. I HAVE SUCCESSFULY UPDATED THE USER INTERFACE OF THE PASSWORD GENERATOR APP ON THE PAGE. AND ENHANCE IT 10X AND MADE IT VERY INTERACTIVE AND EFFECTIVE AND ATTRACTIVE . SO I AM REQUESTING YOU TO PLEASE REVIEW THE PR AND REACT FORWARD. @iamrahulmahato
~thankyou

@iamrahulmahato sir please review the changes and let us know the positive response or if you have any extra requirements please let me know . please.


Check gerber content for layer hints

This adds support for files who have the layer type set on its own line within the gerber file. It comes with some overhead as it scans the entire file but it does make the guessing more precise.

Coverage increased (+0.02%) to 81.01% when pulling 9ae238bf7ab4bc74d101605a9dbaddc098b9348d on ju5t:guess-layer into 8301011abd5961ab6edb2bf91e61d82658df6b9a on curtacircuitos:master.

@hamiltonkibbe would it be possible to merge this or do you think it needs any further adjustments?

Merged, Thanks!


ROX-17414: Temporarily cache Compliance RunResults

Description
Compliance dashboards seem to reuse the same data set across multiple graphQL queries. The goal here is to improve the performance of the dashboards by caching the compliance run results for re-use across queries.
Checklist

[ ] Investigated and inspected CI test results
- [ ] Unit test and regression tests added
- [ ] Evaluated and added CHANGELOG entry if required
- [ ] Determined and documented upgrade steps
- [ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
CI should be sufficient for the functional part.

This is complementary to

https://github.com/stackrox/stackrox/pull/7378


Error is occurred if there is a link in the render section

Error is occurred if there is a link in the render section.
I don't know exactly why this error is coming. Maybe it is a bug. Please check it.
I'm running on beta4 version.
Error will be this:
error CS0246: The type or namespace name 'TagHelperContent' could not be found (are you missing a using directive or an assembly reference?)
Sample code:
@{
ViewBag.Title = "MyPage";
}
@section mysection {
Test

}
This will work:
@{
ViewBag.Title = "MyPage";
}
@section mysection {
@*Test*@

}

This and aspnet/Mvc#2595 are duplicates. Closing aspnet/Mvc#2595 since most if not all of the fix will be in this repo.

Self-assigning since aspnet/Mvc#2595 was in this milestone and assigned to me.

113d80c

Thank you.

Thank you, I'll check this out.


[Support]: Conversion of azure-devops PublishTestResults@2 uses very old version of publish-unit-test-result-action and doesn't execute

What operating system are you using?

[X] Linux
[ ] macOS
[ ] Windows

What version of the tool are you using?
gh version 2.20.2 (2022-11-15)
gh actions-importer     github/gh-actions-importer      v1.0
actions-importer/cli    v1.0.14451
What happened?
Jobs that use PublishTestResults@2 for azure-devops end up being converted to<EMAIL_ADDRESS>This is quite old, and I would expect it to target EnricoMi/publish-unit-test-result-action@v1 when targeting the version 1 branch, which is currently at v1.40. The version 2 branch is also available, though it does have breaking changes.
The converted job additionally doesn't execute, complaining that GITHUB_TOKEN isn't set. I'm not sure if this is due to using the very old version or if the v1 branch requires additional options. When updating the jobs to v2 I didn't need to add any extra options to utilize the builtin pipeline token.
Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@akb825 thanks for opening this issue! We regularly audit the versions of actions that are used by GitHub Actions Importer and will be updating this version shortly 🙌

EnricoMi/publish-unit-test-result-action now at latest version


fps?

Have you test the fps in your computer?
when I run the eval.py, it is only 2.17samples/sec, this is too slow for light head? is it?

The backbone is Resnet101, How fast would you think it could be ? In the original paper, resnet101 is used to prove its' performance, another backbone is used for speed side.

@PCML-WangZhe I think 2 fps is quite fast indeed, what machine are you using btw

@PCML-WangZhe The backbone is Resnet101, How fast would you think it could be ? In the original paper, resnet101 is used to prove its' performance, another backbone is used for speed side.

@TreB1eN  What does Xception speed got?

I know it is resnet101,but I think it should be faster than 2fps, because when I run faster rcnn which is writed with pytorch in the same machine(GTX 1080), the speed is almost 10fps, even though consider the image size problem ,I think light head should be more faster, but it is slower than faster rcnn right now, I don't know why

@PCML-WangZhe what's the backbone, which repo, and test image size and proposals number in your comparison, this is not a speed oriented implementation, I dont think there is much point in discussing the speed here

@jinfagang 100fps, according to the paper

the repo is https://github.com/jwyang/faster-rcnn.pytorch
the backbone is resnet101, proposal number is 300 when test, test size is 600, maxsize is 1200

@PCML-WangZhe this repo Test size is 1000


Add mutator/accessor translations

PR for #116
Make $country->{'name:en'} or $country['name:en'] synonym for $country->translate('en')->name
Easier to automate form building/submissions.

Strange, tests failed randomly... Thanks a lot!

Hi,
Is possible to add this improvement for laravel 4.2?

Only via pull request.


How to create empty view with BindingRecyclerViewAdapter when items is empty?

Feature Request
How to create empty view with BindingRecyclerViewAdapter when items is empty?

The two common patterns for handling an empty state for a recyclerview is to show/hide a view on top of it when the list is empty, or have a single empty item in the list. Closing as this doesn't really have to do with this lib.


X11 variants lack Ctrl+A Ctrl+E etc shortcuts

Some apps have builtin Emacs keybindings, but some don't.
For instance, with the new X11 keyboard, I can still Ctrl+A Ctrl+E in VSCode, iterm2, etc
but not in Google Chrome.
Would be great if there's a way to force-enable them on top of any keyboard,
otherwise maybe we need to create "emacs variants".
A list of all Ctrl+ keybindings: https://www.hcs.harvard.edu/~jrus/site/system-bindings.html

Karabiner Elements works, but can be too complex to get right.

I tried used ~/Library/KeyBindings/DefaultKeyBinding.dict, but failed in Google Chrome (maybe it works in other apps).


Error: require GNU grep. Aborting in OSX Mojave 10.14.6

when i've entered the command "./sudomy -d domain.com --all" an error will appear like this "Error: require GNU grep. Aborting".  even though GNU Grep has been installed, the error will still appear as above.

https://github.com/Screetsec/Sudomy/issues/1

This duplicate to this issue https://github.com/Screetsec/Sudomy/issues/1

solved by $ export PATH="/usr/local/opt/grep/libexec/gnubin:$PATH" Thanks @Screetsec!


Create one NPM package for node/browsers/webpack

This uses wasm-pack to build all 3 targets into a 'fat binary' that is useable for any javascript purpose.
Normally package.json is generated but for our purposes we use our own.  Before this is merged the NPM_TOKEN secret will need to be added with access to https://www.npmjs.com/package/prql-js

I've added NPM_TOKEN

@charlie-sanders lmk if you want a hand finishing this — it looks like the only failing test is a pre-commit one; which should be quite easy to fix...

Ok yes please, I thought I had fixed it but looks like it's still failing.  I'll have computer access tommorow.

Ok yes please, I thought I had fixed it but looks like it's still failing. I'll have computer access tommorow.

OK, lmk if OK to hit the buttan

OK — am merging — @charlie-sanders are you OK to handle any follow-ups? Thank you!

OK — am merging — @charlie-sanders are you OK to handle any follow-ups? Thank you!

Yep! ill be back in town tomorrow night.


Update README on how to add a new lint

This really document really isn't helpful to get anyone getting started.
Do we have a template file for example to be copied in? Does newLint.sh work correctly? (If not, why doesn't it exist?)
How are you creating test certs? I've repeatedly heard that you're using Golang. Do you have a simple script that you can put in that let's get someone started?


Julia for Data Science | Video 10: Neural nets

References:
Speaker: Dr. Huda Nassar
Video title: Julia for Data Science - Video 10: Neural Nets, by Dr. Huda Nassar (for JuliaAcademy.com)
Link: https://www.youtube.com/watch?v=Oxi0Pfmskus
0:00 - Intro and MIST data
5:05 - Build the network
6:40 - Loss and accuracy
8:01 - Train the network
9:04 - Test the network
10:13 - Outro

We can now close this issue.

The video already has timestamps made by [Ahm3rIjaz(https://github.com/Ahm3rIjaz), so we can close the issue.


docs(toArray): added description and example for toArray

Description: Added description and example for toArray
Related issue (if exists): #4613

Thank you @waseemahmad31!


cio: should not open fifo for stderr if terminal

fix: #4342
Signed-off-by: Wei Fu<EMAIL_ADDRESS>

ping @gcapizzi

https://github.com/containerd/containerd/blob/a0696b2bc6a0110263b711b3af1db4f2e483fbc6/pkg/stdio/platform.go#L29
For console case, the stderr looks like duplicate with stdout input. Is there any case using stderr when terminal is set?

After this operation, 521 kB of additional disk space will be used.
Ign:1 http://azure.archive.ubuntu.com/ubuntu bionic-updates/main amd64 libseccomp-dev amd64 2.4.1-0ubuntu<IP_ADDRESS>
Err:1 http://security.ubuntu.com/ubuntu bionic-updates/main amd64 libseccomp-dev amd64 2.4.1-0ubuntu<IP_ADDRESS>
  404  Not Found [IP: <IP_ADDRESS> 80]
E: Failed to fetch http://security.ubuntu.com/ubuntu/pool/main/libs/libseccomp/libseccomp-dev_2.4.1-0ubuntu<IP_ADDRESS>_amd64.deb  404  Not Found [IP: <IP_ADDRESS> 80]
E: Unable to fetch some archives, maybe run apt-get update or try with --fix-missing?
##[error]Process completed with exit code 100.

The url is weird

ping @AkihiroSuda

ping @AkihiroSuda ~


Revisiting the ADAPT Representation System

The existing ADAPT Representation System (https://github.com/ADAPT/ADAPT/blob/develop/source/Representation/Resources/RepresentationSystem.xml) contains standard descriptions of data points passed around in the model.  It was donated by John Deere and has been extended with a few additional features (mapping to ISO11783-11 DDIs, 1-2 dozen new data representations).  Some of the features (recommended units of measure, localization into 38 languages) are artifacts of Deere's implementation and are not necessarily kept current as new things are added.
Going forward, there is discussion of simplifying this system in parallel to the removal of detailed machinery awareness within ADAPT. The breakdown of numeric representations is based a different set of business needs and is not necessarily relevant for the ADAPT standard.    E.g.s
-Particularly in the yield Numeric Representations, there is confusion between Wet, Dry and Total representations, and separation of Forage and Cane (ostensibly simply to associate different default units).
-There is separation between "Setpoint" and "Target" values, conceivably representing the delta between machine capability and agronomic intention.
-There are types defining things handled differently in ADAPT.  E.g., vrABRowSpacing.  Guidance Pattern has SwathWidth property that simply takes a number and unit of measure.
-There are numerous types for use cases that are, at best, at ADAPT's edges:  E.g., vrMaxBestFitDrainCutDepth: Maximum cut depth for Best Fit Drain
-There are types for specific business cases that are not a one-to-one fit into ADAPT.  E.g., vrSeedsPerContainer, vrSeedsPerBag, vrSeedsPerSack
Issues for discussion
-Should we simplify the Representation System in the forthcoming ADAPT Standard to start with a smaller subset of types for common use cases, defining a process to expand these types as new use cases emerge?
-Do Numeric Representations and Enumerated Representations belong in the same system?
-Should we require all data be passed in a common unit defined in each Numeric Representation?

-Should we simplify the Representation System in the forthcoming ADAPT Standard to start with a smaller subset of types for common use cases, defining a process to expand these types as new use cases emerge?
In my opinion, yes.
-Do Numeric Representations and Enumerated Representations belong in the same system?
This is a bit tricky. I think that we need to be able to handle variables that collect string, numeric, bool, and enumerated values. How those objects are modeled in a world that seems to be drifting away from strong typing - I don't know.
-Should we require all data be passed in a common unit defined in each Numeric Representation?
Yes, though keep in mind that there are "unitless" numeric values we may want to capture. I fully support designating a default UoM that the data will be expressed in. If that UoM differs from what the producer or consumer uses internally, it becomes an exercise left to the implementer to do the proper conversions.

See #87 and its child issues for eventual resolution of these issues.  Final Data Type Definitions are published here:  https://github.com/ADAPT/Standard/blob/main/adapt-data-type-definitions.json


Preferred kernel not selected by default

VS Code Version: Insiders
OS Version: Mac

Steps to Reproduce:

Install latest insiders
Install VS Code Jupyter extension
Open an empty workspace
Select Python interpreter and set to xyz
Create a blank jupyter notebook,
The kernel should be set to xyz

However its set to Select Kernel & when running a cell, user gets prompted to select a kernel.
User should not get prompted to select a kernel in this case (there are no other kernel providers, VS Code should just use the preferred kernel)
FYI - This is a regression & i understand you are in the end game
@jrieken @rebornix /cc

Does this issue occur when all extensions are disabled?: Yes/No

Note: This doesn't just apply to new notebooks, but existing notebooks as well.
Hence existing users will now have to always select a kernel (whether its a new or existing notebook)

Going back to read the old thread here. I believe this change is intentional based on this comment here:
https://github.com/microsoft/vscode/issues/122415#issuecomment-829109276
It's actually good here that there is a distinction now between VS Code's selection and what items might be set as preferred. In that comment there are two cases listed:

Only one kernel available, none selected. Implicit selection happens so that kernel shows in the UI and
Multiple kernels available, none selected. No kernel selected in status bar. Kernel picker is shown on execute. Preferred items get bumped to top of picker list

I believe our request to discuss here is the following:
3. Multiple kernels available, none selected, one preferred. Implicit selection happens so that the one preferred kernel is shown in the UI and is launched automatically without selection.
4. Multiple kernels available, none selected, > 1 preferred. No kernel selected in status bar. Kernel picker is shown on execute. Preferred items get bumped to top of picker list
This way if Jupyter does what we discussed (we always set just one preferred controller for documents) we get the behavior that we want where a customer can open a new blank notebook or pull a notebook from the internet and be able to click launch right away with our best guess as to a correct kernel. But it's still fair to the multiple extension cases where we don't get special priority if another extension is also setting a preferred controller.
I do get that from an API perspective this might be thorny. As extensions can add affinities on the fly. So having a UI difference between one preferred and many preferred might be difficult.

Multiple kernels available, none selected, one preferred. Implicit selection happens so that the one preferred kernel is shown in the UI and is launched automatically without selection.

Yes this is desired behavior.

Multiple kernels available, none selected, > 1 preferred. No kernel selected in status bar. Kernel picker is shown on execute. Preferred items get bumped to top of picker list

Yes this is desired behavior.

But it's still fair to the multiple extension cases where we don't get special priority if another extension is also setting a preferred controller.

Yeah, maybe fair but it likely encourages extensions to always mark their controllers a preferred.
What concerns me most is the UX - how do users learn that there is a difference between multiple kernels and 1 preferred vs multiple kernels and 0 or 2+ preferred? Today the model is very simple: 1 kernel -> that's suggested, 2+ kernels -> pick one.

Today the model is very simple: 1 kernel -> that's suggested, 2+ kernels -> pick one

Shouldn't it be 2+  from multiple extensions then pick one.
Ie if only one extension is providing multiple, why should the user pick one when that extension is in control of all kernels.
It makes sense if there are multiple from multiple extensions, else to me preferred has very little value (just sorts stuff).

On a side note, I'm not sure how we're supposed to run any test at this stage.
We have > 2 kernels on our machines (& CI) & now all of them hang because of this prompt.

Another challenge we have today is with testing.
We have a lot of kernels on CI servers, generally around 4 , sometimes more , and we end up with prompts in each test when we run cells (with new temporary notebook files). The current work around (hack) is to executes the command that forces vscode to use the defined kernel Id. That had it's own problems, to be expected, as they are workarounds (that's what one gets with hacks).
This would  be a problem for other extension authors too, testing with multiple kernels.
Changing our tests to have a single kernel, is going to take a lot of effort.

@jrieken after having seen some of the issues users ran into in diary study, i think the current behaviour is the right UX (discussed with @claudiaregio).
I'm closing this issue for now, as there's no point leaving this open.


sharing model

https://github.com/gytis-ivaskevicius/flake-utils-plus/blob/51cb739c9c9c2258bc70747eb7bc22975ae244bd/systemFlake.nix#L5
What is the rational of defaulting to inputs rather than have people make use of inputs in their top level flake?
The inputs thought model as extraArgs neglects the "receptionist" thought model of the flake.nix.
I think flake.nix as receptionist is an important feature of the distributed future of flakes, "to show new guests around": Hans visit's James' repo on github. flake.nix gives him the map and tells at least the outline of the story.
What cases cannot be covered by receiving inputs in flake.nix and dispatching eg overlays via input.otherflake.overlay or inputs.otherflake.nixosModules?
I think it was not an afterthought to make those two public static api of flakes themselves, in the first place by the designers of flakes.
One thing that comes to my mind ...
In a sharing model based on overlay it is not always trivial to pull in the following:
let
   bob = {
     kakoune = {};
     chrome = {};
   };
   alice = {
     kakoune = {};
     chrome = {};
   };
in
[
  bob.kakoune
  alice.chrome
]

flake-utils.lib.simpleFlake solves this by implicitly doing a namespacing of overlays.
Should we really go down that route and just globally pass through inputs, by default?
→ What this does not do is expose and promote encapsulation via the top level flake api.
/cc @Pacman99 @nrdxp

Just to clarify, why is it passed globally like so:

Input can be any git repo which is very convenient for package sources
Usually people implement it as inputs instead of  { inherit inputs; } (if that makes sense) - this creates an issue of input name conflicts
Currently fup uses it to implement a pretty dam cool repl and to auto-populate nix.registry

Nice!
ad 1. devos implements this feature as srcs coming from pkgs/flake.nix
ad 3. No interference with sharing model, here.
ad. 2. Are there known use cases where inputs bring additional attributes that cannot be captured via sharedModules & sharedOverlays?

inputs is just an attribute - there is no problem passing it manually using  sharedModules / sharedOverlays but that just adds additional complexity to the user, and really - I do not see any downsides in passing it automagically

If I had a crazy idea and needed a package from inputs somewhere and did:
{
  nativeBuildInputs = [
    inputs.some.packages.${pkgs.stdenv.system}.mypkgs
  ];
}

Does this have potential serious odds?
It is a somewhat hyper-namespaced alternative to namespacing pkgs as simpleFlake does?
To understand my concerns, one has to understand that I'm mainly concerned with the sharing model, not if it "works for me". Nixos sharing model has potential, since traditionally everyone cooked their own soup and users where just proficient enough to plug things together. But that model clearly fails for less proficient users, so I'm trying to figure out what is it that we should strongly encourage and strongly discourage so that people present a nice sharing interface effortlessly, by default.

Maybe we have indeed to fall back to pkgs namespacing to unbreak the "blessed" (overlays) sharing model.
If we can make the built-in model work, people might not be in need of a workaround that clutters best practices.

I've seen users do inputs.some.packages.${pkgs.stdenv.system}.mypkgs and I am not a fan of this approach. In this particular case, I recommend just defining a custom overlay and using it
As for sharing issue - this is what I do:
My Overlays definition
Normal packages are named properly (like lightcord), packages that I wrapped/customized are prefixed with g-. This means that other people can import my overlay without worrying about conflicts.
Also, may I add that I am exporting packages which are just overlays overlayed on top of nixpkgs (if that makes sense) Flake implementation
In case overlay overwrites something like in the example above - this solution pops into my mind:
final: prev: {
   alice = inputs.alice.overlay final prev;
}

this way we can install alice's kakoune by simply typing pkgs.alice.kakoune

Nice context you give!
final: prev: {
   alice = inputs.alice.overlay final prev;
}

Unfortunately, if alice also defines a module using one of her packages, we would break that, if we also tried to use it.

I don't think that I caught the issue. Can you elaborate on the use case that this does not cover?

Currently, fup does not a minimum consensus default on how to export packages, it expects the user to provide a packageBuilder, instead.
On the other hand, it might promote the overall sharing model.between fup uswrs, if fup would adopt sane packges outputs by defult.
To lay the foundations on package filtering, I opened an upstream issue https://github.com/numtide/flake-utils/issues/27
For fup, the task would be to re-export overlays per system as packages, similar to what simpleFlake does.

So now there is a +- usable version packagesFromOverlaysBuilderConstructor & overlaysFromChannelsExporter in staging, which closes this issue.
There is also modulesFromList exporter which expands the sharing model to modules.
None of them are currently "activated" by default. We could discuss doing soe once they are completely stable and validated.


enlarge ui and add suggestions

enlarge ui and add suggestions.
edit: add paused when open console.

@LauraWebdev
Updated, check plz.
I have reworked the messy UpdateUi() method and now it has become much better.
If there are any shortcomings, please point them out as I am a beginner in C#.


Listen for grammar changes of the editor and update store

Noticed while testing #665 and frequently switching grammars:

Looks good. I had noticed that issue as well. Nice that you found a fix without adding complexity.


Hard crashes when you click on any UV related buttons

OS: MacOS 10.15.7
Unity: 2020.3.* (tested on several minor builds)
RCSG: 1.562 (Asset Store) and 1.569 (GitHub)
When you click on any of these buttons in the surfaces window, a hard crash occurs.

Looking at the dump it refers to the С++ function GetSurfaceMinMaxTexCoords.

Cannot reproduce, can you supply us with the scene you used when it crashed?

Do you still have this issue @olegmrzv?

Closing this issue as I am unable to reproduce, and OP has not responded in several months.


Multiple file uploads on same page

I have multiple file uploads in same page(let's say each uploader for each element) and when I click 'browse' on third  element's uploader, it gets third element's model first and then click event is again triggered and gets the first element's model(it also has uploader) and the uploading takes place in wrong element in the end. Why is this happening? Multiple file uploaders is not supported or is there any other logic happening behind?
PS: One uploader added in the component for all elements, but has different names(model's ID is givan as name.)
Thanks.

@aysha-rafa I had this same issue. It took me a while to realize I was not passing unique values for the for option to the file-upload component. Obviously, I'm just guessing here. Example (using concat from Ember Composable Helpers
{{#each model as |item|
  {{#file-upload name=(concat 'upload-' item.id)
                 for=(concat 'upload-' item.id)
                 accept="image/*"
                 multiple=true
                 onfileadd=(action "uploadImage")}}
     Add an Image.
  {{/file-upload}}
{{/each}}

I hope that helps.

@jayvarner Thank you for the reply. I was passing a unique ID, but my problem was this  which is solved now.


Mapping between workspaces and  FF containers

Description
This is a feature request which, in my use case, would be a killing feature. I'm sure that is not straightforward but it would be really interesting to have the option to map workspaces to firefox containers. In a a simple way, it would be convenient to have the possibility of running each workspace in a different container. Let me know if further explanation is required.

Hello @imyelmo! Yes, please expand. What are workspaces in the context of Zen?

Sorry but I was on vacations til yesterday.
In zen seems to be possible to have different workspaces:

I think that it would be extremely useful that each workspace will run in a different firefox container. It is something interesting but if it is not considered initially, its inclusion in the development could not be straightforward later.

This isn't the Zen you're looking for: https://github.com/zen-browser/desktop


potential bug: unexpectd Initial ACK frame

An Initial packet containing only ACK, PADDING frames is always being sent by the client, even if the server sends the CRYPTO frame containing the ServerHello in a Handshake packet instead (e.g., Caddy).
Other QUIC client implementations including Firefox and Chrome will only send Initial ACK frames only if the server responded in Initial packet space.
I wonder if is this an intended behavior for quic-go with a specific reason or not? Thanks for your time.

In the case where this Initial packet is considered unexpected, it neither breaks the quic connection nor is essential to the quic connection. The communication could be established normally with or without this packet.
Essentially this Initial ACK has no effect when a server is not responding in Initial packet space.

Code to replicate this behavior is attached.
package main

import (
	"context"
	"crypto/tls"
	"net"
	"time"

	"github.com/quic-go/quic-go"
)

func main() {
	udpLaddr, err := net.ResolveUDPAddr("udp4", ":0")
	if err != nil {
		panic(err)
	}

	udpConn, err := net.ListenUDP("udp4", udpLaddr)
	if err != nil {
		panic(err)
	}

	udpRaddr, err := net.ResolveUDPAddr("udp4", "quic.tlsfingerprint.io:443")
	if err != nil {
		panic(err)
	}

	ctx, cancel := context.WithTimeout(context.Background(), 3*time.Second) // 3s handshake timeout
	defer cancel()
	_, err = quic.Dial(ctx, udpConn, udpRaddr, &tls.Config{
		ServerName: "quic.tlsfingerprint.io",
	}, &quic.Config{})
	if err != nil {
		panic(err)
	}

	select {}
}

Some quick experiments showed that neither Firefox nor Google Chrome (presumably Chromium) sends this Initial ACK frame, unless the server sends its ServerHello in Initial packet space (www.google.com does this).

Are you sure the server didn't send an Initial packet as part of a coalesced packet? It's not possible to send an ACK if there's nothing to acknowledge.

@marten-seemann I think you are right. The server DOES send an Initial Packet as part of the coalesced packet.
Are you aware of any compelling reason for these well-known implementations to be not ACKing to the InitialPacket?

For your reference: The first packet received from the server is made of two packets, one in Initial with ServerHello and one in Handshake with EncryptedExtensions.
And thank you for responding so quickly!

RFC 9001 says:

a client MUST discard Initial keys when it first sends a Handshake packet and a server MUST discard Initial keys when it first successfully processes a Handshake packet

The only purpose of this ACK is therefore to allow the server to generate an RTT measurement.

Just before this line:

The successful use of Handshake packets indicates that no more Initial packets need to be exchanged, as these keys can only be produced after receiving all CRYPTO frames from Initial packets.

I would interpret this as "once client sees a Handshake packet from the server, no more Initial packet needs to be sent (and therefore dropping the ACK)". Perhaps we could make this an optional configurable feature?
What's your opinion on this?

That's not correct. See the sentence that I quoted, which exactly specifies when Initial keys are dropped.

I understand what you were saying, the initial keys are dropped (so no more Initial Keys CAN be sent) exactly after successfully sending a Handshake packet.
However, I would interpret "no more Initial packets need to be exchanged" as "you MAY, but you don't have to exchange any initial packets". It is weaker than "a client MUST discard Initial keys". And this also aligns with what we see from modern web browsers.

"you MAY, but you don't have to exchange any initial packets".

That's correct, and exactly what we're doing. As I mentioned above, giving the server the chance to generate an RTT measurement has value, which is why we send this ACK.

Would you welcome a PR adding a switch to turn off this behavior?

Why would you want to make this configurable? I see no advantage in not sending an ACK.

It is more of a fingerprint-ability issue. Like uTLS, we are also creating an experimental uQUIC which is based on quic-go to prevent QUIC fingerprinting as demanded by censorship circumvention community.
Therefore we would like to be able to at least replicate all plaintext communications, i.e., everything sent in Initial packet space.
If you don't feel it would be a useful addition, we would keep this feature solely in uQUIC.

I see, thanks for adding some context here.
What's the reasoning behind not sending the ACK in Chrome and Firefox?

reasoning behind not sending the ACK in Chrome and Firefox?

Unfortunately I don't have an answer for that, it could be just one of their standard-compliant nuances which contributes to their special fingerprints.

I'm going to close this issue. There are tons of fingerprinting opportunities in a QUIC stack: frame ordering, transport parameter ordering, greasing values used, just to name a few that apply to the very first packet sent. There's a lot more after that.
quic-go has never tried to hide its identity, and replicating Chrome's / Firefox's behavior in all details what be massive endeavor (and a moving target).


Plan B to Heroku

If anyone is still looking for Plan B to Heroku, I have been using Azure for a little while now since it is back and both the NightScout website and this wonderful write app from Libre to NightScout works seamlessly.
Follow the same steps as migrating nightscout website from the link below and replace where necessary with librelink-up configurations
https://nightscout.github.io/vendors/azure/migrate/

Getting this error anyone know what I need to do?
2022-11-07T21:31:31.511Z INFO - Logging is not enabled for this container.
Please use https://aka.ms/linux-diagnostics to enable logging to see container logs here.
2022-11-07T21:31:33.652Z INFO - Initiating warmup request to container libre2nightscout_0_9ad090eb for site libre2nightscout
2022-11-07T21:31:54.087Z INFO - Waiting for response to warmup request for container libre2nightscout_0_9ad090eb. Elapsed time = 20.4351059 sec
2022-11-07T21:32:09.210Z INFO - Waiting for response to warmup request for container libre2nightscout_0_9ad090eb. Elapsed time = 35.5578431 sec
2022-11-07T21:32:24.374Z INFO - Waiting for response to warmup request for container libre2nightscout_0_9ad090eb. Elapsed time = 50.7215846 sec
2022-11-07T21:32:41.141Z INFO - Waiting for response to warmup request for container libre2nightscout_0_9ad090eb. Elapsed time = 67.4891618 sec
2022-11-07T21:32:58.170Z INFO - Waiting for response to warmup request for container libre2nightscout_0_9ad090eb. Elapsed time = 84.5175334 sec
2022-11-07T21:33:15.160Z INFO - Waiting for response to warmup request for container libre2nightscout_0_9ad090eb. Elapsed time =<PHONE_NUMBER> sec
2022-11-07T21:33:31.143Z INFO - Waiting for response to warmup request for container libre2nightscout_0_9ad090eb. Elapsed time =<PHONE_NUMBER> sec
2022-11-07T21:33:47.214Z INFO - Waiting for response to warmup request for container libre2nightscout_0_9ad090eb. Elapsed time =<PHONE_NUMBER> sec
2022-11-07T21:34:02.297Z INFO - Waiting for response to warmup request for container libre2nightscout_0_9ad090eb. Elapsed time =<PHONE_NUMBER> sec
2022-11-07T21:34:17.397Z INFO - Waiting for response to warmup request for container libre2nightscout_0_9ad090eb. Elapsed time =<PHONE_NUMBER> sec
2022-11-07T21:34:32.489Z INFO - Waiting for response to warmup request for container libre2nightscout_0_9ad090eb. Elapsed time =<PHONE_NUMBER> sec
2022-11-07T21:34:47.582Z INFO - Waiting for response to warmup request for container libre2nightscout_0_9ad090eb. Elapsed time =<PHONE_NUMBER> sec
2022-11-07T21:35:02.777Z INFO - Waiting for response to warmup request for container libre2nightscout_0_9ad090eb. Elapsed time =<PHONE_NUMBER> sec
2022-11-07T21:35:18.352Z INFO - Waiting for response to warmup request for container libre2nightscout_0_9ad090eb. Elapsed time =<PHONE_NUMBER> sec
2022-11-07T21:35:24.762Z ERROR - Container libre2nightscout_0_9ad090eb for site libre2nightscout did not start within expected time limit. Elapsed time =<PHONE_NUMBER> sec
2022-11-07T21:35:25.207Z ERROR - Container libre2nightscout_0_9ad090eb didn't respond to HTTP pings on port: 80, failing site start. See container logs for debugging.
2022-11-07T21:35:25.709Z INFO - Stopping site libre2nightscout because it failed during startup.

Hi were you successfull with this? I understand how to get the webpage via Azure up but not sure how to make Azure talk to Libre3 ? This code do not have easy option that points to deploy to Azure similar to drploy to Heroku. Can you help?

Hi @durgok, Yes I was able to keep this running by deleting the webapp and starting fresh in Azure.
When setting up use the repository instead of nightscout
And similar to nightscout add relevant username and password in configuration

@Chapagain43 Thanks, but I went with paid service nsdbe10. Heroku shut overnight because I used all the dynos in the month’s quota and I needed something immediately as my daughter was looping. So opted for the service, they charge me ~$5 every month, and it was so seamless service. And they help manage all the other stuff like upgrades/DB size/security etc., Since I was anyhow paying the same for Heroku, just switched over. Appreciate you getting back though. Am sure this response will be useful to others who come searching for Azure option.


[SPARK-9578] [ML] Stemmer feature transformer

jira: https://issues.apache.org/jira/browse/SPARK-9578
Classical Porter stemmer, which is implemented referring to scalanlp/chalk
https://github.com/scalanlp/chalk/blob/master/src/main/scala/chalk/text/analyze
@jasonbaldridge Let me know if you're interested.
I compared the following implementations:
http://tartarus.org/martin/PorterStemmer/scala.txt
https://github.com/ifesdjeen/jReadability/blob/master/src/scala/main/com/jreadability/main/Stemmer.scala
https://github.com/aztek/porterstemmer/blob/master/src/main/scala/com/github/aztek/porterstemmer/PorterStemmer.scala

Test build #47606 has started for PR 10272 at commit ddb26da.

Test build #47606 has finished for PR 10272 at commit ddb26da.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * class Stemmer (override val uid: String)\n

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47606/
Test PASSed.

There are a few quirks regarding formatting.
Also, I'm wondering if the different "step" methods should be documented or renamed so we get what they're doing without skimming over the code.

@BenFradet, Thanks for helping review. I added some comments, yet listing all conditions may not be necessary.

Test build #47903 has started for PR 10272 at commit ff03152.

Test build #47903 has finished for PR 10272 at commit ff03152.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * class Stemmer (override val uid: String)\n

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47903/
Test PASSed.

LGTM

FWIW, I did a Scala adaptation of a Java PorterStemmer here too: https://github.com/utcompling/Scalabha/blob/master/src/main/scala/opennlp/scalabha/lang/eng/PorterStemmer.scala

Thanks @jasonbaldridge for taking a look. If you're interested, I'll close the PR thus you can send a new one. I can help review and compare the performance.
Thanks @BenFradet for helping review.

@hhbyyh: I'm pretty slammed right now with other things, so if you'd like to go ahead and compare and choose whichever, that's totally fine with me. Thanks!

Current implementation still outperforms the one from https://github.com/utcompling/Scalabha/blob/master/src/main/scala/opennlp/scalabha/lang/eng/PorterStemmer.scala, by about 70%.
This is ready for review now.

@hhbyyh I would try to avoid maintaining a stemmer implementation in MLlib. This is not a distributed algorithm and there exist several implementations from NLP libraries. The best option is to introduce a dependency and wrap the stemmer implementation there. If we made some improvements to an existing stemmer implementation, we should consider contributing to it.

@mengxr
Thanks for taking a look.
The comment from Joseph (https://issues.apache.org/jira/browse/SPARK-5571?focusedCommentId=14632052&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14632052) seems to prefer to add the code directly.
We can put this on hold if necessary.

@hhbyyh Joseph's comment was about carefully introducing new dependencies. If we pick a library that doesn't depend on many others, it should be safe for us. There are several packages containing Porter stemmer, e.g., lucene, CoreNLP, and chalk. Lucene is a lightweight library, but used by many other systems. So it is not a safe choice. CoreNLP is licensed under LGPL, so not an option here. chalk seems okay to me by looking at its dependencies.
I'm a little worried about the cost if we maintain our own implementation in MLlib. We cannot leverage other NLP projects (where the experts are) on possible improvements. So could you take a look at chalk?
@jasonbaldridge To add chalk as a dependency, we need chalk releases for both Scala 2.10 and 2.11. But I only see 2.10 releases on maven central. Do you have plans for publishing new releases for both 2.10 and 2.11?

@mengxr Chalk isn't under current development unfortunately, and I'm not sure whether I'll be getting back to it. Another option might be to add it to the lib-text library, which is being maintained and updated, and might have some other useful things for you:
https://github.com/peoplepattern/lib-text
cc @eponvert and @dlwh

(Totally not paying attention to this issue.)
Epic has a PorterStemmer as well. (Well, I re-took it from Chalk.)
https://github.com/dlwh/epic
I've been neglecting Epic to some extent of late, but it's there and
available.
I'm not sure it's worth adding a large dependency just for that, but just
FYI.
On Mon, Mar 21, 2016 at 9:21 PM, Jason Baldridge<EMAIL_ADDRESS>wrote:

@mengxr https://github.com/mengxr Chalk isn't under current development
unfortunately, and I'm not sure whether I'll be getting back to it. Another
option might be to add it to the lib-text library, which is being
maintained and updated, and might have some other useful things for you:
https://github.com/peoplepattern/lib-text
cc @eponvert https://github.com/eponvert and @dlwh
https://github.com/dlwh
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/10272#issuecomment-199631674

IMO more specific or complex domain-specific stuff should live outside of core, until such time as there is clear demand across a wider user base that justifies the maintenance cost of including it. Already Spark ML has a large maintenance & code review burden just with the algos and feature transformers that are already in there.
The whole point of an API for pipelines is to enable external libraries for more specific use cases. This is doubly the case when well-known and robust libraries already provide the functionality. As you can see from your PR, implementing one's own stemmer transformer using one of the external NLP libs is a few lines of code.
Things like NLP (and image, video and audio processing, for example) should start life as a Spark package. How about looking at contributing to https://github.com/mengxr/spark-corenlp and wrapping the CoreNLP stemmer functionality as a transformer?

@jasonbaldridge @mengxr I went ahead and opened https://github.com/peoplepattern/lib-text/issues/21, though I've not scheduled it for a target release yet

I'm fine to put this on hold like previously stated.Thank you all for the discussion. It should have provided enough information for anyone interested in using Stemmer with Spark.
I plan to put this in the topic modeling Spark package for now. Will send a link here afterwards.

@MLnick CoreNLP is GPL, so I would worry that some people would use it as though it were part of an ASL suite. Spark-CoreNLP is correctly licensed as GPL, but there should be some big warning flags in the README so that people don't inadvertently use it in a way that is inconsistent with the GPL.
FWIW, I would strongly prefer an "NLP for Spark" package to be ASL, so Spark-CoreNLP isn't useful (though it's great stuff, objectively speaking).


Added properties in DropdownButtonFormField to match DropdownButton

Description

Added properties available in DropdownButton to DropdownButtonFormField.
Added dart docs for each properties of DropdownFormField.

This PR also solves the problem where DropdownButtonFormField overflows when DropdownMenuItem's child is larger than the viewport as shown below:
Before

DropdownButtonFormField(
              value: _selectedCompany,
              hint: Text('Select a company'),
              onChanged: (value) => setState(
                () => _selectedCompany = value,
              ),
              items: [
                'ADBL (Agriculture Development Bank Limited)',
                'CORBL (Corporate Development Bank Limited)',
                'GMFIL (Guheshowori Merchant Bank & Finance Co. Ltd.)',
              ]
                  .map(
                    (company) => DropdownMenuItem(
                      value: company,
                      child: Text(company),
                    ),
                  )
                  .toList(),
            ),

After

DropdownButtonFormField(
              isExpanded: true,
              isDense: true,
              icon: Icon(Icons.ac_unit),
              style: TextStyle(color: Colors.blue),
              elevation: 24,
              autovalidate: true,
              value: _selectedCompany,
              hint: Text('Select a company'),
              onChanged: (value) => setState(
                () => _selectedCompany = value,
              ),
              items: [
                'ADBL (Agriculture Development Bank Limited)',
                'CORBL (Corporate Development Bank Limited)',
                'GMFIL (Guheshowori Merchant Bank & Finance Co. Ltd.)',
              ]
                  .map(
                    (company) => DropdownMenuItem(
                      value: company,
                      child: Text(company),
                    ),
                  )
                  .toList(),
            ),

Tests
I added the following tests:

autoValidate test in dropdown_test.dart
Tests for other changes are covered by DropdownButton's test itself.

Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I signed the [CLA].
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read [Handling breaking changes]). Replace this with a link to the e-mail where you asked for input on this proposed change.
[x] No, this is not a breaking change.

@googlebot CLA checked.

@shihaohong I have already added dart docs for isExpanded and isDense. Since DropdownButtonFormField is actually wrapping DropdownButton, so the test for DropdownButton should cover DropdownButtonFormField too as all those newly added properties are actually passed into inner DropdownButton.

@sarbagyastha Right, but let's say I forget to pass down isDense to DropdownButton in the DropdownFormField implementation, aka a future developer accidentally removes your line that passes it down into DropdownButton. Then, even though I'm correctly calling:
DrowdownFormField(
  ...
  isDense: true
  ...
),

the result is a non-dense widget. It's still appropriate to include tests for DropdownButtonFormField for the new parameters you've introduced.

@shihaohong Added tests for isExpanded and isDense.

@shihaohong Added more tests and formatted as you suggested.

@shihaohong couldn't write test for elevation. Since the elevation is actually a BoxShadow painted using BoxPainter. I couldn't find the RenderObject to test.

@sarbagyastha If you look at /packages/flutter/test/material/chip_test.dart, there are examples of elevation tests in there. See the test "Chip elevation and shadow color work correctly".

@shihaohong the elevation used there is different from the one in this PR. In chip_test, it was actually a material widget so finding Material was not a problem. But in dropdownbutton, It's using integer shadows which is being mapped into BoxShadows(using kElevationToShadow) and is implemented using BoxPainter. :confused:

@shihaohong What shall i do now? I guess we can skip the test for elevation.

I think @HansMuller might have a pointer tohow to test for elevation

It's a little tricky, but you can verify that the menu is painting a shadow. Here's an example of doing this: https://github.com/flutter/flutter/blob/master/packages/flutter/test/material/mergeable_material_test.dart#L226

@shihaohong  @HansMuller I tried everything but couldn't create the test for elevation. :tired_face:

@sarbagyastha What seems to be the blocker? What happens when you try to implement it in a similar way with the example that Hans provided?

@shihaohong couldn't get how rrect boundaries could be defined.

I'll try to see if I can come up with something in the next day! In the meantime, you can continue trying to get something working :)

I figured out how to do it:
  testWidgets('Dropdown form field - default elevation', (WidgetTester tester) async {
    final Key buttonKey = UniqueKey();
    debugDisableShadows = false;
    await tester.pumpWidget(buildFormFrame(
      buttonKey: buttonKey,
      items: menuItems,
    ));
    await tester.tap(find.byKey(buttonKey));
    await tester.pumpAndSettle();

    final BoxShadow boxShadowOne = kElevationToShadow[8][0];
    final BoxShadow boxShadowTwo = kElevationToShadow[8][1];
    final BoxShadow boxShadowThree = kElevationToShadow[8][2];

    const Rect originalRectangle = Rect.fromLTRB(0.0, 0.0, 800, 208.0);
    final RRect rrectOne = RRect.fromRectAndRadius(
      originalRectangle.shift(
        boxShadowOne.offset
      ).inflate(boxShadowOne.spreadRadius),
      const Radius.circular(2.0),
    );
    final RRect rrectTwo = RRect.fromRectAndRadius(
      originalRectangle.shift(
        boxShadowTwo.offset
      ).inflate(boxShadowTwo.spreadRadius),
      const Radius.circular(2.0),
    );
    final RRect rrectThree = RRect.fromRectAndRadius(
      originalRectangle.shift(
        boxShadowThree.offset
      ).inflate(boxShadowThree.spreadRadius),
      const Radius.circular(2.0),
    );

    final Finder customPaint = find.ancestor(
      of: find.text('one').last,
      matching: find.byType(CustomPaint),
    ).last;

    expect(
      customPaint,
      paints
        ..save()
        ..rrect(rrect: rrectOne, color: boxShadowOne.color, hasMaskFilter: true)
        ..rrect(rrect: rrectTwo, color: boxShadowTwo.color, hasMaskFilter: true)
        ..rrect(rrect: rrectThree, color: boxShadowThree.color, hasMaskFilter: true),
    );
    debugDisableShadows = true;
  });

Please make a test for custom elevation as well. If you can, see if you can simplify the logic as well.

Great @shihaohong ! Added tests for default and custom elevations.

@shihaohong Thanks to you too for coming up with the tests and guiding. I got to learn a lot regrading tests and style guides

@shihaohong I guess it's ready to get merged now.

@shihaohong @HansMuller What shall be done next?

@sarbagyastha This looks fine to me, but let's wait until @HansMuller finishes his review. He should be back at the start of next week :)

@HansMuller

@HansMuller Cloud you please review the changes. 😁

@HansMuller

The changes LGTM. We should be able to land this when the build is green.


Refactor globalVars into a Separate Class for Improved Encapsulation

This PR refactors the globalVars implementation to improve code structure and encapsulation:

Moved globalVars from extension.ts to a Separate File

Isolated the global variables to a dedicated file.

Converted from Namespace to Class with Getters and Setters

globalVars is now a class with getter and setter methods, for better encapsulation and allowing controlled access and modification of global state.

LGTM thanks @Achal1607  for making these changes


Updates toolbox instalation instructions

fix #118

Since the project uses Python 3. Shouldn't we use python -m pip install serenata-toolbox?
What do you think @jtemporal ?

@lipemorais I think that's totally "doable" but it should be in another issue. Also would open up the issue on the other repos to do the same 😉
Right now the focus is only to show that serenata-toolbox is only a pip install away.
@cuducos any thoughts?

I completely agree @jtemporal

Coverage remained the same at 85.358% when pulling c92e51043686120e79b1ebec7ff786b904404743 on updates-documentation-pypi into 114b44ab350d35416d06d8eff8ec26874aa18f1b on master.

Coverage remained the same at 85.358% when pulling 9f8dcf816172e4f57ab64cb5f7b866937a7b0763 on updates-documentation-pypi into 32ec5c1975b50580406feda31585bd4ffa5502ab on master.


[9.x] Allow disabling checks for expired views

One of the changes @nunomaduro made in #44487 reminded me of a similar PR (#31206) I'd done about 2 years ago that was merged and then reverted due to a reported edge case bug.
The reported bug was that the view:cache was not correctly caching registered vendor views. I wish I'd have looked closer 2 years ago, because I've been testing all day, and everything seems to be working well to me, and I can't find any noticeable change in the framework that would've changed this behavior.
With that said, this is a resubmission of that original PR (#31206), with essentially no changes.  This change takes Nuno's optimization on compiled views a step further. The root of the problem is the $this->compiler->isExpired($path) method is a relatively expensive call, and it can get called a large number of times if:

you have a large number of views/components necessary for the page rendering
you render the same view/components multiple times on the page

So the goal is to avoid unnecessary calls to isExpired(). Nuno's change in #44487 solves scenario 2 by building up a local cache of file paths already compiled, and then checking for the path value in the array on subsequent calls.  The changes in this PR actually solve both scenarios because you can now explicitly tell the CompilerEngine to never check if the path is expired and to assume the file is compiled validly.  This is intended to be a production only feature, and will only work if you run php artisan view:cache as part of your deployment.
So to start, could I first get a little confirmation from others that vendor registered views are being compiling with artisan view:cache. After that the PR should be good to review.
Check out the previous PR for the performance improvement data.

How does this address the issues we had with the original PR?

Temporarily converting to draft while I figure out the best course forward for the edge cases.

Hi there. We fixed the failing tests on 9.x which are causing this PR to fail as well. Can you please rebase your PR with 9.x? Thanks

@browner12 feel free to resend this once you've figured out a way forward here. Thanks.


Installation error

Describe Your Problem:

Trying to install the plugin, I get these errors in the log below.
I just tried to install an older version 1.0.6 and didn't encounter the same errors. But also it doesn't seem to do anything and there is no plugin config info in the plugin config info window.
Logs:
[26/07/2024, 08:50:05] [Homebridge UI] Running Command: /opt/homebridge/lib/node_modules/homebridge-config-ui-x/plugin-upgrade-install.sh homebridge-midea-platform 1.1.0 /var/lib/homebridge/node_modules
[26/07/2024, 09:03:25] [Homebridge UI] [homebridge-midea-platform] Custom UI created.
[26/07/2024, 09:03:25] [Homebridge UI] [homebridge-midea-platform] node:events:497
      throw er; // Unhandled 'error' event
      ^

Error: write EPIPE
    at target._send (node:internal/child_process:879:20)
    at target.send (node:internal/child_process:752:19)
    at UiServer.ready (/var/lib/homebridge/node_modules/homebridge-midea-platform/node_modules/@homebridge/plugin-ui-utils/dist/server.js:105:17)
    at new UiServer (/var/lib/homebridge/node_modules/homebridge-midea-platform/homebridge-ui/server.js:149:10)
    at /var/lib/homebridge/node_modules/homebridge-midea-platform/homebridge-ui/server.js:276:10
    at Object.<anonymous> (/var/lib/homebridge/node_modules/homebridge-midea-platform/homebridge-ui/server.js:277:3)
    at Module._compile (node:internal/modules/cjs/loader:1358:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1416:10)
    at Module.load (node:internal/modules/cjs/loader:1208:32)
    at Module._load (node:internal/modules/cjs/loader:1024:12)
Emitted 'error' event on process instance at:
    at node:internal/child_process:883:39
    at process.processTicksAndRejections (node:internal/process/task_queues:77:11) {
  errno: -32,
  code: 'EPIPE',
  syscall: 'write'
}

Node.js v20.16.0

Plugin Config:
{
    "platform": "midea-platform",
    "name": "midea-platform"
}

Screenshots:

Environment:

Plugin Version:
Homebridge Version:  Homebridge v1.8.4 (HAP v0.12.2) (Homebridge 6E1A)
Node.js Version: Node.js v20.16.0
NPM Version: I don't know, where can I find that?
Operating System: hb-service

This is not an error with my plugin, but I updated the dependency with the just now released v1.1.1. Could you please try with this version?

This is not an error with my plugin, but I updated the dependency with the just now released v1.1.1. Could you please try with this version?

Thanks, I'll try the new version next week, I'm away from home on a trip until next tuesday.

Have you managed to try this out?

Have you managed to try this out?

Sorry for the delay! Things have been busy and I forgot about it. I just saw this notification and went back to try.
I didn't try 1.1.1 specifically, I went straight to v1.1.2-beta.6 and it worked perfectly. I was able to add the AC and it works. No errors on installation.
Thank you so much! :)


Feature request: Require a noop deployment before a real one

Hi :wave:
As far as I can tell, it's not possible to require noop deployments before real ones.
I think it would be a great feature to add (if it's possible 😅).
For instance: Require terraform plan (the noop) to run before terraform apply.

I have often thought about this feature myself but haven't looked into it. I'm sure there is a way to accomplish this, but need to make sure it is the correct way. I'll look into this as an experimental feature when time permits. Thank you for the suggestion!
Also, it should be noted that my team does kinda implement this in a way through reviews. If a user opens a PR for repo x my team is requested as a codeower for review. That user can run .deploy noop but not .deploy until they have an approval on their PR. We will not give an approval until the user has run .deploy noop and we have validated the changes of the noop look correct. This in a way enforces .deploy noop to happen before .deploy but that process is done by humans and is subject to mistakes. I like your suggestion much better and I'll do some research to see how this can be accomplished.
Thanks!

@jumoel I'm going to go ahead and close this issue out. If you would like to revisit this, please feel free to re-open and give me a ping!
The TL;DR of what we do at GitHub today is "gating" a full deploy (non-noop) through reviews. Engineers cannot do a full deploy until they have approvals on their PRs. However, an engineer can always do a .deploy noop before they get reviews to preview changes (assuming that CI is passing of course)


Has anyone tried to see if this will work with 0.8.2 beta?

Just wondering if anyone has tried this cookbook to deploy 0.8.2 Beta?

I've successfully installed v<IP_ADDRESS>, upgraded to v0.8.2-beta and finally upgraded to v<IP_ADDRESS>, so I'm gonna go ahead and close this.


github: reuse existing deployment when sending new notification

Try to list existing deployment with the same sha/environment/ref and reuse it if found. This allows ArgoCD notification to update the same deployment with multiple deployment status instead of creating a new one on each deployment notification.

Hi @jaredtbates I saw that you mentioned in your Initial pull request that you didn't find a way to preserve deployment id, what do you think about this patch?

@MrFreezeex could you please resolve conflicts and make sure that you have added tests?

@MrFreezeex could you please resolve conflicts and make sure that you have added tests?

Hi @pasha-codefresh! I just rebased the code but I can't realistically add test unfortunately. This whole function is not tested and AFAIK there are no tests on any Send method anywhere. The tests are up until the actual Send is done but then it's always untested code afaik,

LGTM, thank you


Add structured logging support for command logging

Align command logging messages to specification
Add command logging unified tests

A couple of leftovers that I opened separate tickets for:

Support for max document length: this involves plumbing the value all the way down from MongoClientSettings
Support for failureIsRedacted in spec tests.  This involves a bit of a rethink of how the value matcher is being used

JAVA-4752
JAVA-4753

Fixed the first TODO.  Removed the second after confirming that the assertions are properly executed.  I'm not sure anymore what I was concerned about.

I'm not sure what I was thinking was so hard about failureIsRedacted, but when I went back to look at it again it was all clear and simple so I just pushed the fix onto this PR.

Merging on command line since trying to preserve two commits


Cocoapods: Kingfisher classes not recognized

I am developing a framework that we archive as a .framework file and drop into a container application.
I have made sure to include the import Kingfisher statements at the top of the source files I am referencing Kingfisher classes in.
Getting compiler errors like "Use of undeclared type 'Resource'", "Use of undeclared type 'ImagePrefetcher'", etc when I set "Build Active Architecture Only" to "No" in Build Settings.
When I set it "Build Active Architecture Only" to "Yes", these errors disappear and I am able to build. Need to be able to set to "No" so our framework archives for all architectures.
Using v2.6.1 on Xcode 7.3.1.
Edit: Using v2.6.1, not v2.6.0; mentioned that I am using import statements

How did you include Kingfisher into your framework? You also need to build Kingfisher for all valid architectures if you want your own framework to be compatible with all architectures.

I've since updated to Xcode 8.3, Swift 3, and am using Kingfisher 3.6.2. Same issue. How would I specify to build Kingfisher for all valid architectures?

Maybe you need to specified it in the Pods target?
Some Ref. And I guess it could be done by a post install action in the Podfile.

And I am not sure if you are on the correct way to develop a framework which depends on another.
If my understanding is correct, you are trying to develop your own framework (and archive it as a .framework file) which depends on Kingfisher. First, the CocoaTouch framework (I am not sure whether your framework is a CocoaTouch Framework target, but Kingfisher is) is a dynamic framework and you may need to use lipo to combine x64, arm7 and arm64 architectures to a fat binary. Then when you are archiving it in your final app, you may need to resign it (and optionally strip the unused x64 part). The same thing has to be done for Kingfisher if you want to embed Kingfisher into the Framework folder in your framework. It is not a trivial work at all.
If you have no idea about what I am talking about, I suggest to use CocoaPods or Carthage to manage the dependency. You could just specify Kingfisher in your podspec or Cartfile, then deliver your framework as a dependency. It would be much easier.

Maybe related to #647
I suggest to upgrade both CocoaPods and Kingfisher version to see whether it could work.


Provide current mouse position with each mouse event

This PR will allow the mouse events to be shared across multiple positional-affecting input as the event name will eventually be changed to PositionalEvent or similar.
Breaking change
UIEvent.MousePosition should not be used unless targeting the mouse position directly, use MouseEvent.CurrentMousePosition instead.

I still don't understand what this is really achieving, and your breaking change comment isn't helping.
Beyond the rename, are you eventually hoping to pass a different "current" position in for each touch point? So:
MousePosition: The current cursor position from a dominating pointing device.
CurrentMousePosition: The current cursor position from any pointing device.

If so, this feels exceedingly convoluted, you can't use the terminology CurrentMousePosition for this.

Yes! that's pretty much the goal of this, sorry for not including it in the PR.


Bump version to 0.56.1-SNAPSHOT post release

I was under impression that doing this is a part of the release checklist?
Please merge this if it LGTY

I will merge this for the time being, since it is no worse than the current situation, and in the background will think on how we can make this work with the wiki-prescribed version string.


improved contrast ratio of colours based on accessibility test

i modified the colours to make sure they all rank well in terms of contrast ratio and accessibility. here are the "new" colours with very good rankings.
ideally, all our colours for text would be above 7.0 (AAA). i lightened the background from 13% to 16% so the scores are a bit off. but this is more or less what i'm proposing.

closes #167

Really liking these changes in general! It's looking good and easier to read. I'm going to revert the background lightening and bright text darkening because I feel like the design loses a lot of 'pop' and contrast with those changes. Also going to revert the button change because they should be done in the @nylira/vue-button lib, and I have some ideas for improving that (in a separate pr)


Initial commit

BREAKING CHANGE: first release (this message was added because of semantic-release version up rule ).

:tada: This PR is included in version 2.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Possibility for "authWithOAuth2" to bind to specified existing user

If I understood correctly and from what implentations I did, authWithOAuth2 automatically creates a new user if the email does not exist. There is no option to bind the provided auth to an existing account.
Study case: I create account using a gmail, but then I want to loggin (bind) twitch as social to same account. BUT my twitch is on another email, a yahoo mail.
So currently i cannot create a list of Connections (like discord has) for current user.
In my case i.m using twitch auth keys and google keys ( if provider by user ) to link and offer him some extra features from his youtube, twitch streams.

There is no option to bind the provided auth to an existing account.

Outside of the godocs, this at the moment is not documented very well, but you can link an OAuth2 account to an existing user as long as:

the OAuth2 account is not already linked to another user
there is a valid Authorization header sent with the authWithOAuth2 request (aka. the user is already authenticated)
the OAuth2 authentication happens in the same auth collection as the user

Or in other words, the following should work:
await<EMAIL_ADDRESS>"1234567890")
...
await pb.collection("users").authWithOAuth2(...)

Ok. I.ll try again, maybe i missed something. Thankyou.


chore: disable go test optimization for the mock framework limitation

Ref: https://github.com/bytedance/mockey?tab=readme-ov-file#error-function-is-too-short-to-patch

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Additional documentation e.g., design docs, usage docs, etc.:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.05%) to 58.441%

Files with Coverage Reduction
New Missed Lines
%

pkg/engine/operation/watch.go
1
85.93%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
7325

Relevant Lines:
12534

💛  - Coveralls


Error occurs when clicking on Try Again button

Steps to re-produce

Link a bank account form plaid sandbox
Make the account status to INVALID_UPDATED_USERNAME
Try to re-authenticate this account in update mode
provide username / passcode: user_good / pass_good, then provide OTP as 1234 and go to last step
It shows message saying "Incorrect username" , now click on Try Again button.

Plaid loader wraps up entire screen.
NOTE:

while trying to get item information using plaid-sdk it says 'ITEM_LOGIN_REQUIRED' instead of 'INVALID_UPDATED_USERNAME'

hi @arungogosoon thanks for the report. Two things here.

First, this is not an issue with react-plaid-link itself (this repo is just a little react wrapper around our standard Javascript SDK). This appears to be an issue within Link javascript code itself.
Second, the stack trace for this indicates to me this is from some legacy code that is going to be replaced very shortly. This rollout has already happened in production, but we have not rolled it out to sandbox yet.

Thank you for the report! I can send an update here once we roll out the change to sandbox so you can test this same scenario.


Custom warning class PyfarDeprecationWarning

We decided to implement a custom class for DeprecationWarnings called "PyfarDeprecationWarning" to avoid using warnings.simplefilter("always").

I usually use PendingDeprecationWarning and this works very well. The warning raises in my case.

I usually use PendingDeprecationWarning and this works very well.
We initially also used that Warning but it is not shown to the user by default. Thats why we thought making a new class is the cleanest way. If we wanted to use PendingDeprecationWarning, we had to manually switch it on every time we use it.


Zenodo saved trained model and results

downloading from zenodo although there are no check to see if the file downloaded correctly. Could add this to the download_data.sh script instead?

Thanks! is this ready to be merged?

not sure, but I think there's some extra stuff here because you don't have nbstripout installed. @Imraj-Singh can you manually run it and see if it makes any difference?

Hi @KrisThielemans, I did install it prior to adding, commiting and pushing that changed notebook. I ran nbstripout 4_setup_training.ipynb. I reran nbstripout 4_setup_training.ipynb, git added and it does not seem like it made a difference. Is there anything else I can do?

I'll merge it. @samdporter can then complain later :-)


Add BatarongOS

Add BatarongOS to your website

@1jammer1 Where is the actual code? The repository doesn't look to have any code.

https://github.com/batarongOS/ is GPL-3

@hsn10 https://github.com/batarong/batarong-installer is using unlicence this is the current version of the project and that version is no longer maintained

@artob im sorry for not responding earlier this is just a legacy repo this used to hold the code the code for batarong os now resides in multiple repos across the org this is the main config script for the distro resides here https://github.com/batarong/batarong.github.io/blob/main/install.sh


Staff page mouse over , image issue by sir

staff pe mouse over pe aisa aa raha hai isme photo aana chahiye, staff ka kyunki woh google se aati hai

@mitulpanch sir  image not convert different size.


Added STL and STEP case models for v1.0.

Added case models for v1.0. Includes bottom, top, and shell components. These have not been printed to establish that the fit is good #9

@MeisterGig Thanks :-). Re: the pins/anchors, I'm honestly not sure about whether these will snap when attaching the top and bottom pieces. Happy to tweak this to reduce how far they extend out.
Keen to hear how this prints!

@MeisterGig I've shortened the pins by 2 mm to minimise the risk of snapping when assembling the two halves.

Unfortunately i wasn't able to close it 🥺 The locking Pins interferes with the lip going around the edge. The lip prohibits the Case from bending to that the lib can snap in place.
This part would need to be able to bend out a little bit so the pin gets ito the hole:

Otherwise it seams to fit

When one pin is broken of though, i am able to close it at an angle

Although i just realized, that the USB Port ist now to high.

Hmm, that's not ideal. Thanks for taking the time to print this out.
The snap joints definitely sound like they're too flimsy for the job. I'll look into replacing these with something more robust; perhaps a wedge-style snap similar to what's been done here.
I hadn't considered that the cutout for the USB port wouldn't be high enough. Oversight on my part. I'd developed the model assuming 8.4mm female header pins and a Wemos D1 mini ESP8266. Is that a Wemos S2 Mini that you've used? I'll rework the design to accommodate.
Any other issues with the build?

@MeisterGig - I've reworked the design to (hopefully) fix the issues that you flagged. I've modified both the top and bottom pieces, so you'll want to reprint both if you'd like to check. Notably I've removed the snaps and swapped to a raised-ridge style join to connect the top and bottom, and I've tweaked the power cable access hole to allow enough clearance for the S2 Mini and similar modules.

Nice Thank you.  I've printed it and i can assemble it. The USB port also fits. So i am gonna merge it now :)  There might still be the problem, that the light sensor isn't recieving enough light, but i haven't tested that yet.


Can not pass hashes to set_info_plist_value

Re-opening a clone of ticket that was just closed #913
A PR was submitted but never accepted. Fix is a one-liner (but I think is_string is now deprecated, so not sure how best to handle this as you need to pass in a type and I want it to accept any type).
--- /Library/Ruby/Gems/2.0.0/gems/fastlane-1.67.0/lib/fastlane/actions/set_info_plist_value.rb  2016-03-23 14:44:18.000000000 +0000
+++ /Library/Ruby/Gems/2.0.0/gems/fastlane-1.68.0/lib/fastlane/actions/set_info_plist_value.rb  2016-03-30 16:23:31.000000000 +0100
@@ -34,6 +34,7 @@
           FastlaneCore::ConfigItem.new(key: :value,
                                        env_name: "FL_SET_INFO_PLIST_PARAM_VALUE",
                                        description: "Value to setup",
+                      is_string: false,
                                        optional: false),
           FastlaneCore::ConfigItem.new(key: :path,
                                        env_name: "FL_SET_INFO_PLIST_PATH",

Thanks for confirming @hammertoe. We'll look into this.

Hi @hammertoe - thanks for the report, and apologies about your previous PR not getting attention.
I've put your patch up as #4106 :+1:

This will go out in the next fastlane release! :rocket:

Awesome thanks! :)
-Matt

On 7 Apr 2016, at 18:47, Michael Furtak<EMAIL_ADDRESS>wrote:
Closed #4103 https://github.com/fastlane/fastlane/issues/4103.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub https://github.com/fastlane/fastlane/issues/4103#event-618877022

—
Matt Hamilton
Quernus
<EMAIL_ADDRESS>+44<PHONE_NUMBER>
49b Easton Business Centre
Felix Road, Easton
Bristol, BS5 0HE
Quernus Ltd is a company registered in England and Wales. Registered number: 09076246


Other files don't open when launched with an unsaved workspace

Type: Bug

From an empty VSCode session, open a folder, then add another folder to the workspace (produced an unsaved workspace)
Optional: Disable all extensions in this workspace (to confirm reproducible without extensions)
Close VSCode (persists the temporary workspace such that it'll reopen on the next launch)
Open a file with VSCode, either via the "Open with Code" action or with VSCode as the default file type handler

At this point, the unsaved workspace reopens, but not the requested file.  Attempting to open the file again (through the above methods or by dragging it into the VSCode window) also does not open it.  Reloading the window and then attempting to open the file succeeds (including the developer option to reload with extensions disabled).
VS Code version: Code 1.86.2 (903b1e9d8990623e3d7da1df3d33db3e42d80eda, 2024-02-13T19:40:56.878Z)
OS version: Windows_NT x64 10.0.19045
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz (12 x 2592)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
15.81GB (2.71GB free)

Process Argv
D:\test.txt

Screen Reader
no

VM
0%

Extensions: none

(This was originally tested against 1.85.2, so probably a long-standing issue rather than one with the latest release.)


MWPW-134664: Safeguard autoblocks

Description
Faulty links can lead to the whole page crashing. While it makes the issue visible, so no "faulty" links should be authored - with all the migrations and automatic imports going on this is hard to control.  We don't want end users to end up on white pages due to JS errors.
Resolves: MWPW-134664
Test URLs:

Before: https://main--milo--adobecom.hlx.page?martech=off
After: https://do-not-error-autoblocks--milo--mokimo.hlx.page?martech=off
Before: https://main--helpx-internal--adobecom.hlx.page/content/help/en/aem-forms/internal-kb/not-able-to-edit-adaptive-form-on-aem-forms
After: https://main--helpx-internal--adobecom.hlx.page/content/help/en/aem-forms/internal-kb/not-able-to-edit-adaptive-form-on-aem-forms?milolibs=do-not-error-autoblocks--milo--mokimo

I'd agree that just wrapping the URL would be preferable - but in this instance the whole method handles a lot of logic, so IMO it's safer to just wrap the whole function. Especially given that this method failing leads to a potentially completely white page.
I changed the lana log message to reflect that anything in the method could fail.

Okie, since both of you agree on that I changed the check accordingly


app.use not working

app.use  working only index file.

app.use("/",(req,res)=>
{
res.json("Welcome My Project");
});

but the same time i call router

const infoRouter = require("./router");
app.use("/info",infoRouter);

not working, error Cannot GET

Could you also send here your router file?
BTW. you can use:
```js
code
```
and your code will be displayed better:
const infoRouter = require("./router");

Hello @kayatechindia it is often hard for us to diagnose issues without a working code example to demonstrate the issue.
Our first request would be for you to create a very small program that demonstrates the issue and the http requests you are making to it.
My first guess is that use is generally not used for routing.  Maybe something like
app.get('/', (req,res) => {
  res.json({
     "hello": "world"
  });

would be better.
then when you use the router and supply the prefix route are you calling the route with the prefix or the base route still ?

@kayatechindia  use app.get and not app.use here.

@kayatechindia
try

app.get("/",(req,res)=>
{
res.send("Welcome My Project");
});

when importing routes use
const infoRouter = require("./router");
app.use(("/info",infoRouter)

Then run your app to whatever localhost://port/info/

@kayatechindia  Can you please show your
./router file for more info.

@kayatechindia, plz provide all the code so we can help you

To make us better understand and help you solve the problem you should share some more information or code that you wrote more than above thanks!

This is so obvious that your problem is caused by your routes file and you did not share it

I'm going to close this as it has been a few months and the OP never followed up with the requested information.


tests: require explicit token in outputs

Otherwise amount could be passed to contract as token. That introduced hard
to debug errors in test execution. Also, explicit > implicit.

Merged as a part of #46, closing.


We should lock version of jemalloc

Any followups in this?

Will lock the jemalloc version in the next release.


[Layer Bottom Sheet] Remove unused help icon.

Fixes #2325

[x] Removes unused help icon.

PTAL @gino-m @scolsen

The failing unit tests is unrelated to this PR. Re-triggering the gcb build.


Implement eth/68 (EIP-5793)

PR description
Implement eth/68

https://eips.ethereum.org/EIPS/eip-5793

The NewPooledTransactionHashes message announces transaction hashes, allowing the peer to selectively fetch transactions it does not yet have.
EIP-4844 introduces a new transaction type for blob transactions. Since these blob transactions are large, naively broadcasting them to sqrt(peers) could significantly increase bandwidth requirements. Adding the transaction type and the size to the announcement message will allow nodes to select which transactions they want to fetch and also allow them to load balance or throttle peers based on past behavior.
The added metadata fields will also enable future - upgradeless - protocol tweaks to prevent certain transaction type (e.g. blob transactions) or certain transaction sizes (e.g. 128KB+) from being blindly broadcast to many peers. Enforcing announcements only and retrieval on demand would ensure a much more predictable networking behavior, limiting the amplification effect of transaction propagation DoS attack.

Modify the NewPooledTransactionHashes (0x08) message:
(eth/67): [hash_0: B_32, hash_1: B_32, ...]
(eth/68): [[type_0: B_1, type_1: B_1, ...], [size_0: B_4, size_1: B_4, ...], [hash_0: B_32, hash_1: B_32, ...]]
Fixed Issue(s)
fixes #4715
Documentation

[x] I thought about documentation and added the doc-change-required label to this PR if
updates are required.

Changelog

[x] I thought about the changelog and included a changelog update if required.

@fab-10 Thanks for the review.

Then there is another part of this implementation that should cover our new strategy on how we broadcast transactions taking in account the new type and size parameters, it is fine to do it in another PR to keep them small, have you already planned to do it?

Yeah, that makes sense. This PR only implements the protocol specs, implementing the ability to send and receive NewPooledTransactionHashes with their sizes and types when using eth68 ensuring that we are compatible with older protocol versions.
What the client should do with the type and size and how it impacts broadcast transaction strategy are not part of the scope. I believe it would be better to handle it in a separate PR.


Update sbt to 1.3.6

Updates org.scala-sbt:sbt from 1.3.5 to 1.3.6.
Release Notes/Changelog
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

labels: sbt-plugin-update, semver-patch

Codecov Report

Merging #365 into develop will decrease coverage by 0.13%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #365      +/-   ##
===========================================
- Coverage    61.31%   61.18%   -0.14%
===========================================
  Files           50       50
  Lines         1538     1538
  Branches        39       39
===========================================
- Hits           943      941       -2
- Misses         595      597       +2

Impacted Files
Coverage Δ

...a/com/twitter/bijection/CollectionInjections.scala
93.54% <0%> (-3.23%)
:arrow_down:

.../main/scala/com/twitter/bijection/Bufferable.scala
90.9% <0%> (-0.83%)
:arrow_down:

Continue to review full report at Codecov.

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


Update .pre-commit-config.yaml

YAML comments are of the form # line.
I don't think it makes sense to copyright this file from another project, its not exactly a Portieris contribution.
The wording changes could be contributed to the source https://github.com/IBM/detect-secrets/blob/master/user-config/.pre-commit-config.yaml.


[skrifa] autohint bug fixes to match FT

This is a largish collection of small changes that bring the autohinter into a state where we match with FreeType for at least the Google Fonts and Noto collections.
Also updates fauntlet to use the new hinting API.
Builds on #1144 which should land first.

cc @drott


on:click not working on Badge component

Describe the bug
Adding an on:click handler to the Badge component does not trigger on click.
Reproduction
<Badge {color} on:click={() => alert('clicked')}>{ name }</Badge>
Flowbite version and System Info
MacOS, `"flowbite-svelte": "^0.44.15"`

Badge component dose not have on:click event.
https://flowbite-svelte.com/docs/components/badge#Component_data

@shinokada I know, that's what my issue was :) Is that by design? Why not include an on:click?

When you click on the dismiss button, Badge emits close event. And when it opens - open.
<Badge on:close={...} on:open={...} />


add drop feature

Add the ability to drop a column or list of columns from a StreamingDataFrame via:
SDF.drop("col_a") or SDF.drop(["col_a", "col_b"])

Absolutely perfect timing for this as I needed it tomorrow 🙌🏽


Performance difference between converted models and official models

Hi, thanks for providing this wonderful benchmark for ssl models! I noticed on we have two kinds of ssl models on huggingface website, one is from the official repository like facebook/hubert-base while the other is the converted models by s3prl.
I am evaluating the above two hubert base model on query-by-example task using the last layer's output, but found a different performance, which is suprised to me:

evaluated with the facebook repo: python run_downstream.py -u hf_hubert_custom -k facebook/hubert-base-ls960 .... I get the maxTWV of 0.06696204.

evaluated with the s3prl converted mdoels: python run_downstream.py -u hubert And the maxTWV is 0.07005093.

Can you please tell me why this performance gap existed as the two results are assumed to be the same/similar.
Thanks in advance!

Hi,
Please use the converted models which are simply the official Fairseq models with a few format changes without touching the weights.
The Hugginface models produce slightly different frozen representation values due to their re-implementation in my personal experience.
The difference might not be noticeable when fine-tuning the SSL model or using a trainable downstream model.
However, it can cause significant difference when running DTW algorithm directly on the frozen representation, as shown by your QbE result.
Hence, the difference between two implementations is somewhat expected, and I would suggest to stick to the Fairseq version when reproducing the SUPERB result.
(Please note that I do not indicate that there is any correctness issue.)


Update _index.md

Original:
subscriptionId=$(az account show --query id --output tsv)
Changed:
added a "$" to subscription, since this is a variable $subscriptionId=$(az account show --query id --output tsv)

since this is bash shell we dont need a $ for variable assignment.


Fix basic queries without "from" clause.

What changes were proposed in this pull request?
Just fix  issue #523 first ignore UT
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
Previous ignore UTs.

Let's fallback to bypass the issue now. We need to make clear why it fails without fallback.


"Plugin 'RiderLink' failed to load because module 'RD' could not be loaded."

"Plugin 'RiderLink' failed to load because module 'RD' could not be loaded.  There may be an operating system error or the module may not be properly set up."
Attempted to build Unreal Engine 5.4.2 Project through Rider Version 2024.1.3 and 2024.2 EAP3. Same message pops up in both versions and the project fails to build. Disabling UnrealLink in the plugin settings doesn't help either.
I added the crash log for better insight and some screenshots
Playground.log

Got the same issue, still not found the solution

I seem to have found a solution. I've been struggling with this issue and ended up trying several ways to fix it. Finally, I noticed a hint that I'd been ignoring all along: "Visual Studio 2022 compiler version 14.40.33811 is not a preferred version. Please use the latest preferred version 14.38.33130."
Since I had been using version 14.40.33811 for quite a long time in UE5.4.1 without any problems, I upgraded to UE5.4.2, and suddenly it stopped working.
So here's how I fixed it: I simply used the latest preferred version, and everything worked fine.
https://forums.unrealengine.com/t/build-failed-in-unreal-5-4/1789560/17


Support preserving whitespace in tags other than pre

What problem does this feature solve?
The SFC compiler only preserves whitespace in pre tag, if you want to preserve whitespace when passing content to custom components or tags that require whitespace preservation you have to contort yourself to pass content wrapped in a pre tag. It would be nice to have some directives that preserve whitespace in any arbitrary tag so this isn't necessary.
What does the proposed API look like?
<template>
  <CodeBlock language="javascript" v-preserve-ws>
const name = "World";
console.log(`Hello, ${name}!`);
  </CodeBlock>
</template>

html does not allow stacking regular space; it is a feature of HTML, not vue.

from a comment of https://stackoverflow.com/questions/65381611/add-whitespace-in-vue.

You can already build such a directive.
In your example, the content inside CodeBlock is passed to the default slot. You can intercept it in the directive, then convert the content to a string and display it.