Displaying Category image and Description

Description
Fixes #3604 and #4527
What changes did you make and why?
Avail the images in for category for making it better distinguishable, getting thumbnails and description from API calls, and putting it in layout.
Tests performed (required)
Tested latest master on Pixel 3 API 30
Screenshot

@nicolas-raoul Images are available now on display. Please review it once. And I decided to display the description in this PR. I am trying to find the API but I am not getting it. I will try again.

@nicolas-raoul Should I use this API for getting the description?
https://commons.wikimedia.org/w/api.php?action=query&format=json&prop=description&titles=Category%3ARabbit (zodiac)

Looks good indeed! :-)
If possible, make the loading asynchronous so that it does not slow down the rest of the information from showing.

@nicolas-raoul I added description also. And I think I already made the call asynchronous and for me, It's not slowing down anything. Isn't it?
If there is any problem with the PR, please let me know. I will try to fix that.

It is very helpful and loads fast without slowing down anything, that's great!
Two things I have noticed:

Many categories have the description "Wikimedia category", which is not super useful. In such cases maybe it would be possible to hop to the category's main topic's description? That can be left as a future enhancement, though.
Descriptions and thumbnails could be cached a little longer? They seem to disappear as soon as I scroll, which does not happen for depictions. If you don't want to fix this now feel free to create a separate issue about this.

Thanks! It is making category selection much more reliable, and thus will make all uploaded pictures more findable by Wikipedia editors :-)

@nicolas-raoul Yes, category's main topic's description will more useful. But I searched everywhere in MediaWiki APIs. I can't find any endpoint from where indirectly or directly I can get the category's main topics description. Can you give me some insight on how can I get category's main topic's description?

I am telling you everything that I observed here. To get the thumbnail and description I need to do API calls. Now there 2 ways I can found to get thumbnail and description.

1. BlockingGet
Pros: The thumbnail and description will not appear and disappear with scrolling.
Cons: The UI will get very slow and buggy while fetching categories. Because BlockingGet will block the thread until it got the result which I don't think you wanted.
2. Subscribe
Pros: Will does not block any thread image and description will visible when it is will be available without any kind of lag in UI. So I followed this approach.
Cons: I think because of the above reason it is disappearing and appearing with scrolling. Because each item will try to fetch the thumbnail and description when the view will be attached to the data.
Now Depicts are not behaving like that because in depicts for thumbnail and description we are not doing other API calls. in depicts, every information will come with a single API call while loading and we are getting it from data class instance.
Still, I will try to do it in another way to make it more efficient (If possible).
Thanks for your appreciation.

Understood for the difference with Depicts, thanks!
A "subscribe" strategy is good. I guess there is a way to cache its results momentarily, though?

@nicolas-raoul I was thinking about getting the thumbnail and category along with searching the category. I mean like depicts, the API call will fetch category names along with thumbnail and description and display them when everything is available (how it's working for depicts). What's your thought on this idea?
If you know any way to make it better, please let me know about that.
My exams are very close now so I am a little busy with them. I will start to work on it soon.

It sounds like it would make the UI slower, unfortunately. For Depicts we
can get a lot of details with just one query.
How about maintaining in memory a
HashMap category→(thumbnail, description)
?
Of course, please prioritize your exams, that's the most important, good
luck! 👍
On Wed, 4 Aug 2021, 15:35 Ayan Sarkar, @.***> wrote:

@nicolas-raoul https://github.com/nicolas-raoul I was thinking about
getting the thumbnail and category along with searching the category. I
mean like depicts, the API call will fetch category names along with
thumbnail and description and display them when everything is available
(how it's working for depicts). What's your thought on this idea?
If you know any way to make it better, please let me know about that.
My exams are very close now so I am a little busy with them. I will start
to work on it soon.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/commons-app/apps-android-commons/pull/4531#issuecomment-892403427,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAYKBU36LB7645UJ4T64MDT3DNUNANCNFSM5BHCRM3Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

@nicolas-raoul For categories also I think we can get all the details with thumbnail and description.
See this: https://commons.wikimedia.org/w/api.php?action=query&format=json&formatversion=2&prop=categoryinfo|description|pageimages&piprop=thumbnail&pithumbsize=70&generator=allcategories&gacprefix=ra&gaclimit=25&gacoffset=0
I was talking about using this API call and do it like Depicts.

Sorry for the delay!
The API call you found looks great, please feel free to proceed with it, thanks :-)

@nicolas-raoul But If we proceed with this API we will get the thumbnail and description only in the production server. With the commons beta URL, we can't get thumbnail and description. But get can do this GET call with the prod URL in beta. Should I do that?
I also have a PR regarding using prod URL for some beta GET calls. You can check #4551

Production is the most important.
If the same does not work on the beta server, it is sad but should not prevent us from developing useful features like this one.
Afterwards we can continue the discussion at #4551

@nicolas-raoul I made the required changes. You can review it now.
Thanks.

Changed the logo. Is it ok now @nicolas-raoul?

Thanks for changing the logo!
I am testing the PR now. It seems that the app does not see the categories I selected. Here I selected two categories, but strangely only one is visible on screen, and even more strangely I get the "No Categories Selected" popup:

The uploaded file indeed does not have any of the two categories I selected.

@nicolas-raoul I tested the PR on both beta and prod but Can't reproduce this kind of behavior. Can you please recheck it once?

The problem seems to not happen every time, indeed.

@nicolas-raoul Are you able to produce this issue on prod? I tested a lot of times and found out that sometimes in beta this issue is occurring. But in prod, It never happens for me.
Actually, I am trying to find the cause of this unexpected issue, I was thinking that the cause might be the URL but I am not sure.

Yes, that was in prod. Not sure if related, but I had a depiction selected, which resulted in a category being pre-selected.

Sorry would you mind rebasing on master?

Oops sorry you read m mind already! Thanks a lot, testing it :-)

@nicolas-raoul I tried a lot to find the cause of this problem but can't find any change in my code which can occur this type of problem. I just added a description with CategoryItem everywhere in the codebase and show it as same as category name. Also, it's very hard to reproduce it so I can't test it properly, it occurred twice in 5 days for me. Can you check if this issue is happening with this PR or master too?

I uploaded 3 pictures today and got the problem twice.
You are right, I will check whether it happens on master.
Le lun. 20 sept. 2021 à 14:13, Ayan Sarkar @.***> a
écrit :

@nicolas-raoul https://github.com/nicolas-raoul I tried a lot to find
the cause of this problem but can't find any change in my code which can
occur this type of problem. I just added a description with CategoryItem
everywhere in the codebase and show it as same as category name. Also, it's
very hard to reproduce it so I can't test it properly, it occurred twice in
5 days for me. Can you check if this issue is happening with this PR or
master too?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/commons-app/apps-android-commons/pull/4531#issuecomment-922873125,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAYKBV333NNZG47ET6CEZLUC4QQJANCNFSM5BHCRM3Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Thanks, @nicolas-raoul please let me know about the master.  Don't know why it's very hard to reproduce for me. Can you upload some images for which this issue is occurring?

@nicolas-raoul I changed something. Issues are not occurring for me but I am not sure as it's not occurring everytime. Can you check issues are still there or not?

@Ayan-10 Few of the tests are failing, can you take a look into that :)
Also, can you rebase your branch with the master?
> Task :app:kaptBetaDebugUnitTestKotlin
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/category/CategoriesModelTest.kt: (45, 25): Type mismatch: inferred type is List<String>! but List<CategoryItem>! was expected

e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/category/CategoriesModelTest.kt: (45, 37): Type mismatch: inferred type is List<String> but List<CategoryItem>! was expected
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/category/CategoriesModelTest.kt: (48, 65): The boolean literal does not conform to the expected type String
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/category/CategoriesModelTest.kt: (48, 70): No value passed for parameter 'thumbnail'
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/category/CategoriesModelTest.kt: (48, 70): No value passed for parameter 'isSelected'
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/category/CategoriesModelTest.kt: (68, 61): Type mismatch: inferred type is List<String> but List<CategoryItem> was expected
> Task :app:compileBetaDebugUnitTestKotlin FAILED
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/category/CategoriesModelTest.kt: (71, 25): Type mismatch: inferred type is List<String>! but List<CategoryItem>! was expected
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/category/CategoriesModelTest.kt: (71, 55): Type mismatch: inferred type is List<String> but List<CategoryItem>! was expected
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/category/CategoriesModelTest.kt: (79, 25): Type mismatch: inferred type is List<String>! but List<CategoryItem>! was expected
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/category/CategoriesModelTest.kt: (79, 37): Type mismatch: inferred type is List<String> but List<CategoryItem>! was expected
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/category/CategoriesModelTest.kt: (80, 63): Type mismatch: inferred type is List<String> but (Mutable)List<CategoryItem!> was expected
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/category/CategoryClientTest.kt: (37, 27): Type mismatch: inferred type is List<String> but List<CategoryItem>! was expected
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/category/CategoryClientTest.kt: (40, 27): Type mismatch: inferred type is List<String> but List<CategoryItem>! was expected
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/category/CategoryClientTest.kt: (62, 27): Type mismatch: inferred type is List<String> but List<CategoryItem>! was expected
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/category/CategoryClientTest.kt: (65, 27): Type mismatch: inferred type is List<String> but List<CategoryItem>! was expected
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/category/CategoryClientTest.kt: (87, 27): Type mismatch: inferred type is List<String> but List<CategoryItem>! was expected
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/category/CategoryClientTest.kt: (107, 27): Type mismatch: inferred type is List<String> but List<CategoryItem>! was expected
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/upload/CategoriesPresenterTest.kt: (70, 92): The boolean literal does not conform to the expected type String
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/upload/CategoriesPresenterTest.kt: (76, 68): The boolean literal does not conform to the expected type String
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/upload/GpsCategoryModelTest.kt: (23, 64): Type mismatch: inferred type is List<String> but List<CategoryItem> was expected
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/upload/GpsCategoryModelTest.kt: (24, 40): Type mismatch: inferred type is List<String> but List<CategoryItem>! was expected
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/utils/StringSortingUtilsTest.kt: (15, 26): Type mismatch: inferred type is String! but CategoryItem! was expected
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/utils/StringSortingUtilsTest.kt: (41, 26): Type mismatch: inferred type is String! but CategoryItem! was expected
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/utils/StringSortingUtilsTest.kt: (63, 26): Type mismatch: inferred type is String! but CategoryItem! was expected
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/utils/StringSortingUtilsTest.kt: (93, 26): Type mismatch: inferred type is String! but CategoryItem! was expected
e: /home/runner/work/apps-android-commons/apps-android-commons/app/src/test/kotlin/fr/free/nrw/commons/utils/StringSortingUtilsTest.kt: (123, 26): Type mismatch: inferred type is String! but CategoryItem! was expected

Despite the Paris category having both an image and a description, I do not see them on the app with the current code of this branch, is it intended? Thanks! :-)

@nicolas-raoul I can't reproduce the issue that you are facing. As you can see in the screenshot Category Paris is appearing with an image and description.
But I can see the UI is not working properly now. Don't know how? It was fine when I checked it last time. I will fix the UI.

Maybe I had a network problem or something.
Understood, I will test again after your fix.
Thanks!

@nicolas-raoul I fixed the UI issue. Also, another issue that you are facing is actually occurring. When you will search Paris in the search box it will not occur, as you can see in my screenshot the image of Paris is showing as expected. But when the Paris is coming from a selected depiction (without searching Paris), it will not show the thumbnail and description because when we are getting categories from depiction the thumbnail and description is not available in the server. This is the same problem that I mentioned in  #4539 .

@nicolas-raoul The problem is it's not occurring every time. I tried with the exact same steps but it's occurred only once. Also, I didn't change any logic in selecting categories, don't know why it's happening. And as it's very hard to reproduce and very randomly occurring, I am not able to test to find the cause of the issue. Still, I fixed something to fix the issue. Please review it once. And also If you observe any specific event for which the issue is occurring please let me know.
Thanks.

@nicolas-raoul @neslihanturan I fixed the issue (selected categories disappearing). Can you please check it once?

@nicolas-raoul I checked those issues. I was on another PR. I installed this PR (without uninstalling the app with another PR). These issues occurred for me. Then I uninstalled the app and then installed this PR, after that these issues are not occurring anymore in this PR. I rechecked by doing the same process and the same thing happened (Issues are occurring if I install the app without uninstalling the previous version).
Can you please check it once after uninstalling the current version and then installing this PR?
Thanks.

Thanks Ayan for finding this out!
Would you mind creating a new issue about it?

@nicolas-raoul The issue has been created.

@nicolas-raoul I was using stream() function for java classes to extract and map objects and as stream() is only supported till API level N, I need that annotation. Now I removed all steam() functions and did it in another way so API level doesn't matter anymore.
Thanks

I also love streams!
Unfortunately we have to support old versions of Android, as many of our most important users do not have the financial means to use anything but old phones.
Thanks for the modification!

@nicolas-raoul I hope the TH Liner is coming from selected depictions that you selected in the previous screen. That is why it's not showing any other information other than the name. I explained the reason in this comment.
Other than that If you search TH Liner in the search box it will show the description and thumbnail. As you can see in the screenshot.

Answering my TH Liner question above:
It is probably because of using the Mediawiki API's pageimages method, which does not get images from {{Wikidata Infobox}}.
An improvement would be getting the P18 from the Wikidata item of the category if pageimages returns nothing. Do you want to try? If not, I can just merge this for now, and create an enhancement issue. Like you want :-)

Ah sorry I just saw your reply! Oh, looks like my intuition was wrong :-)

Thank you for linking to https://github.com/commons-app/apps-android-commons/pull/4531#issuecomment-1000094870 and sorry for forgetting that!
In such cases, image/description could be retrieved from Wikidata. Do you want to try, or should I merge this as is, and create a separate issue?

And to answer your question, yes, TH Liner came because I had selected the TH Liner depiction :-)

I would like to work on that as a separate issue.
Thanks.

Understood. Would you mind creating the issue?
Thanks for this very useful PR!

Sure, I will create the issue.


Alternate Radial Navigate and Select buttons

The current radial memu is nice, but not very configurable. It'd be nice to be able to use the left stick and an alternate button to select in the radial (like X switches from swap hands to Activate Radial)

You can change the select button in the bindings (it's the button to open/close the radial menu). I can add an option to switch the thumbsticks though

you don't get what i mean, i mean a seperate activation button, since what'd it be nice to be able to do is hit left on the dpad to open the radial, then X to select the thing i want to do.

I can't add an alternate select button since the radial menu is not a real GUI like the player's inventory. Adding a different select button to the one that is used to open/select it would take up another button on the controller which is already full. I did though add the option to change the thumbstick in https://github.com/MrCrayfish/Controllable/commit/34e41e37b57de31045482a508cf7633e3bca091e and you can do what you originally planned in OP but open/select radial menu would be just the swap hands button, not different buttons.


Fix websocket port config [Finishes #153297153]

Please refer to commit message for details.

I tested this locally running two nodes locally (and checking that both nodes started, and checking $ netstat -ltna | grep '3[01][12][0-9]' for seeing the port used).

sync test suite fixed in force-pushed commit.

All Linux CI jobs passed.


Cannot drag both on langflow and flowise at all.

Describe the bug
Load on https://huggingface.co/spaces/Langflow/Langflow-Preview
cause the following error on console:
145Third-party cookie will be blocked. Learn more in the Issues tab.
Dark Reader: unexpected error during message passing.
2translator.js:566  Uncaught TypeError: Cannot read properties of undefined (reading 'lastError')
at HTMLDocument.mouseup (translator.js:566:32)
    [NEW] Explain Console errors by using Copilot in Edge: click
     to explain an error.
    Learn more
    Don't show again

Langflow-Preview:1  Uncaught (in promise) Error: A listener indicated an asynchronous response by returning true, but the message channel closed before a response was received
11translator.js:566  Uncaught TypeError: Cannot read properties of undefined (reading 'lastError')
at HTMLDocument.mouseup (translator.js:566:32)
and i cannot drag to add more not on paint board.
Browser and Version

Browser [EDGE]
Version [124.0.2478.51]

To Reproduce
1.go to this web , ceate a flow , try to drag node.
Screenshots

Really strange question.

Flowise is a JavaScript-based project, and Langflow is a Python-based project. The main contributors of these two projects are different, and the operational logic itself varies, making them incompatible with each other.

Hey @LeoYelton
Could you try it on a different browser?

Almost try on all available browsers，wondering for it，I'm not sure Is that any relation between I just try to use symbol link to make my cdriver bigger and i move Most of file user local to driver d，And make link to them，I find some of application could not run so i moved them back to c driver again. And I'm not sure how the browser calls And interact to to user local file，I just cannot Drag any of the node，In ie chrome and Firefox.
---- Replied Message ----
| From | Gabriel Luiz Freitas @.> |
| Date | 04/27/2024 01:02 |
| To | @.> |
| Cc |<EMAIL_ADDRESS>|
| Subject | Re: [langflow-ai/langflow] Cannot drag both on langflow and flowise at all. (Issue #1762) |
Hey @LeoYelton
Could you try it on a different browser?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Try using it in Huggingface Spaces

Try using it in Huggingface Spaces

Really strange. I did not have access to the website 4days then i can drag now. Is it some js compiler bug caused by chromium?


Adding some features from Sprint 3

Team Hendrik: #186, #12

Coverage remained the same at 100.0% when pulling a466c64fb2e6529f6f7fe787d558edf9ec2f3a6f on dev into 5a6ee295770bfd09a41b4da9347c4a46586a21b9 on master.

Coverage remained the same at 100.0% when pulling d9add9e4403ba0786d7e2db939204105dacdcea6 on dev into 5a6ee295770bfd09a41b4da9347c4a46586a21b9 on master.

Obsolete because we changed the deployed branch to the production-branch which was already up to date with dev.


Validating consistency fails for unknown reason (cache invalidation issue?)

I'm updating things (crane input, fenix input, cargo update)  in my crane-wrapping Nix-library/toolkit (flakebox): https://github.com/rustshop/flakebox/pull/148
Seems to be building fine etc.
But when I attempt to update project using flakebox to this new version ... the binary fails to compile (flakebox-bin from flakebox flake, compiled using crane), because of vendored stuff differences?
Executing cargoSetupPostPatchHook
Validating consistency between /build/source/Cargo.lock and /build/flakebox-0.0.1-vendor.tar.gz/Cargo.lock
7c7
< version = "1.1.3"
---
> version = "1.1.0"
9c9
< checksum = "8e60d3430d3a69478ad0993f19238d2df97c507009a52b3c10addcd7f6bcb916"
---
> checksum = "0f2135563fb5c609d2b2b87c1e8ce7bc41b0b45430fa9661f457981503dd5bf0"
16c16
< version = "0.6.13"
---
> version = "0.5.0"
18c18
< checksum = "d96bd03f33fe50a863e394ee9718a706f988b9079b20c3784fb726e7678b62fb"
---

...

Either there's some caching in Nix that I'm not aware, or crane is doing some caching that fails to invalidate vendored stuff.

Nvmd. It seems I have an output that is not using crane and I forgot about it. Sorry for the noise.


SGE cluster hanging?

Hello,
I had gotten the pipeline working and started to align, but stopped to adjust a parameter (decreased align_split_size). Since then, I haven't been able to get the alignment to run. I'll get to the point where the output says something like:
[2015-07-30T02:22Z] node0188: Resource requests: ; memory: 1.00; cores: 1
[2015-07-30T02:22Z] node0188: Configuring 1 jobs to run, using 1 cores each with 1.00g of memory reserved for each job

a number of times, then will just sit there for hours. I couldn't see anything in the logs that would suggest what is wrong except the occasional No heartbeat message. I was trying to run with 1000 cores previously, but now I'm trying with just 1 core and the sge log is full of No heartbeat messages (48 in a row currently). The out gets to ipython: disambiguate_split now and has been there for almost an hour.
Likely something is wrong on my end, but I'm not sure how to diagnose at this point.
python -c 'import socket; print socket.gethostbyname(socket.gethostname())' returns an IP address.

Tyler;
Sorry about the issue. Hanging without any output is frustrating. I'm not sure what is happening but can offer some suggestions:

When you run in non-distributed mode (-n 1) on a local machine, does it still hang?
How many samples are you running together? Since you mention 1000 cores, I'm assuming it's a lot. You might want to batch them up into smaller chunks (<100 together) as the iPython framework can get overwhelmed with a ton of messages.
Changing align_split_size in the middle of an alignment may not work cleanly if some blocks already got aligned. bcbio will re-run all of the split alignments since the endpoints changed, so you'll end up with a larger number of total BAM files.

Hope these help some with diagnosing the issue.

Hi Brad,
Thanks for the suggestions. I think the issue was related to the number of alignment jobs submitted. I got things running again with a larger align_split_size (with alignments starting over). Do you have an idea of the upper limit on the number of jobs ipython parallel can handle?

Tyler;
Thanks for following up. That makes sense, as there is a limit to the scalability of the IPython controller and messages. I've used 100 samples x 250 regions = 25k concurrent messages without issues but you can probably push a bit more. The major issue becomes memory limitations on the controller machine so it is somewhat machine specific. We're actively working to integrate with the common workflow language (CWL) where we can run on alternative schedulers that hopefully won't have these limits. Thanks again.

It sounds like you are able to align more than one sample at a time, but my run seems to be running less than half as many alignment jobs as requested cores. Is there a parameter I can fiddle with to see if I can get more jobs running? Maybe amount of memory on the controller machine (I'm assuming this is where the bcbio-c job is running)?
When I was having issues bcbio was running 1 sample split over <1000 cores.

Tyler;
align_split_size is the right command to give you more chunks to process on. I haven't tried splitting up a single sample this dramatically but it should be possible. Generally what bcbio tries to do is use as many cores as possible on a single machine (set in bcbio_system.yaml) so if you have 32 cores/machine, you'd want to target 1000 / 32 ~= 32 splits, so divide the number of reads in your fastq files by 32 and use this as align_split_size to maximize cores.
Practically the splits all need to get merged together in the second step so there may not be a huge advantage of splitting more aggressively than that. Hope this helps.


Registry controller: Add support for create and update manifest

Currently, we have a proto defined for the manifest definition.
In order to store the proto in the registry, we need support for create and update commands. The command might look something like this:

registry controller create manifest {yaml_file_path}
registry controller update manifest {yaml_file_path}
registry controller execute manifest --manifest_ids="" (execute all manifests if ids are not specified).

yaml_file: Will include the manifest definition, project_id and a manifest_id. The manifest_id will be used to create a project level artifact with artifact ID like "manifest:{manifest_id}". We can also add more annotations to the artifact id to differentiate between internal manifests and external (customer defined) manifests.

CC @timburks

This is a good breakdown of necessary tasks.
The create and update commands are essentially working with artifacts, so let's see if those might be formulated more generally.
I think it would be ok to have the execute operation work on a single manifest.
Let's say that apihub- is a reserved prefix for artifact names, and that our default manifest would be stored at projects/{projectid}/artifacts/apihub-manifest. If customers defined their own manifest, they could put it in a different artifact and they or we could schedule a different worker to run the controller for the customer manifest.
To me, execute suggests that the manifest is imperative, and exposing controller in the name might be unnecessary. Perhaps we could use something like registry {verb} /projects/{projectid}/artifacts/manifest_id to perform the tasks specified in the manifest. For verb we might use make, reconcile, or something similar.
WDYT?

For create and update actions, there is currently no generic create command for artifacts (or any resources for that matter) in the registry tool, I believe that will happen through the auto-generated apg commands. So my understanding is that, this create functionality through the registry tool will upload only the manifest artifacts and not anything else.  Hence, mentioning upload-manifest or something similar will make it clear what this command supports.
The reason why we don't want to make it generic upload is because, registry tool is offering add-on capabilities and not basic create/modify operations on the resources in the registry (those are offered through apg command). Makes sense?
While typing out the above explanation I realized we can also do a single command like:
registry upload-manifest  {manifest_file_path} or
registry upload {manifest_file_path}
No separation of create and update required.
I like the idea of execute operation taking a single manifest as an argument. Reserved prefix also makes sense.
Thanks for those suggestions, how about this?
registry resolve projects/{project_id}/artifacts/{manifest_id}

Summarizing the above comment. We will add two registry commands:

registry upload-manifest {manifest_file_path}
registry resolve project/{project_id}/artifacts/{manifest_id}

apihub- prefix will be reserved for internal controller job.


Explicitly cast numeric literals to type byte to avoid ESP32 and ESP8266 compile errors

This is a fix for issue #6.  The ESP32 and ESP8266 code uses templates to define the min/max function.  This causes a compile error because the numerical literals are of implicit type (int) being compared to a byte parameter.
Even though the types are compatible, this type mismatch causes a compile error in the template used to define the min/max function in the ESP32 and ESP8266 code.
This fix explicitly casts the numeric literals in min() and max() to type (byte) and eliminate the compile error.

*** DARN I MISSED A MERGE ***  I will rebase and resubmit.


Is this library working yet?

If not what needs to be done...maybe I can help?

It is indeed! It's currently being used by crates.io as well.
If you've got anything you'd like to see added though, feel free to submit a PR!


Running the debugger opens godot engine menu

Operating system or device, Godot version, GPU Model and driver (if graphics related):
Godot 3.0 aplha 2 - Windows 10
Issue description:

I run godot's debugger and it showed godot engine main menu instead of my project. The bug still happened randomly after I restarted the editor.
This is what console prints in normal way:
running cmdline: "D:\Godot Engine\Godot_v3.0-alpha2_win64.exe" "--path"
"D:/Godot%20Engine/Projects/3.0%20networking" "--remote-debug" "<IP_ADDRESS>:6007" "--
allow_focus_steal_pid" "10748" "--position" "171,84"

and what it prints when the bug appeared (if I remembered):
running cmdline: "D:\Godot Engine\Godot_v3.0-alpha2_win64.exe" "--path"  "D:/Godot%20Engine/Projects/3.0%20networking" "--remote-debug" "<IP_ADDRESS>:6007" "--
allow_focus_steal_pid" "10748" "--position" "171,84" "res://lobby.tscn"

lobby.tscn is my main scene.

Duplicate of #11391?

Duplicate of #11391?

Looks like it yes.


Mention the DEBUG environment variable

I found out about the DEBUG env var from this issue. I think it could be useful to mention it in the readme since it might be able to help others if they encounter any issues.

thank you for making our documentation better :)


initial work towards moving away from Event.Trailing (closes #5)

Closes: https://github.com/lrstanley/girc/issues/35
Closes: https://github.com/lrstanley/girc/issues/19
See also: https://github.com/lrstanley/girc/pull/15#issuecomment-413845482

From: https://modern.ircdocs.horse/

Here are some examples of messages and how the parameters would be represented as JSON lists:
  :irc.example.com CAP * LIST :         ->  ["*", "LIST", ""]

  CAP * LS :multi-prefix sasl           ->  ["*", "LS", "multi-prefix sasl"]

  CAP REQ :sasl message-tags foo        ->  ["REQ", "sasl message-tags foo"]

  :dan!d@localhost PRIVMSG #chan :Hey!  ->  ["#chan", "Hey!"]

  :dan!d@localhost PRIVMSG #chan Hey!   ->  ["#chan", "Hey!"]

As the last two examples show, a trailing parameter (a parameter prefixed with ':') is another regular parameter. Once the ':' is stripped, software MUST just treat it as another param.

Essentially, Event.Trailing is now just an additional entry in the Event.Params slice.
I'm still opting to have the concept of Trailing as a semantic wrapper (and has nothing to do with the rfc's use of "trailing").  Event.Trailing(), which now  just returns the last parameter/argument (from Event.Params directly), which will help as it is much more frequent to get the last element, than any other element (and Event.Params[len(Event.Params)-1] is very verbose). The other benefit is it defaults to an empty string. Maybe I'll rename it to Last?
This probably has bugs in it. Tests run fine, did some initial connections to servers, but I'd like some contributors to test this out before I merge into master:

@nmeum
@42wim
@puffrfish
@bmeh
@Bevinsky
@Kethsar

Also, @qaisjp, this has that spelling correction.

Codecov Report

Merging #36 into master will increase coverage by 0.45%.
The diff coverage is 43.01%.

@@            Coverage Diff             @@
##           master      #36      +/-   ##
==========================================
+ Coverage   55.96%   56.41%   +0.45%
==========================================
  Files          13       13
  Lines        2339     2315      -24
==========================================
- Hits         1309     1306       -3
+ Misses        922      904      -18
+ Partials      108      105       -3

Impacted Files
Coverage Δ

format.go
97.16% <ø> (ø)
:arrow_up:

commands.go
0% <0%> (ø)
:arrow_up:

cap.go
22.77% <0%> (ø)
:arrow_up:

cap_sasl.go
0% <0%> (ø)
:arrow_up:

client.go
70.27% <0%> (ø)
:arrow_up:

ctcp.go
69.49% <100%> (ø)
:arrow_up:

event.go
69.96% <42.5%> (+2.68%)
:arrow_up:

builtin.go
61.59% <62.5%> (+0.79%)
:arrow_up:

conn.go
59.34% <83.33%> (+0.81%)
:arrow_up:

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 c1e59a0...d505e33. Read the comment docs.

First of all: Thanks for this patch, this is really useful to me.
I did some quick tests and the first thing I noticed is that /me messages are currently formated incorrectly bei Event.Pretty(). The code for formating them in event.go looks as follows:
if ctcp.Command == CTCP_ACTION {
    return fmt.Sprintf("[%s] **%s** %s", strings.Join(e.Params, ","), ctcp.Source.Name, ctcp.Text), true
}

and should prorably be changed to:
if ctcp.Command == CTCP_ACTION {
    return fmt.Sprintf("[%s] **%s** %s", strings.Join(e.Params[0:len(e.Params)-1], ","), ctcp.Source.Name, ctcp.Text), true
}

Otherwise they are formated as [#channel,ACTION foo] **user** foo instead of [#channel] **user** foo.
Might also be nice to add a helper method for getting all parameters except the trailing one (e.g. e.Params[0:len(e.Params)-1]) since this idiom is used in quite a few places now, but that's up to you.

@nmeum -- Latest commit should have fixes for that and TOPIC's.
I did decide to rename Event.Trailing() to Event.Last(), as Trailing() may mislead (since it's just the last param, not the trailing param necessarily).
I am not sure I want to add a helper to grab all but the last, because it's both not used too frequently (I think this is the only occurrence actually), but it's also not beneficial without also keeping track of if the last item was a trailing item (which means adding a field in Event that says it had a trailing argument).


[Repo] Upstream sync

Description

TODO

closing in favor of #183 cause I don't want to deal with this one


Doc Update

I've made some document changes based on the problems I encountered in #175
Description of changes:

Added notes about limitations with Presign functions.
Clarified the text regarding the Lambda support.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Hi @mousedownmike,
Thanks for opening this PR. Would you mind rebasing this branch with master ? I am going to merge this PR to let customers know about this issue until we fix it.

@bhautikpip I think I was able to do that in web UI, let me know if this works.


App crashes after disconnecting watch, won't open again

Describe the bug
App crashes after disconnecting watch for the first time, crashes on open afterwards.
To Reproduce
Steps to reproduce the behavior:

Connect watch.
Change settings.
Press button on watch to disconnect
See app crash.

Expected behavior
App Shouldn't crash.
Watch:

G-Shock dw-b5600

Smartphone:

Device: Pixel 6a
OS: Android 14
Version: 10.5

Thanks for the report. I could not find a crash report on Google Play and I don't see this problem on my phone/watch.
It might me related to Android 14, I have not tried it on that. Unfortunately, emulation does not work with this app, so I could not try Android 14.
Do you mind helping narrow down the issue? If you have some time, try the following:

Stop the App. Go to setting, select the app and "clear storage". Restart the app and see if problem persists.
If you have, try on a different phone, preferable running older version of Android.
I am not sure if you are able to generate any stack-traces to see where the problem is. If not, one thing you can try is installing the app from the Google App Store (https://play.google.com/store/apps/details?id=org.avmedia.gshockGoogleSync&hl=en_CA&gl=US) and reproduce the problem. This should create a stack-trace which I can see in the play-store.
Thanks for your help.

In the codeblock below you will find a logcat of android studio, i ran the app via debug mode and used my own steps to reproduce on my pixel 6a with android 14, i don't have another device on hand to test with sadly.
Compat change id reported: 78294732; UID 10368; state: ENABLED
Compat change id reported: 247079863; UID 10368; state: ENABLED
=============== >>>> *** Waiting for connection... ***
FineLocationPermissionGranted
isLeEnabled(): ON
could not find callback wrapper
isLeEnabled(): ON
onScannerRegistered() - status=0 scannerId=4 mScannerId=0
Installing profile for org.avmedia.gshockGoogleSync
Compat change id reported: 265103382; UID 10368; state: ENABLED
isLeEnabled(): ON
connect() - device: XX:XX:XX:XX:98:07, auto: true
registerApp()
registerApp() - UUID=2bfb4c24-850a-4e4a-9b6a-ac93def5e393
onClientRegistered() - status=0 clientIf=6
onClientConnectionState() - status=0 clientIf=6 device=C0:A7:5A:57:98:07
... Do not interrupt...
discoverServices() - device: XX:XX:XX:XX:98:07
onConnectionUpdated() - Device=C0:A7:5A:57:98:07 interval=6 latency=0 timeout=500 status=0
onSearchComplete() = Device=C0:A7:5A:57:98:07 Status=0
Discovered 4 services for C0:A7:5A:57:98:07.
Service 00001801-0000-1000-8000-00805f9b34fb
Characteristics:
|--
Service 00001800-0000-1000-8000-00805f9b34fb
Characteristics:
|--00002a00-0000-1000-8000-00805f9b34fb: READABLE
|--00002a01-0000-1000-8000-00805f9b34fb: READABLE
Service 00001804-0000-1000-8000-00805f9b34fb
Characteristics:
|--00002a07-0000-1000-8000-00805f9b34fb: READABLE
Service 26eb000d-b012-49a8-b1f8-394fb2032b0f
Characteristics:
|--26eb002c-b012-49a8-b1f8-394fb2032b0f: WRITABLE WITHOUT RESPONSE
|--26eb002d-b012-49a8-b1f8-394fb2032b0f: WRITABLE, NOTIFIABLE
|------00002902-0000-1000-8000-00805f9b34fb: EMPTY
|--26eb0023-b012-49a8-b1f8-394fb2032b0f: WRITABLE, NOTIFIABLE
|------00002902-0000-1000-8000-00805f9b34fb: EMPTY
|--26eb0024-b012-49a8-b1f8-394fb2032b0f: WRITABLE WITHOUT RESPONSE, NOTIFIABLE
|------00002902-0000-1000-8000-00805f9b34fb: EMPTY
|--26eb002e-b012-49a8-b1f8-394fb2032b0f: WRITABLE WITHOUT RESPONSE
|--26eb002f-b012-49a8-b1f8-394fb2032b0f: WRITABLE, NOTIFIABLE
|------00002902-0000-1000-8000-00805f9b34fb: EMPTY
configureMTU() - device: XX:XX:XX:XX:98:07 mtu: 517
onConfigureMTU() - Device=C0:A7:5A:57:98:07 mtu=512 status=0
ATT MTU changed to 512, success: true
setCharacteristicNotification() - uuid: 26eb002d-b012-49a8-b1f8-394fb2032b0f enable: true
onConnectionUpdated() - Device=C0:A7:5A:57:98:07 interval=36 latency=0 timeout=500 status=0
Wrote to descriptor 00002902-0000-1000-8000-00805f9b34fb | value: 0x01 00
Notifications or indications ENABLED on 26eb002d-b012-49a8-b1f8-394fb2032b0f
2023-11-01T22:10:54.370283 Wrote to characteristic 26eb002c-b012-49a8-b1f8-394fb2032b0f | value: 0x10
---> Received data 0x10 07 98 57 5A A7 C0 7F 01 03 0F FF FF FF FF 24 00 00 00
PhotoAction: orientation: BACK
PhoneDialAction
2023-11-01T22:10:54.813193 Wrote to characteristic 26eb002c-b012-49a8-b1f8-394fb2032b0f | value: 0x22
---> Received data 0x22 62 E0 77 24 1A EE E0 59 98 B4 02
connectionLayout: hide
... Can be interrupted...
MainLayout: show
OnBackInvokedCallback is not enabled for the application.
Set 'android:enableOnBackInvokedCallback="true"' in the application manifest.
2023-11-01T22:10:55.127904 Wrote to characteristic 26eb002c-b012-49a8-b1f8-394fb2032b0f | value: 0x18
---> Received data 0x18 00 0A 00 00 00 00 00 00 00 00 00 00 00 00
>>> setting watch name to CASIO DW-B5600
2023-11-01T22:10:55.326959 Wrote to characteristic 26eb002c-b012-49a8-b1f8-394fb2032b0f | value: 0x1F 00
---> Received data 0x1F 00 41 4D 53 54 45 52 44 41 4D 00 00 00 00 00 00 00 00 00
2023-11-01T22:10:55.551417 Wrote to characteristic 26eb002c-b012-49a8-b1f8-394fb2032b0f | value: 0x28
---> Received data 0x28 13 19 00 00 00 02 00 00
battery level row value: 19
Skipped 47 frames!  The application may be doing too much work on its main thread.
Davey! duration=865ms; Flags=0, FrameTimelineVsyncId=32963464, IntendedVsync=172391193565651, Vsync=172391976812191, InputEventId=0, HandleInputStart=172391982657986, AnimationStart=172391982662258, PerformTraversalsStart=172391983453111, DrawStart=172392040751125, FrameDeadline=172391276828121, FrameInterval=172391982247301, FrameStartTime=16665672, SyncQueued=172392053028591, SyncStart=172392053300360, IssueDrawCommandsStart=172392054897325, SwapBuffers=172392058841010, FrameCompleted=172392059722683, DequeueBufferDuration=12614, QueueBufferDuration=163697, GpuCompleted=172392059722683, SwapBuffersCompleted=172392059057644, DisplayPresentTime=172381859787050, CommandSubmissionCompleted=172392058841010,
onClientConnectionState() - status=19 clientIf=6 device=C0:A7:5A:57:98:07
onConnectionStateChange: status 19 encountered for C0:A7:5A:57:98:07!
Got error 19
Disconnecting from C0:A7:5A:57:98:07
close()
unregisterApp() - mClientIf=6
connectionLayout: show
... Can be interrupted...
MainLayout: hide
onDisconnect
Not connected to C0:A7:5A:57:98:07, cannot teardown connection!
=============== >>>> *** Waiting for connection... ***
isLeEnabled(): ON
could not find callback wrapper
FATAL EXCEPTION: DefaultDispatcher-worker-1
Process: org.avmedia.gshockGoogleSync, PID: 20671
java.lang.IllegalArgumentException: XX:XX:XX:XX:98:07 is not a valid Bluetooth address
	at android.bluetooth.BluetoothDevice.<init>(BluetoothDevice.java:1370)
	at android.bluetooth.BluetoothDevice.<init>(BluetoothDevice.java:1394)
	at android.bluetooth.BluetoothAdapter.getRemoteDevice(BluetoothAdapter.java:1087)
	at org.avmedia.gshockapi.ble.BleScannerLocal.startConnection(BleScannerLocal.kt:51)
	at org.avmedia.gshockapi.io.WaitForConnectionIO.waitForConnection(WaitForConnectionIO.kt:39)
	at org.avmedia.gshockapi.io.WaitForConnectionIO.request(WaitForConnectionIO.kt:21)
	at org.avmedia.gshockapi.GShockAPI.waitForConnection(GShockAPI.kt:75)
	at org.avmedia.gShockPhoneSync.MainActivity.waitForConnectionCached(MainActivity.kt:295)
	at org.avmedia.gShockPhoneSync.MainActivity.access$waitForConnectionCached(MainActivity.kt:48)
	at org.avmedia.gShockPhoneSync.MainActivity$run$1.invokeSuspend(MainActivity.kt:96)
	at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
	at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:106)
	at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.kt:570)
	at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask(CoroutineScheduler.kt:750)
	at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker(CoroutineScheduler.kt:677)
	at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:664)
	Suppressed: kotlinx.coroutines.DiagnosticCoroutineContextException: [StandaloneCoroutine{Cancelling}@4228752, Dispatchers.Default]
Sending signal. PID: 20671 SIG: 9

@jrvgr
Well, I'm not 100% sure why the app thinks "C0:A7:5A:57:98:07" is an invalid Bluetooth address. Maybe something in the latest SDK 34, for android 4.
In any case, I am now handling this error, and the app should no longer crash. If it cannot re-use the existing address from previous connections, it will scan and connect to the watch.
If you are building from source, the latest code is checked in. You can also get the app from the play-store (https://play.google.com/store/apps/details?id=org.avmedia.gshockGoogleSync&hl=en_CA&gl=US). Make sure it is version 10.6. BTW, you can check the version by pressing the (i) in the top-right of the initial screen, and then scrolling to the bottom.
Thanks for your help.

Issue seems to be fixed, thanks for the quick support!


Some changes work in DEBUG but not RELEASE

Hello
I added one feature.
When i test it in Debug mode it works correctly.
But when i use it in Release mode it doesnt work.
I also added this feature in assembly to be True or False.
What is problem ?

I can only guess. Show me the code.

Same for File-Explorer feature. When I use the Release Version of Quasar Server I don't get Directory Listing in the ListView of the File-Explorer Window. When i click the Refresh button the Client Disconnects.
If I use the Debug Version of Quasar Server the File-Explorer works without any problems!

I noticed this as well, happens when running release mode configuration in IDE.  Some features will not work correctly, others will.

The release configuration is not meant to be used in the IDE, debugging won't work too. Only use the release configuration when you also want to build your client via the in-built builder, otherwise use the debug configuration.

If live-debugging isn't working in Visual Studio, how are you debugging your client ?

Use debug configuration.

@0xE232FE @d3agle What exactly do you mean? I've built the client & server both in release and debug mode and everything works flawlessly, even tested to connect the debug client to the release server. The File Manager worked in all cases. Can you tell me the steps to reproduce the issue?

I have encountered issues while playing with the code.
If client/server packets are differents, the Serializers (client/Server side) will be differents.
=> The TypeMapping will be different too, so if you send an GetAuthentication() packet from the client, the server will understand it as a DoClientUninstall().
Use step-by-step debugging & check if Serializer() (both client/server side) have the same number of registered/mapped packets.
It was the solution for me.

@Perturabo Yes, that's intended as NetSerializer provides no versioning of the types but is therefore faster and has a smaller footprint. Every change to the packets will break the communication with the old server.

Imo the system is fine.
Back to the original problem, you should first check differences between DEBUG & RELEASE mode.
Try finding all occurences in Visual Studio of 'DEBUG'.
Code like "#if DEBUG [...]" might change the behaviour of your app in debug/release mode.


[Vectorized][New Feature] Support min_by/max_by function.

Proposed changes
Issue Number: open #7678
Problem Summary:
Support min_by/max_by on vectorized engine.
Checklist(Required)

Does it affect the original behavior: (No)
Has unit tests been added: (Yes)
Has document been added or modified: (Yes)
Does it need to update dependencies: (No)
Are there any changes that cannot be rolled back: (No)

Further comments

This is an initial patch, I will add the document later.
For now, aggregate functions only supports 1 intermediate type, but min_by/max_by needs 2 type information, I've did a tricky work-around int this initial patch, but we need to discuss that do we really need to change the intermediate type to an intermediate type list? @morningman @HappenLee  @dataroaring. Another solution is to serialize the type information in the intermediate data.
To support these 2 functions in the non-vectorized code needs to pay much more efforts, we may need to do some refactor. Do we need to support this?

we can add docs for these new functions @zhannngchen

we can add docs for these new functions @zhannngchen

Yes, I'll update the patch later today


Llama 3.2

Bring up Llama 3.2 model family on Wormhole, T3K and TG

[ ] Add MeshDevice fixture to all unit tests
[ ] Debug correctness in unit tests

Llama3.2-11B-Vision bringup

[ ] TtModelConfig creates state_dict_prefix
Text model
[ ] MLP
[ ] ATTN
[ ] Decoder
[ ] Model

Vision model

[ ] LayerNorm
[ ] Conv2dPatch
[ ] ImageFeedForward
[ ] ImageAttention
[ ] ImageTransformerBlock
[ ] ImageTransformer
[ ] VisionEncoder
[ ] TilePositionEmbedding
[ ] CrossAttention
[ ] CrossAttentionTransformerBlock
[ ] CrossAttentionTransformerVision
[ ] CrossAttentionTransformerText
[ ] CrossAttentionTransformer


add a "released" timestamp to all packages in the repo

so we can identify "stuck" packages (where autorebase didn't work) more easily

not necessary anymore, as our package update workflow seems to be quite stable.


Pester Task Faling

Running on Linux Azure Agent
Using Latest Pester Test Runner: 10.2.4 (Latest)
When i run the task with a HASH TABLE including Parameters it fails

task: Pester@10
inputs:
scriptFolder: '@{Path=’$(System.DefaultWorkingDirectory)\server.template.tests.ps1′; Parameters=@{ResourceGroupName=''$(azure_resource_group_name)''}}'
resultsFile: '$(System.DefaultWorkingDirectory)\server.template.tests.XML'
usePSCore: true

Failure:
##[error]Get-ChildItem: /usr/local/share/powershell/Modules/Pester/5.0.4/Pester.psm1:3165
Line |
3165 |              Get-ChildItem -Recurse -Path $p -Filter "*$Extension" -Fi …
|              ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| Cannot find path
| '/home/vsts/work/1/s/@{Path=’/home/vsts/work/1/s/' because it
| does not exist.
##[debug]Processed: ##vso[task.issue type=error;]Get-ChildItem: /usr/local/share/powershell/Modules/Pester/5.0.4/Pester.psm1:3165%0ALine |%0A3165 |              Get-ChildItem -Recurse -Path $p -Filter "$Extension" -Fi …%0A     |              ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~%0A     | Cannot find path%0A     | '/home/vsts/work/1/s/@{Path=’/home/vsts/work/1/s/' because it%0A     | does not exist.%0A%0A
##[debug]task result: Failed
##[error]Get-ChildItem: /usr/local/share/powershell/Modules/Pester/5.0.4/Pester.psm1:3165
Line |
3165 |              Get-ChildItem -Recurse -Path $p -Filter "$Extension" -Fi …
|              ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| Cannot find path
| '/home/vsts/work/1/s/@{Path=’/home/vsts/work/1/s/' because it
| does not exist.
When I run the Task without parameters and modify the script to not require parameters it works:

task: Pester@10
inputs:

scriptFolder: '$(System.DefaultWorkingDirectory)/server.template.tests.ps1'
resultsFile: "$(System.DefaultWorkingDirectory)/server.template.tests.XML"
usePSCore: true
run32Bit: False
Running in  PowerShell
Starting discovery in 1 files.
Discovery finished in 665ms.
[+] /home/vsts/work/1/s/server.template.tests.ps1
2.36s (1.38s|344ms)
Tests completed in 2.39s
Tests Passed: 1,
Failed: 0,
Skipped: 0
NotRun: 0
Pester Script finished
Finishing: Pester

That'll be because Pester 5.0 doesn't support the hashtable syntax for passing parameters to tests (or parameters on tests at all). There's work to add that for 5.1 but it's not there yet.
If you need it and aren't using any other 5.0 features then you can set the task version to 9 and it'll use Pester 4.x where that syntax is accepted

Hi Chris,
thanks alot for this.
I have made the change to use Pester 4 and it is now working:

task: Pester@9
inputs:
scriptFolder:
"@{Path='$(System.DefaultWorkingDirectory)/server.template.tests.ps1';
Parameters=@{ResourceGroupName='$(azure_resource_group_name)'}}"
resultsFile:
"$(System.DefaultWorkingDirectory)/server.template.tests.XML"
usePSCore: true
run32Bit: False
PesterVersion: 'OtherVersion'
preferredPesterVersion: '4.10.1'

How do we pass parameters to pester 5 tests?
Regards
Peter
On Wed, Oct 14, 2020 at 4:24 PM Chris Gardner<EMAIL_ADDRESS>wrote:

That'll be because Pester 5.0 doesn't support the hashtable syntax for
passing parameters to tests (or parameters on tests at all). There's work
to add that for 5.1 but it's not there yet.
If you need it and aren't using any other 5.0 features then you can set
the task version to 9 and it'll use Pester 4.x where that syntax is accepted
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pester/AzureDevOpsExtension/issues/50#issuecomment-708476922,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKVAWERTEGYTLH7OQXT2JTTSKW7ELANCNFSM4SQR5QXA
.

Currently we can't. 5.1 should bring that functionality back and beta versions of that are available. I'll need to look into it closer to when it is released

Thanks for your help
On Thu, Oct 15, 2020 at 1:25 PM Chris Gardner<EMAIL_ADDRESS>wrote:

Currently we can't. 5.1 should bring that functionality back and beta
versions of that are available. I'll need to look into it closer to when it
is released
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pester/AzureDevOpsExtension/issues/50#issuecomment-709279338,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKVAWESAZL5A3GCVDF6YZNLSK3S3XANCNFSM4SQR5QXA
.


Fix missing updates on terminal resize

I found a case where the positions weren't updating, specifically when splits were rebalanced by an autocmd on VimResized. This change seems to cover that gap.
I tried replacing WinScrolled but it seems to pick up changes to manually resized vertical splits that VimResized misses.

Thank you!


GDScript: class_name

Show how to register classes without and with an icon
Creating a node that extends a registered class (only extends the gdscript file)
Using registered classes for type checks in code

Video done.


DNN Upgrade Cannot Process on Azure WebApps on S1 Instance

Description of bug
I see the following error when upgrading from 9.2.0 to 9.2.2.
50% ERROR occured - Installing Library - Dnn.EditBar.UI_01.06.01_Install.zip
My DNN site is hosted on Azure.  Though when I duplicate my site on my local machine and attempt to upgrade it, it succeeds in upgrading just fine.  It is my live site on Azure that seems to always fail when it begins to process the module Dnn.EditBar.UI_01.06.01_Install.zip when upgrading.
Affected version

[x] 9.2.2
[x] 9.2

Are you using Azure App Service?  Or an Azure VM?
What size of Azure environment are you in?

I am using Azure App Service of size "Standard: 1 Small".
Do let me know if you need any other information.

There was some improvements recently in 9.3.something and more improvements in 9.4.0 which I am 99% sure solves this issue, if you can still reproduce after testing 9.4.0, please open a new issue. Warning 9.4.0 is at this time in RC state, so be careful not to use it in production yet, you can copy the site and try it or wait for the official release and report back.


return e2e test exit code and change log file name on failure

With this change, we records e2e test exit code. It has the following effects:

It is used to signal result in Jenkins and send emails with results. We should be able to get email like

Subject: Jenkins Job Succeeded (Failed)
Text: (link to storage place)

We are able to rename the log file to "kubemark-log-fail.txt" if e2e test failed.

Still testing in jenkins. Will update the status here.

We had a successful run at https://jenkins.coreos.systems/job/kubernetes-scale-100nodes/12/
Merging this as a checkpoint.
Next, I'm gonna clean up the scripts, make it easier to use, and expose comparable results.
@xiang90


Add patch release notes and docs fixes

Summary
Closes #1401. Our release notes currently doesn't include notes of the patch releases before the most recent minor release. This PR updates release_notes.rst to build previous patch releases (0.3.1 isn't rendering but I think it's too old to be useful now anyways). The new process prior to release is that release notes from the most recent minor version's patch releases should be deleted from the main branch prior to a new minor release, and release_notes.rst needs to have a new directive added for the previous stable branch.
Details and comments
A few other minor changes:

Wraps the output text of jupyter-sphinx blocks
Update the contributor guide to include release note subcategories
Suppress the debugger warning during the docs build by setting PYDEVD_DISABLE_FILE_VALIDATION=1
Add a link to the main branch to the dev docs banner

I just noticed the notes in the github build only goes back to the 0.4 prelude, which I didn't catch because my local build went back to 0.2: https://qiskit-extensions.github.io/qiskit-experiments/dev/release_notes.html. In the logs, reno only saw 0.5.x and 0.4 in stable/0.5, maybe because it doesn't know about the older tags? I suppose we may as well put in a PR to backfill the old notes manually since we want to do that anyways.


Where is the peek result for StreamUtils.debounce?

Where is the peek result when i test:
StreamUtils.debounce(aStream,10,TimeUnit.SECONDS)
.peek(x ->  System.out.println("utilPeek2:"+x) )
.collect(Collectors.toList());
after run, i can not see the printing "utilPeek2:xxx"
The detail code is:
List rs = StreamUtils.debounce(
StreamUtils.schedule(
Stream.of(1,2,3,4,5).peek(x->System.out.println("utilPeek1:"+x))
, "* * * * * ?", ThreadPools.getStandardRetry()
).connect(), 10, TimeUnit.SECONDS
).peek(x -> System.out.println("utilPeek2:"+x)).collect(Collectors.toList());
System.out.println("utilResultList:" + rs);
and when i use ReactiveSeq.debounce().peek(), its is OK(i can see result of "peek2:a"):
HotStream hotStream = ReactiveSeq.of("a", "b", "c", "d", "e")
.peek(x -> System.out.println("peek1:" + x))
.schedule("* * * * * ?", ThreadPools.getStandardRetry());
System.out.println("resultList:" + hotStream.connect().debounce(10, TimeUnit.SECONDS).peek(x->System.out.println("peek2:" + x)).toListX() )
so i think there may be something bug with the StreamUtil.debounce().peek()

The version is "1.0.0-RC2.2", i use gradle to run cyclops-react

Sorry, it is my fault! The result is OK, there is something wrong with my eyes, Sorry, Sorry!

Ah cool, just looking at it now and they seemed similar - will add these as tests to protect future releases!
Thanks for this, I'll update the issue now.


Tool to merge to hex files into one large hex file

What's the purpose of this tool?

We used it to merge the hex of the bootloader and the otas

We were able to include the .bin file with the gcc tools, so I don't think we need this.


[$2000] Email tooltip of the user avatar in the chat report page is incorrectly positioned.

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

Action Performed:

Open any chat report that has more than a page worth of messages.
Scroll until the user avatar of any of the report item is partially behind the report header.
Hover over the user avatar of report item at that instance.

Expected Result:
The tooltip should open at the bottom direction such that header is not blocked.
Actual Result:
Tooltip still opens up at the top. The distance between tooltip and the hover position looks a lot as well.
Workaround:
unknown
Platforms:

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

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

Version Number: 1.2.77-3
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/222031412-15a2b77e-8968-480c-9297-ee3edff97563.mov

Expensify/Expensify Issue URL:
Issue reported by: @allroundexperts
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1677623211493269
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01ee40ab52e0ee2090
Upwork Job ID:<PHONE_NUMBER>944919552
Last Price Increase: 2023-03-13

Triggered auto assignment to @miljakljajic (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

Alright, @shawnborton agrees this is an improvement worth making and it's nice and straightforward for an external issue. Adding the relevant labels

Job added to Upwork: https://www.upwork.com/jobs/~01ee40ab52e0ee2090

Current assignee @miljakljajic is eligible for the External assigner, not assigning anyone new.

Triggered auto assignment to Contributor-plus team member for initial proposal review - @aimane-chnaif (External)

Triggered auto assignment to @yuwenmemon (External), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

Proposal
Please re-state the problem that we are trying to solve in this issue.
The tooltip in a reportComment is covering the header
What is the root cause of that problem?
It is because we have a fixed GUTTER_WIDTH, which checks if the tooltip should render above or below set to 16px
https://github.com/Expensify/App/blob/b6ebac3587d4be39ab74abda9740cb1fe34d40bf/src/styles/getTooltipStyles.js#L91
What changes do you think we should make in order to solve the problem?
Since we have a header at top, we want to add the height of that header with the GUTTER_HEIGHT while checking if the tooltip should be above or below.
But we cannot just increase the GUTTER_HEIGHT since that will effect other sections like Details which have no header.
So we need a way to increase the GUTTER_HEIGHT dynamically acoording to the requirement.
Therefore, we can add a parameter to the getTootltipStyles called topPadding that gets added when the checking happens in getTooltipStyles.js
 const shouldShowBelow = (yOffset - tooltipHeight) < GUTTER_WIDTH + topPadding;

And for the tooltips in the reportAction we can just pass the height of the header as prop in Tooltip
After implementation (topPadding set to 32)
https://user-images.githubusercontent.com/77237602/222500186-e4b16eea-39d5-4d43-9e5a-aa2ad09bca85.mov
What alternative solutions did you explore? (Optional)
Option 2

I also tried using the intersectionObserver to check if a tooltip child was actually visible and only render the toolTip if it was.
But it would add a lot of complexity and we would have to instantiate intersectionObserver for every tooltip which has very bad performance implications.
https://www.bennadel.com/blog/3954-intersectionobserver-api-performance-many-vs-shared-in-angular-11-0-5.htm
Option 3

If instead of displaying the tooltip at the bottom, we want to have the tooltip hidden under the header, (which means that the tooltip is still at the top, but hidden under the header).
To achieve that, we can do the following:
Currently, we are rendering tooltips with a portal directly to the body of the html page. So, we would need to create a portal for the header as well and give it a higher z-index

Wow @esh-g love the speedy proposal! @aimane-chnaif, what do you think of the suggestion to include the header as part of the GUTTER_HEIGHT (adjusted dynamically)?

Proposal
Please re-state the problem that we are trying to solve in this issue.
The tooltip of avatar component is overlaying over the chat header when the avatar is partially hidden behind the header.
What is the root cause of that problem?
We're not making the tooltip change the position based on the relative position of the tooltip child compared to the FlatList boundary.
What changes do you think we should make in order to solve the problem?

Here's a generic solution that will work for all ScrollView/Flatlist and Tooltip inside that ScrollView/Flatlist.

Let's look at the above screenshot, we'll call the red box the Boundary of the ScrollView, meaning all of the scrollable content inside the ScrollView will stay inside that boundary.
Meanwhile the blue box is the Tooltip target, the Tooltip target could go outside of the Boundary and be partially or wholly hidden behind the Boundary.
We need to do the following:

We get the position of the Boundary, for example by using getBoundingClientRect inside https://github.com/Expensify/App/blob/adb0d8a52fc7210e8e48e4439f83d932a064e1f6/src/components/InvertedFlatList/index.js#L64. Now we know the relative location of the Boundary compared to the ViewPort.

We store the above information in a context so that all children of the ScrollView will have access to it

When we render the tooltip, we also get the position of the tooltip target relative to the ViewPort (for example by using getBoundingClientRect)

If the the top position of the Tooltip target is less than the top position of the Boundary, we know that the Tooltip target is partially hidden behind the boundary top. In this specific case that means the Avatar is hidden behind the Header.

In this case we'll render the tooltip below the Tooltip target, by setting the shouldShowBelow here to true https://github.com/Expensify/App/blob/b6ebac3587d4be39ab74abda9740cb1fe34d40bf/src/styles/getTooltipStyles.js#L91

We can apply the above logic to all ScrollView & Tooltip and it will work the same way, if needed we can check if the Tooltip target is hidden behind the Boundary top/bottom/right/left to set the tooltip position properly as well. The above is example of the hidden behind boundary top case.

Also as a note we'll need to capture the bounding client rect correctly if the position of the element changes as well so it will work for dynamic elements, not just getting the first time when it renders.
What alternative solutions did you explore? (Optional)
NA

@aimane-chnaif I use onLayout event handler in the FlatList. This capture the layout of component when it first renders and on layout changes.

@aimane-chnaif Check this out: https://github.com/tienifr/App/tree/fix/15586

We can apply the above logic to all ScrollView & Tooltip and it will work the same way

@aimane-chnaif That uses a different list component SectionList rather than InvertedList. I think I also need to apply my logic to this component too. As a draft branch, I just test it with InvertedList to prove it works.

@yuwenmemon @miljakljajic @aimane-chnaif this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

@aimane-chnaif I tried implement with SectionList and it works just fine. As this is a Context, when the SectionList call setter to set its bounding rect, all of its children also gets updated. Here Tooltip gets the boundary rect value from this context to set yPadding so yPadding gets updated as well.

Upwork job price has been updated to $2000

Consider this case when the target (i.e. the avatar) is overlapped but the coordinate (xOffset, yOffset) (marked as star) is not covered by the Header. My solution is more generic and can handle this since it check whether the target is outside of the Wrapper. I know this is an edge (rare) case but it's worth considering.

📣 @hoangzinh You have been assigned to this job by @yuwenmemon!
Please apply to this job in Upwork and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑💻
Keep in mind: Code of Conduct | Contributing 📖

Reviewing label has been removed, please complete the "BugZero Checklist".

The solution for this issue has been :rocket: deployed to production :rocket: in version 1.2.89-0 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

https://github.com/Expensify/App/pull/16370

If no regressions arise, payment will be issued on 2023-04-03. :confetti_ball:
After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

[ ] External issue reporter
[ ] Contributor that fixed the issue
[ ] Contributor+ that helped on the issue and/or PR

As a reminder, here are the bonuses/penalties that should be applied for any External issue:

Merged PR within 3 business days of assignment - 50% bonus
Merged PR more than 9 business days after assignment - 50% penalty

I agree, @aimane-chnaif !

Ah forgive me - the title of the GH threw me off! Let's hold on and see. Should be fine!

No regressions, right? I will get the final payment done today if so.

Paid, contracts ended. No regression testing required, so closing out.


KVM nic-attach errror

Description
If the nic-attach fails due to libvirt bug (VM can not eject cdrom) the nic appears in the VM (although without proper configuration) but it's not visible on OpenNebla VM
To Reproduce
Setup OpenNebula on Centos8
run VM
onevm stop
onevm resume
onevm nic-attach
Expected behavior
Idally the nic-attaed is successful, but until the libvirt bug is fixed avoid inconsistent state where the nic is attached in the VM but in OpenNebula it's not.
Details

Affected Component: kvm drivers
Hypervisor: KVM
Version: 5.13

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

PRs to merge in 6.4 & master:

code
docs


Add dependency locks for Bitbucket and Medium

Purpose
Adding dependencies locking for the connectors: bitbucket, medium
Automation tests

Unit tests

Done

Integration tests

Done

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes
Ran FindSecurityBugs plugin and verified report? no
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

Test environment

JDK 11, Ballerina Swan Lake Alpha5

The Dependencies.toml should have only the following two entries
[[dependency]]
org = "ballerina"
name = "http"
version = "1.1.0-alpha8"

[[dependency]]
org = "ballerina"
name = "url"
version = "1.1.0-alpha8"


feat: Add forward group for https listener

Description
Add ability to define forward groups for HTTPS listeners.
Motivation and Context
Blue/Green ECS deployments with AWS Codepipelines. Similar to #269
Breaking Changes
There is no breaking changes but I added a check in default_action block for forward/empty action to make sure that only target_group_arn or forward group is defined for that block but not both
How Has This Been Tested?

[x] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects

[x] I have executed pre-commit run -a on my pull request

This issue has been resolved in version 9.0.0 :tada:


Request: increase verbosity of pcluster cli when Importing CDK times out

I'm trying to create a cluster using the pcluster cli from my local machine
When I have errors in the configuration it responds with the correct errors and FSids that are in my account, but when the configuration of my cluster is correct nothing happens;
❯ pcluster create-cluster -n frankfurt-test-2 -c pcluster-example-config.yaml --dryrun true
{
  "message": "Request would have succeeded, but DryRun flag is set."
}

However, when turning off the Dry-run flag:
❯ pcluster create-cluster -n frankfurt-test-2 -c pcluster-reduced.yaml --debug

It just keeps hanging on this command. Adding the --debug flag does not change anything.
Note that I am using granted to gain access to my aws account and that I'm able to deploy resources using terraform and I can u se the aws cli to interact with my resources.
I checked the logs of my pcluster ~/.parallelcluster/pcluster-cli.log and saw that it was stuck on the following line:
cdk_builder.py:34:build_cluster_template() - importing CDK.....
I used nvm to switch to a newer version of cdk, and now it runs, but it took me quite a long time to find this specific error.
ParallelCluster version:
3.11.0

Cluster name:
frankfurt-test-2
Output of pcluster describe-cluster command.
N/A - cluster would not create
[Optional] Arn of the cluster CloudFormation main stack:
N/A - cluster would not create

Bug description and how to reproduce:
I don't get a but, just no feedback at all.
For issues with Slurm scheduler, please attach the following logs:

From Head node: /var/log/parallelcluster/clustermgtd, /var/log/parallelcluster/clusterstatusmgtd (if version >= 3.2.0), /var/log/parallelcluster/slurm_resume.log, /var/log/parallelcluster/slurm_suspend.log, /var/log/parallelcluster/slurm_fleet_status_manager.log (if version >= 3.2.0) and/var/log/slurmctld.log.
From Compute node:  /var/log/parallelcluster/computemgtd.log and /var/log/slurmd.log.

Hello Maurits,
Sorry for the late reply. To help us reproduce the issue, can you provide the version of CDK which were giving you troubles?
Thank you!

I just started setting up ParallelCluster for the first time this week, and this is by far the biggest headache and time waster I stumbled upon.
Steps to reproduce on Ubuntu 22.04.5, using a manually compiled Python 3.12:
# this installs ancient Node.JS version 12, which triggers this issue
sudo apt-get install nodejs
# setup virtual environment, upgrade pip and install latest aws-parallelcluster
python3.12 -m venv ~/.local/opt/pcluster
~/.local/opt/pcluster/bin/pip install -U pip
~/.local/opt/pcluster/bin/pip install -U aws-parallelcluster
# install symlink to pcluster
ln -sft ~/.local/bin/ ~/.local/opt/pcluster/bin/pcluster
# create-cluster silently hangs, with "Importing CDK..." in ~/.parallelcluster/pcluster-cli.log
pcluster create-cluster --cluster-name test --cluster-configuration cluster-config.yaml --debug

Output of pcluster/bin/pip freeze:
attrs==23.2.0
aws-cdk.assets==1.204.0
aws-cdk.aws-acmpca==1.204.0
aws-cdk.aws-apigateway==1.204.0
aws-cdk.aws-applicationautoscaling==1.204.0
aws-cdk.aws-autoscaling==1.204.0
aws-cdk.aws-autoscaling-common==1.204.0
aws-cdk.aws-autoscaling-hooktargets==1.204.0
aws-cdk.aws-batch==1.204.0
aws-cdk.aws-certificatemanager==1.204.0
aws-cdk.aws-cloudformation==1.204.0
aws-cdk.aws-cloudfront==1.204.0
aws-cdk.aws-cloudwatch==1.204.0
aws-cdk.aws-codebuild==1.204.0
aws-cdk.aws-codecommit==1.204.0
aws-cdk.aws-codeguruprofiler==1.204.0
aws-cdk.aws-codestarnotifications==1.204.0
aws-cdk.aws-cognito==1.204.0
aws-cdk.aws-dynamodb==1.204.0
aws-cdk.aws-ec2==1.204.0
aws-cdk.aws-ecr==1.204.0
aws-cdk.aws-ecr-assets==1.204.0
aws-cdk.aws-ecs==1.204.0
aws-cdk.aws-efs==1.204.0
aws-cdk.aws-elasticloadbalancing==1.204.0
aws-cdk.aws-elasticloadbalancingv2==1.204.0
aws-cdk.aws-events==1.204.0
aws-cdk.aws-fsx==1.204.0
aws-cdk.aws-globalaccelerator==1.204.0
aws-cdk.aws-iam==1.204.0
aws-cdk.aws-imagebuilder==1.204.0
aws-cdk.aws-kinesis==1.204.0
aws-cdk.aws-kms==1.204.0
aws-cdk.aws-lambda==1.204.0
aws-cdk.aws-logs==1.204.0
aws-cdk.aws-route53==1.204.0
aws-cdk.aws-route53-targets==1.204.0
aws-cdk.aws-s3==1.204.0
aws-cdk.aws-s3-assets==1.204.0
aws-cdk.aws-sam==1.204.0
aws-cdk.aws-secretsmanager==1.204.0
aws-cdk.aws-servicediscovery==1.204.0
aws-cdk.aws-signer==1.204.0
aws-cdk.aws-sns==1.204.0
aws-cdk.aws-sns-subscriptions==1.204.0
aws-cdk.aws-sqs==1.204.0
aws-cdk.aws-ssm==1.204.0
aws-cdk.aws-stepfunctions==1.204.0
aws-cdk.cloud-assembly-schema==1.204.0
aws-cdk.core==1.204.0
aws-cdk.custom-resources==1.204.0
aws-cdk.cx-api==1.204.0
aws-cdk.region-info==1.204.0
aws-parallelcluster==3.11.0
boto3==1.35.42
botocore==1.35.42
cattrs==23.1.2
certifi==2024.8.30
charset-normalizer==3.4.0
click==8.1.7
clickclick==20.10.2
connexion==2.13.1
constructs==3.4.344
Flask==2.2.5
idna==3.10
importlib_resources==6.4.5
inflection==0.5.1
itsdangerous==2.2.0
Jinja2==3.1.4
jmespath==0.10.0
jsii==1.85.0
jsonschema==4.23.0
jsonschema-specifications==2024.10.1
MarkupSafe==3.0.1
marshmallow==3.22.0
packaging==24.1
publication==0.0.3
python-dateutil==2.9.0.post0
PyYAML==6.0.2
referencing==0.35.1
requests==2.32.3
rpds-py==0.20.0
s3transfer==0.10.3
setuptools==69.5.1
six==1.16.0
tabulate==0.8.10
typeguard==2.13.3
typing_extensions==4.12.2
urllib3==2.2.3
Werkzeug==2.3.8

I can reproduce the issue. Per my understanding, the issue is because of the outdated NodeJS, right? I am working on improving ParallelCluster code
Thank you,
Hanwen


Client performance when moving notebook panels

Moving panels around in my browser appears to be more performance intensive than I would have expected. I can easily push chrome to 100% usage by dragging borders between panels around. Is this expected?
I'm moving a notebook and a terminal to reproduce the issue.
Migrating here from https://github.com/phosphorjs/phosphor/issues/148

Note: there are potentially two issues here:

Drag-Drop panels which cause a reconfiguration of the dock panel.
Resizing individual panels by dragging a splitter handle.

There are reports that some browsers are slow on (1). I am working on a new dock panel internal layout mechanism which will fix this and also provide better support for iframes.
Number (2) is really what we should pay attention to here. There is a known optimization we can make by limiting the number of code mirror widgets in a notebook. If a cell does not have input focus, it should be a static rendering rather than a live code mirror cell. This will cut down on the number of reflows per resize event.

Number (2) is really what we should pay attention to here. There is a known optimization we can make by limiting the number of code mirror widgets in a notebook. If a cell does not have input focus, it should be a static rendering rather than a live code mirror cell. This will cut down on the number of reflows per resize event.

Agreed, and tracked at #25.

Yeah, I saw some performance problems today opening some medium sized notebooks...


feat: create releases with ollama-webui version numbers

Is your feature request related to a problem? Please describe.
I would like to package Ollama-WebUI for the NixOS Linux distribution , but missing versions would make it difficult for users to distinguish newer versions from older ones (currently, we only have Git hashes).
Describe the solution you'd like
Create releases or pseudo-releases with version numbers (as frequent as feasible and with whatever breaking changes, changelog or version numbers as you please).
Additional context
NixOS or rather the Nix package manager (available for Mac and other Linux distributions) makes it very easy to get and run the app with a single, short terminal call. It will likely be as follows:
nix run nixpkgs#ollama-webui

Link to NixOS PR: https://github.com/NixOS/nixpkgs/pull/275448
Btw, thanks for this awesome UI (to me it's the best i found so far for Ollama).

Hi, Thanks for creating this feature request! I'm actively looking into this and I was wondering if you're aware of any ways to somehow automate this? I suppose I could manually create new releases with any changes but that seems suboptimal, Thanks!

We could use the version in package.json and bump that every time a PR is merged to master using GHA. If there was only a change to the backend, in that case too we modify the version as the whole app is packaged together. Wdyt?

@anuraagdjain Thank you for the suggestion, but the version "tracking" is being done from the backend now: https://github.com/ollama-webui/ollama-webui/blob/13881beeddb81824fc75f4272dc0088efec05216/backend/config.py#L34
I've been manually updating the version number from the line above, so I'm not so sure if using the version in package.json would be the best choice. Any other suggestions? Thanks as always!

To avoid manually bumping the version, I recommend something as done in this article https://medium.com/@johnathanmiller/github-actions-npm-version-bump-using-labels-on-pr-merge-8c716a488a64 .

Automatically publishes a new version when PRs are merged.
using the package.json's version you can add tags to git commit or generate changelogs ( https://blog.logrocket.com/automatically-generate-and-release-a-changelog-with-node-js/)
You can read the package.json file and get the version property and return the value via the API. No need to maintain the version when npm can handle it.

Since we've package.json, for above reasons I think its best to use it.

I think we should have a feature in which version update notification should be displayed if not running latest version of ollama-webui in settings and update button for downloading latest update should be displayed next to it.
here we can also have a drop down menu to jump to particular version or nightly version that uses latest github code for latest update by pulling latest github repo code .

@rcgtnick this can be done in one particular way.
1 pull the latest docker image.
2 delete container id
3 restart the container id of the latest running image.
Also this upgrade button can be shown to only those who are running it using docker by detecting how the software is run or how it is installed.

@hemangjoshi37a an app I am running in a container should not have that level of privilege in a production system.
That kind of architecture decision would make this app unsuitable for use outside of a development environment.

Agree with many here, I don't want an upgrade method within the app, Docker and Kubernetes already facilitate version control once we start getting versioned images. I can understand things are in too much flux right now for that, but I imagine once v1.0 is ready it would be appropriate to start tagging the Docker images with version numbers. For now I'm fine pulling the latest image periodically and redeploying my container, this is the Docker way.

How can we make a UI that is user-friendly to a dumb person proof? that is by providing dev level control at ease and with great UI/UX , for incorporating this we have to make this button possible. I have noticed in various opensource projects is that they lack UI/UX  most of them to say the least. If we can improve upon this then we can beat proprietary App world.

Another way to look at it is whether or not updates to the web app are the concern of the web app itself. I think notifying of updates in the UI is fine, but actually trying to perform the update from within the web UI is bound not to work for all users.

I agree with this sentiment. By showing a small UI and a link to the article showing how to upgrade would be much beneficial than focusing on having the auto update mechanism. Again, it depends on the project maintainer to decide the roadmap so we could help them.

Auto update might be a good thing to bundle if this issue https://github.com/ollama-webui/ollama-webui/issues/470 is going to have more focus.

Indeed, releases are much needed for pulling the code using version numbers in various workflows.

Indeed, releases are much needed for pulling the code using version numbers in various workflows.
Using automated releases you could also take advantage of the then newly created release to push the docker image with the same tag version.

All in the plan, Lan. I contributed a modified docker build action that will accomplish exactly this once we start tagging releases.

@justinh-rahb
As properly versionned releases are not a thing right now, why not use the git commit hash in an automatic release workflow in the meantime?

@justinh-rahb
As properly versioned releases are not a thing right now, why not use the git commit hash in an automatic release workflow in the meantime?

@Lanhild I had something like that in mind, ya. What do you think @tjbck do you think we can find a way to replace the build part of the version string with the hash during the docker build?

@justinh-rahb I've opened https://github.com/open-webui/open-webui/pull/839. That would add an automatic GitHub release workflow that tags releases with the short-form of the commit hash.
The same behavior could easily be adopted in the Docker building workflow.
Regards.

Looks like we got an extra v somewhere @tjbck

@justinh-rahb #850 should fix it!


I would like to buy from you

Hello developer, I made a tour of your store. Do you have shipping all over the world?

Please send a message to Discord dm

Hello developer, I made a tour of your store. Do you have shipping all over the world?

Please send a message to Discord dm


Config: preserve the load balancer factory when merging CIOs if there's no new

load balancer factory provided.

Commit Message: Config: preserve the load balancer factory when merging CIOs if there's no new load balancer factory provided.
Additional Description:
Risk Level: low / med
Testing: modified test include testing this behavior
Docs Changes: na
Release Notes: na
Platform Specific Features: na
[Optional Runtime guard:] Config guarded feature
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/assign @RyanTheOptimist

/retest


Remove unnecessary signature from on-the-wire messages

This is the first attempt for my very first task! Passed tests under /net locally.

cc @dgtony @requilence: Here we're removing the payload signatures in the p2p API.

@merlinran before merging, can you remove the "[wip]" from the PR title? We have started using the draft mode of GH PRs to make WIP status more clear.

Thanks @sanderpick ! btw I don't seem to have the permission to merge.

Oop! I think you should have access now.


docs: fix subscription example and add output-location note

Fixes #504 and Fixes #502

Made changes. Pls review @manekinekko @sgollapudi77


Non-functional in HA 2023.5 -- Fix Below

For those who are still trying to use this in 2023, make the following change below on line 24 in init.py:

Thanks! Can you double check the code in https://github.com/joncar/ha-fpa/tree/user/joncar/fix-2023.5 works for you? I don't have the device anymore to check for myself.

Seems to work. Do you plan on merging to master so we can integrate the github repo into hacs?

Done. Thanks!


Update around Stripe's new service agreement

The problem
Stripe's service agreement is being updated, and, because we use Stripe's custom accounts to run Daemo, we are responsible for updating the community with these changes.
The proposal
Put out an update in a forum post and link to the new materials.

To break consensus use this template.

Is duplicated by https://github.com/crowdresearch/collective/issues/4


Get hash of last message to prevent reposts on app restart

Closes #6
Messages will now be posted with a hash included in the context of the message:

On start-up, get the last bot message and get that hash, and then use that to make sure that we do not post duplicate messages.
Also changed things so that ignore "blank" alerts completely and assume that alerts before/after should be compared and unique, and if the same message, won't repost as it's probably a RPI error then?
Did not properly test with slack itself, please let me know best way to do this.

I will merge this into the dev branch later and do a test with another server. It looks good, but testing would always be good.


Assignment creation has all courses in dropdown

My understanding is that this page is only reached from an existing Course page. I think that means that there really shouldn't be a dropdown here at all since we know what Course the assignment is being created for.

Possibly related: #718. v2 is no longer being maintained.


Redesign Homepage

🚀 Description
Redesign the homepage as described in the issue, this requires some new components and API implementations.
CC: @mslowiak

📄 Motivation and Context
Closes #87

🧪 How Has This Been Tested?
Ran the application using npm run start and it worked, also performed npm run test

📷 Screenshots (if appropriate)

📦 Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

✅ Checklist

[x] My code follows the code style of this project(Do your best to follow code styles. If none apply just skip this).
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Sorry for the missing reply the last days, but it works now, that is the most important thing anyway. I also created #124 to merge the rebranding branch with the one in master.

Hey @mslowiak :wave: Sorry for bothering you, but could you check this PR again, please?

@Tobi406 looks fine for me :)


Phase names and log names inconsistent

The name of the phases should be consistent with the names of the log files.

e.g. phase onnx-import has its log file called torch-onnx.log.

https://github.com/nod-ai/SHARK-TestSuite/pull/73
Good observation, thanks for reporting, uses current phase name now.

Fixed it. Please check.


fix copyright yaer

This pr is to fix copyright year in pkg/ddc/base/volume.go

you should add -s when you use command "git commit"


Svelte 1.0

Roadmap for Svelte 1.0
Basics

[x] Setup Package
[x] Setup Testing
[x] Setup Storybook
[x] Setup Template (svelte-kit)
[ ] Docs

Core

[x] Anatomy
[x] Collection
[x] Factory (asChild)

Providers

[x] Environment
[x] Locale

Components

[ ]  Accordion
[x]  Avatar
[ ]  Carousel
[ ]  Checkbox
[ ]  Clipboard
[ ]  Collapsible
[ ]  ColorPicker
[ ]  Combobox
[ ]  DatePicker
[ ]  Dialog
[ ]  Editable
[ ]  Field
[ ]  Fieldset
[ ]  FileUpload
[ ]  Format
[ ]  Frame
[x]  Highlight
[ ]  HoverCard
[ ]  Menu
[ ]  NumberInput
[ ]  Pagination
[ ]  PinInput
[ ]  Popover
[ ]  Portal
[ ]  Presence
[ ]  Progress
[x]  QrCode
[ ]  RadioGroup
[ ]  RatingGroup
[ ]  SegmentGroup
[ ]  Select
[ ]  SignaturePad
[ ]  Slider
[ ]  Splitter
[ ]  Steps
[ ]  Switch
[ ]  Tabs
[ ]  TagsInput
[ ]  TimePicker
[x]  Timer
[ ]  Toast
[ ]  Toggle
[ ]  ToggleGroup
[ ]  Tooltip
[ ]  TreeView

@cschroeter
Hi
In Svelte, when you destructure a reactive variable ($state, $derived), it loses reactivity.
Code like this:
const { dir } = useLocaleContext()
const { getRootNode } = useEnvironmentContext()

Should be written the same way as Vue code:
const env = useEnvironmentContext()
const locale = useLocaleContext(DEFAULT_LOCALE)

Like this one:
https://github.com/chakra-ui/ark/blob/be4e356070b1c37384a9501b1db4e6dee8904793/packages/svelte/src/lib/components/qr-code/use-qr-code.svelte.ts#L13

Would it be OK to contribute on some of the components?

@babakfp Hey mate - thanks for reaching out. Yes this is a good catch. We found also some issues in Zag that we fixed in the latest version. Feel free to convert any of the components left if you have some time to contribute <3

Hello! I'm waiting for ark for Svelte, and trying to work on the incomplete items in the Components section from the progress above. I'm attempting to work on the Frame component first - is there anything I should know beforehand? Or are there any good first issues(component) you'd recommend?


Update Japanese translation.

Many thanks, I'll merge this in just a few minutes (but it may be a bit until the release).


IME mode isn't automatically escaped when pressing Ctrl + S in Source Code Mode, causing user to save twice.

Description
If user tries to save file with Ctrl + S in Source Code Mode, IME mode isn't automatically escaped.
Pressing Ctrl + S first time escapes IME mode, but don't save the file. So user must press Ctrl + S again to save file.

[x] Can you reproduce the issue?

Steps to reproduce

Make sure that you're using language with IME mode, such as Chinese, Japanese, Korean etc.
Write some texts on Editor Mode
Save document with Ctrl + S on Editor Mode
It will be saved with automatic IME mode escape
Toggle to Source Code Mode by pressing Ctrl + E
Write some texts on Source Code Mode
Make sure you don't escape IME mode with Space or Enter or arrow key etc.
Save document with Ctrl + S on Source Code Mode
First attempt will fail while escaping IME mode
Second attempt will save file correctly

Expected behavior
When pressing Ctrl + S, IME mode should be escaped and file should be saved too, just like in Editor Mode.
Actual behavior
Only IME mode is escaped when pressing Ctrl + S while at IME mode.
Versions

MarkText version: Local build from 01e6922
Operating system: Windows 11 22H2

Additional Note
I'm not sure if this is related to https://github.com/marktext/marktext/pull/3419 but I really don't know why this problem happens.
This problem won't be easy to reproduce for people not in East Asia because they don't have IME mode in their keyboard/language. You might need to install CJK language pack and switch to corresponding keyboard layout to reproduce.
Instead of posting issue in marktext/marktext, I choose to post it here because this repository is much more active than the origianl.

Thank for for the detailed bug report.

This problem won't be easy to reproduce for people not in East Asia because they don't have IME mode in their keyboard/language. You might need to install CJK language pack and switch to corresponding keyboard layout to reproduce.

Yes, unfortunately this will take some time and research on my part. I can't commit to doing so in the near-term. I added the "help wanted" label to this issue, in case someone is able to help fix it.


reference state is fixed

Currently, the reference state is fixed.
To change the reference state, one has to change the helmholtzCoefficients.idealPower.
Examples.Validation.ReferenceState contains some more explanation

When changing the reference state, SMIN, SMAX, HMIN, HMAX, SCRIT and HCRIT have to be adapted, too.


Added new method go() to allow opening of non-whitelisted apps in iOS

iOS allows canOpenURL() only for whitelisted app-schemes, but openURL() is allowed for every app-scheme. I have added an additional method go() next to your existing methods check() and start(), that calls openURL() without first testing with canOpenURL().
This allows for launching apps that have not been added to LSApplicationQueriesSchemes in the plist file.

I have merged so that there are no conflicts.

Hello, is it possible to give an example how to use this function go? When i try to use it I get an error that the function is not found
let app = startApp.set(params.uri).go(function() {                         successCallback("App Launched");                     }, function(error) {                         errorCallback(error);                     });