RUMM-1354: Update AGP version to 4.2.1, build with JDK 11

What does this PR do?
This change updates AGP to version 4.2.1, modifies Docker image to build with JDK 11, bumps Gradle version to 6.9.1, also updates Android Command Line Tools version and Kronos and Gson versions.
This is intermediate step before jumping to AGP 7/Gradle 7/Kotlin 1.5.30 in the future.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #708 (41797e1) into master (0944c99) will decrease coverage by 0.11%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #708      +/-   ##
============================================
- Coverage     88.92%   88.82%   -0.11%
+ Complexity     1713     1708       -5
============================================
  Files           189      189
  Lines          5651     5651
  Branches        694      694
============================================
- Hits           5025     5019       -6
- Misses          392      393       +1
- Partials        234      239       +5

Impacted Files
Coverage Δ

...id/log/internal/domain/event/LogEventSerializer.kt
94.44% <0.00%> (-5.56%)
:arrow_down:

...droid/rum/tracking/ActivityViewTrackingStrategy.kt
84.62% <0.00%> (-3.85%)
:arrow_down:

.../android/error/internal/DatadogExceptionHandler.kt
93.10% <0.00%> (-3.45%)
:arrow_down:

...oid/src/main/kotlin/com/datadog/android/Datadog.kt
77.27% <0.00%> (-1.14%)
:arrow_down:

...roid/rum/internal/domain/scope/RumResourceScope.kt
93.51% <0.00%> (-0.65%)
:arrow_down:

...atadog/android/core/configuration/Configuration.kt
95.90% <0.00%> (-0.37%)
:arrow_down:

...g/android/rum/internal/domain/scope/RumEventExt.kt
84.68% <0.00%> (ø)

...m/internal/domain/scope/ExternalResourceTimings.kt
67.65% <0.00%> (ø)

...g/internal/domain/event/DdSpanToSpanEventMapper.kt
91.84% <0.00%> (ø)

...l/net/info/BroadcastReceiverNetworkInfoProvider.kt
77.08% <0.00%> (ø)

... and 1 more


Tibber integration

This is an integration for the Norwegian electricity company Tibber. They also operate in Sweden and Germany (and I think they are moving into more countries as well)
The integration uses OAuth to log the user into their Tibber account, and it returns daily activities for each of the users "homes". A user can have more than one home registered in Tibber.
Closes #38

@corradio I've also gotten a OAuth client id and secret from Tibber that I've used in my testing. Not sure how you handle that for integrations like this

Hey @torbjornvatn thanks for your contribution!
What exactly do you mean by Not sure how you handle that for integrations like this?. Is it a third party client id / secret ? (I have not looked at the API you're calling yet)
I'll review the PR next week

What exactly do you mean by Not sure how you handle that for integrations like this?. Is it a third party client id / secret ? (I have not looked at the API you're calling yet)

Yes, I've gotten a client id / secret from the Tibber folks that they've configured with callbackurls. I was just wondering where you put those when you go live with the integration. I've been using an env.json file locally when testing.

@pierresegonne I've written to you on the Tomorrow Slack about the 3d party id & key

Ok I've been testing the integration on my side and there are a few things that need to be done to make the integration work:

Fix conflicts
Make sure that the client secret and id you transmitted to me are valid (I receive the following error

)
When I try to open the webview.

Hi @torbjornvatn,
Have you seen that we will be sunsetting north? https://www.tmrow.com/blog/sunsetting-north/
Because of that, I'll have to close down this PR. I am very sad to have to do that and not see your work come live on the App. Thanks a lot for the efforts you invested into this, hopefully we can re-use parts of what you contributed in the future.
Let me know if you have any question!


Remove Bootstrap and replace by Tailwind

Currently we are using a bootstrap theme and classes. Remove the  from index.html, remove the theme.css and replace the Bootstrap styles in all HTML elements by Tailwind.

@divyaYK  is working on #30, so we can use complete this issue with that one.
What do you say? @tiagostutz

@divyaYK is working on #30, so she can complete this issue with along that one.
What do you say? @tiagostutz

I offered @ishagupta1548 so she can start with an issue. I think they could go in parallel, by @divyaYK using Tailwind in #30 and @ishagupta1548 in this issue here.
How about that?

Sounds good to me 😄

Hey @ishagupta1548! 😊
Are you working on this issue?
If not, I'd like to take it up. Please let me know!

Hi @divyaYK ! I offered the issue to @ishagupta1548, but as she didn't give a sign that she wants to take that I'm assigning it to you and creating another issue to her.
Thanks! 🚀


added flappy bird game

What does this PR do?
Adds code for an android project of creating the famous flappy bird game
Related PRs and Issues
solves #550
Have you read the Contributing Guidelines?
Yes
I will add the readme after discussion with the maintainer

@AntoineDoniel please add a proper readme

Please check PR #567


Don't compile the source SolarNews

Hi friend,
When I clicking to test on device I got the next log on XCode
Undefined symbols for architecture x86_64:
"OBJC_CLASS$_RCTSRWebSocket", referenced from:
objc-class-ref in libReact.a(RCTInspectorPackagerConnection.o)
"OBJC_CLASS$_RCTReconnectingWebSocket", referenced from:
objc-class-ref in libReact.a(RCTPackagerConnection.o)
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
Is something wrong or omitted with your code?
thanks in advance.

Hi,
can you plz try this solution :
Target ->Linking -> other linker Flag and add $(inherited) in other linker flag in both Debug and Release


fix for issue #316

Description
Simple fix for when opengl doesn't have support for the sample's required extension.  This should now fail gracefully and display a reason.
Fixes #316
General Checklist:
Please ensure the following points are checked:

[x] My code follows the coding style
[x] I have reviewed file licenses
[x] I have updated the header Copyright to reflect the current year (CI build will fail if Copyright is out of date)
[x] I have commented any added functions (in line with Doxygen)
[x] I have commented any code that could be hard to understand
[x] My changes do not add any new compiler warnings
[x] My changes do not add any new validation layer errors or warnings
[x] I have used existing framework/helper functions where possible
[x] My changes build on Windows, Linux, macOS and Android. Otherwise I have documented any exceptions
[x] My changes do not add any regressions
[x] I have tested every sample to ensure everything runs correctly
[x] This PR describes the scope and expected impact of the changes I am making

Sample Checklist
If your PR contains a new or modified sample, these further checks must be carried out in addition to the General Checklist:

[ ] I have tested the sample on at least one compliant Vulkan implementation
[ ] If the sample is vendor-specific, I have tagged it appropriately
[ ] I have stated on what implementation the sample has been tested so that others can test on different implementations and platforms
[ ] Any dependent assets have been merged and published in downstream modules
[ ] For new samples, I have added a paragraph with a summary to the appropriate chapter in the samples readme

Merging now that the checks are passing (per discussion on last Samples call)


System.NullReferenceException when using Globalsearch in a template column with related data

I'm using version 1.11.1 and I get  an error when I have a template column that uses shows a field from a related model. The error is System.NullReferenceException: 'Object reference not set to an instance of an object.'
Here is what the offending column look like:
<Column TableItem="Model" Title="Region" Field="@(x => x.Region.Name)" Sortable="true" Filterable="true" Width="5%">
	<Template>
		@(context.Region != null ? context.Region.Name : string.Empty)
	</Template>
</Column>

Here is my entire table definition:

	<Column TableItem="Model" Title="Name" Field="@(x => x.Name)" Sortable="true" Filterable="true" DefaultSortColumn="true" Width="10%" />
	<Column TableItem="Model" Title="Phone Number" Field="@(x => x.PhoneNumber)" Sortable="true" Filterable="false" Width="5%" />
	<Column TableItem="Model" Title="Email" Field="@(x => x.EmailAddress)" Sortable="true" Filterable="false" Width="10%">
		<Template>
			<a href="mailto<EMAIL_ADDRESS>		</Template>
	</Column>
	<Column TableItem="Model" Title="Locality" Field="@(x => x.Locality)" Sortable="true" Filterable="true" Width="5%" />
   <Column TableItem="Model" Title="Region" Field="@(x => x.Region.Name)" Sortable="true" Filterable="true" Width="5%">
		<Template>
			@(context.Region != null ? context.Region.Name : string.Empty)
		</Template>
	</Column>
	<LoadingDataTemplate>

Loading...

	</LoadingDataTemplate>
	<Pager ShowPageNumber="true" ShowTotalCount="true" />

Everything works great if I comment out the "Region" column.
Originally posted by @tkennedy13 in https://github.com/IvanJosipovic/BlazorTable/issues/135#issuecomment-649041668

I'm not the owner of the repo, but when I was looking at the source code I think it still trying to access the related entity (which would fail when the Region is null in this case, obviously) when it runs the global search.
Essentially, AFAIK it doesn't search based on the text in the template, but rather the data in the field you pass in.
I bet you could load your data into a DTO where you don't have a related entity, just a primitive field "RegionName" that way you won't run into null reference exceptions.

Hi, please try the latest release 1.12.0.

I installed version 1.12 and it works great! Thank you for the fix and thank you for the package it has been very helpful.


Added ability to add extra tags to DataDog  (#11)

This is the first proposal for fixing issue #11
The pull request adds the ability to add tags to DataDog.

Tags can be added by reading a file, or entering a properties file that follows the standard java property file syntax: {k}={v}. The value can here be something that is expanded by Jenkins, for example an environment variable or a build parameter or a fixed value.
The tags are added to event, metric and the service check.
Comments are welome.

Thank you for this pull request! Please check this document for how the Jenkins project handles pull requests.

Hey @MadsNielsen! Thanks for the PR :)
Would you mind making this pull request against our dev repo?
https://github.com/DataDog/jenkins-datadog-plugin


[terra-paginator] Cometic effects of text wrapping

Description

At certain dimensions the Paginator text wrapping can be aggressive. We should get UX feedback on the best way to handle this scenario.

Steps to Reproduce

Navigate to the terra-ui Paginator example page.
Slowly resize the window while observing the Paginator example.
Once the text wraps  observe the visual state of the component.
Navigating to double digit page numbers will exaggerate the effect.

Expected Behavior

We should reach out to UX and discuss what the expected behavior should be.

Removing overflow-wrap here seems to work fine.
Your thoughts, cc @neilpfeiffer .

After having discussions with @neilpfeiffer about the history of this component, we've decided that this is not a bug.  This was determined acceptable behavior when it was originally created.  I'd like to change this project into an enhancement with some UX guidance on what the proper way to fix all permutations of small screen sizes.  Possibly something similar to collapsible menu.


Not rendering properly when using alongside with tcomb-form-native

I just opened an issue on tcomb-form-native. When I use this library alongside with tcomb-form-native, some fields not showing properly.
e.g. Fumi, Kohan are not rendering properly but Sae does.
I just wanted to mention it, thanks.

Could you post screenshots of the problem and some example code to reproduce it @newfylox? I'll try taking a look at the problem when I have some time.

Closing due to inactivity.


Does it work well on Discord bots

Hey Joshua I've been using sqlite the promise based version of sqlite3 for my discord bots, until I found yours but am hesitant to change over because I'm not sure if this would work well with Discord bots which are also promise based.

Promises naturally compose with synchronous values, so it should be fine.


Convert to REP 143

We want to be able enable the pull request builders and other enhancements

Resolved in #10309


Support credential-scopes

Python example: https://github.com/Azure/azure-sdk-for-python/blob/c509c6bf548ebe4918cd5f3503bcbd47ca982170/sdk/synapse/azure-synapse-accesscontrol/azure/synapse/accesscontrol/_configuration.py#L51
Example in readme: https://github.com/Azure/azure-rest-api-specs/blob/master/specification/synapse/data-plane/readme.python.md#python

This have already been supported.


typos: the laravel configuration file has spaces in the env values instead of underscores

The file located here https://spatie.be/docs/ray/v1/configuration/laravel
Has these 5 env values with a space in it. Those values would be invalid if set in an .env file
env('SEND CACHE_TO_RAY', false)
env('SEND JOBS_TO_RAY', false)
env('SEND QUERIES_TO_RAY', false)
env('SEND REQUESTS_TO_RAY', false)
env('SEND VIEWS_TO_RAY', false)

Could you send a PR to fox this?

@freekmurze Didn't realize I could.. done! https://github.com/spatie/ray/pull/366


Add healthchecks for the register service

This builds on top of https://github.com/m-lab/autojoin/pull/28 and updates the register container to the latest (service) version, which keeps refreshing the registration every configured interval rather than doing it just once and then exiting.
All the other services in the file depend on the register container being healthy, which is signaled by the HTTP status code returned by its /ready endpoint.
I'm going to replace latest with a tagged version after the PR above is merged.

This change is

I can't write a reply in reviewable right now - but I didn't have any particular rationale in mind - I just didn't know what went into your choices so wanted to understand first. They really were "clarifying" questions!
LGTM


v2.3.0: Tab selects autocomplete Tags, instead of jumping to next form element

Hello,
with the new version, the tab key selects the selected entry in the autocomplete list. We use tab to navigate through a table / forms and when we tab into a tags-input field, we got a problem.
Can this be an option, so we can disbale the behaviour and use tab as intended? Or is this a new bug introduced and was never planned that way?
Kind Regards

That wasn't introduced in 2.3.0; it's been around since forever, I guess. But the tab key is only processed when the autocomplete list is open. When it's closed, that keystroke is ignored and the browser should handle it normally.

That is true, but I am using load-on-focus and would like to keep doing so.
Kind Regards
Sent from my iPhone

On 02.04.2015, at 06:14, Michael Benford<EMAIL_ADDRESS>wrote:
That wasn't introduced in 2.3.0; it's been around since forever, I guess. But the tab key is only processed when the autocomplete list is open. When it's closed, that keystroke is ignored and the browser should handle it normally.
—
Reply to this email directly or view it on GitHub.

There's an old requested feature to enable custom hotkeys and I intended to work on it for the next release. So maybe it'll be easier to turn off the tab keystroke.
In the meantime I suggest that you fork this repo and disable the tab processing yourself. All you have to do is change this line.

Can this issue be closed?

Closing due to inactivity (the issue can always be re-opened if necessary).

Hope you get to this. It's very unintuitive having tab make a selection from the autocomplete list.


Can't reproduce F1 scores

After training the model using the default parameters:
python train.py --data_dir data/msra --bert_model_dir bert-base-chinese-pytorch --model_dir experiments/base_model
I ran evaluation with
python evaluate.py --data_dir data/msra --bert_model_dir bert-base-chinese-pytorch --model_dir experiments/base_model
and got the following results, which are very much lower than those described in the "detail results on the test set" from the repo README:

Test metrics: loss: 00.05; f1: 58.29
precision    recall  f1-score   support
  PER      35.36     90.09     50.79      1372
  ORG      35.24     85.70     49.94      1287
  LOC      53.90     92.11     68.01      2790

avg / total      44.83     90.09     59.41      5449
Any advice on how to reproduce the expected scores would be greatly appreciated.

Thank you for your feedback, sorry for replying to you for so long.
According to my experiment, the reason for your poor result should be the pytorch-pretrained-bert  version. When I try to use the latest version, it will have a very bad effect. When I switch to version 0.4.0, it will return to normal.
It is recommended to install a lower version of pytorch-pretrained-bert using pip install pytorch-pretrained-bert==0.4.0, and I will update the code later.

I re-clone the code again, the results of the first two epoch are as follows, [default hyper-parameters] (https://github.com/lemonhu/NER-BERT-pytorch/blob/master/experiments/base_model/params.json) has been updated.
100%|███████████████████████████| 1400/1400 [08:54<00:00, 2.72it/s, loss=0.023]
Train metrics: loss: 00.01; f1: 94.26
Val metrics: loss: 00.01; f1: 92.31
Found new best F1
Epoch 2/20
100%|███████████████████████████| 1400/1400 [08:33<00:00, 2.63it/s, loss=0.007]
Train metrics: loss: 00.00; f1: 96.90
Val metrics: loss: 00.01; f1: 94.16
Found new best F1

Thanks for the quick feedback. I've pulled the new version of the code and installed pytorch-pretrained-bert==0.4.0.  It's taking me longer than it should to verify the results because I keep running out of GPU memory (even after dropping the batch size) but will take another shot. Thanks again for your help!

The problem is that the GPU memory is not enough. You need to reduce the size of the batch_size in the hyper-parameter configuration file file params.json, such as, set batch_size = 8.

Dropping to batch_size=8 did the trick.  The first three epochs are looking great:
Epoch 3/20
100%|█████████████████████████████████████████████████████████████████████████████████████████████████████████████| 5250/5250 [37:59<00:00,  2.30it/s, loss=0.010]

Train metrics: loss: 00.01; f1: 96.39
Val metrics: loss: 00.02; f1: 91.76

Thanks for the help!

Here are my new evaluation results:

Test metrics: loss: 00.02; f1: 90.97
precision    recall  f1-score   support
  LOC      92.84     92.97     92.91      2790
  PER      92.42     93.29     92.85      1372
  ORG      82.54     87.41     84.91      1287

avg / total      90.30     91.74     91.00      5449
Much better!

Yeah, feel free to re-open if needed.


Propagate containerd-registry setting to Rancher

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Problem:

Starting in Rancher 2.7.5, Rancher has taken over managing the /etc/rancher/rke2/registries.yaml file. The end result is that it ends up overwriting Harvester's changes to that file as Harvester handles updates to the containerd-registry setting.
Solution:

In order to avoid dataloss for the containerd-registry setting when upgrading to a version of Harvester with embedded Rancher >= v.2.7.5:

Update the settings controller to sync the containerd-registry secret into Kubernetes secrets and update the cluster.provisioning.cattle.io object so that Rancher can render the registries.yaml file.
"Replace" the containerd-registry setting in the upgrade path with its current value so that the new settings controller syncs the secrets that Rancher needs.
Remove the SUC plan that Harvester once used for syncing the registries.yaml.

Related Issue: https://github.com/harvester/harvester/issues/4156
Test plan:

Default/empty containerd-registry setting:
1.1. Starting from a Harvester cluster that is using the default value for setting/containerd-registry, upgrade to a build of Harvester including this change
1.2. cat /etc/rancher/rke2/registries.yaml -> file either doesn't exist or is empty

containerd-registry setting only has mirrors
2.1. Configure the containerd-registry setting to use a mirror (i.e., bogus values like "example.com" with endpoint "http://example.com:5000" is fine for the purpose of the test)
2.2. Ensure cat /etc/rancher/rke2/registries.yaml consists of the mirror example.com with endpoint http://example.com:5000.
2.3. Upgrade to a build of Harvester that includes this change
2.4. cat /etc/rancher/rke2/registries.yaml consists of the mirror example.com with endpoint http://example.com:5000.

containerd-registry setting has mirrors and configs
3.1. Configure the containerd-registry setting with some values:
3.1.1. mirror: example.com
<IP_ADDRESS>. endpoint: http://example.com:5000
3.1.2. config
<IP_ADDRESS>. FQDN: example.com:5000
<IP_ADDRESS>.  Pass: 1234
<IP_ADDRESS>. Auth: YWRtaW46MTIzNA== (echo -n "admin:1234" | base64)
<IP_ADDRESS>. Identity token: identity1234
3.2. Ensure cat /etc/rancher/rke2/registries.yaml consists of the mirror and configs specified above
3.3. Upgrade to a build of Harvester that includes this change
3.4. Ensure cat /etc/rancher/rke2/registries.yaml consists of the mirror and configs specified above. Note that Rancher will probably serialize this as JSON instead of YAML (which Harvester serialized it as).
3.5. Check the Rancher Cluster object's .spec.rkeConfig.registries field (kubectl get cluster.provisioning.cattle.io local -n fleet-local -o yaml | yq '.spec.rkeConfig.registries') and ensure the mirrors match what you entered prior to the upgrade. Check that the configs section includes the FQDN entered before the upgrade, and record the value from authConfigSecretName which should exist.

# kubectl get cluster.provisioning.cattle.io local -n fleet-local -o yaml | yq '.spec.rkeConfig.registries'
configs:
  example.com:5000:
    authConfigSecretName: harvester-containerd-registry-6c0650f89de1ad48
mirrors:
  example.com:
    endpoint:
      - http://example.com:5000

3.6. Confirm the values stored in the secret referred to by authConfigSecretName are correct and what was entered prior to the upgrade:
# for key in password username auth identityToken host ; do kubectl get secret harvester-containerd-registry-6c0650f89de1ad48 -o yaml -n fleet-local | yq ".data.$key" | base64 -d  && echo "" ; done
1234
admin
YWRtaW46MTIzNA==
identity1234
example.com:5000

Just some feedback for the upgrade script, the rest is LGTM. Upgrade from v1.1.2 works perfectly for me; thanks for adding the detailed test plans.


Add more reference benchmarks

[ ] WebGoat
[ ] vuln-flask-web-app
[ ] WebGoat .NET
[ ] #4
[ ] FlowBot
[ ] SecurityShepherd
[ ] #7
[ ] Damn Small Vulnerable Web
[ ] IssueBlot
[ ] go-test-bench
[ ] vulnerable-spring-boot-application
[ ] CodeQL tests
[ ] gosec tests
[ ] go-sec-code
[ ] Go Vulnerable Web Application
[ ] swiss-cheese
[ ] java-web-apps
[ ] #6
[ ] ant-application-security-testing-benchmark

Benchmark
Lang
Kind

WebGoat
Java
web application

vuln-flask-web-app
Python
web application

WebGoat .NET
C#
web application

Juliet C#
C#
synthetic

FlowBot
C#
synthetic

SecurityShepherd
Java
web application

skf-labs
Python, Java
synthetic

Damn Small Vulnerable Web
Python
web application

IssueBlot
C#
web application

go-test-bench
Go
web application

vulnerable-spring-boot-application
Java
web application

CodeQL tests
Java
synthetic

gosec tests
Go
synthetic

go-sec-code
Go
synthetic

Go Vulnerable Web Application
Go
synthetic

swiss-cheese
Python
synthetic

java-web-apps
Java
web application

sast-rules
Java, C#, Python, Go
synthetic

Here is the ground truth for some of the benchmarks above.


AttributeError: module 'mxnet.contrib.symbol' has no attribute 'YoloOutput'

Got AttributeError: module 'mxnet.contrib.symbol' has no attribute 'YoloOutput' when trying to run the demo.

I am having the same issue.....

same as #34

Even after building mxnet from source ,i am getting the same error.
AttributeError: module 'mxnet.contrib.symbol' has no attribute 'YoloOutput'
I followed the following link for building the mxnet from source.
http://mxnet.incubator.apache.org/install/index.html

Hi,
I looked into the file mxnet/python/mxnet/contrib/symbol.py
There is no function called YoloOutput. But demo.py is redirecting to
out = mx.contrib.symbol.YoloOutput(data=pred, num_class=num_classes,
num_anchor=num_anchor, object_grad_scale=5.0, background_grad_scale=1.0,
coord_grad_scale=1.0, class_grad_scale=1.0, anchors=anchors,
nms_topk=400, warmup_samples=12800, name='yolo_output')
I checked for  'YoloOutput' in dir(mx.contrib.symbol)
which returns False only.

@vikash0837 Checkout the latest master, I forgot to update darknet symbol file in the previous update
It should be called Yolo2Output

@zhreshold  ,I rebuild with latest master. Still after building ,i am not getting the Yolo2Output function.
After build i tried
import mxnet as mx
'Yolo2Output' in dir(mx.contrib.symbol) which returns False only

@zhreshold apart from the existing error there is also a bug in your latest changes.
inside symbol/symbol_darknet19_yolo.py
from symbol.symbol_darknet19 import conv_act_layer, stack_neighbor
you can't import stack_neighbor from symbol_darknet19.py as stack_neighbor function is not present in symbol_darknet19 .py
it has to be imported from common.py

@vikash0837 By latest master I mean the submodule mxnet in master branch, not official incubator-mxnet. Maybe you are mixing different mxnet.
Try
python -c "import mxnet; print(mxnet)"

to see where it locates.
Anyways, thanks for spotting the errors.

@zhreshold Thanks for your prompt reply. I am using submodule mxnet from your git only.
Here is the link . You can correct me if i am wrong.
https://github.com/zhreshold/mxnet.git
I couldn't find the module Yolo2Output anywhere in that repository. It is not populating after build as well .
Response of <python -c "import mxnet; print(mxnet)">

<module 'mxnet' from '/home/ubuntu/yoloDemo/updated-yolo/mxnet/python/mxnet/init.pyc'>

@zhreshold, could you please tell me how to resolve this problem. I am still getting this error.
Am I missing something.


Optionally enable incremental build

This enables faster iterations by doing an incremental compilation. Just compile with
VLLM_INCREMENTAL_BUILD_TORCH_PATH=`python -c "import torch; print(torch.__path__[0])"` pip install -e . --verbose

On my system, this reduces incremental compilation time from 2:40 min to 1:30 min, therefore enabling faster development iterations.
This works by making sure the torch include files are not from a temporary directory (if we don't do this, the temporary file path will be different on every invocation, which re-triggers a full build).
If you want even faster, you can develop/compile directly in /tmp/vllmcompile and use ninja to build from there (skipping the setuptools overhead). This reduces the compilation time to 30s.
It is quite possible that there is a less hacky version of this. Curious for any suggestions :)

One thing I noticed is the old python setup.py develop is also doing fast incremental compilation, but unfortunately that's deprecated and probably most people don't use it any more (e.g. it prints a big warning banner that it shouldn't be used any more and links to https://github.com/pypa/setuptools/issues/917).
The reason why python setup.py develop is fast is that it uses a stable temporary directory for the build, and it also uses the system torch path by default -- basically what this PR hacks pip install -e . to to also :)

I was thinking about this a little more, maybe it is best to just document ’python setup.py develop' since it has even less overhead. The other approach uses an internal variable in torch so not clear it is better. I'll make that change and ping you!


Use numpy.AxisError

Rel: #281
Jenkins is failing
(tests/cupy_tests/sorting_tests/test_sort.py:TestSort.test_external_sort_invalid_axis)

There is no AxisError on numpy 1.11.2.

https://github.com/cupy/cupy/issues/342

Closing, as this fix is problematic as @gwtnb pointed out.


gardener-resource-managers secret controller watches the right cluster

How to categorize this PR?

/area usability
/kind regression bug
What this PR does / why we need it:
This PR fixes a bug that prevents Shoot deletion for clusters that were created prior v1.59. On deletion, GRM is redeployed but its secret controller watches the wrong cluster (target instead of source). Hence, it does not cleanup the finalizers.
For Shoots created with v1.59 the deletion works because the secret controller was never adding the finalizer in the first place.
Which issue(s) this PR fixes:
Introduced with #6865
Fixes #6961
Special notes for your reviewer:
This regression was not discovered earlier because

integration tests only have one cluster (source=target)
e2e/TestMachinery tests don't cover this case (ref https://github.com/gardener/gardener/issues/2692)

Release note:

A bug has been fixed which caused stuck `Shoot` on deletion because their `Namespace`s in the seed cluster were not cleaned up properly. It only affected clusters created prior<EMAIL_ADDRESS>

/cherry-pick release-v1.59
cc @acumino, this does not affect v1.58.


Import Annotation bug

API Version: 6.4.0
MC Version: 1.18.1
What I did:

I tried to write the whole argument out, still error
Tried to write just @A and then see if it pops up in the menu

Result: Gave error
What should have happened: Should not have given an error and able to use it
Pictures:

CommandAPI version 6.4.0 is not compatible with 1.18.1 - you should be using version 6.5.2 or later.
(Also, this could be due to a myriad of other dependency-related issues, which you've not provided the information (e.g. pom.xml or build.gradle) for me to look into).


match SpecialKey to Comment instead of NonText

The SpecialKey highlight group has really low contrast with the background. Unlike NonText which is decorative in vim's UI, SpecialKey is actually part of the file being edited. I personally run into the problem a lot when editing vim macros or :s replacements.
This PR proposes replacing the highlight of SpecialKey to match that of Comment instead, which makes it readable while keeping the colors relatively muted.
Before/After screenshot:

Could you share the content of the buffer and its filetype?
SpecialKey is also used for listchars (pseudo characters for tab indent). Comment is too strong for that case...
Current impl:

Comment:

My screenshot was just an example, but as a usecase, if I want to get rid of blank lines, I might have a vimscript that contains:
%s/\n\n/^M/

This can be created in vim with:
set ft=vim
call feedkeys("i%s/\\n\\n/\<C-v>\<cr>/")

Did a little digging. Neovim has put listchars into a separate Whitespace highlight group (https://github.com/neovim/neovim/pull/6367), but I couldn't find reference to something similar in vim. Maybe we make this change just for neovim?

Neovim has put listchars into a separate Whitespace highlight group

Cool. It seems that Vim doesn't have an extra highlight group such as Whitespace. So my thought is...:

for Vim: waiting for Whitespace, keeping the current implementation
for Neovim: dimmed color for Whitespace and adding a link from SpecialKey to Comment

Pushed a commit. I've hardcoded the colors like everything else, but let me know if you want me to hi link it instead.

Thank you for fixing it. Because Iceberg uses a template engine to generate a result, I should also apply changes to the source file...

Is there an easy way to add conditional rules in the template? The only vim/neovim difference is in the terminal colors, and those require new member variables. (Sorry if I'm not familiar with pgmnt.)

Merged. Because SpecialKey is non-text, it should not have the same color as comment so I fine-tuned the color.


It's not clear which versions of node are supported

I couldn't find in docs which versions of node are supported. I want to use this tool on a library, which supports 0.10, 0.12, 4 and 6. It seems the linter consistently crashes on 0.10 and 0.12, so I'll have to run it conditionally in my CI.
(I know 0.10 and 0.12 versions should be doomed already, but still - what rocket science this linter uses that it can't run on those?)
It's not clear from any docs here whether crashing on 0.10 and 0.12 is a bug or feature.

Thanks for pointing this out. The package supports LTS, thus >= 4. Should add this to package.json .engines and a section into Readme.

but still - what rocket science this linter uses that it can't run on those?

FWIW even mainstream linters, like eslint, have dropped 0.10 and 0.12 support. Upgrading not only helps to ensure you're running on a Node that gets security fixes it also helps shape the support landscape as having EOL versions of Node in your CI test matrix can cause a false inflation of usage reports for those version.

Thanks. I knew 0.10 is dead, but didn't notice 0.12 will be with us only until the end of this year. Good!

Documented https://github.com/marionebl/conventional-changelog-lint#supported-nodejs-versions 🚀


Improve tooltip experience on both desktop and mobile.

Improve bubble menu tooltip.
Use Tippy.js library.

Looks clean. Can probably be closed.


Licensing for https://github.com/naudio/varispeed-sample

Hi Mark,
Your lib is beyond awesome!
I'd also like to use the varispeed-sample project (modify it a bit) in a commercial app. What's the licensing for it? Is it still MS-PL (besides soundtouch's licensing which is LGPL 2.1)?
Thanks!
Best,
John

hi John, yes the code I've written is all MS-PL. SoundToich has its own license.

Awesome, thanks!
I was asking, since the varispeed sample didn't have a license :)


Coordinates for muon impact parameter

The muon fitting method implemented by @AMWMitchell returns x and y coordinates of the impact.
The method implemented by @pcumani and me returns the radius and the polar angle of the maximum.
I have a few questions concerning these return values:

In which coordinate system are those x and y coordinates?

What is preferable?
I chose the polar representation, because the radius is the value that enters most calculations concerning light intensity.

What are the transformations needed to get from one to the other?

If the direct output from the fit is X,Y, I would like to save them for debugging/verification purposes.
But if, as you say, the radius is heavily used we can think about storage it, so that it does not have to be calculated every time, although I do not see that this calculation is so heavy computing demanding, specially considering coordinate transformation libraries, isn't it?

The ring fitting algorithm returns the X and Y coordinates of the centre of the ring, at the moment in camera coordinates, as well as the radius.
The intensity fitting algorithm returns the impact parameter only (not X and Y), and does not return an angle to the maximum.
Are you sure that our code returned the X and Y of the impact position? For the impact, the impact parameter (radius of impact) is usually more important than X and Y, as long as the mirror is near symmetric. For the circle fitting, the X and Y of the centre of the ring in the camera as well as the ring radius are useful.
The transformation between coordinates (camera frame to nominal, for example) can be done using  the coordinates class: ctapipe/coordinates
A polar/cartesian transformation can probably be done with numpy.

I was talking about the members of the MuonIntensityContainer class. I wanted to update my code to return this Container. It has the members
https://github.com/cta-observatory/ctapipe/blob/muonmaster/ctapipe/io/containers.py#L219
    impact_parameter_pos_x, impact_parameter_pos_y:
        position on the mirror of the muon impact

These are optional; with these classes, you don't have to fill in all parameters, as long as you fill in the main results of the fit. We should make more clear which parameters are those absolutely necessary.

Just remember that what you put in the container is what ends up in the output files as columns. So it could be useful to have both representations (similar to having ALT/AZ and RA/Dec coords in the event list file, even though you can compute one from the other).  Of course transforming from cartesian to polar coords is pretty simple, so maybe just pick a standard.

Ok. But then I would just give each "operation" it's own container. There should be imho no columns which are always nan or empty.
This is how IceCube does it, for example. And it works quite well for them from what I hear from our icecube folks.

I think Max's idea is good. Optional columns should have their own data structure if there are many of them. The intenion of the data writer code will be to be able to do something like this:
writer = Serializer(list_of_containers)
for event in source:
    writer.write(list_of_containers)

where the list of containers could be a single container (basic muon parameters) or several, in the case that an algorithm creates more information.  All containers in the list will end up as columns in the output (either merged into one table with the name prepended, or as separate tables, depending on what the user wants). Therefore, having muon code produce several outputs is fine.


Cant get styles to load

Followed the steps outlined. Components seem to be loaded, but the styles are missing or not being rendered.
<q-button>Primary</q-button>
      <q-button theme="secondary">Secondary</q-button>
      <q-button theme="link">Link</q-button>
      <q-button loading>Primary</q-button>

Hi, @FuriousLlama, can you show your full setup code?

Sure thing:
package.json
{
  "name": "my-website",
  "version": "0.1.0",
  "private": true,
  "scripts": {
    "serve": "vue-cli-service serve",
    "build": "vue-cli-service build",
    "lint": "vue-cli-service lint"
  },
  "dependencies": {
    "@qvant/qui-max": "0.18.0",
    "vue": "^3.2.13",
    "vue-class-component": "^8.0.0-0",
    "vue-router": "^4.0.3",
    "vuex": "^4.0.0"
  },
  "devDependencies": {
    "@typescript-eslint/eslint-plugin": "^5.4.0",
    "@typescript-eslint/parser": "^5.4.0",
    "@vue/cli-plugin-eslint": "~5.0.0",
    "@vue/cli-plugin-router": "~5.0.0",
    "@vue/cli-plugin-typescript": "~5.0.0",
    "@vue/cli-plugin-vuex": "~5.0.0",
    "@vue/cli-service": "~5.0.0",
    "@vue/eslint-config-typescript": "^9.1.0",
    "eslint": "^7.32.0",
    "eslint-config-prettier": "^8.3.0",
    "eslint-plugin-prettier": "^4.0.0",
    "eslint-plugin-vue": "^8.0.3",
    "prettier": "^2.4.1",
    "typescript": "~4.5.5"
  }
}

main.ts
import { createApp } from "vue";
import Qui from "@qvant/qui-max";
import '@qvant/qui-max/styles';

import App from "./App.vue";
import router from "./router";
import store from "./store";

const app = createApp(App);
app.use(Qui);

app.use(store);
app.use(router);

app.mount("#app");

Everything else is the default of the vue-cli setup.

@FuriousLlama
it's working in sandbox https://codesandbox.io/s/upbeat-wood-1egosc
Do you have some console/app errors ?
Have you tried reinstall node_modules with removing package.lock | yarn.lock ?

Hi i'm having an issue of the same type, css isnt loadind, here are my setup files :
package.json :
{
"name": "frontend",
"version": "0.1.0",
"private": true,
"scripts": {
"serve": "vue-cli-service serve",
"build": "vue-cli-service build",
"lint": "vue-cli-service lint"
},
"dependencies": {
"@qvant/qui-max": "^0.19.0",
"core-js": "^3.8.3",
"vue": "^3.2.13",
"vue-router": "^4.4.5"
},
"devDependencies": {
"@babel/core": "^7.12.16",
"@babel/eslint-parser": "^7.12.16",
"@vue/cli-plugin-babel": "~5.0.0",
"@vue/cli-plugin-eslint": "~5.0.0",
"@vue/cli-service": "~5.0.0",
"eslint": "^7.32.0",
"eslint-plugin-vue": "^8.7.1",
"sass": "^1.80.6",
"sass-loader": "^16.0.3"
},
"eslintConfig": {
"root": true,
"env": {
"node": true
},
"extends": [
"plugin:vue/vue3-essential",
"eslint:recommended"
],
"parserOptions": {
"parser": "@babel/eslint-parser"
},
"rules": {}
},
"browserslist": [
"> 1%",
"last 2 versions",
"not dead",
"not ie 11"
]
}
main.js :
import App from './App.vue';
import { createApp } from 'vue';
import Qui from '@qvant/qui-max';
import '@qvant/qui-max/styles';
const app = createApp(App);
app.use(Qui);
app.mount('#app');
and app.vue :

here is the render :

I even completly duplicated you version here : https://codesandbox.io/s/upbeat-wood-1egosc and have this render :

Maybe local development so the point is on installation i think

'On my version'
I'm sure components are loaded, at inspect we can see the html code is right :

but the style is empy, so css/scss (idk which is used) is not linked

i ended rolling back on the old in style per component import see :

I had to resort to the old-school style import method instead of the one outlined in the documentation. To this day, I still don’t know why it doesn’t work as described.


add .close() to view

This adds support for the latest flumedb, which requires each view
has a close() method for tidly closing all views.

ooo, zing, I don't have permission to publish this.
I've tagged it and stepped the version up. @christianbundy can you publish the master branch and perhaps add me to the flumedb org, or add me permission to publish this?

I'll publish this, but I think Dominic is the only one with permissions to add things to the flumedb org. I can ask him to increase my permissions.


Reload before view controller and after view controller

When I use selectedViewController(:), I thought it is a way to reload data.
I have ever used UIPageViewController, which has the same behavior.
And it would be better than using removeAllViewControllers().

Hi @nathanwhy! I'm not really sure I understand why this change is needed. Do you want to re-create the sibling view controllers every time you select a view controller, even if they already exist?

Some of them may disappear. e.g. [before, selected, after] -> [before, selected, nil].
after view controller may disappear.
Do you have any suggestions?

They shouldn't disappear unless the data source has changed and selecting items does not update the data source. You can reload the data source by calling pagingViewController.reloadData()

I don't need menu items, so I use EMPageViewController directly.  I saw the implementation about pagingViewController.reloadData(), it just remove all view controllers and load again. Considering that data source updates may be frequent, is there a better way?

I don't think there's any point of using Parchment if you're not showing the menu items. It's probably better to use UIPageViewController or EMPageViewController directly then. Another good option is Pageboy: https://github.com/uias/Pageboy

Thanks for your reply. I am using EMPageViewController directly.


Add a help me feature.

If you don't know the answer you can click the help-me button and maybe it gives you one or other digit and you have to provide the other one.

Hint is the word I was looking for. Press hint once and four buttons appear, including the right answer. Press again and one wrong answer disappears. Press again and another one disappears. Answers without hints score 5 points, say, and using hints lowers the score for a right answer.


How to use it

I am interested in your AMAR  model but I am confused how to use it . Even if you have written instruction in READEME.md. Can you give details about your examples? For example, how to create a proper '.txt' file and how to make the configuration file. If I have done it, how to use the torch to test it.

Hi @daicoolb,
first of all, thank you for your interest in our project. We really appreciate it!
The file that you need to create is a "txt" file whose content is the description of the item. It should not have a specific file format. For example, you can consider this description for the movie "Jumanji":

Jumanji is a 1995 American fantasy adventure film, directed by Joe Johnston. It is an adaptation of the 1981 children's book of the same name by Chris Van Allsburg. The film was written by Greg Taylor, Jonathan Hensleigh, and Jim Strain, and stars Robin Williams, Kirsten Dunst, David Alan Grier, Bonnie Hunt, Jonathan Hyde, and Bebe Neuwirth. The special effects were provided by Industrial Light & Magic for computer graphic elements and Amalgamated Dynamics for animatronics components.

Obviously, you can apply the natural language processing pipeline that you want to use on the original text and then use the processed text as input to the model. In our case, we have separated files for the original descriptions and the processed descriptions. We use the processed description as input to the model.
As you can see from the function read_items_data it reads all the text descriptions contained in a given path and create a proper data structure that is exploited in the training process. The training process can be started by using the train_amar_rnn.lua file. It requires a configuration file whose structure is specified in the readme.
If you require additional information please feel free to ask and we will answer to you as soon as possible.
Best regards,
Alessandro

@aleSuglia  ,Sorry for my late feedback ~~
I am poor in torch as well as Lua. But I am interested in your model . For my question, it seems to be clumsy which is how to use your code to test.  Maybe I should read some tutorial about Lua and torch first.

Don't worry about that.  You need to install Torch first. You can follow the official guide which precisely describes how to install it.
I close this issue. If you have any other problem with our code feel free to ask.


Remove Guides and AMs as admins from existing policies

Background

Project tracking issue: Expensify/Expensify#401707
Some AMs have elected to remain as admins on workspaces for specific customer accounts. These exceptions are listed in the following document: List of AM Exceptions. We need to ensure these AMs are not removed as admins on their workspaces.
A list of all existing Guide email accounts can be found here, and a list of all AM accounts can be found here.

Tasks

Review the List of AM Exceptions:

Verify the AM exceptions listed in the provided document.
Ensure these domains and AMs are excluded from the removal process.

Run the Script:

Use the script from Web-Expensify#42227 to remove Guides and AMs from policies.
Check that the script does not remove AMs listed in the exceptions document.

Ensure Continuation in #admins Room:

Verify that Guides and AMs remain included in the #admins rooms in NewDot after the script has been run

Non-engineering tasks

Test the change once live with Guides and AMs

Note for myself:
puneet
7 minutes ago
@tgolen
I think what we could do for your issue perhaps is just augment Ionatan's script with an optional flag of like addToAdminsRoom which would add the Guide/AM back to the admins room after removing them from the policy

Daily Update

I have created a draft Web-E PR to allow some users to optionally be re-added to the #admin room after the policy is unshared with them
I am currently trying to test this out locally to ensure it works first (I am running into an insufficient permissions issue, so I think I need to update Auth)
@davidcardoza I would like to get a CSV of accounts to run with this bulk tool. I believe you can use the lists you posted to create the CSV. It needs to have three columns:

email of the person to remove
policyID of the policy to remove them from
keepInAdminRoom (value can be either "y", "n", or blank)

Next Steps

@tgolen once I have it tested and ensure it's working locally, I'll open the PR for full reviews.
@iwiznia Give the draft PR a look to see if you spot any issues with what I've added
merge and deploy the PR
@davidcardoza construct the CSV file and deliver it to @tgolen
run the necessary removal script

ETA

PR deployed by Thursday, June 13
Script ran by Friday, June 14

@puneetlath I've assigned this to you since you are taking it over and figuring out plan of attack.

Ok, so here's what I think we need to do.
Step 1: Remove Guides/AMs from all policies via script

To do this, we will use the BulkUnsharePolicies.php script
We will pass it the BulkUnshare.csv
This has about 63,000 policies, so with a 1 second sleep will take ~18 hours
Removing them from the policy will also remove them from the #admins room

Step 2: Add Guides/AMs back to #admins rooms via BulkCQ

Now we will add the Guides/AMs back to the admins rooms using the BulkCQ.py script
We will pass it BulkCQ.csv
We will use the query below

INSERT INTO sharedReports (
    accountID,
    permissions,
    reportID )
SELECT {0}, 'read, write', JSON_EXTRACT(value, '$.chatReportIDAdmins') as reportID
FROM nameValuePairs
WHERE name = '{1}'
ON CONFLICT (accountID, reportID) DO UPDATE SET permissions = excluded.permissions;

Plan looks good. TIL about the excluded syntax.
For the bulkCQ you can run probably something like 100 at a time

👍 Gonna first try this with 100 items, and then do the rest

That worked fine, so we're going to do the rest of the AMs, which are about 12K
AMBulkCQ.csv.gpg.zip

Updated query to ignore conflicting lines
INSERT OR IGNORE INTO sharedReports (
    accountID,
    permissions,
    reportID )
SELECT {0}, 'read, write', JSON_EXTRACT(value, '$.chatReportIDAdmins') as reportID
FROM nameValuePairs
WHERE name = '{1}'

👍 to bulkCQ, we're going to use this file with 2600 entries
AMRedoBulkCQ.csv.gpg.zip

Ok, so far we have:

Ran the BulkUnsharePolicies.php script for all AM accounts
Ran the CQ above to add all AMs back to #admins rooms
Ran the BulkUnsharePolicies.php script for all Guides

Next we need to re-add the Guides to the #admins rooms. To do that we will run BulkCQ with this query in batches of 100.
INSERT OR IGNORE INTO sharedReports (
    accountID,
    permissions,
    reportID )
SELECT {0}, 'read, write', JSON_EXTRACT(value, '$.chatReportIDAdmins') as reportID
FROM nameValuePairs
WHERE name = '{1}'

It should take about 10 min to run.

👍 to bulk CQ, we'll run it with this file with 1s between batches GuidesBulkCQ.csv.gpg.zip

We ran the bulkCQ and got 509 200 OK responses :tada

Waiting for an update on the state of things here https://expensify.slack.com/archives/C0745PP38PR/p1718643604728389?thread_ts=1718402793.809139&cid=C0745PP38PR

This is done and it looks like it did the trick, so closing


TOPS-745 fix description causing NIG12

TOPS-747 fix description causing NIG12 business rule failure
The commodity code importer is not processing updates to commodity code description dates
There was an EU update to a commodity code description end date in TGB22197 that was not processed by the importer
Consequently, an NIG12 business rule is firing because a description validity date is now wrong
What
This PR creates a migration that edits the description date on a particular commodity code description (as the EU have done).
Tests have been written for this migration so recommend running them.
Ticket link:
https://uktrade.atlassian.net/jira/software/projects/TOPS/boards/221?selectedIssue=TOPS-745&text=745

Codecov Report
Base: 93.32% // Head: 93.02% // Decreases project coverage by -0.30% :warning:

Coverage data is based on head (202e2a1) compared to base (486f6dc).
Patch coverage: 28.12% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #743      +/-   ##
==========================================
- Coverage   93.32%   93.02%   -0.31%
==========================================
  Files         346      370      +24
  Lines       23264    23767     +503
  Branches     1805     1834      +29
==========================================
+ Hits        21712    22109     +397
- Misses       1216     1312      +96
- Partials      336      346      +10

Impacted Files
Coverage Δ

commodities/tests/test_migrations.py
28.12% <28.12%> (ø)

quotas/views.py
98.59% <0.00%> (-1.41%)
:arrow_down:

urls.py
60.00% <0.00%> (ø)

common/jinja2.py
84.44% <0.00%> (ø)

quotas/models.py
100.00% <0.00%> (ø)

settings/common.py
76.92% <0.00%> (ø)

common/tests/test_views.py
100.00% <0.00%> (ø)

reports/tests/test_report_models.py
83.13% <0.00%> (ø)

reports/urls.py
100.00% <0.00%> (ø)

reports/reports/base_chart.py
82.35% <0.00%> (ø)

... and 20 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[SIEMINT-78] DDS: PingFederate Integration

What does this PR do?

PR for a new integration PingFederate 1.0.0
Additional Notes

-- OOTB detection rules JSON would be shared separately with the required teams as a part of separate repository .
-- Since during the standard attribute remapping we are not preserving the source attributes as per suggested best practices, it would result in filters using these standard attributes populating the values of other integrations as well as per current datadog behaviour.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Changelog entries must be created for modifications to shipped code
[ ] Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
[ ] If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

I'm curious, what ddev command did you use to create this integration?
There's no python logic, but somehow the __about__.py is present. The changelog is managed manually, but there's also a changelog.d file. This mix is confusing.

I have used this command ddev create <INTEGRATION-NAME> --type logs to create this integration, and these files were created by default with the folder structure.

@rushikeshshinde-crest @akanshi-crest I have pulled in some recent changes from master. they contain a check that I'd like to run on this PR.
You can just pull the branch to get these changes incorporated on your development machines.

PR looks good from a logs perspective, added the label to deploy to staging. Will test and approve the PR by tomorrow

Hi @vishalshah-dd  As per the minor changes for remapper in TDD, we have changed  it in PR also. Can you please verify the changes once?

Please update the dashboard design to align with the Datadog design style that we discussed a couple weeks ago.


Displayed image width

Hi!

In this example I have set the image of size 295w, but it takes the container size of 750 , my question is if there is a way to get  the width of 295 not 750 as
$(document).on('lazybeforesizes', function(e){
    console.log(e.detail.width); // 750
   console.log(e.target.width); // 750
});

this returned
Regards!!!

Not 100% sure what you want to achieve here. The sizes attribute is all about how the image is displayed and not about what image candidates you have.
You can influence the sizes by using CSS or JS:

$(document).on('lazybeforesizes', function(e){
    e.detail.width = 295;
});

From the documentation:
Important: How sizes is calculated: The automatic sizes calculation uses the width of the image. If it is below 40 (can be configured through the minSize option), lazysizes traverses up the DOM tree until it finds a parent which is over 40 and uses this number. Often the following general CSS rule might help: img[data-sizes="auto"] { display: block; width: 100%; } (see also specifying image/iframe dimensions).
The data-sizes="auto" feature only makes sense if you use the data-srcset attribute with width descriptors which allows the most appropriate image can be selected (It does not make sense if you use the x descriptor or only src.). The width auto-calculated by lazysizes can be modified using the lazybeforesizes event. Alternatively, the parent fit plugin can be used for sizing images to fit a parent / container, and is the only solution when an image's height needs to be taken into account when fitting it to its container.


ECONNRESET Error When Trying to Post to Threads

I've been trying to use the threads-api library to post to my Threads account programmatically. However, I'm encountering an ECONNRESET error and I'm unsure how to proceed.
Here's the script I'm using:
import { ThreadsAPI } from 'threads-api';
const main = async () => {
const threadsAPI = new ThreadsAPI({
username: 'my_username', // My actual username is replaced here
password: 'my_password', // My actual password is replaced here
});
await threadsAPI.publish({
text: '🤖 Hello World',
});
};
main();
When I run this script, I get the following error:
AxiosError: read ECONNRESET
...
The error suggests that the other side of the TCP conversation (in this case, the Instagram server) is unexpectedly closing the connection. I'm not sure why this is happening. My internet connection seems to be stable, and I've verified that my username and password are correct.
Could you provide any guidance on what might be causing this issue and how I could resolve it? Any help would be greatly appreciated.

I figured this one on my own – i am from Russia and Instagram is blicking nay traffic from my country. So i used VPN instead and everything worked.

@a-burlakovv Thanks for the follow-up! 🙏 Too bad to hear Instagram blocking Russia 😭


seek.Enable(false) not transparent

How to cancel Seek without leaving it transparent?

Grateful for the attention. I'll try!


adjacent is not working if first argument requires more than one item to succeed

let x = short('x').argument::<f32>("X");
let y = short('y').argument::<f32>("Y");
let p = short('p').req_flag(());
construct!(p, x, y)

Without p this fails since adjacent detects left most item with magic...

rethink adjacency detection?
expose more items somehow?
document that adjacent block must start with a flag?
use "left off the first item" logic only when there are positional items and narrowing window otherwise

document that adjacent block must start with a flag?

Is this related to the bugs experienced with a subcommand with overlapping shorts?: https://github.com/pacak/bpaf/issues/325

Is this related to the bugs experienced with a subcommand with overlapping flags (short names)?:

I don't think so. Current logic to restrict scope to adjacent values is something like this:

run the parser showing it each unconsumed item individually, parser will fail, but it might start consuming items. Set this position as a starting point for the region
run the parser again, showing it items from the start of the region until the end of the span of adjacent unconsumed items
if consumed blocks are all adjacent to each other - that's the result, if not - narrow the span to adjacent block and goto 2.

This works if first item in the parser wants to consume let's say -p, but it doesn't work if it wants let's say -x 3 - -x part will be available, 3 - won't be and argument parser checks that both name and value are present. It works this way because some unrelated reasons.


ROX-25461: Expose feature flag service on sensor

Description

Let sensor expose its feature flags to allow synchronization from central configuration source on secured cluster components.
User-facing documentation

[ ] CHANGELOG is updated OR update is not needed
[ ] documentation PR is created and is linked above OR is not needed

Testing

[ ] inspected CI results

Automated testing

How I validated my change

Deployed local RHACS (central + sensor)
Forwarded sensor HTTP port locally
Query /v1/featureflags with curl on locally forwarded port and check there is a response.

Work deprioritized.


x/pkgsite: package removal request for github.com/jfrech/tbp

What is the path of the package that you would like to have removed?
github.com/jfrech/tbp
Are you the owner of this package?
Yes.
What is the reason that you could not retract this package instead?
I want this package's cache trace expunged. The repository has been deleted.

Done, and should take effect within the next few hours. Please comment if not.

@findleyr The package entry appears to have vanished. Thank you very much.


Added internal interests table

Added internal interests table
Moved internal interests into accelerator, updated translations and configs
Renamed migration

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#2359
#2358  👈
#2347 : 1 other dependent PR (#2348 )
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tommylau523 and the rest of your teammates on  Graphite

Merge activity

Sep 3, 8:41 PM EDT: @tommylau523 started a stack merge that includes this pull request via Graphite.


Exam mode: Add exam rescheduled event

Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] Important: I implemented the changes with a very good performance and prevented too many (unnecessary) and too complex database calls.
[x] I strictly followed the principle of data economy for all database calls.
[x] I strictly followed the server coding and design guidelines.
(https://docs.artemis.cit.tum.de/dev/guidelines/server/#rest-endpoint-best-practices-for-authorization) and checked the course groups for all new REST Calls (security).
[x] I documented the Java code using JavaDoc style.

Client

[x] Important: I implemented the changes with a very good performance, prevented too many (unnecessary) REST calls and made sure the UI is responsive, even with large data (e.g. using paging).
[x] I strictly followed the principle of data economy for all client-server REST calls.
[x] I strictly followed the client coding and design guidelines.
[x] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[x] I documented the TypeScript code using JSDoc style.
[ ] I added multiple screenshots/screencasts of my UI changes.
[x] I translated all newly inserted strings into English and German.

Motivation and Context

When the exam start and end dates are adjusted while the exam is already visible, but the duration remains unchanged (e.g., postponing the exam by 10 minutes), this currently triggers a working time update event in the client, which can be confusing since the working time is not affected. This pull request introduces a new live exam event type called ExamRescheduled to handle such cases more clearly.
Description

A new event called ExamRescheduled has been implemented on the server side. This event is triggered only when the exam is visible and before its start date, as postponing an exam that has already started would be irrelevant. The client side has also been updated to handle this new event type appropriately. The notification is shown only if the user is currently viewing the exam start page. If the user navigates to the exam page after the event has been sent, it is automatically acknowledged, as the exam page already reflects the updated exam information.
Steps for Testing

Prerequisites:

1 Instructor
1 Student
1 Exam

Log in to Artemis
Navigate to Exam Management as an instructor.
Make the exam visible and ensure that the exam start date is in the future.
In a different browser/incognito mode, navigate to the exam as a student.
As an instructor, postpone/bring forward the exam while ensuring that the working time is not affected.
Switch to the student view and verify that the ExamRescheduled event was received. Acknowledge the event.
As a student, navigate to another section of Artemis.
As an instructor, postpone/bring forward the exam again.
As a student, return to the exam page.
Verify that no pop-up notification was shown this time.
Start the exam as a student (you can adjust the start date as an instructor to a past date, it will trigger a working time update event).
As an instructor, postpone/bring forward the exam.
As a student, verify that no event was received.

Testserver States

[!NOTE]
These badges show the state of the test servers.
Green = Currently available, Red = Currently locked
Click on the badges to get to the test servers.

Review Progress

Performance Review

[ ] I (as a reviewer) confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance even for very large courses with more than 2000 students.
[ ] I (as a reviewer) confirm that the server changes (in particular related to database calls) are implemented with a very good performance even for very large courses with more than 2000 students.

Code Review

[ ] Code Review 1
[ ] Code Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Exam Mode Test

[ ] Test 1
[ ] Test 2

Performance Tests

[ ] Test 1
[ ] Test 2

-->
Screenshots

Summary by CodeRabbit

New Features

Introduced support for rescheduled exam events, allowing users to receive updates on new start and end dates.
Added a new event type for rescheduled exams in the event handling system.
Enhanced user interface to display messages regarding rescheduled exams.
Improved responsiveness of components to live updates for exam rescheduling.

Bug Fixes

Improved logic for notifying students about changes in exam scheduling and working times.

Documentation

Updated internationalization files to include translations and descriptions for rescheduled exam events.

Tests

Enhanced test coverage for exam participation logic, including scenarios for handling rescheduled exams.

Tested on TS2. Everytime I postpone the exam, a pop up notifications is displayed. whether in step 9 or step 13. Is it not a good behaviour to notify students when the exam is postponed or have I misunderstood the testing instructions 😅? Also should students also be notified when the start date of the exam is being pushed ealier? I tried setting the start time of the exam to be earlier as well as due time so that the working time remains the same, students don't see any notifications

Notifications should appear in the list when you click the button in the top-right corner (as shown in your picture). However, the modal should not open automatically if you navigate to the exam page after the exam has been rescheduled. The pop-up should only appear automatically if the reschedule event occurs while you are already on the exam page.


replace custom ThreadLocalRandom with standard one

Seems to be leftover from Java < 7

Java 7 only just recently got dropped, so good timing on your part!
thanks for the fix.

@busbey standard ThreadLocalRandom was introduced in Java7, so timing was actually late for several years.


Support transaction

Dear Sir,
From the below result of searching 'transaction' in source code, sounds that blueprints-arangodb-graph does not support transaction. Do you have plan to support transaction?

ArangoDBGraph.java  -  \blueprints-arangodb-graph-master\Src\Main\Java\Com\Arangodb\blueprints
FEATURES.supportsTransactions = false;
FEATURES.supportsThreadedTransactions = false;
FEATURES.supportsThreadIsolatedTransactions = false;
ArangoDBBatchGraph.java  -  \blueprints-arangodb-graph-master\Src\Main\Java\Com\Arangodb\blueprints\Batch\

data into a ArangoDB graph. This is a single threaded, non-transactional bulk
FEATURES.supportsTransactions = false;
FEATURES.supportsThreadedTransactions = false;

Hi Wilfried，
Any plan for this feature implementation?

Hi liuguoguo,
We're currently starting out on Spring Data; after that we may head over to blueprints again.

Hi Wilfried,
Just want to know your current work on Spring Data, it means if "spring data for arangodb" or "spring data for any graph db which supports blueprint/gremlin". We hope more improvements in blueprints-arangodb implementation, for example, transaction support, more data types support in property, and so on.
Thanks,
Zhiqiang

Hi @liuguoguo,
(if) its going to be an arangodb backend for spring data, so you will be able to access arangodb via spring data.

Sharing my thoughts because I've looked into this for a customer: the way ArangoDb transactions work, plus the way the driver is written, makes this a tricky task.
ArangoDb doesn't support primitives like begin() or commit(). It only allows you to execute a bunch of javascript functions in one call.
So, inside the tinkerpop driver, you'd have to set some sort of flag to true when calling begin(). That flag would be reset by commit() or rollback(). The flag is checked inside calls like addVertex, updateVertex, addEdge and so on: if the flag is set to true, then that call needs to be transformed to the equivalent javascript call, and put into a list. If rollback() is called that list is discarded, nothing happens. If commit() is called, then you put that list of javascript calls inside a call for a transaction and send it via the java driver.
You'd have to be very careful in the way you document this, as it won't work the way people doing SQL would expect. It would probably only support TransactionalGraph, not ThreadedTransactionalGraph

Currently all operations are translated into AQL and executed server side. As a result all operations are atomic and transactions are not supported. Although some bridge to JavaScript is plausible, making this work is very difficult. Any operation will have to query both the db directly and the "temporal store" of uncommitted information. Another option like loading the graph into memory fails for large graphs.
The lack of transactions is a limitation of how the Java Driver works and overcoming this will result in half baked solutions.


reproduce the results

Hi, since the panda-gym cannot set the random seed as gym, where you can use env.set(seed) to reproduce the results.
When I use env.reset(seed=10), if the random seed is the same, I get the same return value, such as desired_goal. But if I don't set the seed, I can't guarantee the consistency of the state of the subsequent env.resets. Is there a way to guarantee the consistency of the state of each subsequent reset just like gym?

Hey, thanks for the question. It seems like a bug, I'll fix it.

I've only managed to solved the issue with some environments. I can't solve it for the others.
The discussion is also ongoing here: https://github.com/Farama-Foundation/Gymnasium/issues/1111

[x] PandaReach-v3
[x] PandaReachJoints-v3
[x] PandaReachDense-v3
[x] PandaReachJointsDense-v3
[x] PandaSlide-v3
[x] PandaSlideJoints-v3
[x] PandaSlideDense-v3
[x] PandaSlideJointsDense-v3
[ ] PandaPush-v3
[ ] PandaPushJoints-v3
[ ] PandaPushDense-v3
[ ] PandaPushJointsDense-v3
[ ] PandaPickAndPlace-v3
[ ] PandaPickAndPlaceJoints-v3
[ ] PandaPickAndPlaceDense-v3
[ ] PandaPickAndPlaceJointsDense-v3
[ ] PandaStack-v3
[ ] PandaStackJoints-v3
[ ] PandaStackDense-v3
[ ] PandaStackJointsDense-v3
[ ] PandaFlip-v3
[ ] PandaFlipJoints-v3
[ ] PandaFlipDense-v3
[ ] PandaFlipJointsDense-v3

To reproduce
import panda_gym
import gymnasium as gym
from gymnasium.utils.env_checker import data_equivalence

env = gym.make("PandaPickAndPlace-v3")

action_0 = env.action_space.sample()
action_1 = env.action_space.sample()

_, _ = env.reset(seed=10)
obs_00, _, _, _, _ = env.step(action_0)
_, _ = env.reset()
obs_01, _, _, _, _ = env.step(action_1)

_, _ = env.reset(seed=10)
obs_10, _, _, _, _ = env.step(action_0)
_, _ = env.reset()
obs_11, _, _, _, _ = env.step(action_1)

assert data_equivalence(obs_00, obs_10)
assert data_equivalence(obs_01, obs_11)


[HOLD App Navigation Reboot] React navigation StackNavigator causes screen content to load slowly

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

(long lasting LoadingIndicators on many screens)
Action Performed:

Open app
Go to "Search"/"New Chat"/"New Group" screen (any Screen that renders an OptionsSelector)
Wait for the loading spinner to end

Expected Result:
After the screen transition is done, the screen content should render relatively fast.
Actual Result:
Instead, even after the screen transition has ended a LoadingIndicator keeps spinning for a few (milli-)seconds.
That's because the OptionSelector component is waiting for the transitionEnd event to be emitted by react-navigation. (here) The transition will end at earliest though, once the screen content is rendered completely.
Workaround:
This is not blocking any flow.
Platforms:

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

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

Version Number: 1.2.49-0
Reproducible in staging?: yes
Reproducible in production?: yes
Email or phone of affected tester (no customers<EMAIL_ADDRESS>| iPhone 14 Pro (Simulator)
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by: @marcaaron & @chrispader
Slack conversation: https://expensify.slack.com/archives/C035J5C9FAP/p1669856870089949
View all open jobs on GitHub

@marcaaron could you add that issue here: https://github.com/Expensify/App/issues/11768

Hi @marcaaron did you want a job created and @chrispader hired?

No the issue is just on hold for now thanks @jliexpensify

@marcaaron if this is on HOLD, could this be updated to a Weekly or Monthly?

Yes 👍

Still on hold

On hold

Not overdue, on HOLD

Not overdue, on HOLD

Still on hold

@chrispader I think this one can be closed now, please correct me if I am wrong.


#1168: Incorrect TLS cert instructions for Intermediate CA

Proposed changes
Fixed incorrect TLS cert configuration instructions for Intermediate CA on UCP versions 2.0 and 2.1
Related issues (optional)
Fixes #1168

@clocklear thanks for contributing. Did you get a chance to test this on a real deployment, or you're just following what #1168 tells you to fix?
I'm asking because I have a deployment where

CA cert- just the CA public cert
server cert- the public cert for the domain + the public cert for the intermediary (in this order)

Making the CA cert be the CA public cert + intermediary (in this order) also works, but seems redundant.
Were you able to run into a scenario where you had to add the intermediary to the CA cert to make this work?

No, I'm currently unable to test this, as I don't have any apps inside
docker cloud. I followed what #1268 suggested.
Based on what you are suggesting, it seems as though both solutions would
work, but are a bit redundant.
On Wed, Apr 19, 2017 at 2:04 PM Joao Fernandes<EMAIL_ADDRESS>wrote:

@clocklear https://github.com/clocklear thanks for contributing. Did
you get a chance to test this on a real deployment, or you're just
following what #1168
https://github.com/docker/docker.github.io/issues/1168 tells you to fix?
I'm asking because I have a deployment where

CA cert- just the CA public cert
server cert- the public cert for the domain + the public cert for
the intermediary (in this order)

Making the CA cert be the CA public cert + intermediary (in this order)
also works, but seems redundant.
Were you able to run into a scenario where you had to add the intermediary
to the CA cert to make this work?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/docker/docker.github.io/pull/2881#issuecomment-295393925,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAl-fSWl_3wDJe_qjv2PF9d9-axCLpdmks5rxlrPgaJpZM4NA--W
.

I'd like to have @programmerq opinion about this. Interestingly enough he will be giving a talk today about troubleshooting techniques, and I think he is going to talk how to set up your certificates in DDC.

Absolutely. Not at my computer at the moment, but will address ASAP.
On Fri, Apr 21, 2017 at 8:02 PM Joao Fernandes<EMAIL_ADDRESS>wrote:

@joaofnfernandes requested changes on this pull request.
In datacenter/ucp/2.0/guides/configuration/index.md
https://github.com/docker/docker.github.io/pull/2881#discussion_r112795363
:

@@ -39,11 +39,13 @@ and click Certificates.

Upload your certificates and keys:
-* A ca.pem file with the root CA public certificate.
-* A cert.pem file with the TLS certificate and any intermediate CA public
+* A ca.pem file with the root CA public certificate followed by any intermediate CA public
After several tests and a conversation with @programmerq
https://github.com/programmerq I can confirm that this is what you
should do:

CA - the CA public certificate
Server cert - The public certificate for your domain, and any
intermediate public certificates, in this order.

Can you revert your change to line 42 and update 44 make make this clear?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/docker/docker.github.io/pull/2881#pullrequestreview-34143052,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAl-fVWjxiE82ty9gGLs3v92-TxevOscks5ryUN9gaJpZM4NA--W
.

No need to rush! We're all recovering for DockerCon 😂

Whoops.  Sorry 'bout that.  Forgot it was in two places.


Systemd service file has Documentation key in wrong section

Describe the bug
The systemd service file has the Documentation key in the wrong section. The Documentation key is currently in the Service section, but should be moved to the Unit section. See systemd documentation here. Current line causing issues here.
Example warning generated in logs:
systemd[1]: /usr/lib/systemd/system/dynatrace-otel-collector.service:13: Unknown key name 'Documentation' in section 'Service', ignoring.

To Reproduce
Steps to reproduce the behavior:

Install Dynatrace OTel Collector
Start/restart the service via systemd
error is shown in /var/log/messages or via journalctl

Expected behavior
Service shouldn't generate warning
Screenshots
N/A
Desktop (please complete the following information):

OS: AlmaLinux 9.4
Systemd: 252-32.el9_4.6

Additional context
Add any other context about the problem here.

thanks for posting this! i'll fix it


How to use sign in in local?

I changed the supabase key to my own and added the profiles to the supabase table, but an error popped up and it was not resolved.
I want to implement login and model upload in the local environment, is there any information I can refer to?

Hey
What was the error?


[AppBar] Rendering Children of AppBar before the right icon

Currently no tests for the children other than that they render, but happy to add tests for render order if you guys see this as a change that you want to merge?
Description: see #6866
Happy also to rebase so that the commit messages are according to the projects standards (AFAIK: [AppBar] changed order of children output

This change has already been proposed here #3004 but would break #3005.
This is introducing a breaking change, as you can see in the ROADMAP we focus on bug fixes for the master branch and avoid breaking changes as much as possible. I think that we should close that PR. Thanks for the suggesting though!

No worries! Thanks for the feedback :)


Doc change tasks

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

I find our you just correct the NOTE section in 3fd5e72. but please remove the HTML syntax I metion above https://github.com/apache/dolphinscheduler/pull/10639#discussion_r907974483


Describe the bug

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Alternatively link your repo with a sample project that can be run.
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Sorry, it was by mistake. I was trying to open an issue


v12 support

I know this is a bit early to make this request, but considering the new Scene Regions in v12, I think that much of the work is now done by the system, merely needing a few tweaks to integrate into the existing UI and I wanted to put the bug in your head to consider using them rather than updating the  region code used in the module. As close as the built-in capability comes to a full implementation, I half expected the core team to throw manual FOW in as a surprise feature late in development, but as we rapidly approach release, it now seems unlikely.
Anyway, thought I'd suggest it. I'm not familiar with any of the code, so it could very well be that implementation with scene regions is infeasible for reasons I'm unaware of, in which case this is just an embarrassingly early reminder to update for v12.
Thanks for maintaining such an essential tool!

I'm holding off on upgrading until simplefog supports v12. Hopefully it can be done!


Add client side metadata cache

in fuse openFile, a getStatus RPC is issued, but openFile will also issue a getStatus RPC internally, this redundant RCP is reduced in this PR.
when getting status of a file, or listing status of a directory, cache the status results so that future getStatus and listStatus calls do not need to issue RPCs to master.

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/Alluxio-Pull-Request-Builder/7026/
Test FAILed.

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/Alluxio-Pull-Request-Builder/7030/
Test FAILed.

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/Alluxio-Pull-Request-Builder/7035/
Test FAILed.

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/Alluxio-Pull-Request-Builder/7036/
Test FAILed.

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/Alluxio-Pull-Request-Builder/7042/
Test FAILed.

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/Alluxio-Pull-Request-Builder/7047/
Test FAILed.

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/Alluxio-Pull-Request-Builder/7048/
Test PASSed.

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/Alluxio-Pull-Request-Builder/7055/
Test PASSed.

@apc999 PTAL

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/Alluxio-Pull-Request-Builder/7077/
Test PASSed.

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/Alluxio-Pull-Request-Builder/7078/
Test FAILed.

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/Alluxio-Pull-Request-Builder/7079/
Test PASSed.

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/Alluxio-Pull-Request-Builder/7085/
Test PASSed.

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/Alluxio-Pull-Request-Builder/7087/
Test FAILed.

Refactored out CachingFileSystem, @apc999 @calvinjia PTAL

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/Alluxio-Pull-Request-Builder/7110/
Test PASSed.

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/Alluxio-Pull-Request-Builder/7111/
Test FAILed.

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/Alluxio-Pull-Request-Builder/7115/
Test FAILed.

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/Alluxio-Pull-Request-Builder/7113/
Test FAILed.

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/Alluxio-Pull-Request-Builder/7114/
Test PASSed.

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/Alluxio-Pull-Request-Builder/7109/
Test FAILed.

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/Alluxio-Pull-Request-Builder/7116/
Test PASSed.

@calvinjia PTAL

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/Alluxio-Pull-Request-Builder/7137/
Test FAILed.

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/Alluxio-Pull-Request-Builder/7139/
Test FAILed.

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/Alluxio-Pull-Request-Builder/7141/
Test FAILed.

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/Alluxio-Pull-Request-Builder/7144/
Test FAILed.

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/Alluxio-Pull-Request-Builder/7152/
Test PASSed.

@calvinjia @apc999 PTAL

@calvinjia sure, they can all be implemented in CachingFileSystem, but then the logic in getStatus(AlluxioURI, OpenFilePOptions) and openFileInStream will be duplicated in both implementations. These functions can be viewed as utility functions exposed by BaseFileSystem for extended classes, what's your concern about exposing them?

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/Alluxio-Pull-Request-Builder/7182/
Test PASSed.

@calvinjia refactored BaseFileSystem#openFile so that CachingFileSystem does not need to override openFile, much cleaner.

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/Alluxio-Pull-Request-Builder/7183/
Test PASSed.

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/Alluxio-Pull-Request-Builder/7184/
Test PASSed.

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/Alluxio-Pull-Request-Builder/7185/
Test FAILed.

jenkins, test this please

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/Alluxio-Pull-Request-Builder/7188/
Test PASSed.

alluxio-bot, merge this please.

alluxio-bot, cherry-pick this to branch-2.1 please.


Add support for www.glowfic.com bridges

So sorry, hadn't noticed the CI space/tab standard. Ought to be fixed now.

And the last change apparently broke the spaces/tabs rule, so re-CIed. Tests pass now.

Merged. Thanks!


MINOR: Fix flaky tests in DefaultStateUpdaterTest

Found a few flaky tests while reviewing another PR. The root cause seems to be with changing the return behavior of when in mockito. Fixed those without using reset and also bumped a couple debug log lines to info since they could be very helpful in debugging.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

ping @cadonna @lucasbru for a quick review.
Before this PR, local runs fail about 50% of the time; after this PR it succeeds with about 10 runs.

Thanks for investigating it! This test did fail quite frequently recently. But I'm not completely understanding this:

The root cause seems to be with changing the return behavior of when in mockito. Fixed those without using reset

What does the "return bahavior change" in mockito?

I looked into this flakyness before and I don't quite understand the behavior of mockito here. Could you explain this, or did you stumble upon the solution, and maybe add a comment somewhere explaining the solution?
Either way, results speak for themselves so approving

I can see the test result is better, but there are still failed tests in DefaultStateUpdaterTest
    Build / JDK 17 and Scala 2.13 / kafka.server.DynamicBrokerReconfigurationTest.testTrustStoreAlter(String).quorum=kraft
    Build / JDK 8 and Scala 2.12 / org.apache.kafka.streams.processor.internals.DefaultStateUpdaterTest.shouldPauseStandbyTaskAndNotTransitToUpdateStandbyAgain()
    Build / JDK 8 and Scala 2.12 / org.apache.kafka.streams.processor.internals.DefaultStateUpdaterTest.shouldPauseStandbyTaskAndNotTransitToUpdateStandbyAgain()

Honestly I do not quite understanding how mockito is messing here.. take shouldPauseActiveTaskAndTransitToUpdateStandby for example, after task1 (named topologyA) and task2 (named topologyB) are updating, then there's the line when(topologyMetadata.isPaused("A")).thenReturn(true);. In succeeding cases, task1 is indeed paused, and everything works just fine; however in those flaky failing cases, task2 is paused instead, indicating topologyMetadata.isPaused("B") returns true but topologyMetadata.isPaused("A") returns false... I honestly have no idea why that could happen, but after changing that to when(topologyMetadata.isPaused(task1.id().topologyName())).thenReturn(false).thenReturn(true); (i.e. let it to return false on the first call when adding task, and then return true on the second call to pause) I did not see that test failing any more. I checked mockito's docs but still do not see how when could mess up on the return values, so I kinda fixed it but do not know exactly why I fixed it..

I checked the latest failure on shouldPauseStandbyTaskAndNotTransit.. and found out that it's because of the test condition itself is timing dependent. What it wants to verify is the following sequence:
task1 added, task2 added, and then task1 paused. In this case the transitToUpdateStandby should be called once.
But it is possible that the following sequence can also happen:
task1 added, task1 paused, task2 added (because the updater thread can interleave its runOnce with the addTask call), in which case the transitToUpdateStandby could be called twice.
I've relaxed that condition and I think this test should not fail anymore.

Merged to trunk.


BUG: This graph was stopped working.

Recent I found this graph was not working anymore even the link of the website that was using to show the graph is no longer accessible.

same

same

@Raziur306 , @SoulNaturalist , @cbhan0102 , @feduke-nukem @tranthaituananh , try this host instead
https://github-readme-activity-graph.vercel.app/
[![Ashutosh's github activity graph](https://github-readme-activity-graph.vercel.app/graph?username=Ashutosh00710)](https://github.com/ashutosh00710/github-readme-activity-graph)

The underlying issue is

@Ashutosh00710 Thank you for your solution, it has resolved the issue.

@Ashutosh00710 Thank you, this fixed my problem:

@Ashutosh00710  thank you so much

Thanks a lot, I have made some changes to the code and now it is working
for me.
On Mon, May 29, 2023 at 19:53 Nguyễn Hải Dương @.***>
wrote:

@Ashutosh00710 https://github.com/Ashutosh00710 thank you so much
—
Reply to this email directly, view it on GitHub
https://github.com/Ashutosh00710/github-readme-activity-graph/issues/197#issuecomment-1567205837,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE4LJNOQD5WFJICFRQVQTIDXISWMJANCNFSM6AAAAAAYMFTIYI
.
You are receiving this because you are subscribed to this thread.Message
ID: <Ashutosh00710/github-readme-activity-graph/issues/197/1567205837@
github.com>

@Raziur306,@SoulNaturalist,@cbhan0102,@feduke-nukem @tranthaituananh,@Tgentil probeer in plaats daarvan deze host
https://github-readme-activity-graph.vercel.app/
[![Ashutosh's github activity graph](https://github-readme-activity-graph.vercel.app/graph?username=Ashutosh00710)](https://github.com/ashutosh00710/github-readme-activity-graph)

Het onderliggende probleem is

@Raziur306,@SoulNaturalist,@cbhan0102,@feduke-nukem @tranthaituananh,@Tgentil probeer in plaats daarvan deze host
https://github-readme-activity-graph.vercel.app/
[![Ashutosh's github activity graph](https://github-readme-activity-graph.vercel.app/graph?username=Ashutosh00710)](https://github.com/ashutosh00710/github-readme-activity-graph)

Het onderliggende probleem is

@Jommeke22 ,  thanks for pointing out the ongoing issues with the graph. The issue you're referencing has indeed been closed, this issue can be resolved by deploying a personal instance of the activity graph. If you set up your own instance following these instructions, you should find that it resolves the problem for you as well. Here’s an example with my own instance:

https://github.com/Ashutosh00710/github-readme-activity-graph.git

https://github.com/Ashutosh00710/github-readme-activity-graph.git


Core: Add multiple attribute to email method.

Using @Sven-Mad's method. Fixes #1722.

I think this PR still has merit.

Hi @RobJohnston,
Sorry for taking so long to review this PR. I only have one concern about this implementation. That no_multiple argument will always be true whether you are setting the rule using inline HTML or in JS.
What I would like to suggest is, we add support for passing an object to email method in JS like the following:
$("form").validate({
  rules: {
    field: {
      email: {
        multiple: true | false
      }
    }
  }
});

The no_multiple parameter will need to be changed to param (or another convenient name).

Hi @Arkni .  It's now my time to apologize to you for the delay!  I've been away from front-end development for some time.  I didn't quite understand your comment when I first read it and not sure if I do now.
From the W3, the multiple attribute is what they call a boolean attribute, but that doesn't mean that it can be set to "true" or "false".  It must be multiple="multiple" or simply just 'multiple'.  Does that change your suggestion where you wrote:
multiple: true | false


Cache and restore forwarder state on resume

This allows contiguous sequence number and VP8 picture ids.
TODO: Need to figure out if there is state saving needed with DD extension and what should be saved.

This is fantastic! Would AV1 use the VP8 path or RTP? Will we need to handle it differently there too?

I do not know enough about the dependency descriptor extension. I need to understand what the implications there would be. I will file some issues to keep track.

@boks1971  The resumed downtrack's dependency descriptor extension will be refresh by the key frame, so it don't need extra process.

@boks1971 The resumed downtrack's dependency descriptor extension will be refresh by the key frame, so it don't need extra process.

Awesome, thank you for looking at this and confirming @cnderrauber