Exception shown when navigating to endpoint URL in browser

This bug affects the developers familiar with Bot Framework 3.x. Launching bot projects built with v3 resulted in the browser automatically navigating to this URL that now, with v4, displays the exception. Do note that the bot project will keep up running fine, but because this GET operation now results in the exception, it interrupts your development when you start looking for the way to fix the issue.
Steps to reproduce:

Have any simple bot project (echo bot will do)
Launch the project (localhost)
When the project is running, in your browser, navigate to localhost:<port>/api/<controller> e.g. localhost:1234/api/messages
Be shocked to see an IoC exception "InvalidOperationException: Could not create an instance of type 'Microsoft.Bot.Builder.IBotContext'."

This is a great observation. I'll see what I can do about fixing it.

@cleemullins are you tackling this or is it open?

I'm not yet tackling it. If someone wanted to help, that would be awesome.
For extra, extra credit, have the web site for the bot be (optionally) the WebChat control hosted via a  ! 🥇

For details here's what the exception looks like: Internal Server Error.html.txt

Can this issue be reproduced now? I can't find a way to do it, localhost:/api/ is replying a blank page on samples projects

I’m not sure. Drew’s changes to the WebApi and Asp.net core integration may have fixed this.
Room for improvement certainly remains. Click to launch the emulator, the WebChat Control, or even a live bot-stats page would all be awesome!

From: Nicolas<EMAIL_ADDRESS>Sent: Thursday, March 8, 2018 1:45:55 AM
To: Microsoft/botbuilder-dotnet
Cc: Chris Mullins; Mention
Subject: Re: [Microsoft/botbuilder-dotnet] Exception shown when navigating to endpoint URL in browser (#195)
Can this issue be reproduced now? I can't find a way to do it, localhost:/api/ is replying a blank page on samples projects
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fbotbuilder-dotnet%2Fissues%2F195%23issuecomment-371435548&data=04|01|cmullins%40microsoft.com|1a08461a55984318aa1808d584db8764|72f988bf86f141af91ab2d7cd011db47|1|0|636561000760381927|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwifQ%3D%3D|-2&sdata=EJ6SFnbcUoJy0dYM7dkPUjy7nNSLTp7gAVvJjRjJh6k%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABHICY5Rqd6lMzawxb2kedDXaBPYWpCQks5tcP3TgaJpZM4STavc&data=04|01|cmullins%40microsoft.com|1a08461a55984318aa1808d584db8764|72f988bf86f141af91ab2d7cd011db47|1|0|636561000760381927|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwifQ%3D%3D|-2&sdata=8k0ByxSsHbfkNnBGgu54zdp7%2FxxSslSYQKaOR%2BTB9KU%3D&reserved=0.

Sure. I had a quick look to Drew's change, it's nice to be able to easily move the main bot's controller in UseBotFramework!
So for the improvement, do you really think this "quick access page" should be hosted on the api path? Wouldn't it be better to host it in the default page?

Great point!
The default page is where I envision it, not on the API endpoint.

From: Nicolas<EMAIL_ADDRESS>Sent: Thursday, March 8, 2018 2:55:06 AM
To: Microsoft/botbuilder-dotnet
Cc: Chris Mullins; Mention
Subject: Re: [Microsoft/botbuilder-dotnet] Exception shown when navigating to endpoint URL in browser (#195)
Sure. I had a quick look to Drew's change, it's nice to be able to easily move the main bot's controller in UseBotFramework!
So for the improvement, do you really think this "quick access page" should be hosted on the api path? Wouldn't it be better to host it in the default page?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fbotbuilder-dotnet%2Fissues%2F195%23issuecomment-371453565&data=04|01|cmullins%40microsoft.com|5c609458f6ab49b8218908d584e30dc6|72f988bf86f141af91ab2d7cd011db47|1|0|636561033079697787|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwifQ%3D%3D|-2&sdata=rmzLSbBeOxLs9zG%2B60NGhQ2pDNOY4EGM1jKLeLv%2FXw0%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABHICb3lrEvBwUkj1xWVreU5QMzA8hnVks5tcQ4JgaJpZM4STavc&data=04|01|cmullins%40microsoft.com|5c609458f6ab49b8218908d584e30dc6|72f988bf86f141af91ab2d7cd011db47|1|0|636561033079697787|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwifQ%3D%3D|-2&sdata=wqURyACPqcvfnSdMePZyFPEWGj1PDgK44LZcZvMJ%2BhU%3D&reserved=0.

I agree.
I like the idea of this quick access page, but at the moment I can't see how:

Webchat Control =>it will be difficult to add a Webchat Control that will connect to the localhost bot as the Webchat needs that the bot to be deployed (and getting a DirectLine secret)
Click to launch the emulator => can't launch it on user's computer without knowing the location, etc

So for the improvement, do you really think this "quick access page" should be hosted on the api path?

The samples currently have UseStaticFiles() as well, so for anything related to trying to use the WebChat control, I would suggest just making sure to include a default.htm and hosting the control in there. That said, it sounds like WebChat won't work for localhost dev? 🤔

Click to launch the emulator => can't launch it on user's computer without knowing the location, etc

This is probably a good issue to file over on the emulator's repo, but if they registered a protocol handler, say maybe ms-bot-emulator:, then other apps could just launch it via that without having to know where it's installed. Another approach would be for them to just set an environment variable on install (which seems to be coming back into favor these days)... maybe MS_BOT_EMULATOR_PATH. Or, finally, just add themselves to PATH itself on install so all people would ever have to do is set the command to the botframework-emulator.exe executable itself.

The new Bot Emulator is registering a protocol handler, and once that's in we'll update the auto-generated pages to support that
Holding off on resolution until I can verify the behavior has been fixed.

This was fixed in the last iteration of the ASP.Net Integration code.


Expose tile as a switch (not fan)

Hi,
Firstly thank you for this plugin, it has been very helpful to get my vacuum into homekit.
One little problem I have with this plugin however is that because it is exposed as a fan, if I ask Siri to turn on my fan, it would also turn on this vacuum as they are in the same room.
Would it be possible to add a function to expose this plugin as an on/off switch.
Thanks

That's an interesting approach. How would it work for the modes? One switch for each "speed"?

That's an interesting approach. How would it work for the modes? One switch for each "speed"?

Well, I can only speak for myself, but realistically as soon as you choose the suction speed inside the Mi app, you have very little reason to change it.
The reason I find for wanting to expose the vacuum inside of homekit is to be able to run it when certain conditions are met. This is only possible to do inside of homekit, so in my case, just having a switch to start vacuuming, with no modes, would work well.
However I do like your idea, of having one switch exposed inside of homekit, where the default speed can be defined inside of the homebridge config, and if not speed is defined inside of homekit, it will use the speed it is currently set to.

I too would find exposing as a switch useful as I have the multiple fan problem. I would not mind a basic approach where only one speed could be configured via the plugin, or multiple switches for each speed.

same here, would like switch

With the latest refactor I just went through, I think this request would be fairly easy to apply.

Published as https://github.com/homebridge-xiaomi-roborock-vacuum/homebridge-xiaomi-roborock-vacuum/releases/tag/v0.25.0
Set "serviceType": "switch" in your config and let me know if that works as you expect 😇


Move UserView to Infrastructure layer

Fixes #22
I was having problems with the previous merge 😢
So this is the final PR.

I'll add a schema validate to the ci test suites

This time i will do a proper rebase 😃

Pull Request Test Coverage Report for Build 29

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.118%

Totals

Change from base Build 28:
0.0%

Covered Lines:
433

Relevant Lines:
465

💛  - Coveralls

Thanks for the contribution!


[Feature request] Add example gradle project for selenide with selenoid

Thanks for selenide-selenoid!
I try add selenoid to https://github.com/Drill4J/realworld-java-and-js-coverage/tree/Selenoid
Run: ./gradlew clean :build1:test -Dselenide.remote=http://localhost:4444/wd/hub
But selenide use local browser in windows.
Please, Add example gradle project for selenide with selenoid.
Thanks!

There is a sample project: https://github.com/selenide-examples/selenoid-sample
(It uses Maven, not Gradle - but it doesn't matter for your problem).
In your project, you set Configuration.remote AFTER calling open.
That's why method open (in @BeforeAll method) opens a local browser. Configuration.remote is not yet set at that moment.


fix build cmd

https://github.com/open-cluster-management/backlog/issues/10627
Signed-off-by: Song Song Li<EMAIL_ADDRESS>

/lgtm
/approved


No need to manually force port 80 when X-Forwarded-Port is not specif…

…ied.  This results in a bug when having X-Forwarded-Proto as https and X-Forwarded-Port not specified. It will produce a URL with https and port 80 appended at the end.  This bug is prevelant on the most current App Engine Java Flexible environment. It specified the scheme, but no port which causes the code to fail and redirect to a non-existant url.

@habuma please merge

@habuma


Create a setup.py script

I don't really have much experience in the way of packaging and publishing python modules, but that's probably something I should get around doing if I want people to be able to use this.

You are absolutely right about that.
1/You can look at a barebone example that I have written recently here: https://github.com/blackfireio/python-sdk/blob/master/setup.py
2/You should definitely have some kind of unit testing. I would suggest Travis CI.
3/ I suggest following flow: when you tag something in master branch, a release job will be triggered in CI which builds the python package and converts it to a .whl via using python setup.py bdist_wheel. Then you will publish the package to PyPI. For publishing: twine is your friend. It is very simple to use and integrate.
At this point the package should be installable just by pip install asgi-server-timing-middleware
4/ Then announce in python-announce, reddit or whatever. You might know these better than me:)
BTW, an idea: asgi-server-timing-middleware seems a bit too verbose. Maybe more compact name? Not sure. This is coming from a guy who wrote yet another ... profiler :)

1/ You can look at a simple one that I have written recently here: https://github.com/blackfireio/python-sdk/blob/master/setup.py

Thanks! That was a pretty good starting point.

2/ You should definitely have some kind of unit testing. I would suggest Travis for CI.

Yeah, I've been telling myself the same thing since I created this repo, but I'm still trying to figure out how one is supposed to unit-test a timing/profiling tool, given execution time falls outside of the function-as-a-unit concept. There's also how the entire module is basically one class with a constructor and a __call__ method right now, so it's not exactly setup for testing, but I'll try and figure something out.

3/ I suggest following flow: when you tag something in master branch, a release job will be triggered in CI which builds the python package and converts it to a .whl via using python setup.py bdist_wheel.

I'll keep that part on ice until I figure out how I'm going to go about number 2/

Then you will publish the package to PyPI. For publishing: twine is your friend. It is very simple to use and integrate.
At this point the package should be installable just by pip install asgi-server-timing-middleware

That's going to be next on my list. PyPI has a test instance to let people test package uploading, so I'll be sure to try it on there first in case I mess up.

4/ Then announce in python-announce, reddit or whatever. You might know these better than me:)

Hmm, I might want to at least raise it on the FastAPI issue and on awesome-asgi.

Do not hesitate to ask for help at any of these. I think I can help.

After all you've already done on yappi those past months to even make this possible? I'd still want to have some merit left for myself. ;)

BTW, an idea: asgi-server-timing-middleware seems a bit too verbose. Maybe more compact name? Not sure. This is coming from a guy who wrote yet another ... profiler :)

I'm keeping the name descriptive and to the point because I don't think I would want to "brand" this project. It holds in less than 100 lines of python right now and I don't really expect it to grow past 200. Note that the class is just called ServerTimingMiddleware, which makes it sound like a bit less of a mouthful.

Yeah, I've been telling myself the same thing since I created this repo, but I'm still trying to figure out how one is supposed to unit-test a timing/profiling tool, given execution time falls outside of the function-as-a-unit concept. There's also how the entire module is basically one class with a constructor and a call method right now, so it's not exactly setup for testing, but I'll try and figure something out.

Yeah the testing might be tricky. Not sure if this helps, but you might want to have a look at here maybe: https://github.com/florimondmanca/ddtrace-asgi/blob/master/tests/test_trace_middleware.py

I'm keeping the name descriptive and to the point because I don't think I would want to "brand" this project.

Makes sense.


Add Type Hinting and Doc Strings

Describe the solution you'd like
A description of what you would like pyglet to do, or to do differently.
Add Type Hinting and Doc Strings to functions and classes that Don't Have it.
This should just exist as a reminder to anyone who is looking for something to do.
EDIT: I know this is sort of a duplicate of https://github.com/pyglet/pyglet/issues/643, but I wanted to add doc strings.

I've already started on this for font loading. This is going to be a long journey since the codebase is probably older than some of the contributors, and may not be suited to modern typing or import practices at the moment.


including previouly missing graphics and improving the text of cards 💄

Adds graph images that were previously accidentality not included in previous commits & replaces NaN values on cards with "Not Tested"

Error produced in card generation by string values such as inactive


FluentDOM 5.x 6x slower than 4.x ?

Why is this piece of code 6x slower with FluentDOM 5.x compared to 4.x ? (HHVM or not)
$fd = FluentDOM($html, 'text/html');
$r = array();
foreach ($fd->find("//tr[@class='product']") as $fd_child)
{
	$rr = array();
	$rr['imgsrc'] = $fd_child->find("td[@class='image']//img")->attr("src");
	$h3 = $fd_child->find("td[@class='specs']//h3");
	$rr['url'] = $h3->find("a")->attr("href");
	$rr['title'] = $h3->text();
	$rr['desc'] = $fd_child->find("td[@class='specs']")->xml();
	$rr['price'] = $fd_child->find("td[@class='purchase-info']//span[@itemprop='price']")->text();
	$rr['savings'] = $fd_child->find("td[@class='purchase-info']//p[@class='savings']")->text();
	$r[] = $rr;
}

Which FluentDOM 5.x? I changed find() in 5 to match the jQuery behavior. But this had an impact on the performance. So I optimized it with 5.2 and added the other behavior as an option.
Have you tried it with 5.2?

4.1.5 vs. 5.2.1 (same for 5.1.x)
Ubuntu 14.04
PHP 5.5.9 / CLI mode

Version
Time

4.1
1.20

5.2 (no optimization)
2.47

5.2 (1. opt: avoid sort)
1.89

5.2 (Document)
0.72

Thanks for your help, I will work on a sample script (with an HTML file) to help you track the problem.

@ThomasWeinert Here is the PHP sample script (http://pastebin.com/B4jMxWqQ) and HTML data (http://pastebin.com/FtDmzjjJ)
With FD 4.1.2 :
# time php -f test_fd.php
php -f test_fd.php  3.18s user 0.03s system 99% cpu 3.217 total

With FD 5.2.0 :
# time php -f test_fd.php
php -f test_fd.php  26.73s user 0.10s system 99% cpu 26.919 total

(Ubuntu 14.04, PHP 5.5.9)

Besides this "performance" problem, FD (tested with the same script) is (memory) leaking A LOT with HHVM 3.5.0 ; but I think it's more on HHVM's side...

Could you please test [https://github.com/FluentDOM/FluentDOM/commit/848a4ee18435b116f659b6e3a93ced18ab178182]?

It's a lot better, but it's still 30% slower.
php -f test_fd.php  4.05s user 0.12s system 99% cpu 4.173 total

vs.
php -f test_fd.php  3.17s user 0.05s system 99% cpu 3.227 total

I will keep looking into the xdebug profiling files, but for the moment I don't see something where I could optimize it. Here are new features in the background (namespace handling for example) - so here is some more code to be executed.
btw Could you give me your numbers for this approach? https://gist.github.com/ThomasWeinert/11452b95d6f93376189f

You approach is now faster...
php -f test_fd.php  2.16s user 0.18s system 99% cpu 2.346 total

My job is now to understand what was wrong with my approach since yours is different in many ways.
Anyway, many thanks for your help :-)
I think you can close this issue.

The document based approach uses mostly shortcuts. It basically works a lot more directly with the ext/dom and a lot less php userland function calls.
For example:
$rr['imgsrc'] = $child("string(td[@class='image']//img/@src)");

Is in ext/dom
$rr['imgsrc'] = $xpath->evaluate("string(td[@class='image']//img/@src)", $child);

I can not reproduce the memory leak. If I output the memory_get_peak_usage(TRUE), it will increase with the calls (1MB for a single call, 3.1MB for 250 calls). But if I add gc_collect_cycles() to trigger the GC after each call it only increases to 1.3 MB with 2, 250 or 2500 calls.

OK, thank you for the explanation.
I think you can close this issue.

Added FluentDOM\Nodes::FIND_FORCE_SORT as an option for FluentDOM\Nodes::find() to disable the optimization.
Added a cache for QualifiedName validations (avoids multiple PCRE calls for the same tag name.)


sample app crashes on second attempt

Hello Aidan,
I know it's too early to report an issue if this is irrelevant please ignore it. There is a bug in the sample project included in this library.
Affected Device(s):
Nexus 5X with Android 7.1.1(in AVD)
Samsung Galaxy S6 edge plus with Android 7.0
Describe the Bug:
App crashes when I try to launch the second activity for the second time(without closing the   app after first attempt).
To Reproduce:
Step 1-> Open up the sample app and press on the "launch second activity" button.
Step 2-> Step 1 will open an activity enter something in the Edittext box and press "Success" or
              "Failure" button this will finish the activity and show the result as a Toast.
Step 3-> When I try to do Step 1 again(without closing the app after first attempt) the app crashes.

Logcat:
535-8535/com.afollestad.inlineactivityresultsample E/AndroidRuntime: FATAL EXCEPTION: main
    Process: com.afollestad.inlineactivityresultsample, PID: 8535
    java.lang.IllegalStateException: There is already a pending request for request code 20
        at com.afollestad.inlineactivityresult.InlineActivityResult.schedule(InlineActivityResult.kt:34)
        at com.afollestad.inlineactivityresult.ActivitiesKt.startActivityForResult(Activities.kt:56)
        at com.afollestad.inlineactivityresultsample.MainActivity$onCreate$1.onClick(MainActivity.kt:25)
        at android.view.View.performClick(View.java:5637)
        at android.view.View$PerformClick.run(View.java:22429)
        at android.os.Handler.handleCallback(Handler.java:751)
        at android.os.Handler.dispatchMessage(Handler.java:95)
        at android.os.Looper.loop(Looper.java:154)
        at android.app.ActivityThread.main(ActivityThread.java:6119)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:886)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:776)

Screenshots -----

Hello Aidan,
There is another bug similar to this one but doesn't fit into the normal use case, And it's hard to reproduce.
I will add the details below.
Affected Device(s):
Nexus 5X with Android 7.1.1(in AVD)
Samsung Galaxy S6 edge plus with Android 7.0
Screenshots:

To Reproduce:
Place the mouse pointer on the marked spot(please see the screenshot). And click the buttons(first activity "launch second activity button" and second activity "success" button) continuously(don't mind the activity change(button 1 launch second activity and button 2 finish second activity)). In AVD it's too easy to reproduce but in the real device, it's very hard.

Logcat:
java.lang.IllegalStateException: There is already a pending request for requestCode 69.
        at com.afollestad.inlineactivityresult.internal.InlineActivityResult.schedule(InlineActivityResult.kt:34)
        at com.afollestad.inlineactivityresult.ActivitiesKt.startActivityForResult(Activities.kt:56)
        at com.afollestad.inlineactivityresultsample.MainActivity$onCreate$1.onClick(MainActivity.kt:27)
        at android.view.View.performClick(View.java:5637)
        at android.view.View$PerformClick.run(View.java:22429)
        at android.os.Handler.handleCallback(Handler.java:751)
        at android.os.Handler.dispatchMessage(Handler.java:95)
        at android.os.Looper.loop(Looper.java:154)
        at android.app.ActivityThread.main(ActivityThread.java:6119)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:886)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:776)

Oh, that's expected because the request code isn't changed. Clicks would probably be denounced at the app level or different request codes would need to be used each time.

Thank you for the explanation.


fixed Promise syntax in openPR function

This PR fixes #99
Corrected some syntax in the function that creates a pull request which was returning an empty array.
This empty array triggers the error page even though the submission and the PR have been made.
@lacabra I noticed this on the PR. It seems mine failed with this error
[
  {
    status: 'rejected',
    reason: RequestError [HttpError]: Validation Failed: {"resource":"PullRequest","field":"head","code":"invalid"}
        at /Users/nathanfletcher/Documents/DEVELOPER/unicef/submission-digitalpublicgoods/node_modules/@octokit/request/dist-node/index.js:86:21
        at runMicrotasks (<anonymous>)
        at processTicksAndRejections (node:internal/process/task_queues:96:5)
        at async composeCreatePullRequest (/Users/nathanfletcher/Documents/DEVELOPER/unicef/submission-digitalpublicgoods/node_modules/octokit-plugin-create-pull-request/dist-node/index.js:290:10) {
      status: 422,
      response: [Object],
      request: [Object]
    }
  }
]

It is not blocking the submission, however, I'll address it in another PR so that this can be pushed to make the form work again.

This error is due to the fact that you still have NEXT_PUBLIC_GITHUB_OWNER=unicef set in your .env.local. Since unicef/publicgoods-candidates has been migrated to DPGAlliance/publicgoods-candidates, when octokit module tries to open a PR against unicef/publicgoods-candidates it fails because that repo does not exist, and does not take redirects. This environment variable was updated to DPGAlliance in production when the migration took place

@lacabra I noticed this on the PR. It seems mine failed with this error
[
  {
    status: 'rejected',
    reason: RequestError [HttpError]: Validation Failed: {"resource":"PullRequest","field":"head","code":"invalid"}
        at /Users/nathanfletcher/Documents/DEVELOPER/unicef/submission-digitalpublicgoods/node_modules/@octokit/request/dist-node/index.js:86:21
        at runMicrotasks (<anonymous>)
        at processTicksAndRejections (node:internal/process/task_queues:96:5)
        at async composeCreatePullRequest (/Users/nathanfletcher/Documents/DEVELOPER/unicef/submission-digitalpublicgoods/node_modules/octokit-plugin-create-pull-request/dist-node/index.js:290:10) {
      status: 422,
      response: [Object],
      request: [Object]
    }
  }
]

@nathanfletcher My experience is that the empty array was the result of the error you displayed above, and the issue is properly handling the error, not the promises. My proposed fix, which has been fully tested, is included in #101, and thus I propose closing this PR without merging.


Use simd_json

simd_json is much faster than serde_json (sometimes almost double, see https://github.com/serde-rs/json-benchmark). Since responses are usually tossed after deserializing, simd_json is a perfect fit for reqwest's use-case.

Is there an effort to get those performance improvements into serde-json directly? That seems like the most impactful solution. I'd prefer that to changing the dependency in reqwest.

No, and there also couldn't be because simd_json modifies its input. For the most part, simd_json is a drop-in, so we can just condition like tls engines. I wouldn't mind taking that on as long as you approve this issue.

I don't want to include support for both as Cargo features. I also don't want to change what dependency is used in serde-json, unless it becomes obvious the ecosystem has moved on.
Perhaps improvements that don't modify the contents could be made to serde-json.

Ok, then you can close this issue. It isn't possible to reach simd_performance without modification I think. The performance has strong ties to it.
For anyone curious, a workaround is to create a sealed extension trait with a json method that uses their preferred engine.


Issue with Bootstrap

Hi,
My Windows machine can run reghdfejl successfully once. However, when I was trying to combine Bootstrap with reghdfejl, it ends up with an error of updating the temp files.
bootstrap, reps(100) seed(1): reghdfejl ln_wage grade age ttl_exp tenure not_smsa south, absorb(idcode year)
file C:\StataTemp\xxx.tmp cannot be modified or erased; likely cause is read-only directory or file
r(608);
This issue exits on multiple Windows machines that I have tested. Changing the directory of temp files will not resolve the issue. Also, running Bootstrap with the original reghdfe comment works.
Thank you!

I will look at it. But why don't you use the vce(bs...) option instead? It is much faster (though slow on the first use). There is an example in the help file.

I was not aware of the new option you provided. Yes, that should solve the
problem. Many thanks!
Regards,
Wayne Xinwei Wan
On Sun, Dec 1, 2024 at 12:24 PM droodman @.***> wrote:

I will look at it. But why don't you use the vce(bs...) option instead? It
is much faster (though slow on the first use). There is an example in the
help file.
—
Reply to this email directly, view it on GitHub
https://github.com/droodman/reghdfejl/issues/6#issuecomment-2509511480,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMWXGEXSF7NWQ7KXSMRXMJL2DJQMTAVCNFSM6AAAAABSZCI2DKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBZGUYTCNBYGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi,
Sorry for troubling you again. I think the issue comes from the
post-regression analysis after the first run of the command. Your new
option resolves the iterations in the original bootstrap command, but the
issue persists if some other post-regression analysis is required.
Take the simple outreg2 as an example. If run the following code together:
webuse nlswork
reghdfejl ln_wage age ttl_exp tenure not_smsa south, absorb(year occ_code)
vce(bs, cluster(occ_code) reps(1000) seed(42) procs(4))
outreg2 using F:\test.xls,  bdec(4) sdec(4) append addstat(Mean, e(ymean))
label adec(3)
Error message:
file F:\StataTemp\ST_4e34_000002.tmp cannot be modified or erased; likely
cause is read-only directory or file
nothing to restore
could not restore sort order because variables were dropped
On Sun, Dec 1, 2024 at 12:49 PM Wayne Wan @.***> wrote:

I was not aware of the new option you provided. Yes, that should solve the
problem. Many thanks!
Regards,
Wayne Xinwei Wan
On Sun, Dec 1, 2024 at 12:24 PM droodman @.***> wrote:

I will look at it. But why don't you use the vce(bs...) option instead?
It is much faster (though slow on the first use). There is an example in
the help file.
—
Reply to this email directly, view it on GitHub
https://github.com/droodman/reghdfejl/issues/6#issuecomment-2509511480,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMWXGEXSF7NWQ7KXSMRXMJL2DJQMTAVCNFSM6AAAAABSZCI2DKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBZGUYTCNBYGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

That is working for me in Windows, at least once I drop the addstat() and adec() options:
. reghdfejl ln_wage age ttl_exp tenure not_smsa south, absorb(year occ_code) vce(bs, cluster(occ_code) reps(1000) seed(42) procs(4))
(MWFE estimator converged in 6 iterations)

HDFE linear regression with Julia                 Number of obs   =     27,973
Absorbing 2 HDFE groups                           F(   5,     13) =     542.80
Statistics cluster-robust                         Prob > F        =     0.0000
                                                  R-squared       =     0.4105
                                                  Adj R-squared   =     0.4098
                                                  Within R-sq.    =     0.1650
                                                  Root MSE        =     0.3671

                               (Replications based on 13 clusters in occ_code)
------------------------------------------------------------------------------
             |   Observed   Bootstrap                         Normal-based
     ln_wage | coefficient  std. err.      t    P>|t|     [95% conf. interval]
-------------+----------------------------------------------------------------
         age |   .0042979   .0011326     3.79   0.002      .001851    .0067448
     ttl_exp |   .0308828   .0029162    10.59   0.000     .0245826    .0371829
      tenure |   .0157172    .001835     8.57   0.000     .0117529    .0196814
    not_smsa |  -.1615383   .0153971   -10.49   0.000    -.1948018   -.1282748
       south |  -.1210285   .0204315    -5.92   0.000    -.1651681   -.0768889
------------------------------------------------------------------------------

. outreg2 using test.xls,  bdec(4) sdec(4) append addstat(Mean, e(ymean)) label adec(3)
check eret list for the existence of e(ymean)
c:\ado\plus/o/outreg2.ado
test.xls
dir : seeout

. outreg2 using test.xls,  bdec(4) sdec(4) append label
test.xls
dir : seeout

.

If you want, you can do set trace on and send me the log file.

The earlier bootstrap command also works for me. Here too, you can send me the trace log:
. bootstrap, reps(100) seed(1): reghdfejl ln_wage grade age ttl_exp tenure not_smsa south, absorb(idcode year)
(running reghdfejl on estimation sample)

Bootstrap replications (100): .........10.........20.........30.........40.........50.........60.........70.........80.........90.........100 done
(dropped 550 singleton observations)
(MWFE estimator converged in 12 iterations)

HDFE linear regression with Julia                 Number of obs   =     27,541
Absorbing 2 HDFE groups                           F(   5,      .) =          .
                                                  Prob > F        =          .
                                                  R-squared       =     0.6762
                                                  Adj R-squared   =     0.6185
                                                  Within R-sq.    =     0.0531
                                                  Root MSE        =     0.2939

------------------------------------------------------------------------------
             |   Observed   Bootstrap                         Normal-based
     ln_wage | coefficient  std. err.      z    P>|z|     [95% conf. interval]
-------------+----------------------------------------------------------------
       grade |          0  (omitted)
         age |   .0114497   .0112983     1.01   0.311    -.0106945     .033594
     ttl_exp |   .0323758   .0018779    17.24   0.000     .0286952    .0360563
      tenure |   .0104689   .0010556     9.92   0.000        .0084    .0125378
    not_smsa |  -.0914148   .0120741    -7.57   0.000    -.1150796   -.0677501
       south |  -.0640471   .0117199    -5.46   0.000    -.0870177   -.0410764
------------------------------------------------------------------------------

Here is the log file from my machine. Thank you very much!
On Sun, Dec 1, 2024 at 1:45 PM droodman @.***> wrote:

That is working for me in Windows, at least once I drop the addstat() and
adec() options:
. reghdfejl ln_wage age ttl_exp tenure not_smsa south, absorb(year occ_code) vce(bs, cluster(occ_code) reps(1000) seed(42) procs(4))
(MWFE estimator converged in 6 iterations)
HDFE linear regression with Julia                 Number of obs   =     27,973
Absorbing 2 HDFE groups                           F(   5,     13) =     542.80
Statistics cluster-robust                         Prob > F        =     0.0000
R-squared       =     0.4105
Adj R-squared   =     0.4098
Within R-sq.    =     0.1650
Root MSE        =     0.3671
                           (Replications based on 13 clusters in occ_code)

         |   Observed   Bootstrap                         Normal-based
 ln_wage | coefficient  std. err.      t    P>|t|     [95% conf. interval]

-------------+----------------------------------------------------------------
age |   .0042979   .0011326     3.79   0.002      .001851    .0067448
ttl_exp |   .0308828   .0029162    10.59   0.000     .0245826    .0371829
tenure |   .0157172    .001835     8.57   0.000     .0117529    .0196814
not_smsa |  -.1615383   .0153971   -10.49   0.000    -.1948018   -.1282748
south |  -.1210285   .0204315    -5.92   0.000    -.1651681   -.0768889
. outreg2 using test.xls,  bdec(4) sdec(4) append addstat(Mean, e(ymean)) label adec(3)
check eret list for the existence of e(ymean)
c:\ado\plus/o/outreg2.ado
test.xls
dir : seeout
. outreg2 using test.xls,  bdec(4) sdec(4) append label
test.xls
dir : seeout
.
If you want, you can do set trace on and send me the log file.
—
Reply to this email directly, view it on GitHub
https://github.com/droodman/reghdfejl/issues/6#issuecomment-2509536777,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMWXGERULSAHNUCKM4N64RL2DJZ4RAVCNFSM6AAAAABSZCI2DKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBZGUZTMNZXG4
.
You are receiving this because you authored the thread.Message ID:
@.***>

Also, make sure you have the latest reghdfejl and jl with:
ssc install reghdfejl, replace
ssc install julia, replace

It looks like the log file won't come through that way. My email address is in the help file.

Well the error is coming from outreg2 when it tries to preserve the data. So I don't think it's really a reghdfejl issue. See https://www.statalist.org/forums/forum/general-stata-discussion/general/1405940-file-cannot-be-modified-or-erased-likely-cause-is-read-only-directory-or-file. You can also try contacting the author of outreg2.


x-forwarded-proto: https added to HTTP inter-service calls by app name

This issue is a:

[x] bug report

Issue description
Calling from inside of the custom Vnet-integrated internal ACA environment:
curl **http**://api
On the api side in logs I find
Request starting HTTP/1.1 GET **http**://api
...
x-forwarded-proto: **https**
...
Request finished HTTP/1.1 GET **https:**//

This e.g. breaks internal IdentityServer calls that emit the https:// issuer instead of http://
Ingress configuration:
"ingress": {
    "external": false,
    "targetPort": 8080,
    "transport": "Auto",
    "traffic": [
        {
            "weight": 100,
            "latestRevision": true
        }
    ],
    "allowInsecure": true,
},

UseForwardedHeaders() is being used in our Apps.
Expected behavior
x-forwarded-proto equals to http when the request is HTTP
Actual behavior
x-forwarded-proto equals to https when the request is HTTP

I was able to repro this and see where the confusion is coming from.
Short names like api are routed through the envoy sidecar container, which then uses https to communicate with the main envoy instance. That intermediate https is what causes envoy to add x-forwarded-proto: https.
Workaround: use the FQDN to bypass the envoy sidecar and communicate directly with the main envoy instance using http.
http://api.internal.kindwater-1f6ab775.azurecontainerapps.io.
We'll consider if there are other ways to enable this scenario in the future.

Unfortunately, our solution heavily relies on simple names for inter-service calls. We resorted to a workaround to set the dynamically resolved URLs to fixed ones.


changed token is not updated

due to
https://github.com/stdevel/ansible-sentinelone_client/blob/367968cb32594063f473bc4efe1b9a94c666ba03/tasks/main.yml#L34-L37
finding that /opt/sentinelone/.INITIALIZATION_COMPLETE already exists

Hi @azrdev,
@chrisvanmeer supplied a PR for your request - looks good to me.
What do you think? Do you want to have a look at it?


[C+ Checklist Needs Completion] [$250] Category - First item in the search result is not highlighted when parent category is selected

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.75-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4571713
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:
Precondition:

Workspace has a regular category, parent category and category in Parent: Child format.

Go to + > Submit expense > Manual.
Proceed to confirmation page.
Click Category.
Select a regular category.
Click Category.
Search for the regular category via search field.
Note that the first item in the search result is highlighted.
Select a parent category (not the child category).
Repeat Step 7 to 8.

Expected Result:
When parent category is selected, the first item in the search result will be highlighted when searching for it.
Actual Result:
When parent category is selected, the first item in the search result is not highlighted when searching for it.
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/cc299973-a075-47f9-acc5-eab96eb1730b

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

Upwork Job URL: https://www.upwork.com/jobs/~01f99c23349d15f2f1
Upwork Job ID:<PHONE_NUMBER>844476928
Last Price Increase: 2024-06-04
Automatic offers:

tienifr | Contributor | 102601877

Issue OwnerCurrent Issue Owner: @greg-schroeder

We think this issue might be related to the #collect project.

Adding to #wave-collect as this is definitely functionality that came up via the old wave 6

@tienifr's proposal LGTM.

Okay I think you should assign them then, yes? @lakchote

I am assigning you @tienifr as you were selected above

Nice work, PR in review

Ah, shoot, Lucien is on parental leave, sorry about that. I'll request a new engineer to review

No, I'll ping again

I think the second deploy + payment date is a red herring ... because the PR was CP'd to staging possibly? Either way I'm going to take the original payment date as the one to focus on here

Payment summary:
Contributor: @tienifr - $250 - paid via upwork
C+: @Ollyws - $250 - you can make a manual request via ND

All that's left is the checklist cc @Ollyws

Oh, got it, I didn't realize that @tienifr - thanks for doing that. I'll adjust the payment summary. Can you post a screenshot showing the refund on Upwork just for our records when we come back to confirm payment for the manual request? Just to make sure we have our bases covered.

Oh. You did post a screenshot. I swear it wasn't loading! I am on airplane wifi I blame that. Ignore!

$250 approved for @Ollyws

$250 approved for @tienifr

filed regression test, closing


Fewer ford->clay subscriptions

Tracks which beams are used for which update-listener, instead of sending a request to clay per dependency per web request. (also pulls in some prior ford clean up)

So, when I run master currently — is this not in there?

Looks like it developed merge conflicts, so it hasn't been merged into master yet. @ohAitch do you think you could get this merged after the eyre work you're doing?


Update 4_Median_of_Two_Sorted_Arrays.cpp

Test cases have been changed to include cases where one of the vectors is empty

File(s) Modified: 4_Median_of_Two_Sorted_Arrays.cpp
Language(s) Used: C++
Submission URL: _https://leetcode.com/submissions/detail/788164416/_

Thanks, @samirangode!


AttributeError: Module 'scipy' has no attribute 'asarray'

This is somehow related to the same file mentioned in #165 . Also seems pretty easy to fix - just have to import the specific asarray module from scipy. This is a well known issue with scipy, see this post on SO
Fail yang bermasalah itu ialah malaya/graph/fast_pagerank.py
Traceback (most recent call last):
  File "/root/test.py", line 40, in <module>
    r = model.sentence_level(isu_kerajaan)
  File "/root/pr/lib/python3.10/site-packages/herpetologist/__init__.py", line 100, in check
    return func(*args, **kwargs)
  File "/root/pr/lib/python3.10/site-packages/malaya/model/extractive_summarization.py", line 212, in sentence_level
    original_strings, ranked_sentences, top_words, cluster_top_words = self._vectorize_sentence(
  File "/root/pr/lib/python3.10/site-packages/malaya/model/extractive_summarization.py", line 132, in _vectorize_sentence
    scores = pagerank(similar + 1e-6, retry, **kwargs)
  File "/root/pr/lib/python3.10/site-packages/malaya/graph/pagerank.py", line 19, in pagerank
    r = load_fast_pagerank.pagerank(G, **kwargs)
  File "/root/pr/lib/python3.10/site-packages/malaya/graph/fast_pagerank.py", line 37, in pagerank
    r = sp.asarray(A.sum(axis=1)).reshape(-1)
  File "/root/pr/lib/python3.10/site-packages/scipy/__init__.py", line 139, in __getattr__
    raise AttributeError(
AttributeError: Module 'scipy' has no attribute 'asarray'

Can I be assigned this

Pull request is active

Weird, cannot replicate

https://colab.research.google.com/drive/1EymLN98s4eoqxyL63px7sA9r4vbnHUx_?usp=sharing cannot replicate


RuntimeError (EmberCLI Rails require ember-cli NPM package version to be ~> 0.1.5 to work properly

RuntimeError (EmberCLI Rails require ember-cli NPM package version to be
~> 0.1.5 to work properly. From within your EmberCLI directory
please update your package.json accordingly and run:
$ npm install
):
ember-cli-rails (0.1.8) lib/ember-cli/app.rb:175:in check_ember_cli_version!'   ember-cli-rails (0.1.8) lib/ember-cli/app.rb:162:in prepare'
ember-cli-rails (0.1.8) lib/ember-cli/app.rb:32:in run'   ember-cli-rails (0.1.8) lib/ember-cli-rails.rb:83:in block in each_app'
ember-cli-rails (0.1.8) lib/ember-cli-rails.rb:83:in each'   ember-cli-rails (0.1.8) lib/ember-cli-rails.rb:83:in each_app'
ember-cli-rails (0.1.8) lib/ember-cli-rails.rb:47:in run!'   ember-cli-rails (0.1.8) lib/ember-cli/middleware.rb:23:in enable_ember_cli'
ember-cli-rails (0.1.8) lib/ember-cli/middleware.rb:8:in call'   rack (1.6.0) lib/rack/etag.rb:24:in call'
rack (1.6.0) lib/rack/conditionalget.rb:25:in call'   rack (1.6.0) lib/rack/head.rb:13:in call'
actionpack (4.2.0) lib/action_dispatch/middleware/params_parser.rb:27:in call'   actionpack (4.2.0) lib/action_dispatch/middleware/flash.rb:260:in call'
rack (1.6.0) lib/rack/session/abstract/id.rb:225:in context'   rack (1.6.0) lib/rack/session/abstract/id.rb:220:in call'
actionpack (4.2.0) lib/action_dispatch/middleware/cookies.rb:560:in call'   activerecord (4.2.0) lib/active_record/query_cache.rb:36:in call'
activerecord (4.2.0) lib/active_record/connection_adapters/abstract/connection_pool.rb:647:in call'   activerecord (4.2.0) lib/active_record/migration.rb:378:in call'
actionpack (4.2.0) lib/action_dispatch/middleware/callbacks.rb:29:in block in call'   activesupport (4.2.0) lib/active_support/callbacks.rb:88:in call'
activesupport (4.2.0) lib/active_support/callbacks.rb:88:in _run_callbacks'   activesupport (4.2.0) lib/active_support/callbacks.rb:734:in _run_call_callbacks'
activesupport (4.2.0) lib/active_support/callbacks.rb:81:in run_callbacks'   actionpack (4.2.0) lib/action_dispatch/middleware/callbacks.rb:27:in call'
actionpack (4.2.0) lib/action_dispatch/middleware/reloader.rb:73:in call'   actionpack (4.2.0) lib/action_dispatch/middleware/remote_ip.rb:78:in call'
actionpack (4.2.0) lib/action_dispatch/middleware/debug_exceptions.rb:17:in call'   web-console (2.1.0) lib/web_console/middleware.rb:37:in call'
actionpack (4.2.0) lib/action_dispatch/middleware/show_exceptions.rb:30:in call'   railties (4.2.0) lib/rails/rack/logger.rb:38:in call_app'
railties (4.2.0) lib/rails/rack/logger.rb:20:in block in call'   activesupport (4.2.0) lib/active_support/tagged_logging.rb:68:in block in tagged'
activesupport (4.2.0) lib/active_support/tagged_logging.rb:26:in tagged'   activesupport (4.2.0) lib/active_support/tagged_logging.rb:68:in tagged'
railties (4.2.0) lib/rails/rack/logger.rb:20:in call'   actionpack (4.2.0) lib/action_dispatch/middleware/request_id.rb:21:in call'
rack (1.6.0) lib/rack/methodoverride.rb:22:in call'   rack (1.6.0) lib/rack/runtime.rb:18:in call'
activesupport (4.2.0) lib/active_support/cache/strategy/local_cache_middleware.rb:28:in call'   rack (1.6.0) lib/rack/lock.rb:17:in call'
actionpack (4.2.0) lib/action_dispatch/middleware/static.rb:113:in call'   rack (1.6.0) lib/rack/sendfile.rb:113:in call'
railties (4.2.0) lib/rails/engine.rb:518:in call'   railties (4.2.0) lib/rails/application.rb:164:in call'
rack (1.6.0) lib/rack/content_length.rb:15:in call'   thin (1.6.3) lib/thin/connection.rb:86:in block in pre_process'
thin (1.6.3) lib/thin/connection.rb:84:in catch'   thin (1.6.3) lib/thin/connection.rb:84:in pre_process'
thin (1.6.3) lib/thin/connection.rb:53:in process'   thin (1.6.3) lib/thin/connection.rb:39:in receive_data'
eventmachine (1.0.7) lib/eventmachine.rb:187:in run_machine'   eventmachine (1.0.7) lib/eventmachine.rb:187:in run'
thin (1.6.3) lib/thin/backends/base.rb:73:in start'   thin (1.6.3) lib/thin/server.rb:162:in start'
rack (1.6.0) lib/rack/handler/thin.rb:19:in run'   rack (1.6.0) lib/rack/server.rb:286:in start'
railties (4.2.0) lib/rails/commands/server.rb:80:in start'   railties (4.2.0) lib/rails/commands/commands_tasks.rb:80:in block in server'
railties (4.2.0) lib/rails/commands/commands_tasks.rb:75:in tap'   railties (4.2.0) lib/rails/commands/commands_tasks.rb:75:in server'
railties (4.2.0) lib/rails/commands/commands_tasks.rb:39:in run_command!'   railties (4.2.0) lib/rails/commands.rb:17:in <top (required)>'
bin/rails:8:in require'   bin/rails:8:in <top (required)>'
spring (1.3.3) lib/spring/client/rails.rb:27:in load'   spring (1.3.3) lib/spring/client/rails.rb:27:in call'
spring (1.3.3) lib/spring/client/command.rb:7:in call'   spring (1.3.3) lib/spring/client.rb:26:in run'
spring (1.3.3) bin/spring:48:in <top (required)>'   spring (1.3.3) lib/spring/binstub.rb:11:in load'
spring (1.3.3) lib/spring/binstub.rb:11:in <top (required)>'   bin/spring:13:in require'
bin/spring:13:in <top (required)>'   bin/rails:3:in load'
bin/rails:3:in `'
Packages.json
1 {
2   "name": "nemo",
3   "version": "0.0.0",
4   "description": "Small description for nemo goes here",
5   "private": true,
6   "directories": {
7     "doc": "doc",
8     "test": "tests"
9   },
10   "scripts": {
11     "start": "ember server",
12     "build": "ember build",
13     "test": "ember test"
14   },
15   "repository": "",
16   "engines": {
17     "node": ">= 0.10.0"
18   },
19   "author": "",
20   "license": "MIT",
21   "devDependencies": {
22     "broccoli-asset-rev": "^2.0.0",
23     "ember-cli": "0.2.0-beta.1",
24     "ember-cli-app-version": "0.3.1",
25     "ember-cli-babel": "^4.0.0",
26     "ember-cli-content-security-policy": "0.3.0",
27     "ember-cli-dependency-checker": "0.0.7",
28     "ember-cli-htmlbars": "0.7.4",
29     "ember-cli-ic-ajax": "0.1.1",
30     "ember-cli-inject-live-reload": "^1.3.0",
31     "ember-cli-qunit": "0.3.8",
32     "ember-cli-rails-addon": "0.0.11",
33     "ember-cli-uglify": "1.0.1",
34     "ember-data": "1.0.0-beta.15",
35     "ember-export-application-global": "^1.0.2",
36     "express": "^4.8.5",
37     "glob": "^4.0.5"
38   }
39 }
Rails 4.2.0 Ruby 2.2.0

It explicitly tells you that "ember-cli NPM package version to be ~> 0.1.5 to work properly"
Your version is "0.2.0-beta.1", which does not conform with "~> 0.1.5" version restriction. Also, there's already an issue for this: https://github.com/rwz/ember-cli-rails/issues/88


Error while making all three animations start at the same time and stop them one after another with the specified delays

I wanted to make the three reels start at the same time when the draw button is clicked and stop them one after another with the specified delays.
But seems like when the animation is about to end on each slot, the result changes from the result that was about to show when it was about to stop.

All codes were edited on Slot.ts file.
These are the codes that I added.
private createReelAnimation(container: HTMLElement | null): Animation | undefined {
    return container?.animate(
      [
        { transform: 'none', filter: 'blur(0)' },
        { filter: 'blur(1px)', offset: 0.5 },
        // Here we transform the reel to move up and stop at the top of last item
        // "(Number of item - 1) * height of reel item" of wheel is the amount of pixel to move up
        // 7.5rem * 16 = 120px, which equals to reel item height
        { transform: `translateY(-${(this.maxReelItems - 1) * (7.5 * 16)}px)`, filter: 'blur(0)' }
      ],
      {
        duration: this.maxReelItems * 100, // 100ms for 1 item
        easing: 'ease-in-out',
        iterations: 1
      }
    );
  }

  private createGenreReelAnimation(container: HTMLElement | null): Animation | undefined {
    return container?.animate(
      [
        { transform: 'none', filter: 'blur(0)' },
        { filter: 'blur(1px)', offset: 0.5 },
        // Here we transform the reel to move up and stop at the top of last item
        // "(Number of item - 1) * height of reel item" of wheel is the amount of pixel to move up
        // 7.5rem * 16 = 120px, which equals to reel item height
        { transform: `translateY(-${(this.maxReelItems - 1) * (7.5 * 16)}px)`, filter: 'blur(0)' }
      ],
      {
        duration: this.maxReelItems * 150, // 100ms for 1 item
        easing: 'ease-in-out',
        iterations: 1
      }
    );
  }

  private createStyleReelAnimation(container: HTMLElement | null): Animation | undefined {
    return container?.animate(
      [
        { transform: 'none', filter: 'blur(0)' },
        { filter: 'blur(1px)', offset: 0.5 },
        // Here we transform the reel to move up and stop at the top of last item
        // "(Number of item - 1) * height of reel item" of wheel is the amount of pixel to move up
        // 7.5rem * 16 = 120px, which equals to reel item height
        { transform: `translateY(-${(this.maxReelItems - 1) * (7.5 * 16)}px)`, filter: 'blur(0)' }
      ],
      {
        duration: this.maxReelItems * 200, // 100ms for 1 item
        easing: 'ease-in-out',
        iterations: 1
      }
    );
  }

// Play the spin animation for all three reels simultaneously
    reelAnimation?.play();
    reelGenreAnimation?.play();
    reelStyleAnimation?.play();

// Wait for all animations to finish
    await Promise.all([
      new Promise<void>((resolve) => {
        reelAnimation?.addEventListener('finish', () => {
          reelAnimation?.removeEventListener('finish', () => {});
          reelAnimation?.finish();
          resolve();
        });
      }),
      new Promise<void>((resolve) => {
        // Stop the genre reel animation after 1 second
        reelGenreAnimation?.addEventListener('finish', () => {
          reelGenreAnimation?.removeEventListener('finish', () => {});
          reelGenreAnimation?.finish();
          resolve();
        });
      }),
      new Promise<void>((resolve) => {
        // Stop the style reel animation after an additional 1 second
        reelStyleAnimation?.addEventListener('finish', () => {
          reelStyleAnimation?.removeEventListener('finish', () => {});
          reelStyleAnimation?.finish();
          resolve();
        });
      }),
    ]);

   Array.from(reelContainer!.children)
      .slice(0, reelContainer!.children.length - 1)
      .forEach((element) => element.remove());

This part is the code that I deleted except the three .play() lines.
// Play the spin animation
    const animationPromise = new Promise<void>((resolve) => {
      const onAnimationFinish = () => {
        reelAnimation?.removeEventListener('finish', onAnimationFinish);
        reelGenreAnimation?.removeEventListener('finish', onAnimationFinish);
        reelStyleAnimation?.removeEventListener('finish', onAnimationFinish);
        resolve();
      };

      reelAnimation?.addEventListener('finish', onAnimationFinish);
      reelGenreAnimation?.addEventListener('finish', onAnimationFinish);
      reelStyleAnimation?.addEventListener('finish', onAnimationFinish);

      reelAnimation?.play();
      reelGenreAnimation?.play();
      reelStyleAnimation?.play();
    });

    await animationPromise;

Ref: https://chat.openai.com/c/3df15647-6c9d-40d5-a962-28d8edae8b5d


Create full SignUp flow

Figma Reference
https://www.figma.com/file/uEckQY4EaQ5rXwyISg6d1g/Fuel-Wallet?node-id=206%3A9374
Screens

[ ] Welcome screen
[ ] Show mnemonic screen
[ ] Write mnemonic screen
[ ] Create password
[ ] Done

Blocked by

#4
https://github.com/FuelLabs/fuel-ui/issues/119
https://github.com/FuelLabs/fuel-ui/issues/120
https://github.com/FuelLabs/fuels-wallet/issues/19

This issue should also include tests, I suggest using cypress.


ShipmentCharge is missing in the PaymentInformation parameter of the shipment method

ShipmentCharge is missing in the PaymentInformation parameter of the shipment method

Please make sure you use the correct model which is ShipmentPaymentInformation. The PaymentInformation model is to be used with the Tracking API and has different fields.


vendor: update go-git and go-billy to their keybase fork master

cc: @taruti

I don't see references to keybase repos. Did you meant to add "origin" field like https://github.com/keybase/kbfs-server/pull/384/files ?

I edited vendor.json directly; not sure if there's a way to do with the govendor command

Oh I think I figured it out.  govendor update gopkg.in/src-d/go-git.v4::github.com/keybase/go-git


assure usability on multi DB development stack

Hi, thanks for making your project opensource.
Problem
To my knowledge alembic-postgresql-enum assumes that migrations are being generated by postgres, and independently of the source database, it issues postgres-specific queries to check for existing enums.
This is relevant specially for the common SQLite in development, and postgres in staging/production workflow, where currently this library is not able to fill the same needs as it does currently for a standard development in postgres from local/development to production.
currently, using this library it is not possible to generate simple migrations on a SQLite, that is compatible both for SQLite and Postgres.  Its also not possible to turn off the hooks to dispatch_for("schema").
e.g. what happens when trying to generate new migrations having previously imported alembic-postgresql-enum - from alembic_postgresql_enum/compare_dispatch.py. obviously this query is only valid for postgres.
sqlalchemy.exc.OperationalError: (sqlite3.OperationalError) near "(": syntax error
[SQL:
        SELECT
            pg_catalog.format_type(t.oid, NULL),
            ARRAY(SELECT enumlabel
                  FROM pg_catalog.pg_enum
                  WHERE enumtypid = t.oid
                  ORDER BY enumsortorder)
        FROM pg_catalog.pg_type t
        LEFT JOIN pg_catalog.pg_namespace n ON n.oid = t.typnamespace
        WHERE
            t.typtype = 'e'
            AND n.nspname = ?
    ]

Proposed solution
As I understand it (after a quick look),
@alembic.autogenerate.comparators.dispatch_for("schema")
def compare_enums(..)

is the entrypoint for the other hooks to alembic to check enums and modify enums.
as a simple solution for this library to not be code breaking after import, is to add a check for the dialect, i.e. defined_enums.py.
@alembic.autogenerate.comparators.dispatch_for("schema")
def compare_enums(..):

    if not autogen_context.dialect.name == "postgresql":
        print("Not using PostgreSQL, skipping enum comparison")
        return
    (...)

it would also be relevant that migrations created using this library would be applied only if upgrading a postgres database (my code above doesn't solve this issue).
Final Statements
if this is in the interest of the mantainers, i would make myself available to generate a pull request.
thanks for reading, and my apologies for any mistakes or if not relevant for you.

This proposal is interesting. However I don't understand how this library will benefit projects that use SqlLite locally. No commands will be generated so what is the point of using it?

You're correct, with this change this library will not generate its custom commands when making migrations based on a sqlite3 database, but that's important.  The main point is that it should be possible to have this lib imported and still do migrations on an sqlite3.
The needs and motivated outlined in alternatives.md, also apply for a sqlite3/postgres parallel development. Enum synchronization in a postgres environment still needs to be done.
this library helped me to easily:

identify with enums are out of sync with my staging/production postgres databases
generate a synchronization migration
correct my postgres databases

however, since now I have a working migration I cannot generate new migrations based on sqlite3 (due to the error I mentioned in this issue's original text - the import from alembic_postgresql_enum import TableReference as it triggers the alembic hook bindings starting from compare_enums.
and now there's a few options:

change the development workflow and use postgres in local development as well

suboptimal

go back and change the migration to not use this library and make custom commands in the migration file

suboptimal as it would be preferred to not have this concern on our team

ensure this lib doesn't break current workflows

from these options, number 3 i think is the preferred solution, so i created this issue.
with this change:

assures compatibility for the original intended use case and current users
2 broadens the usability of this library in more diverse workflows

Good, I'll make this change

Very happy to hear that :)
when you have time, please also look into this quote of mine, as it would make the use of this lib as seamless as it is today but also for the my described use case, with backward compatibility aswell.

it would also be relevant that migrations created using this library would be applied only if upgrading a postgres database (my code above doesn't solve this issue).

if you need anything else i'll be available! Thank you.

There is new version 1.2.0a1. Please check whether these changes fulfill your request. If everything is alright write back and full release will be dropped

Hi, Sorry for the delayed response. I can confirm that generating new migrations in 1.2.0a1 no longer breaks the existing workflow, and it shows the corresponding warning. Thanks.
any feedback on this?

it would also be relevant that migrations created using this library would be applied only if upgrading a postgres database (my code above doesn't solve this issue).

edit: i see you also changed in sync_enum_values. Dunno if its also usefull in other ops. thanks

Sqlalchemy operations such as sa.Enum().create() will not do anything in dialects that do not have native enums


Added event triggered when closing popup

When clicking the X in the popup an close event is triggered on the popup object.
It can then be used like this:
var popup = new ol.Overlay.Popup();
popup.addEventListener('close', function(){
    alert('close');
});

Happy to add an event but I think I'd prefer it if we used the OpenLayers event system for consistency. As we're an ol.Overlay we can make use of ol.Overlay#dispatchEvent.


[DOCS] Add documentation on how to use wasm-tools compose

wasm-compose is a powerful tool for composing components at build-time, and we should have documentation on when to use wasm-compose and how to use it.

"when to use" addressed by #405 https://github.com/wasmCloud/wasmcloud.com/pull/405/files#diff-e56389e0069912b638e0df5bdd352762adcbbf4690b26cff8c816054b75f636f

https://wasmcloud.com/docs/concepts/linking-components/linking-at-build#composing-a-component


[FEATURE] Rating of multiple responses

Currently I didn't find a way in Argilla to define one question with multiple responses, which I want to rate independently. I can only create a ranking question or multiple rating questions.

hi @hanshupe007 ! Could you provide some more details of what it is you would like to achieve? If you could also attach some screenshot that would help a lot.
Thanks!


Crash on using Agricraft Seed analyzer

According to your testing, does the issue occur with ONLY Immersive Portals mod installed?
No
Forge Version
40.1.86
ImmersivePortals Version
1.4.12-mc1.18.2
Latest Log
https://gist.github.com/Haerar/1f25586e44a8b1577ec6b56ef04350af
Crash Report (if applicable)
https://gist.github.com/Haerar/61b0477c3475ec89f44cd68708f1a4ad
Steps to Reproduce
With Immersive Portals installed alongside Agricraft, try to use the seed analyzer with or without seeds placed in it.
No portal needs to be nearby or even generated/activated
What You Expected
Viewing of the seed (or lack thereof) via the Agricraft seed analyzer - this zooms in the player's view at an angle, and if a seed is present, renders a double-helix on-screen which can be scrolled through to rotate and analyze the genetics of the seed.
What Happened Instead
CTD with appropriate error message:
The game crashed whilst unexpected error Error: java.lang.NullPointerException: Cannot invoke "net.minecraft.client.renderer.entity.EntityRenderer.m_114505_(net.minecraft.world.entity.Entity, float)" because the return value of "net.minecraft.client.renderer.entity.EntityRenderDispatcher.m_114382_(net.minecraft.world.entity.Entity)" is null
Additional Details
This is clearly an edge-case but agricraft has been a reasonably popular mod for a while - so low priority but probably worth looking at.
I suspect it's the rendering of the aforementioned double-helix that's causing the crash, given the log stating a "null entity" that's trying to be (pre)rendered for immersive portals to use at qouteall.imm_ptl.core.render.context_management.RenderStates.updatePreRenderInfo
If an entity is null for this function call (updatePreRenderInfo), could it not be gracefully handled by simply returning and not attempting to render? Or does this have some knock-on chain effect down the line? Sorry, Java coder here, I think about these things too much.
Please Read and Confirm The Following

[X] I have confirmed this bug can be replicated without the use of Optifine.
[X] I have confirmed the details provided in this report are concise as possible and does not contained vague information (ie. Versions are properly recorded, answers to questions are clear).
[x] I have confirmed this does not happen on the fabric version (If it does please report to here instead).
[X] I have confirmed I'm not using a hybrid server (Magma, Cauldron) or a different (non official) Forge Server release.
[X] I have confirmed this issue is unique and has not been reported already.

Looks like OP's dead. Mind if I butt in?

Superseded by #132


docs(combobox): Close command tag in combobox documentation

Close command tag in combobox documentation.

Duplicate: https://github.com/shadcn-ui/ui/pull/4402

Closed as a duplicate of https://github.com/shadcn-ui/ui/pull/4330. Thank you.


T1w-SyN: Is contrast inversion really necessary? (at least for mutual-information scales)

I think I've heard @mattcieslak asking about this, please let me know if I'm wrong (whether you made the comment and/or the content of the comment).
The question is that, most of the linear and nonlinear registration steps of our SyN implementation use Mattes. With this cost function, the contrast inversion just does nothing - while it is one more massaging step where we can harm the reliability of the overall process.
Maybe we could limit the contrast inversion only for the final CC antsRegistration step.
For the case of BOLD, it would be doubtful we need to invert the contrast at all in some cases. Because we are picking the frames with the strongest T1 contrast across all runs in the session with share shimming settings, the inversion is probably not needed anymore (even for the CC metric).
Thoughts?
(I'm giving this very low priority, but nonetheless noting it down for future reference)

Related: #68.

That sounds correct, but worth verifying with some before/after shots of the specific steps.


Integrate findAncestor method

Algorithm no longer takes advantage of the Dijkstra algorithm
findClosestAncestor function of CFG is now integrated with BranchObjectiveFunction
Renamed CFG to ControlFlowGraph
Replaces #169

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
0

Relevant Lines:
0

💛  - Coveralls


Add star pattern in C

Description
Task

For the given input n the number of rows printed should be 2*n-1 for ex:

Note

Filename should be Pattern32.c

@Vishal21121 hey I would like to work on this can you assign this to me?

ok, @bishalbera  you can work on this issue


chown: allow setting arbitrary numeric user ID

~Help needed: the test fails because it requires root privileges to change the owner of a file. I couldn't quite follow how the other tests in this file are accomplishing this. The test passes if you run it as root. Any recommendations for what to do about this? Just skip the test if the user is not root?~

This pull request updates chown to allow setting the owner of a file to a numeric user ID regardless of whether a corresponding username exists on the system.
For example,
$ touch f && sudo chown 12345 f

succeeds even though there is no named user with ID 12345.
Fixes #3380.

Any recommendations for what to do about this? Just skip the test if the user is not root?

Haven't looked into it recently but as far as I remember yes.
I tried to make this more convenient here https://github.com/uutils/coreutils/pull/2692

I've updated this pull request to use the run_ucmd_as_root() helper function introduced in pull request #2692. Let's hope it works :crossed_fingers:

Hmm something's still not quite right:

---- test_chown::test_chown_only_user_id stdout ----
current_directory_resolved:
run: id -u
touch: /tmp/.tmpzLMcnj/test_chown_file1
run: /target/x86_64-unknown-linux-gnu/debug/coreutils chown 1001 --verbose test_chown_file1
result.stdout =
result.stderr = thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Custom { kind: NotFound, error: "No such id: 1001" }', src/uucore/src/lib/features/perms.rs:153:48
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

thread 'test_chown::test_chown_only_user_id' panicked at 'Command was expected to succeed.
stdout =
 stderr = thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Custom { kind: NotFound, error: "No such id: 1001" }', src/uucore/src/lib/features/perms.rs:153:48
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
', tests/common/util.rs:174:9

---- test_chown::test_chown_only_user_id_nonexistent_user stdout ----
current_directory_resolved:
touch: /tmp/.tmpCsFABD/f
run: sudo -E --non-interactive whoami
thread 'test_chown::test_chown_only_user_id_nonexistent_user' panicked at 'called `Result::unwrap()` on an `Err` value: Os { code: 2, kind: NotFound, message: "No such file or directory" }', tests/common/util.rs:1070:14

---- test_chown::test_chown_owner_group_id stdout ----
current_directory_resolved:
run: id -u
run: id -g
touch: /tmp/.tmpIfke8U/test_chown_file1
run: /target/x86_64-unknown-linux-gnu/debug/coreutils chown 1001:121 --verbose test_chown_file1
result.stdout =
result.stderr = chown: invalid group: '1001:121'

thread 'test_chown::test_chown_owner_group_id' panicked at 'Command was expected to succeed.
stdout =
 stderr = chown: invalid group: '1001:121'
', tests/common/util.rs:174:9

failures:
    test_chown::test_chown_only_user_id
    test_chown::test_chown_only_user_id_nonexistent_user
    test_chown::test_chown_owner_group_id

Executive summary: I believe my change revealed an issue in the perms.rs module in uucore. I'll fix that issue in a separate pull request.
Details: Let's take a look at one of the failures, test_chown::test_chown_only_user_id stdout:
According to the logged messages, the test method first calls id -u to get the ID of the current user. This seems to return 1001 in the Ubuntu CI environment. Then it creates a file named test_chown_file1. Finally it runs chown 1001 --verbose test_chown_file1, which panics.
I think that before the change I'm proposing in this pull request, chown terminated with an error message like invalid user: 1001, which caused the test to be skipped due to this block: https://github.com/uutils/coreutils/blob/e5d973718c127547bcd591b3b989370d22008bf3/tests/by-util/test_chown.rs#L404-L409
The panic is happening now because the command-line argument 1001 is (correctly) interpreted as a user ID, and chown attempts to set the owner of the file to the user with that ID. That code is ultimately in perms.rs in uucore. When that code tries to produce the info message required by the --verbose flag, it assumes (by way of an unwrap() call) that the given user ID always corresponds to a named user: https://github.com/uutils/coreutils/blob/e5d973718c127547bcd591b3b989370d22008bf3/src/uucore/src/lib/features/perms.rs#L150-L155
The assumption that a user ID always corresponds to a named user is incorrect. Here's an example of how chown can work with a nameless user ID:
$ touch f && sudo chown 12345 f && sudo chown --verbose 12345 f
ownership of 'f' retained as 12345

However, the username still appears in the verbose message even if it is given as a user ID in the argument:
$ touch f && chown --verbose $(id -u) f
ownership of 'f' retained as jeffrey

I added a commit that updates the perms.rs module.

It looks like my changes to perms.rs resolved the test failure in test_chown_only_user_id, but two failures still remain:

failures:

---- test_chown::test_chown_only_user_id_nonexistent_user stdout ----
current_directory_resolved:
touch: /tmp/.tmplmvqpC/f
run: sudo -E --non-interactive whoami
thread 'test_chown::test_chown_only_user_id_nonexistent_user' panicked at 'called `Result::unwrap()` on an `Err` value: Os { code: 2, kind: NotFound, message: "No such file or directory" }', tests/common/util.rs:1070:14

---- test_chown::test_chown_owner_group_id stdout ----
current_directory_resolved:
run: id -u
run: id -g
touch: /tmp/.tmpsWIZLg/test_chown_file1
run: /target/x86_64-unknown-linux-gnu/debug/coreutils chown 1001:121 --verbose test_chown_file1
result.stdout =
result.stderr = chown: invalid group: '1001:121'

thread 'test_chown::test_chown_owner_group_id' panicked at 'Command was expected to succeed.
stdout =
 stderr = chown: invalid group: '1001:121'
', tests/common/util.rs:174:9

failures:
    test_chown::test_chown_only_user_id_nonexistent_user
    test_chown::test_chown_owner_group_id

I'll diagnose them later this week.

I'm trying to use run_umcd_as_root(), but I'm getting the following error output
---- test_chown::test_chown_only_group_id_nonexistent_group stdout ----
current_directory_resolved:
touch: /tmp/.tmpM827bl/f
run: sudo -E --non-interactive whoami
thread 'test_chown::test_chown_only_group_id_nonexistent_group' panicked at 'called `Result::unwrap()` on an `Err` value: Os { code: 2, kind: NotFound, message: "No such file or directory" }', tests/common/util.rs:1070:14

I think this is saying that it tried to run sudo -E --non-interactive whoami but the call to Command::spawn() returned an error, namely a "No such file or directory" error. Could it be that whoami is not accessible from the CI environment? @jhscheer do you have any ideas?


GetIt install fails

When I try to install it via GetIt on RAD Studio 11, then I get this error:

And it simply fails then:

So I can't get it installed.

Is it happening on the new Delphi 12?
It can be because it didn't have yet the project version for Delphi 12.  I've just merged your pull request with the support to it, thanks @shaunroselt.

No. It happened on Delphi 11.3.
WakaTime isn't available on GetIt for Delphi 12 yet. You guys have to email Embarcadero again.

WakaTime still isn't available on Delphi 12 via GetIt btw.


is there any possible to support third party gpt api

is there any possible to support third party gpt api, such as poe.com.
and is it nessesary to make a fake openai api to integrity?

The fake OpenAI API handles messages in the same format as the official API does, but is based on a third-party (POE) service.Or to put it another way, just like reverse proxy.

The fake OpenAI API handles messages in the same format as the official API does, but is based on a third-party (POE) service.Or to put it another way, just like reverse proxy.

it's not necessary for this project. It is helpful, but that would be pretty involved and beyond the scope of this project. There are some useful repos that do this for LLMs and I will look into integrating them.
For POE or any LLM available through API, I can utilize any service and make a client class that will work.


Modal for Contact Form Submission

This PR Contains the  modal for Contact Form Submission

Hey @sid86-dev
Could you co-author commits in this PR?


Analyze viability of implementing dict copy method with current OPCodes

a = {1: '1', 2: '2', 3: '3'}
b = a.copy()

Could not find a equivalent to Opcode.PACK and Opcode.UNPACK for dictionaries. Probably will need to create an Opcode.NEWMAP and use Opcode.KEYS and Opcode.VALUES and put the pair manually in the NEWMAP. Maybe Opcode.MEMCPY could be usefull, but I'm not too sure how exactly Buffer works. Asking for Neo to create a new Opcode is also an option.


Video Iframe in Video Block shrinks with window height

The height of the video Iframe is set to 45 vh, which leads to a rather weird sizing on different window heights:

Tested, not currently a problem


Shortcuts not sorting properly in Recycle Bin

Describe the bug
When the Recycle Bin is sorted by "Date deleted," any shortcut (.lnk) files will either appear on top or on the bottom.
To Reproduce
Steps to reproduce the behavior:

Create a shortcut using the Windows Explorer, let's say, with the letter K.
Move that shortcut to the Recycle Bin.
Open the Recycle Bin in the Files app, and sort it by "Date deleted."
Even though the name of that shortcut is in the middle of the alphabet, that shortcut will appear either on the top or the bottom.
This may not seem like a big deal, but it is quite annoying when I have a lot of shortcuts that I have to scroll through before I can reach what I am trying to find.

Expected behavior
I would expect the shortcut files to also follow the same sorting rules you specified for the entire Recycle Bin.
Screenshots

Desktop (please complete the following information):

OS Version: Windows 10 Version 20H2 (OS Build 19042.746)
App version: v<IP_ADDRESS>

Additional context
This problem has always occurred since the Recycle Bin could be sorted by date deleted. I have also noticed that the text under date deleted reads "0 seconds ago" for all the shortcuts. Could this possibly be because it is referring to the date deleted of the item the shortcut points to, and not the shortcut itself? Because the actual file was never deleted.
Log file
debug.txt
debug_fulltrust.txt

@Tech-How This issue should be fixed with #3462.


[BUG] Cannot boostrap downstream RKE2 cluster (non ready boostrap machines)

Rancher Server Setup

Rancher version: 2.8.2
Installation option (Docker install/Helm Chart): Helm chart, 1.27.11, RKE2
Proxy/Cert Details: no proxy, lets encrypt with cert manager, layer 4 LB in front

Information about the Cluster

Kubernetes version: 1.27.11
Cluster Type (Local/Downstream): Downstream, custom cluster with ready machines (trying to add a control, etcd VM and a worker VM for start)

Custom = Running the provisioning command displayed on rancher (no docker install just containerd pulled from rancher), Ubuntu 22.04 OS on the nodes

User Information

What is the role of the user logged in?
Admin

Describe the bug
Cannot bootstrap the downstream cluster. Provisioning command is executed with no error but the boostrap node stays for in this state:
[INFO ] non-ready bootstrap machine(s) custom-736e05be332d and join url to be available on bootstrap node
To Reproduce
Have tried multiple times with the same results.
Prepare 2 VMs with Ubuntu 22.04, create a new cluster in rancher with RKE2 type, use the default settings, copy the provisioning command to 2 VMS, one for control plane,etcd and one for worker.
Result
Cluster is stuck on bootstrapping without progress.
Expected Result
Bootstrap should finish and cluster should become active
Screenshots

Additional context
Docker logs from  cluster-register pod on the bootstrapping machine have this repeated from time to time:
root@eu1-control-1:~# /var/lib/rancher/rke2/bin/crictl --runtime-endpoint unix:///run/k3s/containerd/containerd.sock logs 9f00d5465f44b

time="2024-03-27T08:11:34Z" level=error msg="Failed to install system chart rancher-webhook: pod cattle-system/helm-operation-s7zxv failed, watch closed"
time="2024-03-27T08:12:36Z" level=error msg="Failed to install system chart rancher-webhook: pod cattle-system/helm-operation-bkzjs failed, watch closed"
time="2024-03-27T08:16:34Z" level=error msg="Failed to install system chart rancher-webhook: pod cattle-system/helm-operation-klw7b failed, watch closed"
time="2024-03-27T08:17:36Z" level=error msg="Failed to install system chart rancher-webhook: pod cattle-system/helm-operation-9b8f5 failed, watch closed"
time="2024-03-27T08:21:34Z" level=error msg="Failed to install system chart rancher-webhook: pod cattle-system/helm-operation-p42v6 failed, watch closed"

There no special settings, using the defaults and have done the same multiple times on different installations. This is a fresh install of rancher on a RKE2 cluster and the first downstream cluster I am trying to provision through it.
Any ideas on how to debug more? Never had any issue provisioning RKE2 downstream clusters on custom machines...

I have the same problem, posted the information here: https://github.com/rancher/rancher/issues/43701#issuecomment-2022232092

I have the same issue.
Step to reproduce:

Install fresh Rancher docker/HA instance using Let's encrypt
Install custom RKE2 cluster (everything works fine, node connected to the cluster)
Reboot server/nodes, where rancher is installed.
Install custom RKE2 cluster (it's stuck on [INFO ] non-ready bootstrap machine(s) custom-xxxxx and join url to be available on bootstrap node without error)

Expected
The boostrap node join the cluster. Also after reboot server/nodes where the rancher is.
Rancher version: 2.8.2

We also had this issue, it is related to #44929.
The cluster provisions correctly after using one of the workarounds described there.
The workaround that worked for us was to set the cluster.x-k8s.io/cluster-name on the kubeconfig secret in the upstream cluster.

@qhris Can you please explain where to add this label exactly? Sorry for my lack of knowledge, I never had issues provisioning clusters up to now. Is the kubeconfig created as soon as you create a downstream cluster? Thank you in advance.

It should be a secret in the fleet-default namespace named {cluster_name}-kubeconfig (in the local/upstream/rancher cluster).
There are no labels by default so I just added:
metadata:
  labels:
    cluster.x-k8s.io/cluster-name: my-cluster-name

@qhris - I've added the label to {cluster_name}-kubeconfig as you described and it's work for me.
Thanks

@dylanthepodman same issue for me - did you get it up again?
@baskinsy In my case I try to recover a broken cluster and tried to add a triple node as described in the docs.

@JannikZed What is your rancher version? Have you upgraded to 2.8.3.

@baskinsy 2.7.10. wasn't sure if upgrading is save in this state ..

I face the same issue:

Rancher 2.9.1
RKE v1.27.16+rke2r2
Cluster itself is healthy, node is Ready
All systemd services are green, also agent-service

I was stuck at log line "non-ready bootstrap machine(s) custom-4ff27335d03c and join url to be available on bootstrap node".
In my specific case, I only had to remove the config item rke_config.machine_global_config.disable-cloud-controller to get the cluster provisioned.
So it's my conclusion, that it's possible to configure the cluster using rke_config, but this specific config parameter must be left out. Maybe the cloud-controller is required even if no any cloud services are used. It seems that then the RKE2 embedded cloud provide is being used.

Same issue today, however I successfully created a cluster with the same method successfully just 6 days ago. Besides the k8s version nothing has changed, though deploying using the version of the older cluster also has the same problem.
Rancher 2.8.2 running on 1.21.10 k3s2
attempting to deploy 1.27.16 rke2r2
Stuck at Non-ready bootstrap machine(s) new-cluster-pool1-6k5cs-ztlzk and join url to be available on bootstrap node.


feat: Add assets

:tada: This PR is included in version 1.0.0-alpha.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Installation problem

Hello,
When I install boxbilling and get to step 3, it tells me to wait for the database except nothing happens. even after the database has been installed.
How to do ? I have no error logs

Since it is a 500 (Internal Server Error), the issue could be related to .htaccess file configurations if you are using apache.
You can start with your .htaccess file.

Hi @CrIKxSLCD Did you manage to solve this issue?

@CrIKxSLCD You are using a content manager such as word press and you have a blocking extension causing the post method to fail on the install. I would remove the .htaccess file first and see if your problem disappears. Aside from that comment I am closing this issue due to the website mentioned in the issue account at hosting provider has been suspended. http://momonheberg.tk/cgi-sys/suspendedpage.cgi


Edited import task fails if target table was changed

System information:

Operating system (distribution) and version Win10
DBeaver version 7.1.4

Describe the problem you're observing:
Edited import task fails if target table was changed in the wizard
Steps to reproduce, if exist:

create an import task via Database Task Panel and execute it
open the task again and change target table
Press Next
Actual result: Resource file cannot be selected

I cannot reproduce it anymore


Unable to reset user.dir system property in Java 11

Context
Our daemon will reset user.dir system property in switching build environment. However, this is implicitly forbidden in Java 11.
https://github.com/gradle/gradle/blob/561b7ec949184b6bc41605b1818d15161fd5209e/subprojects/native/src/main/java/org/gradle/internal/nativeintegration/processenvironment/AbstractProcessEnvironment.java#L92
    public void setProcessDir(File processDir) throws NativeIntegrationException {
        setNativeProcessDir(processDir);
        System.setProperty("user.dir", processDir.getAbsolutePath());
    }

JDK team's reply https://bugs.java.com/bugdatabase/view_bug.do?bug_id=JDK-8202127 is :

This is not a bug. The "user.dir" property should never be changed in a running VM.

We need to find another solution for this.

Relying on user.dir is always wrong in Gradle. It breaks down whenever you invoke your build from a different directory, e.g.

a subproject directory
as part of a composite build
using the -p switch.

Users should always use Project.file or construct files relative to some existing thing like project.buildDir.
The question here is really just: How do we catch code that constructs relative paths from scratch and fail eagerly?

@blindpirate Can you point me at any test that was failing because of this?

@oehme https://builds.gradle.org/viewLog.html?buildId=14474480&tab=buildResultsDiv&buildTypeId=Gradle_Check_Platform_Java11_Oracle_Linux_ide

Thanks!

I've fixed the tests to be ignored on Java 11+ and made our recommendation against relying on user.dir stronger. See https://github.com/gradle/gradle/pull/6139


add callout boxes

Update callout boxes
Catching us up with OTTR's fancy new callout boxes.
Resolves #103
Also adds examples of the boxes (and how to use them) to Chapter 2
Update dictionary
I also removed some non-AnVIL words from the dictionary (e.g. ITCR), just to keep things clean, since I was editing it anyway.  I'm not sure what "UE" and "UE5" are...can those go?
Plan for updating repos
Nobody is currently syncing with AnVIL_Template, and the other AnVIL and GDSCN repos branched off of OTTR or AnVIL Templates at all different stages.  I'm planning on going through them one-by-one and updating their styles to match AnVIL_Template, then setting them up to sync with AnVIL_Template style going forward.  This PR updates us so we match up with the latest version of OTTR's style, which I wanted to take care of prior to setting up syncing.

Looks good! Yes, I think UE and UE5 refer to the grant and can be removed.


[RFC] Decouple the internal QueryGenerator to prevent future breakage of ORM package

The internal QueryGenerator is doing a number of things, which was already taken care of, but the Orm package extends this class to reuse the Query structure generating logic for DQL queries.
As this class is now marked as @internal we should not longer extend this class, and tread it as a final class!
Instead of extending the QueryGenerator should be split into two (or more) classes:

The QueryGenerator which takes care of the structure, this doesn't change for DQL.
The (to be named; Values, Dialect) class which takes care of the Column and values transforming, the QueryGenerator will delegate most of the work to this class, basically this functions the same way as the Platform does for Doctrine.

I think Platform is a good name, and this will also remove the SearchMatch as this can be integrated directly in the platform. Dql will be named 'orm' as platform.
The platform class is stateless, configuration is passed to the methods.
Removing the state is however a problem for caching of column strategies, but the entire result be cached anyway.


Matcher support for any objects inheriting Comparable<T>

NOTE: A reimplementation for release/2.0 branch from #122.
Added support for matchers comparing two objects inheriting Comparable.
Test conditions reads closer to natural language, e.g. value1 shouldBe gt(value2) as opposed to (value1 > value2) shouldBe true.

:+1:

Thanks!

Do we really want to introduce new code annotated with @Deprecated?

@helmbold I'm not fussed if it was removed. I just thought it would provide some context to new KotlinTest users since the surface behaviours are the same with existing matchers.

What do you think @sksamuel : should we relase a new version with deprecated methods? If I would be a solo developer I would remove them ...

Probably take them out.
On 3 Mar 2017 19:38, "Christian Helmbold"<EMAIL_ADDRESS>wrote:

What do you think @sksamuel https://github.com/sksamuel : should we
relase a new version with deprecated methods? If I would be a solo
developer I would remove them ...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kotlintest/kotlintest/pull/126#issuecomment-284047513,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAtZGu5FeUTyGPhJrlsNQ7OQoy_273aFks5riGnPgaJpZM4LQXfs
.

OK, do you guys want me to remove it? If so, from which branch?

Release 2.
On 6 Mar 2017 1:37 p.m., "Herdy Handoko"<EMAIL_ADDRESS>wrote:

OK, do you guys want me to remove it? If so, from which branch?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kotlintest/kotlintest/pull/126#issuecomment-284397492,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAtZGsCSSeUwsIBthGQ1f2CukVIA3PJNks5rjAwSgaJpZM4LQXfs
.

@sksamuel I just had a look at release/2.0 branch and the @Deprecated methods for Comparable<T> matchers are not there. Did you already remove it?

The deprecated methods were removed in commit 41854941f095e95fcc7f2bb92dc665260fe8d4a8 by Aleix Morgadas.

OK, cool 👌


Auth: Add AWS Cognito authentication backend

Added new authentication backend for AWS Cognito.
This is based on OAuth2.
Testing Plan:
Tests created for it and tested against real AWS Cognito pool.

@chdinesh1089 You've done the process of adding GitLabAuthBackend which was also a simple backend like this, can you take a look if we didn't forget some  step here?

I think I have addressed all the comments now.

Hi @brainrecursion I have implemented your commit in my local development, but when I click the login with AWScognito. it redirects to cognito with an error. Also I have attached my user pool app client settings, if this is the right setup.

Hi @brainrecursion, Also i have tried to logged in using the Hosted UI in cognito, it redirects back to the login page with the id_token from cognito.. so it is not successfully logged in the user.. Is this pull request is already tested and working?
http://localhost:9991/login/#id_token=eyJraWQiOiJiZ1lLRmRHVnVIM1pKZlhSN1A0Y1JERjRodHdzTVBKa0o5WUE3YkUybUJjPSIsImFsZyI6IlJTMjU2In0.eyJhdF9oYXNoIjoiemR1WGFNVmtHY0c0MDVLVlQwUlJjZyIsInN1YiI6Ijk1ZjRmOWFlLWMxNTQtNDNhOC04ZTY3LWNhNGM3OThiNzQwOSIsImF1ZCI6IjV0aDg0bzRrdmlkZnJnMmI3N2d1bHA3NDg4IiwiZW1haWxfdmVyaWZpZWQiOnRydWUsImV2ZW50X2lkIjoiNzhhY2E3MmEtMDk4Mi00MDRmLWExYTktZmJhZGMyYWQ3MDIyIiwidG9rZW5fdXNlIjoiaWQiLCJhdXRoX3RpbWUiOjE2MDMyNzIwNzMsImlzcyI6Imh0dHBzOlwvXC9jb2duaXRvLWlkcC5jYS1jZW50cmFsLTEuYW1hem9uYXdzLmNvbVwvY2EtY2VudHJhbC0xX29VWjBMVGxWZiIsImNvZ25pdG86dXNlcm5hbWUiOiI5NWY0ZjlhZS1jMTU0LTQzYTgtOGU2Ny1jYTRjNzk4Yjc0MDkiLCJleHAiOjE2MDMyNzU2NzMsImlhdCI6MTYwMzI3MjA3MywiZW1haWwiOiJqdW5hcmQubysxMjM0QGR2bWVsaXRlLmNvbSJ9.FMddBX8AMAb9MZf0yZpMbzvKXXzsutaYkjMwwz0Y9v3AeWTnrMaEUZfUsY--r9U7IPIaFJjlintdyLecvypmT5nKtkOoRAcvlGHv0m5pxOXiyg_rbJEoWxPScYRJNUwbO-cAAAesEhmbsIMugtjN4JNkSbKUKVgRq6Zhcv7nTMG4rUDL-85RUyZ3Uacbfzc6Jt41gKXDcLV0fGYubOf8xS6L9DpnkGHJefvKZcAtcSa7-jmw0Dw9Nyr153fgmVrwXyO74Jm7V3qEtYXJD0lvIxpD1_4Ax0QlTYuQScXrc7UArOFVSOmDkECwpvc6LhL8AAbky62YVhiwP-2_WgUIEg&access_token=eyJraWQiOiJ1YjhHU1RZVWdUWENmVGlMbmU1Z3hlSVg5NDF4OWhTaWdvN1lqVFZRT2FjPSIsImFsZyI6IlJTMjU2In0.eyJzdWIiOiI5NWY0ZjlhZS1jMTU0LTQzYTgtOGU2Ny1jYTRjNzk4Yjc0MDkiLCJldmVudF9pZCI6Ijc4YWNhNzJhLTA5ODItNDA0Zi1hMWE5LWZiYWRjMmFkNzAyMiIsInRva2VuX3VzZSI6ImFjY2VzcyIsInNjb3BlIjoiYXdzLmNvZ25pdG8uc2lnbmluLnVzZXIuYWRtaW4gb3BlbmlkIHByb2ZpbGUgZW1haWwiLCJhdXRoX3RpbWUiOjE2MDMyNzIwNzMsImlzcyI6Imh0dHBzOlwvXC9jb2duaXRvLWlkcC5jYS1jZW50cmFsLTEuYW1hem9uYXdzLmNvbVwvY2EtY2VudHJhbC0xX29VWjBMVGxWZiIsImV4cCI6MTYwMzI3NTY3MywiaWF0IjoxNjAzMjcyMDczLCJ2ZXJzaW9uIjoyLCJqdGkiOiI2ZjVjNTcxOS00NzkzLTQ4YWYtOTdhNS1lYzdkZTM0NjYxM2IiLCJjbGllbnRfaWQiOiI1dGg4NG80a3ZpZGZyZzJiNzdndWxwNzQ4OCIsInVzZXJuYW1lIjoiOTVmNGY5YWUtYzE1NC00M2E4LThlNjctY2E0Yzc5OGI3NDA5In0.mLj8ZAtFCO4Nun-pzU_IbCwejejqAcAnQB0aJkZCUAwSzcc_NRbdXx-MGcvSi3XNSfBcG4W8jPRCFF-9c5Ag9mKEeApuAC8S5ODiGFXuDsHNpMEMb36C4VT7saGa_qkGNoJA2OG5evpLMcbwDrRtoSsLd1YQM8RJ1qKD2Y6JjhNUItV1xAofBWU3GPP2_4L0_GyXrwj9U_Lxb4kdmh3PaCD6urAZEy1uCakq1Llt2ABZTLBJdrHT9cn03KKIREqwhyDYh1Q_PtMFf-otVjOs_WraFTVVoLropicGZ_CTwinJPSIhoqwF0lJnWPAJIN9GOBYQELQQxrMdV2_UYlF0iA&expires_in=3600&token_type=Bearer

@junardoclarit The error you are getting is Cognito complaining the callback URL does not match what you have specified. I had this problem using the development environment, the fix is to set EXTERNAL_HOST to localhost otherwise the development environment defaults to zulipdev.com which advertises the callback URL as http://zulipdev.com:9991/complete/cognito/

@brainrecursion I have set the EXTERNAL_HOST to localhost:9991 by running it in the terminal export EXTERNAL_HOST=localhost:9991 then I run the ./tools/run-dev.py --interface=''. But I get a new error now when redirected to cognito.. now the error is unauthorized_client
https://login-upbook.auth.ca-central-1.amazoncognito.com/error?error=unauthorized_client&client_id=5th84o4kvidfrg2b77gulp7488

@junardoclarit It might be you need to enable authorisation code grant instead of implicit grant, I can't remember. If you still have problems the best thing to do is use the browser Chrome/Firefox developer tools and look on the network tab. You will see a request to Cognito containing the callback URL etc. This needs to match the callback URL. Cognito has terrible error handling.

Hi @brainrecursion where should the /complete/cognito/ route go? I mean do you what method is this calling?

@junardoclarit I am not sure what method it calls, it is all handled by the python-social-auth library.

@brainrecursion huge thanks for doing this.  I did some small documentation style tweaks and pushed back to this PR.  The main thing that needs to change is that Amazon appears to call the product "Amazon Cognito", not "AWS Cognito".  We should probably mirror that choice; AWSCognito => AmazonCognito, etc.
Once that's resolved (and we figure out how to tweak the instructions to avoid the issues @junardoclarit is seeing), I think this should be good to merge.

@timabbott I have renamed everything to Amazon Cognito and improved the documentation. Hopefully ready for merge now.


Finding path in the 2D Maze

💥 Proposal
I want to add a folder under Python/graphs named Finding_path_in_the_Maze which will house to different pathfinding algorithms. The argument of all these functions will be a Binary maze( 1 is way and 0 is a wall), the source cell and the destination cell. The pathfinding algorithms will be as follow.

Breadth-First Search
Depth-First Search
Dijkstra Algorithm
A*(astar) Algorithm

The maze will be considered as a graph and one can move in all four directions, in the current cell is (x,y) then the four possible moves will be (x+1,y)-DOWN, (x-1,y)-UP, (x,y+1)-RIGHT, (x,y-1)-LEFT. One has to print the path from the Source to Destination in the form of string if it's possible to reach. The string will contain L, R, U and D denoting the respective direction.
Have you read the Contributing Guidelines on Pull Requests?
YES
I request @iamrajiv @atarax665 and @HarshCasper to assign me this issue in Python

I would like to work on this in Java under DWOC
I request @iamrajiv @atarax665 and @HarshCasper to assign me this issue and please add DWOC label to it


Support Rocky Linux

https://hub.docker.com/_/rockylinux
Centos 8 has reached EOL. Make sure OS resolution tasks support Rocky Linux in the CentOS family of distros instead.

And AlmaLinux?

Want to be mindful because neither of these are listed in Tailscale's supported distros but should be low effort to include. Sure, can you file a separate ticket for that distro?
https://tailscale.com/kb/1031/install-linux/


libInMemoryFrontend

We're running SourceKit on a diskless server. When the user opens a project that has dependencies, we need the server to compile the dependencies and pass them to SourceKit. This PR adds a simple function swift::inmemoryfrontend::compileSwiftModule that lets the server do this.
The only change that I really need in the Swift tree to support this is the swift::serializeToMemory function -- swift::inmemoryfrontend::compileSwiftModule could be implemented in the server code instead of in the Swift tree. However, hiding as much complexity as possible behind the simple swift::inmemoryfrontend::compileSwiftModule function interface makes it easier to maintain compatibility when the Swift tree changes. So I would like to include a simple interface if possible.
See https://forums.swift.org/t/building-swiftmodules-for-sourcekit/28030?u=marcrasi for more discussion of this use case.

Idle thought: I wonder if there is a more general way to provide the buffer mappings.  Off the top of my head, we also have emit-objc-header, and also the module source locations files.

We already respect VFS mappings. If the normal frontend were instructed to ingest a VFS map pointing to locations on a RAM disk it seems like you get everything you want here, right?

The VFS interface only has read methods, so no, it does not let you prevent writing to disk.

Please update the base branch to main by Oct 5th otherwise the pull request will be closed automatically.

How to change the base branch: (Link)
More detail about the branch update: (Link)