Use calcfunction for getitem in For loop workgraph

To track the provenance in the for loop we now use a calcfunction to track provenance for the For loop workgraph. I would like to allow more types than integers for the sequence, but I am not sure how to generically test for types that are supported by aiida core. Some orm types have a _type class attribute, but not all of them.
TODO:

[ ] add notebook reason why to choose this over a normal for loop (less tasks created), or at least move this to an issue

Hi @agoscinski , thanks for the work.
There is a function orm.to_aiida_type which can serialize the data automatically, and you can check which Python data types it support.
It's also would be good if the user can supply a list of AiiDA data as the input for sequence, e.g.,
[structure0, structure1, ...]

Because AiiDA does not support a list of AiiDA data as input, internally, we can change it to a dict with keys, the keys keep the order information.
{"sequence_0": structure0, ...}

and then, in the WorkGraph engine, if the sequence is a dict, we get the AiiDA data directly without serialization.

Thanks @superstar54 applied your suggestion. The test pass, but it does not test the other use cases you mentioned. I tried to implement a dictionary, but it is a bit opaque to me how to change the iterator to be compatible with dictionaries that use hashable keys. It seems the iterator is hard coded to only work for collections that can be accessed by incremental increasing numbers like lists.  Maybe we should generalize this for arbitrary generators. Not sure if this PR is the right place.

Codecov Report
Attention: Patch coverage is 87.09677% with 4 lines in your changes missing coverage. Please review.

Project coverage is 80.76%. Comparing base (5937b88) to head (d9f611f).
Report is 76 commits behind head on main.

Files with missing lines
Patch %
Lines

aiida_workgraph/workgraph.py
76.47%
4 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #283      +/-   ##
==========================================
+ Coverage   75.75%   80.76%   +5.00%
==========================================
  Files          70       66       -4
  Lines        4615     5151     +536
==========================================
+ Hits         3496     4160     +664
+ Misses       1119      991     -128

Flag
Coverage Δ

python-3.11
80.68% <87.09%> (+5.01%)
:arrow_up:

python-3.9
80.71% <87.09%> (+4.98%)
:arrow_up:

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

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

As mentioned in my previous comment, if you use a list of orm.data as the sequence, it will raise an error, because we need to change the list to a dict.

That is for non json serializable data types like orm.StructureData for others it will automatically convert it and is not a problem. My problem with the dynamic workaround is that in this case it is shown differently in the provenance graph and also will create a lot of links.
That was the initial idea
@calcfunction
def __getitem__(seq, idx):
    return orm.to_aiida_type(seq[idx.value])

node = __getitem__([1,2,3], 0)
generate_node_graph(node.pk)

but the dynamic input ports approach gives
import numpy as np

@calcfunctione
def __getitem__(**kwargs):
    idx = kwargs.pop('idx')
    list_of_orms = list(kwargs.values())
    return list_of_orms[idx.value].clone()

inp = {f'list_of_orms_{i}': orm.StructureData(cell=np.random.rand(3,3)) for i in range(3)}
inp['idx'] = orm.Int(0)
node = __getitem__(**inp)
generate_node_graph(node.pk)

There is also the new StructureData object that might be json serializable because of the BaseModel that resolves this issue. I am also concerned by performance, since we now have to convert a Generator to a list which can be quite time consuming. I feel more and more that we should drop this PR. With the For workgraph we can maybe go more into a similar direction as with the while loop and just focus and a nice GUI representation.


Issues with high DPI - control alignment order

The controls at the top are aligned out of proper order when using High DPI.

This is still an issue. I contained those inside their own panel, with the label aligned to the left, button to the right, and edit is client. Yet the label still scoots over.

Well, the filter itself is fixed. But now there's new issues when changing system font size...

Fixed this not necessarily by handling DPI change messages, but rather by changing how these particular controls are positioned. They're no longer "aligned" but instead have absolute position within the panel - in exception of the sort and filter options, which have their own alignment self-contained.


[Question] Linux GUI

Are there any plans to release a linux GUI or a multiplatform (webbased or something) GUI?
Thanks

I'm also curious about this. I want to use optick to profile my code, but not really sure how I can visually check the results on my linux machine

What about Wine or Proton? Did anyone had success?
I used Bottles in pop_os! I had to install wine-mono as dependency, but then it crashes.


How to use websocket request with private key ?

I applied for an Ed25519 API key, but I don't know how to use this API key for websocket requests.

Hello @zhangk, you have several examples in the docs, but basically you just need to do:

api_key = "TwEW0XXXXXXXXXXXXXXXXXXXXXXXXXX"  # your api_key here
privateKey = "-----BEGIN PRIVATE KEY-----\nMXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX\n-----END PRIVATE KEY-----"

client = AsyncClient(api_key, private_key=privateKey)


Add new relic to pomegranate

Coverage remained the same at 100.0% when pulling c87eb84bfdf5e04974c653150ffb2e20d9b4727f on new_relic into 449cb9ef0c517645dce00cc30ea4f797041a5e9d on master.

:+1:


[bug-398]: Support storage system names/IDs with capital letters

Description
viper returns configuration values in lowercase even if the actual value has uppercase characters so we need to read the storage-systems.yaml file directly and extract the contents. Also removed the return statements for each storage type update so if PowerFlex systems update fails (for example), PowerMax and PowerScale will still be attempted.
Changed the default LOG_LEVEL of the RPM installation to debug.
Added missing error handling when asking OPA for a decision.
Changed some error responses to lowercase to be consistent.
GitHub Issues
List the GitHub issues impacted by this PR:

GitHub Issue #

https://github.com/dell/csm/issues/398

Checklist:

[x] I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
[x] I have verified that new and existing unit tests pass locally with my changes
[ ] I have not allowed coverage numbers to degenerate
[ ] I have maintained at least 90% code coverage
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Backward compatibility is not broken

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration
Unit tests for reading the configuration file. Manually verified that connecting to a PowerScale cluster with capital letters in the cluster name works with both deployment paths.

run e2e test


SEOImages: User Story 1463880, Part 31

Summary
Updating image file names and references to improve SEO.
Fixes #1463880

@mairaw This PR is ready for review. CATS report: https://contentqacats.azurewebsites.net/CheckResults/2019032000095/Report/
There are 9 image names identified by CATS as being too short. They should be handled at a later time as they are not part of the SEO image without hyphen work.

@mairaw I've made the changes. New CATS: https://contentqacats.azurewebsites.net/CheckResults/2019040200028/Report/


Result Reproducibility

Hi, @edpizzi congrats on your work!
I am trying to reproduce the results from your paper, and I checked that the code provided in this repo will have different results from the paper.
Would it be possible to provide the results of the exact training code from this repo on the DISC dataset?
If possible, could you provide these results using the same parameter described in the paper:
--entropy_weight=30 --augmentations=ADVANCED --mixup=true
Kind regards,
João Phillipe

Thanks for your question and for your interest in our work @phillipecardenuto!
We expect some amount of deviation from paper results. We performed the paper experiments on an internal codebase, and this repository is a reimplementation. The only difference we know of are different augmentation files (eg. using freely available fonts in this version).
Looking at my notes from before we released this codebase, I have the following results recorded:
This OSS codebase:
micro AP: 62.3
micro AP SN: 72.9
The corresponding paper results:
micro AP: 61.5
micro AP SN: 72.5
Are you seeing modest deviations in accuracy, or more significant changes?

Hi @edpreezi,
Sorry for the late reply.
Due to my limited infrastructure, I had some trouble using the same batch size as used in the paper and in this repo. Thus, I could not reproduce the exact same experiments as you did.
In my case, the largest batch size I could use is 1200 (yours is 4096).
The results I got were similar to yours, and the difference might be explained by the different batch sizes.
My Results:
micro AP: 59.0
micro AP SN: 71.0
Thanks for your quick answer and for providing your results.
Kind Regards,
João Phillipe

Yea, that’s close, and we expect a bit of degradation at smaller batch sizes.
In the paper supplemental we explore batch size and see ~2% degradation by batch size 2048, but we explored it at entropy weight 15.
We could consider changing this to support larger batch sizes using gradient checkpointing. Marking as closed for now.


[ColorDropdown] Create a new ColorDropdown component

Part of issue #1753
What package is this for?

[x] React
[ ] Angular

Summary
A color dropdown component is need for the HotSpotEditorModal.
Additional context

:tada: This issue has been resolved in version 2.141.0-next.11 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 2.141.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Index is cut off when names are too long for PersonCard #323

Resolves #323
Before:

After: (Index modified)

v1
@MightyCupcakes submitted v1 for review.

[v1 1/1] PersonCard: Set id label to use preferred width

(:books: Archive)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/338/1/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you want to fetch this PR to.

v2
@MightyCupcakes submitted v2 for review.

[v2 1/1] PersonListCard.fxml: Set id label to use preferred width

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v1 and v2)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/338/2/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you want to fetch this PR to.

v3
@MightyCupcakes submitted v3 for review.

[v3 1/1] PersonListCard.fxml: Set id label to use preferred width

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v2 and v3)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/338/3/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you want to fetch this PR to.

v4
@MightyCupcakes submitted v4 for review.

[v4 1/1] PersonListCard.fxml: Set id label to use preferred width

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v3 and v4)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/338/4/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you want to fetch this PR to.

V4
Added comments. Although I am not so sure what is our stand on multi-line XML comments.

v5
@MightyCupcakes submitted v5 for review.

[v5 1/1] PersonListCard.fxml: Set id label to use preferred width

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v4 and v5)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/338/5/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you want to fetch this PR to.

Minor nit: Also add a full stop to the second paragraph. :)

v6
@MightyCupcakes submitted v6 for review.

[v6 1/1] PersonListCard.fxml: Set id label to use preferred width

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v5 and v6)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/338/6/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you want to fetch this PR to.

@chao1995:
label to be replaced with an ellipsis.

Or can say truncated.

v7
@MightyCupcakes submitted v7 for review.

[v7 1/1] PersonListCard.fxml: Set id label to use preferred width

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v6 and v7)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/338/7/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you want to fetch this PR to.

Going to merge soon.


[Snyk] Security upgrade php from 8.2-cli-alpine to 8.0-cli-alpine

This PR was automatically created by Snyk using the credentials of a real user.Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Changes included in this PR

alpine/php/cli/Dockerfile

We recommend upgrading to php:8.0-cli-alpine, as this image has only 0 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

Requested change will downgrade PHP to 8.0, rejecting.


How to apply a moss patch?

I have copied the .mpk file on my cellphone's external storage; How do I apply it t moss?

Try selecting the .mpk with a file explorer such as Amaze, and when asked which application to open the file with, select Moss. The .mpk will be installed into your configurations.

Yay! Thank you :) :+1:


My old address on my network that I seem Helium device. However, it does not seem to have that address on network Solano. The transfer process has been made by me are also in question. In my application I use the helium former address (14r89LXAzszgn1WnepbYKkGJJH1NFYgikfnujy8d 1dp7avs7u2x)While the device is still the same address as it appears on doesn't belong to me, I don't know who it belongs to or transfer to an address that appears to be there.

Not enough information here. Need screenshots and more description of what is happening, and what is expected.

You can't. I show you signing with the HsM address to transfer to Hty. You also transferred the hotspot from HsM to Hty.
https://explorer.solana.com/tx/4StQWcWLLpSLMorju3XpetNJzMtMSPGKQ72MkG2N6XYQcdZTMEtxsnRGjWgAwGvdtRc4tYyQXUXqMPRmVRCAqLiV
We can't move anyone's devices for anyone else. This is all on the blockchain, only the holder of those private keys can do the transfer. If you don't have the private keys to Hty, why transfer devices to it?

I did not transfer the device. I don't even know who owns this address. It's not even an address registered in my address book.

All I have to go off of is the transaction history, which show that someone with your private keys did this transfer. Maybe someone compromised your keys?

It happened after the new implementation. I had no problems with the old application. I only have 12 words. nobody else knows. Could there be a gap in the application?

@Perronef5 could speak more to this, but there's no way for the seed phrase to leave the device, save for you exporting it.
The only way for those transfers to have happened would be if you physically used the app and did a transfer. Which all the evidence here suggests. Unfortunately there's not much I can do beyond that. The NFTs have been moved, and they are with the wallet they were transferred to. We don't have the ability to transfer hotspots from other wallets.

What a pity. I've spent so much time and money on this project. The result was not supposed to be like this. If I had really done this process, I wouldn't have bothered at all, but the device was transferred to another account before I did it.


Do not apply headers for requests made via console

Fix #2

@mkubenka, thank you very much for contribution!


AIX: Implement shared_target = "aix-solib" support

This builds shared libraries as libxxx.so, libxxx.so.ver and static libraries as libxxx.a.  For shlib_variant builds, it builds libxxx.so, libxxxvariant.so.ver, and libxxxx.a.  libxxx.so is a linker import library that when causes the linker to embed a run-time dependency reference to libxxxvariant.so.ver.  Only libxxxvariant.so.ver is needed at runtime.
Fixes #21518

Checklist

[x] documentation is added or updated

We will need a CLA for this. @johnkohl-hcl could you please submit a CLA - I assume we will need both ICLA and CCLA for you. https://www.openssl.org/policies/cla.html

We will need a CLA for this. @johnkohl-hcl could you please submit a CLA - I assume we will need both ICLA and CCLA for you. https://www.openssl.org/policies/cla.html

Yes, I'm working through this with my employer (HCL Technologies).  We have an internal procedure that should have already resulted in a CCLA being submitted, and I'm going through the ICLA procedure now.

We will need a CLA for this. @johnkohl-hcl could you please submit a CLA - I assume we will need both ICLA and CCLA for you. https://www.openssl.org/policies/cla.html

I've submitted my ICLA just now.  I'm still waiting on my company open-source coordinators to verify they've done the CCLA, and/or to have them submit it.

We will need a CLA for this. @johnkohl-hcl could you please submit a CLA - I assume we will need both ICLA and CCLA for you. https://www.openssl.org/policies/cla.html

I've submitted my ICLA just now. I'm still waiting on my company open-source coordinators to verify they've done the CCLA, and/or to have them submit it.

Sorry for the delay, I'm still pushing on my corporate team to get that submitted.

This PR has the label 'hold: cla required' and is stale: it has not been updated in 30 days. Note that this PR may be automatically closed in the future if no CLA is provided.  For CLA help see https://www.openssl.org/policies/cla.html

Still waiting for my corporate contacts to execute the CLA.  They say "soon".

This PR has the label 'hold: cla required' and is stale: it has not been updated in 30 days. Note that this PR may be automatically closed in the future if no CLA is provided.  For CLA help see https://www.openssl.org/policies/cla.html

Corporate CLA has been submitted to<EMAIL_ADDRESS>
Any word on whether legal team has reviewed the Corporate CLA and can remove the hold on this PR?

On Fri, Nov 3, 2023 at 1:44 PM Matt Caswell<EMAIL_ADDRESS>wrote:

Hi John,
Many thanks for sending us your CCLA, which I have registered on our
system. We will additionally require ICLAs for each of the individual
contributors before they are able to contribute to the project.
Many thanks
Matt

The "hold: cla required" tag on this PR can be removed now, right?

@johnkohl-hcl we've received the CCLA but we need also ICLA from you unless you've already sent it.

@johnkohl-hcl we've received the CCLA but we need also ICLA from you unless you've already sent it.

See above, July 26th I submitted my ICLA

@mattcaswell can you please look where the ICLA got lost?

@mattcaswell can you please look where the ICLA got lost?

Any news?

Anything more I can do to help here?  I could rebase to latest and force-push if you like?

As there are no conflicts, there is no need to rebase.
Ping @openssl/committers for second review.

Merged to the master branch. Thank you for your contribution.


Question regarding dilation for transformer

Hi,
Thanks for making the source open, it is really a great chance to learn from you!
After reading the paper, I got a simple question about the dilated transformer module. In the paper, it seemed that there's a dilation rate r that grows exponentially from 2^0 to 2^8, and the r are different across the layers. But when I read the code, I found that the idx variable used for initializing Demixed_DilatedTransformerModel was not utilized in dilation (please correct me if i am wrong). Therefore, I am wondering where is this r.
Thanks!

Hi,
Thanks for making the source open, it is really a great chance to learn from you!
After reading the paper, I got a simple question about the dilated transformer module. In the paper, it seemed that there's a dilation rate r that grows exponentially from 2^0 to 2^8, and the r are different across the layers. But when I read the code, I found that the idx variable used for initializing Demixed_DilatedTransformerModel was not utilized in dilation (please correct me if i am wrong). Therefore, I am wondering where is this r.
Thanks!

Hi,
In our implementation, the dilation rate r = 2^layer, where layer ranges from 0 to 8.
When initializing the dilated Transformer layers, however, there is no need to initialize this dilation rate. Dilation is performed when calling the layers.
Please refer to line 64 of DilatedTransformer.py. we assign argument layer=layer. This layer argument (0, 1, ..., 8) is later used in the kv_roll( ) method (line 74 in DilatedTransformerLayer.py) to perform dilation.
Hope this helps!

Thank you! This detailed answer really helped me a lot.
I would like to have a follow-up stupid question on dilation if you don't mind😂
In the paper,

The dilation rate grows exponentially from 2^0 to 2^8, stretching to a receptive field of 47.51 seconds.

Given the frame_step(as in Spleeter)=1024, sr=44100 (as in the jupyter notebook), there would be approximately 44100/1024 frame in a second.
Total frames in receptive field = 47.51 * (44100/1024), which is approximately 2^11.
Does this mean the r is 11 in the last layer?

Thank you! This detailed answer really helped me a lot.
I would like to have a follow-up stupid question on dilation if you don't mind😂
In the paper,

The dilation rate grows exponentially from 2^0 to 2^8, stretching to a receptive field of 47.51 seconds.

Given the frame_step(as in Spleeter)=1024, sr=44100 (as in the jupyter notebook), there would be approximately 44100/1024 frame in a second.
Total frames in receptive field = 47.51 * (44100/1024), which is approximately 2^11.
Does this mean the r is 11 in the last layer? Thank you!

The receptive field here refers to the accumulated receptive ranges throughout the 9 layers. Also note that we have a dilated window of size w=5. At layer l, (l from 0 to 8), the receptive field R =  (w-1) * 2^l + (w-1) *(2^l-1) + 1. Here, the first term (w-1) * 2^l is contributed by the dilated window on layer l. The second term (w-1) *(2^l-1) is contributed by previous layers on both sides of layer l's window. Let l=8 and w=5, we approximately have R = 2^11.


Default option for selector and drag and drop

New option for selectors actions when you can choose the default option and a drag and drop for options

First step: migrate actual options to React component


Change suffix from Api to Client for Ruby

This fixes #670 for Ruby.
I could make changes in super classes GapicContext.java and SurfaceNamer.java such that other languages are fixed too, but I found that triggers massive changes, which seems to be quite risky.
It's safer to wait until other language fixes this issue too, then we can do refactoring to pull up the changes to the super classes.

Current coverage is 83.70% (diff: 60.00%)

Merging #762 into master will decrease coverage by 0.01%

@@             master       #762   diff @@
==========================================
  Files           296        296
  Lines         11283      11288     +5
  Methods           0          0
  Messages          0          0
  Branches       1516       1516
==========================================
+ Hits           9446       9449     +3
- Misses         1415       1417     +2
  Partials        422        422

Powered by Codecov. Last update 592830d...394151b

Why don't we simply update the common GapicContext class and SurfaceNamer class?
Anyways we want to update the suffix for all languages, right?

cc: @garrettjonesgoogle

@jmuk , I think I agree with @swcloud 's approach of one language at a time - we want to control when the renaming is pushed out with respect to other changes in each language. This is a pretty disruptive renaming, so some care is necessary.


To Lua #41: Green enemies are zooming

Compared to the previous pack, which I downloaded (background update), the green enemies are much faster and my wood division skills aren't able to deal with this. The blue dot enemy tho is still travelling at the same pace as before, so I don't think it's the problem with the FPS as previously.

👀 so on which package/version did you notice that issue?

I used the Package #40. I used the workflow from Branch: lua_#41 without Debug configuration, cos my VS has run out of trial ver (I don't really think I had a trial one, but ok Microsoft).

Closing as discussed on dc


TypeError in notification process

Hi,
We just migrated to release-2.18.0, which uses Python3. We use this in our production system for security reporting, and has been impacted.
We are seeing the following error as part of notification process:
"serverMessage": "Error occurred during the notification process: 'Traceback (most recent call last):
  File \"/usr/local/lib/python3.6/dist-packages/forseti_security-2.18.0-py3.6.egg/google/cloud/forseti/services/notifier/service.py\", line 129, in _run_notifier
    self.service_config)
  File \"/usr/local/lib/python3.6/dist-packages/forseti_security-2.18.0-py3.6.egg/google/cloud/forseti/notifier/notifier.py\", line 187, in run
    InventorySummary(service_config, inventory_index_id).run()
  File \"/usr/local/lib/python3.6/dist-packages/forseti_security-2.18.0-py3.6.egg/google/cloud/forseti/notifier/notifiers/inventory_summary.py\", line 296, in run
    details_data = self._get_details_data()
  File \"/usr/local/lib/python3.6/dist-packages/forseti_security-2.18.0-py3.6.egg/google/cloud/forseti/notifier/notifiers/inventory_summary.py\", line 267, in _get_details_data
    details_data = self.transform_to_template(details)
  File \"/usr/local/lib/python3.6/dist-packages/forseti_security-2.18.0-py3.6.egg/google/cloud/forseti/notifier/notifiers/inventory_summary.py\", line 197, in transform_to_template
    return sorted(template_data, key=lambda k: k['resource_type'])
TypeError: '<' not supported between instances of 'NoneType' and 'str'
'"

Could you please help in figuring this out?
Thank you,
Abhishek

@abhishekupadhyaya Would you be able to change the following line here in your server VM to:
return sorted(template_data, key=lambda k: (k['resource_type'] is None, k['resource_type']))
Then run the below to apply the changes:

sudo python3 setup.py install inside the /home/ubuntu/forseti-security folder within the VM
sudo systemctl restart forseti

and see if that works?
Additionally, we have a Slack channel where you can post questions here.

Thanks for your reply.
We disabled inventory summary in config file, which took care of the error since inventory_summary was not called.
However, could you please fix this as part of your next release.
Thank you.

Thanks @abhishekupadhyaya we have tagged this issue to be fixed at our next Sprint planning.

Hi @abhishekupadhyaya we will attempt to get this in in the 2.20 release.  In any case we will keep you apprised.


Update build-script-helper.py to Python 3

When running locally, it looks like we just need to change the shebang.
https://github.com/apple/indexstore-db/issues/150

@swift-ci Please test


Makefile in data - No need to unzip all the files

Hello @mingujo
As I said, we don't need to unzip all the files when we download them.
We definitely don't need to unzip the BOLD data.
I think we should remove the "unzip" part of the Makefile and see if the Travis can pass the test.

@soazig
I got it. should I make another branch to do this?

Yes, maybe. Also, the Travis is not working.
I tried to check the team alpha, it seems like we need to specify the location of our tests somewhere but I can't figure this out.

I did it, see the branch called data.
Still working on the tests functions.


couldn't pull private repository

I couldn't pull my private repos.
and my os is Win 10 64bit

Are you logged in? Did you confirm your account?

Closing this for lack of activity in the past 15days.


cmake error

Hello, this is the second time that I have deployed jetson inference, but I have some problems this time.
--Configuring incomplete,errors occurred!
See also "/home/m217/jetson-inference/build/CMakeFiles/CMakeOutput.log"
I don't know why this mistake happened, please help me
CMakeOutput.log

Hi @Troyechild, can you copy/paste the console terminal log from when you run cmake ../ instead of CMakeOutput.log?   I can't actually tell the error(s) from CMakeOutput.log
Also, for the time being perhaps you could run the jetson-inference docker container which has it pre-compiled.

Hi @Troyechild, can you copy/paste the console terminal log from when you run cmake ../ instead of CMakeOutput.log? I can't actually tell the error(s) from CMakeOutput.log
Also, for the time being perhaps you could run the jetson-inference docker container which has it pre-compiled.
I solved this problem, and the reason for the problem was that cuda and other libraries were not completely downloaded

Thank you for your prompt reply

Hi @Troyechild, can you copy/paste the console terminal log from when you run cmake ../ instead of CMakeOutput.log? I can't actually tell the error(s) from CMakeOutput.log
Also, for the time being perhaps you could run the jetson-inference docker container which has it pre-compiled.

I found that I had not finished downloading cuda and other libraries. Now I have solved these problems. Thank you for your timely reply


Gallery, Landscape, Animation: circles and squares are clipped or off screen

On iOS, open gallery, go landscape, tap into Animations.
Notice how the circle is clipped and one of the rectangles is off screen.

On landscape, we now see both boxes. However, only on the iPhone do we see the complete lower red box. On the nexus 5 and 6p, the red box's bottom border is not visible.
User testing shows that this UI is hard/very-hard for evaluators to use. Consider re-thinking this UI to make it easier for the developer to play with this example.

iirc we removed this animation demo.


alias with absolute path on fuse-box@v4

Hi, recently i moved to v4 from v3.
On v3, i was using homeDir option for alias some files on my project.
With homeDir, i could use "~/" for aliasing path from project root.
But on v4, there is no homeDir option.
So i tried to use absolute path for aliasing instead of "~/", but it didn't work.
Is there anyway to alias file of absolute path? or path from project root?

@zeniroy you can still use aliases with fusebox, on top of that you can tsconfig baseDir to define your home directory and then referencing that directly
import "components/something"

or
import "src/foo"

Depends on your baseDIr in the tsconfig

How about "js" file? also influenced by baseDir of tsconfig?

yes, no problem there too.

Ok, i see. Thanks a lot!


Updated deprecated links in admin-interactive-dialogs _index.md

Summary
Updated deprecated links in admin-interactive-dialogs _index.md
Ticket Link

NA

Hello @Rutam21,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

@cwarnermm Please review and suggest changes if any and also add the appropriate labels for Docathon.


dump_data -train create huge files even for small input

It seems that there is no test to see if the input ends on the second pass. So if the input data has less than 5M frames it will still generate huge features files.
I suggest changing https://github.com/mozilla/LPCNet/blob/master/src/dump_data.c#L264 to
if (!training || one_pass_completed) break;

Hi @zvik ,
Thanks for this tip. I need also to have training/feature data for only one pass. However, I could not understand how the change you have suggested could be applied because the line you have referred to is regarding allocation of an array.
Would it be possible to instead remove the count*FRAME_SIZE_5MS>=10000000 at line https://github.com/mozilla/LPCNet/blob/3a7ef33dcbade95ec08b85839b6268e35b8d3366/src/dump_data.c#L232 ?
The only thing which is changed from one pass to another is the random number generations, isn't it?
Best


add rerun in Grinder link in release summary report

I can take this one :)

For this issue, it's adding a rerun in Grinder link for each parent test. Is this correct?

Yes, this should be for all failed test builds. For this one, I think there is no modification needed. Ideally, we can directly use the rerun in Grinder link received from the API.


Only create blocks if there are pending transactions

https://github.com/HydraChain/hydrachain/commit/0c05d368753374947a32e4d36fae43d213d0dbdb


Application hangs after many connections are saved

Hi,
After adding all of my SQL connections and saving them in various group Azure Data Studio has become extremely slow when you touch anything that might interact with the settings file (that's my assumption at this point)
This is what I'm experiencing now when ever I change a setting or make a connection from a saved server connection, this will go on for 2-3 minutes and Azure Data Studio will utilize quite a lot of CPU and Memory while it's ongoing.

It might be a little on the extreme side but at the moment I'm having 2200 saved connections (it's really useful for my job, since I can easily search the list and find the connection I need to work on) hence my settings file has grown quite a lot.
If i remove all my saved connections Azure Data Studio works fine, with absoultely no application hang, I'm in doubt whether this should classify as a bug or feature request, I mean it works but it hangs and it's pretty annoying to the point where it's becoming unusable for daily operation.
Azure Data Studio version: azuredatastudio 1.28.0 (13e362762762e5cb76a5c8afd2bc650f48c2d3d8, 2021-04-15T00:24:15.710Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8650U CPU @ 1.90GHz (8 x 2112)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.85GB (5.48GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (13)

Extension
Author (truncated)
Version

admin-pack
Mic
0.0.2

agent
Mic
0.48.0

cms
Mic
0.8.0

dacpac
Mic
1.8.0

import
Mic
1.3.0

managed-instance-dashboard
Mic
0.4.2

profiler
Mic
0.11.0

query-history
Mic
0.1.0

schema-compare
Mic
1.10.0

server-report
Mic
0.2.3

whoisactive
Mic
0.1.4

vscodeintellicode
Vis
1.2.1

gruvbox-concoctis
whe
10.30.27

(1 theme extensions excluded)

@rgk85
Checking in, do you still experience this issue with latest versions of ADS?

No, I'm not longer experiencing this issue using the latest version, thanks!


05/03/2018 17:58 : Create Pull Request for branch master -> prTesting1_2018050317

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


feat(STONEINTG-233): add surface checks back into e2e tests

Description
Add surface checks that were removed to unblock e2e tests in infra-deployments back into e2e tests
Issue ticket number and link
STONEINTG-233
Type of change

[ ] 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 not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist:

[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added meaningful description with JIRA/GitHub issue key(if applicable), for example HASSuiteDescribe("STONE-123456789 devfile source")
[ ] I have updated labels (if needed)

/retest-required

MVP-demo failed to find the PaC PR after 5 minutes of polling github

/retest

/lgtm

/approve

@psturc can you approve this?

/retest

/retest


[SIL] Fix tuple keypath verifier

Don't compare a lowered type to an unlowered type because we run into the situation where a @thick Any.Type != Any.Type causing something like the following to crash:
typealias Hello = (a: Int, b: Any.Type)

let x = \Hello.b

@swift-ci please smoke test

@swift-ci please smoke test Windows

@swift-ci please smoke test Windows


Reads s3 responses using an underlying stream

Current implementation is using AsyncResponseTransformer.toBytes which loads the full s3 response in memory.
This replaces the AsyncResponseTransformer by one Fs2StreamAsyncResponseTransformer using the underlying reactivestreams Publisher to subscribe the returned fs2.Stream.
It should avoid loading the full s3 response in memory returning a fs2.Stream that returns chunks with the paginated s3 response.
I understand this is introducing a breaking change on S3.create method that now requires a ConcurrentEffect instead of just a Concurrent.
Not pretending to merge as-is. Just checking you opinions of this approach

I did some research on my side about this issue.
I actually used ZIO-AWS library which takes your approach.
Here are my observations.
For the big files, we set the file chunk as 5Mb, and used AWS SDK to process files by chunks. ZIO-AWS library takes the suggested in this PR approach, which is basically converting the reactive-streams into ZStream (in your case fs2).
As the result we started to observe following exceptions
software.amazon.awssdk.core.exception.SdkClientException: Unable to execute HTTP request: The channel was closed. This may have been done by the client (e.g. because the request was aborted), by the service (e.g. because there was a handshake error, the request took too long, or the client tried to write on a read-only socket), or by an intermediary party (e.g. because the channel was idle for too long).

It looked weird to me, and i decided to profile the app, and found this
io.netty.channel.AbstractChannelHandlerContext.fireExceptionCaught(Throwable) AbstractChannelHandlerContext.java:273
  io.netty.handler.timeout.ReadTimeoutHandler.readTimedOut(ChannelHandlerContext) ReadTimeoutHandler.java:98
  io.netty.handler.timeout.ReadTimeoutHandler.channelIdle(ChannelHandlerContext, IdleStateEvent) ReadTimeoutHandler.java:90
  io.netty.handler.timeout.IdleStateHandler$ReaderIdleTimeoutTask.run(ChannelHandlerContext) IdleStateHandler.java:504
  io.netty.handler.timeout.IdleStateHandler$AbstractIdleTask.run() IdleStateHandler.java:476

So, the Netty client detected idle channel and decided to close it, this is weird, since it shouldn't take 30 seconds (this is idle timeout config in aws sdk client by default) to download 5 megabytes chunk.
So i decided to look at sockets telemetry in the profiler, and found that Socket opens, then i see immediate reads from socket for a fraction of second and then almost 30 seconds idle and Socket close.
This last Socket close was triggered by the Netty Idle handler, see the above stack-trace.
So my explanation here is that in zio-aws and the design you suggest, the GetObject call and actual read the data from the response body can be separated in time due to the fact that Functional Effects responsible for reading the 5Mb chunks are handled by Fibers which are scheduled by Cats-Effect or ZIO. In this scenario, if you have many fibers registered to be executed the IO scheduler can execute the fiber that  starts downloading the chunk, then yield and resume the rest later, and this "later" can be too late and socket can be closed by Netty by that time.
So as the result i made some changes in our code so that draining this stream of bytes happening immediately in one piece and deserialized into the List[Byte] (kind of revert to original design), this change fixed the problem with Idled Sockets.
I suggest to keep the implementation as it is, and if you need to download huge file use the readFileMultipart S3 algebra we provide, where every chunk downloaded in memory in one piece. Setting the chunk size small enough you won't saturate the heap.
Happy to discuss this further


Any tips for deployment?

Hi, great job on this game!
I'm working on a personal fork of this project with a few aesthetic tweaks... and was trying to deploy to a netlify site to test, but having some issues.
So far I just did npm run-script build, which made a dist folder. Dropped that in netlify and this is what is happening...https://reacteroids-test.netlify.app/
Any tips you can provide on building and deploying would be much appreciated.
Thanks!

Actually figured it out, so will close this. But still interested in any elaboration for deployment if you have a sec at some point, thanks


Python install fails because of error in setup.py

Just experienced the following error:
sebastian@wreck ~/p/c/o/ortools_examples> python setup.py install --user
Traceback (most recent call last):
  File "setup.py", line 6, in <module>
    make sure you use \"""" + sys.executable + """\" during the installation"""
NameError: name 'sys' is not defined

Changing it from sys.executable to executable made it run without a hitch. Not sure if that's what it should be, so I'll refrain from opening a PR. Any thoughts?

Hello skosch,
Thank you for the report. I uploaded a new archive with the fix.
Regards,
Driss


How to set playback rate

How do I set playback rate?
This node package can set playback rate.
https://github.com/react-native-community/react-native-video#usage
rate={1.5}

Can react-native-youtube set playback rate also?

It depends on the platform specific YouTube module

Sorry. I talked about not quality but speed.

@maltz the official YouTube plugins we use in this component do not provide this kind of functionality

any update on this feature?

any update on this feature?

I also would love to know if there is an update to the YouTube plugin that would allow this.


sreekanth kamineni peoject

I have added project files

@sreekanthkamineni  thank you for your submission.
The team reviewed your work and we are looking for a couple more items from your end:

Some documentation about your project
an automated mechanism to build and run the project
Some tests

Once you provide the following we will re-review and get back to you with our decision.


Status Code 400 instead of 412

Status code 412 Precondition Failed is defined for the very specific use-case of optimistic locking, i.e. when checking one of the If-... headers.
The status code 422 Unprocessable Entity would be more appropriate... or a generic 400 Bad Request.

Hi ! Thanks for the issue. Mapping response codes correctly is very tricky ! Changing this now would be a breaking change. How about making it configurable (so the user can decide what code to map to) with the default still 412, so it's backward compatible. What do you think ?

Yeah, people claiming REST was simple have absolutely no idea ;-)
Changing the result code could break some things, you're right. I have no idea as to how much code actually relies on this specific error code, but it's at least theoretically possible.
OTOH it might be that some tools (client apis, service meshes, maybe more) will misunderstand or even block a 412, but that's also just theory.
I don't really like to have to configure something to make it standards compliant, though. Maybe go to a version 2.0 instead, with this change well documented?

We are in the process of moving this (microprofile-ext) to smallrye, so as part of that I can change it. Still maybe making it configurable but let the default be 422. How does that sound ?

@t1 - Made the change here (as described above). It will more like this to smallrye. However we can release a new version here if you need it. Let me know

That looks fine! And I don't need a extra release here :-)


Gorm V2 - Adding Hooks for callback before/after database connection

Describe the feature
Current gorm v2 literary only supports before/after callback for DB CRUD operations. Wondering if Gorm can provide hooks for before/after database connection.
Motivation
I would like to add tracing/metrics for how long the connect operation takes to debug performance issues.
Related Issues
https://github.com/go-gorm/gorm/issues/3904

@mayuragarwal24  hello, conn's creation  is done by the driver, so gorm do this thing is not very suitable. you can refer to https://github.com/go-gorm/gorm/issues/3904 's recommendation.


Finalize UI

@rashikaranpuria I was working on this branch from branch add-blobstore. For actual UI changes, you can start reviewing from commit ec94943


Remove wrong comment [skip ci]

per https://github.com/oauth-xx/oauth-ruby/commit/0c7915c038ab579a8f80aa635b1436a4562b97e8 I guess jeweler is not used anymore.

thank you


[DESIGN] - Contacts Component on Control Center.

Is your feature request related to a problem? Please describe.
As an Airy user, i need to be able to enable/disable the automatic contacts creation on Airy core.
Describe the solution you'd like
To give the user this control, we need to add Airy CONTACTS as a component to Install/uninstall and enable/disable it.
We need to offer such component as a feature and give the user the space to disable this system contacts automatic creation.
This solution would also help with clients that are syncing with a CRM system to allow user to switch off the automatic contacts component and only ingest contacts from the external source integration.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

redundant ticket.. Closed for this ticket


Why compute eigenbases before accumulating GG' and G'G?

I'm curious as to why in Algorithm 3 in your paper you compute the eigenbases QL and QR before updating the moving averages L = beta2 * L + (1 - beta2) * G * G'  and R = beta2 * R + (1 - beta2) * G' * G. Typically second-order statistics accumulators are updated before computing the update step (e.g. both Shampoo and Adafactor which this method is based on, as well as Adagrad, Adam, etc.).
It is even mentioned in the code that this was a deliberate choice, but I can't find anything discussed in the paper, so I'm curious if it was an empirical finding or if there is a theoretical reason. I've implemented SOAP with both pre- and post-step updating of the L and R factors and have not noticed any difference, but maybe you have.
Btw, the algorithm works amazingly well, better than Adam, Schedule-free Adam, and Shampoo on a fairly heavily-tuned variational inference problem I've been working on for a while now, so congratulations on the great work!

Thanks! It would be great to see your results (you could perhaps post them in the "Discussions" section of github) whenever you are ready to make them public.
We found it to work better (or be more stable wrt other hyperparameters) at preconitioning frequency 1, for larger frequeuncies it does not make a difference. Another way to see this is that if at the the first step we used the gradients to compute L, R, Q_L, Q_R then the first step gradient (G) after projection in the Q_L, Q_R basis would become a diagonal matrix and then the Adam exp_avg_sq would be 0 for non-diagonal entries. This is numerically unstable (if not for eps) which is not a good situation to be in. Will try to add some experiments about this in the next version.

Ahh, thanks for the explanation, that makes perfect sense. I see now that apart from the first step, in general G will not be exactly diagonalized by the transformation to Shampoo's eigenspace and so V will be nonzero everywhere by the second step. I suppose that increasing epsilon to e.g. sqrt(machine eps) would also help to mitigate the issue, but I agree it's cleaner to just skip the first step.
And sounds good, I'd be happy to share my results when they're ready; in the process of writing the paper currently.


New project from wasm-pack new fails to build with wgpu, needs edition or resolve change in Cargo.toml

🐛 Bug description
If I create a new project with wasm-pack new, and add wgpu as a dependency it fails to build.   Adding resolver = "2", or "edition = "2021" (which implies the resolver = "2") to Cargo.toml fixes it.  I'm not sure if this is a wasm-pack issue or a wgpu issue.
🤔 Expected Behavior
It should build without errors.
👟 Steps to reproduce
$ wasm-pack new test-wgpu
$ cd test-wgpu
$ cargo add --features webgl wgpu
    Updating crates.io index
      Adding wgpu v0.15.1 to dependencies.
...
$ wasm-pack build
... fails..

Full build output
🌍 Your environment
nment.
wasm-pack version: wasm-pack 0.11.0
rustc version: rustc 1.68.2 (9eb3afe9e 2023-03-27)

Looks like this is documented in the wgpu tutorial, but it would be less surprising if the generated project included the newer edition.


auto running

Hi!
Thank you for the addon.
Is it possible to run it automatically on a closed file? I'm thinking about the ones generated by preprocessors. Something like Sass AutoCompile does.
Thank you for your attention.

I would be open to a pull request that implemented a Format On Save option, if that's what you're asking for.
See: https://github.com/sindresorhus/atom-esformatter/blob/3b020123faad0dc96cb0bc716656a680472852d5/index.js#L44-L60

Almost. In the case I'm thinking about, two options:

when saving an .scss, the plugin need to wait until SASS saves the css, then open it and run the prefixing;
or the plugin needs to watch the file system and detect when the css changes;

But maybe this is too complex, too out of scope for your addon.

Definitely out of the scope for this plugin, and IMHO out of scope for an editor plugin. That's something that should be done in a build system.
I'm sure there's a build system type plugin for Atom. Just set it up to run the atom-workspace:autoprefixer command on changes. See Atom docs for how commands work.

Thank you, @sindresorhus !


type 'Null' is not a subtype of type 'List' in type cast

I have a table called city that has id, a1, and a2 columns, I want to do a union all query and get the sum of the particularly same rows, this is my endpoint:
  Future<List<List<dynamic>>> fetchMatchedCity(Session session) async {
    session.log("querying match...");
    final start = DateTime.now().millisecond;

    final result = await session.db.query(""" select id1, id2, sum(score) from (
	    select t1.id as id1 , t2.id as id2, 1 as score from city as t1, city as t2 where t1.a1 = t2.a1 and t1.id <> t2.id union all
        select t1.id as id1 , t2.id as id2, 1 as score from city as t1, city as t2 where t1.a2 = t2.a2 and t1.id <> t2.id
    ) as final
    where id1 > id2
    group by (id1, id2) order by sum(score) desc  """);

    return result;
  }

in main.dart:
....
onPressed: () async {
  await client.city.fetchMatchedCity();
}

I removed the usage of the result of await client.city.fetchMatchedCity(); due to reduce the risk of errors! but I still get following error:
flutter: FormatException: No deserialization found for type dynamic
[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: FormatException: No deserialization found for type dynamic
#0      SerializationManager.deserialize (package:serverpod_serialization/src/serialization.dart:73:5)
#1      Protocol.deserialize (package:hlafit_client/src/protocol/protocol.dart:107:18)
#2      Protocol.deserialize.<anonymous closure> (package:hlafit_client/src/protocol/protocol.dart:96:40)
#3      MappedListIterable.elementAt (dart:_internal/iterable.dart:413:31)
#4      ListIterator.moveNext (dart:_internal/iterable.dart:342:26)
#5      new _GrowableList._ofEfficientLengthIterable (dart:core-patch/growable_array.dart:189:27)
#6      new _GrowableList.of (dart:core-patch/growable_array.dart:150:28)
#7      new List.of (dart:core-patch/array_patch.dart:51:28)

P.S: Query is working on db management applications like postico.

Similar/duplicates of #426 #432

@yahu1031 I don't understand these issues' points, https://github.com/serverpod/serverpod/issues/432#issuecomment-1339478060 didn't work for me, and https://github.com/serverpod/serverpod/issues/426 needs PR I think. So there is no fix for this issue now and I have to wait for the PR, am I right? Thanks, @yahu1031 for answering.

You can do one thing. Place a debugger there and see what is the data you get and runtime type. As the issue states it  is null, we need a deserializer for Null data type too. Check if you have that in serverpod deserialize package. I will update it once I check. @alizera

It's currently not supported to return dynamic objects from Serverpod. You will need to specify a serializable type (the ones you create in the yaml-files). It's possible we can add support for this in the future.

If I get your points and example project correctly:
1- I created a new module called serverpod_test_shared (actually copied it from the example project)
2- Add serverpod_flutter: ^0.9.21 into this modules dependencies
3- Modify the FreezedCustomClass that already exists in this shared module
4- Run flutter pub get and run the build_runner to generate the freezed class of  FreezedCustomClass
5- Modify my endpoint:
 Future<List<FreezedCustomClass>> fetchMatchedCity(Session session) async {
    session.log("querying match...");
    final start = DateTime.now().millisecond;

    final result = await session.db.query(""" select id1, id2, sum(score) from (
	    select t1.id as id1 , t2.id as id2, 1 as score from city as t1, city as t2 where t1.a1 = t2.a1 and t1.id <> t2.id union all
        select t1.id as id1 , t2.id as id2, 1 as score from city as t1, city as t2 where t1.a2 = t2.a2 and t1.id <> t2.id
    ) as final
    where id1 > id2
    group by (id1, id2) order by sum(score) desc  """);

    return result as List<FreezedCustomClass>;
  }

6- Call it from my app and now a new error happens:
flutter: Failed call: city.fetchMatchedCity
flutter: ServerpodClientException: Internal server error. Call log id: 172, statusCode = 500
[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: ServerpodClientException: Internal server error. Call log id: 172, statusCode = 500
#0      ServerpodClient.callServerEndpoint (package:serverpod_client/src/serverpod_client_io.dart:83:9)
<asynchronous suspension>
#1      HomePage.build.<anonymous closure> (package:hlafit/pages/home_page.dart:60:19)
<asynchronous suspension>

My query is working properly on my SQL app. Can you please help me for this? Or fix me if I'm wrong? Thanks guys for your awesome job and support ❤

I didn't get exactly that point 2, but to check what the error is, please head over to the logs table in the DB and check for the ID-172 👉🏻 Call log id: 172, statusCode = 500 as it mentioned. @alizera

Check with this please,
type '_PostgreSQLResult' is not a subtype of type 'List<FreezedCustomClass>' in type cast

@yahu1031 I understood the error, but I don't know how to fix it. At the end line of my endpoint method, I cast it to List<FreezedCustomClass> that may cause the error, but before it, it was List<List<dynamic>> that didn't work either.
Can you please help me to fix it? I think I'm mixing something stupidly easy.

@alizera type casting won't work all the time cause the API endpoint query is not returning the FreezedCustomClass, It returns List<List<dynamic>>. List<List> cannot be typecasted as custom models/Objects. Hope this will clear your doubt.

I think I'm missing something stupidly easy.

Not actually cause many will get into issues at this point by directly typecasting the response. (Even I do sometimes 🤣)

Dear @yahu1031, I know what you're saying but I don't know what to do about my issue!!
Please look at my first comment, I was using List<List<dynamic>> as the return type of the method and I was getting deserialization error!
I think I can't get the idea behind serialization (I've read its documentation twice and yet... I don't get it).
Finally, you can see my whole code and database and query and the result of the query (that consists of 2 int and one double columns), can you give me a straightforward solution, please? 🥺
Thank you again.

@alizera I can actually help you.

I was getting a deserialization error!

IS because it was getting null at that time. But later it was having the result(non-null value) but is not castable. That was the issue you get. Can you post the result here? Or you can contact me in twitter/linkedin

We have not revisited this in a while, Is this still an issue or can we close this? There has been som major work regarding serialization and database communication to stabilize the interface.


chore: Version bump wasmtime-provider to 1.10.0

@pkedy thanks! BTW, I've noticed we're missing the wasmtime-provider-v1.8.0, wasmtime-provider-v1.9.0 and wasmtime-provider-v1.10.0 tags. Can you push them, I can do that otherwise

@flavio Done. Good catch. Thanks!


[CCXDEV-7133] Bulk selector tests improve

Check disabling multiple clusters button is also modified
Test for test of selector

:tada: This PR is included in version 1.4.7 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Initializing double local variables with types that requires cast causes exceptions

What about other types? (int looks to be working)
What about fields? Default parameter values?

Test: https://cecilifier.me/?gistid=412b60e61b2b2158bda06dab73efcbb5

need more investigation; Cecilifier is generating invalid IL when assigning non double constants to double variables.


Add useSearchState method

This adds a useSearchState method, which behaves similarly to useState but with search params. This is useful for situations when using a Link or navigate() directly is unergonomic or impossible.
Quick example of the proposed API:
  const [sort = 'asc', setSort] = postsRoute.useSearchState({
    key: 'sort',
  })

Here's a comparison from my original userland draft of the before and after for users, having access to this hook makes for much cleaner code in the situations it's needed:

I am interested in some abstraction APIs around search, but we're currently exploring this pattern along with many others at Nozzle right now. I don't feel confident enough in both our APIs or this one (it's pretty good as well!) to commit them to the core. I would recommend maintaining this in userland and report back in a few months how well it holds up! We will need to address this eventually though.

No problem!
@tannerlinsley there is some prior art here which you might be interested in: https://github.com/pbeshai/use-query-params/tree/master/packages/use-query-params
I've been using that library for a few years, and it's pretty proven for me, plus being so similar to useState there's some evidence it's a good pattern.
The great thing is the new router hoists out some of the cruftier bits like encoder/decoder objects for types, so it can be way simpler here!


Add Tap Handlers to TableRow

Use case
While laying out a drawer menu I decided to give Table a try because it seems a good fit if you want to show icons before the entries, sometimes another icon behind in a regular grid.
When the user wants to select a menu entry it should be possible that he taps anywhere on the Row.
Adding GestureDetectors for every cell seems not a good solution
IMHO the use case to make w whole row selectable is quite common for instance highlightning a row on tap.
Proposal
Add an OnTap handler to TableRows that adds a GestureDetector to a row is a Handler is set.

Isn't this currently achievable using ListTile widget? Using leading and trailing parameters to show icons. They also have gesture recognizers built in.

If I only have three columns that would be an option. Also List Tile doesn't allow to configure colum widths as flexible

That's right, ListTile has Material Design specifications, this proposal would be really nice for more flexible layouts by providing a kind of more generic/flexible ListTile.

Glad to see this is getting the attention it deserves from the Flutter team lol

@escamoteur Do you propose to have ink well affect as well when a row is tapped?

Yes.

Would probably be nice to have this optional

Any update on this? Or atleast a workaround till the time this gets done?

Any update? Or some ideas of how to do this?


Try using the latest version of CasADi rather than our own fork

@jgillis merged the user-defined sparsity feature: https://github.com/casadi/casadi/issues/2074
The only difference remaining to address is changes we made to casadi's FindIPOPT.cmake. We could apply these changes as a patch.


Specify a device support policy

Specifies a policy for supporting a device as discussed in https://github.com/bitcoin-core/HWI/issues/382#issuecomment-782913737

utACK e56f8fd

There's a preview of the docs build of this PR here: https://hwi--469.org.readthedocs.build/en/469/

utACK 6361a18

utACK 6361a18


Explain the .dat File created by 'tmptool.exe -gatherlogs'

How can the .dat files convered to a readable format.

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

ID: 8ca4b399-8d38-b98c-99eb-80aebeb1d4f2
Version Independent ID: df0242c5-746b-f4d5-c254-7a8573b400ce
Content: tpmtool
Content Source: WindowsServerDocs/administration/windows-commands/tpmtool.md
Product: windows-server
Technology: windows-commands
GitHub Login: @ashleytqy
Microsoft Alias: wscontent

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


az webapp create --resource-group $RESOURCEGROUPNAME --plan $SERVICEPLANNAME --name $WEBAPPNAME The behavior of this command has been altered by the following extension: appservice-kube

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az webapp create Extension Name: appservice-kube. Version: 0.2.2.
Errors:
The command failed with an unexpected error. Here is the traceback:
__init__() got multiple values for argument 'linux'
Traceback (most recent call last):
  File "/opt/az/lib/python3.8/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/az/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 658, in execute
    raise ex
  File "/opt/az/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 721, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/az/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 713, in _run_job
    return cmd_copy.exception_handler(ex)
  File "/home/sreedhar/.azure/cliextensions/appservice-kube/azext_appservice_kube/commands.py", line 42, in _polish_bad_errors
    raise ex
  File "/opt/az/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 692, in _run_job
    result = cmd_copy(params)
  File "/opt/az/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 328, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/az/lib/python3.8/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/home/sreedhar/.azure/cliextensions/appservice-kube/azext_appservice_kube/custom.py", line 505, in create_webapp
    helper = _StackRuntimeHelper(cmd, client, linux=(is_linux or is_kube))
TypeError: __init__() got multiple values for argument 'linux'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az webapp create --resource-group {} --plan {} --name {}

Expected Behavior
Environment Summary
Linux-5.4.0-1073-azure-x86_64-with-glibc2.28 (Cloud Shell), Common Base Linux Delridge (quinault)
Python 3.8.12
Installer: DEB

azure-cli 2.34.1

Extensions:
appservice-kube 0.2.2
ai-examples 0.2.5
ssh 1.0.0

Dependencies:
msal 1.16.0
azure-mgmt-resource 20.0.0

Additional Context

@sreedhar2406, you're using an old version of the appservice-kube extension. Please run az extension remove -n appservice-kube followed by az extension add -n appservice-kube to get the correct version


Extract Type Information

Summary
Enso Studio uses the types of data in order to colour nodes and connections. As a result, it is of paramount importance that we provide the ability to query type information at runtime.
Value
Type information is available for query from the interpreter and can later be provided to Enso Studio via the language server.
Specification

Expand the value extraction API already available with the ability to query arbitrary introspected values for their runtime type (and later their typechecked type).

Acceptance Criteria & Test Cases

Introspected values can be queried for their type.
This feature is well-tested and robust.

Superseded by a new plan.


Documentation shows white screen only

Not sure where to post this, but the docs at https://defstudio.github.io/telegraph/ show only a white screen.
My browser console shows no relevant errors (only two icons missing). I have no experience with Nuxt but will take a look if I can fix it. Right now I can't start using the package 'cause there's no documentation :)

I think this commit: b5de4fe erased all of the docs by accident and was put live in the 1.38.2 release a few days ago, which would have triggered a rebuild of Github pages.


Add properties file for meshery server

Signed-off-by: Lidor Ettinger<EMAIL_ADDRESS>Description
The meshery server image will include a properties.yaml file with a release-tag and git-sha.
When the build isn't a release then the release-tag will be the <previous_tag>-
This PR fixes #962

Description

This PR fixes #
Notes to reviewers
Checklist

[] Signed commits.
[] PR title includes [<name of area>]:<description>

Yay!!! 🥳 🎈

Hi @naturalett, this PR has broken the docker build & push action. Can you please take a look into it?
https://github.com/layer5io/meshery/runs/701315774


Complete ign to gz migration

Precisely what the title says. Not ready yet, as mission tests against latest lrauv-application are failing -.-

mission tests against latest lrauv-application are failing -.-

Aha! They are failing on main too, see https://github.com/osrf/lrauv/pull/248. I'll solve that there, then come back.

@caguero This is getting hairy. Tests that use the LRAUV controller fail seemingly at random, or at least I cannot see the pattern just yet.

Finally passing. @adityapande-1995 @arjo129 PTAL!


Add User Agent strings for Hosts package and Engine version.

The Sumerian Hosts now automatically add a version string for both the hosts package as well as the engine integration.
The Hosts version number is the git commit hash. This is set by github actions during a build, but using a local build will default to 'development'. We may want to switch this to release number which should be a simple change to the webpack file
The Engine version number identifies the version of Babylon or Three being used.
There are some unit tests
I validated by looking at the network inspector in the integration tests/demos for both lex(babylon-only) and polly
Example header from the babylon lex sample
X-Amz-User-Agent: aws-sdk-js/2.1117.0 SumerianHosts-development<EMAIL_ADDRESS>promise
The header does not get added to the presigned URLs we use for the MP3 download from Polly.
I don't think this is a problem, however, as we always send a second request to get the visemes which does have this header.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

"What's the behavior of HOSTS_VERSION when this is consumed by other packages through npm". When using NPM, consumers will use the built artifacts which have the version number baked into it. HOSTS_VERSION/GITHUB_SHA only comes into play when using webpack to build the package. I only expect this to happen if a consumer checks out the package and does npm link or otherwise builds it locally. In this case it will have version development so we don't have bugs misattributed to whatever commit they are using

"What's the behavior of HOSTS_VERSION when this is consumed by other packages through npm". When using NPM, consumers will use the built artifacts which have the version number baked into it. HOSTS_VERSION/GITHUB_SHA only comes into play when using webpack to build the package. I only expect this to happen if a consumer checks out the package and does npm link or otherwise builds it locally. In this case it will have version development so we don't have bugs misattributed to whatever commit they are using

I guess i might misunderstand the workflow here. We don't include any build artifacts in the repo anymore which means the version number from github action is not captured?

We don't check in the built code anymore, but we still need to build it before publishing. Just like an Amazon git package does not have the same contents as the built package in the versionset. NPM is sorta like the versionset. That said, looks like we need to add the build step to our publish workflow, as this is not complete: https://github.com/aws-samples/amazon-sumerian-hosts/blob/mainline/.github/workflows/npm-publish.yml


libGDX Project Setup - FAILURE: Build failed

Hi,
I used "libGDX Project Setup" and then I get a error message:
https://i.imgur.com/zx7qkPm.png
Did I do something wrong? I have JDK etc installed.

Most likely you can ignore that error. Open the generated project in idea or other ide if you like pain.
Dunno whats the point of the error box in the setup app, it just confuses people.

@piotr-j Actually, it is a Gradle error and it still appears if you try to import the project in IDEA or AS.

@georgwarden thank you for your solution :) I almost gave up
@DarthYuber I had the same problem and made it work now by following georgewarden's solution but i also had to remove the following line from the AndroidManifest.xml to make it work in AS.
<uses-sdk android:minSdkVersion="9" android:targetSdkVersion="28" />


Check if batch size changed when resuming.

Proposed by @achalddave, opening as a different issue to keep separate.

Closing this for now as the dataloader does not rely on this to work.


onAfterRenderQuestion doesn't work correctly in vue platform

It pass html element without title and errors

btw, what's the correct behaviour?
Should the event be triggered on each question render or only if model changed?
After #684 it was on each render (let's say, after page change) and now it shows the laziness in full, afaiu.

@snay It passed the incorrect html element into the event. Right now, the current wrong version, passes into options.htmlElement the input html element of the question, where it should pass an element that contains: question title, question description, question errors and question input elements.
I was doing an example: https://surveyjs.io/Examples/Library/?id=survey-afterrender and it works for all platforms except vue. I have fixed the issue.
Right now we are calling onAfterRenderXXX events on mounted function. We do not call it on updated function at the current moment.
Thank you,
Andrew

@snay You may also be interesting in this enhancement: https://github.com/surveyjs/surveyjs/issues/714
You have function isDefaultRender that will return true, so the default rendering will be available.
I am thinking about adding updateRender() function in the custom widget, It will be called on updated function. I am not sure, do we need it or not.
Also you may just use your own vue component, instead of the default vue question component.
Thank you,
Andrew


Add Cilium dev workflow to readme

This should probably be documented under Cilium documentation, but someone
asked about it so I figured I'd post it.

Probably this PR should be superseded or merged with https://github.com/cilium/cilium/pull/25217 . I think that cilium/cilium is the right place for most of these instructions.

No need for this PR any more, Cilium official docs have usage instructions with LVH.


Buy order fails

What happened?
Wanted to buy a fine NFT on kodadot.xyz, but after clicking buy i receive this error message:

Please reproduce in steps

I went into the collection via this link posted on Discord link
browsed the collection Dog Sunset
decided to go for no. 22
connected my wallet via talisman
clicked on buy
received the error code as shown in the screenshot

Expected Behavior
After clicking the buy button, i expected to have this fine piece of art in my wallet. I don't know what else to say here :
What browsers are you seeing the problem on?
Chrome
At which address did you encounter bug?
kodadot.xyz
Are you logged in?
Yes
Which wallet you are using?
Talisman
At which chain did you encounter bug?
 Sometimes it's the easy stuff that keeps you going 💯 Thanks for your help!

It's should be fixed on https://kodadot.xyz as well in ten minutes! :)
Enjoy!
Ht for meme contribution ;)
https://github.com/kodadot/nft-gallery/releases/tag/0.0.17


svtplay.se

Checklist

[x ] I'm reporting a broken site support
[ x] I've verified that I'm running youtube-dl version 2020.09.06
[ x] I've checked that all provided URLs are alive and playable in a browser
[x ] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x ] I've searched the bugtracker for similar issues including closed ones

Verbose log

PASTE VERBOSE LOG HERE

Description

WRITE DESCRIPTION HERE
tle@tl64:~$ youtube-dl -i --verbose https://www.svtplay.se/video/28101832/det-finns-sa-manga-vagar-en-film-om-john-holm
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-i', u'--verbose', u'https://www.svtplay.se/video/28101832/det-finns-sa-manga-vagar-en-film-om-john-holm']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2020.09.06
[debug] Python version 2.7.17 (CPython) -<EMAIL_ADDRESS>[debug] exe versions: ffmpeg 3.2.4, ffprobe 3.2.4, rtmpdump 2.4
[debug] Proxy map: {}
[debug] Using fake IP <IP_ADDRESS> (SE) as X-Forwarded-For.
[SVTPlay] 28101832: Downloading webpage
ERROR: Unable to extract video id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/home/tle/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 797, in extract_info
ie_result = ie.extract(url)
File "/home/tle/bin/youtube-dl/youtube_dl/extractor/common.py", line 530, in extract
ie_result = self._real_extract(url)
File "/home/tle/bin/youtube-dl/youtube_dl/extractor/svt.py", line 235, in _real_extract
webpage, 'video id')

See https://github.com/ytdl-org/youtube-dl/issues/26428#issuecomment-689823144 for a fix


Getting Started: Start-Up Command Fails

Attempting to follow getting-started.md
Steps

Ran npx create-epic-app@latest

❌ Error received
user@machine epic-stack % npx create-epic-app@latest

 remix   v2.4.1 💿 Let's build a better website...

   dir   Where should we create your new project?
         /test-website

      ◼  Template: Using epicweb-dev/epic-stack...
      ✔  Template copied
node:internal/errors:866
  const err = new Error(message);
              ^

Error: Command failed: npx --yes create-remix@latest --install --no-git-init --init-script --template epicweb-dev/epic-stack
    at checkExecSyncError (node:child_process:890:11)
    at execSync (node:child_process:962:15)
    at createEpicApp (file:///Users/user/.npm/_npx/cac6a35e04d35842/node_modules/create-epic-app/index.js:7:3)
    at file:///Users/coreybyrum/.npm/_npx/cac6a35e04d35842/node_modules/create-epic-app/index.js:10:1
    at ModuleJob.run (node:internal/modules/esm/module_job:217:25)
    at async ModuleLoader.import (node:internal/modules/esm/loader:316:24)
    at async loadESM (node:internal/process/esm_loader:34:7)
    at async handleMainPromise (node:internal/modules/run_main:66:12) {
  status: 1,
  signal: null,
  output: [ null, null, null ],
  pid: 38446,
  stdout: null,
  stderr: null
}

Node.js v20.9.0

Any help would be appreciated!

I'm a little confused by this step:

Pulled the repo onto my local file system

You shouldn't have to pull any repo. Simply run npx create-epic-app@latest wherever you want the project to go. It will ask you for a name and will create the directory for you.

I see! Apologies! Fairly new to Remix and utilizing templates/stacks.
Although the issue above seems to persist when running npx create-epic-app directly from an arbitrary folder (not a repo). I can update the description to reflect the issue.
Could it be my node version? What's the expected node range for create-epic-app?

That's interesting. It seems to be working fine for me. I'm on node v20.10.0 so I don't expect that to be the issue. It does appear that your error is coming from something internal to node. Unfortunately it doesn't really look like there's much helpful info in the error message. Sorry! Maybe someone else has seen this before 🙏


Customize skills level

Thank you for your work, very nice tool !
I would like to contribute by transforming select skills level by a simple input, this would allow more flexibility and all langages support.
What do you think ?

https://github.com/AmruthPillai/Reactive-Resume/pull/251

Thanks for the PR, this has been merged to production now :)

Thank you for merging :) I don't see it on online, do you know when it will be deployed in production ? Thanks ! 😊

You might need to refresh your cache. Still trying to figure out how to uncache all these sites :)


Please add the option to turn off update notifications in the app....

Description of the new feature / enhancement
Windows 10 only allows me to turn on or off ALL store app updates, not on an app by app basis, and there are a couple of important apps that I like to keep updated, and Powertoys is not one of those.
While I am sure you guys are very proud of your copious updates, I find the importance of these updates to my own small existence less than shatteringly important, and it is VERY annoying having them constantly intruding on my concentration while working at my computer.
Your app currently only gives me the option of either turning on or off the automatic updating feature, not of turning off the notifications that there is another new update available.
Please add the option of also turning off notifications of new updates - thank you!
Scenario when this would be used?
All situations.
Supporting information
No response

What Windows version do you use? It should be possible to select every app/sender.

You can disable notifications for PowerToys in Windows Settings

Related to #2701

This is not true, we discussed this elsewhere, when I reposted this request. PowerToys installed via the Microsoft Store can only be be globally turned off in the OS ie either you get notifications for all MS store apps you've installed, or you do get them for any of them, and I am a keen user of Quicklook, also installed from the store, and it has rare updates, which I am always keen to install.
And an addendum to that other discussion where you offered this incorrect advice is that PowerToys throws up notifications it seems almost every couple of days if you choose not to upgrade the app when an update becomes available.... which is very annoying needless to say hopefully.

@crutkas had a clear comment once that it is PowerToys that sends the message, that has nothing to do with MS Store.

Win32 apps installed from the store do their own updating.  PT is a Win32 app
This would be /dup #2701


Unable to install Nuget package in framework 4.7.2 project

When I tried to install the package into the project (framework 4.7.2), the installation failed with an error.
Error		Could not install package 'SharpNBT 1.2.0'. You are trying to install this package into a project that targets '.NETFramework,Version=v4.7.2', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author.

My deepest apologies, I somehow missed this question and just now noticed it while doing some maintenance.
I am closing this issue for now, as I sincerely doubt any help is still needed. If I am incorrect, feel free to re-open or create a new issue.


fix(test-suite): featureFlags test was awaiting blocks that have not …

Issue being fixed or feature implemented

Feature flags test was awaiting blocks that may have been produced or not (well, in 3 minutes).
What was done?

propagated documents in different broadcasts, plus created an unnecessary transition to create 2 more blocks

How Has This Been Tested?

devnet testing

Breaking Changes

None
Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added or updated relevant unit/integration/functional/e2e tests
[x] I have made corresponding changes to the documentation

For repository code-owners and collaborators only

[x] I have assigned this pull request to a milestone

Backport to master with cherry-pick


Set Remaining Work of tasks based Effort Points of PBI

It's great that we can now add all tasks to a PBI with one click. But we still need to open each task to define how many hours it will take.
My suggestion is to provide a way to automatically set the Remaining Work of a task based on a calculation of the Effort Points of a PBI.
For example
Analyze task: 20 % of Effort Points
Develop task: 40 % of Effort Points
Test task: 40 % of Effort Points
If a PBI has 5 Effort Points, the remaining hours will be Analyze (1), Develop (2), Test (2)
If a PBI has 1 Effort Points, the remaining hours will be Analyze (0,2), Develop (0,4), Test (0,4)
Maybe some rounding could be applied.

I don't know how to +1 this without a comment but we could also use this. We previously set each task remaining time in the scrum planning event, but it is quite time consuming and the only thing we actually use it for, is the burndown chart. Right now we just set a single "Implementation" task time = the PBI effort. 5 effort = 5 hours. Not necessarily true, but better than 0 and shows correct burndown graph (without looking at the values). We also put a 2 on a Test-task, as this can sometimes be time consuming.
With a feature like @tboudrez mentioned, we could automatically put time on all our process tasks.


Multi Adapter Fixes

Multi-Adapter RL, as it currently is, does not reproduce StackLLaMA results and has some design choices that can be improved. This PR fixes some basics and improves the design. This is the first step in the StackLLaMA repro #471 but I won't add more to the current PR to keep it manageable.
the main changes are

add the reward adapter before calling init so that all the modules are already created

makes adding the reward adapter a class method
makes the reward adapter name and score module more explicit, setting them in the init
this feels like it better follows the design structure I've seen in other huggingface PretrainedModels but I'm happy to change it back

refactor compute_reward_score to use model.forward so it works seamlessly with accelerate in multi-gpu, this can be changed to just accelerator.unwrap_model otherwise

I assume that we want to parallelize

correctly set the model.score layer's dtype

otherwise fails if it isn't the same dtype as the model.pretrained_model

set reward model lora parameters to requires_grad = False by using inference_mode = True in the rm_adapter_peft_config

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Sorry I forgot about this PR. Re-merged with main and re-opening. Should have fixed the test error

@younesbelkada can you re-open this or should I make a new PR?


Double inlay hints for .ts files

Type: Bug
Edit a normal .ts file with the extension enabled, all issues and type hints are shown duplicated (once with 'ts' source, once with 'typescript')
Extension version: 0.3.17
VS Code version: Code 1.91.1 (Universal) (f1e16e1e6214d7c44d078b1f0607b2388f29d729, 2024-07-09T22:07:54.982Z)
OS version: Darwin arm64 23.5.0
Modes:

Is the builtin TS plugin disabled? We recommend disabling it.


Framework export options

Add options to export for specific frameworks:

[x] Svelte
[ ] React
[ ] Vue
[ ] Angular

Hi there! This might be too out-of-left-field for you to want to think about, but if you're still considering adding support for more front-end frameworks, a really helpful intermediate step would be to provide something like a JSON export format that exports structured information about the global styles, the details of each artboard, any scripts included, etc. Then, even if this tool can't export a ready-made React component, say, it would be really easy for a user to write their own component and hook it into whatever build system they're using (without having to fall back to things like dangerouslySetInnerHTML).
Even something that's relatively low-level and unopinionated could be really helpful — something along the lines of
{
  centered: true,
  fluid: false,
  maxWidth: 800,
  artboards: [
    {
      id: 'f2h-frame-768',
      maxWidth: 768,
      minWidth: 320,
      aspectRatio: 2.146,
      image: {
        src: 'figma2html/768px.png',
        preloadSrc: 'data:image/gif;base64,R0lGODlhCgAKAIAAAB8fHwAAACH5BAEAAAAALAAAAAAKAAoAAAIIhI+py+0PYysAOw==',
        with: 768,
        height: null,
        alt: 'Image alt text',
      },
      text: [
        {
          text: 'Hello world',
          customAttributes: {
            class: 'custom-class',
          },
          classStyle: 'font-family: Inter; font-style: normal; font-weight: 400; font-size: 32px; text-decoration: none; text-transform: upper; line-height: normal; letter-spacing: 0px; color: rgba(0, 0, 0, 1); mix-blend-mode: normal;',
          inlineStyle: 'top: 46.94%; left: 42.15%; opacity: 1; width: auto; transform: translate(0%, 0%) rotate(0deg); transform-origin: left top; text-align: left;',
          // Alternatively, you could just expose Figma values here directly
        },
      ],
    },
    // ...
  ]
}

Granted, this gets the most complicated with the text segments, especially considering spans where the styling changes within a text segment — but even if this were just exposing all the internal Figma values and you put the burden on the user to render it how they choose, that would be a really powerful option. E.g.,
text: [
  {
    customAttributes: {
      class: 'custom-class',
    },
    x: '42%',
    y: '57%',
    width: 'auto',
    opacity: 1,
    rotation: 15,
    horizontalAlignment: 'LEFT',
    verticalAlignment: 'CENTER',
    segments: [
      {
        text: 'hello ',
        fontName: 'Inter',
        fontWeight: 400,
        fontSize: 12,
        textDecoration: 'NONE',
        textCase: 'ORIGINAL',
        lineHeight: 'auto',
        letterSpacing: '2px',
        fills: '...',
        textStyleId: '...',
        fillStyleId: '...',
        listOptions: '...',
        indentation: '...',
        hyperlink: '...',
      },
      {
        text: 'world',
        fontName: 'Inter',
        fontWeight: 700,
        fontSize: 12,
        textDecoration: 'UNDERLINE',
        textCase: 'UPPER',
        lineHeight: 'auto',
        letterSpacing: '2px',
        fills: '...',
        textStyleId: '...',
        fillStyleId: '...',
        listOptions: '...',
        indentation: '...',
        hyperlink: 'https://www.figma.com/',
      },
    ],
  },
]

Of course, the plugin would/could still provide more opinionated outputs for various frameworks so that users don't have to deal with all the nitty gritty rendering themselves, but users of unsupported frameworks would have all the structured data they would need to roll their own front-end. Heck, pair that with an NPM library that provides some helpful primitives for converting Figma text values to CSS, and you'd have an incredibly robust jumping off point for all kinds of custom front-ends!
Maybe in an ideal world, the plugin would convert the Figma into this intermediate structure and then each output option would some transformation on top of it, making it simpler to add first-class support for additional output formats — thought that would probably require more of a refactor than you're interested in.
Anyways, I've been dreaming of doing something like this for ai2html for a long time, so as I've been digging into figma2html a bit more I just thought I'd share.

Great idea! We already do a fair amount of processing on our end. While we likely can't do such an extreme abstraction for the text, we could give a json file that contains the js, css and html as objects As a team we're focused on other projects for the time being but we'll be revisiting this plugin and others in a couple of months.
I'll pitch this to the team and see what we can do!


Updated line 38 as it's for Destination Cluster

Fixed verbiage on line 38 as it was on ' Source ' when should be ' Destination '.
(Source Cluster is already defined on line # 7.

Thanks for the contribution @ahojman! Merging now.


crd: Add validation using OpenAPI 3.0

Close #437
/assign @jlewi @ScorpioCPH
The feature is restricted and I only add validation for replicas. This avoids the crash problem when replicas < 0.
Signed-off-by: Ce Gao<EMAIL_ADDRESS>

This change is

/retest

If we could find the validation for pod template spec, then we could reuse it here. While I do not find it.

/retest

/retest

/retest

I think the test failures could be kubeflow/kubeflow#869

/hold
Do you still want to submit this in addition to the ksonnet changes? If so then just cancel the hold.
/lgtm
/approve

/hold cancel
I will file a new PR if I find a way to check the podtemplatespec

Coverage remained the same at 55.646% when pulling<PHONE_NUMBER>a7415a61a08c455b5874122a688575 on gaocegege:validation into 98b777eba0b9e91d55be2fd917f25a198cdddeef on kubeflow:master.


Store config after creating a project.

This PR adds the feature to store config per project. #129
Description of changes

Add a new field in the "nlpviewer_backend_project" table of the example database, which is used to store the config.

When the user uploads the ontology, a default config is generated and the user can edit it.

When the user adds a new project, the config JSON will be stored in the database.

Possible influences of this PR
n/a
Demonstration of results

UI

Database

How will the config be used in the viewer?

LGTM


Use native MUI tabs

Probably that one is the easest feature from all the other tickets, so want to give it a try just to see if it can be easy implemented. Just checked gui_mui.c and there nothing about tabs, so probabaly some new functions has to be added ?
I checked gui.c and gui_win32.c (where they have real tabs), and found some vars like FEAT_GUI_TABLINE and FEAT_VARTABS , dunno if any of them related to through..

Seems found, in gui_win32.c those functions seems to cover tabs:
add_tabline_popup_menu_entry()
show_tabline_popup_menu()
gui_mch_show_tabline()
gui_mch_showing_tabline()
gui_mch_update_tabline()
gui_mch_set_curtab()
And the all inside of FEAT_GUI_TABLINE, so probably that it.

Good, that seems pretty straight forward. It's a nice feature to have, looks a lot more native that way.


"undefined reference function" although include header files

Dear every body,
I'm doing unit test project, and i have a issue below:
"src" folder:

src/sub_folder/file1.h
src/sub_folder/file1.c
src/sub_folder/file2.h
src/sub_folder/file2.c
4 files: file1.h, file2.h, file1.c ,file2.c are in same sub folder.
"test" folder:
test/sub_folder/test_file1.c: unit test all function well, have not issue.
test/sub_folder/test_file2.c: test_file2.c have just include "file1.h", because some function in test_file2.c use function in file1.h. But when ceedling build, i have issue 'undefined reference function", this function from file1.h.
test_file1.c and test_file2.c are in same sub folder.
Can you tell me about solution to fix that, please?
Thanks.

The situation as you describe it seems like everything is set up the way it should. test_file1 should build with file1. test_file2 should build with both file1 and file2, because you're including both.
If you didn't want the actual full source code for file1 in your test_file2, you could instead include "mock_file1", but that's not the problem you're running into at the moment.
Since you haven't included any of the real code or configuration here, I'm just taking guesses at what mistake you could have made. Here are some common ones:
If you're on Windows, you're going to need to still make sure that the capitalization matches. If the files are "file1.h" and "File1.c", that's going to be a problem... or if the file is called "File1.h" and you call it "file1.h" in the test, that would also be a problem. Windows command line is case insensitive, but most tools running on it are not.
It's possible the function itself is misnamed in test2. If your release code is building, that is unlikely.
I assume that both your test and src subfolders are added in ceedling's path listings? When you do includes, you are #including just the name of the file, not the folder, correct?
If neither of these are the problem, it would be useful to get more information. For example, is the undefined reference being reported by the linker or the compiler?

Dear you,
I assume that both your test and src subfolders are added in ceedling's path listings? When you do includes, you are #including just the name of the file, not the folder, correct?
-> Yes.
My  test and src subfolders are added in ceedling's path listings.
src/sub_folder**
src/**
test/**
test/sub_folder/**
I'm include only mame of headers files in source code, not path of header files, like this:
#include "file1.h"
#include "file2.h"
For example, is the undefined reference being reported by the linker or the compiler?
-> I will check console log (linux) again.
I'll feedback for you soon.
Thank you.

Dear you,
Below is  information about ceedling version, os which I using.
1.
cong@cong-Inspiron-3470:~/unit_test/neo-firmware/unit_test$ ceedling version
Ceedling:: 0.28.3
CException:: <IP_ADDRESS>
CMock:: <IP_ADDRESS>
Unity:: <IP_ADDRESS>
2.
OS: ubuntu 16.04 LTS 64 bit.

If neither of these are the problem, it would be useful to get more information. For example, is the undefined reference being reported by the linker or the compiler?
-> the undefined reference was reported by the linker.
Currentty, I instead include "mock_file1".
You can tell me about solutions later.
I will close issue now.
Excuse me! Do you know any free static code analysis tool which check C source code?
Example checking: about memory leak...
Thank you so much!

Excuse me!
Beside github, can I contact you quickly via another thing?
It is my another email<EMAIL_ADDRESS>

Cppcheck is free static analysis tool

I think it can. But since you have a simple example you can run it through and see what it reports. Just be aware that it is same as a compiler where --all does not actually enable all checkers. Check the docs.

I think it can. But since you have a simple example you can run it through and see what it reports. Just be aware that it is same as a compiler where --all does not actually enable all checkers. Check the docs.

Thank you.
If I have any problem when i use cppcheck tool, i need your helps.

If neither of these are the problem, it would be useful to get more information. For example, is the undefined reference being reported by the linker or the compiler?
-> the undefined reference was reported by the linker.
Currentty, I instead include "mock_file1".
You can tell me about solutions later.
I will close issue now.
Excuse me! Do you know any free static code analysis tool which check C source code?
Example checking: about memory leak...
Thank you so much!

Hello,
I also have similar situation but mocking the code file is causing some other issues. Do you have any other solutions than using the mock file?
Thank you,
Suryagiridhar