HTTP Basic auth no longer works alongside Uchiwa

It looks as though something is overwriting the Authentication header on some requests (.e.g. /uchiwa/config) but not others, so I while I can authenticate against both uchiwa and basic auth successfully, it doesn't let me past the login page.

Hi,
Thank you for reporting this issue. In order to help us troubleshoot this problem, could you provide us some examples of the requests made against the Uchiwa API?
Thanks

Sure, here are two requests that happen in sequence after entering the uchiwa login credentials:
The actual login passes in the cached Authorization: Basic token correctly and succeeds.
POST /uchiwa/login HTTP/1.1
Host: example.com
User-Agent: <redacted>
Accept: application/json, text/plain, */*
Accept-Language: en-US,en;q=0.5
Accept-Encoding: gzip, deflate, br
Content-Type: application/json;charset=utf-8
Referer: https://example.com/uchiwa/
Content-Length: 31
Cookie: uchiwa_theme=uchiwa-default; <redacted>
Authorization: Basic <redacted>
Connection: keep-alive

HTTP/1.1 200 OK
Server: nginx
Date: Tue, 13 Sep 2016 16:55:04 GMT
Content-Type: application/json
Content-Length: 988
Connection: keep-alive

The next call replaces the  Authorization header with a Bearer token, so nginx rejects it.
GET /uchiwa/config HTTP/1.1
Host: example.com
User-Agent: <redacted>
Accept: application/json, text/plain, */*
Accept-Language: en-US,en;q=0.5
Accept-Encoding: gzip, deflate, br
Authorization: Bearer <redacted>
Referer: https://example.com/uchiwa/
Cookie: uchiwa_theme=uchiwa-default; <redacted>
Connection: keep-alive

HTTP/1.1 401 Unauthorized
Server: nginx
Date: Tue, 13 Sep 2016 16:55:04 GMT
Content-Type: text/html
Content-Length: 188
Connection: keep-alive
WWW-Authenticate: Basic realm="Restricted"

The call after that to uchiwa/bower_components/uchiwa-web/partials/views/events.html also fails in the exact same way.

It looks like you are using both HTTP basic authentication and Uchiwa's built-in authentication? If so, I can simply recommend you to disable one of them since Uchiwa needs to use the Authorization header with a Bearer token as you noticed.
Unfortunately I don't think supporting both authentication mechanisms at the same time is something we should consider nor recommend.


Inconsistent use of 'enum' in mark down docs

The markdown documents have a couple of different ways of describing an enum type.
Within the core.md document, it has the type 'Enum':  Pipeline Run
Within the testing-events.md doc, it is either defined as a String enum:  Testing Case
Or, just as a string Trigger
It would be great to bring these all into alignment.
I'm happy to put a PR together for this. I feel like these should all be straight Enums based on the Design Decisions document, but also wanted to reach out to see it should be standardized to just Enum in the testing events document, or, if we should be using string enum everywhere?

Thanks @obowersa for finding this inconsistency!
I believe that the field type should be noted as String for all enums. The fact that it is limited by an enum should more be considered a format/range restriction of that field. The most explicit way to show that in the docs would probably be to add another column to the tables, called "value format" or "value range" for example. We could later also use that column to limit the range of numerical values for example, if needed.
But such a change would also make the tables larger and maybe harder to read, so I propose for now that we make a smaller change so that the type field show whether it is an enum or not in parenthesis, like this:
| String (enum) |


feat: static from

When building this library with browserify, the Buffer module is bundled internally. To avoid duplicating the code, it would be nice to have a method that will take any input and convert it to a buffer.

I added the encoding parameter.


Unable to run the example app: npx pod-install fails.

Environment
info Fetching system and libraries information...
System:
    OS: macOS 10.15.6
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 659.73 MB / 32.00 GB
    Shell: 5.7.1 - /bin/zsh
  Binaries:
    Node: 13.8.0 - ~/.nvm/versions/node/v13.8.0/bin/node
    Yarn: 1.22.4 - /usr/local/bin/yarn
    npm: 6.13.6 - ~/.nvm/versions/node/v13.8.0/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    iOS SDK:
      Platforms: iOS 13.6, DriverKit 19.0, macOS 10.15, tvOS 13.4, watchOS 6.2
    Android SDK:
      API Levels: 23, 27, 28, 29
      Build Tools: 28.0.3, 29.0.2
      System Images: android-28 | Google APIs Intel x86 Atom, android-29 | Google Play Intel x86 Atom
      Android NDK: Not Found
  IDEs:
    Android Studio: 3.5 AI-191.8<IP_ADDRESS>10548
    Xcode: 11.6/11E708 - /usr/bin/xcodebuild
  Languages:
    Java: 1.8.0_202 - /usr/bin/javac
    Python: 2.7.16 - /usr/bin/python
  npmPackages:
    @react-native-community/cli: Not Found
    react: Not Found
    react-native: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Description
npx pod-install runs with errors. Furthermore yarn run:ios won't complete.
➜ npx pod-install
npx: installed 1 in 1.017s
Scanning for pods...
1.9.3
> pod install
Auto-linking React Native module for target `example`: react-native-slider
Analyzing dependencies
Fetching podspec for `DoubleConversion` from `../../node_modules/react-native/third-party-podspecs/DoubleConversion.podspec`
Fetching podspec for `Folly` from `../../node_modules/react-native/third-party-podspecs/Folly.podspec`
Fetching podspec for `glog` from `../../node_modules/react-native/third-party-podspecs/glog.podspec`
[!] CocoaPods could not find compatible versions for pod "ReactCommon/jscallinvoker":
  In snapshot (Podfile.lock):
    ReactCommon/jscallinvoker (from `../../node_modules/react-native/ReactCommon`)

  In Podfile:
    ReactCommon/jscallinvoker (from `../../node_modules/react-native/ReactCommon`)

None of your spec sources contain a spec satisfying the dependency: `ReactCommon/jscallinvoker (from `../../node_modules/react-native/ReactCommon`)`.

You have either:
 * out-of-date source repos which you can update with `pod repo update` or with `pod install --repo-update`.
 * mistyped the name or version.
 * not added the source repo that hosts the Podspec to your Podfile.

Aborting run
An unexpected error was encountered. Please report it as a bug:
Error
    at CocoaPodsPackageManager._installAsync (/Users/colin/.npm/_npx/78146/lib/node_modules/pod-install/build/index.js:2:85721)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)

➜ yarn run:ios
yarn run v1.22.4
$ react-native run-ios
error Could not find "Podfile.lock" at null.lock. Did you run "pod install" in iOS directory?
error Could not find the following native modules: react-native-slider. Did you forget to run "pod install" ?
info Found Xcode workspace "example.xcworkspace"
info Building (using "xcodebuild -workspace example.xcworkspace -configuration Debug -scheme example -destination id=E34325E5-1147-4A7F-B502-5C564B181E75")
..........
error Failed to build iOS project. We ran "xcodebuild" command but it exited with error code 65. To debug build logs further, consider building your app with Xcode.app, by opening example.xcworkspace. Run CLI with --verbose flag for more details.
Command line invocation:
    /Applications/Xcode.app/Contents/Developer/usr/bin/xcodebuild -workspace example.xcworkspace -configuration Debug -scheme example -destination id=E34325E5-1147-4A7F-B502-5C564B181E75

note: Using new build system
note: Planning build
note: Constructing build description
error: /Users/colin/Documents/code.nosync/react-native-slider/example/ios/Pods/Target Support Files/Pods-example/Pods-example.debug.xcconfig: unable to open file (in target "example" in project "example") (in target 'example' from project 'example')
error: /Users/colin/Documents/code.nosync/react-native-slider/example/ios/Pods/Target Support Files/Pods-example/Pods-example.debug.xcconfig: unable to open file (in target "example" in project "example") (in target 'example' from project 'example')
error: /Users/colin/Documents/code.nosync/react-native-slider/example/ios/Pods/Target Support Files/Pods-example/Pods-example.debug.xcconfig: unable to open file (in target "example" in project "example") (in target 'example' from project 'example')
error: /Users/colin/Documents/code.nosync/react-native-slider/example/ios/Pods/Target Support Files/Pods-example/Pods-example.debug.xcconfig: unable to open file (in target "example" in project "example") (in target 'example' from project 'example')

** BUILD FAILED **

error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Reproducible Demo
#!/bin/sh
git clone git@github.com:react-native-community/react-native-slider.git
cd react-native-slider
yarn
cd src
yarn
cd ..
cd example
npx pod-install
yarn run:ios

It seems that changing
pod 'ReactCommon/jscallinvoker', :path => "#{$node_modules_path}/react-native/ReactCommon"

to
pod 'ReactCommon/callinvoker', :path => "#{$node_modules_path}/react-native/ReactCommon"

fixes the issue for me.
see https://stackoverflow.com/a/60881827/5444033

I created a PR
Idk know honestly if it is a good idea to merge it, because of the huge diff in the lockfile, but there is also no way (for the solution I have tried) to prevent the changes.


On clicking the "Set Color theme" shows "Cannot read property 'id' of undefined" as error

Hi Team,
I have tried with mac os Sierra,
when I tried with installing and click on "Set Color theme" shows the "Cannot read property 'id' of undefined"

Hm, strange. Have you tried shift + ctrl P and then Preferences: Color Theme, and finding the theme in there?

Whatever this is, I'm pretty sure it's not the theme itself, it's VS Code. Feel free to reopen if I'm mistaken!

This error happen to me too after updating VS Code. Closing and opening again did the trick

Same error here, started happening since last week. Restarted VSCode, Macbook & even my fridge, still same. :(

I'm getting the same error. And the theme got deactivated and I am unable to use it. Any luck getting it patched?

Hey all, I'm sorry but I really don't think this is an error with the theme but rather VS Code. Could you please file an issue in their repo?

I got it to work by uninstalling it, then re-installing the theme.

install dracula theme -> file -> Preferences -> Color Theme-> you select theme  preference

I had the same error with Atom Dark Theme after installing the jshint extension of Dirk Baeumer.
The solution is as described above by @sdras  or disabling the jshint extension

Help!

Hm, strange. Have you tried shift + ctrl P and then Preferences: Color Theme, and finding the theme in there?

Thanks, it worked!

Had a theme somehow listed in settings file that was no longer installed....

Hm, strange. Have you tried shift + ctrl P and then Preferences: Color Theme, and finding the theme in there?

thank you

thanks

install dracula theme -> file -> Preferences -> Color Theme-> you select theme preference
thanks man


Notebooks: Prompt the user when links are disabled

Issue Type: Bug
Links in Notebooks do not work, until you click the "Trusted" button on top. This is un-intuitive. When the user clicks a link, we should atleast show a message to the user that the link will not work because of the Untrusted state, and give an option to Trust the notebook right there in the dialog.
Azure Data Studio version: azuredatastudio 1.13.0 (05a53af085861999ed8dc99596cd55829387d490, 2019-10-22T07:41:49.347Z)
OS version: Windows_NT x64 10.0.18362

Did you just create the ADSLivesite label? We should match the standard we have with labels. Area - Live Site

Fixed.

@qifahs @MsSQLGirl is this something that our livesite users are still encountering, or has the work to trust a workspace/folder been sufficient here?


Remove report.ownerEmails that slipped through or were recently added

Details
Follow-up to https://github.com/Expensify/App/pull/22075 where we're working on removing many uses of emails where we now have accountIDs. Here specifically we're removing remaining uses of report.ownerEmail.
Fixed Issues
$ Related to https://github.com/Expensify/Expensify/issues/294647
Tests
To cover our bases, let's create new accounts while performing these tasks:

New task assignee + edit task title & description
Create new workspace
Invite users to workspaces
Close workspace & view archived workspaces + closed report action
New 1:1 & group chats
Send money, request money, split bill

For each of ^ tests, verify:

Avatars & tooltips appear correctly (these accounts should all show defaults)
Display names / logins show where they should (not showing as blank anywhere)

Finally log in to a brand new account, make sure everything works well

[ ] Verify that no errors appear in the JS console

Offline tests
All of the above should be able to be done offline too, except logging in to a brand new account
QA Steps
Same as above

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

The WorkspaceInviteMessagePage prop type error was actually being fixed here - https://github.com/Expensify/App/pull/21201 but I may move that fix to this PR just to have it all in 1

@allroundexperts it would be great if you could super clear how to reproduce those issues you saw :D But I will play around to try to reproduce! It looks like some of those things you tested while offline?

When creating a task, on confirm screen, hovering over the avatar shows the following:

FYI This exists on staging, it's not from my changes 👍

@allroundexperts nope was online 😅  I just briefly tried offline and I basically only see failing pusher requests in the console teehee - I'll try again in about 1/2 hour

@allroundexperts ok so I did a bit more testing of the flow in staging and here's what I saw:
For an account where I am 99% sure I already had personal details in onyx:
https://github.com/Expensify/App/assets/3885503/fcce0dfe-0751-4d8d-ae8d-d46d616e109f
For an account where I definitely didn't have personal details in Onyx:
https://github.com/Expensify/App/assets/3885503/d168c909-a59a-42a7-9439-8d80be9419bb
This was on staging so YES we definitely need to get it fixed (seems like another case of needing to store the optimistic data in Onyx, then clear it on success) BUT it's not related to changes in this PR

If you find any more bugs from testing this PR can you please also test if those bugs exist in staging?? 🙏  Thanks 👍

@allroundexperts Innnnnnteresting, I wonder why we seem to have had different results - you're saying you previously didn't reproduce those on staging? I'm quite interested to know if you can today 🙃

@allroundexperts that bug that you just found loooooks like it's the same as this issue: https://github.com/Expensify/App/issues/21706
Do you agree or disagree? 🤔

@youssef-lr removing you being requested for review since @madmax330 is already assigned to review, but feel free to review if you'd like :D
@madmax330 ready for your review if you have a chance today 🙏

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/madmax330 in version: 1.3.42-0 🚀

platform
result

🤖 android 🤖
cancelled 🔪

🖥 desktop 🖥
cancelled 🔪

🍎 iOS 🍎
cancelled 🔪

🕸 web 🕸
cancelled 🔪

🚀 Deployed to production by https://github.com/jasperhuangg in version: 1.3.42-26 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


certain POST methods fail

It appears that certain post methods succeed and certain methods fail.
For example lists/destroy fails - despite setting the list_id (and the other possible options) Twitter never sees the list_id
I'm guessing this is related to the issue that was closed (without resolution) by the other guy.

same here

@phpfalcon for what it's worth I found the codebird library works perfectly for all twitter api methods and took me about 5 minutes to replace this Twitter library. https://github.com/jublonet/codebird-php

Doesn't really help the community just suggesting a switch. We need to get to the root of this. PRs that prove it is fixed are welcome.

Hello,
got the same issue: works well with statuses/update.json but fail with lists/members/create_all.json
didn't solve the issue but maybe it will help:
I added CURLOPT_VERBOSE and CURLOPT_STDERR in the performRequest. I tried to compare between curl command line working and libcurl from performRequest failing.
The Content-Type was different: multipart/form-data in performRequest as post values are passed in an array. So I tried to pass my post parameters in a string format (ex: list_id=198390090&screen_name=al_rousset)
Then I had an other error "Could not authenticate you", so I used the OAuth Tool from twitter to generate the authenticate header and then it works...
Then I tried a different parameter with a coma: list_id=198390090&screen_name=alainchretien,alain_gest and then I got the "Could not authenticate you". So I'm quite lost now...

I stopped in to say the same thing. I've not been able to do any list add or remove from lists.

+1 for me, I was trying to upload media through #91, and after fixing the headers, the parameters never get sent (update status ignore trim_user and media_ids).
I switched to codebird, thanks @arlogilbert

Media upload now fixed.

This API interface does not enable calls to twitter list related endpoints. I independently tried everything @nouchka described to no avail.  Surprised no one has figured this out in six years....
https://github.com/J7mbo/twitter-api-php/issues/99#issuecomment-78394687


How do I read the information in syso.json in the program, such as ProductName、FileVersion、ProductVersion...

I want read resource information in the program run.

@iquanxin I think you can check it in the executable's properties window. Right-click on the executable and see Properties and go to Details tab to see it.

@iquanxin You can read the EXE version information using ResourceInfoLoad from Windigo library:
resNfo, _ := win.ResourceInfoLoad(win.HINSTANCE(0).GetModuleFileName())
ffNfo, _ := resNfo.FixedFileInfo()
vMaj, vMin, vPat, _ := ffNfo.ProductVersion() // version information, like 1.0.0

block0 := resNfo.Blocks()[0]
productName, _ := resNfo.ProductName(block0.LangId, block0.CodePage)
companyName, _ := resNfo.CompanyName(block0.LangId, block0.CodePage)
// and so on...

@iquanxin You can read the EXE version information using ResourceInfoLoad from Windigo library:
resNfo, _ := win.ResourceInfoLoad(win.HINSTANCE(0).GetModuleFileName())
verNfo, _ := resNfo.FixedFileInfo()
vMaj, vMin, vPat, _ := verNfo.ProductVersion() // version information, like 1.0.0

blocks := resNfo.Blocks() // each block contains one language
productName, _ := blocks[0].ProductName()
companyName, _ := blocks[0].CompanyName()
// and so on...

Perfect.


MESH: Root looses IP when a new node connects to the network (IDFGH-13292)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
Hi there, I am having a problem while connecting a new node to my mesh network:

IDF Version: release 5.2 (commit cc649ea869b8c24d19ba2d94c4d3f1146cb2947b)
Board: custom
Setup: mesh network with one root and one node already installed, trying to connect a second node (although it looses ip also while connecting the first one
Expected Behaviour: the root should accept a new node without losing connection to the router
Actual behaviour: the root looses IP when the new node attemps to connect to the network
sdkconfig:  sdkconfig.txt
log:  log.txt

Is there something in the sdkconfig i should change in order the prevent this issue?
Thank you very much in advance!

@luisyague

which ESP device do you use?
Can you provide a demo for us to test?

Hello, @zhangyanjiaoesp I am using a custom-made board with ESP32 32E, unfortunately as this is production firmware so I can't give you a demo, but I am using a pretty straightforward approach:

I am using both cores (core 0 for stack threads and wifi-mesh and core 1 for my app thread (it is just a heater app which uses adc, spi some digital outputs)
I am using secure-boot + flash encryption (it also happens without flash-encryption)
Then my mesh application just calls some initilizations from esp-mdf with the default values, I know it is no longer supported but it is just a wrapper for esp-mesh so I dont think it is related.

@luisyague
It should related to the application code. Because our official mesh example can works well. And according to the existing log, I can't determine what the problem is.

Sorry for the delay, okei i'll take a look at the application code then, and will make a post with the solution if i find something. Thanks a lot for the support!!!

Found it, pretty weird... I just changed th defaut channel from 1 to 0 and started working without problems.  Hope this helps anyone with the same problem.


Language Packs add a line to Command Palette

I've seen that language packs that localise VSCode to other languages add a line to the command palette with the english command bellow the localised one.
Ie for the french localisation :

I would like to have an option to disable this behaviour (in order to make the command palette much more usable, because now it can't show as many commands as before), or to overwrite the langauge VSCode is using for the command palette and displaying all in english.
I'm ok for this behaviour to exist, because I can work in english without the localisation, but for beginners, or simply ones that would like to have a nice and thin palette having the option(s) would be nice
Edit : I'm totally new to opening issues, don't know if it's the right place to open this one, if i'm wrong or did anything incorrectly, tell me please

This is intentional so that if you're on Stack Overflow and the command is in English, you can still paste the English string into the Command Palette and it will find the command even if you are using a language pack.
Still, if you're ok with not being able to do that, we can have an option... Let's put this in the backlog candidates to see if other users are looking for this.
cc @bpasero in case you know something I don't

Moving to backlog. I think this is reasonable. I'm not actively working on it but if someone wants to pick it up it's likely fairly straightforward.


[Resource Handling] Make sure all features are included in reference doc. @backwind1233

As title.

@backwind1233 is this issue finished?

@saragluna I will check it today.

Close this issue, since new features were included in the reference doc.


Loading the custom getters does not work after getting the dataloading functions from the sftp server

I created a plugin-like system for custom getters in order to separate specific project getters from the main code of the package. The issue now is that when the dataloading functions are downloaded from the sftp server, the localgetters are not included. This is because the import statement tries to read localgetters on the local machine, not from the server.
A solution would be to rewrite the main code as a python package that can be installed, and the local getters are stored on the server. I think in hindsight this would have been preferable from the start.

This has been fixed by the rewrite in 1b05b2c.

After updating the design, this issue is fixed.


Minimize to System Tray

Probably as a result of this, also add functionality for starting the application in the system tray.

Done.


Adding Toolbar and Intro features

This PR includes:
Building of the first two sections (Toolbar & Intro) of the mobile website as per the requirement in the template.
The toolbar consists of a MyLogo icon and a hamburger menu icon.
The Intro section consists of a greeting to the viewer and a small description of myself.

Hi <STUDENT_MENTION>,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that linters check is failing. Kindly fix linter errors and submit for a re-review.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


Class table no longer separates from text columns properly?

--> http://prnt.sc/e7ym6v <--
This suddenly happened to all my class tables. Friends reported the issue to me, so it's not just on my end. Is a fix known? The brews are from an older version of homebrewery, so I suspect some dated code issues.
Additional Details
Brew code to reproduce : Click to expand

##### The Earth Seer

Level
Proficiency Bonus
Stone Strokes
Features
Ki Points
Cantrips Known
Spells Known

1st
+2
1d4
Earth's Fortitude, Stone Strokes
-
-
-

2nd
+2
1d4
Ki
2
2
2 ( 1st lvl )

3rd
+2
1d4
Seer Structure
3
2
2 ( 2nd lvl )

4th
+2
1d6
Ability Score Improvement
4
2
3 ( 2nd lvl )

5th
+3
1d6
Terrene Athlete
5
3
3 ( 3rd lvl )

6th
+3
1d6
Seer Structure Feature
6
3
4 ( 3rd lvl )

7th
+3
1d8
Elemental Understanding
7
3
4 ( 4th lvl )

8th
+3
1d8
Ability Score Improvement
8
3
5 ( 4th lvl )

9th
+4
1d8
Seismic Strikes
9
3
5 ( 5th lvl )

10th
+4
1d8
Tremorsense
10
3
6 ( 5th lvl )

11th
+4
1d8
Seer Structure Feature
11
4
6 ( 6th lvl )

12th
+4
1d10
Ability Score Improvement
12
4
7 ( 6th lvl )

13th
+5
1d10
Heightened Focus
13
4
7 ( 7th lvl )

14th
+5
1d10
Improved Earth's Fortitude
14
4
8 ( 7th lvl )

15th
+5
1d10
Timeless Body
15
4
8 ( 8th lvl )

16th
+5
1d10
Ability Score Improvement
16
4
9 ( 8th lvl )

17th
+6
1d10
Seer Structure Feature
17
4
9 ( 9th lvl )

18th
+6
1d12
Gravity Weaver
18
4
10 ( 9th lvl )

19th
+6
1d12
Ability Score Improvement
19
4
10 ( 9th lvl )

20th
+6
1d12
Planar Fissure Strike
20
4
10 ( 9th lvl )

##### Class Features
As an Earth Seer, you gain the following features.

### Hit Points
**Hit Dice:** 1d8 per Earth Seer level.
**Hit Points at 1st level:** 8 + your Constitution modifier
**Hit Points at Higher Levels:** 1d8 (or 5) + your Constitution modifier per Earth Seer level after 1st
Proficiencies
Armor: Light Armor
Weapons: Daggers, Shortswords, Quarterstaffs, Slings
Tools: (a) Jeweler's Tools or, (b) Mason's Tools
Saving Throws: Strength, Wisdom
Skills: Choose two from Acrobatics, Athletics, History, Insight, Religion, Nature, and Survival.
Equipment
You start with the following equipment, in addition to the equipment granted by your background:

(a) a quarterstaff, or (b) a shortsword
(a) a sling, or (b) two daggers
(a) a dungeoneer's pack or (b) an explorer's pack
Leather armor

Earth's Fortitude
Your innate connection to the earth increases your resilience.
While your are wearing no armor and not wielding a shield, your AC equals 10 + your Dexterity modifier + your Constitution modifier.
Additionally, if you are wearing Light armor, your Constitution modifier may replace your Dexterity modifier when calculating your AC.
Stone Strokes
Starting at 1st level, you may spend a bonus action to coat your limbs in nearby rock, giving you 2 'Stone Strokes' charges.
You may take this bonus action once, and must complete a short or long rest to use it again.
So long as you have at least 1 charge available, you can roll a d4 in place of the normal damage of your unarmed strikes.
This die changes as you gain Earth Seer levels, as shown in the Stone Strokes column of the Earth Seer table.
Additionally, whenever you hit a creature with a spell or unarmed strike, you may expend one or two charges and add their respective damage dice to the damage total as bludgeoning damage.
1
Earth Seer
\page
Ki
Starting at 2nd level, in similar ways to a monk, your connection to the earth allows you to harness the mystical energy of ki. Your access to this energy is represented by a number of ki points. Your Earth Seer level determines the number of points you have, as shown in the Ki Points column of the Earth Seer table.
You can spend these points to fuel various ki features. You start knowing three such features: Refresh and two earth seer spells.
When you spend a ki point, it is unavailable until you finish a short or long rest, at the end of which you draw all of your expended ki back into yourself. You must spend at least 30 minutes of the rest meditating to regain your ki points.
Refresh
You may spend 1 ki points to manifest your stone strokes, without the need to complete a short rest.
Cantrips
Starting at 2nd level, you know two cantrips of your choice from the earth seer spell list. You learn additional earth seer cantrips of your choice at higher levels, as shown in the Cantrips Known column of the Earth Seer table.
Spells Known of 1st Level and Higher
From 2nd level onward, you know two 1st-level spells of your choice from the earth seer spell list.
The Spells Known column of the Earth Seer table shows when you learn more earth seer spells of your choice. The maximum spell level for these spells is shown between parentheses. Additionally, when you gain a level in this class, you can choose one of the earth seer spells you know and replace it with another spell from the earth seer spell list, which must also be equal or lower to your maximum spell level.
Casting Earth  Seer Spells
Unlike other classes adept in the use of magic, Earth Seers (and monks trained in the Way of the Four Elements) tap into their ki to bring forth magic. To cast a spell, you use its casting time and other rules, but you don't need to provide material components unless they have a listed price.
Casting a spell expends ki points equal to the spell level + 1.
Cantrips require no ki point to cast.
When you use ki to cast a spell of 6th level or higher, you must complete a long rest before you can cast another spell of the same spell level.
2
Earth Seer
Spellcasting Ability
Wisdom is your spellcasting ability for your Earth Seer spells, as your magic draws upon your connection to- and the understanding of the very earth itself. You use your Wisdom whenever a spell or class feature refers to your spellcasting ability. In addition, you use your Wisdom modifier when setting the saving throw DC for an earth seer features and spells you cast and when making an attack roll with them.

**Spell save DC** = 8 + your proficiency bonus + your Wisdom modifier

**Spell attack modifier** = your proficiency bonus + your Wisdom modifier
```
```
Seer Structure
At 3rd level, you choose to identify with a Seer Structure: the Structure of Sand, the Structure of Soil or the Structure of Stone, all detailed at the end of the class description. Your choice grants you features at 3rd level and again at 6th, 11th and 17th level.
Ability Score Improvement
When you reach 4th level, and again at 8th, 12th, 16th, and 19th level, you can increase one ability score of your choice by 2, or you can increase two ability scores of your choice by 1. As normal, you can’t increase an ability score above 20 using this feature.
Terrene Athlete
At 5th level, you have learned to empower your feats of athletics by moving the earth beneath your feet to propel you. So long as you are touching rock or earth, your walking speed (and burrowing speed if applicable) increases by 10 and you are always considered to have a running start when jumping. Additionally, if grounded, you may spend 1 ki point to take the Dash action as a bonus action.
Elemental Understanding
Starting at 7th level, you may spend a ki point to speak with rocks and earth related golems as if you shared a language, for up to 10 minutes. Additionally, you learn to speak and write the Primordial language.
Seismic Strikes
Starting at 9th level, your unarmed strikes are considered to deal both Bludgeoning damage and Thunder damage simultaneously, so long as you have at least one active charge of stone strokes. A creature that resists one, but not both types of damage takes damage as if it had no resistance to either. If both types are resisted, resistance is applied as normal.
Tremorsense
Starting at 10th level, your connection with the earth has become strong to the point where you can sense its very vibrations. You gain tremorsense, allowing you to detect any corporeal grounded or burrowing creature within a range of 30 feet, or 60 feet if your skin is directly in contact with the ground. A creature that spends its turn without disturbing the earth cannot be detected in this fashion.
Heightened Focus
Starting at 13th level, whenever you manifest your Stone Strokes, you may expend 2 ki points to gain 2 additional Stone Strokes charges.
\page
Improved Earth's Fortitude
From 14th level onward, your body has adapted to resist even the most potent of poisons. You resist poison damage and you have advantage on Constitution saving throws to resist poison.
 You can still die of old age, however. In addition, you no longer need food or water.
Gravity Weaver
Starting at 18th level, you have learned to manipulate your personal gravitational influence. Once per short rest, you may take an action to impose a Strength saving throw on all creatures within 30 feet. Choose one of the effects below. All creatures that fail the save take 3d6 Force damage and suffer the effect.
Gravitation. Affected creatures are moved 10 feet directly towards you.
Repulsion. Affected creatures are moved 10 feet directly away from you.
Planar Fissure Strike
At 20th level, your connection to the earth has become so powerful, your attacks tear rifts to and from the elemental plane of Earth. Whenever you hit a creature with a spell attack or unarmed strike, you may expend a charge of stone strokes to deal an additional 4d12 Force damage as part of the attack. Additionally, if the attack reduces the target to 50 hit points or less, you banish it to the elemental plane of Earth. A creature that is already on this plane instead vanishes into a harmless demiplane. While there, the target is incapacitated. It remains there for 1 minute, before it reappears in the space it left, or in the nearest unoccupied space if that space is occupied.
You must complete a long rest before you can use this feature again.

Seer Structures
Three Structures of Earth Seers exist throughout the multiverse. Each harbors seers that specialize in its respective field of expertise, though conclaves of all three structures are hosted sporadically, to discuss discoveries or issues with one another, or share pleasantries. All three structures share the same foundations, allowing their students to be schooled together until it is time for them to dedicate themselves to a particular structure.
the Structure of Sand
Seers of the Structure of Sand focus on discovering the mysteries within the depths of the sands. Most live a nomadic life in deserts, though some spend time on the ocean floor, in search of treasures of both wealth and knowledge.
Scorpion's Sting
Starting when you choose this Structure at 3rd level, whenever you take the attack action, you may make one unarmed strike as a bonus action. The attack may use your Stone Strokes damage die, even if you expended your last charge taking the prior attack action.
Ambush Predator
At 6th level, you gain a burrowing speed of 30 feet. In addition, whenever you burrow 20 feet straight toward a target right before hitting it with a melee weapon attack, the target must succeed a Strength saving throw or be knocked prone. A creature you have surprised has disadvantage on the saving throw.
Sidewinder
At 11th level, you can instinctually navigate deserts and other large masses of sand. Traveling in arid environments does not cause you to gain levels of exhaustion and you gain advantage on Wisdom (Survival) checks when trying to navigate the desert. Additionally, you can always tell the time, and which way is north, from the position of the sun and moon.
Kurara's Wrath
Starting at 17th level, You may take an action to cast Sand Storm at 9th level without expending ki points. You must finish a long rest before you can use this feature again.
3
Earth Seer
\page
the Structure of Soil
Those that join the Structure of Soil often do so to better understand and control the earth. Members of this structure often go out of their way to uphold balance in the universe, and their fighting style of weaving in spells between strikes reflects this.
Ki Efficiency
Starting when you choose this structure at 3rd, and again at 6th and 11th level, your maximum amount of spells known increases by 1. In addition, whenever you take an action to attack or cast a spell, you may spend 1 ki point to cast a cantrip from the earth seer spell list as a bonus action.
Arcane Disruption
From 6th level onward, you may choose to expend 2 additional ki points whenever you cast a spell that can deal damage. Choose 1 creature affected by the spell. The excess arcane energy lingers within its body for up to 1 hour. If the creature casts a spell or is hit by a melee weapon attack, the arcane energy detonates. The creature takes Force damage equal to 3 x your stone strokes damage die and must make a Constitution saving throw. If it fails the saving throw, it is also stunned until the end of your next turn.
If it succeeds the saving throw, it becomes immune to the stun effect until it completes a long rest.
Arcane Overdrive
At 11th level, you have learned to use your ki at maximum efficiency, relaying the excess energy straight to your muscles. Once per turn, if you spend ki to cast a spell, your speed increases by 10 feet and you suffer no movement penalty when traversing difficult terrain. This effect lasts until the end of your turn.
Orbital Reach
Starting at 17th level, you have become so adept at manipulating the earth, your influence stretches even beyond the atmosphere.
You gain access to the Meteor Swarm spell. This spell does not count towards your amount of spells known.

Multiclassing Prerequisites
Wisdom 13
Multiclassing Proficiencies
(a) Jeweler's Tools or, (b) Mason's Tools
Optional: Starting Wealth
2d4 x 10 gp

the Structure of Stone
Seers of the Structure of Stone are renowned for their expert stonework and gem cutting. Artisans live as clans, in large, beautiful bastions and castles. Others roam the land to study stone and precious minerals in the field from within the relative safety of their stone constructs.
Rock Juggernaut
Starting when you choose this Structure at 3rd level, you gain superior control over rock, allowing you to muster up enough material to cover your entire body when you activate your stone strokes feature, increasing your size by 1 stage. This second skin of rock can be made to have whatever appearance you desire. Your rock juggernaut form grants you 16 temporary hit points, + 2 for every earth seer level beyond 3rd level. These hitpoints do not absorb Psychic or Thunder damage. Once the temporary hitpoints are depleted, the form collapses into a small pile of rubble.
Your rock juggernaut form cannot be manifested by activating the Refresh feature.
So long as your rock juggernaut form is active, your more precise movements are restricted, and you cannot provide somatic spell components.
Adept Mountaineer
Starting at 6th level, you gain a climb speed of 40, and you make Wisdom (Survival) checks to navigate mountains with advantage.
Form Detonation
From 11th level onward, you may take an action to expend all you remaining charges of stone strokes (2 minimum) to have your rock juggernaut form explode.
Reduce your rock juggernaut hit points to 0. All creatures (except you) in a 15 foot radius must make a Dexterity saving throw.  This damage increases by 1 die for every third Earth Seer level beyond the 11th, and for each charge of stone strokes you expended beyond 2.
Additionally, the rubble left behind by the explosion causes the affected area to become difficult terrain until it is cleared.
You may activate Form Detonation whenever your Rock Juggernaut hit points are reduced to 0, without spending an action to do so.
Diamond Juggernaut
At 17th level, you have learned how to safely subject your stone outer skin to extreme pressure. You may take an action to compress and harden your Rock Juggernaut form. This grants you resistance to all piercing, bludgeoning and slashing damage, and Psychic- and Thunder damage can no longer bypass your rock juggernaut hit points.
4
Earth Seer
\page
Earth Seer Spells
Cantrips (0 Level)
Druidcraft
Magic Stone
Mending
Minor Illusion
Mold Earth
Resistance
Sword Burst
True Strike
1st Level
Absorb Elements
Catapult
Detect Magic
Earth Tremor
Ensnaring Strike
Entangle
Feather Fall
Jump
Tenser's Floating Disk
2nd Level
Cloud of Daggers
Dust Devil
Earthbind
Find Traps
Hold Person
Knock
Levitate
Locate Animals or Plants
Maximilian's Earthen Grasp
Rope Trick
Shatter
Spider Climb
Spike Growth
3rd level
Earth Servant
Elemental Weapon
Erupting Earth
Feign Death
Leomund's Tiny Hut
Meld Into Stone
Melf's Minute Meteors
Nondetection
Wall of Sand
4th Level
Banishment
Blight
Conjure Minor Elementals
Elemental Bane
Fabricate
Freedom of Movement
Hallucinatory Terrain
Sandstorm
Stone Shape
Stoneskin
Mordenkainen's Private Sanctum

5th Level
Animate Objects
Bigby's Hand
Conjure Elemental
Destructive Wave
Hold Monster
Legend Lore
Passwall
Telekinesis
Transmute Rock
Wall of Stone
6th Level
Blade Barrier
Bones of the Earth
Disintegrate
Find the Path
Flesh to Stone
Investiture of Stone
Move Earth
7th Level
Mirage Arcane
Mordenkainen's Magnificent Mansion
Plane Shift
Reverse Gravity
Sequester
8th Level
Demiplane
Earthquake
Maze
Tsunami
9th level
Astral Projection
Gate
Imprisonment
Meteor Swarm (Exclusive to Structure of Soil)
5
Earth Seer
\page
Earth Servant
3rd-level conjuration

Casting Time:  1 minute
Range: 10 feet
Components: V, S, M (A chunk of clay and a drop of water)
Duration: Instantaneous

This spell creates a construct servant.   The target becomes a Terracotta soldier if you chose clay or a Lesser Golem if you chose earth. (the creature's game statistics are listed later on this page.)

On each of your turns, you can use a bonus action to mentally command any creature you made with this spell if the creature is within 60 feet of you (if you control multiple creatures, you can command any or all of them at the same time, issuing the same command to each one). You decide what action the creature will take and where it will move during its next turn, or you can issue a general command, such as to guard a particular chamber or corridor. If you issue no commands, the creature only defends itself or its creator against hostile creatures. Once given an order, the creature continues to follow it until its task is complete.

The creature is under your control for 24 hours, after which it crumbles to dust. To maintain control of the creature for another 24 hours, you must cast this spell on the creature again before the current 24-hour period ends. This use of the spell reasserts your control over up to four creatures you have animated with this spell, rather than animating a new one.

At Higher Levels. When you cast this spell using a spell slot of 4th level or higher, you animate or reassert control over two additional constructs for each slot level above 3rd. Each of the creatures must come from a different pile of clay or earth.
6
Earth Seer
Sandstorm
4th-level evocation

Casting Time: 1 action
Range: 300 feet
Components: V, S, M (A pinch of sand and a Roc's feather)
Duration: Instantaneous

A dervish of wind and sand ravages the ground in a 20-foot-radius, 40-foot-high cylinder centered on a point within range. Each creature in the cylinder must make a Dexterity saving throw. A creature takes 2d8 bludgeoning damage and 4d6 slashing damage on a failed save, or half as much damage on a successful one.

a foot of loose sand turns the storm's area of effect into difficult terrain until the end of your next turn.

At Higher Levels. When you cast this spell using a spell slot of 5th level or higher, the bludgeoning damage increases by 1d8 for each slot above 4th.

Terracotta Soldier
Medium Construct, neutral

Armor Class 10
Hit Points 22 (3d8 + 9)
Speed 30ft.

STR
DEX
CON
INT
WIS
CHA

12 (+1)
10 (+0)
16 (+3)
1 (-5)
1 (-5)
6 (-2)

Damage Immunities poison, psychic
Condition Immunities charmed, exhaustion, frightened, paralyzed, petrified, poisoned
Senses passive Perception 8
Languages -
Challenge 1/4 (50 XP)

***False Appearance. *** While the soldier remains motionless, it is indistinguishable from a statue.
Actions
Spear. Melee Weapon Attack: +3 to hit, reach 5ft., one target. Hit 4 (1d6 + 1)

Lesser Golem
Medium Construct, neutral

Armor Class 13 (natural armor)
Hit Points 25 (3d8 + 12)
Speed 25ft.

STR
DEX
CON
INT
WIS
CHA

15 (+2)
3 (-4)
18 (+4)
1 (-5)
1 (-5)
6 (-2)

Damage Immunities poison, psychic
Condition Immunities charmed, exhaustion, frightened, paralyzed, petrified, poisoned
Senses passive Perception 5
Languages -
Challenge 1/4 (50 XP)

Actions
Slam. Melee Weapon Attack: +4 to hit, reach 5ft., one target. Hit 5 (1d6 + 2)

No developments, yet?

Can you provide the share link to your brew? Your code above isn't copying completely for me for some reason. I'll give it a look.

http://homebrewery.naturalcrit.com/share/rJQ-wfSJx

This looks like a duplicate of https://github.com/stolksdorf/homebrewery/issues/291 which was never really solved.
How does it look when it's printed?

Looks exactly the same. It looks fine for a few moments after I mess with it, and during that time, printing it also makes it look fine, but my other brew: http://homebrewery.naturalcrit.com/share/rk8V-o3- doesn't even get fixed for a single moment :/

**Brew code to reproduce** : Click to expand
##### The Earth Seer

Level
Proficiency Bonus
Stone Strokes
Features
Ki Points
Cantrips Known
Spells Known

1st
+2
1d4
Earth's Fortitude, Stone Strokes
-
-
-

2nd
+2
1d4
Ki
2
2
2 ( 1st lvl )

3rd
+2
1d4
Seer Structure
3
2
2 ( 2nd lvl )

4th
+2
1d6
Ability Score Improvement
4
2
3 ( 2nd lvl )

5th
+3
1d6
Terrene Athlete
5
3
3 ( 3rd lvl )

6th
+3
1d6
Seer Structure Feature
6
3
4 ( 3rd lvl )

7th
+3
1d8
Elemental Understanding
7
3
4 ( 4th lvl )

8th
+3
1d8
Ability Score Improvement
8
3
5 ( 4th lvl )

9th
+4
1d8
Seismic Strikes
9
3
5 ( 5th lvl )

10th
+4
1d8
Tremorsense
10
3
6 ( 5th lvl )

11th
+4
1d8
Seer Structure Feature
11
4
6 ( 6th lvl )

12th
+4
1d10
Ability Score Improvement
12
4
7 ( 6th lvl )

13th
+5
1d10
Heightened Focus
13
4
7 ( 7th lvl )

14th
+5
1d10
Improved Earth's Fortitude
14
4
8 ( 7th lvl )

15th
+5
1d10
Timeless Body
15
4
8 ( 8th lvl )

16th
+5
1d10
Ability Score Improvement
16
4
9 ( 8th lvl )

17th
+6
1d10
Seer Structure Feature
17
4
9 ( 9th lvl )

18th
+6
1d12
Gravity Weaver
18
4
10 ( 9th lvl )

19th
+6
1d12
Ability Score Improvement
19
4
10 ( 9th lvl )

20th
+6
1d12
Planar Fissure Strike
20
4
10 ( 9th lvl )

Class Features

Duplicate of #291


Explicitly import cmath

This fixes a build error encountered on Ubuntu 18.04 using GCC.

Talk about necromancy :P


cert-manager: removing and preventing 'alternate' versions of Helm charts

Hey all 👋
I've been taking a look through various Helm chart repos (i.e. artifacthub, helm hub, chartcenter etc) and have found that there are a number of 'alternate' versions of the cert-manager Helm chart being published and listed on the Helm Hub: https://hub.helm.sh/charts?q=cert-manager
Whilst I am all in favour of openness and allowing people to do what they want with our software, I am concerned that this may 1) confuse end-users and 2) put users on an 'unsupported' release, which we will have little to no recourse to correct as a project.
If there are changes needed in the cert-manager Helm chart, it'd be far more productive if these could be proposed, discussed and merged upstream so that we can properly take care of users.
There are currently three repositories that contain a variant of the cert-manager chart (all of which are either incorrectly published with the wrong version number, contain patches that are now already upstreamed and implemented in a manner that is dangerous for end-users, or otherwise out of date):

infobloxopen (managed by @drewwells) - which appears to just be a direct copy (no changes to the README at least), on an outdated version. The repo in question also reads as if it is cert-manager specific based on the domain name too.

choerodon (managed by @vinkdong) - from what I can tell, this exists to provide a mirror for our Helm chart so it can be used in China, although I am not certain on the version being published here without digging into the chart further myself. I am uncertain whether our main Helm chart (jetstack/cert-manager) can be fetched in China, and do appreciate the efforts from the community to help adoption in this region. If that is the case, it'd be great to work with you @vinkdong to avoid us confusing others users (and to sort out version numbering, or for us to ensure the jetstack repo is accessible in China).

cloudposse (managed by https://cloudposse.com/, added by @osterman) - this is a very old version of our Helm chart that includes CRDs as part of the Helm chart in an unsafe way (via the crd-install annotation) - without going into the conversation around use of this annotation again, to summarise, it prevents the cert-manager team's ability to manage CRD versions. Our own Helm chart already supports this functionality nowadays anyway, in a safe way, via the installCRDs option in our values.yaml.

What's the best way to go about addressing this? If the cc'd maintainers could respond here with their thoughts, and/or make the change to remove these charts, it'd be greatly appreciated.
As an aside, is there any kind of process to handle this in other instances? As these sorts of 'registries' develop, it's going to become more and more of an issue (especially if big corp legal teams get involved 😅)
Thanks! 🚀

@munnerz thanks for bringing this up here.
The first thing I'd like to point out is that multiple charts for the same application are technically ok. For example, someone might provide a chart with a simple install while someone else might provide something far more configurable. Multiple charts from different people are ok.
I'm reminded that there are numerous debian packages for mysql or mariadb out there. Some are not very discoverable but they exist.
That being said, we have long had conversations on highlighting the "official" chart where possible. For the case where the project itself puts out a chart. This is not something we have implemented in the Helm Hub. Might I ask that you file an issue on Artifact Hub about this. I will follow-up on your issue there but that way you can track it. The long term plan for the Helm Hub is up in the air. Ideally, we would like the Artifact Hub to replace it. Either with the software or with the actual site.
The Artifact Hub has the ability for people to star charts. I think that's also a nice way to help indicate which to use. We are looking at other markers and if you have suggestions please file issues.
Quay is listed as being available in China so I would suspect your images are as well. There is a Helm Hub like site for China at https://developer.aliyun.com/hub/. This is done by Alibaba who tries to make sure the charts work in China.
There is no perfect solution to this problem. We can do much better, though. Suggestions are welcome.

@munnerz thanks for raising the issue regarding our cert-manager chart.
We should delete that chart from https://github.com/cloudposse/charts, but will preserve it in our chart repository for people still using it. It ended up not working well, for the reasons Jetstack warned about and that you raised, which is why we do not use it anymore.

@osterman prior to deleting it, you could mark it as deprecated. This would make it disappear from displays by default but still be discoverable.

I created an issue on https://github.com/artifacthub/hub/issues/542 for official versions of artifacts.

So we've gone ahead and deprecated the chart and will subsequently remove it.
Also, not sure if part of the concern is related to the maintainer field. We struggle with this because whilst we became the maintainer of this chart in our repo, we wanted to maintain attribution. We changed very little but wanted to credit the original maintainers so we left that there, even though we were the maintainers of this fork.
That said, to @mattfarina 's point, we don't see anything wrong with having alternate versions of the chart published; that is the open-source way. =) Blocking alternate charts would be very much against the ethos of open-source (and probably the law of open source licenses). More importantly, alternate chart versions provide field tests of alternatives to the "official" chart that the software maintainers can consider in their future plans without requiring them to make an immediate decision. Cloud Posse publishes several charts that we believe are "better" in some substantial way than the "official" charts or else we would not have bothered to create and publish them. Of course, "better" is subjective, and we do not try to persuade anyone to use our charts, but every chart we publish is one we have used successfully at some point in time, so obviously we see value in them as compared to the pre-existing charts.
While we always try to work with chart creators to have them update their charts, the process is often too slow when we are faced with an issue preventing us from using the existing chart right away on our customer engagements. In the case of the cert-manager chart, we had a philosophical difference of opinion, preferring an approach that Jetstack officially rejected, so that certainly was not going to be "merged upstream".
That said, we fully support efforts to highlight the chart published by the maintainers of the underlying software. We're grateful for the hard work Jetstack puts into the cert-manager project which we have relied on for years. Certainly, users should be made aware of who is responsible for the chart being published, and efforts made to prevent confusion over which chart has the support of the software developers. We also look forward to seeing improved means of attribution in chart manifests but oppose anything that would limit the openness of charts in the helm ecosystem.

@osterman You bring up a great point on the maintainer field on a chart. It's not about attribution. We should clarify the meaning and use of it somewhere. The maintainer field is for the point of contact of the maintainers of this instance. If there is an issue, such as a security problem, those are the maintainers to contact. In the case of a fork with changes, it's the forks maintainers.

Now that Helm Hub has moved to Artifact Hub, and we have an issue for this there https://github.com/helm/hub/issues/402#issuecomment-661045843, let's close this one.

We had issues using the upstream chart and reached out several times to negotiate with the upstream maintainer about it. They closed those issues, which left us with requirements that the upstream chart did not support. Now that helm and this chart have matured, the default cert-manager have finally met our requirements and the fork has been removed.
Related to the topic of alternate versions. There are a number of situations where maintainers are not interested in supporting or allowing through configuration to support client use cases. In CertManager's case, we had a requirement that the chart include all dependencies in the chart. I believe this is actually outlined in the original helm charts repo, helm install {chart} should work with default values. For cert-manager, this was not the case until fairly recently. We created a github repo that applied git patches to the Jetstack repo so helm install {chart} works. Thanks to changes in the original repo, we noticed that dependencies are now included by default (yay). So we closed our fork.
We had to make this alternate chart to satisfy our use case. The ability to publish that alternative chart so others could take advantage of the work is fundamental to OSS. Ideally, you can negotiate with the original maintainers so forking is not necessary. It is not always possible to do so and maintaining an internal fork costs nearly as much as publishing an open source one, sometimes it costs a lot more to keep the repo private. It's advantageous to all users and future users that forks are available. If you ask most OSS project founders, they started by loving a tool. Yet, that tool had just one small thing that they didn't like, so they forked or made a similar project that does this one small thing.
I have been on the other side trying to sort out which chart is the 'best version'. Grafana and Grafana operator are good examples of this. It can be confusing to sort out which one to use, but I don't expect Helm Hub to answer that question for me. As long as the alternative chart provides a maintainer contact, then a user can reach out for detailed questions about the nature of this alternative chart.


https://excubo-ag.github.io/Blazor.Grids/

demo site failing
blazor.webassembly.js:1 Error: Failed to start platform. Reason: TypeError: Failed to fetch
at St (blazor.webassembly.js:1:59395)
window.Module.s.printErr @ blazor.webassembly.js:1

I can't reproduce that on my device. Can you please try clearing your cache?

HI @stefanloerwald, no joy clearing cache,
showing verify integrity errors
blazor.webassembly.js:1          GET https://excubo-ag.github.io/Blazor.Grids/_framework/Microsoft.AspNetCore.Components.WebAssembly.dll net::ERR_CONNECTION_CLOSED 200
fiddler reveals a bit more
fiddler.network.https> HTTPS handshake to excubo-ag.github.io (for #8) failed. System.IO.IOException Authentication failed because the remote party has closed the transport stream.
it does smell like a local problem, I'll report back if I find anything else

Update Edge&Brave, both connection refused
Edge also shows
Error with Permissions-Policy header: Unrecognized feature: 'interest-cohort'.
/Blazor.Grids/:1
       GET https://excubo-ag.github.io/Blazor.Grids/ net::ERR_CONNECTION_CLOSED 200

I checked with yet another device and no issues for me... Not sure what I can do about this, sorry.

Closing since it's not reproducible on multiple devices.


🌐 Language request:

🌐 Language name (native)
Italiano
🌐 Language code
it-IT

Italian was added. Thanks for your contribution :)


Read 'browser' main field when target is 'electron-renderer'

What kind of change does this PR introduce?
bugfix
Did you add tests for your changes?
yes
If relevant, link to documentation update:
n/a
Summary
Fixes #6811
Does this PR introduce a breaking change?
no

@ooflorent was going to submit similar PR 😄
I guess the same check has to be added here as well, right?

This could also use a test: https://github.com/webpack/webpack/tree/master/test/configCases/target

Thanks for your feedback. I've also added an integration test.

@ev1stensberg Regarding the early return, I think it is less readable. If others want this change, sure, I'll do it!

@ooflorent no big deal, small picks 👍

Thanks

this PR cause issue
https://github.com/webpack/webpack/issues/7953


Adding ignore pets configuration to NPC Indicators

Summary
This change adds the option to exclude pets from NPC indicators

on (Default) - Exclude all pets from NPC Indicators.
off - Don't exclude any pets. This maintains parity with how NPC indicators currently works.

The reasoning for this is to exclude overloaded npc matches. For example, * dragon matches all dragons, but this includes Prince black dragon.
Screenshots
For the actual highlighting, I added *chaos*,*cano to my "NPCs to highlight" list and stood next to a random pet chaos elemental with my Smolcano.
Off

On

https://github.com/runelite/runelite/pull/15412
Done the same here also just noticed tho my build failed so just need to fix that rq

For changing only rebuild, I don't see how a method like NpcIndicatorsPlugin#onNpcSpawned would avoid highlighting pets if the change isn't made to that if. There is no rebuild called in that method so it doesn't seem like it would ensure that we don't add a pet to the highlights when they spawn in.
As for the pet names, I believe that the change as is would remove pets from the highlights and thus the names as well. I don't see much of a case where pets are being included in highlights but we want to specifically omit their names when we can just omit them wholly.

added in be2581a6b0a1cc12cb896d8de75dcaff6ba9662a


US2_a-T2: Scatter Plots

US2_a-T2: Create Scatter plot to visualize the relationships between different attributes
Estimated effort: 4

@AlbaGarciaRomo I have modified the title and description of this task


fix: correct misspells in pkg/koordlet/metriccache

Ⅰ. Describe what this PR does

Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[x] I have written necessary docs and comments
[ ] I have added necessary unit tests and integration tests
[ ] All checks passed in make test

/lgtm


[Fix]Modify pom to eliminate pb3 affect to master build

Protobuf3 update in protobuf file caused some methods gone, we fix this by pointing the HEAD of proto submodule to a pb2 version and don't let it synchronize with it's master but a fix commit.
Now mvn clean install should work in this PR.

This change is

Concerning issue is here.

@ilovesoup @zhexuany @birdstorm PTAL

Please verify it with a clean clone for both repo.

Also, did you update anchor for submodules?

Clean clone verified.


ssh connection to hosts

an ssh connection must be created for each host

Remaining Work

Start monitoring processes after connecting
Connect to assigned hosts after controller startup
Check other controllers

Status

Connection is established to assigned hosts on controller startup

but still not when the host is added, basically because the host needs to be first assigned and then the assigned controller needs connecting, so the inter-controller communication needs to be built first
no handling of controller failures
host connection issue (like host is down) kills the controller startup, error handling and assynchronous operation is needed here


#55 create class for database

Added a class with some methods that we can fill with functionality.
The class is meant to represent the database which holds a key and value.

Sync this with #44. Locking of database should probably be moved into database methods so the same lock can be used for the database.

Sync this with #44. Locking of database should probably be moved into database methods so the same lock can be used for the database.

Where should the lock be placed? Inside each function or in something like main (or the method) where we have a switch statement choosing what method to call?

Sync this with #44. Locking of database should probably be moved into database methods so the same lock can be used for the database.

Where should the lock be placed? Inside each function or in something like main (or the method) where we have a switch statement choosing what method to call?

Each method in database manipulating the map should have the lock code in them, surrounding any calls to the map.
And the Lock object can be created once in the constructor, because it must be shared so all threads tries to lock on the same object


Skip DefinitionFinder tests for the moment.

See https://github.com/Polymer/tools/issues/170

Merging before tests because this is obviously safe


v1.1.3

This repository contains GitHub workflows for managing releases automatically.
Please name this issue what you would like the new release to be called. E.x. "v1.0.0"
The following slash commands are available to use in comments on this issue:

Command
Description
Parameters

/release
Creates a new released based on the current state of the master branch
frontend=<version>backend=<version>gitapi=<version>status=<version>dispatcher=<version>agnosticv=<version>anarchy=<version>poolboy=<version>

/promote
Promotes the created release to the next environment
N/A

/cancel
Cancels the current release (and closes this issue)
N/A

/finish
Completes the current release (and closes this issue)
N/A

/release frontend=v1.0.6 backend=v1.0.7 gitapi=v1.0.4 dispatcher=v1.0.10 agnosticv=v0.1.2 anarchy=v0.10.4 poolboy=v0.3.7

/promote

/promote

/cancel


AppNotificationBuilder - Code Improvements

Addressing last round of feedback from: https://github.com/microsoft/WindowsAppSDK/pull/2786
Beside nits, one important issue that must addressed in this PR is retrieving the version number. Dev/Common should have a utility function to retrieve the version number. Let's make sure that whatever solution we used, it's the same for all WindowsAppSDK.
From @DrusTheAxe

Ugh. Version checks. That's a minefield (as in, a reliable answer is a minefield...)
Please use functions in dev\common\IsWindowsVersion.h e.g. WindowsVersion::IsWindows10_19H10OrGreater(). That has checks for >=19H1 and >=20H1, and if additional version checks are needed extend as appropos.
That avoids the minefield of GetVersion mechanisms which don't necessarily yield the expected answer

/azp run TransportPackage-Foundation-PR

/azp run TransportPackage-Foundation-PR

/azp run TransportPackage-Foundation-PR


⚠ Bofrost.lu has degraded performance

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

HTTP code: 200
Response time: 1713 ms

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


migrations mainly

Good. If
Sent from my Metro By T-Mobile 4G LTE Android Device
-------- Original message --------
From: Travis Moore<EMAIL_ADDRESS>Date: 12/3/20 10:09 PM (GMT-06:00)
To: FraxFinance/frax-solidity<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: Re: [FraxFinance/frax-solidity] migrations mainly (#8)
Merged #8https://github.com/FraxFinance/frax-solidity/pull/8 into master.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/FraxFinance/frax-solidity/pull/8#event-4070904796, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AREGYW5B4DPN74DYVMELQATSTBOJHANCNFSM4UM44HSA.


NativeStorage.clear()?

hey there
i think a window.NativeStorage.clear() would make sense to have. what do you think?
cheers

This is certainly possible. I will implement it when I've the time. Thanks for the good idea!

Added clear() function (with callback's) with commit 8250903b87de76af037745bb8680ac3b25e16d7f.
Version has been updated tot v2.0.2 and after one last test, it will be published to NPM.

Published on NPM<EMAIL_ADDRESS>


feat: introduce eager loading functions

What
This introduces eager loading functions that make use of the calamine's new DataTypeRef.
This prevents some allocations, resulting in a lower memory footprint.
Caveats

The API is kinda rough for now, it will probably need some cleaning (I mostly wanted to check if the memory gain was interesting here).

The functions need to be eager because DataTypeRef has an explicit lifetime, which is not allowed by PyO3 (lifetimes are hard to enforce on the python side: https://pyo3.rs/v0.20.0/class.html#no-lifetime-parameters)

In order for this to work, some changes are needed in calamine, and we don't know if this is something the library maintainers had in mind. PR and discussion: https://github.com/tafia/calamine/pull/390

Gains
While the speed stays roughly the same (it was even 3~5% faster on my machine on several tests), the memory footprint decreases by almost 25%. . This means that we're almost as good as pandas memory-wise :partying_face:  (they still beat us by a few MBs), while being about 10 times faster
Before

After

Pandas

Glad to see https://github.com/tafia/calamine/pull/409 has been merged. Hopefully we get a new release soon 👍

new data
main
import argparse
from time import sleep
import fastexcel

def get_args() -> argparse.Namespace:
    parser = argparse.ArgumentParser()
    parser.add_argument("file")
    parser.add_argument("-c", "--column", type=str, nargs="+", help="the columns to use")
    return parser.parse_args()

def main():
    args = get_args()
    excel_file = fastexcel.read_excel(args.file)
    use_columns = args.column or None

    for sheet_name in excel_file.sheet_names:
        arrow_data = excel_file.load_sheet_by_name(sheet_name, use_columns=use_columns).to_arrow()
        # sleeping to be really visible on the resulting graph
        sleep(1)
        arrow_data.to_pandas()

if __name__ == "__main__":
    main()

this branch
import argparse
from time import sleep
import fastexcel

def get_args() -> argparse.Namespace:
    parser = argparse.ArgumentParser()
    parser.add_argument("file")
    parser.add_argument("-c", "--column", type=str, nargs="+", help="the columns to use")
    return parser.parse_args()

def main():
    args = get_args()
    excel_file = fastexcel.read_excel(args.file)
    use_columns = args.column or None

    for sheet_name in excel_file.sheet_names:
        arrow_data = excel_file.load_sheet_eager(sheet_name)
        # sleeping to be really visible on the resulting graph
        sleep(1)
        arrow_data.to_pandas()

if __name__ == "__main__":
    main()

New benchmark looks great 😃

Good news, looks like we should be able to have lazy-by-ref once a new calamine version is out :partying_face:
Benchmarks with the latest version:

iterations
owned
by ref

1

20

calamine 0.25.0 should be released soon, meaning I should finally be able to finish this :slightly_smiling_face:  https://github.com/tafia/calamine/issues/435

iterations
master
this branch (lazy)
this branch (eager)

1

20


UPDATE ... FROM ( subquery ) only supported in PostgreSQL

Hi 👋
The UPDATE <table> FROM (SELECT ...) statement is only supported with PostgreSQL dialect.
It should be also supported for other technologies.
Here are a few examples:

BigQuery
Snowflake
Redshift
SQL Server

Thanks for the report. A contribution PR would be most welcome

I'll take a look.


TSC Meeting (13 Feb 2024)

February 2024 @StackStorm/tsc 1 hour meeting:

Tuesday, 13 February 2024, 09:30 AM US Pacific / 06:30 PM EU CET
See https://github.com/StackStorm/community/issues/33 for the schedule and how to join
TLDR; Jitsi link: https://meet.jit.si/StackStormTSC

Meeting Agenda
StackStorm v3.9.0
https://github.com/orgs/StackStorm/projects/31

Target: ????
Release Manager: TSC volunteer Needed!
Release Assistant: TSC volunteer Needed!

Add Support for MongoDB 5, 6, 7
https://github.com/StackStorm/st2/pull/6079
Progress update?
Assistance required?
Eventlet Retirement
https://github.com/StackStorm/st2/issues/6128
Eventlet team has announced its intention to retire eventlet.  They recommend migrating to Python's standard asyncio solution and have stated they'll provide a migration path for their users which is under active development.
Discussion on slack mentioned migrating to gevent or re-write using asyncio.

701 references to eventlet
193 files reference eventlet

41 files reference import eventlet

Release target: ?
Developer Lead: ?
StackStorm Exchange - Pack compatibility with Python3.9/3.10/3.11
With the addition of py3.9, 3.10 and 3.11 to StackStorm, the exchange packs will need to be tested for compatibility.  This will include any python code in the pack itself as well as module dependencies.
Release target: ?
Developer Lead: ?

TSC Minutes (13 Feb 2024)
Attendees

@nzlosh
@ravi
@mamercad
@khaddict
@jk464
@fuhrmannb
@cognifloyd
@winem
Dale Smith
and anothers - sorry, I wasnt able to get everyones names.

Agenda items
StackStorm v3.9.0
@amanda, @nzlosh, @khaddict and others have been making progess on the package builds for Ubuntu and Rocky as well as removing older unsupported distributions.
The CircleCI is failing for Rocky9 due to a newer version of Docker/Docker Compose being used.  @nzlosh is working on fixing this.
@mamercad has offered to help with updating the anisble deployment playbooks.  We still require someone to help with the puppet and oneliner install script.
Add Support for MongoDB 5, 6, 7
The pymongo depends on mongoengine so they have to be updated together, which involves updating function calls through the code base beyond those that were identified for pymongo.
@winem has offered to help with the mongo PR work.
Eventlet Retirement
ravi mentionned that @guzzijones has been looking into alternative concurrency solutions and we should reach out to get more information on what's being done.
gevent appears to be the easiest way forward but we will still watch what eventlet offers as a migration plan.
@cognifloyd mentioned the concurrency stub code that can be used.  It could be possible to re-write async code by component and upgrade StackStorm part by part by creating async and sync versions in parallel.
StackStorm Exchange - Pack compatibility with Python3.9/3.10/3.11
Packs need to be tested for compatibility but it is unknown who actively maintains to carry out this work.  To help determine if a pack has an active maintainer we will create an issue for each pack requests a response from the maintainer.
ravi requested a review for a PR outstanding for the SQL pack  which highlights the lack of pack maintainers.
Agenda additions
Pants
@cognifloyd continue to make progress on pants integration.  A draft PR is open to create a pants virtualenv with tests passing.
The PR must be have all tests passing with nose before migrating them to pytest.   Almost all tests are passing now.
Pack testing bug
@cognifloyd found a bug in the CI that has broken pack testing for at least a year. :(  When the tests were set up in Github actions, an environment variable was used for a logic test which cause pack tests to be skipped.
Website update
Dale Smith has a local setup running for the new web content stack

Astro https://astro.build/
Netlify https://www.netlify.com/ for content creation
Figma https://www.figcomponents.com/

There a question around how to share components.  Dale will set up an initial configuration and the rest of the community can provide feedback for adjustments if necessary.
Since the scope of this work has shifted from the initial issue https://github.com/StackStorm/community/issues/123, a new new issue wil be created to track progression forward.
Kubernetes/Docker PRs
@jk464 Has multiple Kubernetes and Docker PRs open to address security fixes.  These PR need to be reviewed by a maintainer with sufficient experience with Kubernetes/Docker.  It may be necessary to make some changes configurable as they may be specific to the
Next steps

@mamercad has opened https://github.com/StackStorm/st2/issues/6135 to fix the Pack testing bug
Maintainer needed to reivew Kubernetes/Docker PRs.

https://github.com/StackStorm/stackstorm-k8s/pull/401
https://github.com/StackStorm/stackstorm-k8s/pull/400
https://github.com/StackStorm/stackstorm-k8s/pull/399
https://github.com/StackStorm/stackstorm-k8s/pull/398
https://github.com/StackStorm/stackstorm-k8s/pull/397
https://github.com/StackStorm/stackstorm-k8s/pull/396
https://github.com/StackStorm/st2-dockerfiles/pull/66

Maintainer needed to review https://github.com/StackStorm-Exchange/stackstorm-sql/pull/30


QA - Site - Product Detail Page - Validation Message should be proper in name Field.

QA - Site - Product Detail Page - Validation Message should be proper in name Field.

Click on search button and go to search page.
Click on any product and go tot detail page.
Select Place Order tab and Select product Color.
Enter Qty in Qty box.
Select Shipping Address and Click on Address Book List Button.
Click on Add Address Button.
Click on Save Button Without Enter Any data in Fields.

Validation Message Updated.


Teams Column - ApiServer

Description
Frontend changes PR 120
Made changes to ProjectDao.java, Concise.java and ProjectResource.java to add Teams to the schema model. Also, added new query language to find/sort teams when clicked on.

Addressed Issue
Fixes issue 1435

Additional Details

Checklist

[x] I have read and understand the contributing guidelines
[ ] This PR fixes a defect, and I have provided tests to verify that the fix is effective
[ ] This PR implements an enhancement, and I have provided tests to verify that it works as intended
[ ] This PR introduces changes to the database model, and I have updated the migration changelog accordingly
[ ] This PR introduces new or alters existing behavior, and I have updated the documentation accordingly

@jmayer-lm You will need to update some tests in ProjectResourceTest since the response will now include additional fields, for example this one: https://github.com/DependencyTrack/hyades-apiserver/blob/b6c907445e15a521d02eee9e0541ccb53b4c02b3/src/test/java/org/dependencytrack/resources/v1/ProjectResourceTest.java#L315-L352
Separately, I think it would be good to have a test that covers searching by team.

@nscuro would ".formatted(team.getName()));" allieviate the 3 failed tests? Unsure if its a formatting issue as to why different outcomes are showing OR if it pertains to how the project is being created/accessed. Thoughts?

@jmayer-lm Two existing tests are failing now because the API responses now contain the teams field. You will need to extend the expected JSON for these tests:

getProjectChildrenConciseWithAclTest
getProjectsConciseWithAclTest

The build logs provide a detailed explanation of what's missing:
Different keys found in node "[0]", extra: "[0].teams", expected: <{"active":true,"hasChildren":false,"name":"acme-app-a","uuid":"${json-unit.any-string}"}> but was: <{"active":true,"hasChildren":false,"name":"acme-app-a","teams":[{"name":"Test Users"}],"uuid":"5125a66b-0969-495d-99f8-37ecca09d918"}>

So yeah, following the same .formatted approach as with the new tests should work.


Avoid large directories in the repo by creating subdirs with first two characters of image hash

Very large directories of images/thumbnails are possible after depositing. If we store by first two chars, we can avoid this.
@ElevnLi noted that an average_hash can produce hashes with similar prefixes. To avoid this, we can permute one imagehash.ImageHash into another hash with:
_permute_cache = {}

def permute_hash(image_hash):
    arr = image_hash.hash
    if len(arr) not in _permute_cache:
        _permute_cache[len(arr)] = np.random.RandomState(42).permutation(len(arr))
    arr = arr[_permute_cache[len(arr)]]
    return imagehash.ImageHash(arr)

We've implemented a prefix directory structure this for the otherwise-monolithic thumbnails volume, but without the hash being permuted, data distribution is very unbalanced: the majority of image hashes currently start with 0.


fix: setup when 'storyStoreV7' is enabled

What does this change?
This PR fixes behavior when storyStoreV7 option is enabled in Storybook v6.4 or later.
v7 Store requires a prior call to cacheAllCSFFiles before loading a story.
https://github.com/storybookjs/storybook/blob/e9e10ee1f19c57147464ca8d59a07a9ce7609525/MIGRATION.md#v7-store-api-changes-for-addon-authors
Screenshots
none
What can I check for bug fixes?
$ cd examples/storybook6-store-v7/
$ yarn test

References

https://github.com/reg-viz/storycap/pull/605

動く気配がないので閉じ


Instaloader extracts empty posts - please help!

Hello!
I am extracting instagram posts with a certain hashtag (f.e. "bachmannpreis") via the command line of my computer. I use following code:
instaloader --no-pictures --no-videos --no-compress-json --no-profile-pic --post-metadata-txt="{owner_username}\{caption}\{date_local}\{date_utc}\{likes}\{comments}\\\" --filename-pattern={date_utc:%d-%m-%Y}_UTC_{profile} “#bachmannpreis”
As you can see, I am extracting the caption in the metadata.
Everything works just fine, but I noticed that Instaloader extracted a large number of posts for this hashtag that contain no caption, although I am looking for posts with a caption containing a certain hashtag. These posts have no caption and therefore also do not contain the tag I am looking for.
I don't understand why these posts were extracted in my search for a specific hashtag if they do not actually contain this hashtag. Could someone explain why instaloader is extracting these wrong/useless posts? And how can I prevent this?
Thank you in advance!

Posts match that hashtag search if their caption or any of the comments contains the given hashtag. If you only want to include posts that have the hashtag in their caption you could do
instaloader ... --post-filter='"bachmannpreis" in caption_hashtags' '#bachmannpreis'


Question: How does HasComputedColumnSql work?

Using the example given in http://ef.readthedocs.io/en/latest/modeling/relational/computed-columns.html
I would've expected the projected SQL to be:
SELECT PersonId, FirstName, LastName, LastName + ', ' + FirstName as DisplayName FROM Persons
but the actual projected SQL is:
SELECT PersonId, FirstName, LastName, DisplayName FROM Persons
I hope I am not being dumb here. I'm just completely blank at the moment. Why do we even need to pass a SQL statement argument in the HasComputedColumnSql method if it's not used?
EDIT: Or is it only used for migrations?

It works the same way as SQL server's computed columns. You specify computed value for a column and that column value gets computed on insert or updated on edit automatically (you can't change it by hand).
So in your case projected sql is correct, because DisplayName is already calculated and exists in the database as LastName + ', ' + FirstName value.
It is used in migration code to create that kind of column, so you wouldn't have to fallback to altering tables with raw sql.

In addition to being used in Migrations, it tells the Update pipeline not to save any values for that column and to read back any new value for it after updating the row.

@bricelam it doesn't appear to be the case.
I'm using 2.0.0 and EF core tries to update the property I defined with HasComputedColumnSql .
I had to add [DatabaseGenerated(DatabaseGeneratedOption.Computed)] to the property to stop EF from trying to update the property.
Is that a bug? If not, at least the docs should make that clear.

Yes, if you can boil it down to a simple repro, please submit a new issue.

The method HasComputedColumnSql does not work as developers might expected. Official example https://docs.microsoft.com/en-us/ef/core/modeling/relational/computed-columns is wrong.

@art13g - Please file a new issue with repro steps, your expectations and current behavior.


Restore divide_by_vanishing_poly_on_coset_in_place

https://github.com/arkworks-rs/groth16/pull/40#issuecomment-1329664002
What do you think?

But the evaluation domain consists of roots of unity and because of this the vanishing polynomial is constant on the coset domain and we therefore have to evaluate it in only one point like it is done here: https://github.com/achimcc/groth16/blob/master/src/r1cs_to_qap.rs#L201
That should be way more efficient than a general polynomial division?

Since the vanishing polynomial consists of only two terms, general polynomial division is equally efficient.

My point probably was that you don't always have the polynomial in the monomial form


SQLite source url changes according to sqlite version

The download url for SQLite has the year of the release as part of the path. The makefile is using 2014 in SQLITE_URL and thus, to download a recent version you have to change SQLITE_VERSION and the year inside SQLITE_URL.
I think the documentation should mention this, or, have a way to automatically know the year that should be used in the url according to the value in SQLITE_VERSION.

:+1:


[FEATURE] Background image Post Page UI

Description

The current UI pof post page looks very basic and boring.
I would like to give it an attractive look.

Screenshots

Additional information
No response
Record

[X] I agree to follow this project's Code of Conduct
[X] I have checked the existing issues
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

Yes sure, you can work on this! :)


Handle URLs more robustly

code-server version: docker default version
OS Version: docker CE inside Debian

Steps to Reproduce

Establish reverse proxy via nginx with mapping https://xxxx/ide/ to http://localhost:8443/
The page showed GET // Failed.

I also tried other configurations. But this configuration seems to be the most correct. I guess the URL handler should be more robust to handle URLs like this.

Open PR #149


translation of alert when remove compare item in default theme

Bug report
Title
translation of alert when remove compare item in default theme
Issue Description
translation of alert when remove compare item in default theme
Preconditions
1. framework Version- master

Steps to reproduce
1. step1- In default theme, add item to compare & delete all item

Expected result
success alert should be translated
Actual result

fixed https://prnt.sc/tyxae9


Stream response for downloading big files

Is there any way to generate a method using the retrofit package that returns a Stream<Uint8List> so that I can download big files effectively?

Something like @Streaming annotation used in Kotlin would be great along with the ability to have a onReceiveProgress callback like Dio's get() method

I'd also love this feature.

I've found a way for this, not the most convenient but works:
  @GET('/api/documents/download/{id}')
  @DioResponseType(ResponseType.stream)
  Future<HttpResponse> getMyDocumentFile(@Path('id') int fileId);

And then the usage:
  Stream<Uint8List> getDocumentFile(int documentFileId) async* {
    final response = await _apiClient.getMyDocumentFile(documentFileId);
    yield* (response.data as ResponseBody).stream;
  }

@Albert221 Seems good to me for the moment, are you planning on developing a "more native" method?
Many thanks for the library!


The name 'str_ptr' does not exist in the current context (for UNITY_EDITOR || UNITY_STANDALONE_WIN)

When a package is imported, the error shows:
Assets\Scripts\TesseractWrapper.cs(195,58): error CS0103: The name 'str_ptr' does not exist in the current context
When "str_ptr" attribute is replaced with "stringPtr" in TesseractWrapper.cs it can compile and run in the editor when is selected "PC, Mac & Linux Standalone" platform.
`#if UNITY_EDITOR_WIN || UNITY_STANDALONE_WIN
  string recognizedText = Marshal.PtrToStringAnsi (str_ptr);

#else
  string recognizedText = Marshal.PtrToStringAuto(stringPtr);

#endif`

change str_ptr to stringPtr

change str_ptr to stringPtr

Already fixed in https://github.com/Neelarghya/tesseract-unity/commit/867cff44b458770eae131678525cb7651ebe9caa closing issue

Already fixed in https://github.com/Neelarghya/tesseract-unity/commit/867cff44b458770eae131678525cb7651ebe9caa closing issue