Build ingestion image with graviton ARM 64 base

Description
This PR includes the following:

Switches the ingestion image to be built with ARM 64 arch

Fixes #CDD-1818

Type of change
Please select the options that are relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Tech debt item (this is focused solely on addressing any relevant technical debt)

Checklist:

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests at the right levels to prove my change is effective
[ ] I have added screenshots or screen grabs where appropriate
[ ] I have added docstrings in the correct style (google)

For completeness adding a link to related infra PR:
https://github.com/UKHSA-Internal/data-dashboard-infra/pull/556


apex needed

I have already used the following command for inference
"python scripts/inference.py configs/opensora-v1-2/inference/sample.py
--num-frames 4s --resolution 720p
--layernorm-kernel False --flash-attn False
--prompt "a beautiful waterfall"",
but I am still encountering an error requiring the installation of apex.
"Traceback (most recent call last):
File "/./projects/Open-Sora/scripts/inference.py", line 304, in
main()
File "//projects/Open-Sora/scripts/inference.py", line 289, in main
save_path = save_sample(
File "//myconda/envs/opensora/lib/python3.9/site-packages/opensora/datasets/utils.py", line 178, in save_sample
write_video(save_path, x, fps=fps, video_codec="h264")
File "//myconda/envs/opensora/lib/python3.9/site-packages/torchvision/io/video.py", line 132, in write_video
frame.pict_type = "NONE"
File "av/video/frame.pyx", line 193, in av.video.frame.VideoFrame.pict_type.set
TypeError: an integer is required
/myconda/envs/opensora/lib/python3.9/site-packages/colossalai/pipeline/schedule/_utils.py:19: UserWarning: torch.utils._pytree.register_pytree
node is deprecated. Please use torch.utils._pytree.register_pytree_node instead.
_register_pytree_node(OrderedDict, _odict_flatten, _odict_unflatten)
/myconda/envs/opensora/lib/python3.9/site-packages/torch/utils/_pytree.py:254: UserWarning: <class 'collections.OrderedDict'> is already registered as pytree node. Overwriting the previous registration.
warnings.warn(
/myconda/envs/opensora/lib/python3.9/site-packages/colossalai/shardformer/layer/normalization.py:45: UserWarning: Please install apex from source (https://github.com/NVIDIA/apex) to use the fused layernorm kernel
warnings.warn("Please install apex from source (https://github.com/NVIDIA/apex) to use the fused layernorm kernel")"

Hello, how to solve the issue of "TypeError: an integer is required"?

I encountered the same question when using this inference.

I encountered the same question when using this inference. I mean "TypeError: an integer is required"

Hello, how to solve the issue of "TypeError: an integer is required"?

The TypeError was solved when I changed the Line132 "frame.pict_type = "NONE"" into "frame.pict_type = 1" in the function named "write_video". The function is in "/opt/conda/envs/pytorch/lib/python3.10/site-packages/torchvision/io/video.py".

Hello, how to solve the issue of "TypeError: an integer is required"?

The TypeError was solved when I changed the Line132 "frame.pict_type = "NONE"" into "frame.pict_type = 1" in the function named "write_video". The function is in "/opt/conda/envs/pytorch/lib/python3.10/site-packages/torchvision/io/video.py".
Hope this helps~


NFC engagement: Remove DeviceRetrievalMethods map from enagement NDEF message

DeviceEngagement should not contain DeviceRetrievalMethods map at all for NFC.
Zero size map was removed
Test: Manually (used other vendor mDL holder which detects the issue)
Fixes #207

[x] Tests pass

@davidz25 , could you please review by fix update according to your comment?

Gentle ping if you could update the PR with the two simple changes requested - thanks!

Gentle ping if you could update the PR with the two simple changes requested - thanks!

Sorry for the delay.
Also please note that this data is already logged in the constructor now and these 2 lines create duplication.
Anyway, I make as you asked.


(PUP-11897) Handle EOF in order to avoid busy-loop

When EOF is reached, calls to IO.select() will return immediately, resulting in puppet executing the loop without any internal blocking. This causes up to 100% usage of a CPU core until the child finishes.
If the child does its own stdout/stderr redirection before a long operation, for example, this can result in much wasted CPU usage.
There is no need to execute the body of the loop after EOF is reached, so switch the Process.waitpid2 flags to block.

Can one of the admins verify this patch?

Thank you for your contribution @bugfood! We've included your commit in this PR https://github.com/puppetlabs/puppet/pull/9106 so I'm going to close this.

Thank you.


DataStores?

I looked through the test images and didn't see any examples of datastores.  Is it possible?

sorry, I've just found the discussion page... I'll go look there


Add AzFunctions' Scale Controller support for managed identity

Today, the MSSQL backend requires runtime-driven scaling to autoscale in Azure Functions when managed identity is used to connect to the DB.
This ticket tracks providing out of the box support for autoscaling with MSSQL in the Scale Controller.

When completing this work, we should update these docs: https://learn.microsoft.com/en-us/Azure/azure-functions/durable/durable-functions-storage-providers#comparing-storage-providers


Fix editor error handling

closes https://github.com/TryGhost/Ghost/issues/7099

update editor base controller's showErrorAlert method to handle AdapterError or AjaxError objects
update PSM's error handling so that we don't pass an error object wrapped in an array to showAPIError (this was causing the default error message to always be displayed and the global version mismatch/maintenance mode handling to be skipped if an error was encountered when changing a value in the PSM triggered a save)
add missing title attribute to Mirage's Post factory
remove some leftover console.logs

TODO:

[x] add test for editor API error handling

⚡ready for review

LGTM 👍


Reserved icon

Icon Request

Icon name: Reserved
Tags of this icon: Reserve, reserved
Use case: Food and restaurants
Screenshots of similar icons: https://www.flaticon.com/free-icon/food-and-restaurant_1717051#term=reserve&page=1&position=13

This is the reserved icon with filled style.
This is the reserved icon with outlined style.


APDS-9930 proximity issue

Hello
I found this implementation for APDS-9930, I have that module and used it with arduino, but your proximity example always
shows
APDS-9930 - ProximitySensor
APDS-9930 initialization complete
Proximity sensor is now running
0: 0x5
1: 0xFF
2: 0xFF
3: 0xFF
4: 0xFF
5: 0xFF
6: 0x0
7: 0x0
8: 0x0
9: 0x0
A: 0x32
B: 0x0
C: 0x22
D: 0x0
E: 0x8
F: 0x2E
12: 0x39
13: 0x22
14: 0x0
15: 0x0
16: 0x0
17: 0x0
18: 0xE2
19: 0x1
1E: 0x0
Proximity: 462
Proximity: 480
Proximity: 415
Proximity: 459
Proximity: 455
Proximity: 425
Proximity: 406
Proximity: 406
Proximity: 380
Proximity: 409
Proximity: 422
Proximity: 485
Proximity: 447
Proximity: 427
Proximity: 430
values no matter what distance I use. It is Always around 400.
Ambient example look ok:
Ambient: 255.09  Ch0: 239  Ch1: 99
Ambient: 220.65  Ch0: 213  Ch1: 89
Ambient: 169.15  Ch0: 142  Ch1: 59
Ambient: 30.42  Ch0: 29  Ch1: 12
Ambient: 25.75  Ch0: 24  Ch1: 10
Ambient: 20.44  Ch0: 25  Ch1: 10
Ambient: 19.80  Ch0: 21  Ch1: 9
Ambient: 24.46  Ch0: 22  Ch1: 9
Ambient: 19.80  Ch0: 21  Ch1: 9
Ambient: 243.18  Ch0: 229  Ch1: 95
Ambient: 237.23  Ch0: 224  Ch1: 93
Ambient: 231.91  Ch0: 221  Ch1: 92
Do you think I have bad board? Thanks for your work in this field by the way

Duplicate of #7. No your board is not broken, it's crappy just as everybody else's.
Try messing with gain values, may help but no guarantees.


Fix update user

nothing happens on save when you update an existing user

was connection issue. no update needed.


Remove the Upper casing of headers for the listtable output type  and add padding

What this PR does / why we need it

Remove the Upper casing for listtable header and add padding

Which issue(s) this PR fixes

Fixes #
Describe testing done for PR

Release note

Remove the Upper casing of headers for the `listtable` output type  and add padding

Additional information
Special notes for your reviewer

I totally agree that this change makes the output easier to read. I'm wondering when listtable is used? Because if it is used by scripts, then this change is not backwards-compatible.
For example, if my script does tz plugin group search -o listtable | grep GROUP then my grep would fail after this change because it is case-sensitive.
Is it worth the risk?

It is used by some commands for user-readable output. For automation we should be using yaml or json output so I think we should be fine. (I saw the package plugin using that for user-readable output)


Do not display 'Manage Soft Credits' page when user does not have required permissions

Critical Changes
Changes

To access the Manage Soft Credits page, users must have Create, Edit, and Delete permissions on the Partial Soft Credit object.

Issues Closed
New Metadata
Deleted Metadata

@force2b - Ok, thanks. I guess I just assumed it could by any of those perms, but it must be all three?

@rponti-sforg - this is ready for review, just a reminder that it was extra work I picked up so please put it lower on your priority list. Thank you!


Fix for wrong {version.org.drools} instead of {project.version} in fe…

…atures.xml, to allow one-off patches

Merged into both 6.2.x and master. Thanks @lazarotti!


Jagged text on Laptop screen

First, Thank you for this. Great job!
Display texts are jagged or distorted if change the display to scaled (More Space)
Is there a way to improve it?
Looks good on external monitor though. I'm using 1080p external monitor

First, Thank you for this. Great job!
Display texts are jagged or distorted if change the display to scaled (More Space)
Is there a way to improve it?
Looks good on external monitor though. I'm using 1080p external monitor

Hi!
For a less clear font on the laptop screen need to use HIDPI option. For me it's 1366x768.
With this resolution a font is as smooth as possible on monitor laptop.

I hope this helps you:)

Looks like the developer of HIDPI released a new version

when I input 1366x768 to the Manual input resolution option the results for is only 1600x900 max resolution it is not the same as shown on your screenshot.

Once HIDPI is installed. can we reconfigure this on clover instead?

Once HIDPI is installed. can we reconfigure this on clover instead?

You do not need to write anything in clover. Now HIDPI works.


[FEAT]: README.md: Add chapter: What is FREISA

📑 Description

For the time being, just add the presentation at DataBeers Torino Ep. XVII.
As soon as we have more public videos we might add them here.

✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[x] I have updated the documentation as required
[x] All the tests have passed

ℹ Additional Information

@pitdagosti please have a look and merge if you like it.

For now it's ok, since the two previous images were very similar.  As soon as OPENCV gives us the badges, we will change them 💪


Treat error response payloads from Slack as errors

As described in the "More error types" section below, Slack API can return errors with a 200 response code:
https://slack.dev/node-slack-sdk/web-api#handle-errors
This change adds parsing of API response to extract error messages.
Signed-off-by: Anton Tolchanov<EMAIL_ADDRESS>

@simonpasquier are you the best person to review this?

Just rebased to make this merge-able.
@gotjosh, @simonpasquier – any thoughts on this?

is there any way that I can reproduce the issue locally?

Sure:
$ curl -w "\ncode: %{http_code}\n" -X POST 'https://slack.com/api/chat.postMessage'
{"ok":false,"error":"not_authed"}
code: 200

@gotjosh the grafana/alerting code is AGPL, right?
I'm not a lawyer but how are we sure that it won't create an issue?

This situation happened a couple of months ago and I consulted internally what were the steps necessary to allow that.
The result is https://github.com/prometheus/alertmanager/issues/2845#issuecomment-1303341819 as an example.
I'll use similar wording as I approve the PR.

This is something that the Grafana equivalent does much better, can we instead borrow the idea from what we do there?

Sure! Given alertmanager's use of common Retrier and ErrorWithReason it's not particularly straightforward here to re-use parts of that code in a meaningful way, but I have extended this PR to handle plaintext (non-JSON) responses as well. I have not added parsing of string error ids (like channel_is_archived) because they seem to be expressive enough.
Also expanded the test to cover:

1xx HTTP responses;
Plaintext 2xx successful and error responses;
JSON parsing errors;
Classification of errors as retryable;
Text of error messages.

I believe functionally this now should align with what Grafana are doing. PTAL?

@gotjosh @simonpasquier something I missed is that these changes update Alertmanager to understand error responses from both Slack's chat.postMessage API and incoming webhooks. These are two different APIs with different authentication schemes and different features.
Alertmanager supports both, but the fact that it does so seems coincidental rather than intentional (for example slack_configs contains http_config so Bearer token can be set for chat.postMessage). This is also the case in Grafana, and I have suggested to the team in the past to separate Slack into two receivers: one for chat.postMessage and another for incoming webhooks.
I think we should accept this change because I suspect Alertmanager users are using this integration for both APIs. However, I want to highlight that adding error handling for chat.postMessage further encourages the use of this integration for either API, despite the documentation stating that this integration is just for incoming webhooks.

LGTM once https://github.com/prometheus/alertmanager/pull/3121#discussion_r1286974714 has been fixed! Thanks for your contribution! ❤

further encourages the use of this integration for either API, despite the documentation stating that this integration is just for incoming webhooks.

@grobinson-grafana would you be OK With updating the documentation for that?

Sure! I just wanted to confirm that's what we want 🙂

I just tested a couple of different scenarios.

text/plain response with "oops"

ts=2023-08-10T09:27:48.060Z caller=dispatch.go:352 level=error component=dispatcher msg="Notify for alerts failed" num_alerts=1 err="slack/slack[0]: notify retry canceled due to unrecoverable error after 1 attempts: channel \"#test-alerts\": received an error response from Slack: oops\n"

application/json response with invalid JSON "oops"

ts=2023-08-10T09:28:29.707Z caller=notify.go:745 level=warn component=dispatcher receiver=slack integration=slack[0] aggrGroup={}:{} msg="Notify attempt failed, will retry later" attempts=7 err="channel \"#test-alerts\": could not unmarshal JSON response \"oops\\n\": invalid character 'o' looking for beginning of value"

application/json response with valid JSON

ts=2023-08-10T09:29:16.225Z caller=dispatch.go:352 level=error component=dispatcher msg="Notify for alerts failed" num_alerts=1 err="slack/slack[0]: notify retry canceled due to unrecoverable error after 9 attempts: channel \"#test-alerts\": error response from Slack: oops"

PR to update the docs https://github.com/prometheus/alertmanager/pull/3455


Sync application container

Description
Update code on a provisioned device using Resin CLI resin sync command
Steps to reproduce issue

Install resin cli with npm i -g resin-cli
Login with resin login. If testing on staging, prepend RESINRC_RESIN_URL=resinstaging.io to this and all subsequent resin commands.
Clone this test node js application on your laptop and provision a device with it.
Wait until the application is marked as running on the dashboard.
Enable and navigate to the Public Device URL. Confirm that the web server shows a Hello World message.
Edit server.js of the cloned application so that res.send() returns a different message, e.g. res.send('Hello Resin Sync!')
Run resin devices to get the device uuid of the provisioned device.
Ensure that your resin account has at least one RSA or DSA key -preferably RSA- that is also loaded in your ssh-agent. Use ssh-add -l to check the loaded keys.
Run time resin sync <short device uuid> within the test application folder (i.e. the folder of the edited server.js file). For staging run time RESINRC_RESIN_URL=resinstaging.io resin sync <short device uuid>

Expected result
Expected result

The sync process should start with status messages appearing on each step ('Stopping application container...', 'Syncing..', 'Starting application container...').
At the end of resin sync a "resin sync completed successfully!" message should appear.
The device's Public Device URL should now show the new message, e.g. 'Hello Resin Sync!'

Actual result
resin sync completes, but copies the files to the wrong place. The new if I search for server.js, I find it at ./. ./server.js, wherever that might be actually....
Here's the transcript (with a bit of snip to show only relevant info)
greg@nostromo:simple-server-node 18:08:06 (0) <528>
$RESINRC_RESIN_URL=resinstaging.io resin sync ee764be6931fdebd3fe7c56423ddaed42e93d16029a1968cb14806451ecf26
Connecting with: ee764be6931fdebd3fe7c56423ddaed42e93d16029a1968cb14806451ecf26
Application container stopped.
Synced /usr/src/app on ee764be.
Application container started.

resin sync completed successfully!
greg@nostromo:simple-server-node 18:09:01 (0) <529>
$RESINRC_RESIN_URL=resinstaging.io resin ssh ee764be6931fdebd3fe7c56423ddaed42e93d16029a1968cb14806451ecf26
Connecting with: ee764be6931fdebd3fe7c56423ddaed42e93d16029a1968cb14806451ecf26
root@parallella-hdmi-resin-ee764be:/usr/src/app# cd /
root@parallella-hdmi-resin-ee764be:/# find . -name "server.js"
./usr/src/app/server.js
./. ./server.js

Using latest resin-cli version 4.4.0.
Other information
Reported by: Gergely Imreh
Test configuration: Parallella
Test case number: TC14
Test run: https://resinio.testlodge.com/projects/16238/runs/221686?tab=2&run_section_id=231835&executed_case_id=10347339#executed_case_10347339

Just dropped by to say that this issue is nicely filed.

@lifeeth big thanks for those who made the template :)

No further development is planned for this board.


improvements to merge tool

made interactive merge tool a lot smarter

Pull Request Test Coverage Report for Build 617

42 of 114 (36.84%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.3%) to 62.863%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

scripts/merge.py
37
109
33.94%

Totals

Change from base Build 601:
-1.3%

Covered Lines:
931

Relevant Lines:
1481

💛  - Coveralls


config-conventional: fail on "`"

Fail on:
refactor(config): `webpack` loaders

Expected Behavior
Should be valid
Current Behavior
Invalid
Affected packages

[ ] cli
[ ] core
[ ] prompt
[ ] config-angular
[x] config-conventional

Possible Solution
Looks need fix here https://github.com/marionebl/commitlint/blob/a5309818c244fa09b0a37c87078cb404e34d02c8/%40commi
Steps to Reproduce (for bugs)

Use config-conventional
Try to push commit "refactor(config): webpack loaders"

Context
Use "`" for proper package names
Your Environment
not matter

Executable
Version

commitlint --version
not matter

git --version
not matter

node --version
not matter

The link you posted under solution is broken, could you provide a working one? Thanks!

@marionebl I will send PR with fix and tests https://github.com/marionebl/commitlint/pull/326


Apply serialization in calculator, a_star, more_collections, Couple and Container

Use auto_serialize in the calculator example, and remove the old custom JSON serialization.
The a_star module needed special care and could not rely only on auto_serialize. This was to avoid serializing graphs as a deep tree, instead we serialize the nodes first, then the links and we rebuild the graph at deserialization.
Besides that, more collections support the serialization.

+1

+1

:shipit:

Rerolled to please the bootstrap with the older annotation placement in more_collections.


feat(tweaklistoptions): POC/WIP first shot at tweak list options

goal is to minimize memory usage and allow horizontal scaling
I plan to add more details/documentation. Just submitting as a draft at moment

@mjsmith1028 what is the idea here ? For things like that better to create issue first to discuss if this is direction we want to go. There is another feature planned which should make it possible (https://github.com/metacontroller/metacontroller/issues/266) and tbh I would revise  if this complication is something which can be supported or it will add too much maintenance burden .

@mjsmith1028 what is the idea here ? For things like that better to create issue first to discuss if this is direction we want to go. There is another feature planned which should make it possible (#266) and tbh I would revise if this complication is something which can be supported or it will add too much maintenance burden .

Hi @grzesuav, that is a good point about creating an issue to discuss first. I was hacking away at a POC so I just went ahead and opened a draft POC. I took a look at #266 and this PR is for something different if I understand correctly. #266 is offering an alternative to CRDs via a config file. However, this is to introduce the ability to filter what objects metacontroller observes in the cluster.
Currently, metacontroller uses a shared informer to observe the resources in the Composite or Decorator controller CRs. For the particular resource, it get all objects in the entire cluster and stores them in the in-memory cache. I have some use cases where I know specifically the namespaces that the objects will be in and label/field selectors that the objects will have. The specific case I currently have is using the customize hook to include events as related resources so that my controller can perform some action. For example, if a pod is stuck due to a FailedMount event, then the controller can proactively kill it after a certain period of time instead of waiting indefinitely. The problem I am hitting is that heavily used clusters have a high volume of events and that ends up getting stored in metacontroller's cache causing OOM errors. In my case, I am only concerned with events with type=Warning which can significantly reduce the number of items stored in the cache. See below.
kubectl get events --all-namespaces | wc -l
167884

vs
kubectl get events --all-namespaces --field-selector=type=Warning | wc -l
1611

This PR is inspired by: https://github.com/kubernetes/client-go/blob/f6ce18ae578c8cca64d14ab9687824d9e1305a67/dynamic/dynamicinformer/informer.go#L126 tweakListOptions. Which allows a v1.ListOptions object to be applied during the listwatch operation of the shared informer. Now, there is the ability to add field selectors and label selectors. When this filter gets applied it can determine what is put into the cache and in turn what objects are put into the workqueue.
The way I implemented this is to have a config that can be passed in with, namespaces, field selectors, label selectors that will be applied as tweakListOptions during the listwatch operation. So, now I can specify for the resource Event use a field select type=Warning and metacontroller will only act on events of that type and ignoring all other events.
This can also be extended to namespaces, by using a field selector on metadata.namespace. So, you can specify a list of desired namespaces to monitor. Metacontroller will list all namespaces on startup and if the namespace is not in the desired namespaces list it will create a field selector metadata.namespace!=nameofnamespace. This will ignore results from namespaces not included in the namespaces list. The one caveat here is if a new namespace is created after metacontroller is already running, then this would not get picked up until it restarted. The idea behind this work is that I know of about 10 namespaces my controllers will operate in but there are about 70 namespaces in the cluster. So, metacontroller is storing all the pods and other resources in the cache which I know will never be used. This now gives the capability to restrict one or more namespaces. Which could also let you have an instance of metacontroller per namespace, per label, or field selector. Whatever is desired.
So it looks something like this.

User creates a CompositeController CR with Pod resource.
Metacontroller creates/updates the shared informer with  Pod kind
The shared informer performs a list watch on Pod. In this PR, tweaklistoptions are introduced to the list watch.
The result is stored into the in-memory cache.
The composite controller resource event handlers are triggered by entry in the cache
The composite controller resource event handler adds the Pod to the workqueue
The composite controller reads from the workqueue and performs sync

Hopefully that made sense :joy:. Please let me know what you think. Thanks!

Closing, this was just a POC/Experiment.


Performance issue not otherwise specified

puff — 2024-10-14
@williamcpierce would it be possible to use icons instead of the screencapturekit
there is extremely serious performance degradation for anything over 2 accounts
https://discord.com/channels/1295309622445473865/1295309623074750569/1295363551740821636

Teran Feynoir — 2024-10-14
It’s early alpha, not hotkey or anything
I tried quickly and for me it’s too laggy at the moment to be really usable
https://discord.com/channels/1292215822990970981/1292217015204970527/1295354169166925916

Duvismo — 2024-11-25
got a BIG hit in performance tho
it works but DAMN does it not hurt performance and i have to run the game in window mode
https://discord.com/channels/1292215822990970981/1295516316509147166/1310668390897029180

Zed Al'thor — 2024-12-19
Giving this a try out. Using a Mac Studio (M2 Ultra) setup.
So far it’s running good. The performance is okay with 2-4 clients total. It gets really buggy and starts to freeze adding more than 4 total clients.
https://forums.eveonline.com/t/overview-a-native-macos-alternative-to-eve-o-preview-v0-4-1-alpha/464527/7?u=william_pierce

Related issues: #54 #76


small text change, publication pages

The links to the images say
"view larger graphical abstract/ click to view larger graphical abstract"
but these are not always graphical abstracts so we should say
but these are not always graphical abstracts so we should say
"view larger graphic/ click to view larger graphic"

Good point.  I've made that change and it will be visible in the morning.


missing grunt devmode task

Other Px components such as px-app-nav include a grunt devmode task. this component is missing it

PR: https://github.com/PredixDev/px-spinner/pull/4


Rancher min/max version - Not able to launch catalog app when the default chart version is not applicable for rancher version

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):
Precondition:
rancher version: kinarashah/rancher:v2.3.2(custom image for testing)
chart - fluentd-aggregator
chart version: latest -  rancher_min_version: 2.3.2 and rancher_max_version: 2.4.0
chart version: v0.1.0 - rancher_max_version: 2.3.2
chart version: v0.2.1 - rancher_min_version: 2.3.3

Add to project level catalog -- the custom chart and the branch.
Go to Apps--> Launch app - fluentd-aggregator

Expected Result:
Application should launch, and options of versions - chart version: v0.1.0 must be available.
User should be able to select chart version and launch the app
Actual Result:
Application fails to launch. No error in the UI/dev tools. On clicking View Details of fluentd-aggregator, it goes back to Apps page.
Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): rancher:master-head
Installation option (single install/HA): single

verified on latest UI tag - https://releases.rancher.com/ui/latest2/index.html
Able to launch catalog app - with applicable version - 0.1.0 (fluentd aggregator) and no other versions are shown on the UI. (0.2.1 and 0.3.0 are not applicable for this version of rancher - custom version 2.3.2)


Reports: include read_only tag categories

Purpose or Intent
Split off discussion on #9914.
Reports editor hides any read_only tag categories.
I suspect the condition was simply copied (it's exactly same line) from
https://github.com/ManageIQ/manageiq/blob/62874a37/app/controllers/application_controller/tags.rb#L265
https://github.com/ManageIQ/manageiq/blob/62874a37/app/controllers/application_controller/tags.rb#L316
but that code deals with user assigning/unassigning the tags, so forbidding read only makes sense;
reports merely want to read tags which I think is fine.
Effects

first "Columns" tab:

selecting the category in "Columns", allows manipulating it in other tabs ("Consolidation", "Styling" etc)
"Filters" tab, with "Tag" expression:

[Where do read_only tags come from?  I don't know generally, my use case is tags auto-assigned from kubernetes labels (#7460, not yet exposed in UI) which I want read_only so users don't assign them manually — refresh would overwrite that anyway.]
Steps for Testing/QA
AFAIK read-only tags can't be created from UI; the easiest way with rails console:
cat = Classification.create_category!(name: "rocat1", description: "read_only cat 1", read_only: true)
cat.add_entry(name: "roent1", description: "read_only entry 1")

@miq-bot add-labels reporting, ui, bug, darga/yes

There is another such line in
https://github.com/ManageIQ/manageiq/blob/62874a37/app/controllers/application_controller/performance.rb#L1481
which might also need fixing(?), I'm not familiar with that.

@zeari Please review.
How should I test this?  I now only checked UI works.

How should I test this? I now only checked UI works.

You can tag an entity manually and see that it shows up.
@gtanzillo You once said that the reason for omitting read only tags was that there were internal tags that the user wasnt supposed to see, is this change still ok?

OK, here is a problem: they get added under "My Company Tags".
This doc explains "Company Tags" are ones you define and assign, vs "System Tags" which are automatically assigned.  Makes sense, and sounds like read_only tags should be listed as "System Tags".
@gtanzillo I can't find any code dealing with "System Tags".  Do you know if those are actually a thing?  Is it a concept we want to stick to?

@gtanzillo I think there's an open question to you here. The rest LGTM :+1:
@moolitayer @enoodle can you review?
@miq-bot assign cben

LGTM per @gtanzillo's comment here

LGTM :+1:
@miq-bot assign chessbyte

@cben I'm not familiar with this code but I tracked down this 2007!  The author has been suppressed to protect the innocent.  It seems like this was done purposely ON assignment only as you guessed.  If you can confirm after reviewing this commit, I think we can merge.  Please ping me if you're good with it.
commit 590d6b35e5df6a8cf9dae2f5f1eae66757ededf3
Date:   Wed Oct 31 00:27:16 2007 +0000

    BugzId:
    Fixed bug where "read only" tags were displayed in the My Company Tags assignment screen

    git-svn-id: http://miq-ubuntusub.manageiq.com/svn/svnrepos/Manageiq/trunk@4504 9653ab01-789c-4043-9faa-2b6746af0d71

diff --git a/vmdb/app/controllers/application.rb b/vmdb/app/controllers/application.rb
index 269cab4..87880da 100644
--- a/vmdb/app/controllers/application.rb
+++ b/vmdb/app/controllers/application.rb
@@ -670,16 +670,15 @@ class ApplicationController < ActionController::Base

        # Build the classification assignment screen
        def classify_build_screen
-               cats = Classification.categories        .sort{|a,b| a.name <=> b.name}  # Get the categories, sort by name
+               cats = Classification.categories.sort{|a,b| a.name <=> b.name}  # Get the categories, sort by name
                @categories = Hash.new          # Classifications array for first chooser
+               cats.delete_if{ |c| c.readonly != "0" || c.entries.length == 0} # Remove categories that are read only or have no entries
                cats.each do |c|
-                       if c.readonly == "0"            # Only show the non-read only categories
                                if c.single_value == "1"
                                        @categories[c.description + " *"] = c.name
                                else
                                        @categories[c.description] = c.name
                                end
-                       end
                end
                session[:cat] ||= cats.first                                                                                # Set to first category, if not already set

@@ -687,10 +686,13 @@ class ApplicationController < ActionController::Base

                session[:assignments] = Classification.find_assigned_entries(@tagitems[0])              # Start with the first items assignments
                @tagitems.each do |item|
-                       itemassign = Classification.find_assigned_entries(item)                                                                                # Get each items assignments
-                       session[:assignments].delete_if { |a|   !itemassign.include?(a) }                                                       # Remove any assignments that are not in the new items assignments
-                       break if session[:assignments].length == 0                                                                                # Stop looking if no assignements are left
+                       itemassign = Classification.find_assigned_entries(item)                                                 # Get each items assignments
+                       session[:assignments].delete_if { |a|   !itemassign.include?(a) }       # Remove any assignments that are not in the new items assignments
+                       break if session[:assignments].length == 0                                                                                # Stop looking if no assignments are left
                end
+               if session[:assignments].length > 0                                                                                # if any assignments left
+                       session[:assignments].delete_if { |a| a.parent.readonly != "0"}         # remove the ones from read only categories
+    end
                classify_build_entries_pulldown
        end

Also, @cben please see euwe label if desired.

@jrafanie Thanks for doing the archeology.  Makes sense, I'm good with it.
@miq-bot add-label euwe/yes
@simon3z This is necessary for reporting based on auto-tagging.  Do we want it even in darga?

@simon3z This is necessary for reporting based on auto-tagging. Do we want it even in darga? EDIT: I see we originally wanted this darga/yes — do we still?

@cben did we have a BZ with anyone requesting this in 4.1?

We don't have any BZ for this change itself.  @zeari We have various BZs for reporting by tags, and my understanding that some/all of them we also want to work with auto-tagged read only tags — will/did any of these ship in a darga-* release?
E.g. I see #10671 has been darga/backported.
P.S. @jrafanie can you say anything about [this remaining occurence][1]?  Its result gets used [in _perf_options layout][2], for "EmsCluster", "Host", "Storage", "AvailabilityZone" models.  I have no idea about any of that...
[1]: https://github.com/ManageIQ/manageiq/blob/62874a37/app/controllers/application_controller/performance.rb#L1481
[2]: https://github.com/ManageIQ/manageiq/search?utf8=✓&q=perf_options[%3Acats]

will/did any of these ship in a darga-* release?

@cben this one too: https://github.com/ManageIQ/manageiq/pull/10940
A new BZ is probably the way to go

We don't have any BZ for this change itself. @zeari We have various BZs for reporting by tags, and my understanding that some/all of them we also want to work with auto-tagged read only tags — will/did any of these ship in a darga-* release?
E.g. I see #10671 has been darga/backported.

@cben this commit doesn't have a BZ associated with it.  If we want to ship this fix, please add a BZ in the commit message.

P.S. @jrafanie can you say anything about this remaining occurence? Its result gets used in _perf_options layout, for "EmsCluster", "Host", "Storage", "AvailabilityZone" models. I have no idea about any of that...

@cben I would think this was copy/pasted so if the context is building a list of tags for "view" purposes, then yes, it would be the same as this PR, we shouldn't filter them out.  If it's for assigning or modifying tags, then, I believe the check should remain.  Honestly, this logic should left out of the controllers and be a simple scope in the model:   "viewable_tags" or "assignable_tags" or something.  Either way, it's nasty that this conditional is litter everywhere.  For this issue, just fixing them is fine for now but scoping with a good scope name could really help the readability of this code.
@cben let me know when this is ready for merge (BZ in commit message), thanks!

@jrafanie The BZ for this is https://bugzilla.redhat.com/show_bug.cgi?id=1382720
Please merge, we want this in euwe.
cc @simon3z

@cben @zeari this one is really close, just missing a test or an explanation why there's no test and clarification on the commit message (why and what bz)

@cben ping

Thanks for nudging (vacation + was busy with more urgent stuff).
Opened BZ, edited message.
Found another place in chargeback, but still haven't tested & screenshotted it.
Still need to add tests.
The only way I see to test is extract into small method, and test it doesn return a read_only cat.  Does that sound useful?
classifications.rb already has most of the scopes needed to make such queries pretty, but they're not in darga, so I'll do a followup PR.
@miq-bot add-label wip

@zeari please review

Added tests in the report editor.
Also exctracted (with tests) 2 identical codes for entries.
Added screenshot for the chargeback assignments fix.  (Got lazy and didnt add the nearly identical tests there; I don't want to maintain many nearly identical copies with tests, I'll try centralize all such code in Classification in a followup refactoring...)
@jrafanie @gtanzillo This is ready from my side.
@miq-bot remove-label wip

Done, PTAL.
Used methods instead of let because I dislike relying on let's on-demand execution for side effect.

Done, PTAL.
Used methods instead of let because I dislike relying on let's on-demand execution for side effect.

@cben @jrafanie is this mergeable today? Can we make an effort of getting this reviewed/revised/merged/backported as well so we'll deliver the full auto-tagging capability today?

LGTM @simon3z @jrafanie

@cben @jrafanie is this mergeable today? Can we make an effort of getting this reviewed/revised/merged/backported as well so we'll deliver the full auto-tagging capability today?

@simon3z It would be nice to make an effort to fix the simple rubocops. The bot is there so you can get feedback faster than a human reviewer.  I'd rather not have to merge this now and have a followup PR that also needs backport to fix whitespace.  Let's get it right the first time to avoid git conflicts.
@cben looks great other than the whitespace complaints.

@cben Let's get the Rubocops addressed as soon as possible.  /cc @simon3z

Can someone help me understand what Rubocop wants in this instance?
It doesn't reproduce when I run rubocop locally (prob. using wrong config) and the block braces and |arg| look consistent with all the file and our style guide.

Line 1557 and 1558 have trailing whitespace

@cben Thanks for looking into the cops.  I'm not exactly sure what it's complaining about.  I'll give others a few minutes to review to see if they understand why the bot is concerned about those lines.  Then, I'll merge.  Next time, if you see bot comments that seem totally wrong, please speak up so we can keep them useful and not noisy with useless messages.

Changed .size > 0 to !...empty? (which bot didn't mention but local rubocop legitimately suggested).
Still at a loss about whitespace.  Thanks @Fryguy but I do not see any trailing whitespace (with eyes nor grep).  Adding such whitespace on a test file leads to different message – Trailing whitespace detected. so it's not that.

Oh right...I forgot there was a different message for that.
Yeah, I don't know what's going on with the bot...we'll have to take a look at it.

Checked commits<EMAIL_ADDRESS>with ruby 2.2.5, rubocop 0.37.2, and haml-lint 0.16.1
3 files checked, 0 offenses detected
Everything looks good. ⭐

What now bot!!!  👏 🚢 :shipit:

Euwe Backport details:
$ git log -1
commit 93ac929123683ea4bcf2d6e001a19ae5c2d1d667
Author: Joe Rafaniello<EMAIL_ADDRESS>Date:   Tue Nov 1 13:16:29 2016 -0400

    Merge pull request #10459 from cben/chargeback-read_only-tags

    Reports: include read_only tag categories
    (cherry picked from commit 81b121f00df79afc42c6730ccc16374133473edc)

    https://bugzilla.redhat.com/show_bug.cgi?id=1388659

@simaishi Please make Euwe BZ for https://bugzilla.redhat.com/show_bug.cgi?id=1388659

Darga Backport details:
$ git log -1
commit 5ac96cfa6432875b71a2235c4c09f976fd4b4c88
Author: Joe Rafaniello<EMAIL_ADDRESS>Date:   Tue Nov 1 13:16:29 2016 -0400

    Merge pull request #10459 from cben/chargeback-read_only-tags

    Reports: include read_only tag categories
    (cherry picked from commit 81b121f00df79afc42c6730ccc16374133473edc)

@simaishi Please make Darga BZ for https://bugzilla.redhat.com/show_bug.cgi?id=1388659

Darga BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1391980


Terminate all other container processes when the init exits

Previously, if the container init spawned a child process which
inherited its stdio, then exited, gcs would continue waiting until all
child processes terminated. This only occurs when the container does not
have its own pid namespace, as otherwise the kernel terminates all
processes in the namespace automatically. To preserve this semantic even
when the namespace is shared, we now explicitly kill all other container
processes when its init exits.
Signed-off-by: Kevin Parsons<EMAIL_ADDRESS>

@kevpar I don't think you pushed the most recent comment change

Test for this: https://github.com/microsoft/hcsshim/pull/778


[SP-2860] - Backport of PDI-15569 - Maximum Timeout Field in Wait For…

… Job Entry Does Not Resolve Some Variables (6.1 Suite)
@mchen-len-son, @pamval, here is backport of https://github.com/pentaho/pentaho-kettle/pull/2799/files to 6.1. Thanks.

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -Dtestreports.dir=bin/reports/integration-test -f engine/build.xml jacoco-integration checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  engine/src/org/pentaho/di/job/entries/delay/JobEntryDelay.java
  engine/test-src/org/pentaho/di/job/entries/delay/JobEntryDelayTest.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.job.entries.delay.JobEntryDelay

Complexity Change:  + 2.12%
Instruction Change:  + 1.32%
Line Change:  + .78%
Method Change:  + 2.05%

org.pentaho.di.repository.kdr.KettleDatabaseRepositoryCreationHelper

Branch Change: -1.64%:small_red_triangle_down:
Complexity Change: -.69%:small_red_triangle_down:
Instruction Change: -2.43%:small_red_triangle_down:
Line Change: -2.95%:small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -2.35%:small_red_triangle_down:
Complexity Change: -2.70%:small_red_triangle_down:
Instruction Change: -.18%:small_red_triangle_down:
Line Change: -.53%:small_red_triangle_down:

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change: -.07%:small_red_triangle_down:

org.pentaho.di.trans.steps.sortedmerge.SortedMergeMeta

Instruction Change: -.18%:small_red_triangle_down:

org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta

Branch Change: -.53%:small_red_triangle_down:

org.pentaho.di.trans.steps.uniquerows.UniqueRowsMeta

Instruction Change: -.18%:small_red_triangle_down:

Integration test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.trans.step.BaseStep

Complexity Change: -.18%:small_red_triangle_down:
Instruction Change: -.20%:small_red_triangle_down:
Line Change: -.41%:small_red_triangle_down:
Method Change: -.49%:small_red_triangle_down:

org.pentaho.di.trans.step.RunThread

Branch Change: -2.00%:small_red_triangle_down:
Complexity Change: -3.57%:small_red_triangle_down:
Instruction Change: -.15%:small_red_triangle_down:
Line Change: -2.04%:small_red_triangle_down:


fix: also set port on NextServerOptions for Image component support

Description
If a port OTHER than 3000 is used for the Next/Express server in the project templates and custom server example, the Next.js Image component will still try to serve optimized images via port 3000 (default) unless the port is also set in via NextServerOptions, for example...
const nextApp = next({
    dev: process.env.NODE_ENV !== 'production',
    port: PORT
})

Type of change

[x ] Bug fix (non-breaking change which fixes an issue)

Hey @58bits would you mind fixing the prettier errors so the builds can pass please?
Or I think there's a checkbox to let us edit/commit straight to this PR :thinking:
./src/server.ts
45:15  Error: Insert `,`  prettier/prettier
45:15  Error: Missing trailing comma.  comma-dangle

Hi @paulpopus - sure thing. Done.


[Test Framework] Adding debug logs for KUBECONFIG var

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[x] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

@nmittler: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
76b3b45d32b2ee1a757826ecaf7d9a5f321879ab
link
/test release-notes_istio

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

@nmittler: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
30d7f1b6ad7666d345a27d05cb02c0add6261225
link
/test release-notes_istio

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

/test release-notes_istio


Accept iterables and collections as children.

React supports rendering of iterables and collections as childrens. react-tree-walker does not support this at this moment leading to unexpected results when using Iterables (e.g. List from immutable-js) as children. Subtrees that are rendered in react are might be skipped by the visitor.
I created a minimal example that shows the issue: https://codesandbox.io/s/o9kzlrql86
Here are some links from the original react discussion:
Original Issue
Original PR
Would be great to have this feature!

I've accomplished this in a fork using code lifted from the React merge that allowed for iterator functions:
https://codesandbox.io/s/pj998lr3lj
Relevant lines:
https://github.com/jaredLunde/react-tree-walker/blob/master/src/index.js#L15
https://github.com/jaredLunde/react-tree-walker/blob/master/src/index.js#L78
https://github.com/jaredLunde/react-tree-walker/blob/master/src/index.js#L137
https://github.com/jaredLunde/react-tree-walker/blob/master/src/index.js#L259
Would you accept a PR w/ test that clears this issue @ctrlplusb? My version of the library is quite a bit different at this point (removed Preact stuff, breadth-first resolve rather than depth-first to allow for concurrency, added more strict element type checks etc), but I can try to make a PR as simple as possible for you.
Otherwise you can obviously just lift those sections on your own.


Cheatsheet

I am a hackthoberfest contributer and want to add some cheatsheets prepared by me.

I am a hackthoberfest contributer and want to add some cheatsheets prepared by me.

Which topic cheat sheet you want to add?

Note, we do not host books cheatsheets etc. If your sheets are published on a website (not a file-sharing site) you can add a link to it.


[20.03] nixos/tinydns: use local dns server to fix test (backport)

(cherry picked from commit c8a873560fa002b713d90259c31ac3d05eddb416)

Motivation for this change
Force host command to use a local DNS server for resolving.
ZHF: #80379
Backport of #80565
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This change is

@GrahamcOfBorg test tinydns


VNC recording with BrowserWebdriverContainer in Spock-Extension

Revives #726.
Initially done as part of the Hackergarten at Gr8ConfEU 2018 (yes, that long ago :sweat_smile:).
Thanks to Tamer Shahin, Marcin Erdmann and Dawid Kublik for working on this :slightly_smiling_face:  !
Fixes #631.

Thinking about it, this is probably missing support for @Shared containers as of now, but we can add this in a later stage.

This was released in https://github.com/testcontainers/testcontainers-java/releases/tag/1.14.0 🎉
Thanks for the contribution!


Error when trying to upgrade to 15.0 when using HomeBrew on El Capitan GM Candidate

Getting the following when attempting to upgrade flow using 'brew upgrade flow':
fatal error: too many errors emitted, stopping now [-ferror-limit=]
20 errors generated.
Command exited with code 2.
make: *** [build-flow-native-deps] Error 10

Hi @admmasters sorry you're having trouble!
I'm afraid I haven't tested Flow's build for OSX 10.11 and I'm not sure how well Flow builds. It sounds like homebrew doesn't bottle for 10.11 and instead installs from source. For some reason our build seems to fail on El Capitan.
Is there a verbose flag that might show you the build errors? Maybe brew upgrade -v flow? It would be useful for me to see what is failing.
I'm upgrading a laptop to El Capitan to investigate. If it's easy, then I'll try and fix. But it might be hard to keep things working on 10.11...I think Travis CI just runs 10.9 and we just assume if it works on 10.9 then it works on 10.10. If a lot has changed for 10.11 and if we don't have any one testing 10.11 our support will definitely rot :/

Well I have good news and bad news.
The good news is that Flow seems to build fine on 10.11. I tried brew install flow and I also cloned this repo and ran make test.
The bad news is I have no idea what is causing you trouble. Maybe when I see why the build is failing for you I'll be more help. Sorry!

I had similar issues on El Capitan GM, in the end I used the following:
opam install flowtype
And that worked for me. YMMV.

I simply did brew uninstall flow && brew install flow and had no issues. The actual install should work fine, just the update path in brew seems to error.

Odd - uninstalling, then installing manually without using Brew - works. Brew then worked when I tried to install once again.


clicking on prayer change the theme

While clicking on the prayer to disable the notification it changes the Theme from dark to light and vice versa

Done


Cocoapods use_modular_headers!

Hi.
I use Cocoapods to install, but I have an error while compiling.
This is my Podfile
use_modular_headers!

pod 'KingfisherWebP'

I know use this can solve my problem, but i don't want to generate a dynamic library.
Can you help me?
use_frameworks!

pod 'KingfisherWebP'

Any progress or plan for fixing it?

@InsectQY @yichengchen Apologizes for this late reply 😂 Support for static library should be available in latest version.


Enroll support Throttled on DescribeCases operation

Still present in version 1.9.0
Error when running customizations on approximatively 145 accounts, located on aft_enroll_support.py.
Causong the following issue:
{"errorMessage": "An error occurred (ThrottlingException) when calling the DescribeCases operation (reached max retries: 4): Throttle limit exceeded.", "errorType": "ClientError", "stackTrace": ["  File "/var/task/aft_enroll_support.py", line 36, in lambda_handler\n    if not account_enrollment_requested(ct_mgmt_session, target_account_id):\n"
I've managed to make it work with the following:
aft_enroll_support.py
line 36
Original code
if not account_enrollment_requested(ct_mgmt_session, target_account_id):
    generate_case(ct_mgmt_session, target_account_id)

Proposal for workaround:
requested = None
while requested is None:
    try:
        requested =  account_enrollment_requested(ct_mgmt_session, target_account_id)
    except:
        pass
if not requested:
    generate_case(ct_mgmt_session, target_account_id)

Hi @sl-ediquas,
Thank you for the report. We're aware of the issue and have a backlog ticket to address it.

Any news on this? With this bug we can't really roll out anything on our accounts because it fails the moment we target more than 15 accounts. Which, to be honest, is quite the small number.

+1 . Any updates here -  we're hitting this issue as well and we have a large pool of accounts to roll changes out to.

Hi @stumins, any chance this has progressed at all? This is a painful one for us.

Hi all -
AFT 1.10.4 was released last month with improved retry behavior to address these types of throttling issue, including for the Support API referenced in this issue.
Please upgrade to 1.10.4 and let us know if you continue to experience any throttling exceptions.


Calling CommandList.SetVertexBuffer with a null DeviceBuffer throws a NullReferenceException

I'm trying to do this:
commandList.SetVertexBuffer(1, null);

in order to clear any previously-set buffers for that slot. But both CommandList.SetVertexBuffer and the derived implementations of CommandList.SetVertexBufferCore expect the buffer argument to be non-null.
Is this by-design, or a bug? If it's a bug I can create a PR to fix it. If it's by-design, is there a better way to do what I'm trying to do?
I suppose that what I'm trying to do is actually unnecessary, as long as the current pipeline doesn't reference a vertex buffer in that slot (and if it did I should be setting a non-null buffer anyway), but it doesn't feel right to leave the old buffer lying around.

This is by-design. You don't need to un-bind the buffers that aren't used by the current Pipeline -- they are just ignored.
I guess the proper thing to do here is

Document this behavior, so it's less surprising
Throw an exception when a null buffer is passed (perhaps referencing the behavior)

Okay, that makes sense. I guess it's best practice not to unbind vertex buffers.
My only thought was around garbage collection - a vertex buffer might (unlikely) end up still referenced by a command list, long after it was last used. But that's a very minor concern.

I added an extra note in the XML doc for this method which hopefully will help explain the behavior.


Fix German (and some other) translations

Well, there where some pretty funny translations, for instance Iran -> I ran -> Ich rannte or Togo -> to go -> gehen :D
I fixed the German mistakes and in some other languages I used English as a fallback option.
In the other file I removed all the redundant translations, because they will be merged ($.extend()) nevertheless.

great man thank you !!!!


Missing social medial links (Reddit and Mastodon)

Is your feature request related to a problem? Please describe.
Some social media such as Reddit and Mastodon are not included for implementation.
Describe the solution you'd like
I would like to see an easy _config.yml implementation of these social media links.

I am closing this issue as I do not think everyone will be interested in having these links. Besides, we can add other social links by editing _includes/social.html.


本地运行Python脚本报错

修改dolphinscheduler_env.sh中“export PYTHON_HOME=/opt/soft/python”为本机python目录，执行中报“sudo: /opt/soft/python: command not found”错误，求助问题在哪呢？

Please update/add title and description in English, thx

@wgwtransfer You can refer to the similar issue #2548 #2620 #2868 #3853.
By the way, we will improve the HOME environment variables with #5035

@wgwtransfer You can refer to the similar issue #2548 #2620 #2868 #3853.
By the way, we will improve the HOME environment variables with #5035
You need to set PYTHON_HOME to a correct path of python like /usr/bin/python in file conf/env/dolphinscheduler_env.sh

Yes，I sovled this matters.    I set PYTHON_HOME=/Library/Frameworks/Python.framework/Versions/3.8/bin/python3.8, and added PYTHON_HOME to /etc/profile.   ps. It is deployed in MacOX.


Unable to install AzureMaps packages

I cannot get this project to build because it's missing the AzureMapsToolKit package. When I try to dotnet add package this and the .common package, I get this error:
error: Unable to find package AzureMapsToolkit. No packages exist with this id in source(s): nuget.org
error: Package 'AzureMapsToolkit' is incompatible with 'all' frameworks in project 'C:\Users\aa5arzz\OneDrive - 3M\Documents\SourceCode\RefrigeratedTruck\RefrigeratedTruck.csproj'.
At a loss for what to do next.
It would be nice if in the tutorial it mentioned needing to install these packages, including the Azure.Device packages.

Nevermind. Resolved  library issues myself.


refactor(dbm-services): 开区代码优化

关联研发信息
dev_branch: refactor/optimization_open_zone/#1296
target_branch: v1.2.0


"yakut monitor" is throwing errors with SocketCAN

Any ideas?  I just upgraded pyuavcan, yakut, nunavut, and a number of other dependent packages tonight.
Using a PCAN-USB CAN adapter (shows up as a socketcan interface; works fine with can-utils and yakut {pub|sub}).
Python 3.9.1
Yakut 0.5.2
pyuavcan 1.2.4
NodID Mode Health VSSC Uptime         VProtcl VHardwr VSoftware(major.minor.vcs.crc)            Unique-ID                        Name
    1 oper nomina    0     0d00:00:49 ?       ?       ?                                         ?                                ?
    3 ?    ?      ?    offline          1.0     0.0     0.5                                     8d5361a31e63e8b36942aea7855b9d8d org.uavcan.yakut.monitor
   20 oper nomina    0     0d00:01:16 ?       ?       ?                                         ?                                ?
APPLICATION LAYER CONNECTIVITY MATRIX [t/s=transfer/second] Colors: pub/cln│sub/srv│(pub+sub)/(cln+srv)│activity│uavcan.node.port.List is published/not│
MESSG    1   20 ∑t/s ∑B/s
   22   4         4  242    22
   30   2         2   36    30
   31   2         2   33    31
 7509  30    1   31  220  7509
∑t/s   38    1   39      ↖ t/s
∑B/s  525    6       532
RQ+RS    1   20 ∑t/s ∑B/s
∑t/s    0    0    0      ↖ t/s
∑B/s    0    0         0
TOTAL    1   20 ∑t/s ∑B/s
∑t/s   38    1   39
∑B/s  525    6       532
Total transport layer errors:        0         Values averaged over 10.0 sec
2021-04-21 20:15:55 0023949 ERR pyuavcan.application.heartbeat_publisher: HeartbeatPublisher(heartbeat=uavcan.node.Heartbeat.1.0(uptime=8, health=uavcan.node.Health.1.0(value=0), mode=uavcan.node.Mode.1.0(value=0), vendor_specific_status_code=49), priority=NOMINAL, period=1.0) publisher task exception: [Errno 22] Invalid argument
Traceback (most recent call last):
  File "/home/jacob/Workspace/uavcan-v1/venv/lib/python3.9/site-packages/pyuavcan/application/heartbeat_publisher.py", line 203, in _task_function
    if not await pub.publish(self.make_message()):
  File "/home/jacob/Workspace/uavcan-v1/venv/lib/python3.9/site-packages/pyuavcan/presentation/_port/_publisher.py", line 113, in publish
    return await self._maybe_impl.publish(message, self._priority, self._loop.time() + self._send_timeout)
  File "/home/jacob/Workspace/uavcan-v1/venv/lib/python3.9/site-packages/pyuavcan/presentation/_port/_publisher.py", line 190, in publish
    return await self.transport_session.send(transfer, monotonic_deadline)
  File "/home/jacob/Workspace/uavcan-v1/venv/lib/python3.9/site-packages/pyuavcan/transport/can/_session/_output.py", line 206, in send
    return await self._do_send(can_id, transfer, monotonic_deadline)
  File "/home/jacob/Workspace/uavcan-v1/venv/lib/python3.9/site-packages/pyuavcan/transport/can/_session/_output.py", line 163, in _do_send
    if await self._send_handler(transaction):
  File "/home/jacob/Workspace/uavcan-v1/venv/lib/python3.9/site-packages/pyuavcan/transport/can/_can.py", line 327, in _do_send
    num_sent = await self._maybe_media.send(
  File "/home/jacob/Workspace/uavcan-v1/venv/lib/python3.9/site-packages/pyuavcan/transport/can/media/socketcan/_socketcan.py", line 128, in send
    await asyncio.wait_for(
  File "/home/jacob/local/lib/python3.9/asyncio/tasks.py", line 478, in wait_for
    return fut.result()
  File "/home/jacob/local/lib/python3.9/asyncio/selector_events.py", line 446, in sock_sendall
    n = sock.send(data)
OSError: [Errno 22] Invalid argument
^C
Aborted!
(venv) jacob@jacob-desktop:~/Workspace/uavcan-v1$ python --version
Python 3.9.1
(venv) jacob@jacob-desktop:~/Workspace/uavcan-v1$ yakut --version
yakut, version 0.5.2
(venv) jacob@jacob-desktop:~/Workspace/uavcan-v1$ pip freeze | grep pyuavcan
pyuavcan==1.2.4

Is CAN FD enabled on the SocketCAN interface? It can be done by setting the MTU = 72 (64 bytes of payload + 8 bytes of the kernel overhead).
If you don't want CAN FD, use register uavcan.can.mtu (the corresponding environment variable is UAVCAN__CAN__MTU).

I'm not trying to use CAN-FD though, so I have that disabled on the socket... I take it by this you're saying that yakut expects CAN-FD is eanbled by default?

Yes. This is documented in the PyUAVCAN API documentation linked from the
Yakut manual. I expect that most systems that adopt UAVCAN/CAN v1 will be
FD-capable, so it makes sense to default to that.
On Thu, Apr 22, 2021, 06:30 Jacob Crabill @.***> wrote:

I'm not trying to use CAN-FD though, so I have that disabled on the
socket... I take it by this you're saying that yakut expects CAN-FD is
eanbled by default?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/UAVCAN/yakut/issues/21#issuecomment-824511668, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZFIZFIHEBK7UML6BTI6IDTJ6J7FANCNFSM43LSBHZQ
.

@JacobCrabill should I close this issue?

I believe that worked - it runs without errors, but gives me no data now - I'm going to assume for now that's an issue with my setup and worry about it later.
For other's reference:
sudo ip link set can0 up type can bitrate 1000000 dbitrate 1000000 fd on
export YAKUT_TRANSPORT="CAN(can.media.socketcan.SocketCANMedia('can0',16),3)"
export UAVCAN__CAN__BITRATE=1000000
export UAVCAN__CAN__MTU=16

For the purposes of testing a non-FD network (and assuming I were to use a non-FD-compatible adapter such as a Babel), how would I get this to work in that case?

Wait, this isn't right. The docs say (in a slightly roundabout way but nevertheless) that if you use a transport initialization expression, the environment variables are ignored. Also, my fault here is that above I spoke about the Linux kernel MTU setting, not Yakut/PyUAVCAN:

It can be done by setting the MTU = 72 (64 bytes of payload + 8 bytes of the kernel overhead).

When configuring Yakut, you just set the MTU as-is without any overheads included. So, in your case, the configuration to disable FD is:
export UAVCAN__CAN__IFACE=socketcan:can0
export UAVCAN__CAN__BITRATE=1000000 1000000
export UAVCAN__CAN__MTU=8

If you want FD:
export UAVCAN__CAN__IFACE=socketcan:can0

export UAVCAN__CAN__IFACE=socketcan:can0
export UAVCAN__CAN__BITRATE=1000000 1000000
export UAVCAN__CAN__MTU=8

Ok, I simply had the MTU wrong - I've been a little confused as to whether the MTU refers to the overall message size (including header), or the payload alone (so 8 or 16).  I think I've seen references online to both variants.  In this case for non-FD, setting the Yakut / PyUAVCAN MTU to 8 solved the issue.
So the updated environment variable to make these systems all play happily together is:
export YAKUT_TRANSPORT="CAN(can.media.socketcan.SocketCANMedia('can0',8),3)"

Or to use your UAVCAN_CAN_* variables above.

@JacobCrabill fyi #22

Cool.  Thanks @pavel-kirienko!


add thirdpartyresource s3 test service

Adds actions/rules for test thirdparty resource s3 service.

Please update your branch to merge into dev instead of Master.


feat(taiko-client): improve BlobTransactionBuilder to support BlobTxListOffset

If the blob size is 10, block 1 blob size is 6, block 2 blob size is 10, block 3 blob size is 4.
The result is:
block 1 offset 0, blob index 0
block 2 offset 0, blob index 1
block 3 offset 0, blob index 2
rather than:
block 1 offset 0, blob index 0
block 2 offset 0, blob index 1
block 3 offset 6, blob index 0


Peres Testing data. Impact areas shapefile has no string columns

When trying to import the shapefile impact areas for the Peres dataset, i got the following message:

The shapefile has 5 columns, but I guess none of them are strings.

The string columns are used to load the "Unique Name" combobox.

I vote that I remove the check for string columns and just fill the combo with all the columns that are in the shapefile. I think we did the same thing with the combos when importing structures.
@rnugent3 Are you ok with this?

I changed it to accept any column type.

This seems to be working well. Closing.


Enable enforcing coordinator for request

Motivation
cpp-driver provides an option to explicitly set a node to be coordinator for a specific request. It argues for that option this way:

In general, this should not be used, but it can be useful in the following situations:

To query node-local tables such as system and virtual tables.
To apply a sequence of schema changes where it makes sense for all the changes to be applied on a single node.

Solution
Let's expose a setter for explicit coordinator - consider identifying by address or host_id.
If an explicit coordinator is set for a statement, then:

load balancing policy shall be ignored,
speculative execution shall be disabled,
retries shall be only performed on the same node (consider if RetryNextNode decisions should be then changed into RetrySameNode or into DontRetry).

This should be in line with #738 requirements about enforcing a particular coordinator.

This can already be done by setting a custom LBP on the query, which yields only 1 element.

This can already be done by setting a custom LBP on the query, which yields only 1 element.

Yes, it can be done, but it's fairly inconvenient. The driver's duty is to provide convenient and shorthands to operate with Scylla, and by cpp-driver's documentation (see Motivation) the shorthand for setting an explicit coordinator for a statement is useful.


[Bug] Thread DUT commissioning failure with Internal Error

Describe the bug
I'm using nRF Thread dongle (RCP) connected to Raspberry Pi with Matter Test Harness and Silicon Labs EFR32MG Thread Starter Kit with Door Lock sample application as DUT on Thread. I see the internal Error whenever I try to execute a test case. DUT commissioning is unsuccessful with below errors:
Test Suite Error: Error occurred during setup of test suite.FirstChipToolSuite. sent 1011 (internal error) keepalive ping timeout; no close frame received
or
CHIP:TOO Pairing Failure: src/platform/Linux/bluez/BluezEndpoint.cpp: "611:" CHIP Error 0x000000AC: Internal error
Also, the Test Harness doesnt auto launch and will have to manually launch it everytime after boot up using ./certification-tool/scripts/start.sh command
Steps to reproduce the behavior
No response
Expected behavior
No response
Log files
MoT_Test_1_2024_06_28_07_07_31-7.json
MoT_Test_1_2024_06_26_12_50_30.log
PICS file
Door lock Test Plan PICS.zip
Screenshots

Environment
Raspberry Pi 4
Ubuntu Server 22.04 LTS (64-bit)
Thread Border Router RCP - nRF52840 Dongle (FW - rcp_thread_1_3_nrf52840dongle_nrf52840_v2.0.2)
Reference Matter Node (DUT) on Thread - Silicon Labs EFR32MG Thread Starter Kit
Additional Information
Project Config:
{
"config": {
"test_parameters": null,
"network": {
"wifi": {
"ssid": "",
"password": ""
},
"thread": {
"rcp_serial_path": "/dev/ttyACM0",
"rcp_baudrate": 115200,
"on_mesh_prefix": "fd11:35::/64",
"network_interface": "wlan0",
"dataset": {
"channel": "25",
"panid": "0x5b35",
"extpanid": "5b35dead5b35beef",
"networkkey": "00112233445566778899aabbccddeeff",
"networkname": "5b35"
},
"otbr_docker_image": null
}
},
"dut_config": {
"discriminator": "3840",
"setup_code": "20202021",
"pairing_mode": "ble-thread",
"chip_timeout": null,
"chip_use_paa_certs": false,
"trace_log": true
}
},

Also, the Test Harness doesnt auto launch and will have to manually launch it everytime after boot up using ./certification-tool/scripts/start.sh command

Could you send the docker logs after you restart the raspberry?
1 - cd certification-tool
2 - docker compose logs backend
Also you can try to reinstall the TH.
1 - cd certification-tool
2 - ./scripts/stop.sh
3 - ./scripts/ubuntu/auto-install.sh v2.10.1-beta1+spring2024
4 - Check if no erros occurs and reboot the raspberry

@cecille could you help to triage this issue? About the internal error response?

@hiltonlima
Pls find the Docker logs attached here
docker compose logs backend.txt

Also, the Test Harness doesnt auto launch and will have to manually launch it everytime after boot up using ./certification-tool/scripts/start.sh command

Could you send the docker logs after you restart the raspberry? 1 - cd certification-tool 2 - docker compose logs backend
Also you can try to reinstall the TH. 1 - cd certification-tool 2 - ./scripts/stop.sh 3 - ./scripts/ubuntu/auto-install.sh v2.10.1-beta1+spring2024 4 - Check if no erros occurs and reboot the raspberry

@hiltonlima
I'm unable to run the Test harness with the above branch. Backend service is unable to start
matter@ubuntu:~/certification-tool$ ./scripts/start.sh
[+] Running 6/6
✔ Network chip-default                       Created                                                                                0.4s
✔ Network certification-tool_traefik-public  Created                                                                                0.3s
✔ Container certification-tool-db-1          Started                                                                                1.9s
✔ Container certification-tool-proxy-1       Started                                                                                2.6s
✔ Container certification-tool-frontend-1    Started                                                                                1.7s
✔ Container certification-tool-backend-1     Started                                                                                2.8s
Waiting for frontend to start................. done
Waiting for backend to startservice "backend" is not running
matter@ubuntu:~/certification-tool$ git branch --show-current
v2.10.1-beta1+spring2024

@cecille could you help to triage this issue? About the internal error response?

@hiltonlima Any update on this error.
Further observation:
I'm able to successfully run test only once. Next test run almost always fails with below internal error and Raspberry Pi get disconnected with WiFi AP. It works the next time only on power cycle of Raspberry Pi. My team has also reported similar behaviour with CHIP-tool running on Raspberry Pi.
"Test Suite Error: Error occurred during setup of test suite.FirstChipToolSuite. sent 1011 (internal error) keepalive ping timeout; no close frame received"

Hi @SupreethKoushik2209 , unfortunately we don't have any news yet. As soon as we have any information we will let you know.

@SupreethKoushik2209 sorry for this late response...
This error still occurring on the latest branch? v2.11-beta3.1+fall2024

Closing due to inactivity. Please reopen if needed.


At least one label specified must be in y_true

Hi, with the latest changes I got a new error when run run_generation_pipeline.py
Traceback (most recent call last):
  File "path_to_repo/AutoPrompt/run_generation_pipeline_alena.py", line 64, in <module>
    best_prompt = ranker_pipeline.run_pipeline(opt.num_ranker_steps)
  File "path_to_repo/AutoPrompt/optimization_pipeline.py", line 272, in run_pipeline
    stop_criteria = self.step()
  File "path_to_repo/AutoPrompt/optimization_pipeline.py", line 252, in step
    self.eval.add_history(self.cur_prompt, self.task_description)
  File "path_to_repo/AutoPrompt/eval/evaluator.py", line 115, in add_history
    conf_matrix = confusion_matrix(self.dataset['annotation'],
  File "path_to_env/AutoPrompt/lib/python3.10/site-packages/sklearn/utils/_param_validation.py", line 214, in wrapper
    return func(*args, **kwargs)
  File "path_to_env/AutoPrompt/lib/python3.10/site-packages/sklearn/metrics/_classification.py", line 340, in confusion_matrix
    raise ValueError("At least one label specified must be in y_true")
ValueError: At least one label specified must be in y_true

config_ranking.yml and config_generation.yml are not modified.
config_default.yml is
use_wandb: False
dataset:
    name: 'dataset'
    records_path: null
    initial_dataset: ''
    label_schema: ["Yes", "No"]
    max_samples: 5
    semantic_sampling: False # Change to True in case you don't have M1. Currently there is an issue with faiss and M1

# annotator:
#     method : 'argilla'
#     config:
#         api_url: ''
#         api_key: 'admin.apikey'
#         workspace: 'admin'
#         time_interval: 5

annotator:
    method: 'llm'
    config:
        llm:
            type: 'OpenAI'
            name: 'gpt-3.5-turbo-0613'
        instruction: 'Assess whether the text contains a harmful topic.
            Answer Yes if it does and No otherwise.'
        num_workers: 2
        prompt: 'prompts/predictor_completion/prediction.prompt'
        mini_batch_size: 1
        mode: 'annotation'

predictor:
    method : 'llm'
    config:
        llm:
            type: 'OpenAI'
            name: 'gpt-3.5-turbo-0613'
#            async_params:
#                retry_interval: 10
#                max_retries: 2
            model_kwargs: {"seed": 220}
        num_workers: 2
        prompt: 'prompts/predictor_completion/prediction.prompt'
        mini_batch_size: 1  #change to >1 if you want to include multiple samples in the one prompt
        mode: 'prediction'

meta_prompts:
    folder: 'prompts/meta_prompts_classification'
    num_err_prompt: 1  # Number of error examples per sample in the prompt generation
    num_err_samples: 2 # Number of error examples per sample in the sample generation
    history_length: 4 # Number of sample in the meta-prompt history
    num_generated_samples: 10 # Number of generated samples at each iteration
    num_initialize_samples: 10 # Number of generated samples at iteration 0, in zero-shot case
    samples_generation_batch: 10 # Number of samples generated in one call to the LLM
    num_workers: 5 #Number of parallel workers
    warmup: 4 # Number of warmup steps

eval:
    function_name: 'accuracy'
    num_large_errors: 4
    num_boundary_predictions : 0
    error_threshold: 0.5

llm:
    type: 'OpenAI'
    name: 'gpt-3.5-turbo-0613'
    temperature: 0.8

stop_criteria:
    max_usage: 2 #In $ in case of OpenAI models, otherwise number of tokens
    patience: 3 # Number of patience steps
    min_delta: 0.05 # Delta for the improvement definition

I run command
python run_generation_pipeline.py \
    --prompt "Write a good and comprehensive movie review about a specific movie." \
    --task_description "Assistant is a large language model that is tasked with writing movie reviews."

Hi,
Observe that you modify the annotator to be an LLM estimator. However, the prompt for the annotator asks the model to classify 'Yes' or 'No', where the ranker labels are '1','2',..,'5' (see the config_ranking label_schema).
In this case, the annotator provides non-existing labels which results in this error.
An example of a valid instruction for your task:
"Analyze the following movie review, and provide a score between 1 to 5"
One more thing, I see that you using gpt-3.5 for the meta-prompts (and the annotator). This should not work well, especially for the generation tasks, it's important to use GPT-4/4.5 to get optimal performances.

Hi @Eladlev ,
I was working on a generation task. I followed the instructions that in config_generation:
annotator:
    method : ''

and in config_default:
  method: 'llm'
  config:
      llm:
          type: 'OpenAI'
          name: 'gpt-4'
      instruction:
         "Assess this generated message,
         1. does it align with the intent of user input,
         2. does it rephrase user input,
         If all the answers are Yes, then response '1', otherwise response '0'"
      num_workers: 5
      prompt: 'prompts/predictor_completion/prediction.prompt'
      mini_batch_size: 1
      mode: 'annotation'

Is it expected that in the dump/generator/dataset.csv, the 'prediction' and 'score' are all blank?
And can you suggest me the role of 'annotator' in generation tasks?
Thank you

Hi @danielliu99,

At least the predictor should not be blank (after the iteration is completed)
If you are using an LLM ranker, then you should skip the ranking training phase (since you already have an LLM ranker) and you should change this line:
https://github.com/Eladlev/AutoPrompt/blob/7f373f219aa360cd2de38c6aa700c1dff282d7de/run_generation_pipeline.py#L53
To:
generation_config_params.eval.function_params.instruction = ranker_config_params.annotator.config.instruction
In the generation task, there are two phases in phase 1 we train a ranker prompt (this is the phase that should be skipped in your case), in this case, the role of the annotator is similar to the classification task. In the second part, we are not using an annotator (this is the method is left blank). Instead, we are modifying the score function to be the rank of the ranking model and we apply it directly on the model prediction (so there is a need for the annotator part since it's part of the score function).


OnStartClient not called for scene objects

Describe the bug
OnStartClient is no longer being called on scene objects
Repro project
https://drive.google.com/open?id=1ea1xbZlgxsC_PgMgduZiZaIuJrIsWel2
To Reproduce

Open the attached project
build a standalone player
run the standalone player and click on server only
Start the editor and click on LAN Client
The client connects,  but OnStartClient is never called in NetworkedObject

Expected behavior
In the console log,  you should see the message "OnStartClient called"
Desktop (please complete the following information):

OS: MacOS
Build target: standalone
Unity version: 2017.4
Mirror:  master version 728

Additional context
Notice this was working last week.  It could be related to the spawned dictionary refactor

:tada: This issue has been resolved in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


#516 Report invalid values for enums through parser error handler

This patch catches the low-level IllegalArgumentException from the model code and instead reports an invalidValue through the parser's error handler.
Unit test included

Coverage increased (+0.001%) to 87.681% when pulling a9a350754467d5de70c49cde7d2819b09d80af50 on aehrc:feature/parser_exception_handling into e8fad33042529ad8d90e13973e817a2cf25ac0e5 on jamesagnew:master.

Thanks for the contribution!


iOS 14 onload app message popup "MyAPP pasted from {other installed app}"

Please provide all the information requested. Issues that do not follow this format are likely to stall.
Description
With iOS 14 new release, device is detecting that my app is accessing clipboard and coping data from other apps, however my app not even using Clipboard.
React Native version:
Run react-native info in your terminal and copy the results here.
System:
OS: macOS 10.15.5
CPU: (8) x64 Intel(R) Core(TM) i5-8259U CPU @ 2.30GHz
Memory: 170.30 MB / 8.00 GB
Shell: 5.7.1 - /bin/zsh
Binaries:
Node: 11.1.0 - ~/.nvm/versions/node/v11.1.0/bin/node
Yarn: 1.22.1 - /usr/local/bin/yarn
npm: 6.5.0 - ~/.nvm/versions/node/v11.1.0/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
SDKs:
iOS SDK:
Platforms: iOS 13.5, DriverKit 19.0, macOS 10.15, tvOS 13.4, watchOS 6.2
Android SDK:
API Levels: 23, 25, 26, 27, 28
Build Tools: 23.0.1, 25.0.0, 25.0.2, 26.0.1, 26.0.2, 26.0.3, 27.0.3, 28.0.1, 28.0.3
System Images: android-28 | Google Play Intel x86 Atom, android-R | Google APIs Intel x86 Atom
IDEs:
Android Studio: 3.5 AI-191.8<IP_ADDRESS>91312
Xcode: 11.5/11E608c - /usr/bin/xcodebuild
npmPackages:
react: 16.11.0 => 16.11.0
react-native: 0.61.5 => 0.61.5
npmGlobalPackages:
create-react-native-app: 2.0.2
generator-react-native-ignite: 1.13.0
react-native-cli: 2.0.1
react-native-ignite: 1.13.0
Steps To Reproduce
Provide a detailed list of steps that reproduce the issue.

Setup an new project with CRNA
Copy any text from another app
Open the react native app (iOS 14)
then the "MyAPP pasted from {other installed app}" as in screenshot

Expected Results
No to see this message as I'm not accessing Clipboard
Screenshot

Facing the same issue, any news?

We need to be careful with this situation as our apps made with RN might been perceived as "spying on their users" (see tiktok/linkedin & friends reading clipboard on every keystroke visible thanks to this tiny notification - even if that's not on every keystroke for RN apps).
Anyway we know it's probably not a big deal on RN side for iOS: there is probably just an action that might happen too early instead of being done when really necessary.
My knowledge of RN internal made me unable to track where this happens but I have the feeling this can be easy to adjust (but I can be wrong).

I think this might be an expo issue. (My expo-cli is borked right now so I can't debug it).
If I create a new RN app using npx react-native init I cannot repro the issue. Just to double check that my device was capable of repro'ing the issue, I added this small snippet to app delegate:
- (void)applicationDidBecomeActive:(UIApplication *)application
{
  UIPasteboard *pb = [UIPasteboard generalPasteboard];
  NSString *string = pb.string;
}

And was able to see the popup.

In our case we are experiencing this issue and not using Expo

@tomasfrancomediamonks do you have any other custom ObjC code?
Here's why I don't think this is an issue with React Native core:
RN core only has 3 callsites to [UIPasteboard generalPasteboard] (which is the only API that triggers this popup to my knowledge) :

RCTRedBox - Shouldn't be called unless redbox on screen.
RCTTextView - For the "copy" function, makes sense.
RCTClipboard - A native module to support getting & setting the clipboard, makes sense.

Putting breakpoints in all of these^, none of them get hit on startup of a fresh RN app (created with react-native init).

@PeteTheHeat thanks for your support. We don't have custom ObjC code but I looked into our dependencies and the responsible is the FirebaseDynamicLinks Pod.

(For my case, it can be the same, as I have an old version of react-native-firebase as a dependency.)

@MoOx was it solved when you updated react-native-firebase version?

I didn't yet, but it doesn't seems to be a react-native issue from what I have found in RN codebase.

I will remove firebase from my project now, and see if it still happens or not. Thanks.

Cool, so sounds like this is an issue with react-native-firebase.
If updating that dep doesn't solve the problem, open an issue on that repo. Closing this out :)

If anyone facing the same issue, once i commented out Firebase/DynamicLinks dependency in my PodFile is went away.

@tomasfrancomediamonks  Thank you.


Fixed overflowing calculations

i made the font bigger, and it reduces when the number becomes larger and increases when you delete them, i had to add another div to result, had no choice, please review code and test app.
regards🙏

Hi @ify47, I'm sorry but i can't merge your PR because your branch has conflicts with mine

hello friend, just fixed conflicts
regards

Thank you for your work @ify47


bug: Restrictions for users(consumers) who are defined in namespaces that include hyphen(-) in their name do not work

Current Behavior
let's say we define the "consumer1" in namespace "aaa" . the restriction rules (method, apikey) in apisixRoute CRD  like the following works as expected:
- name: consumer-restriction
      enable: true
      config:
        allowed_by_methods:
        - user: "aaa_consumer1"
          methods:
          - "GET"

user is able to send the GET request, but not other methods (as expected)
GET:
curl https://httpbin.apisix.<DOMAIN>/anything -H 'apikey: <consumer1_key>' -i
HTTP/2 200
Post:
curl https://httpbin.apisix.<DOMAIN>/anything -H 'apikey: <consumer1_key>' -d ' ' -i
HTTP/2 403

but as soon as we define the consumer in a namespace like "aaa-bbb" that includes hyphen symbol in it's name, the rule doesn't work as we expect:
- name: consumer-restriction
      enable: true
      config:
        allowed_by_methods:
        - user: "aaa-bbb_consumer1"
          methods:
          - "GET"

user has full access on any method:
GET:
curl https://httpbin.apisix.<DOMAIN>/anything -H 'apikey: <consumer1_key>' -i
HTTP/2 200
Post:
curl https://httpbin.apisix.<DOMAIN>/anything -H 'apikey: <consumer1_key>' -d ' ' -i
HTTP/2 200

Expected Behavior
No response
Error Logs
No response
Steps to Reproduce

Run apisix-ingress-controller using helm chart (chart version: 0.9.3, appVersion: 1.4.1) --> latest version on helm repo
create the same consumer once in "aaa" and once in "aaa-bbb" namespaces
create a route to httpbin service and enable  plugin
add restriction rules like allowed_by_methods, whitelist and blacklist for the specific user
check the behaviour of consumer in namespace with hyphen include with the other one

Environment

APISIX version : 2.14.1 , Image: 2.15.0-alpine
Operating system : Linux
OpenResty / Nginx version : openresty/<IP_ADDRESS>

please add a test case to cover this one, thanks @AlinsRan   @lingsamuel

e2e: https://github.com/apache/apisix-ingress-controller/blob/master/test/e2e/suite-plugins/suite-plugins-security/consumer-restriction.go#L215
ns: https://github.com/apache/apisix-ingress-controller/blob/master/test/e2e/scaffold/scaffold.go#L38
This scenario has been covered.
In e2e, the default ns format is ingress-apisix-e2e-tests-xxx-xxx.

@hesamhamdarsi please check the reply 👆

Hi again,
The scenario @AlinsRan is shared have a section where all "-" will be replaced by "_" (in apisixConsumer probably because of the tracking consumers to they key, user/pass, etc) and then you injected this automatically in your unit test.
https://sourcegraph.com/github.com/apache/apisix-ingress-controller@master/-/blob/pkg/types/apisix/v1/types.go?L538#tab=references
So in your unit test, the namespace format is automatically aaa_bbb, while in reality we have to manually specify the namespace.
if you want to have a real test case, I suggest using statically defined namespace in apisixRoute test.
so instead of this test case:
plugins:
   - name: consumer-restriction
     enable: true
     config:
       whitelist:
       - "%s"

use this one manually (like in real scenario) and see if it works:
plugins:
   - name: consumer-restriction
     enable: true
     config:
       whitelist:
       - ingress-apisix-e2e-tests-xxx-xxx_ jack1

Yeah, the following configurations are actually used in e2e:
plugins:
   - name: consumer-restriction
     enable: true
     config:
       whitelist:
       - ingress_apisix_e2e_tests_xxx_xxx_jack1

In fact, the expectation should be as follows:
plugins:
   - name: consumer-restriction
     enable: true
     config:
       whitelist:
       - ingress-apisix-e2e-tests-xxx-xxx_jack1

In fact this is because APISIX does not allow consumer usernames to contain -
https://github.com/apache/apisix/blob/1aa45caf2fa59e5f7add70db0439eda20cf59eb8/apisix/schema_def.lua#L713
We first need to modify the schema in APISIX


Incompatibility with Pehkui?

Here's the crash report
I'm using the [1.16.5] [Forge] Pehkui 2.0.0 version. A few data-packs depend on it to add certain origins. I'm not entirely sure if it's your mod causing it or not, I'm not too good with crash reports.

This looks more like an issue from the mod performant. Try disabling it for testing and check if the bug still occurs.

That did it, my bad. I do seem to have problems with the data-packs after installing Pehkui but I'm not sure if its a problem on my end or not

If you have a specific datapack that causes issues, please write attach the datapack with the issue.

Fixed with the latest version, cheers


More info dialogs styles

This pull request adds new options to hide elements inside the more-info dialogs of Home Assistant.
Config Options

Config Option
Type
Default
Description

hide_dialog_header_action_items
Boolean
false
Hides all the action items from the header of more-info dialogs.

hide_dialog_header_history
Boolean
false
Hides the "History" icon in the header of more-info dialogs.

hide_dialog_header_settings1
Boolean
false
Hides the "Settings" icon in the header of more-info dialogs.

hide_dialog_header_overflow1
Boolean
false
Hides the top right overflow menu in the header of more-info dialogs.

hide_dialog_history
Boolean
false
Hides the "History" section in the more-info dialogs.

hide_dialog_logbook
Boolean
false
Hides the "Logbook" section in the more-info dialogs.

hide_dialog_attributes
Boolean
false
Hides the "Attributes" section in the more-info dialogs.

hide_dialog_media_actions
Boolean
false
Hides the actions block in the more-info dialogs of media-player entities.

hide_dialog_update_actions
Boolean
false
Hides the actions block in the more-info dialogs of update entities.

hide_dialog_timer_actions
Boolean
false
Hides the actions block in the more-info dialogs of timer entities.

hide_dialog_climate_actions
Boolean
false
Hides the actions block in the more-info dialogs of climate entities.

hide_dialog_history_show_more
Boolean
false
Hides the "Show more" link in the "History" section of more-info dialogs.

hide_dialog_logbook_show_more
Boolean
false
Hides the "Show more" link in the "Logbook" section of more-info dialogs.

1 These elements are not visible by default if the account is not an admin account.

Options through screenshots

More info dialog (media entity)

Options
Hides

hide_dialog_header_action_items
12

hide_dialog_header_history
13

hide_dialog_header_settings
14

hide_dialog_header_overflow
15

hide_dialog_media_actions
16

More info dialog (climate entity)

Options
Hides

hide_dialog_climate_actions
17

More info dialog (update entity / timer entity)

Options
Hides

hide_dialog_update_actions
18

hide_dialog_timer_actions
18

More info dialog (sensor entity)

Options
Hides

hide_dialog_history
19

hide_dialog_logbook
20

hide_dialog_attributes
21

hide_dialog_history_show_more
22

hide_dialog_logbook_show_more
23

Closes #93

Hi @NemesisRE,
Even if these new options should not break the main kiosk functionality, I would release this as a major version.


Why Applicative_Fun instance is local?

We have Local Instance Applicative_Fun in ExtLib/Data/Fun.v. This makes it undiscoverable during typeclass search and I have to write:
liftA2 (AT:=(Applicative_Fun _)) f x y
instead of just:
liftA2 f x y

Let's try to make it global. I think I was potentially worried about it unifying too many places and therefore being difficult to use.

I am tyring to make PR. FYI master branch does not compile:
$ make
Makefile:3: coqdocjs/Makefile.doc: No such file or directory
make: *** No rule to make target 'coqdocjs/Makefile.doc'.  Stop.

the file is indeed missing.

what about Functor_Fun and the rest of the instances in Fun.v?


Update eslint

Our last eslint + airnbnb config update cost us 8000+ code lines change ;-) For some files history becomes useless ;-)


Fix output folder

This fix uses the output folder defined in the config file.

But it is probably a breaking change for people...

Would it not be better to parameterise the output folder? Something like
pelican ${PELICAN_CONTENT_FOLDER:=content} -o ${PELICAN_OUTPUT_FOLDER:=output} -s ${PELICAN_CONFIG_FILE:=publishconf.py}

After that we can add override paramets in action.yml. This will ensure people who are already using the action are not affected as well.

Scratch that. I missed your comment about config file.
This will not be breaking change for existing users as the marketplace action is tagged to a particular version.


Getting started

Would be nice to have a "hello world" example to start. First on the (posix) host like any Unix system. Then an embedded bare-metal version.

Added some docs on running a simple example on POSIX and on Zephyr


Feature/add views

Project requirements

Implement the design from the sneak peek wireframes.

thank you @codepantha


Update ash to 0.34

Again, not super sure what I'm doing here, but I thiink this is the right thing to do.
Checked updating this in rust-gpu (which is where I ran into ash-molten not supporting ash 0.34), and it seems to work.

You may want to wait a little bit though, we're probably releasing https://github.com/MaikKlein/ash/pull/526 very soon - it doesn't change anything to ash-molten vk though as you already properly use default-features=false :tada:

Release 0.35 has been cut!

Nice! Just updated this PR to reference 0.35 instead.


Hytale Tracker

Hytale is an upcoming game, I'm making a website that tracks the latest news and blogposts from the developer's api. I need to use cors.sh to get information from their blog api.
https://github.com/hugovdev/hytale-tracker-astro

Hi. Can you email me so that I can send you an API Key?
Here is my email address<EMAIL_ADDRESS>Thank you for applying.


Document the r string interpolator

I'd like to have some documentation about the very useful r interpolator.
Example use in the REPL:
"hola" match {
    case r"h$j" => j
    case _ => "nothing"
  }

But I don't know how to use it in my own projects.  I find it very useful.

Finally, I've found it in:
https://github.com/lihaoyi/Ammonite/blob/4919dcc2d2046e1083cb984d2e7c80ba99dfde29/ops/src/main/scala/ammonite/ops/Extensions.scala

Probably moot now that ammonite-ops is EOLed