localized components containing i18n tags will produce identical chunk hashes when building with --localize

🐞 Bug report
Command (mark with an x)

[ ] new
[x] build
[ ] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] xi18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Description
I have an application that is being translated in several (21 at the moment) languages using the Angular i18n mechanism. I'm compiling the app AOT, therefore I need a separate build for each locale. To achieve this I setup my Azure pipeline to build the app using multiple agents in parallel, one for each locale, and it works like a charm. However, this is very resource-consuming, so I'm updating to Angular 9 in order to use the --localize option, which perfectly fits my needs.
After the update I can build the app once and translate it to all languages in a few seconds, which is amazing.
I am now facing this issue: when I build my application for all locales, the resulting chunk hashes are identical. As a result, browser cache is not invalidated and files are not re-requested, so the app is not translated.
🔬 Minimal Reproduction

clone this repo
run npm install
run ng build --localize
check the dist folder: file content for main.js is different for each locale (template text is translated), while the hashes are the same

🌍 Your Environment

Angular CLI: 9.1.0
Node: 10.19.0
OS: win32 x64

Angular: 9.1.0
... animations, cli, common, compiler, compiler-cli, core, forms
... language-service, localize, platform-browser
... platform-browser-dynamic, router
Ivy Workspace: Yes

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.901.0
@angular-devkit/build-angular     0.901.0
@angular-devkit/build-optimizer   0.901.0
@angular-devkit/build-webpack     0.901.0
@angular-devkit/core              9.1.0
@angular-devkit/schematics        9.1.0
@ngtools/webpack                  9.1.0
@schematics/angular               9.1.0
@schematics/update                0.901.0
rxjs                              6.5.5
typescript                        3.8.3
webpack                           4.42.0

Anything else relevant?
In my scenario, the user language preference is stored in the cookies, and a shell app serves the corresponding folder. I can't add the locale to the URL.
Therefore in my angular.json I set baseHref="/" for all locales.
I am aware of this issue and the PR mentioned there, however the issue is still not resolved.

Duplicate of https://github.com/angular/angular-cli/issues/16526

Seems like it was supposed to be fixed with https://github.com/angular/angular-cli/pull/16817 but I am having the same issue with<EMAIL_ADDRESS>

I'm also having this issue.
My repro: https://github.com/pjm0616/pastebin/tree/202004-angular-localize-outputhash
@angular and @angular-devkit version: 9.1.3
I've also included the build output in the repo.
The dist-localize folder contains the build output with localize=true, and the dist folder was built separately for each locale without localize=true.
$ npm run buildall
$ sha1sum dist{,-localize}/*/main-es5*.js
6e0407f00e27d877cb8bbcf0d7b069dfcc42cb8b  dist/en/main-es5.27d2d00de67fb6cace7d.js
bc09ba481672f092a359949c6d55b14e319fc63a  dist/ko/main-es5.54a148db9b0b5fb29549.js
6e0407f00e27d877cb8bbcf0d7b069dfcc42cb8b  dist-localize/en/main-es5.54a148db9b0b5fb29549.js
bc09ba481672f092a359949c6d55b14e319fc63a  dist-localize/ko/main-es5.54a148db9b0b5fb29549.js

#16526 was never addressed and was closed without resolution. #16817 fixes a separate issue, where the hashes would remain the same when rebuilding after changing translations and not changing code.
This exact case of the file hashes being the same for different locales is still present. It was considered a regression in #16526, but has since been reclassified as a feature, it seems.

I agree with @bridzius that #16817 fix only a symptom of the more deep issue of #16526.
I also have the same hash for multiple main.hash.js files that don't have the same content due to Angular i18n post compilation change.
(Side note, I think that, even if I'm not 100% sure, it mess with Angular service worker itself when I switch between translations of my app).
@clydin in https://github.com/angular/angular-cli/issues/16526#issuecomment-582366860 you say I quote:

Removing the use of webpack’s file hashing and moving the file name creation to a later stage of the build process is currently planned. However, such a change is out of scope within the current release candidate period.

It seems like the right things to do to fix this issue.
But you didn't post any link. Can you tell where we could upvote for this solution, or even help?
(@nas90 thanks for (re-)opening this issue!)

@alan-agius4 any updates on this one?

Recently migrated to Angular 9 and faced this issue. Was really disappointing to see this. Even the different configurations for different languages (for example --configuration=production,de and --configuration=production,fr, plus properly set angular.json as Documentation advises) lead to the same hashes. This means that there is only a one way to get the flat files deployment working –  to use the deprecated build parameters --i18nLocale, --i18nFile and --i18nFormat 😞

Just tried on latest Angular (10.0.5) and I'm still facing this issue 😢

This issue is still open hence it hasn’t been addressed yet.
While this is on our radar, there are issues which have a higher priority.
PRs to implement this feature are welcome.

I am having the same issue with angular 11.1.0.

I having the same issue (exactly this particular one) using Angular 11.2.11.
Is there a quick workaround to get a different hash for all locales?

also running into this issue on Angular 12.0.0.  Using the --localize hash values are the same for every locale

I'm facing same issue (v12.2.2)

I can confirm this is still a problem. The workaround posted by @ryansheehan works, but this greatly increases build time. We have a large app with 13 different languages and migrating to Angular i18n effectively increases our build time x13. We cannot take the risk of having the same file hashes across different languages.

We have the same issue. I think this issue makes --localize unusable. We also build all languages separate.

I confirm this is still an issue in v13, the workaround to build the different locales separately can become faster using run-p of npm-run-all.

As a different workaround that doesn't involve running a separate build for each language, I wrote a shell script that adds language codes to the file hashes and their occurrences after the build finishes. This ensures uniqueness between different languages.
There is probably a more efficient way of doing this and I haven't tested it on different setups, but here it is in case it helps anyone:
locale-utils/fix-hashes.sh
#!/bin/bash -eu

# Appends language code to file hashes for js files
# (and references to those files) in dist/
# Motivation: https://github.com/angular/angular-cli/issues/17416

cd ./dist/

LOCALES=($(ls))

cd "./${LOCALES[0]}"

# Find js files | extract hash | filter duplicates
# Hashes are the same for every language (this is the problem we are trying to solve)
# So we only need to do this once.
HASHES=($(find . -maxdepth 1 -name '*.js' | sed -E 's|.*\.([a-z0-9]*)\.js|\1|' | sort -u))

cd ..

for LOCALE in ${LOCALES[@]}
do
    echo "Updating file hashes for $LOCALE"
    cd "./${LOCALE}"

    # Replace occurrences of each hash.
    for HASH in ${HASHES[@]}
    do
        HASH_SUFFIXED=${HASH}_${LOCALE}
        # echo "Replacing occurrences of hash: $HASH with $HASH_SUFFIXED"
        find . -maxdepth 1 -type f -exec sed -i -E "s|$HASH|$HASH_SUFFIXED|g" {} \;
    done

    # Rename files to update their hashes too.
    for FILE in *.js
    do
        # echo "Renaming $FILE"
        mv "$FILE" "${FILE/.js/_${LOCALE}.js}"
    done

    cd ..
done

I added it as a postbuild script in package.json so it runs automatically after each build:
        "postbuild": "./locale-utils/fix-hashes.sh",

I wrote a shell script that adds language codes to the file hashes and their occurrences after the build finishes. This ensures uniqueness between different languages.

I want to point out that this won't fix the problem of source maps not working properly
https://github.com/angular/angular-cli/issues/16526#issuecomment-572142021

Have the same problem with Angular 13 and didn't find a workaround. No matter what I do I always end up with different main-bundles that have the same hash..

Is there is an update on this issue?
Having the same hash for different file contents is counter-intuituve, especially as this breaks cache busting, and make it impossible to do a location.reload() when the language is set by a cookie, which is the case for Firebase.
This issue makes it impossible to deploy Angular localized apps on Firebase

That would be great.
I still don't quite understand though why this is not considered a bug: when building an angular app with multiple locales, the command outputs a main.js file for each locale, containing locale-specific data, so files with different content, but all these files have the same hash.
Could the locale be appended to the filename, as a quick fix, basically producing main.en.[hash].js, main.fr.[hash].js,...

I have updated the workaround script in my previous comment to improve performance and provide support for macOS.

Looking over the comments quickly, it sounds like most of the motivation here comes down to wanting to put multiple locale outputs into a single flat directory, is that the ask here? Why exactly do folks want to do this, does it make some server configuration or URL pathing easier? I think a better understanding of the motivation here would help us identify the right solution.

Looking over the comments quickly, it sounds like most of the motivation here comes down to wanting to put multiple locale outputs into a single flat directory, is that the ask here? Why exactly do folks want to do this, does it make some server configuration or URL pathing easier? I think a better understanding of the motivation here would help us identify the right solution.

For our projects we do path based routing in our ingress, so we opted to do localization with other methods (ie cookies).  Without different hashing on the localized files, the browser cache will return the same locale even though the user switched locales, and a different index file is being served.

For our projects we do path based routing in our ingress, so we opted to do localization with other methods (ie cookies). Without different hashing on the localized files, the browser cache will return the same locale even though the user switched locales, and a different index file is being served.
We dont need to flatten anything. Our server-side picks up the cookie and serves the right files.

That sounds like the Cache-Control header is wrong then? If the response is contingent on a cookie which the browser doesn't key its cache by, then I think it's not safe to let the browser cache that URL.
I can see that marking those URLs as uncacheable could reduce performance for users who don't change languages, but I think that's a consequence of not putting the locale in the URL. Using a different hash is just another form of making the URL locale-dependent so the locale is taken into account in the browser cache key. Whether that's done via a locale-dependent hash, or an explicit locale path / query parameter is an implementation detail to the browser cache.

I don't think this is related to caching at all. You can reproduce it like explained above with just a regular "ng build --translate" - you end with 2 folders, one for each locale - so far so good!
But if you deep-dive it you notice that your main.js bundles (you have one of those for each language) is messed up. You end up with different bundles because there are differences due to language. But the HASH of all these main.ts bundles is the same.
Now depending on your solution you might end up with rewrites that (for the users browser) appear to be from the same path - that would be no problem, but since your ENGLISH-main.ts and your GERMAN-main.ts have the same hash you end up with issues when you switch the language.
Given, there may be ways to host it differently - but still: how could that NOT be a bug? The Hash of my german main.ts file is obviously based on the content of my english main.ts file. That's just wrong, it should be based on the content of my german main.ts, right?

Given, there may be ways to host it differently - but still: how could that NOT be a bug? The Hash of my german main.ts file is obviously based on the content of my english main.ts file. That's just wrong, it should be based on the content of my german main.ts, right?

As I understand it, I believe we use Webpack's [hash] replacement here, not [contenthash]. The hash value is not based on file content, it's only guaranteed to be unique within a build. This SO answer seems to have a good breakdown of the differences. @clydin is that understanding correct?
Given that generated resources shouldn't be loaded into the same page (you shouldn't be loading a French chunk into a German application), then I don't think it's necessarily wrong to see duplicate hashes across different locales.

Thanks for explaining and the SO link! I think I understand, but it still feels wrong and there might be some bugs with that implementation - if I build my project while only changing the german translation file the hash would still be unchanged, right? That would lead to people not getting the newest version of my german main.ts file! So basically only changing translations without touching a source file is not supported with the current build?
The question is: is there a good reason NOT to use [contenthash] or maybe add an optional flag for that? This puts a strong opinion/restriction on how files are served. For me it also seems a bit also unintuitive (because of what I believe a file-hash should do)
Example
will try to explain my approach while hosting my mini-homepage https://budisoft.at that lead to this problem:

on my server there is a folder de_DE and another folder en_US, containing the two localized applications
I set some rules on my reverse proxy (based on "Accept-Language" and a Custom Cookie) to rewrite all URLs to /de or /en_US accordingly
the basehref is just set to "/" - my rules ensure that the correct bundles are loaded

This was a naive - but I think reasonable - approach and would work perfectly with [contenthash], but fails with [hash] (wrong language bundle get's loaded from cache)
Solution
Thanks to your response I think I finally understand what I'd have to do instead of dirty workaround:

rewrite requests just for the index.html.
keep the baseHrefs like default, e.g. "en_US" and "de"
now my english index.html would request a different file location (e.g. https://budisoft/en_US/main.03b15c0004fb5dfc.js) than my german version, even when the hash is the same.

I will implement it that way, but still hope that there can be a solution that enables some flexibility while serving the bundles sometime.

Hey Bart, I think you are a bit in the same "mental trap" as I was. Yes, URLs to a specific content-page should not contain a language url, to ensure deep-linkability across languages.
Example: https://your-page/order/123456 <= this is a link to one specific order and should work, regardless of language.
But this is only important for your own pages/routes! This doesn't matter for JavaScript bundles though. Let me explain:
If you give the link above to a german user your reverse-proxy will probably use your cookie (+ fallback) to determine the correct index.html and retrieve it from a file system like this:
/opt/data/your-page/de for german OR
/opt/data/your-page/en-US for english
In each index.html you have

This issue causes problems when we upload source maps to Sentry. We have to create a release for each Language in Sentry. This then breaks resolved in the next release working correctly.

Hey, I just rewoved the script-workaround (thank you for that @BartCorremans !) from my homepage build https://budisoft.at and now everything is working fine. What I did was completely removing "baseHref": "/"  in my angular.json - so the baseHrefs stay at default. I end up with the following after build:

german index.xml contains
english index.html contains

I changed my rewrite-rules to only rewrite for my index.html - all other urls/files can be loaded as-is
Now my bundles (e.g. main.1234567.js) are loaded from different URLs and it doesn't matter any longer that they have the same filename.
One small issue I had with that, was that routing did include the base-href at first. (e.g. routing to /articles would bring me to https://budisoft.at/en_US/articles). After consulting the documentation I added a provider in my app-module to bring the context back to /:
{provide: APP_BASE_HREF, useValue: ''}
With that everything seems to work as expected and I am happy

Thanks for explaining and the SO link! I think I understand, but it still feels wrong and there might be some bugs with that implementation - if I build my project while only changing the german translation file the hash would still be unchanged, right? That would lead to people not getting the newest version of my german main.ts file! So basically only changing translations without touching a source file is not supported with the current build?

The hash probably should be dependent on the translation files, as I think it's reasonable to expect these hashes to update on any text content changes. If that's not the case, then that sounds like a separate issue we can look at.
Since all the hashes are currently tied together across locales, that would mean that your French build is actually dependent on all the other translations, which is a bit weird and bad for build caching. It would make more sense that the French build only depends on French translations. @alan-agius4 would there be any build improvements from better isolating languages this way? In theory I think it would be better, but in practice everything is done via a Webpack compilation which probably preprocesses everything for this hash, so I'm not sure this would actually have a meaningful effect?

Thanks to your response I think I finally understand what I'd have to do instead of dirty workaround:

rewrite requests just for the index.html.
keep the baseHrefs like default, e.g. "en_US" and "de"
now my english index.html would request a different file location (e.g. https://budisoft/en_US/main.03b15c0004fb5dfc.js) than my german version, even when the hash is the same.

Yes, I think that is the right approach. If you want "language-agnostic" URLs, those should be limited to the URLs actually visible to users (such as HTML pages). JS bundles aren't typically user-visible or stable (hashes change on each deployment), so putting them in a locale-specific URL doesn't negatively impact the user.
This feels like it also addresses @BartCorremans' issue.

This issue causes problems when we upload source maps to Sentry. We have to create a release for each Language in Sentry. This then breaks resolved in the next release working correctly.

I'm not very familiar with Sentry, but it sounds like they're keying sourcemaps by file name? If so, that makes it a subset of the flattening locale resources use case. Given that Sentry has direct Angular support I wonder if that might be better addressed on their end? Seems to me like a sourcemapping utility should be able to handle multiple files with the same name in different directories, but I'm not an expert in that area and I'm sure there's more nuance to it than that.

The question is: is there a good reason NOT to use [contenthash] or maybe add an optional flag for that? This puts a strong opinion/restriction on how files are served. For me it also seems a bit also unintuitive (because of what I believe a file-hash should do)

I agree that this can be unintuitive. Devs have a strong bias to interpret a hash in a file name as a hash of that file's contents. That's not always true (as in this case), but it is often the initial expectation. One other advantage to consider about a [contenthash]-based approach is that files can be cached across deployments for any chunk that didn't change. This can result in some performance improvements, but it's hard to measure the exact impact of that.
I think the main reason not to change the current behavior is that Webpack chunking happens before localization as I understand it. So just changing to [contenthash] doesn't solve the problem because subsequent edits to these files to merge translations would alter the file and change its hash. The Angular CLI then needs to manually update all the files (and their chunk dependencies on each other) to use the new hashes. I think this also doesn't solve the problem of file conflicts, since a JS file without translations will be duplicated across locales, though in that case it is at least the same for each locale. That basically means we'd be implementing our own hashing system on top of Webpack's.

Hey Bart, I think you are a bit in the same "mental trap" as I was. Yes, URLs to a specific content-page should not contain a language url, to ensure deep-linkability across languages. Example: https://your-page/order/123456 <= this is a link to one specific order and should work, regardless of language.
But this is only important for your own pages/routes! This doesn't matter for JavaScript bundles though. Let me explain:
If you give the link above to a german user your reverse-proxy will probably use your cookie (+ fallback) to determine the correct index.html and retrieve it from a file system like this: /opt/data/your-page/de/index.html for german OR /opt/data/your-page/en-US/index.html for english
In each index.html you have

Following up with this conversation, really appreciate everyone's feedback here. It looks like we've mostly figured out how to meet these needs with the CLI's current behavior. I think the recommendation for anyone finding this issue in the future:

Hashes are not keyed by the locale. That means if you want to cache the same file across different locales, that locale must appear in the URL.
This only applies to JS files generated by the CLI, not user-visible links like HTML files. If you don't want user-visible links to have locales (like /my_home_page.html), you can automatically infer the user locale via cookies or backend state, and then link to /${locale}/main.abc123.js to load your application. That way user-facing links are locale-independent, but still load localized and cacheable JS sources.

I agree this behavior can be confusing, and in a perfect world asset hashes would probably be locale dependent. However, there is a lot of complexity and nuance to make that work and maintain it over time. Given that this can be worked around, I don't think changing this is the best use of our limited time and energy. So I'll close this issue for now, and we can reevaluate if it becomes a bigger challenge in the future.


Follow-up regarding GH Actions updates

Follows up https://github.com/quarkus-qe/beefy-scenarios/pull/307
Updating below github actions:
From DeLaGuardo/setup-graalvm@master to graalvm/setup-graalvm@v1
From jitterbit/get-changed-files@v1 to<EMAIL_ADDRESS>

Blocks https://github.com/quarkus-qe/beefy-scenarios/pull/308

We should be using GraalVM 22.3, but that can be handled in follow-up PR

@rsvoboda I already opened it this morning https://github.com/quarkus-qe/beefy-scenarios/pull/308 but I was waiting for this PR to be merged first.
So will rebase that one later today or tomorrow


Deployment

Deployment
User Story
As a boot camp student,
I want the prework notes to be structured on a webpage,
So that I can easily find and read the information.
Acceptance Criteria
GIVEN a Prework Study Guide website,
WHEN I visit the website using the URL,
THEN I can access my website from any browser

just deployed my first website!


bintrayPublishSettings vs bintraySettings

publish with
seq(bintrayPublishSettings:_*) gives java.lang.RuntimeException: Repository for publishing is not specified.
but seq(bintraySettings:_*) works

closing. it shouldn't really matter anymore with the latest release http://notes.implicit.ly/post/118407794254/bintray-sbt-0-3-0 you shouldn't have to think to much about appending these settings lists any more


Strange behavior when resizing with autocmd

Hi, I am currently trying to get nvim to resize accordingly when I resize the terminal.
I tried to set it up with an autocomand like
autocmd VimResized * <cmd>lua require"focus".resize()<cr>
but this produces a blank screen and then, after what looks like a timeout, redraws without a resize.
Is this related to no window currently being focused? Can I work around this somehow?
Thanks and best regards!

Hey @thraizz I see your issue on my end too.
Noticed that if I hit  or any key for that matter after resizing my terminal window that the windows resize correctly again.
Perhaps adding an  at the end of that aucmd there might solve the issue?

Hello,
I have the same issue with the following autocmd :
vim.api.nvim_create_autocmd("VimResized", {
  desc = "Resize split.",
  group = vim.api.nvim_create_augroup("Focus.nvim", { clear = true }),
  pattern = "*",
  callback = function(args)
    require("focus").resize()
  end,
})

There is a message in the :messages with the following error :
Error detected while processing VimResized Autocommands for "*":
Error executing lua callback: ...ck/packer/start/focus.nvim/lua/focus/modules/resizer.lua:68: E592: 'winwidth' cannot be smaller than 'winminwidth'
stack traceback:
	[C]: in function '__newindex'
	...ck/packer/start/focus.nvim/lua/focus/modules/resizer.lua:68: in function 'split_resizer'
	...vim/site/pack/packer/start/focus.nvim/lua/focus/init.lua:54: in function 'resize'
	/home/clement/.config/nvim/lua/user/windows/init.lua:46: in function </home/clement/.config/nvim/lua/user/windows/init.lua:43>

Hope this helps :)

Ah yes, this is a common problem. In 0.8 is has been allowed to set winminwidth to 0
This should provide a stable path to fix this problem.
I will have a look

The refactor branch has tests now. We should write tests for resizing vim.

I'm getting the following error trying to open a plenary.nvim popup for lsp code actions:

I'll look into this today - would you mind opening a separate issue and providing a minimal init?

Yes, will do.


Update scrypto deps to bottlenose

This PR updates scrypto-math with latest Radix Engine changes for oncoming bottlenose protocol update.
For now tag bottlenose-9396c507 is used but eventually it shall be switched to v1.2.0 once bottlenose is released.

@fpieper since scrypto 1.2.0 has been released maybe it make sense to update scrypto-math?
I can amend this PR if required or you can create your own if it makes more sense for you 😄

@lrubasze Thanks for the PR. Will update the examples too and merge it then.
Nice that you now have published Scrypto on crates 👍


remove provisional prototypical code for new haskell nodes

Issue Number

Overview

[x] I have removed extra dependencies for the Haskell Shelley prototypical code
[x] I have removed the provisional networking interface for the new Haskell nodes

Comments

This isn't used and comes at a great build cost and cache size increase. So, unless
required, we are better off without it.

bors r+


Adding Department Tagging

Added Department = devops tag to Azure Virtual Machine Resource

Confirming my own Pull Request to add tagging.


Force the passage of project ID when listing variables for a deliverable, use this to return variables that have values for a deliverable but are outdated

#2339  👈
main

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

Join @nickgraz and the rest of your teammates on  Graphite


WaitForFileChooser() never resolves but fileChooser opens up

Use case
When trying to upload file through file chooser using the code presented in the docs, we encountered situation where waitForFileChooser() never resolves and fails on timeout, but file chooser opens up and is not torn down after executions ends.
Steps to reproduce
Puppeteer version: 13.3.2
Platform / OS version: Windows 10
Node.js version: 12.22.4
Running this code from the docs
var futureFileChooser = page.waitForFileChooser();
// some button that triggers file selection
await page.click('#upload-file-button');
var fileChooser = await futureFileChooser;

await fileChooser.accept([File('myfile.pdf')]);

or, alternatively:
const [fileChooser] = await Promise.all([
  page.waitForFileChooser(),
  page.click('#upload-file-button'),
]);
await fileChooser.accept([File('myfile.pdf')]);

results in execution failing with following error:  TimeoutError: waiting for waiting for file chooser failed: timeout 30000ms exceeded. WaitForFilesChooser() never resolves, but actual file chooser opens up and is not torn down when execution ends.
While looking into similar issues, we assumed there might be some race condition preventing data from click() action to be received in time. But by the time click() invokes, nothing on the page may cause that. Furthermore, following code was working fine on puppeteer 4.0.0 before we updated the package to 13.3.2.
From what we observe, it seems that file chooser is not being intercepted, as it would not appear on the screen otherwise. It opens but not reported by the browser over CDP.
Is there something we're doing wrong or we stumbled upon a bug?

Please check out the latest Puppeteer version and documentation at pptr.dev. The docs you pointed to are not supported.

Please check out the latest Puppeteer version and documentation at pptr.dev. The docs you pointed to are not official.

One of the code snippets in the report is identical to what is used in the official docs on FileChooser you shared.

I had the same issue before, however my issue was that the clicked element selector is wrapped by a root-shadow, so to solve it I used something like:
page.click('>>> #upload-file-button')

Do we have any solution for this. Encountering same issue.

I had the same issue in an older version of Puppeteer, then upgrading the versions got me stuck into a different bug Protocol error (DOM.resolveNode): Either nodeId or backendNodeId must be specified., that I see there is there is another issue opened for here :
https://github.com/puppeteer/puppeteer/issues/7234#issuecomment-2481698839


3.6.8

Codecov Report

Merging #157 (e57736b) into master (20ae695) will increase coverage by 2.60%.
The diff coverage is 65.49%.

@@            Coverage Diff             @@
##           master     #157      +/-   ##
==========================================
+ Coverage   65.93%   68.54%   +2.60%
==========================================
  Files          70       68       -2
  Lines        1641     1580      -61
  Branches       88      105      +17
==========================================
+ Hits         1082     1083       +1
+ Misses        505      423      -82
- Partials       54       74      +20

Impacted Files
Coverage Δ

...an/hashchecker/components/clipboard/Clipboard.java
0.00% <0.00%> (ø)

...hecker/components/hashcalculator/api/HashType.java
84.00% <ø> (ø)

...caldatastorage/impl/ormlite/OrmLiteConfigUtil.java
0.00% <ø> (ø)

...r/components/localdatastorage/models/DbEntity.java
40.00% <ø> (ø)

...omponents/localdatastorage/models/HistoryItem.java
56.00% <ø> (ø)

...features/hashcalculator/input/TextInputDialog.java
100.00% <ø> (ø)

.../features/hashcalculator/lists/actions/Action.java
100.00% <ø> (ø)

...calculator/lists/actions/types/UserActionType.java
100.00% <ø> (ø)

...eatures/history/view/list/HistoryItemsAdapter.java
100.00% <ø> (ø)

...r/features/history/view/loader/HistoryPortion.java
100.00% <ø> (ø)

... and 65 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 faaedad...e57736b. Read the comment docs.


wrong padding size generated from different arch

A generated code via cross compiling environment is difference from naitive generated code.
The cross compiling environment is x86_64 linux .
And a target environment is RaspberryPi(arm-unknown-linux-gnueabihf).

bindgen 0.35.0
llvm 3.9.1

Input C/C++ Header
typedef int aligned[64] __attribute__((__aligned__(8)));
typedef int array[64];

typedef struct {
  aligned buf; // 256
  int x;
} a_record_t;

typedef struct
{
  array buf;
  int x;
} b_record_t;

Bindgen Invocation
On x86_64 linux, I have generated code as gen_arm.rs, and send compiled binary to my raspberrypi.
x86_64 $ mkdir pi
x86_64 $ sshfs pi@raspberry.local:/usr pi
x86_64 $ TARGET=arm-unknown-linux-gnueabihf bindgen --output gen_arm.rs wrapper.h -- -I pi/include/clang/3.9.1/include --sysroot=/work/pi -I pi/include/arm-linux-gnueabihf
x86_64 $ rustc -C linker=armv6-rpi-linux-gnueabihf-gcc --target arm-unknown-linux-gnueabihf --test gen_arm.rs
x86_64 $ scp gen_arm pi@raspberry.local:.

On RaspberryPi, I have generated code ordinally and compile it using native rust compiler.
rpi $ bindgen --output gen_rpi.rs wrapper.h -- -I /usr/lib/clang/3.9.1/include --sysroot=/usr -I /usr/include/arm-linux-gnueabihf
rpi $ rustc --test gen_rpi.rs

Actual Results
Below is the result of the layout test which is generated on x86_64 linux environment.
rpi $ ./gen_arm
running 2 tests
test bindgen_test_layout_b_record_t ... ok
test bindgen_test_layout_a_record_t ... FAILED

failures:

---- bindgen_test_layout_a_record_t stdout ----
        thread 'bindgen_test_layout_a_record_t' panicked at 'assertion failed: `(left == right)`
  left: `260`,
 right: `264`: Size of: a_record_t', gen_arm.rs:13:5
note: Run with `RUST_BACKTRACE=1` for a backtrace.

failures:
    bindgen_test_layout_a_record_t

test result: FAILED. 1 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out

On the other hand, below is the result of the same test above.
But the test code have been generated on RaspberryPi.
rpi $ ./gen_rpi
running 2 tests
test bindgen_test_layout_b_record_t ... ok
test bindgen_test_layout_a_record_t ... ok

test result: ok. 2 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

Expected Results
What is expected of results of tests, both results are matching.
rpi $ ./gen_arm
running 2 tests
test bindgen_test_layout_b_record_t ... ok
test bindgen_test_layout_a_record_t ... ok

test result: ok. 2 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

Can you show the generated file generated in your raspberry PI? Wanna know what's the expected size / alignment.
There are a couple of uses of size_of::<*mut ()>() and align_of::<*mut ()> in the tree that should more than probably be replaced by actual target information.

generated code
This is generated code in raspberrypi and it's command.
# command
pi$ bindgen --output gen_rpi.rs wrapper.h -- -I /usr/lib/clang/3.9.1/include --sysroot=/usr -I /usr/include/arm-linux-gnueabihf

// generated code
pub type aligned = [::std::os::raw::c_int; 64usize];
pub type array = [::std::os::raw::c_int; 64usize];
#[repr(C)]
#[derive(Copy, Clone)]
pub struct a_record_t {
    pub buf: aligned,
    pub x: ::std::os::raw::c_int,
    pub __bindgen_padding_0: u32,
}
#[test]
fn bindgen_test_layout_a_record_t() {
    assert_eq!(
        ::std::mem::size_of::<a_record_t>(),
        264usize,
        concat!("Size of: ", stringify!(a_record_t))
    );
    assert_eq!(
        unsafe { &(*(::std::ptr::null::<a_record_t>())).buf as *const _ as usize },
        0usize,
        concat!(
            "Offset of field: ",
            stringify!(a_record_t),
            "::",
            stringify!(buf)
        )
    );
    assert_eq!(
        unsafe { &(*(::std::ptr::null::<a_record_t>())).x as *const _ as usize },
        256usize,
        concat!(
            "Offset of field: ",
            stringify!(a_record_t),
            "::",
            stringify!(x)
        )
    );
}
#[repr(C)]
#[derive(Copy, Clone)]
pub struct b_record_t {
    pub buf: array,
    pub x: ::std::os::raw::c_int,
}
...(snip test cases for b_record_t)...

In the generated code for the same header on my (x86_64) linux environment, there is no __bindgen_padding_0: u32 field of a_record_t type.
expected results
In the above generated code, ::std::mem::size_of::<a_record_t>() = 264usize is expected (1st case of bindgen_test_layout_a_record_t).
And the generated code on x86_64 linux expects same size (264usize) for a_record_t. (But the actual resut is 260.)
For further experiments, I wrote the below C code.
// test.c
#include <stdio.h>
typedef int aligned[64] __attribute__((__aligned__(8)));
typedef int array[64];

typedef struct
{
  aligned buf;
  int x;
} a_record_t;

typedef struct
{
  array buf;
  int x;
} b_record_t;

#define print_sizeof(ty) \
                printf("sizeof %s: %zu\n", #ty, sizeof(ty))

int main (int argc, char * argv[]) {
        print_sizeof(a_record_t);
        print_sizeof(b_record_t);
        return 0;
}

This code outputs same result to native generated rust code.
rpi $ gcc -o test test.c && ./test
sizeof a_record_t: 264
sizeof b_record_t: 260

So, I guess that cross-generating is wrong.


Regression: PulsarAuthorizationProvider behavior

Describe the bug
Existing implementations of AuthorizationProvider that extend PulsarAuthorizationProvider may regress following PR https://github.com/apache/pulsar/pull/12064.  The initialization method is not called.  The reason seems to be that PulsarAuthorizationProvider::initialize(ServiceConfiguration, PulsarResources) does not call PulsarAuthorizationProvider::initialize(ServiceConfiguration, ConfigurationCacheService).   In other words, the PulsarAuthorizationProvider defeats the back-compat logic in AuthorizationProvider.
To Reproduce
Steps to reproduce the behavior: use MultiRolesTokenAuthorizationProvider or AuthorizationProviderOAuth.  Observe that the provider is not initialized.
Expected behavior
Initialization of authorization provider.

@merlimat please take a look.


Change port number

How can I change the port number, if it is already in use?

There is no option for that yet, but I can add that if that is useful!

In the latest version, there is a new configuration file at: '~/.omega/config.yaml', which has a 'port' parameter you can change.


Updates to installation guide regarding forbidden error in Apache

Describe the task
When setting up the portal, @amisha296 faced an error that throws a page saying "Forbidden". There were some changes to the virtual host file that made it work.
Expected behavior
An optional step should be added to the installation guide at the time of virtual host step, with the proper error message in case a user sees and what to change/modify in the virtual host file.

The issue has been fixed refer screenshot


Version checker

Would be nice if ASB Explorer would automatically check for latest version and tell me if a new version is available. As a later update, it would be nice if it would also auto-update. I want to try and add the versioning support and so want a simple issue to track

I was hoping to use https://api.github.com/repos/paolosalvatori/ServiceBusExplorer/releases/latest to get the latest version and compare it to the apps version. But it looks like the API endpoint cannot be called without authentication. Any ideas?

I was able to invoke the link in incognito mode.

Yeah, thats why I was hopeful. But when I tried it via httpClient it failed. Looking at fiddler, GitHub provides some cookies that I think allows it to work. Still looking.

Figured it out, when calling from code, api.github.com needs one to send the "User-Agent" header. Otherwise it returns a 403: Forbidden error.

If not mistaken, to use any of GitHub API, user agent header must be present. Good luck 🙂

Here is what I was thinking,

have a user setting that allows user to set number of days to check for new version check.
Default it to 30 days (0 would be never check)
When a new version is detected - notify user via a dialog and give them a link to GitHub releases page.

In the future, make it so, it will download and update automatically.
What do you think?

Sounds good.

Very good. However I assume we will need a installation program, preferably packaged as an MSI for the automatic updating.

As a later update, it would be nice if it would also auto-update.

Separate issue.

Interesting idea, I'm going to merge your PR


hdf files

Dear developer:
In my data, The fragment length is 300bp, the length of sequenced reads is 150bp. However, in SYNAPSE, there is not "hg38_fragLen300_kmer150", how can I get or generate "hg38_fragLen300_kmer150" file?
@timreddy @Young-Sook

Hi,
If there is no covariate file (.hdf) that matches with your data, I'd recommend using correctBias subcommand instead of correctBias_stored subcommand. To run correctBias subcommand, you need to have mappability file for 150mer, though.
If you don't have mappability file for 150mer, I'd recommend using hg38_fragLen300_kmer100 hdf files with  correctBias_stored subcommand. We haven't tested how much the fit of the model is sensitive to different parameters such as kmer length, but I believe hg38_fragLen300_kmer100 will still provide pretty reasonable fit.
In terms of runnning time, correctBias subcommand takes longer time than correctBias_stored subcommand as written in README file. The running time of correctBias subcommand is still reasonable, unless the data is not whole genome.
Let me know if you have any more question!
Thanks,
Young-Sook

Hi,
If there is no covariate file (.hdf) that matches with your data, I'd recommend using correctBias subcommand instead of correctBias_stored subcommand. To run correctBias subcommand, you need to have mappability file for 150mer, though.
If you don't have mappability file for 150mer, I'd recommend using hg38_fragLen300_kmer100 hdf files with correctBias_stored subcommand. We haven't tested how much the fit of the model is sensitive to different parameters such as kmer length, but I believe hg38_fragLen300_kmer100 will still provide pretty reasonable fit. Another option is not correcting mappability bias. You can choose the type of biases you want to correct with -biasType parameter in correctBias_stored subcommand.
In terms of runnning time, correctBias subcommand takes longer time than correctBias_stored subcommand as written in README file. However, the running time of correctBias subcommand is still reasonable unless the data is not whole genome.
Let me know if you have any more question!
Thanks,
Young-Sook

Thank you very much for your answer.
I will try to analyze my data according to your suggestions.


[Core-Test] Revisit redaction strategy

In Azure Core Test, RecordingRedactor class.  JSON_PROPERTIES_TO_REDACT pattern, we redacted any value if the keys matched in
"authHeader"
"accountKey"
"accessToken"
"accountName"
"applicationId"
"apiKey"
"connectionString"
**"url"**
"host"
"password"
"userName"

If we have some json file,
"url":"offset" The offset will be redacted in the file, for example, the REDACTED in the json string should be "offset".
\"entities\":[{\"REDACTED\":21

We probably need to revisit the redaction strategy and open to discussion.

We have the new azure core test proxy now.


Project settings window not refreshed after an editor plugin adds/removes project setting(s)

Godot version
v4.1.1.stable.official [bd6af8e0e]
System information
Godot v4.1.1.stable - Windows 10.0.22621 - Vulkan (Forward+) - dedicated AMD Radeon RX 6700 XT (Advanced Micro Devices, Inc.; 27.20.21034.37) - AMD Ryzen 7 5700X 8-Core Processor (16 Threads)
Issue description
When a project's settings are added or removed by an editor plugin, the changes are not visible until the project settings window is closed and reopened. I initially did not think of this as a bug, but it appears to actually be one, according to https://github.com/godotengine/godot-proposals/issues/7525#issuecomment-1685008504
Steps to reproduce

Have an editor plugin add or remove project setting(s)
Check the "General" tab
No changes
Reopen the project settings window
Changes are now visible

The MRP I have attached includes a small plugin to demonstrate this bug when enabling and disabling the plugin. Take a look at the script for more details.
Minimal reproduction project
Project settings problem.zip

I have prepared some changes: https://github.com/godotengine/godot/commit/09c1fe3ad4378d49fb5e492d39d2a8b46faee424
After clicking on a tab window (General or Input Map: #25865) an update is forced. Will that be enough or should changes also be monitored?
https://github.com/godotengine/godot/assets/41921395/437218a2-424c-46eb-89f4-42594838e252
my test plugin.gd file:
@tool
extends EditorPlugin

var counter = 0

func _process(delta):
	counter += 1
	ProjectSettings.set_setting("application/new_category/counter", counter)

func delete_setting(name):
	ProjectSettings.set_setting(name, null)

func init_setting(name, value):
	if not ProjectSettings.has_setting(name):
		ProjectSettings.set_setting(name, value)
		ProjectSettings.set_initial_value(name, value)
		ProjectSettings.set_as_basic(name, true)

func _enter_tree() -> void:
	# Tab 'General'
	init_setting("application/config/test", "test")
	init_setting("application/new_category/test", "test")
	init_setting("application/new_category/counter", 0)

	# Tab 'Input Map'
	ProjectSettings.set_setting("input/my_input", {"deadzone": 0.5, "events": []})

func _disable_plugin() -> void:
	# Tab 'General'
	delete_setting("application/config/test")
	delete_setting("application/new_category/test")
	delete_setting("application/new_category/counter")

	# Tab 'Input Map'
	delete_setting("input/my_input")

This solves the immediate problem, but I am not sure if there are other stuff that also needs forced updates. @Calinou?

Hi, this is still happening in 4.2 it's not a big deal people just have to close and open again the project settings, but it would be way smoother for people to use add ons without this issue.

Hi, this is still happening in 4.2 it's not a big deal people just have to close and open again the project settings,

Yeah, the fix got axed hard. I feel sorry for you.


feat: add experimental conformance support

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #

thanks for working on this @Xunzhuo !
do you think we should manually run this, run this in CI on every merge and copy into the latest release notes, run this before every semvar release ?
and how often should we publish this data upstream in the gateway api repo

        apiVersion: gateway.networking.k8s.io/v1alpha1
        date: "2023-08-10T03:35:39Z"
        gatewayAPIVersion: v0.8.0-rc1
        implementation:
          contact:
          - https://github.com/envoyproxy/gateway/blob/main/GOVERNANCE.md
          organization: envoyproxy
          project: envoy-gateway
          url: https://github.com/envoyproxy/gateway
          version: latest
        kind: ConformanceReport
        profiles:
        - core:
            result: success
            statistics:
              Failed: 0
              Passed: 29
              Skipped: 0
            summary: ""
          extended:
            result: success
            statistics:
              Failed: 0
              Passed: 9
              Skipped: 0
            summary: ""
            supportedFeatures:
            - HTTPRouteSchemeRedirect
            - HTTPRoutePathRewrite
            - HTTPRouteQueryParamMatching
            - HTTPResponseHeaderModification
            - HTTPRoutePortRedirect
            - HTTPRoutePathRedirect
            - HTTPRouteHostRewrite
            - HTTPRouteRequestMirror
            - HTTPRouteMethodMatching
          name: HTTP
        - core:
            result: success
            statistics:
              Failed: 0
              Passed: 11
              Skipped: 0
            summary: ""
          name: TLS

thanks for working on this @Xunzhuo ! do you think we should manually run this, run this in CI on every merge and copy into the latest release notes, run this before every semvar release ?

I think experimental conformance can run in main CI, and having conformance report in every commit.

and how often should we publish this data upstream in the gateway api repo

I think each time we bump gwapi version ?

should we only run & publish this every time we import a new gateway api version ?
so we can add a new GHA that checks if charts/gateway-helm/crds/gatewayapi-crds.yaml path has changed and if so it runs this specific make target as well as publishes the Conformance Profile as a GH comment on the PR 😎 , which we can then copy over to upstream

should we only run & publish this every time we import a new gateway api version ? so we can add a new GHA that checks if charts/gateway-helm/crds/gatewayapi-crds.yaml path has changed and if so it runs this specific make target as well as publishes the Conformance Profile as a GH comment on the PR 😎 , which we can then copy over to upstream

Sounds good : )

I tested and made changes to gwapi.yaml, https://github.com/envoyproxy/gateway/actions/runs/6117087341?pr=1780, it worked normally.


:boom: Update saddle-points implementation based on feedback

Also sets status = "beta" for these two recently added exercises as they may change further

Thanks!


Set the inventory key binding to letter "I" and set Inventory button to be unfocusable

Hi, @ASteinheiser. Forwarding my usability concerns from Reddit:
https://www.reddit.com/r/reactjs/comments/aylp2p/i_wrote_a_post_on_making_an_rpg_with_react_redux/ei4lcjt
On desktop, when I want to read my Inventory, I use the mouse & click on the button, then I have to close it with the same controls, but the Inventory button does not lose focus. Here, [Space] and [Enter] are used as attack action keys, but by browser design, they also mean a "click" user action on <button> elements, so, when I close the Inventory, then attack a monster on my next move, the Inventory comes up. It's really annoying.
There really needs to be something to tackle this:

Set the Inventory button's tabIndex to -1.
https://github.com/ASteinheiser/react-rpg.com/blob/master/src/features/inventory/index.js#L54

class Inventory extends Component {
  // ... other component class methods

  render() {
    const { newItemIndicator } = this.state;
    const { disabled, dialog, sideMenu } = this.props;
    const { inventory: open } = dialog;

    return (
      <div className='flex-row inventory__container'>
        {
          !disabled &&
            <Button
              ...otherButtonProps
              tabIndex={-1} // unfocusable button
            />
        }
      </div>
    );
  }
}

Set the Inventory key binding to [I],
https://github.com/ASteinheiser/react-rpg.com/blob/master/src/features/player/controls.js#L120

// ... other module imports

import isGamePaused    from '../dialog-manager/actions/is-game-paused';
import toggleInventory from '../dialog-manager/actions/toggle-inventory';
// inventory visibility is controlled by the global app state; good for us!

import { ANIMATION_SPEED } from '../../config/constants';

const Controls = ({ isGamePaused, attackMonster, movePlayer }) => {
  // ... other component constants & functions

  function handleKeyDown(event) {
    event.preventDefault();
    // move with 'WASD' or Arrow keys
    switch(event.keyCode) {
      case 37:
      case 65:
        return movePlayer('WEST');

      // ...
      // ... other `event.keyCode` case handlers
      // ...

      case 73:
        return toggleInventory(); // "I" => show / hide the inventory!

      default:
        // some console.log comment I couldn't be bothered to copy! :)
    }
  }

  return null;
}

const actions = {
  attackMonster,
  movePlayer,
  toggleInventory, // controls should be able to modify Inventory visibility, too!
  isGamePaused,
};

export default connect(null, actions)(Controls);

I'm sure that this will open up more questions from the audience about key binding usability, but I think it's a reasonable place to start.

I didn't create a pull request to staging yet because of Contribution step 2: Discussion First and since I've never made community pull requests before, I'm very careful about what I'm doing.
In the meantime, @ASteinheiser, you can check out the changes I did on my fork:
https://github.com/ASteinheiser/react-rpg.com/compare/staging...KajiTetsushi:master

You can go ahead and open a pull request. My intention with that second step is mainly so the community has a chance to discuss on Github before we start merging PRs.
I think this is a good change, and I imagine if we have a Stats dialog in the future, there could be a keybinding for that as well.

@ASteinheiser, PR's up.
I feel there's an even better way to do what I just did 3 days ago. Check out my PR #55. Basically, inventory toggling keydown is handled inside the inventory button rather than through the player controls component, which is already quite busy.
The idea is that the inventory will deal with its own key bindings rather than have some other component listen for it, so, if you decide to e.g. deprecate the button, you can do so without having to jump over to the player <Controls /> component. The same mentality could be applied to your Stats dialog idea.

@KajiTetsushi I understand the Controls component is getting busy, however I still want to have all the controls in one place. Maybe we just need to break out the specific controls into functions to declutter it?
I can see your point on not wanting to have to remove the keybinding from Controls if Inventory is removed, however I don't believe that will happen. I really just want to be able to go to one place for all the player's controls.
Ideally, the Controls should also have the key listeners for closing dialogs and chests as well, since the player attack/move is "disabled" at that point anyways. This way, we wouldn't need to "disable" the player controls, since the Controls component would dispatch a different action than "attack" or "move", for example.

So... are we still doing this?
I'd like to see the inventory get bound to "I" or "B", the only issue I had with the PR was that it made the button unable to be targeted by various screen-readers, etc. Perhaps we can simply unselect it when the inventory gets closed, so that Enter and Spacebar would never trigger it accidentally?

@Alaricus Good suggestion. I think that would be the best of both worlds.
I would prefer we bind the key to "I", then on the inventory button, change the text from "INVENTORY" -> "[I]NVENTORY" to indicate that you can press I.

@ASteinheiser, another question then. Personally I don't use (or test for) the mobile apps, but it appears that the experiences would, at some point, become divergent out of necessity. For instance there is little reason to have the [I] bit show up on a mobile app. Same for the onHover events we discussed earlier. How do you want to go about it? Should there be different repos? Should we fight the divergence and create a unified experience? What's your vision for the long term?

Really sorry about the silence, guys. I've just changed jobs recently, trying to adapt to the new work environment and all...

@Alaricus: you assumed that I would've gone with my first suggested solution (disabling button focus, because yes, that was my initial recommendation), to which I say: "no", I eventually didn't. I saw it coming, too. Thus, I left the button focus event untouched. If you see this here commit: a7dd028, I've left the button alone. But I've seen you reintroduce my approach in #60, with the extra UI bits in it, so yeah, it's all you now. 👍

I really just want to be able to go to one place for all the player's controls.

@ASteinheiser: I understand.
That being said, my first approach was, indeed, to place that keybinding in the controls.js file (see here: a7dd028). I switched to the cf9da09 approach because in my previous experience, putting stuff together in one monolith of a file lead into all kinds of coupling problems. Perhaps that's because the software is big. Perhaps this project is not the case.
However, if this is no concern to you, then, by all means, I've already did it: a7dd028 is my take on solution. I believe @Alaricus has gone with PR #60, which does the same thing (putting the action in controls.js), plus a few bells & whistles on the visuals which I haven't done. I'd gladly defer to that PR.

@KajiTetsushi, yes, you are absolutely right, I did assume! Sorry about that! =)
By the way, I have absolutely no problem deleting my PR if you want to update yours. The only reason I submitted it, is because I saw you being gone for several days and didn't want this issue remain unaddressed. That said, since you are back and this is your idea in the first place, I'll happily step back.
Let me know!

@Alaricus I definitely don't think that we should have separate repos for the mobile and desktop versions. I honestly think the amount of divergence should be small enough that we can deal with it on a case by case basis.
For example, the game instructions dialog check if it's mobile or desktop to show different controls. Does this sound reasonable to you?
@KajiTetsushi No worries, I'm about to start a new gig next week too, so I will be a little slower soon!
I can see what you're saying about the controls, but I don't envision too many controls needing to be bound.
So, between you guys, should I just accept @Alaricus PR or do you want to submit yours @KajiTetsushi ?

@Alaricus that's all right, buddy. no hard feelings.

Yeah, I can take point again @ASteinheiser @Alaricus, since I was the one who asked for it. I might as well finish what I've started. I'll be available to create a PR again in about 6~8 hours once I leave work. You guys cool with that?

I'm about to go to sleep, so I've no problem with this. 😃 In fact, if you want to, feel free to just copy/paste my PR and add whatever you were going to.

Alright, I will wait for @KajiTetsushi to make a PR, then I'll close @Alaricus PR and merge the new PR in. Thanks guys!


adding hannahguo

To add your site to the webring, you must be a current student or an alum of the undergraduate Software Engineering program at the University of Waterloo (Waterloo, ON, Canada).
Full Name: Hannah Guo
Cohort Year: 2025
Full website URL: https://hannahguo.me/
So we know you're human:
LinkedIn or a similar profile: https://www.linkedin.com/in/hannah-guo/
Don't forget to also add your entry to the end of the allSites array in the se-webring.js file (name, year, personal website), and to add the add site label to your pull request. Assign as a reviewer either @simcard0000 or @janakitti.

Doesn't look like I can add labels/assign reviewers to the PR, lmk if there need to be any changes :) @simcard0000 @janakitti

Looks all good to me! Thanks for submitting and don't forget to tell other people about this :)


Warning on _prescalerIndex not initialized

eclipse (sloeber 4.4.0) shows two warnings:
Member '_prescalerIndex' was not initialized in this constructor

megaAVR_TimerInterrupt.hpp

line 126

Code Analysis Problem

Same in line 136
Version used 1.6.0 on Arduino Nano Every. Platform version 1.8.7.
Looks to me that _prescalerIndex is defined but never used. So I assume that this is no real problem.

Hi @pischky
Done in megaAVR_TimerInterrupt releases v1.6.1.
Thanks,

Release v1.6.1

Suppress warnings when _TIMERINTERRUPT_LOGLEVEL_ < 2
Remove unused _prescalerIndex


DB_query_builder: Adding having_in(), or_having_in(), having_not_in() and or_having_not_in()

For generating SQL query such as
SELECT country, count(1) FROM user GROUP BY country HAVING count(1) IN(2, 3) OR country NOT IN('US', 'UK')

Still too much indentation, but I guess that's because the rest of that code gives a bad example ... I'll fix that myself later.


Failed to resolve: org.kodein.di:kodein-di-framework-android-core:5.2.0

same problem with another versions of kodein-di-framework-android-core. org.kodein.di:kodein-di-generic-jvm:5.2.0 resolved well

As a quick fix you can add to your repositories:
maven { url "https://dl.bintray.com/kodein-framework/Kodein-DI" }

https://dl.bintray.com/kodein-framework/Kodein-DI/org/kodein/di/kodein-di-framework-android/
contains max 5.1.1, but 5.2.0 is missing

@089apps that's doesn't matter because 5.1.1 also can't be resolved

@mateherber your fix didn't work for me

@089apps you're using the wrong package
it should be kodein-di-framework-android-core
@ofdrykkja
It's there, you sure you added to the right place?: https://dl.bintray.com/kodein-framework/Kodein-DI/org/kodein/di/kodein-di-framework-android-core/

i had tried to add maven { url "https://dl.bintray.com/kodein-framework/Kodein-DI" } into the buildscript { repositories { and into allprojects { repositories of project build.gradle

@ofdrykkja For me the allprojects one worked :( :

This component do not exist in Kodein 5.2.0 and has been replaced by either:

org.kodein.di:kodein-di-framework-android-core:5.2.0 (for only core Android)
org.kodein.di:kodein-di-framework-android-support:5.2.0 (For core Android and Support library)
org.kodein.di:kodein-di-framework-android-x:5.2.0 (For core Android and AndroidX library)

I have the following lines in my build.gradle files:
Project:
allprojects {
    repositories {
        google()
        jcenter()
        maven {url "https://jitpack.io"}
    }
}

Module:
implementation 'org.kodein.di:kodein-di-generic-jvm:5.2.0'
implementation 'org.kodein.di:kodein-di-framework-android-support:5.2.0'

All I get is this:
Failed to resolve: org.kodein.di:kodein-di-framework-android-core:5.2.0
So should I add the exact download link to use the library? Why isn't it available in the major repositories?

@nologinatgit, it writes that org.kodein.di:kodein-di-framework-android-support:5.2.0 is not resolved, (also 5.1.1). I added maven { url "https://dl.bintray.com/kodein-framework/Kodein-DI" } and it synced.

Same issue, org.kodein.di:kodein-di-framework-android-core:5.2.0 doesn't resolve properly. It only does after adding maven { url "https://dl.bintray.com/kodein-framework/Kodein-DI" } to the repositories of the project. Not that it's a big deal, but pretty sure people will start opening issues 🤷♂

@089apps you're using the wrong package
it should be kodein-di-framework-android-core
@ofdrykkja
It's there, you sure you added to the right place?: https://dl.bintray.com/kodein-framework/Kodein-DI/org/kodein/di/kodein-di-framework-android-core/

This is work for me
for support androidx
implementation 'org.kodein.di:kodein-di-generic-jvm:6.5.0'
implementation 'org.kodein.di:kodein-di-framework-android-x:5.2.0'
and without Androidx
implementation 'org.kodein.di:kodein-di-generic-jvm:6.5.0'
implementation 'org.kodein.di:kodein-di-framework-android:5.1.1'


Would be great if the engine had a FlutterFragment

For hybrid apps you might want to run flutter via a fragment. Not sure if this is possible.

see #10072

For the record, #10072 is the reverse of the desired functionality. Instead of an Fragment->Activity adapter, we'd want a Fragment to be used as a Fragment by its parent.

This is not something we're likely to work on in the comings months but patches are certainly welcome.
cc @mravn-google

I count this as the fundamental that's necessary so flutter can take off.
Large apps will never start using flutter because you can't just start with one feature and expand progressively. Everything would have to be rewritten at once.

This Fragment now exists within the Android embedding refactor.  That refactor lives in a staging branch and will be merged upon completion.  The overall project for the refactor is here:
https://github.com/flutter/flutter/projects/27
Do not use any part of the refactor until it is merged to master.


Fix -nologo indentation

PR Summary
fixes #13516
The pwsh -help did not display the -nologo option with a correct indentation

PR Context
Opened pull request to fix a very simple issue of 4 missing spaces

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[ ] Summarized changes
[ ] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[ ] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

What is next milestone label?


Add Mac_x64 darwin_lint_podspecs to LUCI

This is the first Mac target to be enabled in LUCI, and should be landed after recipes support CL: https://flutter-review.googlesource.com/c/recipes/+/35303
Bug: https://github.com/flutter/flutter/issues/114373

A friendly ping.

LGTM
(Sorry for the delay, I was out sick for the last couple of days.)

No worries at all. Thanks for the review!


Namespace problem

This code compiles ok:
namespace Test.MyNamespace

module MyModule =

    type MyType =
        | SomeInt of int
        | SomeString of string
        | None

    type MyClass() =

        let letValue = 3
        member this.memberValue = 4

        member this.SomeMethod1() =
            printfn "%d" letValue

        member this.SomeMethod2() =
            printfn "%d" this.memberValue

and so this one:
namespace Test.MyNamespace

    type MyClass() =

        let letValue = 3
        member this.memberValue = 4

        member this.SomeMethod1() =
            printfn "%d" letValue

        member this.SomeMethod2() =
            printfn "%d" this.memberValue

However, this one
namespace Test.MyNamespace

//module MyModule =

    type MyType =
        | SomeInt of int
        | SomeString of string
        | None

    type MyClass() =

        let letValue = 3
        member this.memberValue = 4

        member this.SomeMethod1() =
            printfn "%d" letValue

        member this.SomeMethod2() =
            printfn "%d" this.memberValue

fails to compile with error:
Cannot find namespace Test.MyNamespace.MyClass
The error message is a bit misleading, as the real problem is with MyType. When reaching it, getRootDecls is being called with rootNs = "Test.MyNamespace.MyClass" (while the correct should be rootNs = "Test.MyNamespace").
When using a module (first example) the problem doesn't occurs. In this case getRootDecls is called with the correct namespace (rootNs = "Test.MyNamespace.MyModule") when it reaches MyType.

Silly mistake, I should have checked it better. Many thanks for the report!


Text commands dont stack right

This is in an attempt to resolve: https://github.com/Khan/KaTeX/issues/676
This should theoretically allow for the stacking of font commands, if the font is available. Note: It currently does not work for \textbf{\textit{x}} because there is no "Main-BoldItalic" metrics available.
However, I've included the SansSerif-Bold/SansSerif-Italic to illustrate that \textsf{\textbf{x}} and \textsf{\textit{x}} will render a SansSerif bold/italic text respectively.

Closing until tests are fixed, and cleaned up.

@kevinbarabash I ran the screenshots locally, and while they are technically failing, I think the new screenshots are correct.
That being said, would you mind taking a look? I am not very fluent in Latex, and I'd like someone with more experience to be able to verify.

@rrandallcainc the screenshots could probably be collapsed into a single screenshot using {aligned}.  I was thinking of doing this for the font screenshots for MathBf, MathCal, MathFrak, etc.  I'll review the code this weekend.

@kevinbarabash Good idea! I'm going to clean this up. I don't like how I approached some of this logic, and I think it could be improved. That being said, I expect it will be ready for this weekend. I'll remove the "needs-revision" label once it's ready.

@rrandallcainc are you still planning to clean up some of the logic?

@kevinbarabash Yes. I have been crazy busy lately. I'll be returning from a trip later this week. I would like to dedicate more time once I return. Thanks for your patience.


Linking with Lightning State

Blocked on https://github.com/Lightning-AI/LAI-CodeRunner-App/issues/18

Everything works! 🎉 Let's get this in.


Support Bitmap Rendering

tvg::Picture should support Bitmap Rendering
auto picture = tvg::Picture::gen();
picture->load(buffer);
...

https://github.com/Samsung/thorvg/pull/122

Sequence Diagram Update
(condition : Data load, opacity < 255, with composite)


Upgrade Spring Boot + Spring Security

Task Description
We need to upgrade to the latest versions of Spring Boot and Spring Security.
Tasks
The following tasks will need to be carried out:

[ ] Make org.carlspring.strongbox.security.authentication.StrongboxAuthenticationFilter extend the org.springframework.security.web.authentication.AuthenticationFilter
[ ] Rework AuthenticationSupplier implementations to AuthenticationConverter implementations
[ ] Rework AuthenticationSuppliers implementation to DelegatingAuthenticationConverter (it should delegate to first AuthenticationConverter that will return not null Authentication object)
[ ] Check that new StrongboxAuthenticationFilter implementation consistent with logic that we currently have
[ ] Provide unit tests that check different authentication scenarios with few mocked AuthenticationConverter implementations

Task Relationships
This task:

Relates to: spring-projects/spring-security/pull/7025

Help

Our chat
Points of contact:

@carlspring
@sbespalov
@steve-todorov

Spring Security: v 5.2.0.RELEASE

@sbespalov ,
Do you know what things will need to be refactored, once we've upgraded to the newest version of Spring Security? Could you add a list to the task, please? Thanks!

@carlspring tasks updated

Hi,
I'm new to contributing and would love to work on this refactoring task. Thanks!

@soapdogg (and anyone else for this matter),
Please, only pick this task, if you have experience with Spring Boot and Spring Security.
I would also like to invite you to join our chat channel where you can find more details regarding what other tasks you might be able to help with.


Feat/mutation test

Codecov Report

Merging #22 (364c384) into develop (8dd5c2b) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##           develop       #22   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           19        24    +5
  Lines          267       274    +7
  Branches        37        26   -11
=========================================
+ Hits           267       274    +7

Impacted Files
Coverage Δ

src/client/util/clientXMLRPCHelper.ts
100.00% <ø> (ø)

src/client/util/adapters/buildAdapter.ts
100.00% <100.00%> (ø)

src/client/util/adapters/testCaseAdapter.ts
100.00% <100.00%> (ø)

src/client/util/adapters/testPlanAdapter.ts
100.00% <100.00%> (ø)

src/client/util/adapters/testProjectAdapter.ts
100.00% <100.00%> (ø)

src/client/util/adapters/testSuiteAdapter.ts
100.00% <100.00%> (ø)

src/schemas/Build.ts
100.00% <100.00%> (ø)

src/schemas/TestCase.ts
100.00% <100.00%> (ø)

src/schemas/TestPlan.ts
100.00% <100.00%> (ø)

src/schemas/TestProject.ts
100.00% <100.00%> (ø)

... and 6 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 8dd5c2b...364c384. Read the comment docs.


fix(backup,plugin): keep plugin connection open while running a backup

This patch keeps the plugin connection pool open when a backup is running. Before, it was closed as soon as the backup request was correctly received.
Release notes
The plugin connection is always kept open when a plugin is executing a backup.

E2e: https://github.com/EnterpriseDB/cloudnative-pg/actions/runs/11179010397

/ok-to-merge

Version 1.22 does not have the plugin code, so this PR should not have been marked to be backported there.


Clean up and restructure README.md

Thanks for this great package.
I looked through the entire README and did:

Fix a few typos
Reorganize Markdown Heading structure
Split long lines into multiple lines
Run prettier --tab-width=2 README.md
Remove unused links
Remove hint for Laravel 4.x

Especially the top section should be a bit easier to get into now.

Thanks!


Implemented matrix/Invert()

Closes #329
The test is currently failing because it looks like numbers in OpenDream are more precise than BYOND's.
Inverting a matrix(1,2,3,4,5,6) in BYOND results in:
-1.66667 0.666667 1 1.33333 -0.333333 -2
while in OpenDream it results in:
-1.6666667 0.6666667 1 1.3333334 -0.33333334 -2
Do I change the test to expect the output we get in OpenDream currently?
Do we want to mimic BYOND numbers exactly?

Do I change the test to expect the output we get in OpenDream currently?
Do we want to mimic BYOND numbers exactly?

Have the test expect OpenDream's output. I wouldn't expect a slightly higher precision to cause any problems so there's no need to exactly mimic byond here.

There's a problem: Inverting the matrix twice doesn't return the original matrix. Starting with matrix(1,2,3,4,5,6), the test inverts it and then inverts it again. The result is: 0.99999994 1.9999999 2.9999998 3.9999998 5 5.9999995, which is not quite the original. I'm not very well-versed in floating-point magic so I'm not sure how to proceed. Is it because the operations I use inside Invert introduce too many errors?

There's a problem: Inverting the matrix twice doesn't return the original matrix. Starting with matrix(1,2,3,4,5,6), the test inverts it and then inverts it again. The result is: 0.99999994 1.9999999 2.9999998 3.9999998 5 5.9999995, which is not quite the original. I'm not very well-versed in floating-point magic so I'm not sure how to proceed. Is it because the operations I use inside Invert introduce too many errors?

Honestly? There is not much else you can do about this. I think this might be the reason why Byond has less precision than usual, to make those irrational .9999999x's round up to integers.
And because the error is only less than 10^-6, I think it is negligible at best. But if it is going to be a problem. How about we add a Round proc to matrix for the intents and purposes of rounding the elements to integers to a desired precision?

I feel like we'd have to round everything if we do that, not just matrix operations, so that everything is consistent with each other and BYOND. Right?

The reason that the invert test is failing is due to a difference in how floats are handled in OpenDream. This issue would be best handled in another PR, could the test be left out for now?

I've commented out the test.


"ERROR: Unable to download f4m manifest" on a previously working website

This is the issue when I try to -F the webpage : 'http://www.france5.fr/emissions/c-dans-l-air/videos/epargne_04-08-2015_851144?onglet=replay&page=1'
It has worked on this website (or http://pluzz.francetv.fr/videos/c_dans_lair_,125915888.html), but it's not working anymore on both sites (pluzz.francetv.fr & france5.fr).
Version : Version : 2<IP_ADDRESS>-1 (Debian Stretch)
youtube-dl -F works on Youtube
$ youtube-dl --verbose -F 'http://www.france5.fr/emissions/c-dans-l-air/videos/epargne_04-08-2015_851144?onglet=replay&page=1'
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--verbose', u'-F', u'http://www.france5.fr/emissions/c-dans-l-air/videos/epargne_04-08-2015_851144?onglet=replay&page=1']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2<IP_ADDRESS>
[debug] Python version 2.7.10 - Linux-3.16.0-4-amd64-x86_64-with-debian-stretch-sid
[debug] exe versions: avconv 2.6.1, avprobe 2.6.1, ffmpeg 2.6.1, ffprobe 2.6.1, rtmpdump 2.4
[debug] Proxy map: {}
[francetv] epargne_04-08-2015_851144: Downloading webpage
[francetv] 125915888: Downloading video JSON
[francetv] 125915888: Downloading f4m manifest token
[francetv] 125915888: Downloading f4m manifest
ERROR: Unable to download f4m manifest: HTTP Error 404: Not Found (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 312, in _request_webpage
return self._downloader.urlopen(url_or_request)
File "/usr/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 1730, in urlopen
return self._opener.open(req, timeout=self._socket_timeout)
File "/usr/lib/python2.7/urllib2.py", line 437, in open
response = meth(req, response)
File "/usr/lib/python2.7/urllib2.py", line 550, in http_response
'http', request, response, code, msg, hdrs)
File "/usr/lib/python2.7/urllib2.py", line 475, in error
return self._call_chain(*args)
File "/usr/lib/python2.7/urllib2.py", line 409, in _call_chain
result = func(*args)
File "/usr/lib/python2.7/urllib2.py", line 558, in http_error_default
raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)

You are using an outdated version of youtube-dl. Read our FAQ if you have troubles updating.

Thank you for your answer.
I will have to consider installing youtube-dl aside from the dpkg system.
Le 08/08/2015 10:59, Sergey M. a écrit :

You are using an outdated version of youtube-dl. Read our FAQ
https://yt-dl.org/update if you have troubles updating.
—
Reply to this email directly or view it on GitHub
https://github.com/rg3/youtube-dl/issues/6496#issuecomment-128940854.


Path dataset and traing / validation

Hi BXuan
I have downloaded the caltech256 dataset but not found dataset4u-trn.txt / dataset4u-val.txt. I guess you created it but I did not found it in your github. I do not know how it looks like but I think it should be similar to dataset.txt file in caltech256 folder after unzipping.
I tried to use this file for running your code, however, unfortunatly, it did not work since "words" array has only one dimension so words[1] is out of range.
Can you take a loot at my comment and show me how to read img from the dataset Caltech 256?
Thanks

Hi BXuan
I am not sure whether dataset4u-trn.txt / dataset4u-val.txt keep image data under a certain format or just keep the folder name (or label).
When I read your code, the way of reading these files in "generate.py" and "targetmodel.py" (Mydataset) is similar. But then the array resulting from reading file (.txt)  is different.
In "generate.py"
cur_img = Image.open(img_trn[k][0]).convert('RGB')
While in "targetmodel.py"
fn, label = self.imgs[index]
Can you share dataset4u-trn.txt / dataset4u-val.txt ?
Thanks


Wp 114 postgres start gabriel

The first commit failed on GitHub Actions because the POSTGRES_PASSWORD variable was defined as an environment variable, which GitHub Actions could not access.
The second commit fixes that by setting POSTGRES_PASSWORD = postgres.


Add cities dataset to latest datasets download list

Fix #78

def TestLatest(TestCase):

    def test_cities_was_added(self):
        self.assertIn('2017-05-22-brazilian-cities.csv', Datasets().downloader.LATEST)

Or even:
    def test_latest(self):
        self.assertEqual(28, len(Datasets().downloader.LATEST))

But the real question is would it worth it?

I don't think that this kind of year worth.
What can be done to protect us from the future ourselves ?
May it should break if any of this files are removed from this list?

... this kind of test* ...

May it should break if any of this files are removed from this list?

I don't think so, because we remove old versions when there are new ones available… although we wold never see the len() of the list get smaller, because we would either replace a file (adding a newer version) or add a new dataset (which was not present in the list before).

Why is this constant important for toolbox?
The LATEST constant is not used anywhere in this file, if I remove it all project tests will pass smoothly.
Is it OK?

It's used here, in a function documented here.
Maybe the test you'd like to enhance is this one.

There is any point against enhance this tests?

Is there any point against enhance this tests?

Not at all ; )


[now dev] GitHub dependencies not supported

With https://github.com/zeit/now-builders/issues/49 we can use GitHub dependencies in now v2 deployments, but they don't currently work with the now dev command.

Hey @denkristoffer. What kind of error do you see? Would you be able to create a GitHub repo demonstrating the issue? Thanks in advance!

My apologies @TooTallNate, I didn't think of this as a bug report, I thought you simply hadn't gotten around to supporting it for now dev yet 🙂 I'll see if I can put together a reproduction.

Closing because I just tested this with the latest Now CLI and it works with now dev as expected. Demonstration repo is here: https://github.com/TooTallNate/now-cli-2381


Auth Microservice POC

User Story
We need a proof-of-concept Auth microservice to determine if our approach is feasible.
Description/Use Case
Have a locally working microservice that can validate an access token against staging okta, forward you along to the submissions microservice, and authorize a scope.
Dev Notes
Use Spring!
Acceptance Criteria

microservice able to be run locally
Rest client calls out to staging Okta
forwards request to submissions project (locally)
submissions project authorizes scope

Hey team! Please add your planning poker estimate with Zenhub @adegolier @arnejduranovic @brick-green @david-navapbc @jack-h-wang @jalbinson @JFisk42 @mkalish @thetaurean


Manually update of color input value not update model

Hi,
when i select the color with the color-picker i have no problems but if i write the color code manually in the input generated with the color picker, the model does not update.
The model contain the old color selected with the color picker.
How i can resolve this problem?
Thanks, bye.

@topperblues I was running into this as well, so I created a pull request with a fix for this issue. Feel free to check it out.


Add to Starlette "Third Party Packages" docs?

Great work! I guess it'd make sense to add Flama to the "Frameworks" section of https://www.starlette.io/third-party-packages/ right?

Yup! This project is a reworking from Starlette API that is listed in that page but I forgot to update it 😅
I'll make a PR to update the docs, thanks for reminder!

Done :)


No example file

The "Verify body tracking" part failed with:
[2019-06-27 12:19:57.576] [error] [t=1660] D:\a\1\s\extern\Azure-Kinect-Sensor-SDK\src\usbcommand\usbcommand.c (305): find_libusb_device(). Unable to open LIBUSB at index 0
[2019-06-27 12:19:57.578] [error] [t=1660] D:\a\1\s\extern\Azure-Kinect-Sensor-SDK\src\depth_mcu\depth_mcu.c (68): usb_cmd_create(USB_DEVICE_DEPTH_PROCESSOR, device_index, NULL, &depthmcu->usb_cmd) returned failure in depthmcu_create()
[2019-06-27 12:19:57.578] [error] [t=1660] D:\a\1\s\extern\Azure-Kinect-Sensor-SDK\src\sdk\k4a.c (129): depthmcu_create(index, &device->depthmcu) returned failure in k4a_device_open()
Open K4A Device failed!

(File: D:\a\1\s\examples\simple_3d_viewer\main.cpp, Function: main, Line: 107)

Also I don't see any example/bin nor any example file after installation.

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

ID: 3b745a96-f55d-5924-0f29-614186393b6d
Version Independent ID: 6e583bdd-485a-98f6-599b-503d2d4f5480
Content: Quickstart - Set up Azure Kinect body tracking
Content Source: articles/kinect-dk/body-sdk-setup.md
Product: kinect-dk
GitHub Login: @qm13
Microsoft Alias: quentinm

@bastiankayser, Thanks for the question! We are taking a look into this and will get back to you soon. In the interim, kindly ensure that all the prerequisites are met as mentioned in the document and also let us know the OS version you're running.

Yes, all prerequisites installed successfully.
OS Version is
Windows 10 Pro Build 17134

@bastiankayser, Thank you for the details. I have assigned the issue to the content owner to review further.

@qm13, Requesting your comments on this issue.

@bastiankayser Since this issue has been addressed. We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


[BUG] Messages sent from DCS Scripting Engine don't appear with Vaicom enabled.

Describe the bug
A lua Script using the DCS Scripting Engine (not MIST/MOOSE) can only send messages via five functions:
trigger.action.outText()
trigger.action.outTextToUnit()
trigger.action.outTextToGroup()
trigger.action.outTextToCountry()
trigger.action.outTextToCoalition()
When a text message is sent via these, with VIACOM enabled, the message is suppressed and not displayed to the player.
To Reproduce
Either from a script or from a tigger in the mission editor run any of those commands and try to send a text message.
Notice that nothing is displayed.
Disable VIACOM and reload mission, and notice that the text is displayed.
Expected behavior
VIACOM should not surpress these types of messages that mission designers and script programmers use to add richness and additional AI communication to players in-game.
Screenshots
N/A
Additional context
N/A

Tested it, can't replicate it. If I send messages from my mission, it works with or without VAICOM enabled.

Ensure hide on-screen text is turned off under the preferences tab. If it is on please let us know so we can close this issue, if it is off then this is a bug.

I'll do some tests and report back.

Hide On-Screen Text is off and I do get text messages from the standard comms to tankers, ATC, etc.
When a script sends messages using trigger.outtexttounit() I don’t get them.

Everyone else who doesn’t have Viacom does and if I disable Viacom I get them too

*Vaicom not "Viacom"
I also cannot replicate this, I get the messages with or without Vaicom running. Would you mind sharing your Vaicom settings?

I experienced it today while playing the F-16 red flag campaign. I am not 100% sure but I think "disable menus" option caused it.

Also I can't reproduce, I have messages displayed as long as hide messages is unchecked in the Vaicom preferences.I'll leave this open in case someone else has the issue however, I believe it must be something with your system/installation.


Wait for prediction endpoint hangs for ever in the case there is no external loadbalancer

Here:
https://github.com/kubeflow/fairing/blob/cd3b0581c2a70c352ac2f66432bc2e023e3fbc7c/fairing/kubernetes/manager.py#L77
We wait for the loadbalancer to be ready; but there will never be a loadbalancer if we configured it to only use cluster ip

Seems we can waiting for same long seconds (such as 300s), and raise error message if the loadbalancer is not ready? comments?


Fix typos in compiler directory

Splitting #43857 by top-level directories.

Nice, Thanks!
LGTM, assuming it passes the tests.

@Molkree can you please resolve conflicts

@rthadur, resolved

Can you please rebase your PR? It seems some files have changed by now. Also, possibly exclude the changes to tf2tensorrt directory, because those seem to trigger additional checks which our workflow to pull in PRs cannot handle (we would have to do it manually).

@akuegel, rebase or merge? I don't see any conflicts in GitHub interface but sure, if you request it.
And fine, I'll remove that directory now.

@mihaimaruseac, I see that this was merged, should I close PR?

@Molkree Thank you for the confirmation.
Seems auto-merge is not happening but the changes are merged into master now, so we can close this PR.


Misleading INCLUDE_DEPLOYMENTS: true for CATS

Description
Deployments CATS are enabled, but since they depend on V3 tests they are skipped in CI
Steps to reproduce
Execute CATS in CI
What was expected to happen
Do not include deployments (https://github.com/cloudfoundry-incubator/eirini-ci/blob/7fd82ea15666ac2e92767cba12d76faf3ac47691/pipelines/modules/run-core-cats.yml#L35) until V3 are enabled
What actually happened
Misleading INCLUDE_DEPLOYMENTS: true param
Suggested fix (optional)
Remove  INCLUDE_DEPLOYMENTS: true from CI
Additional information (optional)
na

Fixed in https://github.com/cloudfoundry-incubator/eirini-release/commit/759c0fdfdd34235475e827c99ed2ea9f85b5563a


Feature request::Query hosts satisfying multiple groups

I'm submitting a ...

[ ] Feature requestdd

Description
Is there a way for me to query for hosts which is satisfying multiple groups instead of getting hosts of all groups. For eg:
Group1 contains host1,host2,host3
Group 2 contains host1,host4,host5
I want to query for hosts belonging to both Group1 and Group2 which is host1
I don't want to have my results as host1,host2,host3,host4,host5
This eliminates the use of zabbix nested host groups which is not really up to my expectation yet. Actually Zabbix should have implemented tags/labels for hosts instead of grouping. Large environments monitoring objects may related multiple environments, application, instances etc.

Hello. I see. So in your view you tend to believe that when querying a host that needs multiple conditions achieved, instead of doing something like querying a group named "Cloud Servers" and another one called "Windows Servers" we instead should use the "Item Type" tags, which I believe the field is called "Application"...
Yes? But what if we want to template stuff even further? Something like:
Let's say I monitor 10 clients, and all of them have multiple servers and services.
They have Linux Servers, Windows Servers, Local and Cloud Servers.
So I would a filter (Which is the case actually in my scenario) that I would do something like:

Select Client Group (Client A, B, C, etc.).
Select Service Type of the Client (Windows Server, Linux, APIs)
Select the Host (Cluster A, Cluster B, Server A, Server B, etc)

Is still the case to implement all this using tags and not host groups?


Svelete internal redirect error

Describe the bug
This is my project's route structure
└── routes
    ├── api
    │   └── v1
    │       └── login
    │           └── +server.ts
    ├── (auth)
    │   ├── app
    │   │   ├── +page.server.ts
    │   │   ├── +page.svelte
    │   │   └── [serverId]
    │   │       ├── create
    │   │       │   ├── +page.server.ts
    │   │       │   └── +page.svelte
    │   │       ├── +page.server.ts
    │   │       ├── +page.svelte
    │   │       └── [panelId]
    │   │           ├── +page.server.ts
    │   │           └── +page.svelte
    │   ├── +error.svelte
    │   ├── +layout.server.ts
    │   └── +layout.svelte
    ├── callback
    │   └── +page.server.ts
    ├── +error.svelte
    ├── +layout.svelte
    ├── login
    │   ├── +page.server.ts
    │   └── +page.svelte
    ├── logout
    │   └── +page.server.ts
    └── +page.ts

When i request /app route i get following error
$ vite dev

  VITE v5.3.4  ready in 3879 ms

  ➜  Local:   http://localhost:5173/
  ➜  Network: use --host to expose
  ➜  press h + enter to show help
node:internal/process/promises:288
            triggerUncaughtException(err, true /* fromPromise */);
            ^

[UnhandledPromiseRejection: This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). The promise rejected with the reason "#<Redirect>".] {
  code: 'ERR_UNHANDLED_REJECTION'
}

Node.js v18.19.0
error: script "dev" exited with code 1 (SIGHUP)

Reproduction
whole Code is here:
https://github.com/KnownBlackHat/Zenotype-Ticket-Bot/tree/main/webapp/src/routes
if i run curl -iL localhost:5173/app on /app: then i get above mentioned error, and following curl output
$ curl localhost:5173/app -iL
HTTP/1.1 307 Temporary Redirect
Access-Control-Allow-Origin: *
location: /login
Date: Fri, 09 Aug 2024 10:18:42 GMT
Connection: keep-alive
Keep-Alive: timeout=5
Transfer-Encoding: chunked

curl: (7) Failed to connect to localhost port 5173 after 0 ms: Couldn't connect to server

Logs
No response
System Info
System:
    OS: Linux 6.5 Kali GNU/Linux Rolling 2023.4
    CPU: (4) x64 Intel(R) Core(TM) i3-4005U CPU @ 1.70GHz
    Memory: 667.30 MB / 3.73 GB
    Container: Yes
    Shell: 5.9 - /usr/bin/zsh
  Binaries:
    Node: 18.20.1 - /usr/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 9.2.0 - /usr/bin/npm
    bun: 1.0.0 - ~/.bun/bin/bun
  Browsers:
    Brave Browser: nightly
    Chrome: 120.0.6099.129
  npmPackages:
    @sveltejs/adapter-auto: ^3.0.0 => 3.2.2
    @sveltejs/kit: ^2.0.0 => 2.5.20
    @sveltejs/vite-plugin-svelte: ^3.0.0 => 3.1.1
    svelte: ^4.2.7 => 4.2.18
    vite: ^5.0.3 => 5.4.0

Severity
blocking all usage of SvelteKit
Additional Information
No response

You're not awaiting userController.getFinalGuild() and so, as the error message indicates, the promise rejection caused by the thrown Redirect cannot be handled.

You're not awaiting userController.getFinalGuild() and so, as the error message indicates, the promise rejection caused by the thrown Redirect cannot be handled.

i didn't wanted to await on server as it would take time so i passed on that promise to client, how do i fix it without resolving that promise on server?


Disable wifi only tests for non-wifi-only network type

This change is

PR is added to the queue for testing as 1. task. (ceb91b2)

Test (Success) 90350722 build is completed (ceb91b2)
See https://github.com/ThaliTester/TestResults/tree/90350722ceb91b2_Disable_wifi_only_tests_for_non-wifi-only_network_type_chapko/ for the logs

Test 90350722ceb91b2(ceb91b2) has failed
See https://github.com/ThaliTester/TestResults/tree/90350722ceb91b2_Disable_wifi_only_tests_for_non-wifi-only_network_type_chapko/ for the fail logs

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

PR is added to the queue for testing as 1. task. (2a91229)

Well, after merging with master this PR doesn't contain any changes :)

Test (Success) 90350722 build is completed (2a91229)
See https://github.com/ThaliTester/TestResults/tree/903507222a91229_Disable_wifi_only_tests_for_non-wifi-only_network_type_chapko/ for the logs

Test 903507222a91229(2a91229) has failed
See https://github.com/ThaliTester/TestResults/tree/903507222a91229_Disable_wifi_only_tests_for_non-wifi-only_network_type_chapko/ for the fail logs


[BUG] - Accordion prevents  or  children from working correctly

NextUI Version
2.1.12
Describe the bug
When using <Accordion> and <AccordionItem>, if the nested children are of <Input /> or <Textarea /> type, their input commands are not registered. For example, you are unable to use the "Spacebar" key and most punctuations.
Your Example Website or App
https://codesandbox.io/s/morning-firefly-yscnvz?file=/src/App.js
Steps to Reproduce the Bug or Issue

Use the base accordion example
Add a text area using the Textarea example
Observe that punctuations, spacebar and at times regularly input is not registered correctly

Expected behavior
As a user, I expect input and textarea to function regularly and as expected when inside the accordion.
Screenshots or Videos
No response
Operating System Version
macOS
Browser
Opera

Fixed, install the latest version v2.1.13

It seems we are not able to use Arrows Keys (Up and Down) when using TextArea inside an Accordion.
I forked @samosaboy sandbox and upgraded to lastest NextUI version.
sanbbox
Please enter at least two lines in a Textarea to test. And the same applies for any other control inside an Accordion, in my case I was using TipTap.

I am temporarily doing this as a workaround:
<AccordionItem
            key={item.$id}
            ...
            onKeyDown={e => e.stopPropagation()}
          >
            ...


Require users to call cmu_graphics.run()

Replaces cmu_graphics.loop() with cmu_graphics.run()
Doesn't attempt to automatically run the mainloop for a user's app
Fixes the issue where an interactive console would be created even when an error happened before cmu_graphics.loop() was called
Prints a help message when shapes are created by cmu_graphics.run() was never called

I think we should check in with the fulltimers to see what they think about loop vs run vs runApp vs something else

Since it looks like we're going with .run(), you could update the README too

@schmave I think this is ready for another look. The build is failing, but only because of a weird segfault in the Python3.7 pip tests


Update from e3e

This is an automated pull request to update from branch e3e

Summary
Start end 2 end testing for zap media and mediafactory. Also pythonized some code and reorganized the files.
I will end up reorganizing the folder structure for a better pip package creation

Zap Media testing is almost done.
Mediafactory testing is done. It's mainly to test deploying a user's media contract address.
Auctionhouse testing to be done when test pip package is made.

Visuals
All tests are passing.
One note on media_factory.deploy_media():
It's failing when running on pytest but passing when running off hardhat node. The eth_tester seems to act weird when it comes to events.