login broken after cheerio 1.0.0 upgrade

With a fresh install of aws-azure-login I'm getting<EMAIL_ADDRESS>installed. When trying to log in, I'm faced with the following error (together with the last line of debug output).
  aws-azure-login Parsing SAML XML +0ms
TypeError: Cannot read properties of undefined (reading 'load')
    at Object._parseRolesFromSamlResponse (/opt/homebrew/lib/node_modules/aws-azure-login/lib/login.js:663:40)
    at Object.loginAsync (/opt/homebrew/lib/node_modules/aws-azure-login/lib/login.js:379:28)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

Downgrading to<EMAIL_ADDRESS>fixes the issues.

kindly help with command to downgrade<EMAIL_ADDRESS>from mac terminal

it is not fixing to me even after downgrading and even i tried removing it.

Same issue for me as well. Also tried to install v3.6.0 and same issue.
Using AWS SAML endpoint https://signin.aws.amazon.com/saml
TypeError: Cannot read properties of undefined (reading 'load')
    at Object._parseRolesFromSamlResponse (/opt/homebrew/lib/node_modules/aws-azure-login/lib/login.js:663:40)
    at Object.loginAsync (/opt/homebrew/lib/node_modules/aws-azure-login/lib/login.js:379:28)

Also facing the same issue:
TypeError: Cannot read properties of undefined (reading 'load')
at Object._parseRolesFromSamlResponse (/opt/homebrew/lib/node_modules/aws-azure-login/lib/login.js:663:40)
at Object.loginAsync (/opt/homebrew/lib/node_modules/aws-azure-login/lib/login.js:379:28)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

Same issue for me as well. Also tried to install v3.6.0 and same issue.

I guess the problem is that the cheerio dependency is defined as the given version or higher, and currently 1.0.0 is the latest.
Changing this to "1.0.0-rc.12" (exact version) and running npm install inside the module folder is how I made it work. Needed to be run with an older version of Node (18.18.1) for some reason.

Thanks @sganslandt, I tried updating my pnpm package.json to the below and still I am seeing the same error after uninstalling the re-installing aws-azure-login with pnpm uninstall aws-azure-login then pnpm install aws-azure-login Forgive me, I am a noob when it comes to JS package management.
{
	"pnpm": {
		"overrides": {
			"cheerio": "1.0.0-rc.10",
			"cheerio-select": "2.0.0"
		}
	},
	"dependencies": {
		"aws-azure-login": "^3.6.3"
	}
}

This is what worked for me @oliverjfletcher:

Go to your global npm node_modules folder (for my Mac it is /usr/local/lib/node_modules)
You should see a folder named aws-azure-login, go to that folder
Edit the package.json file found inside the above folder
Under dependencies you should see an entry for cheerio, change it to 1.0.0-rc.12
Save
Open a terminal inside that same folder and run npm install (sometimes you need sudo)

After it finishes it should work. I haven't used pnpm but I guess it works the same way as npm.
Hope that helps.

Due to lack of TypeScript and JavaScript proficiency I'm unable to fix this myself but I do note that line 892 of login.ts is calling cherio.load(), and I wonder if this indicates the source of the problem?:
https://cheerio.js.org/docs/api/interfaces/CheerioAPI#load is marked as deprecated with the message:

The .load static method defined on the "loaded" Cheerio factory function is deprecated. Users are encouraged to instead use the load function exported by the Cheerio module.

@gintalk  this worked for me too :)

Note: @gintalk your step worked.
I downloaded npm via homebrew, so the directory for the global npm folder is:
/opt/homebrew/lib/node_modules

Same issue for me as well. Also tried to install v3.6.0 and same issue.

I guess the problem is that the cheerio dependency is defined as the given version or higher, and currently 1.0.0 is the latest.
Changing this to "1.0.0-rc.12" (exact version) and running npm install inside the module folder is how I made it work. Needed to be run with an older version of Node (18.18.1) for some reason.

Same for us today. Put Node down to v18, set dependency to exactly "1.0.0-rc.12", running npm install in there, fixed it up.

Hi Everyone, i saw that PR 340 had updates to the code base as well. I thought a simple update to the package.json will fix this and hence I opened a PR for it #341  . Please do leave some comments on it as it's my first ever PR that I've created!

Hi all, I followed all the steps here but am still facing the same issue?

Hi all, I followed all the steps here but am still facing the same issue?

If you're having trouble manually patching it, I'd suggest uninstalling aws-azure-login and installing aws-azure-auth instead with npm i -g aws-azure-auth

@Garethp thank you very much I'll try that now

I am using windows cmd and getting same issue, please help me.

Other commands are like azure-azure-login etc. I am want to setup AWS profiles, will it work?

This is what worked for me @oliverjfletcher:

Go to your global npm node_modules folder (for my Mac it is /usr/local/lib/node_modules)
You should see a folder named aws-azure-login, go to that folder
Edit the package.json file found inside the above folder
Under dependencies you should see an entry for cheerio, change it to 1.0.0-rc.12
Save
Open a terminal inside that same folder and run npm install (sometimes you need sudo)

After it finishes it should work. I haven't used pnpm but I guess it works the same way as npm. Hope that helps.

I have followed steps but still getting same error

While this waits for the maintainer to resolve it, I've forked aws-azure-login into aws-azure-auth. It's published into npm at the moment and over the weekend I'll take a look into putting it onto Docker and perhaps snapcraft if I can. I'll also be looking into bringing in some of the open issues/improvements on this project.
On top of that, I'm hoping to fold in a CDK plugin into that package as well so that it can automatically detect and refresh credentials for an environment that you wish to deploy into

You are a lovely person.  I switched to this and now everything is working perfectly while we wait!

While this waits for the maintainer to resolve it, I've forked aws-azure-login into aws-azure-auth. It's published into npm at the moment and over the weekend I'll take a look into putting it onto Docker and perhaps snapcraft if I can. I'll also be looking into bringing in some of the open issues/improvements on this project.
On top of that, I'm hoping to fold in a CDK plugin into that package as well so that it can automatically detect and refresh credentials for an environment that you wish to deploy into

This one worked only. Thanks @Garethp

I have tried all the above steps but didn't work for me

After a re-install of 3.6.3 I'm still facing the same issue, with the same error. Downgrading cheerio still fixes the issue. Is it only me, and am I missing something in the re-install process?

After a re-install of 3.6.3 I'm still facing the same issue, with the same error. Downgrading cheerio still fixes the issue. Is it only me, and am I missing something in the re-install process?

You need to be using v.3.6.4 which you can get from NPM: https://www.npmjs.com/package/aws-azure-login but it raises the important question how on earth 3.6.3 is still the latest release here on GitHub?!


Technical-proposal needed for compliance and certification

Ref https://github.com/opencontainers/image-spec/pull/452#issuecomment-280160707
The changes for validation need to have a bit of guidance as to the nature of the compliance validation.
This would ideally be a product from the cert and compliance working group to ensure validation of OCI image artifacts satisfies the requirements.

cc @opencontainers/certification-maintainers

On Wed, Feb 15, 2017 at 02:40:57PM -0800, Vincent Batts wrote:

This would ideally be a product from the cert and compliance working
group to ensure validation of OCI image artifacts satisfies the
requirements.

In opencontainers/image-tools#66, I floated “strict” (only contains
OCI-defined content) and “base” (only contains content which the
image-spec REQUIREs unpackers to support) levels for compliance
testing.  Both of those are more restrictive than a “legal” level
(content does not violate any MUST-level image-spec requirements).
Another axis which cert/compliance would be interested in is
MUST-level vs. SHOULD-level validation.  So I think we should make it
easy for callers to validate with any combination of legal/strict/base
and legal/should.
Beyond the cert/compliance working group, I also floated an
“unpackable” level (can image-tools unpack the content).  That would
not impact the cert/compliance working group, but it would be
unfortunate to address only legal/strict/base (or whatever) in
image-spec and leave no way for image-tools to support
legal/strict/unpackable/base.
It would also be nice if callers could register support for additional
media types (e.g. “I can also unpack
application/vnd.example.layer.v3.json”).
And it should be easy for Go callers to figure out about validator
concerns (e.g. “what were the SHOULD-level violations, if any?”) which
is hard if the validation code is printing warnings to stdout 1.
I've taken a few stabs at implementating a framework for this with no
success (e.g. in #403), so I'll leave figuring that out to
runtime-tools maintainers.  The above stuff is my take on the
constraints for that framework.  This issue may just be about
collecting constraints anyway.

@vbatts Would the proposal at: https://github.com/opencontainers/certification/pull/26 fulfill this issue?

@RobDolinMS perhaps so

The conformance issue was closed because it went stale. I still think a conformance/validation piece would be nice here.


Add Rust hosting integration

Closes #206

This pull request introduces caching for Rust packages in the CI workflows, and adds new Rust projects to the solution. The most significant changes include updates to the CI configuration files, modifications to the solution file to include new Rust projects, and the addition of new project files for the Rust applications.
CI Workflow Updates:

Added Rust package caching using Swatinem/rust-cache@v2 in .github/workflows/dotnet-ci.yml, .github/workflows/dotnet-main.yml, and .github/workflows/dotnet-release.yml [1] [2] [3].

Solution File Modifications:

Updated CommunityToolkit.Aspire.sln to change project type GUIDs for existing projects and added new Rust projects (CommunityToolkit.Aspire.Hosting.Rust, CommunityToolkit.Aspire.Hosting.Rust.AppHost, CommunityToolkit.Aspire.Hosting.Rust.ServiceDefaults, and CommunityToolkit.Aspire.Hosting.Rust.Tests) [1] [2] [3] [4] [5] [6].

New Rust Project Files:

Added new project file for CommunityToolkit.Aspire.Hosting.Rust.AppHost with necessary configurations and dependencies (examples/rust/CommunityToolkit.Aspire.Hosting.Rust.AppHost/CommunityToolkit.Aspire.Hosting.Rust.AppHost.csproj).
Created Program.cs for CommunityToolkit.Aspire.Hosting.Rust.AppHost to set up and run the application (examples/rust/CommunityToolkit.Aspire.Hosting.Rust.AppHost/Program.cs).
Added launchSettings.json for CommunityToolkit.Aspire.Hosting.Rust.AppHost to define development environment settings (examples/rust/CommunityToolkit.Aspire.Hosting.Rust.AppHost/Properties/launchSettings.json).
Added appsettings.json for CommunityToolkit.Aspire.Hosting.Rust.AppHost to configure logging (examples/rust/CommunityToolkit.Aspire.Hosting.Rust.AppHost/appsettings.json).
Created project file for CommunityToolkit.Aspire.Hosting.Rust.ServiceDefaults with necessary dependencies (examples/rust/CommunityToolkit.Aspire.Hosting.Rust.ServiceDefaults/CommunityToolkit.Aspire.Hosting.Rust.ServiceDefaults.csproj).

PR Checklist

[X] Created a feature/dev branch in your fork (vs. submitting directly from a commit on main)
[X] Based off latest main branch of toolkit
[X] New integration

[ ] Docs are written
[X] Added description of major feature to project description for NuGet package (4000 total character limit, so don't push entire description over that)

[X] Tests for the changes have been added (for bug fixes / features) (if applicable)
[X] Contains NO breaking changes
[X] Every new API (including internal ones) has full XML docs
[X] Code follows all style conventions

Other information

@aaronpowell, we should set timeout for each test library.
Last build was long time runnig (1h 44m!!!)
Obviously something going wrong when a build take too long.

@aaronpowell, we should set timeout for each test library. Last build was long time runnig (1h 44m!!!) Obviously something going wrong when a build take too long.
https://github.com/CommunityToolkit/Aspire/actions/runs/11777837131/job/32803012099?pr=212

I didn't think GitHub Actions would let it run for this long, unsure how to better resolve it.

@aaronpowell, we should set timeout for each test library. Last build was long time runnig (1h 44m!!!) Obviously something going wrong when a build take too long.
https://github.com/CommunityToolkit/Aspire/actions/runs/11777837131/job/32803012099?pr=212

I didn't think GitHub Actions would let it run for this long, unsure how to better resolve it.

I think the TestSessionTimeout in .runsetting could help us.
See:
https://learn.microsoft.com/en-us/visualstudio/test/configure-unit-tests-by-using-a-dot-runsettings-file?view=vs-2022#runconfiguration-element


ASB-2316: Homepage & project search - content update

Describe your changes
https://pins-ds.atlassian.net/browse/ASB-2316?atlOrigin=eyJpIjoiZmYzOTBhMTg0OGFhNGQyYWE4YzM3ZWE0NTk5ZWQ4ZDEiLCJwIjoiaiJ9
Type of change 🧩

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Other (please explain in the description section above)

Checklist before requesting a review

[x] I have performed a self-review of my own code
[ ] I have double checked this work does not include any hardcoded secrets or passwords
[ ] I have made corresponding changes to the documentation
[ ] I have provided details on how I have tested my code
[ ] I have referenced the ticket number above
[ ] I have provided a description of the ticket
[ ] I have included unit tests to cover any testable code changes

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


Add dynamic data example

Live example at http://jsbin.com/quhapo/1/edit

This is really cool! :sunglasses:

@really nice @jamiesnape !!

LGTM :+1:


refactor 'shrine' to 'waypoint' where appropriate

With the expansion of "special" splits, such as Ganon (and later Divine Beasts) we generalize the term Shrine to Waypoint wherever appropriate.

I also put in #43 based on this branch. Review appreciated there.
Feel free to merge here first, then merge this to master (my preference), or just merge this to master, and merge that one to master after.


Win7 R installation won't start python standalone server

After digging through several issues (one of which involved a missing path variable), the fault I'm encountering lies with the launch() function in comm.R.
Specifically,
launch <- function(){
  optunitydir <- dirname( system.file("optunity", package="optunity") )
  cmd <- sprintf("cd '%s'; python -m optunity.standalone server %s", optunitydir, ifelse(debug(), "", "2>/dev/null"))

  opipe <- pipe(cmd, open="r+")
  portstr <- readLines(opipe, n = 1)
  port    <- strtoi( gsub("^\\s+|\\s+$", "", portstr) )

  if (is.na(port) || port == 0) {
    stop("Optunity error: could not launch python process.")
  }

  ...
}

'optunitydir' is properly defined, and executing pipe() on cmd opens a connection as it should.
However, it appears opipe returns a python-specific object (a Win32 connection) and as such, cannot be operated on by the readLine() function, which inevitably leads to this error when R evaluates the conditional:
Error in if (is.na(port) || port == 0) { :  missing value where TRUE/FALSE needed

After inspecting what 'opipe' returns in R, it appears it returns a numeric vector of zeros, the number of which indicates the port number.  Thus, the line:
port <- length(numeric(opipe))

will return the correct port number as an interger that can then be used in the call to socketConnection() and the construction of the returned 'conn' object.
My modification to comm::launch(), then, appears as follows:
launch <- function(){
  optunitydir <- dirname( system.file("optunity", package="optunity") )
  cmd <- sprintf("cd '%s'; python -m optunity.standalone server %s", optunitydir, ifelse(debug(), "", "2>/dev/null"))

  opipe <- pipe(cmd, open="r+")
  port <- length(numeric(opipe))

  if (is.na(port) || port == 0) {
    stop("Optunity error: could not launch python process.")
  }

  socket <- socketConnection(port = port, blocking = TRUE)
  conn   <- list(socket = socket, port = port, opipe = opipe)
  return (conn)
}

I would submit the changes in a pull request, but I don't have access to gihub from my machine, unfortunately.

It seems the behavior of numeric(opipe) is not very reliable as in Linux it just returns vector of three 0s.
But there is a simple alternative. We can change optunity.standalone server to take an extra argument for a file name where to write the port number to.
@claesenm what do you think about this workaround for establishing communication between R and Python in Windows?

@graffy76  thanks for the very detailed bug report!
@jaak-s the workaround sounds okay. A possible alternative would be to check what OS we're running and then deal with it differently, but usually those approaches cause more issues than they solve.


The PRPL Pattern

During the /web hack week, we have an opportunity to land a dedicated page for the PRPL pattern outside of the Polymer docs.
This would be a description of the pattern that isn't tied to the App Toolbox, but links out to it and Shop as canonical examples of how the pattern could be implemented in practice.
Given the Polymer team created/discovered PRPL, looping in @graynorton @tjsavage for their thoughts here.
We might be able to fit this under https://developers.google.com/web/fundamentals/performance/?hl=en

+1

Will this be established best practice? I am asking because if it is going to live in /web/fundamentals, then  we expect every developer to follow this guidance.  I've heard reports from a few teams that H2 Push can degrade the performance of your site especially if the content is already in the browser cache.  So this pattern seems experimental at best.  Do we have real-world data and guidance from production sites?

This is a good point. I would say it is experimental. We know that Housing.com, Flipkart and the NFL are experimenting with PRPL but you're correct in saying the pattern is more of "here's a pattern we think makes sense" with companies investigating the impact of implementing route-chunking and seeing how far Server Push, H/2, preload etc can take them.
If it wouldn't live under fundamentals, what would be another place /web that this could live? It's not an update (we want something more formal than that) but it would also be better served outside of the Polymer docs.
My suggestion would be an entry under Performance with the experimental label if there isn't a better place for it today. @petele might have suggestions too.

After talking to Arthur, I'm going to take this one as he is heads down on docs for Polymer Summit. I don't think it'll take too long to land.


Bugs in Adversarial attacks

The attacks are pretty weak right now, this needs to be fixed

Could be fixed in d256589ab7657529addd239474b7f3657e19c009. Needs some testing.


feat: add opengb generate sdk button

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

#142
#141  👈
#140
#139
#138
#137
#136
main

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

Join @NathanFlurry and the rest of your teammates on  Graphite


HI-02 Improper Initialization Leading to Insecure Uses

The implementation is liable to replay attacks because encryption/validation keys are
permanent and round values repeat. For example, whenever a coordinator is created (e.g.,
c=Coordinator.new(config)) it will be instantiated with c.dkg_id=0 and this value
cannot be changed by configuration. This means that every time a new coordinator is
instantiated, the dkg_id values repeat and hence, encrypted packets used in previous
rounds can be reused (e.g., a round with dkg_id=0 will happen many times, same with
dkg_id=1, etc). This allows malicious users to reuse broadcasted/public values of earlier
rounds at least causing unavailability and allows them to manipulate the protocol logic.

The serialization functions load/save allow creating Coordinators that do not repeat dkg_id.  However, not all users of wsts use these functions; in particular sBTC does not.


Difference between two reports (or timeline of changes between lot of reports)

Need to see:

what was added
what was removed
which cases changes status
which cases still broken, or failed

this feature will be useful to show managers dynamic of automation tests project

Nice feature... It will be very useful for everyone. :+1:
It is a much awaiting feature for me. Hope it will be available soon.

Will be implements into TMS :)

This would be great -- on a day-to-day basis being able to see which failures are new would help a lot with debugging, identifying product bugs, and determining which test cases failed due to legitimate product changes and need updating.

@dsafar we'll opensource a related product during next months.

we'll opensource a related product during next months.

Can you please close this issue, when this functionality will be available?
It show wrong interpretation that this feature is completed.
Also you have any rough timelines, when will be that feature available.

@arpitgold this functionality will not be available in allure framework. As we say before we'll opensource a related product during next months. And related not means that our new product will create difference between reports. So the issue closed as should not be resolved.


Update F# Getting Started in VS for VS 2017

Updated installation instructions to be for VS 2017 and removed screenshots.

@mairaw I think this one can be exempt because there isn't any support for .NET Core + F# in Visual Studio right now.  When that support lands, it'll also be with the same tooling.  All of the above Workloads also pull down the necessary bits (project system, NuGet, MSBuild), so no modifications would have to be made to this document if it were to "pivot" to create a .NET Core console app in F#, for example.


mouse-actions-gui: init at 0.4.4

Description of changes
Closes https://github.com/NixOS/nixpkgs/issues/269212
Connected to https://github.com/NixOS/nixpkgs/issues/264904
Partially based on @rgri's solution (link)
This PR adds 1 package: mouse-actions-gui
There is a GUI-less version called mouse-actions, but this package also has all of that functionality (can be run GUI-less).
The package is a tauri app:
I tried to use buildRustPackage, like I did for a previous tauri app, but for some reason vite wanted to connect to the development server even after building. (probably some environment variables I couldn't figure out)
Instead I use cargo-tauri to build it as an unwrapped .deb package, which actually takes care of all the renaming of files and the installing of .desktop and icons files.
This was based on how the pot package did it
I added the necessary udev rules based off of @rgri's solution.
(You'll have to add the package to services.udev.packages in your configs for the rules to register)
I am not sure what the 2nd rule does, as it seems to work without it. Could you explain why that was included?
I rephrased the description a bit, but I am still not completely satisfied with it.
I also don't know how one's intended to launch the program at startup... Maybe from the WM startup function or something? Or from a systemd user service?

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Hi! Thanks for the help, I was planning on looking at this eventually but forgot lol.

The second rule (so others can follow) was included as part of the upstream requirements, specifically "To use the main feature "grab event", you need to have the read&write permission on /dev/input/event*." Interesting that you can run it without that- maybe I didn't do thorough testing.
Instead of requiring users to manually add this package to services.udev.packages you should probably write a module like mine which does it for them on enable = true;. This has precendent in the package glasgow.

Hi hi. #265056 just got merged. Would you be interested in working on this together? If you aren't, I'd be happy to take it from here.

Hi hi. #265056 just got merged. Would you be interested in working on this together? If you aren't, I'd be happy to take it from here.

I'd be up to discuss how to deal with the duality of the two packages.
One option would be to keep them separate, as mouse-actions and mouse-actions-gui.
But then most of basic package data would be the same, so that's a bit of code duplication.
Another option would be to have only mouse-actions, but you can enable/disable the gui by overriding the package, with something like this:
mouse-actions.override { withGui = false/true; }
This would need the two packages to be combined into one file. They are quite different, as I the gui package is using buildNpmPackage with cargoSetupHook and cargo-tauri while the non-gui package is using the normal buildRustPackage.
Whichever option is chosen, we'd need to perhaps be able to override the mouse-actions package in programs.mouse-actions
Currently it's hard coded to pkgs.mouse-actions but in most other program modules, theres a .package attribute, in this case programs.mouse-actions.package would be the one needed to be added.
There's also the question of which package should be the default for the service (or for the combined package)

IMHO, for the sake of simplicity I would vote for two packages... From my experience some code duplication is not an issue if it makes things simpler/easier. (But this is just my "two cents" on the matter). =)

Hey all! =)
Any news on this one?

Hey all! =)
Any news on this one?

Hey, thanks for the reminder. I've forgotten about this a little bit.
I'll try to get someone to look at this PR.

I have added a NixOS module in a new commit (untested), however I am not really happy with this current solution, as there are now 2 modules for the (almost) same program (mouse-actions and mouse-actions-gui).
It might be better to have a programs.mouse-actions.package attribute, which can be set to mouse-actions-gui. (Or maybe that should be the default, and you could supply use the non-gui version)

It should also add it to systemPackages, I think. I'll have to test whether it adds a desktop entry and the binary to PATH like this, but I don't think so.
As for the duplicate module, that's fair. How is this used along with the other tool usually? Do you use one or the other, or is it useful to have both at the same time?

The gui app is a superset of the non-gui app.
I am working on updating the mouse-actions module to:

add the package to systemPackages
create a oneshot systemd user service to launch the program
add pkgs.programs.mouse-actions.package for overriding.

maybe instead of this we could have a gui=true; option

@TomaSajt thanks man! Really really thanks! =))

The gui app is a superset of the non-gui app.

Okay, yes, then I would have only the one already existing module and just swap out the package.

Okay, so I've added a systemd user service to the module (I will put it behind a configuration flag later), which ALMOST works, however whenever I run it it turns of my wifi support (????, how?).
I'll check this out on my desktop PC later today.
If anyone knows what better .targets to use, or how to better setup the service dependency tree, please do tell me.

graphical-session.target is fine (though I think it should be bindsTo rather than requires). Are there any related messages in journalctl? Also, how are you setting up wifi?

Anyways, I realize that because this is started as a systemd service, it has no idea about $PATH so it can't open apps without full paths to them, which is not the most convenient thing.

Try setting environment.PATH = mkForce null; in the service. It's NixOS that overrides it by default, user services usually inherit the environment.

I thought I found the right place where it's being handled
https://github.com/NixOS/nixpkgs/blob/ab14a429cb1a6cc7b7b843a8ad2661ad1570e57c/nixos/lib/systemd-lib.nix#L345-L359
But, it turns out this isn't really where it path comes from. It does, if you don't override anything, but if you do path = lib.mkForce [], then because of the mkIf above, environment won't get set, which you'd thing is a good thing. However there's an actual default probably compiled into systemd, which is the same as this list, but can't be overridden. The same thing happens if I use mkForce null on environment.
I am not sure what to do next...

Wait, I might have actually came up with something working!!

It works! Seems like PassEnvironment had to be used together with either mkForce attempt.

Maybe there should be some message instructing the user to add themselves to the uinput group to use this.

Just realized that I messed up the second line of the udev rule

@TomaSajt can you please point me to some NixOS documentation on how to use this locally until it is available in unstable?

Well, I asked around the Matrix chat a week ago, but nobody mentioned a really easy way.
I usually just create a mouse-actions-module.nix file in my configuration directory, copy over the PR contents, and rename the module from programs.mouse-actions to programs.mouse-actions-dev (all 2 places) to avoid naming conflicts. Then include the file in my configuration using imports = [ ./mouse-actions-module.nix ];.
Now you can do
programs.mouse-actions-dev.enable = true; to install the program
and also do
programs.mouse-actions-dev.autorun = true; to enable it to launch at startup. You'll need to restart your PC/session beforehand, probably.
If you also want to use mouse-actions-gui, you can either create some kind of overlay (any way you want to or are able to), and set
programs.mouse-actions-dev.package = pkgs.mouse-actions-gui;
Or you could copy the derivation into a new mouse-actions-gui.nix file in your configuration (you'll need to copy the udev rules file too) and do programs.mouse-actions-dev.package = pkgs.callPackage ./mouse-actions-gui.nix { };

Thank you for the detailed explanation!
I will follow what you suggested and see how far I can go. So far it seems I need to copy nixos/modules/programs/mouse-actions.nix file locally and enable/use that. I did not get the last few points about gui option and udev rules and I'm not sure how to transfer/use files in pkgs/by-name/mo/mouse-actions-gui/* locally. But I will re-read, explore, and try when I get home. Thank you @TomaSajt

I went ahead and did what I wrote down, here it is in my config: https://github.com/TomaSajt/nixfiles/commit/d4e35046cd0b568b15fca0a141d3693a29cbc3c2

This is not relevant but in case you've also run into it, how do you setup a mouse-button-hold + scroll-wheel-up/down trigger? I can't seem to get even the normal mouse-wheel-up working, and those window zones are not working either. I tried using app-image on ubuntu but no luck there either. Maybe mouse-actions does not support all the things easystroke did.

Sorry, I don't have much experience with this program either. If this issue is not nixpkgs-specific, then perhaps you can get more support from the upstream repo.

Hi all! o/
What still need to be done on this PR? Maybe I could help?

Hey! I haven't really received much feedback about the NixOS module yet, so if you could try if the autorun = true option works for you, I'd appreciate it.

I've come up with a small nixos module which you can import into your on config that applies the changes to the module and enables autorun and mouse-actions-gui
{ ... }:

let
  nixpkgs-with-magui = fetchTarball {
    url = "https://github.com/TomaSajt/nixpkgs/archive/mouse-actions-gui.tar.gz";
    sha256 = "1z8zhsm60hvrrbj56qfbqd1qrj2db7222r2jifz06jhzfj3cdpvd";
  };
  pkgs-with-magui = import nixpkgs-with-magui { system = "x86_64-linux"; };
in
{
  disabledModules = [ "programs/mouse-actions.nix" ];
  imports = [ "${nixpkgs-with-magui}/nixos/modules/programs/mouse-actions.nix" ];

  programs.mouse-actions = {
    enable = true;
    autorun = true;
    package = pkgs-with-magui.mouse-actions-gui;
  };
}

Place this file next to your configs, name it something like magui.nix and the import it from your configuration with
{
  imports = [ ./magui.nix ];
}

Mate... It WORKS! Perfectly! :heart:


Switch to graphs of Observables

The current architecture is fragile as it is based on a mix of synchronous caches of data, synchronous fetches of data and asynchronous fetches of data from the server. This needs to be resolved.
This needs to be reimplemented as trees of observables which are only placed in to local variables in order to data bind to UI components.

This is implemented. Hence closing the issue.


Reintegrate camera-side integration code

Per discussion with @rbiasini, really need to have camera-side integration available for a clean upstream port.

Network position auto-detect (R242 vs J533) is tested and working in VW community 0.7.4-VW1.3.


Update namespace of Infrastructure DependencyInjection

Thanks @NicklasHugoy - much appreciated! Hope all is going well. 👍


decouple signing artifacts from pushing repo

As the latest incident has shown, having the signature in the middle of publishing a repo its kind of a bad idea. If something goes wrong with the signatures, the process stops, leaving the repository in a bad state.
This leads me to think that we should do the signing afterwards instead so the repo is untouched and the signatures are done as their own process.
Not only does this avoid breaking repos, it also allows us to independently sign a repo and push the signatures to different places, in case that one of those goes down, it allows us to resign old artifacts, change fulcio urls, change root certificates (suse certs anyone?) and prevent us for being blocked in case signatures format change in the future.
as the plugin verifies the signatures on install, a missing signature would also fail to install, so there is no risk involved in missing a signature for third parties consuming the artifacts as they wont end up with an unsigned artifact as part of their build process (as long as the use the luet-cosign plugin on install) so there is no risk involved there, only a potential delay if an artifact is not signed on our side, and we can have a scheduled job to sign missing artifacts as well, so we dont miss anything for more than a few hours.

This just happened again, should probably bump this to taclke it soon as to not break our releases. Also we should use something like https://github.com/nick-fields/retry + the resigner script to sign the artifacts so it retries. No damage if it fails and we try to resign as the script checks if the signatures are in there so if it fails in the middle it will just ignore signed artifacts and sign the missing ones.
cc @mudler


Redirect back to the device when you delete IP addresses from it.

Netbox 1.7.3

Go to contact the selected device.
Delete the IP address of the device.

Then we get to the window to confirm the deletion.

And then we get to the list of all IP addresses.

Questions:
Now how do I get back to the device from which I removed the IP address?
Back to go to the list and look for them among thousands of devices? And if I have forgotten the name of the device?
Improvement:
You can make sure that when you delete IP address is returned to the same device that produces its modification. Removing IP address, user and other device settings.

This was one of those things that I've wanted to fix for a long time, but just never made a conscious effort to focus on. I'm sure there are other instances where this sort of behavior can be improved as well. Thanks for opening the issue.

Many thanks!


[BUG] Improvements on MemmapTensor

Describe the bug
We need to do some woodwork on MemmapTensor:

[ ] Remove the ownership and replace it by a check over the number of times the array is being opened (for temporary files only)
[x] Solve a bug where calling MemmapTensor(memmap_tensor[idx]) returns the wrong view of the MemmapTensor #193
[ ] Improve MemmapTensor creation args, including: creating a Memmap on a specific location (#189)
[ ] Improve the way we copy a Memmap from one place to another, if we want and if we don't want to change the filename
[ ] the as_tensor method works much faster than _tensor and it should be the default behaviour
[ ] the device attribute is confusing, as the data is never really on another device but gets loaded there if needed. The usage is that one can put the device attr as a destination, then move the Memmap from process to process and when accessing the data it'll be on the right device. However, this may give the false illusion that the content of the tensor is stored on the device...

cc @tcbegley

[ ] AttributeError on Tensor attributes when creating one MemmapTensor from another. (#192 <- needs to be adapted to recently merged #193)

I'll also update #189, though the desired no-op behaviour there is blocked by #192.


You're missing x-functions-key header for postman if using Function Keys and HTTP POST.

If using postman and sending a post request you need to add the header x-functions-key and give it the value of the URL ?code parameter.  These are the Function Keys found in the Manage settings for the function in the portal.

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

ID: a571957f-41ce-e7ea-5f0b-1aefb8b219fa
Version Independent ID: 6c676eb8-581c-c5c5-0047-c7a7531fa423
Content: Testing Azure Functions
Content Source: articles/azure-functions/functions-test-a-function.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Thanks for the feedback! We are currently investigating and will update you shortly.

@n1cho1as Yes, In postman, you can either pass the key value in the query string as ?code= or you can add in the headers as x-functions-key.
Thanks for the feedback. I have assigned the issue to the content author to investigate further and update the document as appropriate.

@n1cho1as I recently re-wrote this article that features strategies on how achieve automated tests for Functions. The approach used in the updated version no longer uses Postman in the examples.
Do you mind checking out the latest version of the doc  to let me know if you have any further suggestions and if we can close out this issue?
Thanks!

Thanks @craigshoemaker! I think we can #please-close this issue. It can be reopened if needed.


Update scodec-bits to 1.1.17

Updates org.scodec:scodec-bits from 1.1.16 to 1.1.17.
GitHub Release Notes - Version Diff
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.
Configure Scala Steward for your repository with a .scala-steward.conf file.
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.scodec", artifactId = "scodec-bits" } ]

labels: library-update, semver-patch

Codecov Report

Merging #1490 into master will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1490      +/-   ##
==========================================
- Coverage   85.88%   85.84%   -0.05%
==========================================
  Files          58       58
  Lines        2197     2197
  Branches      107      107
==========================================
- Hits         1887     1886       -1
- Misses        310      311       +1

Impacted Files
Coverage Δ

...re/shared/src/main/scala/io/circe/KeyDecoder.scala
94.59% <0.00%> (-2.71%)
:arrow_down:

...les/core/shared/src/main/scala/io/circe/Json.scala
80.44% <0.00%> (-0.56%)
:arrow_down:

.../core/shared/src/main/scala/io/circe/Decoder.scala
85.05% <0.00%> (-0.28%)
:arrow_down:

...re/shared/src/main/scala/io/circe/JsonNumber.scala
93.97% <0.00%> (ø)

...rc/main/scala/io/circe/numbers/BiggerDecimal.scala
85.38% <0.00%> (+0.58%)
:arrow_up:

...d/src/main/scala/io/circe/NonEmptySeqDecoder.scala
100.00% <0.00%> (+8.33%)
:arrow_up:

Continue to review full report at Codecov.

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


High battery and Data usage on Android

App uses very high battery due to location tracking.
Data usage is also very high.

From one case on Android, the app consumes 10 GB over 7 days of use.

From one case on Android, the app consumes 10 GB over 7 days of use.

There could be problems on some device model because my Android device just consumes 6.05 MB since 1st Jan

Btw, it needs to be fixed.

There could be problems on some device model because my Android device just consumes 6.05 MB since 1st Jan

Btw, it needs to be fixed.

Another case here
https://www.facebook.com/McDuck/posts/10221519050437110

Another case here
https://www.facebook.com/McDuck/posts/10221519050437110

Me too.
Device Huawei Mate 20 Pro
Android Version 10
enable loction all day.

Me too.
Device Huawei Mate 20 Pro
Android Version 10
enable loction all day.

Seem to have a problem with the Android devices.

Seem to have a problem with the Android devices.

For network consumption issue, I've roughly (very roughly) profiled the app on both iOS and Android via Charles as well. These are some information that might be useful.
Hypothesis
Based on reports (This issue and on social network) Android devices are consuming data more than iOS devices. Therefore, there might be some different behavior on each platform. In this case, data consumption might come from network request during device handshaking.
Devices

iPhone 12 Pro Max, iOS 14.3, non-jailbroken
Samsung Galaxy Z Flip 2, Android 10, non-rooted

Controlled conditions

Morchana app version 2.0.1
Same set of permissions allowed

GPS: When in use only since most of the reports showed that mobile network used on Foreground ✅
Bluetooth ✅
Motion & Activity ✅
Notification ✅

Putting 2 devices next to each other while opening Morchana app. No user event. No force refresh QR code.

Recording period: 2 minutes via Charles proxy to get size of network request
Result
iOS: 13 requests, 26.76 KB each, In total = 347.88 KB
Android: 23 requests, 26.68 KB each, In total = 613.64 KB
Android device (my device) is sending more network requests than iOS (70% more).
With this rate of network request. Let's say user use this app 6 hours in public. It'll consumed at least 93.7 MB of data per day which is a lot of data 😅  (assume that numbers of scanned devices are not affecting uploading rate)
As I roughly check the code (which might not be the same one on store). Request should be sent once every 30 mins? instead of this rate. Also, code on contract-tracer module looks fine for me. So it shouldn't related to ble scanning logic.
Summary

Response size from api should be smaller with this high rate of scanned nearby device uploading
Something wrong with network request rate 😅

Hope this might be helpful for further investigation

For network consumption issue, I've roughly (very roughly) profiled the app on both iOS and Android via Charles as well. These are some information that might be useful.
Hypothesis
Based on reports (This issue and on social network) Android devices are consuming data more than iOS devices. Therefore, there might be some different behavior on each platform. In this case, data consumption might come from network request during device handshaking.
Devices

iPhone 12 Pro Max, iOS 14.3, non-jailbroken
Samsung Galaxy Z Flip 2, Android 10, non-rooted

Controlled conditions

Morchana app version 2.0.1
Same set of permissions allowed

GPS: When in use only since most of the reports showed that mobile network used on Foreground ✅
Bluetooth ✅
Motion & Activity ✅
Notification ✅

Putting 2 devices next to each other while opening Morchana app. No user event. No force refresh QR code.

Recording period: 2 minutes via Charles proxy to get size of network request
Result
iOS: 13 requests, 26.76 KB each, In total = 347.88 KB
Android: 23 requests, 26.68 KB each, In total = 613.64 KB
Android device (my device) is sending more network requests than iOS (70% more).
With this rate of network request. Let's say user use this app 6 hours in public. It'll consumed at least 93.7 MB of data per day which is a lot of data 😅  (assume that numbers of scanned devices are not affecting uploading rate)
As I roughly check the code (which might not be the same one on store). Request should be sent once every 30 mins? instead of this rate. Also, code on contract-tracer module looks fine for me. So it shouldn't related to ble scanning logic.
Summary

Response size from api should be smaller with this high rate of scanned nearby device uploading
Something wrong with network request rate 😅

Hope this might be helpful for further investigation

@kajornsakp I tried my Android 10 device (in foreground, app version 2.0.1) with Charles and saw only few requests in sequence view. Is there any extra config needed to see a lot of requests like you did?

@kajornsakp I tried my Android 10 device (in foreground, app version 2.0.1) with Charles and saw only few requests in sequence view. Is there any extra config needed to see a lot of requests like you did?

@PaperMonster Hmm, that's weird 🤔 . I just opened 2 devices and let it scan for each other. No need to set extra config in Charles

@PaperMonster Hmm, that's weird 🤔 . I just opened 2 devices and let it scan for each other. No need to set extra config in Charles

@kajornsakp I forgot to open 2 devices next to each other. Once I do, I got more requests from both devices, but not as often as you did. iOS took 1-2 minutes per request while Android hardly sent requests after launch. Gotta play with them more later.

@kajornsakp I forgot to open 2 devices next to each other. Once I do, I got more requests from both devices, but not as often as you did. iOS took 1-2 minutes per request while Android hardly sent requests after launch. Gotta play with them more later.

High data usage links to at least two problems:

bills
battery

Free data usage may solved (1) - for likely a limited period of time, as this can't be forever.

High data usage links to at least two problems:

bills
battery

Free data usage may solved (1) - for likely a limited period of time, as this can't be forever.


Add apidocs to #tap and #dup

Added the api documents to #tap and #dup.
Ref: #788 and #791
I hope my understanding is correct, especially on Hash#dup.

@hachi8833 sorry that I should warn you about object_id doesn't actually function like Ruby. Even if you call it on the same object it returns different numbers every time.
I will try to fix this issue first. It that works out well, we can merge this PR after the fix. If that didn't work, you'll need to use different examples in your doc 😞

Thank you for the explanation! I'll try it after the fix.

@hachi8833 the issue of object_id has been fixed and merged, feel free to reopen this PR!


Leave file decls in their original order

Don't attempt to sort the file decls, just leave them in their original
order (the generator sorts them).
Thanks you for your contribution to the Azure-SDK-for-Go! We will triage and review it as quickly as we can.
As part of your submission, please make sure that you can make the following assertions:

[ ] I'm not making changes to Auto-Generated files which will just get erased next time there's a release.

If that's what you want to do, consider making a contribution here: https://github.com/Azure/autorest.go

[ ] I've tested my changes, adding unit tests where applicable.
[ ] I've added Apache 2.0 Headers to the top of any new source files.
[ ] I'm submitting this PR to the dev branch, or I'm fixing a bug that warrants its own release and I'm targeting the master branch.
[ ] If I'm targeting the master branch, I've also added a note to CHANGELOG.md.
[ ] I've mentioned any relevant open issues in this PR, making clear the context for the contribution.

Can one of the admins verify this patch?

Dependent on https://github.com/marstr/goalias/pulls

Fixes https://github.com/Azure/azure-sdk-for-go/issues/1550


Envelope Icon not sending to Feeback Page

Hi Team,
on the Home Page the envelope icon when clicked is sending to FEED instead of Feedback page, can you check it ?

Thanks,
Tiago

@TiagoaTavares I think the feedpages aren't ready yet. It should eventually go to the pages below. Ill change it once they are done.

this is WIP on another branch

Hello guys, just to follow-up on this. Which ID it is ?


fix: auto acknowledge self events before ignoring

Summary
Since #2283 event requests are no longer auto acknowledged in the processEvent function. This behavior was handed over to the autoAcknowledge middleware. This created a gap in the ignoreSelf middleware since the events reaching it were assumed to already be acknowledged. This PR aims to add auto acknowledge behavior into the ignoreSelf global middleware.
Requirements (place an x in each [ ])

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

Does this address the current non-acknowledged events in the Assistant class, or unrelated?

@misscoded this should be unrelated


Carrying extra information in RemoteData.Loading?

Consider a simple SPA, where you have two buttons, "Load Next" and "Load Previous",
and if I click one of them, I expect the one been clicked change it's text to "Loading", and the other one stay put.
Currently, if you want to achieve this, then you need to track the event source (which button gets clicked) in a separate filed somewhere in the model, maybe something like {remoteData : (Maybe ButtonId, RemoteDate.WebData SomeData)}. However, this has a problem, namely, it becomes a Cartesian product, and not every member in the set makes sense, like, when the state is(Nothing, RemoteData.Loading), we always expect a Just buttonId, not Nothing, (assuming you are interested in which button gets clicked). (In case you are wondering, you cannot change the type from Maybe ButtonId to ButtonId, because when you are setting up the initial model, you will need a placeholder there.)
Any opinion?

Let's suppose that next and previous are tied to page IDs, which are just Ints. For something like this I'd usually have a model of Dict Int (WebData Page).
Clicking LoadNext 5 calls { model | pages = Dict.insert 5 Loading model.pages }, fetch 5 ).
Each button can then see if it's loading, available, or not asked, and display them however you feel is appropriate. As a bonus, you get page caching.


new term request: Eoxin biosynthetic process / Eoxin A4 biosynthetic process

Hi,
Pubmed 23242647 describes the synthesis of eoxin A4 by ALOX15 from 15S-hydroperoxy-eicosatetraenoic acid (15S-HPETE).
Could you please create these new tems?
Many thanks.
Guilaine

@Argoudpuy ; we need a little more info (standard request stuff).
Suggested term def.
Suggested is_a parent.

As far as I could tell eoxin A4 synthesis corresponds to GO:1901753 leukotriene A4 biosynthetic process, while eoxin synthesis is GO:0046456 icosanoid biosynthetic process
I added some synonyms.


Update inflation and image export workflows to use Debian Worker-Images from compute-image-import GCP project

Update inflation and image export workflows to use Debian Worker-Images from compute-image-import GCP project
/cc @darthmelonder
/assign @darthmelonder

failed unit test is fixed in PR #122, I think we can safely merge this one.


Fix freeze issue on RecentVideos screen

https://openedx.atlassian.net/browse/MA-1818
#432 deleted the file layout-land/activity_myvideos_tab.xml which caused this issue.
So, I've brought it back. Also, #449 removed the null check on PlayerFragment's re-initialization (https://github.com/edx/edx-app-android/pull/449/files#diff-f06c48353458aca116f653c06a347fdeL207), which caused the video player to freeze after orientation change, so restored that check as well.
@bguertin @1zaman plz review

:+1:


Expand API of CuyZ\Valinor\Mapper\Tree\Message\Message to allow for i18n and API customization?

Currently, Message is only an instance of Stringable: https://github.com/CuyZ/Valinor/blob/396f64a5246ccfe3f6f6d3211bac7f542a9c7fc6/src/Mapper/Tree/Message/Message.php#L9-L11
When using this library, you can expect the most common usage to be (pseudo-code):
try {
    parse_validate_and_map($request);
} catch (MappingError $failed) {
    response_code(422);
    mapping_error_to_failure_response($failed);
}

In order to effectively write mapping_error_to_failure_response(), especially considering concerns like translation, i18n, and general re-shaping of the response, it would be helpful to have:

the internal message (like now), valid for loggers and such
a declared code (currently, Exception#getCode() is used, but unsure if it is stable long-term?)
the original value (which failed mapping - at the level we're at) - this is useful for interpolating in error messages
the wished type (to which we failed to cast/map towards) - this is useful for interpolating in error messages
the logical path of the value, not the string as key in MappingError, useful for processing/filtering errors later on

In practice, the use-case is mostly around i18n of error messages, plus providing a bit more information on each Message.
A good example of such serialization is https://datatracker.ietf.org/doc/html/rfc7807

Hi @Ocramius, thanks for the suggestion.
Instead of modifying the Message signature, I'd suggest something else, let me know what you think about it.
Currently, when the mapping fails, an instance of MappingError is thrown, the only existing implementation by now is CannotMapObject. This is the entry point for users when something goes wrong, and I assumed that giving access to the describe method like below makes sense:
https://github.com/CuyZ/Valinor/blob/ea842e2efb94ded9f9f2ebce1484f8d4e7e29237/src/Mapper/Exception/CannotMapObject.php#L33-L56
My suggestion is this: removing the describe method and give access to a new method:
interface MappingError extends Throwable
{
    public function rootNode(): Node;
}

This way, something like this could be done:
try {
    parse_validate_and_map($request);
} catch (MappingError $failed) {
    $printError = static function(Node $node) use (&$printError): void {
        if ($node->isValid()) {
            return;
        }

        foreach ($node->messages() as $message) {
            if ($message instanceof Throwable) {
                echo "path: {$node->path()}" . PHP_EOL;
                echo "expected type: {$node->type()}" . PHP_EOL;
                echo "message: {$message->getMessage()}" . PHP_EOL;
                if ($message instanceof HasCode) {
                    echo "code: {$message->code()}" . PHP_EOL;
                }
            }
        }

        foreach ($node->children() as $child) {
            $printError($child);
        }
    };

    $printError($failed->rootNode());
}

The missing part in the code above is the original value, which is currently not accessible from outside the Node — but it could be accessed from Node::$shell->value() with a method that gives access to it somehow.
WDYT?

Hi @Ocramius, I've done some work on this topic, could you check #38?
More information in the chapter "Validation" of the README.

I think for the validation side of things it would be nice to have something similar to symfony/validator with ConstraintViolation and ConstraintViolationList and then MappingError::getViolations() or something similar could return a flat list of errors, with messageTemplate and similar thingies would make whole i18n side of things really nice and friendly.

Hi @aurimasniekis, being able to get a flat list of errors might be useful indeed; I already kept an open door for this feature — see https://github.com/CuyZ/Valinor/pull/38#issue-1081593482. Whether it should be part of the library or not is still to discuss, please tell me what you think of it.
Concerning the messageTemplate feature, there's probably still some work to do. With the current status of the API, how would you expect to be able to extend it and add i18n features? TBH I was waiting for @Ocramius's feedback to have a more precise thought of the direction to take; I know he's been very busy lately, we can surely keep up working on this topic in the meantime. 🙂

The messageTemplate would basically provide the needed i18n support because it would give you an error message template with variables, so then u can translate them and pass variables to any translation library.

Hi @Ocramius / @aurimasniekis, I've made major work on this topic, can you check #52 and tell me what you think about it? I think this is a good approach, but you may have feedback on it. 🙂

Hi @Ocramius / @aurimasniekis, I've made major work on this topic, can you check #52 and tell me what you think about it? I think this is a good approach, but you may have feedback on it. 🙂

Hi :wave: looks neat here.
I'm on a journey of deserializing/validating/mapping payloads to domain models, and found this repo.
My main concern is exactly this topic, and i think https://github.com/CuyZ/Valinor/issues/2#issuecomment-1001774650 gets it right.
symfony/validator brings full i18n compatibility: https://symfony-translations.nyholm.tech/, an object constraint mapping system to read from (let it be PHP8 attributes soley), as well as a ConstraintViolationList to help further response serialization.
My main question would be: can we read the constraint attributes from the target object, while applying them on the source model? Thus achieving "validation before hydration" with i18n friendly error handling. And are you willing to build/maintain such a bridge? Or first-class support maybe? :angel:
Note, under the hood it can transform real type info to a Type constraint (if not already present), to leverage its i18n translation messages.

Hi @ro0NL, thanks for your interest!
To answer your main question: yes, the library is able to read the attributes from the nodes (as I call them in the library) — they can come from class properties or construct parameters. Now the question whether these attributes should be used for validating an input or not is another topic, let me explain my thoughts on this.
One of the main aspect of this library that I try to stand to, is that it should be as much independent to the objects it is mapping as possible. These objects are supposed to be DTO/VO that could in theory be mapped manually; in the end this library is just here to do the recursive type casting automatically.
PHP attributes are metadata; adding a "validation attribute" (a constraint in Symfony's vocabulary) to an object does not guarantee it will be used in every case — for instance if the object is instantiated manually.
Take this example:
final class SomeClass
{
    #[\Symfony\Component\Validator\Constraints\Email]
    public readonly string $email;
}

(new Foo())->email = 'not an email';

The email property is hydrated with a value that is not "supposed" to be correct, yet this example makes sense and the state of the object is perfectly valid, yet the business rule associated to it is not fulfilled.
In my opinion, this should be done like this:
final class SomeClass
{
    public function __construct(
        public readonly string $email
    ) {
        if (! some_email_check($email)) {
            throw new Exception('invalid email');
        }
    }
}

new Foo('not an email'); // error

Now I understand the concern you have with the full i18n support from Symfony, and I'm sure we can work on finding a way to have a bridge that can help, without having to use attributes.
There are two things: the first one it being able to translate the messages. Well, that's kind of easy and not really related to the symfony/validator package, for instance:
try {
    return $this->mapperBuilder
        ->mapper()
        ->map(SomeClass::class, [/* … */]);
} catch (MappingError $exception) {
    $node = $exception->node();
    $messages = new \CuyZ\Valinor\Mapper\Tree\Message\MessagesFlattener($node);

    $translator = new \Symfony\Component\Translation\Translator('fr_FR');
    $translator->addLoader('xliff', new XliffFileLoader());
    $translator->addResource('xliff', 'vendor/symfony/validator/Resources/translations/validators.fr.xlf', 'fr_FR');

    foreach ($messages as $message) {
        echo $translator->trans((string)$message);
    }
}

Like this we can use any string that is part of the translated strings from Symfony's libraries.
Now the second part is more tricky, it would be using the constraints API without the attributes.
final class SomeClass
{
    public function __construct(
        public readonly string $email
    ) {
        $validator = \Symfony\Component\Validator\Validation::createValidator();
        $violations = $validator->validate($foo, new \Symfony\Component\Validator\Constraints\Email());

        if (count($violations) === 0) {
            return;
        }

        throw new ThrowableMessage($violations->get(0)->getMessage());
    }
}

Note that with the current API it is possible to "throw" only one constraint, but we could imagine that the library could support aggregate message that would be flattened by the mapper.
What are your thoughts? 🙂

I'd say (business) validation is a service-layer concern. A complex constraint may require eg. a DB connection.
While i strive for "data models in valid state", im not nessecarily fan of them having runtime assertions all over. In that sense i prefer the type system to do it's work (let it be enhanced psalm/phppdoc).
As such, to me new Foo('not an email'); // error is business rule violation, that should bubble up somewhere. Or maybe it's intended to set this value ... that's an implementation detail we don't nessecarily know. Ultimately the validator can still be applied on the mapped object, if sources are uncertain.
However, to satisy any potential system assertion i tend to prefer pre-validating the input, rather than catching some hydration error (and then have this issue of how to derive a i18n error from it). In that sense, after validating any mapping exception can bubble IMHO (or caught for some logging/special handling purpose).
To make this stupidly-simple from a Symfony app POV, i'd aim for a minimal user API:
try {
  $data = map(Some::class, $reques->getContent());
}
catch (InputError $e) {
  return serialize($e->getViolations()); // i18n errors
}
// catch (MappingError $e) {
//   $logger->error(...);
// }

Note i have no super strong opionion about pre-validating the input vs. post-validating the mapped object technically (i think semantically it's more pure though). But post-validating requires another solution for i18n type errors, for this library to solve.

Btw see also https://symfony.com/blog/new-in-symfony-5-4-serializer-improvements#collect-denormalization-type-errors for how Symfony, eh, does it.
IMHO it suffers the same design issue ... that is; a ton of boilerplate and still no i18n type-errors nor i18n value-errors out of the box :joy:
It can also leak a partial (mapped) object .. let's not go there :)
I understand this is maybe a lot of hassle, just to have i18n type-errors :) (assuming we'd post-validator the mapped object still using symfony/validator for the remaining i18n value-errors)
IMHO ideally we get the i18n type- and value-errors in one go ... otherwise i'll take things for granted and have system type-errors facing end-users :shrug: it's not the end of world :grin:
I've no practical experience yet with your library, so to hijack this issue ... if the current API works for OP (@Ocramius) ... feel free to ignore me :)

I'd say (business) validation is a service-layer concern. A complex constraint may require eg. a DB connection.
…
However, to satisy any potential system assertion i tend to prefer pre-validating the input, rather than catching some hydration error

The issue with pre-validating the input is that you have no idea what's inside, you'd have to check the types for every parameter, and worst if the value you seek is deeply nested. The goal of this library is to help with formatting any input to a known nested structure — which can then be used to do the business logic associated to it — while aggregating low-level errors.
As you said complex constraint is a service-layer concern — I agree, and this is out of the scope of this library and shall be handled correctly by the domain of an application, let it be "inside" a command/query bus for instance.

And maybe this library should already consider i18n decoding errors, for which SF has no translations available AFAIK. But that's least of my concern :)

Well we could of course imagine providing i18n for existing errors of course, I'll think about providing a basic list for it.

Btw see also https://symfony.com/blog/new-in-symfony-5-4-serializer-improvements#collect-denormalization-type-errors for how Symfony, eh, does it.
IMHO it suffers the same design issue ... that is; a ton of boilerplate and still no i18n type-errors nor i18n value-errors out of the box 😂

While it looks a bit different from how you'd use Valinor to flatten messages, I don't think that's too much to do. Plus a helper could probably be used to help any controller/service to do it with only one line (IMO).

It can also leak a partial (mapped) object .. let's not go there :)

I fully agree on this one. 😁

IMHO ideally we get the i18n type- and value-errors in one go

For the reasons I listed above, I really think that "complex value errors" can not and will not be somehow handled by this library, although this may change if people really see a benefit to it and we find a proper way to do it.

I've no practical experience yet with your library, so to not hijack this issue ... if the current API works for OP (Ocramius) ... feel free to ignore me :)

I think Ocramius can unsubscribe from this thread if he wants not to be disturbed 😅 — in the meantime I'm very happy to have feedback so don't feel forced to stop the discussion! I think we are still in the scope of the original topic so we can continue there.

The goal of this library is to help with formatting any input to a known nested structure

fair :) then my main concern is still the boilerplate involved to achieve i18n type-errors out-of-the-box-ish.

I really think that "complex value errors" can not and will not be somehow handled by this library

also fair :) for the record, all im suggesting is to invoke symfony/validator either on input, or the mapped object, to activate "its" constraint validation. If we decide to validate the mapped object, then perhaps this is out-of-scope here i agree.

I don't think that's too much to do

i dont need the boilerplate in every controller for sure ... so im curious who will own the ultimate abstraction eventually, time will tell :)

fair :) then my main concern is still the boilerplate involved to achieve i18n type-errors out-of-the-box-ish.

Right, I'll think about it.
In the meantime, please note that @framjet began a Symfony bundle to integrate this library in the framework, see https://github.com/framjet/php-valinor-bundle — maybe that's where we could have some helper for automatic i18n.

Hi there, I recently merged #106 which improves a lot the way the messages can be handled by the library.
You can check out the new documentation chapter about it; I'd love to have some feedback on it. 🙂


Reports containing errors should make it exit with code 1

This is what ESLint does.
Example:
const VFile = require('vfile');
const reporter = require('vfile-reporter');

const f = new VFile();
f.quiet = true;
f.fail('hi');
console.log(reporter(f));

Interesting! How are you using vfile-reporter?
My initial feels are that it’s out of scope, as I intend the project to be more like just an output generator, and not always as the final thing.
P.S. There’s also vfile#hasFailed(), which could be used below your example code like so:
// ...
process.exit(f.hasFailed() ? 1 :0);

Closing as I don’t think this should be done by a reporter! I’ve opened a new issue for a feature which might make hasFailed useful for warnings as well: https://github.com/wooorm/vfile/issues/7.

Agreed, but you should add that to the docs. It's easy to assume it works like that when you have the sentence:

Originally forked from ESLint’s stylish reporter, but with some coolness added.

 And I don’t think they should. No non-CLI should deal with exit codes.
This also sounds sort-of double: the whole reason of fail is to throw fatal errors to stop node (with non-zero), if you’re explicitly preventing that from happening by settings quiet: true, surely you must handle the exit code too?

You're right. No sure what I was thinking...
Unrelated to this issue, but if VFile exposed an errorCount I could do this: https://github.com/sindresorhus/xo/blob/bb76a2d28817a176056c2da37f09112810643b4a/cli.js#L87

👍

Unrelated to this issue, but if VFile exposed an errorCount I could do this: https://github.com/sindresorhus/xo/blob/bb76a2d28817a176056c2da37f09112810643b4a/cli.js#L87

Yeah, that’d be very useful indeed!
TBH, I’m thinking of moving VFile to more of just a plain object, or at least removing lots of methods, that way an external module could do exactly the thing you want, like calculating an exit-code from a file, much better!

I don't think it should be a plain object. There's benefit in being able to detect that it's a VFile.
Let's continue this in a new issue → https://github.com/wooorm/vfile/issues/8


Estimate fees for joining a stake pool

Issue Number
WB-32: #1094 #1096 (Found it easier to tackle both at once)
Overview
Also see commit history, but main points:

[x] Added API endpoint GET /stake-pools/{stake-pool}/wallets/{wallet}/fee
[x] Added feeBalance :: CoinSelection -> Word64. I re-use selectCoinsForDelegation together with feeBalance to implement the joinStakePoolFee handler.
[x] Integration tests:

STAKE_POOLS_ESTIMATE_FEE_01 - fee matches eventual cost
STAKE_POOLS_ESTIMATE_FEE_02 - empty wallet cannot estimate fee
STAKE_POOLS_ESTIMATE_FEE_03 - can't use byron wallets
STAKE_POOLS_ESTIMATE_FEE_04 - invalid pool and wallet ids

bors r+

bors r+

bors r+


MRTK adds several MixedRealityInputModule component to main camera

Describe the bug
Scenario: I have 3 MixedRealityPlayspace object in the scene, but ONLY ONE active per time. Each of them has a Camera as a child in it. Each time I open the editor, I got an error saying "For Mixed Reality Toolkit input to work properly, please remove your other input module(s) and add a MixedRealityInputModule to your main camera." and I found multiple MixedRealityInputModule attached (instead of the only one I put) to the active Camera. I have then to manually remove them each time it happens.
Expected behavior
MRTK to understand that a MixedRealityInputModule is already there.
Screenshots

Setup

Unity Version 2018.3.12f1
MRTK Version 2.0.0 RC1

Target Platform
UWP:

Desktop
HoloLens
HoloLens 2
WMR immersive

@fcagnetta, can you see if this still repros with the latest mrtk_development? RC1 was quite a while ago and we churned a bunch of things since then. Please reopen if it's still happening (along with a min-repro we can follow to hit the issue)


Consider adding leading to BoP

superceded by https://github.com/BostonGlobe/elections-2016/issues/520


These larger tile sets are so reminescent of Julia sets

Is there any chance there could be some kind of connection? :)
Am I missing an obvious reason why this would be the case?
The fractal nature of Penrose tiling has been shown before: https://www.researchgate.net/publication/237519315_On_the_fractal_nature_of_Penrose_tiling
I am wondering if there is a supertile organization that would more exactly match a Julia set (and perhaps the different monotiles in the family match Julia sets with other parameters).
Also crazy idea: what if it is a monotile that does not require reflecting tiles that corresponds to a Julia set?
e.g., is this a monotile: (probably not)

maybe this one? (seems to be periodic :()

For future reference, it seems to me that the Github issues section of the visualizer is not the place to discuss the underlying research.  I'm barely a Github user at all, and this will just confuse me.
It seems to me that the fractal structure brought about by the substitution rules given in the paper is partly accidental, and not fundamental to the nature of this tiling, so I wouldn't read too much into it.  That being said, I'm not an expert on tiles with fractal boundaries.  There could be some reason for the visual similarity, but I wouldn't know where to look for it.
Joseph Myers has exhaustively characterized all other polykites up to size 21 and found no other monotiles, so I know without investigating that your two examples either tile periodically, or not at all.

Thanks @isohedral and sorry for bringing this up at the wrong place.


no taxid assigned reason

Hi there,
This is a great tool thanks for sharing!
Is there any difference between contigs with "no taxid assigned" that have either "no hits to database" vs. "hits not found in taxonomy files"?
Thanks!

Dear @sheaster,
Thanks for your question! If a contig has a "hits not found in taxonomy files", it means there is a DIAMOND hit, so it's not a completely unknown protein, but there is no mapping found for this protein in the NCBI taxonomy files. This may be because there is a small offset between when nr and NCBI taxonomy are updated, or for other reasons that we do not control but is up to NCBI. When the contig has "no hits to database", it means there is really no DIAMOND hit. We keep those reason in the output file so that a user can delve deeper into the hits if that is required, but for the CAT/BAT algorithm itself, both reasons are treated equally.
I hope this helps, let me know if you have more questions!
Best wishes,
Bastiaan


Set OpenID Connect idtoken as String session attribute

I propose to add a string attribute to the session with the idtoken.
This will decouple the id token from the BaseOpenIDConnectAuthenticator.Authorization object.
This way I don't need a compile time dependency to the tomcat-oidcauth library in my web application and still can access the idToken.
It will also avoid possible future conflicts if the Authorization class changes between releases.

It would also make sense to make the redirect URLs available as a List as request attributes. This way the tomcat-oidcauth.jar is not required at compile time in a web application.
I can provide a pull request for that as well, but will first wait if you approve this request.

Thanks for the PR. What's wrong with accessing the Authorization object as a Java bean? The webapp should certainly not have the authenticator as a compile time dependency.

Thanks for reviewing the PR. I don't like using reflection to access java beans methods. But I understand your remark. Accessing it as a java bean will avoid adding any compile time dependencties.
It just would be convient to have the most used properties available as a String object in the session. From the web application perspective that would tipacally be the id token and the list of redirect URLs.

Can someone show an example of accessing the Authorization object as a Java bean without needing the oidc-auth included in the webapp?  Possibly also without JSTL/EL?


In team DM modes, using the invis powerup turns your viewmodels entirely black as opposed to a teamcolor cloak.

this is a visual bug that only you will see, no idea what costs it

Closing as we've changed this to be non-coloured intentionally and I believe the black arms but has been fixed.


Cosim mismatch: black-parrot is not trapping on an illegal instruction

Summary
For some reason Black-Parrot is trying to commit an illegal instruction instead of trapping, even though there were previous successful traps on illegal instruction before this one.
Instruction at PC 0x800070fc is an illegal one (0xfdc9a1e7).
...
3 0x00000000800070f4 (0x03cbecbb) x25 0xffffffffffffff2a DASM(0x03cbecbb)
3 0x00000000800070f8 (0x0253c23b) x4  0xffffffffffffffff DASM(0x0253c23b)
3 0x0000000080012000 (0x340494f3) x9  0x000000008002fcf8 DASM(0x340494f3)
[error] EMU PC<PHONE_NUMBER>012000, DUT PC<PHONE_NUMBER>0070fc
[error] EMU INSN 340494f3, DUT INSN fdc9a1e7
[error] EMU WDATA<PHONE_NUMBER>02fcf8, DUT WDATA<PHONE_NUMBER>007100
[error] EMU MSTATUS a00101800, DUT MSTATUS 00000000
[error] DUT pending exception -1 pending interrupt -1
COSIM_FAIL
$finish called from file "/mada/users/nkabylka/black-parrot/bp_top/test/common/bp_nonsynth_cosim.v", line 121.
$finish at simulation time                87680
           V C S   S i m u l a t i o n   R e p o r t
Time: 87680 ps

Binary: riscv_illegal_instr_test_0.S.riscv.zip
Cosim full trace: illegal_inst_trace.txt
Dromajo stand alone run trace: trace.txt (this is to view instructions that through exception)
Previous successful traps
Before this mismatch, there was a series of illegal instruction occurrences where Black-Parrot did trap:

at PC 0x80006e28 instruction 0x31a4fa5f

3 0x0000000080006e20 (0x018e2323)                        DASM(0x018e2323)
3 0x0000000080006e24 (0x00be0423)                        DASM(0x00be0423)
3 0x0000000080012000 (0x340494f3) x9  0x000000008002fcf8 DASM(0x340494f3)
3 0x0000000080012004 (0x000704b3) x9  0x000000008002fcf8 DASM(0x000704b3)

at PC 0x80006dc4 instruction 0x00006681

3 0x0000000080006dbc (0x004e0023)                        DASM(0x004e0023)
3 0x0000000080006dc0 (0x40025b1b) x22 0x0000000000000078 DASM(0x40025b1b)
3 0x0000000080012000 (0x340494f3) x9  0x000000008002fcf8 DASM(0x340494f3)
3 0x0000000080012004 (0x000704b3) x9  0x000000008002fcf8 DASM(0x000704b3)

Thanks! Confirmed bug.

RISC-V defines that funct3 for JALR == 3'b000.  In our decoder, we do not make this assertion and just check the opcode.
      `RV64_JALR_OP :
        begin
          decode.pipe_ctrl_v = 1'b1;
          decode.irf_w_v    = 1'b1;
          decode.fu_op      = e_ctrl_op_jalr;
          decode.baddr_sel  = e_baddr_is_rs1;
        end

Check out #499 .  I'll leave the issue open because it's possible there are more illegal instruction cases which will be detected by this test.  Those aren't ordinarily executed, so they tend not to be well-tested

Illegal instruction space is fairly well tested now, passing RISCV-DV tests. Will open new issues if any discovered


After and Before triggered only once

Sorry if the tittle is not exactly accurate to the issue. It's the behavior I got.
Versions

vee-validate: 2.0.X, tested with all most all from 2.0.0 to 2.0.9
vue: 2.5.16
vuetify : 1.0.16

Context
I'm trying to validate two v-text-fields with before and after rules that are binded at each other field.
Describe the bug
Validate with after and before does not trigger correctly after changing date value and updated rule inline.
Demo Link
CodePen
Steps to reproduce the behavior:

Select a date for StartDate or EndDate
Select the other date (To reproduce bug : Start Date should be > EndDate)
Error is correctly trigger
Select a new date where StartDate < EndDate
Error is still shown

It's even worst if you remove the data-vv-delay...
Expected behavior
Every time a date is selected in picker, text fields are updated and should be revalidated as rules and v-models are updated.
Desktop (please complete the following information):

OS: Windows 10
Browser : Chrome
Version : Version 67.0.3396.87 (64 bits)

Any hint or workaround on this ? Should I post this on Vuetify repo ?

Was not using date_format.
Did it, and solved issue. Sorry.

Sorry I didn't find the time to address this issue. Glad it worked out.

No prob, thought I had it to my codepen, and that it was deeper than just a missing params.
Sorry for inconvenience. Thanks for your work.


Review Latest Revisions To OpenJS SBOM Best Practices & Secure Releases and CVE Management for the JS Ecosystem

Jordan has made updates to two more docs, let's discuss and provide feedback:

OpenJS SBOM Best Practices  https://hackmd.io/ujVlBEjqQCSvxfspqd3gcw
Secure Releases and CVE Management for the JS Ecosystem https://hackmd.io/1fnYDGmySt2jQDV8zXgiRw

Doc look generally good. I am wondering for the secure releases, there is still some contention I think between "publish locally" which is what this doc goes over primarily and "publish from ci". Would it be good to call that out in here? And maybe have a section about securely publishing from CI?
Ideally I would say that for a secure CI publish you would have the following:

A granular access token on npm
A CI workflow for release triggered off SCM push (typically to main, maybe PR and staging releases)
Access control is managed at the SCM layer (github user permissions, etc)

There are some systems which also add on user provenance information to these types of setups, but I would guess it is better to start with limited opinions and build from there?

Additionally, "provenance" tells you nothing except which workflow published the package, so I think it's premature to recommend it as a "security" practice.

Yeah I see where you are coming from, I am not claiming it is perfect or even "great", but I think it will be something end users ask, and ideally anything is better than nothing if they go off and do some CI publish that is even further from a recommended path. Anyway, Not super opinionated here, just a gap I noticed.

Resolving as discussion concluded, comments will be integrated into doc by @ljharb


please add  contextMenuBuilder for markdown

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
rt
Proposal
rt

When opening an issue, please try to provide all information indicated in the template so that we can address this issue properly (See bug issue template here)
Closing this issue, since no information has been provided. Once you updated the issue with more information, please write in the comments and I will re-open this for further discussion. Thanks!


C++ callback API: properly handle and test unimplemented RPC method

Fixes and tests an important issue by making sure that callback-only servers properly handle unimplemented methods. This is an easy follow on after #18239 . This is an easy way to do this instead of creating a separate UnknownSyncMethod call (as in the sync API) or UnimplementedAsyncRequest/Response types (as in the async API) because the callback API is based around the idea of background polling; as a result, as long as the callback CQ exists for some purpose, it will be able to also drive an additional service that the user doesn't even know about and which has a default implementation that treats RPCs as unimplemented.

Only failure is python gevent on Windows so purely unrelated


Sheetsconnection

Complete record scores TODO. Merging so I can pull from main.

Not merging since api not created for club account yet


docker build does not copy linked file(s) outside of buildpath

Use Case:
There are many cases where it would be helpful to allow referenced file(s) (outside of the current build path) to be used but I'll add just one for now to make the issue a bit more concrete...
When having to build many docker images which rely on common configuration files (e.g. repo files, system config, etc.), it is not optimal to have to copy these files into the Dockerfile directories that depend on them.  If a change is needed to one or all of these files, one must ensure that all the directories are appropriately updated. If file links (symlinks) were allowed, this would enable one to maintain & update a single file that can be used across all Dockerfiles
eg. Directory structure:

.../DockerImages/

configfile
image1/

Dockerfile
configfile -> ../configfile

image2/

Dockerfile
configfile -> ../configfile

e.g. Dockerfile
...
COPY configfile /etc/program/
...
e.g. Docker image1 build
$ cd DockerImages/image1
$ docker build -t images:image1 ./
Proposal:
I am not proposing that we persist the links, rather copy/dereference the linked file into the build-path (on the client side) prior to sending it off to the server to create/build the image.  My understanding is that on the client side, the Dockerfile directory is tar'd up prior to sending to the server.  You could use the [--dereference (-h) flag] (http://www.gnu.org/software/tar/manual/html_node/dereference.html) to achieve this behavior.
docker version
Client:
Version:      1.9.1
API version:  1.21
Go version:   go1.4.2
Git commit:   a34a1d5
Built:        Fri Nov 20 13:25:01 UTC 2015
OS/Arch:      linux/amd64
Current Docker version:
Server:
Version:      1.9.1
API version:  1.21
Go version:   go1.4.2
Git commit:   a34a1d5
Built:        Fri Nov 20 13:25:01 UTC 2015
OS/Arch:      linux/amd64
docker info
Containers: 0
Images: 6
Server Version: 1.9.1
Storage Driver: devicemapper
Pool Name: docker-253:0-201129991-pool
Pool Blocksize: 65.54 kB
Base Device Size: 107.4 GB
Backing Filesystem: xfs
Data file: /dev/direct-lvm/data
Metadata file: /dev/direct-lvm/metadata
Data Space Used: 12.56 GB
Data Space Total: 15.2 TB
Data Space Available: 15.19 TB
Metadata Space Used: 79.18 MB
Metadata Space Total: 17.05 GB
Metadata Space Available: 16.97 GB
Udev Sync Supported: true
Deferred Removal Enabled: false
Deferred Deletion Enabled: false
Deferred Deleted Device Count: 0
Library Version: 1.02.107-RHEL7 (2015-10-14)
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 3.10.0-229.14.1.el7.x86_64
Operating System: Red Hat Enterprise Linux Server 7.2 (Maipo)
CPUs: 32
Total Memory: 188.6 GiB
Name: deployer-01.cloud.svl.ibm.com
ID: OEPB:5NYE:KJDO:QGSZ:2BCD:5LXZ:KH22:NLOY:2JGK:2XUR:NHVF:O4YV
Debug mode (server): true
File Descriptors: 18
Goroutines: 26
System Time: 2016-01-08T11:51:50.047754961-08:00
EventsListeners: 0
Init SHA1: ac8f1bfea8577235f1e9c2dfaace54207d64ee49
Init Path: /usr/libexec/docker/dockerinit
Docker Root Dir: /var/lib/docker

See also #18789

Follow #18789. Closing


Fix for package upgrading

pkgin has an issue where it sorts packages by string.  This fix will allow upgrading to happen successfully.

Fix for #3811

Don't know how to fix the tests but my image only has varnish 3.0.5 and your test is failing on varnish 2.3.4.
~# pkgin se varnish
varnish-3.0.5nb1     High-performace HTTP accelerator

=: package is installed and up-to-date
<: package is installed but newer version is available
>: installed package has a greater version than available package

Those are unit tests which run on Linux and Windows (which definitely doesn't have varnish installable via pkgin), so the tests must be stubbing the underlying pkgin calls.  Something in your patch has broken the unit tests and needs to get fixed (likely the unit tests were incorrect and also need fixing for the new behavior).

This looks like it'd be easy enough to salvage, but we just don't have the time to do it ourselves.

I apologize for not responding to this.  I just don't know enough about the unit testing to be of assistance.  The main goal of the request was to organize the results so the latest version would be on bottom.  By default Chef will choose the last package in the list even if the version is not the latest.  I also don't have time to help solve this so I guess close it as a 'known issue' or hope someone else can pick it up.  Thanks.


chore: version bump of trivy to 0.24.2

Signed-off-by: Engin Diri<EMAIL_ADDRESS>This PR fixes #1007, this is needed to ensure the #1003 is working, as we need the TRIVY_INSECURE flag.

@dirien it is required also to update matcher.go and matcher_test.go

@danielpacak: hiere the CLA is with a green checkmark. Don't know what is going on in the other PR 😅


Implement cache in Local File System

WHY
To create cache of the result of bundle install
WHAT
Put/Get a directory as tarball cache on local file system.
TODO

[x] Deflate tarball with relative path
[x] Put tarball cache to local directory
[x] Get tarball cache from local directory

@wantedly/infrastructure plz review this :octocat:


Heavy gear 4e v 0.26. Feature added Creature sheets, bugfixes,

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description

Ver 0.26
Bugs
Spray and stun do not work in Inf Weapon (Fixed)
Deleting inf weapon does not work (fixed)
Silent weapons displaying no damage (fixed)
Changes
Change to Max Ammo. (fixed)
toggle melee in inf and gear
Ammo trait should be called Mode (Fixed)
Incomplete
Damage types not complete
Token bars changer
See description should display the see desc text
Known Bugs
Gear Weapon adder not working. Doesn't populate the field properly, click checkbox to fix
Ideas
Add Creature sheet (Implemented)

Character Sheet Info Roll20 Internal Use only.


Added dataset member to HTMLElement object.

Added partial dataset support. Will be updated once Proxy object available for nodejs.
Signed-off-by: Egor Bolgov<EMAIL_ADDRESS>

Will create new pull request.


ivy-avy is not loaded by default

What did you expect to happen?
ivy-avy is called in the ivy minibuffer when pressing its keybinding (C-').
See https://oremacs.com/swiper/#key-bindings-for-single-selection-action-then-exit-minibuffer
What actually happened?
C-' is undefined

Additional details:
My private config: https://github.com/Andre0991/dotfiles/tree/master/.doom.d
Also, ivy-avy is not defined.
Evaluating (require 'ivy-avy) fixes it, but I'm not sure what's the Doom way of fixing this - I suppose it involves modifying the ivy module.
Steps to reproduce:

Press SPC SPC
Press C-'
ivy-avy is not called.

System information:

emacs   version    26.3
        features   JPEG RSVG IMAGEMAGICK GLIB NOTIFY ACL GNUTLS LIBXML2 ZLIB TOOLKIT_SCROLL_BARS NS MODULES THREADS LCMS2
        build      Aug 30, 2019
        buildopts  (--disable-dependency-tracking --disable-silent-rules --enable-locallisppath=/usr/local/share/emacs/site-lisp --infodir=/usr/local/Cellar/emacs-plus/26.3/share/info/emacs --prefix=/usr/local/Cellar/emacs-plus/26.3 --with-xml2 --without-dbus --with-gnutls --with-imagemagick --with-modules --with-rsvg --with-ns --disable-ns-self-contained)
        windowsys  ns
        daemonp    server-running
doom    version    2.0.9
        build      grafted, HEAD -> develop, origin/develop, origin/HEAD e7b7f67 2020-06-12 17:03:19 -0400
        dir        ~/dotfiles/.doom.d/
system  type       darwin
        config     x86_64-apple-darwin18.5.0
        shell      /usr/local/bin/bash
        uname      Darwin 19.4.0 Darwin Kernel Version 19.4.0: Wed Mar  4 22:28:40 PST 2020; root:xnu-6153.101.6~15/RELEASE_X86_64 x86_64
        path       (~/.jenv/shims ~/.jenv/bin ~/.nvm/versions/node/v11.6.0/bin ~/.opam/system/bin ~/Dropbox/nu/commands ~/.rbenv/shims ~/dev/nu/ios-cli ~/sdk-flutter/bin/cache/dart-sdk/bin ~/sdk-flutter/bin ~/dev/nu/mini-meta-repo/monocli/bin ~/dev/nu/nucli ~/dev/nu/nudev /usr/local/bin /usr/bin /bin /usr/sbin /sbin /Library/TeX/texbin /usr/local/MacGPG2/bin ~/go-workspace/bin /usr/local/opt/go/libexec/bin ~/Dropbox/bin_path /usr/local/opt/fzf/bin /usr/local/Cellar/emacs-plus/26.3/libexec/emacs/26.3/x86_64-apple-darwin18.5.0/)
config  envfile    envvar-file
        elc-files  0
        modules    (:completion company ivy :ui doom doom-dashboard doom-quit hl-todo hydra indent-guides modeline ophints (popup +defaults) vc-gutter vi-tilde-fringe window-select workspaces :editor (evil +everywhere) file-templates fold lispy multiple-cursors snippets :emacs dired electric undo vc :checkers syntax :tools (eval +overlay) lookup macos (magit +forge) :lang clojure emacs-lisp markdown org sh :config (default +bindings +smartparens))
        packages   (n/a)
        unpin      (n/a)
        elpa       (n/a)

Resolved now that #3414 is merged.


Move testing data for statistics query to staging server

We need some testing data for the statistics query.
What we need:

The loopdata tables for the entire last week of January (ending today the 30th)
Loopdata tables for the previous 6 fridays preceding today. (Total of 7 fridays worth of data)

We also need to pull data out of the large aggregated table, but let's do that as a second project as it's not going to be a straightforward table dump.

done, left the loopback tables .sql files in my public directory (/home/pgicking/Public) incase we have to redump.


Login page before enter into Admin Dashboard

Issue type
Unable to find lode for Login and Signup
I'm submitting a ...  (check one with "x")

[ ] bug report
[ ] feature request
[x] question about the decisions made in the repository

Issue description
I can see the login under Auth in the dashboard, but not able to find the code code Login and Signup components. When i search, it is showing the code in mode_modules. I don't think we update that code.
Current behavior:

I can see the login page in the admin dashboard, but not as home page before we access the admin dashboard.
Expected behavior:

I would like to load the Login and Signup page before we give an access to the Dashboard. Once user signed in successfully, then we should redirect to the Dashboard. I have followed many articles, but didn't find the exact solution.
Steps to reproduce:

Related code:

insert short code snippets here

Other information:
npm, node, OS, Browser
<!--
Node, npm: `node --version` and `npm --version`
OS: Windows (7/8/10). Linux (incl. distribution). macOS (El Capitan? Sierra?)
Browser: Chrome/Safari/Firefox/etc?
-->

NPM - 6.4.1
Node - v10.15.1
Chrome - 72.0.3626.121
Angular, Nebular
<!--
Check your `package-lock.json` or locate a `package.json` in the `node_modules` folder.
-->

ngx-admin - 2.0.0
Nebular - 3.1.0
Angular - 7.0.3

Hi @venukommu!
You can use auth guards to prevent accessing routes for unauthenticated users. See guide for details.

Contact me on Skype: lms_ict
I can fix it for you.

Thank you @Prefix1802 for your kind help. Actually i have moved to a different template since we were blocked and unable to fix this issue after long effort. But definitely i contact you when we get some time to look into it since we liked this template very much.

@venukommu Hi, I'm having the same issue, what template are you using now?

I am also getting the same issue? any updates?

@iecorp Hi, There are many free templates which you can use. I use one template from Creative Tim's templates.

Hello there, Can anyone help?
I am having the same issue. Will anyone be willing to help?

I am having the same issue. Is there anyone who can help?


Let the user decide how to close bounds in range_count

The implementation of range_count could benefit from letting the user decide how to close intervals:
def range_count(x: TIME_SERIES_T, lower: float, upper: float) -> INT_EXPR:
    """
    Computes values of input expression that is between lower (inclusive) and upper (exclusive).

    Parameters
    ----------
    x : pl.Expr | pl.Series
        Input time-series.
    lower : float
        The lower bound, inclusive
    upper : float
        The upper bound, exclusive

    Returns
    -------
    int | Expr
    """
    if upper < lower:
        raise ValueError("Upper must be greater than lower.")
    return x.is_between(lower_bound=lower, upper_bound=upper, closed="left").sum()

The new version would be:
def range_count(x: TIME_SERIES_T, lower: float, upper: float, closed: Literal[‘both’, ‘left’, ‘right’, ‘none’]) -> INT_EXPR:
    """
    Computes values of input expression that is between lower (inclusive) and upper (exclusive).

    Parameters
    ----------
    x : pl.Expr | pl.Series
        Input time-series.
    lower : float
        The lower bound, inclusive
    upper : float
        The upper bound, exclusive
    closed: Literal[‘both’, ‘left’, ‘right’, ‘none’]
        How to close the interval.

    Returns
    -------
    int | Expr
    """
    if upper < lower:
        raise ValueError("Upper must be greater than lower.")
    return x.is_between(lower_bound=lower, upper_bound=upper, closed=closed).sum()

If you think that's worth it I can submit a PR for this. Let me know

I added the closed option just like in Polars in the latest commit in your fix/clean-function branch.


Add limitations in terms of countries

If the goal is not to have an international list please discard this.
I noticed that some shops in the list do not deliver outside America, if the goal is to have a single list for the whole world, then it would be nice to add the information on limitations in terms of countries of delivery :-)

Yeah, I'm assuming US, roughly, sorry about that. #10 adds a EU source, and that's totally fine as far as I'm concerned.
Longer term, if I turn this into a real app, I'll have some sort of shipping info. But for now it's just a rough list so we can keep tagging stuff like #10 does.


Enable credo again

Quick question: why does credo not like the abuse of Elixir pipes?

Here is the explanation for it:
https://github.com/rrrene/credo/blob/master/lib/credo/check/refactor/pipe_chain_start.ex
We can disable that check instead. Please let me know which way to go.


KAFKA-4864 added correct zookeeper nodes for security migrator

@simplesteph : Thanks for the patch. LGTM