bin/zig is not created if the path-link directory is not in $PATH

$> zigup 0.11.0-dev.1393+38eebf3c4
install directory '/Users/pfg/zig'
mkdir '/Users/pfg/zig'
compiler '/Users/pfg/zig/0.11.0-dev.1393+38eebf3c4' already installed
error: path-link '/Users/pfg/bin/zig' is lower priority in PATH than '/usr/local/bin/zig'

$> zigup 0.11.0-dev.1393+38eebf3c4
install directory '/Users/pfg/zig'
mkdir '/Users/pfg/zig'
compiler '/Users/pfg/zig/0.11.0-dev.1393+38eebf3c4' already installed
error: the path link '/Users/pfg/bin/zig' is not in PATH

Expected:
~/bin/zig is still created or updated despite the warning
What happens:
~/bin/zig is not created, or if it exists it is not updated
I would expect path issues to be a warning rather than an error, something like:
$> zigup master
… downloading
Zig installed to ~/bin/zig
warning: ~/bin is not in your PATH.
or:
warning: There is already a zig compiler in your PATH with higher priority.

I've fixed zigup to update the path symlink even if it's not the highest priority in PATH.  I've still left it as an error for now though since the user is explicitly asking to set the default compiler and they can always skip that request by doing a "zigup fetch VERSION".

Thanks!


containerd-release moved, install.tools target broken

Description
In https://github.com/containerd/containerd/pull/2735, the containerd-release tool was move to a different repo (containerd/project).  The current build process for cri-o is now broken:
if [ ! -x "/opt/cri-o/_output/bin/containerd-release" ]; then \
        go get -u github.com/containerd/containerd/cmd/containerd-release; \
fi
package github.com/containerd/containerd/cmd/containerd-release: cannot find package "github.com/containerd/containerd/cmd/containerd-release" in any of:
        /usr/share/go/src/github.com/containerd/containerd/cmd/containerd-release (from $GOROOT)
        /opt/cri-o/_output/src/github.com/containerd/containerd/cmd/containerd-release (from $GOPATH)
Makefile:224: recipe for target '.install.release' failed
make: *** [.install.release] Error 1

Steps to reproduce the issue:

Checkout cri-o
Run make install.tools

Describe the results you received:
Unable to build install.tools target
Describe the results you expected:
Build install.tools target
Additional information you deem important (e.g. issue happens only occasionally):
I was able to fix this by:
go get -u github.com/containerd/project/cmd/release-tool
mv /root/go/bin/release-tool /opt/cri-o/_output/bin/containerd-release
Output of crio --version:
(paste your output here)

Additional environment details (AWS, VirtualBox, physical, etc.):
Debian 9

This is fixed with #1865. Closing. Feel free to reopen if you still see it. Thanks!


Could not find cursor theme Manjaro Linux

Manjaro Linux (4.11-ck) Godot 2.1.3
Issue description:
I'm using Godot from Steam. And this error comes out:
WARNING: initialize: Could not find cursor theme    At: platform/x11/os_x11.cpp:346.
Steps to reproduce:
abc@abc-pc ..am/steam/steamapps/common/Godot Engine % ./godot.x11.opt.tools.64
Is it an X error or some thing else, i'm not sure.
Any help would be appreciated...

Here are other reports of the same issue, seems to be related with some desktop environments https://godotengine.org/qa/14171/how-to-fix-cursor-theme-error-on-linux

@Supatier That's "just" a warning, not a real error. Most of the time you can safely ignore it.
Is there anything not working as expected related to the mouse cursor? e.g, does it have a different theme/not look how it's supposed to?

@Hinsbart Everything works fine, but what does the error mean?

@Supatier it means that we failed to get information about the cursor theme currently used by the X server and just use the default one. However, for the average user this information isn't useful at all (imo), especially since X11 is such a fragmented environment.. every window-manager seems to handle things a bit differently than the next. During my tests, I found that on some WMs this method will always fail.
That's why I recently removed the warning in the master branch.
TL;DR "Not much"


Line charts

First attempt at introducing line charts. Creating this draft PR for taking input and feedback.

Great work 👏
A couple of points to consider:
* Please run `./gradlew ktlintFormat` and format accordingly

* Would be nice to add optional or customisable `Legend` like in `Pie` or `Bars`

Will look into that! 👍

Great work 👏 🎉

Thanks! Lots of improvements we can do still, but the basics should be here now. I submitted a small update, after you approved, to allow for different axis label colors. Please have a quick look.
Let me know if there's anything more I need to do before this can be merged and a new release published.

Looks good, just needs a ktlintFormat run 👍

Looks good, just needs a ktlintFormat run 👍

Keep forgetting that. Done now 👍


Bitmask segmentations with per-channel obsTypes

User story
In certain cases, a segmentation bitmask will have different obsTypes in each channel. E.g., cells in channel 0, glomeruli in channel 1, tubules in channel 2, etc. which do not have a one-to-one mapping of observations across channels (unlike cells/nuclei/cell boundaries/nuclei boundaries in existing HuBMAP use cases).
Preferred solution
Allow specifying obsType on a per-channel basis for bitmask segmentations, and then rendering multiple segmentation layers that use different obsTypes.
Possible alternatives

Fixed by https://github.com/vitessce/vitessce/pull/1581


pr to wiki

i added some doc about vs development
the changes are on update_doc_1 topic branch of my forked wiki repo
you can see the changes on my published wiki fork

removed nunit from required software (is downloaded from nuget)
add visualfsharp logo to Home
add visual F# tools dev guide

btw, since this repo does not use collaborators, maybe make the public the wiki (removing collaborators only restriction) can help people to write docs?
The wiki is a git repo, it's easy to revert errors

Looks nice. I've pushed your changes to wiki on this repo.


GNU+Linux incorrectly referred to as "Linux"

GoAccess incorrectly refers to the GNU+Linux operating system as "Linux" in its user interfaces and source code.
GNU is an OS started in 1983 and Linux is a kernel started in 1991. The combination of these two components today forms the GNU+Linux operating system.
See also:
https://www.gnu.org/gnu/linux-and-gnu.html
https://www.gnu.org/gnu/gnu-linux-faq.html
https://www.gnu.org/gnu/why-gnu-linux.html

Please feel free to submit a PR with the relevant changes. Thanks

Merged. Thanks!

:)


Storage issue with multiple AuthenticationAdapter

Hello everyone,
I ran into an issue using multiple auth adapters and their $this->isSatisfied() function.
In File src/LmcUser/Authentication/Adapter/AbstractAdapter.php on line 24 the Storage is initialized using the current class name:
$this->setStorage(new Storage\Session(get_class($this)));
If I use multiple classes, e.g Db.php and a custom Ldap.php the class_name differs. If authentication is satisfied using Ldap (first adapter), Db should be skipped. But as the class_name differs a new storage is created an authentication fails.
My temporary workaround was to change this line to $this->setStorage(new Storage\Session('LmcUser')); and all AuthenticationAdapters are using the same storage.
This works for me, but is this the correct way ?
Best regards,
Alexander

Hello Alexander,
I would say you are right.
It's not a good idea that the Storage Session Name depends on the class.
This should be a unique name for the LmcUser Module.
I will look into it and make a PR.
Regards Alex

Hello Alexander,
I would say you are right.
It's not a good idea that the Storage Session Name depends on the class.
This should be a unique name for the LmcUser Module.
I will look into it and make a PR.
Regards Alex

Thank you for your reply and confirmation of my workaround.
I can create a PR too, but unfortunately I already made an open pull request for CSRF which is not yet resolved. So I'm not able to create a second PR

Thank you for your reply and confirmation of my workaround.
I can create a PR too, but unfortunately I already made an open pull request for CSRF which is not yet resolved. So I'm not able to create a second PR


PSS Restricted v2022 installation fails because of missing K8sPSSRunAsNonRoot CRD

I have tried installing "PSS Restricted v2022" bundle via Config Sync to an Anthos on Bare Metal cluster with

Template Library
Pod Security Standards Baseline v2022
Pod Security Standards Restricted v2022

bundles installed via Policy Controller in Cloud Console.
In Config Sync I see multiple error messages about missing CRDs:

When I'm trying to install "PSS Restricted v2022" bundle from this project directly using the installation command
kubectl apply -k https://github.com/GoogleCloudPlatform/gke-policy-library.git/anthos-bundles/pss-restricted-v2022

I get the following error during installation:
error: resource mapping not found for name: "pss-restricted-v2022-running-as-non-root" namespace: "" from "https://github.com/GoogleCloudPlatform/gke-policy-library.git/anthos-bundles/pss-restricted-v2022": no matches for kind "K8sPSSRunAsNonRoot" in version "constraints.gatekeeper.sh/v1beta1"
ensure CRDs are installed first

Where can I find the K8sPSSRunAsNonRoot constrainttemplate definition?

The K8sPSSRunAsNonRoot has been introduced in Policy Controller 1.17.3. Once I upgraded to that version the bundle was successfully installed. Closing the ticket.

thank you for documenting this! I ran into this today.


Exclude unused tasks in FlakyTestQuarantine

I noticed that there are a lot of unused tasks running in FlakyTestQuarantine: https://ge.gradle.org/s/gmi4nfvdah5g4/timeline?name=docs&page=12
This PR excludes them. The build scan after the change: https://ge.gradle.org/s/nleu6iqnnldno/timeline?name=docs

Your PR is queued. See the queue page for details.

I've triggered a build for you.


Enable GameMode on linux

What problem does this solve or what need does it fill?

GameMode is a daemon/lib combo for Linux that allows games to request a set of optimisations be temporarily applied to the host OS and/or a game process.
GameMode was designed primarily as a stop-gap solution to problems with the Intel and AMD CPU powersave or ondemand governors, but is now host to a range of optimisation features and configurations.

What solution would you like?
Use the com.feralinteractive.GameMode dbus interface (or org.freedesktop.portal.GameMode when running inside flatpak) to activate game mode by default when staring the game.
What alternative(s) have you considered?
Manually run the game using gamemoderun (or edit the launcher settings to use it) to enable game mode without changing the game. This won't help if the user doesn't know about the existence of GameMode in the first place (like me until today), but does have it installed. (I wouldn't be surprised if there is some gaming focuses linux distro that has it installed by default)
Additional context
https://github.com/FeralInteractive/gamemode

Taking a look at this, there's a client library available in C. It's basically
#define GAMEMODE_AUTO
#include "gamemode_client.h"

But it feels like it might be better to make a plugin for it, and consider including it by default?

I feel like this should be a plugin, because it only affects Linux and only a certain kind of app. As a plugin, it can also be developed now and independently :)

Windows had an equivalent, but it appears to be partially deprecrated? Worth digging into more, but further details probably belong in their own issue.


python3Packages.pvo: init at 0.1.0

Motivation for this change
Python module to interact with the PVOutput API
https://github.com/frenck/python-pvoutput
Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 151842 run on x86_64-linux 1

2 packages built:

python38Packages.pvo
python39Packages.pvo


Why don't we just set all the fences to dirty?

I noticed a lock before resetFenceDirty() in iHipWaitActiveStreams(https://github.com/ROCm/clr/commit/3c6505c2d5988050f8282f556ff654ab3b2e31a1). The question I have is why don't we just set all the fences to dirty after dispatchBarrierPacket(https://github.com/ROCm/clr/blob/b825decd422500b58e983f1a2f79c9c5f9df3cbd/rocclr/device/rocm/rocvirtual.cpp#L1053).
Would adding another lock and loop for this hurt performance?
Why can't other CacheState just set dirty like kCacheStateSystem?
Thanks and hope you answer!

Hi @Dustray. Apologies for the lack of response. Internal ticket has been created to assist with your question. Thanks!

Hi @Dustray, is this question still relevant? I'll reach out to our internal teams if it is, but our codebase has changed in the meanwhile and I don't see iHipWaitActiveStreams anymore. Let me know if this has just been refactored and your question is still relevant, or if you have any questions about the current code.

Ok I found that the locking judgment against isFenceDirty() has been removed in iHipWaitActiveStreams（now is Device::WaitActiveStreams()）.

Thanks for following up!


Resources are not being applied on Razor Page Input models

Hello,
I have an example where the Localizer is not affecting the ViewModel classes' properties' data annotation, it's a plain MS template project code:
public class ChangePasswordModel : PageModel
{
// ...
        [BindProperty]
        public InputModel Input { get; set; }

        public class InputModel
        {
            [Required]
            [DataType(DataType.Password)]
            [Display(Name = "Current password")]
            public string OldPassword { get; set; }

            [Required]
            [StringLength(100, ErrorMessage = "The {0} must be at least {2} and at max {1} characters long.", MinimumLength = 6)]
            [DataType(DataType.Password)]
            [Display(Name = "New password")]
            public string NewPassword { get; set; }

            [DataType(DataType.Password)]
            [Display(Name = "Confirm new password")]
            [Compare("NewPassword", ErrorMessage = "The new password and confirmation password do not match.")]
            public string ConfirmPassword { get; set; }
        }
//...

I have tried several versions to make it work:
Areas/Identity/Pages/Account/Manage/ChangePassword.ka-GE.resx
Areas/Identity/Pages/Account/Manage/ChangePassword.cshtml.cs.ka-GE.resx (this was read as CZ)
Areas/Identity/Pages/Account/Manage/ChangePasswordModel.ka-GE.resx
Maybe you could create a Razor pages example?
I know how to correct this issue, but with another approach, but it would be better if all localization tasks will be dealt with in same manner.
Thanks!
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d1833676-ff1b-c587-cc9c-81afffa8ef68
Version Independent ID: ad989395-3f49-4d55-521e-cf6fe56281ee
Content: Globalization and localization in ASP.NET Core
Content Source: aspnetcore/fundamentals/localization.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@DavidShukvani  looks like you scaffolded Identity and tried to localize it?
My guess is the fully qualified name is not correct.
cc @hishamco

Yes, exactly, but I tried dozens of combinations for matching, but no luck.

@DavidShukvani could please share a minimal repo to reproduce the issue?

@hishamco please see: https://github.com/DavidShukvani/IssuePresentationForLocalizer
I have setup the registration and login pages to localize Georgian (ka-GE) and English (en-GB).
Thank you!

After I saw your demo, your remind me with a weird bug that AspNetCore doesn't solve yet, but I already fixed this in OC and blogged about few years ago
Just avoid to use inner classes, so move your InputModel into LocalizerIssue.Areas.Identity.Pages.Account and it should work

Ok, thanks!

I would suggest you add an clause in article in this part.


Will this library work with this RFID RC522 module ?

Hello.
My RFID RC522 module looks like this:

The pinouts are labeled: NRST, GND, SWIM, VCC and GND, RX, TC, VCC
I cannot find any documenation, prodocol it uses, product name, etc, nothing :/

Does anyone know what interface is this ?
And does this work with this library ?
Could it write too or it is just for reading RF ?

Thank you.

I cannot find any documenation, prodocol it uses, product name, etc, nothing :/

That is the worst case. 💥
Your mentioned labels let me suggest i2c as supported protocol, which is not supported by this library. You can find libraries with i2c by github search or my untested fork, see #522.


Improve browser support

What?
This PR bumps pino to v5.12.3 to include browser fix from https://github.com/pinojs/pino/pull/612
Why?
It enables hapi-pino to be used within Hapi embedded in WebExtension context:  https://github.com/ipfs-shipyard/ipfs-companion/issues/664 https://github.com/ipfs-shipyard/ipfs-companion/issues/716

Coverage remained the same at 98.808% when pulling 3767ed6b67601831e176e084ed82ba4ed9f726e6 on lidel:feat/improve-browser-support into d50a86766932b0d7b5b168c963f4cc66d4be2aa7 on pinojs:master.

Coverage remained the same at 98.808% when pulling 3767ed6b67601831e176e084ed82ba4ed9f726e6 on lidel:feat/improve-browser-support into d50a86766932b0d7b5b168c963f4cc66d4be2aa7 on pinojs:master.

This should not be needed, updating your dependencies would get this anyway.


chore: updated dependencies and CRC to OpenShift Local

Coverage remained the same at 96.97% when pulling c1006855f8e733c8ccb53197dfd948e4af9250fc on pnaik1:updates into b379a61a04fdbddc9c6bb4c64a4572fc09393dca on nodeshift-starters:main.


bug: Explain opens up in main tab now not in sidebar (6.0.20) Also tabbing away blanks tab.

IDE Information
Version 6.0.20 Nightly
Platform: Windows (Rider 2024.1.4)
Bug Description
When issuing commands like "Explain" / "Code Smell",  the response now opens in a new tab in the main window,  previously it would open in the sidebar.
This seems "weird".   Also tabbing away from this tab and back again, the content will be empty.
It is uncertain if this is a new "feature" or if it is a bug, in which case one may expect the output to be in the sidebar for Cody.

Additional context
No response

The tab clearing on navigating away and coming back appears to have resolved itself after a restart of Rider IDE.
I can only assume moving the response into the main tab is a design choice, not a defect.
Closing issue.


Arkansas Onboarding - Validate Test Data

Acceptance Criteria

[ ] Created and sent data using fake data generator
[ ] Created and sent data using synthesized data generator
[ ] Confirmed with STLT that data passes validation

To Do

[ ] RS engineer: update this ticket with comments with every interaction with receiver
[ ] RS team member who originates this ticket: add any dependencies, as needed
[ ] RS team member who originates this ticket: complete the following when creating this Issue.

[ ] Pipelines: Onboarding & Operations (New)
[ ] Labels: "Onboarding-ops" and "Receiver" and state label
[ ] Epics: [select the epic for this API onboarding sender]

Reply from Eddie Butler (AR DOH) on test data.
"Hi all.
The message looks very sound, containing all the required fields as far as I can tell.  Would love to get some actual production data flowing into Staging so we can further evaluate.
Thanks,
Eddie"


fix: Add chromedriver version override to  JS-Functional-Tests-BrowserBot-yaml

Fixes #minor
Description
This adds a manual override that lets the build admin quickly adjust the chromedriver version when the build is broken.
Background
New Chrome browser versions are released about monthly. The latest chromedriver versions released on npmjs.com are often ahead of the browser version releases, and the relationship is erratic.
Our pipeline gets the 2nd-latest chromedriver version. That works most of the time, but still the pipeline occasionally breaks.
Specific Changes
Add pipeline var ChromeDriverVersionOverride. When set, it forces the chromedriver to a specific version.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 84.622%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-lg/src/templateExtensions.ts
1
84.48%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
19935

Relevant Lines:
22322

💛  - Coveralls


Implemented add/remove page buttons, also added page number display

Fixes #955
Changes made:

Added a state variable for pageCount
Implemented a button that increments pageCount by 1
Implemented a button that decrements pageCount by 1 which is disabled if pageCount!==1
Styled both buttons using reactstrap Button component
Repeat the Output component pageCount number of times
Passed a pageNo prop to the Output component, to display page number along with word count
Styled page number like word count

Screenshot:

While downloading/exporting, except the first page the remaining are not downloaded

While downloading/exporting, except the first page the remaining are not downloaded

Should I try and fix the download in another PR?


Injected identity

Description of your changes

Adding some documentation and an example regarding how to setup the provider using AWS IRSA.
Fix a bug where InjectedIdentity was wrongly advertised as a valid credentials source for the provider config.
Added an extra commit with some linter fixes to the README, I can revert it if you don't agree with my linter setup...

Fixes #70
I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
I have manually tested the setup in a real EKS cluster.

These were automatically done by my IDE. I'll revert them.

Done, I've reverted the commit with the linter changes, should I open a separate PR for that?


1.2.11 version 1

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Hi @dominick01234,
There are a few problems with your submission. Fortunately, ChiaNetwork.GUIforChiaBlockchain is already in the repo! You could just copy ChiaNetwork\GUIforChiaBlockchain\1.2.11 to CactusNetwork\GUIforCactusBlockchain\1.2.11 and change the details in the manifests. Good luck 😄 .


dbt docs generate adding both source and model column names (source and model share name)

Describe the bug
When running dbt docs generate and dbt docs serve the documentation shows the columns from both my source data and model data in the model documentation and in source documentation. I'm guessing this is because they share a name case_info and dbt doesn't know to keep them separate when building the catalog.
Snippet from sources.yml
    database: RAW_DEV
    tables:
      - name: area_info
        identifier: PSO_RESULT_AREA_INFO
      - name: case_info
        identifier: PSO_RESULT_CASE_INFO

I have a models/pso_results/case_info.sql file
{{
  config(
    alias="PSO_RESULT_CASE_INFO",
  )
}}

with clean_case_info as (
    select "//CaseID" as CASE_ID,
           "Name" as "NAME",
           "TaskID" as TASK_ID,
           ISO as ISO,
           "LoadDateTime" as LOADED_AT
from {{ source('pso_results', 'case_info') }}
)

select * from clean_case_info

when I serve docs I see this

Expected behavior
I expected dbt docs to understand and keep separate a source and a model regardless if the name matched.
System information
The output of dbt --version:
installed version: 0.14.3
latest version: 0.14.3

The operating system you're using:
Mac OSX 10.15
The output of python --version:
Python 3.7.3

Hey @jkornblum - we had an issue over here https://github.com/fishtown-analytics/dbt/issues/1708 which was fixed in https://github.com/fishtown-analytics/dbt/pull/1774. Going to close this one as a duplicate - the fix will be out in 0.15.0 later this month!


Save password parameter default values to user secrets

Initial implementation for feedback.
Fixes #1151
Microsoft Reviewers: Open in CodeFlow

This feels like an opportunity to design a better API for parameter defaults.

What did you have in mind?

Couple of things playing around with this locally.

Seems to have broken pgadmin ... is it working for you?
CatalogDbApp is crashing with an error saying it can't authenticate.

This feels like an opportunity to design a better API for parameter defaults.

Are you talking about supporting binding to KeyVault etc?

@DamianEdwards I'm pretty happy with this. I made a minor change so that the secrets file is formatted when we save it.

@eerhardt can you review please


Add auto merge

@conda-forge-admin, please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/dtool_lookup_server_plugin_scaffolding-feedstock#2.


LibreSSL can not find a valid cert file

Bug Report
LibreSSL always read an invalid cert file, like /home/xxx/tikv/target/debug/build/openssl-sys-4e08c83cb3f270e4/out/openssl-build/install/ssl/cert.pem. For now the bug does not affect tikv's TLS feature, but it will fails requests from tikv to other HTTPS websites and raise the error:
Io(Custom { kind: Other, error: "failed to create bucket an error occurred trying to connect: error:14007086:SSL routines:CONNECT_CR_CERT:certificate verify failed:/rust/git/checkouts/libressl-src-df2f639f2f727162/1e3a34a/libressl/ssl/ssl_clnt.c:1121: (unable to get local issuer certificate)" })

OpenSSL solves this by reading an env var SSL_CERT_FILE @ https://github.com/openssl/openssl/blob/bcc0025d0b5d231ee3662cf7f3c522ba43079858/crypto/x509/by_file.c#L49
And golang solves this by hardcoding https://github.com/coreos/bugs/issues/2262#issuecomment-346161278

Fixed via #5384 .


mtree_set preserving the old value in the stack

Describe your changes
Closes #646 .
~This didn't follow the exact data layout requested on the issue, the change copies the old copy of the element after the previous data. The stack will look like this after the op:~
[R', V, V_old ...]

edit: updated the code to use the same format as defined in the issue.

We probably need to update mtree_cwm instruction as well. The new format for this instruction could be: [d, i, R, V', ...] -> [R', V, R, ...]. But I also haven't thought about it much.

I created an issue for that https://github.com/0xPolygonMiden/miden-vm/issues/653

Btw, we should also make sure the cycle count is updated in the docs


Fix static Analysis errors introduced in last commit for ssh keyboard interactive option

Pull Request Functional Test Report for #1743 / d4c81b69085f0abec6f7b359bb91dfa485ca53f8
No tests identified for validation.
Help


Add real framerate calculation in debug

When using the --debug flag, vid2led will now report the actual average frame rate of the video once the video is done.

This is causing extreme slowdown when not simulated, especially for longer videos.

Rolling frame rate calculation is better than the previous solution, but still slows down videos significantly. This solution slows down all videos uniformly, while the previous solution effected longer videos the most.

Closing, will not include this functionality in 0.1.4.


allow npm package to work in nodejs

pieced together more info of how axios works with webpack, node, and web from several sources including these:
https://github.com/axios/axios/issues/456
https://lmiller1990.github.io/electic/posts/server_side_hydration_with_vue_and_webpack.html
Tested local package with nodejs and react dummy projects and openlaw-backend and all seem to be working fine. Changes in this pr also include some cleanup.
If this looks good and merged into master, I can patch update to 0.1.4 and publish again.

could you just update the openlaw-core dependency to 0.1.3?


9020 and 9040 errors

As in #78 and #80
I can repro when Outlook is Need password state.  Consider suppressing:
ERROR: Async Response
{
"status": "failed",
"error": {
"name": "GenericResponseError",
"message": "The operation failed.",
"code": 9020
}
} : The operation failed.
callbackEws()@https://localhost:44358/Scripts/2861fbc/GetHeadersEWS.min.js:79:21
and
ERROR: Unable to obtain callback token.
Fallback to EWS.
{
"status": "failed",
"error": {
"name": "HTTPRequestFailure",
"message": "The request has failed. Please look at the diagnostics object for the HTTP error code.",
"code": 9040
},
"diagnostics": {
"ClientRequestId": "{7563B2F6-9B43-4341-83DF-FB5969C234FF}",
"HTTPCode": "4294967295"
}
} : The request has failed. Please look at the diagnostics object for the HTTP error code.
Anonymous function()@https://localhost:44358/Scripts/2861fbc/GetHeadersRest.min.js:37:21

haven't seen this in ages


Typescript error vscode FormKitComponent

Reproduction
https://formkit.link/e6e026b413c717931d3dc9524a174080
Describe the bug
Hi,
Look at the screenshot I don't know why there's this typescript error on vscode ?

Environment
• OS: MacOS Ventura
• Browser chrome

Gonna need a bit more details on this one. Can you provide your implementation of the component?

It's not the same error, but I am also getting a TS error relating to the Multi-step plugin for VS Code. My #stepNext override is returning a TS error in the most recent update.

This is the exact same example from https://formkit.com/plugins/multi-step

There is an updated version that fixes errors with typings for the multi-step that came with 0.18.

Gonna need a bit more details on this one. Can you provide your implementation of the component?

@justin-schroeder yes paste this : https://formkit.link/e6e026b413c717931d3dc9524a174080 into vscode :)

@justin-schroeder yes paste this : https://formkit.link/e6e026b413c717931d3dc9524a174080 into vscode :)

I meant things like — your version of formkit, your version of formkit pro, your version of vue etc. This error should not happen if you have 0.114.x and beta 0.18.x installed.


Restructure feedback metadata for VPN

Task/Issue URL: https://app.asana.com/0/488551667048375/1208627799127675/f
Description
Add auto exclude broken apps status in VPN feedback metadata + restructure some other info in VPN feedback
Steps to test this PR
QA-optional

#5373  👈 (View in Graphite)
develop

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


Update nivetha.md

Second PR

Ask in Twitter group by giving link to this pr for review and approval.

Ask in Twitter group by giving link to this pr for review and approval.

Okay 👍🏼


[AutoPR postgresql/resource-manager] correct tier enum for postgres

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/2629

(message created by the CI based on PR content)

Codecov Report

Merging #2143 into restapi_auto_postgresql/resource-manager will not change coverage.
The diff coverage is n/a.

@@                            Coverage Diff                            @@
##           restapi_auto_postgresql/resource-manager    #2143   +/-   ##
=========================================================================
  Coverage                                     53.55%   53.55%
=========================================================================
  Files                                          5135     5135
  Lines                                        123513   123513
=========================================================================
  Hits                                          66144    66144
  Misses                                        57369    57369

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 dc6ea95...0fd843a. Read the comment docs.


refactor(wash-cli): deprecate wash app list in favor of wash app get

Feature or Problem

Merging wash app list and wash app get commands.
I do need one clarification: we probably want to keep the periodically watched part of the list to the wash app get without a name, am I correct? If yes, I'll re-add it, and will write some tests. I just tested locally, but would bring more confidence if those would be automated.
Related Issues

fixes: #2884
Release Information

Consumer Impact

Should reduce the number commands for the same thing.
Testing

Unit Test(s)

Acceptance or Integration

Manual Verification

1, start a host
wash up -d

2, create two applications (e.g.:)
wash new component hello --template-name hello-world-rust
wash new component hello2 --template-name hello-world-tinygo

build and deploy the apps (some renaming in the wadm.yamls can make things easier)

cd hello && wash build && wash app deploy wadm.yaml && cd ..

cd hello2 && wash build && wash app deploy wadm.yaml && cd ..

Validate the help output of clap

cargo run -- app --help
cargo run -- app get --help

Validate the output of the command (rawstring[yaml] and json, yq|jq helps to parse/highlight yaml)

cargo run -- app get | yq
cargo run -- app get -o json | jq

cargo run -- app get {{Appname.From.Step.2}} | yq
cargo run -- app get {{Appname.From.Step.2}} -o | jq

tl;dr
a, move the current wash app list functionality under the wash app get when no name
b, remove the wash app list and wash app get should return the yaml/json
c, something else
@vados-cosmonic
no, I was about to check the tests and expected them to fail due to the removal of the other command. So I'll look at the test failures, but I still do not know if we just wanted to yoink all the yaml/json to the screen. Or we want to move the current wash app list render? Putting back to draft meanwhile
@brooksmtownsend you are the creator of the ticket can you (or anyone else) help me out?

no, I was about to check the tests and expected them to fail due to the removal of the other command. So I'll look at the test failures, but I still do not know if we just wanted to yoink all the yaml/json to the screen. Or we want to move the current wash app list render? Putting back to draft meanwhile

I think moving the functionality around is definitely a good start -- if it just did exactly the same thing as before that'd be fine for a start I think!

Rebasing to get the ocistep fix in here so you can stop seeing red in the CI runs! :)
Thanks again for working on this

Yes, I was kinda lost why are the golang tests failing if I change the command line parameters.
I will re-add then the wash app list and and rebase. (will come back to this tomorrow) Thanks all for the feedback.


Add Exponential Retry Mechanism with Idempotency Headers

In order to enhance the resilience and reliability of our SDK, we would like to introduce an Exponential Retry mechanism for retrying failed requests. Additionally, to ensure the idempotent processing of requests, it's vital to incorporate support for providing an Idempotency Key as per the draft specified in the HTTP Idempotency Key Header Field.
The key requirements for this implementation include:

Exponential Retry Mechanism:

The SDK should retry failed requests following an exponential backoff strategy to minimize the contention and impact on the systems involved.
The SDK should ensure that the retry mechanism is configurable (e.g., max retries, initial delay, maximum delay).

Idempotency Key Provisioning:

The SDK should allow for either automatic or manual provisioning of an Idempotency Key for each request.
The Idempotency Key should conform to either CUID, ULID, or UUID formats as specified in the draft.
The Idempotency Key should be included in the HTTP Header as Idempotency-Key and following the standards outlined in the draft.

Configuration and Documentation:

The SDK should provide configuration options for enabling/disabling the Exponential Retry mechanism and Idempotency Key provisioning.
Comprehensive documentation should be provided explaining the configuration options, operational behavior, and the benefits of using the Exponential Retry mechanism along with Idempotency Keys.

Acceptance Criteria:

[ ] Implementation of the Exponential Retry mechanism with configurable parameters.
[ ] Provisioning of Idempotency Keys, either automatically or manually, conforming to specified formats (CUID, ULID, or UUID).
[ ] Adequate unit and integration testing to ensure the robustness and reliability of the implemented features.
[ ] Comprehensive documentation on the usage and configuration of the Exponential Retry mechanism and Idempotency Key provisioning.
[ ] Adherence to the specifications outlined in the HTTP Idempotency Key Header Field draft.

Update: You can reference the go-lang library to keep the method signature and configuration the same.
https://github.com/novuhq/go-novu/pull/62
Please refer to the draft for further details on the HTTP Idempotency Key Header Field and ensure adherence to the specified standards while implementing this feature in the SDK.

@Cliftonz I'm working on this.

Please, did you ask to be assigned to this?
On Thu, Oct 19, 2023 at 7:32 AM Anjorin Damilare @.***>
wrote:

@Cliftonz https://github.com/Cliftonz I'm working on this.
—
Reply to this email directly, view it on GitHub
https://github.com/novuhq/novu-php/issues/57#issuecomment-1770149557,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAWPNUM32BDIKSZAMYPKX4LYADCRNAVCNFSM6AAAAAA5WJBI6OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZQGE2DSNJVG4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I can see that you have a draft PR up @dammy001. So I'll assign you to the issue.
For next time, please ask to be assigned so that multiple folks are not working on the same thing at the same time.

I can see that you have a draft PR up @dammy001. So I'll assign you to the issue.
For next time, please ask to be assigned so that multiple folks are not working on the same thing at the same time.

@unicodeveloper Yeah, thanks.


Missing information in inbound autodiscover rule requirements

The "Additional endpoints not included in the Office 365 IP Address and URL Web service" page lists in row 10 the following information:

In my view the wildcard DNS names cannot be utilized to filter inbound traffic on an on-premise infrastructure.
Without exact DNS names it does not make sense to have wildcard DNS entries (*.acompli.net, *.outlookmobile.com, *.outlookmobile.us) listed as the source.
Asks:

either publish the source IP ranges associated with the these URLs
or publish the exact DNS names that translate to the IP addresses of the service

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

ID: 5de1e69c-aee8-2cd4-3f69-5288253d9b2a
Version Independent ID: 3bcee52a-c7be-6ce3-05a9-2409ec30661c
Content: Additional endpoints not included in the Office 365 IP Address and URL Web service - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/additional-office365-ip-addresses-and-urls.md
Service: o365-administration
GitHub Login: @kelleyvice-msft
Microsoft Alias: kvice

@kelleyvice-msft  Please add your comment and help us in resolving this issue. Thanks.

Looking into this.

I've requested information from the product group about this, and will update when I get a response.

I have updated this information in the article, and also removed the obsolete domain *.acompli.net.


Multi column ordering does not work?

When I shift click columns on the samples they do not multi-order.

When I inspect the queryable, it looks okay:
{SELECT
    [Extent1].[ID] AS [ID],
    [Extent1].[Name] AS [Name],
    [Extent1].[Position] AS [Position],
    [Extent1].[Extn] AS [Extn],
    [Extent1].[StartDate] AS [StartDate],
    [Extent1].[Salary] AS [Salary],
    [Extent1].[OfficeID] AS [OfficeID],
    [Extent2].[ID] AS [ID1],
    [Extent2].[Name] AS [Name1],
    [Extent2].[AddressID] AS [AddressID],
    [Extent3].[ID] AS [ID2],
    [Extent3].[Street] AS [Street],
    [Extent3].[Building] AS [Building]
    FROM   [People] AS [Extent1]
    INNER JOIN [Offices] AS [Extent2] ON [Extent1].[OfficeID] = [Extent2].[ID]
    INNER JOIN [Addresses] AS [Extent3] ON [Extent2].[AddressID] = [Extent3].[ID]
    ORDER BY [Extent1].[Name] ASC, [Extent3].[Street] DESC, [Extent1].[Position] DESC}

Nevermind, looks like its working, I couldn't see enough of the data to tell...


Angular distribution files not found

🐞 Bug report
Description
Get an error while trying to generate static build with Scully after successful ng build.
🔬 Minimal Reproduction
ng build && npm run scully
💻Your Environment
macOS 10.15.5
Angular Version: 10.0.3

Angular CLI: 10.0.1
Node: 10.18.0
OS: darwin x64

Angular: 10.0.3
... animations, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router
Ivy Workspace: Yes

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.1000.1
@angular-devkit/build-angular     0.1000.1
@angular-devkit/build-optimizer   0.1000.1
@angular-devkit/build-webpack     0.1000.1
@angular-devkit/core              10.0.1
@angular-devkit/schematics        10.0.1
@angular/cdk                      10.0.1
@angular/cli                      10.0.1
@angular/material                 10.0.1
@ngtools/webpack                  10.0.1
@schematics/angular               10.0.1
@schematics/update                0.1000.1
rxjs                              6.5.5
typescript                        3.9.6
webpack                           4.43.0

Scully Version:

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
🔥 Exception or Error

Angular distribution files not found, run "ng build" first

Seems like there was something wrong with my project (or angular cli version). Created a new project and it worked.


Sublime Text 3 plugin usage

Hi,
I'm on a Mac and have no idea how to use this plugin, tried to copy the subl3 folder to ~/Library/Application Support/Sublime Text 3/Packages , restart ST3, got a reloading plugin gocode.gocode message, but it doesn't look like working properly, There's no command palette, no auto popup after inputting a ., and triggering it by hotkey the popup window shows lots of messy stuffs.

console log has error:
Traceback (most recent call last):
  File "/Applications/Sublime Text.app/Contents/MacOS/sublime_plugin.py", line 591, in on_query_completions
    res = callback.on_query_completions(v, prefix, locations)
  File "/Users/yee/Library/Application Support/Sublime Text 3/Packages/gocode/gocode.py", line 132, in on_query_completions
    stdin=subprocess.PIPE, stdout=subprocess.PIPE)
  File "./python3.3/subprocess.py", line 819, in __init__
  File "./python3.3/subprocess.py", line 1448, in _execute_child
FileNotFoundError: [Errno 2] No such file or directory: 'gocode'
using gpu buffer for window
Unable to open /Users/yee/Library/Application Support/Sublime Text 3/Packages/Default/Preferences.sublime-settings
using gpu buffer for window
Unable to open /Users/yee/Library/Application Support/Sublime Text 3/Packages/Default/Default (OSX).sublime-keymap
Traceback (most recent call last):
  File "/Applications/Sublime Text.app/Contents/MacOS/sublime_plugin.py", line 591, in on_query_completions
    res = callback.on_query_completions(v, prefix, locations)
  File "/Users/yee/Library/Application Support/Sublime Text 3/Packages/gocode/gocode.py", line 132, in on_query_completions
    stdin=subprocess.PIPE, stdout=subprocess.PIPE)
  File "./python3.3/subprocess.py", line 819, in __init__
  File "./python3.3/subprocess.py", line 1448, in _execute_child
FileNotFoundError: [Errno 2] No such file or directory: 'gocode'

I've already go get gocode:
which gocode
/Users/yee/dev/go/bin/gocode

So I wonder how to make this plugin working, should I start gocode process by hand first? Thanks in advance.

If "which" shows the gocode it only means there are correct enviornment variables in your shell. But you also need to add them to your text editor. Google it. Not sure if that's how you do it: https://packagecontrol.io/packages/Environment Settings
On my machine I set it for the whole system. X11 is started by shell and shell has the right env vars set. Don't know how to do it on mac.

@nsf Thanks you for responding, I've modified the gocode.py, reading gocode path via sublime-config and it's working now.

Hi, could you please tell me how you modified the gocode.py? I have the same problem. @jostyee

@wsszh https://github.com/jostyee/gocode-subl3


Feature(Schedule): Metrics, Fees, Resource Usage + Global Property

Summary of the change:

Implementation for hapi-fees usage calculation of Create, Sign, Delete and GetInfo operations
Implementation for node resource usage calculation for Create, Sign, Delete and GetInfo operations
Added new GlobalDynamicProperty for ledger.schedule.txExpiryTimeSecs -> the time for which Scheduled TX expire

External impacts:
None.
Applicable documentation

[ ] Javadoc
[ ] HAPI protobuf comments
[ ] README

Todo: verify the code coverage

Todo: verify the code coverage


Plot two axes for same chart

Hi! New user here, thanks for maintaining this package.
Is your feature request related to a problem? Please describe.
I would like to plot two axes for the same chart. One represents the price, and the other is some non-linear (but still monotically increasing) function of price. Some metric of price that is useful to our customers. An example in the chartjs library is here. I'm attaching a screenshot so its clear what the feature is.

Describe the solution you'd like
Not sure what the ideal way to go about this is, but maybe a way to set a "mirror" arg and a PriceFormatterFn?
Maybe something like
chart.priceScale('left'}.applyOptions({
    mirror: 'right',
    format: MyPriceFormatter
})

Looking through the source, it's hard to tell how difficult this is to implement, and what other quirks may arise but I have some progress going on a fork. Let me know if this is something you guys are interested in, I could make a PR if I figure this out.
As far as I can tell this isn't possible with a plugin? A plugin does not allow you to mess with the Axes from what I can see.

Also note in the example I provided the ticks are different. For my use case they don't need to be, and it removes a lot of the complexity.

I think this example is what you are looking for: Two Price Scales | Lightweight Charts

A plugin does not allow you to mess with the Axes from what I can see.

You can use a plugin to implement a completely new axis. As you can see on this simple example, it is possible to draw anything you want, however re-implementing a price scale in a new plugin wouldn't be the quickest task.
There is also this somewhat similar plugin example of how to create a price scale which appears above the chart: Lightweight Charts - Overlay Price Scale Plugin Example

Thanks for the response. It is nice to know I was wrong about not being able to use a plugin to implement a new axis.
The two price scales examples isn't exactly what I'm looking for because each dataset is only attached to one price scale. I briefly thought about making two LineSeries with the same data but figured that wouldn't work too well for other reasons. Let me know if I'm misunderstanding that about price scale. Otherwise, I can close and make this as a plugin.

Ahh looking at it it really may be easier to just continue working on a fork than implementing another price scale. The difficulty is that I don't know where the PriceMarks are on the right side, and it would be difficult to coordinate between both scales. Further I do want them to look similar so it would essentially be me copying as much as I can from PriceAxisWidget

Unfortunately I do not have a screenshot but I can describe my use case. The screenshot above, albeit kind of silly
I am plotting gambling odds across different sportsbooks to see if one can find an edge by placing bets on different books. Each sportsbook is a line series. So we are not exactly working with financial data.
Idk if you're familiar, but american odds can look like +400, -400. A probability value can be associated with these odds. Here is a table of some values.

American Odds
Implied Probability (%)

-1000
90.91

-500
83.33

-200
66.67

+100
50.00

+200
33.33

+500
16.67

+1000
9.09

You can see the scale is nonlinear, and the american odds explode at high/low values of implied probability. Implied probability is really what should be charted, as it corresponds linearly with a gamblers expected value and the values near 1 and 0 do not dwarf all the values in the middle. Two books having a discrepancy of -4000 vs -8000 is not that large; -200 to -600 is quite a massive one though. Charting with american odds on the Y axis just doesn't make much sense.
However sports gamblers who are our end users are quite used to the american odds, and it is important to display the american odds. What we currently do in my prototype is chart implied probability and use a formatter to convert to american odds. But ideally I would want to show both values, with the probability using the other axis. This is the best way to analytically process a discrepancy across books without the user having to do any quick math. It would be the same data, same tick mark locations, just a different formatter.
And yes, it would be easier to use Chartjs for this use case, but our team is going to be making lots of time series charts and we rather like the out of the box features and the UI of this library. Maintaining a fork is indeed a pain which is why I would like to contribute if there is interest in adding this feature. However I understand our use case is nonstandard although I can imagine there is use for showing two metrics of the same data in financial charting as well.

If I understand your goal correctly, you can achieve it by using two different line series and two different price scales.
Plot the exact same data for both line series and assign one series to the left price scale and the other to the right price scale. This will make the lines overlap and appear as a single line. Then, apply a different price formatter to one of the scales to display the numbers differently. Finally, disable the user's ability to manually scale or shift the price scale, ensuring that the lines always stay on top of each other. The resulting chart will resemble this:
https://codesandbox.io/p/sandbox/lwc-mirror-price-scales-js5lxg

Sorry for the late response. That is what I was experimenting with but I am not a big fan of disabling scaling/shifting. I was able to get decent results on a fork. Let me know if this could be something you're interested in integrating and I can clean it up and open a PR.

If you've got a working fork then we would be interested in having a look. 😊
One of our main concerns with new features, or improvements, is the effect on the bundle size of the library. If the change in bundle size is minimal then it is something that could be merged into the library.
You can check the size with the following commands:
npm run clean

npm run build:prod

npm run size-limit

sweet! thanks for your quick responses, and you may see a PR from me soon.


Release 1.1.0

Add the "Not working on Windows operating system" notice
Rename TimeoutException to TimeoutError

Coverage decreased (-3.03%) to 96.97% when pulling b171e4bc7d61ddede875769011fc64b42eb90709 on develop into 04abc230ae279d52cd9008d825369dbd43ffd264 on master.

Coverage remained the same at 100.0% when pulling 16909daf30ea1592bf79b65769a4002eca12bf68 on develop into 04abc230ae279d52cd9008d825369dbd43ffd264 on master.

Coverage remained the same at 100.0% when pulling 16909daf30ea1592bf79b65769a4002eca12bf68 on develop into 04abc230ae279d52cd9008d825369dbd43ffd264 on master.

Coverage remained the same at 100.0% when pulling 281949ed9791704bbd8747817993abcd8f253aa4 on develop into 04abc230ae279d52cd9008d825369dbd43ffd264 on master.


Submit Issue to Get Orangueen Added to QOwnNotes

https://github.com/pbek/QOwnNotes/issues/2405

https://github.com/pbek/QOwnNotes/issues/2405 submitted.


Expose buffers in views

When reading the buffers to other columnar formats, an iterator of options is quite inefficient. Since this data is already initialized, we can allow the users to access it and thus can expose it in the public.

Thanks! Agreed this is a valuable feature.

Since this data is already initialized, ...

See #161: We can not make a statement about the contents of the NULL values in the buffer. Nor can we make any claim about the contents at the end for the parts of the buffer, which are unused.
IMHO I still feel this interface should go behind an unsafe guard. Safety can usually achieved, by exposing a less powerful, but more specify interface for the usecase in question. What is it concretly you need the raw access to the buffers for? My guess it is for some kind of copying / aggregating? Maybe there is an even better feature lurking in there, which we could do instead?
Some technicalities

The number of valid rows should be taken into account, then exposing the slices.
I'd be fine with breaking changes and provide the additional abstraction of a TextColumnView of which either operations on the whole buffer can be performed, or an iterator obtained.

The next two points I should put into the Contributing.md at some point.

This crate tries to provide a test written from the perspective of the library user for each feature. Optimally that test demonstrates the usecase.
Comments on functions, should not only describe that the function does, but more importantly even, give the user a hint of why he/she may want to call it.

I know, quite the high bar to pass, but it is the only way I can maintain this thing, with my limitted time.

Pull Request Test Coverage Report for Build 52c04ebb35355ca2f83430348b87d340cb36207c-PR-162

0 of 12 (0.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 83.908%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

odbc-api/src/buffers/bin_column.rs
0
6
0.0%

odbc-api/src/buffers/column_with_indicator.rs
0
6
0.0%

Totals

Change from base Build a7462cdb02fae7613c680f4d8b0b6296175858b7:
-0.2%

Covered Lines:
3624

Relevant Lines:
4319

💛  - Coveralls

When reading data from any in-memory format, the details of how the data is layed out in memory are crucial. The design decision of using an iterator assumes that the consumer wants to consume them from an iterator.
In particular for arrow, the memory arrangement of &[T] (where some values are random) is exactly the arrow format, so in this case we only need to deserialize the indicators:
fn deserialize_bitmap(values: &[isize]) -> Option<Bitmap> {
    MutableBitmap::from_trusted_len_iter(values.iter().map(|x| *x != -1)).into()
}

fn deserialize_p_optional<T: NativeType>(
    data_type: DataType,
    values: &[T],
    indicators: &[isize],
) -> PrimitiveArray<T> {
    let validity = deserialize_bitmap(indicators);
    PrimitiveArray::from_data(data_type, values.to_vec().into(), validity)
}

Compared to iterating over each item individually, this has a different scale factor in O(N), since it only requires a memcopy of the values a "packing" operation of the indicators (vs an expensive check on every item).
An equivalent situation happens in the binary/utf8 arrays, where we can first deserialize the offsets and validity, then the values based on the offsets, without performing bound checks over the values (since we can validate the offsets as part of the first iteration).
Also, note that because the iterators are not marked as TrustedLen (still unavailable in stable rust), std::Vec can't specialize and must check that the size_hint on the hot loop of the iteration is still consistent with capacity.
I will improve the docs and add a test. I would advise against using unsafe here, but let's discuss that at #161

IMHO what you have written here would make a good documentation for the usecase of these functions!

IMHO what you have written here would make a good documentation for the usecase of these functions!

cool. I've added documentation and some tests (I am still relying on CI to run tests, so sorry for the spam on the CI).

I am still relying on CI to run tests, so sorry for the spam on the CI

I was hoping to achieve a local development experience like this:

Open in vscode
Click reopen in Container

Tests and everything works
Your situation is slightly more complicated, because you work remote, but you can try running the topl level docker-compose.yml and connect vscode the dev container, after.

Ok, I was able to run this locally, but it passed on my machine. The error in CI is

State: 40001, Native error: 1205, Message: [Microsoft][ODBC Driver 17 for SQL Server][SQL Server]Transaction (Process ID 61) was deadlocked on lock resources with another process and has been chosen as the deadlock victim. Rerun the transaction.

which I am not sure is related to a change in this PR?

@jorgecarleitao No, it has not. It is very likely related to the fact, that the test which verifies that the api can be used to list all tables, also locks all tables. It seems the Microsoft driver does checks lock hierarchies (kudos). So long story short: I think the tests due to their parallel nature do not always lock the table in the same order. Since the tests share the same ODBC environment, the MSSQL driver probably assumes there might be a deadlock in my application (it can't know that tests are not waiting on each other). Also the the list tabels test probably is blocked for a while (since it needs all the tables), which probably only adds to the suspicion the driver has. I'm still thinking about how to handle this flap. Better to deactivate for now.

@jorgecarleitao Please don't invest too much time in this PR, I've some breaking changes in mind. I would like to discuss them with you on sunday.

@pacman82 , would it be possible to also expose the raw slices in the NullableSliceMut? We also need those to set slices instead of via an iterator.

@jorgecarleitao I don't see why not. Since all values in value buffers are POD, I'd even say exposing the value buffer is safe. Not sure about the indicator buffer, though. Would you need direct access to both? Or is setting NULL / not NULL, via random access enough?

Closing in favor of the other PRs. Thanks a lot for the guidance, @pacman82 !


<EMAIL_ADDRESS>hash check failed

Console Output
Installing 'teamviewer' (15.27.3) [64bit]
TeamViewerPortable.zip (40.5 MB) [============================================================================] 100%
Checking hash of TeamViewerPortable.zip ... ERROR Hash check failed!
App:         extras/teamviewer
URL:         https://download.teamviewer.com/download/version_15x/TeamViewerPortable.zip
First bytes: 50 4B 03 04 0A 00 02 00
Expected:    407d7dcfa113860759b08a774b0ee8275c38d5bf0694da6a042aa0e9954eb993
Actual:      a63cd4bb8ac75957b58b55d90ec1ba607149d15a941b9d08ab9fd2143db7d85d

Scoop Configuration

{
  "lastupdate": "2022-05-11T13:32:56.2324673-03:00",
  "SCOOP_REPO": "https://github.com/ScoopInstaller/Scoop",
  "SCOOP_BRANCH": "master"
}

#8201 doesn't work.

You reported version 15.27.3, but the latest available version is 15.29.4.
Run scoop update; scoop update teamviewer --force

This is not a valid response, as I stated on the original comment.

@rashil2000 Could you please help me? Thanks.


remove babel-preset-env

Follow up to #1107, it seems babel-preset-env was conflicting with other babel plugins - in this case, spread operators weren't being transpiled.

Deploy preview for netlify-cms-www ready!
Built with commit 387da122a83084ba2e97d81d43ee843329f4d14a
https://deploy-preview-1127--netlify-cms-www.netlify.com

Deploy preview for cms-demo ready!
Built with commit 387da122a83084ba2e97d81d43ee843329f4d14a
https://deploy-preview-1127--cms-demo.netlify.com


[Bug] No exported member named 'TransactionService'.

Describe the bug
First off, thanks for putting this out there. The SagePay/Opayo encryption spec do not seem to be working for me with crypto-js.  This is just what I'm looking for.
I've installed this package within my Nuxt3 project, imported the example code and I receive the error.
'"opayo-form-integration-js"' has no exported member named 'TransactionService'. Did you mean 'ITransactionService'?
What is the expected behavior?
No response
Other information
EDIT - Closing this as the source files in this package helped me successfully use crypto-js. Very helpful, thanks.

I will still have a look into this for you and make sure that this works as expect with Nuxt

@premiermatt this should be fixed in latest release now. Thanks for raising the issue.


Setup orientdb in intellij idea

Hello
Can you give me some introduction or tips to set up orientdb in intellij? I want to contribute with code.Thanks!

Your contribute is VERY welcomed. Take a look at: http://orientdb.com/docs/last/Contribute-to-OrientDB.html.


adding is-dark modifier to input

Description
add a is-dark modifier to inputs.
Compatibility
is not breaking.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Swagger documentation erroneously consumes application/json

When our code annotates a PATCH with @ApiOperation and sets consumes = "application/merge-patch+json", the Swagger 2.5.0 HTML documentation erroneously shows that the method consumes both "application/json" and "application/merge-patch+json". And worse, the erroneous "application/json" choice is the first and default one in the drop-down.
I traced the behavior in the debugger, and this is apparently caused by springfox.documentation.spring.web.scanners.MediaTypeReader being first in DocumentationPluginsManager's operationBuilderPlugins, and because the consumesList is empty at that time, its method apply(OperationContext) adds MediaType.APPLICATION_JSON_VALUE.
Later, SwaggerMediaTypeReader calls method springfox.documentation.swagger.readers.operation.SwaggerMediaTypes.apply(OperationContext), which adds "application/merge-patch+json" to the OperationBuilder's consumes Set, which already contains MediaTypeReader's erroneous first entry.
It seems that if the plugin SwaggerMediaTypeReader could be applied before plugin MediaTypeReader, then MediaTypeReader would not see an empty consumesList and thus would not add "application/json".

This should be opened on.... Springfox.


Update opam repo with latest release.

Please update the opam repo to version 0.6.18.

0.6.18 is not yet released (there are no new features yet, I just pushed some updates for the web site).


Docs: Usage

Hi @andrewosh 👋 I came to this module when I was looking how to properly create a swarm of hyperdrives (v10). On the datproject/sdk repo, there is a branch called dat2 where this module was used to create a swarm and seed the drives.
I'm doing something similar but I don't see any connection. I've added some logs using the debug function but with no luck yet.
Any guidance/hint on how to effectively create the swarm using hyperdrive or a link to a project example can work.
Thanks in advance!!!

Hey @dpaez, we'll be doing a big docs pass soon as part of the v10 release (which is coming together pretty quickly now).
In the meantime, maybe take a look at the way the hyperdrive daemon uses this: https://github.com/andrewosh/hyperdrive-daemon/blob/9862ed662a457de5824bc5dfc630fe8658f93b59/index.js#L57
You give it a corestore instance, and some networking options (though the defaults should be sufficient generally), then you call listen to start listening for connections.
To join the swarm, you can call seed and unseed with a discovery key. Importantly nothing in this module is persistent, so you have to maintain your own index of cores to seed (the daemon uses leveldb for this), and reseed everything when your process restarts.
Hope that's helpful -- there will be a lot more to work with soon.

Oh thanks!, I made some progress on this front recently. I will check out the hyperdrive-daemon example, though.
I made some tweaks to this lib to be able to handle multiple corestores (my use case involves multiples hyperdrives each one with their own corestore).
Let me know if I can help with docs or anything.
Once again, thanks for your answer Andrew!


Function not found: CONTAINS_SUBSTR

It seems like the CONTAINS_SUBSTR BigQuery function is missing from this package.

Hey @hansottowirtz!
Thank you for creating this issue. You are right, this function is not a part of ZetaSQL library that we use. We will add support of this function in the next extension release.

Hey @hansottowirtz. I added this function in new version v0.29.0. Please check if it helps. Also please let me know if you have another missing functions.


fixed changes mentioned in review

Changes made to code

Fixed the code and security loophole issues

Fixed Home mount issue raised.

[AMD Official Use Only - AMD Internal Distribution Only]
Let me expand a bit on the reproducible builds. It would be very nice if we could have two people, say in India and California, both starting a build on their local machines and end up with two sets of packages which are identical. There are lots of things that work against this, but we are tackling them. Obviously we want to start with the same source code. We get this by using the “repo” tool and a manifest which gets us the same files checked out in the same locations. We have a version controlled build environment, so we know we are using the same tools (compilers, shells, grep, make and so on). We know how to set timestamps embedded in files to uniform values, rather than the local time so this takes care of building in different time zones at different times. We would like any generated files whose names end up embedded in the product to be the same. This suggests that we should set HOME to a known value, and maybe LOGNAME and USER as well.
Sparry, Icarus
PMTS Software Development Eng. | AMD
ML PMO&Validation US
D 8342106tel:8342106
From: ammallya @.>
Sent: Monday, August 26, 2024 2:59 PM
To: ROCm/ROCm @.>
Cc: Sparry, Icarus @.>; Review requested @.>
Subject: Re: [ROCm/ROCm] fixed changes mentioned in review (PR #3627)
Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding.
@ammallya commented on this pull request.

In tools/rocm-build/docker/ubuntu20/docker_run.shhttps://github.com/ROCm/ROCm/pull/3627#discussion_r1731871331:

@@ -30,10 +33,14 @@ docker run -ti \

 --mount="type=bind,src=${fake_shadow},dst=/etc/shadow,readonly" \

 --mount="type=bind,src=${fake_group},dst=/etc/group,readonly" \

 -v ${HOME}/.ccache:/.ccache \

-v ${HOME}:/home/${CURRENT_USER} \

I see I did it this way because while using the make command the script was failing because it could not find /home/user
maybe I can just make a directory with the name of the current user instead of mounting it.
—
Reply to this email directly, view it on GitHubhttps://github.com/ROCm/ROCm/pull/3627#discussion_r1731871331, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AXRWIKRMFWSIVNPURDX6DDDZTOQKXAVCNFSM6AAAAABM4OTP4GVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDENRRG4ZDAOJUGY.
You are receiving this because your review was requested.Message ID: @.***>


njitted array getitem accepts out-of-bounds array indices

Reporting a bug

[x] I am using the latest released version of Numba (most recent is visible in
the change log (https://github.com/numba/numba/blob/master/CHANGE_LOG).
[x] I have included below a minimal working reproducer (if you are unsure how
to write one see http://matthewrocklin.com/blog/work/2018/02/28/minimal-bug-reports).

Consider example session:
>>> import numba as nb
>>> import numpy as np
>>> def foo(x, i): return x[i]
...
>>> getitem = nb.njit(foo)
>>> getitem(np.array([1,2]), 2)
140182891159624

In the last expression, the following IndexError would be expected:
>>> np.array([1,2])[2]
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
IndexError: index 2 is out of bounds for axis 0 with size 2

I think Aaron is working on this in #4432.

For anyone finding this ticket, the reason bounds checking is not on by default is because checking the every access would prevent vectorization of loops and dramatically decrease performance as a result. The boundscheck feature being worked on in #4432 will make such checks opt-in, so that they can be used whilst developing code and switched off if desirable in production to get the extra performance.

This issue is a duplicate of https://github.com/numba/numba/issues/730

Closing, #4432 implemented optional bounds checking.


fix the bug #209943

fix for the bug bug "Cannot hide breadcrumb icons" #209943
https://github.com/microsoft/vscode/assets/22959614/0526a8ca-0182-485b-b6ba-e7649370613f

closes #209943

Does this bug still need any help?


Hide sidebar by default and move link to Spark UI to the menu

Hey @andypetrella,
we haven't found much use of the sidebar which doesn't seem to be finished yet (the style was odd for a while, chat seems to be in a very early/experimental phase, list of terms isn't that useful (especially with ever-growing list of resN variables..)) and thus we're constantly hiding it. Obviously, this constant hiding is very irritating and such unfinished part of the product increases learning curve for newbies, we've decided to simply hide it by default. What do you think?
We could also use browser cookies to remember last sidebar visibility choice (i.e. people who do like the sidebar could have it always open, otherwise it would be always hidden), but this requires to pick some library to manage cookies :) if you like this idea, please tell me what's your policy with new JS dependencies?
Here's how it looks after the change:

cc @vidma

good stuff :clap:
I say :rocket:, if you tested it ;)

Another place to put Spark UI link, might be the kernel indicator. But I think it's a button now, so wouldn't be as easy...
so good as is...

Interesting, actually I'm using the list of term (but I have an RStudio XP ^^), indeed, we have to enhance it, to maybe hidding the resX (and a switch to show them), also ordering and search would be helpful.
I'm also considering a shortcut using the info there that would open a input text where we can go-to the cell where the variable name you type in is defined (a la CTRL+SHIFT+R in sublime).
The chat is a toy so far, but I'd like to see it evolving, it can be useful!
Hiding the sidebar by default is nice but newcomers won't know it... they don't RTFM!
Hence, I'm proposing two things we can do either or both:

auto hide when the notebook opens but it slides on the right instead of being absent.
keep a small rectangle that would hide/show the sidebar on click. Or we can create a gutter a la Intellij where the icon for the sidebar (or per widget) would be presented.

Something that is mandatory however, is that the auto hide should at the very least be in the global configuration and could be extended by either or both the notebook's metadata or a cookie like you've proposed.
NOTE I guess that the sidebar is more useful now because I've recently added a log panel (for error right now) :-).

auto hide when the notebook opens but it slides on the right instead of being absent.

keep a small rectangle that would hide/show the sidebar on click. Or we can create a gutter a la Intellij where the icon for the sidebar (or per widget) would be presented.

as toolbar is still rather hackish, I'd rather hide it by default.
And sure we can think more about it, and add such features later ;)
P.S. preserving the user's default inside a cookie should be rather easy.

IMHO good enough. could revisit if sidebar gets more prod ready...
If you like, we could rename Toggle sidebar into smf more descriptive: Toggle sidebar (error msgs, variable definitions...)


.svgrrc is never called from rollup or webpack.

🐛 Bug Report
As is title, .svgrrc is never called from rollup or webpack. Is this Right? Is this habit to expected?  If it's not, or someone think this is good idea to able to do this, I'm thinking I'll create patch PR.
To Reproduce
Versions
"@svgr/rollup": "2.3.0"
"@svgr/webpack": "2.3.0"

Relate codes and PR

rollup/index.js: https://github.com/smooth-code/svgr/blob/master/packages/rollup/src/index.js#L32
convert.js: https://github.com/smooth-code/svgr/blob/master/packages/core/src/convert.js#L9
config.js: https://github.com/smooth-code/svgr/blob/master/packages/core/src/config.js#L3
https://github.com/smooth-code/svgr/pull/100

Expected behavior
Link to repl or repo (highly encouraged)
Run `npx envinfo --system --binaries --npmPackages

This is not the expected behaviour. .svgrrc should be enable for Rollup plugin and webpack loader. PR welcome to fix it.

webpack still can't read svgr config


QuickSight disable in China region

Summary
Reporting error in China region
Steps to reproduce

deploy solution in china region
create pipeline
step4 reporting

What is the current bug behavior?
Internal server error
What is the expected correct behavior?
Disable check reporting in China region
Relevant logs and/or screenshots

Possible fixes

This is :bug: Bug Report

Perhaps the QuickSight API is deploying China region, the different exceptions to be returned when calling the API.


Fix FastCountingDeque not recording correct size.

Remove assert to avoid counting not works at production env.
--
I'm still investigating the issue of https://github.com/lettuce-io/lettuce-core/issues/466 . Because we recently have same problem again, and found my setting of requestQueueSize not works.

Thanks for your pull request and for raising the issue. The change rather removes guards than fixing the issue. I was about to release Lettuce 4.4 these days. I will wait with releasing a few more days.

Out of interest: How big do you set the queue size? I asking, because I'm not super happy with the current implementation. ConcurrentLinkedDeque yields the best performance during modifications but calling size() costs O(n) (time). Reasonably small queue sizes could fit into ArrayBlockingQueue but ArrayBlockingQueue pre-initializes the array size with the limit. A queue size limit of 1,000,000,000 elements would probably kill your JVM because of pre-allocated empty arrays.
/cc @CodingFabian

I set blockingQueueSize as 1000(issue #466 hard to find clue, so I set it to 1000 as safe net), and because of assert, the checking at https://github.com/lettuce-io/lettuce-core/blob/master/src/main/java/com/lambdaworks/redis/protocol/CommandHandler.java#L302 not works.

From that perspective, I'd suggest to use:

ConcurrentLinkedDeque if no limit is set - ConcurrentLinkedDeque is the fastest unbounded implementation although it generates some garbage. Without bounds, we don't need size()
ArrayBlockingQueue if a reasonable limit is set - ArrayBlockingQueue creates no garbage on modifications but requires a bound. We won't use the blocking aspect of the queue
LinkedBlockingQueue if a high limit is set - ArrayBlockingQueue pre-allocates elements and we don't want to exhaust the heap for pre-allocations.

Regarding the reasonable limit: I'd set the threshold to 100,000 elements (customizable via system property) to decide whether we use ArrayBlockingQueue or LinkedBlockingQueue. We fall back using the queue's internal counting mechanism and so we don't rely on our own bound checking. ForwardingDeque and  FastCountingDeque get obsolete and can be removed.

we do not know what a good limit is, so we just set it as safeguard. And its currently set to 100k

btw: the change looks good to me. without -ea this code will not do anything, will it?

I had a bit of time to reconsider CommandHandler and concluded to revisit the notion of queue, commandBuffer and transportBuffer.
I have a change prepared to use each queue for a particular use case (queue is just a queue to communicate commands between dispatching and netty and no longer acts also as the protocol stack).
Long story short: The queues change a bit in their responsibility which allows true bounded queue usage if requested. I will come back after the weekend with a fix.

I pushed a change that makes FastCountingDequeue obsolete by changing the way how queues are used. Lettuce now uses netty futures to attach (queue) commands and a AtomicIntegerFieldUpdater to keep track of queue size limits.
Closing this PR because it's no longer applicable.

Short upgrade regarding the performance:
4.3.3
Benchmark                                  Mode  Cnt   Score   Error  Units
CommandHandlerBenchmark.measureNettyWrite  avgt    5  56,134 ± 4,749  ns/op
CommandHandlerBenchmark.measureUserWrite   avgt    5  82,923 ± 5,548  ns/op

## 4.4.0

(unbounded)
Benchmark                                  Mode  Cnt   Score   Error  Units
CommandHandlerBenchmark.measureNettyWrite  avgt    5   7,040 ± 2,372  ns/op
CommandHandlerBenchmark.measureUserWrite   avgt    5  40,471 ± 5,700  ns/op
(bounded)
Benchmark                                  Mode  Cnt   Score   Error  Units
CommandHandlerBenchmark.measureNettyWrite  avgt    5   7,959 ± 0,928  ns/op
CommandHandlerBenchmark.measureUserWrite   avgt    5  53,400 ± 4,662  ns/op

## 5.x (unbounded, ConcurrentLinkedDeque)

Benchmark                                  Mode  Cnt   Score   Error  Units
CommandHandlerBenchmark.measureNettyWrite  avgt   16  6,413 ± 0,658  ns/op
RedisEndpointBenchmark.measureUserWrite    avgt   16  39,052 ± 0,836  ns/op


Prepare release 3.1.3

@danielpassos I guess we want to publish this manually. Normally we do that by bumping all packages


support run rayjob in pipeline

Currently paddleflow pipeline cannot run distributed job like spark / ray / vcjob, here I add a patch to support run rayjob in pipeline.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.dengkai02 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


[workflows] reduce workflows used in PRs

This PR consolidates workflows to conserve resources used in github actions. I wanted to use if logic in a github action to accomplish the same tasks in one workflow.
This combines sauce, sauce-ie11, lint, changeset, and local-tests into one github workflow. They all consume the same resources.
One effect is if a changeset or lint fails, no tests will run. Likewise, Sauce tests will only be run if local tests pass and IE tests will run only if sauce is successful.
This reduces our saucelabs bandwidth and effectively only runs remote tests when local tests pass. This is also the exact use-case for saucelabs: to test browser behavior past what we can do locally.

Here's an example of what it looks like when all tests pass, notice the Remote Test Failure Check does not run on a successful pass
As opposed to remote tests failing, where the failure check will output what remote tests failed:

This combines sauce, sauce-ie11, lint, changeset, and local-tests into one github workflow. They all consume the same resources.

I'm not sure we should be combining this many things. I thought we talked about combining lint + changeset, and all the sauce browsers once we stabilized IE.
I think the criteria for grouping should be around what dependencies the jobs have (ie, lint and changeset don't require an npm install), how fast the results can come back (lint and changeset can both return in < 1 min, local tests < 10 min, sauce tests maybe a bit longer, benchmarks ~30min), and whether independent results will be useful to us, and how much we'd get out of parallelism. Running Node tests, then local browser tests, then Sauce tests then IE tests in sequence is just going to take a long time and we won't see job results until the whole thing is done. And before IE is stabilized we won't have a separate signal that that failed that we can consider before merging.

sounds good. i'll make a separate PR for the lint + changeset and keep this one as a draft.
I thought it was quite successful and greatly reduced our workflows and bandwidth.

This is something I'd like to move towards though in terms of workflows and testing. Most of our workflows run on ubuntu. And they all build the same resources.
An approach like this one would provide a gated and measured way of progressing to expensive resources.

lack of changesets and lints would block any local tests ($ early bail, cheap)
lack of local tests passing would block any remote tests ($$ costs time)
sauce labs tests only runs when local tests run ($$$ expensive, actual browsers, saucelabs bandwidth)

I feel this is more reflective of staging and progressing from a dev like test environment to a prod like test environment.
Something to aim for I feel


Wallet Balance doesn't update

UI Version: 0.7.0
Description: After getting voyager up for the first time on a fresh install and importing my keys i was told i held no tokens. I went to a faucet and requested some but still no tokens. Went to the block explorer and looked for a recent block with any transactions and found mine (i think it was mine). Went back to wallet and still no balance and then remembered there's a refresh button. maybe wallet balance should be automatically refreshed on the mounted event?

use RPC to watch for new TXs then trigger LCDs

wrapper.vm.$nextTick and wrapper.update() helped when moving components over when there were timing errors in tests

Added one more task to this issue

[ ] remove update button

Very similar to #679


Reuse Use Case - Separate Unattended

Once I get an set of disks built for a VM, I will want to copy them for each new VM.  Because I do not have all the info (e.g., computer name and OU), perhaps the injection of the unattended file should be separate.  Then the unattended can be applied to a copy of a VHDX.  Are there other things that should be separate?
Also, from what I've read online, it appears that both x86 and amd64 sections for the some thing will both run on an amd64 system if both are present.  The WinPE pass needs to be x86 or amd64, depending upon what WinPE is using.  The other passes are only in the x86 or amd64 of the OS deployed?  Can the WinPE and OS architectures be detected from the image?
Expected Behavior

The unattended file generated should have one section per architecture.  The architecture of the section needs to be appropriate for both the WinPE and OS deployed.  The unattended file will be specific to each deployment, so it needs to be applied (or re-applied) after the base image is copied.
Current Behavior

The unattended can have both or have x86 when deploying to amd64.  If I copy the VHDX files for a new VM, I will need to create a new function to call New-UnattendedXMl and replace the existing unattended file.  Or I will need to manually mount the VHDX and update the unattended file.
Possible Solution

Use only amd64 in the template.  Change the attribute to x86 in WinPE pass components if WinPe is x86.  Change the other components proc attribute to x86 is the OS is 32 bit.  In those rare cases that the x86 and amd64 components for the same thing are different, then include both in the template and later remove any that do not apply.
The code should be x86 and amd64 agnostic unless there is a real difference in how the xml is constructed.  The x86/amd64 attribute can be altered as needed for those that are otherwise identical.
Steps to Reproduce (for bugs)

Context

I'm testing with a windows server 2019 image.  I changed all "x86" entries in the New-UnattendedXML template to "amd64" and removed duplicates.  Then I updated the code to insure there was no duplicate processing or specific test for x86.  Works for the amd64 only case.
Your Environment

Module version used:
Operating System and PowerShell version:

In what case is having both architecture causing you an issue?
None of my testing has ever shown an bad side effect to having both.

I think it's been working, but Microsoft warns about using both in the same file.  (See the end of the documentation at the following link.)  I do not know how this all works, but it if x86 works for both x86 and amd64, then why bother with defining a section for amd64?
https://docs.microsoft.com/en-us/previous-versions/windows/it-pro/windows-7/dd799255%28v%3dws.10%29
I will try just x86.  My attempt to add letters to the extra partitions was ignored.  Probably because the drives are offline.  I added a section to change the SAN policy that prevents them from coming online, but it was ignored.  I used amd64, but the documentation indicated x86.  Maybe that's the issue.  If the oflineServicing pass is after the windowsPE pass, then perhaps this can't work.  I have never tried this before.  You're module inspired me to see if I can write a script to built a SQL Server unattended.
<?xml version="1.0" encoding="utf-8"?>
<unattend xmlns="urn:schemas-microsoft-com:unattend" xmlns:wcm="http://schemas.microsoft.com/WMIConfig/2002/State" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
  <settings pass="offlineServicing">
    <component name="Microsoft-Windows-PartitionManager" processorArchitecture="amd64" publicKeyToken="31bf3856ad364e35" language="neutral" versionScope="nonSxS">
      <SanPolicy>1</SanPolicy>
    </component>
  </settings>
  <settings pass="windowsPE">
    <component name="Microsoft-Windows-Setup" processorArchitecture="amd64" publicKeyToken="31bf3856ad364e35" language="neutral">
      <DiskConfiguration>
        <Disk wcm:action="add">
          <DiskID>0</DiskID>
          <ModifyPartitions>
            <ModifyPartition wcm:action="add">
              <Order>1</Order>
              <PartitionID>3</PartitionID>
              OS
              <Letter>C</Letter>
            </ModifyPartition>
          </ModifyPartitions>
        </Disk>
        <Disk wcm:action="add">
          <DiskID>1</DiskID>
          <ModifyPartitions>
            <ModifyPartition wcm:action="add">
              <Order>1</Order>
              <PartitionID>2</PartitionID>
              SQL
              <Letter>G</Letter>
            </ModifyPartition>
          </ModifyPartitions>
        </Disk>
        <Disk wcm:action="add">
          <DiskID>2</DiskID>
          <ModifyPartitions>
            <ModifyPartition wcm:action="add">
              <Order>1</Order>
              <PartitionID>2</PartitionID>
              BACKUP
              <Letter>H</Letter>
            </ModifyPartition>
          </ModifyPartitions>
        </Disk>
        <Disk wcm:action="add">
          <DiskID>3</DiskID>
          <ModifyPartitions>
            <ModifyPartition wcm:action="add">
              <Order>1</Order>
              <PartitionID>2</PartitionID>
              DATA
              <Letter>I</Letter>
            </ModifyPartition>
          </ModifyPartitions>
        </Disk>
        <Disk wcm:action="add">
          <DiskID>4</DiskID>
          <ModifyPartitions>
            <ModifyPartition wcm:action="add">
              <Order>1</Order>
              <PartitionID>2</PartitionID>
              LOG
              <Letter>J</Letter>
            </ModifyPartition>
          </ModifyPartitions>
        </Disk>
        <Disk wcm:action="add">
          <DiskID>5</DiskID>
          <ModifyPartitions>
            <ModifyPartition wcm:action="add">
              <Order>1</Order>
              <PartitionID>2</PartitionID>
              TEMPDB
              <Letter>T</Letter>
            </ModifyPartition>
          </ModifyPartitions>
        </Disk>
      </DiskConfiguration>
    </component>
  </settings>
etc.

I changed the san policy and disk config components to x86 and they are still ignored.  I might end up using Set-Disk and Set-Partition to bring them online and set the label and letter.


How to change contract-scope when IDs are not 'stable'?

Describe the bug
In many cases we would like to change contractScope (for example: energy supplier, holon) of a contract, by using a 'change attribute' operation. However, since contractScope refers to the ID of another actor, and IDs are changed when a scenario is cloned, it is not possible to put in a static value for the ID in the slider-configuration. How can we achieve the desired functionality of being able to change to a different energy supplier?
This functionality is fundamental to many aspects of our stories/challenges.

I understand the idea for such a feature, and I feel like the desired behavior would already be achievable with current features.
In order to move an actor from one contract to another, you could use the add and remove functionality to remove the actor from one contract, and add it to others, like so:

Filter on contracts with original_id to move from
Remove the actor from these contracts

Filter on contracts with original_id to move to
Add a template actor to these contracts
Modify the new template actors under these contracts however you want

Would this result in the behaviour you would like to see?

After discussing with @TAVMWB, the following steps are proposed to change achieve a change of contract-scope:

Filter all contracts with the 'old' contract-scope (for example: energysupplier1) based on contracttype and relational attribute filters to select the contracts from a certain group of connectionOwner-actors.
Remove all contracts
Prepare a template contract with the 'new' constractScope (for example: energyholon 1)
Filter the same group of connectionOwner-actors as above
Add contracts the template contract with new contractScope to these connectionOwner-actors. This 'add' action sets the correct 'actor' (contract-holder) field for all the newly instantiated contracts.

related bug found: parent foreign key field for contract was named "actor_owner", but should be "actor".


Need status filter for trigger: on_pod_update

Now, its supporting only the name, namespace and labels as filters.
I would like to take some action based on the pod's status.
For example,

I want to delete the pod when its status is Evicted
I want to change the memory and cpu of the deployment when its status is OOMKilled

Basically the functionality will be same as on_crashloopbackoff, but a more generalized version of it

Thanks @mvkrishna86
These sounds like useful triggers.
We'll prioritize it and add it to Robusta

can I work on this issue ?


[Integration] It's impossible to delete chat (error 500 in console) if there any shared file exists in the structure

Create chat1 and chat2
Share chat1 (open shared link by another user) to have shared icon appeared
Delete chat2 thru context menu -> error in console -> bug
Refresh the browser -> deleted chat appears

Note: verified on new user, that chat is deleted ok if there is no any shared chat
Note: blocker #696

Hello @VolhaBazhkova
It was an issue on core side and @Maxim-Gadalov already fixed
Please recheck it

This particular case was verified successfully on staging


Use common memcached.GetMemcachedServerListWithInetString()

Use common func to retrieve the memcached server list for a memcached instance.

@stuggi: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:
Test name 	Commit 	Details 	Required 	Rerun command
ci/prow/swift-operator-build-deploy-kuttl 	76aec54 link 	true 	/test swift-operator-build-deploy-kuttl
Full PR test history. Your PR dashboard.

Failure looks like it could indeed be related to the PR:
case.go:366: resource Swift:swift-kuttl-tests/swift: .status.conditions.message: value mismatch, expected: Setup complete != actual:  Memcached instance has not been provisioned

@stuggi: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:
Test name 	Commit 	Details 	Required 	Rerun command
ci/prow/swift-operator-build-deploy-kuttl 	76aec54 link 	true 	/test swift-operator-build-deploy-kuttl
Full PR test history. Your PR dashboard.

Failure looks like it could indeed be related to the PR:
case.go:366: resource Swift:swift-kuttl-tests/swift: .status.conditions.message: value mismatch, expected: Setup complete != actual:  Memcached instance has not been provisioned

Ah nope, it's the missing hashes issue affecting all repos:
 Warning  Failed          23m (x4 over 24m)     kubelet            Failed to pull image "quay.io/podified-antelope-centos9/openstack-memcached@sha256:8347d7e59d00117771cdc68c39c2ccebcfb6dbd197689bd2231a1e9fd8fb44c6": rpc error: code = Unknown desc = reading manifest sha256:8347d7e59d00117771cdc68c39c2ccebcfb6dbd197689bd2231a1e9fd8fb44c6 in quay.io/podified-antelope-centos9/openstack-memcached: manifest unknown

/test swift-operator-build-deploy-kuttl

/hold


Dangerous weak-typing of comparison of types

Description
Normally, Typst is strongly-typed, which is good:
a: #(1=="1")   // false

b: #(1<"2")   // cannot compare integer and string

But not when comparing string and type object:
#(type([a])=="content")  // true

Even more dangerous, the name of the constant for the type object is not always the same as its string representation:
#(type([a])=="content")

#(type("a")=="str")

#(type(1)=="int")

#(type(1.2)=="float")

#(type([a])==content)

#(type("a")==str)

#(type(1)==int)

#(type(1.2)==float)

#content

#str

#int

#float

Reproduction URL
No response
Operating system
Linux
Typst version

[ ] I am using the latest version of Typst

This is temporary behaviour as a grace period for compatibility with earlier Typst versions. It will be removed in a future version. The names are different specifically because that was the old output of the type function. See here: https://typst.app/docs/reference/foundations/type/#compatibility


[cpp][absl] Include str_cat.h for StrCat

Explicitly include str_cat.h for StrCat to make sure there are no build failures.
After building to
https://github.com/abseil/abseil-cpp/commit/9e1789ffea47fdeb3133aa42aa9592f3673fb6ed for abseil-cpp, the grpc repo won't build due to missing StrCat definition. Include the str_cat.h before grpc start building with the updated absl-cpp release.
Tested:
Checked the missing header with
files=($(git grep StrCat | awk -F ':' '{print $1}'))

for file in "${files[@]}"
do
  if [[ "${file}" =~ "third_party" ]];
  then
    continue
  fi
  cat "${file}" | grep "str_cat.h"  > /dev/null && continue
  echo "${file}"
done

All StrCat usage now include the header.

oh, I don't think I have permission to add release notes: no as the label, or maybe I just don't know how to do it.

oh, I don't think I have permission to add release notes: no as the label, or maybe I just don't know how to do it.

No worries. I added the label.

I think we're gonna want a // IWYU pragma: keep alongside each of these

I think we're gonna want a // IWYU pragma: keep alongside each of these

I have updated to include the IWYU comment.

please resolve conflicts

please resolve conflicts

Resolved.

please run /tools/distrib/sanitize.sh

oh, the error's actually
src/core/lib/iomgr/tcp_posix.cc:49:1: error: duplicate include [readability-duplicate-include,-warnings-as-errors]
#include "absl/strings/str_cat.h"  // IWYU pragma: keep
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Suppressed 7371 warnings (7322 in non-user code, 49 NOLINT).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
1 warning treated as error

I believe we've removed IWYU anyway, so once you resolve conflicts again, we should be able to merge it.

I believe we've removed IWYU anyway, so once you resolve conflicts again, we should be able to merge it. Sorry for the churn.

Sorry for the delay, I have pushed another revision of this. Hopefully I didn't miss anything.


Incorrect str->camelize()

How to reproduce
string str1 = "some_string";
string str2 = "someString";

echo str1->camelize(), PHP_EOL;
echo str2->camelize(), PHP_EOL;

Actual
SomeString
Somestring

Expected
SomeString
SomeString

PHP's workaround
$string1  = "some_string";
$string2  = "someString";

$stringParts = explode('_', $string1);
$stringParts = array_map('ucfirst', $stringParts);
$string1 = implode('', $stringParts);

$stringParts = explode('_', $string2);
$stringParts = array_map('ucfirst', $stringParts);
$string2 = implode('', $stringParts);

echo $string1, PHP_EOL;
echo $string2, PHP_EOL;

SomeString
SomeString

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/26815643-incorrect-str-camelize?utm_campaign=plugin&utm_content=tracker%2F280146&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F280146&utm_medium=issues&utm_source=github).

Hi looks good I'm seeing correct camel case in the getters/setters now for camel case table field names.
I've just noticed the classname (and therefore php file name), is also being created in title case when the table name is camel case - e.g for a table profileAddress, I get Profileaddress.php with a class Profileaddress - can this be fixed in the same issue as I guess it must use similar camelize function?

@GavinMcL Create please separated issue at Devtools repo

ok thanks


update README.md flickity 2.0.5

Coverage remained the same at 86.881% when pulling c88c601ec7e72ad5c3de385b7f6742e19e6a7c0f on kukac7:patch-3 into 629dc88bed067cf523703d7e341cbf1d44664571 on benjamincharity:master.