the nmdc-schema may contain some Biosample slots that belong exclusively to the submission-schema

For example, rna_volume is defined in nmdc-schema and associated with Biosample. We even have some tests for it.
If it is important to capture that information in the SubmissionPortal, so that it can be passed on to user facilities, but it is never going to be saved into MongoDB, maybe it doesn't belong in nmdc-schema?
Right now, I don't think we have a way to introduce slots into submission-schema other than by extraction from nmdc-schema, but that doesn't seem like a big technical challenge.
see this example data file, which I revised in a currently-unmerged PR.

below is a list of slots that are in nmdc schema, but are only needed in the submission portal. These slots need to be store on the submission portal side, and able to be exported for the JGI metadata template. But do not need captured by NMDC

emsl_store_temp
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/emsl.yaml
EMSL
Note

sample_shipped
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/emsl.yaml
EMSL

sample_type
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/emsl.yaml
EMSL

dna_absorb1
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metagenomics.yaml
JGI
IF we keep it, it should map to processed sample, not biosamples

dna_absorb2
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metagenomics.yaml
JGI
IF we keep it, it should map to processed sample, not biosamples

dna_collect_site
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metagenomics.yaml
JGI
I think we can remove this slot & just put the extension that is selected. so 'soil' but a JGI team member should weigh in

dna_cont_type
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metagenomics.yaml
JGI

dna_cont_well
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metagenomics.yaml
JGI

dna_container_id
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metagenomics.yaml
JGI

dna_dnase
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metagenomics.yaml
JGI

dna_organisms
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metagenomics.yaml
JGI
Remove completely?

dna_project_contact
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metagenomics.yaml
JGI
Will need for JGI API

dna_samp_id
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metagenomics.yaml
JGI
this is effectively an alterntaive identifier for a processed sample which is the output of Class Extraction. JGI doesn't really expose these identifiers externally so low priority to keep. Would be worth keeping in the submission portal postgres in case we ever need to pul this in.

dna_sample_format
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metagenomics.yaml
JGI
not informative for workflows

dna_sample_name
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metagenomics.yaml
JGI
Would map to processed sample if we do decided to keep it

dna_seq_project_pi
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metagenomics.yaml
JGI
Will need for JGI API, principal_investigator will be available on OmicsProcessing/DataGeneration

dna_seq_project_name
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metagenomics.yaml
JGI
Will pull from JGI API

dna_volume
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metagenomics.yaml
JGI

proposal_dna
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metagenomics.yaml
JGI
Will pull from JGI API

dna_concentration
https://github.com/microbiomedata/nmdc-schema/blob/df7bd5e6a31e5c46b9afcd844a66d251e612b2b1/src/schema/basic_slots.yaml#L84
JGI
Would map to processed sample if we do decided to keep it. Why is it in basic_slots and not jgi_metagenomics.yaml

proposal_rna
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metatranscriptomics.yaml
JGI
Will pull from JGI API

rna_absorb1
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metatranscriptomics.yaml
JGI
IF we keep it, it should map to processed sample, not biosamples

rna_absorb2
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metatranscriptomics.yaml
JGI
IF we keep it, it should map to processed sample, not biosamples

rna_collect_site
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metatranscriptomics.yaml
JGI
I think we can remove this slot & just put the extension that is selected. so 'soil' but a JGI team member should weigh in

rna_concentration
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metatranscriptomics.yaml
JGI
Would map to processed sample if we do decided to keep it. Why is it in basic_slots and not jgi_metagenomics.yaml

rna_cont_type
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metatranscriptomics.yaml
JGI

rna_cont_well
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metatranscriptomics.yaml
JGI

rna_container_id
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metatranscriptomics.yaml
JGI

rna_organisms
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metatranscriptomics.yaml
JGI
Remove completely?

rna_project_contact
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metatranscriptomics.yaml
JGI
Will need for JGI API

rna_samp_id
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metatranscriptomics.yaml
JGI
this is effectively an alterntaive identifier for a processed sample which is the output of Class Extraction. JGI doesn't really expose these identifiers externally so low priority to keep. Would be worth keeping in the submission portal postgres in case we ever need to pul this in.

rna_sample_format
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metatranscriptomics.yaml
JGI
not informative for workflows

rna_sample_name
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metatranscriptomics.yaml
JGI
Would map to processed sample if we do decided to keep it

rna_seq_project_pi
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metatranscriptomics.yaml
JGI
Will need for JGI API, principal_investigator will be available on OmicsProcessing/DataGeneration

rna_seq_project_name
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metatranscriptomics.yaml
JGI
Will pull from JGI API

rna_volume
https://github.com/microbiomedata/nmdc-schema/blob/main/src/schema/portal/jgi_metatranscriptomics.yaml
JGI

@pkalita-lbl @turbomam See the table above.
Next step for these JGI and EMSL slots I propose

Remove from the linked .yaml files
Add to NMDC submission schema. @pkalita-lbl Can you add some direction on how I can do this (what file and where to add/move these slots to)

The slots would need to be defined in this TSV file: https://github.com/microbiomedata/submission-schema/blob/main/schemasheets/tsv_in/slots.tsv
The association of slots to classes happens in this TSV file: https://github.com/microbiomedata/submission-schema/blob/main/schemasheets/tsv_in/classes.tsv. You can see that a lot of the slots in question are already associated with the appropriate Interface classes. Those would just need to be reviewed to make sure they're still accurate.

From slack
@turbomam "If some UF slots are gong to be moved to submission-schema in the near future, then I would prefer not to move them around within the nmdc-schema modules. I think Alicia and I moved some slots because she was using them outside of their original UF use-case, and that kept the schema from building. We should have included you ion that decison.
I'm working on an issue to make all modules self sufficient (ie build on their own), and this will address the question you have asked along the line "how do I know which module to put new content in?" Can we leave the UF slots where they are until then?"
In summary
for user facility slots we want to keep in NMDC schema, I'll add alias. In a later task we'll complete this issue, getting the user facility slots that NMDC does NOT need to track removed from NMDC schema, and only have them in NMDC submission portal. We will also later decide if the slots that we DO capture should remain in separate .yaml files or be moved to basic_slots.

See also https://github.com/microbiomedata/metadata-template-transformation/issues/16

This is blocked until we resolve how. Converting to a discussion.


Fix Moonwell Rewards

Fixing #258
Test subgraph: https://thegraph.com/hosted-service/subgraph/dmelotik/moonwell?version=pending

NIce catch!


Pypi setup for Python IDL files

Add setup.py and necessary files to generate pypi packages. In the future releases will automatically push to pypi.

LGTM but I'd ask the illustrious @jonathanburns for sign off

Closing since it's not needed after we've moved the proto files to flyteidl


feat(go): template improvements

🧭 What and Why
🎟 JIRA Ticket: https://algolia.atlassian.net/browse/APIC-660
With https://github.com/algolia/api-clients-automation/pull/1423, the initial setup merged but as explained in that PR we need to make more improvements to finalize the generation.
This PR is meant the close the requirement gap on the go client. There is still some improvement room for the implementation but the needed requirements should be fulfilled with this PR.
Changes included:

templates/go/api.mustache updated to handle parameters properly.

Optional parameters, precisely the optional body parameters, created problems with the generated code.
It was converted to a struct approach to creating the correct structure for all request param. It is similar to the default go template but it is altered according to our needs.

templates/go/utils.mustache added to prevent code generation errors.

The nullable primitive types are not generated automatically, with this file, all needed nullable types are added. As of now, we only need NullableBool but I didn't remove the rest since it may be needed in the future.

transport layer is added as the original client and the templates are updated to use the transport layer.

It is mostly moved from algoliasearch-client-go and removed unnecessary pieces with preserving the functionality.
There are some improvement points on the moved codes but it is better to do these improvements after we set the tests and when we are sure that the client is working as expected. There are some complex operations so moving and updating at the same time could bring some unnoticed bugs. This can prolong the development time which is needed to make this client ready. We can improve this internal layer without breaking changes later on.

🧪 Test
The playground example was briefly updated to ensure the new request structure is working. Also, I couldn't find any static check errors on the code after the changes.

oh also I noticed that the go client is using the default values in the models, we shouldn't use it and use nil instead, to leverage the default value of the API directly (because the API client might not be up-to-date)

@millotp Sorry, I couldn't fully understand. So this means we shouldn't utilize the default value on the specs in any of the models and if the value is not set by the user, we should marshal it as empty?

we shouldn't utilize the default value on the specs in any of the models and if the value is not set by the user, we should marshal it as empty

exactly, this is what we do in other languages, the default is purely informative for the user, but we don't rely on it

we shouldn't utilize the default value on the specs in any of the models and if the value is not set by the user, we should marshal it as empty

exactly, this is what we do in other languages, the default is purely informative for the user, but we don't rely on it

@millotp Oh alright, I haven't noticed this at all. I am not sure if the changes will be small or not. If you don't mind, I would like to tackle this with another PR. So we will not make this PR bigger.

You can ignore this comment, it turned out pretty simple to do so I added it in this PR. When it is removed from the initialization function, the rest is automatically handled. If the user is not using specifically NewI{{param}}WithDefaults, we will not set the default values.

just for context about the default values, they are set at the engine level so we don't need them in the API clients, that's why they are only in the specs


Dynup Slow Shutdown

Let Dynup simple action server not directly shut down so we can use it to make the robot sit down. Currently the HCM is still using the sitdown animation in that case, because the DynUp server will directly be shut down. It looks like this can not be changed easily, since the execution loop of the simple action server directly shuts down.
Describe alternatives you've considered
We could keep using the sitdown animation like now.

We don't plan to sit down


Has the create button been discontinued? How do I deploy a contract now?

Has the create button been discontinued? How do I deploy a contract now?
Hi there,
I haven't used Remix for a long time. Didn't there use to be a red create button which opened up MetaMask when clicked?
How do I deploy after compiling now?
BM

@bitcoinmeetups You can go to the run tab and can click deploy after choosing your contract to deploy.


Deprecate ResizeImage()

I think ResizeImage() is of flawed design, and should be removed from the Flutter library.
I can see two opposing perspectives:

ImageProvider provides the image, and ResizeImage resizes that provided image
ImageProvider should not be given these additional un-resized bytes in the first place, and therefore ResizeImage is doing resize work which should be handled by ImageProvider, or another class which is independent of ImageProvider

I'm more in favor of the second perspective, but regardless the perspectives are opposing, and that hints to me there is a flaw in the design.
Let me know if I'm missing anything in regards to these perspectives.

Hi @octocat-1, I'm not entirely sure I understand what you're saying.
Can you provide a bit more detail wrt the opposing perspectives that you've shared above?
Thank you

@danagbemava-nc
What I'm saying is that ResizeImage should not exist. It only seems logical, based on how the widgets are named. "The ImageProvider provides the image, therefore it needs to provide the image to ResizeImage".
But in actuality, the ImageProvider is passive, and being given image data which it will not need once ResizeImage resizes the image data. Therefore, ImageProvider should either be responsible for resizing the image data, or the image data should already be resized before given to ImageProvider.


Delete automerged branches

Currently, whenever a web submission is automerged, its corresponding branch is not deleted, which is causing a rapid increase of the amount of branches in the repository.
The process of automatic branch deletion after automerge should be added to the automerge workflow in order to maintain an organized codebase.

exhibit A


RTM and REST error details

No, but you could subscribe to it.
https://github.com/fitpay/fitpay-android-sdk/blob/9f13d09053b818e3750dd2aa8ecd6a3407d30aa2/fitpay/src/main/java/com/fitpay/android/webview/impl/parser/RtmParserV5.java#L48-L49

@skynet67 I think we want to log it as well.
Also lets revert the version bump until we are ready to cut a release.

@jeremiahthomasharris without increasing the version it will not be possible to test it. As an option we could name it like 0.6.1-snapshot


Either disable mongo with false and point to your running mongodb_url or stop yours and set to true, etc.


[Storage] Does BlobClient.Exists() function correctly for blobs encrypted with CPK?

https://github.com/Azure/azure-sdk-for-net/issues/20361

Will implement this fix in Python

To clarify, the scenario is calling BlobBaseClient.Exists() with a BlobClient that does not have CPK, on a blob that does have CPK.  We get 409 - blob encrypted with user-provided key from the service.  Since the blob clearly exists, we should return true instead of throwing.

Yep! makes sense


DiagnosticVerifier.VerifyDiagnosticResults should cope with non-deterministic Roslyn behavior

Currently DiagnosticVerifier.VerifyDiagnosticResults does check the order in which expected diagnostics occur. This occasionally leads to failed tests, because the order in which diagnostics are produced by Roslyn is not (always) deterministic at the moment.
Also see dotnet/roslyn#372 and dotnet/roslyn#1506 for details

Marking this as a :bug: since we shouldn't have been making these assumptions from the start.

Grabbed


Logger configuration Edits

This PR just comments out the call to the logger configuration.
logging.basicConfig(format="%(levelname)s: %(name)s: %(message)s", level=logging.INFO)

It has been argued that this is the prerogative of the user to do, and more importantly that this results in duplicate error messages when using software that also uses logging.
I'm a bit unclear on whether we need to do anything else to change the behavior, and if we need to warn users of the change.

I don't know why this drug along so many commits. I literally pulled the master branch just before making the edits. Anyways, you can see in the files changes, that there are just three lines commented out.

Okay, just for clarity here. The output historically looks like this when we set the noise parameters:
INFO: enterprise.signals.signal_base: Setting J0030+0451_430_ASP_efac to 1.1365193945368486
INFO: enterprise.signals.signal_base: Setting J0030+0451_430_PUPPI_efac to 0.9799469185765327
INFO: enterprise.signals.signal_base: Setting J0030+0451_EFF.EBPP.1360_efac to 1.1131774722050722
INFO: enterprise.signals.signal_base: Setting J0030+0451_EFF.EBPP.1410_efac to 0.7317297647944114
INFO: enterprise.signals.signal_base: Setting J0030+0451_EFF.EBPP.2639_efac to 0.9816169162003063
INFO: enterprise.signals.signal_base: Setting J0030+0451_JBO.DFB.1520_efac to 1.0323718949754754
INFO: enterprise.signals.signal_base: Setting J0030+0451_L-wide_ASP_efac to 1.1237984445609916
INFO: enterprise.signals.signal_base: Setting J0030+0451_L-wide_PUPPI_efac to 1.1395353301435847
....
INFO: enterprise.signals.signal_base: Setting J2322+2057_JBO.DFB.1400_efac to 0.8577091151959042
INFO: enterprise.signals.signal_base: Setting J2322+2057_JBO.DFB.1520_efac to 1.428677565632148
INFO: enterprise.signals.signal_base: Setting J2322+2057_NRT.BON.1400_efac to 1.7167103970794015
INFO: enterprise.signals.signal_base: Setting J2322+2057_NRT.BON.1600_efac to 1.1806656373839755
INFO: enterprise.signals.signal_base: Setting J2322+2057_JBO.DFB.1400_log10_equad to -6.758190630422875
INFO: enterprise.signals.signal_base: Setting J2322+2057_JBO.DFB.1520_log10_equad to -7.029452492639104
INFO: enterprise.signals.signal_base: Setting J2322+2057_NRT.BON.1400_log10_equad to -7.1857386295101495
INFO: enterprise.signals.signal_base: Setting J2322+2057_NRT.BON.1600_log10_equad to -6.736449974170599

However it seems like INFO message will no longer appear with out the configuration set, i.e. the default level is WARNING. So for instance the following warning still appears,
WARNING: ... No sksparse library. Using scipy instead!

but it looks like this:
No sksparse library. Using scipy instead!

There are plusses and minuses to this change, the main minus being that the user base currently expects a bunch of output when setting WN parameters. Personally, I would rather only see messages when there is a warning, as that's when I should pay attention and often the "WARNING some parameter was not set " message gets lost in all the INFO messages. I would still like to hear from folks though.

Any user who wants to see INFO messages just has to run logging.basicConfig(level=logging.INFO); this can be plonked into example notebooks or scripts or whatever. The problem is putting it in the library.


Dockerfile.ocp: enable GC trace

/hold
Enabling GC traces to track memory usage during upgrades.

/test e2e-aws-upgrade

/retest


CMake ERROR

I key in this comment
./vcpkg install realsense2
and something went wrong
below are the error messages
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:85 (message):
Command failed: /bin/bash ./../src/3ae1ed06fa-f8218372d8.clean/configure --disable-silent-rules --verbose --disable-shared --enable-static --prefix=/home/chungyutseng/vcpkg/installed/x64-linux/debug --bindir=${prefix}/../tools/libusb/debug/bin --sbindir=${prefix}/../tools/libusb/debug/sbin --includedir=${prefix}/../include --datarootdir=${prefix}/share/libusb
Working Directory: /home/chungyutseng/vcpkg/buildtrees/libusb/x64-linux-dbg
Error code: 1
See logs for more information:
/home/chungyutseng/vcpkg/buildtrees/libusb/config-x64-linux-dbg-out.log
/home/chungyutseng/vcpkg/buildtrees/libusb/config-x64-linux-dbg-err.log
Call Stack (most recent call first):
scripts/cmake/vcpkg_configure_make.cmake:540 (vcpkg_execute_required_process)
ports/libusb/portfile.cmake:53 (vcpkg_configure_make)
scripts/ports.cmake:79 (include)
Error: Building package libusb:x64-linux failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with ./vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: libusb:x64-linux
Vcpkg version: 2020.06.15-unknownhash
Additionally, attach any relevant sections from the log files above.
somebody please help me

Hi @chungyutseng   I should first emphasise that I have not got personal experience of vcpkg on Ubuntu.  Virtually the same error has been documented as occurring when using vcpkg with Windows though.
https://github.com/IntelRealSense/librealsense/issues/6131
https://github.com/IntelRealSense/librealsense/issues/6462
The second of those links provides much more detail about the problem.    In that second case, the information that provided a solution was in the comment linked to below:
https://github.com/IntelRealSense/librealsense/issues/6462#issuecomment-640006475
Bearing in mind that you are using Linux instead of Windows, please let me know if the links above do not solve your particular error with vcpkg.  Thanks!

Hi @chungyutseng   Do you require further assistance on this case, please?  Thanks!

Case closed due to no further comments received.

Hi @MartyG-RealSense I'm facing the same issue on Ubuntu 18, couldn't exactly understand how to use instructions given on #6462 , any updates would be really helpful. Thanks!

Hi @adityag6994   As https://github.com/IntelRealSense/librealsense/issues/6462 relates to installing vcpkg on Windows, the short section of guidelines in the official librealsense documentation may be a better starting point.
https://github.com/IntelRealSense/librealsense#building-librealsense---using-vcpkg
vcpkg is a Microsoft tool, and their own general installation documentation (not specific to librealsense) is structured similarly to the librealsense notes.
It is worth mentioning that the Microsoft version adds the recommendation for Linux users to install git if not done so already and also use g++ version 6 or newer.
https://github.com/Microsoft/vcpkg#quick-start-unix


Get-AzDataLakeGen2ChildItem returns only 5000 entries

With az.storage 1.13.3+, Get-AzDataLakeGen2ChildItem only returns 5000 items. With 1.13.1 and before there was no such restrictions. We need to be able to get all child items to properly apply the ACL to all subfolders and files.

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

ID: 562aa838-243f-bed8-c3cb-9164af5efe71
Version Independent ID: 369544f3-5260-3773-68f2-3479c6e9587e
Content: Azure Data Lake Storage Gen2 PowerShell for files & ACLs (preview)
Content Source: articles/storage/blobs/data-lake-storage-directory-file-acl-powershell.md
Service: storage
Sub-service: data-lake-storage-gen2
GitHub Login: @normesta
Microsoft Alias: normesta

@dckfnuance Thanks for your question. We are checking on this and will respond to you soon.

@normesta Can you please take a look at this?

My team also needs Get-AzDataLakeGen2ChildItem to manage the ACL.
When running the script from Azure CLI (PS with Az.Storage 1.14.0) it only gets ~4800 entries but today it seems that, by running the script from Azure Cloud Shell, we can get all the items.
However we cannot use this workaround since Az Cloud Shell has a 20min timeout.

Can we get an update on this please?

@blueww Would you be able to answer this?

@dckfnuance
Would  you like look at the sample Example 3: List items recursively from a Filesystem in multiple batches in https://docs.microsoft.com/en-us/powershell/module/az.storage/get-azdatalakegen2childitem?view=azps-3.8.0?
Please see if the ContinuationToken of the last item get with Get-AzDataLakeGen2ChildItem is not null. If so, it means server not return all items in one request. Then you need follow the sample to list all items.

The code in example 3 works but the question remains: Why is there a limit of 5000 entries returned by Get-AzDataLakeGen2ChildItem? If this is not a bug, then it should be documented and the reasoning behind it explained.

@dckfnuance
As I get from feature team: this is expected, the service returns ~5000 items per page.
I plan to add a warning message to the cmdlet, when the list result has continuation token (means not all items are listed).

Adding a warning message in the cmdlet is nice but the documentations need to be updated also.
There should be a note about the 5000 limit in:
https://docs.microsoft.com/en-us/powershell/module/az.storage/get-azdatalakegen2childitem?view=azps-3.8.0
and in:
https://docs.microsoft.com/en-us/azure/storage/blobs/data-lake-storage-directory-file-acl-powershell
E.g. In the section Set ACLs on all items in a file system , it is not clear that the following command would only update the ACL for 5000 entries and not "all items" as the title says.
Get-AzDataLakeGen2ChildItem -Context $ctx -FileSystem $filesystemName -Recurse | Update-AzDataLakeGen2Item -Acl $acl

@dckfnuance
The returned number can change with server behavior change, this is not controlled by powershell code. So we don't plan to add this to Powershell doc.
Besides that, the cmdlet calls rest API https://docs.microsoft.com/en-us/rest/api/storageservices/datalakestoragegen2/path/list. In the rest API doc, it will be updated with server behavior.

I still think that the following document should be updated to indicate the limitation. Otherwise people would think the code below will update the ACL for all the child items which may not necessarily be true.
https://docs.microsoft.com/en-us/azure/storage/blobs/data-lake-storage-directory-file-acl-powershell#manage-access-permissions
<EMAIL_ADDRESS>From: Wei Wei<EMAIL_ADDRESS>Sent: Wednesday, April 29, 2020 1:37 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Chan, David<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: [EXTERNAL] Re: [MicrosoftDocs/azure-docs] Get-AzDataLakeGen2ChildItem returns only 5000 entries (#52473)
@dckfnuancehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_dckfnuance&d=DwMCaQ&c=h-HJQ5E3_Jo9moAvrfQA-w&r=WEDlJd_qajI6DLeF_lzy0GSR5eE5xJ2eYXmDNh-VIbI&m=BA3fT0MQViuk_Xynx5p9ijl3jpFBYY3PPsM4Y-N0TuA&s=eMDf20d0srYxFY4-GPy6q1TBwlMsYr6CCuBijuIc3a0&e=
The returned number can change with server behavior change, this is not controlled by powershell code. So we don't plan to add this to Powershell doc.
Besides that, the cmdlet calls rest API https://docs.microsoft.com/en-us/rest/api/storageservices/datalakestoragegen2/path/listhttps://urldefense.proofpoint.com/v2/url?u=https-3A__docs.microsoft.com_en-2Dus_rest_api_storageservices_datalakestoragegen2_path_list&d=DwMCaQ&c=h-HJQ5E3_Jo9moAvrfQA-w&r=WEDlJd_qajI6DLeF_lzy0GSR5eE5xJ2eYXmDNh-VIbI&m=BA3fT0MQViuk_Xynx5p9ijl3jpFBYY3PPsM4Y-N0TuA&s=stROz-i50k9_j5z2a00Pk_MonQy_o3uLjhixw8fnuHI&e=. In the rest API doc, it will be updated with server behavior.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_MicrosoftDocs_azure-2Ddocs_issues_52473-23issuecomment-2D621000873&d=DwMCaQ&c=h-HJQ5E3_Jo9moAvrfQA-w&r=WEDlJd_qajI6DLeF_lzy0GSR5eE5xJ2eYXmDNh-VIbI&m=BA3fT0MQViuk_Xynx5p9ijl3jpFBYY3PPsM4Y-N0TuA&s=49iwPJDyIkCahVNkdavlr4ymTRllkKz4LJuR6zkbjrQ&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_APGJNGS4PD45MQYA4DVDT6LRO64JLANCNFSM4MIXMTNA&d=DwMCaQ&c=h-HJQ5E3_Jo9moAvrfQA-w&r=WEDlJd_qajI6DLeF_lzy0GSR5eE5xJ2eYXmDNh-VIbI&m=BA3fT0MQViuk_Xynx5p9ijl3jpFBYY3PPsM4Y-N0TuA&s=zlbJf7rFf-5bBjI95lBHdBWbrCOSOZCi-zeaNTP97uU&e=.

Have contact the doc owner with mail.
We can update the scirpt to update all items in filesystem:
$filesystemName = "my-file-system"
$acl = set-AzDataLakeGen2ItemAclObject -AccessControlType user -Permission rw-
$acl = set-AzDataLakeGen2ItemAclObject -AccessControlType group -Permission rw- -InputObject $acl
$acl = set-AzDataLakeGen2ItemAclObject -AccessControlType other -Permission -wx -InputObject $acl

$Token = $Null
do
{
     $items = Get-AzDataLakeGen2ChildItem -Context $ctx -FileSystem $filesystemName -Recurse -ContinuationToken $Token
     if($items.Length -le 0) { Break;}
     $items | Update-AzDataLakeGen2Item -Acl $acl
     $Token = $items[$items.Count -1].ContinuationToken;
}
While ($Token -ne $Null)

Thx @blueww! I updated the article with this example. That change should appear live in this afternoon's build (3ish)

#please-close

Hi all,
i'm getting this limitation also. I need to export all ACL from all file and folder in our datalake, but with 5000 limits is a bit hard. Is possible to avoid the limit?
Thanks a lot

@emanuele1986
With current rest API design, if you have many Items which can't be returned in single request, need to list them by ContinuationToken.
So in Powershell, you need to list them by ContinuationToken.
Let us know if you meet any difficulty to write the script with ContinuationToken.


3.0.0-snapshot-459, CI index on multiple properties

OrientDB Version: 3.0.0-snapshot-459
Java Version: 1.8
OS: win 10
ci index on multiple properties seems to not work
it works well on a single property
it woks on 2.2.17 (on single and multiple properties)

Hi @santo-it
how are you creating the index?
Thanks
Luigi

Issue7264() in OGitHubIssuesIT.java

Fixed in develop branch
Thanks!
Luigi


RFC-0004: Unify “AuthAction” and “ResrouceOperations” in Pulsar Authorization Permissions

Motivation
Unify “AuthAction” and “ResrouceOperations” in Pulsar Authorization Permissions

@zymap As we discussed, I don't think we need this RFC, correct? If so, we can mark this proposal as rejected.


Transaction requests forwarder

Codecov Report

Merging #274 (a4c1815) into main (34350ef) will decrease coverage by 0.44%.
The diff coverage is 10.58%.

@@            Coverage Diff             @@
##             main     #274      +/-   ##
==========================================
- Coverage   30.56%   30.11%   -0.45%
==========================================
  Files          45       46       +1
  Lines        4790     4861      +71
==========================================
  Hits         1464     1464
- Misses       3130     3201      +71
  Partials      196      196

Impacted Files
Coverage Δ

api/server/v1/tx.go
0.00% <0.00%> (ø)

server/midddleware/auth.go
50.63% <ø> (-2.94%)
:arrow_down:

server/midddleware/forwarder.go
0.00% <0.00%> (ø)

server/midddleware/middleware.go
0.00% <0.00%> (ø)

server/services/v1/api.go
0.00% <0.00%> (ø)

server/transaction/manager.go
2.72% <0.00%> (+0.01%)
:arrow_up:

server/midddleware/header.go
45.45% <83.33%> (+45.45%)
:arrow_up:

server/midddleware/timeout.go
27.02% <100.00%> (ø)

store/kv/fdb.go
57.44% <100.00%> (ø)

Continue to review full report at Codecov.

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


feat: add parents property on page

Add parents property on page, simplify breadcrumb navigation implementation complexity
community tricks see: https://discourse.gohugo.io/search?q=breadcrumb

in HTML:
{{- $page := . -}

{{- range .Parents.Reverse }}

{{ .LinkTitle }}

{{- end }}

{{ .LinkTitle }}

in JSON-LD:
see https://developers.google.com/search/docs/appearance/structured-data/breadcrumb
{{- $breadcrumbItems := slice -}}
{{- range $index, $page := append .Parents.Reverse . -}}
  {{-
  $item := dict
    "@type" "ListItem"
    "position" (add $index 1)
    "name" $page.Title
    "item" $page.Permalink
  -}}
  {{- $breadcrumbItems = append $breadcrumbItems $item -}}
{{- end -}}

I agree that the idea is good, but Parents isn't a good name (a Page has never more than 1 Parent).
I suggest we use .Ancestors.
Also, could you:

Squash the commits and add one commit message that follows the commit message guidelines in CONTRIBUTION.md
Add a short description in the Page variables section about this new method.

@jmooring for me this is an obvious improvement, agree?

this is an obvious improvement, agree?

Yes. Took it for a spin. :+1:


Add Launcher::setExecutable() to manually set executable path.

Add a method to Launcher to manually set the executable, skipping automatic detection.
Solves #6 and part of #3.

Done!


import NodeTypes from '@vue/compiler-core', NodeTypes is undefined

Describe the bug
import { describe, expect, it } from 'vitest';
import { NodeTypes } from '@vue/compiler-core';

describe('suite name', () => {
  it('import NodeTypes', () => {
   // TypeError: Cannot read properties of undefined (reading 'ATTRIBUTE')
    expect(NodeTypes.ATTRIBUTE).toMatchInlineSnapshot();
  });
});

why NodeTypes is undefined? I can't sure that is vite's reason.
Reproduction
https://stackblitz.com/edit/vitest-dev-vitest-ycjtaj?file=test/suite.test.ts
System Info
System:
    OS: macOS 12.2.1
    CPU: (8) x64 Intel(R) Core(TM) i5-8259U CPU @ 2.30GHz
    Memory: 94.02 MB / 8.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.13.0 - ~/.nvm/versions/node/v16.13.0/bin/node
    npm: 8.1.0 - ~/.nvm/versions/node/v16.13.0/bin/npm
  Browsers:
    Chrome: 103.0.5060.114
    Safari: 15.3
  npmPackages:
    vitest: ^0.17.0 => 0.17.0

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

NodeTypes is a const enum, it exists only in Typescript. It is compiled out of Vue, every instance is replaced with appropriate value. That is why you cannot import it.
And from Vite docs: https://vitejs.dev/guide/features.html#typescript

.. because esbuild only performs transpilation without type information, it doesn't support certain features like const enum and implicit type-only imports.

em, thks.


Spark v3 support

Hello,
The documention mentions that spark3 branch should be used for Spark v3.
Could not find spark3 branch, and also there is no spark-greenplum-connector release for Spark v3.
Does spark-greenplum-connector support Spark v3?
Please let me know how to find the sources to build or released artifact for Spark v3.

@hovercraft-github , could you please assist?

Spark v3 has quite a different internal API - that's why connector for it need a separate release.
Yes, we have released connector for Spark v2 only so far.
We have no a strict schedule, but hope v3 connector will be available in the next month.

Hello @hovercraft-github ,
Are there any updates on Spark v3 support?

Just released v3.1.0 for spark3


Relative destination directory on CopySpec is not relative

The relative destination directory on CopySpec is not relative, as documented. Instead, the absolute path is being concatenated to the absolute path of the working directory, leading to unexpected results.
This can be seen by using the following configuration:
githubPages {
  // ...
  pages {
    from 'build/generated-site'
    into '.'
  }
}

The targetDir used to locate files to wipe out should match the workingDir. Instead, it is matching workingDir + workingDir.

This bug manifests because of a lack of test. I will send a PR with a fix and a test.

Quick and dirty workaround untill your PR will be merged:
githubPages {
  pages.metaClass.getRelativeDestinationDir = {
    destPath ?: new File('.')
  }
  ...
}


Training for custom dataset

Hi @yewzijian,
Thanks for sharing your work. I would like to ask you whether you could elaborate with some details about how someone could train the model for a custom dataset.
Thanks.

You can look at the implementation of the data loaders in src/datasets. Note that depending on your dataset, the network parameters may also need to be changed.

Hi @ttsesm, the fields that are not in common are not necessary and most of them can be removed. However certain fields, e.g. tgt_raw might be used for computing the evaluation metrics.

Hi @yewzijian! I am trying to train the network on custom data. As I went through the configuration, I found that for feature loss config., I need to set(r_p, r_n) which according to the paper are(m,2m), where m being the "voxel distance used in the final downsampling layer in the KPConv backbone". How do I figure out m for my dataset?


Is there any way to get the local coordinates of the face mesh?

Have I written custom code (as opposed to using a stock example script provided in Mediapipe):

I used the following code.
https://github.com/google/mediapipe/blob/master/mediapipe/examples/android/src/java/com/google/mediapipe/apps/facemeshgpu/MainActivity.java
If you code it like this you can get the screen coordinates, but I don't want that.
float eyePoint1 = multiFaceLandmarks.get(0).getLandmarkList().get(374).getY()*1920f;
Is there a way to get local coordinates such that the center of the face is the origin?

OS Platform and Distribution (e.g., Linux Ubuntu 16.04, Android 11, iOS 14.4):

Android

Solution (e.g. FaceMesh, Pose, Holistic):

Face Mesh

Programming Language and version ( e.g. C++, Python, Java):
Java

Hi @pekochun, If you are looking for face coordinates you can find it here https://github.com/google/mediapipe/blob/master/mediapipe/modules/face_geometry/data/canonical_face_model_uv_visualization.png  and android connections of eyes and others can be seen here https://github.com/google/mediapipe/blob/master/mediapipe/java/com/google/mediapipe/solutions/facemesh/FaceMeshConnections.java


chore: add some more owners

Adding @debsmita1 @divyanshiGupta and myself :ninja: as owners to merge plugins update.
/cc @invincibleJai @kadel @nickboldt

/approve
/lgtm

/cherrypick 1.3.x


Typescript formatter extremaly outdated (atom-beautify: 0.3.2, while its 2.1.0 already), formatting issues

Typescript formatter is version 2.1.0 already, while atom-beautify uses 0.3.2. It's horribly outdated and causes many formatting issues (espcially with readonly attributes).
Please update typescript formatter used in atom-beautifier ASAP!
https://github.com/vvakame/typescript-formatter/

Published to v0.29.7


[DO NOT MERGE] benchmark : v2 set operation for #3864

Just to share a way to reproduce https://github.com/coreos/etcd/issues/3864.
So if you parallelism all v2 http request with this code:
./benchmark set --total=10000

10000 / 10000 Booooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo! 100.00 % 4s

Summary:
  Total:	4.6908 secs.
  Slowest:	4.6194 secs.
  Fastest:	0.3813 secs.
  Average:	2.7533 secs.
  Stddev:	1.1057 secs.
  Requests/sec:	1803.7409

Response time histogram:
  0.381 [1]	|
  0.805 [247]	|∎∎∎
  1.229 [546]	|∎∎∎∎∎∎
  1.653 [601]	|∎∎∎∎∎∎∎
  2.077 [167]	|∎∎
  2.500 [3192]	|∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎
  2.924 [543]	|∎∎∎∎∎∎
  3.348 [561]	|∎∎∎∎∎∎∎
  3.772 [134]	|∎
  4.196 [1027]	|∎∎∎∎∎∎∎∎∎∎∎∎
  4.619 [1442]	|∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎

Latency distribution:
  10% in 1.2384 secs.
  25% in 2.1364 secs.
  50% in 2.4432 secs.
  75% in 4.1155 secs.
  90% in 4.3224 secs.
  95% in 4.3583 secs.
  99% in 4.4333 secs.

Error distribution:
  [1539]	client: etcd cluster is unavailable or misconfigured

You get client: etcd cluster is unavailable or misconfigured, which is returning from https://github.com/coreos/etcd/blob/master/client/client.go#L337-L369:
resp, body, err = hc.Do(ctx, action)
if err != nil {
	cerr.Errors = append(cerr.Errors, err)
	...

This happens in 2.2.0 ~ master branch.

Use this script to find out why we are getting this error. Next time, will use gist.


Support for observers

It'd be nice to have VRouter or VMaterialApp support observers.  It looks like this functionality is basically there for VMaterialApp, the class just needs a passthrough variable to give VRouterDelegate.

It looks like this functionality is basically there for VMaterialApp
I don't understand what you mean. VMaterialApp does not have an observable property.

Could you specify your use case? Do you want to observe VRouter ? A navigator ? Could you provide a small code snippet of what you are trying to achieve and how you see it maybe ?

I think I understand what you mean. The issue is that VRouter creates nested navigator.
Describing your use case you really help me understand your needs then.

Sure...the firebase analytics package has a FirebaseAnalyticsObserver that reports on route changes.
https://pub.dev/documentation/firebase_analytics/latest/observer/FirebaseAnalyticsObserver-class.html
I need to use this to observe navigation throughout the app, including in nested navigators :)

I see, I would need to make more tests but I should be able to make this work.
You will have to wait same time though since it's not as simple as passing the navigator observer down, maybe a week, max 2 I would say.
Thanks for bringing this feature of Firebase btw, very interesting!

navigatorObservers can now be given to VRouter.
The name that will be observed in the \FirebaseAnalyticsObserverwill be either the name of theVWidget(if you give it one) or the path of theVWidget`, with the pathParameters replaced.
I keep this open because I would love to have a feedback on this since I did not extensively test it. Feel free to close it if you don't want/can't leave a feedback :)

Did you have the chance to have a look ?

Hi, I didn't try with FirebaseObserver, but my project uses BotToastNavigatorObserver and my own RouteObserver to implement RouteAware (even tho in some cases we can achieve the same result with VGuard) and it's working good!

Oh great, thanks for the input!

Since it's working for @evandrmb I'll close this issue. @CosmicPangolin feel free to reopen it or open a new one if you have an issue with this.

Also, sorry for the late feedback:

I've simulated some bugs in my software and Sentry's observer worked just fine.👍

No worries! Thanks for taking the time to respond anyway. I'm glad it works then 😉


removed none delimited style for the refresh

Removed the non-delimited style per the refresh designs.
Otherwise no change to list-inline.

Looks good to me 👍


Add Eligibility screens

Changes in this PR

Add three new screens for entering eligibility information

Screenshots of UI changes

Release checklist
Release process documentation.
Pre-merge checklist

[ ] Are any changes required to dependent services for this change to work?
(eg. CAS API)
[ ] Have they been released to production already?

Post-merge checklist

[ ] Manually approve
release to test
[ ] Manually approve
release to preprod
[ ] Manually approve
release to prod

 Add Eligibility screens


docs(readme): added readme

copied content from modules/client/components/Home.md
repo w/o readme.md looks incomplete 😄, feel free to close the PR if you disagree.
Thanks!

Thanks, but I added a simpler README instead :)


@sveltejs/adapter-node gained undocumented runtime dependencies

Describe the bug
We use adapter-node to build and deploy our app to staging. We normally copy the build folder and run npm ci --omit=dev --ignore-scripts to install runtime dependencies we use. SvelteKit used to package everything into the build folder. Over that last two weeks, a change was made, and the build now has three runtime dependencies that aren't installed unless you install dev packages. Not sure if it was an oversight in the documentation or the development. We worked around the issue by adding these packages as runtime dependencies in the package.json file.
  "dependencies": {
    "cookie": "^0.5.0",
    "devalue": "^2.0.1",
    "set-cookie-parser": "^2.5.1"
  }

Reproduction

Clone https://github.com/sserdyuk/bug-08-18
Install all packages npm ci
Build the app npm run build
Run node build/index.js to confirm that the app starts
Reinstall only runtime dependencies npm ci --omit=dev --ignore-scripts
Run node build/index.js to confirm that the app now fails

Logs
@> npm ci

added 207 packages, and audited 208 packages in 2s

41 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities
@> npm run build

><EMAIL_ADDRESS>build
> vite build

vite v3.0.8 building for production...
✓ 30 modules transformed.
.svelte-kit/output/client/vite-manifest.json                                         2.09 KiB
.svelte-kit/output/client/_app/immutable/components/layout.svelte-e105e7df.js        0.53 KiB / gzip: 0.35 KiB
.svelte-kit/output/client/_app/immutable/components/error.svelte-6f84932b.js         1.97 KiB / gzip: 0.91 KiB
.svelte-kit/output/client/_app/immutable/components/pages/_page.svelte-4f83cedf.js   0.81 KiB / gzip: 0.47 KiB
.svelte-kit/output/client/_app/immutable/chunks/0-6eff01a1.js                        0.12 KiB / gzip: 0.11 KiB
.svelte-kit/output/client/_app/immutable/chunks/index-df6a2127.js                    6.73 KiB / gzip: 2.74 KiB
.svelte-kit/output/client/_app/immutable/chunks/1-ca9c5b7c.js                        0.11 KiB / gzip: 0.11 KiB
.svelte-kit/output/client/_app/immutable/chunks/2-5a5e5908.js                        0.12 KiB / gzip: 0.12 KiB
.svelte-kit/output/client/_app/immutable/start-153896cf.js                           22.58 KiB / gzip: 8.67 KiB
vite v3.0.8 building SSR bundle for production...
✓ 35 modules transformed.
Generated an empty chunk: "hooks"
.svelte-kit/output/server/vite-manifest.json                   1.11 KiB
.svelte-kit/output/server/index.js                             60.96 KiB
.svelte-kit/output/server/entries/fallbacks/layout.svelte.js   0.23 KiB
.svelte-kit/output/server/entries/fallbacks/error.svelte.js    1.32 KiB
.svelte-kit/output/server/entries/pages/_page.svelte.js        0.31 KiB
.svelte-kit/output/server/chunks/index.js                      3.10 KiB
.svelte-kit/output/server/chunks/hooks.js                      0.00 KiB

Run npm run preview to preview your production build locally.

> Using @sveltejs/adapter-node
  ✔ done
 @> node build/index.js
Listening on <IP_ADDRESS>:3000
^C
 @> npm ci --omit=dev --ignore-scripts

up to date, audited 1 package in 310ms

found 0 vulnerabilities
@> node build/index.js
node:internal/errors:464
    ErrorCaptureStackTrace(err);
    ^

Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'devalue' imported from /home/sergei/Projects/Svelte/ddd/build/server/index.js
    at new NodeError (node:internal/errors:371:5)
    at packageResolve (node:internal/modules/esm/resolve:930:9)
    at moduleResolve (node:internal/modules/esm/resolve:976:18)
    at defaultResolve (node:internal/modules/esm/resolve:1078:11)
    at ESMLoader.resolve (node:internal/modules/esm/loader:530:30)
    at ESMLoader.getModuleJob (node:internal/modules/esm/loader:251:18)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:79:40)
    at link (node:internal/modules/esm/module_job:78:36) {
  code: 'ERR_MODULE_NOT_FOUND'
}

System Info
System:
    OS: Linux 5.4 Ubuntu 18.04.6 LTS (Bionic Beaver)
    CPU: (16) x64 AMD Ryzen 7 5700G with Radeon Graphics
    Memory: 18.21 GB / 30.74 GB
    Container: Yes
    Shell: 4.4.20 - /bin/bash
  Binaries:
    Node: 16.14.0 - ~/.bin/node/bin/node
    npm: 8.3.1 - ~/.bin/node/bin/npm
  Browsers:
    Chrome: 104.0.5112.79
    Firefox: 103.0

Severity
serious, but I can work around it
Additional Information
No response

Duplicate of #3176, #4285, #5222, #5944, #5968, #6027. I've commented with my own workaround in https://github.com/sveltejs/kit/issues/3176#issuecomment-1216498286.


[BUG]:Raw data html value is not converting to htmltext control

In the company detail news screen we have htmltext control to display html formatted data in text area . But it is not converting into html and displaying as plane text with html code .
It is using global variable of selectedNews.Details .
Have any one encounter this issue .
I also followed standard way of datacard but on this screen layout not found datacard
Followed this :
https://powerusers.microsoft.com/t5/Webinars-and-Video-Gallery/Handling-SPO-Rich-Text-Columns-in-PowerApps/td-p/138498
When I am directly putting the html data in text property it is getting converted properly .
I have made my own form with keyvalue and card and their it is converting .

Resolved on an earlier call.
Summary:
If you want to paste HTML content into the article of Company News via the admin app, the rich text editor does not consume raw HTML in tags (text). You are able to paste in rendered, previewed HTML instead, but it may not render it well depending on what's inside.
For raw HTML it is best to go to the source SharePoint list for company news and add the raw HTML there.


Fix fixgridatmos command

Was missing a !


Fix variable names in config/internal_router/gen_keys.sh

https://github.com/enowars/bambictf/blob/8a65436da547303089647ee9aa266da2a163424c/config/internal_router/gen_keys.sh#L59-L65
It still uses team* names because we copied it from there

This was fixed at some point and is live on the dev branch


import broken

I am getting this error:
ImportError: cannot import name 'RockStar'

how did you install rockstar

@avinaash with pip3

check your PATH


dts: arm: stm32: add dts support for ADC2/3 of stm32f7

This PR enables the ADC peripheral 2 & 3 for the stm32F7 soc series from STMicroelectronics.
Signed-off-by: Vincent Genevès<EMAIL_ADDRESS>

Yes, our boards uses the 3 ADCs (for voltage measurement) and read values are accurate.
So I would say it works as expected. Though we do only measurements one by channel at a time and at low speed.

The PR, is approved, but the branch is out of date.
Do you need an action from my part to be able to merge ?

The PR, is approved, but the branch is out of date. Do you need an action from my part to be able to merge ?

Yes, at least fixing the error in
Compliance Checks / Run compliance checks on patch series (PR) (pull_request)

with  sign-off in commit message (no special french char ): author  "Vincent Geneves vgeneves@kalray.eu)"


Add method signature to PHPDoc of the Feature facade

Description
Add method signature of the methods provided by the FeatureManager that are exposed via the Feature facade.
Motivation and context
Adding the method signatures to support type hinting for the facade in the IDE. This way the developer knows the method that are available via the facade without peeking in the FeatureManager class.
How has this been tested?
Verified in PHPStorm the supported methods are completed on the facade.
Screenshots (if appropriate)
Types of changes
What types of changes does your code introduce? Put an x in all the boxes that apply:

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

Checklist:
Go over all the following points, and put an x in all the boxes that apply.
Please, please, please, don't send your pull request until all of the boxes are ticked. Once your pull request is created, it will trigger a build on our continuous integration server to make sure your tests and code style pass.

[x] I have read the CONTRIBUTING document.
[x] My pull request addresses exactly one patch/feature.
[x] I have created a branch for this patch/feature.
[x] Each individual commit in the pull request is meaningful.
[ ] I have added tests to cover my changes.
[ ] If my change requires a change to the documentation, I have updated it accordingly.

Thanks a lot for your contribution! Will tag it in the next release!


Discovered license filter options are blank

https://clearlydefined.io/definitions/git/github/nigels-com/glew/7d72257b0ff2aadab92d9b627beb44c36b072e52

@qtomlinson  @bduranc The PR for this issue got merged. Let us know if that fixes this issue?

@fossygirl How long before it is live on the site?

I'm not sure. @jeffwilcox @MichaelTsengLZ  is there a regular deployment job?

@capfei Deploy to prod just now. And will go to live in around 10 minutes.

@fossygirl : It looks fixed to me. Thanks all

@fossygirl I can see it now. Thank you!


RKE2 Template: Nodepools and other arrays not showing in UI

(spoke with Daishan about this offline -- known limitation of UI until further logic is implemented)
currently, there is no way to add multiple nodepools through the UI in RKE2 cluster templates

This is an issue for other array types as well, such as agentEnvVars.
workaround: user can add arrays to the yaml via the edit as yaml button. i.e.
nodepools:
- name: pool-1
  quantity: 1
  etcd: true
  worker: true
  controlplane: true
  image: ubuntu-20-04-x64
  backups: false
  ipv6: false
  monitoring: false
  privateNetworking: false
  region: sfo3
  size: s-4vcpu-8gb
  userdata: null
  sshPort: 22
  sshUser: root
- name: pool-2
  quantity: 1
  etcd: false
  worker: true
  controlplane: false
  image: ubuntu-20-04-x64
  backups: false
  ipv6: false
  monitoring: false
  privateNetworking: false
  region: sfo3
  size: s-4vcpu-8gb
  userdata: null
  sshPort: 22
  sshUser: root

**Repro Steps:**
on master-head (6b430f3)
* deploy RKE2 templates by adding a repo with cluster templates
* go to deploy a cluster via the template
* go to nodepools in the UI
-> no option to add more than 1 nodepool

parent ticket: https://github.com/rancher/dashboard/issues/2972

Related to https://github.com/rancher/dashboard/issues/5754


Fixed wrong calls to free_func

vc_vector_set and vc_vector_set_multiple shouldn’t call free_func if
there is one because they will try to free pointers which were not
allocated.
Suppose you have a vector of pointers to char (a vector of variable
sized strings) and you insert elem at 0. insert calls vector_set and
vector_set will try to call free_func on vector_at(vector, 0), which
really is a free(* (char ** )s), but that doesn’t work since we never put
anything there in the first place. The same happens with append. Check test_vc_vector_with_strfreefunc to see an example and try it with and without the free_func calls.

Hi. Thanks for your pull-request.
It is assumed that the "set" will replace the existing item. To create a new element you need call  'insert' or 'push_back'

Yes, but insert calls set and pushback calls append which calls set_multiple. Both will break when calling set or set_multiple if there's a free_func != NULL like in the test case I put there. Try to run the test case (for the vector of char * I wrote there) without removing the calls to free_func in set and set_multiple and see what happens.

You're right.
I applied your PR, thanks a lot!
But I replaced 'set' methods on 'replace' with calling deleter.


Reconnect running engine

When an engine with a running method is disconnected/fails to post updates, it should buffer up (some) changes
If it reconnects successfully, it should post the buffered changes
Aggregator should support these actions

Detect engine disconnect and persist this fact
Accept reconnect from running engine and allow saving its data

What if engine never reconnects?
What if engine reconnects with new uod?

Engine should support aggregator shutdown, keep running and reconnect with exponential backoff


Faded text in Feature Info panel

When Austrade user clicked on Mineral Exploration, chose Gold and WA (IE, 2D), got this view of the Feature Info panel with faded text:

I could replicate the issue in IE and 2D mo

de


I use various codes but there was still two errors appears

Challenge Size your Images has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.110 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.

Here are some next steps that you should follow before we can help you with your query:

Check these troubleshooting steps first.
Please use the Chat Rooms or try looking through our forum for help with a specific challenge and any coding assistance.

IMPORTANT:
Please NOTE that this tracker is for reporting bugs and enhancements to freeCodeCamp's code base only.
Happy Coding!


Fix link markup in README.rst

Was originally using markdown syntax.
Surprisingly, rst doesn't allow nested markup, which is why there are two emphasis blocks
http://docutils.sourceforge.net/FAQ.html#is-nested-inline-markup-possible

Thanks!


Send tags to content-store

When tags are created/edited, they are sent to the content-store.

:+1:


Extract variable shouldn't be offered when selection contains statement.

a: str = "hello"

select a: str = "hello"
expected: no extract  variable offered
actual: it is offered, and end result is wrong.
when there is type annotation on assignment, it is a statement that can't be extracted to right value of new assignment statement.
code action shouldn't be offered.

This is a dupe of #5339? We should disable extract variables on assignments.

this is dupe of https://github.com/microsoft/pylance-release/issues/5400
for now, I specifically allow assignment, but if we want, we could only allow nested assignment  (ex, a = b = 1) but not full (a = b = 1)


Android vs android audio is too low

We're using React Native WebRTC in an Android and Web app at the moment
Android vs Web audio when the test does not have any problems
But Android vs Android Simulator Audio is too low, really very too low
return getUserMedia({       audio: true,       video: {         mandatory: {           minWidth: 500,            minHeight: 300,           minFrameRate: 30         }       }     });
Does the audio stream have more options?  Or use the default speaker is loudspeaker?
React Native version: 0.47.2
Plugin version: 1.58.2
OS: Android Simulator Samsung S6 vs Samsung S7 Android 6.0 vs Android 7.0

This plugin does not manage sound volume. It must be an artifact of whatever emulation / translation the simulator is doing. As you already verified, it works well on real devices. We use the earpiece by default.


⚠ Bofrost.es has degraded performance

In 8e70b19, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1749 ms

Resolved: Bofrost.es performance has improved in 71c6c01 after 11 minutes.


Modify url, hello name

This commit modifies the names of link page-views -> views.
This commit also modifies the message hello world -> hello james

http://jtan-sps-summer20.appspot.com/ > press contact > 'hello james' appears


How to send a notification to a particular user

Hi @michalvasko, how can I send a notification to a client which connected with a particular user.

You cannot, that is not how NETCONF is designed. You are supposed to manage notifications on the client side, subscribe to the notifications you want to receive, the server always sends them to all the interested clients.


Torrent search: Fix for Safari on iOS.

keyup.enter event does not seem to trigger for Safari on iOS. Trigger update-code on blur-event too to workaround.
Tested on local bitmagnet installation and seems to work fine.
This fixes https://github.com/bitmagnet-io/bitmagnet/issues/117.

Thanks for this! Please could you rebuild the embedded UI with npm run build -- -c embedded?

Sure. Like this?

Yep, thanks!


Ignore unknown fields in data plane contract

Fixes #3334

Proposed Changes

Use the unmarshalling option to discard unknown fields in the json

Release Note

The data plane contract now supports downgrades from versions with new fields

/cc @pierDipi

/cherry-pick release-1.11
/cherry-pick release-1.10

/test upgrade-tests

/retest-required

/test channel-integration-tests-sasl-ssl

/test channel-integration-tests-sasl-ssl


Remove suffix #id\d{1} as part of openjdk testcase name

Jtreg doesn't support to specify testcase name suffixed with #id\d{1}
the special character in url without encoding will also lead to wrong behaviour

Fix #4385
Signed-off-by: Sophia Guo<EMAIL_ADDRESS>

Jtreg doesn't support to specify testcase name suffixed with #id\d{1}

One way to run openjdk test using JTreg is to specify testcase name, for example: java/lang/String/StringRepeat.java. When specify as java/lang/String/StringRepeat.java#id1 it won't work.


Validation error: Impure callable passed to @Pure parameter

Is your feature request related to a problem?
A parameter with the @Pure annotation only accepts pure callables. Currently, this is not checked, however.
Desired solution
Show an error if an impure callable is passed to a @Pure parameter. This also applies to the default value of that parameter. If the function is marked as @Pure, this implies that all parameters are @Pure.
Possible alternatives (optional)
No response
Screenshots (optional)
No response
Additional Context (optional)
No response

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

npm package (@latest dist-tag)
npm package (@latest dist-tag)
v0.4.0
GitHub release

Your semantic-release bot :package::rocket:


Problem with Dropbox login dialogue

Cant login after login press
I think login button is not working
i have attached the screenshot...take a look on it.

I do not see a problems. I have everything working correctly.

Windowx 7 x64
Internet Explorer 11
Nemiro.OAuth.LoginForms v<IP_ADDRESS>3

Yes its working fine after updating the IE8  to IE11,
thank you so much you saved my life.

me also same problem


Probleme with vsftpd ?

Hi,
i got a probleme, when i changethe vsftpd port to 22 i cant no more connect to ssh.
I dont know why (ssh is on port 22 too)
But if i use 22 for ssh and a random for vsftp the thing is working ! Do you know why ?
Last time i install it, it was working fine like this ?
Thanks for your help

You can't use the same port for different services as the network port identifies the application or service. In addition, you should avoid default ports at all cost concerning security. Here is a list of them: https://en.wikipedia.org/wiki/List_of_TCP_and_UDP_port_numbers.
Please close this issue and read up if you have any further questions!

Ok thanks


test/secmemtest: test secure memory only if it is implemented

Fixes #7322
Checklist

[x] tests are added or updated

@paulidale thanks for the green light. I forgot to set the branch labels, does your approval hold for 1.1.1. too?
I have another question: can someone of @openssl/committers tell me, whether the OPENSSL_NO_SECURE_MEMORY test case is covered if I rerun the ci with [extended tests]?

Aehm, no, you need to do that test locally.

Aehm, no, you need to do that test locally.

Thanks for the information. And yes, I tested both cases locally.

This is a bug fix not a feature, so yes approved for 1.1.1 too.
As an aside, there are a number of defines that are not covered by the CI loops :(


Documenting pitfalls of registry usage in cosign policy init and cosign policy sign

Background
cosign policy init allows users to define a policy around how many signatures and from whom should be considered necessary to consider an artifact as "trustworthy" -- e.g., two or more of [alice, bob, carla] must sign the policy before the signed thing is okay to run in prod. This policy is stored in a content-addressed blob in the registry, referenced by a manifest with a mutable tag.
cosign policy sign allows a user to add their signature to the list -- e.g., alice has signed the policy, and now only one of bob or carla needs to sign for the artifact to be used in prod. It does this by reading the policy config blob, adding the new signature details, and writing it back, referenced by the same original mutable tag.
Problem
Because the policy (who can sign, how many are needed) and the signatures (who has signed so far) are stored in the same blob in the same repository in the same registry, all of alice, bob and carla must have push access to the registry. Other users might have access, either because they're other contributors to the project (e.g., dave) or an attacker who's merely gained push access to the registry (ernest). In this example, even though his signature isn't necessary to meet the threshold of the policy, dave can run cosign policy sign, or even just perform registry operations to modify the policy that's referenced by the mutable tag, to add their signatures.
Because the policy config and the signatures are in the same object in the registry, any user that can contribute their signature can also modify the policy config, either to change the set of required maintainers (now [alice, bob, dave]) or lower the threshold of require maintainers. This means an attacker with push access to the repo can change the policy from "two or more of [alice, bob, carla] must sign" to "one of [ernest] must sign; here's ernest's signature😈 ". In theory they could also change the threshold to 0, effectively disabling the policy, regardless of who's signed so far. They could also remove other maintainers' signatures.
Without registry push audit logs, it would be impossible to know whether this policy was changed, if those pushes aren't also published to Rekor (which presumably a malicious actor just wouldn't do)
Proposal
We should document these pitfalls, and recommend some safeguards against these scenarios:

Clients verifying policies should lookup changes to the policy in Rekor, and fail if the latest version seen by Rekor is not the same as the current policy in the registry -- this guards against untracked modifications to the policy in the registry, and keeps Rekor as a transparent source of truth about the history of the policy.
Clients verifying policies should lookup changes to the policy in Rekor, and fail* if any change modified the policy config, and didn't just add a signature -- that is, all versions of the policy seen by Rekor have the same signed section.

*We might want to let users change their policy over time -- e.g., "oh whoops I forgot dave should also be allowed to sign; sorry dave" -- how can a verifying client detect in general whether the policy edit was not malicious? Was adding dave malicious? 🤷

Thoughts? Is there anything else I'm missing? Where's the best place to document this?
cc @lukehinds @jyotsna-penumaka (for sget side) @asraa

Thanks @imjasonh good to go over this topic.
It's vital that we reject anything not in rekor, I have a section in the proposal document about this.

The other option outlined is a kind of spin on known_hosts, but this is less useful as many sget operations will run within an unattended environment (CI) which is ephemeral.
I do wonder if there are other TUF principles we can leverage here to provide a more robust policy management.


Allow SCITT users to select individual Apply applications to import

Adds the ability for SCITT users to selectively import applications from Apply that are not yet in recruited status.
The intention is that this is only used in exception - where the trainee needs to be registered, but for one reason or another, their conditions are not all met yet.
New link on homepage

Application selection page

Confirm page

Flash message

 Solve the problem of some trainees needing to be registered before they have met all of the conditions for their course on Apply for teacher training

Imported applications become draft trainee records in this service.
Given the last paragraph was on Apply, is it a bit confusing to use 'this service' here?
are for a course with a start date between the beginning of last month and the end of next month
This feels a little bit confusing to me. So if you import today, it could be from the start of February or the end of April? Is there a clearer way we could state this? Could we maybe also be a bit more forgiving here with the dates - a trainee could start in August, but they only really register when they're finalising census records in October. Particularly apprenticeships there's a bit of an a grey area.
'Conditions' are not available over the API at the moment, if we have to get rid of this, is there anything else we think we need here?
Otherwise, looks great, thank you


Appointments List should provide more information on listed appointments

🚀 Feature Proposal
Appointments list should provide more information on listed appointments, like location, type and/or reason of the appointment.
Motivation
Currently, only the start time of the appointment is displayed in the list, but the search bar does only search on the location, reason and type field. It would also make the list more intuitive for the user.
Current appointment list

Hey, is it possible to attempt this one?

Sure @darrylpargeter!

@jackcmeyer @tehKapa
I started looking at this and found this comment on a PR https://github.com/HospitalRun/hospitalrun-frontend/pull/1823#pullrequestreview-358683785 which mentions that this should use the Table component once it has been done.
However, from a look at the component, it still seems to be a WIP and is in need of a bit TLC to be considered usable.
From my point of view the table is missing, in terms of this issue:

An on click event on the rows themselves, though this does seem to be being worked on here https://github.com/HospitalRun/components/issues/330
Showing the pagination controls dynamically or by a flag on the component or within the tableProperties object (not sure if you guys have a prefered way of this working?)
A new property in tableProperties to control the thead class. So the thead-light class from bootstrap can be added to keep the table style inline with the preexisting tables in the app (I'm suggesting a new property so it can be changed on the fly, over hardcoding it within the table comp)
a Cell function to the date type in the generateColumns function to allow the creation of a more user-friendly/ location-based format of the dates

Would there be any objections from me adding (2-4) in the prefered way to the table comp?

@darrylpargeter I think that these are all great ideas. I'm thinking we may want a different interface to the Table component.
I've opened an issue here for discussion: https://github.com/HospitalRun/components/issues/411

Hello,
If this issue is still up for grabs, I was wondering if I could attempt it?

@janmarkusmilan I've assigned this to you!

Just a quick inquiry.
If the goal is to just add more information about the appointments (type, location, reason), then would either of these work:

I think the information that you have provided is good! However, I think it would be better displayed in a table. What do you think?

Sounds good!


Prevent deletion of contests when contest voting file is present

Modified delete_assessment so that if a corresponding contest voting file has been uploaded, deleting the contest is not allowed (400 error code) until the contest voting file has been deleted first.

coverage: 95.353% (-0.04%) from 95.388% when pulling c2a6f8c26cc01eb74b647e67d88af876c22c3b1e on kjw142857:master into fdeb111ade9c38319bd92488412aced6c03cac6c on source-academy:master.

Update:

Tests have been added
Additional check during update_final_contest_entries has been added to ensure that insert_voting is never called if the voting question references a nonexistent contest, which could possibly happen due to data corruption or direct modification of the DB


Possible to access WinRT components with ACUR?

is it possible for ms-local-stream:/// uris to access WinRT components just like what we can do for http:// https:// and ms-appx-web:/// uris?

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

ID: 4ec6eae1-657b-ef10-0a83-28061f6ea64e
Version Independent ID: 53402129-a6ac-ab50-8b0f-f6e6fc955c79
Content: WebView.NavigateToLocalStreamUri(Uri, IUriToStreamResolver) Method (Windows.UI.Xaml.Controls) - Windows UWP applications
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.ui.xaml.controls.webview.yml&version=GBlive&_a=contents
Product: uwp
Technology: winrt-reference
Git Login: jwmsft
Microsoft Alias: jimwalk

What do you mean by "access WinRT components"? This API is designed as a hook so that you can load HTML out of an in-memory blob for WebView.

I want to have the javascript code from ms-local-stream:///  to be able to call Windows runtime APIs.
but it seemed not achievable.

I'm not sure if it's possible. @MikeHillberg do you know?

@jwmsft @jevansaks @MikeHillberg Could you provide some details here? From the docs it pretty unclear.
On one side, Content URIs can be configured for a UWP app in the Package.appxmanifest using ms-appx-web:/// scheme. From there, a WebView can navigate to locally bundled web app provided the web app is aware of the Windows environment and adapts it's global routing (e.g. ms-appx-web:///WebApp/build/<path>). This is not feasible.
On the other hand, when the same web app is bundled with the app the WebView can use a custom UriResolver but only using the ms-appx:/// scheme which loses access to the WinRT APIs. This again is not feasible.
Given that an ordinary web app is bundled locally for offline serving, how can WinRT APIs be accessible then?

@jwmsft @jevansaks @MikeHillberg Can we get an answer to the above please?


Cursor width should respect dpi settings

Windows Terminal version
1.18.2822.0
Windows build number
10.0.22621.0
Other Software
None
Steps to reproduce
Set display scale to 200% and use the bar cursor.
Expected Behavior
Cursor width should respect dpi settings.
Actual Behavior
On 200% scale monitor, drawing the bar shape cursor use native 1px, not 1 logical pixel (2px). This makes it a little hard to locate, especially in light theme.
screenshots: terminal, WinUI control, browser

In AtlasEngine we use half the underline width (thinLineWidth) for the cursor width. This is wrong and it should use options.cursorPixelWidth instead.
Windows Terminal in general always returns 1 in Terminal::GetCursorPixelWidth() which is also wrong because it should use SystemParametersInfoW(SPI_GETCARETWIDTH) instead.


Lower max number of parallel simulators (#2653)

Summary

Motivation

Testing

Changelog

Withdrawal please

Summary

Motivation

Testing

Changelog

Summary

Motivation

Testing

Changelog

Withdrawal please


Why only log resolver path?

I am looking for a graphql logger that logs exactly what GraphQL query is passed, and with which arguments. Do you know if any library does this?

Not sure but you could probably extend this one to make it happen!


Not an extension, this applies 1 direction and is not undoable (Cannot enable/Disable)

This "extension" applies to the users Webui permanantly and is not easily removable, quite annoying, and hard to locate files for customization.

Realized my problem, since this extension is using a default name I kept glazing over it unable to find it and hence modify/disable it. Thanks for the extension, I suggest giving it a unique name!

Yeah, unfortunately due to Catppuccin naming conventions, the repo name is just "stable-diffusion-webui",and WebUI gets extension names from repo names. I have proposed a solution to this here. If you want a better chance of this being solved, then you should upvote that issue :)

Though, given that this can cause issues and be confusing, I'm going to add some clarification in the readme.


Only read CGNS rind values if environment variable CGNS_USE_RIND set.

Description
After I added support for reading Rind values from CGNS files and using that to add ghost zones, some of the CGNS tests using delta.cgns started failing. It turns out it has a single block and there are ghost zones on the entire exterior of the structured grid. When ghost zones are added, all the external faces get removed, resulting in a blank image for plots like mesh and pseudocolor. I don't have a good solution for this, so rather than back out the change since there are users that are waiting for this, I added logic to only use the Rind values if the environment variable CGNS_USE_RIND is set. This should preserve the existing behavior and still allow those that need the new behavior to also get what they need by setting that environment variable.
Type of change

[X] Bug fix
[ ] New feature
[ ] Documentation update
[ ] Other

How Has This Been Tested?
I ran the CGNS.py test from the test suite on quartz and it passed. For completeness, I set the environment variable CGNS_USE_RIND and it failed like it did previously.
Checklist:

[X] I have followed the style guidelines of this project.~~
[X] I have performed a self-review of my own code.~~
[X] I have commented my code where applicable.~~
[X] I have updated the release notes.~~
[ ] I have made corresponding changes to the documentation.
[ ] I have added debugging support to my changes.
[ ] I have added tests that prove my fix is effective or that my feature works.
[X] I have confirmed new and existing unit tests pass locally with my changes.~~
[ ] I have added new baselines for any new tests to the repo.
[X] I have NOT made any changes to protocol or public interfaces in an RC branch.~~
[X] I have assigned reviewers (see VisIt's PR procedures for more information).~~

Yes, this is just temporary for the RC. I will add a proper solution on develop. I'm not 100% sure what the correct solution is, but a proper Read option is probably the correct solution.


[Help]: MaskGCT Phoneme Support

Hello,
Thank you for releasing the MaskGCT project. The sound quality is great.
In the english model, can you pass phonemes directly to the model for pronunciation?
Thank you

Is there any update on this?

Nothing still


app: Default voice pack doesn't makes sense

Currently, the default voice pack doesn't makes any sense. It reads something else instead of what is written.
https://user-images.githubusercontent.com/35926530/153918424-45571c7c-d833-4af3-8008-d9cbf02b8c2c.mp4

It reads something else instead of what is written.

What do you mean? That is how voice packs work. It doesn't have all combinations of all words in all tones in the voice pack, there is no such a thing :P.
You might be talking about voice endpoints which are a totally different technology we use, on cloud GPUs?
This should be documented.

I guess we are confusing between things maybe. I wanted to say that the sishi voice pack is like as if it is reading something else as compared to what is written.
For example here in below video you can here the same sentence being read by sishi voice pack and any other voice pack. The other one reads exactly what is written, but the sishi one doesnt sounds like it. Maybe it is made that way.
https://user-images.githubusercontent.com/35926530/154285888-7cdf82b6-36cb-469b-a04a-139a727c151d.mp4

For example here in below video you can here the same sentence being read by sishi voice pack and any other voice pack. The other one reads exactly what is written, but the sishi one doesnt sounds like it. Maybe it is made that way.

You are switching between voice packs and voice endpoints in that video. They both do voice but in a completely different way.
Voice packs are very fast and small downloads that give you Banjo-Kazooie vocals on anything.
Voice endpoints are AI synthesis endpoints which give you cloud-rendered voice.
I am saying this should be documented.


Docs build v3

Docs Build status updates of commit 8079c03:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

dynamics-nav-app/ui-personalization-user.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.


安装过程死循环了

版本号
~ master
什么问题
~ npm install 死循环刷屏
gyp verb command install [ '12.13.1' ]
gyp verb install input version string "12.13.1"
gyp verb install installing version: 12.13.1
gyp verb install --ensure was passed, so won't reinstall if already installed
gyp verb install version not already installed, continuing with install 12.13.1
gyp verb ensuring nodedir is created /data/qsapi/vendors/node_modules/node-sass/.node-gyp/12.13.1
gyp WARN EACCES user "root" does not have permission to access the dev dir "/data/qsapi/vendors/node_modules/node-sass/.node-gyp/12.13.1"
gyp WARN EACCES attempting to reinstall using temporary dev dir "/data/qsapi/vendors/node_modules/node-sass/.node-gyp"
gyp verb tmpdir == cwd automatically will remove dev files after to save disk space
gyp verb command install [ '12.13.1' ]
gyp verb install input version string "12.13.1"
gyp verb install installing version: 12.13.1
gyp verb install --ensure was passed, so won't reinstall if already installed
gyp verb install version not already installed, continuing with install 12.13.1
gyp verb ensuring nodedir is created /data/qsapi/vendors/node_modules/node-sass/.node-gyp/12.13.1
gyp WARN EACCES user "root" does not have permission to access the dev dir "/data/qsapi/vendors/node_modules/node-sass/.node-gyp/12.13.1"
gyp WARN EACCES attempting to reinstall using temporary dev dir "/data/qsapi/vendors/node_modules/node-sass/.node-gyp"
gyp verb tmpdir == cwd automatically will remove dev files after to save disk space
gyp verb command install [ '12.13.1' ]
gyp verb install input version string "12.13.1"
gyp verb install installing version: 12.13.1
gyp verb install --ensure was passed, so won't reinstall if already installed
gyp verb install version not already installed, continuing with install 12.13.1
gyp verb ensuring nodedir is created /data/qsapi/vendors/node_modules/node-sass/.node-gyp/12.13.1
gyp WARN EACCES user "root" does not have permission to access the dev dir "/data/qsapi/vendors/node_modules/node-sass/.node-gyp/12.13.1"
gyp WARN EACCES attempting to reinstall using temporary dev dir "/data/qsapi/vendors/node_modules/node-sass/.node-gyp"
gyp verb tmpdir == cwd automatically will remove dev files after to save disk space
gyp verb command install [ '12.13.1' ]
gyp verb install input version string "12.13.1"
gyp verb install installing version: 12.13.1
gyp verb install --ensure was passed, so won't reinstall if already installed
gyp verb install version not already installed, continuing with install 12.13.1
gyp verb ensuring nodedir is created /data/qsapi/vendors/node_modules/node-sass/.node-gyp/12.13.1
gyp WARN EACCES user "root" does not have permission to access the dev dir "/data/qsapi/vendors/node_modules/node-sass/.node-gyp/12.13.1"
gyp WARN EACCES attempting to reinstall using temporary dev dir "/data/qsapi/vendors/node_modules/node-sass/.node-gyp"
gyp verb tmpdir == cwd automatically will remove dev files after to save disk space
gyp verb command install [ '12.13.1' ]
gyp verb install input version string "12.13.1"
gyp verb install installing version: 12.13.1
gyp verb install --ensure was passed, so won't reinstall if already installed
gyp verb install version not already installed, continuing with install 12.13.1
gyp verb ensuring nodedir is created /data/qsapi/vendors/node_modules/node-sass/.node-gyp/12.13.1
gyp WARN EACCES user "root" does not have permission to access the dev dir "/data/qsapi/vendors/node_modules/node-sass/.node-gyp/12.13.1"
gyp WARN EACCES attempting to reinstall using temporary dev dir "/data/qsapi/vendors/node_modules/node-sass/.node-gyp"
gyp verb tmpdir == cwd automatically will remove dev files after to save disk space
gyp verb command install [ '12.13.1' ]
gyp verb install input version string "12.13.1"
gyp verb install installing version: 12.13.1
gyp verb install --ensure was passed, so won't reinstall if already installed
gyp verb install version not already installed, continuing with install 12.13.1
gyp verb ensuring nodedir is created /data/qsapi/vendors/node_modules/node-sass/.node-gyp/12.13.1
gyp WARN EACCES user "root" does not have permission to access the dev dir "/data/qsapi/vendors/node_modules/node-sass/.node-gyp/12.13.1"
gyp WARN EACCES attempting to reinstall using temporary dev dir "/data/qsapi/vendors/node_modules/node-sass/.node-gyp"
gyp verb tmpdir == cwd automatically will remove dev files after to save disk space
gyp verb command install [ '12.13.1' ]
gyp verb install input version string "12.13.1"
gyp verb install installing version: 12.13.1
gyp verb install --ensure was passed, so won't reinstall if already installed
gyp verb install version not already installed, continuing with install 12.13.1

如何复现此问题
mkdir yapi && cd yapi
git clone https://github.com/YMFE/yapi.git vendors --depth=1
cp vendors/config_example.json ./config.json
# edit
cd vendors
npm install  --registry https://registry.npm.taobao.org # 安装依赖

什么系统（Linux, Windows, macOS）
centos 7.7

nodejs建议8.x，换成8.16.2验证ok了。


add open ark

Added OpenArk An open source reversing and analysis tool that include many portable tools!
Also added a link for win32 coding conventions

@MustCodeAl Appreciate the PR. I feel including OpenArk so early in the tutorials and then never mentioning it again would confuse readers. However, I did include the link you mentioned in 555c002 since that is helpful context. Closing out this PR, but feel free to submit any other feedback you have!


Install / config management?

Just wondering if there is a tutorial, readme, chef script, docker container, or some other info/resources describing how to get this stack up and running.

I'm going through similar getting started issues. @jmandel were you able to get this project up and running? Any notes to share? Thank you for checking back on a 2 year old comment!
So far this looks like a standard maven project and the output from mvn package will be a WAR. I'm getting a build going now and I assume that the WAR can be deployed to any Java app server.

I don't fully recall, but I don't believe I pursued this. Code appears not to have been documented, updated, or maintained.

@jmandel, @jmandel, these projects are standard Java/Spring maven projects. You first need "mvn clean install" these two projects under ThirdParty folder. Other projects depend on the artifacts from these two projects. You can  then build the projects by going to DS4P folder and run "mvn clean install".  OBHITA consent2share was ended in 2014. but Consent2Share among other projects are still in development under BHITS (https://github.com/BHITS). We do have installation guide for Consent2Share and we will do a better documentation on Github for source code released under BHITS.

@tlin-fei, you are correct that this is a fairly clear maven project.
There are a few gotchas I found as I worked through the build process today. My raw notes follow.
Some of my notes are specific to my environment (eg maven version) or hacky (eg removing the logging exclusion). I think some of the dependencies may not be available from the given repositories. If my schedule permits I'll write up an INSTALL.md and send a pull request next week.
I apologize for not providing more formal feedback but I was using some free time to experiment and did not plan for proper feedback to the project.

mvn enforcer very picky about java, mvn, other versions. Altered it to >3, so my install of mvn 3.2.5 would work
run mvn install on common library for unit testing and common library projects
update org.herasaf.xacml.core version from 1.0.0 M3 to 1.0.1.RELEASE as only 1.0.1 is available in
repos
echosign dependencies
in Third Party, run mvn install in ehcosign project
update repos to find one that hosts net.taldius:clamav

remove herasaf repo
add linpki repo which has the clamav library available
comment out enforcer for commons logging, clamav drags it in; this will probably impact logging; right way to fix it is to add an exclusion to the net.taldius dependency I think

now mvn package in the DS4P project works.
tried to deploy consent2share WAR to Glassfish, got errors. Not sure if this is a build issue or glassfish issue


Block merging code/features into 2.12 until all flaky tests are fixed.

Is your feature request related to a problem? Please describe.
As of the time of writing this there are 91 flaky tests. Even with retries, your chances of having a passing gradle check are close to none.
Describe the solution you'd like
In a discussion on slack we proposed not to merge anything other than flaky test failures until we hit zero into 2.x (2.12).
Describe alternatives you've considered

Run after people to fix flaky tests.
Create an economy where to get 1 feature in you need to fix 1 flaky test.
For each PR that failed CI on an unrelated flaky test, only merge when those tests are fixed.

@andrross how do I get the recent numbers on the worst offenders?

I like the intent behind "block merges until all tests are fixed" but in reality we should probably aim for something like "block merges until tests reliably pass" and use a combination of fixing tests, adding retries limited retries to existing flaky tests, muting known bad tests, and assigning owners to all documented issues in order to get there.

-1. We should never block incremental progress (even on minor branches) for PRs unrelated to the flaky tests.
I propose:

Continue to mute the flaky tests to not block any incremental progress.
Ensure #1715 is current, update it if it's not. (I think there's another similar issue we may need to merge with #1715 but I haven't gone hunting for it)
Start assigning those issues up among the core maintainers (put an @{maintainer_id} next to the issue. Make the maintainer the owner but they don't have to be the one to fix it, just the one to seek out the expertise to fix.

I am not against muting tests (1), (2) is an obvious yes.
Doing (3) means we're asking maintainers to run after people, which I am a -1. I think given the state of affairs we need more than a carrot (a stick?) for contributors that wrote flaky tests to come back to them and fix them. I feel pretty strongly that this work shouldn't be dumped on others, while the authors of the flaky tests race to build more features with more flaky tests.

I feel pretty strongly that this work shouldn't be dumped on others, while the authors of the flaky tests race to build more features with more flaky tests.

It's not that simple. We've (informally) defined a  "flaky test" as one that only "periodically" fails and can't be reproduced by the maintainer that discovered it. "If a test fails and you haven't seen it fail in a week (or ever) and I can't repro in the heat of battle it must be a flake, right?"
This loose definition has been broadly applied by ALL contributors/code authors when their PR triggers an unrelated test (that they didn't write) to fail. The author/contributor proceeds to flag that test as "flaky" by (if we're lucky) adding it to the list so it's thrown behind a retry, commenting on the PR that it's a "flake", muting the test, and re-pushing to the PR in an effort to get their contribution to pass (go green) so they can merge the contribution and move on with life.
This means the original authors (not always a maintainer) of the now deemed "flaky test" are most likely in the dark that their test actually failed. This is a bigger problem caused by us only unit and integ testing contributions on PRs. The test framework uses a Random seed!! And that randomization is all over the test harness (to include Lucene DirectoryFactory choices in LuceneTestcase). So only testing scenario based contributions like SegRep and Remote store once (or even a handful of times) is likely not going to catch esoteric corner cases until some random unrelated PR picks the right seed on the right test container under the right conditions that aren't reproducible on a developers dev environment. (As a side note this is why Lucene has "beast" testing which we used extensively when working on BKD and Geo).
What we're finding now, are many of the SegRep "flakes" are not flakes at all but actual bugs that can be reproduced. Those original authors have been working hard to widdle down that list and not dump it on others.

Doing (3) means we're asking maintainers to run after people..

Maybe. It's also asking maintainers to maintain the codebase by notifying original authors (that are most likely unaware) when a flake is discovered in their test. This is why I said that maintainer doesn't have to fix the test themselves, just find the expertise to help. This is the responsibility of a maintainer of an OSS project.
I also propose:

Turn on Jenkins random testing so that we can run the full test suite more often than just on PRs. Bonus points for posting those failures to a mailing list, or a dedicated slack channel (#core-build)

we proposed not to merge anything other than flaky test failures until we hit zero into 2.x (2.12).

I think this proposal goes against the fundamental tenants of this repository - unilaterally blocking 'feature' contributions will immediately be challenged and exceptions will be cut.  If there is no enforcement mechanism than this is a signal of intentions, which is nice, but not equally applied or visible to contributions creating friction.
I think there might be other ways to tackle the goal of reducing flaky tests, I'll make them their own comments for 👍 👎

Proposal: Improve quality by requiring more approvals
By changing the main branch approval count required from 1 to 2, there is more opportunities for feedback and improve PR quality - and therefore improve test quality.
Pros:

Trivial to implement for maintainers
Clearly articulated to all contributors via PR merge UX
By its nature more maintainers will have a deeper understanding about changes that are being merged.

Cons:

Maintainers will need to invest more time in approving more PRs
Contributors will might to response to to more feedback
PR approvals will take longer

Proposal: Increase the number of successful CI runs to 3 to block merging
This would make the pain of flaky test harder to get around. Note; this might effectively 'block all feature development' until substian progress is made - maybe that is a feature not a bug.
Pros:

Trivial to implement for maintainers
Clearly articulated to all contributors via PR merge UX

Cons:

Contributions no matter the size/area will be impacted deeply
3x increase in utilization of CI resources

Proposal: Require 3 passing gradle check runs to merge PRs

@peternied this is something we have discussed with @andrross, the idea is close enough but the process differs slightly:

we keep single Gradle check as it is now
BUT have a manually triggerable premerge Github action that runs the Gradle check N times (at least 10) on Jenkins right before pull request is about to be merged

Maybe. It's also asking maintainers to maintain the codebase by notifying original authors (that are most likely unaware) when a flake is discovered in their test. This is why I said that maintainer doesn't have to fix the test themselves, just find the expertise to help. This is the responsibility of a maintainer of an OSS project.

to @nknize's point where maintainers help identify the author/ owner to fix the flaky test. Also, prevent merging any new PRs from them until flaky tests are fixed.

There is no clean gradle check run these days.

@CEHENKLE @macohen Do you have thoughts on what we do with this issue.  Doesn't seem right to mark this issue as triaged as we don't have a clear outcome/owner

As far as this specific proposal of blocking the merge of new code/features I think it's clearly not accepted. I appreciate everyone who contributed a comment or suggestion here about how to fix our flaky tests, but as far as this issue goes, the right thing to do is to close it.
Flaky tests obviously remain a problem as we went from 91 to 120 between when I opened this issue now me writing this. If someone has a good idea about how to tackle that, please open a new issue.


Connected filter frontend to backend

when user selects filters and submits search, communicates to backend which filters were selected by updating global variables
in future, can reduce scope of these variables to prevent unexpected mutation

Issue(#14)

Thanks for creating the issue. Does this PR resolve the issue? If so you'll want to add closes #14


Support translation/rotation/scaling of environment objects

Use a gizmo and translate/rotate/scale environment objects like traditional 3D editing environments.

34d67b82ad7b53ac9163f7c3c90401e67b6afd2c Gizmo now tracks the selected object, getting closer to gizmo useage.

Closed by #19

Latest commits with the 4way viewport have disabled the gizmos. Seems mouse events aren't going through.


SB-1138: Implement tests for raw proxy/group repositories

Thanks for the advice and help, guys! :)


Bugfix temporal sub-windows period

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.5%) to 81.372%

Totals

Change from base Build<PHONE_NUMBER>7:
0.5%

Covered Lines:
2444

Relevant Lines:
2870

💛  - Coveralls

@wpreimes, @nfb2021  can I merge Daniel's changes, or you still want to take a look at them?

lgtm