Kata: uprade to 3.2.0.azl1

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] If you are adding/removing a .spec file that has multiple-versions supported, please add @microsoft/cbl-mariner-multi-package-reviewers team as reviewer (Eg. golang has 2 versions 1.18, 1.21+)
[x] Ready to merge

Summary

What does the PR accomplish, why was it needed?
Upgrade kata and kata-cc to 3.2.0.azl1
Change Log

kata-containers
kata-containers-cc
kata-packages-uvm

Does this affect the toolchain?

NO
Test Methodology

kata upgrade: https://dev.azure.com/mariner-org/mariner/_build/results?buildId=562013&view=results
buddy build: https://dev.azure.com/mariner-org/mariner/_build/results?buildId=562036&view=results
image build: https://dev.azure.com/mariner-org/mariner/_build/results?buildId=562229&view=results
test: https://dev.azure.com/mariner-org/mariner/_build/results?buildId=562258&view=results

Holding onmerging this due to https://github.com/microsoft/azurelinux/actions/runs/8942080803/job/24563866266?pr=9006#step:9:72. Trying to upload sources and cargo to mariner cache


Property doesn't render correctly if property, used as a part of it is mapped to the label

Discussed in https://github.com/serilog-contrib/serilog-sinks-grafana-loki/discussions/137

Originally posted by Depechie October  7, 2022
Was wondering if we are doing something wrong, or misjudging the features?
Currently we use Serilog to format our log entries and use the loki sink to push it in json format.
If we look at a result in Loki we get this

If we now use the propertiesAsLabel feature we get following result

So the actual properties are indeed transformed, but that seems to also impact the payload structure... the Message line now no longer maps the data but only show the placeholders, like {RequestPath}.
Is this expected behaviour?
Which version of Serilog.Sinks.Grafana.Loki are you using?
v8.0.0

The problem is TextFormatter gets LogEvent without properties, that will be mapped to labels. This behavior exists because we don't want to create duplicates (properties, which would be both labels and body rendering).
So the way should be found how pass it to formatter and use for message rendering, but not render as a part of the body

Seems like it is not possible to do with current abstractions, should be a point of review and breaking changes

Honestly for our use case leaving the labels as properties (and thus having a "duplicate" would be an acceptable solution. At least one we could opt in too. Basically leave them in the property list and just extract them towards labels, it's how it used to work for v7 and earlier it seems.

Honestly for our use case leaving the labels as properties (and thus having a "duplicate") would be an acceptable solution. At least one we could opt in too. Basically leave them in the property list and just extract them towards labels, it's how it used to work for v7 and earlier it seems.
Say an option that opts out of this partitioning: https://github.com/serilog-contrib/serilog-sinks-grafana-loki/blob/master/src/Serilog.Sinks.Grafana.Loki/LokiBatchFormatter.cs#L185

Yep, this is the problem. To formatter we pass the properties, were not mapped to labels. So while formatter tries to render a Message (for example), rendering input doesn't consist of all needed tokens for rendering.
This is a reason why it is not possible without breaking contracts or scary workarounds fix it in v8.
But this is the point of fix for v9

I'll whip up a quick fork for our internal use that could be released in the v8 line of versions if you want. It's a bit of a workaround (the opt-out) but it also doesn't break anything.

Yep, see your solution. Seems nice as a workaround for now. Let me think

Cool, if you need me to rename the parameter or anything like that let me know.

Thanks for your work, @EraYaN!
I think we should add it to the current versions as a workaround for the situation. Will prepare the release

Thanks once again!
Released as v8.1.0


recaptcha

add support for Recaptcha
you could steal what I did off staticman.

Somehow related (spam block) I've added experimental support for Akismet in commits https://github.com/shaftoe/static-comments/commit/58850b3f48444412ed19be3d9215d17768d1e9af and https://github.com/shaftoe/static-comments/commit/c69ad6bf38483e60ba29a37215c63fcdd82dd27f, it does the job for me for the time being.


Fix to support removal of --short option from kubectl version

kubectl version --short is no longer supported at 1.28. Switching to using --output=json and parsing this with tr and sed.

@greg-wootton Thank you for this PR.  Unfortunately, the project team is a little underhanded at the moment due to illness and other unavoidable absences.  However, we hope to operational again by next week.  At that point, we will review this PR and sort out a plan for incorporating it into project.  We'll keep you updated.
Regards,
Greg

We decided to handle this issue with slightly different syntax (pushed as #652).  Abandoning this PR.


Unable to do an rpc with a function that uses arguments annotated with an typed Array

Godot version
Godot_v4.0-beta6_win64
System information
Windows 11
Issue description
Godot gives me an error when I try to execute an rpc function that has an argument annotated as Array[int], but it works fine when I remove the type annotation or change it to just Array. The data I'm sending is an Array[int].
This is the code of the function (it doesn't work when it has the argument type annotation):
@rpc
func client_print_array(host_array: Array[int]):
    print("Host array: ", host_array)
    print("My array: ", self.array)

When I try to trigger the rpc from the host it gives me this error:
E 0:00:04:0771   _process_rpc: RPC - 'Node(Main.gd)::client_print_array': Cannot convert argument 1 from Array to Array
  <C++ Source>   modules/multiplayer/scene_rpc_interface.cpp:294 @ _process_rpc()

Removing the annotation fixes the problem.
Steps to reproduce

Open the Minimal reproduction project
Run the projects as 2 instances
In the first instance press "Host"
In the second instance press "Join"
In the first (host) instance press "Send message"
Observe errors in the debugger.

E 0:00:04:0771   _process_rpc: RPC - 'Node(Main.gd)::client_print_array': Cannot convert argument 1 from Array to Array
<C++ Source>   modules/multiplayer/scene_rpc_interface.cpp:294 @ _process_rpc()

Modify the Main.gd script by removing type annotation on line 28
Repeat steps 1-5
Observe expected messages in the log output

Host array: [1, 2, 3, 4]
My array: [4, 3, 2, 1]

Minimal reproduction project
RpcIntArrayIssueExample.zip

4.1 hasn't been released, can you be more explicit?

Sorry, the commit is 8d501a2dc31f3bef6d5a7f6b0d060c8915082011

I think the reason is the same as #76841: binary serialization does not currently support typed arrays, so type information is lost in multiplayer.
Text serialization:
https://github.com/godotengine/godot/blob/eb86dabee07e8dfce3b06cbd557b50b74afd3d6c/core/variant/variant_parser.cpp#L1927-L1958
Binary serialization:
https://github.com/godotengine/godot/blob/eb86dabee07e8dfce3b06cbd557b50b74afd3d6c/core/io/marshalls.cpp#L1590-L1609

@dalexeev , @akien-mga , I can still reproduce this one on v4.2.
This is causing an error:
@rpc("call_remote", "any_peer", "reliable")
func characterclasscomponent_sync_class_change(n: String, c: Array[String]):

E 0:00:29:0132   _process_rpc: RPC - 'Node(sync.gd)::characterclasscomponent_sync_class_change': Cannot convert argument 2 from Array to Array
  <C++ Source>   modules/multiplayer/scene_rpc_interface.cpp:290 @ _process_rpc()


Unsupported protocol version 'Asterisk Call Manager/2.10.0 (openNMS)

Dears
I need conect with opennms plataforms  but i can´t because i have this error

The version of asterisk is the "13.15.0"
The version opennms plataform is "19.1.0"
The ami Configuration OpenNMS

asterisk-configuration.properties.txt
maillog-asterisk.txt
manager-conf-asterisk.txt
Helpme please
Thanks!

opennms?
You are going to have to provide more detail as to what you are trying
to do.
On 02/06/17 07:34, clandestinox wrote:

Dears
I need conect with opennms plataforms but i can´t because i have this
error
image
https://cloud.githubusercontent.com/assets/29131990/26701540/8f4a9ac6-46ef-11e7-87fb-8cbb3193b01b.png
The version of asterisk is the "13.15.0"
The version opennms plataform is "19.1.0"
Helpme please
Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/asterisk-java/asterisk-java/issues/200, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA29OOHCroRrZCPt5Hsg1oXjWW_IpTRkks5r_y6DgaJpZM4Ntky_.

The openNMS is a monitoring plataform  and this  use "asterisk-java" to connect  to asterisk pbx, when i want to connect  it rejects the connection
2017-06-02 10:24:13,179 WARN  [Asterisk-Java ManagerConnection-0-Reader-0] o.a.m.i.ManagerConnectionImpl: Unsupported protocol version 'Asterisk Call Manager/2.10.0'. Use at your own risk.
2017-06-02 10:24:14,190 ERROR [Thread-2113] o.o.n.n.a.AsteriskOriginateNotificationStrategy: Error originating call for notification.
org.opennms.netmgt.asterisk.utils.AsteriskOriginatorException: Authentication failure logging in Asterisk manager connection
at org.opennms.netmgt.asterisk.utils.AsteriskOriginator.originateCall(AsteriskOriginator.java:203) ~[opennms-asterisk-19.1.0.jar:?]
at org.opennms.netmgt.notifd.asterisk.AsteriskOriginateNotificationStrategy.send(AsteriskOriginateNotificationStrategy.java:68) [opennms-asterisk-19.1.0.jar:?]
at org.opennms.netmgt.notifd.ClassExecutor.execute(ClassExecutor.java:69) [opennms-services-19.1.0.jar:?]
at org.opennms.netmgt.notifd.NotificationTask.run(NotificationTask.java:267) [opennms-services-19.1.0.jar:?]
Caused by: org.asteriskjava.manager.AuthenticationFailedException: Authentication failed
at org.asteriskjava.manager.internal.ManagerConnectionImpl.doLogin(ManagerConnectionImpl.java:616) ~[asterisk-java.jar:?]
at org.asteriskjava.manager.internal.ManagerConnectionImpl.login(ManagerConnectionImpl.java:480) ~[asterisk-java.jar:?]
at org.asteriskjava.manager.internal.ManagerConnectionImpl.login(ManagerConnectionImpl.java:465) ~[asterisk-java.jar:?]
at org.asteriskjava.manager.DefaultManagerConnection.login(DefaultManagerConnection.java:309) ~[asterisk-java.jar:?]
at org.opennms.netmgt.asterisk.utils.AsteriskOriginator.originateCall(AsteriskOriginator.java:197) ~[opennms-asterisk-19.1.0.jar:?]
... 3 more

This does not appear to be an Asterisk-Java issue, but a configuration one. Feel free to re-open if you believe that is not the case.


Cleanup source-level ignores that work-around evolving language features

In the interest of integrating new Dart SDKs, we have had (and will continue to have) to make local changes to work around / accommodate evolving language features.  This issue tracks the effort to cleanup these ignores as the language continues to evolve and the need to use them is lessened by features or new idioms.
Analysis errors that are suppressed as work-arounds (or otherwise benefit from a revisit) should reference this issue like so:
Rectangle : growRectangle, // ignore: error_code, https://github.com/flutter/flutter/issues/5771

To help keep us honest, we'll add a count of ignored issues to flutter analyze (and possibly add it to the dashboard) [#5772].
References:

PR where a lot of this conversation began: https://github.com/flutter/flutter/pull/5689
Doc exploring language ideas:  Flutter covariant callbacks

FYI @leafpetersen,@jmesserly

I'm not sure what to do with this bug.  @pq  is this still something you want to track?

Maybe not.  I think @Hixie is tracking this elsewhere.  Let's close.  These are only the tip of the iceberg anyway.


discuss: add apachecon toc and articles

Discuss description

Now
First, add an page in the events as the table of contents shared by ApacheCon activities.
Then, add a title, description, sharer, etc. about the shared content, and add a link to an article page with no other link entry.
The article page will be similar to the current existing blog page and contain both articles and videos.
This can be quickly implemented and available on the official website.

Future
In the future, the active directory page will consider using custom pages to achieve a more flexible and friendly layout.

Screenshots of the issue
TOC

Articles (Just like blog)

It would be better to put ApacheCon 2021 to Blog, as there are many

I think ApacheCon Asia 2021 video would be more appropriate in Events or use a separate tab on nav. What do you think?

How about designing a Resource template?

How about designing a Resource template?

good idea~! In fact we have several types of articles now, blogs, events, releases and so on, a distinction should indeed be made for them.
how about improve them in new prs, and we can first post the Apache con online?
@Serendipity96 @juzhiyuan

LGTM, need @Serendipity96's confirmation


Add onAuthenticationFailure callback to Authenticator

Add onAuthenticationFailure() callback to the Authenticator lifecycle methods so that authenticator instances are made aware of authentication failures as well - which would truly complete the lifecycle callbacks.

Thanks for the PR @iyogi!
Have you signed the Contributor License Agreement for contributions to DataStax open source projects? If not you can find it at https://cla.datastax.com/. Thanks!

Thanks for the PR @iyogi!
Have you signed the Contributor License Agreement for contributions to DataStax open source projects? If not you can find it at https://cla.datastax.com/. Thanks!

Yes, have just signed it.


Project OpenStreetMap or Bing maps on point clouds

How can I project OpenStreetMap or Bing maps on my point clouds? Or show my point cloud toghter with some imagery or map. Just to have a sense of where my point cloud is located on a map. There is an example at: http://potree.org/demo/experimental/potree_map_projections/examples/viewer_proj.html

very nice

To enable the minimap on Potree v1.6 you just have to add the projection to your cloud or to edit the "cloud.js" of your converted cloud and add the projection there.

@MamatsasDhmhtrhs Hi, I am new to the georeferencing side. So will you provide the steps how to get point cloud referenced and how to get the minimap in potree please

Hello @nikhilbv,
There are plenty of ways to georeference your point cloud.
To geo-reference the point cloud, you may use the 'Point-pair based alignement' tool, see:
https://www.cloudcompare.org/doc/wiki/index.php?title=Align
You can use this tool with only one cloud and input the reference points (e.g. GPS ones) manually.
Another option is to create a cloud with the GCPs and use it as second cloud. You can find  also videos on youtube (simply search for 'cloudcompare georeference' on youtube.
Also it depends on the method you used to capture/create your point cloud if you created it by you own.
To check if your cloud is georeferenced open it with CloudCompare click on 'Point list picking' and click on your cloud. If you see something like that:

The pointcloud is georeferenced.
The minimap will be automaticaly be enabled if you insert the georeference system to your cloud
-or-
You can edit the "cloud.js" file from your converted pointcloud and add the projection there as shown below in PROJ.4 format

(Thats the GGRS87 / Greek Grid. If you want to find yours go to https://epsg.io/)
Thats all.

@nikhilbv
I can't see your output. Try to reupload it.
What do you mean when you say basetiles ?
Take a look at https://mamatsasdhmhtrhs.github.io source code

Hi,
Base tiles are the tiles in the minimap. Mine is plain and your minimap
has base tiles.
I am struck with getting that in my minimap.

On Wed, Jul 25, 2018 at 1:00 PM, MamatsasDhmhtrhs<EMAIL_ADDRESS>wrote:

@nikhilbv https://github.com/nikhilbv
I can't see your output. Try to reupload it.
What do you mean when you say basetiles ?
Take a look at https://mamatsasdhmhtrhs.github.io http://url source code
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/potree/PotreeConverter/issues/193#issuecomment-407661438,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AmhFCE-evXqXznQRZN_pQaAQpH3XoR58ks5uKB6agaJpZM4Ia6_X
.

Hi,
As I am new to georeferencing side, I did not notice that I had got base
tiles in minimap.
But the code which you gave is for EPSG 4326 or WGS 84 coordinates. How do
I change it to EPSG 3857 or WGS 84 / Pseudo-Mercator coordinates.
Because I have only  EPSG 3857 or WGS 84 / Pseudo-Mercator coordinates as
references.
On Wed, Jul 25, 2018 at 2:06 PM, Nikhil B V<EMAIL_ADDRESS>wrote:

Hi,
Base tiles are the tiles in the minimap. Mine is plain and your minimap
has base tiles.
I am struck with getting that in my minimap.

On Wed, Jul 25, 2018 at 1:00 PM, MamatsasDhmhtrhs <
<EMAIL_ADDRESS>wrote:

@nikhilbv https://github.com/nikhilbv
I can't see your output. Try to reupload it.
What do you mean when you say basetiles ?
Take a look at https://mamatsasdhmhtrhs.github.io http://url source
code
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/potree/PotreeConverter/issues/193#issuecomment-407661438,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AmhFCE-evXqXznQRZN_pQaAQpH3XoR58ks5uKB6agaJpZM4Ia6_X
.

The last 2 lines from my previous messenge
(Thats the GGRS87 / Greek Grid. If you want to find yours go to https://epsg.io/)
😁

Used epsg.io to grab the proj4 text for the coordinate system in my pointcloud however it is putting the minimap location to a different part of the world, typically the northern hemisphere.
Anyone know if this struggles with minus values?

@ianh88
This shouldn't have any problem with north/ south hemisphere. Are you sure you did the search correctly?

@ianh88
This shouldn't have any problem with north/ south hemisphere. Are you sure you did the search correctly?

Hi, sorry missed this one. Yes figured it out. The UTM formatting just wasn't correct. Working fine now.
I would like to have the tiles off by default in the minimap. Do you know of a way to ensure the toggle is set to off in the potree.js file?
ifh88

@ianh88
Just delete the projection from "cloud.js" of the specific cloud. The cloud will remain georeferenced but the tile won't be visible in the minimap.


92 calculate eta

Can create order, can't view status page.

can now view order, but the IP banning has stopped working. I enter a wrong password and the terminal report:
Date: Fri Sep 15 2017 15:36:23 GMT+1200 (NZST)
Error Code: 003
Error Message: Unable to fetch/update Polymorph Order
But there is no failed log being recorded to mongo. And therefore no blacklist entry being created.


Remove Yeoman dependency

Yeoman is a fairly heavy dependency and the cli takes forever to install. It would be nice if we rolled all generator functions directly into the cli project in a more lightweight fashion.

This is done in the 2.0 version. We are now just using Commander and Metalsmith.


Mount Dropbox "Failed" Message after allowing access to the files and folders in your Dropbox

I successfully downloaded File System for Dropbox. When I run the app, and attempt to mount my Dropbox account, I successfully get logged in to Dropbox, but then get a failed message after giving permission to Allow access to the files and folders in your Dropbox. When I look at the Dropbox linked applications list, File System for ChromeOS shows in the list as Access Type: Full Dropbox.

@hcoffey Probably, some error occurred. Could you confirm the debug log on the console tab of the DevTools by the steps: https://github.com/yoichiro/chromeos-filesystem-dropbox/wiki/How-to-Capture-the-Console-Log-of-DevTools

I close this issue because @hcoffey didn't provide the logs to fix this issue.


Support verifying SBOM of a container image at admission

What would you like to be added?
A Software Bill of Materials (SBOM) for a container image is a document that provides details about the components used to build a container image. It is a key building block in software security and software supply chain risk management. Ratify should support verifying SBOM of a container image and provide data for admission policy to make decision. The SBOM can be in SPDX format or in CycloneDX format.
Scenario-1: Users can define an admission policy that can verify the integrity and authenticity of a SBOM during deployment. This policy can audit and prevent SBOM evaluation if the SBOM is not signed.
Scenario-2: Users can define an admission policy that can evaluate a SBOM of a container image during deployment. This policy can audit and prevent image deployment if the SBOM contains software with non-compliant licenses.
Scenario-3: Users can define an admission policy that can evaluate a SBOM of a container image during deployment. This policy can audit and prevent image deployment if the SBOM contains software with known vulnerabilities.
Anything else you would like to add?
Detailed experience will be updated later.
Are you willing to submit PRs to contribute to this feature?

[ ] Yes, I am willing to implement it.

Spent some time reading on existing SBOM validations and would like to share my learnings. Feel free to add any details or corrections.
Tool to generate SBOM
There are variety of tools that can generated SBOM. Some tools like syft can generate both spdx and cyclonedex format. Microsoft sbom tool generates SPDX 2.2 compatible SBOMs.
- Syft
- microsoft/sbom-tool/
- Tern
- Kubernetes BOM tool
- spdx-sbom-generator
SBOM Format
The most popular formats are Spdx and Cyclonedx . Spdx generated are in json format, Cyclonedx is in xml format.
still need to compare the SPDX output generated by different tools ( e.g. Syft vs microsoft/sbom-tool)
Tools that consumes SBOM
- Synk ( Snyk’s backend takes an SBOM and returns vulnerability data)
- Kyverno (sample policy to validate Cyclonedx SBOM exist)
- Grype ( https://github.com/anchore/scan-action invokes the Grype scanner and returning the vulnerabilities found, and optionally fail if a vulnerability is found with a configurable severity level)
- Ratify

Discussion items:

which format do we need to support,  which ones does our customer use?
Should we support one to start with?
Ratify currently have two verifiers that looks at the spdx sboms. License checkers verifier allows user to configure a deny list, and sbom verifier validates the schema of the SBOM is valid). For Ratify, should we look into adding more verifiers to validate against the SBOM? or should we be invoking a library like Grype.
It is a common requirement to consume SBOM and report vulnerability, there are also a few mentions in various talks/blogs that tools should be able to detects drift/changes in package/license from one build to the next. Is this use case in scope for Ratify?

@susanshi Some answers from my side:

The format we should support is SPDX and Cyclonedx.
This is related to implementation, so I suggest a design doc to cover it.
Report vulnerability based on SBOM is not the scope for this policy and Ratify. The three scenarios in this issue are the current scope of evaluating SBOM. There are tools that can generate vuln reports based on SBOM, which can improve the efficiency and experience of generating vuln reports, otherwise tools need to pull and analyze image layers for vulnerabilities, however SBOM can provide the ingredients already. The 3rd scenario that is described this issue can help prevent vulnerable image if some software is reported vuln after vuln reports are generated.


[HELM]: Publish Helm 0.2.9

As per the issue, this PR

pushes the helm 0.2.9 version
and also prepare the chart for release 0.3.0

cc: @xiangfu0

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 0.00%. Comparing base (59551e4) to head (8010935).
Report is 499 commits behind head on master.

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #13230       +/-   ##
=============================================
- Coverage     61.75%    0.00%   -61.76%
=============================================
  Files          2436     2459       +23
  Lines        133233   135346     +2113
  Branches      20636    20981      +345
=============================================
- Hits          82274        0    -82274
- Misses        44911   135346    +90435
+ Partials       6048        0     -6048

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <ø> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <ø> (ø)

java-11
?

java-21
0.00% <ø> (-61.63%)
:arrow_down:

skip-bytebuffers-false
0.00% <ø> (-61.75%)
:arrow_down:

skip-bytebuffers-true
?

temurin
0.00% <ø> (-61.76%)
:arrow_down:

unittests
?

unittests1
?

unittests2
?

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

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


[tesseract:x64-windows] build failure "WebP::WebP not found"

Host Environment

OS: Windows Version 10.0.19043.1055
Compiler: Visual Studio 2019 Community

To Reproduce
>vcpkg install tesseract

Failure logs
Starting package 1/1: tesseract:x64-windows
Building package tesseract[core]:x64-windows...
[DEBUG] CreateProcessW("C:\vcpkg\downloads\tools\cmake-3.20.2-windows\cmake-3.20.2-windows-i386\bin\cmake.exe" "-DALL_FEATURES=cpu-independed;training-tools;" -DCURRENT_PORT_DIR=C:/vcpkg/ports/tesseract -D_HOST_TRIPLET=x64-windows -DFEATURES=core -DPORT=tesseract -DVCPKG_USE_HEAD_VERSION=0 -D_VCPKG_DOWNLOAD_TOOL=BUILT_IN -D_VCPKG_EDITABLE=0 -D_VCPKG_NO_DOWNLOADS=0 -DCMD=BUILD -DDOWNLOADS=C:/vcpkg/downloads -DTARGET_TRIPLET=x64-windows "-DTARGET_TRIPLET_FILE=C:\vcpkg\triplets\x64-windows.cmake" -DVCPKG_BASE_VERSION=2021-05-05 -DVCPKG_CONCURRENCY=13 -DVCPKG_PLATFORM_TOOLSET=v142 -DGIT=c:/code/depot_tools/git.bat -DVCPKG_ROOT_DIR=C:/vcpkg -DPACKAGES_DIR=C:/vcpkg/packages -DBUILDTREES_DIR=C:/vcpkg/buildtrees -D_VCPKG_INSTALLED_DIR=C:/vcpkg/installed -DDOWNLOADS=C:/vcpkg/downloads -DVCPKG_MANIFEST_INSTALL=OFF -P "C:\vcpkg\scripts\ports.cmake")
-- Using C:/vcpkg/downloads/tesseract-ocr-tesseract-4.1.1.tar.gz
-- Cleaning sources at C:/vcpkg/buildtrees/tesseract/src/4.1.1-81819c4317.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source C:/vcpkg/downloads/tesseract-ocr-tesseract-4.1.1.tar.gz
-- Applying patch fix-tiff-linkage.patch
-- Using source at C:/vcpkg/buildtrees/tesseract/src/4.1.1-81819c4317.clean
CMake Deprecation Warning at scripts/cmake/vcpkg_check_features.cmake:182 (message):
  calling `vcpkg_check_features` without the `FEATURES` keyword has been
  deprecated.

      Please add the `FEATURES` keyword to the call.
Call Stack (most recent call first):
  ports/tesseract/portfile.cmake:15 (vcpkg_check_features)
  scripts/ports.cmake:139 (include)

-- Found external ninja('1.10.2').
-- Configuring x64-windows
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:105 (message):
    Command failed: ninja -v
    Working Directory: C:/vcpkg/buildtrees/tesseract/x64-windows-rel/vcpkg-parallel-configure
    Error code: 1
    See logs for more information:
      C:\vcpkg\buildtrees\tesseract\config-x64-windows-out.log

Call Stack (most recent call first):
  scripts/cmake/vcpkg_configure_cmake.cmake:323 (vcpkg_execute_required_process)
  ports/tesseract/portfile.cmake:25 (vcpkg_configure_cmake)
  scripts/ports.cmake:139 (include)

[DEBUG] cmd_execute_and_stream_data() returned 1 after 16813596 us
Error: Building package tesseract:x64-windows 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: tesseract:x64-windows
  Vcpkg version: 2021-05-05-9f849c4c43e50d1b16186ae76681c27b0c1be9d9

Additionally, attach any relevant sections from the log files above.
[DEBUG] C:\A\1\121\s\src\vcpkg\install.cpp(511)
[DEBUG] Exiting after 20379374 us (20331897 us)

-----------
[\vcpkg\buildtrees\tesseract\config-x64-windows-out.log]
...
-- Configuring done
CMake Error at C:/vcpkg/scripts/buildsystems/vcpkg.cmake:537 (_add_executable):
  Target "tesseract" links to target "WebP::WebP" but the target was not
  found.  Perhaps a find_package() call is missing for an IMPORTED target, or
  an ALIAS target is missing?
Call Stack (most recent call first):
  CMakeLists.txt:515 (add_executable)

Additional context
libwebp IS installed.
config-x64-windows-out.log

Similar happens with port vtk:
CMake Error at C:/vcpkg/scripts/buildsystems/vcpkg.cmake:575 (_add_library):
  Target "IOImage" links to target "WebP::WebP" but the target was not found.
  Perhaps a find_package() call is missing for an IMPORTED target, or an
  ALIAS target is missing?
Call Stack (most recent call first):
  CMake/vtkModule.cmake:3333 (add_library)
  IO/Image/CMakeLists.txt:41 (vtk_module_add_module)

Hi @traumedy
Thanks for posting this issue.
Can you remove libwebp and update vcpkg to the latest to rebuild? I cannot reproduce this on my machine.

vcpkg was up to date (git pull and bootstrap-vcpkg), however, removed libwebp, removed buildtrees directory and tesseract (and vtk) compiles successfully now.
Closing issue.
Thank you for your time.


nographics breaks build

Bug description
Unity 2019.4.11.f1
WebGL, Android, Linux, Windows.
both unity-builder v1.5 and 2.0-alpha-1 which introduced mandatory nographics breaks my build which worked before without nographics (I tried to set nographics in custom param before when build was fine and it was actually breaking the build too).

ALSA lib confmisc.c:767:(parse_card) cannot find card '0'
ALSA lib conf.c:4732:(_snd_config_evaluate) function snd_func_card_driver returned error: No such file or directory
ALSA lib confmisc.c:392:(snd_func_concat) error evaluating strings
ALSA lib conf.c:4732:(_snd_config_evaluate) function snd_func_concat returned error: No such file or directory
ALSA lib confmisc.c:1246:(snd_func_refer) error evaluating name
ALSA lib conf.c:4732:(_snd_config_evaluate) function snd_func_refer returned error: No such file or directory
ALSA lib conf.c:5220:(snd_config_expand) Evaluate error: No such file or directory
ALSA lib pcm.c:2642:(snd_pcm_open_noupdate) Unknown PCM default
FMOD failed to initialize the output device.: "Error initializing output device. " (60)
(Filename: ./Modules/Audio/Public/AudioManager.cpp Line: 1489)
Forced to initialize FMOD to to the device driver's system output rate 48000, this may impact performance and/or give inconsistent experiences compared to selected sample rate 48000
(Filename: ./Modules/Audio/Public/AudioManager.cpp Line: 1425)
ALSA lib confmisc.c:767:(parse_card) cannot find card '0'
ALSA lib conf.c:4732:(_snd_config_evaluate) function snd_func_card_driver returned error: No such file or directory
ALSA lib confmisc.c:392:(snd_func_concat) error evaluating strings
ALSA lib conf.c:4732:(_snd_config_evaluate) function snd_func_concat returned error: No such file or directory
ALSA lib confmisc.c:1246:(snd_func_refer) error evaluating name
ALSA lib conf.c:4732:(_snd_config_evaluate) function snd_func_refer returned error: No such file or directory
ALSA lib conf.c:5220:(snd_config_expand) Evaluate error: No such file or directory
ALSA lib pcm.c:2642:(snd_pcm_open_noupdate) Unknown PCM default
FMOD failed to initialize the output device.: "Error initializing output device. " (60)
(Filename: ./Modules/Audio/Public/AudioManager.cpp Line: 1489)
FMOD initialized on nosound output
PlayerPrefs - Creating folder: /github/home/.config/unity3d/niwrad
PlayerPrefs - Creating folder: /github/home/.config/unity3d/niwrad/niwrad
Unable to load player prefs
[Subsystems] No new subsystems found in resolved package list.
[Package Manager] Done registering packages in 0.56s seconds
Targeting platform: WebGL
Refreshing native plugins compatible for Editor in 74.56 ms, found 0 plugins.
Preloading 0 native plugins for Editor in 0.00 ms.
Initialize engine version: 2019.4.11f1 (2d9804dddde7)
[Subsystems] Discovering subsystems at path /opt/unity/Editor/Data/Resources/UnitySubsystems
[Subsystems] Discovering subsystems at path /github/workspace/Assets
Forcing GfxDevice: Null
NullGfxDevice:
Version:  NULL 1.0 [1.0]
Renderer: Null Device
Vendor:   Unity Technologies
Segmentation fault (core dumped)
Build failed, with exit code 139

https://github.com/louis030195/niwrad/runs/1370864071?check_suite_focus=true
How to reproduce
I have no clue, it can be related to my dependencies which need graphics some says ?
manifest.json:
{
  "dependencies": {
    "com.unity.2d.sprite": "1.0.0",
    "com.unity.2d.tilemap": "1.0.0",
    "com.unity.ads": "3.4.9",
    "com.unity.ext.nunit": "1.0.0",
    "com.unity.ide.rider": "2.0.7",
    "com.unity.ide.visualstudio": "2.0.2",
    "com.unity.ide.vscode": "1.2.2",
    "com.unity.inputsystem": "1.0.0",
    "com.unity.test-framework": "1.1.16",
    "com.unity.textmeshpro": "2.1.1",
    "com.unity.timeline": "1.2.17",
    "com.unity.ugui": "1.0.0",
    "com.unity.modules.ai": "1.0.0",
    "com.unity.modules.androidjni": "1.0.0",
    "com.unity.modules.animation": "1.0.0",
    "com.unity.modules.assetbundle": "1.0.0",
    "com.unity.modules.audio": "1.0.0",
    "com.unity.modules.cloth": "1.0.0",
    "com.unity.modules.director": "1.0.0",
    "com.unity.modules.imageconversion": "1.0.0",
    "com.unity.modules.imgui": "1.0.0",
    "com.unity.modules.jsonserialize": "1.0.0",
    "com.unity.modules.particlesystem": "1.0.0",
    "com.unity.modules.physics": "1.0.0",
    "com.unity.modules.physics2d": "1.0.0",
    "com.unity.modules.screencapture": "1.0.0",
    "com.unity.modules.terrain": "1.0.0",
    "com.unity.modules.terrainphysics": "1.0.0",
    "com.unity.modules.tilemap": "1.0.0",
    "com.unity.modules.ui": "1.0.0",
    "com.unity.modules.uielements": "1.0.0",
    "com.unity.modules.umbra": "1.0.0",
    "com.unity.modules.unityanalytics": "1.0.0",
    "com.unity.modules.unitywebrequest": "1.0.0",
    "com.unity.modules.unitywebrequestassetbundle": "1.0.0",
    "com.unity.modules.unitywebrequestaudio": "1.0.0",
    "com.unity.modules.unitywebrequesttexture": "1.0.0",
    "com.unity.modules.unitywebrequestwww": "1.0.0",
    "com.unity.modules.vehicles": "1.0.0",
    "com.unity.modules.video": "1.0.0",
    "com.unity.modules.vr": "1.0.0",
    "com.unity.modules.wind": "1.0.0",
    "com.unity.modules.xr": "1.0.0"
  }
}

Others:

Nakama Unity v2.4.0
TextMeshPro
UniTask
Protobuf (C++ version not Protobuf.NET)

Expected behavior
On unity-builder 1.2 WebGL, Windows and Linux were building properly (no -nographics), Android was failing (NDK stuff).
I expect it to build.
Additional details
My csharp project builder
https://github.com/louis030195/niwrad/blob/master/Assets/Scripts/Editor/ProjectBuilder.cs

This is a known bug with Unity itself. Any version after 2019.3 doesn't work on Linux. They don't know when they will support it.
The -nographics flag is a workaround that we didn't prefer, but it's the only way it works.
If you're using global illumination or other techniques that require graphics processing during build time, you're out of luck at the moment.

See also See also unity-actions#66 and unity-actions#70.

Closing this as I don't think there's anything we can do except add windows and macos based images in the docker repo.


Detect lightweight java language server

Fixes #12
Detects if java ls started in lightweight mode and prompts user to switch to standard. The user ends up getting a second prompt from vscode-java to ask if they are sure.

@fbricon could you give us feedback please about this fix.

See https://github.com/microsoft/vscode-java-debug/commit/20cd68cbb112f170b4b0aaf849604cf37853091a for reference

I agree with trying to avoid starting up lsp4mp if the java language server is in lightweight mode.
I opened an issue https://github.com/redhat-developer/vscode-java/issues/1571 to try and get some clarifications about the best way to go about implementing this. I can detect the current server mode on start up. But since vscode-java can start in lightweight and switch over to standard after activation, if we detect lightweight at start up it can display a warning to switch to standard even though vscode-java is in the process of switching over to Standard which i dont think makes for a good UX.

After some discussion in this issue: https://github.com/redhat-developer/vscode-java/issues/1571 It seems that it can be problematic to check the java language server mode immediately on extension activation. This is because when you import a new project vscode-java starts off in lightweight mode and prompts the user to switch to standard.
The flow for importing a new java project seems to be:
Someone imports a new Java project. -> Java language server starts in Lightweight. They are then prompted to import the project. If they click yes -> Java language server will switch to Hybrid as it starts up the Standard server, and then to Standard once the standard LS is ready.
So for new Java projects, if we check the mode immediately on extension activation it will return lightweight, even though the user could be in the process of switching to standard mode using the prompt from vscode. Which means we could end up displaying an unnecessary warning message.
For an example:
I click yes to import the project and switch to standard. But since immediately at activation vscode-java returns Lightweight we end up with a warning message from vscode-microprofile to switch to standard.

It seems for all the other Java extensions, they detect server mode only when java features are needed instead of immediately on extension activation.
My thoughts are that would could do something like only check the Java language server mode after a user has opened a file or something similar.
Any thoughts @fbricon @angelozerr @datho7561 ?

Update the PR to only start the MicroProfile language server once the java language server is in hybrid mode.
After testing https://github.com/redhat-developer/vscode-java/pull/1581 it fixed the problem when a new importing a project for the most part.
If someone selects yes to import a new project before vscode-java is done activating, they will no longer receive a second prompt while the standard server is in the process of launching.
You can still get two prompts when importing a project if the user has not selected Yes to import the project by the time vscode-java has finished activating. But i think this is acceptable?
Ex:

I think we can start that way.

Updated the options for the prompt to switch to Standard to be Yes and Later to match vscode-java.


Leverage more efficient get_*_key functions

Whenever we call riak_ql_ddl:get_partition_key or riak_ql_ddl:get_local_key we have access to the relevant DDL module name, so pass it along to avoid unnecessary calls to make_module_name.
Depends on basho/riak_ql#52

@macintux I'll look at this once the QL PR merges

:+1: 91f4d42

@borshop merge


Update, root of plugin manager

[ ] Add fisher update all to update all plugins concurrently

What's wrong with
fisher -l | fisher update

nothing is wrong with fisher -l | fisher update, but I'm expecting some users who are not proficient with the cli to struggle with this.
Anyway, i'd suggest fisher update --all or fisher update -A instead, fisher update foo should be reserved for updating the foo plugin.

@jethrokuan, I suggest fisher update all which @bucaran suggest to me.
@bucaran, I did some research, the report is here:
Using xargs, -P means --max-procs:
fisher -l | cut -c 2- | xargs -n 1 -P 0 fish -c 'fisher update'

Using parallel (not installed by default in many distros probably):
setenv PARALLEL_ENV (functions fisher_update|perl -pe 's/\n/\001/')
fisher -l | cut -c 2- | parallel fisher_update

Differences (some taken from man parallel):

xargs deals badly with special characters (such as space, ' and ")
xargs can run a given number of jobs in parallel, but has no support for
running number-of-cpu-cores jobs in parallel.
 Have you actually tried that? What were your results? Maybe try running some benchmarks.
:+1:

@pickfire great stuff! parallel doesn't come with my distro. I'd personally like xargs as default, and parallel as part of a variable that can be set to choose the parallelization option.

I think I like xargs more as parallel have longer startup time (what they said so) but too bad xargs doesn't have any support for functions.

It seems like the xargs command still doesn't work, the parallel command works instead:
fisher -l | cut -c 2- | parallel fisher_update

For the current benchmarking:

fisher -l | cut -c 2- | parallel fisher_update: 14s
fisher -l | cut -c 2- | fisher_update -: 24s
fisher -l | cut -c 2- | xargs -d "\n" -P0 -I {} fish -c 'fisher_update {}': Command too long (FAIL)

I have 4 cores, updating debug, fishtape and spin.

@pickfire Make sure you always use fisher_update -.

@bucaran As I said, it won't work as there is * in front of the words, I really hope that it is deleted. Output:
> fisher l | fisher u -
fisher: ' debug' is not a valid name, path or URL.
fisher: ' fishtape' is not a valid name, path or URL.
fisher: ' spin' is not a valid name, path or URL.
No plugins were updated.

@pickfire That's a bug in 1.0.0-rc, and it's now fixed in my local build. We'll be pushing VERY soon :smile:

Ah, this works:
> fisher l -b | fisher u -
Updating >> debug
Updating >> fishtape
make: warning: jobserver unavailable: using -j1.  Add '+' to parent make rule.
Updating >> spin
Aye! 3 plugin/s updated in 25s

@pickfire That's sweet. If only parallel was installed by default everywhere.

Okay, I will submit a pull request for this.

I'm not sure if this will work: fisher list | xargs -n 1 -P20 fish -c "fisher u -"
It says for me:
No plugins were updated.
No plugins were updated.
No plugins were updated.
No plugins were updated.
No plugins were updated.
No plugins were updated.
No plugins were updated.
No plugins were updated.
No plugins were updated.
No plugins were updated.
No plugins were updated.
No plugins were updated.
No plugins were updated.

@jethrokuan, it won't work, there is no input for that.

For this, only parallelism for fisher update all is useless, I think I will add support the update feature.
And I need to split another update function to just update a single plugin.
I am not sure how to use the debug thing, for me I need to split:
            if set -q plugins[1]
                printf "%s\n" $plugins
            else
                __fisher_file
            end | __fisher_update_plugin

So that it can update concurrently but there is a fallback which is that those variables won't be shared with other functions.

@pickfire Update has an --all option now which is why I closed this issue. But we can create a new one to track the parallel feature. I can assign it to you too if you wish.


879 dev

Q
A

New feature?
[ X]

Related issues?
partially #879

What's in this Pull Request?
Termstore picker can now use the UI language of the user.
Details:
With the new Control Property useUserLanguage you can change the suggestion result language to the user Default Language.

I will change the default behavior for the suggestions so I will remove the additional Property


Trying to use with paymaster

I've added the custom method 'with_paymaster' to set paymaster params.
But then it fails when I try to send it as transaction wtih :
called Result::unwrap() on an Err value: ErrorResp(ErrorPayload { code: -32602, message: "Invalid params: invalid type: string "0x1234", expected a sequence.", data: None })
where 0x1234 was the value inside paymaster input params.
probably some issue with bytes serialization

my current workaroudn was to do:
fn serialize_foo<S: serde::Serializer>(bytes: &Bytes, serializer: S) -> Result<S::Ok, S::Error> {
    serializer.serialize_bytes(&bytes.0)
}

and then
pub struct PaymasterParams {
pub paymaster: Address,
#[serde(serialize_with = "serialize_foo")]
pub paymaster_input: Bytes,
}

Ah, right, serializing Bytes is different from serializing Vec<u8> in RLP. It's handled here for factory deps, but I forgot to implement the same fix for paymaster_input.
If you will have time to submit a fix, that'd be greatly appreciated. Otherwise, I'll try to work on it this week.


Update devcontainer-features.md

To give it a bit more context on naming. Following @Chuxel's suggestion in https://github.com/devcontainers/devcontainers.github.io/pull/17.

Thanks for the quick review @joshspicer!


Add details for time parameters

@navyasric,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@navyasric
The content guidance wiki says...
If you are the listed author for an Azure technical article, you need to use the private repository (azure-docs-pr?) for all content updates and new articles.
Closing this pull request so that you can make updates in the private repo.


billing: Support Zulip Standard free trial

[x] Make upgrade and billing system support free trial

[x] Update /billing to show that the plan is on Free trial and would be upgraded to Standard on this date.

[x] Backend support for downgrading from free trial

[x] UI for downgrading

[x] Message that shows FREE trial is enabled in /upgrade page

[x] Backend tests

[x] Frontend tests

Free Trial

Normal upgrade

@timabbott This is ready for a review.

I merged the first couple commits; will look at this in more detail tomorrow.  One thing I notice is we should probably have a .billing-loading set of classes rather than having 5 IDs listed for each CSS rule :).

@timabbott Yup. I will create a followup PR for cleaning some of that after this is merged :)

Merged, after fixing a s/would be/will be/ English issue in one of the strings.  I appreciate that it's off-by-default so we can continue staging changes cleanly.
Can we change trial length units to days, not months?  "60 days" is a lot clearer than "2 months", and I suppose would better support other lengths in the future?
I think we'll want to do more tweaks to the strings; maybe you can do a call with Trent to walk through the flow?  I feel like at the very least some of the capitalization around "Free Trial" is wrong, and we should always refer to it as "N day/month free trial" when we reference it, rather than just "free trial" in some places.


No able to install

Hi, It's throwing this error while installing:
go<EMAIL_ADDRESS>unrecognized import path "gitlab.nobody.run/tbi/core": reading https://gitlab.nobody.run/tbi/core?go-get=1: 404 Not Found
server response: 404 page not found
Could you please fix that?
Thank You!

Hi, we have fixed  the issue in 1.0.1 !
Regards,


Fix typos and errors in the code documentation

Fix some of the documentation errors currently failing our CI. I'm not sure what to make of the remaining ones, which complain about ambiguous references (that I think are not actually ambiguous):
File "src/repr/type_intf.ml", line 48, characters 11-20:
Reference to 'string' is ambiguous. Please specify its kind: val-string, val-string, val-string, val-string.
File "src/repr/type_intf.ml", line 51, characters 11-19:
Reference to 'bytes' is ambiguous. Please specify its kind: val-bytes, val-bytes, val-bytes, val-bytes.
File "src/repr/type_intf.ml", line 56, characters 6-16:
Reference to 'Unboxed' is ambiguous. Please specify its kind: module-Unboxed, module-Unboxed, module-Unboxed, module-Unboxed.
File "src/repr/type_intf.ml", line 107, characters 17-26:
Reference to 'Of_set' is ambiguous. Please specify its kind: module-Of_set, module-Of_set, module-Of_set, module-Of_set.
File "src/repr/type_intf.ml", line 137, characters 41-54:
Reference to 'empty' is ambiguous. Please specify its kind: type-empty, type-empty, type-empty, type-empty.
File "src/repr/type_intf.ml", line 144, characters 6-13:
Reference to '(|+)' is ambiguous. Please specify its kind: val-(|+), val-(|+), val-(|+), val-(|+).
File "src/repr/type_intf.ml", line 145, characters 6-23:
Reference to 'sealr' is ambiguous. Please specify its kind: val-sealr, val-sealr, val-sealr, val-sealr.
File "src/repr/type_intf.ml", line 150, characters 11-18:
Reference to '(|+)' is ambiguous. Please specify its kind: val-(|+), val-(|+), val-(|+), val-(|+).
File "src/repr/type_intf.ml", line 150, characters 49-57:
Reference to 'sealr' is ambiguous. Please specify its kind: val-sealr, val-sealr, val-sealr, val-sealr.
File "src/repr/type_intf.ml", line 199, characters 16-23:
Reference to '(|~)' is ambiguous. Please specify its kind: val-(|~), val-(|~), val-(|~), val-(|~).
File "src/repr/type_intf.ml", line 200, characters 17-34:
Reference to 'sealv' is ambiguous. Please specify its kind: val-sealv, val-sealv, val-sealv, val-sealv.
File "src/repr/type_intf.ml", line 205, characters 37-44:
Reference to '(|~)' is ambiguous. Please specify its kind: val-(|~), val-(|~), val-(|~), val-(|~).
File "src/repr/type_intf.ml", line 206, characters 6-14:
Reference to 'sealv' is ambiguous. Please specify its kind: val-sealv, val-sealv, val-sealv, val-sealv.
File "src/repr/type_intf.ml", line 353, characters 13-45:
Reference to 'generics' is ambiguous. Please specify its kind: section-generics, section-generics, section-generics, section-generics.
File "src/repr/type_intf.ml", line 400, characters 46-50:
Reference to 't' is ambiguous. Please specify its kind: type-t, type-t, type-t, type-t.
File "src/repr/type_intf.ml", line 419, characters 37-46:
Reference to 'random' is ambiguous. Please specify its kind: val-random, val-random, val-random, val-random.
File "src/repr/type_intf.ml", line 464, characters 25-43:
Reference to 'encode_json' is ambiguous. Please specify its kind: val-encode_json, val-encode_json, val-encode_json, val-encode_json.
File "src/repr/type_intf.ml", line 504, characters 34-42:
Reference to 'case0' is ambiguous. Please specify its kind: val-case0, val-case0, val-case0, val-case0.
File "src/repr/type_intf.ml", line 505, characters 34-42:
Reference to 'case1' is ambiguous. Please specify its kind: val-case1, val-case1, val-case1, val-case1.
... etc.

CC @Julow, any idea what's going on here?

This is a bug in Odoc. I desperately needed a clear reproduction for it, thanks !

This is being fixed in Odoc (https://github.com/ocaml/odoc/pull/761).
I've added some more fixes to references https://github.com/CraigFe/repr/pull/2, there's no fatal warnings left so this can be merged.

Thanks @julow and @maiste for the contributions! Merging now.


[CI] ConfigSpace stability on CI

Description of changes:

ConfigSpace must be installed after numpy is updated so it is not using 1.20.x coming from conda when installing - this is causing numpy compatibility errors:
ValueError: numpy.ffff size changed, may indicate binary incompatibility. Expected xxx from C header, got yyy from zzz

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-922-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-922/1/index.html


Any Plan to monitoring search and indexing rate.

Any plan to monitoring search rate / latency and index rate/latency  ?

@ishara hey there. I'm currently not actively developing kopf anymore, since amongst other things, elasticsearch 5.0 won't support site plugins(like kopf).
I'm currently developing another application to replace kopf(https://github.com/lmenezes/cerebro), but at this moment it does not have all of kopf features. hopefully it will have, and more.


Metrics for levenshtein spelling correction model

Wanted to calculate a metric score of a levenshtein model for the SpellRuEval dataset (F1 = 56.25 according to the doc page).
But python3 -m deeppavlov evaluate levenshtein_corrector_ru -d got the following error:
Traceback (most recent call last):
  File "/usr/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/home/vimary/code-projects/Pilot/deeppavlov/__main__.py", line 3, in <module>
    main()
  File "/home/vimary/code-projects/Pilot/deeppavlov/deep.py", line 87, in main
    train_evaluate_model_from_config(pipeline_config_path, to_train=False, start_epoch_num=args.start_epoch_num)
  File "/home/vimary/code-projects/Pilot/deeppavlov/core/commands/train.py", line 106, in train_evaluate_model_from_config
    if 'evaluation_targets' not in train_config and ('validate_best' in train_config
TypeError: argument of type 'NoneType' is not iterable```

Scores in the documentation were calculated using an external scorer that's not included in the library yet. We can share it on request.


Changed survey.survey_id to survey.id

Changed survey.survey_id to survey.id for clarity.  All tests were update.

Codecov Report

Merging #45 (8c92d4f) into main (5dc7e4b) will decrease coverage by 0.01%.
The diff coverage is 52.94%.

@@            Coverage Diff             @@
##             main      #45      +/-   ##
==========================================
- Coverage   73.07%   73.06%   -0.02%
==========================================
  Files         156      156
  Lines        7812     7808       -4
==========================================
- Hits         5709     5705       -4
  Misses       2103     2103

Impacted Files
Coverage Δ

mt_metadata/timeseries/channel.py
88.52% <ø> (ø)

...ata/timeseries/stationxml/xml_network_mt_survey.py
91.01% <ø> (ø)

tests/tf/test_survey.py
97.91% <ø> (ø)

tests/timeseries/test_survey.py
98.43% <ø> (ø)

mt_metadata/base/metadata.py
66.66% <12.50%> (-0.41%)
:arrow_down:

mt_metadata/base/schema.py
42.33% <50.00%> (ø)

...data/timeseries/filters/channel_response_filter.py
65.11% <100.00%> (ø)

mt_metadata/timeseries/survey.py
88.28% <100.00%> (ø)

tests/test_standards_exist.py
92.59% <100.00%> (ø)

tests/timeseries/stationxml/test_network.py
98.93% <100.00%> (ø)

... and 1 more

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 5dc7e4b...8c92d4f. Read the comment docs.


Implemented tree traversing api for DOM elements

Created  methods:
bfs = breadth-first search
dfs = depth-first-search . With some options: 'reverse' to search from right to left and 'order' to set the type of traversing algorithm -> Preorder, Postorder or Inorder.

Coverage increased (+0.03%) to 99.163% when pulling fbbe24163ecdee282e38320153e684cd4f538fe6 on antoniomdk:master into 34b95d498d7e951667802f1e6dc395cc80024fac on Hrabal:master.

I'm going to fix that issues.

Coverage increased (+0.03%) to 99.163% when pulling fea2637c86db226fb934d719f26673f380b0840a on antoniomdk:master into 34b95d498d7e951667802f1e6dc395cc80024fac on Hrabal:master.

Coverage increased (+0.03%) to 99.162% when pulling 67012b789ee9bf7a6bf480967f33946bd0df2fa2 on antoniomdk:master into 34b95d498d7e951667802f1e6dc395cc80024fac on Hrabal:master.

Coverage increased (+0.03%) to 99.162% when pulling 67012b789ee9bf7a6bf480967f33946bd0df2fa2 on antoniomdk:master into 34b95d498d7e951667802f1e6dc395cc80024fac on Hrabal:master.

Coverage increased (+0.03%) to 99.162% when pulling 67012b789ee9bf7a6bf480967f33946bd0df2fa2 on antoniomdk:master into 34b95d498d7e951667802f1e6dc395cc80024fac on Hrabal:master.

Coverage increased (+0.03%) to 99.162% when pulling 67012b789ee9bf7a6bf480967f33946bd0df2fa2 on antoniomdk:master into 34b95d498d7e951667802f1e6dc395cc80024fac on Hrabal:master.

Coverage increased (+0.03%) to 99.162% when pulling 67012b789ee9bf7a6bf480967f33946bd0df2fa2 on antoniomdk:master into 34b95d498d7e951667802f1e6dc395cc80024fac on Hrabal:master.


BENCH: move rank to be the last parameter

Moving rank to be the last parameter makes the dtype and nthreads the first two parameters and I think improves the output format. See this comment for an example.

LGTM

LGTM


User interface discussion

I hope you don't mind, @pdcgomes but I'd love to start a bit of a conversation about what the UI for XCActionBar might look like.
Initial thoughts are that matching Xcode's Open Quickly makes a lot of sense:

There might need to be a small settings icon somewhere on the panel, for the eventual customisable hotkeys and other prefs.
Thoughts, ideas, suggestions? I realise it's early days, but I'd like to contribute and building out a nice UI like this is probably a couple of hours once there's agreement on what should be done.
/cc @orta

I'm also a big fan of the UI of LaunchBar 6 if we wanted to go for a chunkier, less Xcodey look:

http://www.obdev.at/products/launchbar/index.html

No not at all, I think it's great that people are showing so much enthusiasm!
I'd say that ultimately making it look like Open Quickly is probably the way to go if the feature set remains similar, i.e., there's no hierarchical navigation of any sort and you're only allow to select an item from the list.
Having that said, I do have some ideas for actions for which the user can specify additional input, and also cases where the user "collect" things (blocks of text for example) and operate on them in bulk, meaning something a bit more flexible would be needed.
I'm also thinking of adding a "scoped" search where the user can type a prefix like s: or d:, etc. to narrow down search results to certain domains (for example s: would only search code snippets, d: would only operate on the debug console, etc.) a bit like Dash does. This would probably rely on having the typed prefix transform into a custom TokenField or something similar.
I'm personally a big fan of Alfred and have it setup like this:

As you can see, the settings icon (which is can actually be turned off) fits quite nicely (CMD+, works as expected too).
Just throwing my immediate thoughts out there so you have a rough idea of where I'm planning on taking this.

Given that there are a few types of actions at the minute, the open quickly-like would feel quite good ( you may even be able to outright re-use the classes too, negating the need for building UI )
Then there could be custom icons for

Menu Actions
Built in actions
Custom actions
Code snippets

Repurposing the Open Quickly classes for UI and interfaces for data providing and string matching would definitely reduce the code base and it's something to look at for sure.
But given future plans, it would probably not quite serve the purpose, so I'm happy to keep a separate UI implementation for it - also makes the whole thing a bit less dependent on Xcode internals (granted this is a plugin, but still...).
To make it look like Open Quickly we'd need:

Similar chrome (rounded window surrounding the text input area)
Tweak input field so that font size matches
Add "Clear" button to input field
No floating space between input field and results table
Results table expands/shrinks with results (up to fixed maximum)
Results table animates while expanding/shrinking
Tweak results cells so they match Open Quickly's
Come up with icons to represent the different kinds of actions (Font Awesome probably has some good enough placeholder assets we can use)

Think your reasoning is totally cool, can help with assets too if needed

I just saw this from Dave Verwer's newsletter and I already fell in love with it; kinds of feels the spot that CodePilot left.
My 2 cents in the UI talk I think going for the Open Quickly look & feel would work beautifully for the actions at hand and it can grow "easily".
If needed help I can contribute to the project with some of my :hourglass: can't help with assets because I suck at :art: though

OK, great! I'll try to squeeze in a little work on a PR for this tomorrow night when the kids are asleep. It sounds like going for a look consistent with "Open Quickly" is the logical choice, so I'll concentrate on that first :smile:

@esttorhe I won't have a massive amounts of time over the next few weeks but I'll try to bring some bring some structure and create a few issues/milestones based on some of the ideas that I've got right now. Based on that we can see how people could best contribute in a way that makes sense!
Either way, at this stage I'd say that the most relevant aspect is to gather feedback and ensure the foundation is rock solid and can be built upon with confidence.

A settings panel ( for things like the keyboard shortcut ) could be done using my Preferences plugin. I made it with this kind of thing in mind.

Sorry, took me a while to get to this.
I've checked your plugin, and it could make sense - especially if we could get other people to integrate it with their own plugins.
On the other hand...
I sort of like the approach of fully separating the settings into a separate process (Alfred and a few other apps do it), so I might just go with that approach - it'll also make up for a lighter and stabler plugin, I can simply rely on XPC or some other mechanism to pass data back and forth and can develop and test the settings app independently (which is always my preferred way).
I probably won't get to this until next week or so, so I'll give it some more thought.

Fair, I know this is a bit of an odd request, but I'd love to see this work go into a generic "XCOpenQuicklyWindow" pod. I'm currently replicating this kind of thing in another app.

I can make it a library or modular code, @orta, sure!
</semantics>

Hahaha :hear_no_evil: :speak_no_evil: :see_no_evil:


Is there any way to use ssl

Yes, with a reverse proxy, if you use Apache2 then this tut should do it: https://www.cherryservers.com/blog/how-to-install-and-configure-apache-reverse-proxy-server-with-ssl-encryption
Otherwise search for reverse proxy for nginx or the web server solution of your election.


Make HTTP verbs uppercase is developer portal

The issue has been fixed, will publish PR once Beta has been released.


Feat/pass options to init method

Hello @egidiusmengelberg ,
This PR should add the feature I mentioned here #1 .
Please review and let me know if it is good.

Looks good! Thanks!


[suggestion] way to read/write from blocks

It's possible to read and write from entities and items in hand, but it's currently not possible with blocks.
This would allow for modifying slates in spell circles on the fly (at least in a more practical way)
it would also allow for integrations with pedestals from other mods as an alternative to item frames
and finally would be useful for addons lol

Writing to Scrolls in world would also be quite nice <3 I could do "/patterns hex" in game!

I am planning for messing with blocks to be the purview of spell circles, this is pending the giant casting context refactor.

I think this is pretty well covered by HexGloop - does it still need to be in the base mod?

This post was prior HexGloop. Though being part of the base mod would be nice.

I semi purposefully haven't implemented any direct read/write block stuff in hexgloop since idk if it'd fit best in moreiotas, base hex, or just in gloop. Currently gloop just has pedestals which hold a non-despawning item entity and it has a framework for block reading/writing with loci, but nothing useable yet.

Me and Sam talked about this a bit ago, and I think we decided a good way to implement this from a player perspective would be to allow passing a vector into the Chronicler patterns to read from / write to the block at that position.


I got this to work in Windows

I just followed the setup instructions in the Readme file, there are some changes
Step 0: Setup GOPATH ENV in your machine
Step 1: go get -d github.com/huydx/hget
Step 2: cd <GOPATH>/src/github.com/huydx/hget   // Replace GOPATH with the actual path you set
Step 3: Edit Line 17 in the Makefile to be go build -o 'hget.exe'
Step 4: If you have Make installed you can use the command make install in the hget directory
Step 5: If you dont have Make you can execute these command one by one

go get -d github.com/fatih/color
go get -d github.com/mattn/go-colorable
go get -d github.com/mattn/go-isatty
go get -d gopkg.in/cheggaaa/pb.v1
go get -d github.com/fatih/color
go build -o 'hget.exe'

That's all.

@skadimoolam Thank you for your effort! However did you try to run this on windows? It may be build-able but hget has some logic just work only on unix, for example get home directory by os.Getenv("HOME")

Is there any reason not to use "os/user" for getting the current user's home dir?

@huydx You are right, it does not work, I forked this project and I have started to implement a fallback for windows.

Confirmed not working on Windows. Crosscompiled from source via gox to no avail.


disableNoUnusedVariableRule: false breaks import lookups

Hi
(I'm not sure if I should open this issue here or in vscode-ts-tslint.)
I recently installed vscode-ts-tslint (v0.0.7) in place of vscode-tslint, and I set "disableNoUnusedVariableRule": false in the tsconfig.json for tslint-language-service. I noticed that in this case the import lookups (including the auto-import) are broken. If I set this rule to true, or remove it from the config, lookups work again.
VSCode version: 1.24.0 (bug was the same with previous version too)
Let me know if you need further details.
UPDATE:
Apparently this happens to modules without type definition. See an example setup in this repo:
https://github.com/winston01/vscode-ts-tslint-no-unused-test
Here you can see that for example minimist (w/ a type definition) is recognized correctly, but moment (w/o type def.) is not. If you comment out "disableNoUnusedVariableRule": false in the tsconfig.json and reload the settings, you'll see that moment works too.

Apparently it's related to this issue pending since a long while:
https://github.com/palantir/tslint/issues/2649
Closed.


Pouch: Restaurants demo fails on step #2 to choose time.

Repro steps:

Visit web shell with new Pouch user ex. http://localhost:9119/shells/web-shell/index.html?log=&plannerStorage=volatile&storage=pouchdb://local/foouser
Choose Find restaurants near you. suggestion and wait for results to load.
Choose You are free at 19:00... suggestion.

Note error as:
Uncaught (in promise) Error: can't find store of id !96693423723245:foo--LbyEnYae6T_NtQYj7Rv:21
    at assert (assert-web.ts:12)
    at Arc._instantiateParticle (arc.ts:437)
    at arc.ts:576
    at Array.forEach (<anonymous>)
    at Arc.instantiate (arc.ts:576)
    at Object.onArcLoadRecipe (particle-execution-host.ts:231)

There are three earlier errors in the log as:
StorageProxy '!96693423723245:foo--LbyEnYae6T_NtQYj7Rv:8' received stale update version 1; current is 1

FYI @lindner

Firebase fails with one can't find store error, and a bunch of StorageProxy '!994857782637212:foo--LbyHohOlGll2YQCXWIf:21' received stale update version 3; current is 3 sorts of errors, but eventually is operational. Pouch seems to never get to an operational state.

I had to whittle down the hundreds of calls in description to help make sense of this.  #2905 makes this easier.
With that we see that the requests to read the the handles are executed off the onClick handler, which invalidates the state and leads to recalculating the descriptions.
  | create | @ | description.ts:51
  | (anonymous) | @ | slot-composer.ts:157
  | initializeRecipe | @ | slot-composer.ts:157
  | instantiate | @ | arc.ts:583
  | async function (async) |   |
  | instantiate | @ | arc.ts:523
  | instantiatePlan | @ | arc-host.js:97
  | set plan | @ | arc-host.js:54
  | update | @ | web-arc.js:79
  | _update | @ | xen-async.js:36
  | _doUpdate | @ | xen-base.js:36
  | _doUpdate | @ | xen-debug.js:42
  | _validate | @ | xen-state.js:122
  | Promise.then (async) |   |
  | _async | @ | xen-state.js:97
  | _invalidate | @ | xen-state.js:102
  | _invalidate | @ | xen-debug.js:50
  | _invalidateProps | @ | xen-state.js:67
  | _setProperty | @ | xen-state.js:42
  | _setProperty | @ | xen-debug.js:8
  | set | @ | xen-element.js:36
  | _set | @ | xen-template.js:254
  | set | @ | xen-template.js:216
  | set | @ | xen-template.js:342
  | _doUpdate | @ | xen-base.js:42
  | _doUpdate | @ | xen-debug.js:42
  | _validate | @ | xen-state.js:122
  | Promise.then (async) |   |
  | _async | @ | xen-state.js:97
  | _invalidate | @ | xen-state.js:102
  | _invalidate | @ | xen-debug.js:50
  | _setState | @ | xen-state.js:92
  | _setState | @ | xen-debug.js:15
  | set state | @ | xen-async.js:7
  | onChooseSuggestion | @ | web-shell.js:325

Immediately after this the Handle data is set via onHandleSet
  | set | @ | pouch-db-variable.ts:216
  | onHandleSet | @ | particle-execution-host.ts:92
  | before | @ | api-channel.ts:383
  | _processMessage | @ | api-channel.ts:237
  | APIPort._port.onmessage | @ | api-channel.ts:215

I investigated and Firebase has some code that allows an initial set to trigger the initialized state:
   if (this.version == null) {
      assert(!this.localModified);
      // If the first modification happens before init, this becomes
      // init. We pick the initial version which will be updated by the
      // transaction in _persistChanges.
      this.version = 0;
      this.resolveInitialized();

Still looking at the best way to support that, or alternates.
That said it seems brittle and there should be some way of blocking the plan update until after the handles are written.

I think we have to loop in @shans, @mmandlis et al on this.
Re the first stack trace:
| create | @ | description.ts:51
| (anonymous) | @ | slot-composer.ts:157
| initializeRecipe | @ | slot-composer.ts:157
| instantiate | @ | arc.ts:583
I suspect everything below here is irrelevant, arc.instantiate is where the magic happens.

I tried it on both - locally and arcs-live and this is the exception I'm getting:
assert (assert-web.ts:11)
_instantiateParticle (arc.ts:440)
(anonymous) (arc.ts:579)
instantiate (arc.ts:579)
onArcLoadRecipe (particle-execution-host.ts:243)
await in onArcLoadRecipe (async)

and it is followed by (but i this is probably caused by the one above):
description.ts:132 Uncaught (in promise) TypeError: Cannot read property 'rawData' of null
    at Function._getPatternByNameFromDescriptionHandle (description.ts:132)
    at async Function._createParticleDescription (description.ts:105)
    at async Promise.all (/arcs-live/shells/web-shell/index 7)
    at async Function.initDescriptionHandles (description.ts:91)
    at async Function.create (description.ts:51)
    at async Promise.all (/arcs-live/shells/web-shell/index 0)
    at async DomSlotComposer.initializeRecipe (slot-composer.ts:157)
    at async Arc.instantiate (arc.ts:583)

The particle it fails for is ReservationMultiAnnotation

I dug into this a little deeper and there are two calls into onArcLoadRecipe
The first is ReservationAnnotation
arc.ts:510 instantiate recipe
  use '!833132910810778:user--LcgletXh5K_M-3Q6Jtk:21' as handle0 // Event {...}
  use '!745969886451346:user--LcgletXh5K_M-3Q6Jtk:inner30:1' as handle1 // Restaurant {...}
  slot '!745969886451346:user--LcgletXh5K_M-3Q6Jtk:inner30:2' as slot0
  ReservationAnnotation as particle0
    event = handle0
    restaurant <- handle1
    consume annotation as slot0

 activeRecipe = Recipe {_requires: Array(0), _particles: Array(1), _handles: Array(2), _slots: Array(1), _localName: undefined, …} (2)
resultingHandles = [Handle, Handle]

Note the 2 handles.  One is !833132910810778:user--LcgletXh5K_M-3Q6Jtk:21
Next comes the load of ReservationMultiAnnotation
instantiate @ arc.ts:510
arc.ts:510 instantiate recipe
  use '!833132910810778:user--LcgletXh5K_M-3Q6Jtk:21' as handle0 // Event {...}
  use '!745969886451346:user--LcgletXh5K_M-3Q6Jtk:inner30:3' as handle1 // Restaurant {...}
  slot '!745969886451346:user--LcgletXh5K_M-3Q6Jtk:inner30:4' as slot0
  ReservationAnnotation as particle0
    event = handle0
    restaurant <- handle1
    consume annotation as slot0 Recipe {_requires: Array(0), _particles: Array(2), _handles: Array(3), _slots: Array(2), _localName: undefined, …}

resultingHandles = [Handle]

The mergeInto has removed the handles from the active recipe.
Later on in arc.instantiate things blow up because of this.

So I've managed to move the problem around.
Adding async-mutex around usages of the active recipe in the Arc prevents some of the above errors, but then issues appear further along.

With the mutex'd instantiate and changes to make Pouch stateless I am seeing reservation slots appear.
Will teast apart into smaller PRs to find out what the real solution is.  (And hope that removing logging does not cause race conditions)

Also there's a race condition, in construct.  Reference mode is enabled after the object is constructed so you have to deal with that as well.

Retested on head and this is now working properly without the stateless pouchdb changes.
I would like to land the refactor still, but this is now much lower priority now that this is working.


Add tests to the CI

It would be great to run tests in the CI to ensure there are no issues.
I can add this pretty quickly once I have some free time.

This would be handy. Thanks Brooklin!


add timestamp to global state table for lease table cleanup

Added LastAccessTime Datetime column to the GlobalState table. For accommodating existing triggers, added the column with default getutcdate() value.
Added the clean up scripts to the BindingsOveriew doc, I could move them over to a more appropriate place if folks have better suggestions.

Merging this in, will send a separate PR for the test.


Repo sync for protected CLA branch

The pull request is created from master637511046219972539 to master to fix git push error for protected CLA branch

@ghogen : Thanks for your contribution! The author(s) have been notified to review your proposed change. @evanwindom


Is there an easy way to adding an image to the tab's button?

I'm new to react native please help

You can make tabbar custom via this


Website | Ortelius Google calendar connection refused

Symptom:
Connection refused 403
We're sorry, but you do not have access to this document. That’s all we know.
https://ortelius.io/events/

Seems its isolated to my Chrome browser


Dark mode support for block images

Possible solutions

a rounded background color (sort of like apps on an iPhone) might do

These images can be user provided. So this will have to be pretty flexible.

Closed by https://github.com/PrefectHQ/prefect-ui-library/pull/1142


[docs] add link to ML Flow and treesnip in LightGBM docs

This PR adds a link to the LightGBM integration with ML Flow. If you're curious, you can see more details on the individual methods at https://mlflow.org/docs/latest/python_api/mlflow.lightgbm.html#module-mlflow.lightgbm.

@jameslamb As we are touching this list now, could you please add some R-related projects (I'm not familiar with them at all)?
Maybe some from tydimodels ecosystem ?
https://github.com/curso-r/treesnip
https://github.com/tidymodels/tidymodels

@jameslamb As we are touching this list now, could you please add some R-related projects (I'm not familiar with them at all)?
Maybe some from tidymodels ecosystem ?
https://github.com/curso-r/treesnip
https://github.com/tidymodels/tidymodels

I don't think that {tidymodels} has any LightGBM integration. Searching "lightgbm" in their repo returns 0 results.
I'll add {treesnip} though

There is also lightgbm for mlr3 framework: https://github.com/mlr3learners/mlr3learners.lightgbm

Thanks @t-wojciech . Wasn't familiar with {mlr3}. I'd welcome a pull request to add that link


fix(alert) Rename alert-dismissable to alert-dismissible

This change coincides with the bootstrap rename change: 13845

Landed in d631af5a05a692ce9781f61caa38fef60bb0c4a4


gchart not responsive

I've tried so many ways to make responsive but not working, how to make it responsive, the chart is cropped

This library's output is basically not responsive at the moment, sorry :)

See #1053

is it still not responsive? already any solution?


[Query] Partial response doesn't work if one of stores returned timeout

Hi, Thanos team!
What happened:
Thanos Query returns error (context deadline exceeded) if one of requested stores failed with timeout.
Partial response parameter is enabled.
Expected behaviour:
Thanos returns part of successfully fetched data and warning that one of stores failed.
In the same time there is an other case: if one of stores is not available at all (process is not running e.g.) Thanos Query returns data partially and warning.
It seems that behaviour should be the same in both cases.
@bwplotka what do you think?
Thanos v0.2.1

Well, is it because the request to one of the stor  actually took too long to wait, and either your client or Thanos Query server timeout killed the request? (: So it's not partial erorr - whole request took too long, even though the root cause was one slow store.
So now is the question what would you expect in this case? (:

Yes, In this case Thanos Query killed request, but some stores already returned data.
I think its better to support partial degradation.

up

Hm.. yes, as long as store.read_timeout is smaller the client timeout - that would make sense. But overall it's a tradeoff between availability & conistency. Partial response can indeed surprise you, especially when you use 3rd paty tools like Grafana where warning is not even there.
I am fine to implement your idea though to have this internal timeout to be configurable for Thanos Querier :+1:

@bwplotka Hey! Here it is :)
https://github.com/improbable-eng/thanos/pull/895

Hm.. so @povilasv fixed this issue IMO with this: https://github.com/improbable-eng/thanos/pull/928

so what https://github.com/improbable-eng/thanos/pull/895 is adding? ;p

Moving discussion to https://github.com/thanos-io/thanos/issues/1453


LintDiff violation for 204 response from ArmResourceActionNoContentAsync

In https://github.com/Azure/azure-rest-api-specs-pr/pull/17524, I attempted to move to use ArmResourceActionNoContentAsync.
restrictMovement is ArmResourceActionNoContentAsync<VirtualMachine, VirtualMachineRestrictMovement>;

This generated these responses:
        "responses": {
          "202": {
            "description": "Resource operation accepted.",
            "headers": {
              "Retry-After": {
                "type": "integer",
                "format": "int32",
                "description": "The Retry-After header can indicate how long the client should wait before polling the operation status."
              },
              "Location": {
                "type": "string",
                "description": "The Location header contains the URL where the status of the long running operation can be checked."
              }
            }
          },
          "204": {
            "description": "Action completed successfully."
          },
          "default": {
            "description": "An unexpected error response.",
            "schema": {
              "$ref": "../../../../../common-types/resource-management/v5/types.json#/definitions/ErrorResponse"
            }
          }
        },

LintDiff then complains about the 204 response code, sayding:

PostResponseCodes, RPC-Async-V1-11, RPC-Async-V1-14
Long-running POST operations must have responses with 202 and default return codes. They must also have a 200 return code if only if the final response is intended to have a schema, if not the 200 return code must not be specified. They also must not have other response codes.

This is by design. The recommendation is to use ArmResourceActionNoResponseContentAsync or to simply use ArmResourceActionAsync with a never response type.


Add coercion for Array and Hash to Virtus::Coercion::Object

Add #to_array and #to_hash methods to Virtus::Coercion::Object so that they coerce any value that responds to #to_ary (and #to_a), or #to_hash respectively. If the value does not respond to one of those methods then it should be returned directly as-is.
Consider similar coercions for objects into String using #to_str and #to_s, and Integers using #to_int and #to_i.

Yes, I like that. We should also think about making sure that coercions are 'symmetrical' - we've discussed this once I believe.

@solnic sure, where it's possible. I believe we could make a method that coerces an Array into a Hash by passing it into Hash[], for example. I think we already have stuff to coerce Hash into an Array.
The general idea is that if Ruby provides conventions for conversions into specific primitives, then I want to use those methods by default if they're available. Coercion subclasses can override that if there's a more idiomatic way to convert to the primitive.

Ok, I added #to_hash, #to_string and #to_integer to Virtus::Coercion::Object. Marking this issue as closed.
I decided to only use the explicit coercion methods (#to_hash, #to_str, and #to_int respectively) because the other methods are used in a bit "fast and loose" within ruby core/stlib. Also nearly every object responds to #to_s, and we don't want to coerce stuff into strings and end up with the inspect output -- with virtus' coercions we're looking for cases where the coercion is explicit and in many cases can be reversed because there was no loss of information.


[rabbitmq] Added data collection for exchanges

What does this PR do?
This PR adds functionality to the RabbitMQ integration to collect metrics on exchanges in addition to data already collected for nodes and queues.
This supersedes the same request made in the old repository under https://github.com/DataDog/dd-agent/pull/3026.
Motivation
Inability to see what is going onto an Exchange and not necessarily into a queue.
Testing Guidelines
An overview on testing
is available in our contribution guidelines.
Additional Notes
Think I did everything required, although the test coverage report doesn't say what isn't covered and PyCharm doesn't merge results of Mock and Integration tests.

Will review the failures and get back once they are green.

@wholroyd Did you have a chance to dig into the failures?

@gmmeyer can you have a look at this since you're doing some work around the check?

@masci yes I can

@wholroyd are you still interested in this PR?


Proposal for a refined publish flow

I'd like to spend some time on this module to make the flow a little smoother (and add some features that are a bit manual right now. The bullet points:

Better support for non-interactive publishing
Github integration
Prevent accidental publishing from the wrong branch (or forgetting to pull latest first)

The Github integration should fulfill the following requirements:

[ ] Works with public Github and Github Enterprise
[ ] Can run in CI (e.g. via a CI secret with an access token)
[ ] The access token doesn't appear in logs, no matter how verbose they are
[ ] Can be configured locally, preferably with nice instructions if the config is missing

I think this is moot now that nlm is a thing.


Add heightmap on the client

At this point it's not supported in the NullEngine
https://forum.babylonjs.com/t/heightmap-on-nullengine-with-havok/45126/15


Extension is not available in Azure Pipelines even after installing from Market place

I have already installed Psake marketplace extension for my organization in Azure DevOps services but i am still not able to add the task from build definition. When i am searching with name Psake, it does show up the extension and installed button is grayed out since i have already installed it but it won't give me any option to add any task, it does not appear in build section of task either.

Did you ever figure this out?  I'm having the same issue.

Hi @pyropete,
It may be that the task is too old and the execution handler required by the task is not available anymore on the azure pipelines agent.
Can you share the agent version that your are using?


missing attributes in multisimilarity.py

Hi!
I noticed that  "self.ALLOWED_MINING_OPS" "self.REQUIRES_BATCHMINER" and "self.REQUIRES_OPTIM" are missing as attributes in the multisimilarity criterion class.
This repo is really awesome btw!

Thanks for the kind words and pointing out the issue! If fixed it now in the latest commit :).


Bump dependency versions for react, webpack, babel, eslint, graphiql etc.

Bump dependency versions for react, webpack, babel, eslint, graphiql, graphql, redux, redux-saga.
Changes proposed in this pull request:

Re-order keys in package.json
Updated dependencies - Following are the major ones:
react
webpack
babel
eslint
graphiql
graphql
redux
redux-saga
Updated Webpack scripts to match latest ones from create-react-app.
Switched from tslint to eslint, to match what create-react-app uses.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


feat!: Remove snap packaging

Removing snap packaging in accordance with the TSC vote to discontinue release of snaps as of v3.1.

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/device-rfid-llrp-go/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:

[x] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[x] I am not introducing a new dependency (add notes below if you are)
[ ] I have added unit tests for the new feature or bug fix (if not, why?)
[ ] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[ ] I have opened a PR for the related docs change (if not, why?)

Testing Instructions

New Dependency Instructions (If applicable)

Codecov Report

Merging #261 (911b656) into main (878a6e6) will decrease coverage by 0.04%.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main     #261      +/-   ##
==========================================
- Coverage   41.25%   41.21%   -0.04%
==========================================
  Files          20       20
  Lines        8952     8952
==========================================
- Hits         3693     3690       -3
- Misses       4087     4090       +3
  Partials     1172     1172

see 2 files with indirect coverage changes


fixes #31

Changes Unknown when pulling d27ad71b313a0f1257c2ce1dedea26e17528a673 on mdreizin:status-fix-length-issue into ** on guigrpa:master**.

Sorry for the hiccup, and thanks


build-release-tarball: Add --py3 option for python3 shebang lines

(Split from #2378 and updated with a new option for build-release-tarball.)

Automated message from Dropbox CLA bot
@andersk, it looks like you've already signed the Dropbox CLA. Thanks!

Merged since it'll make our CI better; hopefully I'll find time to test this on chat.zulip.org soon...


Color updates 2

set z-index for nav titles so that they stand out as white text.
if mc-radiogroup is in a guideline, give it the answer/feedback v-alert a mostly black background.

Navigation drawer updates:

Avatar is now 'warning' red.
Make text of navigation stages white/black.
Add a black line to the bottom of headers (to help distinguish when dialogue content has scroll overflow.

Change app bar src image (Still room to adjust, but the new image has less activity, so perhaps simplifies the design.)

darken the background of the piggy bank chip
add divider to scaffolds
turn speech-synthesizer into icon/button
make text of v-card-actions white/black
remove background from v-card-actions

This PR is tied to https://github.com/cosmicds/hubbleds/pull/122 in the hubbleds repository.


My first 2 test cases (BDD-style) checking status returning code for …

…the webhost package #5 #36 #37

I like this test but do you think we should write it within the testing package instead of using an imported library? I was kind of thinking we should keep imports to a minimum due to this eventually being deploy-able software.
To me it looks like all the tests you wrote within this test file can be rewritten in just golang with the testing package without convey package.
What does everyone else think?

Good point, the whole reason I did that is there's a trend to move to Behavior Driven Development  nowadays (Rails did it, React did it, in my experience...), where people can/should read English instead of reading how the test is made or executed (declarative vs imperative). Go doc also provides good facilities for documentation but I want some full blown testing environment (ideally testers are separated from developers, if possible at best). Check it out how this has been done from a maintenance point of view (picture below).
I'm cool if the majority of us would like to go back to the bare-bone Go style ~ but eventually (if we are keen enough ;) ), it will become like this:

Source

For now rewrite your tests using the httptest package. We're really trying to limit the overhead on packages until we get some sort of a real working version of this software up and running. @hoangdangninh


proto: Unmarshal takes more than 100 seconds at 22k QPS

Is this normal performance?
mass goroutine are blocked at  table_unmarshal.go:117
unmarshalInfoLock.Lock().
207251 @ 0x43105f 0x441619 0x4415ef 0x44138d 0x474939 0xc9025a 0xc901de 0xc6ed8c 0xc6e8f1 0xce167c 0x45e8f1
#	0x44138c	sync.runtime_SemacquireMutex+0x3c							/home/zhoudazhuang/usr/local/go1.12.1/go/src/runtime/sema.go:71
#	0x474938	sync.(*Mutex).Lock+0x108								/home/zhoudazhuang/usr/local/go1.12.1/go/src/sync/mutex.go:134
#	0xc90259	acs/vendor/github.com/golang/protobuf/proto.getUnmarshalInfo+0x39			/home/zhoudazhuang/db11/jm/pro/src/acs/vendor/github.com/golang/protobuf/proto/table_unmarshal.go:117
#	0xc901dd	acs/vendor/github.com/golang/protobuf/proto.(*InternalMessageInfo).Unmarshal+0xdd	/home/zhoudazhuang/db11/jm/pro/src/acs/vendor/github.com/golang/protobuf/proto/table_unmarshal.go:59
#	0xc6ed8b	acs/vendor/github.com/golang/protobuf/proto.(*Buffer).Unmarshal+0x1eb			/home/zhoudazhuang/db11/jm/pro/src/acs/vendor/github.com/golang/protobuf/proto/decode.go:425
#	0xc6e8f0	acs/vendor/github.com/golang/protobuf/proto.Unmarshal+0x170				/home/zhoudazhuang/db11/jm/pro/src/acs/vendor/github.com/golang/protobuf/proto/decode.go:343
#	0xce167b	acs/comet/client.(*Client).HandleRequest+0xecb						/home/zhoudazhuang/db11/jm/pro/src/acs/comet/client/client.go:626

I use pb like that:
s := time.Now()
		err := pb.Unmarshal(req.Data, registerInfo)
		e := time.Now().Sub(s).Seconds()
		if e > 1 {
			fmt.Printf("pb unmarshal cost1: %+v\n",e)
}

I got :
pb unmarshal cost1:<PHONE_NUMBER>86
pb unmarshal cost1:<PHONE_NUMBER>51
pb unmarshal cost1:<PHONE_NUMBER>6
pb unmarshal cost1:<PHONE_NUMBER>91
pb unmarshal cost1:<PHONE_NUMBER>05

$ go version
go version go1.12.1 linux/amd64
$ go env
GOARCH="amd64"
GOBIN="/home/zhoudazhuang/gobin/"
GOCACHE="/home/zhoudazhuang/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/zhoudazhuang/db11/jm/pro"
GOPROXY=""
GORACE=""
GOROOT="/home/zhoudazhuang/usr/local/go1.12.1/go"
GOTMPDIR=""
GOTOOLDIR="/home/zhoudazhuang/usr/local/go1.12.1/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build631445118=/tmp/go-build -gno-record-gcc-switches"

My local pb version is obtained with go get, how do I get its version?

func getUnmarshalInfo(t reflect.Type) *unmarshalInfo {
	// It would be correct to return a new unmarshalInfo
	// unconditionally. We would end up allocating one
	// per occurrence of that type as a message or submessage.
	// We use a cache here just to reduce memory usage.
	unmarshalInfoLock.Lock()
	defer unmarshalInfoLock.Unlock()
	u := unmarshalInfoMap[t]
	if u == nil {
		u = &unmarshalInfo{typ: t}
		// Note: we just set the type here. The rest of the fields
		// will be initialized on first use.
		unmarshalInfoMap[t] = u
	}
	return u
}

Any kind of pb.Unmarshal will call this method? I also call pb.Unmarshal elsewhere, and it is blocked here too.Equally time consuming

top - 17:12:03 up 41 days, 18:45,  7 users,  load average: 11.42, 12.44, 12.89
Tasks: 644 total,   2 running, 642 sleeping,   0 stopped,   0 zombie
Cpu(s): 18.1%us, 15.9%sy,  0.0%ni, 54.1%id,  0.0%wa,  0.0%hi, 12.0%si,  0.0%st
Mem:  99061716k total, 50930052k used, 48131664k free,    16052k buffers
Swap:        0k total,        0k used,        0k free, 32538128k cached

My server load is very low. Not the same unmarshal object

It is difficult to diagnose this without knowing:

what version of the golang/protobuf package you are using.
what the structure of the Go type being passed to Unmarshal looks like.
related to 2, is whether you are passing in messages generated by golang/protobuf/protoc-gen-go?

golang protobuf version: v1.3.1 (I update for the lastest, And it recurrent)
the structrue:

// RegisterInfo 客户端向服务发送的注册信息.
type RegisterInfo struct {
	Appver   *string `protobuf:"bytes,1,opt,name=Appver" json:"Appver,omitempty"`
	// app id 用于区别不同的用户
	BundleID *string `protobuf:"bytes,2,opt,name=BundleID" json:"BundleID,omitempty"`
	// 系统类型。0 Iphone, 1 Android, 2 Ipad, 3 AndroidPad
	System    *int32  `protobuf:"varint,3,opt,name=System" json:"System,omitempty"`
	OSVersion *string `protobuf:"bytes,4,opt,name=OSVersion" json:"OSVersion,omitempty"`
	Brand     *string `protobuf:"bytes,5,opt,name=Brand" json:"Brand,omitempty"`
	Model     *string `protobuf:"bytes,6,opt,name=Model" json:"Model,omitempty"`
	UID       *string `protobuf:"bytes,7,opt,name=UID" json:"UID,omitempty"`
	UUID      *string `protobuf:"bytes,8,opt,name=UUID" json:"UUID,omitempty"`
	// 上次更新的patch id
	LPID *string `protobuf:"bytes,9,opt,name=LPID" json:"LPID,omitempty"`
	// 上次更新的schema id
	LSID             *string `protobuf:"bytes,10,opt,name=LSID" json:"LSID,omitempty"`
	XXX_unrecognized []byte  `json:"-"`
}

I confuse about what you say. I accept []byte throught tcp connection, can Unmarshal correctly, no error . what dose it mean of messages generated by golang/protobuf/protoc-gen-go? I generated the pb go file only once

What are the methods on RegisterInfo?

this?

If that's the full method set, it seems that RegisterInfo has not be regenerated in some time. Since v1.1.0, each message type has an XXX_{Marshal,Unmarshal} method that provides an optimized implementation.

Yes, that's the full method set. no XXX_{Marshal,Unmarshal}.
I take over the project which is start at few years ago.The code is old ,And we did't have mass user a few years ago.
Does it mean I should generate the proto buf code by lastest version ? And use the XXX_Marshal instead of the Marshal?

Did you have the performance bottleneck about this lib?
how much qps it can be reached on normal use?

Does it mean I should generate the proto buf code by lastest version ?

Yes, please.

And use the XXX_Marshal instead of the Marshal?

Nope. The proto.Unmarshal and proto.Marshal knows to automatically use the XXX_Marshal and XXX_Unmarshal methods if present. They are never intended to be called directly from user code. If fact, doing so is likely to cause issues.

how much qps it can be reached on normal use?

It's pretty much impossible to provide QPS metrics because the cost of a "query" is dependent on the user's workload.

Did you have the performance bottleneck about this lib?

The golang/protobuf library is intended to be as fast as reasonably can be. It is heavily used in production by many services at Google. I believe your performances issues come from messages that have not been generated in a long time. There seems to be heavy lock contention trying to grab the unmarshalInfoLock lock. That lock is only obtained when the optimized XXX_Unmarshal method is not present.

This has a certain effect，not particularly large.

thx very much.
It’s not as big cost as it used to be.
I'll go and see other optimization problems of the system.

... It Reappeared.
And the system load and cpu cost is low when it reappearing. Any other ideas ?

Any other ideas?

Unfortunately not. Without a minimal reproduction, it is very difficult for us to diagnose this.

@dsnet PTAL at https://github.com/TennyZhuang/protobuf-lock-reproduce