cp: correct --verbose --parents output for both files and directories

This pull request corrects the behavior of cp --parents --verbose when the source path is a file and when the source path is a directory so that it prints the copied ancestor directories. For example,
$ mkdir -p a/b d
$ touch a/b/c
$ cp --verbose --parents a/b/c d
a -> d/a
a/b -> d/a/b
'a/b/c' -> 'd/a/b/c'

And
$ mkdir -p a/b/c d
$ cp -r --verbose --parents a/b/c d
a -> d/a
a/b -> d/a/b
'a/b/c' -> 'd/a/b/c'

Fixes #3332.

I guess the GNU regression is legit. could you please have a look?
Error: GNU test failed: tests/cp/src-base-dot. tests/cp/src-base-dot is passing on 'main'. Maybe you have to rebase?

I think it is related to https://github.com/uutils/coreutils/pull/3894#issuecomment-1236041009 I will take a look.

Okay, I've made a change to the second commit in this branch. By adding this condition:
    if source_absolute.is_dir()
        && !ends_with_slash_dot(&source_absolute)
        && !local_to_target.exists()
    {

it should also fix #3897.

I broke things by introducing some code to set the current working directory of the child process produced by UCommand. I'll need to adjust that a little bit to get this working.


Replace Help us plan section with Unconference summary

Replace with vague summary inspired by info from prospectus, like this:

This is taken care of by PR 'https://github.com/elm-camp/website/pull/29'

I think we can close this for now and make improvements as we notice them.


Mixed performances with the same name quartet

There is a new quartet in SUN Easy Street form my chapter. They name was used before 4-5 times with different names. How to remove the performances from this NEWBIE #504093, as they are just getting ready for their first official contest?

I don't think we need to list previous performances of quartet members in quartets OTHER THAN the named entry. I know these guys and neither one of them sang as Easy Street. Paul Carter and Bryn Havel sang in the same quartet that competed in SUN, so did Charlie Nelson and maybe David Cross. Why not list this under Performers?

i'm sorry, but i'm not fully understanding.
the system thinks of entities in terms of BHS ID (the primary key).  so is this a new group, with a new ID, or the same group, with the old ID but a new name?

Reopen - closed by accident

There are several quartets - current and past performers with the name Easy Street. BHS now lists just one

Last time I checked barberscore there were 4 with the name Easy Street. None of them listed any performance, except ES with Dave Cross as lead. I was not sure whether these performances were of the previous Easy Streets or of the previous quartets where current members of Easy Street sang. Either way I find listing performances where current members sang is confusing in the Groups tab. Better place in the Persons tab, IMHO

i agree.  we'll change this.


All the links under Core Packages are dead

Not sure how this happened, but all the links under the header are dead.
E.g. https://github.com/primer/primer-css/blob/master/packages/primer-core

Fixed in https://github.com/primer/primer-css/pull/243


Include JWT on all portal requests (fix for auth-only portals)

PULL REQUEST
Overview
The ky requests to other subdomains on the portal were not including the JWT cookie, causing auth errors.
ky doesn't seem to expose fetch's credentials option, so I've replaced methods with fetch where needed.

The app is usable at https://3004q31gj0ilrtpj40j8s2mq3st2mbgtgc3eq068qjs07at0cvmufgg.skynetpro.net/
Here is the current error for homescreen.hns.skynetpro.net for resolving an HNS name:

If able to resolve an HNS name (say, homescreen.hns because it's same-origin), you can get the metadata, but you can't call HEAD and then GET on the skylink:

Following feedback on Discord, Karol preferred that we keep all calls using ky. Last 2 commits replace ky-universal dependency with ky (we don't need the nodejs context here) and return to the original code, but adding {credentials: "include"}

Deployed to https://3001covtrto3u7jbf394ieers01t5bskoph2lofr9c9s4nq7curmtu8.siasky.net/Skylink: sia://GAAWY_3fcD8ea3jSSTnb4APSr5TGYirh-0sTwl9HZ7du-Q

https://300buoi415mmk4mcu0mnsmoa6hk6s5bpcn6gbbppcng0j73iksolatg.skynetpro.net/


Input doesn't work when LANG=eo on Wayland

Under Linux/sway, Alacritty appears to be unresponsive to keyboard input when LANG is set to eo. Changing LANG to en_US.UTF-8 or unsetting WAYLAND_DISPLAY causes it to work again.
Probably related error logs:
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:83:29: string literal is not a valid UTF-8 string
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:84:29: string literal is not a valid UTF-8 string
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:85:29: string literal is not a valid UTF-8 string
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:86:29: string literal is not a valid UTF-8 string
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:87:29: string literal is not a valid UTF-8 string
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:88:29: string literal is not a valid UTF-8 string
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:89:29: string literal is not a valid UTF-8 string
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:90:29: string literal is not a valid UTF-8 string
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:91:29: string literal is not a valid UTF-8 string
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:92:29: string literal is not a valid UTF-8 string
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:93:29: string literal is not a valid UTF-8 string
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:93:29: too many errors
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:93:29: failed to parse file
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:83:29: string literal is not a valid UTF-8 string
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:84:29: string literal is not a valid UTF-8 string
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:85:29: string literal is not a valid UTF-8 string
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:86:29: string literal is not a valid UTF-8 string
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:87:29: string literal is not a valid UTF-8 string
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:88:29: string literal is not a valid UTF-8 string
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:89:29: string literal is not a valid UTF-8 string
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:90:29: string literal is not a valid UTF-8 string
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:91:29: string literal is not a valid UTF-8 string
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:92:29: string literal is not a valid UTF-8 string
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:93:29: string literal is not a valid UTF-8 string
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:93:29: too many errors
xkbcommon: ERROR: /usr/share/X11/locale/iso8859-3/Compose:93:29: failed to parse file

Does this work when you set your locale to eo_EO.ISO8859-3 or eo_EO.UTF-8?

Those locales don't appear to exist? Maybe I'm missing something

They aren't listed in the locale.gen file but they are present in the compose.dir file, which is why I was curious if this would work by setting it explicitly to one of them. Since it seems like xkbcommon is trying to read an iso8849 file in utf8.

eo_EO.ISO8859-3 still causes the same issue, and eo_EO.UTF-8 doesn't seem to affect programs' language

If you clone the https://github.com/rust-windowing/glutin example and run it using cargo run --example window, can you reproduce this issue?
It should be possible to just check stderr for the xkbcommon issues, so actual mouse input shouldn't be necessary.

The window example also shows Compose parse errors

Reported upstream in https://github.com/rust-windowing/winit/issues/1125.

Fixed upstream.


No clipboard on OS X

It seems there's no clipboard functionality at all in the OS X client.

Hi @zippanto. Do you want to copy logs?

Yes mainly, but should be possible in any parts of the application where it's possible to highlight text

So, it's really strange because my friends and I checked this and on macOS copying works well.
What is your macOS and DockStation versions?

hi @igor-lemon apologies not sure what was going on here. I just tried again and it's working now.


[Feature Request]Hide games that don't drop treasure in the channel list or show only the games in the priority list

Hide games that don't drop treasure in the channel list or show only the games in the priority list

Hello o/
I'm assuming you mean this as a purely visual filtering feature? Because right now, the channel list represents 1:1 to application's memory, including which channels are being tracked, their status, game they're playing, etc., including the order of the channels on the list, matching the order of the channel objects in-memory. This order is considered when picking the next channel to watch after the one currently being watched goes offline.

I've looked into the mechanics of making this possible. The issue is, the table structure that's used to display the channels, has no easy option to "keep things ordered" once I start hiding rows from it. As in, "hiding" a row involves removing it from the current position, and all other rows change their indexes afterwards - reinserting at the same index is not possible since the list can have arbitrary rows removed, and the indexes shift every time that happens, so reinserting wouldn't happen at the same place the original row was removed from. This wasn't an issue with the inventory viewer since it had a "hide" option that'd remember the original index internally. The channel list was never designed with hiding in mind.
Is there a real need for this feature to exist? I mean, all online channels are already kept near the top of the list, their games are already displayed. Is it really a trouble to sift through the list, looking for a channel to watch? I'd appreciate other people's input on this topic here.

Thanks for your response, in fact this feature is not necessary for most users, but it can be handy to switch to a given game if the drop activity of a game is about to end, because for me the list of games shown in the list includes games I have not added to the priority list, many and hard to identify.
If the hidden feature is hard to add, is it possible to add a different color for each item of the game list, for example, the channel text color that supports drop or the background color of the item can be changed to a prominent color, unfortunately I'm not a Python developer and can't help you with your development.
Thanks again for your response.

I see. The colors solution is also possible, but just as tricky to implement (maybe only a little less). The thing is, the indexes issue I just mentioned could be helped with some index tracking, but since it'd take time (I have IRL work to do) and additional code to implement, I'd like to hear some feedback from other people on this request first. Anything from "I'd also like this feature to exist" to filtering option suggestions (like the "drops enabled" or "priority list games" ones you suggested) will be appreciated.
Until then, the status of this issue is "maybe, once more people would like this to exist". Same applies to #32.


Setting up puppeteer in a docker environment using sail

Hello guys!
Please I need help with setting up my puppeteer in a docker environment
I'm using laravel sail and will appreciate having a container with chrome setup where I can connect my puppeteer to, I have seen someone do it with node, something like this:
chrome:
    image: 'browserless/chrome'
    ports:
        - '${BROWSERLESS_CHROME_PORT:-3000}:3000'
    networks:
        - sail
    environment:
        - CONNECTION_TIMEOUT=-1

later in the code
const browser = await puppeteer.connect({
    browserWSEndpoint: "ws://chrome:3000",
});

I'm also curious about this. @Joemires, did you manage to make it work?

Yes I did @dsebastien
I'll send the structure once I'm settled

Yes I did @dsebastien Your docker compose configuration
chrome:
    image: 'browserless/chrome'
    ports:
        - '${BROWSERLESS_CHROME_PORT:-3000}:3000'
    networks:
        - sail
    environment:
        - CONNECTION_TIMEOUT=-1

later in the code
$client = $puppeteer->connect([
    'browserWSEndpoint' => env('BROWSER_WS_ENDPOINT') ?: 'ws://localhost:300'
]);

@dsebastien


I added a dot as part of training

training purpose - please allow me to make this change Stephen

Thank you Sev for the full stop


Fix warnings generated by purescript-0.8.0

Updated for prelude changes, but depends on purescript-strings being updated first.

Same story here as elsewhere :smile: :raised_hands:


my rabbitmq consumer/producer must never fail!

how can we write rabbitmq producers and consumers that can survive even the zombie apocalypse?
types of network failures, partitions and how consumers can handle this
consumer ack choices
fischer impossibility results & implications for "deliver just once" ideal scenario in bounded time (ref https://decentralizedthoughts.github.io/2019-12-15-asynchrony-uncommitted-lower-bound/ for example)

I think looking at this from the client side is an interesting perspective - self-healing resumable examples are what we need in production, but the tutorials are largely conceptual understanding, and the gap is quite large.


Make items editable by all content admins

Make items editable by all content admins (not just the depositor) -- may be achieved with groups + changes to workflows.

Belongs in other repo


Can't find artifacts in maven central

Although the README says they are there.

I am getting too many errors while compiling. This is not complete at all.


Use string representation for matching for GREL cross() function

Close #2461
As a conclusion to the discussion under #2461, when doing a match with the GREL cross() function, two values will be taken as identical if and only if they have the same string representation.
That's to say, the Integer 1 and Long 1 will match, while 1 and 1.0 won't, because they don't have the same representation.
Also changed the help text for the cross() function to make it more clear.

Nicely done.  Help text is perfect for explaining current handling.
matching column !@#b with !@#c

matching column !@#b with itself

However, curious where it gets confused with the record index here?
Seems OK...

and the record handling is unchanged I think with this PR (a good thing)


Gitlab migration: Migration fails when there are both issues and MRs in the repo

Description
When migrating a gitlab repo to gitea, the migration will almost always fail if both MRs and issues are being migrated.
This is because gitlab assigns a seperate number sequence for each of them, so there is an issue 1, an MR 1, issue 2, MR 2, etc.
This doesn't work with gitea, which has a single number sequence for both and doesn't want them to overlap.
I sadly trashed my logs so I do not have them, my apologies :(
I will reproduce it on a test instance if necessary and provide them, please let me know if wanted.
Gitea Version
1.9.3
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
gitea was being ran using the gitea helm charts found at https://gitea.com/gitea/helm-chart/
Database
PostgreSQL

This is #13884


Windows cookbook dependencies should be updated for Chef 12.5

Pending https://github.com/chef-cookbooks/windows/pull/278
@chef/client-windows

:+1:


System is being selected as x86_64-darwin on aarch64-darwin machine

Describe the bug
My devshell is always being selected as x86_64-darwin (or at least devenv is exporting env vars to suggest so) while my system is aarch64-darwin.
To reproduce
I can reproduce this on a fresh devenv init in an empty directory.
firework@FW-Mac0401 in ~/Workspace
~> cd tmp-dev
direnv: loading ~/Workspace/tmp-dev/.envrc
direnv: loading https://raw.githubusercontent.com/cachix/devenv/95f329d49a8a5289d31e0982652f7058a189bfca/direnvrc (sha256-d+8cBpDfDBj41inrADaJt+bDWhOktwslgoP5YiGJ1v0=)
direnv: using devenv
direnv: using cached devenv shell
hello from devenv
git version 2.44.0
direnv: export +AR +AS +CC +CONFIG_SHELL +CXX +DEVENV_DOTFILE +DEVENV_PROFILE +DEVENV_ROOT +DEVENV_RUNTIME +DEVENV_STATE +GREET +IN_NIX_SHELL +LD +LD_DYLD_PATH +MACOSX_DEPLOYMENT_TARGET +NIX_BINTOOLS +NIX_BINTOOLS_WRAPPER_TARGET_HOST_x86_64_apple_darwin +NIX_CC +NIX_CC_WRAPPER_TARGET_HOST_x86_64_apple_darwin +NIX_CFLAGS_COMPILE +NIX_DONT_SET_RPATH +NIX_DONT_SET_RPATH_FOR_BUILD +NIX_ENFORCE_NO_NATIVE +NIX_HARDENING_ENABLE +NIX_IGNORE_LD_THROUGH_GCC +NIX_LDFLAGS +NIX_NO_SELF_RPATH +NIX_PKG_CONFIG_WRAPPER_TARGET_HOST_x86_64_apple_darwin +NIX_STORE +NM +PATH_LOCALE +PKG_CONFIG +RANLIB +SIZE +SOURCE_DATE_EPOCH +STRINGS +STRIP +ZERO_AR_DATE +__darwinAllowLocalNetworking +__impureHostDeps +__propagatedImpureHostDeps +__propagatedSandboxProfile +__sandboxProfile +cmakeFlags +configureFlags +mesonFlags +name +system ~PATH ~XDG_DATA_DIRS

firework@FW-Mac0401 in ~/Workspace/tmp-dev
~> env | rg x86
system=x86_64-darwin
NIX_CC_WRAPPER_TARGET_HOST_x86_64_apple_darwin=1
NIX_PKG_CONFIG_WRAPPER_TARGET_HOST_x86_64_apple_darwin=1
NIX_BINTOOLS_WRAPPER_TARGET_HOST_x86_64_apple_darwin=1

firework@FW-Mac0401 in ~/Workspace/tmp-dev
~> nix repl --expr 'import <nixpkgs>{}'
Nix 2.21.2
Type :? for help.
Loading installable ''...
Added 21628 variables.
nix-repl> system
"aarch64-darwin"

I expect the aarch64 shell to be selected. I am not using rosetta in my terminal.
Version
~> devenv version
devenv 1.0.5 (x86_64-darwin)

The system string is actually hardcoded into the binary at build time, which means that the devenv you're running was built targetting x86.
How did you install devenv?

Through home manager, I noticed the flake.nix there has a hardcoded system string... 🤦 thanks!


RPi2

Hi
is it compatible with RPi2?
Thanks... Béla

Yes.   pigpio should work on all the current Pis (including the compute module).

Thanks

When you say:

Yes. pigpio should work on all the current Pis (including the compute module).

I am just starting to experiment with Pigpio and I am excited by the possibilities that it offers to a 'C' coder (who has virtually no experience of Python  :snake:)!
Reading through the man pages first, as a responsible coder does, :angel: : How does the code work when the range of usable GPIO pins on the Compute Module is nearly all the 120-odd total that the BCM2835 has - does the current code detect the platform is a Compute Module and allow all 120 pins to be used but on the other RPi products reject access to pins that are used "internally"?

I don't know how many pins the compute module has.
It has the same number of GPIO (54) as all the other Pis - however on the compute module you have physical access to all 54 GPIO.
GPIO 0-31 are fully supported by pigpio - sampling at up to 1 MHz, callbacks, PWM generation, waveforms.  The other GPIO (32-53) are restricted to the  other functions (read/write/set mode/get mode, hardware clock, hardware PWM etc.).

Ah further checking on the Farnell website here and here reveals that there are definitely 45 free GPIO pins (Of two more, GPIO 46 is HDMI detect and 47 seems to be used to control booting from the on-board Flash and for the "Activity" LED) - I guess that a naked BCM2835 device core may have 120 pins but the remainder are tied up in the HDMI interface, 2x RPi camera interfaces (each with a I2C bus BTW!), 2x RPi display and a composite video (no audio) output.


Turn rz/angular into it's own npm package

Per conversation, we will turn each programming language into it's own npm package. Razroo is currently not set up as a mono repo, but it makes sense to do so, so we can keep all npm packages in the same github repo. We will look into how to do so via nrwl nx, lerna, or perhaps even just our own package.json.

@giteshsarvaiya this here if interested in working on

How and where to learn the 'how to', can you pls suggest some resources.


MutableArray in Buffer is not UNPACKed

https://github.com/konsumlamm/rrb-vector/blob/204a6437c6fa170429555cc7bf524b3d3d916cb5/src/Data/RRBVector/Internal/Buffer.hs#L19-L20
Unpacking the MutableArray would likely give a small performance benefit. Or better, Buffer could use a SmallMutableArray directly since it does not need to support slicing.

You're right, although looking at the core, Buffer seems to be always unpacked and the start & length fields are optimized away. Feel free to open a PR anyway!

Nice, I hadn't checked the core.
I'll make a PR 👍


feat: delete redundant zones

Description
Delete redundant zones when drag last widget out or remove it.
https://www.notion.so/appsmith/Removing-redundant-zones-8a2cf907c97246adb618664940e298b0
Fixes #34854
Automation
/ok-to-test<EMAIL_ADDRESS>:mag: Cypress test results

[!TIP]
🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/9892015631
Commit: 2262349f5564baf6b40f741b226f848dd8dfa893
Cypress dashboard.
Tags: @tag.Anvil
Spec:
Thu, 11 Jul 2024 13:28:39 UTC

Communication
Should the DevRel and Marketing teams inform users about this change?

[ ] Yes
[x] No

Summary by CodeRabbit

New Features

Introduced new utility functions for handling widget properties, including checking if a widget is empty, if it has enabled JS properties, and retrieving widget children.
Added functions to validate and manage zone widgets, including determining zone widgets and redundant zone widgets.

Bug Fixes

Improved widget management by introducing functions to handle deletion of redundant zones during widget movements.

Tests

Added comprehensive test cases for new utility functions related to widget properties and zone validations to ensure robustness and reliability.

/build-deploy-preview skip-tests=true


squid:S1197 - Array designators "[]" should be on the type, not the variable

This pull request is focused on resolving occurrences of Sonar rule
squid:S1197 - Array designators "[]" should be on the type, not the variable.
You can find more information about the issue here:
https://dev.eclipse.org/sonar/rules/show/squid:S1197
Please let me know if you have any questions.
George Kankava

Hi,
Thanks for the pull request. Changing now the NumericLexer.java, unfortunately, will be fruitless because it is automatically generated from the NumericLexer.jflex specification. Every time we run jflex on that specification the types will be wrongly assigned.
Thanks,
Rodrigo


ENH: Improve support for choosing image model weights

This PR allows the user to determine whether to use ResNet weights pre-trained on ImageNet or BioViL, or whether to have a randomly initialised model: fixes #617  .

Converting to draft until we've updated from main, after merging the high-priority PR

#830

@markpinnock and @fepegar: Thanks for creating the PR and proposing this feature. I would like to recommend that nn.Module class definition and state dictionary initialisation are conceptually different topics, and it's conventionally handled separately in DL libraries. Moving forward we probably should remove the pretrained_model_path argument from ImageModel class instead of adding more argument because "weights or bnorm stats" do not characterise how computation graph is defined.
For your specific use case, I would recommend writing a simple helper function like the one below, which should enable your use-case and help us to have a more modular design. In case, I am missing a feature that you intend to have, please reply to this thread so that we can identify an alternative solution. Thanks.
def load_pretrained_encoder(specified_state_dict_path, model_args_dict):
     my_module = ImageModel(**model_args_dict)
     state_dict = torch.load(specified_state_dict_path)
     my_module.encoder.load_state_dict(state_dict)
     return my_module

Thanks for your contribution, @markpinnock!


C++ API developer documentation

This pull request adds developer documentation through a link to xml output from doxygen using breathe. I have also reorganized doxygen to use different input files when running through cmake and on rtd along the lines of github.com/NaluCFD/NaluWindUtils. You can see a preview of the added documentation here: http://ganesh-openfast.readthedocs.io/en/latest/source/dev/cppapi/index.html

@sayerhs The Breathe functionality seems very useful, but taking on the task of adding the source code documentation into sphinx in the near future is unlikely. Since the doxylink usage is minimal, missing doxylinks in the pdf are acceptable.
However, thanks for pointing out the possibility of Breathe. @gantech is using it in pull request #53 (C++ API documentation), and its clearly worth considering again in the future.

Closing this pull request to merge all further documentation updates to another pull request.


Talent | Profile

A user should be able to navigate to the learner detail page

React-Router route: /learners/:githubHandle


Add background card to DOB input

FRONTEND PULL REQUEST
Related Issue

Fixes #4460

Changes Proposed

Add a white background card to the DOB input, fixing the color contrast issues and bringing it in line with the rest of the site

Testing

Visual inspection of the DOB card

Screenshots / Demos
Before: Grey-on-grey text inaccessible

After:
Desktop

Mobile

Checklist for Author and Reviewer
Accessibility

[x] All changes have run through the Deque automated testing

Design

[ ] Any UI/UX changes have a designer as a reviewer, and changes have been approved

Merging this because as-is the grey text is inaccessible, but @kenieh if you have any suggested changes let me know and I'm happy to open a new PR for them.

LGTM!


woocommerce_api_authentication_error, Invalid Timestamp

I'm working on a local Wordpress server and a local NodeJS server.
Everything was working fine until I changed some products on Woocommerce, and I started getting this error when trying to call the API.
{
  errors: [
    {
      code: "woocommerce_api_authentication_error",
      message: "Timestamp inválido"
    }
  ]
}

It does not seem to be an issue with this REST API wrapper.
Maybe it's cache, try to clear the entire cache of your WordPress, but this is not the right place to ask for support not related to this wrapper.


Missing word in sentence

Reporting Failure shows the IoT Edge devices that have reported back to the service that one or modules have not been deployed successfully.
Should be:
Reporting Failure shows the IoT Edge devices that have reported back to the service that one or more modules have not been deployed successfully.

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

ID: 9c2fa43c-0132-8f96-ab4f-e35292d64888
Version Independent ID: c3041c24-5b96-8ff4-757a-218ad1d34c55
Content: Automatic deployment for device groups - Azure IoT Edge
Content Source: articles/iot-edge/module-deployment-monitoring.md
Service: iot-edge
GitHub Login: @kgremban
Microsoft Alias: kgremban

@perkops We are checking this and will get back to you.

Thanks for the feedback @perkops , fix on it's way ;)!


Compile time is too long

system version ubuntu 14.4
bazel version 0.9.0
gcc version 4.8
computecpp version 0.5.1
GPU:r9 270x 4g
ComputeCpp Info (CE 0.5.1)

Toolchain information:
GLIBC version: 2.19
GLIBCXX: 20150426
This version of libstdc++ is supported.

Device Info:
Discovered 2 devices matching:
platform    :
device type :

Device 0:
Device is supported                     : UNTESTED - Device not tested on this OS
CL_DEVICE_NAME                          : Pitcairn
CL_DEVICE_VENDOR                        : Advanced Micro Devices, Inc.
CL_DRIVER_VERSION                       : 1800.11 (VM)
CL_DEVICE_TYPE                          : CL_DEVICE_TYPE_GPU
Device 1:
Device is supported                     : UNTESTED - Vendor not tested on this OS
CL_DEVICE_NAME                          : AMD FX(tm)-8300 Eight-Core Processor
CL_DEVICE_VENDOR                        : AuthenticAMD
CL_DRIVER_VERSION                       : 1800.11 (sse2,avx,fma4)
CL_DEVICE_TYPE                          : CL_DEVICE_TYPE_CPU
If you encounter problems when using any of these OpenCL devices, please consult
this website for known issues:
https://computecpp.codeplay.com/releases/v0.5.1/platform-support-notes

when i run sudo bazel build --local_resources 2048,.5,1.0 -c opt --config=sycl //tensorflow/tools/pip_package:build_pip_package
.............
WARNING: /home/zzt/.cache/bazel/_bazel_root/37963c447511203e3d4ee1aa24f9805c/external/grpc/WORKSPACE:1: Workspace name in /home/zzt/.cache/bazel/bazel_root/37963c447511203e3d4ee1aa24f9805c/external/grpc/WORKSPACE (@com_github_grpc_grpc) does not match the name given in the repository's definition (@grpc); this will cause a build error in future versions
WARNING: /home/zzt/tensorflow-dev-amd_gpu/tensorflow/core/BUILD:1832:1: in includes attribute of cc_library rule //tensorflow/core:framework_headers_lib: '../../external/nsync/public' resolves to 'external/nsync/public' not below the relative path of its package 'tensorflow/core'. This will be an error in the future. Since this rule was created by the macro 'cc_header_only_library', the error might have been caused by the macro implementation in /home/zzt/tensorflow-dev-amd_gpu/tensorflow/tensorflow.bzl:1150:30
WARNING: /home/zzt/tensorflow-dev-amd_gpu/tensorflow/contrib/learn/BUILD:15:1: in py_library rule //tensorflow/contrib/learn:learn: target '//tensorflow/contrib/learn:learn' depends on deprecated target '//tensorflow/contrib/session_bundle:exporter': No longer supported. Switch to SavedModel immediately.
WARNING: /home/zzt/tensorflow-dev-amd_gpu/tensorflow/contrib/learn/BUILD:15:1: in py_library rule //tensorflow/contrib/learn:learn: target '//tensorflow/contrib/learn:learn' depends on deprecated target '//tensorflow/contrib/session_bundle:gc': No longer supported. Switch to SavedModel immediately.
INFO: Analysed target //tensorflow/tools/pip_package:build_pip_package (257 packages loaded).
INFO: Found 1 target...
INFO: From Compiling tensorflow/core/framework/op_gen_lib.cc [for host]:
tensorflow/core/framework/op_gen_lib.cc:269:13: warning: unused function 'RenameInDocs' [-Wunused-function]
static void RenameInDocs(const string& from, const string& to, OpDef* op_def) {
^
1 warning generated.
In file included from :1:
./tensorflow/core/framework/op_gen_lib.cc:269:13: warning: unused function 'RenameInDocs' [-Wunused-function]
static void RenameInDocs(const string& from, const string& to, OpDef* op_def) {
^
1 warning generated.
INFO: From Compiling tensorflow/core/common_runtime/gpu/process_state.cc [for host]:
tensorflow/core/common_runtime/gpu/process_state.cc:54:6: warning: unused function 'useCudaMallocAllocator' [-Wunused-function]
bool useCudaMallocAllocator() {
^
tensorflow/core/common_runtime/gpu/process_state.cc:60:6: warning: unused function 'useCudaMemoryGuardAllocator' [-Wunused-function]
bool useCudaMemoryGuardAllocator() {
^
2 warnings generated.
In file included from :1:
./tensorflow/core/common_runtime/gpu/process_state.cc:54:6: warning: unused function 'useCudaMallocAllocator' [-Wunused-function]
bool useCudaMallocAllocator() {
^
./tensorflow/core/common_runtime/gpu/process_state.cc:60:6: warning: unused function 'useCudaMemoryGuardAllocator' [-Wunused-function]
bool useCudaMemoryGuardAllocator() {
^
2 warnings generated.
INFO: From Compiling tensorflow/stream_executor/stream.cc [for host]:
tensorflow/stream_executor/stream.cc: In function 'std::string perftools::gputools::{anonymous}::ToVlogString(perftools::gputools::dnn::DataType)':
tensorflow/stream_executor/stream.cc:184:1: warning: control reaches end of non-void function [-Wreturn-type]
}
^
INFO: From Compiling tensorflow/core/framework/op_kernel.cc [for host]:
tensorflow/core/framework/op_kernel.cc:962:8: warning: unused function 'FindKernelRegistration' [-Wunused-function]
Status FindKernelRegistration(const DeviceType& device_type, const Node& node,
^
1 warning generated.
In file included from :1:
./tensorflow/core/framework/op_kernel.cc:962:8: warning: unused function 'FindKernelRegistration' [-Wunused-function]
Status FindKernelRegistration(const DeviceType& device_type, const Node& node,
^
1 warning generated.
INFO: From Compiling external/snappy/snappy-sinksource.cc [for host]:
cc1plus: warning: command line option '-Wno-implicit-function-declaration' is valid for C/ObjC but not for C++ [enabled by default]
cc1plus: warning: unrecognized command line option "-Wno-shift-negative-value" [enabled by default]
INFO: From Compiling external/snappy/snappy-stubs-internal.cc [for host]:
cc1plus: warning: command line option '-Wno-implicit-function-declaration' is valid for C/ObjC but not for C++ [enabled by default]
cc1plus: warning: unrecognized command line option "-Wno-shift-negative-value" [enabled by default]
INFO: From Compiling external/snappy/snappy.cc [for host]:
cc1plus: warning: command line option '-Wno-implicit-function-declaration' is valid for C/ObjC but not for C++ [enabled by default]
external/snappy/snappy.cc: In member function 'void snappy::SnappySinkAllocator::Flush(size_t)':
external/snappy/snappy.cc:1403:38: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
for (int i = 0; i < blocks.size(); ++i) {
^
At global scope:
cc1plus: warning: unrecognized command line option "-Wno-shift-negative-value" [enabled by default]
INFO: From Compiling external/snappy/snappy-c.cc [for host]:
cc1plus: warning: command line option '-Wno-implicit-function-declaration' is valid for C/ObjC but not for C++ [enabled by default]
cc1plus: warning: unrecognized command line option "-Wno-shift-negative-value" [enabled by default]
INFO: From ProtoCompile tensorflow/contrib/lite/toco/toco_flags.pb.cc:
bazel-out/k8-opt/genfiles/external/protobuf_archive/src: warning: directory does not exist.
bazel-out/k8-opt/genfiles/external/protobuf_archive/src: warning: directory does not exist.
INFO: From Compiling tensorflow/contrib/fused_conv/ops/fused_conv2d_bias_activation_op.cc [for host]:
tensorflow/contrib/fused_conv/ops/fused_conv2d_bias_activation_op.cc:31:8: warning: unused function 'GetAllActivationModeAttrString' [-Wunused-function]
string GetAllActivationModeAttrString() { return "activation_mode: {'Relu'}"; }
^
1 warning generated.
In file included from :1:
./tensorflow/contrib/fused_conv/ops/fused_conv2d_bias_activation_op.cc:31:8: warning: unused function 'GetAllActivationModeAttrString' [-Wunused-function]
string GetAllActivationModeAttrString() { return "activation_mode: {'Relu'}"; }
^
1 warning generated.
[1,238 / 1,429] 8 actions, 1 running
Executing genrule //tensorflow/python:bitwise_ops_pygenrule; 31425s local
[Sched] .../contrib/input_pipeline:input_pipeline_ops_pygenrule; 31433s
Target //tensorflow/tools/pip_package:build_pip_package failed to build
Use --verbose_failures to see the command lines of failed build steps.
ERROR: build interrupted
there is a lot of warning and compile time is very long .stuck in one action for 9 hours.i tried gcc4.9/5.4.but it doesn't work .and i change bazel option opt to fastbuild.also doesn't work.so i need some help .

I just successfully compiled the thing, on fglrx 1800.11. No problems to report.
Yes, it took 20 hours...  On a cpu with the power of a toaster. Almost 10 times less than yours.
Only remotely sensible reason I can think, is your ancient OS.

Once this https://github.com/lukeiwanski/tensorflow/pull/214 gets in, the compilation time should be reduced ( as we finally remove calls to python )
Please re-open if it's still a problem


Merge [] and at() methods code

The implementation is a direct copy of the previous [] code.

The methods were already present before I started adding new ones :smile:


Add support for counters and multiple values in pseudo-elements

Adds support for <var> <attr> <counter> in content attributes for pseudo-elements. Also adds the possibility of defining counter-reset and counter-increment in styles.
Fixes #256.

wow, super cool @mikevoets
could you update the docs with basic info on what you enabled here? sorry I didn't notice this sooner!

@jrit Glad you think it's cool!
I assumed being able to use count, var and attr worked "out of the box", which is the reason why I haven't updated any documentation. Where in the documentation would it be appropriate to clarify this functionality?

sorry I keep missing notifications on this! I merged what you had so far, I guess that is a good point, when I make a release I'll add it to the changelog. thanks again, and sorry for taking so long to notice things :)

sorry I keep missing notifications on this! I merged what you had so far, I guess that is a good point, when I make a release I'll add it to the changelog. thanks again, and sorry for taking so long to notice things :)

what a good day! expect the release version!

what a good day! expect the release version!

@mikevoets could you tell me how to use this feature？

@guanpengchn Sorry for the late reply.
It works like this. If you have the following input:

 Title 1

 Title 2

 Title 3

This resolves to:

1 Title 1

Bar - Baz Title 2

2 Title 3


Makes the share and back icons in the single speaker view easier to see

This samples a pixel from the top right corner of the speaker's image and uses that to check whether the icons should be black or white, then applies the color filter. Fixes issue #961

I'm sorry, but I'm not 100% sure what's causing this error. I ran gradlew build on my own PC, and it worked fine.

It's not able to find the drawable you have referenced. Please check. Thanks!

Probably the drawables were not pushed, happens a lot in android studio.
Please manually git add and then commit those drawables.

--
Best,
Harshit Dwivedi

Okay, thanks. I added the drawables to the repository and pushed, and now it works

Looks good but there seems to be a merge conflict.
Please resolve it so that i can merge it.

Sadly there seems to be a merge conflict again.
Please resolve it.
I'll make it sure that there are no more merge conflicts for this PR.

Okay, I'll see if I can sort everything out.


issue-1862: upgrade commons-collections:commons-collections3.2.2 to c…

…ommons-collections4:4.4
This is to close issue 1862 in strongbox PR-1900

The branch name is incorrect, thus deployment will be skipped. The correct branch name is issue-1862.
You would need to close this PR, rename the branch and create a new one.


transformation old data to new index

Goal:
Build a flink job which reads all "old" data from the RawTwitterData table and writes the corresponding TermIndex entries.
Details:

use the flink dataset api
see example how to read data from accumulo using the dataset api
write data to accumulo using the AccumuloOutputFormat and the HadoopOutputFormat and Dataset.output
see accumulo-schema

see https://github.com/IIDP2016/OSTMap/wiki/Accumulo-schema

save username and screenname both as user
remove number of appearances when number of appearances is 0


Add Nethermind eth1 client as an option

Keep an eye on https://github.com/NethermindEth/nethermind/pull/2308 and consider offering Nethermind as an eth1 option when pruning is in.

As well as https://github.com/NethermindEth/nethermind/pull/2383

This is in thanks to @adrienlacombe . Nethermind has issues with DB corruption potentially, but that's upstream.


[Question] How to use the convert token logic inside Django?

I want to convert the token from OAuth to the Django session one, but I need to do it inside Django. How can this be achieved?
Tried to use requests and making a POST call internally but it's not working.

I'm trying to convert the token from the Google OAuth callback, getting the token from the Django session and return to another application. But when I deploy the server, it's not being able to request to itself
def complete_google(request: HttpRequest) -> Response:
    code: str = request.GET.get("code")
    url: str = "https://oauth2.googleapis.com/token"
    payload: Dict[str, str] = {
        "client_id": SOCIAL_AUTH_GOOGLE_OAUTH2_KEY,
        "client_secret": SOCIAL_AUTH_GOOGLE_OAUTH2_SECRET,
        "code": code,
        "redirect_uri": f"{DJANGO_BASE_URL}/complete/google-oauth2/",
        "grant_type": "authorization_code",
    }
    headers: Dict[str, str] = {"Content-Type": "application/x-www-form-urlencoded"}
    response: Response = requests.post(url, headers=headers, data=payload)

    response_json: Dict[str, str] = response.json()
    google_auth_token = response_json.get("access_token")
    url: str = f"{DJANGO_BASE_URL}/auth/convert-token/"
    payload: Dict[str, str] = {
        "grant_type": "convert_token",
        "backend": social,
        "client_id": APPLICATION_CLIENT_ID_GOOGLE,
        "client_secret": APPLICATION_CLIENT_SECRET_GOOGLE,
        "token": google_auth_token,
    }
    headers: Dict[str, str] = {"Content-Type": "application/x-www-form-urlencoded"}
    response: Response = requests.post(url, headers=headers, data=payload)
    return Response(
        response.json(),
     )

What is your django base url?
You could do it with self.request.build_absolute_uri('/')[:-1]
Or something like that.
Finishing the day here. Hope you find it out!

Succeeded?

No, I'm moving this to a different application, that's going to be in charge of validating the users. So I guess I will not encounter this problem. But if is there a way to make it to call itself would be ideal. Is there any way to call the view responsible for converting the token ?

I believe I managed to do what you were discussing @digoburigo.
class GoogleLoginView(APIView):
permission_classes = [AllowAny]
def post(self, request):
    google_token = request.data.get('credential')

    if not google_token:
        return Response({'error': 'No Google token provided'}, status=status.HTTP_400_BAD_REQUEST)

    token_exchange_url = request.build_absolute_uri('/auth/convert-token')

    # Perform token exchange with backend OAuth provider
    response = requests.post(token_exchange_url, data={
        'token': google_token,
        'backend': 'google-identity',
        'grant_type': 'convert_token',
        'client_id': settings.GOOGLE_APP_OAUTH2_CLIENT_ID,
        'client_secret': settings.GOOGLE_APP_OAUTH2_CLIENT_SECRET
    })

    if response.status_code == 200:
        # Return the access and refresh tokens to the frontend
        return Response(response.json())
    else:
        print(f"Token exchange failed. Status: {response.status_code}, Response: {response.text}")
        return Response({'error': 'Token exchange failed'}, status=status.HTTP_400_BAD_REQUEST)

Careful, you should never return the client secret like that. It should not be exposed for security reasons.


Add knee_identification.py file

knee_identification.py contains methods for obtaining descriptors of knee behavior of capacity fade for a given cell.

@pasinger Thanks for the contribution! This PR could probably use some unit tests if possible though

Hi @pasinger just to check if you still would like to merge this PR/if this will be come part of the new PR you are writing? Thank you


feat: Brillig IR refactor

Split brillig IR in low level instructions and higher level codegen files that compose the instructions and restrict visibilities. This should not have any business logic change. This also fixes debug_show by only printing the low level instructions, previously we were printing both the high level codegen and the low level instructions and it was a mess to read.

Benchmark results
Metrics with a significant change:

note_trial_decrypting_time_in_ms (8): 72.1 (+121%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit c34a299e and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
5,668
18,820
36,356

l1_rollup_calldata_gas
66,184
239,080
469,004

l1_rollup_execution_gas
666,865
953,990
1,336,361

l2_block_processing_time_in_ms
1,317 (-1%)
4,878 (+1%)
9,416 (-1%)

note_successful_decrypting_time_in_ms
163 (-28%)
628 (+14%)
1,048 (+11%)

note_trial_decrypting_time_in_ms
:warning: 72.1 (+121%)
44.7 (-38%)
72.8 (+2%)

l2_block_building_time_in_ms
18,711
70,110 (+1%)
138,538 (+1%)

l2_block_rollup_simulation_time_in_ms
8,470 (+2%)
29,669 (+2%)
57,557 (+2%)

l2_block_public_tx_process_time_in_ms
10,217 (-1%)
40,376
80,875 (+1%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
13,940 (+1%)
26,916 (+1%)

note_history_successful_decrypting_time_in_ms
1,269 (+7%)
2,430 (-2%)

note_history_trial_decrypting_time_in_ms
101 (+24%)
153 (-11%)

node_database_size_in_bytes
19,054,672
35,692,624

pxe_database_size_in_bytes
29,859
59,414

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
285
44,370
27,700

private-kernel-ordering
219
52,324
14,326

base-parity
1,913 (+6%)
128
311

base-rollup
727 (+1%)
177,083
925

root-parity
1,520 (-4%)
1,244
311

root-rollup
68.8
4,487
789

private-kernel-inner
642
73,229
27,700

public-kernel-app-logic
455 (+1%)
35,262
28,215

public-kernel-tail
178 (+1%)
40,926
28,215

merge-rollup
8.35 (+1%)
2,696
925

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
512 leaves
1024 leaves
2048 leaves
4096 leaves
32 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.2 (+2%)
16.2 (+1%)
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.6
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.591 (+1%)
0.498 (+1%)
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
47.3 (+1%)
73.3 (+2%)
235 (+2%)
448 (+1%)
877 (+1%)
1,742 (+2%)
N/A

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
96.0
159
543
1,055
2,079
4,127
N/A

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.484 (+1%)
0.451 (+2%)
0.426 (+1%)
0.420 (+1%)
0.416 (+1%)
0.416 (+1%)
N/A

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
54.2 (-1%)
108 (+1%)
344 (+1%)
662
1,338 (+1%)
2,636 (+1%)
N/A

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
104
207
691
1,363
2,707
5,395
N/A

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.481 (-2%)
0.484
0.463
0.455
0.460 (+1%)
0.455
N/A

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
61.6 (+1%)

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
109

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
0.538 (+1%)

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes

tx_size_in_bytes
22,014

Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes

tx_pxe_processing_time_ms
3,283 (-1%)
1,758

Metric
0 public data writes
1 public data writes

tx_sequencer_processing_time_ms
12.6 (+10%)
1,274 (+1%)


openssl.org has incorrect content-type HTTP header for source tarballs

andy@bark ~> curl -I https://www.openssl.org/source/openssl-3.3.0.tar.gz
HTTP/2 200
x-guploader-uploadid: ABPtcPrU7RWpmtP2LvsiHnlVY92nvTOlj8qGHcedRJGsr0pPXTv0zHaq3_BXOrBG0VjO8_s05S4
x-goog-generation:<PHONE_NUMBER>724876
x-goog-metageneration: 1
x-goog-stored-content-encoding: identity
x-goog-stored-content-length: 18038030
x-goog-meta-goog-reserved-file-mtime:<PHONE_NUMBER>
x-goog-hash: crc32c=1321TQ==
x-goog-hash: md5=yLBjr76oXYZ+Fh7LiBbPqQ==
x-goog-storage-class: STANDARD
accept-ranges: bytes
server: UploadServer
via: 1.1 google
date: Fri, 12 Apr 2024 22:04:51 GMT
age: 1440
last-modified: Tue, 09 Apr 2024 12:45:50 GMT
etag: "c8b063afbea85d867e161ecb8816cfa9"
content-type: application/x-tar
content-length: 18038030
cache-control: public,max-age=3600
strict-transport-security: max-age=31536000
alt-svc: h3=":443"; ma=2592000,h3-29=":443"; ma=2592000

These are incorrect HTTP headers. The content-type should be instead application/gzip, application/x-gzip, or application/tar+gzip, or it should use transfer-compression: gzip.
Or you could just omit the content-type header, or use octet-stream. Then it would not be incorrect.
https://github.com/ziglang/zig/issues/19636

@quarckster could you look into this please?
https://www.iana.org/assignments/media-types/application/gzip
application/gzip seems like the right content type I think

Nice, thanks!


dts: arm: nordic: "okay" status for wdt30 of nrf54l15_cpuapp

Change sets status of wdt30 to "okay". The wdt30 is used for watchdog0 alias. Change aligns behavior with our other SoCs.

Actually my commit only aligns the approach with other Nordic SoC's. The approach is ok for me, because it simplifies configuration and you still need to enable the CONFIG_WATCHDOG to enable the Zephyr watchdog driver.
If we want to change approach I would suggest to do it globally (outside of scope of this PR).

Actually my commit only aligns the approach with other Nordic SoC's. The approach is ok for me, because it simplifies configuration and you still need to enable the CONFIG_WATCHDOG to enable the Zephyr watchdog driver.
If we want to change approach I would suggest to do it globally (outside of scope of this PR).

The rule should be: is this peripheral always needed, i.e., on all builds? If not, it should be enabled on either board or, application overlay.

Thanks for your input. I will move enabling the wdt30 to DTS overlay of my application


Fixes #134 - Does not create http_forward when aws_lb_target_group.default is disabled

What
Fixes #134 - Does not create http_forward when aws_lb_target_group.default is disabled

disables aws_lb_listener.http_forward when default_target_group is not enabled
target_group is required when type is 'redirect'

Why
See #134.  Fixes this Validation error when default_target_group_enabled == 0 :
module.alb.aws_lb_listener.http_forward[0]: Creating...
╷
│ Error: creating ELBv2 Listener
(arn:aws:elasticloadbalancing:...:...:loadbalancer/...):
ValidationError: A target group ARN must be specified
│       status code: 400, request id:
7cf9d727-fc77-4d32-a160-deadbeefcafe
│
│   with module.alb.aws_lb_listener.http_forward[0],
│   on .terraform/modules/alb/main.tf line 150, in resource
"aws_lb_listener" "http_forward":
│  150: resource "aws_lb_listener" "http_forward" {

references
closes #134

Edited to remove unnecessary guard rails for the other aws_lb_listener blocks; they work fine when the default target group is disabled.
This leaves a small, clear change.  Please reply with any questions and feel welcome to reformat or cherry-pick as necessary.

/terratest

/terratest

@spazm Thanks. For the failing test, can you do
README.md is outdated. Please run the following commands locally and push the file:
  make init
  make readme


Simplify provided examples

Try to simplify current set of examples. ideally, make them fewer, but don't lose any value.
Remove examples which do not run "out of box" (need additional extra work). Those should be done as full example documents.
Also try to maintain set naming convention on installation files.

Resolved in v1.0.8


Panic on manual run when connection with Forge lost

Component
server
Describe the bug
If connection broken while you are running pipeline manually, you'll get unknown error and panic on Server.
Steps to reproduce

Run Forge
Run Woodpecker server, log in
Open some repository
Click Run pipeline
Turn off Forge / scale down to 0 / disallow WP-Forge connection via firewall
Close Trigger a manual pipeline run dialog by pressing Run pipeline button
See unknown error and not closed overlay
Error
Check server logs and find panic recovered
server-panic.log

Expected behavior

Should not be panic on server
As dialog closes, overlay should disappear also
More meaningful error than unknown

System Info
`2.7.0`, Gitea

Additional context
No response
Validations

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Checked that the bug isn't fixed in the next version already [https://woodpecker-ci.org/faq#which-version-of-woodpecker-should-i-use]

next-bbdeb4622c
While Should not be panic on server was fixed,

As dialog closes, overlay should disappear also
More meaningful error than unknown

are not addressed. Still unknown error (HTTP 500) and overlay doesn't close automatically.


await in repl no longer works

evcxr used to support this, however even the simple case of:
>> async fn foo() -> i32 { 3 }
>> foo().await
A compilation error was found in code we generated.
Ideally this shouldn't happen. Type :last_error_json to see details.
error: couldn't read lib.rs: No such file or directory (os error 2)

>> :last_error_json
   ^^^^^^^^^^^^^^^^
{"rendered":"error: couldn't read lib.rs: No such file or directory (os error 2)\n\n","children":[],"code":null,"level":"error","message":"couldn't read lib.rs: No such file or directory (os error 2)","spans":[]}

No longer works.

Sorry about that. Looks like I broke that when I added some code that validates newly added dependencies. Will send a fix shortly.


Verify nonExistentEmoji is a set

Summary
When using the mobile app without blacklisting the reducer for nonExistentEmoji it gets converted into an array instead of a Set and then when the action gets called it crashes the app, this PR makes sure that nonExistentEmoji is always a Set

@hmhealey I could try and implement a transform for it in the mobile app to see if that works, but this way we ensure that even without a transform it will work in any client

I'd prefer to fix the transform. The problem with doing it this way is that we should then theoretically add this check to anywhere we access one of the Sets which would add a lot of unnecessary code. The problem here is with the mobile app's persist code messing up the state, not something that needs to be fixed in the redux library


Can't handle try-catch for importing optional dependencies

Some modules use a try-catch pattern as seen here to import an optional dependency or fallback. This does not seem to work with ember-browserify.
Example app: https://github.com/jonesetc/browserify-demo
Build output:
Build failed.
The Broccoli Plugin: [object Object] failed with:
Error: Cannot find module 'bufferutil' from '/home/kyle/code/js/browserify-demo/node_modules/ws/lib'
    at /home/kyle/code/js/browserify-demo/node_modules/ember-browserify/node_modules/browserify/node_modules/resolve/lib/async.js:46:17
    at process (/home/kyle/code/js/browserify-demo/node_modules/ember-browserify/node_modules/browserify/node_modules/resolve/lib/async.js:173:43)
    at ondir (/home/kyle/code/js/browserify-demo/node_modules/ember-browserify/node_modules/browserify/node_modules/resolve/lib/async.js:188:17)
    at load (/home/kyle/code/js/browserify-demo/node_modules/ember-browserify/node_modules/browserify/node_modules/resolve/lib/async.js:69:43)
    at onex (/home/kyle/code/js/browserify-demo/node_modules/ember-browserify/node_modules/browserify/node_modules/resolve/lib/async.js:92:31)
    at /home/kyle/code/js/browserify-demo/node_modules/ember-browserify/node_modules/browserify/node_modules/resolve/lib/async.js:22:47
    at FSReqWrap.oncomplete (fs.js:82:15)

The broccoli plugin was instantiated at:
undefined

If you manually go and update the import at node_modules/ws/lib/BufferUtil.js to skip the optional dependency and just load the fallback, the build gets past this easily. There is another one you have to update here and then the whole build works just fine.

I'm trying to understand how this addon works, so I could be far off, but I think it might have to do with this snippet. It throws an error if the package isn't found locally, regardless of if that is tolerable or not.

This is a limitation of browserify. I think there may be an option to tell it to ignore certain package names.

Any way to get around this?

You may want to experiment with browserify's ignore option. See also the discussion in https://github.com/substack/node-browserify/issues/901

The following seems to work in config/environment (mutatis mutandis):
browserify: {
  ignoreMissing: ['WNdb', 'lapack'],
}

For an addon, you can put in tests/dummy/config/environment.js and/or add a config to index.js (former to ignore during testing, the latter to ignore when installed as dependency to some other project.
I was using sentencer to test an addon, but didn't want dependent projects to have to include as it is very large. I used the above snippet in test/dummy/config/environment.js, and in ./index.js I have:
  config(env, baseConfig) {
    var browserify = baseConfig.browserify = baseConfig.browserify || {};
    var ignoreMissing = browserify.ignoreMissing = browserify.ignoreMissing || [];
    if(ignoreMissing.indexOf('sentencer') === -1) {
      ignoreMissing.push('sentencer');
    }
  }

... seems to work! :)


Bug in 'Sets new account password'

https://github.com/openware/barong/blob/80571172653c6cc9ae75bddf68b1b906fef5ed5a/app/api/v2/identity/users.rb#L267
Here it should be payload[:email] in place of params[:email]

#1357


[Submission]: Test gtardif/my-awesome-extension

Docker Hub repository name
gtardif/my-awesome-extension
Terms of services

[X] I accept the term of services

/validate


[Security] Workflow code-quality.yml is using vulnerable action wagoid/commitlint-github-action

The workflow code-quality.yml is referencing action wagoid/commitlint-github-action using references v1. However this reference is missing the commit bf83d2b35c4177779d047f464b48d9907f2c5201 which may contain fix to the some vulnerability.
The vulnerability fix that is missing by actions version could be related to:
(1) CVE fix
(2) upgrade of vulnerable dependency
(3) fix to secret leak and others.
Please consider to update the reference to the action.

Hello, @igibek! 😁
If I'm understanding you correctly, you'd like to upgrade wagoid/commitlint-github-action, used in the code-quality pipeline, from v1 to v4?
It's true that the first version of the project used v1, but it was upgraded to v4 in August of 2021: https://github.com/aryella-lacerda/react-native-accessibility-engine/pull/70
Can you confirm this is what you meant?

As stated above, this problem seems to have been addressed already. Closing!


VSTS build badge by repo does not work (with YAML definitions?)

🛑 IMPORTANT: You can get your feedback addressed faster if you use the comment section for the article in which you encountered a problem.
There is no comment section on that page
Link to article:
https://docs.microsoft.com/en-us/rest/api/vsts/build/badge/get build badge data?view=vsts-rest-4.1
Problem:

The parameter branchName is required, too.
Otherwise you get the following error:

{
"$id": "1",
"innerException": null,
"message": "Value cannot be null.\r\nParameter name: branchName",
"typeName": "System.ArgumentNullException, mscorlib",
"typeKey": "ArgumentNullException",
"errorCode": 0,
"eventId": 0
}

There is no hint where I can find the project Id. I use https://{myworkspace}.visualstudio.com/_apis/projects

Even with correct parameters I just get a setup now badge:

I think this is because I am using YAML build definitions.

@wnjenkin Can you take a look at this REST API question?

badges arent supported yet for yaml definitions https://github.com/Microsoft/vsts-agent/blob/master/docs/preview/yamlgettingstarted-features.md

If you can use the new 5.0 (preview) API, there is a new, preferred endpoint for displaying the badge that will work for yaml definitions as well:
https://docs.microsoft.com/en-us/rest/api/vsts/build/status/get?view=vsts-rest-5.0
Note that if your project is not marked "public", then the user will be required to log in to see the badge.

Thank you @moswald , this works fine!


@conda-forge-admin, please add bot automerge

Additional comment:
No response

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


bibdata refactor changes

The following changes impact this gem, as elements moved to bibdata/ext from bibitem. All elements specific to ISO must be populated in bibdata/ext; please ask me if there are any changes not covered in the following.
The grammar of ext is:
BibDataExtensionType =
        doctype?, editorialgroup, ics*, structuredidentifier

editorialgroup = element editorialgroup {
  technical-committee+, subcommittee*, workgroup*, secretariat?
}

technical-committee =
  element technical-committee { IsoWorkgroup }

subcommittee =
  element subcommittee { IsoWorkgroup }

workgroup =
  element workgroup { IsoWorkgroup }

IsoWorkgroup =
    attribute number { xsd:int }?,
    attribute type { text }?,
    text

secretariat =
  element secretariat { text }

ics = element ics {
  element code { text },
  element text { text }
}

structuredidentifier =
  element structuredidentifier {
    attribute type { text }?,
    (documentnumber, tc-documentnumber? )
}

DocumentType =
     "international-standard" | "technical-specification" |
     "technical-report" | "publicly-available-specification" |
    "international-workshop-agreement" | "guide"

The type of standard, e.g. "international-standard", is no longer set in bibitem/@type. It is now set in bibdata/ext/doctype

title-main is replaced with title[@type = 'title-main']. The lang attribute is retained.

title-intro is replaced with title[@type = 'title-intro']. The lang attribute is retained.

title-part is replaced with title[@type = 'title-part']. The lang attribute is retained.

A title[@type = 'main'] shall be added, consisting of title-main, title-intro, title-part, joined by space-endash-space. The lang attribute is retained.

editorialgroup moves to bibdata/ext.

ics moves to bibdata/ext.

The structured ISO identifier, using project-number and part-number attributes, is moved from bibitem/docidentifier to bibdata/ext/structuredidentifier. In bibitem/docidentifier, the ISO identifier given should be the identifier as it appears on the site, with the part numbers incorporated; e.g. <docidentifier type="ISO">ISO 631-1</docidentifier> vs <structuredidentifier><project-number part="1">ISO 631</project-number></structuredidentifier>

For all of these, let me know if you need XML samples; the easiest thing to do is to change the spec examples.

@opoudjis was the status moved to the BubliographicItem class?

The status is in bibitem, and there is therefore no validation of its vocabulary by enum any more in the grammar: that is flavour-specific, and it is done in the consuming gems for bibdata.

@opoudjis there is a /bibitem/note element in IsoBibliographicItem. We pass the content of the element as an argument of to_xml function. In isostandard.rnc grammar there is a definition of the biblionote which could have type and reference. Should we be able to pass type and reference to to_xml function?

Ah. I was wondering about that.
biblionote now already has the type attribute in relaton; so the type attribute must be passed in.
The reference attribute is an error: I'm removing it. I had it in there because ISO bibliographies are rendered with footnotes, if there is a draft reference. But the footnote is inserted at the isodoc level, if it detects a note entry at all: the footnote is not present in Metanorma XML.

@opoudjis if pass a note argument to to_xml function we get an xml with /bibitem/note element. Should we parse the element when create IsoBibliographicItem from xml?

I do not understand the question. Note is correctly parsed in bibitem, and isobib does not change anything about it,  now that I have removed the spurious reference attribute.

@opoudjis in IsoBibliographicItem we pass note ooption in to_xml function, not in constructor:
    def render_xml(builder, **opts)
      builder.send("iso-standard", xml_attrs(type)) do
        ...
        if opts[:note]
          builder.note("ISO DATE: #{opts[:note]}", format: 'text/plain')
        end
       ...

so it doesn't save the option in the model. Should we change the behavior?

Ah. I can see why that's done, and why we would want to inject a note into the reference. Preserve this behaviour: any note included in the options should be added to any notes already included in the screenscraped reference.
The user should also have the option of including a type for their note, although I can see that would get awkward. It would also be nice to allow there to be multiple notes added by the user, as an optional array.

Just to be clear, we want to accept personal notes in Relaton XML/YAML right? This was Relaton can be used in bibliographic managers.

You can inject notes; I'm just saying it won't cache them, because the calling client could be inserting a note for any number of transitory reasons. And if your client is a bib manager, surely it doesn't need to cache, because it is storing the results itself.


some games do not unlock achievements (RA)

I have found that some games such as mega man (nes), pepsiman (ps1) or Hugo (ps1) have functional achievements, however, after unlocking some, this is not reflected in sucess story, although both in retroachievements or retroarch, they appear as unlocked.
Update: suddenly they worked, but I had to delete the achievements of those games and then hit "update". if I gave update with the placed achievements, nothing happened. However, it is still rare since this did not happen before. I seem to have to do the same for every retroachievements game (although with figthing force and adventure island 2 I work normally).

Can you post your extension.log after a game session where the issue occurs?

https://github.com/Lacro59/playnite-successstory-plugin/commit/f123fd63714c16774a74e4bd0c7d1ab9e7b167d4
https://github.com/Lacro59/playnite-successstory-plugin/commit/d9d37068823520049e85955b946ae26a0c83bdf6
https://github.com/Lacro59/playnite-successstory-plugin/commit/be7228f0c7ecb97b98d46a377e00e5ef0b8010ff


[$250] Pay-On pay money, description in preview changed to expense after few seconds

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.63-0
Reproducible in staging?: Y
Reproducible in production?: Y
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:

Go to https://staging.new.expensify.com/home

Tap on a report

Enter an amount and tap next

Enter description

Tap pay elsewhere

Note IOU preview details

Expected Result:
When we pay money, description or merchant must be displayed without getting altered.
Actual Result:
When we pay money, description is shown in preview and after few seconds it is changed to "expense"(merchant)
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/39a96dc4-9ccd-45e2-a7a9-ec30f3b54775

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~018753cc78d651b6ba
Upwork Job ID:<PHONE_NUMBER>571158016
Last Price Increase: 2024-05-07

We think this issue might be related to the #vip-vsb.

BE is setting the merchant to Expense for pay request type we can align it in optimistic data to align to it but I think the BE should return none same as submitting request when merchant is not set.

Proposal
Please re-state the problem that we are trying to solve in this issue.
The send money request shows merchant as Expense after a few seconds.
What is the root cause of that problem?
When we create a new send money request, we always set the merchant to "(none)" since the user isn't allowed to set a merchant for sending money. After the request is completed, BE returns the merchant as "Expense".
This confuses the user because of the value inconsistency between FE and BE. It could be a BE issue, but maybe we intentionally change it since the user is not allowed to set a merchant for a send request.
What changes do you think we should make in order to solve the problem?

Set the merchant value for send money optimistically to "Expense".
https://github.com/Expensify/App/blob/a04f40e09e6a933488957b7f6300edd6290c1ebd/src/libs/actions/IOU.ts#L4819
const optimisticTransaction = TransactionUtils.buildOptimisticTransaction(amount, currency, optimisticIOUReport.reportID, comment, undefined, undefined, undefined, 'Expense');

Adding to vip-split.

Taking over as C+

First of all, we need to clarify the expectations for this case. When creating a new send money request, should we add the default merchant as "Expense"?
@shawnborton Could you help to give a thought from the UX perspective ?
🎀👀🎀 C+ reviewed
Also want to hear an option from the internal engineer

No context I'm afraid. I agree the default merchant Expense we need under the hood shouldn't take priority over a description entered. Only a user entered merchant should. 👍

It's not clear to me yet that this is a backend issue. Sounds more like a frontend thing where we want to change the logic for what to display when we have a description vs. when we don't.
Like, let's say there is no description... are we expecting "(none)" or "Expense"? I think it's the latter still, no?
I see @mountiny recently updated this from "Request" to "Expense" so maybe they have some opinion.

Proposal
Please re-state the problem that we are trying to solve in this issue.
When we pay money, description is shown in preview and after few seconds it is changed to "expense"(merchant)
What is the root cause of that problem?
https://github.com/Expensify/App/blob/7483d10e686e240d07d9fa3e43e89ffdfcda9cbd/src/components/ReportActionItem/ReportPreview.tsx#L231-L237
We prioritize displaying the merchant if it isn't empty. In this case, the BE return merchant: "Expense", the supportText turn into "Expense"
What changes do you think we should make in order to solve the problem?
If the merchant is empty or default value
-  If description is valuable, we will display description
- If description is empty, we will display default merchant
if (formattedMerchant && formattedMerchant !== "Expense") {
            return {isSupportTextHtml: false, supportText: formattedMerchant};
        }
        if (formattedDescription ?? moneyRequestComment) {
            const parsedSubtitle = new ExpensiMark().replace(formattedDescription ?? moneyRequestComment);
            return {isSupportTextHtml: !!parsedSubtitle, supportText: parsedSubtitle ? `<muted-text>${parsedSubtitle}</muted-text>` : ''};
        }
        if (formattedMerchant === "Expense") {
            return {isSupportTextHtml: false, supportText: formattedMerchant};
        }

What alternative solutions did you explore? (Optional)
NA

@DylanDylann Updated proposal
While testing my solution I see another bug, I updated proposal to add additional fix

@mountiny Could you help to check this comment?

Waiting for a confirmation from @mountiny

@mountiny any thoughts on  the comments above?

Recently we have just updated the default merchant name from Request to Expense to match the changes with the split expense.
I have tested this locally and I can confirm there is another bug @nkdengineer mentioned because I am seeing no description or merchant in the send money flow.
I think we need to:

fix the number of the transactions
update the default_merchant const to be Expense and not Request, this was missed in the split expense change
Update the support_text logic to show the description if defined and the merchant if default or partial.

@marcaaron Feel free to reassign to me if you want to focus on your projects. I am happy to see this one through to the finish since I have worked on the Split Expense project

As for this requirement, @nkdengineer's proposal looks good. Let's go with them

As for this requirement, @nkdengineer's proposal looks good. Let's go with them
🎀👀🎀 C+ reviewed

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
[@DylanDylann] The PR that introduced the bug has been identified. Link to the PR: NA This caused by BE changes
[@DylanDylann] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: NA
[@DylanDylann] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: NA
[@DylanDylann] Determine if we should create a regression test for this bug. Yes
[@DylanDylann] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.
Regression Test Proposal

Click on FAB button
Click on Pay Someone
Enter amount
Go to confirmation page
Enter description
Click "Pay Somewhere"
See that the description still remains and has not been changed

Do we agree 👍 or 👎

Regression test created and payments made in upwork. All set


Mention watch probes in release notes

@bot-gradle test and merge please.

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

OK, I've already triggered a build for you.


search in all available repositories

This might come in handy for larger setups where "access-control" to packages is realized by having a multi-stream setup.
In such a szenario, the CLI choco search works just as expected, searching via the GUI becomes cumbersome.
(searching for packages by typing in the search box should not be limited to the currently selected stream - maybe an option to "search in all streams" could be added to global settings)

will create an option to ignore source - entry points should be https://github.com/chocolatey/ChocolateyGUI/blob/develop/Source/ChocolateyGui/Services/ChocolateyService.cs#L167


GIF Memory Leak - VM: ImageIO_GIF_Data

Hi, I'm using SDWebImage on iOS 7 and am noticing a very high increase in memory usage any time a GIF is loaded from a URL. I'm not sure if it's a problem with SDWebImage or just iOS 7, as I can't find the exact like the allocations occur at but I have noticed that the problem doesn't occur when a cached version is loaded instead.
Here's a screenshot after inspecting it in Intsruments:

Same issue.
I suspect that some gif images have many image frames so that using UIImageView to render and show these frames requires a lot of memory.
Now my solution is that limit gif data size in method +(UIImage*)sd_imageWithData:(NSData*)data.
+(UIImage*)sd_imageWithData:(NSData*)data
{
    UIImage *image;
   if([data sd_isGIF] && data.length < GifLimitSize)

I created a swizzle for SDWebImage + FLanimatedImage and works great.
https://gist.github.com/exherb/1257263e350831d6c205

We have replaced our GIF component with a library from Flipboard (FLAnimatedImage). Could you try our 4.x branch (instructions) and see if it helps?

Closing. The FLAnimatedImage solution replacing our own GIF is integrated into the 4.x branch.


DataLakeServiceClient when used, with ClientCredentials, trough PrivateLink, getting self.continuation_token errors:" This request is not authorized to perform this operation"

Package Name: azure-storage-file-datalake,azure-identity
Package Version: azure-storage-file-datalake=12.17.0,azure-identity=1.15
Operating System:Linux(CentOS Linux release 8.5.2111), Windows 10
Python Version: 3.9.18  on Linux(CentOS Linux release 8.5.2111), 3.11.9 (Windows 10)

Describe the bug
DataLakeServiceClient when used, with ClientCredentials, trough PrivateLink, getting self.continuation_token errors:" This request is not authorized to perform this operation", however when allowing the DataLakeServiceClient  host IP storage account through firewall the authorization is successfull. The behaviour appears inconsistent given that using the same Service principal passed before as ClientCredentials, but through the Azure CLI:
az login --allow-no-subscriptions --service-principal -u SP1 -p SP1Secret --tenant Teant1
and listing through the command is also successful
az storage fs directory list --file-system  --account-name Test--auth-mode login
however using the below described script errors with:
File "/root/test.py", line 19, in
for container in containers:
File "/usr/local/lib/python3.9/site-packages/azure/core/paging.py", line 123, in next
return next(self._page_iterator)
File "/usr/local/lib/python3.9/site-packages/azure/core/paging.py", line 75, in next
self._response = self._get_next(self.continuation_token)
File "/usr/local/lib/python3.9/site-packages/azure/storage/blob/_models.py", line 544, in _get_next_cb
process_storage_error(error)
File "/usr/local/lib/python3.9/site-packages/azure/storage/blob/_shared/response_handlers.py", line 186, in process_storage_error
exec("raise error from None")   # pylint: disable=exec-used # nosec
File "", line 1, in
azure.core.exceptions.HttpResponseError: This request is not authorized to perform this operation.
RequestId:a6be6dd4-501e-0066-7c0f-11f005000000
Time:2024-09-27T19:00:06.9692023Z
ErrorCode:AuthorizationFailure
Content: AuthorizationFailureThis request is not authorized to perform this operation.
RequestId:a6be6dd4-501e-0066-7c0f-11f005000000
Time:2024-09-27T19:00:06.9692023Z
To Reproduce
Steps to reproduce the behavior:

Create ADLS Storage Account and create SP and secret and grant SP  Azure Storage Bob Data Contributor.
Configure network private links and run az cli connectivity validation test.
Create test.py with  the following code and run:
from azure.identity import ClientSecretCredential
from azure.storage.filedatalake import DataLakeServiceClient

Replace these values with your actual Azure AD credentials and Storage account details
tenant_id = 'YYYYYYYYYYYYYYYYYYYYYYYY'
client_id = 'XXXXXXXXXXXXXXXXXXXXXXXXXXXX'
client_secret = 'ZZZZZZZZZZZZZZZZZZZZZZZZ'
account_url = 'https://MyStorageAccount.dfs.core.windows.net/'
Authenticate using service principal
credential = ClientSecretCredential(tenant_id=tenant_id, client_id=client_id, client_secret=client_secret)
Create DataLake ServiceClient
service = DataLakeServiceClient(account_url=account_url, credential=credential)
List containers in the Blob service
file_systems= service.list_file_systems(logging_enable=True)
for file_system in file_systems:
print(file_system['name'])

Instead of listing the ADLS filesystem, just like the az cli does,  I get the error:
Traceback (most recent call last):
File "/root/test.py", line 19, in
for container in containers:
File "/usr/local/lib/python3.9/site-packages/azure/core/paging.py", line 123, in next
return next(self._page_iterator)
File "/usr/local/lib/python3.9/site-packages/azure/core/paging.py", line 75, in next
self._response = self._get_next(self.continuation_token)
File "/usr/local/lib/python3.9/site-packages/azure/storage/blob/_models.py", line 544, in _get_next_cb
process_storage_error(error)
File "/usr/local/lib/python3.9/site-packages/azure/storage/blob/_shared/response_handlers.py", line 186, in process_storage_error
exec("raise error from None")   # pylint: disable=exec-used # nosec
File "", line 1, in
azure.core.exceptions.HttpResponseError: This request is not authorized to perform this operation.
RequestId:46ec260c-d01e-006b-0559-13dd82000000
Time:2024-09-30T16:55:41.0299784Z
ErrorCode:AuthorizationFailure
Content: AuthorizationFailureThis request is not authorized to perform this operation.
RequestId:46ec260c-d01e-006b-0559-13dd82000000
Time:2024-09-30T16:55:41.0299784Z

Expected behavior
Expect that if I am able  use a Service Principal login through and list its filessystems through  Azure CLI successfully , to be able using
DataLakeServiceClient with ClientCredentials to have the same behaviour of success from the python DataLakeServiceClient , as the documentation states that it support and TokenCredentials (ClientCredentials included) :https://learn.microsoft.com/en-us/python/api/azure-storage-file-datalake/azure.storage.filedatalake.datalakeserviceclient?view=azure-python#parameters, no matter if the access is done publicly through Firewall or Privately through private endpoints.
I have tested also Azure key vault ( with a similar Private link setup) and the unlike DataLakeServiceClient the AKV ServiceClient
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Hi @elisabetao, do you have the correct RBAC roles setup on your service principle to access Storage? Your service principle will need to have one of the built-in Blob Storage RBAC roles (or a custom role with the right permissions) to access Storage using a TokenCredential with the SDK.
Azure CLI can talk to Storage without these specific roles, with the Contributor role for example, as they use that management plane to fetch the account key so they can cache and use that for Storage requests.

Hello @jalauzon-msft , as mentioned above the storage account already has  Azure Storage Bob Data Contributor. Do you see specifically a role that could create issue. Didi you get the chance to test and try to replicate on your side?
Thanks a lot


Config: update lexer to match string start

To prevent this error only match against "config"
keyword if it's the start of the string:
bpftrace -e 'struct Foo {int config;} BEGIN { $x = (struct Foo*)0; $x->config; }' -d
stdin:1:56-66: ERROR: syntax error, unexpected config
struct Foo {int config;} BEGIN { $x = (struct Foo*)0; $x->config; }

This should be fine since the config block needs
to be at the top and start the line.
Checklist

[ ] Language changes are updated in man/adoc/bpftrace.adoc and if needed in docs/reference_guide.md
[ ] User-visible and non-trivial changes updated in CHANGELOG.md
[ ] The new behaviour is covered by tests

so we'd ideally find a solution that makes them all work in user-defined types

This change still may be a good temporary solution while we work on one that covers a wider range of keywords, especially if "config" is more common.

I don't think it'd be too hard to fix this properly for all keywords: we'd just need to update any rules which should accept user-defined type/field names (such as type and postfix_expr) to accept "keywords" as well as plain idents, e.g.:
keyword:
                config     { $$ = $1; }
        |       if         { $$ = $1; }
        |       while      { $$ = $1; }
                [...]
                ;
postfix_expr:
                [...]
        |       postfix_expr DOT ident      { $$ = new ast::FieldAccess($1, $3, @2); }
        |       postfix_expr DOT keyword    { $$ = new ast::FieldAccess($1, $3, @2); }

It'd probably be cleaner to create a new rule along the lines of:
external_name: ident | keyword;

And then use that in place of ident where necessary:
postfix_expr:
                [...]
        |       postfix_expr DOT external_name      { $$ = new ast::FieldAccess($1, $3, @2); }

@ajor Great suggestion! Is something like this what you had in mind? https://github.com/bpftrace/bpftrace/pull/3004

Actually, I'm realizing we might just be able to add all the other keywords to the ident definition. All the builtins are there in addition to things like STACK_MODE
ident:
                IDENT         { $$ = $1; }
        |       BUILTIN       { $$ = $1; }
        |       CALL          { $$ = $1; }
        |       CALL_BUILTIN  { $$ = $1; }
        |       STACK_MODE    { $$ = $1; }
                ;


Overwrite mixin not remapping in forge correctly.

Mixins are being mapped to mojang srgs in development when they shouldnt.
Mixin apply failed debugify-common.mixins.json:client.mc148149.OperatingSystemMixin -> net.minecraft.util.Util$OperatingSystem:

org.spongepowered.asm.mixin.transformer.throwables.InvalidMixinException Overwrite target "m_137648_" was not located in target class net/minecraft/util/Util$OperatingSystem

[ -> MAIN Applicator Phase -> debugify-common.mixins.json:client.mc148149.OperatingSystemMixin -> Apply Methods -> (Ljava/net/URI;)V:m_137648_]

Also, when building, it seems to not generate any sort of refmap entry.
This could relate to 817bdb9
Here is my mixin
@Mixin(Util.OperatingSystem.class)
public abstract class OperatingSystemMixin {
    @Shadow
    public abstract void open(URL url);

    @Overwrite
    public void open(URI uri) {
        CompletableFuture.runAsync(() -> {
            try {
                this.open(uri.toURL());
            } catch (MalformedURLException e) {
                System.err.println("Couldn't open uri '" + uri + "'");
                e.printStackTrace();
            }
        });
    }

    @Overwrite
    public void open(File file) {
        CompletableFuture.runAsync(() -> {
            try {
                this.open(file.toURI().toURL());
            } catch (MalformedURLException e) {
                System.err.println("Couldn't open file '" + file + "'");
                e.printStackTrace();
            }
        });
    }
}

this works fine on fabric.
This seems to be an issue with the Overwrite annotation. Using the equivelent with Inject at head with cancellable works fine

Can also confirm this issue with Overwrite.
Mixin apply failed healthoverlay_common.mixins.json:PlayerTabOverlayMixin -> net.minecraft.client.gui.components.PlayerTabOverlay:

org.spongepowered.asm.mixin.transformer.throwables.InvalidMixinException Overwrite target "m_94530_" was not located in target class net/minecraft/client/gui/components/PlayerTabOverlay

[ -> MAIN Applicator Phase -> healthoverlay_common.mixins.json:PlayerTabOverlayMixin -> Apply Methods -> (Lnet/minecraft/world/scores/Objective;ILjava/lang/String;IILnet/minecraft/client/multiplayer/PlayerInfo;Lcom/mojang/blaze3d/vertex/PoseStack;)V:m_94530_]

I have the same problem too.


Bounty: An automated reproduction for issue #417

In issue #417 we see at least two distinct types of crashes: TypeError: Cannot read property 'pop' of undefined and TypeError: Cannot read property 'get' of undefined (see issue for full traces). We believe that this issue is the most common cause of crashes Ganache and ganache-cli.
We believe that these crashes share an underlying root cause, and while we have some idea of where this is happening, we don't yet have a reliable reproduction which demonstrates how ganache-core is causing this problem to occur.
Bounty acceptance criteria:

Submission must be in the form of a pull request containing one or more automated tests
Tests must fail reliably with deterministic results
Tests must exercise ganache-core to demonstrate the specific process by which ganache-core triggers these bugs in the merkle-patricia-tree module, or whichever module is ultimately responsible for the behavior observed.
Test must demonstrate the root cause of both crashes described in the description of issue #417 and mentioned above.
Submissions may make reasonable modifications to ganache-core to improve its testability for the purpose of this task, provided that they are logically equivalent to the unmodified code and that they do not impact normal runtime performance

Note that while the intention is for us to merge the submitter's PR so that the tests therein can guard against the future regression of this issue, in certain cases we may elect not to do so. If this happens, provided the above criteria are still met, the bounty will still be awarded.
Bounty details, including award amount, will be available via GitCoin once the issue is funded.

Reddit discussion on this bounty: https://www.reddit.com/r/ethdev/comments/7l5ug9/the_truffle_team_is_offering_up_a_05_eth_bounty/

@avdaredevil i see that you've claimed the issue but dont see any comments from you.  do you have a reproduction of the issue ready?  what is your plan?  let us know.
i hope you understand that, out of respect to the other community members, we can't keep your claim active for a long time unless it's clear that this is a top priority for you and you're going to solve it soon.
best,
kevin

I haven't had the chance to actually run these tests yet, but on first glance they certainly look like they quality for the bounty -- great work, @0xNPE! I'll pull & run the tests first thing tomorrow morning and report back.
In the mean time, given that it's Christmas, I expect it might be a couple of days until @owocki is able to get things sorted for you.

No problem, and no rush. Merry Christmas to the both of you!

@avdaredevil no showed, so i'm rejecting his claim.  @0xNPE feel free to claim at https://gitcoin.co/funding/details?url=https://github.com/trufflesuite/ganache-cli/issues/450

I've merged your PR, @0xNPE -- thanks so much for this! @owocki is going to clear out the invalid claim now so that you can go ahead and register your claim to receive the bounty. I expect he'll pay it out quickly once you're able to register your claim.

Also @0xNPE if you want to continue on with your work on this, the next bounty on this will be to fix the underlying problem. I'm writing that bounty up now - should be good to go in 15-30 mins.

Thanks guys, I've placed a claim on gitcoin. I'd love to work on the fix for this issue too, but I have some higher priority things to finish first. If no one has taken a stab at it in a week or so, I'll give it a go.

just posted a bounty for https://github.com/trufflesuite/ganache-cli/issues/453


Species Guess being set inconsistently

As per this question in the google group, there's confusion over why these two observations with seemingly identical histories have different species_guesses:
Observation.find_by_id(1424712).species_guess => "True Crabs"
Observation.find_by_id(1424715).species_guess => "Fiddler Crabs"
User claims the former did say 'Fiddler Crabs' for a time but changed to 'True Crabs' on its own.

Rather than fix this, I suggest we implement improvement #540

closing due to #540


Type inference from element expressions within tuple literal arguments

The Roslyn compiler allows type inference from element expressions within tuple literal arguments.
F((1, null), (default, "")); // ok: F<int, string>()

static void F<T, U>((T, U) x, (T, U) y) { }

However, it looks like 12.6.3 Type inference only covers inference from the tuple argument type, not from the tuple element expressions.

FWIW, for collection expressions there is a similar requirement, and the collection expressions proposal extracts the rules from the first phase to an input type inference section to allow recursive inference from collection expression elements.


Possible conflict with Alpine 3.13 and OpenJDK

Hello. First, I have to say that this Minecraft Docker is so handy and well documented! Thanks for the time you put in!
I recently had trouble running a Paper server with the floodgate-bukkit plugin in the latest Java version (Java 11, Alpine) of docker-minecraft-server.
I opened a ticket in the Floodgate repo and it seems that it could be an upstream issue with the Docker container as opposed to the Floodgate plugin itself.
I was eventually able to get the plugin to work without crashing when I switched to the multiarch (Ubuntu) image.
It was suggested that I open a ticket on this repo to see if there are potential issues with the container. Hopefully this isn't a duplicate issue.
Here's a link to the ticket I've opened on the Floodgate repo: https://github.com/GeyserMC/Floodgate/issues/120
Cheers!

In fact, just after posting this ticket, I see another that may be related, but am not sure: https://github.com/itzg/docker-minecraft-server/issues/783

Yes that’s a related issue and the root cause is further upstream
https://github.com/AdoptOpenJDK/openjdk-docker/issues/520

Okay. Thanks much for verifying.


Old state

Description

Fixes

Fixes #XX
Fixes #XX

Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Refactoring (internal implementation changes)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (no changes to the code)
[ ] CI change (changes to CI workflows, packages, templates, etc.)
[ ] Version changes (changes to the package or dependency versions)

Testing

N/A
Other Pull Request Checklist
Please confirm the PR meets the following requirements.

[ ] Code changes pass pre-commit (e.g., ruff, mypy, etc.) via pre-commit run --all-files.
[ ] Tests have been added to show the fix is effective or that the new feature works.
[ ] New and existing unit tests pass locally with the changes.
[ ] Docs have been updated and reviewed if relevant.

revert to old shell.nix