How to enter multiple coordinates?

Hi,  I have a question, how to enter multiple coordinates?
python remove_anything.py \

--input_img ~/input/test1.png \
--point_coords 58 127 778 127 58 366 778 366 \
--point_labels 4 \
--dilate_kernel_size 15 \
--output_dir ~/output \
--sam_model_type "vit_h" \
--sam_ckpt ~/sam/sam_vit_h_4b8939.pth \
--lama_config ./lama/configs/prediction/default.yaml \
--lama_ckpt big-lama

Traceback (most recent call last):
File "/root/Inpaint-Anything/remove_anything.py", line 75, in
masks, _, _ = predict_masks_with_sam(
File "/root/Inpaint-Anything/sam_segment.py", line 28, in predict_masks_with_sam
masks, scores, logits = predictor.predict(
File "/root/Inpaint-Anything/segment_anything/segment_anything/predictor.py", line 154, in predict
masks, iou_predictions, low_res_masks = self.predict_torch(
File "/root/miniconda3/lib/python3.9/site-packages/torch/autograd/grad_mode.py", line 28, in decorate_context
return func(*args, **kwargs)
File "/root/Inpaint-Anything/segment_anything/segment_anything/predictor.py", line 222, in predict_torch
sparse_embeddings, dense_embeddings = self.model.prompt_encoder(
File "/root/miniconda3/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
return forward_call(*input, **kwargs)
File "/root/Inpaint-Anything/segment_anything/segment_anything/modeling/prompt_encoder.py", line 155, in forward
point_embeddings = self._embed_points(coords, labels, pad=(boxes is None))
File "/root/Inpaint-Anything/segment_anything/segment_anything/modeling/prompt_encoder.py", line 84, in _embed_points
points = torch.cat([points, padding_point], dim=1)
RuntimeError: Sizes of tensors must match except in dimension 1. Expected size 8 but got size 2 for tensor number 1 in the list.

Hello, may I ask about the reason for wanting to input multiple coordinates? Are you attempting to refine the mask of a single object?"

Hello
I want to mark more accurately by multiple coordinate points. This display supports multiple coordinate points. Will it support multi-coordinate points in the future?
usage: remove_anything.py [-h] --input_img INPUT_IMG --point_coords POINT_COORDS [POINT_COORDS ...] --point_labels POINT_LABELS [POINT_LABELS ...]
[--dilate_kernel_size DILATE_KERNEL_SIZE] --output_dir OUTPUT_DIR [--sam_model_type {vit_h,vit_l,vit_b}] --sam_ckpt SAM_CKPT
[--lama_config LAMA_CONFIG] --lama_ckpt LAMA_CKPT

Hello! Thank you so much for your suggestion.
We're happy and willing to add support for multi-coordinate points soon, which will greatly improve the accuracy of our mask generation. Stay tuned for updates!

Hello
I want to mark more accurately by multiple coordinate points. This display supports multiple coordinate points. Will it support multi-coordinate points in the future?
usage: remove_anything.py [-h] --input_img INPUT_IMG --point_coords POINT_COORDS [POINT_COORDS ...] --point_labels POINT_LABELS [POINT_LABELS ...] [--dilate_kernel_size DILATE_KERNEL_SIZE] --output_dir OUTPUT_DIR [--sam_model_type {vit_h,vit_l,vit_b}] --sam_ckpt SAM_CKPT [--lama_config LAMA_CONFIG] --lama_ckpt LAMA_CKPT

Hello! Thank you so much for your suggestion.
We're happy and willing to add support for multi-coordinate points soon, which will greatly improve the accuracy of our mask generation. Stay tuned for updates!

looking forward to support for  multi-coordinate points soon

Thank you. Will be supported.

Hi, Does the multi-coordinate points come?  Is it feasible for me to try to record multiple coordinate points using a csv file and then iterate to read the points for processing？

2024年2月120.期待官方的多座标点的更新！！

Hey, guys! I have tried to implement the remove_anything for multiple coordinates and labels. Please check it out and give feedbacks :)

Hey, guys! I have tried to implement the remove_anything for multiple coordinates and labels. Please check it out and give feedbacks :) @chenjun0210 @geekyutao @RuoyuFeng @ChamnLee @tsqt2023 @weijiafs

awesome! Thanks!


Define metadata necessary to populate a community component catalog

If a team or a domain creates a component, and we want to publish it into the catalog, what is necessary to provide a consistent, informed, experience for consuming those components.
e.g.

Component name
Common alternate names
One line description
Thumbnail image or url
URL to component guidance doc (can link to Salt satellite site, Figma, Storybook, or a repo)
Usage documentation follows template Y/N?
There is a Figma spec and kit Y/N?
Component has a11y specifications Y/N?
Component has code Y/N?
Code passes a11y tests Y/N?

Connected to #1205

Added this as a task of #1207, which is the epic for a community component index. Also, #1195 seems related, so have added that to the same epic as well.


Tag version 0.2.0

Could the project kindly create a tag for 0.2.0?

@eric-murray as this has to be done by one of the codeowners - would you be so kind to take it?

@hdamker Yes, I can do this.

@eric-murray the release looks good for me!
Regarding protecting the release-* branches you are right - they should be protected. I suppose it would make sense to create a rule for that in all sub projects  @shilpa-padgaonkar what do you think? Could that be an issue for commonalities (small addition to release guidelines) and then we can implement it across the project?

Release LGTM as well.
Yes, makes sense to add it as a rule in the release guidelines. @hdamker : Would you like to open a PR? :)

All branches with *release* are now protect with same settings than main`.
Issue to update release guidelines created in Commonalities: https://github.com/camaraproject/WorkingGroups/issues/186


Javalike method pointer groovy 7772

This is a change for method pointer semantics in groovy discussed in following issue.
https://issues.apache.org/jira/browse/GROOVY-7772
ref.
http://docs.groovy-lang.org/latest/html/documentation/index.html#method-pointer-operator
https://docs.oracle.com/javase/tutorial/java/javaOO/methodreferences.html#type

I merged in the test case additions commit from this PR. The other commits have been superseded. Thanks!


Fix button error

What does this fix exactly?

Please hurry up :trollface:

https://github.com/Cubed-Development/Modern-Warfare-Cubed/issues/156

So did you forget to add it or intentionally delete it? This is the statement included in the previous version

So did you forget to add it or intentionally delete it? This is the statement included in the previous version

?

This is not something I added myself, it was included in the previous development version

If you are not aware that this statement was previously included, please add it


Update MeltanoData instance setup guide to use new create_instances script

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/1935
Originally created by @DouweM on 2020-04-02 00:19:10

View 3 previous comments from the original issue on GitLab


fix: fixes demo app fetching job

Overview

Fixes #
Checklist
My PR contains...

[ ] No code changes (changes to documentation, CI, metadata, etc.)
[ ] Bug fixes (non-breaking change which fixes an issue)
[ ] Improvements (misc. changes to existing features)
[ ] Features (non-breaking change which adds functionality)

My changes...

[ ] are breaking changes
[ ] are not breaking changes
[ ] If yes to above: I have updated the documentation accordingly

Documentation

[ ] My changes do not require a change to the project documentation
[ ] My changes require a change to the project documentation
[ ] If yes to above: I have updated the documentation accordingly

Tests

[ ] My changes can not or do not need to be tested
[ ] My changes can and should be tested by unit and/or integration tests
[ ] If yes to above: I have added tests to cover my changes
[ ] If yes to above: I have taken care to cover edge cases in my tests

Code Coverage

File
Coverage [53.54%]

atala-prism-sdk/src/commonMain/kotlin/io/iohk/atala/prism/walletsdk/prismagent/PrismAgent.kt
53.54%

Total Project Coverage
25.83%


client-auth.c: Fix authentication denial for valid logins

According man pages, getgrouplist() always return non-zero number, so we have to handle only the case when user is in more groups than we have static array for.
This happens when you run PAPPL based printer application with -o auth-service=password-auth -o admin-group=wheel.

From the getgrouplist man page on my Mac:
RETURN VALUES
     The getgrouplist() function returns 0 on success.  If the size of the
     group list is too small to hold all the user's groups, getgrouplist()
     returns -1 to indicate failure.  In this case, the group array will be
     filled with as many groups as will fit.

OpenBSD's implementation seems to match macOS:
RETURN VALUES

The getgrouplist() function returns 0 if successful and -1 if the size of the group list is too small to hold all the user's groups. Here, the group array will be filled with as many groups as will fit.

But glibc's implementation is different:
RETURN VALUE

       If the number of groups of which user is a member is less than or
       equal to *ngroups, then the value *ngroups is returned.

       If the user is a member of more than *ngroups groups, then
       getgrouplist() returns -1.  In this case, the value returned in
       *ngroups can be used to resize the buffer passed to a further
       call to getgrouplist().

The fact that you are seeing issues with 32 groups means we need to increase the size of that array, too.

But glibc's implementation is different:
RETURN VALUE

       If the number of groups of which user is a member is less than or
       equal to *ngroups, then the value *ngroups is returned.

       If the user is a member of more than *ngroups groups, then
       getgrouplist() returns -1.  In this case, the value returned in
       *ngroups can be used to resize the buffer passed to a further
       call to getgrouplist().

The fact that you are seeing issues with 32 groups means we need to increase the size of that array, too.

We can do that too for sure, but the issue was with the condition - the function probably returns 0 only when user would not be in any group (IMHO it is not possible, if the user exists, so it is not mentioned in man page), so it returned non-zero for my user (positive number of groups, since I'm not in more than 32 groups), so the execution got into the if scope and returned failure.

@michaelrsweet the message in Changelog is incorrect - it was not issue about user being in more than 32 groups. Auth failed if user was in any group (the function returned number of groups he's in) with glibc.


[SECURITY] Denial of service because of unsafe regex processing

I have tried to contact you by<EMAIL_ADDRESS>and created https://github.com/jflyfox/jfinal_cms/issues/22 asking for the contact. Nobody replied.
The JFinal_cms is vulnerable to regex injection that may lead to Denial of Service.
User controlled path and contextPath are used to build and run a regex expression (first argument to replaceFirst):
https://github.com/jflyfox/jfinal_cms/blob/1a9653264be5a2fcf3641ae2aa5f14fc188fd192/src/main/java/com/jflyfox/modules/filemanager/FileManager.java#L929-L949
Since the attacker controls the string and the regex pattern he may cause a ReDoS by regex catastrophic backtracking on the server side.

Thank u  for feedback .
for easy, There is a problem.

Hi, is there any update on this?

Mitre assigned CVE-2021-38244 ID for the issue.


Dependabot/maven/org.jetbrains annotations 26.0.1

name: Pull Request
about: Propose changes to the codebase
title: ''
labels: ''
assignees: ''

Description
Please provide a brief description of your changes.
Changes

List of changes
...

Verification

[ ] Changes have been tested locally.
[ ] Documentation has been updated accordingly.

Related Issues
Fixes # (issue number)
Additional Information
Add any other information here.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.89%. Comparing base (ba54c39) to head (5a5d36b).
Report is 21 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##               main      #69   +/-   ##
=========================================
  Coverage     96.89%   96.89%
  Complexity      811      811
=========================================
  Files            51       51
  Lines          1835     1835
  Branches         99       99
=========================================
  Hits           1778     1778
  Misses            2        2
  Partials         55       55

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Reddit backfeed appears reversed

Thanks for fixing the reddit backed issue. However there appears to be a new issue.
I now see a list of reddit comments that bridgy has found, but they are in reverse order. That is, on the first page I see the oldest posts and comments. If I click the "Older->" button it actually takes me to newer comments, and if I keep clicking it I eventually get to the last page, which has the most recent comments.

Sorry for the confusion! Responses on https://brid.gy/reddit/rattroupe aren't perfectly ordered by timestamp, but by when Bridgy found them. Looks like it found pretty much all of those comments at the same time. Going forward, new comments will show up at the top as Bridgy finds them.


Correct the default DNS resolver to ares

The default value for GRPC_DNS_RESOLVER is changed since https://github.com/grpc/grpc/pull/17723 which released in v1.19.0. So, this part of doc should be corrected.

@apolcyn, isn't the default per-platform these days?  Or is that just internally?

@apolcyn, isn't the default per-platform these days? Or is that just internally?

^ yeah it's per-platform.
It's probably good to mention something like:
"ares is the default on most platforms, but not on iOS, Android, or Node"

The description about per-platform default value is added.


[BUG]: Action 'Parse_JSON' failed - Audit Log Components

I get following error "Action 'Parse_JSON' failed". Any pointers how to solve this?

We had a similar issue earlier that was closed with the last release
Can you verify that you're on the latest release?
Can you look at this thread to see what further information we would need to help solve this, if you're on the latest release and still experiencing issues
https://github.com/microsoft/powerapps-tools/issues/672
If you have recently upgraded, can you confirm you have followed the steps here to remove unmanaged layers. this is required to receive the latest upgrade
https://docs.microsoft.com/en-us/power-platform/guidance/coe/setup#installing-updates
Thank you
Manuela

After last update error was gone. A new is introduced, but I make another thread for that. Thanks appreciated.

Great thank you - I will close this issue for now, but let me know if it occurs again so we can look into it!
Thank you for using the CoE
Manuela


handle commands with bot username

In Group conversations, there will be many bots. So /some-command-here@my-bot will only work.
Remove @my-bot part before comparing with list of commands

Should username in command@username match bot's(TelegramClient instance) actual username?

Ah, yeah. there's that. I'm sure all the libraries I came across, doesn't care about the username.
Only one instance you need to type in the username of the bot when the group privacy is turned on. When that happens, bot will receive update only when it's username is called.
We can safely assume that it is our bot's username and skip this username validation part.


Use update rather than save in assign_chants_to_segments

In testing the assign_chants_to_segments command (introduces in #1434) for #1437, I was finding the command was taking a very long time (about 20,000 chants assigned over the course of about 12 hours). This meant we were looking at a weeks-long command run, which seemed ripe for things going wrong!
This PR changes the command to use the update method to add the segment to the chants in a source. Run-time is considerably reduced in tests.
@lucasmarchd01 raised the point in #1434 that batching chants would reduce the possibility for memory error. The use of update also obviates that need because chant objects are never loaded into memory here, and the updates happen entirely on the SQL side (as such, the update method does not work on sliced query sets).

@lucasmarchd01 raised the point in #1434 that batching chants would reduce the possibility for memory error. The use of update also obviates that need because chant objects are never loaded into memory here, and the updates happen entirely on the SQL side (as such, the update method does not work on sliced query sets).

Good to know, thanks!

Good to know, thanks!

Yeah, it was a nice discovery this morning!

Something to keep in mind is that when using the update method on a chant wont call the on_chant_save method in signals.py won't be triggered. I don't think it will matter in this case but maybe a check to make sure would be good!

Yes...I checked here and the segment change won't implicate any of the signals, but an important consideration for other uses.


feature request: script to fetch machine readable version and install in RTIR/OTRS

As discussed in the meeting in Paphos 9/19, it would be great to have a script that pulls the latest version of the machine readable file and may install it as custom fields into RTIR.
(possibly the same is doable for OTRS?)
Future versions of RTIR could hopefully include the taxonomy as custom fields by default

Dear Community. We had a meeting with OTRS today. At the moment, we are adding the taxonomy manually. They see a benefit and also have customers who are using the taxonomy. They will clarify whether they can integrate this as a new feature. If you have questions or suggestions, Jens Bothe, from OTRS, can be contacted directly<EMAIL_ADDRESS>

OTRS (STORM) version now pulls directly with a web service the machinev1 version to fill up.
Thanks to Jens


Tool Deletion/Dupe

Somehow Swapper 1.03 has duplicated my Lumber Axe and overwritten my Hammer. The tools in the screenshot are (Iguana) Tinkers (Excavator, LAxe, LAxe), Draconic Evolution Sword and Reliquary Soujourner's Staff.
I'm really glad I had a backup of my tools in a creative chest ;)
The modpack is basically FTB Infinity 1.3.1 with some mods (IC2, iChun's and Headcrumbs) removed and a few added (Reliquary, Iron Backpacks, Iguana Tinker Tweaks, Compact Machines)
Thanks.

The modpack is basically FTB Infinity with some mods (IC2, iChun's and Headcrumbs) removed and a few added (Reliquary, Iron Backpacks, Iguana Tinker Tweaks, Compact Machines)

Do you know what you did to cause this?  What are the steps to reproduce?

Unfortunately not. I only noticed it when I tried to remove the ineffable glass in the screenshot and it only removed a single block instead of all 6. I've reset the swapper to the original tools and I'll try and isolate some sort of cause.

With any luck this'll prevent this bug from happening again.  Watch for the new version and please reopen this ticket if it happens again.


Project filtering does not show empty projects

Internal Reference: SURE-4312

Create a new Project called 'empty' - don't add any namespaces.
In the project list you will see the project.
In the filter box, type 'empty' - the list will be empty and does not show the project, even though it matches the filter text.

without filter:

✅  Passed Dev-Validate

Image Name
Image Tag
Image Digest

rancher/rancher
v2.6.3
f411ee37efa38d7891c11ecdd5c60ca73eb03dcd32296678af808f6b4ecccfff

Reproduction Steps:

run rancher version v2.6.3 docker run -d --restart=unless-stopped -p 80:80 -p 8443:443 --privileged rancher/rancher:v2.6.3
provision a new downstream cluster (digital ocean droplet)
Create a project named "empty" without any namespaces
filter search for the name empty
observe that no projects are shown

Validation Steps:

run rancher version v2.6-head docker run -d --restart=unless-stopped -p 80:80 -p 8443:443 --privileged rancher/rancher:v2.6-head

Image Digest: df15cf3e3c93329658cdfde7ebf3b5dc3c8f20a22ad85f3621d189890402b95a
Version: v2.6-6739a84ede27905dfcccbc9675c875d1752d438a-head

provision a new downstream cluster (digital ocean droplet)
Create a project named "empty" without any namespaces
filter search for the name empty
observe that created project is shown

Validation Status: Pass
@sowmyav27


name_debug to RPC fix, resubmitted

Transform name_debug* calls to actual RPC calls.

Hello! Is this broken or wrong somehow? I would be happy to make any corrections necessary or suggested and resubmit.


Issues with email format and contents in mobile view (web-app)

When viewing the template on mobile view (outlook app and web-app), the contents of the template are inconsistent.
Steps to reproduce:

Open the email editor
Drag a text to the editor

edit the text to "We’re giving you."
set the font size to 28px and align center
set font weight 700

Drag a button underneath the text

set padding 15px 55px 5px 55px
set the text to "on your next purchase within 14 days*"
set the font size to 24px and align center
set font weight 700

Drag a text again under the button

set padding 5px 55px 25px 55px
set the text to "Shop at your local Discount Drug Store today"
set the font size to 20px and align center
set font weight 700

Observation:
If we send this template in the email, the elements are not consistent in mobile view
web-app and outlook app.
Below if the screenshot of how it appears in the outlook mobile web-app (Here I'm using Safari). Here we can see the text is not wrapped correctly and button is also missing.

Expectation:
The email is supposed to look like the screenshot below taken from the outlook app on mobile phone.

Please check if the button link is valid, as some platforms may remove the button if the link is invalid.

Thanks @m-Ryan, for the button we figured that the default # that gets added to the Href when we drag a button which was causing the button to disappear in some platforms.
However there still are issues with text wrapping it seems. The texts are breaking when we send test emails. Please view the screenshot below:

https://www.caniemail.com/search/?s=word-break
The default style of MJML is word-break: break-word, you can override it.
https://developer.mozilla.org/en-US/docs/Web/CSS/word-break
But I don't have a better way to recommend.
Relatived:
https://github.com/mjmlio/mjml/issues/1728
https://github.com/mjmlio/mjml/issues/2615
https://github.com/mjmlio/mjml/issues?q=word-break


handle users getting invited to multiple projects via email at once

per https://github.com/sillsdev/languageforge-lexbox/pull/567#discussion_r1491276240 this could happen but right now would get lost. But it's out of scope for that PR.

2 theoretical scenarios that could break an invite link:

For a course, a teacher invites every student to a FLEx project and a LIFT project. Students would only be able to join one.
A user registers an account after getting an invite email.

Besides fixing this, #1043 also checks that the email address in the invitation link matches the email address in the JWT. Without that check, someone who manages to get access to an email invitation that isn't theirs (perhaps by persuading the genuine recipient to forward the email to them) could easily rewrite the URL in the invite link to include their email address. We verify that the JWT is genuine, but then we trust the email address from RegisterAccountInput without validating it. So #1043 closes that hole.


[Suggestion] Add use of  parameters shards_num and using in example of creating collection

Add parameters shards_num and using in https://milvus.io/docs/v2.0.0/create.md (Create Collection - 1) with example values.
Add notes that if they are optional and their default values. (Please double check with dev: shards_num = 2, using = 'default' as default)
Maybe also add in https://milvus.io/api-reference/pymilvus/v2.0.0rc5/api/collection.html (APIs References > class pymilvus.Collection > Example)

@jaelgu Thank you. Working on it.


Java build failure during SDK release job

What happened?
         *   @pattern: \d+(?:-\d+)?
             ^
23 errors

It appears that the release workflow https://github.com/pulumi/pulumi-gcp/actions/runs/6881931301/job/18720430202#step:3:6581 did not build (or publish) the Java SDK properly.
Example
N/A
Output of pulumi about
N/A
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

A rerun resulted in the same error: https://github.com/pulumi/pulumi-gcp/actions/runs/6881931301/job/18728079843#step:3:2830

Ah I see I guess we're expecting 7.1.x patch to relese but we forfeit having 7.1.0 in https://central.sonatype.com/artifact/com.pulumi/gcp/versions .. slightly unfortunate but does indeed resolve the problem. Thank you.


WAIT: Add maelstrom script

Resolves #5
Also using namespace http://example.com/ here.

We probably could use the src/properties.tsv template from #6 instead of including the two annotation properties in maelstrom.ttl. I can update this after we merge that one.

@beckyjackson Please update this PR.

Updated and ready to merge

@beckyjackson Please update file locations if necessary, then let me know and I'll review.

This is updated and should be ready to go!


Under the heading Prime the Output

What is this part of the document saying? "## One possible search query is:"
Are we telling it to look for keywords or... why do we suddenly have hashtags in the prompt example in this doc?

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

ID: cb9e5229-1e4d-fc14-8845-013ec692aad4
Version Independent ID: cc651542-802d-1104-af78-6f6764f3b859
Content: Prompt engineering techniques with Azure OpenAI - Azure OpenAI Service
Content Source: articles/cognitive-services/openai/concepts/advanced-prompt-engineering.md
Service: cognitive-services
GitHub Login: @suhridpalsule
Microsoft Alias: mbullwin

@MSFTTracyP
Thanks for your feedback! We will investigate and update as appropriate.

Hi @MSFTTracyP
The section titled "One possible search query is:" is an example of how to use a prompt to prime the model to produce a specific output. The prompt is used to communicate the desired output to the model and can be used to remove hedges that the model might typically add.
The use of hashtags in the prompt example is not related to keyword searching. Instead, it is used to differentiate between different sections of the prompt and to prime the model to produce a specific output.
If there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

@MSFTTracyP
We are going to close this thread, if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.
#Please-close

Hi @Naveenommi-MSFT
In this case, what could be the prompt to test this scenario? Please guide.

Hello @nanddeepn
In this case, the example search query is being used to generate a prompt that includes information about a specific topic. The hashtags (#) are being used to indicate keywords or topics that are relevant to the prompt.
For example, if the search query is "climate change", the resulting prompt might include information about the causes and effects of climate change, as well as potential solutions and policy recommendations. The hashtags in the prompt might include #climatechange, #globalwarming, #environment, and so on.
To test this scenario, you could try using a similar search query to generate a prompt and see what kind of results you get. You can experiment with different search queries and hashtags to see how they affect the resulting prompts.


[HOLD for payment 2023-09-29] [$250] Tooltip flickers when click 3 dots menu on task report

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:

Go to any task repot
On report header, hover 3 dots menu
Verify that tooltip with "More" text shows
click 3 dots menu

Expected Result:
Tooltip should be hidden or should keep showing
Actual Result:
Tooltip flickers - disappears and re-appears
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

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

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

Version Number: 1.3.64-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/c8f945d3-7cbd-4eb5-ac71-44da61525cc3
https://github.com/Expensify/App/assets/43996225/b63fbd94-950f-4374-8d97-583779ebb97e
Expensify/Expensify Issue URL:
Issue reported by: @situchan
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1693310046653569
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01e970a8827f09ba1a
Upwork Job ID:<PHONE_NUMBER>958922752
Last Price Increase: 2023-09-14
Automatic offers:

0xmiroslav | Reviewer | 26737206
bernhardoj | Contributor | 26737208
situchan | Reporter | 26737209

I'm going to reduce the bounty to $250 since this is a very minor design bug that a user is unlikely to notice.
Out of curiosity, why doesn't this happen with the global create menu?

Proposal
Please re-state the problem that we are trying to solve in this issue.
Tooltip flickers when clicking the three-dot menu
src > components > ThreeDotsMenu > index.js
wrap PopoverMenu with the  tag. – it works
Reasons:
In React, layout and rendering can be affected by the nesting of components within  elements. Wrapping the PopoverMenu component in a  might help stabilize the layout and rendering, ensuring that it behaves as expected.
By enclosing the PopoverMenu within its own , you create a clear separation between the button (the trigger) and the menu. This separation can prevent unexpected interactions between the two components, which might have been causing the flashing or flickering.
Sarah Papierz

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~017e3c2bd26318290f

@0xmiroslav thoughts on the proposals?

After reevaluating this problem, I believe the issue lies within the functions that run outside of a controlled React component mounting lifecycle in the PopoverWithMeasuredContent.js file. The conditional logic responsible for measurement and visibility is floating within the function. This uncontrolled timing might not trigger at the right moment, potentially leading to inaccurate measurements and popover positioning. To ensure consistent behavior, we can address this by adding a useEffect that runs as early as the component mounts, handling the measurement and visibility checks reliably.
Here is the new proposed fix:
useEffect(() => {
    if (!isVisible && props.isVisible) {
        // When the popover is shown, recalculate
        setIsContentMeasured(props.popoverDimensions.width > 0 && props.popoverDimensions.height > 0);
        setIsVisible(true);
    } else if (isVisible && !props.isVisible) {
        setIsVisible(false);
    }
}, []);

This is consistent with the popover being used for the Start a Chat if you take a look in the BaseVideoChatButtonAndMenu.js file. you'll notice that the measurement logic is placed inside a useEffect that runs when the component mounts:
This organized approach ensures that the measurement and visibility checks are executed consistently and at the appropriate times.
This solves the flickering issue when clicking.

@0xmiroslav friendly reminder on the checklist so that we can pay this out later this week 🙂

Payment summary:
@situchan - $50 for reporting (paid in Upwork)
@bernhardoj - $250 + $125 bonus (paid in Upwork)
@0xmiroslav - $250 + $125 bonus (to be tracked by @0xmiroslav separately)
Thanks everyone!


Treemap Chart Type

Is your feature request related to a problem? Please describe.
The project I am working on requires Treemap Charts in the near future, and is currently working on a plan to deprecate our current charting library.
Describe the solution you'd like
Add the Treemap Chart type to Apex Charts.
Describe alternatives you've considered
ngxCharts supports tree maps, however, ngxCharts does not come close to having the same set of features that Apex has.
Additional context
Apex Charts feature-set exceeds our requirements for a charting library, that would allow us to move off of ngxCharts.

bump

any news on this?

Alright, I have started working on it.

The first version done.

Most probably, demos and docs will be ready in 7-15 days with a new ApexCharts release.

Treemap charts are released with 3.21.0
Here are the demos - https://apexcharts.com/javascript-chart-demos/treemap-charts/

Treemap charts are released with 3.21.0
Here are the demos - https://apexcharts.com/javascript-chart-demos/treemap-charts/

Great improvement!! Thx.
How can I make the categories larger? I mean, how to make the label font size base a large proportion from the rectangle?

@fguimara Please open a new ticket for that.

Done

@junedchhipa Having great problem in label size.

Already tried setting fontSize. Does not work. Please help.

It automatically changes size to small.

Also tooltip goes out of chart.


"panic: runtime error: " when upgrade is attempted with source service that does not exist.

rancher-server version - v0.29.0-rc1
rancher-compose version - v0.2.3-rc1
Attempt to upgrade with source service that does not exist - "test3" doesnot exist.
./rancher-compose -p testyo upgrade test3 test4 --batch-size 4 --scale 12 --interval "30000" -w
root@sangee01:/home/sangeethahariharan1/v23/rancher-compose-v0.2.3-rc1# ./rancher-compose -p testyo upgrade test3 test4 --batch-size 4 --scale 12 --interval "30000" -w
INFO[0000] Creating service test4
INFO[0002] Upgrading test3 to test4, scale=12
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x0 pc=0x5f24d2]

goroutine 1 [running]:
github.com/rancherio/go-rancher/client.(*ServiceClient).ActionUpgrade(0xc208030390, 0x0, 0xc20826d500, 0x3, 0x0, 0x0)
	/source/Godeps/_workspace/src/github.com/rancherio/go-rancher/client/generated_service.go:203 +0x122
github.com/rancherio/rancher-compose/upgrade.Upgrade(0xc208040180, 0x7ffd14b9e89c, 0x5, 0x7ffd14b9e8a2, 0x5, 0x4, 0x7530, 0xc, 0x101, 0x0, ...)
	/source/gopath/src/github.com/rancherio/rancher-compose/upgrade/upgrade.go:69 +0x7d1
github.com/rancherio/rancher-compose/app.Upgrade(0xc208040180, 0xc2080b40c0)
	/source/gopath/src/github.com/rancherio/rancher-compose/app/app.go:73 +0x236
github.com/docker/libcompose/app.func·001(0xc2080b40c0)
	/source/Godeps/_workspace/src/github.com/docker/libcompose/app/app.go:30 +0x160
github.com/codegangsta/cli.Command.Run(0xa12ca0, 0x7, 0x0, 0x0, 0xacb610, 0x28, 0x0, 0x0, 0x0, 0x0, ...)
	/source/Godeps/_workspace/src/github.com/codegangsta/cli/command.go:113 +0x1038
github.com/codegangsta/cli.(*App).Run(0xc20800ad00, 0xc20800a000, 0xd, 0xd, 0x0, 0x0)
	/source/Godeps/_workspace/src/github.com/codegangsta/cli/app.go:156 +0xcf7
main.main()
	/source/main.go:63 +0xeee

goroutine 5 [chan send]:
github.com/docker/libcompose/logger.generateColors()
	/source/Godeps/_workspace/src/github.com/docker/libcompose/logger/colors.go:27 +0x191
created by github.com/docker/libcompose/logger.init·1
	/source/Godeps/_workspace/src/github.com/docker/libcompose/logger/colors.go:33 +0x25

goroutine 6 [syscall]:
os/signal.loop()
	/usr/src/go/src/os/signal/signal_unix.go:21 +0x1f
created by os/signal.init·1
	/usr/src/go/src/os/signal/signal_unix.go:27 +0x35

goroutine 7 [chan receive]:
github.com/docker/libcompose/project.(*defaultListener).start(0xc2080552a0)
	/source/Godeps/_workspace/src/github.com/docker/libcompose/project/listener.go:37 +0x7b
created by github.com/docker/libcompose/project.NewProject
	/source/Godeps/_workspace/src/github.com/docker/libcompose/project/project.go:50 +0x291

goroutine 13 [runnable]:
net/http.(*persistConn).readLoop(0xc20809a210)
	/usr/src/go/src/net/http/transport.go:928 +0x9ce
created by net/http.(*Transport).dialConn
	/usr/src/go/src/net/http/transport.go:660 +0xc9f

goroutine 14 [select]:
net/http.(*persistConn).writeLoop(0xc20809a210)
	/usr/src/go/src/net/http/transport.go:945 +0x41d
created by net/http.(*Transport).dialConn
	/usr/src/go/src/net/http/transport.go:661 +0xcbc
root@sangee01:/home/sangeethahariharan1/v23/rancher-compose-v0.2.3-rc1#
root@sangee01:/home/sangeethahariharan1/v23/rancher-compose-v0.2.3-rc1# cat docker-compose.yml
test3:
  tty: true
  image: sangeetha/testupgrade:v2
  stdin_open: true
test4:
  tty: true
  image: sangeetha/testupgrade:v1
  stdin_open: true
root@sangee01:/home/sangeethahariharan1/v23/rancher-compose-v0.2.3-rc1#

rancher-server version - v0.30.0-rc3
rancher-compose version v0.2.4-rc1
when upgrade is attempted with source service that does not exist , following error message is presented to the user:
FATA[0000] Failed to find service test1


Error

Can you help me with this error?

You spelled treee instead of tree 😉

sorry

Have you imported the package in your main module?
import { TreetableModule } from 'ng-material-treetable';

@NgModule({
    ...
  imports: [
    ...
    TreetableModule // <--- this
  ],
  ...
})
export class AppModule { }

yes

And I suppose GridModule is correctly imported in AppModule?
Can you reproduce this on stackblitz so I can have a proper look at it?

Thank you. Working


Automated cherry pick of #5181: aws listener sync fix

Cherry pick of #5181 on release/2.13.
#5181: aws listener sync fix

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.13@5fc0b07). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.13   #5183   +/-   ##
==============================================
  Coverage                ?   6.81%
==============================================
  Files                   ?     530
  Lines                   ?   90011
  Branches                ?       0
==============================================
  Hits                    ?    6137
  Misses                  ?   83366
  Partials                ?     508

Flag
Coverage Δ

#aFlag
6.81% <0%> (?)

Impacted Files
Coverage Δ

pkg/multicloud/aws/loadbalancerlistener.go
0% <0%> (ø)

pkg/multicloud/aws/instance.go
0% <0%> (ø)

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 5fc0b07...e06ebe8. Read the comment docs.

/lgtm

/approve


remove DOWNLOAD_CODES

suggest to remove the use of this environment option. if you want to build a code that needs to be downloaded, just download (if not already present...).

Yes, good idea.
I think this was introduced to prevent huge downloads happening too often in the time when we had to build all of AMUSE, so this was made obsolete by creating packages and the ability to make specific modules (make (module).code).

work in progress

wip


Issue 41 redux toolkit

develop
GitHub Board
Issue link
#41 Redux toolkit
Story link
#11 Story
Code reviewers

[ ] @github_username

Summary of issue

[ ]  Summary of issue
[ ]  Summary of issue

Summary of change

[x]  Update store & reducers to use Redux Toolkit
[x]  Create thunks for async operations
[x]  Update components appropriately

merged latest develop, resolved conflicts

CI is passing


Fixed 2 typos

Hi @bartbilliet, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

@asteen Please review these proposed changes. Thanks.


Add filter Thread Status

I would like to suggest an enhancement to our thread status functionality by adding a filter option for thread status. Currently, all processes are displayed without any filtering. However, I believe it would be beneficial to introduce a feature that allows users to selectively view specific processes based on their needs.
Why a Filter Option Matters:

Improved Usability: With a filter, users can uncheck irrelevant processes and focus on what matters most. This streamlines the trace view and reduces visual clutter.

Example:

This is a random example of what it should look like

Thanks for the interest in Trace Compass. Yes, it would be great to be able to filter (show/hide) rows to be shown. The tree and the timegraph on left are synchronized, that means one line on the tree corresponds to a line on the graph (left). Removing lines have to be done on the tree and on the graph. Enable the checkboxes on the left would allow do deselct rows, but there will needs to be a different control to get the hidden rows back. Maybe having a separate tree dialog (from the menu) where a checkbox tree exists and users can select and deselect threads (rows).
Interesting would also to be able to select multple rows on the tree, right mouse click and select a hide menu item. This is just a shortcut to hide items.

First of all, I want to thank you for your response and support for enhancing Trace Compass functionality. Your comment regarding the synchronization of the tree with changes on the graph is extremely important. Allowing users to focus only on essential information, rather than getting confused with unnecessary details, is crucial for improving the user experience.
Thank you once again for your engagement and support.

@bhufmann I am a student currently working on a student project that utilizes Trace Compass. I would like to add this functionality, but I'm encountering difficulties in following the instructions for starting the project from the README. Could you help me to start the project and provide the necessary support so that I can begin working on this functionality?

Hey @NikolaKalinic I want to help out too! :) First off, anything missing in the README?

Hey @MatthewKhouzam , thank you so much for expressing your willingness to help out! It's fantastic to have support as I dive into this project.
Regarding the README, everything seems clear, but I've encountered some difficulties in getting started. Here's where I stand: I've got Node.js version 18.20, Java 17, and Yarn 1.8 installed on my system. I've also cloned the repository and tried running
yarn tswatch

as instructed, but I seem to be facing some issues.

Do you have any insights or suggestions on how to troubleshoot this? Any help would be greatly appreciated!

Exactly, I only have one repository 'theia-trace-extension'. I thought I could fetch the other necessary components using commands like 'yarn download:server', but it seems that's not how it works the way I imagined. Can you tell me which repositories I need to have and where they should be located? Also, do I need to follow instructions for setting up each one or something else? Can you give me baby steps (rubber ducking) :joy: :joy:  ?

Ok, that helps! and Yes, I love the enthusiasm. ;)
Basically, yarn tswatch says "Hey, use the local libraries, don't donwload any.
If you're still at the begining, you have a few choices.

clean up your git... (git clean -fdx, git reset hard)
Erase everything restart with no yarn tswatch
the best, download the other modules like timeline-chart and link them.

Hope this helps.

I've successfully run the project! Thank you so much for your time and the invaluable assistance you've provided to help me through this. Your support mean a lot, and I truly appreciate everything you've done. I'm now ready to move forward with this feature! Once again, a big thank you!

Looking forward to seeing the progress!

Hey @MatthewKhouzam, I need small help again. I run application with yarn start:server and yarn start:browser, but now when I want to change small part of code like add some id on some element, my change is not appeared in DOM, but if I check that in Sources I can see that. Does I need to rebuild something or what?


FORMS-10259 - Custom properties - Refactor code to new design aligning to styles

Jira: https://jira.corp.adobe.com/browse/FORMS-10259

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes and the overall coverage did not decrease.
[ ] All unit tests pass on CircleCi.
[ ] I ran all tests locally and they pass.

@TalmizAhmed Please fix the test case failures

PR rebase GH action failed, can you rebase with base branch ?


Minor differences between this and TechNet Gallery

There are (somewhat) minor differences between the script here and the on TechNet Gallery. Which one is newer, and which one do you recommend using?

Hi @webash , this GitHub version is the latest and recommended version.

Your blog posts currently link to the TechNet gallery; so hence my confusion :)

...while I'm here - thanks massively for this basis. We've adopted it. I've made some changes to make the scripts a bit more resilient to failure which I can share back in case you're interested. Additionally it looks like the latest version of an Azure Function uses PS Core 6.1.2, which has some fundamental differences, which I've factored in. Watch out for a PR in the next few weeks.

Thanks and looking forward to it!


UI

Create a new document seen as you try to create a new document looks like a field that is supposed to be filled but is instead a locked field. From a UI perspective this should look different. May be have that region have a different color and maybe centered.

Thanks. I'll see what can be done about this.


Idea: include debug variant images, ko debug

⚠ I'm not sure this is a good idea, but it's something I thought of, and I'm curious for feedback.
Basing images on distroless/static is great (don't get me started), but it means folks who want to debug things won't have a shell if they want to kubectl exec in and poke around.
What if ko built two images for each platform, one based on distroless/static and one based on a (configurable) debug variant which includes a shell. ko would need to make sure to put the "real" image first in the manifest list, so container runtimes don't accidentally pick up the debug variant all the time, since both images would have the same platform value.
Then we could have some UX for swapping out the "real" variant with its debug counterpart, e.g., ko debug <pod>, where it would look into the Pod for any ko-built images, and update the podspec's images (which is allowed, surprisingly!). This unfortunately restarts the container with the new image, but it would restart it with a new debuggable image. ko debug could even wrap the kubectl exec part, and swap the image back when you're done. Mischief managed. 🪄
I also have no idea how this would work with Windows, so let's just ignore that.
Ephemeral containers are supposed to be the answer for this, but unfortunately they're still in beta so they're not available many places in the wild.
cc @jedsalazar who made me think about this

Any chance this can be reconsidered?


Removed region tags.

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

I know that these region tags are not used today. But why we can't keep them?

Repeating Iana’s message: These code samples can’t be used for documentation anyway. That’s why they are all located in separate folders. They can be used only for interactive tutorials, because sometimes we ask user to replace some pieces of code manually. Amanda asked us to remove them se we won’t confuse anybody with the purpose of these code samples

Repeating Iana’s message: These code samples can’t be used for documentation anyway. That’s why they are all located in separate folders. They can be used only for interactive tutorials, because sometimes we ask user to replace some pieces of code manually. Amanda asked us to remove them se we won’t confuse anybody with the purpose of these code samples

Discussed offline, that we're not gonna use the interactive tutorial samples in cloud doc, so SGTM to remove the tags.


test

test

Hi, I have some suggestions to help solving this issue!
Here is a list of similar issues:

1. [test test](https://github.com/fjsj/python-social-auth/issues/3)

2. [test](https://github.com/fjsj/python-social-auth/issues/1)

3. [test](https://github.com/fjsj/python-social-auth/issues/4)

4. [test](https://github.com/fjsj/python-social-auth/issues/9)

5. [test](https://github.com/fjsj/python-social-auth/issues/10)

Hope this is useful!
Cheers,
Cinnabot.
PS: Liked my suggestions? Install me in your repository! Visit: getcinnamon.io.

test

aaa


Text overflow is handled incorrectly for english text and cannot be modified

Right now, when text reaches the end of the line it will break immediately and without hyphen:

Setting css style of the Editor component sadly does not fix this, so it would need to be done internally. Tested using this class:
.hyphenate {
  word-wrap: break-word;
  overflow-wrap: break-word;

  -webkit-hyphens: auto;
  -moz-hyphens: auto;
  hyphens: auto;
}

Default break-all, keeps width of each line consistent. To override it:
.md-content .md-preview,
.md-content .md-html {
  word-break: break-word;
}

Thank you!


Endpoint appended by "prisma deploy" to prisma.yml is wrong

Describe the bug
When prisma.yml does not contain an endpoint. Command prisma deploy tried to append the endpoint to prisma.yml by asking the user for it. However, if we do not specify the service and stage, the generated URL should have defaults.
To Reproduce
Steps to reproduce the behavior:

prisma.yml

datamodel: `d.graphql`

A valid datamodel.graphql
Run prisma deploy and fill it with the following

prisma.yml will have endpoint: http://localhost:4466// because the defaults are missing
Running prisma deploy again works as defaults kick in but the appended URL is incorrect.

Expected behavior
The first append should be endpoint: http://localhost:4466/default/default
Screenshots
Done above
Versions (please complete the following information):

OS: OS X High Sierra
prisma CLI: prisma/1.10.2 (darwin-x64) node-v10.4.0
Prisma Server: 1.10

Additional context
NA

This was resolved earlier 🎉


Refactoring of go-dhl

Refactor of Express method was reviewed here: https://github.com/shipwallet/go-dhl/pull/1
Freight method: https://github.com/shipwallet/go-dhl/pull/2
Repo cleanup & update wasn't yet reviewed.

Changes applied. Thanks for such an insightful review(s). :)


Disclose render mode

Describe the problem
The performance characteristics of SPA vs SSR apps is very different. While many frameworks only support one or the other, SvelteKit can run in different modes and supports both. It quite hard to compare SvelteKit to an SSR-only framework or SPA-only framework because our numbers are grouped together despite these being totally different deployment modes. Being able to report on this would help us generate better reports at https://cwvtech.report/ to understand where users are succeeding at creating fast sites and where we can better guide them.
https://github.com/HTTPArchive/cwv-tech-report/issues/32#issuecomment-1892965582
Describe the proposed solution
One possibility would be to generate something like:
<meta name="generator" content="SvelteKit 3" />

And then mark feature usage for stuff like ssr|prerendered|spa|mpa and enhanced-img:
https://www.w3.org/TR/user-timing/#dfn-mark_feature_usage
This could be controlled by an option similar to Svelte's discloseVersion option.
Alternatives considered
No response
Importance
nice to have
Additional Information
No response

Possibly related: It'd be useful to see a tree of routes with the rendering method listed for each, to review for performance opportunities, likely as part of build output.


Add jQuery easing to navigation scroll

Use easeInOutExpo easing type

Why do we need this?

I just think it looks better, but we can talk it over... I just added a issue to remind myself about it.

What are your thoughts?

Most modern sites like Code for America, Microsoft, and Apple just use the browser's default scrolling feature. I'm not sure that this library really adds anything significant to the site to make it worth adding. For example, the toastr library added a notification system so the user knows if their message was successfully sent our not. This library just adds a scrolling animation, which to me isn't worth the extra KBs of code.

None of those websites have a single page scrolling functionality. I don't think you see what I'm trying to do. I'm not trying to hijack the ability to scroll the page, or use the mouse wheel. The functionality I'm suggesting is when you click on the navigation bar. Rather than have it jump to the section (like it was), I adding a scrolling animation (like sites like Jatex, and SLAMR). Adding jQuery easing will allow to change the scrolling animation from linear to a different curve.

Ok, then go for it. I'd just prefer to not add any more dependencies than we have to. Also while you're working on that what are your thoughts about adding Bootstrap scrollspy feature as well? It's already a dependency so nothing new would have to be added with that. We'd just need to add data-spy="scroll" to the opening body tag. We'd also need to add a data-target attribute as well I'm just not sure what we'd have to point it at for it to work.

scrollspy is cool, but it really do what I was talking about. Honestly, I think you're right. No reason to induce new dependencies without a huge need. I'll just leave the javascript animate property set as is, but not induce easing. (Honestly, that'll be better on us so we can call the website complete, and move on)


[fix][logstash] - Use configmap name instead to use annotations

To identify configmap, use name instead annotation of type config, piepline or pattern

It's ok


filter_shapenet files

Hi Lirui,
Thanks again for this awesome work. Is there any chance that the full shapenet training data can be made available? I assume these are the objects listed in experiments/object_index/filter_shapenet.json.
I tried following the README instructions for processing the shapes data myself but have run into various issues when trying to use OMG-Planner's python -m real_world.process_shape. For example, I'm having a little difficulty getting meshlabserver installed and running properly on my machine.

Hello, unfortunately I think Shapenet objects require licenses so we cannot publicly share. Regarding data processing (it has been a while), I recalled it only required a python interface to the meshlab.

Hi Lirui, thanks for the fast reply. I got meshlab working, though there are a few other issues I'm stuck on:
(1) I followed the acronym instructions to generate simplified meshes. I put each simplified mesh in the following format:
data
|----objects
|    |----Stapler_8d152be34b41785677937146265c551a
|    |    |    model_normalized.obj
|    |----[next object]
|    |    |    model_normalized.obj

After running python -m OMG.real_world.process_shape -a -f Stapler_8d152be34b41785677937146265c551a the size looks off. Maybe I missed an actual normalization step?

(2) Additionally, where do I put the acronym grasps? Acronym gives a folder of h5 files such as Xbox360_eabea01e420a96a3c55b18f95951c1e1_0.012159769145183007.h5. Do I put those in data/grasps/simulated/? The existing grasps there are in .npy format.

I think the scale should come from the acronym dataset "/object/scale" (and there should be a resizing step here). Could you try visualizing the grasps in acronym dataset with this code and this repo, and see if it looks good to you? Yes, the grasps should be in /simulated.

I added some scripts from my old codebase to the repo. Feel free to take a look at those and see if it is helpful.

Manually resizing the OBJ files using trimesh.apply_scale before process_shape (using the value /object/scale from acronym dataset) appears to work for most objects (though some of them like TV_dadf3f55c02c4dbe71357e42f9203ca5 don't seem to appear at all when I load them into the pybullet environment).
For the grasps, I can't seem to get expert planning to work (0% SR). It seems like expert planning uses this code which is for YCB objects. Does it work the same for acronym objects?
I'll try the visualization scripts you linked when I have some time in the next few days, thanks!

Some examples:

(In the last one the object doesn't seem to appear at all)

Yes the grasp loading code should also work for shapenet objects. It looks like the object is not given a stable pose before simulation. Probably would require some debugging in the loading object and env reseting code even without the robot or planner

That's helpful, I did some more debugging at reset time. It looks like it's still some sort of geometry or processing issue though. The object is dropped in a fairly stable pose, but when it hits the table the physics don't look right at all and it's center of mass seems off.

Right it seems that some of the physics parameters might be off. Maybe see if bullet provides some ways to recompute center of mass and inertia of objects after loading.


fix(router): check if there is certificates to generate

Debugging an error I found this output from the router:
/bin/generate-certs: line 21: syntax error near unexpected token `done'
/bin/generate-certs: line 21: `done < /etc/ssl/deis_certs'

Code LGTM.

Code LGTM. Good fix, @aledbf. This will necessitate a 1.6.1 release.


Modify Readme file and update packages

Update the Readme file with the screenshots of the projects.
Also, update the package/dependencies to the latest versions.

Successfully updated the README.md and pubspec.yaml


Component: Accordion

Need to add an Accordion component (thanks for the heads up @paaljoachim)!
We already have Collapsible and  Panel components that could help with this.
Collapsible is the primitive that handles the collapsible state + animations. Panel a rendered UI layout component that would be used in places like the sidebar (using Collapsible underneath).
I gave this one some thought 🤔
Panel gets us 90% of the way there. The issue is that it doesn't synchronize state across multiple Panel components. In other words, if there are 3, only 1 should be allowed to be open at a given time.
See this example:
https://github.com/WordPress/gutenberg/issues/21584#issuecomment-695775809
At the moment, individual Panel open and close as they please.
Component API
We could add a new component like this:
<Accordions>
    <Accordion>
        <AccordionHeader>...</AccordionHeader>
        ...
    </Accordion>
</Accordions>

Or!! Maybe we can leverage existing Collapsible and Panel components, and do something like this!
<Accordion>
    <Panel>
        ...
    </Panel>
    <Panel>
        ...
    </Panel>
</Accordion>

Any Collapsible or Panel component placed within Accordion can have their states synced in the "only 1 can be open" kind of manner.
This "one" or "any" behaviour can also be customized at the <Accordion /> level via props.

Will this in any way help the discussion going on here:
https://github.com/WordPress/gutenberg/issues/21584
The discussion going in JS on the frontend vs using details HTML tag that is not accessible. See the various bottom comments.


DAT failure: Url encoding issue

For the DAT tests multi_partitioned and multi_partitioned_2 In the duckdb delta extension I'm seeing doubly url encoded paths.
To reproduce, run DuckDB v1.0.0:
force install delta from core_nightly
from delta_scan('delta_kernel_rs/acceptance/tests/dat/out/reader_tests/generated/multi_partitioned/delta');

results in:
IO Error: Cannot open file "../../../acceptance/tests/dat/out/reader_tests/generated/multi_partitioned/delta/letter=%252F%252520%2525f/date=1970-01-01/data=hello/part-00000-a29a3f63-5a26-4127-aeff-e6d27c077917.c000.snappy.parquet":

Note that the letter column contains the partition value %252F%252520%2525f which should be %2F%2520%25f. This means that somewhere the value is urlencoded again where it shouldn't.
These paths are exactly what I get back in the callback passed to the ffi::visit_scan_data function, so I'm pretty sure the issue is kernel side?
Weirdly this is fine when I query the tables using delta-kernel-rs/kernel/examples/read-table-multi-threaded:
    Finished dev [unoptimized + debuginfo] target(s) in 0.12s
     Running `/Users/sam/Development/delta-kernel-rs/target/debug/read-table-multi-threaded -e sync ../../../acceptance/tests/dat/out/reader_tests/generated/multi_partitioned/delta`
Reading file:///Users/sam/Development/delta-kernel-rs/acceptance/tests/dat/out/reader_tests/generated/multi_partitioned/delta/
+--------+------------+------------+--------+
| letter | date       | data       | number |
+--------+------------+------------+--------+
| b      | 1970-01-01 | f09f9888   | 7      |
| /%20%f | 1970-01-01 | 68656c6c6f | 6      |
+--------+------------+------------+--------+

Yeah, so the issue is that kernel deals all in urls. So the if your code isn't expecting a URL, then you need to decode it.  %2F%2520%25f is the actual path name, while %252F%252520%2525f is the url encoded version. It's not been encoded "again", just once, it's just that the actual path looks encoded in this case.
So the file is at:

Url version: file:///path/to/acceptance/tests/dat/out/reader_tests/generated/multi_partitioned/delta/letter=%252F%252520%2525f/date=1970-01-01/data=hello/part-00000-0b6e3269-8890-4556-9dca-d4032ea9a952.c000.snappy.parquet
fs path version: /path/to/acceptance/tests/dat/out/reader_tests/generated/multi_partitioned/delta/letter=%2F%2520%25f/date=1970-01-01/data=hello/part-00000-0b6e3269-8890-4556-9dca-d4032ea9a952.c000.snappy.parquet

What makes this tricky is that we pass these as strings, because we can't pass a "url class" over ffi. If you have url decoding in duckdb, the right thing to do would be to create the url and then decode it. Also, from the error it looks like you're using the path that the user types in as the "table root". Once #203 finally merges I'd recommend using snapshot_table_root, which will give you the actual expended root url of the table. Then you can just parse the whole thing as a url.
If you want to use a file-path, I think it should just work to url-decode the value that comes in as the path to visit_callback.

ah yea that makes sense, okay I will just fix this DuckDB side by url decoding


Feat Container basic widgets

Basic Container components

DecoratedBox
LimitedBox
Shadow
ShapeBorder
etc...

Directional

Directional trait for different text direction dependency widget
Directionality for inherited TextDirection

Now some kurbo and frui geometry struct are mixed because of

kurbo lacks some convenient methods
Soon or later, we will have an abstract layer for the backend painting system.

Align with TextDirection::Rtl in ancestor Directionality


Merge branch-21.06 into branch-21.08

This resolves a merge conflict with branch-21.06.  The merge conflict was the same as with #2561 except that PR was squashed instead of merged, so the merge conflict remained.
NOTE: This PR must be commited with a merge commit, not squashed!

build


circleci init

@JimBTek if you think CircleCI will be better than Travis lets please press on with this PR. I only fixed the current one just to ensure the release was being tested (though i was also running tests locally) and also to update the build status on the README. ;-)


zed_components zed_camera_component.cpp will not compile

Preliminary Checks

[X] This issue is not a duplicate. Before opening a new issue, please search existing issues.
[X] This issue is not a question, feature request, or anything other than a bug report directly related to this project.

Description
I have ROS2 Humble on Ubuntu 22.04 and have installed the ZED_SDK 4.1. I have tried following the instructions you have listed in the README but have had no luck.
Steps to Reproduce

Setup ROS2 Humble environment
Setup ZED_SDK
Follow instructions on the README
There are no rosdep dependency issues

Expected Result
zed_components should successfully build
Actual Result
/home/roverteam/Desktop/roverSim/camera_proj/src/zed-ros2-wrapper/zed_components/src/zed_camera/src/zed_camera_component.cpp:9561:5: error: template argument 1 is invalid
9561 |     RCLCPP_WARN_STREAM(
|     ^~~~~~~~~~~~~~~~~~
/home/roverteam/Desktop/roverSim/camera_proj/src/zed-ros2-wrapper/zed_components/src/zed_camera/src/zed_camera_component.cpp:9561:5: error: ‘’ is not a template [-fpermissive]
9561 |     RCLCPP_WARN_STREAM(
|     ^~~~~~~~~~~~~~~~~~
/home/roverteam/Desktop/roverSim/camera_proj/src/zed-ros2-wrapper/zed_components/src/zed_camera/src/zed_camera_component.cpp:9561:5: error: template argument 1 is invalid
9561 |     RCLCPP_WARN_STREAM(
|     ^~~~~~~~~~~~~~~~~~
/home/roverteam/Desktop/roverSim/camera_proj/src/zed-ros2-wrapper/zed_components/src/zed_camera/src/zed_camera_component.cpp:9561:5: error: ‘’ is not a template [-fpermissive]
9561 |     RCLCPP_WARN_STREAM(
|     ^~~~~~~~~~~~~~~~~~
/home/roverteam/Desktop/roverSim/camera_proj/src/zed-ros2-wrapper/zed_components/src/zed_camera/src/zed_camera_component.cpp:9561:5: error: template argument 1 is invalid
9561 |     RCLCPP_WARN_STREAM(
|     ^~~~~~~~~~~~~~~~~~
/home/roverteam/Desktop/roverSim/camera_proj/src/zed-ros2-wrapper/zed_components/src/zed_camera/src/zed_camera_component.cpp:9564:26: error: ‘mLastClock’ was not declared in this scope
9564 |         static_cast(mLastClock.get_clock_type()));
|                          ^~~~~~~~~~
/home/roverteam/Desktop/roverSim/camera_proj/src/zed-ros2-wrapper/zed_components/src/zed_camera/src/zed_camera_component.cpp:9566:5: error: ‘mClockAvailable’ was not declared in this scope
9566 |     mClockAvailable = false;
|     ^~~~~~~~~~~~~~~
/home/roverteam/Desktop/roverSim/camera_proj/src/zed-ros2-wrapper/zed_components/src/zed_camera/src/zed_camera_component.cpp: At global scope:
/home/roverteam/Desktop/roverSim/camera_proj/src/zed-ros2-wrapper/zed_components/src/zed_camera/src/zed_camera_component.cpp:9571:1: error: expected declaration before ‘}’ token
9571 | }  // namespace stereolabs
| ^
gmake[2]: *** [CMakeFiles/zed_camera_component.dir/build.make:104: CMakeFiles/zed_camera_component.dir/src/zed_camera/src/zed_camera_component.cpp.o] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:164: CMakeFiles/zed_camera_component.dir/all] Error 2
gmake: *** [Makefile:146: all] Error 2
Failed   <<< zed_components [22.6s, exited with code 2]
There is a lot more, but I thought this was probably enough to understand the problem.
ZED Camera model
ZED2
Environment
OS: Ubuntu 22.04
ROS2: Humble
ZED SDK: Version 4.1

Anything else?
I have tried modifying the cpp program and using earlier branches, but I cannot figure out the issue. I was able to get it to work with  Jetson Nano running on ROS2_Eloquent using an earlier branch, so I think the issue is more likely with my setup.

The ZED ROS2 Wrapper repository does not yet support the new ZED SDK v4.1.
We plan to release the new version at the beginning of the next week.
Meanwhile, you can use the ZED SDK v4.0.8:
https://www.stereolabs.com/developers/release/4.0

Hello, When are you releasing the support for ZED SDK v4.1 ?
thanks


2.6.0 package is broken and can't be loaded

version.rb tries to load gem_version.rb which is not part of the package.
This results with
`version.rb:1:in `require_relative': cannot load such file -- .../the_sortable_tree-2.6.0/gem_version (LoadError)`

@bblimke oh, I have to check it. Sorry.

@bblimke please try
gem 'the_sortable_tree', '2.6.1'

thank you for report!


Experiment with attrs class to define CSV outputs

As my mind starting wandering toward the future challenge of standardization, I realized that, at some point, we're going to need to provide a mapping between the raw fields provided by our sources and the common clean fields of our consolidated file.
While that mapping is beyond the scope of this repository, which should stay limited to scraping, I had a thought. What if we provide a simple, loose schema for our source files within the scrapers?
This patch contains a test implementation using the attrs spin on dataclasses.
It has the following pros:

We would explicitly document each source's headers in our code, making each state easier for a newbie to grok.
When the time comes to merge all the sources, we won't have to crack open CSV files to see what the source provides.
The attrs system is loose and doesn't require any strict typing or changes to the raw data. We can flow it through without changing anything.
We can standardize how we write out the CSV files using a utility like the one I've drafted here
We can stop torturing ourselves by writing special case code to pluck out headers. Instead, we can define the expected headers in the class and then just skip header rows in the scrapes, as you see me do here.

Here are the cons I can think of:

It's more code to write
If there are cases where a source provides multiple files with different headers, I'm not sure what we'd do with this approach.

That's my musing. I'd love your take on this @zstumgoren. And @chriszs, I bet you'll have an opinion too. What do you think?

@palewire Standardization is def the next big challenge and I'm open to any ideas that would nudge us that way. I do share your concerns about complexity-level of coding (i.e. raising the bar to entry for contributors, especially students who regularly work with Big Local). And I also wonder what it would look like for scenarios where data structure/headers evolve over time for a single state/source. For this project, I had assumed we'd have to do the usual slog of mapping non-standard structures/field names to a canonical set of names (or some subset for places that don't capture all the relevant data, e.g. company address). Probably need to talk through it while looking over your shoulder on Zoom to make a definitive decision, but based on cursory review of branch code and attrs docks, I'd say I'm 50/50 either way.

Illinois definitely changes headers between years. I do like the idea of standardizing column names. When we did a similar project (with many collected by hand), we saved that for last, read all the CSVs into a giant nightmare data frame, and then did our column merging there based on the handful that were relevant. I think in our case, saving that until after collection was the right call. But I think the CSVs themselves are probably the primary source for what column names analysts will try to start with.

giant nightmare dataframe was enough to trigger me

I found a way to solve this problem downstream with marshmallow so I'm going to withdraw my idea as beyond the scope and needs of this stage of the pipeline.


20220205 issue18 cmake

Summery of changes:

Comments added to /conf/cmake/ configuration files.
Some improvements in and replacement in cmake configs.
New file - pre-project.cmake - added for some settings which must be added before project().

In this phase, I must say some problems might be encountered during cross-compiling for specific scenarios, though I'm trying to solve them.
Open to any modification or suggestion :)

@aregtech and @biwiki please  review new modification.
Is this what did you have in mind?

@aregtech and @biwiki please review new modification. Is this what you had in mind?

Nearly perfect :-)
I tried to compile mcrouter with shared lib option, and the shared lib is outputed to:
areg-sdk/product/build/OFF/UNIX-x86_64-Release/lib/libareg.so
Please check if that is the expected path.

library output defined in conf/user.conf which indicates:
set(ProjBuildPath "build/${CrossCompile}${Toolset}/${OpSystem}-${Platform}-${Config}")
set(ProLibDir "${ProjOutputDir}/lib")

which I believe is same as Makefile's output, is that correct?
Though I have modified Toolset to have corresponded value as compiler which is little bit different from Makefiles.

library output defined in conf/user.conf which indicates:
set(ProjBuildPath "build/${CrossCompile}${Toolset}/${OpSystem}-${Platform}-${Config}")
set(ProLibDir "${ProjOutputDir}/lib")

which I believe is same as Makefile's output, is that correct?
Though I have modified Toolset to have corresponded value as compiler which is little bit different from Makefiles.
Output executables changed to mcrouter${AREG_BING}.

By using Makefile, the shared library path is areg-sdk/product/build/g++/linux-x86_64-release/bin/libareg.so
The shared library should be placed in 'bin' if I could remember correctly. For Windows, it is expected that the executable and library are in the same directory ...
And should mcrouter be placed in 'bin' as well?

I assumed based on /conf/makefile/user.md is should be like that.
# The project static library directory
ProjLibDir      := $(ProjOutputDir)/lib
# The project binary output directory
ProjBinDir      := $(ProjOutputDir)/bin

Nevertheless, I've changed it to bin, however for executable output couple issues are in the way.
So I suggest we move on for now and after refining configs structure we get back to it.

Specifically, due to some issues with commands that must be defined before some else, keeping current Makefile structure is tricky.

@Ali-Nasrolahi ,
Jus want to inform the logic of path structure:

all outputs should be in the folder product. This folder right now contains only 1 subfolder build. Later, it will contain subfolder gen (or generated) for all generated files. Probably proj for all project files (done for IDE)
The build subfolder contains other subfolder(s) with information about HW architect, compiler, OS and the build information (release / debug). Right now I don't remember by heart the sequence. The best would be to compile with make and see the outputs.
Finally, all object files are output in obj subfolder. All static libraries are output in lib subfolder. In Windows, when compile as a shared library (DLL), it generates areg.lib and areg.dll files. The areg.lib is located in the lib subfolder. And the bin subfolder contains all shared libraries and executables, as well as configuration files. In the same example of Windows, the areg.dll goes to bin. Under Linux it is straight forward --  areg.a is in the lib and areg.so in the bin.

The purpose is to simplify compiler / linker options:

add product/gen in the include compiler option;
add xxx/lib in the include of linker option;
collect all binaries / executables in the xxx/bin.

Could you please let me know whether your changes have these structure of file paths?

Thank you @biwiki. That actually clarified some important spots.
Currently, Cmake output does not fully structured on what you have explained, but shortly will  be.
Again the reason is, I tried to replicate Makefile structure, so it'll be easier for you to maintain later on. However as I mention those issues, I had to change it a little - that is exactly why I added pre-project.cmake.
Please let me know if any other structure has conflict with what you have in mind so I work on them and the new structure becomes as close as possible to your standard. (Hopefully in close future PRs)

Gents,
This is not a comment, but an improvement to consider.
CMake allows us to put all its outputs to a build directory. Please take a look at this chapter for source vs binary directories
https://cmake.org/cmake/help/book/mastering-cmake/chapter/Getting Started.html#directory-structure
Having said so, all the binaries including the library should go to CMake binary directory.
Suppose that someone tries to compile areg-sdk with cmake like:
$ mkdir build
$ cd build
$ cmake ../
$ make -j 8
Then build directory will become CMake binary directory ....
The great benefit of this is to allow parallel compilation of the source base, here we can think about compiling all the projects with all different supported compilers/options in parallel on a powerful server.
autotools has similar feature like that ...
areg-sdk/product/build is a common path in the source directory, and using it for the build outputs may break parallel builds in the above scenario.
PS: I was looking for libareg.so in the CMake binary directory :cry:

@biwiki ,
i didn't get really what is the exact problem. Maybe I'm missing something.

areg-sdk/product/build is not a common path for the sources, it is common path for the binaries.
The common path for the sources are are-sdk/gen, which will contain generate files, split by subfolders. For example, the file located in my-project/some/service/MyService.siml will generate files in the folder areg-sdk/product/gen/my-project/some/service/...
Depending on compiler, HW, OS and build option we have path separation like areg-sdk/product/build/linux-x86-gcc-debug/xxx or areg-sdk/product/build/win32-x86-vc142-debug/xxx, where each compiles makes own outputs in the xxx

Please correct me if I understood wrong your question.

@biwiki and @Ali-Nasrolahi,
Gents, what do you think to have regular video call?
Let's organize regular community meetings.
There are some projects that use the engine, maybe they would be also interested to join the community.

@biwiki and @areg,
By removing/commenting this line from framework/CMakelists.txt libareg will be included in binary directory.
set_property(TARGET areg PROPERTY LIBRARY_OUTPUT_DIRECTORY ${ProBinDir})

I must discuss the fact that we do not need to restructure the outputs directoriey if the only feature your looking for is multi-threaded compiling.
I've tried this method in areg root directory.
$ cmake CMakeLists.txt -B build/ -D CMAKE_RUNTIME_OUTPUT_DIRECTORY="../product/build"
$ cd build
$ make -j 8

By using CMAKE_RUNTIME_OUTPUT_DIRECTORY, we specify binray output directory - in our case mcrouter.out.
It works nicely.
Although reconsidering the structure would not be necessarily an unpleasant idea.(i.e. maybe some features you needed that wouldn't be convenient to implement with bare Makefiles.)
In any case, I'm open to any idea or reconsideration.

@Ali-Nasrolahi and @biwiki
Guys, let's have a video call and discuss these things there. If needed, we can share the screen for better understanding.

I have no problem with video call, but due to time differences I assume my free period will be imconvient for you.
Though if you believe my presence is beneficial and necessary , then we'll do something about it.
It's your decision @aregtech.

Yeah, this should be up to @aregtech to consider and decide :smiley:
And it is about parallel builds, not the multi-thread build with -j
Let's assume that an user decides to make use of the library and build their continuous integration pipelines like:
Build    #1                     #2                             ...
         <setup binary dir>     <setup binary dir>
         make -j 8              make areg=shared -j 8
         make test              make test
         <packaging>            <packaging>
         make clean             make clean

As the current design, both build #1 and #2 share the same output directory under areg-sdk/product, they can both write to the same file at the same time?
And what would the user do at , if they clean up everything at areg-sdk/product then that would cause a big issue since files are being used by other pipelines.
 is supposed to remove every output from the build, and if they have to remove individual files then they have to know exactly which files are generated from the build.
By using modern build systems, normally people keep all their outputs in the binary directly as stated in the document.

Sorry, I must have understood your comment in wrong way. I agree that's different.
P. S. guys could add me to your discord server so we could continue this discussion there.

Currently, make clean removes the complete product/build folder. True that it will cause problem and true that it should only remove the "own" binaries. I would appreciate very much if you fix this issue. But also not a big problem right now.
About video call / regular meeting, we are already a community. If we want to established and grow, we should have regular meetings for various topics. I'll move this discussion offline.


module must have its parameters and buffers on device cuda:0 (device_ids[0]) but found one of them on device: cuda:1

I run the code on cuda:1.
But when the code executes the "calc_clean_fid(net, e)", the code will report the "RuntimeError: module must have its parameters and buffers on device cuda:0 (device_ids[0]) but found one of them on device: cuda:1"

Thank you for reporting the error.
Can you give me a more detailed location on the implementation code where the error occurred?

compute FID of a model with mnist_test-1024 statistics
FID model:   0%|                                                                                | 0/20 [00:00<?, ?it/s]
Traceback (most recent call last):
File "main_fsvae.py", line 335, in
calc_clean_fid(net, e)
File "main_fsvae.py", line 206, in calc_clean_fid
fid_score = clean_fid.get_clean_fid_score(network, glv.network_config['dataset'], init_device, num_gen)
File "/home/wangbo/codes/FullySpikingVAE/metrics/clean_fid.py", line 28, in get_clean_fid_score
score = fid.compute_fid(gen=sample_from_vae, dataset_name=dataset_name,
File "/home/wangbo/anaconda3/envs/pytorch/lib/python3.8/site-packages/cleanfid/fid.py", line 525, in compute_fid
score = fid_model(gen, dataset_name, dataset_res, dataset_split,
File "/home/wangbo/anaconda3/envs/pytorch/lib/python3.8/site-packages/cleanfid/fid.py", line 251, in fid_model
np_feats = get_model_features(G, model, mode=mode,
File "/home/wangbo/anaconda3/envs/pytorch/lib/python3.8/site-packages/cleanfid/fid.py", line 228, in get_model_features
feat = get_batch_features(resized_batch, model, device)
File "/home/wangbo/anaconda3/envs/pytorch/lib/python3.8/site-packages/cleanfid/fid.py", line 88, in get_batch_features
feat = model(batch.to(device))
File "/home/wangbo/anaconda3/envs/pytorch/lib/python3.8/site-packages/cleanfid/features.py", line 25, in model_fn
def model_fn(x): return model(x)
File "/home/wangbo/anaconda3/envs/pytorch/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/home/wangbo/anaconda3/envs/pytorch/lib/python3.8/site-packages/torch/nn/parallel/data_parallel.py", line 157, in forward
raise RuntimeError("module must have its parameters and buffers "
RuntimeError: module must have its parameters and buffers on device cuda:0 (device_ids[0]) but found one of them on device: cuda:1

Do you solve the problem？
I met the same problem as you on cuda：9

Do you solve the problem？ I met the same problem as you on cuda：9

Sorry, I didn't solve it. What about you?


Add new events to manage Delete and Rename

Add new event on RenameController
Add new event on DeleteController

Allow users to manage listeners on rename and delete files (for SQL
manipulation for example : it’s my case).

Thank you @ejeanneaubc !


Deserialize enum on from same level other struct fields.

I have enum that used in lots structs:
#[derive(Debug, Deserialize)]
#[serde(rename_all = "lowercase")]
pub enum Ident {
    Name(String),
    Pattern(String),
}

#[derive(Debug, Deserialize)]
#[serde(deny_unknown_fields)]
pub struct Func {
    ident: Ident,
    typ: String,
    /*
    some other fields
    */
}

I where way to deserizalize struct Func from string where "name" or "pattern" data is on same level as "typ" and other Func fields? Ex. for JSON from r#"{"name":"func1", "typ":"str"}"# or r#"{"pattern":".+", "typ":"str"}"# ?
I created repo with tests  for this problem.

Yes I would like to support this! We are tracking this pattern in #119. It may look something like:
#[derive(Deserialize)]
struct Func {
    #[serde(flatten)]
    ident: Ident,
    typ: String,
}

Thanks for pointing that issue


Cards property not being populated

When I make a payment I am getting the following error in stripe-customer.js, line number 61.
var card = customer.cards.data[0];
TypeError: Cannot read property 'data' of undefined

thanks! I'll check this out

It works fine now that I switched to customer.sources instead:
var card = customer.sources.data[0];

Ah. I see. There were major api changes: https://stripe.com/docs/upgrades#2015-02-18
I'll create a pr to update. Thanks again for reporting this!

@cpustar can you explain a little more how you resolved it?

Thanks for working on this project - it's been a nice starting point. Any progress on this issue though? I just ran into it myself.

@benrlodge this should be fixed with #14

Thanks for the update. I ran into another error on the change - I'll comment on the PR.


Creating pages from frontend

I'm trying to utilise the Kirby API to create pages using the front-end of Vue (with a form) — with the exact same purpose as described int this Cookbook recipe: https://getkirby.com/docs/cookbook/forms/creating-pages-from-frontend
Any guidance on how this is done via your Vue starterkit?

Since the default Kirby templates are replaced by the Vue frontend in this starterkit, this cookbook is not applicable as-is.
You could instead create a custom API route, post data to it and then parse everything from within your custom route plugin.


Wildcard serialize doesn't apply to getter properties

http://jsbin.com/cefaza/3/edit?js,console
Getter properties don't get serialized when serialize is set ike this:
'*': {
  serialize: true
}

Does it work if you serialize it explicitly?

Yup. That's shown in the demo too.

Closing for: https://github.com/canjs/can-map-define/issues/4


Use original menu sound effects

I really like the concept of this app, but I miss the original menu interaction sounds
https://youtu.be/ChkgsU7HMZI

That's actually a good idea. Should be possible.

+1


Retrieving intrinsic parameters after post-processing

| Camera Model                       | { D400 } |
| Operating System & Version |   {Linux 18.04} |
| SDK Version                            |  { legacy / 2. }                          |
| Language                            |  {python}                          |
Hello,
In the documentation, after post-processing the frames with 'decimation filter', the intrinsic parameters will be recalculated.
I apply the filters to the frame however 'get_intrinsic()' function only works for the rs2::video_stream_profile object. How could I retrieve these new intrinsic parameters after post-processing?

Hi @shgold
Python 2.7.14 (v2.7.14:84471935ed, Sep 16 2017, 20:19:30) [MSC v.1500 32 bit (Intel)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import pyrealsense2 as rs2
>>> d = rs2.decimation_filter()
>>> p = rs2.pipeline()
>>> p.start()
<pyrealsense2.pyrealsense2.pipeline_profile object at 0x008DEEE0>
>>> fs = p.wait_for_frames()
>>> df = fs.get_depth_frame()
>>> processed = d.process(df)
>>> prof = processed.get_profile()
>>> video_prof = prof.as_video_stream_profile()
>>> intr = video_prof.get_intrinsics()
>>> intr
width: 640, height: 360, ppx: 316.603, ppy: 181.593, fx: 322.773, fy: 322.773, model: Brown Conrady, coeffs: [0, 0, 0, 0, 0]

Thanks! It works!


add 2 test skins

So exciting!
I was imagining these being a bit more minimal. Basically just what's required to send the success messages on startup.
It actually gave me another question: Would it be better to have a bunch of small skins that each act as a test of one specific assertion (or type of assertion) or would it be better to have a single skin who's XML acts as a test harness and then we can load arbitrarily many maki scripts into that skin, adding XML as needed.

I didn't really intend on making it so we guaranteed the same XML harness for every test. I think it is an interesting idea, but like you said, will probably end up getting unwieldy as we start trying to test more things.
I see how the tests could be more basic, but also wanted to have enough there that it would be easy to start making a bunch more examples in the future.  Since we already know we can render this basic skin (it actually is a somewhat stripped down version of the simple skin), hopefully its enough to get started on the testbed? We can always make simpler (and more complicated) tests :D

added as part of #881


InvalidSignature on latest testnet 0.29.1

Is there anything that can be done to fix sign process, latest testnet version 0.29.1 report this error with previosly working code:
Maybe it's relted to blake2b changes, not sure.
    "error": {
        "code": -32000,
        "message": "Invalid user signature: InvalidSignature { error: \"Invalid signature was given to the function\" }."
    }

The same
code below
privKeyBytes, err := hex.DecodeString("xxxxxx")
	if err != nil {
		return "", err
	}
	privateKey := ed25519.NewKeyFromSeed(privKeyBytes)
	resp, err := cli.ExecuteSignedTransaction(
		context.Background(),
		*txSinged.SignSerializedSigWith(privateKey),
		&types.SuiTransactionBlockResponseOptions{ShowInput: true, ShowEffects: true, ShowEvents: true, ShowObjectChanges: true, ShowBalanceChanges: true},
		types.TxnRequestTypeWaitForLocalExecution)
	if err != nil {
		return "", err
	}

Please use the v2 version and try it again


[Workers] Runtime API nav revision

BEFORE COMMENTING ON THIS PR, refer to PCX-6575 for a report on industry comparison, design principles and user research for this change.
Related to #9592

merge conflict resolution by #9861


carddav object can't be called

Error
JSX> j.clients.carddav
Thu 13 12:24:06 dav/CarddavClient.py -  46 - <module>                           : EXCEPTION:
    ModuleNotFoundError("No module named 'lxml'",)
--TRACEBACK------------------
<stdin> in <module>
    1
/sandbox/lib/jumpscale/jumpscale_generated.py in carddav
    1068    from JumpscaleLibs.clients.carddav.CarddavFactroy import CarddavFactory
/sandbox/lib/jumpscale/JumpscaleLibs/clients/carddav/CarddavFactroy.py in <module>
    3    from .CarddavClient import CarddavClient
/sandbox/lib/jumpscale/JumpscaleLibs/clients/carddav/CarddavClient.py in <module>
    46    import lxml.etree as ET
-----------------------------

No module named 'lxml'

should use elementtree for that (whoever going to work on that)


Add an example of object segmentation (ClipSeg)

What does this PR do?
The ClipSeg model has been validated using graph mode on Gaudi 2. Adding this as an example for users.
Performance number:
Without graph mode

n_iterations: 20
Total latency (ms):<PHONE_NUMBER>802734
Average latency (ms): 33.50095748901367

With graph mode

n_iterations: 20
Total latency (ms): 95.40128707885742
Average latency (ms): 4.770064353942871

Fixes # (issue)
N.A.
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

@regisss
Please take a look at this one. This is a validated model and this is an example script for users.

@cfgfung I see that you have a couple of more PRs which has the same change.
https://github.com/huggingface/optimum-habana/pull/814/
https://github.com/huggingface/optimum-habana/pull/826
Which is the valid one? and can you please close any not valid PR?

@cfgfung I see that you have a couple of more PRs which has the same change. #814 #826
Which is the valid one? and can you please close any not valid PR?

@yeonsily
They are not the same. Each PR is for a dedicated model.  Each model requires a different output and dedicated preprocessing /postprocessing codes. They only share a similar start script.

@cfgfung Your result looks good. Could you also port relevant test case for your model and make sure those are passing?
https://github.com/huggingface/transformers/blob/main/tests/models/clipseg to https://github.com/huggingface/optimum-habana/tree/main/tests/transformers/tests/models

@cfgfung Your result looks good. Could you also port relevant test case for your model and make sure those are passing? https://github.com/huggingface/transformers/blob/main/tests/models/clipseg to https://github.com/huggingface/optimum-habana/tree/main/tests/transformers/tests/models

Actually, above model test cases are optional, it's good to check but not necessary.
For the test cases for this ClipSeg model, please add similar test cases as your other PR(SAM).

@cfgfung Your result looks good. Could you also port relevant test case for your model and make sure those are passing? https://github.com/huggingface/transformers/blob/main/tests/models/clipseg to https://github.com/huggingface/optimum-habana/tree/main/tests/transformers/tests/models

Actually, above model test cases are optional, it's good to check but not necessary. For the test cases for this ClipSeg model, please add similar test cases as your other PR(SAM).

Hi, I have added the test and followed the same format as the PR of SAM.
Ran 'make style' and the unit test. Also aligned the folder structure and the readme.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@cfgfung  all looks good. Could you please make sure that you ran "make style" and upload the fix? I still see many style error.

@cfgfung all looks good. Could you please make sure that you ran "make style" and upload the fix? I still see many style error.

Hi,
There might be some updates in the rules and hence triggered the new errors.
I have re-ran "make style" and the result is listed below.


Able to add candidates with same email

Also, if candidate Id is found to be duplicate, there is no alert. Consider adding let's say a snackbar. Consider using material-ui.

Currently, we're just keeping the IDs unique.
@19BCS1114 What you think? Should we allow the cases where two candidates can have the same email or fix to check for duplicates?

always room for improvement @abhpd

@sarthakmalik0810 Okay so we will not have duplicate emails and IDs. it should give an alert if a user is trying to add a duplicate entry.

We already have a check in place for unique IDs here: https://github.com/abhpd-team/OEP-Frontend/blob/main/src/components/classes/class.jsx#L87
We just need to check for email as well.


plurality of formats

Hi Adrian.. Thanks your 192-foil Best Hits a repast from reality for an hour or so..
Pedantic issues & Impressions:
Is this big enough BigData? training an LLM I thought requires many Gigs of fodder..
the many different formats also might confound;
...unless there's an LLM with polyglot format conversion capabilities at par or better than SearchIt of yore..
beyond just extracting text from PPTs and PDFs,
the visually aphoristic style of presos, the  missing duality of dialogue seems confounding of 'sensible' machine interpretations.
endless non-auto-wrapped lines in .txt of Medium posts make HiTL browsing of .txt files rather too challenging.

I downloaded pdfs, and .pptx's to view them, which github apparently doesn't grok -
Guessing one could view the .txt files in a browser or autowrap in textedit.
all as far as 'issues'; Hope you are well.
  This adventure motivated me also to sign up for Mastadon.. lots of cats.

Interesting I harbor a similar objective, training -something- on thousands of handwritten 4x6 notecards.
The less-than chatBot objective: simple OCR of my own handwriting and fulltext search/retrieval ;
maybe a tag Cloud Zettelkasten.
Hundreds of topics I'l like to bounce off you :^)   guess those will have to wait

Thanks for the input. The way RAG works is that it takes lots of diverse kinds of input, but not many gigs of it, and indexes it for retrieval. I figure if I make a nice clean well documented data set, the LLMs will figure out how to use it eventually.


Worker nodes cannot be scaled down "using the slider"

So - how can they be scaled down? Presumably if you're being specific about the slider then there is a way?

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

ID: 54ee9b3e-0838-5cde-b644-ba2e2b9fdb2a
Version Independent ID: b3003462-e2f3-a9fa-72b1-419391833ed3
Content: Scale server group - Hyperscale (Citus) - Azure Database for PostgreSQL
Content Source: articles/postgresql/howto-hyperscale-scale-grow.md
Service: postgresql
Sub-service: hyperscale-citus
GitHub Login: @jonels-msft
Microsoft Alias: jonels

@robertlagrant Thanks for the feedback! We are currently investigating and will update you shortly.

I confirmed that the feature of decreasing the number of Hyperscale (Citus) worker nodes is not available through the Azure portal. In exceptional situations, we could do this via a support request. Normally compute and storage scaling is infrequent, and scaling nodes is even less frequent.
If you would like to discuss specific needs for your workload, please feel free to reach out to our team at Ask Azure DB for PostgreSQL<EMAIL_ADDRESS>


additional package managers to PackageIsNotLatest and a test for that rule

More OS and language package managers for the rule and a test for the rule.
list of ansible package modules
I don't think the new test was added to tox based on the TravisCI output. What do I need to do for that?

The tests are definitely showing in the Travis output:
test_command_changes_negative (test.TestPackageIsNotLatest.TestPackageIsNotLatestRule) ... ok
test_command_changes_positive (test.TestPackageIsNotLatest.TestPackageIsNotLatestRule) ... ok

although that does show that the test method names should be changed to something like test_package_not_latest_negative

New push.

New push.

soft ping

Merged, thanks (and nice reminder :) )

Updated version to 3.2.0 - applied tag and pushed to pypi.