chore(main): release mise 1.16.0

:robot: I have created a release beep boop
1.16.0 (2024-06-26)
Features

fzf: add in-file-search function with vim integration (0ee3f5e)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

mise-v1.16.0
:sunflower:


generating recommendations from metric profile

Description
Please describe the issue or feature and the summary of changes made to fix this.
Fixes # (issue)
Type of change

[ ] Bug fix
[ ] New feature
[ ] Docs update
[ ] Breaking change (What changes might users need to make in their application due to this PR?)
[ ] Requires DB changes

How has this been tested?
Please describe the tests that were run to verify your changes and steps to reproduce. Please specify any test configuration required.

[ ] New Test X
[ ] Functional testsuite

Test Configuration

Kubernetes clusters tested on:

Checklist :dart:

[ ] Followed coding guidelines
[ ] Comments added
[ ] Dependent changes merged
[ ] Documentation updated
[ ] Tests added or updated

Additional information
Include any additional information such as links, test results, screenshots here

@msvinaykumar - Can you please resolve the conflicts.

Duplicate of Shreya PR


Arm NN Inference type

What is the inference type of the TFLite model deployed through the Arm NN?  Is it the same as the TFLite model converted from the TF? For example, the TFLite model converted from TF is float-point32, and the TFLite model deployed through the Arm NN is also float-point32.

Hey @ivs-pychen
yes, that's right :+1:  An fp32 model in TfLite is deployed as fp32 model in ArmNN


Load modules from /usr/lib/memgraph/query_modules

Cannot use the transformation module if it is placed in /usr/lib/memgraph/query_modulesfolder

Has been fixed. Memgraph will now look for modules in /usr/lib/memgraph/query_modules and /transformations.


Add GRPC API support

What type of PR is this?
/kind feature
What this PR does / why we need it:
This allows us to create custom data transferred between containers
within SPOD daemonset pods. For now the API is not being exposed,
whereas the server listens only on localhost.
One use case of this interface is to update the metrics from different
containers, for example from the log enricher.
Which issue(s) this PR fixes:

None
Does this PR have test?
None

Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

None

/retest

/test pull-security-profiles-operator-test-e2e

/lgtm


Loss of Energy Controls - Error fetching Teslemetry Energy Site Info XXXXXX data: Multiple 5xx failures

Checklist

[X] I have filled out the issue template to the best of my ability.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] This issue is not a duplicate issue of any previous issues..

Describe the issue
As reported in Discord the Tesla site is returning 500 status errors, which results in the Energy controls going unavailable.

I'm wondering if a delay between calling for live_status & site_info may smooth things out a bit?
Reproduction steps
Config log attached.
config_entry-teslemetry-9148783ba548ef27e70b73108cd5fd97 (9).json
Debug logs
2024-11-08 16:04:29.800 DEBUG (MainThread) [tesla_fleet_api] Requested: https://na.teslemetry.com/api/1/energy_sites/2252097574301386/live_status
2024-11-08 16:04:29.801 DEBUG (MainThread) [tesla_fleet_api] Response Status: 200
2024-11-08 16:04:29.801 DEBUG (MainThread) [tesla_fleet_api] Response JSON: {'response': {'solar_power': 1961, 'percentage_charged': 100, 'battery_power': -200, 'load_power': 6130, 'grid_status': 'Active', 'grid
_power': 4369, 'generator_power': 0, 'island_status': 'on_grid', 'storm_mode_active': False, 'timestamp': '2024-11-08T16:04:29+10:00', 'wall_connectors': [{'din': '1529455-02-E--PGT22325087207', 'wall_connector_
state': 2, 'wall_connector_fault_state': 2, 'wall_connector_power': 0, 'ocpp_status': 1, 'powershare_session_state': 1}, {'din': '1529455-02-E--PGT22325086977', 'vin': '5YJ3F7EC5LF698201', 'wall_connector_state'
: 4, 'wall_connector_fault_state': 2, 'wall_connector_power': 0, 'ocpp_status': 1, 'powershare_session_state': 1}]}}
2024-11-08 16:04:29.802 DEBUG (MainThread) [custom_components.teslemetry] Finished fetching Teslemetry Energy Site Live<PHONE_NUMBER>301386 data in 0.672 seconds (success: True)
2024-11-08 16:04:43.126 DEBUG (MainThread) [tesla_fleet_api] Requested: https://na.teslemetry.com/api/1/energy_sites/2252097574301386/site_info
2024-11-08 16:04:43.126 DEBUG (MainThread) [tesla_fleet_api] Response Status: 500
2024-11-08 16:04:43.128 DEBUG (MainThread) [tesla_fleet_api] An error occurred while processing the request.
2024-11-08 16:04:43.129 ERROR (MainThread) [custom_components.teslemetry] Error fetching Teslemetry Energy Site Info<PHONE_NUMBER>301386 data: Multiple 5xx failures
2024-11-08 16:04:43.130 DEBUG (MainThread) [custom_components.teslemetry] Finished fetching Teslemetry Energy Site Info<PHONE_NUMBER>301386 data in 5.240 seconds (success: False)
2024-11-08 16:04:45.335 DEBUG (MainThread) [custom_components.teslemetry] Streaming received vehicle_data from 5YJ3F7EC5LF698201
2024-11-08 16:04:45.335 DEBUG (MainThread) [custom_components.teslemetry] Manually updated Teslemetry Vehicle 5YJ3F7EC5LF698201 data
2024-11-08 16:04:47.385 DEBUG (MainThread) [custom_components.teslemetry] Streaming received state from LRWYHCFS9PC851970
2024-11-08 16:04:47.385 DEBUG (MainThread) [custom_components.teslemetry] Manually updated Teslemetry Vehicle LRWYHCFS9PC851970 data

Hey Mark do you have any evidence that other services work when Teslemetry doesnt? I am thinking I'll need to start running a test with Tessie, Teslemetry, and the Fleet API directly to see if there is a correlation.
Here is the number of status 500 messages for energy sites over the last 7 days and the number of users it impacted. A small number of people constantly have issues, but there are also spikes.

Ongoing issue. I mainly use teslemetry for my controls exclusively so haven't noticed the effects on any other platforms.
Is there user ability to reduce or back off the numbers of requests being sent in my behalf?
This error originated from a custom integration.

Logger: custom_components.teslemetry
Source: helpers/update_coordinator.py:414
integration: Teslemetry
First occurred: November 14, 2024 at 7:29:18 PM (77 occurrences)
Last logged: 5:30:43 AM

Error fetching Teslemetry Energy Site Info<PHONE_NUMBER>301386 data: Multiple 5xx failures

This isdue is only for energy sites, so Tessie is not relevant.
Here are some comparisons, both are going unavailable at similar times.
Entity display from Teslemetry:

Entity display from Tesla Fleet:

Hey Mark,
Updates are run every 30 seconds, you can take control of this as such:

On the Integration entries page, you can click the three dots besides your Teslemetry config entry and select system options. Here you can disable polling for updates. You would then need to setup your own automation to update an entity from each coordinator on whatever schedule you want.

There are three energy coordinators (info, live and history) so if you need help identifying a entity from each let me know.
I am now running a test between Teslemetry, Tessie, and Tesla Fleet API directly to try and identify if I can replicate your issue and isolated it to Teslemetry or not. (Tessie does support the whole Fleet API including energy)

This is your requests over the last 24 hours:

And this is mine:

You are making more POST requests to change things than me, so I may need to start changing things randomly to see if that contributes.
I will also turn up the logging on 5xx errors to see if we can learn something new.

Its probably no help, but you have more errors on the energy APIs than any other Teslemetry user. Today you have been 45% of all 5xx errors on the energy APIs. Every single one, was a response from Tesla saying upstream internal error.
My only two theories are: there is a connectivity issue to your Powerwall, or because you are frequenty changing your configuration, the powerwall itself is unable to respond to API requests at times.
I will continue to monitor.

I do seem to have a lot of devices and entities, but I would image that isn't part of the root cause..:

George (Model Y) was a hire car I haven't had access to for 6 months, but I don't seem to be able to delete..

Does it still show up in your Tesla app or products endpoint? There is a seperate issue where I need to do device cleanups.

Looks like NetZero is seeing increased 5xx errors as well:
https://github.com/netzero-labs/netzero/discussions/55#discussioncomment-11353609

Looks like some correlation when my sensors are unavailable and the Tesla app reports 3rd party updates.


[SPARK-18674][SQL] improve the error message of natural join

What changes were proposed in this pull request?
The current error message of USING join is quite confusing, for example:
scala> val df1 = List(1,2,3).toDS.withColumnRenamed("value", "c1")
df1: org.apache.spark.sql.DataFrame = [c1: int]

scala> val df2 = List(1,2,3).toDS.withColumnRenamed("value", "c2")
df2: org.apache.spark.sql.DataFrame = [c2: int]

scala> df1.join(df2, usingColumn = "c1")
org.apache.spark.sql.AnalysisException: using columns ['c1] can not be resolved given input columns: [c1, c2] ;;
'Join UsingJoin(Inner,List('c1))
:- Project [value#1 AS c1#3]
:  +- LocalRelation [value#1]
+- Project [value#7 AS c2#9]
   +- LocalRelation [value#7]

after this PR, it becomes:
scala> val df1 = List(1,2,3).toDS.withColumnRenamed("value", "c1")
df1: org.apache.spark.sql.DataFrame = [c1: int]

scala> val df2 = List(1,2,3).toDS.withColumnRenamed("value", "c2")
df2: org.apache.spark.sql.DataFrame = [c2: int]

scala> df1.join(df2, usingColumn = "c1")
org.apache.spark.sql.AnalysisException: USING column `c1` can not be resolved with the right join side, the right output is: [c2];

How was this patch tested?
updated tests

cc @rxin @gatorsmile

Test build #69481 has started for PR 16100 at commit 9461839.

This is a good fix! Will review it today.

Test build #69481 has finished for PR 16100 at commit 9461839.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class UsingJoin(tpe: JoinType, usingColumns: Seq[String]) extends JoinType

Merged build finished. Test PASSed.

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

LGTM

Merging to master and 2.1. Thanks!


[tests-only][full-ci]add test for antivirus user group sharing

Description
This PR adds the API tests for user group link share with antivirus service. The scenarios added in this PR are

upload a file with virus from user group sharing
upload a file with virus from password protected user group sharing

Related Issue

Part of: https://github.com/owncloud/ocis/issues/6255

Motivation and Context

there was no test coverage for user group link share with antivirus service. so, this PR covers the require scenario test case

How Has This Been Tested?

test environment:
locally
CI

Types of changes

[ ] 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 change)
[ ] Technical debt
[x] Tests only (no source changes)

Checklist:

[ ] Code changes
[ ] Unit tests added
[x] Acceptance tests added
[ ] Documentation ticket raised:

E2E tests failed: https://drone.owncloud.com/owncloud/ocis/23157/90/1
:boom: To see the trace, please open the link in the console ...

npx playwright show-trace https://cache.owncloud.com/public/owncloud/ocis/23157/tracing/public-link-for-space-alice-2023-6-9-11-59-44.zip

@nabim777 address the above review and please rebase with ocis master and push. I can approve after that


Fix wrong ogr import in tests

Fix the ogr import in conftest.

Discovered when testing https://github.com/packit-service/ogr/pull/236

@jpopelka thanks


Update set-up-anti-phishing-policies.md

updated the documentation for more emphasis of the first contact safety tip as it is a strong recommendation and adoption is increasing drastically (~10K tenants so far)

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks


Relationship classification ON custom dataset

hi yakumada,
greeting,
are trying to perform relation extraction on custom dataset. we are trying to generate Stanford_Ner, Stanford_pos, Stanford_deprel, Stanford_head for the dataset using Stanford core nlp. could you help us how to generate these values.
thank you.

Hi, thank you for opening the issue. However, the example code is provided for reproducing experiments reported in our paper. Therefore, you need to modify the example code to support custom dataset.

can we just use subject, sub-type, sub_start, sub_end, object, object_type, obj_pos,  for the model instead of using all the 8 columns that are mentioned above. (Stanford core Nlp)

Hi, thank you for your reply! I do not know the details of the Stanford CoreNLP. Please refer to the corresponding code for its detailed specification. The example code corresponding to the relation classification task is available here. I think if you convert your dataset to the same format as TACRED, you can do inference based on the dataset.

I close this issue as there is no activity.


Add upload image functionality in image space

Fixes #111

@kdodia can you take a look? There are two issues that still need to be solved:

get the redirect working here: https://github.com/ContinuumIO/memex-explorer/pull/188/files#diff-b5099d1f0106025f31781ccbe0a59d13R614

and fix some bugs on where to find the location of the images in the compare of uploaded files:


HDDS-7156: Reset pending delete block count

What changes were proposed in this pull request?
Reset pending delete block count to 0 if a container has no transactions Each container has a counter that tracks the number of pending delete blocks in the delete_txns table for that container. In some cases, this counter can remain positive when all delete transactions are cleared, and the counter will never be reset. This means it will continue to be chosen for deletion instead of other containers that actually have delete txns.
After this change, if a container is chosen for deletion because it has

0 pending delete block count but upon deleting from the container, it
is found that there are actually no delete txns, reset the pending delete block counter to 0. This way it will not be chosen for block deletion until it gets more delete transactions.

A new unit test for TestBlockDeletingService was added, and the whole TestBlockDeletingService suite passes.
Author: Ethan Rose
Change-Id: I43e6815962f2db83a95a396020cfa6b7195a8f5f
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-7156
How was this patch tested?
Unit tests included

This is related to and duplicated https://github.com/apache/ozone/pull/3703

LGTM, but I technically wrote these so someone else should officially +1.

+1 LGTM. Feel free to commit once CI comes back clean.

Sorry for late reply, LGTM.


EHRbase-Docker-Container can't be terminated with Ctrl+C

Configuration information

Docker-Container ehrbase/ehrbase:0.19.0 and ehrbase/ehrbase:next
Operating System: Linux
Environment: docker

Steps to reproduce
Start ehrbase-Database and ehrbase as docker-containers (not with docker-compose)
Try to terminate the ehrbase-container with Strg+C
docker network create ehrbase
docker run --rm --name ehrbasedb --network ehrbase -e POSTGRES_PASSWORD=postgres -e EHRBASE_USER=ehrbase -e EHRBASE_PASSWORD=ehrbase ehrbase/ehrbase-postgres:13.4

# open a new console because the previous one is blocked
docker run --rm --name ehrbase --network ehrbase -p 8088:8080 -e SPRING_DATASOURCE_URL=jdbc:postgresql://ehrbasedb:5432/ehrbase -e SPRING_DATASOURCE_USERNAME=ehrbase -e SPRING_DATASOURCE_PASSWORD=ehrbase -e SERVER_AQLCONFIG_USEJSQUERY=false ehrbase/ehrbase:0.19.0

Press Ctrl+C

Actual result
The ehrbase-container is not terminated. You just see ^C
The ehrbase-postgres container can be terminated by pressing Ctrl+C
Expected result (Acceptance Criteria)
The ehrbase-container is terminated
Definition of Done
The EHRbase container can be terminated with Ctrl+C
Maybe the root cause is described in [1] and using the recommended json-syntax [2] can solve the issue. But that's currently just a guess.
[1] https://engineeringblog.yelp.com/2016/01/dumb-init-an-init-for-docker.html#process-behavior-inside-docker-containers
[2] https://docs.docker.com/engine/reference/builder/#run

I’ve seen this happen with other containers also on some docker versions on some operating systems.
Can you verify if it’s only the docker image or if it happens with all images?

It is only the ehrbase-image. Other images (e.g. the ehrbase-postgres-image) terminate with Ctrl+C
Maybe it is just enough to change the last line in the docker-file with CMD [./docker-entrypoint.sh] or "/bin/bash", "-c", "./docker-entrypoint.sh"
But as I said, it's just a guess.

I can confirm this behaviour (also only at the ehrbase image).
docker -v
Docker version 20.10.12, build e91ed57

sw_vers
ProductName:	macOS
ProductVersion:	12.3
BuildVersion:	21E230

Normally if you want to terminate a container process using ctrl + c you have u add -t -i to the docker run command.
Here an example:
docker run -t -d ubuntu /usr/bin/top -b
Control-C
The container is still listed
If you run:
docker run -t -i -d ubuntu /usr/bin/top -b
Control-C
The container is not there
I've tried with the ehrbase container and it worked

Thanks, with -t -i it works. But other containers work without it (actually I never used it before). That's why I think a change would be possible.
But for me it has low priority.

Hello All,
As per @agustinpallares94 , I see that solution was provided already.
Closing ticket.
CC: @vidi42


Adaptive Cards for Web Chat

I've tried implementing Adaptive Cards based on this. I could see that Activity supports attachments parameter and accepts inputs as shown in the above link to implement Adaptive Cards. I could not understand the contentUrl though. I've filled in the remaining fields and tried this. i'm encountering the below issue.
My Activity
Activity(
        type=ActivityTypes.message,
        channel_id=request_activity.channel_id,
        conversation=request_activity.conversation,
        recipient=request_activity.from_property,
        from_property=request_activity.recipient,
        text=text,
        service_url=request_activity.service_url,
        attachments=Attachment(
            content_type="application/vnd.microsoft.card.adaptive",
            content={
                "type": "AdaptiveCard",
                "version": "1.0",
                "body": [
                    {
                    "type": "TextBlock",
                    "text": "Hello World!",
                    "size": "large"},
                    {
                    "type": "TextBlock",
                    "text": "*Sincerely yours,*"},
                    {
                    "type": "TextBlock",
                    "text": "Adaptive Cards",
                    "separation": "none"}],
                    "actions": [
                    {
                        "type": "Action.OpenUrl",
                        "url": "http://adaptivecards.io",
                        "title": "Learn More"
                    }
                    ]}),)

Error Log

ERROR:root:Error is:
Traceback (most recent call last):
File "D:\home\site\wwwroot\main.py", line 150, in messages
app.connector.conversations.send_to_conversation(activity['conversation']['id'], response_activity)
File "D:\home\python364x64\lib\site-packages\botframework\connector\operations\conversations_operations.py", line 224, in send_to_conversation
body_content = self._serialize.body(activity, 'Activity')
File "D:\home\python364x64\lib\site-packages\msrest\serialization.py", line 473, in body
errors = _recursive_validate(data_type, data_type, data)
File "D:\home\python364x64\lib\site-packages\msrest\serialization.py", line 124, in _recursive_validate
return data.validate()
File "D:\home\python364x64\lib\site-packages\msrest\serialization.py", line 184, in validate
validation_result += _recursive_validate(attr_name, attr_type, value)
File "D:\home\python364x64\lib\site-packages\msrest\serialization.py", line 116, in _recursive_validate
for content in data:
TypeError: 'Attachment' object is not iterable

My target is to have a card with a button. I've already implemented the SuggestedActions with buttons, but I need the button to be always available just in case the user wants to visit the link again. Is there any issue with the way I implemented this? Any samples towards this will be helpful.

I should've googled better. @stevengum97 has already provided some guidance here


passing dns resolvers out of a plugin

We need to define the where in the returned JSON metadata a plugin can tell the caller what the DNS configuration should be. This will essentially be a priority ordered list of IP addresses:
"dns-resolvers": ["<IP_ADDRESS>", "<IP_ADDRESS>"]

I can think of three appropriate ways of providing nameservers:

entirely out-of-band with respect to CNI (e.g., if Kubernetes runs its own DNS server);
in the configuration of a network;
supplied by an IPAM plugin.

It's a bit of a thorny one though, because it's very sensitive to the configuration of other components (e.g., you want the nameserver to supply addresses you can reach); and, you can't compose answers, so you usually have to pick one.

@squaremo see #76


self links for jsonapi_resource routes

It appears that the link generation for resources that are singleton resources (jsonapi_resource) are not generating correctly and instead it produces a link using jsonapi_resource rules. For example I would expect a self link in the singlton case to be:
http://example.com/api/myobject

but I am instead getting
http://example.com/api/myobjects/123

Looking through the source it appears this is not handled, am just missing it? Thanks!

This comes from self_link, which in turn calls resource_path.
resource_path checks if the resource is a singleton resource, but not how it's declared in the routes.
Perhaps this can be solved with something like red._single_routed = true in the routing_ext.rb, and then have the resource_path method check for that to determine how to correctly link?


Release Process

Tracking issue for AwsCommunity registry extension publishing
Let's use this issue as a place to discuss our release process. We will not actually release any extensions publicly until the process has been approved by appsec. We are open to feedback and suggestions!
See RELEASE.md

https://aws.amazon.com/blogs/devops/reducing-docker-image-build-time-on-aws-codebuild-using-an-external-cache/


Update to 1.20.2

Hello, are there plans to update the mod for 1.20.2?

Hello; there are plans to update to Minecraft 1.20.2 once Quilted Fabric API is updated itself


QNX: Build fails, issue in src/unix/qnx.c

Hello, I'm trying to port libuv to QNX 8.0, but I ran into some issues in the src/unix/qnx.cc file.
Specifically, I get the following error when building:
In file included from /home/avchen/libuv/src/unix/qnx.c:29:
/home/avchen/libuv/src/unix/qnx.c: In function 'uv_cpu_info':
/home/avchen/libuv/src/unix/qnx.c:128:57: error: 'struct syspage_entry' has no member named 'new_cpuinfo'; did you mean 'cpuinfo'?
  128 |     (struct cpuinfo_entry*)_SYSPAGE_ENTRY(_syspage_ptr, new_cpuinfo);

So I was wondering what the new_cpuinfo member  in the function int uv_cpu_info(uv_cpu_info_t** cpu_infos, int* count) is coming from. It is not a member present in  he syspage_entry structure, at least when I've looked into the syspage.h file, it has cpuinfo instead.
I've tried changing this line:
struct cpuinfo_entry* cpuinfo =
    (struct cpuinfo_entry*)_SYSPAGE_ENTRY(_syspage_ptr, new_cpuinfo);

to
struct cpuinfo_entry* cpuinfo = SYSPAGE_ENTRY(cpuinfo);

and also
struct cpuinfo_entry* cpuinfo =
    (struct cpuinfo_entry*)_SYSPAGE_ENTRY(_syspage_ptr, cpuinfo);

Both changes ends up building but I was wondering if it accomplishes the same thing as having new_cpuinfo and where is new_cpuinfo coming from?

Version:  v1.46.0
Platform:
❯ uname -a
Linux BB78327197 5.15.0-76-generic #83~20.04.1-Ubuntu SMP Wed Jun 21 20:23:31 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Target Platform: QNX 8.0 aarch64le and x86_64
Compiler: The C compiler identification is QCC 12.2.0

Maybe @Arenoros or @mjvankampen know, they contributed the qnx port.


bugfix: pr gaurdrails fix

Description
Resolves #75

Please fix code linting issues in next pull request @PritamIT2023


IndexSizeError in /admin/communication/blocks/cb6d002d-c564-40da-be54-91c4e481a915/edit

Error in Osuny
IndexSizeError in /admin/communication/blocks/cb6d002d-c564-40da-be54-91c4e481a915/edit
The index is not in the allowed range.
View on Bugsnag
Stacktrace
../plugin-inline-script-content/inline-script-content.js:149 -

View full stacktrace
Created automatically via Bugsnag

An error linked to this issue has been reopened in Bugsnag
IndexSizeError in /admin/communication/blocks/2b71c422-1b41-47fc-817c-b4f9038ed0a5/edit

An error linked to this issue has been reopened in Bugsnag
IndexSizeError in /admin/pt/communication/blocks/0e9786f4-31a1-4483-a66a-86c97cb93588/edit

An error linked to this issue has been reopened in Bugsnag
TypeError in /admin/en/communication/blocks/0db7a2bb-c064-4055-b788-27108a516c40/edit


Bump timeout to 5400

This is to help deal with slow nodes.
Signed-off-by: Paul Belanger<EMAIL_ADDRESS>

Pull-request updated, HEAD is now dc0e91822e295948ffdb987034b11f406397ad4f


Update choose-aspnet-framework.md

part of #3551
review URL

@tdykstra  I added a note to #3545 to update the link when it's complete (hopefully before Aug 7th). I don't want to send folks to the MVC series when we should be sending them to Razor Pages. The Razor Pages is almost a scenario. It's not perfect but should work until I complete #3545
See my other change.


Improve cache cleanup log message

Change phrasing of log message about last cleanup to sound less like it's a call to action, and demote it to debug level.

Build: https://builds.gradle.org/viewQueued.html?itemId=11196172

Thanks @lptr .


Allow environment to inject base node ports

Currently the node port numbers are derived by adding a node's ordinal identifier to a static constant.

E.G. $CCTL_BASE_PORT_RPC + node_idx  where CCTL_BASE_PORT_RPC=11000 and <=1 node_idx <= 10.

There is a requirement to allow the deployment environment to override the static constant.  This can be achieved via the use of an environment variable per port type.  If the env-var is declared then it will be utilised, otherwise the default constant is utilised.

Adding support for the following env vars that override the static constant:

CSPR_BASE_PORT_NETWORK
CSPR_BASE_PORT_BINARY
CSPR_BASE_PORT_SPEC_EXEC
CSPR_BASE_PORT_REST
CSPR_BASE_PORT_RPC
CSPR_BASE_PORT_SSE


Debug and Prod | Push Notification

Debug!!
and
Prod!!
that's all!!

Pull Request Test Coverage Report for Build 142

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 0.096%

Totals

Change from base Build 140:
0.0%

Covered Lines:
1

Relevant Lines:
1040

💛  - Coveralls


Multiple exclusions?

Hello,
I'm trying to use this plugin in a CI environment to fail when my dependencies are outdated. However, I have exceptions that I would like to ignore.
This is my current output:
$ yarn outdated --check
➤ YN0000: ┌ Checking for outdated dependencies
➤ YN0000: └ Completed in 47s 653ms

➤ YN0000: Package                 Current   Latest   Package Type
➤ YN0000: @pinia/testing          0.0.13    0.0.16   devDependencies
➤ YN0000: @vitejs/plugin-vue2     1.1.2     2.2.0    devDependencies
➤ YN0000: @vue/test-utils         1.3.5     2.3.2    devDependencies
➤ YN0000: start-server-and-test   1.15.4    2.0.0    devDependencies
➤ YN0000: vue                     2.7.14    3.2.47   devDependencies
➤ YN0000: vue-router              3.6.5     4.1.6    devDependencies
➤ YN0000: vue-tsc                 1.3.8     1.6.4    devDependencies

➤ YN0000: 7 dependencies are out of date
➤ YN0000: Failed with errors in 47s 662ms

I would like to exclude both vue and vue-router (I can't upgrade just yet), but I can't seem to find a way to do it. This is what I've tried so far:
Multiple globs. This doesn't seem to work at all:
$ yarn outdated '!vue' '!vue-router' --check
➤ YN0000: ┌ Checking for outdated dependencies
➤ YN0000: └ Completed in 32s 72ms

➤ YN0000: Package                 Current   Latest   Package Type
➤ YN0000: @pinia/testing          0.0.13    0.0.16   devDependencies
➤ YN0000: @vitejs/plugin-vue2     1.1.2     2.2.0    devDependencies
➤ YN0000: @vue/test-utils         1.3.5     2.3.2    devDependencies
➤ YN0000: start-server-and-test   1.15.4    2.0.0    devDependencies
➤ YN0000: vue                     2.7.14    3.2.47   devDependencies
➤ YN0000: vue-router              3.6.5     4.1.6    devDependencies
➤ YN0000: vue-tsc                 1.3.8     1.6.4    devDependencies

➤ YN0000: 7 dependencies are out of date
➤ YN0000: Failed with errors in 32s 82ms

Single glob with list. This works, but for some reason also excludes all the namespaced deps, I'm not sure why:
$ yarn outdated '!(vue|vue-router)' --check
➤ YN0000: ┌ Checking for outdated dependencies
➤ YN0000: └ Completed in 18s 665ms

➤ YN0000: Package                 Current   Latest   Package Type
➤ YN0000: start-server-and-test   1.15.4    2.0.0    devDependencies
➤ YN0000: vue-tsc                 1.3.8     1.6.4    devDependencies

➤ YN0000: 2 dependencies are out of date
➤ YN0000: Failed with errors in 18s 671ms

I'm not really sure what else to try. Is there a way I can use this and ignore multiple outdated deps? Thank you.

This should work.
yarn outdated '@*/*' '!(vue|vue-router)' --check

This plugin uses micromatch, so anything they support should be supported. Namespaced deps are a little weird though since the / makes micromatch think it's a directory.

Thank you, this seems to work well for my use case.


copy edits and update ZOOKEEPER_URI in examples

Fixed a few typos.
Updated ZOOKEEPER_URI in examples to match the defaults created by kk install zookeeper

@tbaums can you please sign off the commit? the DCO check is failing


license: permit no license for non-core taps, add exempted license groups

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

With this pull request, it's OK for non-core taps to either not specify a license or to say that the license is NOASSERTION.
Furthermore, the GitHub license detection system is not perfect. Create groups of permitted licenses where the license will be considered matching if both the GitHub license and the formula license are members of the same groups. Currently this just applies to GPLv2 and GPLv3. cc @Rylan12 and @chenrui333 for feedback on if they've seen other bad license detection.

As far as I know, the GitHub API only returns the identifiers listed here.
From those, it looks like the only issues we'll have are with:

AGPL-3.0 --> AGPL-3.0-only or AGPL-3.0-or-later
GPL-2.0 --> GPL-2.0-only or GPL-2.0-or-later
GPL-3.0 --> GPL-3.0-only or GPL-3.0-or-later
LGPL-2.1 --> LGPL-2.1-only or LGPL-2.1-or-later
LGPL-3.0 --> LGPL-3.0-only or LGPL-3.0-or-later

From those, it looks like the only issues we'll have are with:

AGPL-3.0 --> AGPL-3.0-only or AGPL-3.0-or-later
GPL-2.0 --> GPL-2.0-only or GPL-2.0-or-later
GPL-3.0 --> GPL-3.0-only or GPL-3.0-or-later
LGPL-2.1 --> LGPL-2.1-only or LGPL-2.1-or-later
LGPL-3.0 --> LGPL-3.0-only or LGPL-3.0-or-later

Yeh, that's what I was thinking. If the "key" there is specified: require/allow one of the "values".

Ok, I've tweaked the license matching code against GitHub. This should be ready assuming full tests pass.

Thanks for the reviews!


Remove docs from the main repo

Before doing this, I'll go through PRs to try my best merging docs change.


Reminder

Coverage remained the same at 63.495% when pulling 95b035753f17efd9b22ba7d1d665b0231b28d4a7 on reminder into fed0af14694e90698e4f042c3855c0271a690e8f on master.


update critools

Updates to the latest critools. Uses new critools make install-crictl.
Signed-off-by: Mike Brown<EMAIL_ADDRESS>

@mikebrow You also need to update all crictl sandboxes. It is changed to crictl pods.

Fixes https://github.com/containerd/cri-containerd/issues/534.

You also need to update all crictl sandboxes. It is changed to crictl pods.

Thx will adding a commit for that, was skimming over the prs in cri-tools and got interrupted forgot to finish my skim checking for changes :-)

LGTM with nits

Merge this PR directly. I'll address comments in a commit in https://github.com/containerd/cri-containerd/pull/603.


[Terraform] Add for_each support to prevent false positives

Expected Behavior
Not throwing a Redis Publicly Accessible Vulnerability error when using a for_each on  azurerm_redis_firewall_rule resource
Actual Behavior
Vulnerability Redis Publicly Accessible found on resource azurerm_redis_firewall_rule when using a for_each.
If I remove the  for_each from the azurerm_redis_firewall_rule resource and instantiate with the same IP defined on the redis variable the error will not show up.

Steps to Reproduce the Problem

Instantiate azurerm_redis_firewall_rule resource
Use a for_each to dynamically  create resources
Set start_ip and end_ip to the for_each value using each.value

locals {
  redis_allowed_ips = ["<IP_ADDRESS>"]
}

resource "azurerm_redis_cache" "redis" {
  name                = "redis"
  location            = "location"
  resource_group_name = "resource_group_name"
  capacity            = 1
  family              = "P"
  sku_name            = "Premium"

  redis_configuration {
  }
}

resource "azurerm_redis_firewall_rule" "firewall_rule" {
  for_each = local.redis_allowed_ips

  name                = "redis_firewall_rule_${replace(each.value, ".", "_")}"
  redis_cache_name    = azurerm_redis_cache.redis.name
  resource_group_name = "resource_group_name"
  start_ip            = each.value
  end_ip              = each.value
}

terraform {
  required_providers {
    azurerm = {
      source  = "hashicorp/azurerm"
      version = "=2.86.0"
    }
  }
}

Specifications
(N/A if not applicable)

Version: v1.5.5
Platform: Terraform
Subsystem: N/A

I just noticed that in my example above I used locals but in my project I used variables.
So this issue is not related with the problem.
A new example without locals:
variable redis_allowed_ips{
  type = set(string)
  default = ["<IP_ADDRESS>"]
}

resource "azurerm_redis_cache" "redis" {
  name                = "redis"
  location            = "location"
  resource_group_name = "resource_group_name"
  capacity            = 1
  family              = "P"
  sku_name            = "Premium"

  redis_configuration {
  }
}

resource "azurerm_redis_firewall_rule" "firewall_rule" {
  for_each = var.redis_allowed_ips

  name                = "redis_firewall_rule_${replace(each.value, ".", "_")}"
  redis_cache_name    = azurerm_redis_cache.redis.name
  resource_group_name = "resource_group_name"
  start_ip            = each.value
  end_ip              = each.value
}

terraform {
  required_providers {
    azurerm = {
      source  = "hashicorp/azurerm"
      version = "=2.86.0"
    }
  }
}

Hello @cxAndreFelicidade, thank you for the information, I will be waiting for an update.

@cxAndreFelicidade
I changed the issue title to prevent further duplicates such as #5312

@leandroyou thank you for being so attentive!

Currently we don't have any plans to support this.


Roll Flutter Engine from e6d3bac5c723 to 2c4c1108aa87 (1 revision)

https://github.com/flutter/engine/compare/e6d3bac5c723...2c4c1108aa87
2023-09-25<EMAIL_ADDRESS>Bump archive from 3.3.9 to 3.4.2 in /lib/web_ui (flutter/engine#46253)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jonahwilliams@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Update for getting etl-plugin latest version.

Update the code for getting the latest version of  etl-plugin. Include the parsing logic for the omnia-ivy-releases artifactory repo

Apart from the minor comments nice work! Can you please bump the version number as well.


BUG: error for wrong method

We have to add an exception in case a method is used that is not supported by this package:
for instance this method does not exist
method="insert"

instead of throwing an error, it appends the dataset to the sql database.

Fixed in: https://github.com/zypp-io/df_to_azure/blob/75bad0a4d554b8b0060baee65b4a1c56c3bddbd7/df_to_azure/parse_settings.py#L25-L31


add repo context

Issue
Running the command mentinoed on checklist item one gives below error.
pramodyadav@Pramods-MBP selenide % gradlew check chrome_headless firefox_headless
zsh: command not found: gradlew

Proposed changes
Running this command with current repo context works fine (as below).
pramodyadav@Pramods-MBP selenide % ./gradlew check chrome_headless firefox_headless
Configuration on demand is an incubating feature.

> Task :modules:core:compileJava
Jabel: initialized

Note: This maybe a tiny thing to know for gradle users but it can throw a new gradle user off; unnecessary making them spend time trying to troubleshoot/fix this issue.

Checklist

[ ] Checkstyle and unit tests are passed locally with my changes by running gradlew check chrome_headless firefox_headless command

No they didn't. Considering my current change is a typo fix in a PR template, I think these failures already exist from some other PR from past.

[ ] I have added tests that prove my fix is effective or that my feature works

NA.

[ ] I have added necessary documentation (if appropriate)

NA.

@asolntsev : will try on my windows machine and confirm today.

@asolntsev : It does work on windows. I forgot that powershell now does not open executables by default if provided as a path.
Does that mean, we should update the statement to reflect it for windows/mac users? i.e. run:
for windows: gradlew check chrome_headless firefox_headless
for mac/linux: ./gradlew check chrome_headless firefox_headless

Does that mean, we should update the statement to reflect it for windows/mac users? i.e. run:
for windows: gradlew check chrome_headless firefox_headless

It won't work for windows. Correct one is   .\gradlew.bat foo bar

Does that mean, we should update the statement to reflect it for windows/mac users?

It would be the correct way, yes.
BUT
I don't think we really need such complexity.
I think current wording "by running gradlew check chrome_headless firefox_headless command" is absolutely fine.
Any gradle user know how to run gradle.

@asolntsev : Okay. We can close the PR.
I was trying my hands on gradle for the first time and came across this issue so thought if we can avoid it for others but completely okay to close this PR without a change.


[vNext] Documentation: Fix code samples

Description
This PR fixes broken samples and makes sure that sample serialization is used with System.Text.Json and not Newtonsoft.Json
Type of change

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

Closing issues
Closes #1590
Closes #1593
Closes #1591

Thank you Matias


Outline scenarios with examples are not working as expected

Hello, first of all thank you for this awsome project 💖
👓 What did you see?
I am testing it with outline scenarios with examples. Undefined steps are not showing as undefined steps.
✅ What did you expect to see?
I expected my invalid step to be reported as problem
📦 Which tool/library version are you using?

Reproduced on the Live Demo to remove variables.
🔬 How could we reproduce it?

Visit https://cucumber.github.io/monaco/
Use the feature file below,

@foo
Feature: Hello
  Scenario Outline: Hi
    Given I have <qty> cukes in my belly
    And this is an undefined step

  Examples:
    | day            | answer |
    | Friday         | TGIF   |
    | Sunday         | Nope   |
    | anything else! | Nope   |

📚 Any additional context?

This text was originally generated from a template, then edited by hand. You can modify the template here.

Curious issue @oliverde8! The root cause lies within the Cucumber Language Service: it disables diagnostics for undefined steps if the test is a scenario outline (has an examples table). My reasoning is that this is due to the challenges in ensuring clear and reliable behaviour with respect to examples table lookup to consider whether the values match the step definition. This is in comparison to steps without parameters, where we can pass the step text to a validation function in its raw format.
I have captured an issue detailing proposing this as a feature request (cucumber/language-service/#149). I would greatly appreciate if you could take a look and provide any feedback.

@oliverde8, issue resolved - thanks for raising! Welcome further requests and if you observe any edge cases.


Backport the test functions refactoring

Did you notice that this is completely failing on Jenkins?

Did some poking and this branch is very wrong. It refers to the storage and network code in a lot of places despite none of that existing in LXD 2.0.

I'm looking into the changes, but the jenkins failure is about "Target branch stable-2.0 isn't the master branch."

jenkins: test this please


Feature request: save image with metadata

Hi, thanks for making this ComfyUi addon.
As you mentioned yourself, ComfyUi's metadata is quite a mess. I wasn't able to reconstruct the prompt and settings from an image I made earlier. It seems reconstructing the complete workflow is the only option.
With that being said: would it be possible to add a node to this addon which saves an image with metadata in the selected format? That way, I can first use ComfyUi to reconstruct the image workflow, then add the SaveImageWithMetadata node and select the metadata format. I then have an image I should be able to load using LoadImageWithMetadata.
My goal eventually is to have a folder containing images with just the following metadata:

positive prompt
negative prompt
checkpoint name

This will be enough for me to run an upscaler script for each of the image files in the folder.

Sounds reasonable to have this step added to the initial workflow, as you might have a clear view of what goes into the prompts and we don't need to reconstruct this later by going through the node tree.
I saw this a few days back on reddit, maybe this is already what you need:
https://www.reddit.com/r/comfyui/comments/15pz7ak/node_to_save_image_and_option_to_save_text_prompt/
https://www.reddit.com/r/comfyui/comments/15tjz0b/biggest_let_down_of_comfyui_you_loose_generation/
The later seems to be more 'professional' but i did not have a close look at any of these.

Github link: https://github.com/giriss/comfy-image-saver
Seems to have a minor compatibility issue at the moment. This causes CFG scale to not be readable by my node.

For anyone looking for a all in one solution (save and load nodes) made by the original author of the prompt reader, please use this node collection here:
https://github.com/receyuki/comfyui-prompt-reader-node
I guess this one can be loaded via comfy manager as well.


[4.12] OCPBUGS-6789: make the bootstrap kube-apiserver honor cluster-wide featuregates

this is a backport of https://github.com/openshift/cluster-kube-apiserver-operator/pull/1419
/cc @deads2k @ibihim

/approve
/assign @ibihim
for lgtm

/label cherry-pick-approved


LibGDX, also add [Framework] section

Some reference to the engine, such as the developer site or a wikipedia article.
https://github.com/libgdx/libgdx
SteamDB links
https://steamdb.info/app/646570/
Possible way to detect it
No response
Also, please add [Framework] section for frameworks like LibGDX.

Sections are not special, so they can be added trivially.

Sections are not special, so they can be added trivially.
I don't see libGDX files in Slay the spire, does it have a different name?

You can check it by opening final .jar, there wil be com.badlogic package. But probable there is no files which 100% specific exactly for LibGDX. So you can't detect this framework onlt by filenames )=

Welp.


Appstudio update hacbs-test-project-v4vb

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI has failed.There was an error creating the PipelineRun: hacbs-test-project-v4vb-on-pull-request-
creating pipelinerun hacbs-test-project-v4vb-on-pull-request- in namespace rhtap-demo-cyxb-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request


[Aspire] I can't figure out what Configuration is

The Overview docs mention configuration without really explaining what that is.  Is it related to IConfiguration?  Is it declarative yaml/json/xml files (I don't think so)?  Is it component-specific?
A couple particularly confusing bits:

Applies ServiceBusClient configurations either inline through code or through configuration

capable of flowing their configurations through dependencies

I think this has been fixed, as I cannot find this content.


Non–late-escaping XSS sniff

The XSS sniff that is currently a part of the standards is designed to enforce a philosophy of late escaping—everything should be escaped immediately before output. However, strict adherence to this is currently difficult, especially for themes, because of some patterns used in WordPress core. (Many WordPress functions return a string, rather than echoing it directly, and do not always escape everything that is incorporated into the output.) There are also times, like when reviewing code, where one is mostly interested in detecting low-hanging fruit, not forcing a late-escaping policy on another developer.
To remedy this, a new sniff may need to be introduced, though it is possible that the existing sniff could also be modified to have a less strict "mode". The idea would be to only flag unescaped output that incorporated known-to-be-dangerous values, like $_POST, $_GET, etc.
Related: #748, https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/pull/844#issuecomment-279714754

What about using error for the low-hanging fruit and warning for anything in strict mode?

What about using error for the low-hanging fruit and warning for anything in strict mode?

I think it would depend on the ruleset.
So we will be having a list of three different types of functions.
Insecure function

get_theme_mod
get_options
$_GET
$_POST

Mostly secure functions

get_the_category_list

Totally secure functions

Any non-pluggable function that returns a known limited set of safe static values not directly based on the input arguments

category_description

My last question would be for the totally secure function the data would need to be escaped after the filter and not before?

What about using error for the low-hanging fruit and warning for anything in strict mode?

That is another possible approach. It really doesn't matter to me, it depends on the different applications that people have for this. As @grappler, it might depend on the ruleset.

So we will be having a list of three different types of functions.

Yes, probably something like that.

My last question would be for the totally secure function the data would need to be escaped after the filter and not before?

Yes.

I've been going through the list as well, and so far I've found 2 deprecated functions in the $autoEscapedFunctions list.
Deprecated functions
comments_popup_script
comments_rss_link

bloginfo() is safe as it echoes
echo get_bloginfo( $show, 'display' );

The display filter is safe to use (as commented here).
So far on my list of $autoEscapedFunctions are the following functions
allowed_tags
bloginfo
body_class
calendar_week_mod
category_description
checked
comment_author_IP
comment_form
comments_link
count
delete_get_calendar_cache
disabled
do_shortcode

I'm still going through the list, will update as I check more.

get_bloginfo()  is not valid for $autoEscapedFunctions as the data can be modified via a filter that is why I have added to the Mostly secure functions.
Some of the items on your list conflict with the items on my list. https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/issues/845#issuecomment-279834686

I just checked if they are safe, I didn't check for 'mostly safe', that's why I have these still in the allow :)
And get_bloginfo isn't on the $autoEscapedFunctions list, so we're good there I think.

@dingo-d Regarding deprecated functions being on the lists: IMHO they should be (correctly detected) for the purposes of these lists.
There is a separate sniff to alert about the use of deprecated functions.

@jrfnl So the fact that they are on the list doesn't matter? Seems like a bit of an unnecessary clutter. If we are sorting these out, and have a separate sniff for deprecated functions, I think there's no need for them to be here.

@dingo-d I do not agree. These are two separate issues and should be sniffed for independently of each other. Someone might have either one of these sniffs disabled or use a ruleset which doesn't include one of them (VIP), the other sniff should still function correctly in that case. So deprecated functions should stay in the list.

@JDGrimes @jrfnl I have gone through the current list of $autoEscapedFunctions and have seperated them into different sections. https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/issues/845#issuecomment-279834686
I have just used the functions that we currently have listed. There are some functions missing but I think it would be easiest to add them once we have the changes.
There are a few functions that need to be double checked and a few that need a decision what to do with them.

I have gone through the current list of $autoEscapedFunctions and have seperated them into different sections. #845 (comment)

Looks good. (I haven't double checked any of them yet, though.)

I have just used the functions that we currently have listed. There are some functions missing but I think it would be easiest to add them once we have the changes.

+1.

There are a few functions that need to be double checked and a few that need a decision what to do with them.

comments_popup_script is deprecated and does nothing.

Yeah, might as well remove it.

delete_get_calendar_cache does not have a return value so it should never be echoed

How did it even get on the list? 😄 (Just a rhetorical question, let's remove it.)

do_shortcode_tag is a private function and in core it is not echoed

I don't see why that would ever need to be echoed directly. It's not what it is designed for, and as you say, it is private, so I see no reason to keep in on the list.

My last question would be for the totally secure function the data would need to be escaped after the filter and not before?

Yes. The only exception would be for functions that are specifically escaping functions. For example, esc_html() has a filter, but it is obvious to anything that is using that filter that they need to be escaping whatever they return. For these functions with filters that aren't specifically for applying escaping to a value, they should not have any filter after they escape the data. If they do, then they don't belong on the totally secure function list.


Broken Pipe

I leave one of my services using boringproxy and it went offline (9 hrs) overnight because of broken pipe.

There was no automatic reconnect/handling of this.

There was no automatic reconnect/handling of this.

After I turned off the server version to increase limits, the client stopped working and when I turned the server back on, the client didn't connect automatically again. https://github.com/boringproxy/boringproxy/issues/34#issuecomment-757496422

After I turned off the server version to increase limits, the client stopped working and when I turned the server back on, the client didn't connect automatically again. https://github.com/boringproxy/boringproxy/issues/34#issuecomment-757496422

I have zero idea why I must constantly restart the client, I one http error I receive is
context deadline exceeded (Client.Timeout exceeded while awaiting headers)

Every 10-15 minutes on a site with many requests the client just stops working. No errors, no console output, I just have to restart it.

I have zero idea why I must constantly restart the client, I one http error I receive is
context deadline exceeded (Client.Timeout exceeded while awaiting headers)

Every 10-15 minutes on a site with many requests the client just stops working. No errors, no console output, I just have to restart it.

I would expect something like this from boringproxy:

I would expect something like this from boringproxy:

Yeah the auto-reconnect functionality is currently very limited. The biggest problem I've run into is sometimes if the client crashes, the server doesn't close its side of the SSH tunnels, and the client can't reconnect. Only solution I've found so far is to killall sshd on the server, which is quite annoying. However, this has been much less of an issue since the client is more stable and hasn't crashed for me in a long time.

Yeah the auto-reconnect functionality is currently very limited. The biggest problem I've run into is sometimes if the client crashes, the server doesn't close its side of the SSH tunnels, and the client can't reconnect. Only solution I've found so far is to killall sshd on the server, which is quite annoying. However, this has been much less of an issue since the client is more stable and hasn't crashed for me in a long time.

I tested the client on a site that gets thousands of requests to specific files and hundreds of visitors a day and the client just freezes/dies.
Is there anything I can give you to help workout why this happens? Or do you know why and need time/contribution to fixing it?

From: Anders Pitman<EMAIL_ADDRESS>Sent: 11 January 2021 17:16
To: boringproxy/boringproxy<EMAIL_ADDRESS>Cc: voarsh2<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [boringproxy/boringproxy] Broken Pipe (#33)
Yeah the auto-reconnect functionality is currently very limited. The biggest problem I've run into is sometimes if the client crashes, the server doesn't close its side of the SSH tunnels, and the client can't reconnect. Only solution I've found so far is to killall sshd on the server, which is quite annoying. However, this has been much less of an issue since the client is more stable and hasn't crashed for me in a long time.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/boringproxy/boringproxy/issues/33#issuecomment-758096564, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AONDJP7PGRMSWSHPJJXA473SZMW5RANCNFSM4V4O6HFA.

I tested the client on a site that gets thousands of requests to specific files and hundreds of visitors a day and the client just freezes/dies.
Is there anything I can give you to help workout why this happens? Or do you know why and need time/contribution to fixing it?

From: Anders Pitman<EMAIL_ADDRESS>Sent: 11 January 2021 17:16
To: boringproxy/boringproxy<EMAIL_ADDRESS>Cc: voarsh2<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [boringproxy/boringproxy] Broken Pipe (#33)
Yeah the auto-reconnect functionality is currently very limited. The biggest problem I've run into is sometimes if the client crashes, the server doesn't close its side of the SSH tunnels, and the client can't reconnect. Only solution I've found so far is to killall sshd on the server, which is quite annoying. However, this has been much less of an issue since the client is more stable and hasn't crashed for me in a long time.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/boringproxy/boringproxy/issues/33#issuecomment-758096564, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AONDJP7PGRMSWSHPJJXA473SZMW5RANCNFSM4V4O6HFA.

You'll probably need a developer with access to the site and the boringproxy code to debug something like this. I haven't had any similar issues since the very early versions. It's probably either a load problem (the machine running the client isn't powerful enough) or boringproxy has a memory leak or similar. Easiest thing to start with is monitoring your memory usage to see what's happening there.
If you share your tunnel settings that might give more insight but unlikely.

You'll probably need a developer with access to the site and the boringproxy code to debug something like this. I haven't had any similar issues since the very early versions. It's probably either a load problem (the machine running the client isn't powerful enough) or boringproxy has a memory leak or similar. Easiest thing to start with is monitoring your memory usage to see what's happening there.
If you share your tunnel settings that might give more insight but unlikely.

Also check your CPU usage.

Also check your CPU usage.

I belive this may be fixed by setting the ClientAliveInterval on your SSH server. See here:
https://github.com/boringproxy/boringproxy/issues/23#issuecomment-766160657

I belive this may be fixed by setting the ClientAliveInterval on your SSH server. See here:
https://github.com/boringproxy/boringproxy/issues/23#issuecomment-766160657


Fix failing Konflux tests

Konflux pipeline runs are failing with
[Violation] tasks.required_tasks_found
  ImageRef: quay.io/redhat-user-workloads/multiarch-tuning-ope-tenant/multiarch-tuning-operator/multiarch-tuning-operator-bundle@sha256:ec6f55f3eaac12394e654fd6e8c158e5c626aef1b49009f50931789973aebfff
  Reason: Required task "rpms-signature-scan" is missing
  Title: All required tasks were included in the pipeline
  Description: Ensure that the set of required tasks are included in the PipelineRun attestation. To exclude this rule add
  "tasks.required_tasks_found:rpms-signature-scan" to the `exclude` section of the policy configuration.
  Solution: Make sure all required tasks are in the build pipeline. The required task list is contained as
  xref:ec-cli:ROOT:configuration.adoc#_data_sources[data] under the key 'required-tasks'.|

Added rpms-signature-scan to fix issue
Using https://github.com/enterprise-contract/golden-container/blob/d3f9845fa73a46b77275db3365b57dbfb974f8c8/.tekton/build-pipeline.yaml#L322-L343 as a template
Docs: https://github.com/konflux-ci/build-definitions/tree/main/task/rpms-signature-scan/0.2

Hi, this looks good to me. Can you verify the failures in the EC before "/approve"?
/lgtm

/retest

Ran again to get the logs. The errors are all part of enterprise contract

/approve


Conservation checker interface

Purpose

Move existing energy check to src and extend to water conservation.
Content

[x] ConservationChecker module
[x] move energy checker code to src
[x] extend energy checker to water mass
[x] add unit tests for accumulator
[x] add unit tests for visualization
[x] demonstrate implementation in a new driver (coupler-driver_modular.jl)
[x] add buildkite hook
[x] add documentation

Notes

the energy check plots in Buildkite are identical to those produced using the non-modularized version of the conservation checker, called by coupler_driver.jl.
conservation is not to the required precision. Investigating this is already a TODO in #135
generalizing the contained functions further requires a re-design of our simulation objects. This will be deferred to the upcoming SDI on optimization and performance.

[x] I have read and checked the items on the review checklist.

@szy21 and @juliasloan25 , thank you both for the review! 🚀

Sure, we can add this as an new feature in the next PR. As you say this PR is already long, and its main purpose is to make existing code modular.

It's really no problem to add it in here temporarily, if it's useful. But in the long term we should probably have check for coupled and standalone runs. I'll put it on the AMIP improvements list and maybe we can reassess in Jan?

bors r+


Support system library subdirectories

Hi!
I have not properly tested this out, do not accept this code- I'm submitting this to have a discussion with you:
I'd find it really useful to support things like require('when/node'), I think this more or less gets it in there - I'd love to get this in main rather than hack my local :D

Coverage remained the same at 50.0% when pulling 3650845e45ad8d8a9749e0e9f31c4c90e88053a0 on davidsketchdeck:patch-1 into e8a17e99d01e31f865d22d31cf2504674866dcef on socketstream:master.

Reimplemented it for 0.4
https://github.com/socketstream/socketstream/commit/669f000ae9bc22df87b1fb8f064f2c4877d8722a

A) you guys rock
B) sorry i was so slow
--
David MackCTO & Co-Founder, SketchDeck
sketchdeck.com
See examples of our work
On Sat, Apr 4, 2015 at 1:53 AM, Henrik Vendelbo<EMAIL_ADDRESS>wrote:

Reimplemented it for 0.4
https://github.com/socketstream/socketstream/commit/669f000ae9bc22df87b1fb8f064f2c4877d8722a
Reply to this email directly or view it on GitHub:
https://github.com/socketstream/socketstream/pull/519#issuecomment-89530588

A) Glad for the thumbs up
B) You can help by trying out 0.4 and reporting any issues you have as soon as possible, it is on the next branch and will be released in beta soon

Sure! I’ll try it on an experimental branch of sketchdeck
--
David MackCTO & Co-Founder, SketchDeck
sketchdeck.com
See examples of our work
On Tue, Apr 7, 2015 at 11:40 AM, Henrik Vendelbo<EMAIL_ADDRESS>wrote:

A) Glad for the thumbs up
B) You can help by trying out 0.4 and reporting any issues you have as soon as possible, it is on the next branch and will be released in beta soon
Reply to this email directly or view it on GitHub:
https://github.com/socketstream/socketstream/pull/519#issuecomment-90693999

It is now released as 0.4.0-beta


Compatibility documentation

Hi @naokazuterada,
I have created first shot of a compatibility matrix. If you find that it could provide any value please feel free to include it in the documentation. I will try to maintain it as I pick up information on use or use MarkdownTOC myself with different Markdown tools.
Take care,
jonasbn

This change is

I am not sure whether this should be included, I will let you decide.

@jonasbn Thank you! This is nice!


Remove field.tsv from config

As discussed in #99, we do not actually need the field.tsv configuration table.

Please update the README.md

Done!


Add a FoldedLightCurve class to fix the default plot labels of a folded light curve

This PR introduces a FoldedLightCurve class for the purpose of fixing the default plot labels (cf. #6).
It's not clear to me that there are other differences which justify the difference between LightCurve and FoldedLightCurve, so perhaps the use of inheritance is overkill here, but it's a nifty solution to #6 nonetheless.

Codecov Report

Merging #64 into master will decrease coverage by 0.17%.
The diff coverage is 55.55%.

@@            Coverage Diff             @@
##           master      #64      +/-   ##
==========================================
- Coverage   91.41%   91.23%   -0.18%
==========================================
  Files          17       17
  Lines        1852     1860       +8
==========================================
+ Hits         1693     1697       +4
- Misses        159      163       +4

Impacted Files
Coverage Δ

lightkurve/lightcurve.py
87.73% <55.55%> (-0.98%)
:arrow_down:

Continue to review full report at Codecov.

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

Can FoldedLightCurve have a property 'phase' which just returns time?

Good idea!!

Looks excellent, the user won't even know! 😱 😄 Merge it!

Added a phase property which returns time.  Also updated the unit test.  Let's merge as soon as the tests complete!


[Bug]: Firefox paste of OTP code unreliable; typing code works fine

What happened?
When keying in a code manually, the handleInput call in the otp-input.blade.php updates the component's state for each key-press, resulting in this.state having the full OTP key by the end of the last number.
When pasting in a code, this is not triggered: Pasting in without previously entering any keypresses results in an exception in vendor/afsakar/filament-otp-login/src/Filament/Pages/Login.php on line 112 as the array key 'otp' is undefined.
If you type in 123456, then remove them all to empty then paste, the 'otp' key is 123456 not the key pasted.
This does not affect Google Chrome, Ungoogled Chromium or Edge
How to reproduce the bug
Use Firefox (127.0.2) and paste a valid OTP key into the OTP input on login
Package Version
1.3.1
PHP Version
8.3.6
Laravel Version
11.14.0
Which operating systems does with happen with?
Linux
Notes
No response

Hi Alex,
Thank you for your feedback. I fixed the bugs in v1.3.2, you mentioned. You can check it.
Have a good day, best regards.

Thanks!


feat: add missing new functions for byte types

currently there's no infallible way to create Bytes types
This adds new(array) -> Self functions

Hey @mattsse. There are functions for this here (might need to scroll btw):
https://github.com/ethereum/c-kzg-4844/blob/624aa60d01da524827123506975431aa5454c80d/bindings/rust/src/bindings/mod.rs#L487-L503


Feature: Implement FakeBasicPublishBatch.

Hi,
I've implemented a possible solution for the BasicPublishBatch.

Thanks for your contribution, I'll probably tweak it a bit (style), then merge it back into master.

@odalet  You should add .editorconfig file and maybe style cop to define the code styles you want.
It would make PR's a little more easy.

Yeah, I know, but I didn't bother setting this up, and my own style preferences are somewhat personal and sometimes based on feeling rather than precise rules, hence probably difficult to hard code in a set of rules...
I also tend to understand other people's code by rewriting it. This rewriting is by no way a judgement by the way. So, and at least until I'm overwhelmed with PRs, I don't think I'll enforce anything for now.
Anyway, your contributions where very welcome!


Nil error object returned from gorealis api for error scenarios

Nil error object returned from gorealis api error scenarios. this behavior is for both  thermos and custom executor.
Purposefully set ram to 0 to generate error scenario ...
log for custom executor:
/usr/local/go/bin/go build -o /private/var/folders/jc/ljm2qnwx42l8f88vnxtt6k_82zcfdr/T/customCreatego /Users/kkrishna/work/golang/src/github.com/rdelval/gorealis/examples/client.go
Creating job
error obj:
response: Response({ResponseCode:INVALID_REQUEST Result_: ServerInfo:ServerInfo({ClusterName:aurora-cluster StatsUrlPrefix:}) Details:[ResponseDetail({Message:ram_mb must be greater than 0.0})]})Process finished with exit code 0
thermos logs:
/usr/local/go/bin/go build -o /private/var/folders/jc/ljm2qnwx42l8f88vnxtt6k_82zcfdr/T/thermosCreatego /Users/kkrishna/work/golang/src/github.com/rdelval/gorealis/examples/client.go
Creating job
error obj:
response: Response({ResponseCode:INVALID_REQUEST Result_: ServerInfo:ServerInfo({ClusterName:aurora-cluster StatsUrlPrefix:}) Details:[ResponseDetail({Message:ram_mb must be greater than 0.0})]})
Process finished with exit code 0

Fixed in a715282c00918986f5af48dc31c7986247d75283. Please reopen if the fix doesn't work as expected.


Mark scaleMode as optional for ILineStyleOptions

I have got error if not pass scaleMode to linesStyle as ILineStyleOptions

Fix this problem mark scaleMode as optional

thanks! i'll re-upload the lib soon


ShelleyMA formal - fix timelocks and interval, etc.

Align timelock scripts with code,
fix the ininterval function ie i_s <= s < i_f
align min-value calculation with code
add outputsTooBig check

The size function should get an implementation, as discussed

Yes! Thank you for the reminder. Doing that now.


Misc cmake test cleanup

Does several things:

makes incremental builds where you are editing the cmake files MUCH faster by using configure_file to conditionally replace the *config.h files
makes PyIlmBase tests conditional in the same manner as OpenEXR and IlmBase
makes IlmImfFuzzTest an optional test to run, defaulting to disabled as it is in configure.ac. Unlike the autoconf pass, it still compiles it and adds a "make fuzz" (or whatever your build system) rule to run the fuzz test, mostly for future integration with jenkins
fixes exrdisplay compiling under cmake if enabled
adds additional check in cmake for numpy
fixes compile errors under c++17

merge #328 to develop branch.

actually merge #328 to develop branch.


Aliasing Bug

Here is a testcase that causes Grackle to produce invalid SQL due to an issue with column aliasing. It's not the exact problem I was seeing in my code but hopefully it has the same root cause. It's in the style of SqlPaging2Mapping but does away with the hasNext bit, which isn't relevant to the error.
The query in the test ends up generating the following:
SELECT
  t_observation_nested.c_observation_id,
  t_observation_nested.c_program_id,
  t_program.c_program_id AS c_program_id_alias_0,
  t_program.c_program_id AS c_program_id_alias_1
FROM
  t_program
  LEFT JOIN LATERAL (
    SELECT
      t_observation.c_program_id,
      (t_observation.c_observation_id COLLATE "C")
    FROM
      t_observation
    WHERE
      (
        (
          t_program.c_program_id_alias_0 = t_observation.c_program_id
                    -------------------- 🔥 this isn't a column
        )
      )
    ORDER BY
      (t_observation.c_observation_id COLLATE "C") NULLS LAST
  ) AS t_observation_nested ON (
    t_observation_nested.c_program_id = t_program.c_program_id
  )
WHERE
  ((t_program.c_program_id = $ 1))

Note that the parent and child tables both use the name c_program_id, which forces the aliasing and may be the unusual circumstance that's exposing the issue.

I'm fairly sure this was fixed in #294.


A hook for mounting an extra-patches directory into the jail

I've been using this for over two years. It was recently updated
based upon suggestions:
https://gist.github.com/dlangille/924507ce545de8fbbe78b50b752997ce
Background:https://dan.langille.org/2019/08/10/poudriere-hooks/

Removing the directory produced many errors in my overnight cronjob. I reverted that change and pushed it to my repo. I don't see it here yet.
mkdir: /usr/local/poudriere/data/.m/122amd64-default-primary/ref/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/122amd64-default-primary/01/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/122amd64-default-primary/04/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/122amd64-default-primary/03/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/122amd64-default-primary/02/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/122amd64-default-mysql57/ref/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/122amd64-default-mysql80/ref/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/122amd64-default-pg96/ref/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/122amd64-default-pg10/ref/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/122amd64-default-pg11/ref/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/122amd64-default-pg12/ref/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/122amd64-default-pg13/ref/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/122amd64-default-unifi/ref/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/122i386-default-primaryi386/ref/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/13amd64-default-primary/ref/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/13amd64-default-primary/03/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/13amd64-default-primary/04/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/13amd64-default-primary/01/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/13amd64-default-primary/02/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/13amd64-default-mysql57/ref/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/13amd64-default-mysql80/ref/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/13amd64-default-pg96/ref/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/13amd64-default-pg10/ref/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/13amd64-default-pg11/ref/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/13amd64-default-pg12/ref/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/13amd64-default-pg13/ref/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/13amd64-default-unifi/ref/local-patches: File exists
mkdir: /usr/local/poudriere/data/.m/13i386-default-primaryi386/ref/local-patches: File exists


Added Auto Deploy

Fixes #240
Demo: https://pr-249-ongaku.surge.sh
This PR is for Auto-deployment Feature.

[x] This issue is for enhancement/bug fixing.
[ ] This issue is an addition to JSON database.

[ ] Pull Request Title is of form Added [Anime Name].

[ ] This addition is already on ongaku.

[ ] This PR is an improvement to current addition.

[x] I have checked my addition locally and it is working and won't break existing entries.
[x] I have gone through Contribution Guidelines.
[x] This pull request will be closed if I failed to update it even once in a week.

Link not Working Please see


Initial coordinates in octagonal structure

@yasmensarhan27 in the nacl_octagon.py how did you manage to get the initial position?

Did you follow this?

Hi
I Used the empirical value 2.36 A (0.236 nm)
then put the z=0 for all the points
draw the configuration in  using the equilibrium distance between each Na and Cl

Like this
but please keep in mind that the values are in Angstrom so you have to divide by 10 to have the indices in nm
@reshnashrestha

Is there some kind of code to find out these configurations when we input their bond length?

Oh seems like a lot of hard work

did you figured it out?


Run safety only against real dvclive deps.

We don't own ML framework dependencies and it is the ML framework responsibility to run the safety checks.
We will eventually move framework integrations to the framework itself #221

@dtrifiro I will try to include in this P.R. a proper decoupling of the ML framework


Use metav1.Condition

Kubernetes 1.19 includes a Condition type appropriate for us to
consume. The only significant difference between the conditions we
already define is that reason, message and lastTransitionTime are
required to exist, and reason and lastTransitionTime must have a
value, while message may be empty.
Resolves #126

Discussed in the hangout. Agreed to use the KEP link for now, and open an issue to update the link to a proper documentation when available.

Community to review (if there are any outstanding reviews) and PR to be merged.


error on yarn create r3f-app next my-app -ts

Upon running yarn create r3f-app next my-app -ts per the docs yields the following error:
Error: create-r3f-app tried to access semver, but it isn't declared in its dependencies; this makes the require call ambiguous and unsound.
Full output:
➤ YN0000: · Yarn 4.0.0
➤ YN0000: ┌ Resolution step
➤ YN0085: │ + create-r3f-app@npm:1.6.8, @ampproject/remapping@npm:2.2.1, @babel/code-frame@npm:7.22.13, @babel/compat-data@npm:7.23.2, and 315 more.
➤ YN0000: └ Completed in 1s 679ms
➤ YN0000: ┌ Fetch step
➤ YN0013: │ 319 packages were added to the project (+ 29.93 MiB).
➤ YN0000: └ Completed
➤ YN0000: ┌ Link step
➤ YN0000: │ ESM support for PnP uses the experimental loader API and is therefore experimental
➤ YN0000: └ Completed
➤ YN0000: · Done with warnings in 2s 62ms

/private/var/folders/fr/g4k50xt53x50ymkb71cr551r0000gn/T/xfs-33816afa/dlx-49456/.pnp.cjs:9092
      Error.captureStackTrace(firstError);
            ^

Error: create-r3f-app tried to access semver, but it isn't declared in its dependencies; this makes the require call ambiguous and unsound.

Required package: semver
Required by: create-r3f-app@npm:1.6.8 (via /private/var/folders/fr/g4k50xt53x50ymkb71cr551r0000gn/T/xfs-33816afa/dlx-49456/.yarn/cache/create-r3f-app-npm-1.6.8-d4d1ca0ded-ef12dd8092.zip/node_modules/create-r3f-app/lib/utils/)

Require stack:
- /private/var/folders/fr/g4k50xt53x50ymkb71cr551r0000gn/T/xfs-33816afa/dlx-49456/.yarn/cache/create-r3f-app-npm-1.6.8-d4d1ca0ded-ef12dd8092.zip/node_modules/create-r3f-app/lib/utils/check-for-update.js
- /private/var/folders/fr/g4k50xt53x50ymkb71cr551r0000gn/T/xfs-33816afa/dlx-49456/.yarn/cache/create-r3f-app-npm-1.6.8-d4d1ca0ded-ef12dd8092.zip/node_modules/create-r3f-app/lib/index.js
- /private/var/folders/fr/g4k50xt53x50ymkb71cr551r0000gn/T/xfs-33816afa/dlx-49456/.yarn/cache/create-r3f-app-npm-1.6.8-d4d1ca0ded-ef12dd8092.zip/node_modules/create-r3f-app/bin/create-r3f-app

Node.js v21.0.0
Yarn 4.0.0

Same issues with Node.js v18.17.1 and  Yarn v4.0.1.

@takavfx @bstro using npm works (im using node 21.4.0) : npm create r3f-app next my-app -ts

@ashleymvanduzer 's suggestion worked for me, but only with tailwind, not styled unfortunately.

I'm seeing the same issue, with Yarn v4.0.2 and Node.js v21.6.2 (I also just updated yarn and node to latest, and had the same issue with older versions of both).
Seems like the error comes from Yarn's PnP. I wish there was some way to turn that off during this create command, to see if that helps.
I was also able to use npm and get it to work with that, but nothing looks to be TypeScript? Everything is still .js, not sure if that's unrelated to using npm though.

@Bluhurr nice, I had been meaning to try that, had a feeling npx would be the best option for now.


Add Gradient

Anyone gonna approve or decline the request? Been waiting..

@philipposslicher Sorry, couldn't get to this before. Looks ok, will merge soon.

Merged.


uvwasi team working tasks

As discussed in the recent meeting with @cjihrig, @phated, @ospencer, @guybedford and myself

Refactor build so that uvwasi can be used as shared library

Oscar and Blain will work on PRs for this, Colin indicated its something that uvwasi should accept.

Stabilization

Land the 2 PRs Michael has submitted

https://github.com/nodejs/node/pull/46469
https://github.com/nodejs/node/pull/46434

Wait a bit until we see we have momentum on maintenance
Target to have be stable in Node.js 20 which goes out in April
Will remove need for command line option.

Implementing the 4 stubbed out socket methods

Oscar to talk to Colin to get some help on how to manage streams in order to be able to get started. Will let Michael know once he's got that as he might have some cycles to help with implementation

Snapshot 2

Longer terms as spec is not defined yet.
Guy is going to work on sharing his experimentation/prototyping that he has been doing on this front. Even though this is in JavaScript, agreement is that final implementation for Node.js should be in uvwasi.

Finding additional people to contribute (Done)

Oscar to reach out to people he knows at bytecode alliance to see if he can get some others interested.
We should think about how we make joining our discussion discoverable, something that new people can join and make a call in other ways as well.

If anybody is interested in contributing or joingin the meeting just let one of us know.

I would be interested; it's been on my list of things to do for a while but because I don't have a clear idea of what to do it always ends up low priority.

If there are upcoming meetings regarding WASI, I'd like to join.

@abrown, @tniessen great to hear that you'd like to join the meetings/help out.
I'm just setting up the infra to have issues generated for each meeting (https://github.com/nodejs/create-node-meeting-artifacts/pull/124)  and have added the meeting to the Node.js calendar.  The regular meeting is scheduled for every other Wednesday at 2 ET. The next one is scheduled for Wed March 15.
The zoom for the next meeting will be - https://zoom.us/j/91612484369

Closed in favor of these issues where ongoing work will be tracked:
https://github.com/nodejs/uvwasi/issues/59
https://github.com/nodejs/uvwasi/issues/200
https://github.com/nodejs/uvwasi/issues/199


scaffold command adds an unneeded newline at the end of output

https://github.com/aquaproj/aqua-registry/pull/9556#issuecomment-1411629843

How to reproduce
$ aqua-registry -v
aqua-registry version 0.1.6 (291d904ed1c7b8954068c1dada00dfc7353dd7ae)

$ aqua -v
aqua version 1.32.2 (4f892abd43f2781d1252f794072d59a64a369857)

$ aqua-registry scaffold soh335/shukujitsu

packages:
  - type: github_release
    repo_owner: soh335
    repo_name: shukujitsu
    asset: shukujitsu_{{trimV .Version}}_{{.OS}}_{{.Arch}}.tar.gz
    supported_envs:
      - linux
      - darwin
    checksum:
      type: github_release
      asset: checksums.txt
      file_format: regexp
      algorithm: sha256
      pattern:
        checksum: "^(\\b[A-Fa-f0-9]{64}\\b)"
        file: "^\\b[A-Fa-f0-9]{64}\\b\\s+(\\S+)$"
# unnecessary newline

aqua gr doesn't add a newline.
aqua gr soh335/shukujitsu > test.yaml

packages:
  - type: github_release
    repo_owner: soh335
    repo_name: shukujitsu
    asset: shukujitsu_{{trimV .Version}}_{{.OS}}_{{.Arch}}.tar.gz
    supported_envs:
      - linux
      - darwin
    checksum:
      type: github_release
      asset: checksums.txt
      file_format: regexp
      algorithm: sha256
      pattern:
        checksum: "^(\\b[A-Fa-f0-9]{64}\\b)"
        file: "^\\b[A-Fa-f0-9]{64}\\b\\s+(\\S+)$"

I found the cause.
https://github.com/aquaproj/registry-tool/blob/e7f7b550dea1fe2bb27122609fd461f53d5c08c7/pkg/scaffold/api.go#L72-L74
https://github.com/aquaproj/registry-tool/blob/e7f7b550dea1fe2bb27122609fd461f53d5c08c7/pkg/patchchecksum/patch.go#L55

https://github.com/aquaproj/aqua/pull/1541

I think PatchChecksum isn't needed anymore, because this is for old aqua.
When we released checksum verification, we expected users use old aqua which didn't support checksum verification, so we patched checksum settings in registry-tool.
registry-tool is managed by aqua, so we can force users to update registry-tool automatically.
The drawback of PatchChecksum is that we have to maintain this function.

Updated.

https://github.com/aquaproj/aqua-registry/pull/9594


How to use my own dataset on this implementation?

Hello, sir.
I would like to use my own dataset which consists of 100 grayscale images + binary masks (labeled) and 900 unlabeled grayscale images.
Is there a way to use my own dataset other than the datasets which are provided in the instructions?

You need to write a new dataset similar to https://github.com/LiheYoung/UniMatch/blob/main/dataset/semi.py

Sure. Please refer to this issue: https://github.com/LiheYoung/UniMatch/issues/33#issuecomment-1653531541


style(Change Title To URL)

Update the title so that it is a URL instead. Also add a reminder that you can click on the URL to be taken to the server's channel.

RTM! 🚀


Ownerless group  events missing in Sentinel Audit log query

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

Make a Microsoft Group ownerless (delete the account)
3 types of notifications visible in the audit log (notificaction,response and unattended group)
Sentil log query against audit log has only operations "Response" as result, other both missing
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

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

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Hello @v-amolpatil

Enable new solution
https://learn.microsoft.com/en-us/microsoft-365/admin/create-groups/ownerless-groups-teams?view=o365-worldwide

2.First delete user in Azure AD permanent who is an owner of a Microsoft Group
3.most active user on the group should get offer to accept to be the new owner
In the audit log is the activity Ownerless group notify
If somebody accepts the offer, you will see in in the audit log activity "Ownerless group response"
If nobody accepts the offer you will see in the audit log daily entry "Ownerless group unattended"
All the activities in the audit log have identity "Group Configuration Processor"
In Sentinel query against the audit log you will find only activity Response, notify and unattended missing
Audit log filter:

Sentinel query

Hi @funzel1 are you still facing this issue, please confirm, thanks!

Gentle Reminder: We are waiting for your response on this issue. If you still need to keep this issue active, please respond on it in the next 2 days. If we don't receive response, we will be closing this issue as per our standard procedures, thanks!

Hi @funzel1 Since we have not received a response in the last 5 days, we are closing your issue as per our standard operating procedures. If you still need support for this issue, feel free to re-open at any time. Thank you for your co-operation


⚠ ClamAV has degraded performance

In 91e018e, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 178 ms

Resolved: ClamAV performance has improved in c39ffcb.


Feature/279 when built from Xcode, display something

#279
This PR includes two indicators to show the user that Xi-Mac is being run from within Xcode:
First, a [Debug] .. string in the window titlebar:

Second, a different icon that kinda mimics the look of the Xcode icon:

@cmyr I broke something when I commited. The icon should also work now. I also added the license header.

@terhechte awesome, this is working for me now.
My only lingering concern is that I'm seeing two new compiler errors (Xcode 10). The first is about your #if debug causing an unreachable line, and the other is about a downcast to NSWindowController. If you'd like to clean those up I'd be happy to merge.
Also it's a little embarrassing that our debug logo maybe looks a bit better than our default logo.. 😅💦 (aka, good job on the logo)

@cmyr Thanks! Glad you like the icon :) I've fixed the compiler warnings. If there's interest, I could give the main icon a refresh as well. I'd love to do that.

There's definitely interest from me for that new icon :)

See also xi-editor/xi-editor#51
That issue has a lot of suggestions but sadly no consensus has been reached. I think people expected @raphlinus to make an authoritative decision there but things are quite different now.


CXF-6904 Customize Swagger scan logic so it doesn't hurt in OSGi.

This PR overrides bean config and allows to use most of it's code used for swagger configuration. This keeps default behavior of swagger, with some small difference that when scan on Swagger2Feature is disabled it passes classes which are available on endpoint.

Very nice, significantly simpler than trying to create the application (we can still have that option if needed) - will apply next week

CXF-6904 is resolved so this should be closed as well, shouldn't it?


Command window is too small

Can you include OS and browser and version?

Firefox + (Windows 8 || Ubuntu 14.04)

This is actually fixed with #159 here. @chunkyg and I only saw it with the addition of the new button.


lot of options breaks the layout

https://jsfiddle.net/0afuq7pk/ if u follow this fiddle you will find out that when large number of options are available this just breaks the layout of checkboxes.
This can be fixed if we remove position:absolute from the css for this class .select2-result-label .wrap:before but not sure if this will hamper something else. Please review

fixed with new css. take a look into https://jsfiddle.net/wasikuss/7ak9skbb/

duplicate od #2 and already fixed with new css


Releasing an update

Would you consider releasing an updated version (0.2.0 or 0.1.1) with the newest code?

I will, but I'm on the road without my computer (and hence private key), so I can't publish a hex package until Monday. Sorry for the delay.

No problem.

Done. https://hex.pm/packages/html_entities

+1


Not good that tests need manual creation of user

The fact that the tester needs to create a user manually before running the test is is not good and impedes just doing an npm test. We should register a user if needed before each test.

Fixed


feature request: re-tail identically named resource

- [example-pod-kvfr5] pod deleted

now, i'd be awesome if kt could pickup logging if a identically named resource comes up:

detect resource deletion
wait (in configurable intervals) for the resource to show up
tail, again

why? i'm working on a interactive workflow for https://github.com/GoogleContainerTools/kaniko requiring something like as described above..

@gretel now kt should be able to work as you described.
For example, let's say we have a pod running as shown below:
apiVersion: v1
kind: Pod
metadata:
  creationTimestamp: null
  labels:
    run: test-log
  name: test-log
spec:
  containers:
  - image: alpine:3.10
    name: test-log
    command:
    - sh
    - -c
    - 'while true; do date; sleep 3; done'
    resources: {}
  dnsPolicy: ClusterFirst
  restartPolicy: Never

and we execute kt test-log in a terminal.
Then if we recreate this pod, kt should auto re-tail it:
$ kt test-log
+ [test-log] pod added
test-log[test-log] Mon Feb 17 11:53:09 UTC 2020
test-log[test-log] Mon Feb 17 11:53:12 UTC 2020
test-log[test-log] Mon Feb 17 11:53:15 UTC 2020
test-log[test-log] Mon Feb 17 11:53:18 UTC 2020
test-log[test-log] Mon Feb 17 11:53:21 UTC 2020
test-log[test-log] Mon Feb 17 11:53:24 UTC 2020
test-log[test-log] Mon Feb 17 11:53:27 UTC 2020
test-log[test-log] Mon Feb 17 11:53:30 UTC 2020
test-log[test-log] Mon Feb 17 11:53:33 UTC 2020
test-log[test-log] Mon Feb 17 11:53:36 UTC 2020
test-log[test-log] Mon Feb 17 11:53:39 UTC 2020
test-log[test-log] Mon Feb 17 11:53:42 UTC 2020
test-log[test-log] Mon Feb 17 11:53:45 UTC 2020
test-log[test-log] Mon Feb 17 11:53:48 UTC 2020
test-log[test-log] Mon Feb 17 11:53:51 UTC 2020
- [test-log] pod deleted
+ [test-log] pod added
test-log[test-log] Mon Feb 17 11:54:15 UTC 2020
test-log[test-log] Mon Feb 17 11:54:18 UTC 2020
test-log[test-log] Mon Feb 17 11:54:21 UTC 2020
^C

@knight42 thanks! just tried, but does not pickup the pod, again.

@gretel could you let me know the commands you executed?

job description:
$ kubectl describe job kaniko-kerykeion
Name:           kaniko-kerykeion
Namespace:      ca
...
Labels:         app.kubernetes.io/instance=kaniko-kerykeion
                app.kubernetes.io/managed-by=Helm
                app.kubernetes.io/name=kaniko
                app.kubernetes.io/version=0.17.1
                helm.sh/chart=kaniko-0.3.6
Annotations:    helm.sh/hook: post-install,post-upgrade
...

works, but does not pick up recreated job:
$ kt -v3 -n ca --tail 10 job kaniko-kerykeion
+ [kaniko-kerykeion-p627h] pod added
kaniko-kerykeion-p627h[build] INFO[0010] RUN curl -sSL https://get.helm.sh/helm-${HELM_VERSION}-linux-amd64.tar.gz | tar -xzO linux-amd64/helm -f - > /usr/local/bin/helm     && chmod +x /usr/local/bin/helm
...

filtering by label (as described) does not work:
$ kt -n ca --tail 10 -l app.kubernetes.io/name=kaniko
no matched pods found

Ah got it, I could fix it soon.

@knight42 that'd be awesome!

@gretel The problem may be a bit tricky than I expected. The mechanism that kt find out the pods belong to a job is filtering pods by labels specifed in job.spec.selector.matchLabels, which is sth like:
spec:
  selector:
    matchLabels:
      controller-uid: ac8cb0ea-5174-11ea-9cd8-b22a251591f2

A newly created job ususally has different uid, so kt cannot find out the pods belong to the new job.
The workaround of this problem could be one of:

specify labels in the podTemplate of job and execute kt -n ca -l <selector>
filter pods by regexp, like kt -n ca kaniko-kerykeion

thanks, i was not aware of option 2!
seems like there is a bug lurking though:
- [kaniko-kerykeion-ljsnq] pod deleted
+ [kaniko-kerykeion-rjzw9] pod added
- [kaniko-kerykeion-rjzw9] pod deleted
+ [kaniko-kerykeion-trch5] pod added

it does get lifecycle of the pod, but the log shows up on the first time, only.

ummm could you tell me the output of kt -v 5 -n ca ...? @gretel

tried again - it's working now. looks like some api glitch happenend cluster-side.

🤔well I guess I could close this issue now? feel free to reopen it if needed.

@knight42 sure, thanks!


[DO NOT MERGE]add test case

Add many test cases and correct corresponding bugs.
fixes #136

Codecov Report

Merging #274 (b61ede0) into master (6719b4f) will increase coverage by 0.77%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #274      +/-   ##
============================================
+ Coverage     50.17%   50.94%   +0.77%
- Complexity      713      725      +12
============================================
  Files           122      121       -1
  Lines          3229     3219      -10
  Branches        480      480
============================================
+ Hits           1620     1640      +20
+ Misses         1413     1391      -22
+ Partials        196      188       -8

Impacted Files
Coverage Δ
Complexity Δ

...ava/seedu/address/storage/JsonAdaptedDeadline.java
69.01% <ø> (ø)
13.00 <0.00> (ø)

...n/java/seedu/address/storage/JsonAdaptedEvent.java
45.28% <ø> (ø)
4.00 <0.00> (ø)

.../seedu/address/logic/parser/DoneCommandParser.java
80.00% <100.00%> (+20.00%)
5.00 <3.00> (+1.00)

...va/seedu/address/model/task/deadline/Duration.java
76.92% <100.00%> (ø)
5.00 <2.00> (ø)

.../address/logic/parser/FindLessonCommandParser.java
94.28% <0.00%> (+39.99%)
19.00% <0.00%> (+11.00%)

...ddress/logic/parser/DeleteLessonCommandParser.java
100.00% <0.00%> (+40.00%)
2.00% <0.00%> (ø%)

Continue to review full report at Codecov.

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


Bookmark followups based on feedback

Add the search bar.
Add a message when there are no default/shared bookmarks.
Move the description just below the name in the forms.
Share a personal bookmark to everyone.

Note that this has changes from #4269 to have the final changes in a single branch.

Some UX items:

I'm unable to use the Bookmark's Search input when on the TDD view

Per the Figma mocks, it looks like the Edit and Delete icons should have their own hover state. But it'd be good to check with @jkhwu how that hover state should be reconciled with the hover state of the full bookmark item.

The "Filters" text is subtly a different font style (both size and color) than the other form item labels

Thanks have addressed all your comments except the hover state. Will check that and raise a separate PR with any potential feedback from the demo.


Static Limitation: Implement Static Limitation for Keba

It's a matter of putting Keba model into the templates.
Move to asana.


Moe Sync

This code has been reviewed and submitted internally. Feel free to discuss on the PR and we can submit follow-up changes as necessary.
Commits:
 Turn down ReturnMissingNullable
it is disabled by default, and has bugs.
e5dfb88050741936c3e0121ab902d1541f17dd62

 Finally: Replace `.label` by `.getLabel()`
See #1106 - field removed in JDK12 as per
Fixes #1107
f838f788703c7ec1fcdac4ac593d0f059682087d

 Obtain Descriptions for testing via BugCheckers, rather than via Description's constructor.
To fulfil a TODO, and allow tidying up the API.
1477732a8535eee2492a55ec35826696061f49a9

 Fix two typos (for the price of one)
6744ce2558418a2e2fa2235e1ba5271fae8df6f2

 Enable Javadoc-related checkers by default
363cca174dfccd8c73692d477f6b79dd2ad599a9

Setting CLA: yes from CLA agreement in #1107


Docs: Please use "jsonfield" and not "django-jsonfield"

Hi,
unfortunately there are two projects called "django-jsonfield".
Since "django-jsonfield" is a different project on pypi, I suggest you use "jsonfield" to refer to your project.
Please update at least the README.
Background: I just wasted some time, just because of this confusion.
Thank you for this nice django field!

I've added a note to clear this up.


Popover overflowing past screen when trigger is on the edge of the screen

Please provide the environment you discovered this bug in.
0.0.1-alpha.344
Which area/package is the issue in?
popover
Description
When the button is on the edge of the screen, using popover could push the popover past the screen.
The underlying CDK may be limiting the width of the actual container to the edge of the screen but the underlying content will still be push past it.

In the example image the underlying container has width of 400px but the underlying "cdk overlay pane" is limited to 288px thus the overflowing container.
It may be the case that the overlay should remove the width limit and/or allow some x-offset.
Please provide the exception or error you saw
No response
Other information
No response
I would be willing to submit a PR to fix this issue

[ ] Yes
[X] No

Can you create a stack blitz with this behavior displayed? I'll take a look then

@dez1729 please provide a minimal reproduction for this and I'll take a look