feat(ado/infra): Create a release tag for the ADo extension

Details
Create a release (tag) in GitHub for the ADO extension on releasing.
Motivation
Keep track of the release.
Context

Pull request checklist

[ ] Addresses an existing issue: Fixes #0000
[x] Added relevant unit test for your changes. (yarn test)
[x] Verified code coverage for the changes made. Check coverage report at: <rootDir>/test-results/unit/coverage
[x] Ran precheckin (yarn precheckin)
[x] (after PR created) The Accessibility Checks (pull_request) check should fail. All other checks should pass.

Codecov Report

Merging #793 (132fc76) into main (b0db466) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 132fc76 differs from pull request most recent head c4cdca5. Consider uploading reports for the commit c4cdca5 to get more accurate results

@@           Coverage Diff           @@
##             main     #793   +/-   ##
=======================================
  Coverage   86.50%   86.50%
=======================================
  Files          34       34
  Lines         630      630
  Branches       57       57
=======================================
  Hits          545      545
  Misses         85       85

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 b0db466...c4cdca5. Read the comment docs.


Making variation selection easier to use and more general across the embed methods.

Swapping variation-id without scene-product-id or product-id.
Since we can iterate through the products and their variations, we don't need to store the main product-ids
<plattar-embed id="embed_viewer" scene-id="143d6c50-48fb-11ed-a0d1-33f25dd99e4e" embed-type="viewer" variation-id='<variation-id>'>

Using variation-sku as an attributes
if variation-id and variation-sku clash, then take the newest (and console log clash error)
<plattar-embed id="embed_viewer" scene-id="143d6c50-48fb-11ed-a0d1-33f25dd99e4e" embed-type="viewer" variation-sku='<variation-sku>'>

Swapping embed-type from viewer to facear updates in real time

Having any variation changes propagate upwards into the tag's config-state

@calrk feedback appreciated on my thoughts below
Swapping variation-id without scene-product-id or product-id
This is doable as a search feature internally in the renderer, however it does slightly complicate the implementation for launching AR without the renderer but its still doable.
Currently the following messenger interfaces are available. I also suggest streamlining and making all these interfaces available for all renderer nodes.

selectProductVariation(productID:string, variationID:string): Promise<void>;
sceneSceneProductVariation(sceneProductID:string, variationID:string): Promise<void>;

Solution
I recommend adding the following messenger functionalities (alongside the old ones). This ensures backwards compatibility.

selectVariation(variationID:string): Promise<void>; - Enables a single Variation using a Variation ID
selectVariationSKU(variationSKU:string): Promise<void>; - Enables a single Variation using a Variation SKU
selectVariationList(variations: Array<string>): Promise<void>; - Enables all Variations provided a list of Variation ID's
selectVariationSKUList(variationSKUs: Array<string>): Promise<void>; - Enabled all Variations provided a list of Variation SKU's

The new functions should map into new attributes available for <plattar-embed /> node

<plattar-embed variation-id="" /> - Calls selectVariation
<plattar-embed variation-sku="" /> - Calls selectVariationSKU
<plattar-embed variation-id-list="" /> - Comma separated list of ID's that calls selectVariationList
<plattar-embed variation-sku-list="" /> - Comma separated list of SKU's that calls selectVariationSKUList

Alternatively we can use only the following with dual-purpose functionality

<plattar-embed variation-id="" /> - Accepts either single ID or comma separated list of ID's, calls selectVariation or selectVariationList depending on input
<plattar-embed variation-sku="" /> - Accepts either a single SKU or comma separated list of SKU's, calls selectVariationSKU or selectVariationSKUList depending on input

Concerns
The SKU's are not systematically unique and can be duplicates. This places the burden on content-creation or content-curators to ensure the SKU's they enter into the CMS is all unique.
In cases where SKU's are duplicated (aka, two separate products with variations having same SKU) how should the system handle this? swap the variation on both?
Using variation-sku as an attributes
See above proposal
Swapping embed-type from viewer to facear updates in real time
This is possible by internally swapping out the controller and re-initialising the renderer, however should this maintain state changes done via messenger interface?
Having any variation changes propagate upwards into the tag's config-state
We can achieve this by having messenger callbacks (aka, calling messenger functions on the parent page called by renderer when interfaces are used)

Looks good, would like to see the performance impact of swapping from viewer to facear


Import hardly any street names

Hi followed all steps works wonderfully unitil zooming in to find there are no street names.
In the yaml files tranport layers are there
Dont think the street name data has been imported to postgres
How can i resolve this?
Very limited experience with this

Are they no street names or only a few?
Which area did you import? Which coordinates / streets are you looking at? I could imagine some problem with local translations, but this is just a guess.
Which version of makina-maps are you using? Did you clone it recently?

You can access the database with
cd openmaptiles
make psql

But I am not sure in which table the street names should go, as I was not able to set it up correctly.

@goody85
so you solved the problem of missing openmaptiles-tools by switching to another branch? Not sure if that can be the reason. There is a issue #52 which is closed, but I believe this still needs to be fixed.
Unfortunately due to the missing openmaptiles-tools I currently cannot reproduce the problem.
With the last command (database access) you could look around in the database if the street names were imported at all.
Unfortunately I do not know which tables the street names should be stored in.
With \dt you can see the tables, with select * from table_name_goes_here you can view the single tables. The should be something like transport_name tables and so on.
It might be best to wait until the problem mit openmaptiles-tools is fixed? Or maybe someone else has some advice ...

I found a resolve with it.
As i followed the instructions, coming to the import openmaptiles tools
were missing, so i grabbed a copy , but got the wrong version i didnt look
down the branch for yours. 5.2
Thank you for your help 😊
On Mon, 14 Feb 2022, 15:32 Frédéric Rodrigo, @.***>
wrote:

Street and road names are in transportation_name
https://openmaptiles.org/schema/#transportation_name
You can check with
select tags from osm_transportation_name_linestring;
—
Reply to this email directly, view it on GitHub
https://github.com/makina-maps/makina-maps/issues/55#issuecomment-1039212220,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB5LAFJ6FPUGD7GYVUFW623U3EMYRANCNFSM5OFDPWEQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Add licenseDetails to item info panel

What is it and who's it for?
People looking at digitised items
We should add the license detail to the side panel so that users get a brief description of what the license means, so they don't necessarily have to use the licence link for more information.
We currently expose the license type and credit in the side panel

We should also expose the information from the LicenseDetail

User story
As a researcher
I want to know about the license assigned to this work
So that I can use an item with confidence that I'm using it correctly.
Implementation
Acceptance criteria

'human readable' text description for the specific license is displayed under License: {XXX}

Info from @jennpb:
The credit problem:
When we designed our credit line, we only had Miro images to work with. We followed the recommendations as described here https://wiki.creativecommons.org/wiki/best_practices_for_attribution
Following the TASL format: Title Author Source Licence.
Now that we have more complex titling and the possibility of multiple named contributors (or complete absence of), this template doesn’t work quite right anymore.
Some examples of where it goes wrong:
https://wellcomecollection.org/works/sat2qh4k - contributor already part of the title; has additional two contributors
Credit: La pratique du jardinage / par m. l'abbé Roger Schabol, ouvrage rédigé après sa mort sur ses mémoires, par M.D*** [i.e. A.N. Dezallier d'Argenville] by  Source: Wellcome Collection Public Domain Mark
Archive - only shows title, no author https://wellcomecollection.org/works/bqd39jxt
Credit: Fingerprints. Wellcome Library. Attribution-NonCommercial 4.0 International (CC BY-NC 4.0)
Third-party attribution
https://wellcomecollection.org/works/jrjc2jb6
https://wellcome.slack.com/archives/G5QLV74NM/p1619798388002300
FYI @jtweed
This also applies to the 'Can I use this' section of the works page.

Another example of an archive that only shows title, no author:

I notice that when viewing a digitised archive item, the "Credit" line in "Licence and credit" uses the title, but this is often going to be nonsensical, e.g. in this one, we have a credit of "R-W". Anyone wanting to use the Oscar Wilde letter that comes up top of the list will be pretty confused! https://wellcomecollection.org/works/b7deha34/items?canvas=1 (edited)
https://wellcome.slack.com/archives/C8X9YKM5X/p1625136140058400


feat: add react 19 codemods

adds 7 react 19 codemods
adds ability to run ts transfroms from react-codemod CLI

Testing
Running with react-codemod
npx react-codemod remove-context-provider <path>
npx react-codemod replace-string-ref <path>
npx react-codemod remove-forward-ref <path>
npx react-codemod replace-use-form-state <path>
npx react-codemod use-context-hook <path>
npx react-codemod remove-memoization <path>
npx react-codemod replace-act-import <path>

Running with codemod
npx codemod react/19/remove-context-provider --target <path>
npx codemod react/19/replace-string-ref --target <path>
npx codemod react/19/remove-forward-ref --target <path>
npx codemod react/19/replace-use-form-state --target <path>
npx codemod react/19/use-context-hook --target <path>
npx codemod react/19/remove-memoization --target <path>
npx codemod react/19/replace-act-import --target <path>

This PR contributes 6 new codemods to react-codemods repository designed to streamline migration to React v19. In the root folder of v19 repo, there exists a README file that introduces Codemod CLI and how to run codemods both via Codemod CLI and react-codemod. in addition to that, each codemod has its own folder and for each codemod there exists a README that explains the transformation logic.
h/t @r4zendev for creating the React 19 codemods.

@acdlite , @rickhanlonii who can review this? i would like us to merge this PR, invite a few early adopters before we market it more broadly. thank you.

pnpm i -g codemod

codemod react/19/ref-to-arrow-function
codemod react/19/remove-forward-ref
codemod react/19/replace-use-form-state
codemod react/19/use-context-hook
codemod react/19/remove-memoization
codemod react/19/replace-act-import

@DmytroHryshyn can you allow edits to the PR?

@DmytroHryshyn can you allow edits to the PR?

Sure, added you to the repo @rickhanlonii

We do not recommend removing useMemo and use callback while the compiler is still experimental. The compiler may skip components for various reasons (violations of React’s rules, as-yet-unsupported syntax, etc) and a blanket code of to remove memoization could cause problems.
We strongly recommend waiting until the compiler is stable. We plan to provide a script that will only remove manual memoization where the compiler is running, skipping components/hooks where the compiler would bail out.

noted. we will remove it soon. thanks for flagging.

Thanks!


Use Router Module

I'm using @nuxt/router-module. I've tried to use nuxt-i18n too, but unfortunately, It doesn't work. What should I do?

Specify exactly what you want to achieve and what is not working.

@rchl I've been using nuxt-i18n since I launched my project, now I wanted to use @nuxt/router-module, after installing and configuring @nuxt/router-module, nuxt-i18n stopped working.

Specify exactly what you want to achieve and what is not working.

Maybe try switching modules order.

Maybe try switching modules order.
Otherwise please provide reproduction.

I can't change the order, i18n is located on modules and router-moudle is located on buildModules.

Put router modules in modules temporarily to see if that's the issue.
I assume it's the issue as the router module needs to augment routes first so that nuxt-i18n can see the routes that you've added through it.

Put router module in modules temporarily to see if that's the issue.
I assume it's the issue as the router module needs to augment routes first so that nuxt-i18n can see the routes that you've added through it.

I've done ordering and didn't work.
The default language (en) works now but when I try to switch language by switchLocalePath() function I get the following error

And when I try to switch the language manually by adding the language in the URL I get this 404 Not found error.

I don't know then. You'd need to provide reproduction repo. There is a lot of variables here (modules configuration) and you haven't specified them.

I don't know then. You'd need to provide reproduction repo. There is a lot of variables here (modules configuration) and you haven't specified them.

How can provide reproduction repo?

Create a minimal github repo. Or create Nuxt sandbox at http://codesandbox.io/

Hello i would like to re-open this issue.
@obaidsamadian  I feel you man i'm in the exact same situation here.
@rchl There is nothing to "reproduce". This are dead simple to understand. He's asking how one can plug the two module to be compatible with each other.
Router-module will disable Nuxt ability to generate routes by itself based on the page directory files. Instead we get back control over the Router file and create route ourselves.
This means the router instance is not the same anymore.
Unfortunately the i18n-module seams to perform it's magic on the Nuxt built-in router only.
I really wish i could find a solution to this myself as these two modules are community reference and it believe they should get along.
Does anyone have any clue on a way to having routes declared from the router.js file and then translated by i18n module ?

Put both modules in modules since then you can control the order in which they process things.
Make sure nuxt-i18n is before router-module in the list. If it doesn't work, try to the way around.

If there are still issues, feel free to create a repro case so that I can test it quickly.
(Having router-module in modules rather than devModules doesn't have any downsides besides requiring to put it in dependencies rather than devDependencies. It won't have any impact at runtime otherwise.)

When we use native nuxt router - routing table builded before calling i18n-module.
In case of router-module usage - routing table created in runtime, rather prebuilded.


Using a custom template with frontmatter does not work in edge cases

There are a few edge cases in which the frontmatter present in a template crashes

Released in v1.3.6


[Keyboard Manager] The key remains pressed even after released.

Microsoft PowerToys version
0.69.1
Installation method
Microsoft Store
Running as admin
Yes
Area(s) with issue?
Keyboard Manager
Steps to reproduce
[Phenomenon]

Assign "Alt" or "Alt (Left)" or "Alt (Right)" to "VK 242" using Keyboard Manager.
Press the "VK 242" key.

Note: "VK 242" is the "Katakana Hiragana Romaji" key on the Japanese keyboard.

Even after the key is actually released, "Alt" remains pressed internally.

To escape this state, the PC have to be rebooted.

[Conditions]
This bug seems to occur only when "Alt" is assigned to "VK 242".
Similar bugs have not been confirmed for other combinations.
Occurs when:

"Alt" is assigned to "VK 242".

Does not occur when:

A key other than "Alt" is assigned to "VK 242".
"Alt" is assigned to a key other than "VK 242".

[Report log]
PowerToysReport_2023-04-18-09-42-59.zip
✔ Expected Behavior
When "VK 242" is actually released, "Alt" is also released internally.
❌ Actual Behavior
When "VK 242" is actually released, "Alt" remains pressed internally.
Other Software
No response

dup
https://github.com/microsoft/PowerToys/issues/18102
https://github.com/microsoft/PowerToys/issues/19787

Sorry, I missed similar bug reports in the past (#3397, #13955, #18102, #19787).
I understood that it was an IME problem, not a PowerToys bug.


Edit button colour after selection

I'm using this code to customize the buttons. However, the command to change the background color is not working.
It seems the command .jspsych-survey-multi-choice-option input[type="radio"]:checked + label
It is not correct.
What I'm missing? Should I use plugins instead?
const radialButtonStyles = `
/* Target the containers for each option to apply a horizontal layout */
.jspsych-survey-multi-choice-option {
  display: inline-block; /* Align options horizontally */
  margin-right: 5px; /* Space between options */
}

/* Hide the actual input (radio button) visually but keep it accessible */
.jspsych-survey-multi-choice-option input[type="radio"] {
  position: relative;
  opacity: 100;
  width: 2;
  height: 2;
}

.jspsych-survey-multi-choice-option label {
  background-color: #f0f0f0; /* Light background for the label */
  padding: 3px 20px; /* Padding around the text */
  border-radius: 20px; /* Rounded corners for the labels */
  font-size: 16px; /* Adjust font size as needed */
  cursor: pointer; /* Pointer cursor on hover */
  display: inline-block; /* Ensure label is inline-block for proper spacing */
  transition: background-color 0.3s, color 0.3s; /* Smooth transition for color changes */
}

/* Change label background and text color when radio button is selected */
.jspsych-survey-multi-choice-option input[type="radio"]:checked + label {
  background-color: #ff9900; /* Orange background for selected option */
  color: white; /* White text for selected option */
  opacity: 100;
  width: 1;
  height: 1;
}

What is this you are doing here? This is apparently not inline CSS and yet you are prefixing it with a const, thereby declaring it as a constant. Are you trying to dynamically insert CSS? If so, why not just import it in a CSS file?
And what do you mean by

Should I use plugins instead?

How do you expect the survey-multi-choice plugin to function without using it?

The point is that I'm using cognition.run, perhaps this is not the place for such a question

Hi @Dazzid
Happy to help figure this out. I think in cognition.run you can insert custom CSS files. Is that correct?

Use this piece of code instead:
.jspsych-survey-multi-choice-option label:has(> input[type="radio"]:checked) {
  background-color: #ff9900; /* Orange background for selected option */
  color: white; /* White text for selected option */
}

See here on stackoverflow.


Can I change the color of the text cursor ?

@JordanMartinez
Hello ! Remenber me ?
I changed the CodeArea's background into black yesterday.
After that, I found it was difficault to find the text cursor because it was black, too.
Can I change the text cursor's color into white or other color ?
Forgive me for my poor CSS plz ...

@YoriChan If you read the CSS Reference Guide, you can style the caret via the style class caret.


[JOSS review] is the list of authors in the JOSS submission complete?

Hi there, another question related to the review (https://github.com/openjournals/joss-reviews/issues/3545).
this code is the basis for your master thesis research conducted at University of Osnabrück, correct? Then I was wondering whether there could be any other researches involved in the work underlying this package, e.g., thesis supervisors?

Hi @janfb. The network was created as part of my master's thesis. The actual thesis can be found at repo. However, the PyPI package is not part of my thesis. Also, I have informed my supervisor about a possible publication in the future, but have not received any feedback in the last years. However, I am happy to credit my supervisor as a co-author of this thesis. But I think it is not appropriate to act without his concern. I will follow your recommendation in this case.

Hi @Saran-nns okay, I think this is fine then. I just wanted to make sure the supervisor(s) were considered in this process. Thanks!


Editor: Make "open 2d/3d/script editor" shortcuts configurable.

Also adds a shortcut for opening the AssetLib (defaulting to F4).

I would love to see is two shortcuts (even if they are set to nothing by default), to switch to the next and previous editor. Similar to how you switch between workspaces in some desktop environments. e.g.:
Ctrl+Alt+ Left / Right: :arrow_right_hook: 2D <-> 3D <-> Script <-> AssetLib :leftwards_arrow_right_hook:

Uhm. Ctrl+Alt+ Left / Right are already used by sane linux distros for "Switch workspace left / right"

Hello,
How about Ctrl+Tab and Ctrl+Shift+Tab, as Firefox does to switch between tabs ?

Ah, ctrl + tab and ctrl+shift+tab are already taken for switching scene tabs.
Any alternatives, or should I add the shortcuts and leave them unassigned?

@Hinsbart I think that's definitely a good idea for now, since we (the people who want this feature pretty bad) can start using it while the collaborators decide on sane defaults for the keybinding.

Ok, will leave them unassigned for now. The tab key currently doesn't work with shortcuts anyway, will open an issue about this..

Alright, updated.


Optimize Network to remove click like sound artifact

After 2000 epochs sound quality reach to usable level but only buggy thing remains is metallic click like noise artifact at the end of each generated sample. Needed to optimize and do some more R&D to remove such kind of Noise artifacts.

https://github.com/seungwonpark/melgan/issues/8 Already covering this issue.


Update microsoft-365-guest-settings.md

Correct typo in AAD table- "gorup" to "group"

@MikePlumleyMSFT - PR is ready for final review. This is just a typo issue.
Thank you.


Godot 4 print output doesn't show up in an exported game on Windows without running through command prompt

Tested versions

Reproducible in 4.2.2.stable.mono.official.15073afe3
Not reproducible in 3.5

System information
Windows 10 and Windows 7, works on Linux (Fedora 39 and Flatpak)
Issue description
None of messages printed by the game show up in the process output at all (problem is both for stdout and stderr). This is problematic as I want to get some status messages by reading the stdout stream of my game process. Even error prints don't show up at all.
When running my exported game through command prompt all the game output shows up then, but not if I use another program to start the process and read stdout and stderr. This used to work in Godot 3 perfectly fine (albeit with quite a bit of buffering) but now as I've upgraded my project to Godot 4 this no longer is the case.
Note that all of this works perfectly fine on Linux without issues, the same as it did in Godot 3. Only on Windows it seems Godot 4 behaves differently in this situation. On another computer with Windows 10, I think I saw a few of the engine error prints show up in the output, but none of my GD.PrintErr lines showed up. It was on a Windows 7 laptop that I observed no output whatsoever coming through.
Steps to reproduce

Have GD.Print("Stuff"); lines in a game
Export for Windows
Run the exported game executable using another program (for example through another c# program using https://learn.microsoft.com/en-us/dotnet/api/system.diagnostics.processstartinfo?view=net-8.0) and not command prompt
Observe how the other process gets no output at all from the game even after the process quits (though my game always crashes on quit due to: https://github.com/godotengine/godot/issues/89941 which may be a factor)

Minimal reproduction project (MRP)
Unless there's some output disabling settings for Windows, making a MRP should be as easy as taking any blank project and adding  GD.Print("Stuff"); into it or maybe even print() in GDScript (I would assume that the logging function works the same so that the result would be the same).

CC @bruvzg
AFAIK this is working as intended by Microsoft, and it's not something we can easily support to behave like on Unix.
That's why Windows exports include a _console.exe wrapper, which you can use to actually get proper stdout/stderr output.

I think it should be possible for Godot to check if the stdout and stderr streams are valid even without a command prompt attached. And then just writing to them should get output. Of course that doesn't show up anywhere if there is nothing listening for the input, but in my case I attach streams to the started process from another C# program, so the output should arrive. Even not doing anything and just trying to write to std::cout should work just fine on Windows, as long as there isn't any calls done to detach from the current "console" or tweaking the standard stream properties. Godot 3 works just fine, so I suspect that code has been added to Godot 4 to either detach the output streams if a command prompt is not detected (/ allocating a dummy output) or an explicit check to suppress "useless" output.
Way back when I primarily developed on Windows I looked into this, and I remember there being APIs to check if there is an attached command prompt and even an API to open one for even a GUI application after it has been started. So it should be possible to make everything work fine again in Godot 4.

It should already redirect IO to the parent console, but the issue is Windows console IO is a mess, so it depends on how messages are printed, what terminal is used etc. If you need a fully functional console, you can build the engine with windows_subsystem=console flag to make it a proper console app.

That's what windows_subsystem=console is for, by default export template is a GUI app without its own console, but it's trying to redirect to the parent process console.

I am making a game (https://github.com/Revolutionary-Games/Thrive) but I used the command line tool as an example of how I want to process game output text. So it is not a solution for me to use the console subsystem. It would confuse the average player if a console window appeared when they started the game (based on the fact that I've seen even prospective game devs on the Godot subreddit be confused by it).


Update the Metrics link in README.md

Old Metrics links leads to a 404. This commit updates the URL accordingly

Can you add your name to AUTHORS, squash and force push ?

Can you add your name to AUTHORS, squash and force push ?

Done

Merged.
Thanks for your contribution !


Many pending tasks, but no worker will process them.

hii, guys.
I am sure that all the pending tasks are not "disabled" (retry times=100, and they are processed 0 or 1 times only).
what is the reason?

Many pending tasks, but no worker will process them.

Have you seen http://tarrasch.github.io/luigid-basics-jun-2015/#/? It might give more insights of how luigi works. Also check out the assistant feature.

@Tarrasch, I see that doc, is it means that I have to start task it again (by crontab?) to finish the pending tasks?
Why does not the schduler automatically start a worker to check and do the pending task?

worker-keep-alive: True
I have config it in the client.cfg

In my design, each time I start a task A, a shell script will first check "ps -ef |grep A", if found a procedure, it will quit. otherwise start task A(N+1).
But I found that even task A is not finished, the procedure is gone.

I see in the log:
luigi.rpc.RPCError: Errors (1 attempts) when connecting to remote scheduler 'localhost'

Closing this since not enough background


BoundingBoxOutlineReloaded Don't work in 1.16.1Server

The mod also install in a 1.16.1Fabric Server
but the Boxes doesn't correct display

Please join the discord server for support


daemon: Add md to daemon call

Include a md_dev argument for the daemon executable.
Signed-off-by: Joel Granados<EMAIL_ADDRESS>

Addressed all @jwdevantier comments.


The HTMLImageElement provided is in the 'broken' state

VM2992:16252 Uncaught DOMException: Failed to execute 'drawImage' on 'CanvasRenderingContext2D': The HTMLImageElement provided is in the 'broken' state.
    at NewClass._drawImage (<anonymous>:16252:23)
    at NewClass._drawMarker (<anonymous>:16239:22)
    at <anonymous>:16200:22
    at Array.forEach (<anonymous>)
    at NewClass._draw (<anonymous>:16179:16)
    at NewClass._redraw (<anonymous>:16077:18)
    at NewClass.fire (<anonymous>:1867:11)
    at NewClass._move (<anonymous>:5506:9)
    at NewClass._onZoomTransitionEnd (<anonymous>:5963:8)

Caused by broken icons (img load error), like here: https://github.com/IITC-CE/ingress-intel-total-conversion/issues/187

Fixed in https://github.com/IITC-CE/Leaflet.Canvas-Markers/commits/fixes


Working Directory

To use bmofetch with my nix config it would be nice to allow for a working directory outside of the git repo in terms of bmofetch.conf and bmo.txt. (the nix-store is read-only, so writes are not allowed).
Maybe a new parameter for bmo.txt such that I can change the working directory.
Love,
Anton :heart:

When using bmosay you can now use -p to specify the path to your working directory. Note that the path must be given directly after the flag and must contain bmofetch.conf and bmofetch.txt.
Examples:
./bmosay.sh -p ~/.config/neofetch/bmofetch "Hello World!"
./bmosay.sh -qr -p ~/.config/neofetch/bmofetch ~/quotes.txt
./bmosay.sh  --path ~/.config/neofetch/bmofetch -r ~/quotes.txt


Transaction Visualization

Problem
• Community members regularly go to https://dragonglass.me/ in order to screenshot / visualize transactions on the network and posting them across social media — but in a number of cases, those visualizations and metrics aren't accurate.
• HashScan does a good job of expressing tokenomics related information on the homepage, but is limited to that data.
• Etherscan + network explorers across other EVM chains always have some basic data visualization (Charts & Stats) wrt transaction throughput across various types and it's expected by users to see this information.

Solution

Create a dedicated page on HashScan that offers visualizations of transactions
Add an option in the top navigation for "Charts & Stats"
Include everything that is listed on Etherscan for visualizing (https://etherscan.io/charts)

This should include, since genesis:

Market Data: HBAR Daily Price (USD) Chart, HBAR Market Capitalization Chart, Total Supply & Market Cap Pie (should include the information broken out from (https://hedera.com/treasury-management-report)
Blockchain Data: Daily transactions chart (broken out by service — HCS, HTS, SC, Crypto, File), Unique Addresses Count Chart, Average Gas Price / Fees Chart, Daily Gas Used Chart, Daily Active Hedera Addresses Chart, Daily Active Fungible Token Collections Chart, Daily Active Non-Fungible Tokens Chart, Daily Active non-fungible token collections chart.
Contracts Data: Hedera Daily Verified Contracts Chart (can only be included after we finish work on the daily verified contracts chart)

Alternatives
Alternatively, we can do this via Metrika and link out from HashScan to Metrica.

Hey @SimiHunjan got hit up by Elaine from THF citing this as a common occurrence for them and folks in the ecosystem; was curious if/how we could push to prioritize this feature?
We're contracted with Patches/Splash of Hgraph.io (paying quarterly) under the umbrella of "data science" for some on-chain ecosystem marketing dashboards, if resources are thin and it's possible, we could have them support development of this.
Let me know your thoughts!

hey @bmgentile !
We can review this issue as part of our backlog grooming session for q4 items we can tackle. We have the following items as priority to deliver in q3 https://github.com/hashgraph/hedera-mirror-node-explorer/issues?q=is%3Aopen+is%3Aissue+label%3AP1.
This is an open source project so external contributions are always welcome.
Some features may depend on mirror node providing the data that is needed to create the visualizations.
The next steps would be to:
(1) Design the feature. This would include UI/UX, what mirror node rest APIs would be used to create the visualization, etc
(2) Review the proposed design
(3) Address feedback from the review
(4) Implement the feature
(5) Release to staging
(6) Review the feature in staging
(7) Release the feature to production

To reiterate, in addition to the dashboarding and visualization outlined above, a pretty common use case is the ability to search HashScan via ticker. So, if I search USDC, I should be presented with https://hashscan.io/mainnet/token/0.0.456858

Thanks so much, Simi; the current timeline sounds great for Q3 to
understand prioritization and appreciate the details on how that process
works!
Fwiw, I have communicated this to Patches/Splash of HGraph.io and we have
an ongoing engagement with them — they have agreed they could support us on
the development side when the time comes if we are resource constrained.
My gut says they need to utilize the raw mirror node data (vs hgraph.io)
for this — mostly because we want an open source functionality as part of
the explorer to create visualizations of this data and not be reliant on a
paid external tool.
BRADY GENTILE
DIRECTOR, PRODUCT & ECOSYSTEM MARKETING
SWIRLDS LABS
Cell:<PHONE_NUMBER> <4807351133>
Twitter: @bmgentile https://www.twitter.com/bmgentile
From: SimiHunjan @.> @.>
Reply: hashgraph/hedera-mirror-node-explorer
@.>
@.>
Date: August 16, 2023 at 5:04:49 PM
To: hashgraph/hedera-mirror-node-explorer
@.>
@.>
Cc: brady @.> @.>, Mention
@.> @.>
Subject:  Re: [hashgraph/hedera-mirror-node-explorer] Transaction
Visualization (Issue #615)
hey @bmgentile https://github.com/bmgentile !
We can review this issue as part of our backlog grooming session for q4
items we can tackle. We have the following items as priority to deliver in
q3
https://github.com/hashgraph/hedera-mirror-node-explorer/issues?q=is%3Aopen+is%3Aissue+label%3AP1
.
This is an open source project so external contributions are always welcome.
Some features may depend on mirror node providing the data that is needed
to create the visualizations.
The next steps would be to:
(1) Design the feature. This would include UI/UX, what mirror node rest
APIs would be used to create the visualization, etc
(2) Review the proposed design
(3) Address feedback from the review
(4) Implement the feature
(5) Release to staging
(6) Review the feature in staging
(7) Release the feature to production
—
Reply to this email directly, view it on GitHub
https://github.com/hashgraph/hedera-mirror-node-explorer/issues/615#issuecomment-1681269830,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXFFURR7NZUJJEM7Z6UIO3XVUYXDANCNFSM6AAAAAAYS57MOU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey @SimiHunjan was curious if this is still prioritized for Q4?

Hi @bmgentile ! No we have not scoped this issue and are still working on features needed for smart contracts.

We need mirror node to provide the ability to get time series data to create these kinds of visualizations on the explorer. This would likely result in a specific API to provide.
Things like evolution of hbar price we can use other available services to get that data.

PRD required

Check for charting component that can be used in open source projects

AM Chart
Highchart

Spoke to the Hedera mirror node team and they can support most if not all of the queries we are looking for.
Follow-up conversations with other mirror node providers to see if we can get APIs sooner and later when the Hedera mirror node has the support we can update to use those APIs.

Need to submit HIP for the APIs we need from the mirror node team.


Update Private Link documentation

Update the Private Link page
Add instruction for the what the teams should do

https://www.radix.equinor.com/docs/topic-private-link/


Fix/extend re replacement seq

This PR is an extension of https://github.com/outlines-dev/outlines/pull/763, related to extending the re_replacement_seq regex.
The new NorwAI models use a tokenizer that has the token �., which leads to the same error as was described in the previous issue https://github.com/outlines-dev/outlines/issues/762.
This PR extends the fix from https://github.com/outlines-dev/outlines/pull/763 to deal with this case, as well as adding a unit test to test various tokenizers, and a comment describing why we need the prefix and suffix in the regex.

HUGGINGFACE_API_TOKEN should now be available in actions

HUGGINGFACE_API_TOKEN should now be available in actions

That's great, thanks. Are you able to get access to the following models with that token?

meta-llama/Meta-Llama-3-8B
mistralai/Mistral-7B-v0.3
google/gemma-2b
NorwAI/NorwAI-Mistral-7B-instruct

Those are the culprits of the failing tests.
Also, just to be sure, the HUGGINGFACE_API_TOKEN is an access token and not a token for the Hugging Face inference API, right?

It's probably better to do this "manually" indeed.

@rlouf Changed the test to a "manual" one now, and all tests pass 🙂

Awesome, thank you!


Error in Raspberry Pi

$ python3 eval.py --video="aab.mp4"
** (Detector:2966): WARNING **: Error retrieving accessibility bus address: org.freedesktop.DBus.Error.ServiceUnknown: The name org.a11y.Bus was not provided by any .service files
terminate called after throwing an instance of 'std::bad_alloc'
what():  std::bad_alloc
Aborted
$python3 daemon.py
[INFO] 2018-05-13 17:50:24,420: Start camera loop
terminate called after throwing an instance of 'std::bad_alloc'
what():  std::bad_alloc
Aborted
$python3
Python 3.5.3
[GCC 6.3.0 20170124] on linux

import tensorflow
import cv2
tensorflow.version
'1.4.0'
cv2.version
'3.4.0'

Hi,
How do you install openCV on RPI?
AFAIK opencv-python that is used in this demo do not officially support ARM builds.
Best solution for now is to build openCV from scratch. Here you can found good tutorial.

'std::bad_alloc'
what(): std::bad_alloc
This allocation error is due to Heap Memory so just as mentioned above try to reinstall using some other way..
If you haven't installed please do follow this the best one and you can build from scratch.
https://www.pyimagesearch.com/2017/09/04/raspbian-stretch-install-opencv-3-python-on-your-raspberry-pi/
Goodluck


oc-bifrost-pedant not merged into the repository

Bifrost (the authorization service) has its own set of pedant tests.  They can be found here:
https://github.com/chef/oc-bifrost-pedant (still private for some reason).
These tests also don't run in CI as far as I can see.  Are these tests still important?  If so, we should merge them into this repository and make sure we run them in the CI pipeline.

Fixed by https://github.com/chef/chef-server/pull/761


cant not run nginx with lua

when I follow the step , I can't restart nginx :
# nginx -t
nginx: [emerg] unknown directive "lua_package_path" in /etc/nginx/conf.d/crowdsec_nginx.conf:1

There is some messages:
apt-get install lua5.3 libnginx-mod-http-lua lua-sec
Reading package lists... Done
Building dependency tree
Reading state information... Done
lua-sec is already the newest version (0.9-3).
lua5.3 is already the newest version (5.3.3-1.1ubuntu2).
libnginx-mod-http-lua is already the newest version (1.18.0-0ubuntu1).
0 upgraded, 0 newly installed, 0 to remove and 1 not upgraded.
---
# cat /etc/issue
Ubuntu 20.04.1 LTS \n \l
----
# nginx -V
nginx version: nginx/1.18.0 (Ubuntu)
built with OpenSSL 1.1.1f  31 Mar 2020
TLS SNI support enabled
configure arguments: --with-cc-opt='-g -O2 -fdebug-prefix-map=/build/nginx-5J5hor/nginx-1.18.0=. -fstack-protector-strong -Wformat -Werror=format-security -fPIC -Wdate-time -D_FORTIFY_SOURCE=2' --with-ld-opt='-Wl,-Bsymbolic-functions -Wl,-z,relro -Wl,-z,now -fPIC' --prefix=/usr/share/nginx --conf-path=/etc/nginx/nginx.conf --http-log-path=/var/log/nginx/access.log --error-log-path=/var/log/nginx/error.log --lock-path=/var/lock/nginx.lock --pid-path=/run/nginx.pid --modules-path=/usr/lib/nginx/modules --http-client-body-temp-path=/var/lib/nginx/body --http-fastcgi-temp-path=/var/lib/nginx/fastcgi --http-proxy-temp-path=/var/lib/nginx/proxy --http-scgi-temp-path=/var/lib/nginx/scgi --http-uwsgi-temp-path=/var/lib/nginx/uwsgi --with-debug --with-compat --with-pcre-jit --with-http_ssl_module --with-http_stub_status_module --with-http_realip_module --with-http_auth_request_module --with-http_v2_module --with-http_dav_module --with-http_slice_module --with-threads --with-http_addition_module --with-http_geoip_module=dynamic --with-http_gunzip_module --with-http_gzip_static_module --with-http_image_filter_module=dynamic --with-http_sub_module --with-http_xslt_module=dynamic --with-stream=dynamic --with-stream_ssl_module --with-stream_ssl_preread_module --with-mail=dynamic --with-mail_ssl_module--add-dynamic-module=/build/nginx-5J5hor/nginx-1.18.0/debian/modules/http-auth-pam --add-dynamic-module=/build/nginx-5J5hor/nginx-1.18.0/debian/modules/http-dav-ext --add-dynamic-module=/build/nginx-5J5hor/nginx-1.18.0/debian/modules/http-echo --add-dynamic-module=/build/nginx-5J5hor/nginx-1.18.0/debian/modules/http-upstream-fair --add-dynamic-module=/build/nginx-5J5hor/nginx-1.18.0/debian/modules/http-subs-filter --add-dynamic-module=/build/nginx-5J5hor/nginx-1.18.0/debian/modules/http-geoip2

Should I need install nginx-lua-module by mannul or install openresty?
Thanks !

Final I  try to install openresty ,but still get errors:
Dec 11 11:21:49 nginx-proxy03 systemd[1]: Starting The OpenResty Application Platform...
Dec 11 11:21:49 nginx-proxy03 nginx[4326]:  nginx: [error] init_by_lua error: /usr/local/lua/crowdsec/lrucache.lua:49: attempt to redefine 'lrucache_queue_s' at line 2
Dec 11 11:21:49 nginx-proxy03 nginx[4326]: stack traceback:
Dec 11 11:21:49 nginx-proxy03 nginx[4326]:         [C]: in function 'cdef'
Dec 11 11:21:49 nginx-proxy03 nginx[4326]:         /usr/local/lua/crowdsec/lrucache.lua:49: in main chunk
Dec 11 11:21:49 nginx-proxy03 nginx[4326]:         [C]: in function 'require'
Dec 11 11:21:49 nginx-proxy03 nginx[4326]:         /usr/local/lua/crowdsec/CrowdSec.lua:4: in main chunk
Dec 11 11:21:49 nginx-proxy03 nginx[4326]:         [C]: in function 'require'
Dec 11 11:21:49 nginx-proxy03 nginx[4326]:         init_by_lua:2: in main chunk
Dec 11 11:21:49 nginx-proxy03 systemd[1]: openresty.service: Control process exited, code=exited, status=1/FAILURE
Dec 11 11:21:49 nginx-proxy03 systemd[1]: openresty.service: Failed with result 'exit-code'.
Dec 11 11:21:49 nginx-proxy03 systemd[1]: Failed to start The OpenResty Application Platform.

Hello,
Thanks for the report !
You shouldn't have to use openresty, nginx mod lua should do the trick.
The error nginx: [emerg] unknown directive "lua_package_path" in /etc/nginx/conf.d/crowdsec_nginx.conf:1 looks like lua was actually not installed as a nginx module (this directive is one of nginx).
Can you check if lua is correctly enabled at nginx level ?
Thanks,

@zhixiangjoy any update on this ?

@zhixiangjoy any update on this ?

Thanks for your reply, It's my fault。I had modify the nginx.conf which had delete the line :
include /etc/nginx/modules-enabled/*.conf;
In that situation, nginx can't loading the lua lib.

Thanks for your reply, It's my fault。I had modify the nginx.conf which had delete the line :
include /etc/nginx/modules-enabled/*.conf;
In that situation, nginx can't loading the lua lib.


Use clippy-preview from 2018-07-16

I'm not sure why, but the old way of getting clippy (cargo install) now seems to be broken on travis.

This seems to be uncontroversial, so I'll just merge once green.


Support Torch.Tensor in the variable explorer

Problem Description
I'm a pytorch user, and I'm always thinking that type torch.Tensor might be represented in a proper way. What torch.Tensor type data need is just '.numpy()' to be converted to numpy array objects.
So, is it possible to apply simple scripts for the unsupported data types inside of Variable explorer?(i.e. If Variable explorer saw Tensor type objects, then a corresponding function/method for the type is applied and show the result of it on Variable explorer with distinguishable color.)
def type_conversion(data: torch.Tensor):
  if isinstance(data, torch.Tensor):
    return data.numpy()
  return None

Package Versions

Spyder: 3.2.8
Python: 3.6.4
Qt:
PyQt:
Operating System: macOS 10.13.4

Dependencies

IPython >=4.0    :  6.3.1 (OK)
cython >=0.21    :  0.28.2 (OK)
jedi >=0.9.0     :  0.12.0 (OK)
nbconvert >=4.0  :  5.3.1 (OK)
numpy >=1.7      :  1.14.1 (OK)
pandas >=0.13.1  :  0.22.0 (OK)
psutil >=0.3     :  5.4.5 (OK)
pycodestyle >=2.3:  2.4.0 (OK)
pyflakes >=0.6.0 :  1.6.0 (OK)
pygments >=2.0   :  2.2.0 (OK)
pylint >=0.25    :  1.8.4 (OK)
qtconsole >=4.2.0:  4.3.1 (OK)
rope >=0.9.4     :  0.10.7 (OK)
sphinx >=0.6.6   :  1.7.4 (OK)
sympy >=0.7.3    :  1.1.1 (OK)

Thanks for reporting. We'll take a look at this for Spyder 4.

To unpack this a tad, it seems the proximate issue is support for PyTorch tensors in the Variable Explorer, which could be solved with a few straightforward changes to CollectionsEditor editor selection or within ArrayEditor. However, it seems the broader feature request the user was making, as was reflected in the original title and not in the modified one, was the ability for users to add support for custom types to the Variable Explorer via simple conversion scripts to known times for which we already offer Editors for. Not sure the latter is realistic or feasible, but wanted to make sure we were all clear on what specifically the user was asking for.

What I need is the support for torch.Tensor type.
However, I wrote an idea about the general solution because I was not sure whether it's okay to put just one exceptional case for the specific torch.Tensor type. Hmm.... Could someone tell me which one is more 'Spyder' way?

@ccordoba12 would be the one to ultimately decide that; based on his re-titling of your issue, it seems he's focused more on the former. However, the latter could always be potentially added at some point in the (likely further) future, in the form of some type of modular "plugin" support for the variable explorer's Editor classes. It would be really need to have, but would take some work, requiring refactoring the existing Editors into a modular system and creating an API to interact with third-party editors, etc.
As somewhat of a stopgap between the quick but specific and general but more involved approaches could be Pythonic duck typing, converting unknown types to known ones (principally array, and maybe list) and regressing to the current behavior if an exception, but not sure if it would be worth the tradeoffs vs. checking against a list of types that can be converted to array, list etc. by various means.
The more custom types we supported though, the more we'd want to have the option to view any object as a generic low level __dict__ to see and interact with all its properties, along with the higher-level view we currently use when available—I've been thinking about something like this for a while, since somethings while working the former can be convenient, especially when working with derived types and convertible ones like this. @ccordoba12 , thoughts?

The narrow issue is similar to, perhaps the same as, #5375.
I think there should be an issue about making it easier to support new data types, with how to achieve this to be discussed.

The narrow issue is similar to, perhaps the same as, #5375.

They're related, and both fit under the broader issue of PyTorch tensor support. but not the same (narrowly speaking, at least). Fixing this one (displaying tensors as arrays with ArrayEditor rather than as generic objects) would work around the other one, but not the converse; the latter would be a very simple fix along the lines of #6284 and a few similar issues I've fixed, while the former would be a tad more involved (though not that much more, at least in theory).

I think there should be an issue about making it easier to support new data types, with how to achieve this to be discussed.

Indeed, agreed on my part. @ccordoba12 ?

I don't think it's necessary to wait for ccordoba12 to give permission to open a new issue.

Now i am using spyder 4.0.0b1, but the problem is still existing :(

@donglee-afar We've been busy implementing the overhauled completion/introspection/analysis architecture and the new debugger for the next betas, but more modular support for new Variable Explorer datatypes is something we'd like to add in the future.
In the meantime, though, it should actually not be too difficult to add support for viewing and even edit Tensorflow tensors as Numpy arrays, so long as the output arrays are <4D (same as with Numpy arrays). We already do something similar for Image here; you would just need to add a block below that one following the same outline, except replacing arr = array(value) with arr = value.numpy() and you wouldn't even need a conv_func as a Pytorch tensor and its Numpy array counterpart share the same data locations in memory so they update automatically without having to convert them back.
Other than that, there's only one complexity—you'll need to add a short try-except block here in Spyder-Kernels trying to import tensor from torch, and making it a FakeObject if not. Finally, in CollectionEditor, you'll need to add tensor to the imports from spyder_kernels.utils.nsview.
After you test this and ensure its working with dev versions of Spyder-Kernels and Spyder, there's only one other problem—it will make Spyder crash if it doesn't have the latest dev version of Spyder-Kernels (once your changes are merged). To fix that, at the end when you want to add some backwards compatibility, you can add a try-except around the torch import from spyder-kernels and set torch = FakeObject there (or, you could do the same import check there, and actually enable it if so, but that gets a little messy). Do that change in a separate commit so it can easily be reverted once Spyder-Kernels is out and well-available. Thanks!

Hello, I am a newcomer to PyTorch, I tried to see the tensor variable on the spyder ide, I tried to do what @CAM-Gerlach  directed but still don't know how to solve it. Can I get more detailed steps or teaching?
Thanks!

Hi Spyder team, any update on this matter? Like current developments or even if supporting Pytorch tensors in the debugger and in the variable explorer is a priority? It would be really helpful. Thanks.

Sorry, there are not updates about this. We'll try to do that in the coming months.

I am sure the list of todos is long for spyder but I just wanted to bump this again since it seems like an important feature

Note: This StackOverflow answer seems relevant to implement this feature. However, it shows that it won't be easy to do given the many ways in which you can declare a Torch tensor.

It actually doesn't seem that difficult after having that detailed answer to reference, since what matters is how to convert the tensor to Numpy, not how it is declared. It boils down to a few sequential steps:

If requires_grad = True (via either EAFP try-except RuntimeError or LBYL check of relevant object attribute), call .detach()
If non-CPU tensor (via either EAFP try-except TypeError or LBYL check object attribute), call .to('cpu')
Call .numpy() on the tensor to get the Numpy ndarray form.

In fact, it appears that if neither of the first two are true, tensors should theoretically be editable in-place as Numpy ndarrays (though in practice, that presumably won't work without non-trivial changes on the kernel side, since right now AFAIK objects are pickled back and forth and replaced rather than actually editing them in-place).


Added hand-rolled DiverseKerbalHeads

Works great on my system with ckan install -c.
Alas, had to do the hand-rolled file as it looks like it's only on Curse.

im in my PR, merging my chngz.


Check index is not negative.

Hi, do you have a use case for this fix ?

I was getting an error thrown as i was -1. To be honest I didn't dig much deeper as this got my lint system working on .jsx files with an npm command:
"lint": "eslint . --ext .jsx,.js"

Ok, I found that the following pattern trigger the bug:
class Hello extends React.Component {
  static get propTypes() {} // <- empty body for the propTypes declaration
  render() {
    return
Hello World
;
  }
}

I'll add a this pattern to the test suite.
Thanks for the bugfix!

Thanks for the quick resolution.


Fix SyntaxError: duplicate argument 'ncbarlabel' in function definiti…

…on in exa/mpl.py

Nice catch. There is no sense in working on getting the travis build passing as we will be moving to exa0.4.x soon enough.


Unexpected token: punc (() from UglifyJs

Hello,
When I try to build my app with NPM RUN BUILD command, the response from terminal is:
ERROR in static/js/vendor.fd125947db1218b759b7.js from UglifyJs Unexpected token: punc (() [./node_modules/vue-mce/dist/vue-mce.common.js:1,823][static/js/vendor.fd125947db1218b759b7.js:62883,800]
If I run it with NPM RUN DEV, it works perfectly. I have try to create a new project with VUE CLI, so I use the last version of webpack, vue, etc. , but I get this error when I try to build my project.
Can u help me ?
Giovanni

Hello!
I made some changes and released package under 1.3.6 version
Tested it with vue-cli webpack template

Good Job!
Now it works great!
Thanks about this awesome plugin ;)


Add WebPageTest link when too big

At the moment when the test results are too big for the comment we add this:

Webpage test results that are too big for a comment are available as the action's artifact.

Example:
https://github.com/HTTPArchive/custom-metrics/pull/145#issuecomment-2365230655
It would be nice if we also got a WebPageTest link so we can see them there too. Is this possible @max-ostapenko ?

When I download and open the artifact the URL is there:

Or did you mean something else?

I mean putting the test URL link in the GitHub comment as well.
That would allow me to click on it to copy and waste the custom metric rather than having to download and unzip a file.


Feature request: ignore part of a resource in upgrade

In Helm2, it was possible to use it together with a separate CD system because it would only apply the diff since the last Helm release. But with Helm3, due to the 3-way merge process, this is not possible. For example, if you use a tool like Spinnaker or any other CD tool that is effectively running kubectl set image xxxxx container=nginx:latest, now Helm3 will undo this change on the next chart upgrade.
Is there any way around this?
Would a reasonable feature request be to allow the user to ignore a specific attribute from a resource during an upgrade? For example:
helm diff upgrade ingress01 stable/nginx-ingress --reuse-values --ignore deployment.ingress01.spec.template.spec.containers[0].image

The only other option is to include Helm in the CD process. While this sounds trivial and a lot of people do it, in many cases this is too cumbersome. For example we have many values files, complicated charts, helm-secrets encrypted with AWS KMS etc. To include these toolsets and dependencies in our CD would be a huge pain where right now we essentially just call the k8s API to change the image.
Or any other ideas would be appreciated 🙂

Have you tried setting the value to null, or by matching the current value present in your live state?
https://helm.sh/docs/faq/#improved-upgrade-strategy-3-way-strategic-merge-patches

Have you tried setting the value to null,

I have not. So I would install the chart with a dummy value, set it to spec.template.spec.containers[0].image=null, upgrade the chart, then this key can be managed outside of Helm? I'll try it 🙂

or by matching the current value present in your live state?

So we would get the current image, and then add this as a value like: helm upgrade ingress01 stable/nginx-ingress --values path/to/my/values.yaml --set controller.image.tag=0.29.0?
Could do this but would be prone to error if it's left to a user. We would have to automate something here I think.

@bacongobbler thanks for the reply. Regarding that link "Improved Upgrade Strategy: 3-way Strategic Merge Patches", it says there in the example about sidecar containers:

but live state contains a sidecar container

How does Helm decide to let that sidecar remain after the 3 way merge? Is there specific logic for sidecar containers? Or it just allows extra items to remain when they are in an array, like containers?

the latter. As the documentation suggested, live state was not considered in Helm 2, so helm had no idea there was a sidecar container. When the PATCH request comes in, it effectively removed the sidecar. In Helm 3, we have knowledge of that sidecar and can allow that extra item to remain in the array, as the update (the chart from helm upgrade does not care about modifying the state of the sidecar.

Using null doesn't work as it's rejected by the k8s API (or a local check), for example:
Error: UPGRADE FAILED: cannot patch "el-k8s-event-logger" with kind Deployment: Deployment.apps "el-k8s-event-logger" is invalid: spec.template.spec.containers[0].image: Required value

I thought I could pre-create the deployment and then install the chart with no image set but then I get existing resource conflict
@bacongobbler is there another way to manage a deployment with Helm but not the image? Could I manually remove spec.template.spec.containers[0].image from the release by editing the release secret data?

@bacongobbler do you think this is a valid feature request then?

I have the same issue but with HorizontalPodAutoscaler minReplicas. I have automation which tweaks this value but every Helm deploy it is reset back to the number in Git. I presume this also happens with the replicas count for a deployment, which might have been modified by a HPA but I haven't confirmed this second case.

It's not stale. I would still like to see some idea of a solution here as this is still a major problem for us.

I have automation which tweaks this value

This is exactly what I'm talking about. We have automation for deployments.

:smiley:

We are also facing similar problem with HPA and we want HPA to be ignored by Helm.

We are also facing similar problem with HPA and we want HPA to be ignored by Helm.

Not stale 😢

We are facing the same problem and have not found a solution yet.

It'd be really nice to be able to do special customizations alongside helm and not have helm override it with a specific resource.
Personally, I'd be fine with something like --ignore-resource=<resource name>.yaml, and I could just apply my own version of that resource next to it, or in a different helm chart.

this is not stale and we are facing similar issue. Please let us know how to ignore a specific yaml

@bacongobbler can you change this issue tag to feature?

Can confirm not stale as well, and we are also interested in a possible solution to this.

not stale, experiencing same issues as mentioned previously.

You could almost solve this problem using Helm's lookup function, except, the lookup function breaks a bunch of other automation tools because it's unusable in a dry-run, which means things like helm diff and helmfile apply don't function correctly if you've got a call to lookup anywhere in your code.
For people who don't make any use of helm template, helm upgrade --dry-run, helm diff, or any similar tool, here's a snippet that should solve this for you:
{{- define "example.image" }}
{{- $existing := lookup "apps/v1" "Deployment" .Release.Namespace (include "example.fullname" .) }}
{{- $containers :=  dig "spec" "template" "spec" "containers" (list dict) $existing }}
{{- $container := index $containers 0 }}
{{- dig "image" (include "example.default-image" .) $container }}
{{- end }}

This isn't perfect: it assumes your container of interest is spec.template.spec.containers[0], since Helm doesn't have easy support for anything like jsonpath's<EMAIL_ADDRESS>== 'app')]. But if that's a problem for you, you can find a given container by name using some complicated range-based workarounds.
The bigger problem, as noted above, is that there is no method in Helm to preview what this function will do prior to actually running the install/upgrade, because all preview functions like template and upgrade --dry-run refuse to contact the remote cluster and will always fall back to the designated default image.

/reopen

I don't think it's possible to reopen an issue closed by the stale bot unless you're a maintainer. The issue was open for almost 2 years with no resolution anyway.

This is also nothing helm is made for. Helm in my eyes is a tool to template resource definitions (and apply them).
For complex use cases like that, I deem tools like ArgoCD the much better option.
On November 28, 2022 3:39:09 PM UTC, Max Williams @.***> wrote:

I don't think it's possible to reopen an issue closed by the stale bot unless you're a maintainer. The issue was open for almost 2 years with no resolution anyway.
--
Reply to this email directly or view it on GitHub:
https://github.com/helm/helm/issues/7971#issuecomment-1329314037
You are receiving this because you are subscribed to this thread.
Message ID: @.***>

This is tricky because most of people will use Helm with argocd:

argocd get raw manifests via helm template
argocd apply manifests

So even if the solution is done on helm side, this will not work with argocd (or git'ops tools).
As a (dirty) workaround, but used on production, we generate "mutable ressources" with a kube Job and right RBAC:
# Create the configMap
apiVersion: batch/v1
kind: Job
spec:
  template:
    spec:
      serviceAccountName: patcho
      containers:
      - name: work
        image: bitnami/kubectl:1.24
        command: ["sh", "-c"]
        args:
        - |
          echo "Create status configMap"

          kubectl create configmap \
            my-status \
            --from-literal=status=todo

# To modify the configMap
kubectl patch cm \
            my-status \
            --patch \
            '{"data" : {"status" : "done", "result" : "232 items processed"}}'

We faced a similar issue.  IIUC, when helm supports server-side apply(https://github.com/helm/community/pull/312). we can set different ownership for different fields.

Same issue here, see https://github.com/helm/helm/issues/13506


report_checks -from [all_registers] -to [all_outputs] and report_checks -from [all_clocks] -to [all_outputs] should be equivalent

Describe the bug
The first of these statements is much slower than the second for large designs, but @tspyrou says they are equivalent.
However, for make DESIGN_CONFIG=designs/asap7/mock-array/config.mk nuke final gui_final, the second returns no paths.
report_checks -from [all_registers] -to [all_outputs]

report_checks -from [all_clocks] -to [all_outputs]

Build https://github.com/The-OpenROAD-Project/OpenROAD/pull/5431
untar
reg2out.tar.gz
$ ./run-me-mock-array-asap7-base.sh
OpenROAD v2.0-14707-g194e18f06
Features included (+) or not (-): +Charts +GPU +GUI +Python
This program is licensed under the BSD-3 license. See the LICENSE file for details.
Components of this program may be licensed under more restrictive licenses which must be honored.
read_spef ./results/asap7/mock-array/base/6_final.spef
Startpoint: REG_58$_DFF_P_/QN (internal pin)
Endpoint: io_lsbs_58 (output port)
Path Group: path delay
Path Type: max

  Delay    Time   Description
---------------------------------------------------------
   0.00    0.00 v REG_58$_DFF_P_/QN (DFFHQNx2_ASAP7_75t_R)
  47.72   47.72 ^ _120_/Y (INVx4_ASAP7_75t_R)
  34.06   81.78 ^ output2103/Y (BUFx2_ASAP7_75t_R)
   0.03   81.81 ^ io_lsbs_58 (out)
          81.81   data arrival time

  80.00   80.00   max_delay
   0.00   80.00   output external delay
          80.00   data required time
---------------------------------------------------------
          80.00   data required time
         -81.81   data arrival time
---------------------------------------------------------
          -1.81   slack (VIOLATED)

No paths found.
openroad>

Expected Behavior
Report the same
Environment
https://github.com/The-OpenROAD-Project/OpenROAD/pull/5431

To Reproduce
See above
Relevant log output
No response
Screenshots
No response
Additional Context
No response

@oharboe there are corner cases where the two will not align. Your set_max_delay -from a non startpoint is one of those. The latest opensta now is fast for the original all_registers case. You can switch back to that and not use all_clocks as a workaround now.


Make breakpoints section height adjustable

VSCode Version: 1.2.0-alpha
OS Version: Windows 10

Steps to Reproduce:

Open debug viewlet
Add 7 or more breakpoints

When you have many breakpoints set you have to scroll through them, but all other sections of debugging allow adjustable height sections. The breakpoints section should either be height adjustable or expand to be able to view all set breakpoints at once.

+1.


showzero need to better logic

Show zero is still confusing since it does not apply to all blood component.
We need to think about ways to improve this

currently the grey shader is not working properly

"When you have heatmaps with show zero transfused disabled, the zero cells end up with tooltip values more than 100%. I think we could do something a little different with the label in that case. Also the grey fill might be wrong. All the grey cells have the same shade"

This shouldn't be an issue anymore since now all units can be filtered directly with the new filter system.


can't compile flop

I bought petal pro and am trying to add to an existing phoenix project. I'm getting this error:
Compiling 13 files (.ex)

== Compilation error in file lib/flop/builder.ex ==
** (Ecto.Query.CompileError) `not (is_nil(field(r, ^var!(field))) == ^var!(value))` is not a valid query expression.

* If you intended to call an Elixir function or introduce a value,
  you need to explicitly interpolate it with ^

* If you intended to call a database function, please check the documentation
  for Ecto.Query.API to see the supported database expressions

* If you intended to extend Ecto's query DSL, make sure that you have required
  the module or imported the relevant function. Note that you need macros to
  extend Ecto's querying capabilities

    (ecto 3.6.2) expanding macro: Ecto.Query.dynamic/2
    lib/flop/builder.ex:196: Flop.Builder.build_op/3
    expanding macro: Flop.Operators.build_dynamic/3
    lib/flop/builder.ex:196: Flop.Builder.build_op/3
could not compile dependency :flop, "mix compile" failed. Errors may have been logged above. You can recompile this dependency with "mix deps.compile flop", update it with "mix deps.update flop" or clean it with "mix deps.clean flop"

I'm not sure what deps I should update to make this happy

I've unlocked and updated all my ecto* deps

Hi @atomkirk , hm not sure what is happening here. Seems like Flop is not compiling due to an error. Are you using the latest version of ecto and flop?

@atomkirk closing this for now as it's been a couple of months. Please reopen or open another issue if still a problem.


Make blog admin base path configurable (instead of '/admin/blog')

See #107.

Is this feature still being considered for the next release?

If you change those routes it won't work


6play not working

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2021.12.01. (update instructions)
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
France
Description
I try to download an M6 (french TV channel) replay but it doesn't seem to work. I also tried with --cookie-from-browser.
I let you check the logs.
I think you need an account to view this kind of content. I created this one :
User<EMAIL_ADDRESS>Passwd : xZ(L5SJlV-Et?xY
Thank you in advance
Verbose log
[debug] Command-line config: ['-Uv', '--cookies-from-browser', 'firefox', 'https://www.6play.fr/66-minutes-p_825/cadeaux-louez-vos-jouets-chauffeur-si-tes-champion-noel-coup-de-chaud-chez-le-roi-du-froid-c_12912864']
[Cookies] Extracting cookies from firefox
[debug] Extracting cookies from: "/Users/ackermannd/Library/Application Support/Firefox/Profiles/low04f7u.default-release/cookies.sqlite"
[Cookies] Extracted 1352 cookies from firefox
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, err utf-8, pref UTF-8
[debug] yt-dlp version 2021.12.01 [91f071af6]
[debug] Python version 3.10.1 (CPython 64bit) - macOS-12.0.1-arm64-arm-64bit
[debug] exe versions: ffmpeg 4.4.1 (setts), ffprobe 4.4.1
[debug] Optional libraries: Cryptodome, mutagen, sqlite, websockets
[debug] Proxy map: {}
Latest version: 2021.12.01, Current version: 2021.12.01
yt-dlp is up to date (2021.12.01)
[debug] [6play] Extracting URL: https://www.6play.fr/66-minutes-p_825/cadeaux-louez-vos-jouets-chauffeur-si-tes-champion-noel-coup-de-chaud-chez-le-roi-du-froid-c_12912864
[6play] 12912864: Downloading JSON metadata
ERROR: [6play] 12912864: No video formats found!; please report this issue on  https://github.com/yt-dlp/yt-dlp . Make sure you are using the latest version; see  https://github.com/yt-dlp/yt-dlp  on how to update. Be sure to call yt-dlp with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/opt/homebrew/Cellar/yt-dlp/2021.12.1/libexec/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 1329, in wrapper
    return func(self, *args, **kwargs)
  File "/opt/homebrew/Cellar/yt-dlp/2021.12.1/libexec/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 1412, in __extract_info
    return self.process_ie_result(ie_result, download, extra_info)
  File "/opt/homebrew/Cellar/yt-dlp/2021.12.1/libexec/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 1462, in process_ie_result
    ie_result = self.process_video_result(ie_result, download=download)
  File "/opt/homebrew/Cellar/yt-dlp/2021.12.1/libexec/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 2346, in process_video_result
    self.raise_no_formats(info_dict)
  File "/opt/homebrew/Cellar/yt-dlp/2021.12.1/libexec/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 933, in raise_no_formats
    raise ExtractorError(msg, video_id=info['id'], ie=info['extractor'],
yt_dlp.utils.ExtractorError: [6play] 12912864: No video formats found!; please report this issue on  https://github.com/yt-dlp/yt-dlp . Make sure you are using the latest version; see  https://github.com/yt-dlp/yt-dlp  on how to update. Be sure to call yt-dlp with the --verbose flag and include its complete output.

It seems to be protected by fairplay DRM (hlsfp) or dashcenc DRM
There is a kodi plugin that supports 6play but I haven't looked at the code too much:
https://github.com/Catch-up-TV-and-More/plugin.video.catchuptvandmore/blob/e4a15971152cc23e1902aa535ed36023417aca34/resources/lib/channels/fr/6play.py
It might be possible to port this to yt-dlp

IMO it's more something like Widevine DRM.

Hello.
I have the same issue.
Here is the verbose log.
Best regards
[debug] Command-line config: ['-v', 'https://www.6play.fr/le-1245-p_1056/le-1245-du-20-09-2022-c_12966333'] [debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8 [debug] yt-dlp version 2022.09.01 [5d7c7d6] (pip) [debug] Python 3.10.6 (CPython 64bit) - macOS-10.15.7-x86_64-i386-64bit [debug] Checking exe version: ffmpeg -bsfs [debug] Checking exe version: ffprobe -bsfs [debug] exe versions: ffmpeg 5.1.1 (setts), ffprobe 5.1.1, rtmpdump 2.4 [debug] Optional libraries: Cryptodome-3.15.0, brotli-1.0.9, certifi-2022.06.15, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.3 [debug] Proxy map: {} [debug] Loaded 1670 extractors [debug] [6play] Extracting URL: https://www.6play.fr/le-1245-p_1056/le-1245-du-20-09-2022-c_12966333 [6play] 12966333: Downloading JSON metadata ERROR: [6play] 12966333: No video formats found!; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U Traceback (most recent call last):   File "/usr/local/Cellar/yt-dlp/2022.9.1/libexec/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 1459, in wrapper     return func(self, *args, **kwargs)   File "/usr/local/Cellar/yt-dlp/2022.9.1/libexec/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 1556, in __extract_info     return self.process_ie_result(ie_result, download, extra_info)   File "/usr/local/Cellar/yt-dlp/2022.9.1/libexec/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 1614, in process_ie_result     ie_result = self.process_video_result(ie_result, download=download)   File "/usr/local/Cellar/yt-dlp/2022.9.1/libexec/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 2546, in process_video_result     self.raise_no_formats(info_dict)   File "/usr/local/Cellar/yt-dlp/2022.9.1/libexec/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 1025, in raise_no_formats     raise ExtractorError(msg, video_id=info['id'], ie=info['extractor'], yt_dlp.utils.ExtractorError: [6play] 12966333: No video formats found!; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U

originally posted in #5237 by @dirkf:

Apparently the extractor only gets DRM formats for this show from the API URL that it uses. Also, it only gets one of them because it doesn't look for the DASH/mpd case.

...
                         'type': 'usp_dashcenc_h264',
...
                         'type': 'usp_hlsfp_h264',

The yt-dlp 6Play extractor is essentially identical to the yt-dl version and doesn't understand any special processing for DRM formats.

Related: https://github.com/ytdl-org/youtube-dl/issues/24967


Package metadata is incorrect

Currently MarkupSafe is listed as install_requires, which is incorrect, it should be a pure requires since the package doesn't work without it. install_requires denotes packages which are only necessary to install the package, but if e.g. someone were to build a wheel for WTForms, then these requirements could be dropped from the resulting wheel.
It didn't matter until now since you don't publish wheels for WTForms on PyPI, but it's annoying when it's used with third party tools that would first install the package and then export it into a clean environment, since you will need to explicitly add the MarkupSafe requirement.
This came up when trying to create stubs for WTForms in typeshed, since their metadata parsing ignores install_requires, since those requirements only should come into play when the original package has to be installed into the environment.

Alright the issue was something else I think. I might reopen if it's actually an issue.


SOT-71: Add Sumcheck and GKR protocol slide

Check List

[x] Did you check your grammar / reader proof your document using ChatGPT?
[x] Don't forget to squash commits into meaningful chunks before merging
[x] After merging this, please update your task status!
[x] Did you verify that your math formulas are rendered correctly?

Explain the runtime complexity part in detail please!

I will add it on the slide


Added k2 crash app

Related:
The crash app: https://github.com/samsung-cnct/k2-crash-application/pull/1
K2 integration: https://github.com/samsung-cnct/k2/pull/505

On standby until I get Jenkins building a binary for the crash-app


Stretchy elements should inherit current color

If no color is set explicitly, stretchy elements should still have a mask image with currentColor:
node.style.backgroundColor = 'currentColor';

Otherwise this happens:
$$
\overbrace{x+y}
$$

@ronkok

I was also wondering whether any background settings should be marked !important.  According to this discussion, it seems to help with printing on most browsers...

Sometimes I wish that I had written all of the stretchy element SVG code inline. With what I know today, I no longer have to write it with URL escapes. With just JavaScript escapes, the code is somehow less hideous to look upon.  Colors are easy with inline code. The CSS becomes very straightforward.
Here is the SVG inline code for six different surd characters. There are 36 external SVG files currently in the images folder. Transferring it all to inline would add about 20 KB to the JavaScript.
Any takers?

Oh, and those inline SVG are not background, they are instead <svg> written directly into the DOM. All the background printing issues go away.

Another benefit of the SVG being inline is that if you serve it gzip'd a lot of the duplication will get compressed away.

I expect that this issue is now resolved, since stretchy elements are all inline.

@glebm this should be fixed with the release of (v0.8.3)[https://github.com/Khan/KaTeX/releases/tag/v0.8.3].  Please reopen if the new version doesn't solve this issue.

Confirmed, it works! Thank you!


docs: fix broken logo in readme

Current coverage is 90.08% (diff: 100%)

Merging #555 into master will increase coverage by 0.05%

@@             master       #555   diff @@
==========================================
  Files           176        176
  Lines          1926       1926
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           1734       1735     +1
+ Misses          192        191     -1
  Partials          0          0

Powered by Codecov. Last update 590956e...8d4c7fa


How to stop auto closing comment?

Environment data
version 1.11
VS Code version: 1.14
Steps to reproduce
type /*
Expected  behavior
Just emit /*
Actual behavior
It emit /*  */
I have my habit to switch active block for debugging by using */ and /* around it. But not only the autocomplete not aware of existing */. When typing /*, it emit the unneccessary */ to have me delete it. This behaviour should have setting to shut it somewhere

VS Code has a setting to disable automatic closing brackets. You can use "editor.autoClosingBrackets": false in your settings.json.

@DustinCampbell Always set that to false for my whole life. And it seem it work only on bracket, not a comment

Unfortunately, it doesn't appear that there's a way to do this in VS Code, though there's an issue tracking it: https://github.com/Microsoft/vscode/issues/1966.
We actually don't do anything special for closing brackets and comments in C# for VS Code. We've defined the delimiters here but the auto-closing behavior is in VS Code itself.

I see thank you very much


Show X glyphicon when option isn't selected.

When an option isn't selected, instead of removing the glyphicon, just replace it for the "glyphicon-remove".

I'll try to merge this in in some way, but it 'll be hard


Not saving after certain number lines?

I think I found a bug in the extension. Is there a particular max character or line limit?
I am seeing here that I am unable to save additional information on my notes.
Here are my current notes:
https://i.imgur.com/M3uFmfS.png
I then type additional info:
https://i.imgur.com/cOflqnp.png
I then click Save and it seems to save:
https://i.imgur.com/TD9s14c.png
I then close the tab, and open it again... but I only see this:
https://i.imgur.com/My1gzPU.png
So the last bit of information was lost.
Here is some data about my notes:
275 lines
7879 characters
Any way I can solve this issue?

Hi!
First of all, I apologise for not replying to your email.
Now getting to the issue. I have certainly known about this for long, and the probable cause too. After investigating the issue further I realised that chrome.storage allows only 4096 bytes per item to be stored in their storage area. Usually, the history item takes up most of the available storage and exceeds the quota: 4096 bytes.
Since this is a limitation set by Chrome, there is no easy way to bypass it. I say easy because we can, of course, rely on other means of storage such as a GitHub Gist (or any other). But that won't be as easy, simple and quick to set up for the user and for me, as Chrome storage is.
For now, since I think this is predominantly stemming from history exceeding the quota, it is recommended that you delete some of your history items.
If this issue still persists, please copy your console logs in the Chrome developer console and post them here. I'll go through them.
Thanks :)


Replace "beliebig" in Price with const

1. Why is this change necessary?
beliebig is a really bad value as it isn't descriptive, messing with type (string instead of int), repetitive (-> typo prone) and should be removed for better i18n.
2. What does this change do, exactly?
This PR isn't replacing the value beliebig yet but at least adds an abstraction by replacing all instances (in PHP) with a reference to a newly introduced const Shopware\Models\Article\Price:: NO_PRICE_LIMIT. This way the change isn't breaking.
In the future another PR should replace the default value of Shopware\Models\Article\Price::$to with null as this is a better fit.
3. Describe each step to reproduce the issue or behaviour.
4. Please link to the relevant issues (if any).
5. Which documentation changes (if any) need to be made because of this PR?
Using the const instead of default value is recommended.
6. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Right now this is failing cause there are some very wild $query->getSQL() actions are used to do subqueries without copying their bound parameters...

This PR was closed due to inactivity. If this change is still important to you, feel free to create a new pull request.
For more information about our contribution guidelines, see https://developers.shopware.com/contributing/contribution-guideline/.


chore(CPlusPlus): add negative element

PR Checklist:

[X] My submission is formatted according to the guidelines in the contributing guide
[X] My addition is on refer on the language README.md file
[X] My addition does not have a spelling problem
[X] My submission has a proper and user-friendly description of the algorithm
[X] My submission has the time complexity of the algorithm
[X] My submission has sample input-output of the program (NOT FOR PYTHON)

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[X] New algorithm
[ ] Optimization in previous algorithms
[ ] Code style update
[ ] Refactor
[ ] Documentation
[ ] Other, please describe:

Briefly describe the changes in this PR

Other information:

Hi @Ektakanojia,
Due to the branch is not exist on your repo


Simply does not work. No "I" or "U" action completes.

I am trying to do just as told in the documentation - storing an entity in a table.
I use this msg object:
msg.payload = {
  "tableName": "newtable",
  "action": "I",
  "partitionKey": "part1",
  "rowKey": "row1",
  "data": "test"
};

The Azure node just says "Sending" and that's it. No more, no less.
The table was created when I did this the first time, but no entity was stored.
Trying to read using "R" action fails, too. Unless I enter the exact same data manually using Visual Studio Cloud Explorer. Then reading works fine - but again, no "I" or "U" actions work properly.
Why?

It's propably because "data" itself is not recognized as column but as 'packet' of data.
Look at examples: "data": {"Column":"value"}


Fix execution when current folder is not script folder

If demo scripts were called from another folder it was failing to source the selenium.sh script, e.g. bash shellnium/demo.sh.
Fix that by resolving current script location and sourcing with absolute path.

@Rasukarusan tagging you in case you did not get notification about this PR

@hugares
Thank you for the PR! I'll review it within this week.


Fixed adding extra quote when completing strings

Fixed adding extra quote when completing strings, introduced by #70297
supersedes #70699
closes #70535

Thanks!


1.0.0-alpha2-final : Please bring back Clients.OthersInGroup

Use case is a massive turn-based game in which the caller doesn't need to be notified about the play he already did and obviously already knows.
Groups are small (4 members), but if there are thousands of groups in the hub, using Clients.OthersInGroup will save the server from sending thousands of useless "self notifications"

OthersInGroup functionality is currently in dev c8bd72be361b84ded1b7e6c60f39f725500b867e


Run Credo and Dialyzer on Travis

We now have these cool tools to help us. Let’s run them on Travis.

👍


Test Jmix Application Deployment documentation

https://docs.jmix.io/jmix/0.x/deployment/index.html
Wait until AWS Amazon account will be created

Testing completed.
There is a problem: deploying to AWS only works with the root user.
Waiting for the IAM user to be configured. After that, need to add this information to the documentation


Support for<EMAIL_ADDRESS>

Hi, just ran into some compatibility issues while trying to use this library. Would it possible to release a version with support for otel 0.23.0 ?
Thanks!

version 0.19, for opentelemetry 0.23, is published


Heapster is deprecated

Hi Microsoft Team,
Thanks for these insightful material. I found mention of use of Heapster while exporting metrics to a time-series database. Since Heapster is deprecated, this must be updated to use Kubernetis metrics server.

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

ID: 1160782c-3fd6-3236-1cbd-6ddecf56c3db
Version Independent ID: cbcfb4e3-4fb5-9427-3a17-cb296339b1ad
Content: Monitoring a microservices application in AKS - Azure Architecture Center
Content Source: docs/microservices/logging-monitoring.md
Service: architecture-center
Sub-service: reference-architecture
GitHub Login: @MikeWasson
Microsoft Alias: pnp

Thanks for reporting - this issue is under review.  This is a Microsoft Internal DevOps Bug ID AB#198400


View and set eco/power mode

Add the ability to view, and set, which power mode is currently active.

Dependent on: https://github.com/Phype/purei9_unofficial/pull/9

See https://github.com/Ekman/home-assistant-pure-i9/issues/7

Finishing polishment in #31 . Released as version 1.1.0.


vector_tiler null output

vector_tiler is giving null output when tiling a geojson using the tiles created by raster_tiler

It's possible that VectorTiler isn't set up to split linestrings, only
polygons. Might need to be checked/implemented if it's not.
On Mon, Jul 15, 2019 at 12:28 PM Adam Van Etten<EMAIL_ADDRESS>wrote:

vector_tiler is giving null output when tiling a geojson using the tiles
created by raster_tiler
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/CosmiQ/solaris/issues/197?email_source=notifications&email_token=ADDLNIYO3NKYH4MRW44K2UDP7SQSVA5CNFSM4IDYS75KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G7IQFDQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADDLNI7UDSEM7TDDU2PGQHLP7SQSVANCNFSM4IDYS75A
.

First off, much thanks for the Solaris toolkit and kudos on the initial release...really helpful!
I ran into this same problem of getting null outputs from VectorTiler. It worked as expected with the examples from the tutorials but encountered when I tried tiling my own data samples.
At least in my case, the problem seems to stem from my raster input not being in epsg:4326. When raster_tiler.tile_bounds is generated by RasterTiler, they contain non-4326 coordinates like<PHONE_NUMBER>, 9350190.01551792,<PHONE_NUMBER>974508, 9350249.838515371)
When passing this into vector_tiler.tile(tile_bounds=), clip_gdf() isn't able to find the bounds to clip the input geojson/gpd.
I confirmed and fixed the problem by converting the generated raster_tiler.tile_bounds to 4326 first and then passing that into vector_tiler.tile().
Here's a Colab notebook showing the issue and my quick fix: https://nbviewer.jupyter.org/gist/daveluo/7c54d0fbb1d1f32307d2af1a5b640254#Make-tiles-for-znz029

Thanks a lot for doing the diagnosis and exploration @daveluo! This may have been fixed with #201 - if you feel like checking, you could pip install the version on the dev branch and see if it fixes it.
I'll dig into it myself later tonight and make sure everything's resolved.

Thanks for the heads up! I pip installed the dev branch but the latest changes did not fix the non-4326 tile_bounds issue.

OK thanks @daveluo, I'll check back into it in a little bit.

Alright yeah, there wasn't anything written into the code to handle cases where the raster and vector have different CRSs. I've added an argument to VectorTiler.tile() and VectorTiler.tile_generator(), tile_bounds_crs, where the user can specify the EPSG code for the raster CRS; solaris will check if it differs from the vector CRS and transform the bounds as needed. See #203.
Thanks for bringing this to our attention @daveluo!

Resolved with #203.


Shop with ID Create not found

Hi,
In trying to create shops our users get the following error: shop > Shop with ID create not found. The command they are using is:
/shop create 1 20 20  - trying to create a shop.
Any ideas why this may be happening? I am on 1.12

Are you sure you're using ShopChest? Or do you have another plugin using the command "shop"?

Yep, using shopchest and following the instructions provided:
Make sure you have permission
Make sure you have enough money
Make sure you hold the item you want to sell in your hand
/shop create
You can set buy-price or sell-price to 0 to disable buying or selling
Click the chest which should be the shop
If not already done, put the items inside the chest

But the message you're getting cannot be thrown by ShopChest