[Bug] there has no org.gradle:gradle-tooling-api:8.3

Describe the bug
the newest org.gradle:gradle-tooling-api is 7.3-20210825160000+0000 not 8.3.
To Reproduce

search the keyword  _"gradle-tooling-api" _ and will find them, all of them is 8.3.

# joern/joern-cli/frontends/x2cpg/build.sbt:16
"org.gradle"     % "gradle-tooling-api" % Versions.gradleTooling % Optional,

# joern/joern-cli/frontends/javasrc2cpg/build.sbt:8
"org.gradle"              % "gradle-tooling-api"            % Versions.gradleTooling,

# joern/joern-cli/frontends/x2cpg/build.sbt:11
"org.gradle"               % "gradle-tooling-api"         % Versions.gradleTooling,

# joern/project/Versions.scala:10
val gradleTooling  = "8.3"

Expected behavior
downgrade org.gradle:gradle-tooling-api  to 7.3-20210825160000+0000 or delete it if not uesd
Desktop (please complete the following information):

OS: MacOS 14.2.1
Joern Version: 2.0.285
Java version: 19

extracting structure failed: Structure file /private/var/folders/43/qdfj53kn7jq3h2hff025t9y40000gp/T/sbt-structure.xml is empty
stack trace is suppressed; run last update for the full output
stack trace is suppressed; run last ssExtractDependencies for the full output
(update) sbt.librarymanagement.ResolveException: Error downloading org.gradle:gradle-tooling-api:8.3
Not found
Not found
not found: ~/.ivy2/localorg.gradle/gradle-tooling-api/8.3/ivys/ivy.xml
not found: https://repo.huaweicloud.com/repository/maven/org/gradle/gradle-tooling-api/8.3/gradle-tooling-api-8.3.pom
not found: https://repo1.maven.org/maven2/org/gradle/gradle-tooling-api/8.3/gradle-tooling-api-8.3.pom
not found: https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/org.gradle/gradle-tooling-api/8.3/ivys/ivy.xml
(ssExtractDependencies) sbt.librarymanagement.ResolveException: Error downloading org.gradle:gradle-tooling-api:8.3
Not found
Not found
not found: ~/.ivy2/localorg.gradle/gradle-tooling-api/8.3/ivys/ivy.xml
not found: https://repo.huaweicloud.com/repository/maven/org/gradle/gradle-tooling-api/8.3/gradle-tooling-api-8.3.pom
not found: https://repo1.maven.org/maven2/org/gradle/gradle-tooling-api/8.3/gradle-tooling-api-8.3.pom
not found: https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/org.gradle/gradle-tooling-api/8.3/ivys/ivy.xml
Total time: 5 s, completed 2024年3月7日 15:10:04


Docs: hide "edit this page" on api reference and "Jupyter format" on home

In the api reference part of the documentation, the user should not have an option to "edit this page" as it is not possible. That should be hidden. Similarly, it does not make sense to have "other formats: Jupyter" on the home page. That should be hidden too.

Hiding other formats: https://quarto.org/docs/output-formats/html-multi-format.html#hiding-all-links
Avoid edit this page: https://github.com/machow/quartodoc/discussions/337

Closed by #488


Gradle folder is missing

Yo, just wanted to say, you are using the gradlew system, but you havent committed your gradle/ directory, which makes it impossible to use the gradlew command.

Oh, yes, you're right. We've been always building using IDEA building, so we haven't used ./gradlew command at all
Now fixed, we committed gradle/
Thank you!


Junk in the debug output (possible cause of crashes?)

Describe the bug
When I use the -debug option sometimes I get junk characters in the terminal following the HTTP header. I also had a couple of crashes without the -debug option, I wonder if they are caused by the same issue.
Expected behavior
No junk characters.
Actual behavior
I get characters beyond ASCII.
How To Reproduce?
Run spoof-dpi -enable-doh -debug
Watch YouTube videos (only from Russia?) for some time, scroll the comments, etc.
Client information
OS: Linux (Debian 12 stable)
Version: Git master cloned on August 14, 13:54 UTC
Additional context

@zimbabwe303  Revision number? Does crashes looks like here #139

@zimbabwe303 These broken symbols are the payload of the request. I found something similar here https://www.reddit.com/r/software/comments/5e5z9f/mpchc_trying_to_connect_to_infected_website/

The problem may be if these symbols contain a control sequence. Then the terminal may beep

@zimbabwe303 Revision number? Does crashes looks like here #139?
v0.10.7 but before the release, it was just a clone of the master branch. May be the problem is fixed now, I'll test it.

I cannot tell what the crashes looked like because I rarely leave spoof-dpi running in the terminal, most of the time it's in the daemon mode (&).

The problem may be if these symbols contain a control sequence. Then the terminal may beep

Also, in general, it's better avoid dumping raw unfiltered data into the text console. All sorts of things may happen, depending on the terminal program.

@zimbabwe303 tell your revision number please

@zimbabwe303 tell your revision number please

See above. v0.10.7 but before the release. I forgot to put an extra line after the quote.

let me check if the new dns over https feature have some issue tomorrow

@zimbabwe303 git log --pretty=format:'%h' -n 1

@zimbabwe303 in the cloned repo
git log --pretty=format:'%h' -n 1

4721bd6
Sorry, never used that feature.

@zimbabwe303 update your branch. The problem with random drops was fixed today

@zimbabwe303 revision 3bdf24ca8ab7559aecc41cf020b5a32ed2e72475 should be stable

Thanks. I think I'll better stick to v0.10.7 release for a while. When, when there'll be more upstream changes I'll do pull on my master clone.


Bump golang.org/x/net from 0.21.0 to 0.25.0

Similar to https://github.com/buildpacks-community/kpack/pull/1612 but targeting the release/0.13.x release.
Fixes https://github.com/advisories/GHSA-4v7x-pqxf-cx7m

Thank you @tomkennedy513 it looks like I don't have permission to merge!

can you run go mod tidy?

can you run go mod tidy?

Ooops! Done in https://github.com/buildpacks-community/kpack/pull/1630/commits/12d0c4fe82c404d235b46e99d580e6bddd74c30e
:bow:


Icons don't appear to work

I've tried using icons from both Font Awsome and RPG Awsome without success, I've tried using them like this:
icon: fa-alien-8bit

icon: alien-8bit

icon: ra-hydra

icon: hydra

But the icon doesn't seem to change for me.
I am new to using admonition, am I missing something?

Are you doing this in an Obsidian callout or a code block admonition?
Also:

fa-alien-8bit is not a free FontAwesome icon. Only the free ones are bundled.
You have to download the RPGAwesome icon pack in settings.

I didn't know about it being a non-free icon. using the file icon from FA worked and now RA does too, I might have just needed to restart obsidian. sorry for the trouble!


Change build engine to extract source control positions

Instead of local files, extract canonical source control links. Probably use the git head ref to permalink, and look for common source control repos, eg. git@github.com:TBD54566975/ftl.git -> https://github.com/TBD54566975/ftl on 666085cb8bf11e74e5af25b63876d14ebc5c8668 would be https://github.com/TBD54566975/ftl/tree/666085cb8bf11e74e5af25b63876d14ebc5c8668

IMHO we should keep the existing behaviour for FTL dev, this should just be for release builds.


Is it support Chinese title？

I set Chinese name in common files，but it doesn't show in exported pdf file.

See #46 where someone also had trouble with this, but solved it. It seems to be related to the Chinese fonts installed on the system and available for use by LaTeX. I suggest you test and debug it using LaTeX directly from the command line on a document with Chinese characters.


elasticsearch 8 support

Hello! Thank you for all the hard work, it is very much appreciated.
Is there a matrix that maps elastic version to java-io-elasticsearch version? I noticed rest client 7.9.2
as I was encountering the error below in trying to perform a simple read from elasticsearch 8.3.5.
URI [/_search/scroll], status line [HTTP/1.1 400 Bad Request]
{"error":{"root_cause":[{"type":"illegal_argument_exception","reason":"Cannot parse scroll id"}],"type":"illegal_argument_exception","reason":"Cannot parse scroll id","caused_by":{"type":"e_o_f_exception","reason":"tried to read: 1668254 bytes but only 0 remaining"}},"status":400}
Your help would be appreciated!

During fix of #31640 encountered https://www.elastic.co/guide/en/elasticsearch/reference/7.17/removal-of-types.html likely the cause of this issue. This sounds generic for elasticsearch 8


Adds CursorStyles option for terminal

Summary
Luckily in version 1.2.0 of SwiftTerm, they’ve introduced CursorStyle which provides multiple styles for the cursor. In issue https://github.com/AuroraEditor/AuroraEditor/issues/482, it was mentioned that currently the cursor blocks visibility of a character when going over it. Therefore, by introducing cursor styles, the user can now choose their preference of shape as well as whether they'd like it to blink or not allowing them to see the character behind the cursor.
Screenshots/GIFs

Checklist
Please ensure all of the following are completed before submitting the PR:

[x] Code has been tested and verified.
[x] Documentation has been updated to reflect the changes.
[x] All tests pass successfully.
[x] Code follows the project's coding guidelines and style.
[x] The branch is up-to-date with the latest changes from the main branch.
[ ] Reviewed by at least one other contributor.

Related Issues
This PR addresses the following issue(s): https://github.com/AuroraEditor/AuroraEditor/issues/482

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Looks good to me!

@all-contributors please add @czuria1 for bug.

Thanks for your contribution!


Link hover

Per Dave’s suggestion over at https://github.com/daveliepmann/tufte-css/pull/43#issuecomment-131164733, I’m proposing a PR for colored links on hover.
It’s been 11 years since Jacob Nielsen’s guidelines  and a lot of sites nowadays —notably Medium— don’t offer different styles for hover.
They may only be relevant to desktop users, but in my opinion hover styles are indispensable in how they strongly inform the reader that the underlined phrase they put their cursor on, is actually a link.

I'm ok with a :hover style for links....though I'm not sure that using the same color as the sidenote numbers makes sense.
here is a showcase of some more creative link hover effects if you're interested in playing around with some thing a bit different. They would need to be adapted somehow to fit the aesthetic of tufte-css: http://tympanus.net/Development/CreativeLinkEffects/

I definitely don't want anything unusual or out of the ordinary for hover effects in this project.
I agree that a color change might make sense, but I'm really hesitant to use red. My instinct is to stick to greyscale.

After refinements to project goals with regards to hyperlinks—specifically, emphasizing readability and unobtrusiveness—hover styles have become less compelling. Thanks for your contribution but I'm going to close this.


Speaker registration and representation rework

This PR introduces a new Speaker system and fixes #24
Users can now register as speakers and submit sessions. When a session for an event is published a new PublishedSpeaker that is associated with that session is created. There is a nullable foreign key relationship between PublishedSpeaker instances and Speaker.
Public views use the PublishedSpeaker instance.
The PR introduces an improved speaker registration workflow. Uploaded portrait images can be cropped to the expected aspect ratio by the speaker. All views from the link on the index page, to user registration, to activation mail, to login, to speaker detail input, portrait upload until session submission are linked via a "next" parameter to allow a smooth transition through the required steps to submit a session.
I tested the migration with production data.

Fixed the broken tests and a minor issue with the display of QR codes on user profile pages for attendees of earlier events.

Pull Request Test Coverage Report for Build 144

0 of 0   changed or added relevant lines in 0 files are covered.
20 unchanged lines in 2 files lost coverage.
Overall coverage increased (+2.0%) to 98.759%

Files with Coverage Reduction
New Missed Lines
%

devday/utils/devdata.py
1
98.14%

talk/views.py
19
94.07%

Totals

Change from base Build 139:
2.0%

Covered Lines:
2199

Relevant Lines:
2219

💛  - Coveralls

The speaker photo cropping doesn't work quite right for at lease some images:

The session plan is not filtering rooms by event:

The links on the talk view page http://localhost:8000/devdata18/talk-preview/ are using the title, not the slug for the event. (This is likely true for development too.)

The speaker photo cropping doesn't work quite right for at lease some images:

It appears that the container for the uploaded image has an aspect ratio of about 2:1; if you upload anything that is taller than wide, it still fills the container horizontally, cutting off the top and bottom. I think making the container 1:1, and larger in general, should fix this.

In the program committee voting view, the speaker column is empty.

In the program committee voting view, the speaker column is empty.


Stencil: build fail after upgrading to Nx v17.

Describe the bug
After upgrading to Nx v17 @nxext/stencil executor fails to build the library.

According to the Nx changelog, some deprecated code was removed from @nex/workspace. Not sure if this is the root cause 🤷🏼♂
To Reproduce

Create an Nx workspace with the latest version (v17)
Add @nxext/stencil
Try to build the stencil library

Expected behavior
The Stencil executor runs without issues.

I haven’t tested anything against v17

@DominikPieper it seems that buildable-libs-utils was moved to @nx/js` package.

@DominikPieper if you need a repro to debug, please feel free to look at (chore/major-nx is the branch):
https://github.com/Endava/BEEQ/tree/chore/major-nx
Also here is the failing pipeline: https://app.circleci.com/pipelines/github/Endava/BEEQ/2026/workflows/467e367c-2250-48d4-8a2c-3c278e1b299a/jobs/2085
Thank you so much in advance 🙏🏼

@dgonzalezr will be fixed with the next version. See #1047


keepClassName

As a separate option in plugins.
Original comment by @jacott here - https://github.com/babel/babili/issues/54#issuecomment-252131881
We now have keepFnName as an option in Mangler and DCE. What are your thoughts about having a separate keepClassName ?

@boopathi Looks interesting. I can work on this.


Allow to customize the TSIG algorithm and allow to omit the DNS port

The TSIG algorithm is currently hardcoded as HMAC-MD5. Any other key type cannot be used. Let's add an environment variable that allows to pass through the pseudo domain name used for the HMAC type.
Also append the default DNS port to the target nameserver to be updated, in case none is specified. This is caused by the Go API wanting to know the port for the connection, but we know a sensible destination default.

Thank you for your contribution :)
I added an inline comment for you to review.


Support EIP3860 after Shanghai upgrade

With EIP 3860, the contract size limit should be doubled from 24KB to 48KB?
https://github.com/ItsNickBarry/hardhat-contract-sizer/blob/95b202e75df3cc3fe309332bb8fa2062ad9a6ea2/tasks/size_contracts.js#L22
-  const SIZE_LIMIT = 24576
+ const SIZE_LIMIT = 24576 * 2

I don't think so - that EIP appears to be adding a new, separate limit to initcode rather than deployed bytecode.  Perhaps the plugin could include initcode size in its report as a separate metric.


[remote-shuffle]Refactor

What changes were proposed in this pull request?
This PR

renames some classes, to be more clear, like ShuffleRemoteSorter -> RemoteUnsafeShuffleSorter
Eliminates some comments containing 'HDFS', which is actually general(for all Hadoop compatible filesystem), resolving https://github.com/Intel-bigdata/RemoteShuffle/issues/46
Uses vanilla Spark's configuration as much as possible, avoiding to create new ones, like -> turn to using spark.reducer.maxBlocksInFlightPerAddress

How was this patch tested?
Existed tests

cc @carsonwang Could you please help me review this?


Adding re-learn link to libraries section

There were some weird line endings in the file as well, I cleaned them up!

Thanks!


🐛 Bug Report: common-library plugin not being added

📜 Description
Hi, I created a plugin that can be used for frontend and backend doing this command yarn backstage-cli create --select plugin-common
I creates perfectly but when I try to add it to the backend it doesn't work. It says not found.
👍 Expected behavior
I expectedit to be added to either frontend and backend.
👎 Actual Behavior with Screenshots
Not found.
👟 Reproduction steps

yarn backstage-cli create --select plugin-common
yarn --cwd packages/backend add

📃 Provide the context for the Bug.
No response
🖥 Your Environment
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

@Rugvip https://github.com/backstage/backstage/issues/10283 saw you answered this could you please help me?

You need to run yarn --cwd packages/backend add PACKAGE_NAME you can find the package name in the created folders package.json. Does that still not work?

Hey I just encountered with this error and got stuck now.
Context - I created a common plugin, when trying to add using command yarn --cwd packages/backend add @internal/backstage-plugin-notifications-common, getting below error
❯ yarn --cwd packages/backend add @internal/backstage-plugin-notifications-common
yarn add v1.22.19
[1/4] 🔍  Resolving packages...
error An unexpected error occurred: "https://registry.yarnpkg.com/@internal%2fbackstage-plugin-notifications-common: Not found".
info If you think this is a bug, please open a bug report with the information provided in "/Users/athira.sabu/Athira/backstage/packages/backend/yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/add for documentation about this command.

@benjdlambert


Usage of '/wp' directory telling WordPress that ALL of WordPress is installed at web/wp/, including Plugins/Themes

Background

The "WordPress Root" directory is set in four locations: config/application.php, wp-cli.yml, web/wp-config-pantheon.php, and web/wp-cli.yml.
With the sym-link files running to the web/ directory, setting /wp as the WordPress directory is unnecessary within either the config/application.php and web/wp-config-pantheon.php.
The current Bootstrapping process also tells some WordPress internals that "All of WordPress is installed at web/wp/ instead of web/... including the themes.

Issue
When running a custom theme that relies on "base theme path" (specifically: Sage), none of the styles/js/etc. actually loaded except for those provided by WordPress itself. This is because WordPress attempted to load the styles via /wp/wp-includes/css/dist/block-library/style.min.css?ver=6.2,
What is affected?
I could honestly see this issue affecting anything that uses some sort of Compile step in making themes/plugins/etc. work. GravityForms, ACF, etc. possibly?
The reason it affected the Sage Base theme, for example, is because the base_path() function actually generates based on where the theme is in relation to the codebase, not where WordPress is.

The "working" directory should have been /code/web/app/themes/THEME_NAME/public/STUFF
The "working" directory was being rewritten to load as /code/web/wp/app/themes/THEME_NAME/public/STUFF.

(Text from Screenshot: <link id="app/0-css" rel="stylesheet" href="https://integration-wp-composer-sage-base.pantheonsite.io/wp/app/themes/sage-base/public/css/app.874a0d.css" media="all">)
Recommendation
The sym-link solved the problem that the various wp/ additions were trying to solve in a non-breaking manner.  Worth testing, but I think removing all of them except web/wp-cli.yml would be the correct next-steps.
(The root-level wp-cli.yml would just point to web instead of web/wp. I could potentially see keeping the web/wp-cli.yml around to limit CLI-based triggers and actions potentially failing.)

Tracked internally as BUGS-6130

Clarification:

Both of the wp-cli.yml files should be left alone. 🗡 ❤
The WP_SITEURL within .env.local defaults to /wp as well.

We'll pretend I didn't unknowingly break WP-CLI for a day without realizing it.

Closing in favor of our internal tracking ticket in preparation of moving this repo to private (see https://github.com/pantheon-systems/wordpress-composer-managed/issues/77#issuecomment-1758207493). It's on the GA roadmap.

Fixed in #132


NRF52832 - PWM not working after zephyr update

Describe the bug
Hi, after may be half a year I updated zephyr (pull zephyr, west update) and PWM stopped working on my custom board, so I tried original nrf52dk_nrf52832 with blinky_pwm and that is also not working.  I also tried completely new zephyr installation and that didn't help.
To Reproduce
Steps to reproduce the behavior:

install latest zephyr
cd  zephyr\samples\basic\blinky_pwm
west build -b nrf52dk_nrf52832
west flash
LED1 is always OFF

Expected behavior
LED1 should fade
Logs and console output
[00:03:01.386,840] <dbg> pwm_nrf*** Booting Zephyr OS build v2.7.99-2678-g0813d09df968  ***
PWM-based blinky
Calibrating for channel 17...
[00:00:00.271,789] <err> pwm_nrfx: Prescaler for period_cycles 16000000 not found.
[00:00:00.280,426] <err> pwm_nrfx: Prescaler for period_cycles 8000000 not found.
[00:00:00.288,970] <dbg> pwm_nrfx.pwm_nrfx_pin_set: pin 17, pulse 2000000, period 4000000, prescaler: 7.

Environment (please complete the following information):

OS: Windows 10
Toolchain: gcc-arm-none-eabi-9-2020-q2-update-win32
Commit SHA or Version used: 0813d09df968c75ab38aafd39818cb7129ca8e64

Thanks for help,
Martin

hal_nordic update helped, thanks


Tensorboard logs are not showing in HF Hub

I can see the tensorboard scalar charts in my local correctly, but they are not showing in HF Hub. How could I fix it?
Buggy HF Hub tensorboard page: https://huggingface.co/keremberke/yolov8s-forklift-detection/tensorboard
Related discussion: https://discuss.huggingface.co/t/possible-bug-in-hf-hub-tensorboard-logs-not-showing/30149
I started using the great tensorboard feature of the hub after seeing one of @merveenoyan's posts 🚀 Maybe she can help me with this 🤗

thanks a lot @fcakyon for letting us know, indeed I managed to reproduce it with another repository: https://huggingface.co/merve/vq-vae/tensorboard doesn't appear on older models but with the ones that we upload recently @severo

We had an issue with event on tensorboard this week-end, it's solved now

Yes, we are recovering all misses events. Should be done by now

This seems like it's an ongoing issue, Tensorboard charts seem to load only intermittently.

@severo @rtrompier

Yes, I am also having the same issue right now. When should we expect a fix? As somebody already mentioned above, it seems to be working for some and not others!
Here is a link to a repo, where I tried to play with different setups. But none works! https://huggingface.co/YusuphaJuwara/nli-fever-v2/tensorboard
Note that I'll delete the repo soon, since it was only for testing!
Looking forward to a fix soon! Thank you!

Hmmm, could you share the dataset again? For the two datasets mentioned in https://github.com/huggingface/hub-docs/issues/632#issuecomment-1407712108, the tensorboard traces are empty (the file only contains 40 bytes: https://huggingface.co/keremberke/yolov8m-pothole-segmentation/tree/main and https://huggingface.co/keremberke/yolov8m-table-extraction/tree/main)

I think it's fixed now. Closing.

Thanks! It works now! Love your HF!

Seems the issue is still there. The link provided still cannot see any data:
https://huggingface.co/keremberke/yolov8m-table-extraction/tensorboard
https://huggingface.co/keremberke/yolov8m-pothole-segmentation/tensorboard
I am also coming across this issue myself.

Also here: https://discuss.huggingface.co/t/tensorboard-does-not-load-on-hub-loads-locally-tfevents-files-are-uploaded-to-hub/74917/4

I am also experiencing this issue.


Bump OCM SDK to v0.1.209

This release adds a new 'waiting' cluster state used to wait for user
configuration of operator roles and OIDC provider.

/lgtm
no issues creating sts cluster :+1:


fix(client): Handle expire value for easyrsa version 3

Pull Request (PR) description
EasyRSA version 3 and above has renamed some environment vars and then source by default the "vars" file. So client.pp allow override the "expire" for certificates but do not handle this change.
You can notice here that the "vars" template handle correctly the change https://github.com/voxpupuli/puppet-openvpn/blob/master/templates/vars-30.epp#L59 but not client.pp

Hi, thanks for the PR! Please rebase after https://github.com/voxpupuli/puppet-openvpn/pull/393 got merged

Hi, thanks for the PR! Please rebase after #393 got merged

Hello It's done :)

Hello, Is this ok to be merged ?


Error while running dev

Operating system: Unix
When I run npm run dev, the following error comes:

That EADDRINUSE error suggests the problem might be you have some stale processes running?

Yup, thanks its working now.

OK.    Thoughts?

That would be great! I actually installed a fresh setup, don't have that error anymore.

@timabbott @akashnimare I suggest we should close this issue. The behavior is inherited from electron and nodejs and there is nothing we can do about it. It's not feasible to catch the error by process.on('uncaughtException') in main.js because electron terminates before running that script.

Cool, thanks for investigating @geeeeeeeeek!  @fragm3 if you're interested in helping further here, you could add a troubleshooting guide to our docs that mentions this.

Solution for this problem -

First get the process ID (PID) of the process running on that port (say 30080):

lsof -i tcp:30080

This will return something like:
COMMAND   PID    USER   FD   TYPE             DEVICE SIZE/OFF NODE NAME
node     5738   You   11u  IPv6 0xac745b2749fd2be3      0t0  TCP *:ddi-tcp-1 (LISTEN)

Kill the PID using -

kill -9 5738


disambiguation character

In some files, we still have the "=" sign as disambiguation character, e.g. At=hǝr in manahimanuk.xml.
The new version of the TG (presently available as .docx file in the guides section of the repo project-documentation, soon to be released on HAL-SHS), we have had to decide to adopt ":" instead of "=".
Please check all your files, make any necessary changes of = to : and respond to this issue with "done" when you're done.

Dear all: I haven't received any responses so far.... Could you let me know once you have thoroughly re-read the relebant parts of the current TG (v3) and made any necessary adjustments in "your" files? Thanks.


Frontend : implement one interface

Material UI implementation
login page
list of series

As you can see, I didn't understand anything in this code. It's not your fault, but mine.
Did write all those things or are they generated?

I wrote all those thing


buildpack<EMAIL_ADDRESS>is broken

buildpack<EMAIL_ADDRESS>is broken
refer: https://github.com/paketo-buildpacks/ca-certificates/issues/235
<EMAIL_ADDRESS>dependency cause build to fail with below error message
Expected Behavior
Should be able to build
Current Behavior
Build failed with error message

message: 'validating buildpack<EMAIL_ADDRESS>stack
  io.buildpacks.stacks.jammy is not supported'

Possible Solution
upgrade to<EMAIL_ADDRESS>Steps to Reproduce
try any getting started guide in kpack/pack
Motivations
Getting started tutorial is itself failing and unable to use java buildpack because of its dependency to<EMAIL_ADDRESS>

The fix for this is in<EMAIL_ADDRESS>and that's been merged into the main branch, so the fix will go out in the next release of the paketo-buildpacks/java buildpack. That is usually every Friday.

This has been released. Closing.


Added Huffman Encoder in java

No description provided.

No description provided.

Do this format please
HuffmanEncoder/java/HuffmanEncoder.java


Added support for relative paths, __dirname no longer necessary (#41 followup)

Follow up on #41

Got relative paths working on all the node versions 0.6, 0.8, 0.10 etc.
Used stack trace to get the caller instead of module.parent.filename which caused issues with cached modules

(The property name and absolute path of required file are also passed to the resolve function as in #40)

Does this work for folk who invoke their Node.js with node --stack-trace-limit=0?

Also, please check that this work for the folks who use chakracore Node.js and JXCore, both of which I frequently encounter. From when I recall, those don't emulate things that are V8-specific, which you are doing in Error.prepareStackTrace. This seems to cause issues when I tried to do something similar with modules in the past. I wonder if those two runtimes support Error.prepareStackTrace now or not.

JXCore simulates the V8 JavaScript stack trace API
I found a test that does the same as I am doing in my code: jxcore/test/simple/test-prepare-stack-trace.js
function getCallerFile() {
  var oldPrepareStackTrace = Error.prepareStackTrace;
  Error.prepareStackTrace = function(err, stack) { stack.dd = 3; return stack; };
  var stack = new Error().stack;
  Error.prepareStackTrace = oldPrepareStackTrace;

  return stack[2] ? stack[2].getFileName() : undefined;
}

Great! This will be awesome if this is usable in the ecosystem these days :) Now I just wonder about chakracore, which is the backend being worked on within the Node.js foundation itself as part of the VM neutrality project. They have nightly builds on Node.js here: https://nodejs.org/download/chakracore-nightly/

https://github.com/nodejs/node-chakracore#how-it-works

To enable building and running Node.js with the ChakraCore JavaScript engine, a V8 API shim (ChakraShim) is created on top of the ChakraCore runtime hosting API (JSRT). ChakraShim implements the most essential V8 APIs so that the underlying JavaScript engine change is transparent to Node.js and other native addon modules written for V8.

And I also find all the properties and functions that I use in the source code and tests. So it shouldn't be a problem.

Cool. I mentioned those two runtimes, because the last time I published a module using Error.prepareStackTrace lost of people complained :) I'll see if I can test it if you're not able to to make sure that it does indeed work. Once we know this is working, I'll go though the code to review the code itself, since we know we've entered working territory :D !

Just ran the test with chakracore 7.0.0-pre10 and it worked :D


Windows build

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Trying to get recipe modified to build on Windows.  Used bld.bat from gobject-introspection-feedstock and patterned changes on meta.yaml from there also.
Partially building. Not sure what's wrong yet though. All suggestions appreciated.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

The top level meta keys are in an unexpected order. Expecting ['package', 'source', 'build', 'requirements', 'outputs', 'about', 'extra'].

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to {} for you but ran into some issues, please ping conda-forge/core for further assistance.

@conda-forge/core, a little help here please. Thanks!

You don't want any m* pkg-config.. You want one that looks in the msvc directories instead.

You don't want any m* pkg-config.. You want one that looks in the msvc directories instead.

Where do I get this then?
(base) C:\Users\Tim>conda search "*pkg-config*"
Loading channels: done
# Name                       Version           Build  Channel
m2-pkg-config                 0.29.1               1  pkgs/msys2
m2-pkg-config                 0.29.1               2  pkgs/msys2
m2w64-pkg-config              0.29.1               1  pkgs/msys2
m2w64-pkg-config              0.29.1               2  pkgs/msys2

You may need to build it, is it definitely essential? I am not familiar with meson, how do you tell it where to find things and what compilers to use etc?

I'm actually not very familiar with meson either (or building with msvc either for that matter - I'm mainly a Python programmer.) I'm just following the success @pkgw has had with building gobject-introspection-feedstock and glib-feedstock on Windows.
Re: this PR

I put m2w64-pkg-config back instead of m2-pkg-config. That finds the libraries and things get compiled.
The error I'm finding is with ninja test in bld.bat. If I comment that out, gst-plugins-base gets build, but the build for the  gstreamer output fails.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to {} for you but ran into some issues, please ping conda-forge/core for further assistance.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

How do I avoid the win_c_compilervs2008 check?

You can ignore it

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Can I ask what the status of this is? We're interested in running GStreamer on Windows, and are already using anaconda for everything. The current build status says the windows build is disabled. Thanks!

@nmaxwell I think the status is about what you see here — there's been interest but no one's pushing this over the finish line.

I'd love to get back to this, because I need it too, but I realized that I'm not the right person to be doing this. I forget what I got hung up on, but I kept running into problems building this.
I think that the build process for gstreamer has progressed a bit though, so it may be easier.
For now, I'm using msys2 for gstreamer and gst-python.

cc @seanyen (in case this is of interest to you 😉)


Example of the iteration by all model points

Hello. I have the DXF file and want to get all points this file describes. Provide any example for this, please.

Hello.
I implemented ParsePoint() at parser.go and add an example program at example/point.
https://github.com/yofu/dxf/blob/431b39efb10e93f5ab6aa99b01f9542b21a5df69/example/point
Please refer to it :)

Thank you.

Something wrong with code
package main

import (
	"fmt"
	"log"

	"github.com/yofu/dxf"
	"github.com/yofu/dxf/entity"
)

func main() {
	drawing, err := dxf.Open("point.dxf")
	if err != nil {
		log.Println(err)
	}

	fmt.Printf("Layers: %v\n", len(drawing.Layers))
	fmt.Printf("Sections: %v\n", len(drawing.Sections))

	for _, e := range drawing.Entities() {
		if p, ok := e.(*entity.Point); ok {
			fmt.Println(p.Coord)
		}
	}
}

Console:
$ ./dxf
2017/04/17 20:05:13 line 1935: unknown entity type
Layers: 1
Sections: 6
$


Does not work with Android 12

I tested it on Android 10 and Android 11 but with Android 12 it crashes on all my testing devices :-(. Does anyone else have this problem?

Thanks for opening the issue. I'll have a look asap

Works now on android 12 with v1.0.6


Use VLA for temporary buffer in fcb_append to prevent stack overflow

Make use of a variable length array as temporary buffer in fcb_append to
prevent stack overflow in case flash alignment is bigger than 2 bytes.

Is there a limit to the size of fcb->f_align?

Is there a limit to the size of fcb->f_align?

Theoretically no, but as far I have seen 4 or 8 bytes are common values.

So why not unconditionally increase the size of tmp_str to 8 bytes (or whatever the largest needed value is) to avoid the VLA?

or whatever the largest needed value is

Now that is the problem as it depends on the flash device used.
It is not because 4 & 8 bytes are common values that there is not exotic device out there that uses a different size.
Alternative approach would be to increase the array size and put a if statement in place to check the size at run time and in case the array is to small return an error.

My fear is that such exotic device could then cause a stack overflow at run time anyway - there is no easy way to check how much free stack space there is, so VLA can be dangerous.
That's why Linux purged them (and they also generate more complex code than just a fixed-size buffer), I'm not sure what's the policy about them in Zephyr.

@benpicco updated the code to use a static buffer of 8 bytes

closing and reopening to trigger stale ci jobs


@JsEnum types cannot be used in a union

Steps to repro - use an externs file with an enum that is referenced by a union type. Roughly:
/**
 * @enum {number}
 */
foo.bar.SomeEnum = {
  ONE: 0,
  TWO: 1,
  THREE: 2,
  FIVE: 3,
  SEVEN: 4
};

/**
 * @type {foo.bar.SomeEnum|string}
 */
foo.bar.ObjectWithProperties.prototype.thing;

The error is when j2cl tries to transpile the sources into js, it is reported that the union type is attempting an instanceof on the enum instead of just treating it like a number (which in this case is all it is).
Easy workaround: rewrite ObjectWithProperties.thing's type to be {number|string}, though that defeats the purpose a bit.
Proposed fix (not having given it great thought): The union type should instanceof based on the type label: in this case, number. This seems to be a bit difficult to implement though, since the original @enum might be already made into a java type? The jsinterop.generator.model.Type instances don't keep track of the type label at this time, so that would need to be added? It could potentially also be helpful in general, j2cl itself could implement $isInstance with this knowledge if it were part of the @JsEnum annotation?
Easier error reporting option: instead of waiting for j2cl to fail the generated java, something like this could throw when the generator is trying to emit the union's isFoo() method:
index b72a70a..c9d2e7b 100644
--- a/java/jsinterop/generator/visitor/UnionTypeHelperTypeCreator.java
+++ b/java/jsinterop/generator/visitor/UnionTypeHelperTypeCreator.java
@@ -262,6 +262,10 @@ public class UnionTypeHelperTypeCreator implements ModelVisitor {
       return new ArrayTypeReference(OBJECT);
     }

+    if (typeReference.getTypeDeclaration() != null && typeReference.getTypeDeclaration().isEnum()) {
+      throw new IllegalStateException("Can't use @enum in instanceof, so cannot construct union type");
+    }
+
     // remove Type parameters
     if (typeReference instanceof ParametrizedTypeReference) {
       return ((ParametrizedTypeReference) typeReference).getMainType();

Riffing on this idea a bit more:

It could potentially also be helpful in general, j2cl itself could implement $isInstance with this knowledge if it were part of the @JsEnum annotation?

This could be handy too, in cases like
/**
 * @enum {string}
 */
MyEnum = {
 "Orange":"Foo",
 "Blue":"Bar"
}

where the string keys don't match the enum name or the ordinal doesn't match the value, so you can either do the "proper" java thing and use MyEnum.values()[n] or MyEnum.valueOf(str), or you can do the "its JS, I do what I want" thing of Js.<MyEnum>cast("Foo").

IIRC, we discussed earlier that externs should not have enum definitions. if we are not handling the correctly we should probably have a good warning about it.
I think Closure is willing to accept patches for converting such enums to number.
@jDramaix please correct me if I'm wrong.

Perhaps jsinterop-generator should drop support for @enum, as it currently (correctly) creates native @JsEnum types? I think treating those enum types as if they were the aliased type could make sense too?
Related: I'm having trouble with @typedef as well, getting errors when those are handled. There appears to be explicit support in jsinterop-generator for this (for example, jsinterop.generator.closure.visitor.AbstractClosureVisitor#acceptTypedef), though I don't see tests for it (so I'm unclear of the typedefs in closure's externs are handled wrong, or written wrong). That said, if this typedef mechanism worked, perhaps @enum code could behave a bit like that?

IIRC, we discussed earlier that externs should not have enum definitions. if we are not handling the correctly we should probably have a good warning about it.

Externs used in Elemental2 should not use closure enum as they represent native api. If your extern represents a closure library, it can use closure enum.

 Wouldn't it just be the closure library as input at that point?
Assuming so, could the jsinterop-generator tool detect the presence/absence of @externs in the file to know how it should generate the enum?
Concrete example of "closure-compiler provided externs, which definitely do not refer to a closure-library": https://github.com/google/closure-compiler/blob/master/contrib/externs/maps/google_maps_api_v3.js#L4713-L4728

Can you clarify the distinction then you were making, where "externs used in elemental2 should not use closure enum"? I'm afraid I'm not understanding the point you were trying to convey.

Elemental2 is the jsinterop version of apis provided by the browser. Browser code is not closure annotated code and they never define a closure type enum.
Google maps is not an api provided by a browser but by a third party javascript library. As this library is written using closure type system, it can define closure enum type.

I don't think Google Maps vs Browser change much.
I think it is still controversial to define enum in this case; this is because the extern should never be the source of truth of those values and defining them on enum does that.

@jDramaix right - I'm filing this issue against jsinterop-generator ("Generates Java annotated with JsInterop from JavaScript extern sources"), not elemental2. My only goal in this is to understand/document/expand the limitations of jsinterop-generator when trying to generate jsinterop sources based on arbitrary externs, not to specifically tailor this work to browser apis.

So what is the resolution here? Goktug indicated that a change could be made on closure, but I don't quite follow what that change would be:

I think Closure is willing to accept patches for converting such enums to number.

Would this mean that in generating externs, you would get number instead of the @enum type? Would tsickle also be modified to emit number params instead of enums when externs are generated?
And for projects which have closure JS side by side with Java (see #44), is it expected then that @enum would be outright removed? Or is this project not intended to support closure js/ts to be used with Java (not as externs, but to let j2cl java be compatible with those existing closure js types)?

I'm not following most of your questions/conclusions.
You can just send a patch to closure extern to make it a number.

@gkdn ok, that was one of the options I was trying to offer, sorry if that was not clear. Are you saying that generally when we find an @enum in closure externs it should be a @const with @type {number} members?

Yes, that's what we were told earlier.

Thanks, I appreciate the clarification.


Inconsistent aws_route behavior when using NAT gateways

Expected Behavior
VPC is created with <IP_ADDRESS>/0 routes set to route to NAT gateways.
Actual Behavior
AWS routes are created, but Terraform cannot find the created resources. Retrying would lead to the same error.
Output example when one route is created successfully and other fails:

module.company-vpc.aws_route.private_nat_gateway[0]: Creation complete after 0s (ID: r-rtb-3ef046421080289494)
module.company-vpc.aws_route.private_nat_gateway.1: Still creating... (10s elapsed)

Output example 1:

Error: Error applying plan:
2 error(s) occurred:

module.company-vpc.aws_route.private_nat_gateway[1]: 1 error(s) occurred:

aws_route.private_nat_gateway.1: Error finding route after creating it: Unable to find matching route for Route Table (rtb-571db42b) and destination CIDR block (<IP_ADDRESS>/0).

module.company-vpc.aws_route.private_nat_gateway[0]: 1 error(s) occurred:

aws_route.private_nat_gateway.0: Error finding route after creating it: Unable to find matching route for Route Table (rtb-8302abff) and destination CIDR block (<IP_ADDRESS>/0).

Output example 2:

Error: Error applying plan:
2 error(s) occurred:

module.company-vpc.aws_route.private_nat_gateway[1]: 1 error(s) occurred:

aws_route.private_nat_gateway.1: Error creating route: RouteAlreadyExists: The route identified by <IP_ADDRESS>/0 already exists.
status code: 400, request id: cfbc621a-0b44-4e99-ae6e-bca550e6b922

module.company-vpc.aws_route.private_nat_gateway[0]: 1 error(s) occurred:

aws_route.private_nat_gateway.0: Error creating route: RouteAlreadyExists: The route identified by <IP_ADDRESS>/0 already exists.
status code: 400, request id: 932ad58d-0afb-44ef-9d43-589ddac154bf

Output example 3:

Error: Error applying plan:
2 error(s) occurred:

module.company-vpc.aws_route.private_nat_gateway[0]: 1 error(s) occurred:

aws_route.private_nat_gateway.0: Error creating route: RouteAlreadyExists: The route identified by <IP_ADDRESS>/0 already exists.
status code: 400, request id: 27ef394a-7259-4225-ad41-73ebcd234f3b

module.company-vpc.aws_route.private_nat_gateway[1]: 1 error(s) occurred:

aws_route.private_nat_gateway.1: Error creating route: RouteAlreadyExists: The route identified by <IP_ADDRESS>/0 already exists.
status code: 400, request id: 23ace76b-06f9-4f2f-bb0f-bb093bd3d0c5

Steps to Reproduce the Problem

Use VPC with private subnets and NAT gateway
Run Terraform

Specifications

Module Version: 1.26.0
Terraform Version: 0.11.3
AWS Provider Versions: ~> 1.8 & ~> 1.11

Show the Terraform configuration you are using to be able to help you.

Here is the code for network provisioning:
provider "aws" {
  version     = "~> 1.8"
  max_retries = 5
  region      = "${var.region}"

  assume_role {
    role_arn    = "${var.backplane_aws_sts_assume_role ? format("%s%s%s", "arn:aws:iam::", var.backplane_aws_account_id, ":role/Terraform") : "" }"
    external_id = "${var.backplane_aws_sts_assume_role ? var.backplane_aws_sts_assume_role_external_id : "" }"
  }
}

module "company-vpc" {
  source  = "terraform-aws-modules/vpc/aws"
  version = "1.26.0"

  name = "company - ${var.company_sitename}"
  cidr = "${var.company_vpc_cidr_block}"

  azs              = "${var.company_vpc_subnet_azs}"
  private_subnets  = "${var.company_vpc_private_subnet_cidr_blocks}"
  public_subnets   = "${var.company_vpc_public_subnet_cidr_blocks}"
  database_subnets = "${var.company_vpc_database_subnet_cidr_blocks}"

  enable_nat_gateway               = true
  enable_dhcp_options              = true
  dhcp_options_domain_name         = "ec2.internal"
  dhcp_options_domain_name_servers = ["AmazonProvidedDNS"]

  create_database_subnet_group = false
  create_vpc                   = "${var.vpc_create}"
}

resource "aws_vpc_peering_connection" "installer_backplane_peering" {
  peer_owner_id = "${var.installer_aws_account_id}"
  peer_vpc_id   = "${var.installer_vpc_id}"
  vpc_id        = "${module.company-vpc.vpc_id}"

  tags {
    Name = "VPC Peering between installer and company backplane"
  }
}

############# PEERING ACCEPTER ########################

provider "aws" {
  alias  = "installer"
  region = "us-east-1"
}

resource "aws_vpc_peering_connection_accepter" "peer" {
  provider                  = "aws.installer"
  vpc_peering_connection_id = "${aws_vpc_peering_connection.installer_backplane_peering.id}"
  auto_accept               = true
}

############# ROUTING FROM INSTALLER INSTANCE TO company PLATFORM ###############

data "aws_route_table" "installer_subnet_rt" {
  provider  = "aws.installer"
  subnet_id = "${var.installer_subnet_id}"
}

resource "aws_route" "installer_route" {
  provider                  = "aws.installer"
  route_table_id            = "${data.aws_route_table.installer_subnet_rt.id}"
  destination_cidr_block    = "${var.company_vpc_cidr_block}"
  vpc_peering_connection_id = "${aws_vpc_peering_connection.installer_backplane_peering.id}"
}

############# ROUTING FROM company PLATFORM TO INSTALLER INSTANCE ###############

data "aws_subnet" "installer_subnet" {
  provider = "aws.installer"
  id       = "${var.installer_subnet_id}"
}

resource "aws_route" "company_private_route_1" {
  route_table_id            = "${module.company-vpc.private_route_table_ids[0]}"
  destination_cidr_block    = "${data.aws_subnet.installer_subnet.cidr_block}"
  vpc_peering_connection_id = "${aws_vpc_peering_connection.installer_backplane_peering.id}"
  depends_on                = ["module.company-vpc", "aws_vpc_peering_connection.installer_backplane_peering", "aws_vpc_peering_connection_accepter.peer"]
}

resource "aws_route" "company_private_route_2" {
  route_table_id            = "${module.company-vpc.private_route_table_ids[1]}"
  destination_cidr_block    = "${data.aws_subnet.installer_subnet.cidr_block}"
  vpc_peering_connection_id = "${aws_vpc_peering_connection.installer_backplane_peering.id}"
  depends_on                = ["module.company-vpc", "aws_vpc_peering_connection.installer_backplane_peering", "aws_vpc_peering_connection_accepter.peer"]
}

resource "aws_route" "company_public_route" {
  route_table_id            = "${module.company-vpc.public_route_table_ids[0]}"
  destination_cidr_block    = "${data.aws_subnet.installer_subnet.cidr_block}"
  vpc_peering_connection_id = "${aws_vpc_peering_connection.installer_backplane_peering.id}"
  depends_on                = ["module.company-vpc", "aws_vpc_peering_connection.installer_backplane_peering", "aws_vpc_peering_connection_accepter.peer"]
}

.tfvars file values:

company_vpc_cidr_block = "<IP_ADDRESS>/16"

company_vpc_public_subnet_cidr_blocks = ["<IP_ADDRESS>/24", "<IP_ADDRESS>/24"]

company_vpc_database_subnet_cidr_blocks = ["<IP_ADDRESS>/24", "<IP_ADDRESS>/24"]

company_vpc_private_subnet_cidr_blocks = ["<IP_ADDRESS>/24", "<IP_ADDRESS>/24"]

company_vpc_subnet_azs = ["us-east-1a", "us-east-1b"]

company_sitename = "demo"

Looks good, but I have not had a chance to run this code myself (I will be offline the whole next week).
It can be that this module is not friendly-enough with VPC peering if you mix code and let Terraform/AWS to resolve dependencies automatically.
You can try to split the code into two steps - one create VPC and another create VPC peering and required routes.
Also, you can try to run terraform apply -parallelism=1 to reduce the amount of potential eventual inconsistencies and hopefully find the problematic resources.

I have tried to run the code with -parallelism=1 and still got some inconsistencies:
Error: Error applying plan:

2 error(s) occurred:

* aws_route.installer_route: 1 error(s) occurred:

* aws_route.installer_route: Error creating route: RouteAlreadyExists: The route identified by <IP_ADDRESS>/16 already exists.
        status code: 400, request id: b14ed14a-ea59-406a-a7d3-f30de12c0b2c
* module.frame-vpc.aws_route.private_nat_gateway[1]: 1 error(s) occurred:

* aws_route.private_nat_gateway.1: Error creating route: RouteAlreadyExists: The route identified by <IP_ADDRESS>/0 already exists.
        status code: 400, request id: 6e57e60b-6794-43d0-af71-b20e609f943d

Terraform does not automatically rollback in the face of errors.
Instead, your Terraform state file has been partially updated with
any resources that successfully completed. Please address the error
above and apply again to incrementally change your infrastructure.

Error: Error applying plan:

1 error(s) occurred:

* aws_route.installer_route: 1 error(s) occurred:

* aws_route.installer_route: Error creating route: RouteAlreadyExists: The route identified by <IP_ADDRESS>/16 already exists.
        status code: 400, request id: 073141e2-404c-4dba-a198-78943e930f82

Terraform does not automatically rollback in the face of errors.
Instead, your Terraform state file has been partially updated with
any resources that successfully completed. Please address the error
above and apply again to incrementally change your infrastructure.

I have tried specifying -target=module.company-vpc and omitting targets in next TF apply run (to create remaining resources), but got the same results.

I have tried to completely remove VPC peering and run Terraform. Still getting the same results, so the problem is not caused by VPC peering / VPC peering routes.

@antonbabenko I have tested v1.27.0 and the behavior is still the same. I have completely removed everything that isn't part of the terraform-aws-vpc module, here is the code as tested:
module "company-vpc" {
  source  = "terraform-aws-modules/vpc/aws"
  version = "1.27.0"

  name = "Frame - ${var.company_sitename}"
  cidr = "${var.company_vpc_cidr_block}"

  azs              = "${var.company_vpc_subnet_azs}"
  private_subnets  = "${var.company_vpc_private_subnet_cidr_blocks}"
  public_subnets   = "${var.company_vpc_public_subnet_cidr_blocks}"
  database_subnets = "${var.company_vpc_database_subnet_cidr_blocks}"

  enable_nat_gateway               = true
  enable_dhcp_options              = true
  dhcp_options_domain_name         = "ec2.internal"
  dhcp_options_domain_name_servers = ["AmazonProvidedDNS"]

  create_database_subnet_group = false
  create_vpc                   = "${var.vpc_create}"
}

Error: Error applying plan:
3 error(s) occurred:

module.company-vpc.aws_vpc_dhcp_options.this: 1 error(s) occurred:

aws_vpc_dhcp_options.this: Error waiting for DHCP Options (dopt-1eaeae66) to become available: InvalidDhcpOptionID.NotFound: The dhcpOption ID 'dopt-1eaeae66' does not exist
status code: 400, request id: 80f1419d-4c6c-48ed-8651-8ede00aa18bb

module.company-vpc.aws_route.private_nat_gateway[1]: 1 error(s) occurred:

aws_route.private_nat_gateway.1: Error finding route after creating it: Unable to find matching route for Route Table (rtb-a0385edc) and destination CIDR block (<IP_ADDRESS>/0).

module.company-vpc.aws_route.private_nat_gateway[0]: 1 error(s) occurred:

aws_route.private_nat_gateway.0: Error finding route after creating it: Unable to find matching route for Route Table (rtb-ed355391) and destination CIDR block (<IP_ADDRESS>/0).

Terraform does not automatically rollback in the face of errors.
Instead, your Terraform state file has been partially updated with
any resources that successfully completed. Please address the error
above and apply again to incrementally change your infrastructure.

The DHCP option set error happens in roughly 10% of runs also, but let's not tackle this in this issue.
Additional info:
I am authenticated using IAM role (instance metadata). Here is my aws provider config:
provider "aws" {
  version     = "~> 1.13"
  max_retries = 5
  region      = "${var.region}"
}

If skip DHCP related errors then there are no RouteAlreadyExists errors in your output. RouteAlreadyExists is gone, right? :)

Yes, but errors from original issue post are still around ( Unable to find matching route for Route Table )

RouteAlreadyExists is different and was fixed - this is an important information.
Now, the remaining problems could be fixed by tainting failed resources and recreating them (terraform taint ... and then terraform apply).

@antonbabenko I have successfully tested, and got consistent result with all aws_route resources. The trick is to set custom timeouts for create on all aws_route resources.
Here is what I changed (if you give me access I can create a PR):
resource "aws_route" "public_internet_gateway" {
  count = "${var.create_vpc && length(var.public_subnets) > 0 ? 1 : 0}"

  route_table_id         = "${aws_route_table.public.id}"
  destination_cidr_block = "<IP_ADDRESS>/0"
  gateway_id             = "${aws_internet_gateway.this.id}"

  timeouts {
    create = "5m"
  }

}

and:
resource "aws_route" "private_nat_gateway" {
  count = "${var.create_vpc && var.enable_nat_gateway ? length(var.private_subnets) : 0}"

  route_table_id         = "${element(aws_route_table.private.*.id, count.index)}"
  destination_cidr_block = "<IP_ADDRESS>/0"
  nat_gateway_id         = "${element(aws_nat_gateway.this.*.id, count.index)}"

  timeouts {
    create = "5m"
  }

}

I am creating and destroying the whole environment, not reapplying over some existing infrastructure. This is definitely affecting everyone, since I just want to create a VPC similar (even less complex) from the one in examples ( https://github.com/terraform-aws-modules/terraform-aws-vpc/blob/master/examples/complete-vpc/main.tf ).
The problem is that once NAT or IGW is provisioned, aws_route is created, and usually it takes AWS more than two minutes to create the route. Everyone who uses IGW/NAT is affected by this, but not everyone is having this module used to reprovision VPC ten times in a row.

Yes, I agree with what you are saying. Could you please submit a PR with timeouts?


Set up automatic publishing of Shiny app

Set up GitHub action to publish Shiny app to viz.datascience.arizona.edu

I made a draft PR #42 , but realized this won't work because actions are only run once they're merged I think.  I'll create a fork to test this on and make a new PR.


The structure of the 3D KiUNet model is incorrect

There are some problems with your 3D kiunet. I compared the code with your paper and found that the model details do not correspond. Maybe you should explain these questions. E.g: out = F.pad(out,[0,0,0,0,0,1]), scale_factor=(0.5,0.25,0.25)，(0.5,0.0625,0.0625), (2,0.25,0.25) and so on.
I hope you can reply or update the correct code， Thanks

The same problem!!

@jeya-maria-jose
out1 = torch.add(out1,F.interpolate(F.relu(self.inte3_2bn(self.intere3_2(tmp))),scale_factor=(2,16,16),mode ='trilinear'))
RuntimeError: The size of tensor a (6) must match the size of tensor b (2) at non-singleton dimension 2

I am facing the same issue anybody solved it??
out= torch.add(out1,F.interpolate(F.relu(self.inte3_2bn(self.intere3_2(tmp))),scale_factor=(2,16,16),mode ='trilinear'))
RuntimeError: The size of tensor a (6) must match the size of tensor b (2) at non-singleton dimension 2


Use linenoise-ng instead?

https://github.com/arangodb/linenoise-ng supports Windows.
Is it supposed to supersede this linenoise repo?

Why did you open a new issue???

Because that's a different issue

Hello, there are a few things I would add like UTF8 minimal support (discussed in another issue), however to add support for Windows is currently not the goal, given that there is a fork that handles it. Remember that Linenoise design is specifically focusing simplicity... So as long they are small self-contained additions that are very useful, I'll add them, but if I see complex and/or C++ code, it's unlikely I'll merge. It's great that @fceller is providing an alternative, but linenoise-ng cannot be considered a set of patches on top of linenoise, more a spin-off project that changed language, goals, and so forth.

@antirez (FYI)
There are two other modified versions of linenoise:
@yhirose 's cpp version includes UTF-8 support, and windows:
https://github.com/yhirose/cpp-linenoise
@sonophoto 's version is C, POSIX and supports UTF-8 and ANSI color.
https://github.com/Sonophoto/linenoise
I think we both watch your bug fixes and back-port them as appropriate. I think my last change was the history file permission issue.


I'd like to output to application/dicom+json

I tested by Postman tool (Rest), the result out is XML format (application/dicom+xml_.
How can I config to output JSON (application/dicom+json)?
Actual output (by Postman tool):

Expected output:
[
    {
        "00080005": {
            "Value": [
                "ISO_IR 100"
            ],
            "vr": "CS"
        },
        "00080020": {
....

Edit source
package org.rsna.isn.dcmrsbroker.core.spark.qido;
class QidoResponse
public Object send() throws Exception
// 20190809 Modified by Lam. Fix OHIF pass Accept: application/dicom+json
// Add || this.request.headers("Accept").equals("application/dicom+json"))
// if (this.request.headers("Accept").equals("application/json"))
if (this.request.headers("Accept").equals("application/json")
|| this.request.headers("Accept").equals("application/dicom+json")) {


chore: bdd-spring-1581424476 to 0.0.1

chore: Promote bdd-spring-1581424476 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Transform html files

hi,
Is it possible to transform html file via azure devops.
Thanks,
VP

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

ID: 315ff823-9992-9f9b-62a1-42ce622ed41c
Version Independent ID: 72ec3ffc-1dc9-b2dc-e3d4-4f14deae3720
Content: File transforms and variable substitution - Azure Pipelines
Content Source: docs/pipelines/tasks/transforms-variable-substitution.md
Product: devops
Technology: devops-cicd
GitHub Login: @RoopeshNair
Microsoft Alias: ronai

html is not supported. Can you share your scenarios for html transformation?


⚠ Bofrost.lu has degraded performance

In 88d652b, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2180 ms

Resolved: Bofrost.lu performance has improved in 22edc19 after 6 minutes.


(black screen source) null pointer check proposed fix for createMaterial coming from preload of MTL

When loading the .mtl file, there is an exception:
DartError: type 'Null' is not a 'bool' in boolean expression
dart-sdk/lib/internal/js_dev_runtime/private/ddc_runtime/errors.dart 297:3      throw
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 749:3  booleanConversionFailed
dart-sdk/lib/internal/js_dev_runtime/private/ddc_runtime/operations.dart 742:5  dtest
packages/three_js_simple_loaders/obj/mtl_loader.dart 401:15                      createMaterial
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 84:54               runBody
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 127:5               async
packages/three_js_simple_loaders/obj/mtl_loader.dart 266:18                      createMaterial
packages/three_js_simple_loaders/obj/mtl_loader.dart 260:13                      create
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 84:54               runBody
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 127:5               _async
packages/three_js_simple_loaders/obj/mtl_loader.dart 258:9                       create
packages/three_js_simple_loaders/obj/mtl_loader.dart 238:13                      preload
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 84:54               runBody
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 127:5               _async
packages/three_js_simple_loaders/obj/mtl_loader.dart 236:23                      preload
Please fix this bug. A possible solution is: instead of the block:
if (options != null && options?["invertTrProperty"]){
n = 1 - n;
}
to have the block:
if (options != null) {
if ((options!.containsKey("invertTrProperty"))
&& options!["invertTrProperty"] as bool) {
n = 1 - n;
}
}

Hi @aoncorici,
Thanks for finding the issue I will make sure it is implemented on the next update.


feat(cli): More detailed CSV compliance reports

Updating the CLI CSV output of the compliance reports to closely resemble the CSV exports from UI
JIRA: https://lacework.atlassian.net/browse/ALLY-534

make it so

make it so

make it sooooooooooo


Use dart2_constant for Dart2 compatibility.

@jakobr-google: I have no merge rights on this repo, could you please merge? (... and eventually release?)

I released 0.1.4 2 days ago.


Multiple OS targeting

My project has different compilation paths for Linux, Windows and OSX. What’s the best way to package these together in a single nuget, avoiding custom build scripts and nuspec files if possible?

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

ID: e80fe2fd-cbdb-699e-0dbb-a0a310bd372c
Version Independent ID: 266d08ae-344a-683b-11e2-143b704e21a8
Content: Open-source library guidance
Content Source: docs/standard/library-guidance/index.md
Product: dotnet
GitHub Login: @JamesNK
Microsoft Alias: mairaw

@jherby2k why don't using a multiple nuget runtime package approach (e.g. runtime.linux-x64.Company.Project.nupkg, runtime.osx-x64.Company.Project) and with a meta nuget package (e.g. Company.Project)
see https://github.com/google/or-tools/tree/master/ortools/dotnet for working examples

You may be interested by https://github.com/Mizux/dotnet-native

You may be interested by https://github.com/Mizux/dotnet-native


docs: update the PyPI URL

Summary
This pull request just updates the README.md file a bit.

Added hyphens in words where it's needed
Updated the PyPI URL, as the domain had changed a few months ago

Checklist

[ ] If code changes were made, then they have been tested

[ ] I have updated the documentation to reflect the changes
[ ] I have formatted the code properly by running task lint
[ ] I have type-checked the code by running task pyright

[ ] This PR fixes an issue
[ ] This PR adds something new (e.g. new method or parameters)
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[x] This PR is not a code change (e.g. documentation, README, ...)

... a few months ago ...

?
AFAIK this URL was used everywhere all the time I were in Python (at least last 3-4 years). :l
Just as a general question, nothing special

... a few months ago ...

? AFAIK this URL was used everywhere all the time I were in Python (at least last 3-4 years). :l Just as a general question, nothing special

It was actually 2018

Okay, that's fine, it doesn't really matter much when it changed, but yes the URL did change at some point.


Searching for specific arguments in classes / functions

Current:

Previous (better results):

Not sure if this helps, but perhaps using phrase search will improve the precision:

You can combine phrase search and normal queries in a single search request. In this case, MeiliSearch will first fetch all documents with exact matches to the given phrase(s), and then proceed with its default behavior.


Node12 depricated

To be updated with node16.

I got no idea what node.js to do with Solar2D, but it is the error message I get when I run "Build and Deliver". Here is an error msg, just in case:
build
Node.js 12 actions are deprecated. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/. Please update the following actions to use Node.js 16: actions/checkout@v2

I should've mentioned that I tried changing @v2 to @v3. Error code was still 1, no message about "node.js" version, though...

So probably your error is something else. You should review logs to know what your error is.

Did you check if such a simple change worked on your end and compilation for iOS worked? As for me I got the following msg:
ERROR: with exit code 1 and output:
error: The specified item could not be found in the keychain.
Node12 did not ask for keychain, node16 does. How to fix it?

This has nothing to do with the node. Please check which step is failing for you.

Btw, this should work now, try again


Attach tracking info to frontend responses

Added the configs and logic to attach tracking info to response headers regarding to the responding frontend.
Tests are WIP

Codecov Report

Merging #1040 into master will decrease coverage by 0.01%.
The diff coverage is 83.33%.

@@             Coverage Diff             @@
##             master   #1040      +/-   ##
===========================================
- Coverage     68.31%   68.3%   -0.02%
+ Complexity     4758    4755       -3
===========================================
  Files           379     379
  Lines         30189   30200      +11
  Branches       3847    3849       +2
===========================================
+ Hits          20624   20627       +3
- Misses         8524    8529       +5
- Partials       1041    1044       +3

Impacted Files
Coverage Δ
Complexity Δ

...src/main/java/com.github.ambry/rest/RestUtils.java
94.57% <ø> (ø)
72 <0> (ø)
:arrow_down:

...n/java/com.github.ambry/config/FrontendConfig.java
100% <100%> (ø)
1 <0> (ø)
:arrow_down:

...ambry.frontend/AmbryBlobStorageServiceFactory.java
91.66% <100%> (+0.36%)
2 <0> (ø)
:arrow_down:

...om.github.ambry.frontend/AmbrySecurityService.java
98.8% <50%> (-0.6%)
54 <0> (ø)

...github.ambry.frontend/AmbryBlobStorageService.java
92.69% <85.71%> (-0.14%)
43 <6> (+1)

...rc/main/java/com.github.ambry.store/BlobStore.java
83.06% <0%> (-1.62%)
69% <0%> (-2%)

.../src/main/java/com.github.ambry.store/Journal.java
90.76% <0%> (-1.54%)
24% <0%> (-1%)

...m.github.ambry.clustermap/HelixClusterManager.java
92.39% <0%> (-1.13%)
33% <0%> (ø)

...in/java/com.github.ambry.store/BlobStoreStats.java
70.62% <0%> (-0.65%)
102% <0%> (+1%)

...src/main/java/com.github.ambry.commons/BlobId.java
86.97% <0%> (-0.43%)
59% <0%> (-1%)

... and 4 more

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 b081ce1...484d87c. Read the comment docs.

Added verification of tracking headers in FrontendIntegrationTest and fixed typos pointed out in the previous review.


support replacing in list-type fields

This PR addresses https://github.com/edgars-supe/beets-importreplace/issues/7 by adding support for replacing in the list-type fields returned by the Musicbrainz database.
If the field being modified is a list, the replacements are applied to each item in the list.  This has the potential to (further) increase the processing time when importing based on the number of replacement rules, although pre-compiling and reusing the patterns should help.

Please let me know if you'd like documentation changes and/or additional test cases or fields in the tests.  Thanks!

Thanks for the feedback and suggestions!  Backward compatibility wasn't something I had considered (hooray sunny day case!), but will definitely be looking into it.  I anticipate it taking a little bit of time to test the different permutations to satisfy my curiosity:
beets 1.4.7 was released in May 2018, 1.4.8 added official support for 3.7 and 3.8 in May 2019, and it wasn't until 1.6.0 in Nov 2021 that 2.7, 3.4, 3.5 support was removed.
This means that Python 3.6 was the current version at time of beets 1.4.7 release and 2.7, 3.4, and 3.5 had not met their end-of-life and were supported.
I'm now extremely curious and (un)?lucky for me there are pre-built container images available for trying it out.  At a minimum, definitely with 3.6/1.4.7 and 3.7/2.0.0 (latest beets minimum Python version).

A slight complication- the list-like fields was only added in beets v2.0.0: https://github.com/beetbox/beets/commit/f72261e44f5501a870c43721883f1e4980003324
The plugin code which handles this case should be backward-compatible, but the test case helpers will need to handle this (to avoid unexpected keyword args to TrackInfo and AlbumInfo constructors :))
Also- apparently re.Pattern was only added in Python 3.7, which I'll see if I can get the code to handle.

Apologies for my late reply. And thanks for the extensive research, much appreciated!
I'll preface all of this by saying I'm not a Python dev, I don't use Python in my day-to-day, so I don't really have an opinion (or knowledge) on most of these things, but I'll do my best to look into things and make some decisions.
So, as for the Python version, I think it doesn't make sense to support <=3.6, since it is EOL. And since the plugin uses re.Pattern, I'll set the minimum Python version to 3.7. That will also cover @singledispatchmethod.
As for beets - I'll set the minimum version to 1.5.0, as it was released 3 years ago, officially supports Python 3.7 and will cover the two issues you mentioned.

Changes have been pushed to master. You can remove the compatibility things you've added. :)

Note that all Python versions up to and including 3.8 are now EOL. Though, as @edgars-supe mentioned

As for beets - I'll set the minimum version to 1.5.0, as it was released 3 years ago, officially supports Python 3.7 and will cover the two issues you mentioned.

I think it's a good idea to start with picking the oldest beets version to support - this will decide your Python constraint (actually you have little control here 😅).

Also @edgars-supe happy to see that your pick of beets==1.5.0 aligns with what I have in beetcamp: https://github.com/snejus/beetcamp/blob/b356bf6379ff887d78a281e88f5fc32f93287e3e/pyproject.toml#L30

@xe2io I think you can remove the Python <3.7 and beets <1.5.0 compatibility code. Then we'll get it merged in.


Peer arguments overloading

In Peer constructor port, network and relay is optional arguments. When port is Object and network isn't undefined arguments will be overloaded, but only port and network, not relay.
i.e. instead
// overloading stuff
if (port instanceof Object && !network) {
  network = port;
  port = undefined;
}

should be something like this
var test = Function.prototype.call.bind(Object.prototype.toString)
---
if (test(network) === '[object Boolean]' && !relay) {
  relay = network
  network = undefined
  if (test(port) === '[object Object]') {
    network = port
    port = undefined
  }
}
if (test(port) === '[object Object]' && !network && !relay) {
  network = port
  port = relay = undefined
}
if (test(port) === '[object Boolean]' && !network && !relay) {
  relay = network
  port = network = undefined
}

I'm right?
Of course it's not simple overloading, so it may be better to use opts as in Pool?
Also @param missing for relay.

+1
I think we should use an opts object probably

Agree with the options object.

Options object for params included in: https://github.com/bitpay/bitcore-p2p/pull/48


Upgrade Gradle to 6.9.2, the Gradle plugin to 0.5.3, the base Embulk to 0.10.36, and embulk-util-config to 0.3.1

Thanks!


Support JSON and XML atlasmapping-* files

Add support for reading json file from resourceAsUri
DOES NOT support JSON when specifying mappingUri in Exchange Header, or full content of JSON as exchange Header
Unit test added


if there is a turtorial about how to login by ssh keyfile or codeconfig

#keyfile
scala-ssh looks for files under ~/.scala-ssh
i use a template and it looks like this:
login-type  = keyfile
username    = root
keyfile     = @keyfile
host-name   = @sshHost
port        = 22
command-timeout = 5000
fingerprint = any

write the hostconfig file under ~/.scala-ssh
keyfile is your ~/.ssh/id_rsa
SSH(sshHost) { client => enjoy


chore: add github action to check PR labels

This PR enables a Github Action to check for PR labels. This will allow to enable lerna-changelog usage in future.

Codecov Report

Merging #3992 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3992   +/-   ##
=======================================
  Coverage   99.84%   99.84%
=======================================
  Files         183      183
  Lines        3157     3157
=======================================
  Hits         3152     3152
  Misses          5        5

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 0c042f3...4724b84. Read the comment docs.


Next/Prev keyboard nav in Quick Open

VSCode Version: 1.7.2 7ba55c5860b152d999dda59393ca3ebeb1b5c85f
OS Version: OSX 10.10.5 (14F2009)

next/prev keyboard nav in quickOpen inconsistent with other "inline prompt list" paradigms (e.g.; IntelliSense). Can this be resolved by user config or made default behavior? I'm not seeing a collision w/ existing commands.
Steps to Reproduce:

Hmmm, good spot… You can bind the following to ctrl+p / ctrl+n:
{
	"key": "ctrl+n",
	"command": "workbench.action.quickOpenNavigateNext",
	"when": "inQuickOpen"
},
{
	"key": "ctrl+p",
	"command": "workbench.action.quickOpenNavigatePrevious",
	"when": "inQuickOpen"
}

But careful, those 2 commands behave not exactly the same as when you simply use up-arrow / down-arrow. They open/execute whatever is selected as soon as you lift your finger from the modifier key (try ctrl+tab / ctrl+shift+tab for reference).
A "real" up/down to bind and with those keys as default would be nice to have, so 👍 for the feature request.

Awesome! I hate having to reach for arrow keys almost as much as reaching for mouse. ^ exactly what I was looking for. Thanks!

Is very subtle the diff b/n "real" up/down and using the bindings you shared. Holding modifier key and dancing around w/ n and p feels more natural to me (and saves me an enter keypress, yay!)
But the original reason i raised issue was for consistency (e.g.; with IntelliSense). For that reason, @kaiwood's argument for "real" up/down bindings & defaults is a good one.
Not closing in case maintainers wish to apply feature-request label.

This is probably obvious but would also love to see crtl+p and crtl+n implemented in the command palette as well. Thanks.

https://github.com/Microsoft/vscode/issues/7876


fix: Visualization settings were lost when editing a datasource from Explore

SUMMARY
When working on a visualization in Explore, if there were changes to the visualization that had not been "run" via the "Run Query" button, they would be lost when editing the related datasource via the modal on the page.
Why? The reducer that updated the form_data when a form component changed was broken in such a way that it would update the control settings to make the change visible, but not the form_data settings to make the change durable.
This PR retains the behavior of not updating the encoded data in the URL until the "Run Query" button is pressed.
Question for the community: are there any tests for any of the reducers in Superset? Can someone point me to an example? I'd love to write some tests for this change to avoid a regression.
Also, big props to @nytai for walking me through the Explore front-end code and helping me track this one down.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

@willbarrett there are a few tests for the exploreReducer here

Thanks @nytai - I'll add a few more.

I also noticed this yesterday and was surprised. Thanks for the fix!

@nytai @ktmud @etr2460 I believe this is ready for review, when you have a moment.

Codecov Report

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

@@            Coverage Diff             @@
##           master   #10092      +/-   ##
==========================================
- Coverage   70.63%   69.96%   -0.68%
==========================================
  Files         592      190     -402
  Lines       31191    18256   -12935
  Branches     3191        0    -3191
==========================================
- Hits        22032    12772    -9260
+ Misses       9050     5484    -3566
+ Partials      109        0     -109

Flag
Coverage Δ

#cypress
?

#javascript
?

#python
69.96% <ø> (-0.23%)
:arrow_down:

Impacted Files
Coverage Δ

superset/views/database/mixins.py
59.64% <0.00%> (-21.06%)
:arrow_down:

superset/utils/cache.py
48.00% <0.00%> (-20.00%)
:arrow_down:

superset/db_engine_specs/mysql.py
78.26% <0.00%> (-13.05%)
:arrow_down:

superset/views/database/validators.py
78.94% <0.00%> (-5.27%)
:arrow_down:

superset/views/database/api.py
84.09% <0.00%> (-3.41%)
:arrow_down:

superset/db_engine_specs/postgres.py
97.29% <0.00%> (-2.71%)
:arrow_down:

superset/views/database/views.py
87.17% <0.00%> (-2.57%)
:arrow_down:

superset/models/core.py
83.96% <0.00%> (-2.34%)
:arrow_down:

superset/jinja_context.py
80.00% <0.00%> (-0.96%)
:arrow_down:

superset/security/manager.py
88.69% <0.00%> (-0.35%)
:arrow_down:

... and 407 more

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 be6b9b8...fa0034a. Read the comment docs.


Maximum number of steps ignored and program doesn't terminate

I've noticed that sometimes, the maximum number of steps are reached but the program doesn't terminate? This makes it very challenging to automate running experiments.
You can see timestamps below.

Hi Wilka,
what script are you using to launch the program?
If you can give us the command line as well, that would help.

I'm using a custom builder that's a very close replica of R2D2.
One thing to note is that I launch multiple runs using the Ray toolkit for running multiple jobs.  Ray will launch the program in a sub-process. I'm not sure if that's part of the problem?

I think it's related to running multiple jobs at the same time. I think that when JOB1 calls lp.stop(), it might cancel processes for JOB2 (e.g. the coordinator), leading JOB2 to have running processes which are never killed?

I'm not very familiar with Ray but it may be due to the way ray and LaunchPad interact.
Did you know that you could pass multiple programs as a list to lp.launch? Would this allow you to do what you need purely with LP?

Are JOB1 and JOB2 started from the same process? If so then lp.stop() can be the problem indeed (as it uses signals to notify launching process to terminate started program(s)). But if Ray starts a code that calls lp.launch in a new process then this shouldn't be the problem.

Please reopen in case of any follow-up questions.


migrating to new api

closes #16 and unblocks #15 (right now the only arguments against publishing an initial version is migrating the api). We might want to run a few sanity test in the benchmark repo (just confirming the output is the same as speechpy) but I'd say its achieved MVP status after this gets merged

Migration to new builder pattern api is finished, when tested python side it shaves .03 seconds off repeated calls


Adding custom attribute handling

Hey Aziz,
I've come across this dilemma, where in my forum, we have quoting using blockquote tags; the quotes are converted correctly using this library, but there's this extra attribute "rel" that links to the user you're quoting:

 Something I said

I looked around and didn't see anything indicating that this is a standard convention, so I didn't think it appropriate to make a PR on the original repo. I'm not even sure it belongs here, as it seems unique to a plugin on the Vanilla platform. Do you have any ideas how this could be handled?

yea, rel is not a standard attribute, according to specs
I suggest manually pre-parsing i guess,
I hate to parse html manually, but here's a quick solution. Might run into issues if nested quotes.
var re = /
(.+?)<\/blockquote>/gi;
function replaceQuotes (content) {
   if (!re.test(content)) return content;
   return content.replace(re, '@$1 said
') + content; // leave the html in, this is just a pre-process
}

or i guess it wouldn't hurt to support an optional attribute to blockquote if you want,
you could support 'cite' || 'rel' || 'author' or something like that
somewhere here:
https://github.com/akhoury/html-md-optional_window/blob/master/src/md.coffee#L437

Since the author hasn't merged your PR in months, would you consider keeping this repo open?
-Brian

On Jun 4, 2015, at 9:10 AM, Aziz Khoury<EMAIL_ADDRESS>wrote:
yea, rel is not a standard attribute, according to specs
I suggest manually pre-parsing i guess,
I hate to parse html manually, but here's a quick solution. Might run into issues if nested quotes.
var re = /
(.+?)
/gi;
function replaceQuotes (content) {
if (!re.test(content)) return content;
return content.replace(re, '@$1 said') + content; // leave the html in, this is just a pre-process
}
or i guess it wouldn't hurt to support an optional attribute to blockquote if you want,
you could support 'cite' || 'rel' || 'author' or something like that
somewhere here:
https://github.com/akhoury/html-md-optional_window/blob/master/src/md.coffee#L437
—
Reply to this email directly or view it on GitHub.

sure.  i guess.
it's published on npm - might as well let it live.
don't appreciate the long name, but who cares


fixes in vocabularies

rename superseded specimentype vocab (now material sample object type with new uris)

this has old changes from Danny, I can't get the updates I need in here.


Integration with Azure App Service Authentication (Easy Auth)?

Azure App Service Authentication requires calling an /.auth/me endpoint from the client-side in order to obtain an API access token. The documentation (https://docs.microsoft.com/en-us/azure/app-service/app-service-web-tutorial-auth-aad) says that this can be managed by the ADAL library, but obviously I would prefer to use MSAL since it's the new standard. I'm not seeing any functions in the public API that would allow me to perform this sort of flow. Is it currently supported in MSAL?

The OpenID Connect UserInfo endpoint is not implemented on the v2.0 endpoint.
However, all user profile data that you potentially would receive at this endpoint is available from the Microsoft Graph /me endpoint.

I don't believe so.
The public API's don't give any indication of such support:
https://github.com/AzureAD/microsoft-authentication-library-for-js/wiki/Public-APIs
I grep'd the source for authorityInstance.AuthorizationEndpoint and it didn't look as if it had such a feature or flexibility built-in.

@watabtme I am not sure I understand your scenario. MSAL.js provides authentication with AAD and gets the user info from the id token claims. It does not make the API calls for you. Using the tokens it gets your app can call APIs. In your case, if you have already authenticated in your app using Easy Auth redirecting to AAD, why do you still need MSAL? Is there a reason your app cannot call the /.auth/me endpoint you mention directly like you would call any API endpoint?

@navyasric there is still a need for something like MSAL because Azure Easy Auth only provides the initial login logic. Once the client-side code calls /.auth/me, the application is now responsible for actually maintaining the auth state, refreshing the token when necessary, monitoring validity of the token, etc etc. In other words, all the things that MSAL provides (other than the very first login call).

I was able to get my MSFT Org based login working with this library and additionally getting an access token for App Authentication on an App Service/Azure Functions. The key is that although your primary login might be using v2, the app service auth request expects the v1 designation of resource access requests instead of scopes. If you try to access your resources using scopes similar as to defined below, you shouldn't have a problem.
protectedResourceMap
[
    'https://your-site-url-with-auth.azurewebsites.net',
    ['https://your-site-url-with-auth.azurewebsites.net/user_impersonation']
  ]

consentScopes
consentScopes: [
    ...,
    'https://your-site-url-with-auth.azurewebsites.net/user_impersonation'
  ]

This FAQ is what let me to the answer.
https://github.com/AzureAD/microsoft-authentication-library-for-dotnet/wiki/Adal-to-Msal#scopes-for-a-v1-application

Closing this issue since the above comment provides information to resolve this.


Escape some characters before finding files with globby

Sometimes file search bases will have special characters that will unintentionally change the pattern. Removing those special characters gives intended results.
Fixes #3019

Tests failed, and a commit was made with the wrong email.


Update NG to 1.6.12 to downgrade WG lib version

Task/Issue URL: https://app.asana.com/0/488551667048375/1206399526867562/f
Description
Ditto
Steps to test this PR
Smoke tests VPN

develop

#4103   👈

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

Join @aitorvs and the rest of your teammates on  Graphite


how do I define a datalist for an input field?

Is there a standard way of creating something like this in JSONForm?
<input list="browsers">

<datalist id="browsers">
  <option value="Internet Explorer">
  <option value="Firefox">
  <option value="Chrome">
  <option value="Opera">
  <option value="Safari">
</datalist>

Hi
Datalists are not implemented yet. Their support is still quite partial and buggy (https://caniuse.com/#feat=datalist) so I don't think it's important to implement as of now.
If you find some time and want to contribute, feel free ! ;)
Thanks


Mutually recursive functions are impossible to write

Description
Currently, it seems impossible to write mutually recursive functions, because the compiler is sensitive to the order to function definitions, and AFAICT there is no way to forward-declare a function. This appears to be an unfortunate side effect of functions being "just variables" (because in the case of variables, this order-sensitivity makes sense).
For example, the follwing code:
#let first(n) = {
    if n > 0 {
        second(n - 1);
    } else {
        [Ended in first]
    }
};
#let second(n) = {
    if n > 0 {
        first(n - 1);
    } else {
        [Ended in second]
    }
};
#first(10);

produces the following error:
error: unknown variable: second
  ┌─ impossible.typ:3:8
  │
3 │         second(n - 1);
  │         ^^^^^^

help: error occurred in this call of function `first`
   ┌─ impossible.typ:15:1
   │
15 │ #first(10);
   │  ^^^^^^^^^

I need a pair of mutually recursive functions for rendering a tree of elements where every element is either a leaf (stand-alone data) or a container. I could work this around by inlining the definition of one function to the other, but it's not pretty.
Reproduction URL
No response
Operating system
macOS
Typst version

[X] I am using the latest version of Typst

Duplicate of https://github.com/typst/typst/issues/744


Fix issue xml formatter

Description
currently the web-console will crash when failing to format an Xml string. This commit makes sure it wont crash and the a non formatted string will be returned.
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[ ] Attached issue to pull request
[x] Changelog entry
[x] Code is clear and sufficiently documented
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )

Reviewer Checklist:

[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Code is clear and sufficiently documented
[ ] Correct, in line with design

Processing this pull request

Merged into branches iso5, master in d57daa03fc40d45ec59058c4b06a6e1538a3d449


Align side effect retries with sdk-typescript

See https://docs.restate.dev/services/sdk/side-effects#manually-controlling-the-retry-policy

This discussion went on in https://github.com/restatedev/sdk-java/issues/156 and https://github.com/restatedev/sdk-java/issues/157


I'm so sorry sir,does your Pingtunnel support ICMPv6

I'm so sorry sir,does your Pingtunnel support ICMPv6

no, you can modify it to support


Need object resizing option

Being able to select and resize certain parts of pixels would be greatly helpful.
Thank you guys.

Yes I can help and do that. What do you need?
On Fri, Jan 12, 2024, 5:44 PM cyberfrogJ @.***> wrote:

Being able to select and resize certain parts of pixels would be greatly
helpful.
Thank you guys.
—
Reply to this email directly, view it on GitHub
https://github.com/piskelapp/piskel/issues/1115, or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2IEQMPUG2SO46G56KI65YTYOG4DJAVCNFSM6AAAAABBYZUWGGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGA3TSOBSG4ZDAOI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Rename db.name to db.collection.namespace

Changes
Deprecates db.name and db.redis.database_index in favor of db.collection.namespace
Removes db.mssql.instance_name (includes it into db.collection.namespace).
Open question:
where does the schema belong?

within name:  namespace = mydb, name = dbo.mytable , span name is SELECT mydb.dbo.mytable

If schema is not set in the query, then we won't have it (namespace = mydb  and name = mytable and span name is SELECT mydb.mytable
But it's a best practice to qualify table name with schema

within namespace: namespace = mydb.dbo, name = dbo.mytable,  span name is SELECT mydb.dbo.dbo.mytable

if schema is provided as a part of table name (best practice), we have duplication

The schema (at least sometimes) requires actual database call: postgres jdbc driver, SQL Server JDBC driver.
It's not a connection property per se, but a database server configuration, i.e. can be modified elsewhere and can't be cached on the client.
So in this PR schema stays inside table name (if users provided it in the query) and is not captured inside the database name.
Merge requirement checklist

[x] CONTRIBUTING.md guidelines followed.
[x] Change log entry added, according to the guidelines in When to add a changelog entry.

If your PR does not need a change log, start the PR title with [chore]

[x] schema-next.yaml updated with changes to existing conventions.

Was there any discussion before this PR in some SIG meeting or similar? Is there any issue associated with this PR?

Was there any discussion before this PR in some SIG meeting or similar? Is there any issue associated with this PR?

yes, this is the design approach discussed within DB semconv WG: https://docs.google.com/document/d/1zTi_Z4WBisytPnXTpwtnGOK-4Gv5oSTuMHvIQ8--5hg
It partially addresses #749, clarifies parts of #718, #714

if schema is provided as a part of table name (best practice), we have duplication

wdyt of just SELECT {db.collection.name} for the span name? I'm not sure we need db.collection.namespace in the span name, sort of like HTTP spans include the route but not the server name. you can always group span names further by db.collection.namespace if that's desired

where does the schema belong?

For SQL at least:

db.collection.name will be whatever the user has in the query (may be partially or fully namespaced, or not namespaced at all)
so the question is whether to capture schema in db.collection.namespace or not

it could be useful to capture in db.collection.namespace, since it gives you the "complete namespace" for the collection, which could be useful for grouping(?)
https://stackoverflow.com/questions/7942520/relationship-between-catalog-schema-user-and-database-instance refers to schema as a namespace, which makes me think that it may be more intuitive to include it in db.collection.namespace

As discussed in the last WG call, looked into what we can collect in .NET.
The DbConnection ADO interface has following relevant methods:

DbConnection.DataSource: name of the database server
DbConnection.Database: name of the database
DbConnection.GetSchema(string): returns a DataTable which contains metadata info about the objects in the db including schema and collection. Here is a super useful article on this.

I ran following code snippet with different DBs and libraries:
static void DisplayConnectionData(DbConnection connection)
{
    Console.WriteLine($"DataSource: {connection.DataSource}, Database: {connection.Database}");
    Console.WriteLine("============================");

    DataTable table = connection.GetSchema("Tables");
    foreach (System.Data.DataRow row in table.Rows)
    {
        foreach (System.Data.DataColumn col in table.Columns)
        {
            Console.WriteLine("{0} = {1}", col.ColumnName, row[col]);
        }
        Console.WriteLine("============================");
    }
}

Above code queries the Tables schema collection (so it only returns infos about tables). Here is a list about other collections (e.g. Procedures could be used for stored procedures).
MS SQL Server with System.Data.SqlClient
Output:
DataSource: <IP_ADDRESS>,50312, Database: master
============================
TABLE_CATALOG = master
TABLE_SCHEMA = dbo
TABLE_NAME = spt_fallback_db
TABLE_TYPE = BASE TABLE
============================
....similar info for all tables...

MySQL with MySqlConnector
Output:
DataSource: <IP_ADDRESS>, Database: test
============================
TABLE_CATALOG = def
TABLE_SCHEMA = information_schema
TABLE_NAME = CHARACTER_SETS
TABLE_TYPE = SYSTEM VIEW
ENGINE =
VERSION = 10
ROW_FORMAT =
TABLE_ROWS = 0
AVG_ROW_LENGTH = 0
DATA_LENGTH = 0
MAX_DATA_LENGTH = 0
INDEX_LENGTH = 0
DATA_FREE = 0
AUTO_INCREMENT =
CREATE_TIME = 12.04.2024 14:18:11
UPDATE_TIME =
CHECK_TIME =
TABLE_COLLATION =
CHECKSUM =
CREATE_OPTIONS =
TABLE_COMMENT =
============================
....similar info for all tables...

PostgreSql with Npgsql
Output:
DataSource: tcp://<IP_ADDRESS>:54251, Database: postgres
============================
table_catalog = postgres
table_schema = public
table_name = foo
table_type = BASE TABLE
============================

So, overall it seems, at least in ADO.NET we can get the schema. Issue I see with this is that this seems to query the database - not only that, but  connection.GetSchema("Tables") returns the schema info for all tables, which can be very high overhead.