[FEATURE] provide a warning if persistent storage has not been enabled.

Is your feature request related to a problem? Please describe.
Within the Docker deployments on HF and others (I would say everything not k8s), we've got an issue of people not realising they need persistent storage to retain annotation responses.
Describe the solution you'd like
I would consider adding a PERSISTENT_STORAGE_ENABLED=False env var that shows a small UI pop up whenever someone has not spend time thinking about this.
Describe alternatives you've considered
N.A.
Additional context
This has happened multiple times during the DIBT efforts and also in the past to community members.

The main idea, show a warning within the UI about "Missing persistant storage and riskt of loosing annotated data" whenever the environment variable PERSISTANT_STORAGE_ENABLED has not been set.
This should cover 2 flows.

Hugging Face deployment without PERSISTANT_STORAGE_ENABLED will automatically show these errors when people did not think about this.
All other deployments in any other place will have PERSISTANT_STORAGE_ENABLED=False by default to ensure that people will explicitly need to set the environment variable to be aware they might loose data or at least to emphasize that someone should spend time on actively changing this environment variable to avoid doing that.

Update: internally we can use  SHOW_PERSISTENT_STORE_WARNING but from the HF side PERSISTANT_STORAGE_ENABLED is fixed so we can have some logic surrounding these two examples. Having SHOW_PERSISTENT_STORE_WARNING can also be used for forcefully disabling this message.

Update:  I believe it should be quite clear something in the config has not gone correctly so

Every type of role should be able to see it.
Every time they open the platform the pop-up can be shown


chore: update static-site-generator-webpack-plugin

Pre-flight checklist

[x] I have read the Contributing Guidelines on pull requests.
[ ] If this is a code change: I have written unit tests and/or added dogfooding pages to fully verify the new behavior.
[ ] If this is a new API or substantial change: the PR has an accompanying issue (closes #0000) and the maintainers have approved on my working plan.

Motivation
Upgrade the dependency @slorber/static-site-generator-webpack-plugin after some cleanup and a few PRs merged
New concurrency option added with env variable. Not documented for now, looking for feedback before.
Test Plan
site builds fine, not slower than before
Test links

Deploy preview: https://deploy-preview-_____--docusaurus-2.netlify.app/
Related issues/PRs

https://github.com/slorber/static-site-generator-webpack-plugin/pull/2
https://github.com/slorber/static-site-generator-webpack-plugin/pull/3

Excited that we got rid of a bunch of lodash modules, bluebird, AND cheerio!


Add ArtifexSoftware.mutool

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

I am sure the installer URL/domain is correct and I have tested the PR's manifest locally and it works as expected.
Maybe this might be the case.

This may happen for sites with policies prohibiting distribution or use by third parties.

Regarding the validation executable error, I will try adding uninstall behaviour.

This package is expecting arguments:
mutool version 1.22.1
usage: mutool <command> [options]
        clean   -- rewrite pdf file
        convert -- convert document
        create  -- create pdf document
        draw    -- convert document
        trace   -- trace device calls
        extract -- extract font and image resources
        info    -- show information about pdf resources
        merge   -- merge pages from multiple pdf sources into a new pdf
        pages   -- show information about pdf pages
...

@wingetbot waivers Add Validation-Executable-Error Validation-Domain


Add a sidebar for Draggable Blocks

we can add a sidebar to the studio to add blocks that can be draggable and dropped into the visualizer. This sidebar would contain multiple blocks that can be easily dragged and dropped into the visualizer.
when a block is clicked on the visualizer, the corresponding code for that block should open in the editor, where the properties of the block can be modified for that particular block. In the editor, we have a code to change the properties such as the description of a specific block, message, summary, etc.
we are thinking of creating a modal to add blocks in a draggable manner also, but it’ll reduce the user experience if we drag a block from the modal to the visualizer.
I am looking forward to some discussion around this.
( to learn more about this issue, refer: #265 )
cc- @magicmatatjahu

Hey @wh0sumit, this looks interesting 😄
I'd love to collaborate with you on this :)
Is there anything available yet concerning the blocks that I can look into?

@wh0sumit @magicmatatjahu Is someone working on this issue ?


updating readme to show configuration options

Please check if the PR fulfills these requirements

[x] Followed the Contributing guidelines.
[x] Tests for the changes have been added (for bug fixes/features) with 100% code coverage.
[x] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
updating readme to show configuration options

Thanks for replying.
I checked the code and in the memory cache shouldCloneBeforeSet is set to true by default. As indicated here

Unless I am missing something, please let me know if I did.

@adelbke - LOL I was going too fast. Good catch. Updated here: https://github.com/node-cache-manager/node-cache-manager/pull/645

Yeah...it happens to everyone, even the best 😅

Yeah...it happens to everyone, even the best 😅

😂


Fix importing include multiple times in urlconf

Description
Check for already imported include function when patching urlconf
References
Fix #21
Checklist

[x] I have read the contribution guide
[x] Code lint checked via inv lint
[x] changes file included (see docs)
[x] Usage documentation added in case of new features
[x] Tests added

Pull Request Test Coverage Report for Build 446600160

10 of 10 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.1%) to 99.016%

Totals

Change from base Build 435545326:
1.1%

Covered Lines:
210

Relevant Lines:
211

💛  - Coveralls


Expose a way to enable autoscale on a running cluster

There's a way to enable autoscale when you create a cluster and not a way to enable it after the fact. After implementing it here, we'll want to bring that up to carina as well.

PUT /clusters/{userName}/{clusterName}/autoscale/{true|false}`

Done in #13


fresh 1.0.0 Dockerfile self-hosting

I'm try to run fresh on my server using docker
this is my Dockerfile

FROM denoland/deno:1.23.1
WORKDIR /app
ADD ./webapp ./
RUN deno cache main.ts
CMD ["run", "--allow-net","--allow-read","--allow-env","--allow-write","--allow-run","main.ts"]

docker logs
....
Server listening on http://localhost:8000
fresh-v1  | Download<EMAIL_ADDRESS>

I get auto-load forever !

I set DENO_DEPLOYMENT_ID deno environment variable in Dockerfile ,
FROM denoland/deno:1.23.1
WORKDIR /app
ADD ./webapp ./
ENV DENO_DEPLOYMENT_ID=unique-id
RUN deno cache main.ts
CMD ["run","-A", "--allow-net","--allow-read","--allow-env","--allow-write","--allow-run","main.ts"]

I hope this help 👍🏽

I have a problem related with this. $fresh at import_map.json is absolute so putting the project in a container causes problems.


Windows security warning still showing up

Despite code signing, and a few weeks of downloads, there is still a security warning showing up when downloading and installing Botpress on windows. The warning is particularly scary in firefox.

Likely found the issue and solved it.
Submitted a ticket to Microsoft just to be sure I got this right this time... And here's the answer :
Analyst comments:

The warning you experienced indicates that neither the application nor the signing certificate had established reputation with Microsoft Defender SmartScreen services at the time. We can botpress-setup-12.26.12-a.exe, (SHA256: bfea4e951ea9f683f0a3f1d8da586cacde6ff91751ac9c828694d23d6f7827bc), Clean has since established reputation and attempting to download or run the application should no longer show any warnings. The signing certificate (thumbprint 0422CB66BF0B3159B2DB081AC469230F79411153) is still in the process of establishing reputation. Once it does, all applications that are signed with that certificate should have a warn-free experience from the start.

Looks like we just have to wait for enough people to download and run Botpress, and then we'll be able to use our certificate for different versions of Botpress!
Thanks for the help Abe.
Closing now, if the issue persists, will re-open.


adapt url's after moving the repocheck api's

depends on https://github.com/crowbar/crowbar-core/pull/732

 (3) S49P8 Move upgrade-related API endpoints to the correct namespace


[SPARK-29675][SQL] Add exception when isolationLevel is Illegal

What changes were proposed in this pull request?

Now we use JDBC api and set an Illegal isolationLevel option, spark will throw a scala.MatchError, it's not friendly to user. So we should add an IllegalArgumentException.
Why are the changes needed?

Make exception friendly to user.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Add UT.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

Test build #112978 has started for PR 26334 at commit baf1f45.

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-K8s/17901/
Test PASSed.

Thanks, @HyukjinKwon ! Pending Jenkins.

Test build #112978 has finished for PR 26334 at commit baf1f45.

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

Merged build finished. Test PASSed.

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

@ulysses-you Can you remove the prefix before merging? https://github.com/apache/spark/pull/26334#discussion_r340949098

Thanks for your review @maropu @HyukjinKwon @dongjoon-hyun . I have Fixed it.

Test build #112995 has started for PR 26334 at commit 0de8529.

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-K8s/17918/
Test PASSed.

Thanks, @ulysses-you ! Pending, Jenkins.

Test build #112995 has finished for PR 26334 at commit 0de8529.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Jenkins, retest this please

Test build #113020 has started for PR 26334 at commit 0de8529.

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-K8s/17942/
Test PASSed.

Test build #113020 has finished for PR 26334 at commit 0de8529.

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

Oh ha sorry you merged it. The back ticks are superfluous but it doesn't matter, just leave it.

Oops. Sorry, @srowen . :)

Thanks for merging, @dongjoon-hyun !

Thanks all !


Legion: Seg fault in receive_message when using TRACE_ALLOCATION

I am running S3D on Summit with -DTRACE_ALLOCATION at 256 nodes and seeing the following crash:
Thread 8 (Thread 0x2000dfa1f890 (LWP 594571)):
#0  0x000020000093a114 in nanosleep () from /lib64/power9/libc.so.6
#1  0x0000200000939f44 in sleep () from /lib64/power9/libc.so.6
#2  0x00002000033d5328 in Realm::realm_freeze (signal=<optimized out>) at /gpfs/alpine/cmb103/scratch/seshuy/legion_s3d_nscbc_summit/legion/runtime/realm/runtime_impl.cc:177
#3  <signal handler called>
#4  0x000020000280c0b0 in Legion::Internal::MessageManager::receive_message (this=0x3ff000001400742a, args=0x2016054a3fc0, arglen=48) at /gpfs/alpine/cmb103/scratch/seshuy/legion_s3d_nscbc_summit/legion/runtime/legion/runtime.cc:13133
#5  0x000020000280c12c in Legion::Internal::Runtime::process_message_task (this=<optimized out>, args=0x2016054a3fbc, arglen=<optimized out>) at /gpfs/alpine/cmb103/scratch/seshuy/legion_s3d_nscbc_summit/legion/runtime/legion/runtime.cc:25629
#6  0x000020000280c4c0 in Legion::Internal::Runtime::legion_runtime_task (args=0x2016054a3fb0, arglen=56, userdata=<optimized out>, userlen=<optimized out>, p=...) at /gpfs/alpine/cmb103/scratch/seshuy/legion_s3d_nscbc_summit/legion/runtime/legion/runtime.cc:31058
#7  0x00002000033b5fc0 in Realm::LocalTaskProcessor::execute_task (this=0x51ecbe20, func_id=<optimized out>, task_args=...) at /gpfs/alpine/cmb103/scratch/seshuy/legion_s3d_nscbc_summit/legion/runtime/realm/bytearray.inl:150
#8  0x000020000340b1c0 in Realm::Task::execute_on_processor (this=0x51f8e7e0, p=...) at /gpfs/alpine/cmb103/scratch/seshuy/legion_s3d_nscbc_summit/legion/runtime/realm/runtime_impl.h:378
#9  0x000020000340b384 in Realm::UserThreadTaskScheduler::execute_task (this=<optimized out>, task=<optimized out>) at /gpfs/alpine/cmb103/scratch/seshuy/legion_s3d_nscbc_summit/legion/runtime/realm/tasks.cc:1646
#10 0x000020000340e04c in Realm::ThreadedTaskScheduler::scheduler_loop (this=0x4ec13390) at /gpfs/alpine/cmb103/scratch/seshuy/legion_s3d_nscbc_summit/legion/runtime/realm/tasks.cc:1125
#11 0x0000200003417090 in Realm::UserThread::uthread_entry () at /gpfs/alpine/cmb103/scratch/seshuy/legion_s3d_nscbc_summit/legion/runtime/realm/threads.cc:1337
#12 0x00002000008a7ffc in makecontext () from /lib64/power9/libc.so.6
#13 0x0000000000000000 in ?? ()
Backtrace stopped: previous frame inner to this frame (corrupt stack?)

Using commit 42b768e10fb6afbb0842 of control_replication.

How repeatable is this? The way you get a segfault there is by literally corrupting the first few bytes of the client payload of an inter-node message.

It was pretty repeatable. I ran 3 times and hit the same issue each time.

Also, it only appears at 256 nodes when I turn the trace allocation logging on (-level allocation=2). If I just build with -DTRACE_ALLOCATION but dont turn the logging on it seems to work at 256 nodes. All other node counts from 4 - 128 nodes worked as expected.

@streichler Do you have an option to turn on CRC checksums on messages in Realm? Looks like some data is getting corrupted in a message (literally the first four bytes).

The gasnetex network layer does message checksums by default, so try building/running with that.

@syamajala Which version of GASNet are you using?

Should be gasnetex-2021.3.0.

Can you try this again with all the GASNet-EX fixes that @streichler pushed?

Duplicate of #1159 and fixed.


Delete ideas.txt

this does not belong here. I have this saved and will upload it somewhere else in this repository

I will drop this in patch notes once this is approved of

we should make a folder for these then, just so it doesn't conflict with our main scripts

a seperate folder for future ideas? where would put that in? assets or patch-notes? or have its own thing near the main.py... cuz my thought process it that it should only be dsn py programs with folders on the side
if there is a better place or way of storing it, lemme know cuz for now this is what i think is the best solution

maybe you're right, it might be better to store this not on github and with our programs

that's genius, let's leave it in the code as commented to-do list that we might want to complete in the future

we can leave it as the todo comments in certain areas of the code that we need to work on 👍


The "Overview" section for Key Vault is different between Stable and Preview ToCs

Found via #17311.
Here is a link to the Key Vault Overview in the Preview ToC. It only shows links to the Key Vault sub-packages when instead it should show a more complete page like the one on the second screenshot below.

However, trying to switch to the Stable ToC version of the same section leads to a way more complete page and the table of contents on the left dissapears.

Additionally, if you are looking at a different section like the Keys Overview in the Stable ToC and then click overview, you are led to a broken link (404).

This is an issue that library owners should be able to address by following the documentation we've created for these types of issues here.

This is an issue that library owners should be able to address by following the documentation we've created for these types of issues here.

Keyvault resource manager is having GA version ahead of preview. So do we still need the preview Resource Manager show up in toc?
If you definitely need the preview version, please update the preview section under package.json.
In the section here: https://github.com/Azure/azure-docs-sdk-java/blob/master/package.json#L1071

Keyvault resource manager is having GA version ahead of preview. So do we still need the preview Resource Manager show up in toc?
If you definitely need the preview version, please update the preview section under package.json.
In the section here: https://github.com/Azure/azure-docs-sdk-java/blob/master/package.json#L1071

I have been pushed the fix to keyvault readme. The issue got resolved. Please check

I have been pushed the fix to keyvault readme. The issue got resolved. Please check

When will this be reflected in Microsoft Docs? I still see a disparity between the following links:

Stable
Preview

When will this be reflected in Microsoft Docs? I still see a disparity between the following links:

Stable
Preview

@vcolin7 these look to be the same now, can you verify

@vcolin7 these look to be the same now, can you verify

Correct @alzimmermsft, thanks for checking!


Configuration to read-only apps

Is there a way to only expose query requests through GraphQL and GET requests through the REST API? I have a read-only application and I don't need/want to expose ways to alter my database records.

This is a duplicate https://github.com/platformatic/platformatic/issues/604

You right. Thanks @dmateiu.


Aggregate Icinga alert counts for faster reporting

This introduces two new metric namespaces so that we can easily see
the total number of alerts overall and per-machine. Currently, the
number of metrics is too great to compute this.

Note I followed the example here: https://github.com/graphite-project/carbon/blob/master/conf/aggregation-rules.conf.example

Note I followed the example here: https://github.com/graphite-project/carbon/blob/master/conf/aggregation-rules.conf.example


refactor(client): literal

refactor literal in order to create correct code if users pass object to a function

I need to fix the type issue


[release-v1.13] kafka broker now supports all kafka topic config options (#3671)

Cherry pick of https://github.com/knative-extensions/eventing-kafka-broker/commit/75194cf249c11cf8fadee6ca529fa48db9a301fa

I think we do not use 1.13 at all.
/hold


properly set path and test

Fixes #18 with the updated runtime. Note merge and deploy runtimes before merging this with master.

LGTM


new L5 branch

Second PR following your kind suggestion (and fixed the forgotten L4-style call to config).

Had to create a new pull request from your fork to the new l5 branch. Thanks.


Add RuNNer notebook

Example MD for RuNNer.  I'll post some timings and hook it up to the CI after dinner.

@pmrv CI worked fine: 1:29 (min: sec)

@pmrv CI worked fine: 1:29 (min: sec)

It was 108 atoms for 1000 MD steps, so around .8ms per atom and step.  That probably limits the applications for the third day by a bit.


[DOCFIX]  Update Getting-Started.md of the Chinese version

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/12956/
Test PASSed.

LGTM. thanks for the improvement!


test do not merge

Background

Results

Fixes https://github.com/OctopusDeploy/opentelemetry-teamcity-plugin /issues/...

Before

After

How to review this PR

Pre-requisites

[ ] I have considered informing or consulting the right people
[ ] I have considered appropriate testing for my change.

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


Tx recorded as invalid when it isn't, but it's children are recorded as valid

https://slpdb.fountainhead.cash/explorer/ewogICJ2IjogMywKICAicSI6IHsKICAgICJmaW5kIjogewogICAgICAiZ3JhcGhUeG4udHhpZCI6ICI0ZmUzNzFmM2NiYzViMjJmYjBjYzQ3OTA1M2MxYzgyNDEyZjFhYjNkNTVmYTA2MDRjZTAzNjMxYTg1Mjg5YTgxIgogICAgfSwKICAgICJsaW1pdCI6IDEwCiAgfQp9
{
"slpAmount": "100000",
"address": "simpleledger:qrs77l23pmclphnsjhd82dhxu2nvqssyluugg0jtn5",
"vout": 1,
"bchSatoshis": 546,
"spendTxid": "5f91233a5efc7b81f100d922b3967c82db27187bce22eb2f8a5df434d37dde77",
"status": "SPENT_INVALID_SLP",
"invalidReason": "Output burned in an invalid SLP transaction"
},
looking up that txid with tx.h shows its not in slpdb confirmed collection either
https://slpdb.fountainhead.cash/explorer/ewogICJ2IjogMywKICAicSI6IHsKICAgICJmaW5kIjogewogICAgICAidHguaCI6ICI1ZjkxMjMzYTVlZmM3YjgxZjEwMGQ5MjJiMzk2N2M4MmRiMjcxODdiY2UyMmViMmY4YTVkZjQzNGQzN2RkZTc3IgogICAgfSwKICAgICJsaW1pdCI6IDEwCiAgfQp9

This issue should be fixed now, please reopen if there's still an issue.


Flush ElasticSearch logs asynchronously

Currently, we use reqwest to make synchronous calls to elastic search:
https://github.com/graphprotocol/graph-node/blob/de5465ad3c1ab195cda35ba17ac65b72a904a3ec/graph/src/log/elastic.rs#L248-L250
We should make this asynchronous, for example by replacing reqwest with the hyper client.

Done, we've since upgraded reqwest to an async version.


DateOnly string does not deserialize to DateOnly Object when target is object

DateOnly string does not deserialize to DateOnly Object when target is object. But DateTime strings are deserialized to DateTime objects when the string is in the correct format for a DateTime object (see JsonSerializerSettings.DefaultDateFormatString).
Source/destination types
object[]

Source/destination JSON
["2023-03-13T10:12:47.1830163+00:00",123,"9999-12-31"]

Expected behavior
Deserialize as DateOnly Object
Actual behavior
Deserializes as string
Steps to reproduce
var list = new object[] { DateTime.Now, 123, DateOnly.MaxValue };
var json = JsonConvert.SerializeObject(list);
var listDeserializeObject = JsonConvert.DeserializeObject<object[]>(json);
Assert.AreEqual(list[0], listDeserializeObject[0]); // Working: Type == DateTime
Assert.AreEqual(list[1], listDeserializeObject[1]); // Working: Type == int
Assert.AreEqual(list[2], listDeserializeObject[2]); // Not Working: Type == string, but should be DateOnly

Any news?

I'm have the same issue. I Foked the project to include this function at link https://github.com/CamilloBorges/Newtonsoft.Json. However is not a FULL Validated and have Broken test that.


Problem opening datauristring with image

I have generated a pdf file, saved it into my database as datauristring. When the pdf file has an image then window.open(<datauristring>) opens a blank tab without the generated document. See:

Is this the right way to save generated pdf document to a database?
OR Do I have to save the document differently to my database (bloburi, bloburl, blob, arrayBuffer)?
OR Do I have to care differently about images?
When I create the same pdf document without an image everything work as expected!

After 3 years probably any support is too late :-/

Hey,
Any updates on this?


[BEAM-1205] Auto detect and set "enableAbandonedNodeEnforcement" in TestPipeline

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @tgroh , @kennknowles

To be honest, even though we violate this in a few places, I strongly prefer to have tests always named after a real class, for discoverability.

Makes sense, I'll make the proper arrangements :)
Let me know if you spot any other issues.

@tgroh , thanks for taking a look.
I've addressed your comments, let me know how it looks.

@tgroh , @kennknowles I was wondering if you had any further comments before we can move forward with this?

@tgroh thanks for taking a look!
Hopefully I was able to provide more context on some of the design decisions behind this PR.
Please feel free to let me know if I can provide more info.

@kennknowles , I have revised according to (most of) your comments.
Essentially this leaves with the discussion about "having 1:1 mapping between user-scenarios and tests vs. explicit exception handling inside the tests" (more details in the comments above).

Coverage increased (+0.03%) to 69.968% when pulling 55aca1a408a33f1a41681132dfbe1b7eed0bf295 on staslev:BEAM-1205-auto-set-enableAbandonedNodeEnforcement-in-TestPipeline into 603f4fb0493435f4eb451f52fc5c10aa12001486 on apache:master.

@kennknowles , @tgroh  I caved in to your arguments around the enforcement tests :)
Let me know if there's more or if I have missed anything.

Coverage decreased (-0.1%) to 69.79% when pulling 90fb50507a47ecd53641fc254013efd6ebe046a1 on staslev:BEAM-1205-auto-set-enableAbandonedNodeEnforcement-in-TestPipeline into 603f4fb0493435f4eb451f52fc5c10aa12001486 on apache:master.

@tgroh made some changes following our discussion.
PTAL.

Coverage decreased (-0.6%) to 69.152% when pulling 7ac01d72cf2cdb22aba9bcd00c35d503c61ac227 on staslev:BEAM-1205-auto-set-enableAbandonedNodeEnforcement-in-TestPipeline into d4db4fb2c20e46beac1a1fc0ef47ec30d9a2bd34 on apache:master.

Rebased and moved expectations to occur before throwing statements.
Waiting for tests to pass, then merging.

Coverage decreased (-0.6%) to 69.359% when pulling<PHONE_NUMBER>c4cd39d217d4f9b8dfcb41033800a6 on staslev:BEAM-1205-auto-set-enableAbandonedNodeEnforcement-in-TestPipeline into 00ea3f7d7ffbca398f8deeb1feefc1e1d635ef2a on apache:master.

Reformatted the changed code using the google-code-format plugin for IntelliJ.

Coverage decreased (-0.6%) to 69.288% when pulling caaf89708746fea5ca593fa260d21b5cb9a571c1 on staslev:BEAM-1205-auto-set-enableAbandonedNodeEnforcement-in-TestPipeline into 5fe78440bd603702661595d0d92c33c38d38a51a on apache:master.


Add Postgres setup instructions in README

I spent some time creating the Postgres tables until I found that there exists a setup script. I don't think this is documented anywhere.
It would be nice if the DB and tables were setup upon initialization though, as referenced in #96.

Codecov Report

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

@@            Coverage Diff            @@
##             master     #119   +/-   ##
=========================================
  Coverage     63.03%   63.03%
  Complexity      271      271
=========================================
  Files            66       66
  Lines          1339     1339
  Branches        136      136
=========================================
  Hits            844      844
  Misses          466      466
  Partials         29       29

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 ba577d5...f3b7e03. Read the comment docs.


Example showing connection-specific settings makes no sense

The example showing connection-specific settings is confusing because it's using the same command and members as the global settings.  How can that possibly work?  If this really is how it works, it needs more explanation.

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

ID: 3f809c02-ffa1-56c0-070b-b3879b1aa8a9
Version Independent ID: 687b7248-4b32-a4a5-5dbb-9b2f152503cd
Content: Get-DnsClientGlobalSetting (DnsClient)
Content Source: docset/winserver2022-ps/dnsclient/Get-DnsClientGlobalSetting.md
Product: w10
Technology: windows
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


IPN parser does not parse SellerId

The IPN parser does not parse the SellerId and other attributes for IPNs such as Capture notification.
INotification only provides the notification type and no other common information.
We would like to obtain the SellerId and use it to verify that some random other seller did not set our URL for the IPNs.
Since we pass in the OffAmazonPaymentsServiceConfig when creating the NotificationParserFactory, I thought maybe the SDK was checking the sellerId internally, but this does not seem to be the case. As it is, we would have to parse the message again separately just to get the sellerId....

This item is fixed in the 2.2.1 Maintenance Release.  I apologize for any inconvenience this may have caused.
Notification notification = NotificationFactory.parseNotification(headers, respBody);
...
System.out.println("sellerId=" + notification.getMessageMetadata().getSellerId());


Implement OnyxSelect flyout opening direction and alignment

Open question

Should we also support to manually set a fixed open direction?

Why?
The flyout of the OnyxSelect should detect in which direction it should open (top or bottom), depending on in which direction more space is available.
It should also support to align left/right if not using the full parent width
Design
Figma
Acceptance criteria

[x] if more space is available above the select, its opened to the top
[x] if more space is available below the select, its opened to the bottom
[x] by default the flyout uses the full width of the select
[x] the width can optionally be limited and aligned left/right

Definition of Done

[x] covered by component tests
[x] screenshot tests are updated
[x] follow-up tickets were created if necessary -> #1764
[x] updated version + documentation is deployed
[x] Storybook can show the feature
[x] Storybook code snippet of new/changed examples are checked that they are generated correctly
[x] Set task to In Approval

Approval

Storybook top open direction
Storybook Right alignment
[ ] approved by designer @jannick-ux
[ ] approved by dev

Implementation Details

ViusalViewport
don't change the direction on scroll -> try out how it feels.
prefer to solve the width things with CSS only

@MajaZarkova When using the select with search, the flyout width increases to more than the content width, is this intended?


RFC011 - Custom data sources

RFC 011 - Custom data sources

[ ] Approved in principle
[x] Under discussion
[ ] Implementation
[ ] Shipped

Summary
A replacement for DataSourceAttribute, which is currently .NET Framework only, to make it more modern and bring the support for .NET Core.
Motivation
User needs for DataSource to be supported on .NET Core. https://github.com/microsoft/testfx/issues/233
Detailed design
Instead of having a single giant attribute like DataSourceAttribute, we will divide it into multiple attributes.
XmlDataSourceAttribute
The attribute constructor will have three parameters, (string xmlFilePath, string tableName)
Previously with DataSrouceAttribute, the code used to look like this:
[TestMethod]
[DeploymentItem("MyTests.Configuration.xml")]
[DataSource("Microsoft.VisualStudio.TestTools.DataSource.XML", "|DataDirectory|\\MyTests.Configuration.xml", "MyTable1", DataAccessMethod.Sequential)]
public void TestMethod()
{
    var foo = TestContext.DataRow["MyData"]);
}

XML file:
<?xml version="1.0" encoding="utf-8" ?>
<Rows>
  <MyTable1>
    <MyData>true</MyData>
  </MyTable1>
  <MyTable1>
    <MyData>false</MyData>
  </MyTable1>

  <MyTable2>
    <MyData2>false</MyData2>
  </MyTable2>
</Rows>

Now, it will be:
[TestMethod]
[XmlDataSource("MyTests.Configuration.xml", "MyTable1")]
public void TestMethod(bool myData /*parameters can't be validated at compile-time*/)
{
    // The parameters are the alternative of TestContext.DataRow["MyData"]);
}

The attribute will implement ITestDataSource and may be in a separate package.
CSVDataSourceAttribute
The attribute constructor will have one parameters, (string csvFilePath)
Previously with DataSrouceAttribute, the code used to look like this:
[TestMethod]
[DataSource("Microsoft.VisualStudio.TestTools.DataSource.CSV", "TestData.csv", "TestData#csv", DataAccessMethod.Sequential)]
public void TestMethod()
{
    var header1 = TestContext.DataRow["CsvHeader1"]);
}

Now, it will be:
[TestMethod]
[CSVDataSource("TestData.csv")]
public void TestMethod(string header1, int header2, ... /*parameters can't be validated at compile-time*/)
{
    // The parameters are the alternative of TestContext.DataRow["CsvHeader1"]);
}

The attribute will implement ITestDataSource and may be in a separate package.
SqlDataSourceAttribute
TBD
Concerns:

Security: What's the best way to store connection strings?

Drawbacks
Why should we not do this?
Alternatives
What other designs have been considered? What is the impact of not doing this?
Compatibility
This change isn't a breaking change. But it brings functionality to .NET ("Core") that is done differently on .NET Framework
Unresolved questions
What parts of the design are still TBD?

Linking https://github.com/microsoft/testfx/blob/main/docs/RFCs/007-DataSource-Attribute-VS-ITestDataSource.md.
Ideally we want the new attribute to implement the same interface as the other source attributes. Outside of technical constraints, we would also like to unfold arguments and declare them on the method signature instead of using the TestContext.DataRow[".."] syntax.
Finally, I think we should drop the DataAccessMethod and consider these sources the same way we consider other parameterized tests.

@Evangelink Thanks. I updated the proposal a little bit.

One consideration for not having TestContext.DataRow[...] is how the users are using data sources. Are there any valid scenarios where the XML could contain much more data than needed?
If we go with parameters instead of TestContext.DataRow, users will have to declare everything in the correct order, and it will be noisy if part of the schema is unused for the purpose of the test. I'm not sure if that's how users are using DataSource though.
In that case, we may be able to have the test parameter as DataRow? e.g:
[TestMethod]
[XmlDataSource("MyTests.Configuration.xml", "MyTable1")]
public void TestMethod(DataRow dataRow /*alternative of TestContext.DataRow*/)
{
}

That's a good remark, we probably need wrapper type yeah.

@Evangelink Is it "our own new wrapper type"? Or could we just use "DataRow"? (available in all .NET versions without extra packages)

Thanks for your input @fforjan. I read it as our solution wouldn't be helping you much, am I right?
Giving some more thoughts on it, I am actually wondering if this feature is actually bringing much to our users.
I mean that either we will end up in a too generic solution where user is basically left with a too generic argument type or something too narrow where the user ends up having to write its own extension.
For XML, I don't think we are really providing much value as (nearly) everything is already available out-of-the-box. For CSV, we would need to either re-implement a parser which is a no-go or rely on some library from community (with licensing/security/support concerns) but even there, I have the feeling we will either have too generic type or too narrow which means we don't have the right abstraction. Finally, for databases, this seems to be even more true.
I am more tempted to invest some time in offering a different fixture experience that would give the flexibility to use the library and srong typing the user wants.
I'll sleep on it.

I read it as our solution wouldn't be helping you much, am I right?
yes, the painful part in my solution was to implement the GetData method and as we were successful, we didn't even think you were looking to move away (and raise such request!) from the original DataSourceAttribute from  the .net Framework - we were using it but had to move away to allow the tests to run in both fwk and core.
In our case, we decided that the xml file would be read from the filesystem as such, relative to the cs:

We could have decided to read the file where the DLLs is executed or we could also have read the xml file as a stream from a resource from an assembly. (in the end, I would reference the DeploymentItem here, would the file be a deployment item too?)
_ I have the feeling we will either have too generic type or too narrow which means we don't have the right abstraction. Finally, for databases, this seems to be even more true._
I would agree with you, same  if you unit test is  a sqllite file, where do you read it (see the previous remark on the deploy item) ? Where do you store the credential for a real DB ? etc.. I think everyone would have a different answer.
What would be helpful potentially is more end to end sample/simple for the different use case - XML/CSV/DB- which people can adapt from ?

What would be helpful potentially is more end to end sample/simple for the different use case - XML/CSV/DB- which people can adapt from ?

That's exactly what I was saying to @Youssef1313 yesterday on our offline sync. Thanks for the confirmation.
I'll move forward by closing as won't fix and if we get any good motivation from users we will revisit.


[TASK] cosmos extraction display

Describe the task
Implement into the HMI PDF extractions saved by TDS, like images and text.

@YohannParis where should these show up? Are there designs for this?

@YohannParis where should these show up? Are there designs for this?

This task needs to wait for TDS to implement their counter-part: #1774


add init.py to avoid ModuleNotFoundError when executing metrics-utili…

I was trying the steps with Elijah and found the following error Traceback (most recent call last):   File "/var/lib/awx/venv/awx/bin/metrics-utility", line 8, in <module>     sys.exit(manage())              ^^^^^^^^   File "/var/lib/awx/venv/awx/lib64/python3.11/site-packages/metrics_utility/__init__.py", line 28, in manage     utility.execute()   File "/var/lib/awx/venv/awx/lib64/python3.11/site-packages/metrics_utility/management_utility.py", line 57, in execute     self.fetch_command(subcommand).run_from_argv(self.argv)     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/var/lib/awx/venv/awx/lib64/python3.11/site-packages/metrics_utility/management_utility.py", line 60, in fetch_command     module = import_module(f"metrics_utility.management.commands.{subcommand}")              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/usr/lib64/python3.11/importlib/__init__.py", line 126, in import_module     return _bootstrap._gcd_import(name[level:], package, level)            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "<frozen importlib._bootstrap>", line 1204, in _gcd_import   File "<frozen importlib._bootstrap>", line 1176, in _find_and_load   File "<frozen importlib._bootstrap>", line 1147, in _find_and_load_unlocked   File "<frozen importlib._bootstrap>", line 690, in _load_unlocked   File "<frozen importlib._bootstrap_external>", line 940, in exec_module   File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed   File "/var/lib/awx/venv/awx/lib64/python3.11/site-packages/metrics_utility/management/commands/build_report.py", line 11, in <module>     from metrics_utility.automation_controller_billing.report.factory import Factory as ReportFactory   File "/var/lib/awx/venv/awx/lib64/python3.11/site-packages/metrics_utility/automation_controller_billing/report/factory.py", line 3, in <module>     from metrics_utility.automation_controller_billing.report.report_renewal_guidance import ReportRenewalGuidance   File "/var/lib/awx/venv/awx/lib64/python3.11/site-packages/metrics_utility/automation_controller_billing/report/report_renewal_guidance.py", line 6, in <module>     from metrics_utility.automation_controller_billing.report.renewal_guidance.dedup import Dedup ModuleNotFoundError: No module named 'metrics_utility.automation_controller_billing.report.renewal_guidance'
This is because we were missing the init.py file which was making it not importable

@Ladas Can you please review? 🙇♂

👍


Laravel Version Requirement for 5.5

Currently, the laravel framework requirement is set to 5.0 - 5.4

Yes we need 5.5! If you want it fixed quickly submit a pr. I can't look at it before the weekend.

How can that be done ? I am using this package and I need to upgrade my system to 5.5. I am not able to upgrade my system because of this package.

Thanks. I have tagged 1.0.8 with support for 5.5 now.
Out of interest, why are you still using this package? Its using the old
Mailchimp v2 api.
Niels.
On Thu, Aug 31, 2017, 10:12 Bijaya Prasad Kuikel<EMAIL_ADDRESS>wrote:

How can that be done ? I am using this package and I need to upgrade my
system to 5.5. I am not able to upgrade my system because of this package.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/skovmand/mailchimp-laravel/issues/27#issuecomment-326223473,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABUr_pA0Q5nxRPKA5m5ffWmQo9eTk8TAks5sdmrzgaJpZM4PBq9W
.

Thank you


Docker API Proxy

Current usage of Swarm requires modification of default daemon options or Systemd socket to expose Docker's API over TCP. How about as an alternative, have Swarm be able to open a port that proxies to Docker's default Unix socket. Something like
docker run -d --restart always --name swarm-agent -p 2376:2376 \
-v /var/run/docker.sock:/var/run/docker.sock \
-v /etc/docker/certs.d:/etc/docker/certs.d:ro swarm:latest ...

Thoughts?

Sorry for the slow response - been busy at DockerCon.
We've been talking about this and I kinda like the idea.
A few thoughts:

For backward compatibility, how about leaving swarm join as is (and deprecate it), and move this new stuff into a swarm agent sub-command?
Security is a pain here. It becomes much easier to leave docker in the open (before you explicitly had to alter the docker settings and people were kinda aware of what they were doing). Perhaps we should have an easy way to generate TLS certs (similar to docker machine) and require TLS in swarm agent?

/cc @vieux @abronan @chanwit @jimmyxian

/cc @ehazlett @nathanleclaire

What would be the benefit of this approach?

It means we wouldn't have to expose the engine on TCP or worry about TLS
settings for it as well. It would also reduce the attack area as it's just
listening on the socket.
On Tuesday, July 7, 2015, Nathan LeClaire<EMAIL_ADDRESS>wrote:

What would be the benefit of this approach?
—
Reply to this email directly or view it on GitHub
https://github.com/docker/swarm/issues/990#issuecomment-119283829.


dw listperson: finally, some events!

Some person entries don't have either birth or death elements, instead one or more event elements (representing their burials, gone missing, declaration of death etc.)
So, for each event element in person,

[ ] create a cidoc E5_Event, URI: https://sk.acdh.oeaw.ac.at/[person-xml:id]/event/[n]

[ ] create respective time-span, URI: https://sk.acdh.oeaw.ac.at/[person-xml:id]/event/[n]/time-span

[ ] Then model like this:

<https://sk.acdh.oeaw.ac.at/[person-xml:id]/event/[n]> a ns1:E5_Event ;
rdfs:label "Event: [event/note/text()]" ;
ns1:P4_has_time-span <https://sk.acdh.oeaw.ac.at/[person-xml:id]/event/[n]/time-span> ;
ns1:P7_took_place_at <https://sk.acdh.oeaw.ac.at/[event/desc/placeName/substring-after(@key, "#")]> ;
ns1:P2_has_type <https://sk.acdh.oeaw.ac.at/types/event/[event/@type] .

<https://sk.acdh.oeaw.ac.at/[person-xml:id]/event/[n]/time-span> a ns1:E52_Time-Span ;
rdfs:label "event/desc/date/@when" ;
ns1:P82a_begin_of_the_begin "event/desc/date/@when"^^xsd:date ;
ns1:P82b_end_of_the_end "event/desc/date/@when"^^xsd:date .

@linxOD just create a file e.g. extra_cidoc_utily.py (or whatever) in scripts, copy code from https://github.com/acdh-oeaw/acdh-cidoc-pyutils/blob/cde8558f5718e1e809dd196f67cd3f368eb29c46/acdh_cidoc_pyutils/init.py#L545 into this file (plus needed imports) and include the function in your script
see #https://github.com/acdh-oeaw/acdh-cidoc-pyutils/issues/63

2528e55410581b21db925f7748efcc9a89977f5a and 9eeb441f8f0e32f52ed8e3399feb8b04aff7f6d8


Revisit GeoSearch API design

Potential changes to existing GeoSearch API:

change method names from camel case to snake case
consider replacing QueryRelation in Query with methods (e.g. new_within_query)
rename SimpleGeoSearcher to LuceneGeoSearcher to stay consistent with renamed SimpleSearcher

There was some confusion between @d1shs0ap and I on Pyserini vs. Anserini... so, nm, what we currently have is fine.


Lint full podspec on travis CI

We originally linted the as quick because Travis CI didn't have a version of Xcode / CocoaPods that was compatible with this before.
Changing to full will mean that CocoaPods will build the project on the lowest supported platforms and ensures all the sources work together.

Looks good dude!


ONN Mechanical Keyboard Support

I recently bought an ONN mechanical keyboard (Walmart store brand). The keyboard was good for the price but the software is as may be expected pretty bad. It would be nice if there was an alternative.
As far as I can tell there is no SDK (there's not even a driver download site that I can find, just the CD that came with it) and of course this is not a huge product line, so I fully understand it's not a priority. I make no guarantees, but I would be willing to take a look at sniffing the driver's interactions if someone can point me in the right direction for how to do that.
Also, I suspect this may be a rebrand of a Redragon keyboard.
Device Name: ONN 100004357
Device Website: No real website, closest https://www.walmart.com/ip/onn-Gaming-Mechanical-Keyboard/660379246

Please tell me what the VID and PID of the keyboard are. If it is a redragon rebrand, aurora support is not possible. I've tried this on a keyboard i own and the firmware does not support direct lighting, only writing custom lighting patterns to flash.

Looks like the VID is 3938 and PID is 1205.

I just got one of these keyboards and would like to try Aurora. I will keep an eye on updates and hopefully y'all find a way to get it working.

Forgot about this issue, sorry. The best way to potentially get this supported would be to reverse engineer it and write a controller for OpenRGB. You can find more info on their gitlab and discord: https://gitlab.com/CalcProgrammer1/OpenRGB . Although if it is indeed a redragon rebrand it is not possible to integrate into aurora as those boards don't support real time lighting.

I'm having an issue with this keyboard.  Seems Shift+W doesn't work.  Some other keys act as function keys.  Anyone else experiencing this with the Onn. Mechanical keyboard?

Openrgb killed my redragon k621 keyboard with no way to repair it. So double-check if your keyboard is compatible with openrgb.

Try Fn + CtrL + W    to reset back to regular mode.....no need to unplug it as some others have said

Try Fn + CtrL + W to reset back to regular mode.....no need to unplug it as some others have said

oh my god that worked!  you're amazing - thank you soooo much!

This reset, FN + CTRL + W, works on the Onn Mechanical keyboard, p/n 100004357, as (was?) sold at Walmart.
One day the keyboard stopped accepting regular keypresses (password failed repeatedly), as I presume it was in Gamers/ Gaming Mode. The above reset put it back into "normal" mode, working as expected. I suspect an accidental bump/ combination of keys turns on gamers mode. I'm adding a FN + CTRL + W label at the top of my keyboard!

I’m having an issue with my onn keyboard my A,S,D buttons on my keyboard moved to the shift left, right, up, down, button and I’m wanting to change them back how can I do that

DO NOT INSTALL OPENRGB UNLESS YOU KNOW WHAT YOU DOING! Openrgb killed my redragon k621 keyboard with no way to repair it. So double-check if your keyboard is compatible with openrgb.

Uhm...how the F did it do that?  It didn't update firmware- and if sending it RGB commands bricked the thing, you probably DIDN'T want the thing in the first place...  ;)


syslog-ng WARNING with --tty

After the new syslog-forwarder-less commits, starting the container with --tty now results in regular WARNING messages, as well as at startup.
syslog-ng[21]: WARNING: you are using the pipe driver, underlying file is not a FIFO, it should be used by file(); filename='/dev/stdout'

Is there anything we can do with this? Seems to be a bit of a backwards step that our users cannot use the --tty switch without some bad looking messages appearing regularly.
@Theaxiom @davidhiendl

It's the same: -t, --tty (actually I'm using -it).
A possible solution would be to check the type in a bash script and sed the config file on each startup, I have no other idea about how to solve it. Other than a few warning lines it seems quite harmless, but definitely not a nice experience for users trying out Baseimage.

The warning with "-t" or "--tty" stems from the fact that there is a difference between attaching a tty and logging to file (pipe driver vs file driver). It might be possible to solve this but since you should only see this warning during development and not when running containers in a production environment I'm not sure if it is a good idea.
In order to solve this you would have to detect the current attachment mode during startup of the container and modify the syslog configuration.

This is an annoying issue for me.  dmesg -D does not fix this problem.

@danielpclark feel free to test my PR if you need an immediate fix. For me this fixes the issue.

Trying now…
P.S. setting CONSOLE_LOG_LEVEL=1 in /etc/default/syslog-ng doesn't seem to help either.

You fix works.  The warnings stopped appearing every hour.

Oops I spoke too soon.  I'm still seeing the warning.

That is not a warning or an error. That is simply the log for the cron invocation.

I'm aware that it is neither a warning or an error.  But it is noise on the tty which is why this issue was raised.

The thing is that message actually serves a purpose (in my opinion, which is notifying you that cron was executed). The point of this ticket was the warning that was introduce a while ago when syslog-ng was changed to also log to docker logs via stdout (to allow aggregating logs). It might be better to raise a separate issue to bring attention to this?

@danielpclark that's a totally different issue, and has been present for ages. Most of us actually believe it's a good thing to have feedback about cron jobs running, so it'll probably stay like this.
If you are concerned, please open a separate ticket about that line please.

I'll be content to leave it as is.  This solution wasn't what I thought you were aiming for.  My ncurses server stats display gets distorted on the hour.  But the information there is only needed initially so it's not a huge issue.

How do I suppress this annoying message?
CONSOLE_LOG_LEVEL=1 doesn't do it for me.

@stevenmunro Which message are you referring to?
If you are talking about something like this:

Feb 22 13:17:01 507a5181d782 CRON[881]: (root) CMD (   cd / && run-parts --report /etc/cron.hourly)

It's perfectly normal and a log statement emitted by cron when performing cron tasks.

At the risk of pointing out something that seems obvious (to me?), you could simply prevent cron from running altogether if you don't need that (disable the service) or run it in a different container that isn't in charge of displaying your stats curses app, if you do need that.


 Slack token need to be encrypted

Currently, slack token are mentioned in plain text. Can you please renew the token and use the encrypted one to avoid misuse.
https://github.com/angular-ui/ui-router/blob/master/.travis.yml

Thanks for the report!


Need help with testing and maintaining project

I need help with testing and maintaining project. This project has future and I need help with improving

What do you need to test, specifically?

Is it useful package for easy creating multi language web applications on Laravel? If not, what we need to add/remove from this package?

@akalongman have you seen the PR?

Merged, thank you!

What do you need to test, specifically?
We need test it on new Laravel 5.3, also think how implement page for managing texts (something like laravel-passport which has small "admin panel" for managing tokens)

Also we need increase test coverage


Timeout when Running e2e tests on a fresh installation

Have you experienced this bug with the latest version of the template?
Yes
Steps to Reproduce

Run npx create-remix@latest --template remix-run/indie-stack
Run run test:e2e:dev

Expected Behavior
e2e tests pass
Actual Behavior
Execution times out when trying to run cy.exec as a part of cy.cleanupUser(); call after each test.
Actually, no matter which command is passed to the cy.exec call (like, echo "test"), it still times out. I see the same behavior when running test:e2e:run.
Node version: v20.2.0 (tried with 20.1.0 as well)
Cypress version: 12.13.0
OS: Mac OS Ventura 13.4 (M1)

@artykr Is this still an issue for you? Would you my retrying and get back to us?


Proposal: Start Page for easy access

Feature request
To show start page when excalidraw desktop app opens.

Allows to open recent files quickly
Decide whether to start a new sketch or to open existing from file

Sample Wireframe:

@lipis @pinussilvestrus Would like to hear your opinion on this.

Thanks for your thoughts! I'm wondering whether this is Something better to have for the Main Application instead of the Desktop App?

I'm wondering whether this is Something better to have for the Main Application instead of the Desktop App?

One of our main principles (not published publicly yet) is to reduce friction and time-to-first-draw as much as possible, and that means no welcome screens.
It's tempting (in the early days, I wanted to implement one as well), but currently it would add more cost (UX-wise) than it would add.
Only really valuable thing would be the recently-edited section. We'll figure something out when we support tabs/projects in the future.

The motivation behind opening this discussion is that, people usually download something only when it able to provide more functionality, if it is used numerous times a day or want to offline. In such cases, it would be common to switch between multiple documents/sketches.
Having tabs in the main application might be an answer but still, for a native app, providing options from menus to open recent files is a common behaviour, if not a start page.

That makes sense. I can see how having a list of recently opened documents would indeed be very useful for a desktop app.
That said, it'd have to implemented here, not in the web app.

Maybe a list / menu option for recently opened files would be a good first iteration.
I'd also not go for a huge starting screen for that purpose to keep the quick-start feeling

For reference, here’s what Xcode opens on launch:


Multiple View Layout Framework7 Hybrid Application

I have 2 different layouts. One for before login and one more for after login.
I already read the instructions here. http://framework7.io/docs/views.html
And still cant understand why my apps become like this:
http://i.stack.imgur.com/UU0ZJ.png
It display 2 views at once. Can scroll fully throught both views.
This is the js involved:
var mainView = myApp.addView('.view-main');
var anotherView = myApp.addView('.another-view');
//to call anotherView after login
mainView.router.load(anotherView);

Just add "display:none" to hidden view by default, and then hide login view and show after login view. You can use something like:
$('.view-main').show();
$('.another-view').hide();

etc

Shouldn't this be the default behaviour? It seems very strange that multiple views in one page all get displayed by default.

Split view or views in tabs

var mainView = app.addView('.view-main'); 2.0.0 error
0
app.addView is not a function
var app = new Framework7({
root: ‘#app’,
theme: theme,…Like


Version control

bower, npm, something! same for the other plugins
pretty please? ;p

+1
I need it to have version tag so that I can use it with rails-assets.org


Settings UI: Add modal for viewing invitation details

This PR introduces a new UI feature that allows users to view detailed information about individual invitations in the Settings > Invitations section. Users can access this feature by clicking an eye icon button, which includes a tooltip labeled "View details."
Key Features

The feature is available for both email invitations and reusable invite links.
It allows all users, including admins, to view invite details, even for invitations they did not create themselves.
The design is inspired by the user profile modal for consistency.
The Resend and Revoke buttons are functional within the new invite details modal too.

Backend Changes

Two new REST endpoints were introduced: get_invite_details and get_multiuse_invite_details.
New backend functions were added to retrieve and return invite properties as a dictionary.
The new API endpoints are documented in openapi/zulip.yaml.
Multiple backend tests were added in test_invite.py to validate the new API functionality.

Fixes: #31244
Screenshots and screen captures:
Tooltip:

Invite Details Modal:

Demo:
https://github.com/user-attachments/assets/3919392d-a6dc-4210-b1fa-e4c0153d10b3
Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

@alya This PR is up for review. Please take a look when you get time.

Thanks! Could you please update your commit message to match the commit style guidelines, including proper capitalization/punctuation.
You can post a comment when that's done and the PR is rebased.

@alya Done, but few commit descriptions are >76. Hope it works, otherwise let me know!

Thanks for working on this @reharsh! The individual commits in this PR don't seem to be split into pieces in any coherent way; each one has changed related to other commit titles. Possibly you'll want to squash them, but I think it'll help you to get practice with a smaller preparatory PR, so first please put up a PR for https://github.com/zulip/zulip/pull/31698#discussion_r1800265070, with a commit for that change and a similar one for multi-user invites, and we'll work on your development habits there, and then you can come back to this once that's done.

This still seems very oddly structured as a commit series -- tests and API docs should be added in the same commit that introduces the functionality to be tested, for example.
Check out PRs by core team members for examples.

@timabbott , I've squashed api_docs, tests, and backend functionality into 1 commit so it passes all tests by itself and also reordered to ensure backend commit is before frontend commits, we can also squash tooltip commit into frontend commit if you want.

@timabbott I've wrapped the commit lines under 70 characters, along with the rearrangements mentioned above. Hope it looks good now!


Adding camvid dataset

Hi,
I am starting doing some research on image segmentation. So, I was thinking that it could be useful for everyone to add those datasets on fuel. Here we have the camvid dataset [Camvid]. Currently this dataset is only available on a github page (https://github.com/alexgkendall/SegNet-Tutorial/tree/master/CamVid). I was thinking to create a zip file and upload it on the web, in order to make it downloadable by fuel (After asking the permissions of the authors). Is there any ftp server where I could upload it eventually ?
[Camvid] http://mi.eng.cam.ac.uk/research/projects/VideoRec/CamVid/

Hello,
I am also interested in this data set, and several others that aren't in the fuel examples directory. A few questions/comments.
Questions for fuel maintainers:
The SegNet tutorial and (presumably) data set the OP linked to are released under a CC non-commercial license. Fuel is released under an MIT license, which is incompatible (allows commercial usage). But since the data itself is not being hosted in fuel, is this a problem? Do the maintainers of fuel have a policy about what kind of license a data set must be released under to include it as an example?
It seems like it is a slippery slope to start adding many data sets to the examples, unless the repository maintainers intended the fuel repo to become a catalog of data sets. I believe the examples are intended to illustrate how to create your own loaders/converters in fuel. Is this a correct interpretation, or do you welcome PRs with new data sets?
Question for maintainers/regular fuel users:
What is the recommended workflow that folks should use to share their own fuel loaders/converters for popular datasets that don't make it into the fuel examples directory - should the fuel repo be forked? Or should we create an external package like camvid_fuel that imports fuel and then defines some loaders/converters, and can then be used like import camvid_fuel; X_train, Y_train = camvid_fuel.load_data()?


Repo appears as updated 22 days ago!

This Repo is obsolete and yet it is still at the top of the searches on Github.
It has not been updated for 18 months and yet Github shows 22 days!
If you no longer wish to maintain this repo and in order not to mislead us, please archive it. And encourage users to use:
https://github.com/hardevine/LaravelShoppingcart which is (for now) maintained!
Guys, if you don't want to take care of your repo anymore, leave a message on README and pass it on to motivated people, we understand that you can have other priorities, but don't put thousands of users in the disarray.
Thank you

You can find a list of maintained forks here in the wiki of this repository. Please leave your fork there if you maintain one.


vulkaninfo: fix device ext list having bad items

vkconfig exposed an issue in the vulkan-loader which resulted in two different sizes
returned from vkEnumerateDeviceExtensionProperties(). A full fix requires a change to
the loader, but this commit stops the issue from happening in vulkaninfo. The issue is
because the count might be less on the second enumerate call, so resizing the vector
again trims the extraneous elements away.
Change-Id: I808d1fd13868711675aae5c91ea5100ec8cbc316

Vulkan-Loader issue in question: https://github.com/KhronosGroup/Vulkan-Loader/issues/403

If pPropertyCount is smaller than the number of extensions available, VK_INCOMPLETE will be returned instead of VK_SUCCESS, to indicate that not all the available properties were returned.
The spec for basically all enumerate functions is that the second call will return VK_INCOMPLETE if the count ever increased, which this code handles.

Works for me then.


How to close the socket and stop ffmpeg?

stream.wsServer.close() - as suggested in one of the closed issues, doesn't stop the FFmpeg. The FFmpeg(s) keeps running in the background. How can I close the socket and stop FFmpeg, too?

 You can see it instantiated in the code here.

You would be :zap: if you submitted a pull request to create a stream.end() method that did all this stuff automatically...
As I've mentioned in some issues before, this was built as a Proof-of-Concept a few years ago and I'm not actively maintaining it. I hope it inspires you to do something even better!


Use ? syntax instead of ReturnIfAbrupt

See latest ES spec for usage and examples

I haven't modified specs as indicated in another issue but I've dropped all ReturnIfAbrupt
http://webreflection.github.io/Object.getOwnPropertyDescriptors/

LGTM


Add support for scheduled emails

Coverage decreased (-0.29%) to 83.0% when pulling 2a1d6208332105ed6e8468aae29ab0119a011875 on ondrejcernos:oce-GD-42850 into cebb92198d670abd746dd25eb4c805e40314032b on martiner:master.

ScheduledMailBuilder/ScheduledMail - what are types of to and bcc? Email addresses? Please add javadocs and consider to rename it to something more descriptive. Like toAddress?

I would use some javadoc for recurrency

[ ] add some info into readme

[ ] Please merge ScheduledMailBuilder with ScheduledMail (with retaining handy add* methods etc)

Coverage decreased (-0.32%) to 82.96% when pulling b5b1fbf0068266453271571d4a4afaf528a816da on ondrejcernos:oce-GD-42850 into cebb92198d670abd746dd25eb4c805e40314032b on martiner:master.

Coverage decreased (-0.32%) to 82.96% when pulling f137745893a115bcf18a0daba519f5d5c31c8be0 on ondrejcernos:oce-GD-42850 into cebb92198d670abd746dd25eb4c805e40314032b on martiner:master.

Coverage decreased (-0.41%) to 82.93% when pulling 68980835f53fc3eb41303c47db7ebceb2e1f3817 on ondrejcernos:oce-GD-42850 into a750c453a8b040ba49c45287e6ea699dfed145ad on martiner:master.

close #158


use streaming/iterators, swap eyes, fix audio, add option to use depth video file

I changed the program to work with iterators, which means everything is processed frame by frame and my PC can convert a whole movie without running out of memory.
While I was doing that, I also set the correct FPS for the output video,
fixed the audio (I think moviepy doesn't want to write over a file it's also reading from),
and swapped the views so that the left view is on the left.
I got rid of the depth data cache too. Instead I added an option to read the depth map from a video file.
Sorry about putting so much stuff into the PR. I started with the streaming/iterators and changed some other stuff along the way.
I noticed too late that the fixing_app branch is newer. It should be possible to port these changes over.

@silphendio thanks for your improvements. I checked your changes and it looks fine but can I ask you how you tested your solution?
Is the audio and codecs copied correctly, and is the final video supported by VR headsets, which are the target platform for this project? Unfortunately, I don't have time to test the program at the moment, so it would be great to know how you tested the program?
I haven't updated this project for a long time and I see a lot of changes in my fixing_app branch, although I wouldn't like to merge it into the main branch, since it looks very draft. So it would be great to know about tests, so that I can merge your changes and delete the fixing_app branch.
Thanks.

I converted a few videos. The longest was over 30 minutes (where the old version ran out of memory).
I also tried to produce depth videos, and then make new 3d videos from that depth video. That worked too.
I'm not sure how to properly test this though. It's entirely possible that there are some bugs left.
The resulting video is always saved as mp4, audio is converted to aac.
The final video is just left&right images concenated together. All VR video players can play that. I changed the sides, so the left image is actially left, but even the cross-eyed view is supported by most players.
I haven't really tried the fixing_app branch, but I looked at some of the code.
It has a faster Stereo-conversion. I wasn't sure if I should commit that, but I did it just now in case you delete that branch.


[nrf noup] ci: scripts: add quarantine file

fixup! [nrf noup] ci: scripts: add quarantine file

Shouldn't this be paired with sdk-nrf PR for manifest update?

hmm... This change should have no effect at all on sdk-nrf CI, just affecting sdk-zephyr's CI

hmm... This change should have no effect at all on sdk-nrf CI, just affecting sdk-zephyr's CI. But we will need nrf updated, yes. I will create on

I am asking because the sdk-nrf PRs are also a method to inform that sdk-zephyr is moving forward in sdk-nrf west.
In this case this is not a problem, but if you have some changes that may conflict and you push PR basing on the current sdk-nrf you may end up with conflicts, because sdk-zephyr has extras on top, and then it is worth seeing what is bringing the change in and whether it is safe to rebase your sdk-zephyr PR, or there are extras that you need to wait for.


Added support for logging to a NATS broker

http://nats.io/

This is great! Thanks for contributing! I'll try to release a new version soon with this.

 I found this tool very useful for prototyping.


[Ingress] Do not recreate octavia resources when the controller restarts

What this PR does / why we need it:
If the ingress resource doesn't change, nothing needs to do when the
ingress controller restarts. In order to do that we need to store
the ingress resource version information in the load balancer's
description field.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #311
Special notes for your reviewer:
With this PR, when the ingress controller restarts, for existing ingress that not changed, you can see the log below:
INFO[0000] ingress created or updated, will create or update octavia resources  ingress=default/test-octavia-ingress
DEBU[0000] loadbalancer exists                           name=k8s-default-test-octavia-ingress-lb
INFO[0000] loadbalancer active                           id=d15108bd-1f4a-49d5-8e8a-5c5fb80720d9 name=k8s-default-test-octavia-ingress-lb
INFO[0000] ingress not change                            ingress=test-octavia-ingress

Release note:

The octavia ingress controller stores the ingress ResourceVersion in the description field of the load balancer in Octavia in order to avoid any change unnecessarily.

/ok-to-test

/test cloud-provider-openstack-acceptance-test-flexvolume-cinder

Please add a release note, especially since this changes default behavior and adds an update method.

/test cloud-provider-openstack-acceptance-test-keystone-authentication-authorization

@hogepodge done

/ok-to-test
/lgtm
/approve

/retest

/test pull-cloud-provider-openstack-check

/test pull-cloud-provider-openstack-check


fix(#1320): send all values of same key in post req #1355

fixes - #1320
Currently, if multiple data keys of the same name are sent in a request, bruno only takes the last one
This is because params key/value pairs are stored in an obj, so it gets overwritten with new values (old value is discarded)
Fix:

Updated the code for prepare-request to check if the key is already present In resultant obj, convert the val to arr of values

for the below request:

item is an arr with 2 values

IMO this change should work fine
Additionally, the response formatting might need changes if we want to show the response arr instead of each items in arr separately

Closing this PR as this has been handled by #2964


编译不了呀

[INFO] ------------------------------------------------------------------------
[INFO] Building Coding IDE Backend 0.1.1-SNAPSHOT
[INFO] ------------------------------------------------------------------------
[INFO]
[INFO] >>> spring-boot-maven-plugin:1.4.0.RELEASE:run (default-cli) > test-compile @ backend >>>
[WARNING] The POM for net.coding.ide:frontend:jar:1.0.0 is missing, no dependency information available
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 1.790 s
[INFO] Finished at: 2016-11-14T21:11:11+08:00
[INFO] Final Memory: 17M/227M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal on project backend: Could not resolve dependencies for project net.coding.ide:backend:jar:0.1.1-SNAPSHOT: Failure to find net.coding.ide:frontend:jar:1.0.0 in https://repo.maven.apache.org/maven2 was cached in the local repository, resolution will not be reattempted until the update interval of central has elapsed or updates are forced -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException

从错误信息来看，是没有执行 build 操作
./ide.sh build

ide.sh  没有个脚本呀

参考这里：https://github.com/Coding/WebIDE


React FloatingFocusManager does not perserve tab order with FloatingPortal

Describe the bug
Two bugs actually:
1. With guards={true} Focus gets trapped and cycles between the Floating Element buttons and the last focusable element on the page. It feels odd, but maybe it's expected behavior.
https://github.com/user-attachments/assets/cd384f6e-4978-4f09-93a6-2bb8fac19064
2. With guards={false} FloatingPortal's preserveTabOrder does not appear to be honored
https://github.com/user-attachments/assets/1726747c-fa3c-4c5a-a47d-fa0de887b301
Steps to reproduce the behavior:
Broken in 0.26.0
https://codesandbox.io/p/sandbox/broken-new-version-dnl2hw?file=%2Fsrc%2FApp.tsx%3A49%2C26
Working in 0.23.0
Using 0.23.0 cause it's in the Dialog example and just happened to be working.
Note: This does seem to also break when guards={false}, but it does work as expected without it.
https://codesandbox.io/p/sandbox/wonderful-rumple-4kqnz4?file=%2Fsrc%2FApp.tsx

Focus on the Delete Baloon Button
Hit Enter
Tab through the elements on the page
See error described above.
Turn on and off guards to see the various behaviors.

Expected behavior When focusing on the Delete Ballon Button, you should be able to hit enter, tab into the floating element content, tab through the floating element buttons, and then tab to Button 2. You should be able to shift Tab back into the Floating Element Content button.
Context:

MacOS
Chrome
0.26.20

To unblock my project, I'm just going to downgrade FloatingUI, but I would like to be able to use guards false also eventually!

This is because, since the floating element doesn't close when focus leaves it (like it does by default with non-modal focus management), eventually the focus can end back up inside the floating element as the focus moves to the end of the DOM tree which is where it's located, which starts the loop.
One workaround is to apply the inert attribute to the floating element once focus leaves it, but this is kinda tricky to deal with and I'm not even sure how the library would handle it internally if fixed

Thanks for the fast reply! Focus moving to the end of the DOM tree and looping was my theory, so thanks for confirming.
I'll try the workaround!
For the library fix, would it be as simple as just implementing the inert fix if Portals are being used and the closeOnFocusOut prop is false?

The main issue is shift+tab for instance won't allow you to get back inside the floating element as you may expect if it doesn't close. While pointer users can still access it, keyboard usage will need to traverse back to the triggering element and then move forward.

I hear ya. We ended up reusing a workaround that we used prior for a different component using the https://github.com/KittyGiraudel/focusable-selectors package. I'll throw together a codesandbox tomorrow and you can see if you'd be interested in adapting it.


Compile without Atlas

This is a much neater version of #39 as discussed with @mauricefallon that does not require user intervention/editing of the tobuild files. It converts atlas-collection and bdi-walking to use CMake. Then the entire atlas-collection is skipped when atlas has not been checked out as part of the externals build. Furthermore, flycapture and ptgrey that are only needed for atlas are also moved into atlas-collection.

@patmarion is this a potential resolution to the previous discussion?

My 2 cents, I think we could benefit from a more detailed discussion of build systems before introducing changes like this.  I think this change does not follow good cmake practices (usage of add_subdirectory for independent projects is not correct, regardless of whether it works or not).  Also, this pull request introduces odd design, such as requiring atlas proprietary code in order to build pointgrey and flycapture projects.  Aren't those separate projects that do not depend on atlas?
I feel like I'm the only one reviewing these changes, maybe you could request a few others to review too.
Maybe we can use the README to document the fact that open-source users must comment out the atlas-collection, drivers/flycapture, and drivers/ptgrey from tobuild.txt.  Then we can take our time to discuss the necessary build system changes to support configuration options.

Another thought, I think the state-sync program is used for atlas simulation, too.  Could we get drc-state-sync-atlas to compile without the atlas proprietary driver, then Atlas simulation could be supported in the open-source version of the build.
If that works out, then all we need to disable is the proprietary drivers for atlas and pointgrey.  A separate topic, maybe we look into the pointgrey license issue, perhaps there is a way we can distribute the necessary files to support this driver in the open-source build (disabling it is still a nice option, though)

Another thought, I think the state-sync program is used for atlas simulation, too. Could we get drc-state-sync-atlas to compile without the atlas proprietary driver, then Atlas simulation could be supported in the open-source version of the build.

state sync is not necessary for simulation. (If it is now, that can be easily remedied.) Everything is the atlas-collection can be disabled without effecting simulation. Atlas simulation is currently supported in the open source build. The control directory is now built before atlas-collection for this purpose
Since the pointgrey camera are on the atlas robot it seemed to make sense that it could be grouped together with the atlas driver.

Ok, I wasn't sure, I just looked at config/atlas_sim_mit/robot_v5.pmd and saw that it launches drc-state-sync-atlas.

As a possible CMake design, see externals/libbot/CMakeLists.txt for an example of using ExternalProject to build subdirs as individual projects.

Ok, I wasn't sure, I just looked at config/atlas_sim_mit/robot_v5.pmd and saw that it launches drc-state-sync-atlas.

I can't test this now but it's my understanding that isn't not necessary

Pat, thanks for the feedback and pointer to libbot - I'll have a look at it and will update this PR. Your feedback and expert insights are much appreciated :)

@patmarion I've now declared a macro to add external projects and set the dependency before using the macro. Thus, it is not as neat as the externals/cmake/externals.cmake but more minimal. Is this okay or where/how can I amend to improve? Thanks! :)

Ready to go ahead and merge?

Thanks for all your time, feedback, and suggestions Pat!


New package: MilagroCrypto v0.201.1

Registering package: MilagroCrypto
Repository: https://github.com/blenessy/MilagroCrypto.jl
Created by: @blenessy
Version: v0.201.1
Commit: caaa2f2e7fd29f30c3ed09a95fc90da3cce9e209
Git reference: master

[noblock]
Replaces https://github.com/JuliaRegistries/General/pull/13283 .
The rationale for why we I would like the version number to be 0.201.1 can be found here.

That's fine, ping me in 3 days.

ping as requested @DilumAluthge .


Raise error on invalid Markdown links

EIP-1491 has [WIP] under multiple sections. This is rendered as [WIP] but I'm not sure it is valid or not. Ideally a Markdown parser should be run on the files too in case it reports any failures?

What sort of markdown failures would we want to capture? It seems like [WIP] is valid? I would think that Jekyll would catch any markdown errors. But it would make sense for eipv to validate things like the "are the template sections present", etc.


Flex: Add location group schema and update stop time schema

Describe the problem
We want to be able to support Flex as a part of #1721, we need to add a schema for the new file location_groups.txt and update the stop times schema to include new fields and modify stop_id from required to conditionally required.
Proposed solution
New schema. Test by checking if:

foreign_key_violation is triggered when location_group_id is defined in stop_times.txt but not in  location_groups.txt. Test with modified-flex-feed-4226672-missing.zip. I removed location_group_id 4226672 from location_groups.txt in https://data.trilliumtransit.com/gtfs/islandtransit-wa-us/islandtransit-wa-us--flex-v2-TEST.zip to test this.

Check that missing_required_field no longer triggers when there's a missing stop_id in stop_times.txt. Check with https://data.trilliumtransit.com/gtfs/islandtransit-wa-us/islandtransit-wa-us--flex-v2-TEST.zip

Run performance tests on the implications of adding/checking additional fields in stop_times.txt

Alternatives you've considered
No response
Additional context
No response

Tasks:

[ ] Add location_groups.txt schema
[ ] Update stop_times.txt schema

[ ] Modify stop_id from required to conditionally required
[ ] Add location_group_id
[ ] Add location_id
[ ] Add start_pickup_drop_off_window
[ ] Add end_pickup_drop_off_window
[ ] Add pickup_booking_rule_id
[ ] Add drop_off_booking_rule_id

[ ] Add validation for the all the conditionally required fields (stop_id, location_group_id, location_id, start_pickup_drop_off_window, end_pickup_drop_off_window)
[ ] Unit tests

Blocked until #1750 is completed

As part of QA for this issue:
Test #1776 and #1761


Fully support generators for Chain

This fully supports generators and allows to iterate multiple times over the services, even if they were passed through a generator.
I looked into using CachingIterator but it would require to be rewound before every iteration, additionally it seems to call the next value of the iterator a bit too early.
Follows #111.

Thanks @willemstuursma 👍

Thanks. Could you release as 2.5?

@willemstuursma Done!


Additional execution properties as matchable entities.

Signed-off-by: Katrina Rogan<EMAIL_ADDRESS>TL;DR
This changes add additional parameters to ExecutionConfig which are applied to a workflow in the form of annotations, labels, rawoutputDataConfig , security context which includes the identity to use for the running the user pod.
The order in which this matchable entities are used are as follows

Execution request directly passing this info
Launch plan spec containing this info
Matchable attribute containing this info .
config from you values.yaml for flyteadmin
Flyteadmin defined defaults

The IDL change is used  in flyteadmin in the following way
https://github.com/flyteorg/flyteadmin/pull/378
And from Flytectl
https://github.com/flyteorg/flytectl/pull/298
Refer to the testing section of above two PR's
Type

[ ] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

[ ] Code completed
[ ] Smoke tested
[ ] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
How did you fix the bug, make the feature etc. Link to any design docs etc
Tracking Issue
Remove the 'fixes' keyword if there will be multiple PRs to fix the linked issue
https://github.com/flyteorg/flyte/issues/2070
Follow-up issue
NA
OR
https://github.com/flyteorg/flyte/issues/

Testing details available here https://github.com/flyteorg/flyteadmin/pull/378

+1 although I guess I opened this PR so can't review hah

Can we please add some description

Done. Thanks @kumare3


ci: support tests on amazon-linux

What type of PR is this?
/kind bug
Any specific area of the project related to this PR?
/area CI
What this PR does / why we need it:
This PR should allow us to build and run Falco e2e tests on our self-hosted arm64 runners
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

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

Needs approval from an approver in each of these files:

OWNERS [Andreagit97]

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

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

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

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

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

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,jasondellaluce]

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


Feature request: saving the default image sort method in settings

It seems the image sorting method is always reverted back to the default "Sort by created date by descending" no matter how many times I change the method to something else. It will be nice if the default sort method can be set in Global Settings. Thank you.

Hi there!
i'm sorry but i can't set the default sort order by name descending persists across reboots.
if i set it in the Global settings, it works, thenif i close A1111 (CTRL+C in terminal and close the browser) when i restart the sort order is back to Updated date ascending.
is there something i'm doing wrong?
i'm on Linux, any file to edit to change this for good?
Thanks for this great extension btw.

Hi there! i'm sorry but i can't set the default sort order by name descending persists across reboots. if i set it in the Global settings, it works, then if i close A1111 (CTRL+C in terminal and close the browser) when i restart the sort order is back to Updated date ascending. is there something i'm doing wrong? i'm on Linux, any file to edit to change this for good?
Thanks for this great extension btw.

@ali0une
These settings are stored in the browser and are subject to cross-domain restrictions. If you open IIB with a link from a different domain or in incognito mode, these settings will be reset. Refer to https://developer.mozilla.org/en-US/docs/Web/HTTP/CORS

@zanllp many thanks! Didn't think about that as the site settings are set to be deleted in my Firefox.


Variable name is null on NP_NULL_ON_SOME_PATH

In SpotBugs.xml an NP_NNOSP node has 2 SourceLines: SOURCE_LINE_DEREF and SOURCE_LINE_KNOWN_NULL
OSA fixes this vuln by changing the SOURCE_LINE_DEREF occurrence
To fix this issue, only the role="SOURCE_LINE_DEREF" node should be selected to extract the line number information

For instance the issue is present in arduino-1.8.19 on the first NP_NULL_ON_SOME_PATH vulnerability entry.


addValueCallback() doesn't work unless animData.loop = true

I've always wanted to create dynamic backgrounds, like this:

But wanted to harness Lottie API to have the eyes actually follow the user's cursor, which works very smoothly for only one instance but has a noticeable lag/performance hit with multiple:

I understand what I'm doing here is very taxing, but can it be confirmed whether addValueCallback() only works when animData.loop is true? This seems like odd/undesirable behavior in the instance I'd want to have an initial state (like the eye opening) then dynamically pass a new value in later, because atm it only works when the eyes are constantly flashing open and closed (since it's looping the entirety of the animation) unless I use a completely separate file which doesn't have those keyframes.

Hi, indeed for the values to update, you need the animation to be playing.
You can loop a segment of the animation instead of playing it fully so you won't have the blinking part every time

Hi, indeed for the values to update, you need the animation to be playing.
You can loop a segment of the animation instead of playing it fully so you won't have the blinking part every time

@bodymovin This is good to know, thank you! Do you know if there is a way to update the values without playing the animation? I tried to use gotoFrame() which triggered the enterFrame event, but it didn't redraw. It's the same with triggering renderFrame().
This is basically because my bodymovin export don't have any animation, but a rigged character I can animate by updating individual values (sliders). So I want to use my own requestAnimationFrame method etc.


13-vizs-text-onLandingPage

planning to create a new html page to show high level viz and website about on landing page

viz design changes and fixes are complete on the home page
ref: 15-vizs-chartAddition-mapFixes](https://github.com/bnicp/recall-near-me/tree/15-vizs-chartAddition-mapFixes


Feature request: Allow setting resolutions other than 80x24.

Hi there,
Thanks for publishing this code. Is there any way I can tell ttyrec2video to use a higher resolution output (ie larger number of rows and cols)?
I tried changing this code (lines & columns), but it didn't seem to have any effect:
class ScreenRenderer:
    font_size = attr.ib()
    # Fonts must be PIL.ImageFont instances.  (No, `attrs` validators can't be
    # used to enforce this.)  They are also assumed to be monospaced and of
    # size `font_size`.
    font      = attr.ib()
    bold_font = attr.ib()
    lines     = attr.ib(default=75)
    columns   = attr.ib(default=140)

Oh whoops! Apologies!


PowerShell Daemon/NTService

Summary of the new feature/enhancement
PowerShell as a Daemon/NTService would help with several things:

Startup time will be improved as the console host doesn't need to initialize a runspace as it exists within the Daemon
We need some sort of hosting of PowerShell for JEA scenarios which are currently handled by WinRM but is not cross platform and only supports WS-Man and not SSH based remoting
Support headless management scenarios where you just want an endpoint to execute a script

Some MSFT teams already have something like (Exchange server and others).

Exchange (unless they did something different since) was using IIS Hosted PowerShell (rather than WinRM).

Exchange continues using IIS Hosted PowerShell. I think SharePoint and Skype/Team too. We could think about single-on-system hosted solution and share with all applications.

Any additional information like this about the future of JEA is good, especially since there was no mention of JEA in the PowerShell 7 roadmap post on May 30th.  Thank You

@JasonFossen to be clear, PowerShell Core 6 already supports JEA, but only on Windows via WS-MAN.  Is your interest in JEA for non-Windows (or on Windows via SSH)?  This is a big work item that is currently not in plan for PS7.0.  Perhaps 7.1.

@SteveL-MSFT  Yes, thank you, I should have been more specific: JEA over SSH to both Windows and Linux, and also further integration with Windows Admin Center (WAC).  I have difficulty selling PowerShell to Linux people who are already using bash and Python, but SSH+JEA+WAC for both Windows and Linux on-prem targets, preferably with some pre-configured starter JEA endpoint definitions that could be pushed out through WAC itself, would be very nice.  They immediately "get" the idea when shown a diagram of Browser -> TLS -> WAC gateway -> SSH -> Linux/Windows Targets, since this is similar to what they already do for other products, but they key selling point for PowerShell would be the JEA command restrictions enforced on the targets themselves, not in WAC, using analogies from restricted bash, sudo, setuid, seccomp, etc.

Perhaps this is a reason to port WS-MAN quickly.

I've already started a discussion with WAC team on not requiring WMF and instead use PSCore6 if available as a pre-req.  Once they do that, I can see about having them use SSH if available.


chore(main): release 1.6.2

:robot: I have created a release beep boop
1.6.2 (2024-03-28)
Build System

update packages + add comment to bundle (f5d2dbd)

Continuous Integration

change node version to workflow file (f8a1471)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/Basalt-Lab/basalt-logger/releases/tag/v1.6.2 :sunflower:


Can't specify core-js specific version

Describe the bug
In the doc it mentions that you can specify core-js specific version:

But if I do so, then all core-js imports will be imported. And, also, playground will be showing config error that only number can be passed:

Playground link
https://play.swc.rs/?version=1.3.37&code=H4sIAAAAAAAAA8vMLcgvKlFQT84vStXNKla35uJKzs8rLlFILCpSsFWIjgUKAJl6BaXFGRpGxpoAkcK9%2BTAAAAA%3D&config=H4sIAAAAAAAAA0WOawrCMBCEryL7uxSt4KNHEDxEiNsaaR7sbsVQeneT2Np%2FszszHzPBizW0EwRFjJQVRyfqAy1IDMiaTBCoQDi9OjUwVmBs8CR3FAWt0Ihzskk57jzZTCBUWn6iI%2BTnksoxRT0mC5Cb%2FeGSuIP3jH%2ByNc50MVe1tyF1ebOU64c1mVnWP8b8mMrQwjzBvDHWsXxfgnlEBejepVOWJDywcRp3ac4VCrSgnFBMp%2FaEtxw61s25bmCevx%2FxGDExAQAA
You can notice that it includes imports that already are in all browsers since 2019 (my env target), like:

import "core-js/modules/es.array.push.js";

Expected behavior
Even with specific version, swc should import only needed core-js imports or doc should be changed.
Version
1.3.37
Additional context
Here is the case when I'm specifying core-js as just 3 — https://play.swc.rs/?version=1.3.37&code=H4sIAAAAAAAAA8vMLcgvKlFQT84vStXNKla35uJKzs8rLlFILCpSsFWIjgUKAJl6BaXFGRpGxpoAkcK9%2BTAAAAA%3D&config=H4sIAAAAAAAAA0WOSw7CMAxEr4K87oKPhKBHQOohouCUoOYj2yCqqnfHCS3djT0zTzPBky20E2RDjFQUj1HMB1qQMSNb8lmgAWF9OTMwNuBDTiQdioFW6IWz2mQiu0ShEAiNlZ9whPxYUiVmqEe1APm4P1yUO6TE%2BCcHH70bS9WmkLXLm2ViP6zJwgrp%2FiqPqQ6tzDPMG2Mdy90SLCMawPiunbpE8cA%2BWtzpnCtUaEVFoVFPmwhvGjrN8xdHTE0JKgEAAA%3D%3D , and it works correctly

The doc is correct. This is a problem of schema


sound: promote to main workspace

Summary of the PR
The current implementation seems ready to be promoted to the main workspace since the device supports both playback and recording, and several audio backends.
Also, test coverage is good and aligned with the coverage of crates in the main workspace.
Requirements
Before submitting your PR, please make sure you addressed the following
requirements:

[x] All commits in this PR are signed (with git commit -s), and the commit
message has max 60 characters for the summary and max 75 characters for each
description line.
[x] All added/changed functionality has a corresponding unit/integration
test.
[x] All added/changed public-facing functionality has entries in the "Upcoming
Release" section of CHANGELOG.md (if no such section exists, please create one).
[x] Any newly added unsafe code is properly documented.

v2:

fixed readme link


Apache Pekko based Streaming application is idle and not consuming from kafka topic post rebalancing

Issue: Apache Pekko-based Streaming Application Idle After Kafka Rebalancing
Description: Our application, which runs on Kubernetes with KEDA scaling based on consumer group lag, has encountered issues post-rebalancing. During testing, the application scales up and down as expected. However, after a rebalance, certain Kafka topic partitions are no longer being consumed.
We observed that while the consumer is assigned multiple partitions, some partitions are paused and never resumed, resulting in unprocessed messages. The application only resumes consuming these pending messages when there is another surge of input messages, which triggers pod scaling and, subsequently, another rebalance.

Can you isolate whether this is a pekko-connectors-kafka issue or whether it's due to the underlying kafka client?

also, we need to know which version of pekko-connectors-kafka that you are using - it is recommended to use pekko-connectors-kafka 1.1.0 if you can

it is exactly the same issue as described here. https://github.com/akka/alpakka-kafka/pull/194
it is not underlying client issue but something to do with rebalance which pauses some partition but never resume them again, i checked KafkaConsumerActor file which has poll method where for some partitions we are not calling resume method
i am using pekko-connectors-kafka 1.0.0 version. actually i am running with scala 2.12.