feat: add contributor's when editing a list

Description
Adds the add contributors page to the flow for editing a list.
What type of PR is this? (check all applicable)

[x] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents

Closes #1978
Mobile & Desktop Screenshots/Recordings
Successful Flow
https://github.com/open-sauced/app/assets/833231/ced46036-3337-4473-9d15-5fcef147f4ae
Flow with Error
https://github.com/open-sauced/app/assets/833231/f8be024f-bcf7-4460-8d7d-aa056e5c0576
Navigating to add contributors to a list you don't own results in a 404 page. Same if you are not logged in. e.g. navigating to one of my lists, https://deploy-preview-2016--oss-insights.netlify.app/lists/1cb6a034-c510-468d-b7c1-711588d88275/add-contributors should give you the 404 page.

Desktop

Mobile/Smaller Screens

When An Error Occurs

When Adding Contributors is Successful

Added tests?

[ ] 👍 yes
[ ] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[x] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

@nickytonline answered your comments on the issue, which includes a couple of small changes

@nickytonline answered your comments on the issue, which includes a couple of small changes

@isabensusan, I've made the changes and provide a couple videos in the PR description showing it in action.

thanks for this @nickytonline! A couple of minor style comments:
1. For the empty state image

Can we add some text below the image saying "Search for contributors to add to your list" ?
Can we make the empty state image width 300px? It'll look a bit more balanced I think

2. Can we make the text on this component "x contributors selected"  instead of just "x contributors"?

3. Can we make the header component's width the same as the other components on the page? (meaning the component with the list name, searchbar, timezone dopdown and add to list button)  Ideally it would all be aligned inside these red lines:

I think with those fixes this PR would be ready to ship. There is an edge case of users adding contributors that are already in the list, but we can create a separate issue for that and not worry about fixing it before shipping this PR.

thanks for this @nickytonline! A couple of minor style comments:
1. For the empty state image

Can we add some text below the image saying "Search for contributors to add to your list" ?
Can we make the empty state image width 300px? It'll look a bit more balanced I think

2. Can we make the text on this component "x contributors selected" instead of just "x contributors"?
3. Can we make the header component's width the same as the other components on the page? (meaning the component with the list name, searchbar, timezone dopdown and add to list button) Ideally it would all be aligned inside these red lines:
I think with those fixes this PR would be ready to ship. There is an edge case of users adding contributors that are already in the list, but we can create a separate issue for that and not worry about fixing it before shipping this PR.

I've made the requested changes @isabensusan and here's a quick clip of the grid and header lining up properly now.

Hey! Thanks for the changes. I left a couple of comments on the code, but overall my main concern is that the mobile view is looking a bit wacky.
Let me know what you think!

Sounds good @isabensusan. I'll go ahead and make those changes.

I believe I've captured all the changes in the latest commits @isabensusan. Let me know what you think. 😅

Test


Move jenkins templates to ods-core

They should not need to be customized and are better located in ods-core.

@michaelsauter  - this would require tailor to support upload-templates :)

FYI @kiwo

Not completely convinced yet. Because this will grow core with every new slave.
Probably depends what we define as core:

Everything that does not need to be customized (then it could be moved there)
Foundation for all further functionality provided by ODS (then I wouldn't put it there)

@rattermeyer I think there is a misunderstanding.
I want to:

have everything that is related to "project provisioning" in ods-core
have everything that is related to "component provisioning" in ods-project-quickstarters (which we could rename to ods-quickstarters)

That way we have a clean separation, and we pave the way to multiple "quickstarter repos", e.g. ones that only exist within an organisation on BitBucket with private quickstarters.
I am not proposing to have the slaves in core, just the master templates ...

Done as part of https://github.com/opendevstack/ods-core/pull/227, and further refinement is in https://github.com/opendevstack/ods-core/pull/241. The new ods-quickstarter repo does not have the templates anymore since https://github.com/opendevstack/ods-quickstarters/pull/3.


Request: Set 'foundry.toml' root to a folder deeper in workspace

Just stumbled upon this VSCode extension- 🤩✨
At the moment I'm working on a project where the Contracts are inside the monorepo at
/packages/contracts/
and the plugin is searching at
/
That would be a great setting to have.
Looking forward to using this more 🙏🍀
Thank you for your efforts.

Just stumbled upon this VSCode extension- 🤩✨
At the moment I'm working on a project where the Contracts are inside the monorepo at /packages/contracts/ and the plugin is searching at /
That would be a great setting to have.
Looking forward to using this more 🙏🍀 Thank you for your efforts.

Thanks for the feedback!
I made a new release (0.9.11) with improved root path support.
You can try to set either one of following:

solidity.project.root directly set (inner)root folder so in this case "packages/contracts"
solidity.project.monorepo if set to true, tries to find a foundry.toml and sets that directory as root

lmk if it does not work for you


Include reports from industry in assessment document

Agreed to at JTC Dec 2016 meeting. Was Appendix B for Canadian for 2016 assessment.
Two countries to assimilate separately.

I updated the Canadian one for this year in commit be4bef7d4139a4be511816f54421aa8b25989a99. I'll leave this open so we remember to check if the US AP is going to submit one.

Aaron to email US AP members.

Any chance we'll get this? It will make it easier to update Section 1.4.1 overview of the fisheries.

Response from AP member was that they will do their best but we shouldn't count on getting it in time to include in the pre-SRG document. I have a few notes from the Dec. 6 JTC meeting that I can hunt down to use in revising this section. At minimum, we can simply say that a report from the fishing industry was not available in time for this document so detailed information on the 2016 US fishery is not available at this time.

Let's create a placeholder appendix and say that this is forthcoming, in case it doesn't arrive before mid-day tomorrow (just so SRG knows that US is submitting something and that there is 'balance' with stakeholder input into assessment (via those appendices)....

placeholder created, but will leave this open until we get the report and insert it into the document - may be pre- or post-SRG meeting...

Thanks, it's a good idea to spur then to provide one.

we can close this now that the placeholder is in the document and it will be clear to insert it once it arrives... closing


Simplify swagger-ui updates by downloading on install

To simplify updates just download swagger-ui from github.
This currently works for npm. I don't know about bower et al.
It includes a patch so that no modifications on swagger-ui source necessary anymore and
you can just drop in another swagger-ui.
You could even think of always getting the latest swagger during install. But like that you can still check if it is working.

I'll update the testcases if you think this is an idea worth following.

any updates here @whitlockjc? not interesting?

I appreciate the work but I don't agree with the approach.  swagger-ui is included as a convenience and if you want to use a custom version, or a newer version, there is a simple middleware option to point to another swagger-ui location.  I realize the current approach makes it very easy for the swagger-ui shipped with swagger-tools to become very old but the same could happen with this PR because you will only install swagger-ui when you npm install swagger-tool.  So I'm not sure how this fixes the problem.
I'm not sure if you noticed but the middleware/swagger-ui/index.html file is custom to swagger-tools.  It alters the swagger-ui bootstrap process to get the Swagger document URL from swagger-tools via a simple HTTP call.  This PR would break that feature.
I'm all for figuring out a way to make things better so let's continue the conversation.

I thought about the the downloading as well and I'm not quite sure if it is the best idea to handle it that way. Because the environments are probably very different where the download should work. And if there is a way to download the npm package doesn't mean there is also a way to download the ui package as well.
Thats also why i didn't work on the testcases as well. Was more like a PoC. :-)
But there is a patch included that could be interesting anyway, and perhaps that would be a first start.
As you mentioned, the middleware/swagger-ui/index.html is patched. I reverted that patch and patched the way the UI is invoked by automatically adding a URL parameter to pass the swagger.json data. That way there is no need anymore to patch swagger-ui when upgrading.
https://github.com/apigee-127/swagger-tools/pull/416/files#diff-5c8932074d16e6b49a2248910f364c85R156
I could just pick those commits and make a new pull request for that.

I've updated swagger-ui to the latest as part of 9b5131361ec75712a6a0784b2e106d7664eb5b4f.  I kept the original index.html in repository as index.html.orig so if you can figure out some way to automate the downloading/patching of swagger-ui as part of download/install, I'm all ears.  For now, I'm closing this as the new work would likely be a new contribution.


Support stringified JSON in @TypedQuery

Currently, @TypedQuery isn't as flexible as @TypedBody, making it quite hard to pass complex queries which may require nesting to GET requests.
The Loopback project has something calling "stringified JSON" which addresses this problem. Supporting this, I believe would bring @TypedQuery and @TypedBody closer in terms of flexibility, and ultimately help me port an application from Loopback to NestJS.
Reference: https://loopback.io/doc/en/lb4/Querying-data.html

This is too domestic feature that not defined in the standard spec.

I think you’re missing the point. GET requests don’t have bodies. If you read through the Loopback documentation, you’ll see that it is spec compliant. This is an encoding mechanism ultimately.

I think you'd better to use other methods like POST.
If @TypedQuery() supports JSON typed property, it would be a great hazard due to 2K length limit of URL path.


Display a user-friendly message when deploying, but config has a TOML…

… errror

Intent

Resolves #2374
Type of Change

[x] Bug Fix

[ ] New Feature

[ ] Breaking Change

[ ] Documentation

[ ] Refactor

[ ] Tooling

Approach

Changed agent to return a JSON error which then was detected at the UX and was able to build an error string similar to what we show in our configuration warning.
There was a circular dependency that I had to unwind, which moved internal/services/api/api_errors.go to types}/api_errors.go. This affected the majority of the files within this PR. The important files relevant to the functionality is in internal/services/api/post_deployment.go
Screenshot of error

Automated Tests

Unit tests updated.
Directions for Reviewers

A few questions for the reviewers:

I am planning on updating the error message to contain a button to edit the configuration. Do you think that needs to be included in this blocking PR or can it come later?
You should notice this failure does not update the content-record for the last deployment. For this type of error, which we were previously blocking the deployment for, do you think it should?

Checklist

[ ] I have updated CHANGELOG.md to cover notable changes.

The latest commit addressed a few other ways a TOML parsing error was reported through the agent.
(To handle additional locations where TOML may fail, I added new errors tomlValidationError and tomlUnknownError.) Some errors will be shown with the line number, others will not (when not available). These will be shown when you click deploy.
List of invalid TOML syntaxes used for testing:

Title (line number shown)
Title = (line number shown)
Title = 123 (line number shown)
[Title] (no line number)
title = ['abc'] (no line number)
abc (line number shown)
title = 'abc' title = 123 (two lines... no line number)
type = 'python-fastapi22' (no line number)

Looks good.  Tested the suggested scenarios on MacOS, Windows and Linux AMD+ARM.


[New Feature]: Tournament feature

🎮 feature Request
feature logic and basic description : wants to add a new mode of tournament option where user 1 and 2 will play under a tournament of 3,5 . Can also create a leaderboard functionality where user can view total tournaments win based upon that user can gain points
Point down the features
feature points
1.Tournament feature
2. Leaderboard feature
3. Points wise
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Please assign this issue under gssoc 24

If you have any doubt you can directly send message on our discord (https://discord.gg/rZb46cCMmK), we have created seprate chennals for all projects...
Thank you :)

🎮 feature Request
feature logic and basic description : wants to add a new mode of tournament option where user 1 and 2 will play under a tournament of 3,5 . Can also create a leaderboard functionality where user can view total tournaments win based upon that user can gain points
Point down the features
feature points 1.Tournament feature 2. Leaderboard feature 3. Points wise
Select program in which you are contributing
GSSoC24
Code of Conduct

[x]  I follow CONTRIBUTING GUIDELINE of this project.

👋🏽 open to collaborate on this issue!

please assign me this issue

@ayush-t02 Please can you assign me this issue


STORM-1966 Expand metric having Map type as value into multiple metrics based on entries (1.x)

PR for master: #1560

populate metric before applying filter so that filter can play with populated metrics
add relevant configs to metrics consumer registration and describe them to storm.config.yaml
add unit test

Please refer https://issues.apache.org/jira/browse/STORM-1966 for more details.

@abhishekagarwal87 @harshach @ptgoetz Please review this as well. Thanks in advance!

+1


Tests for untested Ruby files, particularly those dealing with components

[ ] helper_component post-processor
[ ] associate_external_markup
[ ] pattern containers


URI.unescape is obsolete

This issue is a (choose one):

[x] Problem/bug report.
[ ] Feature request.
[ ] Request for support. Note: Please try to avoid submitting issues for support requests. Use Gitter instead.

Checklist before submitting:

[x] I've searched for an existing issue.
[ ] I've asked my question on Gitter and have not received a satisfactory answer.
[ ] I've included a complete bug report template. This step helps us and allows us to see the bug without trying to reproduce the problem from your description. It helps you because you will frequently detect if it's a problem specific to your project.
[x] The feature I'm asking for is compliant with the JSON:API spec.

Description
It's throwing warnings on Ruby 2.7.2 (jr 0.9.10)

2020-12-01T01:31:31.730709+00:00 app[web.1]: /app/vendor/bundle/ruby/2.7.0/gems/jsonapi-resources-0.9.10/lib/jsonapi/request_parser.rb:317: warning: URI.unescape is obsolete

per https://github.com/rails/rails/pull/39733 changing URI.unescape to URI::DEFAULT_PARSER.unescape should be fine

@bf4 Could you please close this issue since your PR is already merged 😃

Closed by https://github.com/cerebris/jsonapi-resources/pull/1340


Use .editorconfig when editing with VSCode

This repository contains .editorconfig file, which is configured for indentation.
However, VS Code, which users who want to contribute to this project may be using, does not read the .editorconfig settings by default, so it is highly likely that you will unintentionally use the wrong indentation.
How about adding .vscode/extensions.json to this project to recommend the EditorConfig.EditorConfig extension?

@2percentsilk Hi, is there any reason why this would not be done?


Show more symbols on profiling builds

This change turns on more symbols in profiling view of DevTools and thus partially solves #95.

The artifact sizes are unacceptably large. Embedders shouldn't have dependencies on the dart runtime.

@swift-kim Could you add -v option on Azure Pipelines?


rolling_process_data example still use too much RAM

Does the rolling_process_data example suppose to solve the problem of training data too large to be loaded into RAM? It fails at self._dump_pre_handler("pre_handler.pkl") because of the following error:
MemoryError: Unable to allocate 67.5 GiB for an array with shape (360, 50321774) and data type float32

@2young-2simple-sometimes-naive
Currently,  it will load the data into memory directly.  The easiest way to load the data is by expanding swap or RAM.
Thanks.


Command Issues

Description

Since the brush command was changed to the new format I no longer am quite sure how to use it. If you wouldn't mind adding example commands to the wiki or just giving examples here I would greatly appreciate that!
Versions

PocketMine version:
BlockSniper version: Latest/Self Compiled
Game version: PE Version 1.0.2

Crashdump and/or console errors

None

Yeah I don't understand the commands either it would be greatly appreciated if someone would upload an example command

Take a look at the Wiki :)

As of the changes, this is how you would make a sphere made out of stone and grass:
/b sh sphere (default already)
/b bl stone,grass
/b ty fill (default already)
You could also do this with the normal ways, which would look like this:
/b shape sphere (default already)
/b blocks stone,grass
/b type fill (default already)

Ohhh thanks!

If I were a youtuber I would gladly give you a tutorial, but too badly I'm not.

But you have TV at the end of you name :P (I know what it means, I'm just kidding xD)
I'll do a tutorial about it

But you have TV at the end of you name 📦

Damnit @HimbeersaftLP xD
And you would? Amazing :D

LOL, OutLook replaced my ":P" with a cardboard box in the notification mail of your reply xD

Lmao, let's not spam the issue too much :3

Did you get the information you needed @Mrfanman345? ^^

Yeah it makes A lot more sense now. Only thing is when I use the height variable (he) on cylinders it sets the height 1 more than I set it to but I'll make a separate issue for that.


Automatically collected Locations?

In some cases it won't be worth considering ignoring a Location. Examples in AV2 include map tiles; fast travel only works to seen save points, and also one win condition will include 100% of the Map revealed.
Hybrid exits are somewhat insufficient, since for in-area movement, it would result in a large amount of extra exits from adjacent spots. Plausibly this could obsolete hybrid exits (at the cost in some cases of introducing a spot in the middle, as for OoT's blue warps).
On the other hand, the current implementation doesn't ignore Locations until the minimization phase, so just having the Location (and the relevant dependency on the respective Item) might be sufficient.

We can decide not to try skipping locations with a time of 0, that's probably all we need for this.


CallbackUrl and redirects reloads application (does not use vue-router)

Environment
No response
Reproduction
No response
Describe the bug
It seems that every callbackUrl or redirect reloads whole page.
If module is developed for nuxt, it would be useful to redirect user (ex. during signIn signOut functions) without reloading whole page. Currently, we have to set { redirect:false } and use router.push() or navigateTo() functions to keep SPA context.
It is an issue especially if you are building an app where user logs in with custom modal (instead of default page provided by nuxt-auth) and you want to keep everything on one page with use of the reactive properties.
Additional context
No response
Logs
No response

Hey @mkierdev 👋
Please provide a reproduction of your issue, thanks!

closing due to inactivity, feel free to re-open this with a working, full reproduction of the problem

Hey @mkierdev
I get the same issue on my nuxt app. How did you solve it ?

Hey @BracketJohn
Dose nuxt-auth has a function for setUser ?

@AliBahadori41 I wish I could help, but after lot of thoughts, we decided not to use this package in our project.

@mkierdev
Which one do you use for your project ?
Share links please.


build dev failed error when starting ionic serve

Short description of the problem:
When starting ionic serve i get this output
ionic-hello-world@ watch /Users/justinwillis/Projects/ionicBeer
ionic-app-scripts watch

[09:53:46]  ionic-app-scripts 0.0.37
[09:53:46]  watch started ...
[09:53:46]  build dev started ...
[09:53:46]  clean started ...
[09:53:46]  clean finished in 25 ms
[09:53:46]  copy started ...
[09:53:46]  transpile started ...
[09:53:53]  transpile finished in 6.99 s
[09:53:53]  webpack started ...
(node:27030) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: [object Object]
[09:53:53]  copy finished in 7.38 s
[09:54:00]  webpack finished in 7.25 s
[09:54:00]  sass started ...
[09:54:00]  build dev failed: [object Object]
(node:27030) PromiseRejectionHandledWarning: Promise rejection was handled asynchronously (rejection id: 1)
[09:54:00]  watch ready in 14.49 s  ```

Note: this does not happen when using rollup

**Steps to reproduce:**
1. clone https://github.com/jgw96/ionicBeer
2. cd into project, run `npm install`, and run `ionic serve`

**Which @ionic/app-scripts version are you using?**
latest master branch. I also get this with the latest `@beta` release.

This has been fixed in the latest beta release.


feat: [EX-3265] format tests for prebid.js adapter

@cpabst Can you create a PR in the Prebid.js repository from this branch?

@maximpakhomov You bet. I'll do that this morning.

PR was merged to the prebid.js repo.


Stormfox causes sunlight to look very bright

Using Stormfox 2 alongside Real CSM causes sunlight to look very bright and white.
I haven't done too much testing, but it might have been because of the env_sun update since I did not encounter this issue before that.
Here are some example screenshots on a sandbox version of Lost Coast map:

CSM without Stormfox

CSM with Stormfox, time of day 10:00 AM
Setting a time in Stormfox after around 4PM will look fine though, but results may vary depending on map

CSM with Stormfox, time of day 5:00 PM
NOTE: I didn't encounter this in Construct or Flatgrass, so this is probably a custom map-only issue.

Do you have hdr enabled in your video settings? this is a HDR thing i haven't accounted for. you should have it on btw

I do have HDR on

try change the stormfox brightness multiplier cvar


[WIP] Responsive display

This patch is intended for grouping a number of UI responsiveness issues.

[X] Version string wrapping in header
[X] Navigation sizing issues at mid-ranges
[X] Search bar/button grouping
[X] Dashboard hero image overlap with text
[X] Navigation visibility in mobile
[ ] Sidebar visibility in mobile
[x] Navigation toggle should expand on click when in collapsed mode

@ezimuel You can track progress as I work on this via the checkboxes. I may add more items as I explore the UI in more detail.

@ezimuel I've got a ton of issues resolved now. The last two, however, are big: the collapse toggle is not making the navigation menu dropdown appear, and at a certain point, the sidebar disappears entirely, and cannot be found. I'll work on those when I can.

Down to the last item: sidebar visibility when on narrow screens.

All set!

@weierophinney the new responsive design looks good, there are only some minor issues:

the search field in the sidebar shows the X button, to reset the search content, over the Search. Moreover the size of the search input text is too long, it's not aligned with the size of the angular-ui-tree (see image below)

the menu at the top of the screen and the size of the main content is not 100% width of the window. I think this should be fixed because folks can take advantages of full screen with pages with a lot of contents, like the Fields page for REST/RPC services (see image below)

for very small window, the delete service button is wrong positioned in the panel-heading (it's over the div). In general all the buttons in the panel-heading have the same issue (see image below)

the menu at the top of the screen and the size of the main content is not 100% width of the window

I had to make this change due to problems when resizing the window. Essentially, when we had the content justified, elements would move to a line below, making the header larger and obscuring content in the sidebar and content area. Additionally, it meant it was really hard to calculate exactly when the menu button should appear, as justified == taking entire width. It was far simpler to achieve this by removing the justification.

the search field in the sidebar shows the X button, to reset the search content, over the Search. Moreover the size of the search input text is too long, it's not aligned with the size of the angular-ui-tree (see image below)

Now fixed!

for very small window, the delete service button is wrong positioned in the panel-heading

Also fixed! Note: if you size really, really small, it's still a slight issue if the button and text cannot fit in the same line. However, I tried on a variety of sizes down to original iPhone resolution, and it's working correctly in each.

Finally got the sidebar behavior I want: the toggle now appears in the header, on the left, and toggles between left and right chevrons. This is ready to merge. :)


Add cpp chrono project for sim

See #35 for more information. For visibility: @ishaanamahajan @harryzhang1018

See #35. Closing.


feat: Allow installing and testing with all database types, including MariaDB

(Draft: This has to be rebased after

https://github.com/justafish/ddev-drupal-core-dev/pull/33
https://github.com/justafish/ddev-drupal-core-dev/pull/31

This PR allows using any database type supported by DDEV, including MariaDB and MySQL. And of course it can still use sqlite.
It currently includes unmerged PRs because drush is required for it.
I guess it still needs tests, will wait until after rebase.

@rfay Could you rebase this against main please? There are a lot of conflicts and I don't know what I'm doing with the code involved.

Ah drat, I hadn't seen this depended on #33.

I don't know if in my fork I want to support both styles of installation, or just using my Composer project template. I was initially thinking I'd support both, if it's not too much faff.

This is rebased against

https://github.com/justafish/ddev-drupal-core-dev/pull/33
which was already up-to-date.

But of course it still depends on #33.

I don't know if in my fork I want to support both styles of installation, or just using my Composer project template. I was initially thinking I'd support both, if it's not too much faff.

All I care about here is using a "normal" database type, and having drush available. I'm pretty sure your template is a fine way to do that, true?

Yup, but what I mean is if I drop support for running this directly in a Drupal core clone, then none of #33 is needed.

Again, any way you get drush works for me :)


Tilde syntax should not expand inside quoted code

If a Gen model includes a tilde expression in a quoted expression, then it is incorrectly replaced with @trace:
@gen function f()
   expr = :(x ~ normal(0, 1))
   return expr
end

f()
# => :(x = @trace(normal(0, 1), :x))

# Output should be: :(x ~ normal(0, 1))

I didn't realize MacroTools.postwalk would send us the insides of quoted expressions as if they weren't quoted! (Or maybe the problem is MacroTools.@capture being overly broad, and matching quoted expressions?)
This suggests that other places we use MacroTools.postwalk (e.g., in the @dist DSL) may have the same bug.

I think postwalk descends into quotes, yes! So yeah, we have to be careful -- I think the static modelling language will have these issues as well. I think we could try and fix this together with #222 in PR #223.

Addressed by #223.


Split password entry from login process (for password manager support)

Done! But need to make the form look nicer.


Host throws exception when initialized multiple times by JS workers

The node-api-dotnet native host throws an exception if its initialize() function is called multiple times in the same process:

.NET is already initialized in the current process. Initializing multiple .NET versions is not supported.

Normally a JS require() will return the same object if called more than once for the same module path. But multiple initializations can happen for either of two reasons:

Multiple .NET TFMs are requested in the same process, e.g. require('node-api-dotnet/net6.0') and require('node-api-dotnet/net7.0').
The module is required (with the same path) from multiple JS workers. Each worker initializes the module separately.

The first case is a valid error, since, as the error message says, multiple .NET versions in the same process are not supported.
But the second case should work. The native host can check if the same .NET version is requested and if so the re-initialization can be allowed. Then the managed host should use a separate assembly load context for each worker.

Just encountered this while calling load just once in a github action.
I suspect the github environment has an already runing .Net session which doesn't even let us initalize this module once.

My dependency graph was actually calling the load function more than once

That should be fine. Subsequent calls to load the same assembly file will just immediately return the already-loaded assembly object.
After looking into this further, I'm not sure my analysis above of the root cause was correct. We actually have a test case that loads .NET from multiple workers in a process, and it works. I'm unable to reproduce the error (except using private APIs that you wouldn't be calling).
Can you share:

The version of node-api-dotnet package you're using
Portion(s) of your JS code that use that package - note I do not have access to Office org private repos.

So I have replicated my issue in a public repo (https://github.com/rijulg/node-api-dotnet-issue), unfortunately even the simple example is a bit involved as the bug I encountered was in a project using nestjs so bear with me. I have enapsulated all my project requirements in a devcontainer for running locally, so it should be easy to run locally if you need to.
Essentially we see the following in github actions (but not when running locally):

Jest runs the src/repositories/DLL.spec.ts test which loads the dll
Jest runs the src/controllers/Example.spec.ts test which has a dependency on the repositories
2.1. Nestjs loads the dependency internally and for that it creates an instance of DLLRepsository
2.2. A new instance of DLLRepository tries to load the dll and fails with the following error:

.NET is already initialized in the current process. Initializing multiple .NET versions is not supported.

You can see the issue over at this github action report:
https://github.com/rijulg/node-api-dotnet-issue/actions/runs/4696432987/jobs/8326519102

This is very likely https://github.com/nodejs/node/issues/6624, or at least related to it. I happen to have investigated an issue for another project a couple years ago that I traced to Node.js mixing up the drive letter case in the context of a test runner, which led to some modules being loaded multiple times when they shouldn't be. (So I know it's not fully fixed in Node.js.) And there are comments from others specifically mentioning it is a problem with jest.
#95 should fix it.

At least the error changed: https://github.com/rijulg/node-api-dotnet-issue/actions/runs/4697994667/jobs/8329728914
I am getting the following error now:

Object reference not set to an instance of an object.

while running the following in second test
import { load as loadDLL } from "node-api-dotnet";

Still the error is just on github actions and not locally which is weird.

The NRE is fixed by https://github.com/microsoft/node-api-dotnet/pull/96
I was able to repro and test it better this time.

Yup, the new fix works
https://github.com/rijulg/node-api-dotnet-issue/actions/runs/4699042432/jobs/8332048634
Waiting on merge so that I can update my dependncies to the main branch. Thank you so much Jason!


RuntimeError: cuda runtime error (2) : out of memory

So finally I was able to get rid of all pytorch version related problems, and at least I can see the network structure when I execute now. However, for some strange reason I always end up with the specified error.:
THCudaCheck FAIL file=/pytorch/torch/lib/THC/generic/THCStorage.cu line=58 error=2 : out of memory
Traceback (most recent call last):
  File "train.py", line 381, in <module>
    model = model.cuda() # model = torch.nn.DataParallel(model, device_ids=[0]).cuda() # Multiple GPU parallelism
  File "/home/user/.local/lib/python2.7/site-packages/torch/nn/modules/module.py", line 216, in cuda
    return self._apply(lambda t: t.cuda(device))
  File "/home/user/.local/lib/python2.7/site-packages/torch/nn/modules/module.py", line 146, in _apply
    module._apply(fn)
  File "/home/user/.local/lib/python2.7/site-packages/torch/nn/modules/module.py", line 146, in _apply
    module._apply(fn)
  File "/home/user/.local/lib/python2.7/site-packages/torch/nn/modules/module.py", line 146, in _apply
    module._apply(fn)
  File "/home/user/.local/lib/python2.7/site-packages/torch/nn/modules/module.py", line 152, in _apply
    param.data = fn(param.data)
  File "/home/user/.local/lib/python2.7/site-packages/torch/nn/modules/module.py", line 216, in <lambda>
    return self._apply(lambda t: t.cuda(device))
  File "/home/user/.local/lib/python2.7/site-packages/torch/_utils.py", line 69, in _cuda
    return new_type(self.size()).copy_(self, async)
RuntimeError: cuda runtime error (2) : out of memory at /pytorch/torch/lib/THC/generic/THCStorage.cu:58

I've tried various GPUs such as Titan V, 1060ti, 1080ti, no luck.  It can't be the case that I need more than 11 GB memory to run the training session, right? Validation is OK, it runs. Training however, no matter which card I tried, does not run. One should perhaps decrease the batch size? Which config file should one edit then? yolo-pose-pre.cfg or yolo-pose.cfg? Or both? Before I change any of these I wanted to ask if that's actually the reason.

The reason is multithreading，You should modify the worker thread to 0.specifically is
utils.py:
options['num_workers'] = '0'

I did that, now it complains about some other file(s) though:
2018-07-11 15:44:14 epoch 0, processed 0 samples, lr 0.000100
THCudaCheck FAIL file=/pytorch/torch/lib/THC/generic/THCStorage.cu line=58 error=2 : out of memory
Traceback (most recent call last):
  File "train.py", line 401, in <module>
    niter = train(epoch)
  File "train.py", line 91, in train
    output = model(data)
  File "/home/eren/.local/lib/python2.7/site-packages/torch/nn/modules/module.py", line 325, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/eren/workspace/singleshotpose/darknet.py", line 91, in forward
    x = self.models[ind](x)
  File "/home/eren/.local/lib/python2.7/site-packages/torch/nn/modules/module.py", line 325, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/eren/.local/lib/python2.7/site-packages/torch/nn/modules/pooling.py", line 143, in forward
    self.return_indices)
  File "/home/eren/.local/lib/python2.7/site-packages/torch/nn/functional.py", line 334, in max_pool2d
    ret = torch._C._nn.max_pool2d(input, kernel_size, stride, padding, dilation, ceil_mode)
RuntimeError: cuda runtime error (2) : out of memory at /pytorch/torch/lib/THC/generic/THCStorage.cu:58

The batch setting should not be greater than 16, otherwise OOM will still appear.

Do you know which config files to edit then? I see already 32 and 64 as batch sizes in config files I could spot now over a minute of looking. Are these the only files?

My gpu is k80,OOM will appear when I set the batch to be greater than 16.so the batch is not set too big，The modified configuration file is: yolo-pose.cfg. Num_worker must be modified

There is no variable called Num_worker in that config file though, I think you confused it with utils.py.
Nevertheless, here is what one finds in yolo-pose.cfg:
[net]
# Testing
batch=32
subdivisions=8
# Training
# batch=64
# subdivisions=8
height=416
width=416
channels=3
momentum=0.9
decay=0.0005
angle=0
saturation = 1.5
exposure = 1.5
hue=.1

So I've changed batch there to 8 and 16, and got the following errors, respectively:
Traceback (most recent call last):
  File "train.py", line 396, in <module>
    niter = train(epoch)
  File "train.py", line 91, in train
    loss = region_loss(output, target)
  File "/usr/local/lib/python2.7/dist-packages/torch/nn/modules/module.py", line 477, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/user/workspace/singleshotpose/region_loss.py", line 205, in forward
    pred_corners[0]  = (x0.data + grid_x) / nW
RuntimeError: The size of tensor a (13) must match the size of tensor b (1352) at non-singleton dimension 3

Traceback (most recent call last):
  File "train.py", line 396, in <module>
    niter = train(epoch)
  File "train.py", line 91, in train
    loss = region_loss(output, target)
  File "/usr/local/lib/python2.7/dist-packages/torch/nn/modules/module.py", line 477, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/user/workspace/singleshotpose/region_loss.py", line 205, in forward
    pred_corners[0]  = (x0.data + grid_x) / nW
RuntimeError: The size of tensor a (13) must match the size of tensor b (2704) at non-singleton dimension 3

first: Num_worker is in utils.py，not in cfg
second:Batch is set in cfg
third：pytorch version must be 0.3，Can't be version 0.4

Okay I got it, works with lower batch_size indeed. Not 16 though, 8 worked for me. Also, it must be stated that I had to deallocate the memory before.
So if anyone else faces this, set the batch_size to lower values such as 8/16, and make sure the GPU is not occupied.
@hhaAndroid  thanks BTW.

With Pascal Titan X, a batch size of 32 works for me. Just make sure that you have enough memory by checking (sudo) fuser -v /dev/nvidia*. If you see some running processes deallocate memory by killing them.


The-Talker

Is your feature request related to a problem? Please describe.
"The Talker" is a Chrome Extension to narrate texts from a webpage through speech. Even a selected text from a webpage can be specifically narrated. It works on text-to-speech principle.
Additional context
PLease assigned it to me.

goo for it


Domain not showing up in account confirmation email

Summary
I just signed up to content curation (https://contentworkshop.learningequality.org) and received the following email:
Hello Ivan,

Here is your activation link for Kolibri Studio:

None/accounts/activate/Iml2Yzzxxxxmci:1dCtL0:KdzzzzzMJ-Tma0C8I/

This link is valid for 7 days.

Thanks for using our site!

The Learning Equality Team

Note the None instead of domain.
Cateogry
BUG

To followup on this I did a little test with FF, Chrome, and Safari and none of them seem to set the HTTP_ORIGIN header.
I believe you can get the host name using from django.contrib.sites.models import Site then Site.objects.get_current().domain.
Another option is this request.get_host(). Since neither of these come with the protocol prefix, you should add https:// to the front.

Fixed: https://github.com/fle-internal/content-curation/pull/328


lib - incorrect struct matching?

So, I was trying to make a SDL2 wrapper, but I found something interesting when linking to SDL2 library. SDL's events are a C unions, so I made a struct Event in Crystal, and put each type of event in the struct, so it's look exactly like in SDL, with the same types, names, order.
When I wanted to process a Keyboard event, it filled the events whose comes before KeyDown event(CommonEvent, WindowEvent) with the Keyboard events data. So e.g. WindowEvent's padding1 member contained KeyboardEvent's state member.
I suppose it's because the WindowEvent and CommontEvents are null on a KeyboardEvent, so it (SDL I think) thinks that the first byte to write is happens to be CommonEvent's data.
I hope I could properly write down the problem

Use a union to bind a union: http://crystal-lang.org/docs/syntax_and_semantics/c_bindings/union.html

https://github.com/weskinner/crystal-sdl2/blob/master/src/lib_sdl2.cr#L158
https://github.com/TheBlueCrane/SDL2-Crystal/blob/master/src/lib_sdl2.cr#L195
https://github.com/farleyknight/crystal-sdl/blob/master/lib/sdl_main.cr#L149
https://github.com/Arcnor/crystal-raw-sdl2/blob/master/src/libsdl2.cr#L911

Thanks, I totally forgot about union type.


Make components headless for Password-input component

Description
Migrate styles from components to themes.
Since component styles in themes inherit styles from other components using mergeStyle etc., duplicate styles should be removed.
Problem Statement/Justification
Move static styles from password-input component to theme.
Proposed Solution or API
No response
Alternatives
No response
Are you willing to participate in implementing this feature and create a pull request include the implementation?
Yes
Additional Information
No response

Closed due to no migration required.


added more to describe CSS tools in the intro

first PR on open source!

[ x] I have read freeCodeCamp's contribution guidelines.
[x ] My pull request has a descriptive title (not a vague title like Update index.md)
[ x] My pull request targets the master branch of freeCodeCamp.
[ x] None of my changes are plagiarized from another source without proper attribution.
[x ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you for opening this pull request.
This is a standard message notifying you that we’ve reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Thank you and happy coding.


HOS : Provide Backend Registration for Bucket Management

Issue/Feature Description:

To Provide Backend Registration .
Use config based Backend addition to limit the feature for the supported Clouds [Iter-3 scope : AWS , Azure, GCP]

- with the available resources, it is possible to support AWS , Azure , GCP

- Other clouds / backends will be done in the next release.

Why this issue to fixed / feature is needed(give scenarios or use cases):
How to reproduce, in case of a bug:
Other Notes /  Environment Information: (Please give the env information, log link or any useful information for this issue)

Done in Isabela Release


feat: support for both HTTP and gRPC protocols in meter/logging/tracing

/claim #1090
/close #1090
Add support for both HTTP and gRPC protocols in meter/logging/tracing in meter/logging/tracing exports

Summary by CodeRabbit

New Features

Introduced a new Protocol configuration field for log, tracing, and metrics exporters, allowing users to specify communication protocols (HTTP or gRPC) for enhanced data transmission.
Enhanced exporter creation functions to accept the protocol parameter, increasing flexibility in configuring log, tracer, and meter exporters.

Bug Fixes

Improved error handling for unsupported protocol inputs in OTLP exporter functions, ensuring clearer feedback to users.

Documentation

Updated documentation to reflect changes made to configuration options and exporter functionalities.

@tolgaOzen Please review this PR. Thank you.

Now I have fixed the inconsistency reviewed by coderabbitai

@tolgaOzen , There are two falling test case: vercel and Coverage / Test with Coverage. is it from my side?

hello @vikashsprem , thank you for your contribution 🚀


Session recording

Hi, Is session recording available? if yes where can we find it.

Hi, for the limited hands-on session format, there is no recording available.
Participants can go through the exercises at their own pace based on the description in this repository.
In the hands-on session, the hosts were just supporting them in case of questions.
The exercise description is detailed enough so that also participants that weren't registered for the hands-on can go trough them. In case of questions, you can raise a new issue here which will be answered by our experts.


https with arn from aws certificate manager not working

NOTE: GitHub issues are reserved for bug reports only.
For anything else, please join the conversation
in Kong Nation https://discuss.konghq.com/c/kubernetes.

Summary
I want deploy Kong/kubernetes-ingress-controller on aws with own https certificate from aws certicate manager.
https service is online but certificate is invalid
Common Name (CN)	localhost
Organization (O)	Kong
Organizational Unit (OU)	IT Department
my service is:

apiVersion: v1
kind: Service
metadata:
name: kong-proxy
namespace: kong
annotations:
# Cloud-provider specific annotations
# GKE
# GKE creates a L4 LB for any service of type LoadBalancer
# TODO figure out how to enable Proxy Protocol on an L4 LB for GKE
# AWS
# Use NLB over ELB
service.beta.kubernetes.io/aws-load-balancer-type: nlb
# Use L4 LB so that Kong can do TLS termination
service.beta.kubernetes.io/aws-load-balancer-backend-protocol: tcp
# Enable Proxy Protocol when Kong is listening for proxy-protocol
service.beta.kubernetes.io/aws-load-balancer-proxy-protocol: ''
# replace with the correct value of the generated certificate in the AWS console
service.beta.kubernetes.io/aws-load-balancer-ssl-cert: "arn:aws:acm:us-west-2::certificate/*"
# increased to '3600' to avoid any potential issues.
service.beta.kubernetes.io/aws-load-balancer-connection-idle-timeout: "3600"
spec:
externalTrafficPolicy: Cluster
type: LoadBalancer
ports:

name: kong-proxy
port: 80
targetPort: 8000
protocol: TCP
name: kong-proxy-ssl
port: 443
targetPort: 8443
protocol: TCP
selector:
app: kong

Kong Ingress controller version
0.5.0
Kong or Kong Enterprise version
kong
Kubernetes version
Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.1", GitCommit:"4485c6f18cee9a5d3c3b4e523bd27972b1b53892", GitTreeState:"clean", BuildDate:"2019-07-18T09:18:22Z", GoVersion:"go1.12.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"13+", GitVersion:"v1.13.10-eks-5ac0f1", GitCommit:"5ac0f1d9ab2c254ea2b0ce3534fd72932094c6e1", GitTreeState:"clean", BuildDate:"2019-08-20T22:39:46Z", GoVersion:"go1.11.13", Compiler:"gc", Platform:"linux/amd64"}
Environment

Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

What happened
Expected behvaior
Steps To Reproduce

Custom https://bit.ly/kong-ingress  with arn certificate

Please refer to: https://kubernetes.io/docs/concepts/services-networking/service/#ssl-support-on-aws on how to configure TLS termination at ELB.


Fix sitemap opportunity

The suggestions are split into two parts:
Errors concerning fetching sitemap or robots file:
{
   opportunityId: 'id',
   type: 'REDIRECT_UPDATE',
   rank: 0,
   data: {
      type: 'error',
      error: 'NO VALID URLs FOUND IN SITEMAP',
   }
}

Errors concerning urls in the sitemap:
{
   opportunityId: 'id',
   type: 'REDIRECT_UPDATE',
   rank: 0,
   data: {
      type: 'url',
      sitemapUrl: 'https://some-domain.adobe/sitemap.xml',
      pageUrl: 'https://some-domain.adobe/foo',
      statusCode: 404,
   }
}

:tada: This PR is included in version 1.45.4 :tada:
The release is available on:

v1.45.4
GitHub release

Your semantic-release bot :package::rocket:


Add keyword shortcut support for date entries

How cool would it be to be able to enter dates like today or 3 months ago for quick entry of dates based on sentence logic! Not sure how easy or feasible this would actually be though.. could be a real can of worms with all the possible ways folks may want to enter those dates. Something worth looking into though.

[ ] today, now
[ ] 1 year ago, 3 months ago, 5 weeks ago, 50 days ago  (takes into account the current day of selection)


Update spring and gradle

BACKEND PULL REQUEST
Related Issue

This is in lieu of https://github.com/CDCgov/prime-simplereport/pull/6705

Changes Proposed

Added Rins security comments
Update to spring 2.7.16
Update to gradle 8.3

updated binaries are found here https://start.spring.io/

LGTM! Thanks for getting this in.


Section and panels don't have style

Hi great project, I really like it.
The section and panels don't have a defined style, and make it difficult to make a nice dashboard.
Maybe you can add or improved the scss on this part, for now I have added custom scss for the panel (from the activeadmin repo)
Here is the scss:
.panel {
  border: 2px;
  margin: 10px 0;
  border-radius: 3px;
  background-color: whitesmoke;
  box-shadow: inset 0 1px 4px #ddd;

  >h3 {
    background-color: #efefef;
    background-image: linear-gradient(180deg, #efefef, #dfe1e2);
    text-shadow: #fff 0 1px 0;
    border: solid 1px #cdcdcd;
    border-color: #d4d4d4;
    border-top-color: #e6e6e6;
    border-right-color: #d4d4d4;
    border-bottom-color: #cdcdcd;
    border-left-color: #d4d4d4;
    box-shadow: 0 1px 3px rgba(0,0,0,0.12), 0 0 1px #fff inset;
    font-size: 1em;
    font-weight: bold;
    line-height: 18px;
    margin-bottom: 0.5em;
    color: #5E6469;
    padding: 5px 10px 3px 10px;
  }

  .panel_contents {
    padding: 10px;
  }
}

Yes it's a good idea, i will check that code 👍


corrections/updates to news contributions of each variable

Corrected how updates are calculated to variables specified by store_idx. In the previous version this incorporated only changes to contemporaneous factors. In this version, when store_idx is low frequency (as in the us_gdp.R example), the gain and thus news contribution of each series incorporates updates to lagged factors as well.

Codecov Report

Merging #53 into master will decrease coverage by 0.25%.
The diff coverage is 40%.

@@            Coverage Diff             @@
##           master      #53      +/-   ##
==========================================
- Coverage   59.69%   59.43%   -0.26%
==========================================
  Files           8        8
  Lines        1424     1425       +1
==========================================
- Hits          850      847       -3
- Misses        574      578       +4

Impacted Files
Coverage Δ

R/dfm_core.R
59% <40%> (-3.63%)
:arrow_down:

Continue to review full report at Codecov.

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


Absolute beginner's guide to install this?

Apparently this software is intended to be used by people who are already experts with node.js. I have never used it before.
I have no idea what I am doing. I am typing commands that are twiddling unknown settings deep in the operating system, and I don't like this at all. I don't want to end up having to format and reinstall because of unknown borkage that might be occurring due to trying to make this work.
Can you please break down the install process for this in as few steps as possible? I am using Ubuntu 16.04.
Steps so far.
sudo apt-get install nodejs
sudo apt-get install npm
sudo npm install -g discord-irc
.... this magically does "something", and it somehow knows what "discord-irc" is and doesn't tell me how it knows what it is. I don't know what's going on here. I might be installing malware. No idea. Fails.
└─┬<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY erlpack@hammerandchisel/erlpack
├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>└── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>Okay. Um. now what. Google. Maybe auto-install can fix up whatever is needed?
sudo npm install -g auto-install
Install succeeded
Try discord-irc again. Nope, still fails on dependencies.
Well let's try those one at a time.......
sudo npm install -g<EMAIL_ADDRESS>Succeeds
sudo npm install -g<EMAIL_ADDRESS>Fails, bunch of error messages.
sudo npm install -g<EMAIL_ADDRESS>Succeeds
sudo npm install -g<EMAIL_ADDRESS>Succeeds
sudo npm install -g<EMAIL_ADDRESS>Fails.
?? Google search, someone suggests make sure package "build-essential" is installed.
sudo apt-get install build-essential
"This is already installed. Install type changed to manual."
Um... ok...
Still doesn't work. Another google search... apparently Ubuntu doesn't make a file named "node" in /usr/bin.
sudo ln -s /usr/bin/nodejs /usr/bin/node
Try again..
sudo npm install -g  bufferutil
Succeeds
sudo npm install -g node-opus
Succeeds
sudo npm install -g opusscript
Succeeds
sudo npm install -g sodium
Fails.
sudo npm install -g libsodium-wrappers
Succeeds (?????)
Try again.
sudo npm install -g sodium
Fails.
Is there some ubuntu package I need to use instead or something?
apt-cache search sodium
sudo apt-get install libsodium18
Succeeds.
sudo npm install -g sodium
Fails
Try developer package instead.
sudo apt-get install libsodium-dev
Try again.
sudo npm install -g sodium
Well..... it isn't the same error as before but now I get a whole pile of "make" errors.
Google has no answers to these. I just find pastebins without comments.
Try this whole thing over again...
sudo npm install -g discord-irc
?? I still get a list of dependency errors of things that supposedly installed.
└─┬<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY erlpack@hammerandchisel/erlpack
├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>└── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>Try uws again.
sudo npm install -g<EMAIL_ADDRESS>Succeeds.... but.... it says
/usr/local/lib
└──<EMAIL_ADDRESS>So it actually installed a newer version of uws, but apparently discord-irc is demanding an older version that I don't know how to install. And sodium still is not working either.
Also I've now done tons of crap to Ubuntu with sudo, and with no idea how to back out of all this.

So, you installed a bunch of things with apt-get – sudo makes your commands privileged (like running in admin mode), apt-get is the operating system package manager.
You've also installed a bunch of things with npm, the Node package manager (see initials), which fetches packages from npmjs.com. Packages are uploaded there, and can be seen there (discord-irc here, individual files not clearly visible).
I have no idea what auto-install is. I don't think it does anything to npm installations. Peer dependencies, or at least these ones (it may not hold for the general case), are optional. Most of these are from the upstream discord.js library, which is the large snippet of code that we don't maintain that we rely on to communicate with Discord. I suppose we could, as they have, put a note in the README that unmet peer dependencies are optional. You did not need to modify anything, from what I can tell, to get it to work past that point.
To back out of this, you could look into the documentation on npm. This can be done, relatively unsurprisingly, with npm help or npm --help. It lists a bunch of commands you might find useful. For example, uninstall. The -g flag makes it apply globally, instead of to the folder you're currently looking at. (It's called a flag because it modifies the behavior, and is indicated by the initial dash; this is a standard thing with lots of UNIX command-line tools, UNIX including Linux and macOS and certain other derivatives.) You may want to go through and remove the npm packages you manually installed, presumably with sudo npm uninstall -g [package-name] (remove the square brackets).
You can also remove the various Ubuntu packages, with apt-get. I'm not on Ubuntu, so I can't be sure, but it looks like you want remove – that is, sudo apt-get remove [package] – for the packages you don't need. You don't need, as far as I know, any of the ones you installed past your first attempt to install discord-irc, which appears not to have failed, but instead just mentioned unmet peer dependencies.
This software is intended, I believe, to be used by people who have passing familiarity with node.js, seeing as it uses node's package manager. I'm not sure if you're coming from a Windows background, here, but you could do this on Windows and you might feel more comfortable there, as node is multi-platform. I do not believe npm tweaks deep internal system settings except insofar as it modifies your path and presumably adds configuration options places that configuration options are usually added.
I'd recommend going through and uninstalling (with npm and apt-get) the packages you don't need (those that you went through and added), and then try using the command discord-irc --help to make sure it responds properly. You should then be able to proceed with the instructions as given.
Summary: it looks like you took the unmet peer dependencies as required, when they're actually not; we should probably add this to the README.

The install might have completed successfully - try running discord-irc. If not you probably need to upgrade your Node.js version, as apt's default version is quite old: https://nodejs.org/en/download/package-manager/#debian-and-ubuntu-based-linux-distributions

Okay, I found how to delete everything and start over, and to update the node.js. I also managed to make all the warnings go away, except it is still complaining about a missing "package.json" and I don't know what to do next.
================
(Removed Everything, cleaned up /usr/lib  /usr/bin ...)
curl -sL https://deb.nodesource.com/setup_8.x | sudo -E bash -
sudo apt-get install nodejs
... (update to 8.x nodejs)
npm install discord-irc
... (fails, unicode/ucsdet.h: No such file or directory)
sudo apt-get install libicu-dev
npm install discord-irc
. . . (it gets past the unicode error, but still has problems)
npm WARN saveError ENOENT: no such file or directory, open '/home/foobar/discord-irc/package.json'
npm WARN enoent ENOENT: no such file or directory, open '/home/foobar/discord-irc/package.json'
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of erlpack@hammerandchisel/erlpack but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN discord-irc No description
npm WARN discord-irc No repository field.
npm WARN discord-irc No README data
npm WARN discord-irc No license field.
... (Try clearing the warnings, don't know if this helps)
npm install uws
npm install bufferutil
npm install node-opus
npm install opusscript
npm install erlpack
... (? It doesn't like this one)
npm install erlpack@hammerandchisel/erlpack
... (Warning goes away for this one goes away.)
npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>npm install sodium
... (fails, it wants "libtool")
npm install libtool
npm install sodium
... (fails, it doesn't want npm libtool apparently)
sudo apt-get install autotools-dev
npm install sodium
... (fails, doesn't seem to help)
sudo apt-get install autoconf
npm install sodium
... (fails, doesn't seem to help)
sudo apt-get install libtool
npm install sodium
... (FINALLY sodium install succeeds)
=======================
But I still don't know what I need to do next.
~/discord-irc$ npm install discord-irc
npm WARN saveError ENOENT: no such file or directory, open '/home/foobar/discord-irc/package.json'
npm WARN enoent ENOENT: no such file or directory, open '/home/foobar/discord-irc/package.json'
npm WARN discord-irc No description
npm WARN discord-irc No repository field.
npm WARN discord-irc No README data
npm WARN discord-irc No license field.

<EMAIL_ADDRESS>updated 1 package in 1.356s
=========================

If I go to ~/discord-irc/node_modules/discord-irc   I see this:
:~/discord-irc/node_modules/discord-irc$ ls -glot
total 188
-rw-rw-r-- 1   2514 Oct  1 09:12 package.json
drwxrwxr-x 2   4096 Oct  1 09:12 dist
-rw-rw-r-- 1   8130 Aug 31 02:41 CHANGELOG.md
-rw-rw-r-- 1 163776 Aug 31 02:41 package-lock.json
-rw-rw-r-- 1   4400 Jul 18 23:45 README.md
-rw-rw-r-- 1   1094 May  6  2016 LICENSE
So..... I don't know what this "package.json" it is still complaining about is for.
Let's copy package.json  and put it in ~/discord-irc and try again.
~/discord-irc/node_modules/discord-irc$ cp package.json ../..
~/discord-irc/node_modules/discord-irc$ cd ../..
~/discord-irc$ npm install discord-irc
npm ERR! code ENOSELF
npm ERR! Refusing to install package with name "discord-irc" under a package
npm ERR! also called "discord-irc". Did you name your project the same
npm ERR! as the dependency you're installing?
npm ERR!
npm ERR! For more information, see:
npm ERR!     https://docs.npmjs.com/cli/install#limitations-of-npms-install-algorithm
???

You don't have to install the other dependencies. Install node.js, then run only npm install -g discord-irc (note the -g for global). Then you should be able to run discord-irc --config config.json.

Another problem, npm has wrong permissions access, even when run with sudo.
I am looking into it . . .
https://docs.npmjs.com/getting-started/fixing-npm-permissions
sudo npm install -g discord-irc
/usr/bin/discord-irc -> /usr/lib/node_modules/discord-irc/dist/index.js

><EMAIL_ADDRESS>install /usr/lib/node_modules/discord-irc/node_modules/iconv
> node-gyp rebuild

gyp ERR! configure error
gyp ERR! stack Error: EACCES: permission denied, mkdir '/usr/lib/node_modules/discord-irc/node_modules/iconv/build'
gyp ERR! System Linux 4.4.0-91-generic
gyp ERR! command "/usr/bin/node" "/usr/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /usr/lib/node_modules/discord-irc/node_modules/iconv
gyp ERR! node -v v8.6.0
gyp ERR! node-gyp -v v3.6.2
gyp ERR! not ok

Is there a difference between installing discord.js or node.js? I've been using discord.js only and have been getting it to run successfully.

Uh. I don't think that's possible, since discord.js is a library that's written in node.js, relying on node.js, and does not replace node.js?
The language used is JavaScript, and this runs here on node.js, and discord.js is a library written in JavaScript to keep all the Node ↔ Discord communication in a standardized block of library code, which is then used by this program to actually do things around Discord.
I can only assume you already had node.js installed, or somehow managed to install it while installing discord.js? (Like it says on discord.js's readme, "Node.js 6.0.0 or newer is required.")

Yeah, I do already have node.js installed on my own computer, but I also tried it on Cloud9, G Cloud and Azure, and I could install discord.js without installing node.js first.

To install it, you would've had to have used npm? And then to run this bot you would need Node, and npm relies on Node being installed.
So if you installed discord.js, I am rather certain you had node.js installed?

I think you're right then. I guess most cloud services just have node.js ready.

Followup comment (this can stay closed).
I was able to get this working, though the npm permission thing is weird.  Running "sudo npm install -g discord-js" somehow manages to fail.
"npm" is running as root and it doesn't have write permission? It appears npm is screwed up in some manner if sudo doesn't give it everything it needs.
I have no idea if npm itself is a script or a linux executable in order to try to analyze this permissions problem.
 Root owns all that stuff in /usr for a very good reason.
In the end I made a new custom directory for global nodejs stuff named "/usr/nodejs" and set that as the default npm install path. So now npm -g works without sudo, which still seems incredibly dumb for a global install for all users. . . but this is still better than setting /usr/bin to owned by my account. Derp.


Project dependency aliases don't resolve in editor

Trying to leverage Pkl projects with the IntelliJ Plugin. When using project dependencies, the plugin loses all context on the dependencies, even after running pkl project resolve.
Plugin Version 0.26.0
IntelliJ IDEA 2023.3.4 (Ultimate Edition)
Build #IU-233.14475.28, built on February 13, 2024
Runtime version: 17.0.10+1-b1087.17 aarch64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
macOS 14.3.1

The red underline for ObjectMeta says "Unresolved reference: ObjectMeta"
If I replace @k8s with its url from PklProject, everything looks good.

Thanks for the great work on this to date!

I tried looking into this myself but not really sure where to start. It would be a big help if this could get some attention

Apologies; missed this. The plugin knows how to hold the CLI, so you should never have to pkl project resolve to inform the plugin. When you open the corresponding PklProject file, do you see a top-bar offering you to Sync Pkl project (as in attached screenshot)? If so, click that Sync and revisit your file.

Doh! That was it! Thank you!


[1.12] Issue Title Placement of the Laser drill in a particular direction is impossible

This mod needs a wrench that can rotate in whatever direction you need. placing the laser drills is the most infuriating thing ive ever experienced in my entire life id rather hang myself by my foreskin than try and place this block. So heres the issue.  SUCH AS FORWARDS.
wan to replicate this? try and place a block in every which direction and i garuntee you will not be able to get it to rotate in 2 of the four directions. This is not the first time ive had this issue and trying to collect ether gas is the most painful thing in existence to do considering this issue. If i didnt need this mod for the atm star id outright delete it. PLEASE HELP

👋 Sorry but Industrial Foregoing has ended support for version 1.12. I'm closing this issue as it will no longer be updated for that version. Thanks for playing with the mod.


Drill 6294: Update Calcite version to 1.16.0

Left the same three Drill-specific commits from the last upgrade to Calcite 1.15.
Also, this pull request contains cherry-picked commits from Calcite master: [CALCITE-2205] & [CALCITE-2201]

+1


Create dev only create user page

We can skip using the CLI if we make a dev-only create a user page.
To close this issue:

Create a "Create User" Page that only shows up in the dev environment.

we will use the command line instead


Upgrade to DJL 0.25.0

Upgrade notebooks to DJL 0.25.0.

I would suggest to close the PR to update each notebook in individual PRs

Upgrading dependencies is not as simple as version bump up. You need to carefully reevaluate the content of each notebook and see if some parameters need to be added/removed.
For some examples, currently vLLM no longer needs to install HuggingFace anymore.

I did reevaluate the content of each notebook. For example, in tnx_rollingbatch_deploy_llama_7b_int8.ipynb, option.load_in_8bit=true was replaced with option.quantize=bitsandbytes8.
I did not know that vLLM no longer needs to install HuggingFace anymore. What else do I need to know? For reference, I have only been referencing "[GitHub] LMI Containers parameters after V7 release" to see what has changed.

Upgrading dependencies is not as simple as version bump up. You need to carefully reevaluate the content of each notebook and see if some parameters need to be added/removed.
For some examples, currently vLLM no longer needs to install HuggingFace anymore.

I did reevaluate the content of each notebook. For example, in tnx_rollingbatch_deploy_llama_7b_int8.ipynb, option.load_in_8bit=true was replaced with option.quantize=bitsandbytes8.
I did not know that vLLM no longer needs to install HuggingFace anymore. What else do I need to know? For reference, I have only been referencing "[GitHub] LMI Containers parameters after V7 release" to see what has changed.

for tnx, option.load_in_8bit should work and also option.quantize. Both should work because we support backward compatibility

Yes, but the document says option.load_in_8bit is deprecated since 0.25.0


Problems with subscriptions

On the client side I use this:
client.log.add({ type: 'logux/subscribe', name: 'user/10' })

it fails and suggests to use sync, so I updated to:
client.log.add({ type: 'logux/subscribe', name: 'user/10' }, {sync: true})

and now it's rejected by the server as using unknown type here.
I thought it may be a bug, but there is spec treating this behavior as intended.
What am I doing wrong? Is there any example of successful subscription from client to server?

Strange O_o.
Can I ask you (since you can repeat issue) to debug this code block: https://github.com/logux/logux-server/blob/master/base-server.js#L174-L200
If you didn’t find anything, please put throw new Error('') here and give me stack trace.

I put console.log before if (action.type === 'logux/subscribe') { as my very first debug attempt and it was never reached. Only then I found that it's rejected by ServerClient filter even before going to that code in base-server.
I will provide you a stack trace as soon as I can.

unknownType (action, meta) {
    throw new Error(action.type)

 FATAL  logux/subscribe at 2017-08-29 09:51:17
        at KetchupServer.unknownType (/Users/mateuszderks/workspace/ketchup-server/node_modules/logux-server/base-server.js:588:11)
        at ServerClient.filter (/Users/mateuszderks/workspace/ketchup-server/node_modules/logux-server/server-client.js:254:16)
        at /Users/mateuszderks/workspace/ketchup-server/node_modules/logux-sync/messages/sync.js:61:31

Thanks. I will look after few hours.

Thanks for testing :).
I found source and fixed it fe53f5e

It works, thank you!


Integrate ledger stats generation

Code from somethingpretentious:
chessstatmodule.txt

https://github.com/michael1241/lichess4545-stats for up to date program.
Integration even lower priority now that tnan has it running too.


504 Gateway Time-out when we want to GET link after the draft export airlock request is created

Describe the bug
In version v0.7.0 which is currently on our DEV environment, we get 504 Gateway Time-out every time when we:

create a draft export airlock request:

GET/api/workspaces/{workspace_id}/requests/{airlock_request_id}/link:

And, this is the log from App Service:

After that, the environment is unavailable for a while with 502 and 504 gateway errors.

Steps to reproduce
Explained above.

Hello @tamirkamara, @marrobi
This is a blocker for us currently.
Could you please help us to resolve this issue?

@tamirkamara, @marrobi
For the import airlock requests we are getting the link, this issue is happening only when we want to do an export.

@tamirkamara, @marrobi
After digging more into app service logs we can see that connection to the airlock export storage account within the workspace is timing out:

You can see the full log here:
connection-time-out-storage-account-stalexintws99b6.txt
Could you pelase advise?

I believe this has been fixed in a more recent update -https://github.com/microsoft/AzureTRE/pull/3215
Sometimes the private link DNS records were not being created correctly.
@tamirkamara think this is the same issue?

@tamirkamara We have just applied the changes proposed by @marrobi in this comment.
However we still have the timeout problem.
I uploaded the full log in this comment.
Log_Default_Docker.txt

@migldasilva do you still hit this? It was resolved, and required changes detailed in the release notes. Thanks.

@marrobi We haven't noticed this issue for a while. Thanks for the clarification, I'll review the release notes.

OK, I'll close the issue, if you see it again, let us know.


md-grid-list, md-grid-tile can't find the way how to align items to left/right #1212

Bug, feature request, or proposal:
iits closed without to say how to do it.
I cat align the text within colums to the left
What is the expected behavior?
What is the current behavior?
What are the steps to reproduce?
Providing a Plunker (or similar) is the best way to get the team to see your issue.
Plunker template: http://plnkr.co/edit/o077B6uEiiIgkC0S06dd
What is the use-case or motivation for changing an existing behavior?
Which versions of Angular, Material, OS, browsers are affected?
Is there anything else we should know?

i did it like this,with a div and a class
.text-inside-grid {
position: absolute;
left: 5px;
}
probably not the best way,but i don't know the standard way within md-grid

any news about this issue?

@yuriyovych
Just came across this one. Solved it by wrapping the mat-grid-tile content with div and setting it to width: 100%; display: flex; align-items: flex-start;


How to solve error (-255 Assertion failed)

face_recognition
Python version:3.9
Operating System: Windows

Description:
OpenCV(4.5.2) C:\Users\runneradmin\AppData\Local\Temp\pip-req-build-kuwfz3h3\opencv\modules\objdetect\src\cascadedetect.cpp:1689: error: (-215:Assertion failed) !empty() in function 'cv::CascadeClassifier::detectMultiScale'
IMPORTANT:

And please do guide me for this warning message too.
Thanking you!

Were you able to solve this ?


Toonily stopped functioning like normal

Source information
Toonily version 1.4.18
Source language
English
Steps to reproduce

Click on browse
Click on Toonily source
No results are shown for latest or browse
Click on arrow next to latest to move into source
Http 500 error is given with error check in webview
Click check in webview
Error connecting to database
Click 3 dots to check via browser
Opens fine via browser

Expected behavior
When clicking into source manga should be shown.
Actual behavior
Error when trying to view through source or saved manga.
Mihon/Tachiyomi version
Tachiyomi R561
Android version
Android 13
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.15.3.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

I am having this error now.


[WIP] OSS Detection and Registration

Ok, so here's the first pass at this. Super simple.
The first quirk I've noticed is that it doesn't have a way to distinguish between NuGet packages we build ourselves vs. packages we depend on. So it wants to register Microsoft.AspNetCore.* as OSS. Now, we could just let that happen and say it's Internal-only to avoid things, or I can investigate ways to suppress them from the output (which may need changes in the tool itself). Thoughts @natemcmaster @Eilon ?
TODO:

[ ] Decide on how to handle our packages
[ ] Change scanType to Submit to actually produce the registrations
[ ] Change governanceProduct to the real one we plan to use

So it wants to register Microsoft.AspNetCore.* as OSS.

Can we give this feedback to the authors of this plugin? I'm afraid this is going to be unbearably noisy, to the point that I'm not sure using their automation is the right fit for the job.

to the point that I'm not sure using their automation is the right fit for the job.

I'm not convinced it goes quite that far. You're not the one who has to manually register our NPM packages ;P

Another option would be to turn this on for NPM packages only. We can configure the specific detectors we want to run.

Sounds like this behavior may be by-design and by-policy now.

@anurse are we good to merge?

I'll rebase and then yes, we should be good.


Node.js functions start execution before modules have been installed

When using continuous integration to deploy an Azure Function written in node.js, when you include a package.json with a set of modules that need to be installed, function execution start before this 'npm install' completes.
Repro steps
Use the canonical event hub sample to repro ala http://github.com/timfpark/ah-eh-test with an Event Hub that has events in it that will immediately fire.

Set app setting AzureWebJobsEventHubReceiver to the connection string for this Event Hub.
Setup continous integration to deploy app from Github repo.

Expected behavior
Azure Functions installs modules before starting execution.
Actual behavior
Azure Function fails a number of times with "can't find module" errors until 'npm install' completes:
2016-12-13T18:32:05.462 Function started (Id=561f215f-4eb8-4510-9d64-5c45215677c0)
2016-12-13T18:32:05.772 Function completed (Failure, Id=d4f35507-c676-42d9-993d-2a7cdc10327e)
2016-12-13T18:32:05.788 Function completed (Failure, Id=d8df8c8d-9238-4589-a605-231fb5f106b3)
2016-12-13T18:32:05.804 A ScriptHost error has occurred
2016-12-13T18:32:05.804 Error: Cannot find module 'async'
at Function.Module._resolveFilename (module.js:455:15)
at Function.Module._load (module.js:403:25)
at Module.require (module.js:483:17)
at require (internal/module.js:20:19)
at Object. (D:\home\site\wwwroot\featureIntersection\intersect.js:3:15)
at Module._compile (module.js:556:32)
at Object.Module._extensions..js (module.js:565:10)
at Module.load (module.js:473:32)
at tryModuleLoad (module.js:432:12)
at Function.Module._load (module.js:424:3)
2016-12-13T18:32:05.804 A ScriptHost error has occurred
2016-12-13T18:32:05.804 Error: Cannot find module 'async'
at Function.Module._resolveFilename (module.js:455:15)
at Function.Module._load (module.js:403:25)
at Module.require (module.js:483:17)
at require (internal/module.js:20:19)
at Object. (D:\home\site\wwwroot\featureIntersection\intersect.js:3:15)
at Module._compile (module.js:556:32)
at Object.Module._extensions..js (module.js:565:10)
at Module.load (module.js:473:32)
at tryModuleLoad (module.js:432:12)
at Function.Module._load (module.js:424:3)
2016-12-13T18:32:05.823 Function completed (Failure, Id=d3000565-6478-49b7-a109-42a8d59be9f1)
Known workarounds
Problem eventually resolves itself when modules are installed.

Thanks. Your repo link above doesn't resolve. Are you using your own custom deployment script to perform the NPM restore?


setState() called after dispose()

I have made use of the widget in a form field when a user is registering for a new account like so
class PasswordValidator extends StatelessWidget {
  final TextEditingController controller;
  final Function() onSuccess;
  const PasswordValidator({
    Key? key,
    required this.controller,
    required this.onSuccess,
  }) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Padding(
      padding: const EdgeInsets.only(
        left: 20.0,
        top: 0.0,
        right: 20.0,
        bottom: 20.0,
      ),
      child: FlutterPwValidator(
        width: MediaQuery.of(context).size.width,
        height: MediaQuery.of(context).size.height * 0.2,
        minLength: 6,
        specialCharCount: 1,
        uppercaseCharCount: 1,
        numericCharCount: 1,
        onSuccess: onSuccess,
        controller: controller,
      ),
    );
  }
}

During the API call, I set the page to be replaced with a ProgressIndicator. When the API calls fails, the user is kept on the same page and the ProgressIndicator is removed and text form fields are reset, in resetting the controller value that the password field is bound to, the following error is thrown
The following assertion was thrown while dispatching notifications for TextEditingController:
setState() called after dispose(): _FlutterPwValidatorState#85b38(lifecycle state: defunct, not mounted)

This error happens if you call setState() on a State object for a widget that no longer appears in the widget tree (e.g., whose parent widget no longer includes the widget in its build). This error can occur when code calls setState() from a timer or an animation callback.

The above error is also thrown anytime the password field is tapped or interacted with at all.

Would it be possible to include the page code as well?

Hey @ArefMozafari and @lbk9
I will help you with this Error and update this library according to your concern.

Great

Sorry for ghosting this, github never notified me of any replies to this...
I think the setState in the widget state may just require a mounted check.

That's fine. Waiting to see @Money1998's solution before I fix this.

@ArefMozafari issue Resolved, please merge code I have created one pull request


EXC_BAD_ACCESS in StaticMetadataInitCanary

What version of gRPC and what language are you using?
Object-C
pod 'BoringSSL-GRPC', '0.0.8'
pod 'Protobuf', '3.12.0'
pod 'abseil', '1.20200225.0'
pod 'gRPC', '1.29.1'
pod 'gRPC-Core', '1.29.1'
pod 'gRPC-ProtoRPC', '1.29.1'
pod 'gRPC-RxLibrary', '1.29.1'
What operating system (Linux, Windows,...) and version?
Mac 12.4
Xcode Version 13.4.1
What runtime / compiler are you using (e.g. python version or version of gcc)
grpc_objective_c_plugin in 1.48.0
What did you do?
Please provide either 1) A unit test for reproducing the bug or 2) Specific steps for us to follow to reproduce the bug. If there’s not enough information to debug the problem, gRPC team may close the issue at their discretion. You’re welcome to re-open the issue once you have a reproduction.
grpc_init_static_metadata_ctx method make g_static_metadata_slice_ctx NULL and got a EXC_BAD_ACCESS crash in StaticMetadataInitCanary method

What did you expect to see?
run normally
What did you see instead?
EXC_BAD_ACCESS
Make sure you include information that can help us debug (full error message, exception listing, stack trace, logs).
See TROUBLESHOOTING.md for how to diagnose problems better.
Anything else we should know about your project / environment?

I can reproducing the bug in this demo https://github.com/OYQ/grpcTest

run pod install and open grpcTest.xcworkspace
run target "grpcTest"


Command to open the open file dialog with a specific folder

I would like the workbench.action.files.openFile command to take a parameter of the folder that should be initially shown in the file picker.
This could be used in keybindings or from extension API.
Currently it is showing the folder of the active editor.
It could be a separate command but I think it can be an optional parameter on the existing command.
Are you accepting PRs on this?

Code reference
https://github.com/microsoft/vscode/blob/d00e0f9f74e69991955f5e203bd50f5ff39e6245/src/vs/workbench/browser/actions/workspaceActions.ts#L50
https://github.com/microsoft/vscode/blob/b1eab983e40dfdca90d0f9833b00afe4233c12c3/src/vs/platform/dialogs/common/dialogs.ts#L336
https://github.com/microsoft/vscode/blob/b1eab983e40dfdca90d0f9833b00afe4233c12c3/src/vs/platform/dialogs/common/dialogs.ts#L104

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


can't insert image

Hi @YunyunZhai to image upload works you will need to configure a server, if you share more details, maybe I could help.

Hellor @YunyunZhai, here there are a working example of a rails app as api rest.
https://github.com/nossas/bonde-server/blob/develop/app/controllers/uploads_controller.rb
If you still have some misconception about how create your own server code, let me know.


3.5.rc1 Editor window fail to close

Godot version
v3.5.rc1.official [e86d840d4]
System information
Windows 7 & 10, GLSE3
Issue description
Closing the Editor causes the window to stop responding, this also occurs when Godot switches from the Project Manager to the Project workspace (the project manager window stops responding and fails to close), this wasn't present with 3.4.4.stable and 3.5.beta5. Other than that it appears that the engine is functioning as expected so far. The reason to this problem is probably a memory leak on the loaded resources.
here is the output with --verbose flags.
D:\Programes\Godot>
Godot Engine v3.5.rc1.official.e86d840d4 - https://godotengine.org
Using GLES3 video driver
OpenGL ES 3.0 Renderer: GeForce 9500 GT/PCIe/SSE2
Async. shader compilation: OFF
OpenGL ES 2D Batching: ON
Batching Options:
        max_join_item_commands 16
        colored_vertex_format_threshold 0.25
        batch_buffer_size 16384
        light_scissor_area_threshold 1
        item_reordering_lookahead 4
        light_max_join_items 32
        single_rect_fallback False
        debug_flash False
        diagnose_frame False
WASAPI: wFormatTag = 65534
WASAPI: nChannels = 2
WASAPI: nSamplesPerSec = 48000
WASAPI: nAvgBytesPerSec = 384000
WASAPI: nBlockAlign = 8
WASAPI: wBitsPerSample = 32
WASAPI: cbSize = 22
WASAPI: detected 2 channels
WASAPI: audio buffer frames: 933 calculated latency: 21ms

CORE API HASH:<PHONE_NUMBER>0375384316
EDITOR API HASH:<PHONE_NUMBER>76539986
Loading resource: C:/Users/Abdera7mane/AppData/Roaming/Godot/editor_settings-3.tres
EditorSettings: Load OK!
Scanning projects at: D:\Programs\Godot\Projects
Found 132 projects.
EditorSettings: Save OK!
Loaded builtin certs
EditorSettings: Save OK!
WARNING: ObjectDB instances leaked at exit (run with --verbose for details).
     at: cleanup (core/object.cpp:2071)
Leaked instance: Image:8441 - Resource path:
Hint: Leaked instances typically happen when nodes are removed from the scene tree (with `remove_child()`) but not freed (with `free()` or `queue_free()`).
Orphan StringName: Image
StringName: 1 unclaimed string names at exit.
ERROR: There are still MemoryPool allocs in use at exit!
   at: (core/pool_vector.cpp:63)

Steps to reproduce
the shortest way to reproduce:

open Godot.
wait for the Project Manager to load projects list.
close the window.

which provides the previous console output
Minimal reproduction project
No response

Can you look into bisecting the exact commit in which the regression started? This will require compiling the engine several times, but it'll greatly speed up troubleshooting.

Took me a while, 2d3f5855c62588c6bd5145aa3b53c3902dbe499f is the first bad commit.

Thanks for doing that work, it's very useful to identify and fix the regression.

Took me a while, https://github.com/godotengine/godot/commit/2d3f5855c62588c6bd5145aa3b53c3902dbe499f is the first bad commit.

Unlike DisplayServer on master, OS is destroyed too late to clean icon. It's probably should be manually removed before ObjectDB is cleared. I'll take a look tomorrow.

Fixed by #61304.


Release v1.1.2-rc9

Deps

GUI: v1.1.2
Installer: v1.1.2-rc9
harvester-network-controller: v0.3.2
harvester-node-manager: v0.1.4
harvester-node-disk-manager: v0.4.11
harvester-load-balancer: v0.1.5

Tested with the upgrade from v1.1.1 to v1.1-head. Also, creating the downstream cluster, PVC, and some basic operations looks good.


Prevent selecting multiple items (cdkDragDrop)

Reproduction
How to prevent selecting multiple items?
When I try to long-press on a draggable item, it's grouping multiple items and drag in a group.

Closing this issue as it does not follow the issue template. Please file a new issue that follows the template, which gives the team the information needed to investigate.


Analyze forms with a custom model part is misleading

If we will train a model with labels, then "Analyze forms with a custom model" sample will not work properly due to: https://docs.microsoft.com/en-us/python/api/azure-ai-formrecognizer/azure.ai.formrecognizer.formfield?view=azure-python
"label_data
Contains the text, bounding box, and field elements for the field label. Note that this is not returned for forms analyzed by models trained with labels."

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

ID: 1e3de1b4-a118-efae-0e72-7953c8e891b3
Version Independent ID: 5ad38c9e-c775-8dbc-16ff-705a40b580e1
Content: Quickstart: Form Recognizer client library - Azure Cognitive Services
Content Source: articles/cognitive-services/form-recognizer/quickstarts/client-library.md
Service: cognitive-services
Sub-service: forms-recognizer
GitHub Login: @PatrickFarley
Microsoft Alias: pafarley

Hello @smereczynski , thanks for taking time to share your valuable feedback! We are taking a look into this and will get back to you soon.

@smereczynski It appears the sample handles this error by substituting name if label_data doesn't exist. Are you getting an error?

#please-close


RFC: Totally Disable zkApp Commands in case of Emergency

An RFC to describe how to disable zkApp commands completely (i.e. not softcap, or hardcap).

!ci-build-me