[$500] Infinite loading when accessing a non-existing workspace

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.41-2
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @hayata-suenaga
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1707848696684749
Action Performed:

Log out
Go to some WS link, e.g. https://staging.new.expensify.com/workspace/adsasddssad/rateandunit (doesn't matter if policy exist or not)
Log in

Expected Result:
Shouldn't be infinite loading
Actual Result:
Infinite loading appears
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~019c756122ca1f4027
Upwork Job ID:<PHONE_NUMBER>083399168
Last Price Increase: 2024-02-13
Automatic offers:

fedirjh | Reviewer | 0
bernhardoj | Contributor | 0

Able to reproduce on staging, but not latest main branch

the issue is not reproducible on the latest main.

I can still reproduce it.

I can also still reproduce this

Proposal
Please re-state the problem that we are trying to solve in this issue.
If we open a nonexistent workspace page after login, the app will load indefinitely.
What is the root cause of that problem?
When we open a nonexistent workspace page after login, the OpenApp request is never called. It will only be called if isReadyToOpenApp promise is resolved.
https://github.com/Expensify/App/blob/574157eb23a0c5f8ffbc101091848e3c35a88015/src/libs/actions/App.ts#L151
App.confirmReadyToOpenApp is used to resolve the promise and it currently is called in both ReportScreenIDSetter and SidebarLinks. When we open the workspace page after login, the report screen isn't added to the navigation stack, so App.confirmReadyToOpenApp is never called in ReportScreenIDSetter.
In SidebarLinks, App.confirmReadyToOpenApp is called only on small screen.
https://github.com/Expensify/App/blob/b0b651b4184230aa6b29af21af1430c5741aed95/src/pages/home/sidebar/SidebarLinks.js#L54-L59
Because the promise isn't resolved, the app shows the loading indicator infinitely.
What changes do you think we should make in order to solve the problem?

SidebarLinks is placed inside the bottom tab navigator which is always exist in the nav stack, so I think we can just have one place to call App.confirmReadyToOpenApp inside SidebarLinks. So, we can just remove the small screen condition.
https://github.com/Expensify/App/blob/b0b651b4184230aa6b29af21af1430c5741aed95/src/pages/home/sidebar/SidebarLinks.js#L54-L59

ReportScreenIDSetter will be mounted alongside ReportScreen.
https://github.com/Expensify/App/blob/634f7666b049aced9f6ac2ae5565508ec07d813e/src/libs/Navigation/AppNavigator/ReportScreenWrapper.tsx#L13-L25
If you are aware, before the ideal nav, in CustomRouter, we have a logic to make sure the report screen (specifically CentralPaneNavigator) is always added to the nav stack on large screen. (old code below)
https://github.com/Expensify/App/blob/1682be47d71b5a84c20c2396e833c0a015829252/src/libs/Navigation/AppNavigator/createCustomStackNavigator/CustomRouter.js#L71-L72
CentralPaneNavigator now includes workspace screen (overview, reimburse, etc.).
https://github.com/Expensify/App/blob/634f7666b049aced9f6ac2ae5565508ec07d813e/src/libs/Navigation/AppNavigator/Navigators/CentralPaneNavigator/BaseCentralPaneNavigator.tsx#L36-L53
The new CustomRouter logic now making sure to add a full screen or central pane navigator to the nav stack
https://github.com/Expensify/App/blob/634f7666b049aced9f6ac2ae5565508ec07d813e/src/libs/Navigation/AppNavigator/createCustomStackNavigator/CustomRouter.ts#L54-L84
In this case, visiting /workspace/{policyID}/rateandunit adds WorkspaceReimburse (belongs to CentralPaneNavigator) to the stack

PR is ready
cc: @fedirjh

@fedirjh can you review this?

Looks like the automated system didn't update.  Deployed to production on Feb 28 - 7 holding day means payment would be due on March 6th

@slafortune can you handle payment here?

Sorry I lost sight of this one!  PAID!
@fedirjh can you please complete the checklist?
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[ ] [@fedirjh] The PR that introduced the bug has been identified. Link to the PR:
[ ] [@fedirjh] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
[ ] [@fedirjh] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
[ ] [@fedirjh] Determine if we should create a regression test for this bug.
[ ] [@fedirjh] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.
[ ] [@slafortune] Link the GH issue for creating/updating the regression test once above steps have been agreed upon:

Thanks @fedirjh !


Fix Parser::match() so it does not accept T_WHERE when T_WITH is supposed to be matched

http://www.doctrine-project.org/jira/browse/DDC-3701

By the way, where should appropriate tests for the match() method go?

@mpdude I think we have only SQL generation tests for this sort of issue

The fix for this issue required a few more changes; it is available in #1395 and I have also added some tests there.


Install the generator: npm install -g generator-esta-webjs => does not work (typo?)

generator-esta-webjs is the right name, and indeed, it's the name given in the package.json, but interestingly, the name of the folder is wrong. We could rename it.
Nevertheless, npm install generator-esta-webjs should work and that's how I installed it.
@morarupasukaru, is npm install esta-webjs-generatorworking?

...esta-webjs/esta-webjs/esta-webjs-generator$  npm install esta-webjs-generator:
npm ERR! Linux 3.19.0-49-generic
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "install" "esta-webjs-generator:"
npm ERR! node v4.3.0
npm ERR! npm  v3.7.5
npm ERR! Unsupported URL Type: esta-webjs-generator:
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     https://github.com/npm/npm/issues
npm ERR! Please include the following file with any support request:
npm ERR!     /home/pascal/Documents/dev/dico/esta-webjs/esta-webjs/esta-webjs-generator/npm-debug.log
....
esta-webjs/esta-webjs-generator$ npm install -g esta-webjs-generator:
npm ERR! Linux 3.19.0-49-generic
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "install" "-g" "esta-webjs-generator:"
npm ERR! node v4.3.0
npm ERR! npm  v3.7.5
npm ERR! Unsupported URL Type: esta-webjs-generator:
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     https://github.com/npm/npm/issues
npm ERR! Please include the following file with any support request:
npm ERR!     /home/pascal/Documents/dev/dico/esta-webjs/esta-webjs/esta-webjs-generator/npm-debug.log
...

Hello,
i successfuly installed the generator once. But i dunno how to reproduce the installation. I'm not a pro of npm...

I looked up the nam registry and I didn't find eta-webjs there. If you are not in SBB network, npm won't find generator-esta-webjs. Until it's in the official npm repo you have to clone it and install it manually into your local repo. @ReToCode, right?

@morarupasukaru as a hint, here is described how to install a node module locally: http://podefr.tumblr.com/post/30488475488/locally-test-your-npm-modules-without-publishing
short cookbook

Clone esta-webjs
build the generator with npm install
pack the module with npm pack (creates a tar ball)
install the packed module into the local registry with npm -g install
then you can start using it via yo esta-webjs

Does this help?

Thank you for the informations. I have installed the module locally as explained. It's working.
I'll reject my pull-request.
Have a nice weekend!


Deploy and test custom metadata changes in MCP DEV

Can you provide an update on this ticket?

UDS v6.4.2 has been deployed to MCP Dev.
Custom metadata search works. For example, curl command to search for granules with last updated time > 2024-02-04
curl "https://d3vc8w9zcq658.cloudfront.net/am-uds-dapa/collections/urn:nasa:unity:uds_local_test:DEV1:CHRP_16_DAY_REBIN___4/items?filter=updated%20%3E%3D%20%272024-02-04T00%3A00%3A00Z%27"


why $cancel() will add a new row

why $cancel() will add a new row,how to cancel it?

demo:
http://vitalets.github.io/angular-xeditable/#editable-row

You have to remove the item that was added to the array when you click add row.
There is an example in editable-table

@bravility can this be closed now?

@ckosloski To some extent, yes. But I would rather Editable row use than Editable table, can I fix it in Editable row? instead of use Editable table

All you need to do is call a function on cancel that removes the last item in your list
$scope.movieCharacterModel.characters.splice(index, 1);


Important: Disable error handler

[EDIT by @guardrex to fix the code blocks.]
Please update this page to include this little bit.  You can get errors in the console, instead of having to print out the contents of the response. At minimum you should add:
builder.ConfigureServices(services => { //Already in the doc
    services.AddMvc(); //Add this line
}).Configure(app => { app.UseMvc(); }); //This is the important line.

This will prevent your Startup.Configure method from running.  The MVC template includes a Startup.Configure method with the following line:
if (env.IsDevelopment())
{
    app.UseDeveloperExceptionPage(); //This is the source of the problem

That causes errors not to be caught by xunit.  My method prevents that line from being called, thus, disabling any error handlers and allowing the error message to drop back to xunit which prints it out to the ITestOutputHelper.  That way you get beautiful text errors with hyperlinks you can click that bring you right to the error.
I'll bet most people don't even know this is possible.  Please fix!
Here's my question on StackOverflow https://stackoverflow.com/questions/55738128/how-do-you-see-errors-from-the-controller-in-xunit

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

ID: 77889d43-fedb-f8b4-8fea-379ec946555c
Version Independent ID: f499b131-8b14-7e10-1728-543fdd5a5656
Content: Integration tests in ASP.NET Core
Content Source: aspnetcore/test/integration-tests.md
Product: aspnet-core
Technology: aspnetcore-test
GitHub Login: @guardrex
Microsoft Alias: riande

Thanks @bluebaroncanada ... I'll mark this for a look by engineering when they do their engineering doc sweep.

It actually turns out that this is more important than I once thought. If you don't do this properly, you won't get proper errors from Continuous Integration.

Moved to #16341: Integration tests MASTER issue


Actually create flowlogs if they don't already exist

Before, if 'FlowLogs': [] then it never reaches create_flowlog()

@jchrisfarris

I think this was fixed in a different pull.


No bearer authorization (JavaScript/TypeScript client)

Dupe of https://github.com/aspnet/SignalR/issues/888


LinkField - a URLField which supports mailto:

Needed for a client site.
The custom field makes it a bit easier to deal with across multiple instances.

:shipit:


Combine map and starmap into single function and automatically determine which based on zipped iterable

This is provided by a new simplified package, parabar: https://github.com/mdmould/parabar.


Add callback support.

I have added a callback parameter, which is called just before sleep is initiated. The callback is passed the time.time() of when the sleep() call is due to finish.

Just a heads up: I'm going to upload my fork to PyPI.


how to set the read txt file

Hello. I use short reads from http://www.ebi.ac.uk/ena/data/view/SRR306607 as input(RepARK paper data),
sample_configuration_10X.txt
reads.txt
and for the configuration and read txt, I use the settings and the running log as denovo.txt as the attachment. But the result file is only 60k, I think I may set something wrong. Can you have a look at those file? Especially I do not know how to set the information about insert. Thanks!

Uploading denovo.txt…

denovo.txt

Hi @bzvew ,
Can u change the "MIN_REPEAT_FREQ 10" (first line in the config file) to a smaller value, like 5, and try again. It's better to use a different output folder (set the "OUTPUT_FOLDER ./test/" in the config file).


ci: add dummy CI checks+wait for it

Even if the GH App is authorized to not go thru the PR process, it still needs to validate the 'Status Checks'.
This PR does that by following the GH doc on the topic:
https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/defining-the-mergeability-of-pull-requests/troubleshooting-required-status-checks#handling-skipped-but-required-checks
TL;DR:

Push the commit
Create a tag on the commit
Create a release on the commit
Release trigger a new dummy workflow that validates the check.
Wait for the check to go green.
Reset main to the commit.
Profit?

Just creating this PR for documenting the change. The impact being on the CI only, I'll move ahead without review.


csvcubed build validation-errors.json file path is invalid

I ran the csvcubed build cmd with the erroneous input below. The command I ran is
csvcubed build csv/errorneous_input/csv_errorneous.csv --out out/errorneous_input/csv_errorneous --validation-errors-to-file
Dimension,Value,Measure,Unit
A,,Length,cm
B,,Length,cm
C,,Length,cm
D,,Length,cm
E,,Length,cm

Then I got the below exception. As shown in the exception, the path csvcubed build is looking at for the validation-errors.json seems to be wrong. I was expecting for it to look at out/errorneous_input/csv_errorneous/validation-errors.json.
2022-08-03 16:53:14,530 - csvcubed.cli.build - ERROR - Validation Error: Observed values missing in 'Value' on rows: {0, 1, 2, 3, 4} (build.py:87) 2022-08-03 16:53:14,530 - csvcubed.cli.build - ERROR - More information: http://purl.org/csv-cubed/err/obsv-val-mis (build.py:89) 2022-08-03 16:53:14,531 - csvcubed.cli.entrypoint - CRITICAL - Traceback (most recent call last):   File "/opt/hostedtoolcache/Python/3.9.13/x64/lib/python3.9/site-packages/csvcubed/cli/entrypoint.py", line 94, in build_command     build(   File "/opt/hostedtoolcache/Python/3.9.13/x64/lib/python3.9/site-packages/csvcubed/cli/build.py", line 59, in build     with open(output_directory / validation_errors_file_out, "w+") as f: FileNotFoundError: [Errno 2] No such file or directory: 'out/errorneous_input/csv_errorneous/out/errorneous_input/csv_errorneous/validation-errors.json'  (log.py:109).

Closing as duplicate of #570


Powertoys fails to get focus when volume slider has just been clicked

Microsoft PowerToys version
0.62.1
Running as admin

[x] Yes

Area(s) with issue?
PowerToys Run
Steps to reproduce

if I click my volume icon in the taskbar.
And then use the hotkey to start Powertoys Run.
Then Powertoys Run fails to properly catch focus. I have to click the powertoys Run window to enable input and be able to search.

This also happens with other trayicons interactions: battery, wirelessnetworks, date&time, notification panel.
Does work with other trayicon interactions such as OneDrive
✔ Expected Behavior
Powertoys Run should catch focus end allow immediate typing to start searching.
❌ Actual Behavior
Cannot search immediately, but have to click the window first.
Other Software
No response

Hi @madcowGit
Can you check under PowerToys Run settings if you have "Use centralized keyboard hook" enabled? If yes, can you try to disable it?

/needinfo

I had it enabled.  Disabling it fixed this issue
I dont remember if it was enabled by default or that I manually enabled it

Done a check and the option is disable by default.
According to the description this is suggested in case of issues with the activation but the side effect is the focus issue.

This is likely related to #20043


Add files via upload

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[x] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Installation failed with exit code -1978335226
2024-10-25 00:52:44.480 [CLI ] ShellExecute installer failed: 1

Automated error analysis suggests -1978335226 may mean Running ShellExecute failed
(Automated response - build 897.)

@wingetbot run

Automatic Validation ended with:

Installation failed with exit code -1978335226
2024-10-30 16:13:29.122 [CLI ] ShellExecute installer failed: 1

Automated error analysis suggests -1978335226 may mean Running ShellExecute failed
(Automated response - build 897.)

Manual Validation also ended with exit code: 1, but the install appeared to be successful otherwise:

@wingetbot run

Automatic Validation ended with:

Installation failed with exit code -1978335226
2024-11-04 12:04:47.589 [CLI ] ShellExecute installer failed: 1

Automated error analysis suggests -1978335226 may mean Running ShellExecute failed
(Automated response - build 897.)

Close with reason: Duplicate of #185683;


لا يكمل التحديث يوجد مرفق

السلام عليكم
بالمرفقات
مالحل ؟

url ??

www.z4z6.com

Kleeja error:
[ No Template ! ]
Requested "admin_php_info" template doesnt exist!
Visit Kleeja Website for more details.

Kleeja error:
[ No Template ! ]
Requested "admin_php_info" template doesnt exist!
Visit Kleeja Website for more details.

maybe you played with the code, because there is not template called admin_php_info,
your kleeja is defined as http  and your server is forcing the call to be https so the resources is not loaded in the page,
make the url in the database to https via phpmyadmin -> klj_config -> siteurl = https://z4z6.com/up/
and make the cookie_secure = 1

no modification
i do it same in siteurl
plz

تعال ع الديسكورد https://discord.gg/Mp3XVKP


Implement frontend MVP

Implemented a "MVP" version of the frontend that contains a form with input fields for each letter of the alphabet. Clicking the submit button will make a request to the API and the results will be displayed under the form.
Removed placeholder content from the frontend
Removed the media query from the global.css file to prevent frontend styles from changing based on user browser preferences.

Tähänkö vintun aikaaa senkin homo

From: Janne Kantola @.>
Sent: Sunday, September 8, 2024 1:08:46 AM
To: ttaiv/bracelet-optimizer-app @.>
Cc: Teemu Taivainen @.>; Review requested @.>
Subject: Re: [ttaiv/bracelet-optimizer-app] Implement frontend MVP (PR #7)
@JanneKantolahttps://github.com/JanneKantola requested your review on: #7https://github.com/ttaiv/bracelet-optimizer-app/pull/7 Implement frontend MVP.
—
Reply to this email directly, view it on GitHubhttps://github.com/ttaiv/bracelet-optimizer-app/pull/7#event-14172739391, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BASFNZ7SGMPAZD3GGCR4AULZVN2O5AVCNFSM6AAAAABN2LZGRCVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJUGE3TENZTHEZTSMI.
You are receiving this because your review was requested.Message ID: @.***>

Äijä 8n kyl kohtuullisen hom8 äih pää

From: Teemu Taivainen @.>
Sent: Sunday, September 8, 2024 2:42:07 AM
To: ttaiv/bracelet-optimizer-app @.>; ttaiv/bracelet-optimizer-app @.>
Cc: Review requested @.>
Subject: Re: [ttaiv/bracelet-optimizer-app] Implement frontend MVP (PR #7)
Tähänkö vintun aikaaa senkin homo

From: Janne Kantola @.>
Sent: Sunday, September 8, 2024 1:08:46 AM
To: ttaiv/bracelet-optimizer-app @.>
Cc: Teemu Taivainen @.>; Review requested @.>
Subject: Re: [ttaiv/bracelet-optimizer-app] Implement frontend MVP (PR #7)
@JanneKantolahttps://github.com/JanneKantola requested your review on: #7https://github.com/ttaiv/bracelet-optimizer-app/pull/7 Implement frontend MVP.
—
Reply to this email directly, view it on GitHubhttps://github.com/ttaiv/bracelet-optimizer-app/pull/7#event-14172739391, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BASFNZ7SGMPAZD3GGCR4AULZVN2O5AVCNFSM6AAAAABN2LZGRCVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJUGE3TENZTHEZTSMI.
You are receiving this because your review was requested.Message ID: @.***>

I pushed some improvements based on the comments to the branch now. I also added more comments for improved readability.

Looks really good to me! 🚀


Why is Docker so Popular

We ask that you please be patient as our team works through approving and publishing all articles/tutorials in a timely manner. Allow 1-3 days for a topic to be reviewed and/or approved - allow 3-7 days for an articles to be reviewed and/or published.
Be sure to visit our Resources Page for tools, resources, and example articles to go over.
NOTE: (Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
NOTE: We tend to stray away or tend not to publish reviews/comparisons of commercial product offerings.
We're looking for the first 2-3 paragraphs of the article that appropriately summarize what your article will be about.
Proposed title of the article
Your title should be descriptive of the article/tutorial. Be Specific. Use keyword research to gain your article's higher ranking.
Why is Docker so popular? What makes it a big deal
Introduction paragraph (2-3 paragraphs):
Docker is an open platform container technology for developers and sysadmins to build, ship, and run distributed applications, whether on laptops, data center VMs, or the cloud. Docker has been a game-changer since its release in 2013. It has become a massively popular containerization technology. In this article, we will discuss why Docker is a big deal.
Key takeaways:
What is Docker?
How popular is Docker?
Reasons Why is Docker so popular and why the rise of containers?
Advantages of Docker Containers
Which companies use Docker?
References:
Please list links to any published content/research that you intend to use to support/guide this article.
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

seems like a interesting topic - approved @mosesreigns
please check that it does not overlap too much with other new incoming articles 👍


update Overview of Data Migration Assistant fta-emea-spring22

@pelinat : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@rajeshsetlem
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"

@rwestMSFT Randolph, this public PR is not assigned to anyone. Could you please take a look? #assign:rwestMSFT


set image type to gif

Is there any option to set/modify image type dynamically as per image type
I require to upload gif [though cropper shows a gif in preview] but cropped image is of type png

Per the readme
type (optional)
Valid values are 'png' or 'jpeg' (might work with webm too, haven't tried it)
I have not tried gif, but I assume gif is not supported.
For firefox it's unclear: https://developer.mozilla.org/en-US/docs/Web/API/HTMLCanvasElement/mozGetAsFile#Specifications
For the other browsers, toDataURL is used, and you can see what types it supports by going to this page on each browser: http://kangax.github.io/jstests/toDataUrl_mime_type_test/

thanks for the information


Increase the key size from 1024 to 2048

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

NONE

/lgtm
/approve


German Localization

I added the files for the German localization.
In addition you  forgot to delete some questions that where removed in @TristanBomb 's repository. I have rewritten the questions.json to include all questions from master in the right order.

Thanks a lot :smile:
Actually I just left the old strings, but they were not on present on the questions.js file, where they must have the same id. Should we rename q56-q58 or just remove them? Maybe a git revert, I dunno

Ah I see, I'll do a revert and adapt my translation. This is easier then changing the question numbers everytime a new question is added (or removed).

I also have a solution for image localization based on CSS like @elifoster proposed. Do you want it?


Take a look at existing GitHub Actions. Currently fail.

Fixed with:

https://github.com/ubiq/pyrus/commit/9cebcd125f0324370f1c109a6c6c65955f620e1d
https://github.com/ubiq/pyrus/commit/21738e31ad943cdadfcef763a878ba3fbbb7cbcd


glGetUniformLocation (1, b’Texture\x00’) 'invalid operation'

First of all, I'm very excited for the close release of 2.0, and I'm sorry to be submitting this issue at this time. I'm fairly certain it's a problem with the OS and not pyimgui, but I couldn't find any concrete results online...
Traceback (most recent call last):
  ...
  File "imgui/integrations/glfw.py", line 12, in __init__
  File "imgui/integrations/opengl.py", line 60, in __init__
  File "imgui/integrations/base.py", line 17, in __init__
  File "imgui/integrations/opengl.py", line 108, in _create_device_objects
  File "src/latebind.pyx", line 51, in OpenGL_accelerate.latebind.Curry.__call__
  File "OpenGL/GL/VERSION/GL_2_0.py", line 430, in glGetUniformLocation
  File "OpenGL/platform/baseplatform.py", line 415, in __call__
  File "src/errorchecker.pyx", line 58, in OpenGL_accelerate.errorchecker._ErrorChecker.glCheckError
OpenGL.error.GLError: GLError(
	err = 1282,
	description = b'invalid operation',
	baseOperation = glGetUniformLocation,
	cArguments = (1, b'Texture\x00'),
	result = -1
)

From what I can tell the issue comes up here, even though the traceback doesn't state it precisely.
I haven't had this issue on any of my systems, and most of my users had no issues with opengl, glfw and imgui. This issue was reported by only 2 users, and both on a Debian based distribution of linux (one on Devuan and one on either Pop!_OS or base Debian). That's why my best guess is that this error is due to some caveat with debian based packages, but I could very well be wrong. From what I recall the first report came up maybe 6 months ago, while the second was 2 months ago. Both users have been completely unable to get a window to show up with my program, even through my regular updates (and I always use the dev/version-2.0 branch). I package it with cx_Freeze, but the issue happens also when running from source.
In case this matters, I know that the user on Devuan is using an NVidia gpu with nouveau drivers, I don't know about the other report.
I had poked at it a few times before with no results, and now I tried to look harder and still managed nothing. Any help, even just tiny hints, is very welcome

From what i found online, the #version 330 sets which GLSL version (GL_SHADING_LANGUAGE_VERSION) to use, and usually the opengl version goes hand in hand with that, for example on my system glinfo output:
GL_VERSION: 4.6.0 NVIDIA 520.56.06
GL_RENDERER: NVIDIA GeForce RTX 2060/PCIe/SSE2
GL_VENDOR: NVIDIA Corporation
GL_EXTENSIONS: ...
GL_SHADING_LANGUAGE_VERSION = 4.60 NVIDIA
GLU_VERSION: 1.3
GLU_EXTENSIONS: ...
GLUT_API_VERSION: 4
GLUT_XLIB_IMPLEMENTATION: 13

and you can see that opengl version 4.6.0 goes hand in hand with the glsl version 4.60. i thought that the integration used a window hint to specify the gl version to use, but impl_glfw_init only does that on macos... it used to do that only on macos when i originally copied it into my project, but i see it was changed to always set the window hints...
i will update my impl_glfw_init, ask the two user to try that and also send the output of glinfo for more context. will report back soon, and thanks.

Looks like that was it! Ran into some more struggles due to typical debian packages being held back, but after that its running just fine! Thanks again.
For now only one of the two users has reported back, but I'm confident this was it, if not I'll comment again

Welp, looks like I have been cursed. While that fixed it for one linux user, it created another issue for a windows user...
Traceback (most recent call last):
  ...
  File "imgui\integrations\glfw.py", line 12, in __init__
  File "imgui\integrations\opengl.py", line 60, in __init__
  File "imgui\integrations\base.py", line 17, in __init__
  File "imgui\integrations\opengl.py", line 84, in _create_device_objects
  File "src\latebind.pyx", line 39, in OpenGL_accelerate.latebind.LateBind.__call__
  File "src\wrapper.pyx", line 318, in OpenGL_accelerate.wrapper.Wrapper.__call__
  File "src\wrapper.pyx", line 311, in OpenGL_accelerate.wrapper.Wrapper.__call__
  File "OpenGL\platform\baseplatform.py", line 415, in __call__
  File "src\errorchecker.pyx", line 58, in OpenGL_accelerate.errorchecker._ErrorChecker.glCheckError
OpenGL.error.GLError: GLError(
    err = 1282,
    description = b'invalid operation',
    baseOperation = glGetIntegerv,
    pyArgs = (
        GL_TEXTURE_BINDING_2D,
        <object object at 0x0000020BD9A54E20>,
    ),
    cArgs = (
        GL_TEXTURE_BINDING_2D,
        array([0], dtype=int32),
    ),
    cArguments = (
        GL_TEXTURE_BINDING_2D,
        array([0], dtype=int32),
    )
)

The previous version of my program, where the context version hints were not being set, worked fine for them and opengl was using version 4.6. But apparently setting it to 3.3 breaks it for them... Like I can understand the previous error where the implementation setup was designed around OGL 3.3, and for one particular user on a newer 4.2 version it caused issues, but here the opposite is happening where setting it to the correct version breaks it while the newer version does not...?
I was tempted to make a try except block and have it attempt 3.3 first and if that fails let it use the default version, but the problem is both errors dont happen at context creation, but much later, so I'd have to stick the whole GlfwRenderer() setup in the try block, and that would require me to enable OpenGL.ERROR_CHECKING which is a huge performance hog, and cant be disabled after the first opengl import...

Yes indeed, using the hint to ensure correct version (3.3 for GLSL compatibility) seems to be the proper way to fix that.
But I am not sure why it would cause some incompatibilities with 4.6 users. glGetIntegerv should work the same in 3.3 and 4.x, as far as I know. (It is however reported to be very slow and may be useless in the present case, thus it could be avoided).
There have been some changes to the integration in #296, which I just merged. It may be useful to test that this doesn't fix it miraculously or breaks it even more.

Thanks for the insight. I now have received another report of the same issue. This user has the same exact situation as the other, was working fine, and my update with the hints showed this issue, same lines in the traceback. They mentioned updating their nvidia graphics drivers yesterday, not sure if that was before or after updating my app, so I’m not sure if that driver opened the app correctly without the hints. Either way this makes me even more sure that this is some weird nvidia shenanigans, because when is nvidia ever not the problem am I right? 🙄
I’ll try a build with the updated integration and ask the affected user to report back.

So while adventuring down the rabbit hole that is page 2 of google i found some mentions of msi afterburner and riva tuner statistics server overlays injecting into opengl applications and causing crashes on nvidia drivers. So i asked both users and indeed, disabling rtss fixed the crash. Also updating to the latest beta of afterburner and rtss fixed the crash while keeping them enabled. So maybe a note could be added somewhere? Either way im happy this is working now, and that its not a problem with my or this great project's code.

TLDR of solutions:

make sure you enable the hints for the correct opengl version when creating your window
some crashes might be caused by msi afterburner and rtss, try updating or disabling them


Improve performance of trace page

Calculate Trace summaries asynchronously.
Reuse results of /api/v2/traces API call in the individual trace page if can.

I'll update unit tests.

as I understand, this improves performance by doing work in parallel correct?

@adriancole
Yes, correct.
And this improves performance by avoiding calling /api/v2/trace API in trace page if not needed.
In other words, reuse results of /api/v2/traces API in the individual trace page

PTAL 🙇

PTAL

thanks much!

And this improves performance by avoiding calling /api/v2/trace API in trace page if not needed.
In other words, reuse results of /api/v2/traces API in the individual trace page.

I haven't thoroughly checked the implementation (sorry), but I'm wondering if a user wants to get the latest data from /api/v2/trace on the trace view screen, is there a way to do this? Does refreshing the web page do that? I can imagine in the case that the trace is in-progress when returned in the search results, a user will want a way to get the up-to-date trace data in the trace detail view.


[LLVMGPU] Set prefetching on translation info

This patch makes prefetch_shared_memory part of translation_info config dictionary, allowing us to control prefetching at dispatch level, instead of globally turning it on/off. Prefetching is still off by default, the flag makes KernelConfig add prefetch_shared_memory unit attribute to config dictionary.

@kuhar You would need to change your tuning script to explicitly put prefetching in tuning config after this patch

cc: @RattataKing ☝

FYI this improved the SDXL benchmarks we run in this repo (on mi250 using rocm):
E2E Benchmark Time: 1330.0 ms (golden time 1661.5 ms)
Scheduled Unet Benchmark Time: 340.0 ms (golden time 450.5 ms)
Prompt Encoder Benchmark Time: 16.8 ms (golden time 17.5 ms)
VAE Decode Benchmark Time: 288.0 ms (golden time 288.5 ms)

(we set --iree-llvmgpu-enable-prefetch=true in https://github.com/iree-org/iree/blob/main/build_tools/pkgci/external_test_suite/sdxl_vae_decode_gpu_rocm_gfx90a.json)

Results were visible on presubmit here: https://github.com/iree-org/iree/actions/runs/9681801742
"Golden" times can be updated at the bottom of this file to pin down the improvements: https://github.com/iree-org/iree/blob/main/.github/workflows/pkgci_regression_test.yml

FYI this improved the SDXL benchmarks we run in this repo (on mi250 using rocm):
E2E Benchmark Time: 1330.0 ms (golden time 1661.5 ms)
Scheduled Unet Benchmark Time: 340.0 ms (golden time 450.5 ms)
Prompt Encoder Benchmark Time: 16.8 ms (golden time 17.5 ms)
VAE Decode Benchmark Time: 288.0 ms (golden time 288.5 ms)

(we set --iree-llvmgpu-enable-prefetch=true in https://github.com/iree-org/iree/blob/main/build_tools/pkgci/external_test_suite/sdxl_vae_decode_gpu_rocm_gfx90a.json)

Results were visible on presubmit here: https://github.com/iree-org/iree/actions/runs/9681801742
"Golden" times can be updated at the bottom of this file to pin down the improvements: https://github.com/iree-org/iree/blob/main/.github/workflows/pkgci_regression_test.yml

This would mean that not using prefetching on the tuned configurations is actually better. Adding a parameter for prefetching to the tuning script might be a good idea @kuhar @RattataKing


Instance check fails

The following line will evaluate to False if a numpy data type is used.
https://github.com/paul019/python_to_latex/blob/b7739e6a3851d7adf3c542faf0be3d49ba0fd154/python_to_latex.py#L126
And, indeed, a numpy wrapper is used due to
https://github.com/paul019/python_to_latex/blob/b7739e6a3851d7adf3c542faf0be3d49ba0fd154/python_to_latex.py#L122
This is why I get an erroneous variable in the results.tex file when I do something like
result_g = (9, 1.0)

In my local clone, I temporarily disabled the isinstance check (also for the error variable) and it worked fine. You still might want to incorporate some instance check somehow.

Not applicable in alpha version anymore.


Keep generated TABLE_NAME

Why is TABLE_NAME and table column names @Deprecated in the current sqldeligth and removed in the new one? We rely on them to monitor table changes. Would you consider keeping them or at least add an option to the generator to maybe generate object classes with table defenitions?

SQL Delight monitors table changes automatically. Do you have any other use for the constant?

Couple uses:

Easily iterate over tables to run DELETE FROM $TABLE statements, so you don't have to write it manually in each .sq file.
Use it in code that doesn't use .sq files or sqldelight to execute sql. For example part of our database doesn't use sqldelight because it has a dynamic schema that comes from config(that comes from server). So we have our own little implementation of android db interface that does table notifications by table names(like Sqlbrite) and we need table name to be able to use it.

I'm sure people have other uses cases, theese are ours.
As I mentioned in the first post you can disable this generation by default, so only people that really need it will get it.

We don't like options. Constants are mostly free so a good argument for these means we'd probably just generate them always.


Use find_package_message for printing messages

Instead of printing the same message over and over again (e.g. if the compiler doesn't support code coverage, but FindCodecov.cmake is called more than once), the find_package_message() function should be used to print these messages only once.

Three years later...
You gotta love those well maintained repositories.


is there something wrong in setsat function in BlendShaderTypes.metal ?

I found this code in setsat function in  BlendShaderTypes.metal .
/* b is max, r is mid, g is min */             c.r = mid(c.g, c.r, c.b, s);             c.b = s;             c.r = 0.0h;
is it should be like this ？
/* b is max, r is mid, g is min */             c.r = mid(c.g, c.r, c.b, s);             c.b = s;             c.g = 0.0h;
😂

Lol. I changed it and I'm blaming the guy who was here before me. Thanks! 🙃

Lol. I changed it and I'm blaming the guy who was here before me. Thanks! 🙃

it also exist in GPUImage2


Move release_notes on old GitHub for Service Fabric to this one.

Given the new repo, I think it may make sense to move release_notes: https://github.com/Azure/service-fabric/tree/master/release_notes, to new Service Fabric GitHub: https://github.com/Microsoft/service-fabric

Release notes have been moved here https://github.com/microsoft/service-fabric/tree/master/release_notes


unlink me from this project

considering today's facts, i don't wanna be a part of this no more, please stop using assets made by me.

Np, they have been removed


1.1.5

Fixed SqrtF, and add Width and Height to HMM_Vec2. Approval ? @bvisness

I say go for it!

Awesome merging!


[BUG] Changelog for 2.12.0 does not mention switching of libraries

Changelog for 2.12.0 (https://github.com/opensearch-project/opensearch-build/blob/main/release-notes/opensearch-release-notes-2.12.0.md) does not mention switch from org.apache.cxf.rs.security.jose to com.nimbus.jose.jwk (https://github.com/opensearch-project/security/pull/3595). This slowed us down considerably when trying to find out what was the breaking change between 2.11.1 and 2.12.0.

[Triage] @jarkko-rantavuori-vincit Thank you for filing this issue. I think the release notes should be retroactively updated to reflect that a change in the JWT library was made in 2.12.


Prepare changelog for 0.4.0 release

Just going to merge this to prepare the release. Since it only affects the changelog.


Input parameters info

Hi Alexey,
I come back to you for a question regarding the parameters used in the SBAS Baseline. In the samples of landslides (Golden Valley, YamchiDam) you set different parameter values like "days", "baseline_pairs", "limit", "iterations", as the following extract:
baseline_pairs = sbas.sbas_pairs(days=60)
optionally, drop dates having less then 2 pairs
#baseline_pairs = sbas.sbas_pairs_limit(baseline_pairs, limit=2, iterations=2)
optionally, drop all pairs connected to the specified dates
#baseline_pairs = sbas.sbas_pairs_filter_dates(baseline_pairs, ['2021-01-01'])
baseline_pairs
Could you please explain me the meaning in order to set my analysis parameters properly?
I have the following AOI to analyse for the last 4 years:
{
"type": "Feature",
"properties": {},
"geometry": {
"coordinates": [
[
[
15.441202320468705,
38.08053176221421
],
[
15.441202320468705,
38.06973406684506
],
[
15.46064854972468,
38.06973406684506
],
[
15.46064854972468,
38.08053176221421
],
[
15.441202320468705,
38.08053176221421
]
]
],
"type": "Polygon"
}
}
Can you suggest me the starting notebook sample and correct parameters to use?

In this context, the terms 'days' and 'meters' for baseline parameters are exactly as named. The appropriate choice depends on the coherence of your area and the accuracy of the DEM. You can begin with the example configurations and adjust them as necessary. Additional parameters and functions are designed to eliminate pairs that lack sufficient connections in the baseline graph.


CSS: Add a new recipe on mobile shows "tapbar" only if you scroll down and is showing out of the page

Also, tapbar image is not showing fully.

fixed


Add collaborators to this repo

Comment below to be added as a collaborator
We will all be using this repository today. Even though it's public right now, you will not be able to make changes until you're given the correct permissions. We've automated this process via the GitHub API. Once you comment, we will add you as a collaborator.
You'll start to receive a lot of emails. (❗) You can slow these down immediately by clicking the Unwatch button at the top of the page and selecting either the Participating and @mentions or Ignore option.

Hello

Hello

Hi

Hello world

hello

Woooo! MARKDOWN

test

Hey @ericdelamancha

Eric is a ghost

Friday


Plugin options to support more use cases

Thanks for the great project and support.
I would like to suggest adding a few options for the plugin:

remote: option to make external image requests optional (false by default). Reason is that SSG frameworks like 11ty or Vitepress have to process all images on the entire site everytime during development due to hot reload and having to fetch remote images everytime can significantly slow down the development speed if the internet connection is not in an ideal state.
publicDir: option to allow local absolute path processing. This will replace the "." below with the specified public directory path.
https://github.com/boyum/markdown-it-image-size/blob/6ee9922e7034229c97419738d5bd6244a8d6f667/packages/markdown-it-image-size/src/index.ts#L39

Example:
const MarkdownIt = require("markdown-it");
const { markdownItImageSize } = require("markdown-it-image-size");
const { cwd } = require("process");

const mdRenderer = MarkdownIt();
mdRenderer.use(markdownItImageSize, {
	publicDir: cwd() + "/public",
	remote: false
});

The idea of the cache option sounds brilliant. If we could realize it and make it a default behavior, I believe the remote option would not be necessary.
Regarding the publicDir suggestion, it actually came out from the fact that I was not using 11ty but Vitepress for my website, I wasn't able to use the package but had to pull the code and make appropriate modifications for it to work for my use case. Thought it'd be good to contribute back to the author, since there are actually some demands. (ref https://github.com/vuejs/vitepress/discussions/1639, https://github.com/vuejs/vitepress/discussions/3738)

#758 should add a file system cache, but I haven't really tried it yet (apart from unit tests) and I'm haven't done anything similar before. Please have a look at that, too (if you want, have time and care for it, no pressure).

Thank you for making the changes, I'll check them out later today or tomorrow and get back to you.

File system cache released in v14.4.0. I improved the unit tests and trust the code a bit more.


updating all scripts to exit on non-zero status

@mporrato good catch on the verify-whitelisted-index-image task, we already take care of this in preflight-trigger but in a more inline way where it is needed... See below as an example:
https://github.com/redhat-openshift-ecosystem/operator-pipelines/blob/main/ansible/roles/operator-pipeline/templates/openshift/tasks/preflight-trigger.yml#L103


Enforce UTF8 when writing DSV

DsvSerializer uses FileWriter1  which means the system default locale will determine the encoding. We discovered this when running in a Docker container where the locale was POSIX, and any non-ASCII characters used for delimiters or in Strings in proc annotations were garbled. It would be better to enforce UTF8, or make the encoding configurable if there is a use case for other encodings.

The constructors of this class assume that the default character encoding and the default byte-buffer size are acceptable. To specify these values yourself, construct an OutputStreamWriter on a FileOutputStream.2

To experience what we did, try the following:
File file = /* ... */;
OutputStreamWriter posix = new OutputStreamWriter(new FileOutputStream(file), StandardCharsets.US_ASCII);
posix.write("¦"); // will write a `?`
posix.close();

Yep, rookie mistake. Thanks for spotting the issue.
Would any of you be willing to try a PR? :)

The testing has sadly been more involved than it should have been, but I've got a proper test and a fix ready to be merged, feel free to review https://github.com/oss-transfer/neo4j-sproc-compiler/pull/57.

I'll do a minor release once this is merged.

@fbiville Thank you. I will try to test this sometime soon.


"none is not an allowed value" when using Union with Any

Checks

[x] I added a descriptive title to this issue
[x] I have searched (google, github) for similar issues and couldn't find anything
[x] I have read and followed the docs and still think this is a bug

Bug
It seems that None is allowed when a field's type is Any (as is expected, stated in the docs), but None is not allowed when a field's type is Union[SomeType, Any].
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
             pydantic version: 1.8.2
            pydantic compiled: True
                 install path: C:\Users\TobyHarradine\PycharmProjects\orchestration\.venv\Lib\site-packages\pydantic
               python version: 3.7.9 (tags/v3.7.9:13c94747c7, Aug 17 2020, 18:58:18) [MSC v.1900 64 bit (AMD64)]
                     platform: Windows-10-10.0.19041-SP0
     optional deps. installed: ['typing-extensions']

Code which reproduces the issue:
>>> import pydantic
>>> class M(pydantic.BaseModel):
...     a: Any
...
>>> M(a=1)
M(a=1)
>>> M(a=None)
M(a=None)
>>> class M(pydantic.BaseModel):
...     a: Union[int, Any]
...
>>> M(a=1)
M(a=1)
>>> M(a="abcd")
M(a='abcd')
>>> M(a=None)
Traceback (most recent call last):
  File "<input>", line 1, in <module>
  File "pydantic\main.py", line 406, in pydantic.main.BaseModel.__init__
pydantic.error_wrappers.ValidationError: 1 validation error for M
a
  none is not an allowed value (type=type_error.none.not_allowed)

This may be somewhat related to #1624.

allow_none attribute for field a remains False. Possible fix may be in pydantic/fields.py:566:
if is_union_origin(origin):
    types_ = []
    for type_ in get_args(self.type_):
        if type_ is NoneType **or type_ is Any or type_ is object**:
            if self.required is Undefined:
                self.required = False
            self.allow_none = True
            continue
        types_.append(type_)

@mykhailoleskiv That almost works, except it doesn't add Any or object to types_ (which eventually becomes ModelField.sub_fields). So M(a="abcd") or something like that no longer works. I'll open a PR


Allow parse content using Twig (.html files).

I was recently thinking about this idea while working on Pico 2.0 (#334) and thinking about Pico 3.0 (#317), but came to the decision, that we shouldn't explicitly allow replacing Markdown with other "content processors" (like Twig).
If you want to use Twig for some "more complex" pages you can already do that. Simply create a content/my_complex_page.md with
---
Template: my_complex_page
---

and a matching themes/my_theme/my_complex_page.twig. Add your contents to the Twig template and you're ready to go.
If you want to create a larger number of such "complex" pages and don't want to create both a (basically empty) .md file and a .twig template, you can also write a plugin that searches for .twig files in e.g. the themes/my_theme/complex_content/ directory, adds them to Pico's pages array (onPagesLoaded event) and intervenes appropriately when a "complex" page is requested (basically hook into the onRequestUrl event to detect the request and change the $templateName argument of the onPageRendering event). However, I don't think that this feature is widely needed, thus this isn't going to make it into Pico's core.
If you rather want to extend the Markdown processor (i.e. the Parsedown class) with some custom features (like #318), you can do this starting with Pico 2.0. There's a new onParsedownRegistered event that passes Pico's $parsedown object as reference, allowing you to replace it with another object (however, the object's class must extend the Parsedown class, otherwise you'll break Pico).
If you want to completely replace Markdown, YAML and/or Twig with something different, you can extend Pico's class and replace the respective methods as you want (Pico is open source). We obviously will neither officially support this nor will this work with all existing plugins or themes out there, but you definitely can do this if you want to.
Thank you for your feedback anyway! 👍


Building image fails with selected SDK being Java 16

The opnjdk:16 probably uses a different base image than 11. Other images should be checked too:
com.github.dockerjava.api.exception.DockerClientException: Could not build image: The command '/bin/sh -c apt-get update && env DEBIAN_FRONTEND="noninteractive" apt-get install -y libcurl4-openssl-dev tzdata && rm -rf /var/lib/apt/lists/*' returned a non-zero code: 127

This is because openjdk:16 is an image based on Oracle Linux Server, which uses rpm as a package manager; while openjdk:11 is based on Debian. I believe, there are Debian-based images for newer versions too, though they require a special tag format.

Do we really need this?

Do we really need this?

Yes, or do you suggest to allow only Java 8 and 11?

At least by for now let's not touch it :)

At least by for now let's not touch it :)

We should also disable these versions in our dropdown selection in order not to confuse the users

Under #1030 we should ensure it at least builds for Java 8, 11 and 17

The reason is that openjdk images starting from version 12 are using Oracle Linux as a base, which is not Debian-based. This issue will be closed by #1030 by swithing to Eclipse Temurin and keeping only LTS versions


ci: add Github Actions' cron to update Kong license

This adds a Github Actions' cron which will try to update the secret with Kong license (named: KONG_LICENSE_DATA) by pulling it from Pulp.
Apart from pulp related secrets (that contain credentials and the URL) this workflow requires Github Personal Token with repo permissions.

Codecov Report

Merging #172 (8c6b26f) into main (87ac1fa) will decrease coverage by 13.80%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main     #172       +/-   ##
===========================================
- Coverage   53.06%   39.26%   -13.81%
===========================================
  Files          44       44
  Lines        3884     3884
===========================================
- Hits         2061     1525      -536
- Misses       1367     2070      +703
+ Partials      456      289      -167

Flag
Coverage Δ

2.0.5
39.26% <ø> (ø)

2.1.4
39.10% <ø> (ø)

2.2.2
39.10% <ø> (ø)

2.3.3
39.10% <ø> (ø)

2.4.0
39.10% <ø> (ø)

2.5.1
39.10% <ø> (ø)

2.6.0
39.10% <ø> (ø)

2.7.0
39.10% <ø> (ø)

2.8.0
?

community
39.26% <ø> (ø)

enterprise
?

enterprise-<IP_ADDRESS>
?

enterprise-<IP_ADDRESS>
?

integration
39.26% <ø> (-13.81%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

kong/mtls_auth_service.go
0.00% <0.00%> (-62.50%)
:arrow_down:

kong/developer_service.go
0.00% <0.00%> (-54.87%)
:arrow_down:

kong/workspace_service.go
0.00% <0.00%> (-51.21%)
:arrow_down:

kong/rbac_role_service.go
0.00% <0.00%> (-48.94%)
:arrow_down:

kong/developer_role_service.go
0.00% <0.00%> (-46.67%)
:arrow_down:

kong/entity_permission_service.go
0.00% <0.00%> (-40.66%)
:arrow_down:

kong/rbac_user_service.go
0.00% <0.00%> (-34.89%)
:arrow_down:

kong/types.go
7.04% <0.00%> (-30.99%)
:arrow_down:

kong/endpoint_permission_service.go
0.00% <0.00%> (-26.42%)
:arrow_down:

kong/exists.go
50.00% <0.00%> (-25.00%)
:arrow_down:

... and 4 more

Continue to review full report at Codecov.

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


Is it possible to list all files in the served directory?

I hope I am not abusing the issues tracker for my question, but I haven't found documentation about this anywhere.
I'd like to use this module for serving files which have a secure hash as filename and I don't want anybody else except those who know the name of the file to access it. Does this module support preventing to list the index of the public directory?

This module doesn't provide a way to get a directory listing, correct.

Ok, so my files are "safe" if nobody knows their name?

At least with this module. This module just takes the given URL and looks up the file on the file system by that name. If it exists it will be served up, otherwise next() will be called.

Ok, thanks for the quick response!


Feature/improve objectfieldvalue for onetomany

this is for the file downloads inside ponds tables

Okay, now after the fix on fix/svg I am not sure this is necessary anymore.
Also, as a side note, using str.join should be avoided in most places where we generate any HTML output. It basically enforces any BaseElements to be converted to strings, which should normally only be done by htmlgenerator's internal methods.

Yes, it's still necessary because it makes it possible to format "candidatereferences.file" as a list of downloadable files...

In that case I suggest we do not use ObjectFieldValue directly but just add a custom BaseElement-tree (or pass a custom formatter to ObjectFieldValue, but that might be less straight-forward).
The problem here is that ObjectFieldValue is a very generic element but join will not always work as expected, because it is not guaranteed that the resolved object is a string. And, maybe more importantly, when using mark_safe we should guarantee that the provided argument will never contain any html-strings, otherwise we have a potential XSS vulnerability.
We could theoretically also use hg.Iterator inside ObjectFieldValue.resolve to avoid those two issues. I am not too sure about that though. Sometimes we want a list of related items to be rendered as UL, sometimes with a comma, sometimes just with a single space, etc. To me it seems easier to just write something like
  hg.Iterator(
    hg.C("row.items"),
    "item",
    hg.BaseElement(hg.ObjectFieldValue("item", "file"), " ")
  )

or similar, instead of letting ObjectFieldValue do everything.

Yes, I agree.
I think it will be better to just the cell parameter each time we want something as complex as candidatereferences.file, ok?


Add stable/prometheus-es-explorer

Signed-off-by: Àlex Pérez-Pujol<EMAIL_ADDRESS>
What this PR does / why we need it:
It adds the prometheus-es-exporter to the stable chart repository.
Special notes for your reviewer:
None.
Checklist

[x ] DCO signed
[x ] Variables are documented in the README.md

I'm not sure why it says that I fail the DCO. I've signed with my main e-mail, but it wants me to set the corporate mail.

@alexppg the DCO checks that the signing email corresponds to the email in the commit itself.
git config user.email

Thanks @vsliouniaev, I've change it. I believe there's nothing to be done on my side until somebody reviews it, the lint that fails won't pass until some maintainer is asigned.

Since you are submitting this chart to the repository, wouldn't it make sense that you should add yourself as the maintainer?

No problem @mvasilenko, but I believe I can't add myself like that and make the tests pass. In a prior commit I added myself to the maintainers section but this check says I'm not validated as a maintainer, and I don't know how it validates it.

ping @viglesiasce @foxish

Just to keep this open as the bot is trying to close this. Is there anything still to be done here?

@ntavares Somebody needs to revise it, as far as I know I don't have to do anything.

hey @alexppg
Have you considered hosting this chart and adding it to helm hub?
https://github.com/helm/hub/blob/master/Repositories.md

I'd like to that, but right now I'm not able. I'll try to convince my company to do it, but for now is no possible. It's a problem @maorfr ?

hey @alexppg, thanks for your reply!
it is an effort problem. chart maintainers are over loaded, so reviewing a new chart usually takes a long time. with that said, the way forward is not this repository, but rather helm hub! :)
Here are some things to consider if you are exploring hosting your own charts repository (copied from a similar PR):
I think that you totally should host your own charts!
This will give you more freedom around management, and you will not be blocked by us (maintainers).
This means that when users want to install your chart, instead of installing them from stable, they will install it from (for example) mycompany-charts:
$ helm install --name my-release mycompany-charts/mychart
Here are some instructions about how to host your own charts repo:
https://github.com/helm/hub/blob/master/docs/github.md#github-pages-as-a-repository
In order for people to find out about your charts repo, you should add it to helm hub:
https://hub.helm.sh/
Here are some instructions about how to add your charts repo to helm hub:
https://github.com/helm/hub#how-to-add-your-helm-charts
Hope that answered all your questions! :)

Thanks @maorfr ! I'll sure like to do that, because of all the reasons you mentioned. But as I said, I don't think this will be possible in a short time, so I'd like to maintain this chart here. Sorry and thanks for your kind explanation!


Add gravatarcr

It is inspired by https://ko.gravatar.com/site/implement/images/ruby/

Are you going to evolve it? There is still "WIP" in readme.

I'll finish it tomorrow :) Sorry for late

Be free to reopen it once it is finished. Thank you


adding trailing hyphen to valid url query endings

URLs that end with a hyphen such as https://www.youtube.com/playlist?list=PL0ZPu8XSRTB7wZzn0mLHMvyzVFeRxbWn- should be valid.

Thanks, #Shipit
Lemme find one more reviewer too.

Also, we'll need that code to be in java, js, and objc as well.

This will be included in #74 once it has another #shipit

merged


Error: Hydration failed because the initial UI does not match what was rendered on the server.

Bug report
Describe the bug
I am testing the Supabase UI with nextjs and using the basic setup, I keep getting this error:
Error: Hydration failed because the initial UI does not match what was rendered on the server.

To Reproduce

Create a simple nextjs app
Add the auth component

import { Auth, Typography, Button } from "@supabase/ui";
import { createClient } from "@supabase/supabase-js";

const supabase = createClient(
	"",
	""
);

const Container = (props) => {
	const { user } = Auth.useUser();

	if (user)
		return (
			<>
				<Typography.Text>Signed in: {user.email}</Typography.Text>
				<Button
					block
					onClick={() => props.supabaseClient.auth.signOut()}
				>
					Sign out
				</Button>
			</>
		);

	return props.children;
};

export default function Home() {
	return (
		<div>
			<Auth.UserContextProvider supabaseClient={supabase}>
				<Container supabaseClient={supabase}>
					<Auth supabaseClient={supabase} />
				</Container>
			</Auth.UserContextProvider>
		</div>
	);
}

if I remove <Auth.UserContextProvider..... it works without an issue.
Expected behavior
Should not have an error
Screenshots

System information

Version of supabase-js: 1.35.3
Version of Node.js: v14.19.3

I get this error from either of the following reasons:

Setting supabase.auth.user() inside my provider. Workaround is to use supabase.auth.session() to get the user object.
Nesting particular HTML tags/elements (refer to https://github.com/vercel/next.js/discussions/35773#discussioncomment-2489865)

I'm facing the same issue when trying to access user's properties (like user.email or user.user_metadata_user_name) when I use i.e. const user = supabase.auth.user();
Also the same thing happens when I try to access the user from the session like supabase.auth.session() as you suggested @madleo-dk

Ran into this when I added a Google oAuth login to the example form, but was able to quickly resolve it by doing two things:

Move the auth update functions out of the Profile function
Move the Profile form to a new file and then dynamically import it with ssr: false

Hope this helps!

This is possibly fixed now in the new version.
The auth component has been moved to it's own repo and package.
github.com/supabase-community/auth-ui.
You can also read the docs for auth ui here.
There are also docs on our nextjs auth helpers that might help with these issues, especially SSR issues.
https://supabase.com/docs/guides/auth/auth-helpers/nextjs

Closing this issue for now since the component has moved to a new repo.

hello @MildTomato , the auth-ui library link you have given is broken, could you help to update it? thank you!

sorry about that, fixed the link @lyqht


Tests

Identifying what areas would benefit from test coverage and writing them.

I propose a replay from every protocol along with the extract from that replay as verified against the heroprotocol python library.
Our tests would be then be to test extraction of that replay, and verify it against the extracted data already in the test/ directory.  It bumps up the size of the repo considerably, but none of this goes in the npm package, so does it really matter?
The other option is to checksum the extracted files and only include the checksum in the tests.


ioBroker Integration

The problem
Hi, many thanks for writing this adapter. I wonder why I can't install it via the ioBroker adapter page (does not show up there although BETA is enabled). Is it only possible to  install  it via GIT?
Version of Node.js
12.22.12
Version of ioBroker js-controller
3.1.5
Version of "husqvarna-automower"-adapter
0
Version of "admin"-adapter
6.1.6
Operating system incl. version running ioBroker
linux
Checklist of files to include below

[X] iobroker.current.log (Please share the logs in debug mode and check that there are no personal information)
[ ] Contents of the JavaScript browser console (always include in cases of issues with the user interface)

Additional information & file uploads
No response

thank you for your interest in this adapter.
Currently it can be installed directly via Github (https://github.com/ice987987/ioBroker.husqvarna-automower.git) or via NPM (https://www.npmjs.com/package/iobroker.husqvarna-automower).

Understood. Is there any reason why it's not in the iobroker repository? Are there any plans to do that in the next future?
Cheers

Would not help much with your outdated versions of
Node.js 12.22.12
js-controller 3.1.5

Would be great to have this adapter included into standard iobroker repository to be able to install it directly.

Bitte diesen Adapter ins Beta Repository setzen.

@ice987987
Please request addition to iobroker repositories once the adapter is working for users.
see https://github.com/ioBroker/ioBroker.repositories?tab=readme-ov-file#requirements-for-adapter-to-get-added-to-the-latest-repository
If you need any help, feel free to ask.
Maybe you could joind our delevoper channels at telegram. Invite links see www.iobroker.dev

Just a kind reminder:
Please request addition to iobroker repositories once the adapter is working for users.
see https://github.com/ioBroker/ioBroker.repositories?tab=readme-ov-file#requirements-for-adapter-to-get-added-to-the-latest-repository
If you need any help, feel free to ask.
Maybe you could joind our delevoper channels at telegram. Invite links see www.iobroker.dev


32 bit appimage segmentation fault

Hello, I'm a novice so forgive me if this turns out to be a waste of your time. I couldn't get the 32 bit appimage version to run without a segmentation fault without first copying the contents of my non-appimage installation located at  "/home/pi/Desktop/Prusa/resources/profiles" to the "/usr/local/share/PrusaSlicer/profiles". After all the files were copied over the 32bit appimage runs correctly.

I also got segmentation error when trying to install the AppImage

@asperger514 Thanks for the report! I was able to reproduce this and it looks like its not picking up the profiles embedded within the AppImage. There is an assert that fails before that as well.. I'll have a closer look soon and will see if there is a solution that doesn't require a workaround.
../src/common/ctrlsub.cpp(232): assert "IsValid(n)" failed in GetClientData(): Invalid index passed to GetClientData(
(prusa-slicer:29360): Gtk-WARNING **: 09:48:06.425: gtk_window_set_titlebar() called on a realized window
[2021-02-19 09:49:06.471381] [0xb13bb010] [error]   stl_open_count_facets: Couldn't open /usr/local/share/PrusaSlicer/profiles/Creality/ender3_bed.stl for reading
Segmentation fault

@sab0276 Hello! Apologies for the issue. Some things to check that might help narrow down the cause of the issue: Is $DISPLAY populated? (check with echo $DISPLAY on the command-line)
..and does running ./PrusaSlicer-2.3.0-GTK3-armhf.AppImage --help produce help output (or does it return the same error straightaway)?

Hello,
Every image for the latest release has been re-generated and re-uploaded. This should let PrusaSlicer find the printers that are embedded inside the AppImage.
The assert that occurs before the segfault is an unsolved issue; it looks like some data is being collected for it upstream.

Still getting the error when trying to install PrusaSlicer-2.3.0-GTK3-armhf.AppImage.

@sab0276 Hello! Apologies for the issue. Some things to check that might help narrow down the cause of the issue: Is $DISPLAY populated? (check with echo $DISPLAY on the command-line)
..and does running ./PrusaSlicer-2.3.0-GTK3-armhf.AppImage --help produce help output (or does it return the same error straightaway)?

I just ran echo $DISPLAY did not return anything.
Running ./PrusaSlicer-2.3.0-GTK3-armhf.AppImage --help did return the help info right away.

@sab0276 Thanks for the information! There was another user that had a similar experience in issue #4 some time ago..
It looks like your dependencies are ok, but PrusaSlicer looks for $DISPLAY during launch. When it is empty it will segfault -- $DISPLAY is typically is not filled over an SSH session.
There are two workarounds when using PrusaSlicer over SSH, using X11 forwarding to your local system (this is typically something like ssh -X<EMAIL_ADDRESS>/home/pi/PrusaSlicer-2.3.0-GTK3-armhf.AppImage) or pre-pending $DISPLAY during the AppImage launch (this will put PrusaSlicer on the monitors attached to your Pi, but will not allow control over SSH): DISPLAY=":0" ./PrusaSlicer-2.3.0-GTK3-armhf.AppImage.
The other alternative use over SSH is to slice purely on the command-line with a GUI (as in issue #4).

Ah gotcha.  I was trying to install the program over SSH, but will be running it on the desktop.  I didn't realize running the AppImage ran the app, I just thought it installed the app to be run later.  So anytime I want to run the app I have to run ./PrusaSlicer-2.3.0-GTK3-armhf.AppImage from the directory I downloaded it to?

There is a way to manually add one. Running these commands should place a launcher in the 'Graphics' category of the menu. Be sure to edit the Exec= line to fit your setup.
curl -SsL https://raw.githubusercontent.com/prusa3d/PrusaSlicer/master/resources/icons/	PrusaSlicer_128px.png | sudo dd of=/usr/share/pixmaps/PrusaSlicer_128px.png

sudo tee /usr/share/applications/PrusaSlicer.desktop <<'EOF'
[Desktop Entry]
Name=PrusaSlicer
Exec=/home/pi/Downloads/PrusaSlicer-2.3.0-GTK3-armhf.AppImage
Icon=/usr/share/pixmaps/PrusaSlicer_128px.png
Terminal=false
Type=Application
Categories=Graphics;3DGraphics;
EOF

Thank you for all your work, I removed the superfluous workaround directory copy of settings from "/usr/local/share/PrusaSlicer/profiles" and the new appimage works great.

This worked great, thank you!


update to mirte logo

Resized the logo I had to 128x64 pixels and changed it to 1 pixel black and white.

Perfect ;)


Ident exposing allocation free &str

I noticed that Ident has to go through to_string for exposing its contained value. That seems to allocate memory, but please, correct me if I'm wrong. I've been checking the rust code and it leads me to String::fmt calling Display::fmt. So my question is easy, in the case that I'm not wrong, why isn't there a way to return an str without extra memory allocations. Thanks!

Currently there's no way to access the value of an Ident without allocating, no. It's not currently planned for upstream proc-macro itself, but it's flexible! Is the profiling motivation perhaps for removing the allocation?

I have not seen this allocation appear significant when profiling Syn. Parsing involves looking at every identifier so I believe it would show up if the allocation were a bottleneck.
Separately, note that in the case of proc macros, allocating a string takes basically zero time compared to compiling all of the generated code. Compiling the generated code probably takes 6-7 orders of magnitude longer than making a String from every Ident in the macro input. So the optimization suggested in this issue would only possibly improve compile time by something like 0.00001%.

No, I haven't done any profiling. Since one of the goals of this crate was bringing proc_macro to other unknown use cases, and one of the rust signatures is zero-cost abstractions, I thought it might be natural to have it in the API. But of course, I see that it could make a lot of sense that the codebase can't afford a new method without real numbers backing it.
Thanks for taking your time responding me, I didn't know that in proc macros, working through the parsed TokenStream would have so little impact in performance compared to compiling the generated TokenStream. That's very interesting.

I'll close for now but let's revisit if anyone has a benchmark where this is significant for their use case.


Remove --path vendor/bundle from Readme.md

Bundler 3 has deprecated the use of --path flag when running bundle install. More info here.
Therefore, we should update the command in our readme from:
bundle install --path vendor/bundle

to
bundle config set --local path 'vendor/bundle'
bundle install

@robzwolf and @kronaemmanuel I'll just deal with this on issue #18

Okay @SnowPrimate, I'll assign this to you then.


Combine some background image with foreground image and mask

Good day!. How to do it using imgaug?

@SergeyMalashenko was you able to solve it?


hitContext.getImageData causing crash

Setting input.pixelPerfectOver=true on a Phaser.Sprite object causes the following error :
Uncaught SecurityError: Failed to execute 'getImageData' on 'CanvasRenderingContext2D': The canvas has been tainted by cross-origin data.
First yes my Access-Control-Allow-Origin "*" is set properly.
Secondly if I turn pixelPerfectOver=false I have no problems with the displaying and interacting through input/events on the Phaser.Sprite.
Example Source:
var i, iTot,
    bPosX = [35, 265, 35, 265],
    bPosY = [75, 75, 305, 305],
    cBtn;

iTot = bPosX.length;
for (i=0;i<iTot;i++) {
    cBtn = game.add.sprite(bPosX, bPosY, 'BuyCards', 'BuyCard'+(i+1)+'.png');
    cBtn.inputEnabled = true;
    cBtn.input.pixelPerfectOver = true;  // <-- commenting out this line will remove the error
    cBtn.input.useHandCursor = true;
    cBtn.events.onInputUp.add(this.btnUp, this);
    this.cardSelAr.push(cBtn);
}

Problem was resolve by adding:
this.load.crossOrigin = 'Anonymous';
in the create: function() {} of my first state, in this case "Boot.js" as per the Phaser Game template.

This is a local configuration issue, rather than a Phaser one. If you've tainted the canvas somehow (or in this case the image it's trying to read from) then this error will occur. There's nothing Phaser itself can do about it, you just have to correctly handle CORs loading of the assets in the first place.

I already stated the local configuration isn't the issue at all here. Even the comments in the phaser.js also point out that crossOrigin param on the Loader object needs to be changed.
I've got'n this same response everywhere its a CORs issue... and it isn't. It only happens on hitcontext but whatever it was resolved but updating and any following devs can see this answer instead of having to argue on every form to fight through this issue.
This was brought up in :
https://github.com/photonstorm/phaser/issues/1684
By you photonstorm.

#1684 is unrelated to this. That is about the base object not actually having a texture that can be pixel checked (i.e. a BitmapText object) that maps to what is displayed on-screen, hence why it was flagged as a feature request and not an issue.
Your issue however is 100% CORs related I'm afraid. From MDN:
What is a "tainted" canvas?
"Although you can use images without CORS approval in your canvas, doing so taints the canvas. Once a canvas has been tainted, you can no longer pull data back out of the canvas. For example, you can no longer use the canvas toBlob(), toDataURL(), or getImageData() methods; doing so will throw a security error.
This protects users from having private data exposed by using images to pull information from remote web sites without permission."
Which is exactly what happens here. The canvas is tainted because the image wasn't CORs loaded, and doing a pixel perfect hit test uses getImageData (as it needs to read pixel values from the image), which is what throws the browser warning. The canvas was always tainted, you just didn't notice it because browsers don't complain about it until you try to do something they may consider a security risk.
There are ways around it, sites like https://crossorigin.me/ exist for example, or if you have control over the origin server then setting the correct response headers works too. Either way it's local to your server set-up and is nothing Phaser can check for.


C# syntax highlighting is wrong of identifiers starting with an underscore

There's been a similar problem before, see:

https://github.com/alecthomas/chroma/issues/507
https://github.com/gohugoio/hugo/issues/8532

Visit https://opentelemetry.io/docs/instrumentation/net/netframework/, notice the _ in _tracerProvider is highlighted differently:

The entire identifier should be highlighted the same, like this:
 private readonly TracerProvider _tracerProvider;

As of the time of writing this, we're using Hugo 0.9.2, which uses v0.9.4, the version of Chroma specified here: https://github.com/gohugoio/hugo/blob/cdf6a0d6228eef4df2a82a1483f016c11f4a6019/go.mod#L7
Maybe this has been fixed in the more recent 0.x version or Chroma v2? Not sure if/when Hugo will jump to v2.
To do:

[x] Test if latest version of Chroma still has the issue
[x] File an issue: https://github.com/alecthomas/chroma/issues/613
[x] Propose a fix: https://github.com/alecthomas/chroma/pull/614
[ ] Wait for the next Chroma release
[ ] Open an issue over Hugo to update Chroma?
[ ] Upgrade Hugo

😅

So I think this can be closed...insofar as the presence of the underscore doesn't seem to impact colorization anymore. The colorization isn't particularly knowledgeable about C# constructs as a whole, but uhhh, it's at least consistently dissatisfying.

Yes, all done.


feat: add rbac options to helm chart

This change allows the Helm chart be installed without including the creation of cluster-wide resources (ClusterRoles/ClusterRoleBindings). Specifically, this change supports at least 2 use cases:

The scenario where the engineer installing the chart has limited (i.e. namespace only) access and cannot install cluster-wide resources. This change allows the cluster-wide resources to be installed separately by cluster administrators while still allowing the engineers to manage the operator itself without having full cluster-wide access.

The scenario where the operator only needs to manage resources in a specific namespace and not the entire cluster.

I believe this is the first pull request into this repository since moving the Aiven charts from its dedicated repository so if a separate PR needs to be made there, let me know and I'd be happy to open one.

Hi @ryaneorth !
Thanks for the contribution! May I ask you to add a line to the changelog, please?
Regarding moving the charts to this rep, it is still in progress. I need to do few things first that may take a week or two. Then we can release this charts.
Is this urgent for you (the new chart)? Just to know if I need to prioritise this. Thanks.

Hi @byashimov - I've added a line item to the change log. This isn't incredibly urgent, but if we could have a new version of the chart released with these changes in the next week or so that would be very helpful!


fermyon-spin: 2.1.0 -> 2.2.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/fermyon/spin/releases.
meta.description for fermyon-spin is: Framework for building, deploying, and running fast, secure, and composable cloud microservices with WebAssembly.
meta.homepage for fermyon-spin is: https://github.com/fermyon/spin
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.2.0 with grep in /nix/store/4j3gmd3hcqvb90fi2kvy1k73cjx07kg7-fermyon-spin-2.2.0
found 2.2.0 in filename of file in /nix/store/4j3gmd3hcqvb90fi2kvy1k73cjx07kg7-fermyon-spin-2.2.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
fermyon-spin

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/4j3gmd3hcqvb90fi2kvy1k73cjx07kg7-fermyon-spin-2.2.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A fermyon-spin https://github.com/r-ryantm/nixpkgs/archive/ffa28e91a7dfbe9bec33cf8d3ef388823664cee6.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/ffa28e91a7dfbe9bec33cf8d3ef388823664cee6#fermyon-spin

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/4j3gmd3hcqvb90fi2kvy1k73cjx07kg7-fermyon-spin-2.2.0
ls -la /nix/store/4j3gmd3hcqvb90fi2kvy1k73cjx07kg7-fermyon-spin-2.2.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

fermyon-spin

Maintainer pings
cc @MGlolenstine for testing.

Result of nixpkgs-review pr 285199 run on x86_64-linux 1

1 package built:

fermyon-spin


Publish in Terraform.io registry to prepare to be available for terraform 0.13

Provider binaries can be automatically hosted by Terraform registry as of MajorRelease 0.13:
cf. https://github.com/hashicorp/terraform/tree/guide-v0.13-beta/provider-sources

@ggongaware What do you think about this issue. It can be sufficient to do this and close #80 .

I opened an issue on Terraform repository because I can't find a way to install proxmox provider with actual implementation. cf. https://github.com/hashicorp/terraform/issues/25172

👋 I work on the Providers / Registry team at HashiCorp, happy to help you all get this published as well.  Not sure if anyone has applied for this provider yet for access to publishing in the closed beta, but to get invited to the closed beta, please email<EMAIL_ADDRESS>We need:

A list of GitHub usernames to add to the beta (you and any team members who will publish)
List of provider repositories you plan to publish
A GPG public key in ASCII-armor format, which you will be using to sign your provider releases

You can use one key for all of your providers, or separate keys if you prefer. If you are publishing from an organization, this key or keys will be associated with that namespace. Once in the beta, you can manage personal keys in the UI as well.

I'm actually interested to take part of this as I am currently testing this provider with terraform 0.13.0-beta1 to use it inside modules

I agree this would be useful, but of course requires the provider and provisioner binaries to be compiled for each supported OS. I suspect you will know already but you can publish the Proxmox Terraform provider and provisioner to a local mirror. I use this approach and can confirm that it is successful, following the docs from HashiCorp here: https://www.terraform.io/docs/configuration/provider-requirements.html#in-house-providers

We have a goreleaser.yml config file that will do all the cross compilation, summing, signing, for you, etc to create registry compatible releases, its referenced in our publishing docs: https://www.terraform.io/docs/registry/providers/publishing.html#creating-a-github-release
We also have a GitHub action that will automatically run GoReleaser for every v* tag: https://github.com/hashicorp/terraform-provider-scaffolding/blob/master/.github/workflows/release.yml

I added the plugin locally using information from
https://learn.hashicorp.com/tutorials/terraform/provider-use
Changes your steps in Copy executables to plugin directory
https://github.com/Telmate/terraform-provider-proxmox/blob/master/docs/installation.md
from  ~/.terraform.d/plugins
to  ~/.terraform.d/plugins/telmate.com/tf/proxmox/0.1/linux_amd64
Add section to main.tf
terraform {
    required_providers {
        proxmox = {
            source = "telmate.com/tf/proxmox"
        }
    }
}

Then terraform init succeeds

If you're running on Darwin (OSX) it will look like this:
$ cp bin/terraform-provider-proxmox ~/.terraform.d/plugins/telmate.com/tf/proxmox/0.1/darwin_amd64
$ cp bin/terraform-provisioner-proxmox ~/.terraform.d/plugins/telmate.com/tf/proxmox/0.1/darwin_amd64

Closed by #228


Friendly API for attribute query.

Adds a friendlier API for querying attributes.

Awesome! Thanks for doing that! =)


Minimum coins problem

What does this PR do?
This PR adds the code to calculate the minimum number of coins required to obtain the sum as the total amount.
Test Plan
No
Related PRs and Issues
This PR is related to issue #577
Have you read the Contributing Guidelines?
Yes

@kavinraju thanks for the contribution.

Welcome @dubesar :)

Hi @dubesar Please add hactoberfest-accepted label this PR.


i don't know what to do about this

I hope you can help,
it says me this error in node.js after I type "npm install" - "npm WARN<EMAIL_ADDRESS>No repository field."
anyone know what is this? xd, I feel a bit dumb about what I ask, but I'm really new about this
EDIT:
When I do "node index..js"
it hosts the map, but no one shows up on that map, it is just like the picture.
I already started it there https://github.com/nakating/PUBG-maphack-map/issues/31
but Nakating said that I should ask you

I don't know why he said to ask me, as I did not make the nodejs version of the map, he did.The problem seems to be something nodejs related that I know nothing about

Could you help me out with your part? like I don't understand much about visual studio and would be thankful if you help me that thing out

i have visual studio 2017

nakating said:  you need the driver, usermode app, and the map. both of them. if anyone of them got some error, the whole system couldn't work. good luck.

so what should I do, I have everything installed, but when I try to debug hackControl, it pops error out that could not find windows sdk version.........

oh, i see what your issue is

like I don't understand much about visual studio

you should learn more about visual studio
hope this helps!

This project has stopped updating, even if you compile it, it can't be used. Because the game is encrypted

oh,

i see what your issue is
like I don't understand much about visual studio
you should learn more about visual studio
hope this helps!
nice 12 year old autism response kiddo.


Error: None

Hi, we followed the Docker submission example you provided, but we encountered an issue where we are unable to submit the Docker image.

The issue is resolved.


Rename samsunginternet.svg to samsung_internet.svg

To be consistent with other multi-word names

I'm going to manually resolve this - can't do it on GitHub.


Fix the fact that childrens classNames are not merged anymore

Following the issue #159 , here is the fix I suggested in the comments.
Just checked if the child.props.className exists, if so, merging these props with the calculated classes from the getSlideClasses method.

My bad... created the fix branch from master instead of 0.14...
Closing this PR.

Thanks @websilone
Can you open a new PR branching from 0.14

Yes I already did that... #161


"Duplicate content entry slug" error occurred with i18n in some pages.

Most appropriate sections of the p5.js website?
Other (specify if possible)
What is your operating system?
None
Web browser and version
No response
Actual Behavior
I have attempted to internationalize some pages.
(In my case, I am working on translating the Japanese version.)
But "Duplicate content entry slug" error occurred when copy "en" folder to non-English lang folder (e.g. es) in pages follows:
/src/content/pages/
/src/content/tutorials/
Expected Behavior
These pages also support internationalization.
(However, I cannot discern whether this is a bug or a policy that does not require internationalization.)
Steps to reproduce
In your development environment.
e.g. Copy /src/content/pages/en to /src/content/pages/es
e.g. Copy /src/content/tutorials/en to /src/tutorials/es
Would you like to work on the issue?
someone

hey @shibomb
can i work on this , i am  new to this community.

hey @shibomb can i work on this , i am new to this community.

Hi @Nitin-Budakoti
After creating this Issue, I continued my research and found a solution, so I created a PR for it.
Thank you for your interest in this issue!


[CS2103-W13-2] SmartLib

SmartLib is a platform for private book loaning service to keep track of books and customers' information digitally.

Codecov Report

Merging #7 (b99ad29) into master (c36220c) will decrease coverage by 0.11%.
The diff coverage is 74.78%.

@@             Coverage Diff              @@
##             master       #7      +/-   ##
============================================
- Coverage     72.15%   72.04%   -0.12%
  Complexity      399      399
============================================
  Files            70       72       +2
  Lines          1232     1234       +2
  Branches        125      125
============================================
  Hits            889      889
- Misses          311      313       +2
  Partials         32       32

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/seedu/smartlib/AppParameters.java
75.00% <ø> (ø)
9.00 <0.00> (?)

src/main/java/seedu/smartlib/Main.java
0.00% <ø> (ø)
0.00 <0.00> (?)

src/main/java/seedu/smartlib/MainApp.java
0.00% <0.00%> (ø)
0.00 <0.00> (?)

.../main/java/seedu/smartlib/commons/core/Config.java
68.18% <ø> (ø)
7.00 <0.00> (?)

.../java/seedu/smartlib/commons/core/GuiSettings.java
69.23% <ø> (ø)
5.00 <0.00> (?)

...n/java/seedu/smartlib/commons/core/LogsCenter.java
78.37% <ø> (ø)
10.00 <0.00> (?)

...ain/java/seedu/smartlib/commons/core/Messages.java
0.00% <ø> (ø)
0.00 <0.00> (?)

...main/java/seedu/smartlib/commons/core/Version.java
82.50% <ø> (ø)
19.00 <0.00> (?)

.../java/seedu/smartlib/commons/core/index/Index.java
100.00% <ø> (ø)
10.00 <0.00> (?)

...ib/commons/exceptions/DataConversionException.java
100.00% <ø> (ø)
1.00 <0.00> (?)

... and 64 more

Continue to review full report at Codecov.

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


workflow engine not started if /TaskHubSidecarService/GetWorkItems is not called

In what area(s)?

/area runtime
What version of Dapr?

edge: v1.11.3-549-gcc9d51e1-dirty

Expected Behavior

I'm testing the the upcoming workflow go sdk and I was getting an error like did not find address for actor dapr.internal.myns.myapp.workflow/a7a4168d-3a1c-41da-8a4f-e7f6d9c718d9 when my app tried to start a workflow. Its expected to not have any errors and my workflow to start successfully.
Actual Behavior

As I said I was getting an error like  did not find address for actor dapr.internal.myns.myapp.workflow/a7a4168d-3a1c-41da-8a4f-e7f6d9c718d9
Digging into my dapr sidecar logs I noticed only two requests were being made in my test:

first a request to /TaskHubSidecarService/Hello
then a request to /dapr.proto.runtime.v1.Dapr/StartWorkflowBeta1

The error I was getting means that the internal actors used for workflow were not actually registered, and looking at the workflow engine code, the internal actors are only registered when the workflow engine is started and I wasn't seeing this log entry https://github.com/dapr/dapr/blob/master/pkg/runtime/wfengine/wfengine.go#L160C17-L160C40
So the root cause must have been that the go sdk was not making a request to /TaskHubSidecarService/GetWorkItems and because of that this callback was not being executed https://github.com/dapr/dapr/blob/master/pkg/runtime/wfengine/wfengine.go#L90, resulting in the workflow engine not being initialized...
A simple solution I can propose consists of adding a callback to autoStart the workflow engine to https://github.com/dapr/dapr/blob/master/pkg/runtime/wfengine/component.go#L53 and that callback would be called on every function of workflowEngineComponent except for the Init function, this way whenever a request to the workflow management APIs it will be guaranteed that the workflow engine is started
Steps to Reproduce the Problem

Try the workflow example included here https://github.com/dapr/go-sdk/pull/487
Release Note

RELEASE NOTE:

cc @cgillum @RyanLettieri

Indeed, this is related to the work we did to start the workflow engine only after we've detected that an app has connected to it. This is unfortunately one of the side effects of that design decision. I think your proposed solution sounds like a good one that still maintains the spirit of the original delayed-start design.
One thing I'll note about that sample, however, is that we strongly discourage developers from using the DaprClient to manage workflows since that API was designed to be generic for any arbitrary workflow engine (a strategy we have since abandoned). They should instead use the DaprWorkflowClient (or equivalent for their language), which has access to the full feature set and metadata for Dapr workflows. I'll need to double check the Go SDK contribution to see whether it's properly exposing this functionality.

Thinking of this, if a client has not called GetWorkItems before it tries to start a new workflow if that fails that is a problem of the application.
I've got my sample go application finally working and the source of my problems was that the GetWorkItems was silently failing for me