Feature/pinning file support

TODO

[x] convert settings keys to the tuple and change evaluate function to take tuple instead ADDON_SETTINGS_USD_RESOLVER_LOG_LVL

Fixed in LakeFs branch

[x] implement a standalone system that generates a pinning file

This Pr tracks a pinning file creation via the Usd API.
Here are some related chats on the AOUSD forums.
https://forum.aousd.org/t/get-all-references-sublayers-recursively-of-stage/1322/2?u=bigroynl
https://forum.aousd.org/t/sdf-format-args-and-custom-ar-resolvers/1660/6
function to get info about the USD version
Usd.GetVersion()

okay.
There is this thread on the AOUSD forums. Where they talk about changes in the behaviour of UsdUtils.ExtractExternalReferences, the version of Usd I developed with outputs the 1001-1100 range, and this creates the problem that we don't have the info about how  URI was resolved and thereby creating the pinning file this way is pointless because we are missing central information.
We could implement a function that gets all the data from the stage ourselves and not rely on the UsdUtils functions. @dee-ynput @antirotor, do you have any opinions about this?
https://forum.aousd.org/t/usdutils-extractexternalreferences-converts-udim-to-1001-in-23-11/1669

I quickly resolved the merge conflicts - would be good to re-test this on your end @Lypsolon

I quickly resolved the merge conflicts - would be good to re-test this on your end @Lypsolon

Big thanks for the Work. i tested on my end (its a simple test script maybe we get to setup tings like this as "Static" tests so that we dont need ayon env)
issue created for moving those tests into the addon Repo https://github.com/ynput/ayon-usd/issues/37

Running this on the simplest USD file that I have on my machine:
from ayon_usd.standalone.usd import pinning
import ayon_api

usd_path = r"E:\asset.usd"
root_info = ayon_api.get_project_roots_for_site(os.environ.get("AYON_PROJECT_NAME"))
pinning_file_path = "E:/asset_pin.json"
pinning.generate_pinning_file(usd_path, root_info, pinning_file_path)

I got:
# Error: 'NoneType' object has no attribute 'pseudoRoot'
# # Traceback (most recent call last):
# #   File "<maya console>", line 7, in <module>
# #   File "E:\dev\ayon-usd\client\ayon_usd\standalone\usd\pinning\__pinning_file_generation_funcs.py", line 292, in generate_pinning_file
# #     pinning_data = get_asset_dependencies(entry_usd, resolver)
# #   File "E:\dev\ayon-usd\client\ayon_usd\standalone\usd\pinning\__pinning_file_generation_funcs.py", line 183, in get_asset_dependencies
# #     recursive_result = get_asset_dependencies(
# #   File "E:\dev\ayon-usd\client\ayon_usd\standalone\usd\pinning\__pinning_file_generation_funcs.py", line 183, in get_asset_dependencies
# #     recursive_result = get_asset_dependencies(
# #   File "E:\dev\ayon-usd\client\ayon_usd\standalone\usd\pinning\__pinning_file_generation_funcs.py", line 160, in get_asset_dependencies
# #     layer.pseudoRoot, layer
# # AttributeError: 'NoneType' object has no attribute 'pseudoRoot'

This error occurs whenever a layer does not exist on disk.
Fixed with: https://github.com/ynput/ayon-usd/pull/22/commits/be375fb46cfb7e438ed9b8b20163397035b26e47
Now it passes those with warnings.
Sidenote: I directly hate the 'nested' python code because it disallows me to quickly live reload the code hehe - now I need to import from the 'fake' private file to reload that instead. Needing me to do:
import ayon_usd.standalone.usd.pinning.__pinning_file_generation_funcs as pinning
import ayon_api

import importlib
importlib.reload(pinning)

usd_path = r"E:\asset.usd"
root_info = ayon_api.get_project_roots_for_site(os.environ.get("AYON_PROJECT_NAME"))
pinning_file_path = "E:/asset_pin.json"
pinning.generate_pinning_file(usd_path, root_info, pinning_file_path)

Anyway, that's my dirty hacks I guess.

Can't create pinning file with no local site root info
The next issue I'm facing @Lypsolon is that I can't generate a pinning file because I don't have any root_info for my site.
root_info = ayon_api.get_project_roots_for_site(os.environ.get("AYON_PROJECT_NAME"))
print(root_info)
# {}

So using my ugly as testing code:
import ayon_usd.standalone.usd.pinning.__pinning_file_generation_funcs as pinning
import ayon_api

import importlib
importlib.reload(pinning)

usd_path = r"E:\asset.usd"
root_info = ayon_api.get_project_roots_for_site(os.environ.get("AYON_PROJECT_NAME"))
pinning_file_path = "E:/asset_pin.json"
pinning.generate_pinning_file(usd_path, root_info, pinning_file_path)

I get:
# Error: both root_info and dependency_info need to be present and an instance of Dict (root_info: {}, dependency_info: {'E:\\asset.usd': 'e:\\asset.usd', 'E:\\payload.usd': 'e:\\payload.usd', 'e:\\payload.usd': 'e:\\payload.usd', './look.usd': 'e:\\look.usd', './model.usd': 'e:\\model.usd'})
# # Traceback (most recent call last):
# #   File "<maya console>", line 12, in <module>
# #   File "E:\dev\ayon-usd\client\ayon_usd\standalone\usd\pinning\__pinning_file_generation_funcs.py", line 301, in generate_pinning_file
# #     root_less_pinning_file_data = remove_root_from_dependency_info(
# #   File "E:\dev\ayon-usd\client\ayon_usd\standalone\usd\pinning\__pinning_file_generation_funcs.py", line 32, in remove_root_from_dependency_info
# #     raise ValueError(
# # ValueError: both root_info and dependency_info need to be present and an instance of Dict (root_info: {}, dependency_info: {'E:\\asset.usd': 'e:\\asset.usd', 'E:\\payload.usd': 'e:\\payload.usd', 'e:\\payload.usd': 'e:\\payload.usd', './look.usd': 'e:\\look.usd', './model.usd': 'e:\\model.usd'})

Non-unique keys for relative anchored files in the dependency_info?
Also note how ./look.usd and ./model.usd are both "relative anchored paths" as keys even though they are in the payload.usd file (which is not the entry file). As such, my feeling is that if we test a scenario where one file has multiple assets, each with same "relative anchored filenames" but (different resolved paths) that the keys in this dict will not be a valid look up table since the entries would be overwriting each other.

---- Rest is Done in Issue -------

Non-unique keys for relative anchored files in the dependency_info?
Also note how ./look.usd and ./model.usd are both "relative anchored paths" as keys even though they are in the payload.usd file (which is not the entry file). As such, my feeling is that if we test a scenario where one file has multiple assets, each with same "relative anchored filenames" but (different resolved paths) that the keys in this dict will not be a valid look up table since the entries would be overwriting each other.

that's a fair point i need to do some more testing for that to see how the resolver is getting those data points. this was not in the testing data on my side i will add it.

Pinning File creation now allows to create a pinning file with relative paths and rootless paths for both key and value.
PS: resolve also has the update but i did not recompile it yet.

New Resolver's Are uploaded to LakeFs Pinning File Branch for testing.

Overall this looks good - However, I'm getting this when running inside e.g. Maya:
# Error: module 'pxr.UsdShade' has no attribute 'UdimUtils'
# # Traceback (most recent call last):
# #   File "<maya console>", line 15, in <module>
# #   File "E:\dev\ayon-usd\client\ayon_usd\standalone\usd\pinning\__pinning_file_generation_funcs.py", line 334, in generate_pinning_file
# #     pinning_data = get_asset_dependencies(entry_usd, resolver)
# #   File "E:\dev\ayon-usd\client\ayon_usd\standalone\usd\pinning\__pinning_file_generation_funcs.py", line 207, in get_asset_dependencies
# #     recursive_result = get_asset_dependencies(
# #   File "E:\dev\ayon-usd\client\ayon_usd\standalone\usd\pinning\__pinning_file_generation_funcs.py", line 207, in get_asset_dependencies
# #     recursive_result = get_asset_dependencies(
# #   File "E:\dev\ayon-usd\client\ayon_usd\standalone\usd\pinning\__pinning_file_generation_funcs.py", line 207, in get_asset_dependencies
# #     recursive_result = get_asset_dependencies(
# #   [Previous line repeated 3 more times]
# #   File "E:\dev\ayon-usd\client\ayon_usd\standalone\usd\pinning\__pinning_file_generation_funcs.py", line 191, in get_asset_dependencies
# #     udim_data = _resolve_udim(identifier, layer)
# #   File "E:\dev\ayon-usd\client\ayon_usd\standalone\usd\pinning\__pinning_file_generation_funcs.py", line 123, in _resolve_udim
# #     resolved_udims = UsdShade.UdimUtils.ResolveUdimTilePaths(
# # AttributeError: module 'pxr.UsdShade' has no attribute 'UdimUtils'

This may be due to USD version that is old in Maya 2024.2 with MayaUSD 0.27? It works in Houdini 20.0.273 however.
from ayon_usd.standalone.usd import pinning
import ayon_api
import os

def get_root_info(project_name=None) -> "dict[str, str]":
    project_name = project_name or os.environ.get("AYON_PROJECT_NAME")
    return ayon_api.get(f"projects/{project_name}/siteRoots").data

usd_path = r"/path/to/entry.usd"
pinning_file_path = "E:/asset_pin.json"
root_info = get_root_info()
pinning.generate_pinning_file(usd_path, root_info, pinning_file_path)

@BigRoy i fixed the udim issue if you can give it a quick test that be great.
i believe everything works but im not sure.
also i just saw that the tray publisher would allow for texture publish (not really Udim ready) but we would need a product that has  in the name and the > is not allowed. not important just wanted to mention (other than that texture resolve via uri works)

@BigRoy i tested the pinning file creation on Maya 2025_5(just installed) it works there gonna install 2024_2 and then test there

As per comment here we'll have to do additional fixes to the logic.

the resolver site of this is done.
could we merge it and do all other fixes as


Is there anyway to use this just with plain javascript?

Hi guys....
I'm just a rookie dev... in fact, I'm just code my portfolio with plane html, css and javascript and well, I thought this can be more easy to implement.
I just wonder if there anyway to use this without typescript or anyway more simple?

Hi Hanu (@callmehanu),
Thank you for your request. Of course it's possible, I will put together a demonstration and get back to you as soon as possible.
Thank you for your patience!
David

Hello Hanu (@callmehanu),
I've created a 👉CodeSandbox to demonstrate how to use Embla Carousel only by including a <script> tag. TypeScript compiles to plain JavaScript so by adding the src to the npm package script  you have access to EmblaCarousel() function globally like so:
<script<EMAIL_ADDRESS><script>
  var emblaElement = document.querySelector(".embla");
  var options = { loop: false };
  var embla = EmblaCarousel(emblaElement, options);
</script>

I hope this helps!
Kindly,
David

Hi Hanu (@callmehanu),
Not to rush you but have you had the chance to check out the CodeSandbox example yet?
Best,
David

Hello Hanu (@callmehanu),
I'm closing this issue due to lack of response. Feel free to open this issue again if you don't think it's resolved.
Kindly,
David


avoid stacking in total metric plots

[x] Closes #463  .
[x] I am familiar with the contributing guidelines.
[ ] Tests added.
[x] Updates entries to docs/source/api.rst for API changes.
[ ] Adds descriptions to appropriate "what's new" file in docs/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).
[x] New code is fully documented. Includes numpydoc compliant docstrings, examples, and comments where necessary.
[x] Maintainer: Appropriate GitHub Labels and Milestone are assigned to the Pull Request and linked Issue.

And make hover on total plots show the full name

hard to test, but should look like

@lboeman are you able to run this against your suite of pathological long names? I'm slightly concerned about how plotly might treat the null characters now and also in future updates to plotly.

I'll test it out.

Interesting behavior, but .. works? Not sure where this suffixing is occuring.But similar names are getting -01, -02 etc. but not in the timeseries legend.

suffixing isn't occuring the changed code, so I think this is good


Add support for Watch and TV

Motivation:
This product should be available on as many platforms as possible.
Modifications:
Add minimum OS versions.
Extend to Int64 when necessary to work around smaller Int on watch.
Result:
Product builds for Watch and TV.

readVarint() I think still a problem for watch.  Could be quite difficult to fix.


Add features section

What this PR does / why we need it:
I was looking for a place to put my admission webhook docs to and thought of introducing a features section in the docs to put that in. Currently I feel like this page is too much implementation details to be a high level page so I used it to create the features page. Let me know what you think :)

sgtm! – merged :)


URGENT: app contacts google.com and discloses IP addresses of all users

Summary
When the app runs (on iOS at least), it periodically requests https://clients3.google.com/generate_204. This is because the app uses react-native-netinfo, which, by default, exhibits this behaviour.
Note: I didn't report this as a security issue because it's not exploitable beyond what Google could already exploit. It just needs to be fixed ASAP.
I urge you to treat this as an urgent security problem that must be fixed ASAP. You are currently leaking PII to Google for all users.
See fixes below!
Steps to reproduce
Running the app at all produces this behaviour, but in order to actually monitor it, you'll need to set up mitmproxy or otherwise watch what e.g. the iOS Simulator is doing via packet capture/Little Snitch, etc.

Environment
I demonstrated the problem on an iOS device, but from what I can tell, this happens to all apps on all platforms that use react-native-netinfo (without changing the defaults).
What's happening?
When the app invokes NetInfo 1 2, it uses an internal Reachability Configuration to determine if the network is present and if it's behind a captive portal.
Unfortunately, the default URL that is used for this is on google.com.
Why is this bad?
This is convenient for most React Native developers, I'm sure, but I'm also sure you'll admit that sending Google the IP addresses of all users, frequently, is bad form for an app that is specifically designed to track its users (and keep them safe while doing so). It's especially disturbing that such an app comes from our government.
But really, is it so bad?
In a word? Yes.
Canada's Privacy Commissioner says:

An Internet Protocol (IP) address can be considered personal information if it can be associated with an identifiable individual. For example, in one complaint finding, we determined that some of the IP addresses that an internet service provider (ISP) was collecting were personal information because the ISP had the ability to link the IP addresses to its customers through their subscriber IDs.

It's plausible and even likely that Google would have enough data on many or most Canadians to be able to correlate IP addresses at a given moment (and the information shed is frequent), with the time-sliced data in the tracking collisions. I'm not saying they are doing this, but I believe they could.
Fixes
To fix this problem two changes are required:

an endpoint within the canada.ca ecosystem that we can trust is not disclosing Personally Identifiable Information to a third party
changes to the app to use this endpoint instead of Google

In the next few moments, I will edit this section to link PRs for both projects.
Server
On the server side, we need to provide a presence URL. Ideally this would return HTTP status 204 to help avoid a situation where a captive portal is wither redirecting (via 30x) or is displaying an interstitial page (as 200).
What's most important here is that the URL is within an ecosystem our users already trust (canada.ca).
I've modified the server app to provide a 204 on: https://retrieval.covid-notification.alpha.canada.ca/exposure-configuration/present
I made a server pull request to implement this.
Client
Once the server is deployed, and the 204 …/present URL is available, a new version of the app must be propagated to users.
This PR configures the app to connect to https://retrieval.covid-notification.alpha.canada.ca/exposure-configuration/present to determine reachability, instead of leaking information to Google.
I made an app pull request to implement this.
Notes

Obviously, feel free to move the URL endpoint elsewhere; I just put it into the config routing to avoid having to change anything else infrastructure-wise.
Global configuration of the netinfo module made the most sense to me, but feel free to move it elsewhere, so long as the Google information leak is resolved.

Thanks for the catch @scoates .
We'll work with the teams to fix this and aim to roll it out with our next release.

Note: the PRs have been replaced by internal ones: #1008, cds-snc/covid-alert-server#242. 👍

Closing this - the updates have been merged and will be in the next release.

Thank you for your quick and professional response!


Publish Updates

Related #87

Sorry guys did not release the tags don't start with v in this repository. @IEvangelist @dcuccia @bradwestness


Backfill the README with instructions and content from the original

lemurheavy's coveralls-ruby repository apparently updated its README after this fork, and it has a lot of detail in it that's helpful when trying to get set up. I think it used to rely on the docs being hosted by coverall's site, but that site has since stopped holding them, and now points to your readme, which doesn't include much detail.
For https://github.com/tagliala/coveralls-ruby-reborn/issues/44

coverage: 94.385%. remained the same when pulling 3ebfd4bda1c23d5bd71514a80a50314385c33c9c on nevinera:nev--update-readme-to-match-upstream into 5440895e47e93559e612617f3c57876c4c6aaee9 on tagliala:main.

Hi, thanks for the PR
Running CI was not necessary but I didn't want to ask to add [ci skip] and it is a chance to test against latest ruby-heads
The changelog is quite accurate: https://github.com/tagliala/coveralls-ruby-reborn/blob/main/CHANGELOG.md#changelog
There should be new CIs supported, but I never used those CIs and I can't provide instruction for those

Thank-you for your speed, sir! That is by far the fastest anyone has ever responded :-D

Welcome and thanks for having the time to submit a PR. Feel free to make adjustments to the readme if needed


What is the purpose of using encryption in the cloud?

What is the motivation for using encryption in the cloud? If I do not use encryption, in this case, what are the potential risks I have?

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

ID: 86b44bac-c760-a06d-e618-149f275e0d0e
Version Independent ID: bcaaf53e-87c1-59aa-c51b-d3359ebb9046
Content: Encrypt disks on a Windows VM in Azure
Content Source: articles/virtual-machines/windows/encrypt-disks.md
Service: virtual-machines-windows
GitHub Login: @cynthn
Microsoft Alias: cynthn

Hi @Dmitresky Thank you for your feedback! We will review and provide an update as appropriate.

@Dmitresky
Azure Disk Encryption helps protect and safeguard your data to meet your organizational security and compliance commitments. It uses the BitLocker feature of Windows and the DM-Crypt feature of Linux to provide volume encryption for the OS and data disks of Azure virtual machines (VMs). It is also integrated with Azure Key Vault to help you control and manage the disk encryption keys and secrets, and ensures that all data on the VM disks are encrypted at rest while in Azure storage. Azure Disk Encryption for Windows and Linux VMs is in General Availability in all Azure public regions and Azure Government regions for Standard VMs and VMs with Azure Premium Storage.
If you use Azure Security Center, you're alerted if you have VMs that aren't encrypted. The alerts show as High Severity and the recommendation is to encrypt these VMs.
What are the potential risks I have?
For some scenarios has been explained in this article, please refer to the below mentioned link:
https://docs.microsoft.com/en-us/azure/security/azure-security-disk-encryption-overview

@DmitWe will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


[Bug] origins:prevent_block_selection doesn't work with non full blocks

When using prevent block selection on for example a chest, selecting it is possible.
Anything that isnt a full block has this behaviour.

*from what i know

Should be fixed since Origins 1.1.0


Enable JSONSerialization Int fastpath for boxed values

Enable the Int serialization fastpath introduced by #923 to be used when the value has been boxed in an NSNumber (for example, via JSONEncoder).
With a simple microbenchmark which serializes a struct and an equivalent dictionary containing 10 keys with Int values, this takes JSONEncoder from ~4x to ~2x slower than using JSONSerialization directly:
4.0-DEVELOPMENT-SNAPSHOT-2017-08-04-a
JSONEncoder (Struct) took<PHONE_NUMBER> ns
JSONEncoder (Dict) took<PHONE_NUMBER> ns
JSONSerialization took 43780.5796 ns

With Int fastpath
JSONEncoder (Struct) took 75462.8424 ns
JSONEncoder (Dict) took 84936.7405 ns
JSONSerialization took 46191.5048 ns

@swift-ci please test

@djones6 what's the status of this?

Im actually working on a PR that adds support for more type in JSONSerialization, which may supersede this (or I could look at incorporating it). I just need to add test cases to it.
https://github.com/spevans/swift-corelibs-foundation/commit/41091434c34b3711698e66d33dcdbac6508d9bbe

Closing for now, as I need to reevaluate now that #1197 has merged, and also see how things look with whole-module-optimization applied to Foundation on Linux.  I'll make sure to address @ianpartridge 's comment (re. overflow) too.


Revert specific handling at setup for macs with apple chips

Docker for apple chips now supports host.docker.internal so can revert the changes done to support it for the dev setup:
https://docs.docker.com/docker-for-mac/apple-silicon/#fixes-since-the-apple-silicon-preview-7
Test coverage results :test_tube:
:white_circle: Total coverage has not changed
No changes to code coverage between the base branch and the head branch

/ok-to-test sha=e03dc5e

/ok-to-test sha=e03dc5e


feat(bridge-ui): rework HORSE faucet

Describe the feature request
It's currently not user friendly to request HORSE.
I'd suggest changing the icon so it looks less like a filter, changing the button text to 'Request HORSE' and align the button left.
Describe alternatives you've considered
Description of the alternatives you've considered here.
Additional context

cc @shadab-taiko

I actually prefer the current layout TBH.

I too prefer the current layout. Will check if there are icons that look more relevant. Closing this issue.


Option to disable "Invalid Character!" red message

Checklist

[x ] Have you read through :h nvim-surround to see if there might be any relevant information there?

Is your feature request related to a problem? Please describe.
The "Invalid Character!" red message is very aggressive. I'd like to turn it off, or replace it with a simple print() statement.
Describe the solution you'd like
An option to disable / change that message.
Additional context
I tried this code and it didn't work, what am I doing wrong?
require("nvim-surround").setup({
	delimeters = {
		invalid_key_behavior = function() print("ok cool") end,
	},
})

Thank you very much.

It appears you have spelled "delimiters" wrong.

Thank you very much :) Silly me :joy:


Update ANDROID_HOME to ANDROID_SDK_ROOT

From the Android Studio user guide:

ANDROID_HOME, which also points to the SDK installation directory, is deprecated.

I was surprised about that rule as well. It makes it sound like ANDROID_SDK_ROOT is the variable being deprecated.

Please also update the Android guide for macOS

It’s already updated, see this diff.

Hello @sonicdoe, sorry, but I have lost a track of this PR for some reason. 😅
It look like the path changes have already been applied within #2991, so I think we can close this PR now.
If you seeing anything incorrect or missing in the parts after the refactor please post the new PR with the suggestions!

thanks too @sonicdoe btw i'm seeing Last updated 2022-03-23 UTC ANDROID_SDK_ROOT is deprecated in favour of ANDROID_HOME

From the Android Studio user guide

so i think Google have made a u-turn 20 days after

https://github.com/facebook/react-native-website/pull/2991 (Mar 3) cc @henryhobhouse @cortinico

@Simek i'm happy to create a new issue and PR top revert if i'm seeing straight 😅
(i'm seeing both variables remain on latest GitHub Actions Virtual Environments, so as mentioned both variables will be checked by AGP for consistency anway)

i'm happy to create a new issue and PR top revert if i'm seeing straight 😅

Yup you're right. We should update the docs to use ANDROID_HOME instead of ANDROID_SDK_ROOT now to follow the official guidance


Minor Attribute Fixes

In the panel for attributes (in composer and the attribute panel context) make it so that labels that come inside the outer field-groups have a slightly different font size and weight, in order to make it obvious that they are subservient to the outer field groups (e.g. add an address attribute to a page and it looks strange.)

This should also work in Express forms.

Make is to that you can control the text displayed next to the checkbox of a Boolean attribute. Right now it is just "Yes." We should save "Yes" as the text for upgrades, and if there is no custom text we use "Yes" as a fallback, but you should be able to populate it with a sentence like "I agree to the Terms of Service."

I'm unsure if it's feasible within this bit of work, but if we could get the "keyword" search function in the Express Entity List block to search against Boolean attributes, I think that would be helpful. In my v8.1.0 setup searching for "yes" or "no" via keywords does not include any boolean attributes and their values.

My fault – this is still open as we need checkbox labelling.


[REEF-479] Introduce time mesuarements in BroadcastReduce and PipelineBroadcadtReduce in Network.Examples.Client

[REEF-479] Introduce time mesuarements in BroadcastReduce and PipelineBroadcadtReduce in Network.Examples.Client
This addressed the issue by

Introducing time measurements using StopWatch in master and slave tasks.
Making the intermediate computations between broadcast and reduce minimal so as to measure the time of round trip correctly.

JIRA:
479

@markus @jwang98052 , the logging function does not give duration in a variable but just prints it. I want to maintain running average of broadcast and reduce over iterations. Hence, will prefer going with StopWatch explicitly unless you guys strongly feel otherwise. Moreover, this is more like a user code than REEF code. Hence, we should be ok using StopWatch.

This is client code. And it requires special calculation for the performance of this particular task. I am fine with it to use stopWatch/startWatch directly here without using LogFunction.

I completed my pass.

Made the changes and pushed the code

WAKE tests keep hanging. I have tried a few times with this build but couldn't pass. I cannot see anything related to this change though.

That's weird. But this one is unrelated to that.
Dhruv
On Jul 17, 2015 00:35, "Julia Wang"<EMAIL_ADDRESS>wrote:

WAKE tests keep hanging. I have tried a few times with this build but
couldn't pass. I cannot see anything related to this change though.
—
Reply to this email directly or view it on GitHub
https://github.com/apache/incubator-reef/pull/299#issuecomment-122204012
.


Hi, I'm getting error in the line 554

The error is in this line
paver.$t.on('mousemove.paver', $.throttle(paver.settings.cursorThrottle, function(e){
I already load the paver css and js.
I'm using jquery v3.1.0

Paver has been thoroughly tested with jQuery v3.1.1 (the latest commit, d13b8ef, uses jQuery v3.1.1 for all demo pages). It seems like you have forgotten to load Ben Alman's $.throttle plugin.

So yes! haha thanks all is working good now. cheers!
2016-09-30 19:40 GMT+02:00 Terry Mun<EMAIL_ADDRESS>
Paver has been thoroughly tested with jQuery v3.1.1 (the latest commit,
d13b8ef
https://github.com/terrymun/paver/commit/d13b8eff3e22eb8ec02262a973d9210a0d7098ee,
uses jQuery v3.1.1 for all demo pages). It seems like you have forgotten to
load Ben Alman's $.throttle plugin
http://benalman.com/projects/jquery-throttle-debounce-plugin/.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/terrymun/paver/issues/10#issuecomment-250806703, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGNxz_RjGFbY_FjaQG4YKHY5oYVF0LtPks5qvUmpgaJpZM4KKfiJ
.


Drivers: Sensor: mc3419: Added LPF Configuration using Kconfig variables

For applications that rely on low noise/variance in accelerometer sample reading. Application can take advantage of integrated LPF Thus this PR adds LPF configuration capability to the mc3419 driver

Why isn't this being set in device tree?

Because of two reasons

Since we already have MC3419_DECIMATION_RATE Kconfig variable that used to configure decimation rate. I don't want to confuse user with one configuration in Kconfig while other in dts. Thus MC3419_LOW_PASS_FILTER_CUT* was introduced
not all values are valid for LPF settings, thus i find it is easy to restrict them while providing some info in help string, using Kconfig choice

@gmarull @nordicjm

Moving only LPF setting to DT, will have one configuration LPF in dts, while other DECIMATION RATE in Kconfig.
Move Both LPF and DECIMATION RATE to DT, and update any existing sample.

Are you guys fine with 1st approach, or 2nd make more sense?

@gmarull @nordicjm
1. Moving only LPF setting to DT, will have one configuration `LPF` in dts, while leave other `DECIMATION RATE` in Kconfig.

2. Move Both LPF and DECIMATION RATE to DT, and update any existing sample.

Are you guys fine with 1st approach, or 2nd make more sense?

Second makes sense, older drivers had options added to Kconfig but they never really should have been put there. Whilst an application with one of these devices is only ever likely to have one, there is no reason it can't have two or more, and if you have two or more and have the options set by Kconfig, that means every device has the exact same configuration and can't be changed, whereas with dts you can have as unique configuration as you want for each device

@gmarull friendly reminder


Validate SDK Milestone #4 deliverable

Validate SDK Milestone #4 deliverable: Browser Kit + Plugins

Uploaded test results for session
test-output.txt

Updated feature spreadsheet with results from tests
https://docs.google.com/spreadsheets/d/1fkKyXkb7NhGCeAJYt7z4FpcSGG_lrCM2lxLBLW-OMrE/edit#gid=0


Enable retries for specified status codes

This PR enables the retriable status code flag in retry_on, this is needed before the status code values can be evaluated for retries.
The PR also reduces the number of retries to 2 as discussed with @rshriram @nmittler and @duderino

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: knrc
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: zackbutcher
If they are not already assigned, you can assign the PR to them by writing /assign @zackbutcher in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @knrc. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test
Wonderful contribution @knrc

@knrc: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
cc7f5bc4f4bc0bc7806e8ad3ec145d4968f42e02
link
/test e2e-simpleTests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@knrc: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
74a4947457da746fe3f2640aa8e76f4b1bda4620
link
/test e2e-simpleTests-cni

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@knrc: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
74a4947457da746fe3f2640aa8e76f4b1bda4620
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
74a4947457da746fe3f2640aa8e76f4b1bda4620
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: knrc, nmittler
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [nmittler]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@knrc: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
74a4947457da746fe3f2640aa8e76f4b1bda4620
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
74a4947457da746fe3f2640aa8e76f4b1bda4620
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
74a4947457da746fe3f2640aa8e76f4b1bda4620
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

I worry that both circleci runs for this PR failed the same required test:
https://circleci.com/gh/istio/istio/343105?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link
TestIngressGateway503DuringRuleChange - pilot
Failed
gateway "istio-ingressgateway" created
destinationrule "destination-rule-c" created
virtualservice "gateway-bound-route" created
destinationrule "destination-rule-c" configured
virtualservice "gateway-bound-route" configured
destinationrule "destination-rule-c" configured
virtualservice "gateway-bound-route" deleted
destinationrule "destination-rule-c" deleted
gateway "istio-ingressgateway" deleted
ingressgateway_test.go:258: Got non 200 status code while changing rules: map[503:2 200:498]

@duderino +1, I'm investigating

@knrc I found one other recent case of that failure that's independent of this change:  https://circleci.com/gh/istio/istio/342827
I'm kicking it off again to see if it fails a third time

@duderino  It would be quite a coincidence if it wasn't related, there must be something to it.

If it fails a third time I'll revert the second commit and try again

/retest

@knrc retest only retriggers prow jobs.  you have to click on the Details link on the circleci failures and clikc on the rerun workflow in the upper right corner

@duderino Ah, I don't have access in that case

Hi. How does the envoy know what status code can be configured? Looking at the istio docs I cannot find the answer.  Thanks.

oh, it's a bit obscure. Looks like you can configure through virtualservice's header values.

It's part of the virtual service

@nmittler oh, it needs to append the retry headers in the virtualservice to tell Envoy to retry on certain status code.

@iandyh

@nmittler oh, it needs to append the retry headers in the virtualservice to tell Envoy to retry on certain status code.

Not exactly ... this has nothing to do with HTTP headers. We're configuring Envoy's routes directly. If you were to provide headers in the request to modify this behavior, Envoy would process those headers normally.

@nmittler Thanks for the correction. Looks likes the status code atm is being hard coded as http.StatusServiceUnavailable? ref: https://github.com/knrc/istio/blob/cc7f5bc4f4bc0bc7806e8ad3ec145d4968f42e02/pilot/pkg/networking/core/v1alpha3/route/retry/retry.go#L34

Yes, that's the default.


Add turn_off_nicely.

Closed in https://github.com/pollen-robotics/reachy-sdk/commit/a12dbfb137391e19690da71e262ba905c4bffb4c.


[DO NOT MERGE]Publish Aug 2019 relnotes to live docs

Deploy preview for docsdocker ready!
Built with commit 1b177dcb7eb064c13b732714cffeffaf27fccc66
https://deploy-preview-9349--docsdocker.netlify.com


add omitempty to Pair.ID

Overview
In order to enterprise TopN queries to return a Key without an ID, then Pair.ID must support json omitempty.
Pull request checklist

[x] I have read the contributing guide.
[x] I have agreed to the Contributor License Agreement.
[x] I have updated the documentation.
[x] I have resolved any merge conflicts.
[ ] I have included tests that cover my changes.
[x] All new and existing tests pass.

Code review checklist
This is the checklist that the reviewer will follow while reviewing your pull request. You do not need to do anything with this checklist, but be aware of what the reviewer will be looking for.

[ ] Ensure that any changes to external docs have been included in this pull request.
[ ] If the changes require that minor/major versions need to be updated, tag the PR appropriately.
[ ] Ensure the new code is properly commented and follows Idiomatic Go.
[ ] Check that tests have been written and that they cover the new functionality.
[ ] Run tests and ensure they pass.
[ ] Build and run the code, performing any applicable integration testing.

I'm going to close this until we decide on a long term direction direction for the proxy.


Non-breaking text flowing out of build log container

Any ideas here?

Hmm, very odd, what version of Firefox are you using? That output seems to work okay for me in Firefox!

I'm not sure of the best way to repro. Maybe just load up a very large RSpec run? I don't think we're doing anything too special...maybe I'll ask in the Slack channel.

Looks like there’s a regression with word-break in Chrome 56. No other browser appears to behave this way except Blink-based ones. Really strange!

I’ve made a small test case which reproduces this issue. Going to send a bug report to Chrome and hope that gets somewhere…

Posted to Chromium’s bug tracker, let us hope it gets somewhere.

Oh, I fixed this with some hacks last week :) Should be fine now!


[Enhancement] Update Game End Canvas.

We get a Earth wins / Alien wins screen when player wins/loses the game along with an option to continue playing. Currently the text highlighting this issue is very small and does not fit the UX requirement.
We need bigger font to highlight the status and add button to continue playing the game.

@DhanushNehru I would like to work on this, can you assign this issue #22 to me

Issue Resolved here #25


deploy RK3588 rknn model in runtime but get infer result as all zero

.rknn model is transform from rknn2 and it can run correctly in rktoolkit2 in PC.
However, it got wrong when using 'runtime' to infer and got all zero result, while the shape of result is correct. The input has already checked.(there is no example in runtime usage, while my model is not used as any way in 'vision' example).

The code is below:
import fastdeploy as fd
from fastdeploy import ModelFormat
import numpy as np
import cv2

camera = cv2.VideoCapture(2)
option = fd.RuntimeOption()

option.set_model_path("mymodel.rknn", model_format=ModelFormat.RKNN)

option.use_cpu()
option.use_rknpu2(rknpu2_name=fd.c_lib_wrap.CpuName.RK3588,rknpu2_core=fd.c_lib_wrap.CoreMask.RKNN_NPU_CORE_AUTO)

runtime = fd.Runtime(option)

input_name = runtime.get_input_info(0).name

while True:
    ret,frame = camera.read()
    cv2.imshow("test",frame)
    cv2.waitKey(1)
    rgb_array = cv2.resize(frame,(320,176))
    hsv_array = cv2.cvtColor(rgb_array,cv2.COLOR_BGR2HSV).reshape((1,176,320,3))
    # print(hsv_array)
    results = runtime.infer({
        input_name: hsv_array
    })
    input_check = hsv_array
    output_check = runtime.num_outputs()
    print(input_check)
    print(output_check)
    print(results)

OK,the problem is at model transformation form .pt to .rknn.I fixed it by transform .pt to onnx then to .rknn.Now it works!


Substitution score from or to Z is 0

As noted in a comment to #56, the substitution score from and to the Z dummy block is always 0. This is used in PBclust to build the distance matrix, and to compare sequences with the --compare option.
To this, @pierrepo commented:

We should not have to compare "z" to a PB since "z" is not a PB. It just means that a PB cannot be assigned to this residue.
So maybe we should rise an error as this point.

How to we want to handle substitution from and to the Z dummy block ? The obvious way, as suggested by @pierrepo is to raise an exception. Yet, this means that we cannot do a clustering when some information is missing, or that we need to find tricks.
Bare in mind that this will affect how we calculate distance matrices. See #62 for more about distance matrix calculations.
cc @pierrepo @alexdb27

I think we can use the "gapopen" value used for structural alignement if a real PB is in front of Z, while it is zero if Z in front of Z. cc @pierrepo @jbarnoud

OK. How much is a gapopen value ?

-600 !!!
Ok, it is x 100 ...


somearrow.strftime('%s') uses local timezone, not somearrow's timezone

Well, I think this is wrong. Here's a test script:
import arrow

sometime = arrow.get('2017-01-01 12:00')
print(sometime.timestamp)
print(sometime.strftime('%s'))
print(sometime.strftime('%Y-%m-%d %H:%M:%S'))

Saved as /tmp/test.py, here's the output of running it with some different system timezones:
actual system timezone - PST:
[adamw@xps13k pagure (more-timezone-fun %)]$ python /tmp/test.py
1483272000
1483300800
2017-01-01 12:00:00

UTC:
[adamw@xps13k pagure (more-timezone-fun %)]$ TZ='UTC' python /tmp/test.py
1483272000
1483272000
2017-01-01 12:00:00

Winnipeg (why not?):
[adamw@xps13k pagure (more-timezone-fun %)]$ TZ='America/Winnipeg' python /tmp/test.py
1483272000
1483293600
2017-01-01 12:00:00

Note that the arrow's timestamp attribute and the strftime('%Y-%m-%d %H:%M:%S') output is the same regardless of the system timezone, which I'd expect: arrows are UTC by default. But the output of strftime('%s') varies by system timezone, which I did not expect. I would expect it to always be the same as the timestamp attribute.
After noticing this in arrow I checked and in fact the bug exists in Python's own datetime, for aware datetime instances: I filed that as Python bug 32988. So if arrow is just handing strftime duties off to datetime behind the scenes, this is essentially the same as that bug, I guess. Still figured it'd be worth reporting.

Update: there's quite a lot about this at Python issue 12750.

Hey thanks for pointing out those bug threads, interesting reading.
You're correct that arrow hands off strftime duties to datetime (https://github.com/crsmithdev/arrow/blob/master/arrow/arrow.py#L1135) so we can't do much about this here.


[BUG] Missing Localization in settings: "Custom weather messages"

In settings the name, description, sub menu button, and climate descriptors are not localized.

Thanks for catching.
Fixed in 1.13.2 and ready for new localization text


Make cpufreq module optional for rt-app calibration

If cpufreq module is not used (e.g., KVM target) rt-app calibration
should still work as usual, no need to set frequency to max in this case
though.
Remove the hard dependecy towards cpufreq module.
Signed-off-by: Juri Lelli<EMAIL_ADDRESS>

@derkling suggested changes on IRC

So I was thinking this could benefit from being packaged in a context manager, like was done for freezing the userspace: https://github.com/ARM-software/lisa/commit/cdf25f2961e22d5cc8b97c68c5a121a54b09adc8
I don't know what would be the best place for it to live - maybe inside devlib's cpufreq module ?
I'd say it would look something like this:
@contextlib.contextmanager
def use_governor(governor, strict=False):
    old_governors = {}

    for domain in target.cpufreq.iter_domains():
        cpu = domain[0]
        governor = target.cpufreq.get_governor(cpu)
        tunables = target.cpufreq.get_governor_tunables(cpu)
        old_governors[cpu] = governor, tunables

    target.cpufreq.set_all_governors(governor)

    yield

    # Restore previous governors
    #   Setting a governor & tunables for a cpu will set them for all cpus
    #   in the same clock domain, so only restoring them for one cpu
    #   per domain is enough to restore them all.
    for cpu, (governor, tunables) in old_governors.iteritems():
        target.cpufreq.set_governor(cpu, governor)
        target.cpufreq.set_governor_tunables(cpu, **tunables)

with use_governor("performance"):
    # Do whatever you want !

And then you can fold in checks for "is the governor I'm asking supported ?".

I think the main issue for having this into devlib is that, if we don't have the cpufreq module (which is the case Juri is trying to fix) we cannot access the method you propose... unless... @valschneider: can we have a context manager defined for a static method?
If the above is not possible, it's fine for me to have this API living in rta.py.

Hmm yeah so that's actually a downside. One way to still use it would be:
if "cpufreq" in target.modules:
    with use_governor("performance"):
        run_calibration()
else:
    run_calibration()

Which isn't great... But we're already facing that problem with userspace freezing: https://github.com/ARM-software/lisa/blob/master/libs/utils/executor.py#L266
Another way would be to define it as a static method in devlib.cpufreq as you propose, and then within the method you can skip everything if the passed target doesn't have the cpufreq module loaded.

I don't have the full background of the discussion but the context manager needs to take "target" as a parameter anyway, so there is no reason for this check not to be included inside the context manager:
if "cpufreq" in target.modules:

However, having a such an API that silently bails out is a recipe for lot of troubles down the line, so this silent behavior should be enabled by a parameter that defaults to raising an exception, even if we end up specifying that parameter all the time.

Hi,
I implemented that governor context manager in here: https://github.com/ARM-software/devlib/pull/316
I'm also cleaning up lots of stuff around LISA, so when it comes to rtapp I'll make it work without the cpufreq module.

It's all implemented in here for now: https://github.com/valschneider/lisa/blob/lisa-next/lisa/wlgen/rta.py#L394
Closing this for now, although you should wait for a proper release ;)


Upgrade github actions and introduce dependabot

Update setup-java and checkout actions to newer version
Integrate dependabot  - see
https://docs.github.com/en/code-security/supply-chain-security/keeping-your-dependencies-updated-automatically/automating-dependabot-with-github-actions?learn=dependency_version_updates

Created https://github.com/Daimler/sechub-plugin-eclipse/issues/15 as a follow up issue to handle gradle and eclipse dependency synchronization when the current sechub-client-java fat jar will be replaced by newer and modular java api version.


Auto-detect PyPy in test suite

Currently, Legion Spy tests fail if pypy isn't available or on PATH. Would be good to auto-detect and call Python for machines that don't have PyPy.

@lightsighter Should be fixed in 2a5cac8f9b6c2de25fbfc07af1f5ea6c825bb64e. Can you confirm?

Fixed. Closing the issue.


Investigate JavaScript support for Jupyter notebook without Spark

Jupyter notebooks most often run with a simple Python or R interpreter and without Spark as a big compute engine. What would it take to enable EclairJS to run in such an environment?

I am not a Jupyter notebooks expert (maybe @bpburns can correct me were I go wrong), the way I understand Jupyter notebooks is in order to support other languages is to add iPython/Jupyter kernels. To run R for example the user would need to install the the iRKernel for Jupyter. This would enable R code in the user's notebook, this would not allow them to use SparkR from the notebook.
EclairJS-nashorn support Jupyter notebooks through the Apache Toree Jupyter kernel. This allows the user to write EclairJS-nashorn code that will run on the Spark cluster.
If the desire is to run JavaScript in Jupyter notebooks without Spark then the user could install iJavaScript

We have a EclairJS kernel for Jupyter notebooks, but "magics" are not working. We need to get magics working do that we support HTML and D3 for visualization of data.

to integrate with Jupyter notebooks. EclairJS extends Toree's org.apache.toree.interpreter.Interpreter for EclairJS support in Jupyter notebooks. Currently Apache Toree only support "magics" in the scala interpreter, in order for EclairJS notebooks to use "magics" we would need to develop a feature enhancement for Apache Toree to handle "magics" globally. Apache Toree has offered to provide guidance but not do the actual development. If EclairJS want to put a Pull Request together they would merge it.


UI endless spinner when offline

What did you do?
Started Perses with default config file and log level debug.
What did you expect to see?
The UI
What did you see instead? Under which circumstances?
The spinner, endlessly

System information
macos
Perses version
0.47.1
Perses configuration file
database:
  file:
    folder: "./perses"
    extension: "json"

schemas:
  panels_path: "./cue/schemas/panels"
  queries_path: "./cue/schemas/queries"
  datasources_path: "./cue/schemas/datasources"
  variables_path: "./cue/schemas/variables"
  interval: "6h"

Logs
% ./perses --config config.yaml --log.level debug
WARN[0000] encryption_key is not provided and therefore it will use a default one. For production instance you should provide the key.

 ___________
\___________/
     ___________      ______
    \___________/     | ___ \
 ___________          | |_/ /__ _ __ ___  ___  ___
\___________/         |  __/ _ \ '__/ __|/ _ \/ __|
 ___                  | | |  __/ |  \__ \  __/\__ \
\___/                 \_|  \___|_|  |___/\___||___/  0.47.1
__________________________________________________________

DEBU[2024-09-18T19:54:01+10:00] No migration file found at cue/schemas/panels/bar/migrate.cue, plugin bar will be skipped  error="open cue/schemas/panels/bar/migrate.cue: no such file or directory"
DEBU[2024-09-18T19:54:01+10:00] No migration file found at cue/schemas/panels/scatter/migrate.cue, plugin scatter will be skipped  error="open cue/schemas/panels/scatter/migrate.cue: no such file or directory"
DEBU[2024-09-18T19:54:01+10:00] No migration file found at cue/schemas/panels/time-series-table/migrate.cue, plugin time-series-table will be skipped  error="open cue/schemas/panels/time-series-table/migrate.cue: no such file or directory"
DEBU[2024-09-18T19:54:01+10:00] No migration file found at cue/schemas/panels/trace-table/migrate.cue, plugin trace-table will be skipped  error="open cue/schemas/panels/trace-table/migrate.cue: no such file or directory"
DEBU[2024-09-18T19:54:01+10:00] No migration file found at cue/schemas/panels/tracing-gantt-chart/migrate.cue, plugin tracing-gantt-chart will be skipped  error="open cue/schemas/panels/tracing-gantt-chart/migrate.cue: no such file or directory"
DEBU[2024-09-18T19:54:01+10:00] No migration file found at cue/schemas/queries/tempo/migrate.cue, plugin tempo will be skipped  error="open cue/schemas/queries/tempo/migrate.cue: no such file or directory"
DEBU[2024-09-18T19:54:01+10:00] BarChart plugin loaded from file cue/schemas/panels/bar
DEBU[2024-09-18T19:54:01+10:00] GaugeChart plugin loaded from file cue/schemas/panels/gauge
⇨ http server started on [::]:8080
DEBU[2024-09-18T19:54:01+10:00] Markdown plugin loaded from file cue/schemas/panels/markdown
DEBU[2024-09-18T19:54:01+10:00] ScatterChart plugin loaded from file cue/schemas/panels/scatter
DEBU[2024-09-18T19:54:01+10:00] StatChart plugin loaded from file cue/schemas/panels/stat
DEBU[2024-09-18T19:54:01+10:00] Table plugin loaded from file cue/schemas/panels/table
DEBU[2024-09-18T19:54:01+10:00] TimeSeriesChart plugin loaded from file cue/schemas/panels/time-series
DEBU[2024-09-18T19:54:01+10:00] TimeSeriesTable plugin loaded from file cue/schemas/panels/time-series-table
DEBU[2024-09-18T19:54:01+10:00] TraceTable plugin loaded from file cue/schemas/panels/trace-table
DEBU[2024-09-18T19:54:01+10:00] TracingGanttChart plugin loaded from file cue/schemas/panels/tracing-gantt-chart
DEBU[2024-09-18T19:54:01+10:00] PrometheusTimeSeriesQuery plugin loaded from file cue/schemas/queries/prometheus
DEBU[2024-09-18T19:54:01+10:00] TempoTraceQuery plugin loaded from file cue/schemas/queries/tempo
DEBU[2024-09-18T19:54:01+10:00] PrometheusDatasource plugin loaded from file cue/schemas/datasources/prometheus
DEBU[2024-09-18T19:54:01+10:00] TempoDatasource plugin loaded from file cue/schemas/datasources/tempo
DEBU[2024-09-18T19:54:01+10:00] PrometheusLabelNamesVariable plugin loaded from file cue/schemas/variables/prometheus-label-names
DEBU[2024-09-18T19:54:01+10:00] PrometheusLabelValuesVariable plugin loaded from file cue/schemas/variables/prometheus-label-values
DEBU[2024-09-18T19:54:01+10:00] PrometheusPromQLVariable plugin loaded from file cue/schemas/variables/prometheus-promql
DEBU[2024-09-18T19:54:01+10:00] StaticListVariable plugin loaded from file cue/schemas/variables/static-list
INFO[2024-09-18T19:54:04+10:00]                                               method=GET status=200 uri=/
INFO[2024-09-18T19:54:04+10:00]                                               method=GET status=200 uri=/main.js

see my comment https://github.com/perses/perses/issues/2195#issuecomment-2358031535 it's because of the wrong naming folder to store the data.
I propose a workaround, that should unblock the situation until the next release

Unfortunately the above linked fix does work for the other issue, it doesn't work for this issue.
database:
  file:
    folder: "./data"
    extension: "json"

% ./perses --config config.yaml --log.level debug == spinner

oh yeah, indeed going offline somehow stop the browser to download the rest of the file. Interesting.


chemed not in pubchem

If the starting molecule is not in pubchem, it just dies.

C/C=C/C(=O)C1CCC(C=C1C)(C)C

Fixed by #33


Fix thread state transition in get_function_pointer

get_function_pointer can be called from any thread. In particular, it can be called from Android thread pool thread when method marshaling is used. We need to ensure those threads stay in the GC safe mode from the runtime perspective once we stop executing user code on those threads. mono_thread_attach switches the thread to GC unsafe mode and it stays there after we stop executing user code. The next GC will try to suspend the thread which cannot succeed since there's no cooperative runtime code running on that thread anymore.
The fix is to use mono_jit_thread_attach which mimics what the iOS/macOS interop does. It creates the necessary MonoVM thread structures for JIT to work but doesn't transition the thread state.
Fixes #8253

/azp run


[SPARK-9478] [ml] Add class weights to Random Forest

This PR adds weight support to the following Predictors in ML.
DecisionTreeClassifier
DecisionTreeRegressor
RandomForestClassifier
RandomForestRegressor
cc @jkbradley

Merged build triggered.

Merged build started.

Test build #43318 has started for   PR 9008 at commit 367a443.

Test build #43318 has finished for   PR 9008 at commit 367a443.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43318/
Test PASSed.

Merged build triggered.

Merged build started.

Test build #43460 has started for   PR 9008 at commit 0ffbdd0.

Test build #43460 has finished for   PR 9008 at commit 0ffbdd0.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43460/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #43482 has started for   PR 9008 at commit 33982fb.

Test build #43482 timed out     for PR 9008 at commit 33982fb     after a configured wait of 250m.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43482/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #43503 has started for   PR 9008 at commit 3273ed4.

Test build #43503 has finished for   PR 9008 at commit 3273ed4.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class ChildProcAppHandle implements SparkAppHandle
abstract class LauncherConnection implements Closeable, Runnable
final class LauncherProtocol
 static class Message implements Serializable
 static class Hello extends Message
 static class SetAppId extends Message
 static class SetState extends Message
 static class Stop extends Message
class LauncherServer implements Closeable
class NamedThreadFactory implements ThreadFactory
class OutputRedirector

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43503/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #43531 has started for   PR 9008 at commit 8f35057.

Test build #43531 timed out     for PR 9008 at commit 8f35057     after a configured wait of 250m.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43531/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #43553 has started for   PR 9008 at commit bd316d6.

Test build #43553 has finished for   PR 9008 at commit bd316d6.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43553/
Test FAILed.

Jenkins failed tests unrelated to this patch. Let's try again.

Merged build triggered.

Merged build started.

Test build #43572 has started for   PR 9008 at commit 822382e.

Test build #43572 has finished for   PR 9008 at commit 822382e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43572/
Test FAILed.

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #43588 has started for   PR 9008 at commit c1785a8.

Test build #43588 has finished for   PR 9008 at commit c1785a8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43588/
Test PASSed.

@sethah I have incorporated your comments in the latest patch. Thank you!
@jkbradley Do you have any comments or suggestions? Much appreciated.

Build triggered.

Build started.

Test build #45206 has started for   PR 9008 at commit c1785a8.

Test build #45206 has finished for   PR 9008 at commit c1785a8.

This patch fails Spark unit tests.
This patch does not merge cleanly.
This patch adds no public classes.

Build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45206/
Test FAILed.

retest this please

Merged build triggered.

Merged build started.

Test build #45315 has started for PR 9008 at commit 32f4548.

Test build #45315 has finished for PR 9008 at commit 32f4548.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * abstract class Writer extends BaseReadWrite \n  * trait Writable \n  * abstract class Reader[T] extends BaseReadWrite \n  * trait Readable[T] \n  * case class GetInternalRowField(child: Expression, ordinal: Int, dataType: DataType)\n  * case class Expand(\n

Merged build finished. Test PASSed.

Thanks for working on this!
Minor: PR title says class weights but actually it's sample weights what is implemented.

@rotationsymmetry I made a pass on this, mostly minor comments. Thanks for working on this, it would be great to get it merged in!

@sethah Thank you very much for your review. I will incorporate the changes in the next few days. Regarding the TODO in BaggedPoint.scala, I want to look into the details to find out the scope of the change.

I noticed a problem with the current implementation regarding the minInstancesPerNode parameter. The number of  instances in each node is now a weighted count where the weights can have an arbitrary scale. For example, a tree built with uniform weights where each weight is equal to 1.0 will build a different tree than uniform weights where each weight is 1.0 / N (N is number of samples). I suppose there are a number of ways to mitigate this.
I checked scikit-learn and they track the actual raw sample counts (unweighted) as well as the sample weights. They use min_samples_leaf to compute validity based on raw counts, and min_weight_fraction_leaf to compute validity based on weighted counts. This will not be possible under the current implementation here because we lose the raw counts when we convert to unadjustedBaggedInput to baggedInput. We could compare weighted split counts vs minInstancesPerNode / N where N is number of training samples, or we could adjust the BaggedPoint class to store counts and weight and proceed ala scikit-learn. I'm not sure what is best, thoughts?

Another issue is that the information gain for candidate splits is not computed correctly with fractional samples. This is because the information gain calculation here uses the sample counts which are converted to Long type. This produces incorrect results in general, and NaN values when the total count is less than 1. The count function here should return a Double type instead. Can we add a test to ensure that the trees are invariant under constant multiplication of the weights?

@rotationsymmetry: Will you have time to work on this? I am more than happy to send a PR to your PR if you do not have time.
@jkbradley @dbtsai Would you mind chiming in on the issue mentioned above about minimum instances per node?

cc @MLnick thoughts on the above comments?

@sethah So to avoid adding any overhead from computing stats for both these params one option would be to selectively compute only the stats that are required (e.g. if they request minInstancesPerNode per node request that and if they requeust min_weight_fraction_leaf compute the stats needed).

@holdenk Thanks for the feedback. Upon some further thought, I think that a.) We need to compute the statistics needed for both minInstancesPerNode and minWeightFractionPerNode and b.) it will not be too hard to compute them both and will not add a ton of extra memory overhead. Selectively computing one or the other could get complicated very quickly.
I am going to have a PR for this ready soon, which will incorporate changes submitted in this PR. I created two JIRAs for issues that I encountered when preparing this PR and submitted patches for each. They are:

SPARK-14610 - PR 12374
SPARK-14599 - PR 12370

@rotationsymmetry Could you please close this?


Comments written by admin to be shown to the students

The comments/feedbacks on the resume submitted for review to be displayed to the individual student

fixed the issue in https://github.com/SE-vrs-organization/SE_Project3_Wolftrack/commit/f0b3fe11ecc33c6c432b35541fa163adf1fdeacb


Switch missing after update "dishcare_dishwasher_option_silenceondemand"

Before update there was a switch to choose "silent mode" for dishwasher.
"switch.xxxxxx_dishcare_dishwasher_option_silenceondemand" There is now only "binary_sensor.xxxxxx_dishcare_dishwasher_option_silenceondemand"
Easy fix to get the switch back?

If it's gone then it's no longer available through the API.

How about theese:

switch.xxxxxx_dishcare_dishwasher_option_extradry
switch.xxxxxx_dishcare_dishwasher_option_variospeedplus
switch.xxxxxx_dishcare_dishwasher_option_halfload

same thing. by the way, the integration doesn't delete entities so I'm not even sure what you mean when you say it's not there.

These 4 switches has been available before update and a re-install.
If they no longer are available, that is the answer :)

Entities are created dynamically. An upgrade doesn't delete entities but if you reinstalled then all the previous entities were removed when you removed the integration. They will be back when you actually use the appliance in a way that gets them back (i.e. select a program that supports them), assuming they are still available through the API.

Sounds nice, will give it a testrun.


Remove reference to "Future" in ML.NET docs

Summary
Removed the notion of future in the ML.NET documentation and added them to the list as they are now available.
Fixes #Issue_Number (if available)
https://github.com/dotnet/machinelearning/issues/4886
Preview Link: https://review.docs.microsoft.com/en-us/dotnet/machine-learning/automate-training-with-cli?branch=pr-en-us-18239

@aslotte when you get a chance, if you're comfortable with the changes I've suggested, can you please merge them into your branch so I can approve and merge this PR. Thanks!

@luisquintanilla how does that look?

@aslotte looks good. Sorry to make you take action though. Since the new version of the CLI is out, @briacht has made updates to this doc that reflect the latest experience. I'll close this PR for now and make sure your changes are included as part of her PR #18782 . Thanks again for the contribution!


.Net Bindings - Implemented CI tests

Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] Pull request title reflects its content
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Description:
There is a new Test-Project which will be run from CI using ctest, producing juinit compatible report which is copied to the other reports for archiving in GitHub Artifacts.
Also the .NET Bindings generation has been removed from all Jobs except for Windows, VS 2022 x64, Debug+Release.

Draft, until PR #172 is merged (this branch is based on the other one).

@Unsttopabull I'm not that well versed with the .NET binding code, would you have some time to take a look at this PR?


add timeout to put/post methods

Fix for #49

add timeout to post/put requests
minor code formatting

Agreed with the *getTimeout, should be more general.
I'll think about a refactor, pending finding some time.


[Error: Initialize client first] when using Recaptcha.fetchClient

I am a React Native developer currently integrating your package into our app. I followed the installation steps and usage guidelines provided in the readme.md file.
The documentation suggests using initClient and execute. But Based on the code of the package,
/**
 * Initialize a reCAPTCHA client
 * @deprecated Use {@link Recaptcha.fetchClient} instead
 * @param siteKey Your application's reCAPTCHA site key
 * @param timeout An optional timeout value in milliseconds
 */
export function initClient(siteKey: string, timeout?: number): Promise<string> {
  let args: Args = {};

  if (timeout) {
    args.timeout = timeout;
  }

  return RecaptchaEnterpriseReactNative.initClient(siteKey, args);
}

/**
 * Execute reCAPTCHA and retrieve a token
 * @deprecated Use {@link RecaptchaClient.execute} instead
 * @param action An action to describe what the user is doing such as "login"
 * @param timeout An optional timeout value in milliseconds
 */
export function execute(
  action: RecaptchaAction,
  timeout?: number
): Promise<string> {
  let args: Args = {};

  if (timeout) {
    args.timeout = timeout;
  }

  return RecaptchaEnterpriseReactNative.execute(action.action, args);
}

these methods are deprecated. So I implemented the following code:
      try {
        await initClient(siteKey);

        // Step 1
        const client = Recaptcha.fetchClient(siteKey);

        // Step 2
        const token = await client.execute({ action: 'kkr_recaptcha' });

        // set the result
        setInitResult('ok');
        setExecuteResult(token);
        console.log(token);
      } catch (error) {
        // error exception
        console.log(error);
        setInitResult(error?.toString());
        setExecuteResult(error?.toString());
      }

However, when I execute the code, I encounter the following error:
[Error: Initialize client first]
This seems to suggest that the client must be explicitly initialized, but initClient is marked as deprecated. Could you clarify how to properly initialize the client with the latest version of the package?
Additionally, I would like to know where I can find up-to-date documentation for the most recent version of this package.
Environment:
•	React Native version: 0.76.1
•	Package version: 18.6.0
•	Platform: iOS/Android
Thank you for your assistance! Looking forward to your guidance.

Thanks for the issue @gyulhana. The two methods can't be used interchangeable. So if you call initClient you can't call the execute from Recaptcha.fetchClient. If you remove the first call await initClient(siteKey); it should work.
I was able to reproduce this behavior you faced by calling the deprecated mode in the example app we have in the React Native repository and calling execute on the new one.

@walterjgsp oh, sorry. It seems like there was a misunderstanding. Initially, I only used fetchClient without calling initClient, but I encountered the error [Error: Initialize client first]. So, I added initClient, and it was working. but since this method is deprecated, I'm unsure how to proceed. Could you provide an example using a non-deprecated approach?

@gyulhana, I tested from my side and this seems to be working. Use the import:
import {
  Recaptcha,
  RecaptchaAction,
  type RecaptchaClient,
} from '@google-cloud/recaptcha-enterprise-react-native';

We create some states to update the UI:
const [recaptchaClient, setRecaptchaClient] =
    React.useState<RecaptchaClient>();
  const [initResult, setInitResult] = React.useState<string>('Not Initialized');
  const [executeResult, setExecuteResult] = React.useState<
    string | undefined
  >();
  const [token, setToken] = React.useState<string | undefined>();

And the buttons to initialize the client and fetch the token:
<View style={styles.container}>
      <Button
        onPress={() => {
          try {
            setRecaptchaClient(Recaptcha.fetchClient(siteKey ?? 'SITEKEY'));
            setInitResult('ok');
          } catch (error: any) {
            setInitResult(error.toString());
          }
        }}
        title="Fetch Client"
        testID="fetchClientButtonId"
      />
      <Text>Fetch Client Result: </Text>
      <Text testID="fetchClientResultId">{initResult}</Text>

      <Button
        onPress={() => {
          if (recaptchaClient) {
            recaptchaClient
              .execute(RecaptchaAction.LOGIN(), 10000)
              .then((newToken) => {
                setExecuteResult(newToken.startsWith('03') ? 'ok' : 'error');
                setToken(newToken);
              })
              .catch((error) => {
                setExecuteResult(error.toString());
              });
          } else {
            setExecuteResult('Recaptcha Client is undefined');
          }
        }}
        title="Execute"
        testID="clientExecuteButtonId"
      />
      <Text>Execute Result:</Text>
      <Text testID="clientExecuteResultId">{executeResult}</Text>
      <Text>Token:</Text>
      <Text testID="tokenId">{token}</Text>
    </View>

We will update the documentation to exemplify how to use the new API. Please let me know if this works for you. If not could you provide a small reproduction of your code in a repository so we can test?

I've actually found the issue with your code by checking the difference on it. There's a bug on how the RN wrapper for reCAPTCHA is working in the current fetchClient API. We do not wait in the wrapper for the client to be initialized and return immediately so you can execute and fetch token. Since there's no wait for the state in fetchClient if you can execute right way the error you're facing will be triggered. In the next release we will fix this issue and also update the documentation. Thanks @gyulhana !

@gyulhana, a new version of the RN SDK was released and should include the changes to fix the fetchClient API and better documentation. Closing this bug for now, please open it again in case it doesn't work for you.

@walterjgsp thx!


String.replaceFirstMapped() and replaceAllMapped() documentation issues

Please see https://api.dartlang.org/stable/1.21.0/dart-core/String/replaceFirstMapped.html

No valid range for [start] argument described. Please, add something like "If start is provided, it must be an integer in the range 0 .. string.length"

If the value returned by calling replace is not a String, it is converted to a String using its toString method, which must then return a string.

Please, add the same statement to the  String.replaceAllMapped() documentation (https://api.dartlang.org/stable/1.21.0/dart-core/String/replaceAllMapped.html)

https://codereview.chromium.org/2637103002/
I don't think we want to add the 2nd paragraph to the comments: the function type clearly requires to get a String back, so the only thing we need to worry about, is null.
@lrhn: should null be allowed or should we explicitly throw?

Agree on the string.
We generally only say if a type includes null, the default is that null is not allowed (which will make non-null by default an easier switch!)
In practice, it works, but I don't want to document it, and when we go non-null-by-default, it will become non-nullable String as return type.

Both replaceAllMapped() and replaceFirstMapped() have the same function type.
https://api.dartlang.org/stable/1.21.0/dart-core/String/replaceFirstMapped.html

String replaceFirstMapped(
Pattern from,
String replace(
Match match
), [
int startIndex = 0
])

https://api.dartlang.org/stable/1.21.0/dart-core/String/replaceAllMapped.html

String replaceAllMapped(
Pattern from,
String replace(
Match match
)
)

But replaceFirstMapped() documentation contains statement about what is going on if not a String is returned, but replaceAllMapped() not. See an example, demonstrating why documentation should be consistent below
class C {
  String toString() {
    return "C";
  }
}

main() {
  print("1231".replaceFirstMapped("1", (m) {return new C();})); // prints C231, this is according to the documentation
  print("1231".replaceAllMapped("1", (m) {return new C();})); // prints C23C, documentation doesn't describe this case
}


tracing: remove stray event_logger code

Remove obsolete kernel event logger code.
Signed-off-by: Anas Nashif<EMAIL_ADDRESS>

Codecov Report

Merging #9811 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #9811      +/-   ##
==========================================
+ Coverage   52.23%   52.24%   +0.01%
==========================================
  Files         212      212
  Lines       25954    25949       -5
  Branches     5594     5593       -1
==========================================
  Hits        13558    13558
+ Misses      10160    10155       -5
  Partials     2236     2236

Impacted Files
Coverage Δ

kernel/sched.c
91.18% <ø> (ø)
:arrow_up:

kernel/sem.c
93.02% <ø> (+9.68%)
:arrow_up:

Continue to review full report at Codecov.

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


Add Bundle Size Check?

https://github.com/siddharthkp/bundlesize
Can integrate this with GitHub checks, but I don't know which files to specify for checking bundle size.

premature


when use getInitialUri function open a closed(coldstart)  app, it will automatic hide in backend (not crashed)

i am use iphone x. if the app is runing in backend.it works fine. but if the app is been killed. when use unilink to start from coldstart. it will auto hide in backend. any one have some problem like this?

have you resolve this issue? can you please share its solution ?
i am facing the same issue.


[Performance] Investigation of performance issues for web components

User Story

As a [user role below]:
IBM.com application offering manager

I need to:
have performant pages in my IBM.com application using web components

so that I can:
score higher in SEO rankings

Additional information

Some initial tests were done on the pages: https://www.ibm.com/partners, and https://www.ibm.com/testing/partners (page without ibm-common.js)
Page itself has image loading opportunities, but should focus more on other optimizations around number of DOM elements, script evaluation time, etc

Acceptance criteria

[ ] Identify opportunities and tasks on how the web components package can be improved in page speed performance
[ ] Playback to the engineering team
[ ] Create issues / next steps

@RobertaJHahn here is another story for dev research, and also a candidate for sprint 8. We should timebox this exercise probably, but need to estimate.

After investigation there are no more improvements to be made to performance at this time. The remaining issues affecting performance exist on the application side. Based on Lighthouse audits the main contributing factor to the low score are long load/paint times. There has been some improvement from https://github.ibm.com/partnerworld/partnerworld-landing/pull/64 but more optimizations will need to be made in the application before more testing can be done.


Allow urllib3 1.23

Changelog: https://github.com/urllib3/urllib3/blob/1.23/CHANGES.rst#123-2018-06-04

Codecov Report

Merging #4669 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4669   +/-   ##
=======================================
  Coverage   66.62%   66.62%
=======================================
  Files          15       15
  Lines        1564     1564
=======================================
  Hits         1042     1042
  Misses        522      522

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 00fd4c8...ac944b7. Read the comment docs.

Can you please bump requests with this fix so we can fix the current version conflicts?
There are a lot of other libs out there that don't cap urllib3 meaning they are failing due to this cap

Hi @bar, I’ve just commented on a PR to address this. We usually won’t modify this until we are ready for a release. It shouldn’t break anything, but simply emits a warning.
We’ll have a release out in the next week or two once urllib3’s new version has had a chance to settle. I’ll try to get master a bit cleaner this evening.

Lots of warning emails send me here. I'm going to edit the local code now because I don't like the coming thousands of emails.


Add analytics dashboard

Purpose
This PR

Refactors the existing widgets to use dimension selector
Adds the new manager analytics and publisher analytics dashboards.
Add drill down across widgets

........................................................................................
Publisher analytics
Overall Usage Page

API Usage Page

API Performance Page

API Faults Page

................................................................................................................
Manager Dashboard
Overview Page

API and Application Page

Developer Statistics Page

Overall API Usage Page

API Usage Page

API Performance Page

Top Faulty Page

API Faults Page

Devices Page

Geo Page

Fixes https://github.com/wso2/analytics-apim/issues/1144


Cannot hear speaker audio

Describe the bug
I cannot hear audio in rooms. The recv transport state is initially connecting for a while and then becomes disconnected. On refresh of the page, I am asked to click an okay button and then the same process happens.
To Reproduce
Specifically on my machine:

Join a room
Don't hear audio

Expected behavior
Hear audio.
Screenshots

What device are you on?
Windows 10 / Google Chrome Version 90.0.4430.85

Make sure to take a screenshot of the console in devtools next time it happens

Closing it as we don't have much information


GitHub workflow for running unit tests for MKL backend on CPU

Description
GitHub Actions workflow to run unit tests for MKL/CPU.
I disabled tests that require GPU by excluding tests where the name ends in gpu. RNG and DFT domains finish in 10 minutes. I am still waiting for BLAS/LAPACK to finish. We could have nightly testing that runs everything and PR testing that runs a smaller set. Any thoughts on how to reduce the testing for PR's?
@mkrainiuk

lapack finishes in 30 minutes and blas takes 1.5 hours. All the time is building the tests. Combining multiple tests into a single binary might make testing faster without reducing coverage.

@hjabird: Added CPU to job name to address: https://github.com/oneapi-src/oneMKL/pull/482#discussion_r1598245390

Any thoughts on how to reduce the testing for PR's?

I guess we can also skip building GPU part with -DENABLE_MKLGPU_BACKEND=False for now since it's not tested anyway, so it will save some build time, and we can run only runtime dispatching tests and ignore compiler time dispatching tests with -E /CT/, it should give us significant testing time reduction.

I made the change. It runs about 25% faster, which makes up for the intel hosted runners being 25% slower than github hosted runners. When you move to onemkl-project org, we can use github hosted runners without contending with other projects.


Shutting down virtual machine.
+> Deleting nanobox files (/Users/lyon/.nanobox/apps/odin)
+> Removing odin.dev domain from /etc/hosts
odin :: (feature/apps) » nanobox dev -v
+> Creating a nanobox
Importing nanobox base image....
Booting virtual machine.......
Configuring virtual network
A fatal error occurred (See ~/.nanobox/nanobox.log for details). Exiting...

cat ~/.nanobox/nanobox.log:
2015-11-11 15:45:14 INFO  ==> odin: Forcing shutdown of VM...
2015-11-11 15:45:15 INFO  ==> odin: Destroying VM and associated drives...
2015-11-11 15:45:15 INFO  ==> odin: Running cleanup tasks for 'shell' provisioner...
2015-11-11 15:45:15 INFO  ==> odin: Running cleanup tasks for 'shell' provisioner...
2015-11-11 15:45:15 INFO  ==> odin: Running cleanup tasks for 'shell' provisioner...
2015-11-11 15:45:15 INFO  ==> odin: Running cleanup tasks for 'shell' provisioner...
2015-11-11 15:46:06 FATAL [commands/create] vagrant.Up() failed -  - exit status 1
2015-11-11 15:46:06 LOG Closing log now

if i do a 'nanobox dev' immediately after the failure it works

UPDATE: i noticed there was no real output for the vagrant command so i cd'ed into the ~/.nanobox/apps/appname folder and ran a vagrant up:
 » vagrant up                                                                                ~/.nanobox/apps/odin
Bringing machine 'odin' up with 'virtualbox' provider...
==> odin: Importing base box 'nanobox/boot2docker'...
==> odin: Matching MAC address for NAT networking...
==> odin: Checking if box 'nanobox/boot2docker' is up to date...
==> odin: A newer version of the box 'nanobox/boot2docker' is available! You currently
==> odin: have version '0'. The latest is version '0.16'. Run
==> odin: `vagrant box update` to update.
==> odin: Setting the name of the VM: odin
==> odin: Clearing any previously set network interfaces...
==> odin: Preparing network interfaces based on configuration...
    odin: Adapter 1: nat
    odin: Adapter 2: hostonly
==> odin: Forwarding ports...
    odin: 22 => 11870 (adapter 1)
==> odin: Running 'pre-boot' VM customizations...
==> odin: Booting VM...
==> odin: Waiting for machine to boot. This may take a few minutes...
    odin: SSH address: <IP_ADDRESS>:11870
    odin: SSH username: docker
    odin: SSH auth method: password
    odin: Warning: Connection timeout. Retrying...
    odin: Warning: Connection timeout. Retrying...
    odin:
    odin: Inserting generated public key within guest...
    odin: Removing insecure key from the guest if it's present...
    odin: Key inserted! Disconnecting and reconnecting using new SSH key...
==> odin: Machine booted and ready!
==> odin: Checking for guest additions in VM...
==> odin: Configuring and enabling network interfaces...
==> odin: Exporting NFS shared folders...
NFS is reporting that your exports file is invalid. Vagrant does
this check before making any changes to the file. Please correct
the issues below and execute "vagrant reload":

exports:1: path contains non-directory or non-existent components: /Users/lyon/pagoda/kernel/kernel-app-dashboard
exports:1: no usable directories in export entry
exports:1: using fallback (marked offline): /

my vagrant file looks like this:
################################################################################
##                                                                            ##
##                                   ***                                      ##
##                                *********                                   ##
##                           *******************                              ##
##                       ***************************                          ##
##                           *******************                              ##
##                       ...      *********      ...                          ##
##                           ...     ***     ...                              ##
##                       +++      ...   ...      +++                          ##
##                           +++     ...     +++                              ##
##                       \\\      +++   +++      ///                          ##
##                           \\\     +++     ///                              ##
##                                \\     //                                   ##
##                                   \//                                      ##
##                                                                            ##
##                    _  _ ____ _  _ ____ ___  ____ _  _                      ##
##                    |\ | |__| |\ | |  | |__) |  |  \/                       ##
##                    | \| |  | | \| |__| |__) |__| _/\_                      ##
##                                                                            ##
## This file was generated by nanobox. Any modifications to it may cause your ##
## nanobox VM to fail! To regenerate this file, delete it and run             ##
## 'nanobox init'                                                             ##
##                                                                            ##
################################################################################

# -*- mode: ruby -*-
# vi: set ft=ruby :

#
Vagrant.configure(2) do |config|

  # add the boot2docker user credentials to allow nanobox to freely ssh into the vm
  # w/o requiring a password
  config.ssh.shell = "bash"
  config.ssh.username = "docker"
  config.ssh.password = "tcuser"

  config.vm.define :'odin' do |nanobox|

    ## Set the hostname of the vm to the app domain
    nanobox.vm.provision "shell", inline: <<-SCRIPT
      sudo hostname odin.dev
    SCRIPT

    ## Wait for nanobox-server to be ready before vagrant exits
    nanobox.vm.provision "shell", inline: <<-WAIT
      echo "Waiting for nanobox server..."
      while ! nc -z <IP_ADDRESS> 1757; do sleep 1; done;
    WAIT

    ## box
    nanobox.vm.box     = "nanobox/boot2docker"

    ## network

    # add custom private network and ip and custom ssh port forward
    nanobox.vm.network "private_network", ip: "<IP_ADDRESS>"
    nanobox.vm.network :forwarded_port, guest: 22, host: 11870, id: 'ssh'

    ## shared folders

    # disable default /vagrant share (overridden below)
    nanobox.vm.synced_folder ".", "/vagrant", disabled: true

    # add nanobox shared folders
    nanobox.vm.synced_folder "~/.ssh", "/mnt/ssh"
    nanobox.vm.synced_folder "/Users/lyon/pagoda/nanobox/odin", "/vagrant/code/odin",
      type: "nfs",
      mount_options: ["nfsvers=3", "proto=tcp"]

    ## provider configs
    # VirtualBox
    nanobox.vm.provider "virtualbox" do |p|
      p.name = "odin"

      p.customize ["modifyvm", :id, "--natdnshostresolver1", "on"]
      p.customize ["modifyvm", :id, "--cpuexecutioncap", "50"]
      p.cpus = 2
      p.memory = 1024
    end

    ## wait for the dhcp service to come online
    nanobox.vm.provision "shell", inline: <<-WAIT
      attempts=0
      while [ ! -f /var/run/udhcpc.eth1.pid && $attempts -lt 30 ]; do
        let attempts++
        sleep 1
      done
    WAIT

    # kill the eth1 dhcp server so that it doesn't override the assigned ip when
    # the lease is up
    nanobox.vm.provision "shell", inline: <<-KILL
      if [ -f /var/run/udhcpc.eth1.pid ]; then
        echo "Killing eth1 dhcp..."


\1 Gives a wrong output

I was making a syntax and when you use "\\1" it gives you "\`" in the output, but the bug is only with 1, with other numbers works great.

@isagalaev  can we check this please?

Could you be more specific, please? I've no idea what you're talking about :-)

When you use "\1" in every language you get "`" instead of "\1", but when you use "\2" or other number it gives you exactly that.

Use where? Are we talking about language definitions or highlighted snippets or something else?

In other words, tell me what exactly to do to reproduce it.

puts \\1
puts \\2

Use this and see the output.

[quote]
puts \\1
puts \\2
puts \\3

[/quote]
Use this and see the output.

Works for me, no backticks: https://jsfiddle.net/zock61nm/
Could you reproduce it in the jsfiddle or set up your own page somewhere showing the problem?
Which version of highlight.js are you using by the way?

The issue is in this repository, they´re using the version 9.2.0 actually, i think it's the latest.
https://meta.discourse.org/t/1-in-code-block-fails/39956

Well, looks like your assumption about it being a bug in highlight.js didn't prove to be correct (as show in the jsfiddle above, and can also be easily tested with highlight.js locally.) My bet would be that it might be cause by the markdown processor they use in Discourse. (But please do test it before filing a bug against the markdown library!)


Cleanup agentclusterinstall condition handling

Cleans up and simplifies the way conditions are checked in the agentclusterinstall resource.
Previously condition handling was overly complicated and not aligned with any other packages of the repository.

/lgtm


Prepare for new release of Bitbucket

Workflow links: https://github.com/atlassian-labs/atlassian-slack-integration-server/actions/runs/10778416891, https://github.com/atlassian-labs/atlassian-slack-integration-server/actions/runs/10778421463, https://github.com/atlassian-labs/atlassian-slack-integration-server/actions/runs/10778425812

Integration tests pass, so current version in a branch release-1.x of Bitbucket DC for Slack is marked compatible with Bitbucket 8.19.9


Users can choose minimizers to optimize inducing features

travis is green now.
I merge this.

Cool!
It would be cool to have a numerical comparison for those, don't you think?
Maybe in some notebook?


Refactor cookie banners on Apply (and Manage) to use JavaScript

Context
We have updated the cookie banner on Apply (and Manage… and Find) to follow the latest guidance from GDS.
The banner on Apply (and Manage) is implemented differently to that on Find, in that it doesn’t use JavaScript.
Changes proposed in this pull request

Refactor the cookie banners to use JS to prevent page reloads and make for a generally nicer cookie 🍪 experience
No-JS fallback banner added. If a user has JS disabled then they will be told to turn it on

Guidance to review

Open up an incognito tab and click through the cookie banners that you see appear. No page reloads!

Accept cookies

No JS fallback banner

Link to Trello card
https://trello.com/c/4yQCrJjz/3267-🍪-refactor-cookie-banner-on-apply-and-manage-to-use-javascript-to-prevent-page-reloads
Things to check

[x] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] API release notes have been updated if necessary
[ ] This code does not rely on the addition/removal of Azure config environment variables in the same Pull Request
[ ] New environment variables have been added to the Azure config

@george-bit @davidgisbey @despo
One of the potential downsides of this PR is reduced system spec coverage (see https://github.com/DFE-Digital/apply-for-teacher-training/pull/4511/commits/0166e68770b36bc9c0029f7abc2a7864d0281cf8) but bear in mind we have JS specs ini place (although they are unit tests not integration tests 😞)
Are we happy with this or do you think we need to add a card to configure RSpec to run JS?

Are we happy with this or do you think we need to add a card to configure RSpec to run JS?

We've got this far without it — we have even had it in the past and removed it — and I'm reluctant to reintroduce it as it introduces extra dependencies and slows things down. If we're working with standard components and we have unit tests (as we do, here) I'd rather not take the hit. Happy to hear other opinions, of course.

Are we happy with this or do you think we need to add a card to configure RSpec to run JS?

We've got this far without it — we have even had it in the past and removed it — and I'm reluctant to reintroduce it as it introduces extra dependencies and slows things down. If we're working with standard components and we have unit tests (as we do, here) I'd rather not take the hit. Happy to hear other opinions, of course.

Yep makes sense to me @duncanjbrown


Subaru legacy: flip driver torque

This resolves permanent steering faults when the user overrides steering and then releases the steering wheel. Thanks to @martinl for this fix.

https://github.com/martinl/panda/commit/8b7da38988a66b269a310479ed08973fb3635807

Requires

https://github.com/sunnyhaibin/panda/pull/6

The user confirmed the issue is fixed. b81cc3ea44acd372|2023-05-28--18-14-13


fix: handle PermissionError in reading libdir.is_dir()

pyodide-build fails with scikit-build-core, because scikit-build-core tries to stat() the libdir given by sysconfig. I'm not sure what the bug is here; should we be able to stat LIBDIR under pyodide-build? I suspect not if we're pretending to be the target environment, but I'm not certain.
This is a very specific fix, and I suspect there's a nicer way of generalising this to the  class of "this isn't a conventional Python distribution" errors. However, this patch as-it-stands fixes pyodide-build, so it's a starting point for a proper fix.

Getting the lib for is extremely unimportant - it only matters at all for Windows, and I believe we’re still ignoring it for FindPython.

What happens if exists is used instead of is_dir? Does it just move the permission error down? (Also would want it to return None for permission errors, but just wondering)
Also there might be a better way to handle cross compiles.

next step from your comment is

I was just says this was fine, finding the lib is not very important.

Thanks!


JSON decode failures for Plugins

The wp.org API returns unexpected values for several plugins, which results in errors when decoding the JSON into the plugin struct. To resolve this we need to check for unexpected values in the author_profile field and return an empty string.
This currently causes around 12 plugins to be incorrectly marked as closed and as such not searched.

This was fixed, and many other similar issues with Theme data, by the wporg repo which is a standalone wrapper for the WP.org API.