Organized version information

Pull request checklist:

[x] CHANGELOG.md was updated, if applicable

Description
Organized version information shown in About page:

moved Qt version information to the top (main motivation)
organized and split compile-time and runtime information since it was getting a bit cluttered in there; diff there is stupid, just read new code
removed ancient irrelevant comments
added new bool isModified() member to Version class - used to determine whether the vcs tree was compiled or not at the time of building the app. Idea adopted from Golang's source code:

Added information about running in DEBUG mode to fullVersion:
At first, I wanted to only add this to AboutPage.cpp but I figured it might be good to be consistent in other places (result of --version and/or title window); however if this change is "too invasive" I will revert it and add this information to AboutPage.cpp only.
Previews:
built in debug mode with modified tree + no nightly:

built in non-debug (release) mode with clean tree + nightly:

while on about page topic, maybe we could move contributors list to chatterino api (since it needs to be curated and we cant use github directly)? so there wont be need to bundle 50 avatars and txt file.

Firstly, I'm focusing on version-related changes here (not just in About page - it's not about reworking that page but reworking logic around version information).
Secondly, No. There is a good reason we include authors' information locally, but I won't go over that again here since this was already raised in https://github.com/Chatterino/chatterino2/pull/2139#issuecomment-716670625

The new wrapped around line of information cannot be copied.

Thanks for catching this!
Before my changes, the structure was something like:
parent layout -> QGroupBox -> QLabel with version
But after I wanted to add a line break in between I turned it to something like:
parent layout -> QGroupBox -> QVBoxLayout -> (QLabel , QLabel)
Even though QVBoxLayout renders its labels vertically, they're in fact completely "isolated" so that it's impossible to select text from multiple QLabel's at once, even with right mouse interaction flags being set.
I've tried to find a reasonable solution to this and getting rid of second QLabel + putting a simple <br> between lines seems to be the only non-overengineered quick solution.
One thing I'm not so sure about this is that with only one QLabel, the QVBoxLayout is technically useless, however I've noticed that its presence makes some nice "padding" that pther QGroupBox'es in About page use, which makes Version's QGroupBox a bit more inline with the other ones (they all have a QVBoxLayout holding all the elements), look at the padding:

Like I said, even though QVBoxLayout is not necessary, the lack of it makes things look a bit more 'ugly':

So I'm thinking I'll just keep the QVBoxLayout here for the sake of being consistent with the paddings, even if it's going to hold only 1 element. Such a minor thing and so much writeup... but alas I've chosen the modification visible on the 1st screenshot in 4a60365d1d14c2f5c0b5dda03e67a2004037c06b.

I decided it'll be cleaner to determine the boolean at qmake/cmake step already, which should also take care of qmake compilation issues. While doing this I've decided it should be impossible to override the value of GIT_MODIFIED since then it'd be pointless to have this "variable" in the first place. And there's no need for any fallback through env variables since if git command fails, Version::isModified() would be set to false by default.

I removed a ; before the built part
I uppercased the running string
Both strings are now independently generated once on startup by the Version class, and returned with buildString() and runningString()


format the input after change the keyboard on safari

Description
When input method is set as [Japanese -Romaji] with Mac Safari, the card number is not separated 4 digits each, so the system does not recognize as a credit card number.  As a result, it becomes an error.  This issue is observed only in Mac Safari.  Also, if a different IME is used, such as Google IME, the problem does not happen.
Why are we making these changes? Include references to any related Jira tasks or GitHub Issues
Jira ticket
Reproduction Steps (if applicable)
Reproduction steps are available in Jira ticket
Evidence
Solution
Since this problem is related to the keypress event listener is not attached on all browsers, in the same way, we need to handle a flag to know if the keypress event is present or not, in the case, the browser is not using the keypress event listener or change because the user changes his keyboard will use the input event listener as a fallback.
Before
https://user-images.githubusercontent.com/990207/158444429-217002e9-690b-4135-b1b4-c89bf9b4363c.mov
After
https://user-images.githubusercontent.com/990207/158446387-a2e26eb4-a966-484d-ad94-676509d29bbc.mov

@garbray  Can you also add a CHANGELOG entry?


Added a few features

Added a few features:

Added getLoadTime(), getHTML()
Fixed output. Output Current HTML or Current Content  must be the one cell of the output array.
Added constructor with path to CasperJS and to temp directory

Cool!
Thanks.
On Wed, Jan 13, 2016 at 9:20 AM, Alexandre<EMAIL_ADDRESS>wrote:

Merged #17 https://github.com/alwex/php-casperjs/pull/17.
—
Reply to this email directly or view it on GitHub
https://github.com/alwex/php-casperjs/pull/17#event-513369446.


unable to log into your salesforce org

When trying to login with salesforce I am getting a message saying it failed and to clear my cache.  Here's what I have done so far and I do not know where to go next.

Used chrome and edge
incognito in both browsers
production org
sandbox org (developer)
test.salesforce.com
my domain in sandbox https://pentairwater--bendev.sandbox.my.salesforce.com
my domain in production

My login history seems ok and I am redirected back to happysoup.io after authenticating with SF.

Hi @cadillacben ,
Thanks for reporting this issue! Do you see a specific error in HappySoup? Also, does your profile have login IP ranges? HappySoup uses Heroku and the IP of Heroku may be out of range.

I had checked the IP ranges before based on some searching, but I do not have any.  I even added <IP_ADDRESS>-<IP_ADDRESS> just in case but it did not make a difference.
When logging in I get the standard happy soup page, then the salesforce login page, and then I am redirected back to happy soup.  It flashes to https://happysoup.io/?oauthfailed=true very quickly and then lands on https://happysoup.io/?logout=true with the message below.

@pgonzaleznetwork
I tried removing MFA as a test and used a different computer, but no luck.  It does seem related to my orgs, but the login seems ok. I could give you a login if it would help.

@cadillacben sorry for the late reply. I have no idea, it must be something with your org as I'm and other people are able to log in just fine.


Switch to embedded PackageIcon

Part of aspnet/AspNetCore-Internal#3146
Local build passes the smell test with these changes. I'm still unsure on whether or not we need to include a reference to the package icon in the .nuspec - the "old way" did so automatically, the "new way" does not (https://docs.microsoft.com/en-us/nuget/reference/msbuild-targets#packing-an-icon-image-file). CC @nkolev92 who may know more about that question.

Looks like we don't need to worry about the .nuspec, as per https://github.com/aspnet/EntityFramework6/pull/1286#issuecomment-534307966

With the last change, this now includes the icon metadata in all .nuspec's, including the manually generated ones - @dougbu PTAL

Closing in favor of https://github.com/aspnet/Blazor/pull/1897


RFC: Autocomplete

This is a request for comments PR for adding an autocomplete component, intended to replace our current typeahead component. This version uses Downshift to help with accessibility, exposing a composable API, which could be used to build custom typeahead components in apps.
This component is intended to:

Add more baked-in accessibility than our current typeahead component
Use a composable API that uses and is compatible with other design system components, including the FormField in #361
Optimize for experimentation changes
Allow consuming apps to control how data is fetched from typeahead services

See the Autocomplete docs. This PR is dependent on the monorepo setup #353

Codecov Report

Merging #371 into next will not change coverage.
The diff coverage is 100%.

@@         Coverage Diff         @@
##           next   #371   +/-   ##
===================================
  Coverage   100%   100%
===================================
  Files        42     43    +1
  Lines       418    448   +30
===================================
+ Hits        418    448   +30

Impacted Files
Coverage Δ

packages/autocomplete/src/index.js
100% <100%> (ø)

Continue to review full report at Codecov.

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

I think this is a good idea, as long as we use the "controlled" props for downshift, i.e. not let it manage any state internally. We need fine control over what happens when an item is selected because for the "current location" feature we do not want to close the dropdown right away, but instead wait until a promise has resolved.

cb8ab87, c5c7cdf and 2befbc7 commits brought this PR up-to-date w. the v2 "structure". Travis CI is also "green" to proceed with the merge.


Update DocSearch to V3

Is Your Feature Request Related To A Problem? Please describe.
DocSearch (Algolia) is on V3 and we are still on V2. It is stated that V2 is not maintained anymore in their docs.
Describe The Solution. Why is it needed?
We should upgrade to V3: https://docsearch.algolia.com/docs/DocSearch-v3
Additional Context.
n/a
Related Issues
┆Issue is synchronized with this Github issue by Unito

Closing this in favor of #316


Changing costumes

I could not test this (edit: beyond the point of seeing the set_costume function is called) because I have no idea how to make the "hat" animation work out in the player scene. At least the default costume works!
@fleskesvor you wanna nudge me in the right direction or even send me some code?
The original code I stole this from uses binary scenes so there's little I can do right now.

@fleskesvor you wanna nudge me in the right direction or even send me some code?

I made a PR on your branch, which adds the second costume to the player scene.
An issue, which I'm not sure if have practical consequences or not, is that if the selected frame set in the AnimatedSprite is different from the current costume, the correct costume won't be set until the player starts moving or the costume is set explicitly from an esc script.
As punto mentioned on my original PR, #33, we also need to save the current costume, or else the default will be used when a save is loaded.

I made a new PR on your branch (mjtorn/escoria#4) and I'm re-posting my comments on it here for posterity:

I moved player.gd's functionality into set_state which was currently stubbed and rewrote the example esc scripts to use set_state for items instead of set_costume. The main advantage of this is that it converges the two very similar concepts and there is already code in place for saving state, so we get that one for free.
I also added the floppy disk, which is a simple example of using item.gd's set_state for something other than just swapping textures and it also makes it easy to verify that state has been saved by :use-ing that, quitting the game and choosing to "Continue" from the menu. (Just playing the scene in the editor does not load the autosave.)
I also think the PR should have a more integrated example of changing player costumes than :use-ing the can and ice cream. Maybe there should be a hat in the room that you can pick up and use from the inventory to toggle the costume? I think a pink bycocket with a green feather on one side would be awesome, but I'm not picky. We can always improve it later.

As I said in the original PR, all this is great, but I would respectfully ask that improving the demo scene is a separate effort.
Due to reasons, I would also hope that this PR gets merged without having to squash the set_state code into the history, as that's an insane effort as well.

As I said in the original PR, all this is great, but I would respectfully ask that improving the demo scene is a separate effort.

Yes, I agree.

Due to reasons, I would also hope that this PR gets merged without having to squash the set_state code into the history, as that's an insane effort as well.

As discussed on IRC, both @StraToN and I are fine with this.
Thanks for the contribution! :smile:

I should also mention that punto's original concerns from #33 are adressed in this PR.


optional argument for copy_to_pandas to create a simplified index

for t in self.all_tables:
    getattr(rtn, t).reset_index(inplace=True, drop=True)

If argument is truthy, and argument can't be truthy unless drop_pk_columns is also Truthy
(or could simply have a better flag - perhaps a special string).

Added reset_index argument option.

Added reset_index argument option.


Remove @testable import from explicit module builds test

@testable may make eventual toolchain integration more difficult (because we'd have to build once for testing and again for integrating), so remove it from the one place it's currently used. This requires making the module dependency graph structures public, which IMO is an ok tradeoff.

@swift-ci test

I think @_spi could help, but I don't think we should consider using it until there's a version of Xcode shipping with swift 5.3. It's already required for all the tests to pass, but ideally people shouldn't need a snapshot to build the driver.

I'm going to go ahead and merge this to wipe out the @testable. Thanks!


BUG: Useing altsrc when flag has multiple names

In this code, you are never using the eachName's functions variable name. This caught my attention then realized that f.Name could be equal to user,u which you will never be set in context. So then I realized that the eachName should wrap everything inside the if f.set.  then changing all f.Name inside to name. Also seems like f.GenericFlag.Name should also be renamed to name.

It does feel like name should be used there, but I'll let @ChrisPRobinson comment in case I'm missing something. Thanks for bringing this up @dixonwille.

I agree with @jszwedko, it seems like a bug, this seems like we should use name instead of f.Name.
I don't think I really had much testing around the scenario where there are multiple of one flag. Are you seeing bad behavior when the short flag and long flag are used in combination?

I am but replacing that single variable didn't seem to work. context.IsSet(f.Name) stops it from ever getting to that loop. f.Name is user,u for example and since that is not a flag (user is a flag and u is a flag) it skips over the whole computation.

@dixonwille perhaps you can give a pointer to your actual code and tests? I'm not exactly sure of the problem. My approach would be to add the test scenario I want to work and evaluate its behavior, sorry if I'm repeating myself from above. I don't have a ton of time to attempt this work myself right now but could review your code if that helps?

Code is on internal Enterprise Github. Here is the snippet though.
altsrc.NewStringFlag(cli.StringFlag{
		Name:  "user",
		Usage: "The SQL `USERNAME`",
}),

I would like to make the name user,u. I am also busy and understand. I'll see if I can get some more info for you or try and fix it myself and make PR. Not sure when I will have time but I will try my best.

So to clarify what you want to do is something like this?
https://github.com/urfave/cli#alternate-names
Based on the example above you would need to change it to this?
altsrc.NewStringFlag(cli.StringFlag{
        Name:  "user, u",
        Usage: "The SQL USERNAME",
})

Then when you put a command line like this it doesn't work?
cmd do-something --u chris --user dixon

Well passing it through as flags it works. It's loading from the file into the flags that is not working. If I add ,u (the only line that I change) then the flag is never set from file. But if I remove it and just have user then loading from a file works great!

In the yaml file do you want to be able to value by both names, u and user?
This may not compile but can you tweak the function below to something like this?
func (f *StringFlag) ApplyInputSourceValue(context *cli.Context, isc InputSourceContext) error {
	if f.set != nil {
		if !(context.IsSet(f.Name) || isEnvVarSet(f.EnvVar)) {
			if value != "" {
				eachName(f.Name, func(name string) {
                                    value, err := isc.String(name)
			            if err != nil {
				        return err
			              }
			           f.set.Set(f.Name, value)
				})
			}
		}
	}
	return nil
}

So I think what its doing for yaml anyway is its trying to do a lookup for "user, u" and not user. Then it doesn't find it and no value is returned.

That is what I would expect to happen anyways. But yes that is what I think is going on. When I have time I will try a few things but starting there.

Hiya! There's a known issue with the current implementation of the CLI, where altsrc is generally clunky and poorly documented. My current idea is that I'm going to move all of the altsrc functionality into the main package in some way, and that's how all of the "altsrc is weird and different???" issues people are having will get resolved. I'm tracking that work here => https://github.com/urfave/cli/issues/833


Don't wrap ListEmptyComponent in an extra view

Motivation
A common UI pattern for list empty states is some text/images centered inside the visible part of the list. This is pretty hard to do currently because we wrap ListEmptyComponent with an extra view with no way to style it so we cannot just use flex: 1 to make it fill the available space.
Test Plan

Added an example of ListEmptyComponent in the FlatList example in RNTester

Before (no way to make ListEmptyComponent fill the space):

After:

Tested some edge cases like returning null from the ListEmptyComponent

Tested in an app that uses FlatList + ListEmptyComponent

Release Notes
[GENERAL] [MINOR] [VirtualizedList] - Don't wrap ListEmptyComponent in an extra view

Is there any information in what version it will be available?

The release process is explained at http://facebook.github.io/react-native/versions.html
db061ea8c7b78d7e9df4a450c9e7a24d9b2382b4 just landed 8 days ago, so I would expect it to make it into 0.56 when the release candidate is cut next month.

@hramos thank you!


Small improvements to the photo cropper in product addition

What

[ ] Small improvements to the photo cropper in product addition

[ ] (Product Addition) There should be a rotate button in the photo cropper
[ ] (Product Addition) The name of the current crop should appear on top of the cropping window

Part of

#1379

I was having a look for an easy  fix for this issue but didn't seem to find one,
instead of our current crop_your_image package was thinking if we could use this https://pub.dev/packages/image_cropper.
It would be more robust with more features(rotate_image), the only downside I could find was it just supports ios and android(no_web)
Suggestions?

We actually used image_cropper until around two months ago, but had problems with it, see https://github.com/openfoodfacts/smooth-app/issues/1038 and it also restricted access to the StatusBar for the whole app. Therefore I switched to the current one here: https://github.com/openfoodfacts/smooth-app/pull/1063

But yes @AshAman999, the name shouldn't be a big problem but rotating isn't supportet. That said I have no problem if we switch the used package

Hey! I was looking through the packages that we are already using and noticed that we are already using the photo_view package and it has support to rotation controls.
So, what I had in my mind was that if we create a IconButton and provide a function which changes the rotation controller by 90 degrees?
DO we have any designs/ Icons which we would like to use and where to place the buttons ?
And I would like to work on this feature, can you assign it me.

If you can do without using new packages that would be great, you go ahead with the work.
Let maintainers assign you the role

I think there are no finalized mocks on how to do it, but I fear your suggestion won't work. We need to save the rotated image because we upload it to the server and not just display it rotated.

@M123-dev we have the option to request a server side rotation, but that's last resort

I haven't followed what you guys are up to with rotations, but rotation is easy to display on a Canvas/CustomPainter:
canvas.save();
final double x = width / 2;
final double y = height / 2;
canvas.translate(x, y);
canvas.rotate(radians);
final bool tilted = radians == 90 || radians == 270;
canvas.translate(tilted ? -y : -x, tilted ? -x : -y);

// display on your canvas...

canvas.restore();

https://user-images.githubusercontent.com/57723319/160780793-b97d4d38-7621-4cb1-90eb-0b952dba7424.mp4
This is of the package that i suggested above,
If it's okay i can do the work and get me assigned

We can sure try to @AshAman999, but as said we had a problems with this package.
Could you try to build a apk with this package set up (flutter build apk --debug) and check if this works

Just did, i guess i didn't found any problems as of now, done the changes for just the product add page to check
Should I send the apk ?
https://user-images.githubusercontent.com/57723319/160786072-d034686f-cf92-4c4f-bcee-cb36d1dfdd43.mp4

No problem, I believe you, just ouf of couriosity, did you add the
<activity
    android:name="com.yalantis.ucrop.UCropActivity"
    android:screenOrientation="portrait"
<EMAIL_ADDRESS>
to the AndroidManifest

Yeah I did ,
had to add i guess

so should I go ahead with these changes?

We can sure test it, but I think it's unlikely that anything has changed. Even if the build works again, it may be that the system status bar disappears again.

okay, I will raise a pr for the same with the changes, will try it
if it works then best else we will have to find some other alternative for the crop feature cuz the current one is real slow as well

Should be fixed right, feel free to re-open if not


eng-1247 account page

The account page is back now :)

Had to update it to fit with the new UI structure. Here it is before: https://github.com/aqueducthq/aqueduct/blob/085cdf6692dc8419377a658e059dad2c243efd53/src/ui/app/pages/account.tsx
Side note: One thing I noticed was that my browser didn't reflect my UI changes until I switched to incognito and restarted the gateway so if you run into the issue you may want to try that.

Thanks for the quick fix! @vsreekanti @agiron123 given we've completely removed account page could you confirm whether this is intentional?
@eunice-chan most changes looks good! I'm wondering if we could bring back the code block for rendering. There are some reference in the older code: https://github.com/aqueducthq/enterprise/blob/main/ui/pages/account.tsx#L148 , https://github.com/aqueducthq/enterprise/blob/main/ui/components/text/codeblock.tsx , https://github.com/aqueducthq/enterprise/blob/main/ui/package.json#L40

It absolutely wasn't intentional, and I'm kinda shocked we somehow did this. Thanks for fixing this @eunice-chan!

@likawind Added syntax highlighting


Prepare the demo data to pass all tests.

According to the state of the test groups, looks like we should prepare the demo data for the tests.
Conditions:

Start point is the default example resources from the package;
All resources should be available as clean JSON resources;
All changes should be produced step by step for providing details via git diff.

There is a table displaying each test along with its related details, providing a clear overview of the progress.
https://docs.google.com/spreadsheets/d/1DSIIi-V2IRSJzUUuhggO9RGxK2llHFl3/edit?usp=sharing&ouid=114127786440816887867&rtpof=true&sd=true
Summary
In general, we have identified several common issues:

Tests attempting to retrieve attributes that should not be present in the profile (8 test groups).
Tests encountering a situation where resources and patient references exist, but the search result is empty (7 test groups).
Tests facing a scenario where resources exist but lack a reference to the patient, resulting in an empty search result (3 test groups).
A single test exhibiting a problem within the source code of the generator (1 test group).
Test groups that should not be present (3 test groups).

There is a table displaying each test along with its related details, providing a clear overview of the progress. https://docs.google.com/spreadsheets/d/1DSIIi-V2IRSJzUUuhggO9RGxK2llHFl3/edit?usp=sharing&ouid=114127786440816887867&rtpof=true&sd=true
Summary
In general, we have identified several common issues:

Tests attempting to retrieve attributes that should not be present in the profile (8 test groups).
Tests encountering a situation where resources and patient references exist, but the search result is empty (7 test groups).
Tests facing a scenario where resources exist but lack a reference to the patient, resulting in an empty search result (3 test groups).
A single test exhibiting a problem within the source code of the generator (1 test group).
Test groups that should not be present (3 test groups).

Tests attempting to retrieve attributes that should not be present in the profile This problem was fixed

@ir4y
I found a problem related to empty responses for some test groups, ex: 1.8 Observation Pathology Result Tests, 1.16 Observation Diagnostic Imaging Result Tests.
The problem is hidden in the generator implementation. The first search is always should have default values because it is the request to the server to get the data. In these examples, the first test is a test related to the code. code is not fixed in these examples.
I will find a way to fix it soon.

I made a few updates related to the observation profile tests. The table with information was updated.


GoMaxProcs

Sometimes, go defaults to a single core.
Force usage of multiple cores by setting GOMAXPROCS in main.

Not needed for go 1.5 and above I think.


Bulk Edit page pagination and filtering

Having this huge library makes bulk edit page almost unusable, it takes more than 1 minute to load the page and it loads every model in my library, resulting in a huge scrolling screen without any way of filtering.

On previous versions (before we removed the synology generated thumbnails) this page didn't load.
I was always getting time out error after some minutes.

Is this the models bulk edit page, rather than files? If so, it should respond to currently-applied filters correctly. I admit it doesn't have pagination though :)

It is indeed the models bulk edit page. I didn't know that it uses the applied filters, it makes sense now that you say it.. A page size (maybe same as the one set on the settings) and a pagination will help a lot with big libraries.

And it should make clear that the filters have been applied!


Toc: finish table of content

Hi!
I don't exactly know what happened in the past, but I managed to get the toc anchors working. What do you think ?

With a few CSS & font awesome it gives this


Duplicated Question Number 12

https://github.com/lydiahallie/javascript-questions#12-whats-the-output
https://github.com/lydiahallie/javascript-questions#12-what-are-the-three-phases-of-event-propagation
See that? They are both Question 12.

And Question 19 disappeared.

Thank you so much! This has been fixed now 👍


Subcommand with no arguments shows full help output

Handles special case of nested subcommands

Whoops, this looks to be superseded by https://github.com/codegangsta/cli/pull/186


Update create-a-phone-system-call-queue.md

Updating to provide clarification on Longest Idle and Presence Based Routing

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@MikePlumleyMSFT  PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

These updates are being handled through a different PR.


recepient -> receiver,

recepient is spelled recipient, but I belive receiver is a batter label here.
I noticed this because never version of cadence actually error if you use the wrong labels!

I think correct one is "royalty recipient" vs "royalty receiver" , but English is not my first language, so I can be wrong.

This would be a breaking change, so I don't think it is worth changing.

Actually right now cadence does not care about the labels of structs created in other contracts, i found out this yesterday. But I am ok if we want to have a spelling error in the contracts forever :)

I agree that this is breaking change, with current handling of breaking changes not worth it, but on the other hand I think we care more than necessary about breaking changes. I mean running after dapps to make them fix contracts is a losing battle.

@bjartek So you're saying that if we did this upgrade now, it would not be breaking, but when that version of cadence is applied, then it would break the contracts that use this? That would be a breaking change anyway, so maybe this is still worth merging and updating

Yes.

merged upstream master.

@bjartek you'll probably also need to run make test to build the assets so we can merge

done

The back and forth on this was taking too long, so I just created a PR myself. I'll close this


Track and use fallback TextAffinity for null affinity platform TextSelections.

Fixes https://github.com/flutter/flutter/issues/40095
When affinity is unknow, we pass null instead of defaulting to TextAffnity.downstream. This allows us to supply fallback affinities computed from libtxt.
Since LibTxt tracks affinities in the getPositionForOffset call, we can use the affinity information here even when the selection itself is overridden by the TextEditingController as a fallback affinity.

Will attempt to do this without the addition of ambiguous

This version makes use of null instead of the explicit "ambiguous" affinity. This adds no additional API, and is no longer dependent on https://github.com/flutter/engine/pull/13787

Wait it was working in my repro, let me try some more cases.

@justinmc are you sure you are running the parched version? Mine seems to work with a variety of TextFields and settings on multiple devices.
Do you mind posting the code you are using?

There is no engine piece needed now, right?  I just checked out your branch and ran a plain app with a multiline TextField.

I'm trying it again and will post my code in a second.

Oh wait this may not work for iOS that's what it probably is

This specifically addresses the fact that android does not pass an affinity. Have not seen what iOS does.

Give me a second and I'll look at the iOS side of it

Ohh that makes sense.  Sounds good.

@justinmc Should be ready for another round. This version is a bit cleaner and fixes the bug on iOS in conjunction with https://github.com/flutter/engine/pull/13852


Can't get dependency_overrides to work

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
Project structure (with only relevant files):
├── app
│   ├── __init__.py
│   ├── main.py
│   ├── routers
│   │   ├── __init__.py
│   │   ├── foo_router.py
│   ├── tests
│   │   ├── __init__.py
│   │   └── routers
│   │       ├── __init__.py
│   │       └── test_foo_router.py

routers.foo_router (minimal, only with relevant parts):
from typing import List, Optional
from fastapi import APIRouter, Depends
from frontegg.fastapi.secure_access import FronteggSecurity, User

router = APIRouter()

@router.get("/bar", response_model=Optional[List[schemas.Foo]])
def list_bar(user: User = Depends(FronteggSecurity(permissions=["foo"]))):
    logger.debug(f"Authenticated user: {user}")
    return []

app.main (minimal, only with relevant parts):
from fastapi import FastAPI
from app.routers import foo_router

app = FastAPI()

app.include_router(foo_router.router, prefix="/foo", tags=["Foo"])

tests.routers.test_foo_router:

from fastapi.testclient import TestClient
from frontegg.fastapi.secure_access import FronteggSecurity, User

from app.main import app

def override_frontegg():
    return User(
        sub="user1",
        tenantId="tenant1",
        type="userToken",
        access_token="abc123",
    )

app.dependency_overrides[FronteggSecurity] = override_frontegg

def test_list_bar(client: TestClient) -> None:
    r = client.get("/foo/bar")

    response = r.json()
    assert r.status_code == 200
    assert response == []

Description
Hey there!
I'm trying to apply that approach on my api, but something doesn't seem to work properly.
I found several other issues here as well as a couple of StackOverflow questions on this matter, but the suggested solutions don't seem to work for me. Unless there's something I've overlooked.
I've attached the relevant project structure and related files in the Example Code section.
However, when I run my tests, I get a 401 response, which means that the override isn't working as expected. It actually makes the call to my external auth provider (FrontEgg) although it shouldn't, based on the override that I've set.
Am I missing something here?
Thanks!
Operating System
macOS
Operating System Details
macOS Monterey 12.2 running on Apple MacBook Pro 13" with M1 chip.
FastAPI Version
0.73.0
Python Version
Python 3.9.9
Additional Context
No response

You cannot override dependencies that receive parameters. Well, you cannot override any dependency (object) that will have a different key value in a dictionary if you recreate it.
What you can do is:
x = FronteggSecurity(permissions=["foo"])

Then you use x as dependency e.g. Depends(x), and you can override it as well with app.dependency_overrides[x] = y.

@Kludex that worked for me, thanks!
I guess that the best solution would be to still support overriding a dependency that takes parameters (kinda like what unittest.patch does) if possible, but if not- I can still work with that proposed solution, definitely got me unblocked.
Thank you!


[SR-15585] Invalid task group list items do not have source locations

Previous ID
SR-15585

Radar
rdar://problem/89641499

Original Reporter
@karwa

Type
Bug

Environment
Swift-DocC 92f7fe11da0cf357e8e9d38aab835055c4c1aa5b
Also present in the Xcode 13.1 build
macOS 11.6

Additional Detail from JIRA

Votes
0

Component/s
Swift-DocC

Labels
Bug

Assignee
None

Priority
Medium

md5: 9c2a4789ba12c4f99e093155320236fb

Issue Description:
If there is a non-link item in a "See Also" list or Topics list in a Swift source file, DocC will emit a warning: "Only links are allowed in task group list items" without a source location.
This makes locating the cause of the warning incredibly difficult. The only way I was able to discover where these things were coming from is by building my own version of DocC with a bunch of "dump()" calls thrown around.
This revealed that DocumentCurator was unable to find a document location for the node reference here: https://github.com/apple/swift-docc/blob/d94139a5e64e9ecf158214b1cded2a2880fc1b02/Sources/SwiftDocC/Infrastructure/DocumentationCurator.swift#L176
Steps to reproduce:
Add an item to a "See Also" list or "Topics" list which is just plain text (not a link), and build documentation. DocC will emit a warning with no location.
/// A thing
///
/// ## Topics
///
/// ### Things about Things
///
/// - bar
///
struct Foo {
}

Thanks for mentioning the warning text, so that I could perform a web search and find this issue 👍
The obscure warning is still there in Xcode 14.3 (14E222b).

I believe that this was fixed by https://github.com/apple/swift-docc/pull/753


search plugin + show_only_matches = performance issues?

Hi,
I have a tree with almost 300 json elements and an input text field to filter a node. Could someone help me find out what is happening, please?
If I configure jstree with "show_only_matches" = true in search section and type something, it freezes Google Chrome for some seconds. If "show_only_matches" is false, it doesn't freeze.
Here is a jsfiddle reproducing the described problem: http://jsfiddle.net/brunolellis/no430s1f/1/
This issue occurs with jstree 3.2.0 and it is working with jstree 3.0.8.

You had the wrong CSS file loaded, you also had the search function running on each keyup, which means a huge DOM update on each button pressed - that should be avoided. Here is a fixed demo (I see no problem with it in FF & Chrome):
http://jsfiddle.net/no430s1f/2/
I will investigate further and try to improve the search performance, but I believe the above fixes should be more than enough for a responsive tree.
Best regards,
Ivan

OK, thanks for your help.
It doesn't seem "natural" filtering while typing in this version, like it is in 3.0.8 version at least. I haven't tried later versions yet.
I will do some more investigation and let you know.

3.2.0 introduces the concept of hidden nodes, so the performance hit probably comes from there, I will investigate and try to speed things up a bit. Thank you for reporting.

I removed one additional full redraw when searching - searching with show_only_matches is two times faster now.
Best regards,
Ivan

There is something weird with the lastest versions: the "show_only_matches" it is just not working at all, and also becoming super slow. Version 3.1.1 works fine so I'm back to that one.

It works (as seen in the jsfiddle in my previous answer), just make sure you update the theme CSS file too - exactly what the author of this issue missed.
If that is not it, please provide a demo of this feature not working so that I can fix it.

I was also having this problem with performance after 3.1.1 and figured out what was going on based on vakata's note about the introduction of hidden nodes in 3.2.0. I'm adding DOM content to nodes "after_open", and after the introduction of hidden nodes this was being applied to hidden nodes as well. Adding ":visible" to my selector solved the problem. For example, I'm adding an info icon with long searchable descriptions and a tooltip, where the tooltip has to be refreshed if/when nodes go in and out of the DOM. Maybe there's a better way to do this? For now, the ":visible" selector works great.
I'm going to double back to my code to see if I can use the


license?

Hi, this looks interesting, and nice for learning C/C++ and PDCurses. Which license applies to this project?

I have attached the MIT license.
As for learning from this code - it can be improved. In particular, the program should not use so many threads recklessly. Ideally, multithreading should be eliminated.

I have attached the MIT license.
As for learning from this code - it can be improved. In particular, the program should not use so many threads recklessly. Ideally, multithreading should be eliminated.

Thanks for making your code available to learn from, and thanks also for the advice! That will be part of the learning process for me (finding what can be improved and how to do it) :)


Add Birthday field to Person

Add Birthday field to Person
Add testcases in BirthdayTest.java
Include parsing of Birthday in all AddCommandParser, EditCommandParser and ParserUtil
Amend existing testcases to include Birthday, which includes tests for storage, model, parser, logic and command

Also editted UG to include birthday


Switch to flux_ranocha / flux_hindenlang_gassner for multicomponent

Exchanging flux_chandrashekar as well as flux_derigs_etal with flux_ranocha and flux_hindenlang_gassner for multicomponent as well as ideal_glm_mhd_1d, see #760

I am a bit confused - does this address both #738 and #760?

I am a bit confused - does this address both #760 and parts of #738?

It addresses #760 (multicomponent euler and mhd 1D/2D) and just a part of #738 (mhd 1D).

Thanks, @Cczernik! I didn't check the new fluxes analytically. If there is someone more experienced with multicomponent equations, please ping them.

Did you run some runtime comparisons of the new fluxes vs. the old ones for the multicomponent equations?
Could you please also check whether we still have tests of flux_chandrashekar (multicomponent Euler) and flux_derigs_etal (multicomponent MHD)? If not, could you please add a simple test, say the EC tests with modifie d flux via keyword arguments of @test_trixi_include?
Please find below some additional questions/comments.

Thank you for your fast and helpful reviews, @ranocha!

Regarding the runtime comparison, the new fluxes look faster than the old ones.

1D Multicomponent Euler EC Test with Flux_Ranocha (left) and Flux_Chandrashekar (right).

2D Multicomponent MHD EC Test with Flux_Hindenlang_Gassner (left) and Flux_Derigs_Etal (right).

Regarding the chandrashekar/derigs_etal testcases: I have intentionally left one test for each equation in each dimension with these fluxes.

Thanks for your help!


Include a new version of buffer to fix Edge/Firefox48+ bug

Fixes #6

Unfortunately didn't work with tests.  Closing.


feat(python): split python import list

Split code like:
from re import search, match, sub...

Thanks a lot for the plugin.

Done at #77

Works. Thank you very much!


Fix: RowSelection now respects enableMultiRowSelection

Simple PR, if enableMultiRowSelection is false, clear out the object so only one item can be selected at a time.

Feels right to me!


Add missing ScaleSet properties

The change adds the following missing properties

"zones" property to VirtualMachineScaleSetVM
DiskSizeInGB to VirtualMachineScaleSetOSDisk
DiskSizeInGB to VirtualMachineScaleSetUpdateOSDisk

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[ ] I have read the contribution guidelines.
[ ] My spec meets the review criteria:

[ ] The spec conforms to the Swagger 2.0 specification.
[ ] The spec follows the guidelines described in the Swagger checklist document.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-python

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-node

Automation for azure-sdk-for-go

Automation for azure-sdk-for-java

@soshekar I don't see any examples updated with the new fields, is there a work item for that?

@jhendrixMSFT  I have now added examples to the PR.

apologies. I have added the reference to the example in the swagger as well.


Fix off by one in blockchain slot

This PR fixes an off-by-one error, where the global slot for the genesis block after the hard-fork has the same global slot as the final block of the network.
IMO this is the incorrect way to fix this: slot time should be tied to wall-clock time -- that is, any slots that pass between the final block of the pre-fork network and the first block in the post-fork network should just be empty -- instead of munging the concept of time when we set up the new network. This also does weird things to timed accounts. Oh well.

!ci-build-me

I just realized the test doesn't use this exporter anyway...

Closing. We all now agree that this was the wrong approach.


Add more wording about listing or adding a business to openstreetmap, to try and improve search results

The current onosm site ranks poorly (8/10; despite the top answers linking to it). It makes it hard to find this when talking about it to people.
Example screenshot:

This change slightly improves the wording and metadata available; which will hopefully improve listings.
It would be also worth exploring additional domain names or landing pages. An alternative could also be a small adwords campaign or similar to encourage people into the right location.

I will have to have a think about this one an i18n

@alexandre-mbm - for a user searching in English via google for the following, onosm ranks poorly or not even on the first page of results:

"How do I add my business to OpenStreetMap"
"how do I list my business on OpenStreetMap"
"how can I add business to OSM"
"list business openstreetmap"

The old description doesn't actually talk about what onosm does, but what OpenStreetMap is:
"OpenStreetMap is an open, global map used by people around the world to navigate and find local businesses. It's easy and free to get on the map!"
Though its a small change, it should better describe what onosm actually does and rank a bit better for searches like that.

I wonder if the javascript-based internationalization stuff will be picked up by Google or if there will have to be real content in the header not generated by javascript.

I've assumed that real markup is going to be indexed, at least for the basic structure of the page.
https://developers.google.com/webmasters/ajax-crawling/docs/learn-more talks about google's advice for crawling generated content, and http://searchengineland.com/tested-googlebot-crawls-javascript-heres-learned-220157 talks about some real world results; which contradict each other slightly


NotificationsWhenBusy

Allows you to receive notifications while your status is set to Busy.

slightly confused about a comment in the latest source, the binary seems to be built from this commit e46c218
 however in the latest source there is this comment stating that the config system is not working

It’s not intentional; probably a result of me creating a project on a Razer laptop. I’ll have to scrutinize some assembly properties a bit more next time I build.


Guys and girls, how to build and run "Hello, World" program from the "Rust and WebAssembly" book? npm says that it could not find the executable file

➤ npm init wasm-app www
npm error could not determine executable to run

I found this one:
#https://github.com/rustwasm/create-wasm-app/issues/206#issuecomment-1519116945

I found this one: #rustwasm/create-wasm-app#206 (comment)

@j4dzg3r I got the same issue here. Could you help provide mode details to resolve it. I don't know where to put --target web, should I run command like npm init wasm-app www --target web? really thanks

This comment is about a workless repo. MDN has a topic about wasm and Rust, that can help you compile your program.https://developer.mozilla.org/ru/docs/WebAssembly/Rust_to_Wasm


Update utils.go

github.com/docker/go-units replaces github.com/docker/docker/pkg/units

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "patch-1" git@github.com:humblec/libcontainer.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

@humblec Libcontainer has been integrated to runc , you can move your contribution there.


Real-Time ASD

Hello,
Thank you for your amazing work, I was wondering since the inference times are so low, would it be possible to use this model in real-time with the input being my webcam for example ? If so, what do you recommend doing to optimize ?
Thank you so much

Hi,
Thank you for your interest! The inference is actually very fast if you compute the per-frame processing time (note that our method processes a graph at a time, which includes many frames). And yes, it's possible to extend our work in real time. We are currently working on this problem and will release it within this year! Please stay tuned.
Thank you.
Kyle


[ADN] Fix

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] Searched the bugtracker for similar pull requests
[x] Read adding new extractor tutorial
[x] Read youtube-dl coding conventions and adjusted the code to meet them
[x] Covered the code with tests (note that PRs without tests will be REJECTED)
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Closes #26963. The extractor doesn't work at all currently.

Thanks for the thorough review! This does not address the subtitles situation.

Thanks for the thorough review! This does not address the subtitles situation.

#27841

#27841

Do you want to be credited as one of the authors of youtube-dl? If so under what name?

Sure, thank you :) That would be »Adrian Heine«.


Error in switch(file_format, .txt = abundance <- readr::read_delim(file,  :    EXPR must be a length 1 vector**

I am trying to run ggpicrust2, and am having an issue. Below is the script I'm running:
Load necessary data: abundance data and metadata
KO_abundance_file <- read_delim("pred_metagenome_unstrat.tsv",
delim = "\t",
escape_double = FALSE,
trim_ws = TRUE)
metadata <- read_delim("Metadata.txt",
delim = "\t",
escape_double = FALSE,
trim_ws = TRUE)
Run ggpicrust2 with input file path
KO_results_file_input <- ggpicrust2(file = KO_abundance_file,
metadata = metadata,
group = "Treatment",
pathway = "KO",
daa_method = "ALDEx2",
ko_to_kegg = TRUE,
p.adjust = "fdr",
order = "pathway_class",
p_values_bar = TRUE,
x_lab = "pathway_name")
When I start the analysis, I receive the below error message:
Starting the ggpicrust2 analysis...
Converting KO to KEGG...
Loading data from file...
Error in switch(file_format, .txt = abundance <- readr::read_delim(file,  :
EXPR must be a length 1 vector
Any suggestions on how to resolve this?
Thanks!

Also stuck here...

Dear @bplet01  and @Salineraptor,
Thank you for bringing this issue to our attention. I apologize for any confusion this may have caused. I believe I can help clarify the problem you're encountering.
The error you're seeing is likely due to how you're passing the data into the ggpicrust2 function. I notice that you've already read the data using read_delim before calling ggpicrust2. In this case, you should use the data parameter instead of the file parameter.
Here's how you can modify your code:
# Your existing code to read the file
KO_abundance_file <- read_delim("pred_metagenome_unstrat.tsv", delim = "\t", escape_double = FALSE, trim_ws = TRUE)
metadata <- read_delim("Metadata.txt", delim = "\t", escape_double = FALSE, trim_ws = TRUE)

# Modified ggpicrust2 call
KO_results_file_input <- ggpicrust2(data = KO_abundance_file, metadata = metadata, group = "Treatment", pathway = "KO", daa_method = "ALDEx2", ko_to_kegg = TRUE, p.adjust = "fdr", order = "pathway_class", p_values_bar = TRUE, x_lab = "pathway_name")

The key change here is replacing file = KO_abundance_file with data = KO_abundance_file. The file parameter is used when you want ggpicrust2 to read the file directly, while the data parameter is used when you're passing an already-read data frame.
Please try this modification and let me know if it resolves the issue. If you encounter any further problems, don't hesitate to reach out.
Best regards,
Chen Yang


λcancer pon pon doesn't work

λcancer pon works as expected, but not λcancer pon pon

:noel:

Done in 0bae24dd20fc9ee254cb3b34eed59c1fa37a1714


Translate ParDo and GroupByKey operations through the Runner API

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

R: @vikkyrk

Coverage decreased (-0.007%) to 70.467% when pulling a18966901e878aabd314ab3900d37078cd572d7c on robertwb:runner-api-pardo into e206b1a41c1f22330cb78fd1082425ea1e74dbb4 on apache:master.

LGTM.

Unrelated java failure. Retest this please.

Coverage decreased (-0.3%) to 70.163% when pulling d8dd27bffe9a15e2878e1d019b3c1973e76c139f on robertwb:runner-api-pardo into e206b1a41c1f22330cb78fd1082425ea1e74dbb4 on apache:master.


Handling Errors from Power Outage Safely - Repair script?

After running successfully for a month, my raspiblitz has hit into a first issue. Appears to have lost power briefly which has caused an EXT4-fs error when rebooted.
Process I think may help will be to unmount the hdd, perform fsck and remount. Hopefully that will be sufficient, but otherwise I may need to reindex-chainstate for bitcoind.
Has anyone successfully resolved a similar issue at all? This could be a useful script to put together for version 1.0 to handle 'repair' functions.

For what it is worth, the fsck & reindex process I mentioned was successful - it took the best part of 14 days to reindex-chainstate though.

I would need more details on what would be the repair-options. So removing this from milestone vor 1.0 and maybe adding repair-scripts later on.

In v1.0 there is now basic detection if re-index/repair is needed and the torrent and scp scripts for getting the blockchain can already be used to replace corrupt data again. I would recommend to make this a better UX and offer repairs more user friendly for v1.1

Also check if anything of the following can be added to a repairscript: https://www.tecmint.com/check-linux-hard-disk-bad-sectors-bad-blocks/

To simulate the "data corruption" (for testing) its possible to just go into the directory /mnt/hdd/bitcoin/blocks/index and echo "" > [latest-ldb-file] and then make reboot.

For v1.1 when on SSH into the RaspiBlitz its detecting a reindex/data corruption it will offer now multiple options:

download torrent again
copy blockchain from another computer
start resync by itself

also a rescue backup option is offerred to do first optional.

testing worked fine. closing issue.


[BEAM-6635] increase maxHeapSize for test tasks

with gradle 5 default maxHeapSize got decreased to 512mb. This increases to a more reasonable default for beam

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.
 It's hard to evaluate whether this will be sufficient without running this in the Gradle 5 context.

We certainly can do that.
AFAIU, we reduce defaults here for gradle4. So if changes work there, they should also work on gradle5.
So to recap:

I ll add that maxHeapSize = 2g to those tests requiring that
do that with that [BEAM-6630] PR

Right?

AFAIU, we reduce defaults here for gradle4. So if changes work there, they should also work on gradle5.

Yes, it sounds like it should "just work" with Gradle 5, assuming there are no other unexpected interactions. It would be safer to test just-in-case.

I ll add that maxHeapSize = 2g to those tests requiring that

See comment above. On further thought I worry about accidentally destabilizing tests by reducing their memory. So I'd recommend just giving all test tasks the 2g for now. If you feel like adding this individually to each project, that would make it easier to eventually revert to the default. But, I don't mind if you defer that to another PR.

do that with that [BEAM-6630] PR

Yes, I think it would be best to validate together.

See comment above. On further thought I worry about accidentally destabilizing tests by reducing their memory. So I'd recommend just giving all test tasks the 2g for now.

Of course as previous default was 1/4th of physical memory, that 2g might incur the same problem. Lets test and see...


Merge pull request #1 from kbwalex/master

updated

I have finally push.
Thanks
On Tue, May 9, 2023 at 7:32 PM Kabir Adebayo @.***> wrote:

Sorry bro,
I don't understand this anymore. I had done the project and I have been
trying to git push but all prove abortive.
On Tue, May 9, 2023 at 4:46 PM Obinna @.***> wrote:

Merged #2 https://github.com/kbwalex/AirBnB_clone/pull/2 into master.
—
Reply to this email directly, view it on GitHub
https://github.com/kbwalex/AirBnB_clone/pull/2#event-9202356868, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A4DHJ6D4YYARRQ7O4VSIEZ3XFJRFVANCNFSM6AAAAAAX3PULCU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Okay bro. Will check it once I’m back online.
Regards.
On Tue, 9 May 2023 at 19:50, kbwalex @.***> wrote:

I have finally push.
Thanks
On Tue, May 9, 2023 at 7:32 PM Kabir Adebayo @.***> wrote:

Sorry bro,
I don't understand this anymore. I had done the project and I have been
trying to git push but all prove abortive.
On Tue, May 9, 2023 at 4:46 PM Obinna @.***> wrote:

Merged #2 https://github.com/kbwalex/AirBnB_clone/pull/2 into master.
—
Reply to this email directly, view it on GitHub
https://github.com/kbwalex/AirBnB_clone/pull/2#event-9202356868, or
unsubscribe
<
https://github.com/notifications/unsubscribe-auth/A4DHJ6D4YYARRQ7O4VSIEZ3XFJRFVANCNFSM6AAAAAAX3PULCU

.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
https://github.com/kbwalex/AirBnB_clone/pull/2#issuecomment-1540716493,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3DJMMZIBI2BXCVRC4HURE3XFKGWFANCNFSM6AAAAAAX3PULCU
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

This is my whatsapp number<PHONE_NUMBER>6
On Tue, May 9, 2023, 9:03 PM Obinna @.***> wrote:

Okay bro. Will check it once I’m back online.
Regards.
On Tue, 9 May 2023 at 19:50, kbwalex @.***> wrote:

I have finally push.
Thanks
On Tue, May 9, 2023 at 7:32 PM Kabir Adebayo @.***> wrote:

Sorry bro,
I don't understand this anymore. I had done the project and I have been
trying to git push but all prove abortive.
On Tue, May 9, 2023 at 4:46 PM Obinna @.***> wrote:

Merged #2 https://github.com/kbwalex/AirBnB_clone/pull/2 into
master.
—
Reply to this email directly, view it on GitHub
https://github.com/kbwalex/AirBnB_clone/pull/2#event-9202356868, or
unsubscribe
<

https://github.com/notifications/unsubscribe-auth/A4DHJ6D4YYARRQ7O4VSIEZ3XFJRFVANCNFSM6AAAAAAX3PULCU

.
You are receiving this because you are subscribed to this
thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
<https://github.com/kbwalex/AirBnB_clone/pull/2#issuecomment-1540716493
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/A3DJMMZIBI2BXCVRC4HURE3XFKGWFANCNFSM6AAAAAAX3PULCU
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
https://github.com/kbwalex/AirBnB_clone/pull/2#issuecomment-1540818239,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4DHJ6GAVUJMXJG4XA5VWQDXFKPJPANCNFSM6AAAAAAX3PULCU
.
You are receiving this because you commented.Message ID:
@.***>


"You stumbled upon a Vike bug" in console in dev

Description
error from console:

Uncaught (in promise) Error<EMAIL_ADDRESS>You stumbled upon a Vike bug. Go to https://github.com/vikejs/vike/issues/new and copy-paste this error. A maintainer will fix the bug (usually under 24 hours).
at createErrorWithCleanStackTrace (createErrorWithCleanStackTrace.js?v=4b702f4d:4:17)
at assert (assert.js?v=4b702f4d:54:27)
at setPageContextPrefetchCache (prefetch.js?v=4b702f4d:73:5)
at prefetchPageContextFromServerHooks (prefetch.js?v=4b702f4d:61:5)
at async prefetch.js?v=4b702f4d:212:21
at async Promise.all (:3000/index 1)
at async prefetchOnEvent (prefetch.js?v=4b702f4d:199:5)
it seems to appear before vite optimize all dependencies and I try to go to another page (in this case also page sometime freezes)
ubuntu 22.04
brave browser
bun + hono

It seems like you're trying to use prefetch(url, { pageContext: true })? It isn't supported yet. If you remove the { pageContext: true } option then the error should go away.
Let me know if you're still having any issues then.

Possibly related: https://github.com/vikejs/vike/issues/1920.


Python crashes when calling compiled function with default argument

Code for reproducing the error:
import numpy as np
from numba import jit

a = np.zeros(2)
@jit
def foo(a=a):
    a[0] += 1
foo()

Result:
Crash of python.exe. At the same time foo(a=a) works fine.
Environment:
import sys
import numba
import platform
print(sys.version)
print(numba.__version__)
print(platform.platform())

Output:
3.5.2 |Anaconda 4.2.0 (64-bit)| (default, Jul  5 2016, 11:41:13) [MSC v.1900 64 bit (AMD64)]
0.30.0
Windows-7-6.1.7601-SP1

Thanks for the report. I can replicate on Linux.
Top few frames:
Program received signal SIGSEGV, Segmentation fault.
0x00007ffff7fec08a in ?? ()
(gdb) bt
#0  0x00007ffff7fec08a in ?? ()
#1  0x00007fffe57d6f48 in ?? ()
#2  0x00007fffe5539748 in ?? ()
#3  0x00007fffe489b708 in ?? ()
#4  0x00007fffe5b015f2 in call_cfunc (locals=0x0, kws=0x0, args=0x7ffff7e5ae48, cfunc=0x7fffe47a58d0, self=0x7fffe57d6f48)
    at numba/_dispatcher.c:339
#5  compile_and_invoke (self=self@entry=0x7fffe57d6f48, args=args@entry=0x7ffff7e5ae48, kws=kws@entry=0x0,
    locals=locals@entry=0x0) at numba/_dispatcher.c:362
#6  0x00007fffe5b01f71 in Dispatcher_call (self=0x7fffe57d6f48, args=<optimized out>, kws=<optimized out>)
    at numba/_dispatcher.c:562
#7  0x00007ffff7954236 in PyObject_Call (func=0x7fffe57d6f48, arg=<optimized out>, kw=<optimized out>)
    at Objects/abstract.c:2165
#8  0x00007ffff7a2e314 in do_call (nk=<optimized out>, na=0, pp_stack=0x7fffffffd4b8, func=0x7fffe57d6f48)
    at Python/ceval.c:4936

compile result is like:
CompileResult(typing_context=<numba.typing.context.Context object at 0x7fffe57caf60>,
target_context=<numba.targets.cpu.CPUContext object at 0x7fffe4840780>,
entry_point=<built-in method foo of _dynfunc._Closure object at 0x7fffe47a58d0>,
typing_error=None, type_annotation=<numba.annotations.type_annotations.TypeAnnotation object at
0x7fffe4840a20>, signature=(omitted(default=array([ 0.,  0.])),) -> none, objectmode=False, lifted=(),
fndesc=<function descriptor 'foo$1'>, interpmode=False, library=<Library 'foo' at 0x7fffe4853e10>,
call_helper=None, environment=<numba.lowering.Environment object at 0x7fffe4853eb8>,
has_dynamic_globals=False)

As of Numba 0.56, this does not segfault, however it doesn't replicate the behaviour without the JIT, the array a is not updated. It should probably also be noted that mutable default values often cause surprises, so I'm not sure that this will be fixed.


Consumer searches for Patient using identifier

i.e. search-type Patient interaction
Should we provide guidance around use of national identifiers (NHS Number, CHI Number etc)?
How strongly should this guidance be enforced?
Should the guidance be provided via implementation guidance OR via conformance resources (is this even possible?)?

Guidance will be provided for implementers to support searches using:

national identifiers
local identifiers
systemless identifiers

Following discussion, it was agreed that it will not be mandatory for an implementer to support any specific identifier type.

Interaction added to CapabilityStatement.fsh, draft documentation in capabilities.md.
Will need decision around creation of identifier systems for CHI, H&C numbers.

The IG will include a comparison with IHE PDQm so that implementers understand how the standards are related. It is anticipated that this IG will include a subset of capabilities from PDQm and that additional capabilities related to patient search will only be added where there is a demonstrable need within the UK.

Need comparison with International Patient Access


Added game

Please merge it.It is working

make sure you follow the guidelines! make sure to star the repo and follow


Concierge Flow 1

https://bithyve-workspace.slack.com/archives/C02ETLWF1BP/p1712859696102429

Flow has been implemented.


Analytics/Tracking feature in pdf.js

What is the best way to track user interaction with pdf's served via pdf.js?
I want to track time spent on each slides by each user.

You could detect page changes via the pagechange event.
It is triggered here: https://github.com/mozilla/pdf.js/blob/63ed6816eb5701c263d89ca604d17968c7db39c4/web/pdf_viewer.js#L131-L151
Example of where it is used:
https://github.com/mozilla/pdf.js/blob/63ed6816eb5701c263d89ca604d17968c7db39c4/web/viewer.js#L1792-L1815

You can use this library
https://github.com/hueper/PDF-Analytics


Create a more standard-Docsy homepage

Create a more standard-Docsy homepage for this version of the website. The homepage should look similar to this one for example:

https://opentelemetry.io

You can find the page source for the site's homepage in the OTel website repo. You can also look at other examples for inspiration, such as https://example.docsy.dev.

You can also look at other examples for inspiration, such as https://example.docsy.dev.

See https://opentelemetry.io/ for an example of how to fix the following:

[ ] The top-nav logo:

[ ] The hero image should be about the same size as for the existing website (the green box in the image below), see the OTel website for an example since it has a hero image that is of about the right size as what you need for in-toto:

And in last point are you talking about the learn more button in in-toto.io

@chalin Fixed the top nav bar as suggested only used logo with default title instead with wordmark. I have searched a lot about how can i change hero image but didnt get anything in layout so that i can change css of that particular block.

I'm not seeing that fixed in the preview, but I'll let Lukas review this next week.
As for the last point, I'm referring to these "Read more" links:

@chalin ... I have searched a lot about how can i change hero image but didnt get anything in layout so that i can change css of that particular block.

Try changing height to max in the following:

{{% blocks/cover title="Welcome to in-toto!" image_anchor="top" height="max" %}}

Also, drop the title field as shown above. Instead, include the logo+wordmark just like the current site has. You can see how this was done for the OTel website here: https://github.com/open-telemetry/opentelemetry.io/blame/dc79770a701688205fd4b82532cf66ee5233ecaa/content/en/_index.md#L19-L25

@chalin refer to this link as preview link https://main--in-toto-docs.netlify.app/

The points raised in the second comment have all been address. Well done @DarikshaAnsari. Closing as done.


[Improve][Security] The size option isn't honored after following a redirect in node-fetch

Description

InLong Component
Other for not specified component
Are you willing to submit PR?

[ ] Yes, I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I would like to work on this issue


plasticity: 24.1.8 -> 24.2.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for plasticity is: CAD for artists
meta.homepage for plasticity is: https://www.plasticity.xyz
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 24.2.0 with grep in /nix/store/sm7rnkga5pyfbih471dhg26f2450nlv0-plasticity-24.2.0
found 24.2.0 in filename of file in /nix/store/sm7rnkga5pyfbih471dhg26f2450nlv0-plasticity-24.2.0

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

plasticity

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/ce10ce52b94f4e8dd48ac13fee7563b951b2c27c#plasticity

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/sm7rnkga5pyfbih471dhg26f2450nlv0-plasticity-24.2.0
ls -la /nix/store/sm7rnkga5pyfbih471dhg26f2450nlv0-plasticity-24.2.0/bin

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

1 package built:

plasticity

Maintainer pings
cc @imadnyc for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

Plasticity removed version 24.1.8 from their github releases, so it's no longer building on nix. Merging this, or even better the latest 24.2.3, would fix that.

I'll just update this for now, and let the updater get the latest version later

@NixOS/nixpkgs-merge-bot merge


99 uk digital twin powsys

This is the UK power grid project.

The biggest problem is that there are hard-coded strings, hard-coded URLs/IRIs, and hard-coded absolute paths on a local file system all over the place. It's going to be very painful getting this to work on another machine, with a triple store in another location, or with different ontology base URLs.

All the problems mentioned in this comment have been solved.

@msff2, please test this on another machine (with a fresh clone). Given the (continued!) presence of user-specific hard-coded file-system paths, it seems highly unlikely it will work in its present state.

@msff2, please test this on another machine (with a fresh clone). Given the (continued!) presence of user-specific hard-coded file-system paths, it seems highly unlikely it will work in its present state.

I have conducted tests on another machine and found that the reproduced figures are identical to Figures 2-6, so I am happy for Wanni to merge her branch into the main.


Modifying an Area2D's shape size emits area_exited and area_entered for all contained areas

Godot version: 3.2.2 Stable Mono 6a9fbafcb
OS/device including version: Manjaro Arch Linux 5.4.44-1-MANJARO
Issue description:
Changing the size of the shape in an Area2D causes the Area2D to emit both area_exited and then area_entered when the area's containing areas (and I suspect other CollisionObjects, but I've yet to test that) may not actually change.
Perhaps this is expected behavior, in which case I believe it should be moved to a docs issue.
Steps to reproduce:
Create a project with two Area2Ds setup (enable Visible Collision Shapes under Debug to ease visibility)
Connect to area_entered with a line containing print("entered") and connect to area_exited with a line containing print("exited")
Script the connected Area2D's shape to grow to encompass the second Area2D when the game starts
Minimal reproduction project:
Area2D_size_test.zip
Q to shrink the orange selector box, E to enlarge the orange selector box

This is expected behavior. You should instead change the scale property of Area2D.

Ah nevermind, I thought I had tried scaling an Area2D before and had it fail, thought I read somewhere scaling didn't work properly for Area2Ds.


fixed legend range

Howdy, do you know how to fix the legend? The values in the legend always change, and I tried to change the legend directly but it will not render anyway. Thanks!

Hello @mzy2240  can you please provide perhaps a screenshot and more detail of what you are trying to achieve?

@danmarshall Hey Dan, I'm trying to make it work with real time data. Now everything is working, unless the values in the legend keep changing.  For example, at beginning all data are within 1-1.2 which is the safe zone, so the color should be all blue; and later some data starts to drop so they should become red or something. However,  right now because the legend value keeps changing, so all the points are always blue to red, so color becomes meaningless.

Thanks for the clarification @mzy2240 . Realtime is a scenario we haven't necessarily considered yet. Which component are you using?

@danmarshall I'm using the pure SandDance core with Vue.

Just a thought: perhaps you can prime your dataset with a couple of rows of data which specify the min and max of the range.

Is it possible to pass a fixed a legend option when calling the render function?

Well, you can always just hide the legend and render your own.

However, hide the legend will still change the colormap

what if I want to have a colormap for data over 1 even if there is no data in that range?

Sorry, I forgot that you can intercept the intermediary scenegraph prior to rendering.
In your options, add an onStage handler. The stage object will contain the legend that you may modify. Sorry that some of these docs are not linkink correctly. 😐

Thanks a lot! Yeah thats another issue I want to report.  Most of the doc links are btoken, it's really hard to code without doc.

Hello, we've got new docs. If this is still an issue, please feel free to re-open it.


generate the materials.json file as is reported in the print statement

Closes #26

I've tested this BTW. Everything works and is fully backward compatible.

There you go, thanks!


[NPU] Update npu.md

Motivation
Fix link errors

Hi @wangjiangben-hw ,
Thanks for your kind contribution. Would you like to fix the lint issue so that we could merge this PR?
The details can be found by clicking the item of lint check as shown below

Please fix the lint issue. You can install pre-commit hook on your machine like following:
pip install -U pre-commit
cd mmdeploy
pre-commit install

Then you can run pre-commit run --all-file to check whether there are lint errors


want to run multiple user sequentially

Expected behavior
i have 800 user and i want to run that the next request is sent only after the prior request is completed?
Actual behavior
they run with same timing
Steps to reproduce the problem
create a multiple thread
add http request
csv file
add listner
JMeter Version
5.5
Java Version
No response
OS Version
No response

This is most probably not a bug, but a wrongly constructed test plan.
Have you had a look at building a web test plan?
I guess, that you have a list of pages in your CSV file, that each user should visit one after another. If that is the case, have a look at Sharing Mode in the docs for CSV Data Set config.


Rename DHT tests such that trial automatically runs them

There are still tests in tests/dht that I didn't rename/move because they aren't currently passing.

trivial enough.  merging


Add editor for sector range fan

Ready for testing.

Merged into the dev branch. Closing this issue.


Add increment parameter for skipping video frames

e.g. 10 will read every 10th frame
Increased processesing speed at the expense of error in score calculation.
fixes #32

This seems to work as expected for me


Export fails: Building accelerated plugins...

Thanks for the excellent code...
I've successfully trained retinanet on coco. The accuracy looks decent. Then, I converted the retinanet_rn50fpn.pth model to .onnx.  Then I built the export and infer utils successfully.  So far, so good.  But when I try to export the .onnx model to .path, it fails a Parameter check.
./export retinanet_rn50fpn.onnx out.path

...... hundreds of lines of output.. and then

 530:Conv -> (256, 80, 80)
Building accelerated plugins...
Parameter check failed at: ../builder/Network.cpp::addConcatenation::167, condition: nbInputs > 0 && nbInputs < MAX_CONCAT_INPUTS
Segmentation fault (core dumped)

I'm using TensorRT <IP_ADDRESS>  on a Ubuntu 16.04 desktop, with Cuda Driver Version: 418.56       CUDA Version: 10.1
Thanks

Okay, I see this is a duplicate issue....  Apparently I need to specify --opset=9 when I create the onnx file.  When I do so, it works.

@JimKlingshirn JimKlingshirn  What do u mean specify --opset?? It already using opset9 in
        import torch.onnx.symbolic_opset9 as onnx_symbolic

I specified the opset when calling torch.onnx.export, using extra_args.  For example:
    extra_args = { 'opset_version': 9 }
    torch.onnx.export(net.cuda(), zero_input, onnx_bytes, verbose, input_names=["data"] ,*extra_args)

@JimKlingshirn  I don't get your point, this repo doesn't using torch.onnx.export export onnx model at all, it using Engine to parse it, which line of code does torch.onnx.export used? Did u using the 19.09 branch or old branch?


Switch to London-comparisons everywhere for isSameDay

The isSameDay function is a bit of a weird historical artefact: it was created in the aftermath of removing the Moment.js dependency from the codebase, and so it supports two definitions of "is it the same day":

is it the same day in UTC?
is it the same day in London?

and when you call the function, you have to say which one you want.
I'm ~95% sure we want London comparisons everywhere, but everything involving datetime logic needs care and attention – we need to make sure what the expected behaviour of the calling code is.
This is a tracking ticket for all the small pieces to gradually refactor this away.

@alexwlchan this one is scary because time based is scary, would be good to catch up on its status! Maybe it's even done?

would be good to catch up on its status

So there are only three places left where we call isSameDay() with UTC (excluding the tests):
https://github.com/wellcomecollection/wellcomecollection.org/blob/01060ff98ea5a5507b537fa405a2a6f93bbe9bf5/common/services/prismic/opening-times.ts#L57C1-L59
https://github.com/wellcomecollection/wellcomecollection.org/blob/01060ff98ea5a5507b537fa405a2a6f93bbe9bf5/common/services/prismic/opening-times.ts#L260C1-L265
https://github.com/wellcomecollection/wellcomecollection.org/blob/01060ff98ea5a5507b537fa405a2a6f93bbe9bf5/content/webapp/services/prismic/transformers/events.ts#L457-L463
All of these are used for grouping events rather than the precise boundary of midnight so I think it would be fine to switch them and nothing bad would happen.
The reason I haven't done it yet is because I've yet to think of a good test case that "proves" London is the correct thing to do. I could change them without changing any of the tests, but that feels a bit icky. In other cases, I was able to find an obvious edge case which differentiated between London/UTC.
I'm going to have a quick look at it this morning; look at how these functions are tested and whether that's a sufficient safety net.


closes #350

Works, but cannot be merged for now, see the issues explained at: https://github.com/odalic/odalic-ui/issues/350


Assess performance and memory prior to production release

Per @wbazant's comments in the original development PR, some items have not yet been addressed that need to be before this service goes into production.  Would be nice to have a new branch (from initial dev) for a developer to perform some testing and address concerns.  Wojtek's original notes:
Some missing work before this is production ready that I couldn't do:

  - running with a big index, seeing how much memory it needs (on disk they're 5x 200 MB at most so maybe 1 GB?)
  - checking that the requests for big queries properly stream the response back
  - checking what happens with a few parallel big requests - ideally they'd both process in parallel
  - observing peak memory usage - I think retrieving one while chromosome will cause the biggest jump per request, but actually, even metazoan chromosomes are just a few megabytes so it shouldn't be too bad
  - choosing good cutoff values for max num sequences and max num bases per request: I propose 1e6 and 1e8, but check it fits within 30 seconds
  - right-sizing the JVM setup and containerisation based on that
  - other devops (where this should run and what files it should have)
  - revisiting defline format in fasta: mostly, I think the deflines will come from request itself, so, .bed files should contain the desired header for each subsequence - the choices of "region only" or "query + region" are for user-defined bulk retrieval

Related: submit a very long list of features, and make sure it's rejected without blowing up the container.

this has been released so closing


Run integration tests only in ccov job

Also run them on one thread only.
Fix #397

Codecov Report

Merging #401 (5914c45) into master (f574584) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #401      +/-   ##
==========================================
- Coverage   75.33%   75.31%   -0.02%
==========================================
  Files          83       83
  Lines       10395    10395
==========================================
- Hits         7831     7829       -2
- Misses       2564     2566       +2

Impacted Files
Coverage Δ

minimint/src/net/peers.rs
92.89% <0.00%> (-0.27%)
:arrow_down:

minimint/src/net/connect.rs
94.94% <0.00%> (-0.26%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

It seems to me that this no longer runs rust-tests.sh. One important thing that run-tests.sh does is run bitcoind and lightningd fixtures. Tests still pass, but they just mocking out bitcoind and lightningd. See MINIMINT_TEST_REAL environment variable.

@justinmoon The "Test workspace" step is cargo test --profile release on the whole workspace, which AFAIU executes integration tests as well. The reason there is no output from this test is because it passed before and Nix doesn't bother running it (cachix).

I think it is still being run:

here  it refers flakes .#cli-test.rust-tests
which is in flakes here and will run this

@justinmoon @dpc

It was disabled atm of @justinmoon comment. I've fixed it (hopefully) since then.

@justinmoon The "Test workspace" step is cargo test --profile release on the whole workspace, which AFAIU executes integration tests as well. The reason there is no output from this test is because it passed before and Nix doesn't bother running it (cachix).

"Test workspace" runs .#workspaceTest, which is this:
        workspaceTest = craneLib.cargoBuild (commonArgs // {
          cargoArtifacts = workspaceBuild;
          doCheck = true;
        });

I don't see cargo test --profile release being run anywhere in our project ...

doCheck = true; means that cargo test --profile release will be run on the workspace. https://github.com/ipetkov/crane/blob/master/docs/API.md#optional-attributes , see cargoCheckCommand and doCheck.
You can verify locally with nix build -L .#workspaceTest (in case it does nothing, make some changes in the test code to invalidate cache).
Now I have disabled it because:
env RUST_TEST_THREADS=1 cargo llvm-cov --workspace --lcov --output-path $out/lcov.info"
is running the same tests (unit + integration). Per https://github.com/taiki-e/cargo-llvm-cov#basic-usage:

By default, run tests (via cargo test), and print the coverage summary to stdout.

To verify: nix build -L .#workspaceCov. (waiting for it to finish myself to double check).


Negative values for passive cooling runtime

Version of the custom_component
v1.3.5
Configuration
HA: 2022.11.2
HASC: 1.28.3
Describe the bug
Negative runtime value for "Runtime Passive Cooling" seems masked by 2^16=65536
65536 + (current value) = gives the value presented by my termia

Can you check the value in the mqtt message as I'm not sure this is in HA. It could be the ThermIQ-mqtt as well. Which fw version are you on?

"app_info": "ThermIQ-mqtt 2.22",
"d110": -31358

Try to update the fw...

is already last fw...

This is not in the HA integration then but in the ThermIQ fw. Update will be coming soon....

Try out the new fw 2.48 where I hope this is resolved

Sorry, Not entirely fixed yet

Likely solved withfrom v2.50 and up


Flutter offline payment

Hi!
I have a flutter online store UI. I want to accept cash on delivery by hand for the customer orders. Please refer some firebase option/project.

Hi @laersel2
This platform is not meant for assistance on personal code.
Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


feat: add projects to the dashboard

Suggested new feature or improvement
Dev home can be the main app for devs to manage their projects, but for that, we need to be able to further manage them in it.
Please, add the possibility to manage local projects to the dashboard.

Personally, I think projects should be added from the dashboard instead of from the "Machine Configuration" tab.
Machine configuration is ideal for settings such as the "install applications" (which is an amazing option, btw), "add DevDrive",  and, possibly, "open PowerToys settings" and a direct link to Windows' "Developer Settings".
Scenario

open dashboard
see the button to add new / pin project
by clicking on the button, see the list of projects that were already cloned in "machine configuration", or, directly see the options to:

clone project(s)
add local project(s)
create new

after adding it, the project will be directly visible from the dashboard
for each added project, have the options to manage it, such as:

open with...
remove from dashboard
open project location

Additional details
No response

Thanks for being detailed about what you're imagining here! You might want to share it over in #702 as well.

We're investigating adding a separate projects page for this functionality: https://github.com/microsoft/devhome/pull/1049
Would adding projects widgets to the dashboard solve what you're looking to do here? With the creation aspect staying within the projects page?

We're investigating adding a separate projects page for this functionality: #1049
Would adding projects widgets to the dashboard solve what you're looking to do here? With the creation aspect staying within the projects page?

hey there! I'm sure that with the addition of a projects page my suggestion would be irrelevant, as long as the options to manage the projects are present as well, rather than only the option to launch them (as per the picture within the PR description).
Thanks for the replies @krschau and @cinnamon-msft . I'll be closing the issue and look forward to the projects page!


How to fix the EOFError

Hello @xxyll
I have seen that you have created an issue about EOFError. Can you please how you have solved this issue?
Thanks

As mentioned in #4, the problem certainly comes from the  "OSError: [Errno 24] Too many open files".

It doesn't seem to come from my code, so you could try some solutions proposed in this stackoverflow thread : Python Subprocess: Too Many Open Files

Yes I understand it. However my question was more about which specific fix did @xxyll used in order to solve this problem.
"sudo sysctl -w fs.file-max=100000" <- this?
or
another one on the comments

Ok problem fixed thanks


Enable (in)equality constraints for use with scipy.optimize.minimize interface

also (re)introduce support for a few more scipy minimisers
redefine _minimizer_callback signature to support "trust-constr" minimiser
add a few quick minimiser tests (with and without constraints)
minor touchups

closes #834

still need to make sure that constraints are correctly passed through to other minimisation schemes apart from simple local ones

closes #842

Note that various of the new minimiser unit tests "fail" (error is caught and reported but not re-raised) due to scipy bugs/quirks. I think pisa shouldn't prevent the user from trying out the various minimisation interfaces and options scipy provides in principle, even if some of these are buggy (either always or only when given certain configurations). This should also lower maintenance cost and raise awareness of existing enhancement potential.


rate_limit: Rate limit the /accounts/find/ endpoint.

Closes #19287
This endpoint allows submitting multiple addresses so we need to "weigh"
the rate limit more heavily the more emails are submitted. Clearly e.g.
a request triggering emails to 2 addresses should weigh twice as much as
a request doing that for just 1 address.

Great catch on making sure we ratelimit per email address, not per submit, @mateuszmandera!
One thing I noticed in reading do_test_hit_ratelimits is that it doesn't assert that we only hit the ratelimit on the 6th request; just that the 6th request is ratelimited.  So if we had a bug where we accidentally ratelimited on the 2nd request, tests would still pass.
Maybe a friendly amendment to add, after
https://github.com/zulip/zulip/blob/37845037f4fff2963f4758bc66309ecbff9eeb51/zerver/tests/test_external.py#L166-L169
if i < 6:
   assert.assertNotEqual(result.status_code, 429)

lgtm, feel free to merge after addressing Alex's last comment.

Ah, good find - this allowed finding an actual test bug in these find_account and create_realm tests where .clear_history() was getting called on the default domain instead of the appropriate one - thus completely failing to clear the rate limiting record. Fixed that as prep commits and merging now.

Awesome, glad we caught that!  This sort of "not testing what you think you're testing" bug is very common and it's always worth thinking about how to make sure tests will fail if the test setup is broken.


LockoutEnabled not enable after fail login attempt

I have turn on user login attempt in controller
  var result = await _signInManager.PasswordSignInAsync(model.Username, model.Password,model.RememberMe, true);

Startup
 services
                .AddIdentity<User, ApplicationRole>(options =>
                {
                    options.Password.RequireDigit = false;
                    options.Password.RequiredLength = 4;
                    options.Password.RequireLowercase = false;
                    options.Password.RequireNonAlphanumeric = false;
                    options.Password.RequireUppercase = false;

                    //lock out attempt
                    options.Lockout.AllowedForNewUsers = true;
                    options.Lockout.DefaultLockoutTimeSpan = TimeSpan.FromMinutes(30);
                    options.Lockout.MaxFailedAccessAttempts = 2;
                 })
                .AddEntityFrameworkStores<ApplicationDbContext>()
                .AddDefaultTokenProviders();

But when user login fail only the LockoutEnd have data. The LockoutEnabled column is always false so user can keep login back again. Any idea about this. Thank you

It should get set on your user after you CreateAsync the user, see: https://github.com/aspnet/Identity/blob/dev/src/Core/UserManager.cs#L477

@HaoK I mean when I already have account I want to login to system and the LockoutEnabled is set to true after 2 times login fail

I don't understand what you are asking.
So there are two lockout related properties on the user.
LockoutEnabled must be set to true for the feature to be enabled for the user at all.
If lockoutEnabled = True and LockoutEnd date is set to a time later than now, the user should be considered locked out.

In this specific case, if a user fails the login for options.Lockout.MaxFailedAccessAttempts = 2; the code is only setting the lockoutEnd date but not the lockoutEnabled so he can basically still login.
Regarding your code quote, that also does not work since it only sets the lockoutEnabled but not the lockoutEnd -> the check in IsLockedoutAsync() will return False if the lockoutEnd is not set.
All this also fails in the VisualStudio Identity.Core sample template as well - in fact, it's even worse there as the AccessFailedCount column never gets incremented.
Cheers

That's correct, if lockout isn't enabled for the user, it won't do anything.  Whether lockout is enabled for a user or not, is a separate field, typically you turn it on via AllowedForNewUsers which is default to on, so typically when a user is registered that flag is set to true.
https://github.com/aspnet/Identity/blob/dev/src/Core/LockoutOptions.cs#L19

Its used in Create by the user manager to set LockoutEnabled here:
https://github.com/aspnet/Identity/blob/a273e349eea2be3a40b16e6947b2deab95f4b5b2/src/Core/UserManager.cs#L477

IsLockedoutAsync() expects BOTH LockoutEnabled AND LockoutEnd, otherwise it will always return FALSE (not locked out).
Please read again my comment!

Yes that's how the code is supposed to work, the user must have lock out enabled, and there must be a lock out end date that's in the future.

I'll try again ;)
If that's how it supposed to work, then why:
a. CreateAsync for new users only sets the LockoutEnabled but not LockoutEnd?
b. failed logins only sets the LockoutEnd and not LockoutEnabled?
In both of those cases the user should be locked out but instead the method IsLockedoutAsync() returns false (so the user can still login).
I'd urge you to read again the whole thread...
Cheers

a. because unless you want your newly registered user to be locked out, you don't send lockout end.
b. failed logins should set login end to trigger lockout if its enabled.
I'm pretty sure I'm not the one that's confused...
Anyways this behavior is by design

The default SigninManager provided with Identity.Core does not do that, and neither does the sample provided by the Visual Studio template.
I am pretty sure it's at least 2 people on this thread trying to tell you the code is NOT working correctly (UserManager in combination with SignInManager), so I suggest you try it out and then we'll see who is confused ;)
Cheers

This code hasn't changed in a long time, since before core, and there are unit tests so while its possible there's an issue, I'd assume it would have been discovered earlier.
If I were to guess what is going on, users were created with lockout disabled (allowedForNewUsers = false, or by some other mechanism) so their lockoutEnabled = false.  If that's the case, irrespective of what lockoutEnd value is there, that user will never be locked out.
The templates indeed do not have this enabled by default as its no longer recommended practice as it allows people to lock out other users.

I understand that things change and designs/templates need to be adjusted, but in the short term at least an update to the documentation would probably be helpful for all sides.
In it's current state the following configuration has absolutely no effect, which I am sure was not the original intent...
//lock out attempt options.Lockout.AllowedForNewUsers = true; options.Lockout.DefaultLockoutTimeSpan = TimeSpan.FromMinutes(30); options.Lockout.MaxFailedAccessAttempts = 2;
Cheers

The new default UI might be making it harder to turn on, did you scaffold any of your pages?

I don't see any issues with the current default UI templates, I tried by just setting
        services.AddDefaultIdentity<IdentityUser>(o => o.Lockout.MaxFailedAccessAttempts = 1)

And then the user on first failure is immediately locked out.

Thanks, but it seems we are using different UI templates :(
I've used the latest on VisualStudio 2017, but I don't have .AddDefaultIdentity, instead only .AddIdentity:
`
services.AddIdentity<ApplicationUser, IdentityRole>(options =>
{
options.Lockout.MaxFailedAccessAttempts = 3;
options.Lockout.DefaultLockoutTimeSpan = TimeSpan.FromMinutes(3);
            options.SignIn.RequireConfirmedEmail = true;
        })
            .AddEntityFrameworkStores<ApplicationDbContext>()
            .AddDefaultTokenProviders();

I am using the latest NuGet:

`
Cheers

Did you turn the lockout flag on in your PasswordSignIn call in the controller/login action?

Yes of course ;)
But really just by looking at the sourcecode (linked in my other replies) this can't work - unless I am looking at the wrong files or missing something...
Cheers