Cleared mnemonic and root key still allows addresses to be generated

Steps to reproduce

Generate a random mnemonic
Delete the mnemonic value (should show feedback 'Blank mnemonic')
Delete the BIP32 root key value (should show feedback 'Invalid root key')
Select one of the tabs in the Derivation Path section
Note the following incorrect behaviors:

the BIP32 Root Key value is magically populated with a valid value
addresses table is populated with values

The expected behavior when there is no mnemonic or root key is to show an error and not generate any addresses.
I suspect this is due to some incorrect caching of the root key value. Smells like a code refactor to remove 'global' variables in index.js may be a good use of time.

See https://github.com/iancoleman/bip39/commit/ed6d9d3905aa581f1a76d37d601c6be39954d1ff - Clear old seed when mnemonic is changed


Allow rake task descriptions in describe_class

Currently, describing a task via describe 'namespace:task' will generate a warning. It'd be nice to allow type: :rake or similar metadata in order to reduce noise when testing rake tasks (yes, some people do that).

Have you considered disabling the cop for whatever your spec/rake directory is?
Also I test my rake tasks too heh. I actually just extract the code to some callable object though so I don't have to go through the madness of like Rake.application['my_task'].invoke! or whatever. Rake should just be a thin wrapper

I ended up just ignoring the whole tasks dir, maybe it really isn't worth the time and trouble. Thanks for the fast response.


Individual app proxy?

I have spent the last hour messing around with the proxy'ing for an app but for the life of me I cannot get it to proxy traffic into my Burp proxy on my desktop. What am I missing? I enter in the proxy details on the local network, and the "Add proxy" button doesn't do anything, no indication of whether or not anything actually happened. Burp is running and sees no traffic. The HTTP hook however sees requests going out. What am I doing wrong?

The proxy through Inspeckage does not actually work on some applications that use Xamarin and Apache Cordova. In these cases you should try the proxy in the android wifi options.


Ollama integration tests fix

Disable tests by default
Improve Testcontainers configuration

this helps :)

@tzolov yeah, sorry, after testing the rebase I forgot to switch that off again. The Testcontainers-based Ollama tests should run ok as part of the GitHub Actions workflow, so we could remove the disabling logic. The error that happened was actually because the tests were running on native Ollama.


build(deps): Bump github.com/canonical/edgex-snap-hooks/v2 from 2.2.0-beta.5 to 2.2.0 in /snap/local/hooks

Bumps github.com/canonical/edgex-snap-hooks/v2 from 2.2.0-beta.5 to 2.2.0.
Signed-off-by: Mengyi Wang<EMAIL_ADDRESS>

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/device-mqtt-go/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:

[x] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[x] I am not introducing a new dependency (add notes below if you are)
[x] I have added unit tests for the new feature or bug fix (if not, why?)
[x] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[ ] I have opened a PR for the related docs change (if not, why?)

Testing Instructions

New Dependency Instructions (If applicable)

Codecov Report

Merging #380 (5146f07) into main (ab5f304) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #380   +/-   ##
=======================================
  Coverage   21.34%   21.34%
=======================================
  Files           5        5
  Lines         492      492
=======================================
  Hits          105      105
  Misses        374      374
  Partials       13       13

Continue to review full report at Codecov.

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


CTRL+F search highlight color is not inline with theme

The default Firefox search highlight color is not matching the nord color scheme, seems like it can be modified by updating the textSelectBackgroundAttention field. See: https://support.mozilla.org/en-US/questions/916946.

I just checked the Firefox theme docs and it seems that the official method of changing the highlight color of the Ctrl+F bottom bar field is the "toolbar_field_highlight" field in manifest.json, and that is in fact set to #5e81ac, which is a color in the Nord Theme. However, perhaps due to the new Proton design, the highlight color of the Ctrl+F field is not #5e81ac even though it is set to that color.
The ui.textSelectBackgroundAttention field is a field in the Firefox about:config page, and I am pretty sure it is incompatible with the manifest.json theme type, unless evidence determines otherwise. Therefore I cannot use that field to change the Ctrl+F highlight color.
I will leave this issue open for any comments, and close if after a week if there are no other comments.
See the following image for reference about the method of changing the Ctrl+F bottom bar field highlight color.


Make model extra functional.

@andrewMacmurray I was just having fun doing this tbh. Do you think it is too much?

@ivanmauricio let's go over this tomorrow, I'm wondering if with the events it might just add an extra layer of complexity that's not actually adding that much to it.
With the postcode, I've actually had a crack at that too and think it might be good to represent it in a different union type instead of a Maybe because we actually want to be able to render the data when the postcode is invalid but Maybe doesn't let us do that
something like
type Postcode
    = Invalid String
     | Valid String


Azure Fails with "Invoking Azure CLI failed with the following error: ERROR: Please run 'az login' to setup account"

Hi,
I'm trying to get a Terratest with Azure working and I'm following the example at https://github.com/gruntwork-io/terratest/tree/master/examples/terraform-azure-example.  I've added environment variables for ARM_CLIENT_ID, ARM_CLIENT_SECRET, ARM_SUBSCRIPTION_ID, ARM_TENANT_ID.
The process appears to work, I can see resources being created and destroyed in Azure but the final result appears to fail with no obvious underlying cause.
The error message is Received unexpected error:                                 Invoking Azure CLI failed with the following error: ERROR: Please run 'az login' to setup account.
A full sanitised log is attached.
I'd appreciate any assistance you can give.
terratest_log.txt

Hi take a look at https://github.com/TsuyoshiUshio/terratest/tree/feature/aks/examples/terraform-azure-aks-example#prerequisite-setup-azure-cli-access, which is a more complete setup guide for Azure terratest. This is a contribution currently in PR review that is slated to merge shortly.

I fixed this by adding ARM_SUBSCRIPTION_ID, ARM_CLIENT_ID, and ARM_CLIENT_SECRET environment variables inside the script and removing az login completely.

Hi All, I am facing the same issue but with terragrunt. I have checked all the environment variables are already set.

Can you file a new issue in terragrunt with a link to this issue? Thanks!

I fixed this by adding ARM_SUBSCRIPTION_ID, ARM_CLIENT_ID, and ARM_CLIENT_SECRET environment variables inside the script and removing az login completely.

Hello, I am running into exact same issue. I am trying to understand where to add these environment variables in the script. Could you please share a snippet of your code for this part?

I am trying to understand where to add these environment variables in the script.

We recommend adding these in outside the go test, in the test runner. This could be either:

Your CI script. E.g., see how we set it up for the github action from the secrets: https://github.com/gruntwork-io/terratest/blob/master/.github/workflows/ci.yml#L75

A helper script that knows how to configure these statically that you run prior to go test (see https://github.com/gruntwork-io/terratest/tree/master/examples/azure#review-environment-variables)


Fix Typo - "SubjectAccessReviewStatus" -> "status"

"SubjectAccessReview" is the value of the "kind" field of the request and response. The webhook has to fill in the "status" field to indicate authorization passed or failed.
For consistency highlight spec better using back ticks

This change is

/lgtm
/approve
...didn't I just approve another PR for this yesterday?

yep that was for authentication @lavalamp :) This one is for authorization. i was reviewing the authorization today when writing a webhook for it

Deploy preview ready!
Built with commit 6e1c227e40de491541dbb3d893ef168b4940ccd7
https://deploy-preview-4527--kubernetes-io-master-staging.netlify.com

Ah, thanks, now it makes sense :)
On Thu, Jul 27, 2017 at 7:32 PM, Davanum Srinivas<EMAIL_ADDRESS>wrote:

yep that was for authentication @lavalamp https://github.com/lavalamp
:) This one is for authorization. i was reviewing the authorization today
when writing a webhook for it
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/kubernetes.github.io/pull/4527#issuecomment-318539746,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAnglscgKuXn3RVk6wBxKP56SMuJO9Tlks5sSUhAgaJpZM4OlaGU
.


Navigation bar not updating

Describe the bug:
After navigating to a particular route by clicking an an attribute element, the navigation bar doesn't update. For example if I click Click in HTML the navigation bar must show abc.com/sample-route but rather it remains the same as original (Before navigation) abc.com
Link to the blitz that caused the error
https://stackblitz.com/edit/node-demo-app?file=index.js
Steps to reproduce:

Go to Above provided link of my stackblitz project
Type node index.js in the terminal
Let the project load, after the loading is complete click Next button in preview screen.
Observe the navigation bar it remains the same as node-demo-app--3000.local.webcontainer.io instead of node-demo-app--3000.local.webcontainer.io/new

Expected behavior:
After clicking the Next button, navigation bar must show node-demo-app--3000.local.webcontainer.io/new
Version of webcontainer:
Hash: 70a1ff62a0f0131ac0df3c30faf487fef744ac90
Desktop (please complete the following information):
Browser name  = Chrome
Full version  = 91.0.4472.124
Major version = 91
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/91.0.4472.124 Safari/537.36
performance.memory = {
  "totalJSHeapSize": 90342262,
  "usedJSHeapSize": 86665498,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}

Yes, that's correct. It's on our radar and roadmap to update the URL bar properly in the preview window. For now, I suggest to just open the URL in another browser tab and the URL bar will update. This is currently only a limitation of the preview window right in the editor.

I also experienced the preview bar's URL not updating which was very confusing.
I am adding my experience as a data point
Steps to reproduce:

Visit the following stackblitz URL:

https://stackblitz.com/edit/github-de7jbv?file=src/App.tsx

Click on one of the links on the page, and observe that the preview pane's URL is not updating.

https://user-images.githubusercontent.com/6743796/144726779-9fb2a554-ec2c-4b0b-a2f6-a229d79f9a0b.mp4
Expected behavior

clicking a link should update the URL
clicking a link should update the URL and if I the refresh icon OR hit enter on the preview pane's URL bar it should reload the page and retain the URL

Hey! Yes, as I mentioned, this is not a bug strictly speaking and rather a limitation of the URL bar for the preview window. I'd suggest to open the page in a separate browser tab / window and then you can just rely on the browser's built-in address bar, at least for the time being. We are working on another generation of the editor which comes with a whole bunch of improvements.

@d3lm - exciting! Out of pure curiosity (no pressure & not demanding anything), do you know when that new editor experience will release?
I have been enjoying stackblitz 💙 a lot lately & can't wait to see what's in store

Glad to hear you've been enjoying it 😍
I can't promise an exact date but the plans are Q1 / Q2 2022.

Duplicate of #611 .


add throttle plugin

Q
A

Bug fix?
no

New feature?
yes

BC breaks?
no

Deprecations?
no

Related tickets
mentioned in https://github.com/php-http/HttplugBundle/pull/460

License
MIT

What's in this PR?
Add documentations for throttle plugin

thanks!


New version: Microsoft.Teams.Exploration version <IP_ADDRESS>211

ARP entries are correct.\r\n\r\n#### Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 492
Microsoft Reviewers: Open in CodeFlow

/rerun-pipelines-please


[Improvement] Way to hide a pokemon from the map after you have captured it

Checklist

[x] I have read the FAQ
[x] I have checked if this issue already exists (both open and closed issues)
[x] I am using the most recent version of pogom from the master branch
[ ] If this is an installation issue, I am posting my OS, OS version, python version, pip version
[ ] I'm posting the complete log
[ ] I'm using a descriptive title for this issue

My Issue
Currently if you are collecting Pidgeys and going through a certain area that is filled with them, it's difficult to tell what ones you have captured.  It would be nice to select the pidgey from the map and click on 'hide' or something like that.

If you add this feature, it would be nice to somewhere have a "unhide all".


Add autoscaler setup CLI and docs

Setup autoscaler on cluster
Update docs

@aluminiumgeek can you check why build failed here.

@aluminiumgeek please can you look at this asap please 😄

Build is failing because the PR is compared to the code in my own repo (it was created before I've got write access to this repo). I'll close this one and open the new one

The code is already in master


feat: add Node API for packaging with more options

This makes the package more usable by adding a Node API that allows setting the entry for the executable. It also allows selecting the package manager, and skip deduplication if it is not needed.
I moved the code from index.mts to lib.mts, so that I can export a function that's usable when imported.
Fixes #13
Fixes #14

We had something like that at some point, but then I decided to give users only one way to interact with @radically-straightforward/package to make things simpler. If necessary, one could always shell out from within JavaScript. That’s what we do in the tests, for example.
Do you have an use case in which this wouldn’t be enough?

But the simplicity doesn't solve the problem. I have an important use case. I distribute executables for bootstrapping the environments without needing to install Nodejs.
https://github.com/aminya/setup-cpp

A package created with @radically-straightforward/package includes Node.js itself. Users of the package don’t need to have Node.js installed.
The API you’re proposing is for developers using @radically-straightforward/package to be able create packages from JavaScript programs instead of the command line.
I’m afraid I don’t see the connection between these two.
In any case, I think that it’s better to support only one way to interact with @radically-straightforward/package—less stuff to maintain is better.
With that, I really appreciate your contribution but I’ll close the pull request for now.


Clobber all files in generated directory

@rupurt Please rebase on top of master. This will go out in 1.0.2.

@justin808 done


Inconsistent assignment of function type with optional fields

Bug Report

🔎 Search Terms
Generics
🕗 Version & Regression Information
This is the behavior in every version I tried.
⏯ Playground Link
Playground link
💻 Code
export type T1 = {
    readonly x: string
}

export type T2 = {
    readonly x?: string
}

type F1 = (t: T1) => void
type F2 = (t: T2) => void

type F3 =  <T>(t: T) => void

declare const x: F1
const y: F3 = x

declare const v: F2
const w: F3 = v

🙁 Actual behavior
The assignment of y doesn't compile with:
Type 'F1' is not assignable to type 'F3'.
  Types of parameters 't' and 't' are incompatible.
    Type 'T' is not assignable to type 'T1'.

However, the assignment of w compiles fine.
🙂 Expected behavior
I would expect both to be consistent. Furthermore, I would expect both assignments to compile, since F3 essentially expects any.

Note that if I change F3 to type F3 = <T extends T1>(t: T) => void, it compiles fine.

This is correct and intended behavior. The assignability is checked here by comparing the argument type to the implied constraint of the type parameter, which here is { }, and { } is assignable to an all-optional type but not a type with a required property.
Different code doing different things is not inconsistency, it's how you're able to write different programs, and being able to write different programs is not a defect.

This is correct and intended behavior. The assignability is checked here by comparing the argument type to the implied constraint of the type parameter, which here is { }, and { } is assignable to an all-optional type but not a type with a required property.

Ah, I didn't know that given a generic parameter, it was implicitly constrained to {}. My expectation was that, like a generic function function foo<T>(t: T) { ... }, T could be correctly bound to either T1 or T2.

Different code doing different things is not inconsistency, it's how you're able to write different programs, and being able to write different programs is not a defect.

No need to condescend.

Apologies it came across that way; not my intent.


Cil devel build tag

closes #304 by removing CIL repos from the DEVEL_BUILD. This is due to backward incompatibility of the structure of the CIL package from version 19.10 on.

closes #309

@paskino, if I'm not mistaken this PR does two things:

Add extra CIL builds to the Travis matrices
Removes code duplication by putting CPPi tags and URLs outside of the if DEVEL_BUILD...ELSE logic, since they were the same anyway.

Is that correct? If so, I'm fine with this.

That's right.


supabase start doesn't pull images if they don't exist

Bug report
Describe the bug
After upgrading the CLI and running supabase start, the command fails with an error saying "no such image".
This is the exact error message:
Error restoring branch main: Branch was not dumped.
Failed to restart storage-api: Error response from daemon: No such container: supabase_storage_example-project
Error: Error response from daemon: No such image: public.ecr.aws/t3w2s2c9/storage-api:v0.20.1

I tried with a clean project as well (delete supabase/ directory, run supabase init). Same error.
I started manually pulling the images one by one, re-running supabase-start, getting a new error message for which image next to pull. So the first image it errored with wasn't actually storage-api. The first image it errored with was gotrue, if I remember correctly.
To Reproduce

Upgrade CLI to v1.5.2 (brew upgrade supabase)
Run supabase stop (if the development environment is still running)
Run supabase start
Realize there are some syntax errors in seed.sql which causes the command to fail. Fix those, then re-run supabase start
Witness "no such image" error

Expected behavior
The CLI would automatically pull missing images, like I remember it doing initially when I started using it.
System information

OS: macOS

Additional context
Could the bug have anything to do with any of the following things?

I had supabase running while upgrading?
When running supabase start after upgrading, there were syntax errors in seed.sql which caused the command to bail out early. Perhaps some state was stored somewhere and not invalidated which caused problems when re-running the command?

It's a bad release on my part. Should be fixed in 1.5.3

Ok! Thanks for the quick response!


Warn about browser bugs in (ok)lch rendering

Requested on social media:

https://social.vasilis.nl/@vasilis/110841553733684928

Browser bugs to track:

chromium
webkit
mozilla

I think this might be important, and we may even want to set display-p3 as the default. The current implementations of (ok)lab/lch colors are so bad, they're almost unusable. (Why did browsers ship it like this?)

Draft Proposal
https://www.figma.com/file/JtQTGIpwoaOcqcWfsoeOk1/Browser-bugs-%26-other-warnings?type=design&node-id=0%3A1&mode=design&t=RuE2m0Xo75cF0fGM-1

@mirisuzanne @stacyk
Here are the warning-related Figma file links.
Note: I don't know why Figma is suddenly displaying green lines around various bits of the mockups, and I haven't found the way to turn that view off yet.
Out of Gamut + Format Groups
https://www.figma.com/file/JtQTGIpwoaOcqcWfsoeOk1/Browser-bugs-%26-other-warnings?type=design&node-id=83%3A11&mode=design&t=4CwzTDrU2fakSiQ4-1
Incorrect implementation + Expanded Color Issue Info
https://www.figma.com/file/JtQTGIpwoaOcqcWfsoeOk1/Browser-bugs-%26-other-warnings?type=design&node-id=12%3A138&mode=design&t=4CwzTDrU2fakSiQ4-1
Background Alpha Alert + Collapsed Color Issues
https://www.figma.com/file/JtQTGIpwoaOcqcWfsoeOk1/Browser-bugs-%26-other-warnings?type=design&node-id=12%3A243&mode=design&t=4CwzTDrU2fakSiQ4-1
Foreground Alpha Alert
https://www.figma.com/file/JtQTGIpwoaOcqcWfsoeOk1/Browser-bugs-%26-other-warnings?type=design&node-id=12%3A348&mode=design&t=4CwzTDrU2fakSiQ4-1
Mobile Alerts + Color Issue Info
https://www.figma.com/file/JtQTGIpwoaOcqcWfsoeOk1/Browser-bugs-%26-other-warnings?type=design&node-id=83%3A10&mode=design&t=4CwzTDrU2fakSiQ4-1
Mobile Alerts + Color Issue Info 2
https://www.figma.com/file/JtQTGIpwoaOcqcWfsoeOk1/Browser-bugs-%26-other-warnings?type=design&node-id=83%3A11&mode=design&t=4CwzTDrU2fakSiQ4-1

Left some comments on the designs. It seems like we should maybe break this out into smaller stories for the specific issues and warnings.

Acceptance criteria-

If an unbounded color space is selected
And a user has a color selected that is out of gamut in the sRGB gamut
Then, show the warning bar.

I removed myself but I didn't assign anyone else (yet, wasn't sure what is left on 146, or if that matters here).

@stacyk I assigned you because I think #146 is waiting for you to re-review.


Target gnuplot instead of Maple in graphing tests

Many of our "unit" tests generate plots instead of checking assertions. Right now they generate Maple input files. The goal here is to generate gnuplot input files instead, since gnuplot is open source and generates nicer plots.
About gnuplot: http://www.gnuplot.info/

Done in ab142af2ad59d1d5e21ca436a81426187fceb9c2.


Ensure tomcat native installs and starts correctly

Fix #115
Fix #114

Apache tomcat does not distribute native archives, so we can only test this on JWS

Apache tomcat does not distribute native archives, so we can only test this on JWS

I was thinking about adding a feature to download the sources for tomcat-native from the ASF to build and test it with the Apache Tomcat distribution as well. Then we could test it outside of JWS as well, if we wanted.


State value to set for "nuki-extended.0.xxxxx" has to be type "string" but received type "number"

Hello and good morning, just checked my log and found this.
Adapterversion 2.4.0
Best regards

Ich hätte auch noch welche (sofern nicht schon oben enthalten)
`

nuki-extended.0
5720
2022-07-24 05:52:42.149
info
Listening for Nuki events on port 51989.

nuki-extended.0
5720
2022-07-24 05:52:41.797
info
State value to set for "nuki-extended.0.bridges.bridge.scanResults.0.rssi" has to be type "string" but received type "number"

nuki-extended.0
5720
2022-07-24 05:52:41.796
info
State value to set for "nuki-extended.0.bridges.bridge.uptime" has to be type "string" but received type "number"

nuki-extended.0
5720
2022-07-24 05:52:41.386
info
State value to set for "nuki-extended.0.smartlocks.haustür.state.lastStateUpdate" has to be type "string" but received type "number"

nuki-extended.0
5720
2022-07-24 05:52:41.256
info
State value to set for "nuki-extended.0.smartlocks.haustür.state.batteryCharge" has to be type "string" but received type "number"

nuki-extended.0
5720
2022-07-24 05:52:40.856
info
Polling Nuki Web API with a frequency of 60s.

nuki-extended.0
5720
2022-07-24 05:52:40.713
info
starting. Version 2.6.5 in C:/iobroker/GLT/node_modules/iobroker.nuki-extended, node: v14.16.0, js-controller: 4.0.23

`

Hier auch noch einer: "nuki-extended.0.smartlocks.wohnungstür._ACTION.LOCK"
`

javascript.0
2023-09-14 19:19:34.266
warn
You are assigning a boolean to the state "nuki-extended.0.smartlocks.wohnungstür._ACTION.LOCK" which expects a number. Please fix your code to use a number or change the state type to boolean. This warning might become an error in future versions.

`


Mbed CLI does not handle multiple output folders

mbed-cli will fail to link if code is compiled to multiple different output folders.
For. ex.
running
mbed compile --build xyz

mbed compile

Will fail to link as it also tries to link everything under xyz/ folder also.
Maybe a simple fix would be that as part of --build , the tools generate .mbedignore file in the root of the  folder.

Internal Jira reference: https://jira.arm.com/browse/MBOCUSTRIA-1158

I have no use case for this, just noticed this behaviour. I consider this quite trivial-level.

Thank you for raising this issue. Please note we have updated our policies and
now only defects should be raised directly in GitHub. Going forward questions and
enhancements will be considered in our forums, https://forums.mbed.com/ . If this
issue is still relevant please re-raise it there.
This GitHub issue will now be closed.

Apologies, this was closed in error by the automated bot - re-opening.

Internal Jira reference: https://jira.arm.com/browse/IOTBTOOL-462


gqlgen: gqlgen.yml mutation schema addition is ignored, mutation resolvers not generated as a result

A new project with ent and gqlgen doesn't create the additional file to ent.graphql.
I have made this project public, temporarily
https://code.icod.de/dalu/ka/src/commit/9f116854173685e7edc891752b6e086a9ed3dad8

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
When you have a gqlgen.yml file with
# schema tells gqlgen when the GraphQL schema is located.
schema:
  - ent.graphql
  - ka.graphql

on top
the ka.graphql is not generated. It should hold the mutations.
Instead the mutations are placed in ent.graphql.
Because the ka.graphql is missing, the mutation resolver file is never generated, for ka.resolvers.go
Expected Behavior 🤔
ka.graphql should be generated and the mutation resolvers as well.
Alternatively if no ka.graphql is generated, the mutation resolvers should be created in ent.resolver.go
Steps to Reproduce 🕹
Init a new repo with ent and graphql, add mutationcreate and mutationupdate annotations, generate
https://code.icod.de/dalu/ka/src/commit/9f116854173685e7edc891752b6e086a9ed3dad8
Your Environment 🌎

Tech
Version

Go
1.23.1

Ent
0.14.1

Database
MySQL

Driver
https://github.com/go-sql-driver/mysql

Even with https://github.com/a8m/ent-graphql-example
and go get -u
and upgrading everything possible to the latest version
and removing *.graphql and *.resolvers.go
the extra files aren't generated and no mutations and mutation resolvers are generated.

I'm going to make the remote repo private soon.
I am lucky to have had old generated graphql mutation definitions, so I just copy pasted and adjusted them.
They look something like
type Mutation {
    # The input and the output are types generated by Ent.
    createProfile(input: CreateProfileInput!): Profile
    updateProfile(id:ID!, input: UpdateProfileInput!): Profile

    createPost(input: CreatePostInput!): Post
    updatePost(id:ID!, input: UpdatePostInput!): Post
    deletePost(id:ID!): Boolean

    createCategory(input: CreateCategoryInput!): Category
    updateCategory(id:ID!, input: UpdateCategoryInput!): Category
    deleteCategory(id:ID!): Boolean
}

I don't remember if this is supposed to be created by hand or not.
If yes, this issue can be closed.

I don't think ent is being worked on anymore, probably too busy with their atlas


Specifying a root dir for openfl resources

When targeting html5 if an openfl application is in a different directory to the server root it seems it cant load the resources.
I found this: https://community.openfl.org/t/set-root-url-for-resources-on-html5-backend/580
But couldnt see how to actually use it... strangely a previous application works fine in a server sub dir, so its possible this is a regression of some kind. Test app:
var data:BitmapData = openfl.Assets.getBitmapData("img/test.png");
var bitmap:Bitmap = new Bitmap(data);
addChild(bitmap);

Cheers,
Ian

Hoped the following would fix this, but no such luck (for me at least).
https://github.com/openfl/lime/pull/1342

Does this help?
https://github.com/openfl/lime/commit/d0f15395a564470e89b333e4d919272c03072b4b

Sure did!


Unable to edit values for Redshift driver properties

When I try to create a new connection with Redshift, I can't set values for ssl and sslfactory. I can only select ? from the dropdown menu.
I also tried to connect with PostgreSQL driver. I can connect fine, but I can't open schemas, because pg_catalog.pg_authid table is missing.

Redshift driver seems to be poorly configured. It says that the only valid value for all properties is "?".
I'll add a workaround in the next version (will make all properties with predefined values editable).
Thanks for report.
(ph_authid issues is a duplicate of  #271)


create a fibonacci number generator function

Fixes #392
Review of colleague's PR #246
Changes proposed in this PR: make codes more readable
-Initial create the Fibonacci number generator function

Also, very sorry for the delay in re-reviewing.

@hsy3418 I saw you have removed the Merge Me label and added the Approved By Colleague label just two days ago. This was approved by @camh-anz almost a month ago now, which means you are able to merge this to master. As Cam also mentioned above remember to squash merge. Please go ahead and do this when you can as this has been sitting stale for a long time now.

Please squash merge when you merge this PR so the fixup commits are merged together with the original code commit. This maintains a clean history without the fixups requested during review making it into the master branch.


Unable to access windows paths via network

Version
Microsoft Windows [Version 10.0.22000.556]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>-microsoft-standard-WSL2
Distro Version
Arch Linux
Other Software
Nvim: v0.6.0
node: 17.19.1
Repro Steps
attempt to install language server packages / run language server packages/ attempt to execute via a UNC WSL path in WSL (\wsl.localhost\Arch\mnt\c\rest-of-path[dosent matter]) you get errors saying you do not have access. things like node dont know how to handle it and just error out as if they cant find it. example:
pyright (failed)
        internal/modules/cjs/loader.js:818
          throw err;
          ^

        Error: Cannot find module '\\wsl.localhost\Arch\mnt\c\Users\storm\scoop\apps\nvm\current\nodejs\nodejs\node_modules\npm\bin\npm-cli.js'
            at Function.Module._resolveFilename (internal/modules/cjs/loader.js:815:15)
            at Function.Module._load (internal/modules/cjs/loader.js:667:27)
            at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:60:12)
            at internal/main/run_main_module.js:17:47 {
          code: 'MODULE_NOT_FOUND',
          requireStack: []
        }
        spawn: npm failed with exit code 1.

Expected Behavior
all execution from within wsl should be allowed to access local network addresses (UNC WSL or not) especially within its own machine....
Actual Behavior
programs cannot access the url they are given as windows dosent allow access to the network location. they should either be given a WSL path or be allowed access to the path.
Diagnostic Logs
No response

Thanks for reporting the issue @Lite5h4dow.
Based on the error it looks like a linux program is trying to access a Windows path \wsl.localhost, which doesn't make sense in Linux.
I'm guessing that this project was originally configured inside WSL, and that you're now trying to access it from Windows, which can't work since paths are different.

No it's neovide running in wsl mode.
On Tue, 12 Apr 2022, 22:23 Blue, @.***> wrote:

Thanks for reporting the issue @Lite5h4dow https://github.com/Lite5h4dow
.
Based on the error it looks like a linux program is trying to access a
Windows path \wsl.localhost, which doesn't make sense in Linux.
I'm guessing that this project was originally configured inside WSL, and
that you're now trying to access it from Windows, which can't work since
paths are different.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/WSL/issues/8278#issuecomment-1097233664,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEJZCGOCUOZNBU2M3WR57QTVEXSV7ANCNFSM5THL7PZA
.
You are receiving this because you were mentioned.Message ID:
@.***>

why close? i access this problem too


Prisma creates incorrect relations if a bogus relation name is set

Describe the bug
If the relation name is set to the name of an existing automatically-generated relation, prisma will generate a CRUD schema with incorrect relations types.
To Reproduce
Create a new service with the following datamodel:
type A {
    field: Int
    b: B
}

type B {
    field: Int
    a: A
    c: C @relation(name: "AToB")
}

type C {
    field: Int
    b: B @relation(name: "AToB")
}

Prisma will associate C.b with A instead of B. All input types and output types will be generated according to this incorrect association.
Attempting to create a node of C with an associated node of type B leads to an internal server error:
mutation test1 {
  createC(data: { field: 10, b: { create: { field: 15 } } }) {
    field
    b {
      field
    }
  }
}

Expected behavior
It should not be possible to create such a relation, the name should be reserved and the creation should fail with an error.

This was fixed by https://github.com/prisma/prisma/pull/3971


Use viem instead of ethers for EVM stuff

Been reading EVM -> AHP bridge and seems that implementation is using ethers.
As there is nothing wrong with ethers, there are major reasons why to use viem

@vikiival Thanks for the suggestion! We will add this to the pipeline and try to optimize this!
With kind regards,
Team ParaSpell


Add dataset: old_bailey_proceedings_online

A URL for this dataset
https://www.dhi.ac.uk/data/oldbailey
Dataset description

The dataset consists of 2,163 transcriptions of the Proceedings and 475 Ordinary's Accounts marked up in TEI-XML, and contains some documentation covering the data structure and variables. Each Proceedings file represents one session of the court (1674-1913), and each Ordinary's Account file represents a single pamphlet (1676-1772).

This dataset represents an interesting text corpus for a number of reasons:

it covers a long time period
the text data represents court transcriptions of court proceedings. As a result, the text in the corpus can be considered somewhat closer to spoken language for the time period and also represents language use from a broader range of speakers i.e. includes more working-class speech compared to parliamentary corpora (particularly for the earlier periods).

Whilst relatively small, I think this dataset could be a useful component of a training dataset for training historic language models.
Dataset modality
Text
Dataset licence
Creative Commons Attribution 4.0 International
Other licence
No response
How can you access this data
As a download from a repository/website
Confirm the dataset has an open licence

[X] To the best of my knowledge, this dataset is accessible via an open licence

Contact details for data custodian
No response

#self-assign

@davanstrien For this dataset, what would be ideally looking to have? There's a lot of extraneous information in these XMLs. Also, what task would this text be used for? There's some NER in the passages
<interp inst="OA16760517n12-2" type="given" value="EDWARD"/>Edward Russel</persName> that suffered, were brought
to this untimely and ignominious End, by the means and seducements of this unhappy Woman. For they together with one A.
 M. going after the former Sessions to a Gentlemans House, to sollicite and engage his Interest, in order to the obtaining of a
Reprieve for a Woman that past for one of their Wives, and was then under Condemnation, they chanced to spie the Maid a
scowring a very considerable quantity of Plate, the glittering sight whereof so much affected them, that when they came back
to <placeName id="OA16760517-geo-8">Newgate</placeName>, to give an account of their business, amongst other
discourse, they mentioned what abundance of Plate they saw. And will you only see it?

but what else would this dataset be used for?

@davanstrien For this dataset, what would be ideally looking to have? There's a lot of extraneous information in these XMLs. Also, what task would this text be used for? There's some NER in the passages

I think there are a few potential use cases would be for training and evaluating language models. For this, I think the main features that could be useful to have is the text and associated dates.
NER would also be a great use case for this, especially as it is 'clean' text rather than OCR which makes it easier to perform an evaluation of the performance of OCR methods across different time periods without also introducing the issue of OCR noise (which is likely to be correlated to time period too). I  think adding NER fields from this text would also be great but I realise this might be a little bit trickier to implement.
For this particular dataset, I think we can also emphasise in the Datacard that we welcome suggestions/pull requests for additional configurations. If you prefer to implement only the simpler ones for now that is also completely fine of course. There may also be some extra XML support being added to the datasets library which could make working with this kind of data simpler (cc @albertvillanova)
also cc'ing @clancyoftheoverflow @cakiki who may have thoughts on this too

For this, I think the main features that could be useful to have is the text and associated dates.

Sounds good! I can leave the dates as is and they can be parsed by whoever is using it

I think adding NER fields from this text would also be great but I realise this might be a little bit trickier to implement.

It shouldn't be too difficult to implement. I will try it out and share them to my HF account and then transfer it to bigLAM after someone's had a chance to look at it

For this, I think the main features that could be useful to have is the text and associated dates.

Sounds good! I can leave the dates as is and they can be parsed by whoever is using it

I think adding NER fields from this text would also be great but I realise this might be a little bit trickier to implement.

It shouldn't be too difficult to implement. I will try it out and share them to my HF account and then transfer it to bigLAM after someone's had a chance to look at it

That sounds great. Thanks for working on this 🤗

#ready-for-review

@davanstrien This dataset is now live at https://huggingface.co/datasets/shamikbose89/old_bailey_proceedings. Can you take a look to check if it looks alright and then I can transfer it to biglam? Thanks!

@davanstrien This dataset is now live at huggingface.co/datasets/shamikbose89/old_bailey_proceedings. Can you take a look to check if it looks alright and then I can transfer it to biglam? Thanks!

Thanks for this! I am AFK most of today, but I'll take a look at this tomorrow 🙂

Hi @shamikbose, thanks so much for working on this. Sorry for the slow response with this. I just caught covid, so I'm unable to work much.
Some initial comments:

I made a small suggestion in a pull request to fix the en language tag
On https://huggingface.co/datasets/shamikbose89/old_bailey_proceedings/blob/main/old_bailey_proceedings.py#L145, I think it would be better to just log an exception here rather than asking for user input (since this might prevent the loading script from working inside other scripts. WDYT @albertvillanova
NIT, you could replace the dictionary here https://huggingface.co/datasets/shamikbose89/old_bailey_proceedings/blob/main/old_bailey_proceedings.py#L42 and just put the URL in a variable

@shamikbose happy for you to move this over to the BigLAM org. I may make a few minor additions to the README if that's okay with you. @albertvillanova or @clancyoftheoverflow would be great if you could look at this one at some point, too (I trust my brain even less than usual today!)
thanks again for working on this (and so many other datasets)!

I've made a few extra changes to the README now. I'll close this for now since I've put a note on the README asking for any suggestions for other configurations in the discussion section of this dataset. Thanks again @shamikbose!


Missing Textures in JEI - Various items

Please use the search functionality before reporting an issue. Also take a look at the closed issues!
Issue description:
Flamethrower, Free runner, energy cubes all missing item texture in JEI. Only thing that shows up is the power bar for the items.
Steps to reproduce:
1.Installed in various instances
2.Issue persists regardless of instance
3.
Version (make sure you are on the latest version before reporting):
Forge: <IP_ADDRESS>95
Mekanism: <IP_ADDRESS>9
Other relevant version:
If a (crash)log is relevant for this issue, link it here: (It's almost always relevant)
[gist/pastebin/etc link here]

Duplicate of #5205


Add Support for Bing Tile functions

It would be super helpful to add functionality to support the bing tile functions to this API like Presto has as listed here: https://prestodb.io/docs/current/functions/geospatial.html#bing-tiles
As part of an effort to detect overlap of 2.9 buildings with water globally I find I need to bucket my queries on quadkey. This has worked great, but unfortunately it only works if the dataset already has quadkeys for all features. I would love the ability to generate bing tiles and quadkeys natively in the Sedona API.
The highest priority function for me is the geometry_to_bing_tiles function that produces a list of quadkeys that the geometry passed in touches.
For now I am having to define a UDF to generate quadkeys and call it to convert geometry to quadkeys.

@atiannicelli Do you have an example of the UDF you implement? We can help you import that to Sedona.

once I finalize it and test it I will post what I have

@atiannicelli any progress on this?

I didn't implement it as a UDF. Instead I used geometry_to_bing_tiles to cut up the geometries into smaller pieces. This is the Athena (trino) query that I use to split the water bodies up into smaller geometries:
                SELECT
                    id,
                    TRY(ST_ASBINARY(st_intersection(bing_tile_polygon(tile), ST_GeomFromBinary(geometry)))) as geometry,
                    TRY(bing_tile_quadkey(tile)) as quadkey
                FROM release.v{overture_table}
                CROSS JOIN UNNEST(TRY(geometry_to_bing_tiles(ST_GeomFromBinary(geometry), 13))) AS t(tile)
                WHERE
                    theme = 'base'
                    AND type = 'water'
                    AND ST_ISVALID(ST_GeomFromBinary(geometry))

Unfortunately I still need to tweak this as some water body geometries (like lake Michigan) are too complex to apply to geometry_to_bing_tiles at that zoom level.


Preserve instance reference relationships by tagging instance IDs

For instance tag ObjId, realize the  serialization of circular reference or multiple references.
在几乎序列化类库中,  都没有指出相同的实例引用在xml中的对应关系. 这就造成了两个问题:
In almost all serialized class libraries, the correspondence of the same instance reference in XML is not indicated. This creates two problems.

A->B->C->A 会造成序列化时无限循环, 即使我们的实例是有限的.
A->B->C->A will cause an infinite loop when serializing, even if our instances are finite.
对同一个实例多次引用后, 经过序列化和反序列化,  结果并没有还原他们的相同引用关系. 比如[A1, A1] 经过序列化和反序列化会变成[A1,A2]
After multiple references to the same instance, serialized and deserialized, the result is that their same reference relationship is not restored. For example, [A1, A1] becomes [A1,A2] after serialization and deserialization

I tagged the instance with the ID, which solved the problem.
The test samples illustrates the functionality more clearly.  Note the _ObjId=XXX.
Example1:
      public static CPack GetSampleInstance()
        {
            CBoxA boxa = new() { Name = "AAA" };
            CBoxB boxb = new() { Name = "BBB" };
            CBoxC boxc = new() { Name = "CCC" };
            boxa.Child = boxb;
            boxb.Child = boxc;
            boxc.Child = boxa;
            return new CPack { Name = "PackLoop", ItWillLoop = boxa };
        }

The result of serialization is
            <CPack xmlns:yaxlib="http://www.sinairv.com/yaxlib/">
              <Int01>666</Int01>
              <Name>PackLoop</Name>
              <ItWillLoop yaxlib:id="1000">
                <Int02>333</Int02>
                <Name>AAA</Name>
                <Child>
                  <Int01>444</Int01>
                  <Name>BBB</Name>
                  <Child>
                    <Int02>555</Int02>
                    <Name>CCC</Name>
                    <Child yaxlib:ref="1000" />
                  </Child>
                </Child>
              </ItWillLoop>
            </CPack>

Deserialization can also pass
  Assert.AreSame(got.ItWillLoop, got.ItWillLoop.Child.Child.Child);

Example2:
     public static CPack02 GetSampleInstance()
        {
            var newone = new CPack02() { BoxList = new List<CBox>() };

            for (int i = 0; i < 5; i++)
            {
                newone.BoxList.Add(new CBox
                {
                    Name = $"Node{i}",
                });
            }
            newone.BoxList[0] = newone.BoxList[2];
            newone.BoxList[4] = newone.BoxList[2];
            return newone;
        }

The result of serialization is
          <CPack02 xmlns:yaxlib="http://www.sinairv.com/yaxlib/">
              <BoxList>
                <CBox yaxlib:id="1000">
                  <Name>Node2</Name>
                </CBox>
                <CBox>
                  <Name>Node1</Name>
                </CBox>
                <CBox yaxlib:ref="1000" />
                <CBox>
                  <Name>Node3</Name>
                </CBox>
                <CBox yaxlib:ref="1000" />
              </BoxList>
            </CPack02>

Deserialization can also pass
     Assert.AreSame(got.BoxList[0], got.BoxList[2]);
     Assert.AreSame(got.BoxList[4], got.BoxList[2]);

Thanks again for the PR. Here are some thoughts after going through the code.
1. YAXSerializationOptions
1.1 Competing options
1.1.1 With this PR we have have

YAXSerializationOptions.ThrowUponSerializingCyclingReferences
YAXSerializationOptions.MarkObjId2AvertSefRef

They can be set at the same time, which is ambiguous. This is disadvantage of using [flag] enums.
1.1.2 Looking at the Json.NET implementation, they are using
a) ReferenceLoopHandling.Error
b) ReferenceLoopHandling.Ignore
c) ReferenceLoopHandling.Serialize
1.1.3 Solution for YAXLib
So we could add a property LoopHandling of type ReferenceLoopHandling to the SerializerOptions:
a) ReferenceLoopHandling.Error: This would be the same behavior as with YAXSerializationOptions.ThrowUponSerializingCyclingReferences. We mark YAXSerializationOptions.ThrowUponSerializingCyclingReferences as obsolete, but care for compatibility: In case YAXSerializationOptions.ThrowUponSerializingCyclingReferences will be set, LoopHandling will be ReferenceLoopHandling.Error and vice versa. No change to the current behavior.
b) ReferenceLoopHandling.Ignore: This is the behavor when YAXSerializationOptions.ThrowUponSerializingCyclingReferences is not set. For serialization we produce just an empty element. No change to the current behavior as shown in this unit test:
https://github.com/YAXLib/YAXLib/blob/19b6d61d52dc3ba657d63fa07c62189d37effe09/YAXLibTests/SerializationTestBase.cs#L2668-L2690
c) ReferenceLoopHandling.Serialize: This would be the serialization implemented with this PR (currently marked with YAXSerializationOptions.MarkObjId2AvertSefRef).
1.2 Not implemented option
Option YAXSerializationOptions.SuppressMetadataAttributes currently has no effect when using YAXLib.Enums.YAXSerializationOptions.MarkObjId2AvertSefRef | YAXLib.Enums.YAXSerializationOptions.SuppressMetadataAttributes
YAXLib.Enums.YAXSerializationOptions.SuppressMetadataAttributes should never add YAXLib metadata attributes (e.g., 'yaxlib:realtype') to the serialized XML (even when they would be required for deserialization.) Useful when the generated XML is targeting third party systems.
Possible solution: When this option is set, we fall back to ReferenceLoopHandling.Ignore.
2. Exception YAXCannotSerializeSelfReferentialTypes
The exception will continue to be thrown for the setting ReferenceLoopHandling.Error-
3. New class Xmlhelper
3.1 Use existing definitions
There are already definitions for YaxlibNamespace and PrefixNamespace in class SerializerOptions.
Attribute names for OBJID_ELEMENT_NAME and OBJID_ELEMENT_NAME should also be implemented there.
3.2 Use existing classes
We have an existing class TypeExtensions, so extensions method rather should be placed there.
4. Miscellaneous
4.1 Method ShouldMarkObjId(this MemberWrapper member) does seem to be in use.
4.2 Replace memtype.FullName == "System.String"with more simple memtype == typeof(string)
Looking forward to your feedback on the comments.


Provider produced inconsistent result after apply if order changed in UI

Affected Resource(s)

coralogix_tco_policy_logs

Terraform Configuration Files

Please include all Terraform configurations required to reproduce the bug. Bug reports without a functional reproduction may be closed without investigation.
terraform {
  required_version = ">= 1.2.3"
  required_providers {
    coralogix = {
      source  = "coralogix/coralogix"
      version = ">=1.10"
    }
  }
}

provider "coralogix" {}

resource "coralogix_tco_policy_logs" "tco_policy_all_logs_to_low" {
  name       = "pawling-test debug to low - tf"
  priority   = "low"
  order      = 10
  severities = ["debug"]
}

Debug Output

https://gist.github.com/chris468/9005c7c947badbcb36dc222bbd0c21aa
I forgot to redirect stderr when I captured the logs, but I was able to get the full debug output from console history for 2 of the files.
Panic Output

N/A
Expected Behavior

Ideally find a way to maintain the desired order, but at least error during apply rather than reporting inconsistent result.
Actual Behavior

│ Error: Provider produced inconsistent result after apply
│
│ When applying changes to coralogix_tco_policy_logs.tco_policy_all_logs_to_low, provider "provider[\"registry.terraform.io/coralogix/coralogix\"]"
│ produced an unexpected new value: .order: was cty.NumberIntVal(10), but now cty.NumberIntVal(3).
│
│ This is a bug in the provider, which should be reported in the provider's own issue tracker.

Steps to Reproduce

Check order of any existing policies: grpcurl -H "Authorization: Bearer $CORALOGIX_API_KEY" -d '{}' ng-api-grpc.coralogix.com:443  com.coralogix.quota.v1.PoliciesService.GetCompanyPolicies | jq '.policies[] | {name: .name, order: .order}'
Result:{
  "name": "End to end test logs",
  "order": 1
}
{
  "name": "Debug, Verbose, and Info to Low",
  "order": 2
}

terraform apply | tee initial_apply.log
Check order: grpcurl -H "Authorization: Bearer $CORALOGIX_API_KEY" -d '{}' ng-api-grpc.coralogix.com:443  com.coralogix.quota.v1.PoliciesService.GetCompanyPolicies | jq '.policies[] | {name: .name, order: .order}'
Result: Terraform-created policy is order 3, even though 10 was specified in resource definition. But overall order is correct.{
  "name": "End to end test logs",
  "order": 1
}
{
  "name": "Debug, Verbose, and Info to Low",
  "order": 2
}
{
  "name": "pawling-test debug to low - tf",
  "order": 3
}

Create another tco policy in the web UI, at the end.
Check order: grpcurl -H "Authorization: Bearer $CORALOGIX_API_KEY" -d '{}' ng-api-grpc.coralogix.com:443  com.coralogix.quota.v1.PoliciesService.GetCompanyPolicies | jq '.policies[] | {name: .name, order: .order}'
Result: New policy has order 4, makes sense{
  "name": "End to end test logs",
  "order": 1
}
{
  "name": "Debug, Verbose, and Info to Low",
  "order": 2
}
{
  "name": "pawling-test debug to low - tf",
  "order": 3
}
{
  "name": "pawling-test verbose to low - ui",
  "order": 4
}

In the web UI, move the manually created one above the one that was created by the stack: swap 3 and 4.
Check order: grpcurl -H "Authorization: Bearer $CORALOGIX_API_KEY" -d '{}' ng-api-grpc.coralogix.com:443  com.coralogix.quota.v1.PoliciesService.GetCompanyPolicies | jq '.policies[] | {name: .name, order: .order}'
Result all policies orders are updated, starting w/ 0 instead of 1:{
  "name": "End to end test logs",
  "order": 0
}
{
  "name": "Debug, Verbose, and Info to Low",
  "order": 1
}
{
  "name": "pawling-test verbose to low - ui",
  "order": 2
}
{
  "name": "pawling-test debug to low - tf",
  "order": 3
}

terraform apply | tee second_apply.log
Result: Error in provider│ Error: Provider produced inconsistent result after apply
│
│ When applying changes to coralogix_tco_policy_logs.tco_policy_all_logs_to_low, provider "provider[\"registry.terraform.io/coralogix/coralogix\"]"
│ produced an unexpected new value: .order: was cty.NumberIntVal(10), but now cty.NumberIntVal(3).
│
│ This is a bug in the provider, which should be reported in the provider's own issue tracker.

Important Factoids

Our use case is that we want a default policy to set Info and below to 'Low', but we have an exception where we have one service that needs Info to be 'High'. We attempted to do this by setting the "default" policy w/ a high order, so that "exceptions" could be inserted above it.
When that didn't work, we tried to set the order to exactly what it should be. But the terraform resource will not allow an order of 0:  Change the order in the configuration to 0 and terraform validate  | tee validate_order_0.log reports:
│ Error: Invalid Attribute Value
│
│   with coralogix_tco_policy_logs.tco_policy_all_logs_to_low,
│   on main.tf line 15, in resource "coralogix_tco_policy_logs" "tco_policy_all_logs_to_low":
│   15:   order      = 0
│
│ Attribute order value must be at least 1, got: 0

References

#113 refers to multiple policies in the same terraform stack. But in these steps I use a single policy.
#178 removed coralogix_tco_policy_override. I believe policy overrides would be a better solution to my use case. Is there a plan to support a policy override resource going forward?

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Hi @chris468.
The current backend design prevents the ability to keep track on policies that where create outside of terraform, therefor it would be a bit difficult to implement your suggestion, even thought I agree it would be better to have this functionality.
However, the backend intend to release a new API where the tco-policies are implemented as a single list, thus solving the whole issues of parallelism, order, etc...

Thanks @OrNovo. Makes sense, that sounds like a better API. Do you know what is the timeframe for the new API?
Focusing less on policies created inside vs. outside terraform, just on the use case of "exceptions" to a "default" TCO policy: Is multiple TCO policies the best approach? Or are policy overrides better? Is the new API the reason the policy override resources were removed from the terraform provider?
In the meantime, we applied another workaround to prevent the terraform provider from erroring so that the rest of our stack will complete. We ignore the order for now:
resource "coralogix_tco_policy_logs" "tco_policy_all_logs_to_low" {
  name       = "pawling-test debug to low - tf"
  priority   = "low"
  order      = 10
  severities = ["debug"
  lifecycle {
    ignore_changes = [
      order
    ]
  }
}

I'm not sure about the time frame, I'll update.
Policy-overrides API will be removed regardless of terraform or API changes, so the best approach is to use only TCO policies.

Worth mentioning that this same issue affects the filter order for the coralogix_sli resource.

@cameronwaterman Not sure I understand your comment.

You receive the same error Provider produced inconsistent result after apply when deploying a coralogix_sli resource if the filter order was changed in the UI.

@OrNovo - I created a separate issue #211 to track the SLO behavior.


Vectara rag mq

Description: another Vectara template for MultiQuery RAG flow
Twitter handle: @ofermend

Erick - sorry, when I run "make lint" locally I only see other (non-vectara-related) files that come up as issues. The failues above say:
Creating virtualenv langchain-monorepo in /home/runner/work/langchain/langchain/.venv
poetry lock --check is deprecated, use poetry check --lock instead.
Error: poetry.lock is not consistent with pyproject.toml. Run poetry lock [--no-update] to fix it.
Error: Process completed with exit code 1.
Again, not sure how to fix it. Can you please advise?

Looks like it's all good now

Hey @ofermend ! I unfortunately don't have push access to the vectara repo to fix a few more things. See the PR mentioned ^
Will land today!


Azure AD Graph Explorer

Rename to Microsoft Graph Explorer.  (You should say that this app is used to allow you to access data in your tenant.)
Graph Explorer supports deep links - so consider a link that will auto-populate the query in your step.

Please make this change in all other topics that reference "Azure AD" Graph Explorer.

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

ID: 3bd35b3e-5e84-8be5-f87b-b4b666a9f7cd
Version Independent ID: c768f626-4ea1-4ede-492d-3a9d9e2178f3
Content: Tutorial: Azure Active Directory integration with Amazon Web Services (AWS) to connect multiple accounts
Content Source: articles/active-directory/saas-apps/aws-multi-accounts-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @jeevansd
Microsoft Alias: jeedes

@dkershaw10 Thanks for your feedback! We will investigate and update as appropriate.

@chetansriv Please review this

@dkershaw10 We have made changes in the article, please review.
#please-close


Table Sorting, Filtering, Pagination

Added sorting, filtering and pagination to Table component
Dependencies:

React Table
React Paginate (for pagination component)

New components:

Pagination - Page selector for table pagination
Pill - Pill for filter option

Will add screenshots soon

Just a clarification - this PR covers the new Table functionality + adding that to the Performance page, right? We can do something similar for settings now, I'm assuming?

Just a clarification - this PR covers the new Table functionality + adding that to the Performance page, right? We can do something similar for settings now, I'm assuming?

@mkanoria Yeah, exactly. However, IIRC Julia said that we won't be having sorting/filtering/pagination for the settings page?

Just a clarification - this PR covers the new Table functionality + adding that to the Performance page, right? We can do something similar for settings now, I'm assuming?

@mkanoria Yeah, exactly. However, IIRC Julia said that we won't be having sorting/filtering/pagination for the settings page?

Ohh perfect, let's get this merged in then 💪🏽 🥳


Post box filter does not recognize swears with repeated characters

this could possibly be fixed by switching to regex

This issue was moved to HulaSamsquanch/swearify#3

FIXED IN REGEX2 BRANCH!!


Motivation for lmdb-go

Hi,
I'm planning to use an LMBD in a go / cosmos project. I know that you did an amazing work with TurboGeth. Could you share please a bit about your motivation for creating this package? Specifically:
Did you consider going bbolt? It offers a bit more features and data security. BoltDB has a bit worse performance, but not sure if this is a bottleneck. On the other hand it  has more data security features.

Hi Robert,

Are you asking why we are using LevelDB (which is what levigo is wrapping) vs LMDB? We need transactionality (ACID or at least ACI) in the database, and LevelDB does not give that. Also, LevelDB is write-optimised, and LMDB (and bolt) are read optimised.
Turbo-Geth was using BoltDB for most of its development and we only recently switched to LMDB. Contrary to what it may seem, LMDB actually has more features and more flexible, it just takes time to learn how to use them.

Warning - I did try to simply replace LevelDB with BoltDB in Cosmos SDK, and it got 2x slower. Simply changing the DB won't help much unless you are also prepared to change your data model, which is much harder

@AlexeyAkhunov , thanks for insights!
Cosmos is currently using IAVL+ for the internal storage.
Wow, I wouldn't expect that BoltDB is 2x slower.
What is the performance diff after the switch from BoltDB to LMDB?

Cosmos is currently using IAVL+ for the internal storage.

Yes, I am aware, I worked with it a tiny bit: https://github.com/AlexeyAkhunov/turbo-geth-archive/tree/turbo-geth-12/avl
:)
Initially when we switched from BoltDB to LMDB, there was no difference. But then we started using features that weren't available in BoltDB, and performance did improve. It is hard to say "how much", because we have variety of workloads in different stages. But in the most challenging stage, I think we reduced the transaction replay time probably by 50-70%.
Again, word of caution - do not expect sudden gains simply from switching the database, you must understand exactly WHY you are switching.

thanks a lot!


[BUG] Version 1.30 is not available on mvn repo, even though the Getting Start Guide indicates to use it.

The installation guide here: https://playwright.dev/java/docs/intro is mentioning to use version 1.30, but it's not available on Maven Repo yet, causing some confusion, which is especially confusing a Getting Started Guide.
I saw there have been similar issues in the past, like https://github.com/microsoft/playwright-java/issues/1159, but maybe it would be better to avoid confusion and only publish the documentation when the release is done?

Thanks for sharing that information, I was unaware of this. I know the statistics on OSS Sonatype are usually 3 weeks behind, but I didn't know publishing is affected now too. Thanks again, I'll close this issue.

@yury-s any updates you know of?  The ticket and https://status.maven.org/ seem to point to issues having been resolved.

Should be available now.


Ordered box

Idea from https://github.com/cdgriffith/Box/issues/45

Hello @cdgriffith! Thanks for submitting the PR.

In the file box.py, following are the PEP8 issues :

Line 523:80: E501 line too long (97 > 79 characters)
Line 556:80: E501 line too long (97 > 79 characters)
Line 564:80: E501 line too long (93 > 79 characters)
Line 580:80: E501 line too long (94 > 79 characters)
Line 586:80: E501 line too long (83 > 79 characters)

In the file test/test_functional_box.py, following are the PEP8 issues :

Line 678:1: E302 expected 2 blank lines, found 1

Coverage increased (+0.3%) to 99.158% when pulling 16d48474c2d418817106da9e76057ad33ef97e64 on ordered_box into 26742f804dfcbb5e11d4ea320ed914a97e057181 on development.


adjusting the model for input images with 4 channels

Hi Yassine,
I want to use your model to train a segmentation model on satellite imagery with 4 channels: RGB and Height. So far I got it to work on training on the RGB images only, now I want to change the model/encoder to ingest a 4th channel. I can't find where the number of input channels is specified. In _PSPModule this is set to 2048. What does this mean?
How do I adjust the code to train on 4-channel images?
Thanks for your help!

Hi @SuzannaLin
The number of input channels is set in the backbone here, the layer name is conv1.


User Creation Improvement / Bug

Translation is Missing in DE and EN

Save Action should have a spinner to signal the user, that something is working
Right now the page just waits until the save is done.
Error after successful saving a user

reread of missing events didnt work if user is not in readmodel


Failure presenting the alert using a button action from another alert

When I try to show CustomAlert when I press on a button from another CustomAlert (both are in the same SwiftUI view), the second CustomAlert does not show up and XCode logs this warning in the console:
[Presentation] Attempt to present a window with key 'CustomAlert' while there is already a window with key 'CustomAlert' presented on the current window scene.

Yeah, this was semi-intentional to prevent showing multiple windows at once, but I don't see a strong reason to keep it, especially if it prevents other flows like your.
Should be fixed in 3.6.1

yeah there are many cases in which a user needs it, thanks for the effort.


ClangdToggleInlayHints must hits twice first time

when I use
      local on_attach2 = function(_, bufnr)
        vim.api.nvim_buf_set_option(bufnr, "omnifunc", "v:lua.vim.lsp.omnifunc")
        require("clangd_extensions.inlay_hints").setup_autocmd()
        require("clangd_extensions.inlay_hints").set_inlay_hints()
      end

{
    "p00f/clangd_extensions.nvim",
    enabled = true,
    ft = { "c", "cpp" },
    opts = { extensions = { autoSetHints = true } },
    lazy = true,
}

the inlayHints auto start,
for the first time I use ClangdToggleInlayHints
I must entering the cmd twice.
Is there some where configure wrong with me？

You don't need to lazy load it, good plugins (including this one) lazy-load themselves.
If the issue still persists, please provide a minimal reproduction using a simpler plugin manager like vim-plug, paq-nvim or :h packages

Author has not responded, closing


rewrite tactic in HoTT library

Jakob and I noticed that the rewrite tactic tends to fail a lot in the HoTT library. I thought that the higher order unification failed to find the correct solution, but maybe there is something going on with universe levels (see below).
To give some examples where rewrite works in Coq, but fails in Lean, here are some rewrite tactics in the Coq-HoTT library:
https://github.com/HoTT/HoTT/blob/master/theories/Types/Forall.v#L153-156
Here is a working port of it, using apply (transport ...) (with explicit type family of the transport) instead of rewrite:
https://github.com/fpvandoorn/lean/blob/rewrite/hott/types/pi.hlean#L109-120
(note: transport_V is just transport along the inverse path, so that rewrite foo corresponds to apply (transport_V _ foo))
(note 2: this repository is just the blessed repository where only types/pi.hlean has been modified)
Below each line are two commented out tactics, the first one is the rewrite tactic as Coq used it, and the second is the rewrite tactic with all arguments supplied to the theorem.
All these rewrite tactics fail.
All of them give the same error message "tactic failed", except rewrite (adj f0 a'), which gives the very weird (but hopefully interesting) error message:
type mismatch at application
  eq
term
  function.compose f0 f0 ⁻¹ (f0 a') = function.id (f0 a')
has type
  Type.{l}
but is expected to have type
  Type.{k}

I really hope these rewrites all fail because there is some minor universe error in the rewrite tactic, instead of a failing higher-order unification.

@fpvandoorn I found and fixed one bug in the HoTT mode.
https://github.com/leanprover/lean/commit/98960cbeda9aadce25e7cf9731d11691582a7e63
Now, we can use rewrite (adj f0), instead of apply (transport_V _ (adj f0 a')).
The rewrite tactic is based on higher-order matching. This is why rewrite adj  does not work. In this case, the left-hand-side would be something like (retr ?f (?f ?x)).
This is not a higher-order pattern. In a higher-order-pattern, a metavariable cannot be the argument of another metavariable (e.g. ?f ?x).
BTW, we can also provide our own patterns to the rewrite tactic
     rewrite {retr f0 _}adj

I did not close the issue because the other rewrite tactics in your file do not work.
Do you expect all of them to work?

@fpvandoorn rewrite -transport_compose also does not work because the left-hand-side is not a higher-order pattern. It has the form
@eq.transport ?B ?P (?f ?x) (?f ?y) (@ap ?A ?B ?f ?x ?y ?p) ?z

The problem are the subterms ?f ?x and ?f ?y.
I managed to workaround the issue by providing my own pattern:
  rewrite -{eq.transport _ (ap _ _) _}transport_compose,

Now, I'm wondering if we should "cleanup" the patterns before we try to match.
For example,
@eq.transport ?B ?P (?f ?x) (?f ?y) (@ap ?A ?B ?f ?x ?y ?p) ?z

does not qualify as a pattern. Then, we replace the offending terms by fresh metavariables.
@eq.transport ?B ?P ?M1 ?M2 (@ap ?A ?B ?f ?x ?y ?p) ?z

BTW, matching is just a quick/efficient way to locate places where rewriting should be considered.
After we locate them, higher-order unification is used anyway.

@fpvandoorn I implemented the trick I described in the previous comment.
I think the rewrite tactic is more convenient to use now.
Take a look at:
https://github.com/leanprover/lean/blob/5cbdd77ad039e3ab1ff06dafca4b94589d3aabe3/tests/lean/hott/433.hlean#L99-L120

This looks very nice!
In the test I'm confused about the comment
-- retr cannot be used as rewrite rule, the resulting type is not an equality
The conclusion of retr is function.compose f f ⁻¹ ∼ function.id which can be unfolded to forall x, function.compose f f ⁻¹ x = function.id x. Does the current rewrite tactic not unfold definitions in the theorem with which you want to rewrite? I think that would be nice to have.

My mistake.
∼ is marked as reducible. So, it will be unfolded by the rewrite tactic.
If we use
rewrite (retr (f1 _) _)

Lean uses the following pattern:
@function.compose (B' ?M_1) (B (f0 ?M_1)) (B' ?M_1) (f1 ?M_1)
                  (@inv (B (f0 ?M_1)) (B' ?M_1) (f1 ?M_1) ?M_2) ?M_3

The rewrite tactic works in the following way:
1- If a pattern is provided, then it is used. If it is not provided, then the left-hand-side of the of the equality provided is used as a pattern.
2- Then, the goal (and/or hypotheses if at is used) is traversed, and Lean tries to match the pattern against subterms. The matching procedure is very restrictive: only constants marked as reducible are unfolded; projections are never unfolded; terms marked as implicit instances (the [a : A] notation) are "ignored"; the matcher only supports higher-order patterns.
3- If a subterm t matches the pattern, then Lean will try to use higher-order unification (and type class synthesis) to synthesize the missing arguments of the lemma provided by the user.
The unification step is also conservative: only constants marked as reducible are unfolded (projections included).
4- If the previous step is successful, then all occurrences of the subterm t are replaced (unless the user provides a subset of the occurrences), and the rewrite step terminates. If the unification step fails the search continues.
We should view the matching step as a heuristic to select potential places where the rewriting step can be applied.

Here is the real reason rewrite (retr (f1 _) _) fails.
The problem is that the function.compose reducibility attribute is "part of" the function namespace.
So, if we don't open function, function.compose is not really reducible.
If we execute
open function

then rewrite (retr (f1 _) _) succeeds.
Here is another possibility:
We define function.compose in the top-level as
definition function.compose [reducible] (f : B → C) (g : A → B) : A → C :=
λx, f (g x)

Then, it will always be reducible. What do you think?

The matching sounds good. It's good that it's more restrictive than full higher order unification, because that tends to make the search space too big.
I think it's good to make function.compose (and also function.id) always reducible.


Need to add a new measure way

❓ Questions and Help
Hi, I am using this Mask R-CNN to implement an auto-checkout dataset.
And for an auto-checkout task, if there is one error in one image, then this image is wrong, counted as 0. Otherwise, it is correct and counted as 1. And the measure way is called "Checkout Accuracy (cAcc)". And the cAcc is the average value among all images that are evaluated.
I read the codes, find I should modify the cocoapi. However, I have no idea how can I add this new measure way for now. Could anyone give me some suggestion?
Thank you very much!

You can implement your evaluation method and add it here.
https://github.com/facebookresearch/maskrcnn-benchmark/blob/master/maskrcnn_benchmark/data/datasets/evaluation/__init__.py
Just implement in the way voc does it. i.e, add another dataset instance and add its own evaluation


Parts Images don't show up

Deployment Method

[ ] Installer
[ ] Docker Development
[x] Docker Production
[ ] Bare metal Development
[ ] Bare metal Production
[ ] Digital Ocean image
[ ] Other (please provide a link Steps to Reproduce

Describe the problem*
Hello there, hope you can help me...
Well, i think a screen capture will speak for itself :

Steps to Reproduce
I made a fresh install of inventree on my synology Nas via Docker Container Manager.

1 : i create a part
2 : i upload an image  (the image file show up in the part_images directory)
3 : image doesn't show up (image broken logo)

Relevant log output

We need the full server information and logs to help with anything.

The /media/ files are not being served correctly. If you have followed the docker setup guide, then your inventree-proxy container should be serving these files.

Did you follow the setup / update guide precisely
Did you read the FAQ

I just checked my installation and it doesn't use the caddy proxy server because i'm on a Synology Nas and i want to use the native reverse proxy of the nas to access inventree.
I tried to reinstall inventree with the proxy but i can't connect trought the nas reverse proxy (i get the "ERR_TOO_MANY_REDIRECTS" error)
According to this page : https://docs.inventree.org/en/stable/start/serving_files/
I tried to set the debug mode to True and when i started it again, my images just show up (that could be a solution, i don't know if it's safe to stay in debug mode)
So i discovered my real problem is that i don't know how to configure my caddy proxy correctly on a Synology Nas :/

I recently started to look at Inventree again for home/personal use with the plan to run it on my Synology NAS and found the same thing. As
mentioned above it will be to do with the Iventree reverse proxy and the Synology reverse proxy not working well together. I came up with a few potential ways to work around this,

Use the Synology proxy to replace the Inventree proxy (which I think you are trying to do)
Don't use the Synology proxy and expose the Inventree proxy on non-standard ports
Use a proxy in front of the Inventree proxy, but not the Synology proxy

Option 1. This might be possible to get working, but you will have to figure out how the Synology proxy works internally on DSM and likely modify the config via the command line. I think it uses Nginx, so the config might be straight forward as Inventree provide a Nginx example in their compose file. However, as you are modifying these files outside of the GUI there is a good chance they won't work after a reboot or DSM upgrade (found this out trying to bodge something else on DSM). For this reason, I discounted option 1.
Option 2. Probably the easiest option, but you're likely wanting to use a reverse proxy so you don't have to manage certificates and you don't have to remember port numbers on the end of your URL. Which is why I discounted this option.
Option 3. The most convoluted setup but was what I ended up using. The downside is putting a proxy in-front of another proxy may causes issues, as you have found out with the Synology proxy. The good news is if you have full control of the proxy, you can most likely fix the issue and you are not tied to Synology if you want to move your containers at a later date.
As a rough outline, this is what I have done.

Created a 'MACVLAN network' and a 'reverse proxy bridge network' in Docker to home my reverse proxy (Nginx Proxy Manager, aka NPM). This has the advantage of allowing NPM to not conflict with any ports Synology may use and have a static IP directly on my main network.

Deployed Inventree on Docker but modified the Inventree 'reverse-proxy' container to run Nginx not only on the default Inventree network, but also my reverse proxy network (hint: the reverse proxy network should be specified as external in docker compose to allow you to take spin up/down Inventree without affecting other services).

I used Nginx as I had odd things happen with Caddy, but I'm less familiar with it so that might have been me messing things up as I was trying various solutions. I also didn't expose the Inventree reverse-proxy container ports as there is no need since the new proxy can access it internally via Docker using the created 'reverse proxy' network. In NPM I didn't have to do anything special to get Inventree to work, as it seemed to pass the required information to the Inventree proxy out of the box.
I'm happy to go over this solution in more detail, including my full config files and how I'm also using Portainer to manage my config/deployments on Synology. However, I feel it might be best to move the conversation over to the 'Discussions' area so the developers can close this issue since it isn't really an issue with Inventree.
Lastly, for anyone reading this, please please be aware what I have done above is NOT supported by the Inventree team so you may have to fix any future issues with running double proxies and compose modifications etc. yourself.


Configure iptable forwarding policy for pods

This is best described in: https://www.altoros.com/blog/kubernetes-networking-writing-your-own-simple-cni-plug-in-with-bash/

Merging this :)


[BUG]: [User Registration] : [Change "Manual verification by uploading government ID"]

Describe the bug
Now, it is showing as "Manual verification by uploading government ID"
Expected behavior
Change "Manual verification by uploading government ID" to "Manual verification by uploading government issued ID"
Screenshots

Hi @devarajd94, please confirm this..


Displaying web content in a DirectX

I am new in DirectX and I'm a little bit lost.
What I would like to do is:

Show a web page using any browser (Chrome?) but within a DirectX app.

Is it possible to do that?

Sorry, this is out of the scope of SharpDX. I don't think that Chrome provides even such a feature (as it is multiprocess, not suitable to integrated into another app like a browser control)


DMMailChimpClient: add get_lists_for_email(...) and permanently_remove_email_from_list(...) methods

To support https://trello.com/c/QROOUpoX/272-users-whose-personal-data-is-removed-are-not-removed-from-mailchimp-mailing-lists
Note this includes a bump to mailchimp3 v3.x.

Each test checks the behaviour of a single method - its return value/exception and any side effects it has. It so happens that the "permanently remove" method has two majorly different behaviours it exhibits, so there are two separate tests.

I think the tests are among some of @risicle's more readable ones 😹 There's a lot going on with the object patches and logging assertions, but I'm not sure how we'd re-write it to be much clearer.


Update _Layout.cshtml

Updated CSS Class on button to make it more purdy!

This looks good, but we will need to remove the styles from site.css if we want to do this. Why don't we update the stylesheet to use the image from DNN?

Ok, I was just using the default bootstrap class... I'll find the DNN icon for this


Webtoons.com extention not working!!

Steps to reproduce
tried, ain't working, would be pleasured if it was fixed
Expected behavior
it should work 😭😭🙏
Actual behavior
it kept making me wait
Crash logs
No response
Mihon version
0.16.3
Android version
11
Device
Samsung A54
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version 0.16.3.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Mihon is a manga/picture local reader. This is a extension issue, report it to where you got the extension from.

We don't provide support for 3rd party extensions

We don't provide support for 3rd party extensions

ok okay, sorry for that


Add compute-approvers

Signed-off-by: James Slagle<EMAIL_ADDRESS>

I'll force merge since tests are useless here.


Nexus/Central analyzers should fail hard when they cannot download some file

Excerpt from NexusAnalyzer:
            } catch (DownloadFailedException ex) {
                LOGGER.warn("Unable to download pom.xml for {} from Nexus repository; "
                        + "this could result in undetected CPE/CVEs.", dependency.getFileName());
            } finally {

In order to have stable scans, I would rather fail (i.e., throw an exception) at this point. Maybe this could be configurable, but I find this reasonable as the default behavior.
Without that, a connection issue can affect the build. That would be OK if that meant build failure (that is, ODC finds out that it misses some important information). This is not OK if it means that ODC pretends everything is OK.
EDIT: I have hit Enter prematurely…

I disagree, I think this warning is appropriate; the repository mananagers do not provide vulnerability information, just a better identification of artifacts in case you're not using Maven.

It depends on type of failure:

If it is kind 404, I agree with you. Failing there would be too strict. Thank you for pointing this out.
For other errors like file failed to write (e.g., you are out of space), bad URI (!), connection errors, TLS errors etc, I still believe ODC should fail hard.

When you look at the fetchFile method, DownloadFailedException is thrown in all the cases, not just in case of 404. And that's the point of the issue.

Thanks for the suggestion. I agree we should differentiate between the reasons for the failure and in some cases fail the execution vs. just outputting a warning.


E2e tests are failing

The e2e tests are failing.

Run pnpm e2e from the root of the monorepo to reproduce.
I found some possible solutions here: https://stackoverflow.com/questions/69692842/error-message-error0308010cdigital-envelope-routinesunsupported.

Downgrading to node 16 could do the trick (not recommended).
On the other side I tryed to set the following env in my powershell console $env:NODE_OPTIONS = "--openssl-legacy-provider" and the above error disabpeard.

@oliverschuerch, re-evaluate and check cypress download mirror.

Its working with the cypress download mirror!


useAsyncResource ArgTypes extends unknown[] too narrow

Hi, i've got a particular use case where the return and arg types for my async function cannot be inferred due to them needing to be passed as generics when called.
I noticed that the useAsyncResource hook accepts 2 generics, the first being the return type and second being the params.
This is great, as i can pass my own generics when calling the hook, the only issue i have is that the generic for args extends unknown[].
My arg type is an object, not an array meaning i get a TS error. Is this by design? It seems to be that enforcing something that extends an array is a bit too narrow?

Hey @sami616
The second is an array because the params get destructured. If your function has only one param and that's of a specific generic type, then the second generic you need to pass to useAsyncResource is that type as an array. Something like this:
function apiFn<CustomType>(param: CustomType): Promise<ResponseType> {
  return fetch(...).then(r => r.json() as ResponseType);
}

...

const [data] = useAsyncResource<ResponseType, [CustomType]>(apiFn, objParam);

If your function has multiple arguments, all generic, again you should pass them as part of an array:
function apiFn<T, U, V>(a: T, b: U, c: V): Promise<ResponseType> {
  return fetch(...).then(r => r.json() as ResponseType);
}

...

const [data] = useAsyncResource<ResponseType, [CustomType1, CustomType2, CustomType3]>(apiFn, param1, param2, param3);

But you should not have to pass any generics to the hook at all, as it is pretty good at inferring all of them for you:

If your case is different, please share more details so I can better help you more specifically.

okay this makes sense, thanks. I've managed to solve the issue i had 👍🏼


[GHSA-r6wg-rrw3-824h] The django-navbar-client package of v0.9.50 to v1.0.1 was...

Updates

Affected products
Summary

Comments
Added title and version numbers.

Hi @NicPWNs, thank you for reaching out about GHSA-r6wg-rrw3-824h. My colleagues and I have chosen not to review GHSA-r6wg-rrw3-824h because, according to the issue where the information was originally reported, only people who download the django-navbar-client from unofficial mirror sites are vulnerable to the malicious backdoor. No versions of django-navbar-client currently available on PyPI are vulnerable, and alerts only go out to users of the official PyPI package. Therefore, we are not accepting the contribution and not publishing the advisory because non-vulnerable users would receive alerts, and users who need alerts would not receive them. Thanks again for your interest in improving the GitHub Advisory Database and have a great week.


[League] Menu background randomly not loading

Like you can see in the video, the menu background (the paper scroll thingy holding the teleport locations) randomly don't load in. The scrollbar and [x] icon to exit always shows up but the teleport menu itself don't.
Sometimes it loads, sometimes it does not. You kinda have to click the teleport compass several time and just hope it loads.
Could be some kind of race condition. The hotkeys work but you can't click any of the invisible teleport links.
I tell me if you need more info or anything else. I am happy to help resolve this issue!

Looks like the same issue: https://github.com/abextm/better-teleport-menu/issues/22
Just in another form.

upload your client.log from ~/.runelite/logs/client.log and if you have a higher resolution video that too

client.log
https://github.com/user-attachments/assets/e3c950a2-05da-4835-b7c0-257adac45336

lol, I read the closing of #22 and tested disabling teleport maps.
Now it works flawlessly

Open an issue on https://github.com/MJHylkema/spirit-tree-map. It probably just needs a few manual priorities.

Done deal: https://github.com/MJHylkema/spirit-tree-map/issues/35


Add support for distributing releases to multiple distribution groups

This PR adds support for specifying multiple distribution groups when executing the appcenter distribute release command. the -g command now supports comma-delimited lists of groups. For example:

-g thisgroup,thatgroup
-g thisgroup,"group with spaces"

Changelog description:

Flexible distribution: The CLI tool now supports distributing a release to multiple distribution groups.

I believe I have covered all my bases, but let me know if any changes/additions should be made.
Fixes #662

Hi @charliealbright! We really appreciate the contribution to this long-living feature request.
Would you mind adding tests for the change to test/commands/distribute/release/release-test.ts?

@russelarms I added tests there and made a new file for testing the utils I wrote for this feature.
Had some trouble with the mocking setup but eventually got it working.

@Jamminroot @DmitriyKirakosyan this should be ready to review again

/azp run

/azp run

Could you please let me know in which version will the fix be released?

@russelarms is there a timeline for when the next release of appcenter-cli is expected to happen? Looking to use this soon!

Hi @szczepanski-k and @charliealbright,
There is no specific ETA. We're planning to batch a few more fixes to the release.


When SSL connect fails, we return 502 success

While debugging a failure in origin cert verification, we saw that the response header included
HTTP/1.1 502 Success
Date: Mon, 27 Feb 2017 15:18:29 GMT
Connection: keep-alive

502 Success seemed peculiar.  Tracked it down to the fact that errno was not set in the SSL_connect failure case.  Added a check to stuff in ENET_CONNECT_FAILED if errno is 0 so we get a reason that is in the ballpark.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/freebsd-github/1661/ for details.

Intel CC build successful! See https://ci.trafficserver.apache.org/job/icc-github/93/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1557/ for details.

clang-analyzer build successful! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/225/ for details.

The fix seems reasonable, but why 7.1.1? It should be 7.2.0?

Ship it.

Yes, I suppose since it is a latent issue and not a regression it doesn't need to be in the 7.1 train.  We will separately pull it into our version.  Set version to 7.2

I'd like to put this in for 7.1.0 I think. Any objections?


Add DELETE support for use with the Duo Admin API

Some operations against Duo's Admin API use the HTTP DELETE method. These include disassociating users from groups, tokens, phones, etc., along with (obviously) deleting any of those entities. Cf. https://duo.com/support/documentation/adminapi#disassociate-group-from-user
This adds support for the full Admin API by adding a delete method. I also bumped the version number; let me know if you'd like me to resubmit without that change, or if anything else is needed.
Thanks!

Nice add!
Could be getting close to wanting to just generate methods for each HTTP Method, but this looks great for now. Could you swap that version to 0.3.0 since we'll be adding functionality? Once you get that in I can release this.
Thanks!

Done.

Released.
Thanks for the contribution!


Feature request: Lazy load only required exports from module

Feature request

Allow lazy loading of only a part of a module (say, a named export).
When a part of a module is lazy loaded, it must not contain the default and other named exports.

What is the expected behavior?

The ability to request only a particular export (or a set of it) from a module while lazy loading it.
When a single export or a collection of such exports are lazy loaded, webpack should deliver only the what is required with the rest of the module's code tree shaken

What is motivation or use case for adding/changing the behavior?

Dynamic import as of now only allows importing of the whole module that includes all the exports
With this new feature, when code splitting for the requested chunks happens at build time, we will have a chunk with a reduced size which will save us bandwidth while requesting the resource from a server

How should this be implemented in your opinion?

Expand the import(specifier) argument list to include a second parameter to mention required named exports
A single named export:

import(specifier, { named }).then(module => { /* Only module.named is available */ })

Multiple named exports:

import(specifier, { named1, named2 }).then(module => { /* Only module.named1 and module.named2 are available */ })

Creating chunks during build time containing only the specific exports of the module (with the rest tree shaken) can decrease the chunk size and reduce latency when it is requested inside the application

Are you willing to work on this yourself?
Unsure

Expand the import(specifier) argument list to include a second parameter to mention required named exports

import() is spec'ed, we can't add arguments.
But something like
import(/* webpackExports: ["named1", "named2"] */ "module")

would be possible.
As workaround you can create a wrapper file which reexports only a part of the module.
// partial-module.js
export { named1, named2 } from "module";

import("./partial-module")

Thank you for addressing my feature request @sokra
Yes, I do agree that populating information as a magic comment is a better and more elegant approach without fiddling with the spec. As for the workaround, the onus is on the developer's code without webpack actively helping out.
I would like to help if this is something that will be useful to implement (However, I will need time and help as I haven't contributed before)

It is good feature, after it we can deprecate https://github.com/webpack-contrib/exports-loader in favor built-in solution

I want to work on this. I think the starting point is ImportParserPlugin. I request some time to present a code solution if I can do so, @sokra

Sure go ahead

I have opened a draft PR (https://github.com/webpack/webpack/pull/10964/files?diff=unified&w=1)

Added getReferencedExports method to ImportDependency class to be able to return the named exports
Added the initial test cases

Do I need to do a similar change for ImportEagerDependency and ImportWeakDependency? How do we proceed?
@sokra @evilebottnawi

Yep, they need similar changes. Check if they inherit from each other

Added similar support for:

ImportEagerDependency
ImportWeakDependency
(These did not inherit from each other but they all inherited from the common 'ModuleDepenency` class)

Considering the following next:

For params that are not strings (Dynamic expressions) are built using ContextDependencyHelper and ImportContextDependency. Working on introducing named exports here too.
"default" should handle resolving module with only the default export

My tests pass intest:build but fail in test:integration (test/TestCasesCachePack.test.js). I need some help in understanding why and eventually resolving this.


QR code reader

Can we install and integrate OS software that reads QR codes from a desktop camera or from images?

Would let a foray Recorder scan a physical Field Slip on a desktop.
For MO users (and maybe Recorders) could eliminate the scanning step:

User includes the Field Slip in the photo(s) for an Observation. (Photo could be mushroom + Field Slip, or a stand-along Field Slip photo).
create_observation scans its Images for a Field Slip. If it finds one, creates the Field Slip and associates it with the Observation.
At forays, participants could send the Recorder photos of the field slips. Recorder could create Observations from those photos. Maybe some kind of bulk create feature? (Recorder drags a bunch of Images to a page, MO then creates Observations and field slips.)

I've looked into this a bit and it appears the most widely used QR reader code is the zxing Java library supported by Google (https://github.com/zxing/zxing).  There are a number of JavaScript wrappers for it, a JavaScript port called ZXing-JS (https://github.com/zxing-js/library) and various tutorials.  There have been some attempts at native Ruby readers, but none of them seem to be maintained.  I found one paywalled video tutorial specifically about doing it from Ruby on Rails at https://www.driftingruby.com/episodes/creating-and-reading-qr-codes.  There's also a somewhat out of date (2020) text one here:https://dev.to/morinoko/qr-code-reader-on-rails-5816.  I believe all of these work on video or image input managed by the browser.  I haven't found anything that I'm sure works on images stored on the server.  I don't think this is show stopper as long as we have a way to have MO know that it should be looking for a live QR code or that the next image upload is expected to have a code.
In the meantime, the workaround for this to use one of many free online QR readers.  I haven't settled on a favorite yet for this workaround.


border is not showing up  around series name in new version

before i was using old version of this but when i updated with new one everything is working fine but before i was able to see border around series name but now with the new version it is not showing border around series name

I think this is more one for stack overflow. Try setting the options as you would do normally with highcharts


Uncaught TypeError: undefined is not a function

Hello,
after the last update the plugin stopped working for me.
I receive this error:

Hi, you have to include speakingurl as a dependency.
<script src="jquery.js"></script>
<script src="speakingurl.min.js"></script>
<script src="slugify.min.js"></script>

If you use bower - it will be installed as dependency for you. If you do not user bower - you can dowload speakingurl here: https://github.com/pid/speakingurl
Please consult the latest Readme for the changes.

Yup, I've missed that.
Thanks for your fast response.


fails to build with latest nightly

please advise thanks!
   Compiling http-body v0.4.5
error[E0635]: unknown feature `stdsimd`
  --> /Users/***/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ahash-0.8.3/src/lib.rs:99:42
   |
99 | #![cfg_attr(feature = "stdsimd", feature(stdsimd))]
   |                                          ^^^^^^^

   Compiling lz4_flex v0.11.1
For more information about this error, try `rustc --explain E0635`.
error: could not compile `ahash` (lib) due to 1 previous error
warning: build failed, waiting for other jobs to finish...

Related to https://github.com/tkaitchuck/aHash/issues/195, please using the following commands to update ahash
cargo update<EMAIL_ADDRESS>cargo update<EMAIL_ADDRESS>

Closing as I believe this has been resolved