How is this related to rocket?

Because i am runnibg uso thru rocket it only works thru rocket byt then mods stopped working

@JMDEMO54311 USO is completely separate from rocket and VERY unstable. I don't recommend using it at all. Also, workshop mods nor USO have anything to do with rocket. They both are SEPARATE things.

Download it and look in the server it runs rocket

Is it hard to understand that USO is a 3rd party software that has nothing to do with rocket?
Just in case you still wanna stick with the idea that USO is part of rocket: there's a public download for rocket in the website. What USO does is download it and install it in your server.
This is their website: https://unturned-server-organiser.com/
This is rocket's website: https://rocketmod.net/
According to the imprints of both websites (due to german law) (USO's imprint: https://unturned-server-organiser.com/imprint.php) (Rocket's imprint: https://rocketmod.net/imprint/), USO is owned by "Deventure Tech Software Solutions" and rocket is owned by "Sven Mawby Software Solutions".
THEY ARE DIFFERENT FROM EACHOTHER. They both have nothing to do with each other. USO just uses rocket when creating the server on your device because rocket is publicly available. USO did not program it. USO is NOT stable either. Some workshop bugs are due to an external program that INSTALLS the workshop content, and fails to CORRECTLY install them.
Just to be clear on the endpoint of this discussion:
Your workshop mods not working = problem with USO, not rocket.
USO =/= Rocket.
USO -> Allows to use a GUI to host and manage servers easily. Rocket -> Allows for plugins to be loaded and includes a few extra features.

I'll just get help elsewhere


Description tab no longer exists

both tests and the test servers need updating

fixed now on ranking test servers


Reject clauses that involves returned sut for init_state

Close #142

Rebased on #122.
Regarding the warning, I agree that the current error message is not very helpful (see commit message). Having slept on it, I'm wondering whether the warning should even be displayed. For example, a file like:
type 'a t
(*@ mutable model contents : 'a list *)

val make : int -> 'a -> 'a t
(*@ t = make i a
    ensures t.contents = t.contents
    ensures t.contents = List.init i (fun x -> a) *)

would generate a correct STM file but still display a warning about the first ensures clause. The function being used for INIT purpose, it is not tested anyway. So I don't see any reason why we should inform the user that we are skipping a clause. The important information is whether there is enough information to generate the init_state function, which will raise an error if not.

Rebased on latest main.
I've also commit a new warning message.
Not completely sure about the formulation though.


Integration broken after 2021.6.3 upgrade

Confirmed this is broken for 2021.6.3 -- working fine for me when I rollback snapshot to 2021.5.5
Error message in 2021.6.3: "...Integration 'myicomfort' not found..."
Originally posted by @tumtumsback in https://github.com/thevoltagesource/LennoxiComfort/issues/25#issuecomment-856876414

@tumtumsback I created a new issue for your report as that other one turned out to be an unsupported mode.  I should (and will) update the title of that other issue.
But to your issue, what version of the integration are you running?Starting in 2021.3 HA started logging a warning for any custom components that did not report their version in the manifest file. Then in 2021.6 HA stopped loading those components.  I added the required field in version 3.0.1 (3.0.2 is the latest) of the integration. Can you check your version and possibly reinstall it thru HACS?

If you have the Home Assistant Community Store (HACS) integration installed in your HA instance it makes managing custom components easier.  However, you can certainly manage it the way you previously done it by by downloading the files and putting them in the appropriate folder.
Either way, yes if you will download the files from the latest release and replace the ones currently in use, the component should properly load when you restart HA.

I do have HACS, just not sure how to find your integration via HACS.

You will need to delete the folder you used for the manual installation first, then from the integrations tab of HACS, click the blue + Explore and Repositories button on the bottom right.

Then in the pop-up type Lennox and this integration will be the only result.

Click the integration, choose install, and after a reboot you should be back to controlling your thermostat from HA.  All of the config file entries remain the same.
3.0.3 is now the latest version.  I fixed some logic this morning to allow better HomeKit control.

It worked -- you are amazing! Where can I 'buy you a coffee?'

I'm glad it is working for you again.  Let me know if you run in to issues in the future.
I really appreciate the the offer of the coffee.  However, I would ask instead that you find an opportunity to help someone in your area of influence and pay it forward.  This integration is my way of giving back to the community for all the work that is put in to the integrations I use, as well as HA itself. I find it super cool that people are using my integration.


[suggestion] Monkey friendly examples

Hello, first thanks for the wonderful package.
Because I suck, I spend ~ 30mins figuring in this example of checking the token
token, err := jwt.Parse(t.Token, func(token *jwt.Token) (interface{}, error) {
        return []byte("AllYourBase"), nil
    })

that []byte("AllYourBase") should be my secret.
It is TOTALLY my fault, i suck, but if others agree you could change that with "token_key_goes_here"
Cheers

Agreed. For my code I opted to use a regular function and it's more clear.
// signingKeyCb is a callback for jwt.Parse. The examples show it as
// an anonymous function, but the "return secret, nil" freaked me out at
// a glance, so here it is as a full function to make things clear.
//
// The examples & docs show a type check for the signature, which is
// probably useful if multiple signing methods are in play, but this
// code only supports one, so I've decided against including it.
func signingKeyCb(t *jwt.Token) (interface{}, error) {
    secret, err := getSigningSecret()
    if err != nil {
        return "", err // TODO: annotate
    }

    return secret, nil
}

func parseJWT(tokenStr string) (out MyTokenThing, error) {
    out.jwt, err = jwt.ParseWithClaims(tokenStr, jwt.StandardClaims{}, signingKeyCb)
    ...
}

I've never been very good at writing examples.  Reading the godocs is far more useful than following the examples.  If you want to open a PR with clearer examples, I'd be happy to review them.

I've always felt this design was a bit clunky. I've seen other libraries where you just pass in the key, but there are a lot of use cases where you need information in the header in order to provide the correct key.
I've seen implementations where different signing methods are used for different type of client/operation. There's also the kid field in the spec (among others).
@tobert I also tend to use regular functions for this, though I've seen cases where the closure style is helpful. Thank you for the useful example.
I'm going to include this ticket in v4 as a reminder to clean up some of these examples.


TestMemoryAwareness.java makes hotspot_jre test fails on JDK8 Hotspot xLinux

Describe the bug
test case TestMemoryAwareness.java fails because of created container OOM
To Reproduce
make _hotspot_jre_1
Expected behavior
test case pass
Screenshots

Additional context
possible related issue in JDK-13: https://bugs.openjdk.java.net/browse/JDK-8224506
verified that set -Xmx to 128M is also a work around:
===============================================
Running test hotspot_jre_1 ...
===============================================
hotspot_jre_1 Start Time: Wed Apr 13 08:42:05 2022 Epoch Time (ms):<PHONE_NUMBER>412
Nothing to be done for setup.
variation: Mode650
JVM_OPTIONS: -Xmx128M
{ itercnt=1; \
mkdir -p "/home/luchang/AQAvit/aqa-tests-0.9.0/TKG/../TKG/output_1649839324437/hotspot_jre_1"; \
cd "/home/luchang/AQAvit/aqa-tests-0.9.0/TKG/../TKG/output_1649839324437/hotspot_jre_1"; \
"/home/luchang/JVM/snapshot/newest/openjdk-1.8.0/bin/java" -Xmx512m -jar "/home/luchang/AQAvit/aqa-tests-0.9.0/test-results/openjdk/jtreg/lib/jtreg.jar" \
-agentvm -a -ea -esa -v:fail,error,time,nopass -retain:fail,error,*.dmp,javacore.*,heapdump.*,*.trc -ignore:quiet -timeoutFactor:8 -xml:verify -concurrency:48   -vmoptions:"-Xmx512m -Xmx128M " \
-w ""/home/luchang/AQAvit/aqa-tests-0.9.0/TKG/../TKG/output_1649839324437/hotspot_jre_1"/work" \
-r "/home/luchang/AQAvit/aqa-tests-0.9.0/test-results/openjdk/report" \
-jdk:"/home/luchang/JVM/snapshot/newest/openjdk-1.8.0" \
-exclude:"/home/luchang/AQAvit/aqa-tests-0.9.0/TKG/../openjdk/openjdk-jdk/hotspot/test/ProblemList.txt" \
-exclude:"/home/luchang/AQAvit/aqa-tests-0.9.0/test-results/openjdk/excludes/ProblemList_openjdk8.txt" \
"/home/luchang/AQAvit/aqa-tests-0.9.0/TKG/../openjdk/openjdk-jdk/hotspot/test:jre"; \
if [ $? -eq 0 ] ; then echo ""; echo "hotspot_jre_1""_PASSED"; echo ""; cd /home/luchang/AQAvit/aqa-tests-0.9.0/TKG/..;  else echo ""; echo "hotspot_jre_1""_FAILED"; echo ""; fi; } 2>&1 | tee -a "/home/luchang/AQAvit/aqa-tests-0.9.0/TKG/../TKG/output_1649839324437/TestTargetResult";
Directory "/home/luchang/AQAvit/aqa-tests-0.9.0/TKG/../TKG/output_1649839324437/hotspot_jre_1/work" not found: creating
XML output with verification to /home/luchang/AQAvit/aqa-tests-0.9.0/TKG/output_1649839324437/hotspot_jre_1/work
Test results: passed: 698
Report written to /home/luchang/AQAvit/aqa-tests-0.9.0/test-results/openjdk/report/html/report.html
Results written to /home/luchang/AQAvit/aqa-tests-0.9.0/TKG/output_1649839324437/hotspot_jre_1/work

hotspot_jre_1_PASSED

Looks like a test or environment setup issue.  In Adoptium the default set up is -Xmx512m and tests passed.
https://ci.adoptopenjdk.net/job/Test_openjdk8_hs_extended.openjdk_x86-64_linux_testList_2/37/console

The test also passed with in recent run with bisheng jdk in  adoptium jenkins https://ci.adoptopenjdk.net/job/Test_openjdk8_bisheng_extended.openjdk_x86-64_linux_testList_2/28/testReport/runtime_containers_docker_TestMemoryAwareness/


Parallel publish requests are being ignored

Hi,
I'm seeing unexpected behavior whilst using pysui to run back-to-back transactions in parallel.
The scenario I saw was:

Deploy a contract (as part of a small PTB)
Deploy a contract (as part of a small PTB - in parallel with the first request)
Await finalization of first contract
Await finalization of second contract

The behavior seen was that the second request was ignored and instead returned a transaction digest matching the first request.
Questions:

Is this expected behavior when running parallel transactions (before finalization) from the same signer?
Except for serializing my requests, is there anything I can do to avoid the risk of a transaction being ignored/rejected?

I am using the default transaction type (i.e. WaitForEffectsCert).

Interesting
Please post the minimal amount of code to reproduce the error.
I do know that in Sui 1.33 they removed WaitforEffectsCert before returning and with that you should poll on the transaction digest you receive, but that you are saying you get the exact same transaction digest does not seem correct.

Thanks for getting back to me.
I will put together something that demonstrates this in a minimal way for you.

Anything on this? The way to confirm a Tx (prior to any changes to pysui) is to take the digest from the result and then do a fetch on that transaction.
But that you said you were getting the same transaction result still interests me. Look forward to the use case that replicates the issue.

Implemented a SerialTransactionExecutor in 0.74.0
Here are the initial docs

@cottrela Any improvements (should be) using SerialTransactionExecutor?


feat(dev): add upx v3.96

upx is itself compressed with upx and it appears that strip exits with an error code when a binary has no section...

upx is already packaged by Debian and Ubuntu as upx-ucl


Fix 12507

Brief Information
fix the bug #12507
This pull request is in the type of:

[x] bug fixing
[ ] new feature
[ ] others

What does this PR do?
Axis min and max values set by functions are not equal to slider min and max values, this pr may fix it
Fixed issues
#12507
Details
Before: What was the problem?

After: How is it fixed in this PR?

Usage
Are there any API changes?

[ ] The API has been changed.

Related test cases or examples to use the new APIs
NA.
Others
Merging options

[x] Please squash the commits into a single one when merge.

Other information

the bug has fixed at next branch


CI: fix filename for suse-java-buildpack.

Description

This PR adds an exception for filepath of suse-java-buildpack while bumping its release image. Since packaging for java buildpack is different from others, we can't simply rely on filepaths from the buildpack package.
Also see: https://github.com/cloudfoundry-incubator/kubecf/pull/952
Motivation and Context

This creates a problem while bumping buildpacks and creates a commit like this https://github.com/cloudfoundry-incubator/kubecf/pull/851/files which is wrong.
How Has This Been Tested?

https://concourse.suse.dev/
Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code has security implications.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

I just wonder if it would be better to change the name of the zip file in the bosh release to make it consistent. This way we wouldn't need any special handling here. I don't have a strong opinion though. Approving.


[feature] (partition) Dynamic partition behavior changes

Proposed changes
Issue Number: close #xxx
In the current version, the dynamic partitioning feature detects the time interval every 10 minutes and removes partitions that are not within the preset range. Now let's make a behavior change and only delete partitions that are smaller than the preset range
example:
today is 2024-4-17 , we create a table with start for -1 and end for 1 , it will be

2024-4-16 2024-4-17 2024-4-18

now we manually insert some partition , it will be

2024-4-02 2024-4-10  2024-4-16 2024-4-17 2024-4-18 2024-5-15 2024-6-14

In the current version , after checking , it will be

2024-4-16 2024-4-17 2024-4-18

but after we changes it , it will be

2024-4-16 2024-4-17 2024-4-18 2024-5-15 2024-6-14

it only remove partitions that are smaller than the preset range

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

by the way, please make title and content of this pr more detailed

run buildall

run cloud_p1 5

run cloud_p1 5

run cloud_p1 5

run buildall


[SPARK-8919][Documentation, MLlib]Added @since tags to mllib.recommendation

Can one of the admins verify this patch?

ok to test

Merged build triggered.

Merged build started.

Test build #36949 has started for   PR 7325 at commit c413350.

@coderxiang Could you help review this PR? Thanks!

Test build #36949 has finished for   PR 7325 at commit c413350.

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

Merged build finished. Test PASSed.

Just realized one issue with adding the @since tag. For overloaded methods, it inherits the doc from its parent if no JavaDoc is provided. However, if we want to add @since, we have to add JavaDoc. Then we need to copy the JavaDoc from parent, which makes it hard to keep docs in sync.
@rxin Is it better to implement @since as an annotation?

How would the annotation work for each method in javadoc?

Annotation supports methods as well. I don't think it affects binary compatibility.
@since("1.3.0")
def func(...)

We can make them show up in the generated doc like @Experimental.

Yea if we can figure out a way for show to show up, I think that's good.

Can one of the admins verify this patch?

I double-checked the tags and LGTM except for the one overloaded train I commented on - probably because its declaration is multi-line, so a grep won't catch it

Merged build triggered.

Merged build started.

Merged build finished. Test FAILed.

test this please

Merged build triggered.

Merged build started.

Test build #37643 has started for   PR 7325 at commit 93156f2.

Test build #37643 has finished for   PR 7325 at commit 93156f2.

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

Merged build finished. Test PASSed.

Merged into master. @vinodkc Next time, please ping other people on the JIRA page before you start coding to avoid duplicate work. Could you send a message the JIRA page? So I know your JIRA account and I can assign this ticket to you. Thanks!

@mengxr , I've added  message in JIRA page from my JIRA account.
Thanks


Cannot author a C++ WinUI 3 component

Describe the bug
Following the discussion here (https://github.com/microsoft/WindowsAppSDK/discussions/3013), it seems like the ability to author a C++ WinUI 3 component is not supported "out the box" in Visual Studio.
Attempting to write a component in C++ and then consume it results in the following error message:
Exception thrown at 0x00007FFB1846474C (KernelBase.dll) in ControlTest.exe: WinRT originate error - 0x80004005 : 'Cannot locate resource from 'ms-appx:///FancyControl.xaml'.'.

Steps to reproduce the bug
Create a C++ WinUI 3 component.
Create a C++ WinUI 3 application.
Add a project reference to the component, in the application's project file.
Attempt to use the component within the XAML of the application.
Expected behavior
To be able to use C++ end-to-end.
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.1.5
Windows app type

[ ] UWP
[X] Win32

Device form factor
Desktop
Windows version
Windows 11 (22H2): Build 22621, Windows 11 (21H2): Build 22000
Additional context
No response

This is still a problem that needs resolving. Either support C++ or don't. Don't leave it in some weird in between state.

C++ components have always been supported. This is the result of another known issue.
The release notes for 1.0 always had a workaround in them. The templates that come with Visual Studio 2022 17.8 also updated the runtime component template so that it doesn't need this.


TabView: TabHandler: Call handleCloseAction method of parent tabView when closeHandler is clicked.

/cc @fatihacet @szkl @gokmen @sinan

ping

@sinan could you guys finalise this PR, we have to merge this or better one ASAP.

I'm closing this and i'm creating a new PR for this.

Here's the new PR: https://github.com/koding/kd/pull/143


Auto cast stopped

Unchecked hook strong(!!), min wait time is set to 7. If !! fish appeared but not reaching the min wait time, the auto cast will stop. This issue does not happen in previous version.

im stupid, should be good now

Should be fixed now


MRG: Actually do projection

@agramfort it's probably good to take a look at this one now, esp. how test_epochs.py had to be changed. Writing out EpochsArray and reading it back by default will have avg ref added and applied.
The least intrusive change might actually be to have add_eeg_ref=False in read_epochs. If people have been using FIF, they've presumably already added the ref by that stage (during Epochs construction), and it avoids breaking people's EpochsArray use.

Ready for review/merge from my end. Once this is in, #2676 should work okay with add_eeg_ref options, whatever those may be. My vote has actually changed to not add the add_eeg_ref argument at all, since users with this PR can do epochs.add_eeg_ref() afterward if they want.

@jasmainak how does this look to you?

Looks like we have failing tests. I check carefully tomorrow ..

Fixing now

@jasmainak let me know if this is good for you. This and your PR are the only things holding up release I think.

 Can you have one last look to make sure you're happy? If so I'll merge once the CIs come back happy.


std::shared_ptr == nlohmann::json compiles, which seem undesirable

Specifically std::shared_ptr<nlohmann::json>() == nlohmann::json() compiles (where one forgot to deference the shared_ptr before comparing).
What is the issue you have?
It is easy to miss dereferencing the pointer and so have the comparison not be as desired.
Please describe the steps to reproduce the issue.
std::shared_ptr<nlohmann::json>() == nlohmann::json()
Can you provide a small but working code example?
std::shared_ptr<nlohmann::json>() == nlohmann::json() (which is true) or std::make_shared<nlohmann::json>() == nlohmann::json() (which is false)
What is the expected behavior?
A compilation error would be really nice as it is very likely a bug.
And what is the actual behavior instead?
Compiles (as far as I know, the comparison result is what you would expect given that it compiles.
Which compiler and operating system are you using?

Compiler: apple-clang 12,13. Visual Studio 2017. Gcc 9.
Operating system: various

Which version of the library did you use?

[X] latest release version 3.10.2
[X] other release - please state the version: 3.9.1
[ ] the develop branch

If you experience a compilation error: can you compile and run the unit tests?

This is due to the implicit conversions/constructors in the library.  If you disable them, then this code no longer compiles.  See here for how to disable them:
https://github.com/nlohmann/json#implicit-conversions

so this issue can be closed, right? works as designed?

Works as designed and I can’t think of a better design.


Add options handling to AWS and GCP deployments

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@2f7926d). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop       #1   +/-   ##
==========================================
  Coverage           ?   36.00%
==========================================
  Files              ?       48
  Lines              ?     4205
  Branches           ?        0
==========================================
  Hits               ?     1514
  Misses             ?     2548
  Partials           ?      143

Flag
Coverage Δ

unittests
36.00% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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 2f7926d...56814f4. Read the comment docs.


streaming: create multiple dispatchers for MV creation

Currently, we have one broadcast per actor to all downstream for MV creation. To make it more reasonably represented in the system, we should change it to multiple dispatchers of 1v1 exchange.

What's the benefit of this? By the way, so there'll be no usage of the broadcast dispatcher anymore?

This sounds more reasonable.

Just make everything sounds more reasonable.
Yes we can deprecate broadcast dispatcher in the future.


How to put data preprocessing layer into model

I am studying at text classification.I find i need to process the text into vector before i put it into model. I want to know is there any method can put the data preprocessing into model ,so i can put the text not the vector into model.If you have any idea,thank you for comment.

I believe you can use a lambda layer and implement that preprocess using that layer

@WittyLLL I think TextVectorization mentioned in this tutorial will help you. Thanks!


Drop error if a Caffe layer references an invalid bottom

This should help address the case when a layer is added to manipulate e.g. the label blob. In that case the layer must include a directive like:
include {
phase: TRAIN
}
Otherwise Caffe will drop an error during inference.
A sanity check is added to DIGITS to catch these errors earlier in the process (i.e. before training).

I have updated the patch to make the new method static and I have added tests.

I have updated the patch to make the new method static and I have added tests.

Nice!


Identify and fix the issues with signer status flipping

this is potentially fixed by https://github.com/alchemyplatform/aa-sdk/pull/808


TypeError: expect.addSnapshotSerializer is not a function

I receive the following error when I try to run my tests with jest-enzyme:
TypeError: expect.addSnapshotSerializer is not a function

I'm assuming you ignored the npm peer dependency warnings :)
Jest added this feature in version 19 hence this [https://github.com/blainekasten/enzyme-matchers/blob/master/packages/jest-enzyme/package.json#L46](peer dep).
You either need to upgrade jest or use a 2.x version of jest-enzyme that supports older versions of jest.

Created a PR to add what you mentioned in the README.
I'm expecting people new to the ecosystem to use create-react-app a lot and stumble upon enzyme eventually. And I'm not really expecting many of them to read peer dependency warnings :).

Awesome! Thanks so much!


Run Functional Test/Test Agent Deployment fails

Hi
I'm having issues using the Test Agent Deployment/Run Functional Test Tasks.
I earlier had problems as well (https://github.com/Microsoft/vsts-tasks/issues/1483) and the answer was that it will be fixed as soon as we update our TFS to 2015 Update 2.
So we did, but still we have problems.
Following is the setup:
We have a TFS 2015 Update 2 on prem and as well a build agent that runs on a server that is on premises. We run the build agent with a service account that is part of our domain.
Then we have virtual machines were we want to run UI Tests on with the "Run Functional Test" task.
These VM's are not part of our domain and only have one admin user. I think this was the problem why it was not working before TFS update 2, but I can't get it to work now either.
It seems like that during the "Visual Studio Test Agent Deployment" Task it tries to configure the test agent and there are some errors. Now I was not sure if/where I have to put credentials from our domain to access TFS?
Thanks for your help
I attached the DTAExecutionHost which shows some more details.
DTAExecutionHost.exe.zip

In the attached log, there are not any concerning errors.
Can you remove the existing log and run the DTA flow and paste the full log in case of any error

Hi
so these lines in the log are normal:
DTAExecutionHost.exe, Communicating to the Token Servicehttp://<url.of.tfs>:8080/tfs/PSAC_RD
DTAExecutionHost.exe, Service task failed. Retry attempt 10.
I will run the tests again as soon as I'm in the office.
Thanks for your help

Hi there
so i ran again the tests and following was shown after some time on the VM:

I aswell attached the logs.
So it detects my one test that fits to the filter criteria, but I think it is not run (on the TFS, I get now since some minutes only:
DistributedTests: Test run '985' is in 'InProgress' state.
DistributedTests: Total Tests : 1, Passed Tests : 0
DTAExecutionHost.exe.log.txt

Hello
Just a little update on that issue. I did reinstall the OS on the VM and then tried again. That seems to have fixed the issue, so probably it was something that was misconfigured before (maybe because I tried as well earlier to run the tests before we updated TFS to 2015 Update 2).
Anyway thanks for your support.


Temporarily fix issuers

The code currently sets the default issuer to
osp-rootca-issuer-internal, which is the label for the issuer - rather than the name of the issuer itself (rootca-internal).
The proper fix for this is to use the label and then call something like GetIssuerByLabel() to get the right issuer - while handling the error if the issuer is not yet available.  We plan to add such a function to lib-common by early next week - and will add code to use it then.
Until then, we can unblock folks trying to test out tls-e by setting the issuer name  correctly.

Even if we could apply what Rabi says about getting the issuer from const values, this MUST be considered a temporary fix

Though it would be better to use the label (which  itself is a constant in lib-common), I don't think using the constants that're used in openstack-operator for the issuer name, would make it as bad as hard-coding it.

recheck


Localize content/en/docs/reference/glossary/annotation.md into Hindi

Fixes #35600

@taniaduggal did you mean to close this PR?


last 2 leds stays pink

my reported issue before wasn't a real issue.
https://github.com/hyperion-project/hyperion.ng/issues/388
On recommendation i changed to a other adalight.ino.
Now the problem is back, this time its pink.
Previously I changed the led count to from 124 to 85 and it worked (the last 2 leds went off)
With the new adalight.ino i changed define max leds to 85. But the last 2 leds are pink.

dont change max leds, unless you have to less memory.
Would be great if you don't make such noise here, this is a bug tracker not chat room.


Don't treat 409 response to conditional request as an error

Contributes to: https://github.com/Azure/azure-sdk-for-net/issues/19982

I talked with @tg-msft and we agreed with a more targeted ResponseClassifier implementation in storage instead of changing the global one.

@kasobol-msft this change would reduce amount of error spans customers see in AppInsights/logs when making CreateIfNotExist calls.

 How can I tell if they are? Do I need to update anything on my end? Because I am still getting these errors in my ApplicationInsights for BlobBaseClient.Exists()


🔧 Repository: Documentation & Guidelines on Local Plugin Development

📜 Description
Backstage and community plugin repo documentation severely lack a description and/or general set of guidelines on how best to actively develop and test code in this repository locally. There should be specific documentation provided that explains how best to clone this repo and develop locally against one or more of the plugins. I believe that this would substantially improve community engagement, and at least create a uniform starting point that conversations around developing these plugins can operate upon.
👍 Expected behavior
A clear and concise README.md or CONTRIBUTING.md file containing step-by-step instructions for local development and contribution to this repository.
👎 Current Behavior
Lack of centralized documentation
👟 Reproduction steps
N/A
📃 Provide the context for the Bug.
Sometimes, we create a test app in order to assist with local development, as in this example. However, this is only present in very few of the plugin directories, and the only way to divine this process is to locate a similar PR or extrapolate from unrelated Backstage guides that sort of indirectly articulate this process.
Roadie's documentation for their plugin monorepo is a great place to start. While their repository structure is different and allows for a shared test app, they at least sort of (and rather imperfectly) describe the steps it takes to get up and running with local development.
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
No, but I'm happy to collaborate on a PR with someone else

https://github.com/backstage/community-plugins/issues/413


Supabase db push should support migration sub directories

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
It could be interesting to have the support for push migration for subdirectories instead of only the main directory of supabase/migrations
that could be helpful in order to have features or scheme separated for each migration, specially in bigger projects i think that something like that is essential.
Describe the solution you'd like
A clear and concise description of what you want to happen.
I would like the script to iterate over
supabase/migrations/**/.sql
instead of only
supabase/migrations/.sql
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
The other alternative is naming convention but that doesn't help to pre filter easily the files in the file explorer
Additional context
Add any other context or screenshots about the feature request here.

We will probably support this by allowing custom migrations path to be declared in config.toml, similar to https://github.com/supabase/cli/issues/2139#issuecomment-2055650677

Will a single custom migration path really be the solution here? I too have an interest in organizing my migrations folder by using subdirectories because it's a huge pain to search through to find the right SQL statements to adjust.
example:

subdirectory for each table I've changed.  That way I know the last file in each folder is the latest sql for that view.

migrations/vw_some_view
migrations/vw_some_other_view

Is the current vision for custom paths in the config.toml really supporting this?

I don’t think so, but that was my same necessity, and making this possible would be a very little change on the supabase cli (just using a recursive file list instead of looking in the main directory).Il giorno 20 nov 2024, alle ore 20:52, Jose Torres @.***> ha scritto:
Will a single custom migration path really be the solution here? I too have an interest in organizing my migrations folder by using subdirectories because it's a huge pain to search through to find the right SQL statements to adjust.
example:
subdirectory for each table I've changed.  That way I know the last file in each folder is the latest sql for that view.
migrations/vw_some_view
migrations/vw_some_other_view
Is the current vision for custom paths in the config.toml really supporting this?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

I too have an interest in organizing my migrations folder by using subdirectories because it's a huge pain to search through to find the right SQL statements to adjust.

That's a slightly different issue. I believe a nice solution for that is to use declarative schemas. So you would declare all your tables once in supabase/schemas directory and use db diff to generate migrations against the declared schema when making changes.
This is currently lacking some docs so I will try to explain the process more in a docs update.
See also https://github.com/supabase/cli/pull/2099

Thanks for the idea @sweatybridge! Much appreciated. I'll give that approach a try.


deployment template list not working for Cloud Saas

Overview
Expected Behavior

It should display the possible templates
Current Behavior

At the moment this does not work out for Cloud SaaS.
In #221 I got the hint about the list parameter.
ecctl platform deployment-template list
I get the error message:
<title>404: Not Found</title>404: Not Found

I enabled verbose and obviously the resulting API call doesn't seem to work against Cloud SaaS
GET /api/v1/platform/configuration/templates/deployments?format=cluster&show_hidden=false&show_instance_configurations=false HTTP/1.1
Host: api.elastic-cloud.com
...

====================  End of Request #0  ====================
==================== Start of Response #0 ====================
HTTP/1.1 404 Not Found
...
<title>404: Not Found</title>404: Not Found

Reference:
https://github.com/elastic/ecctl/issues/221

Hi @111andre111! Thank you for taking the time to open this issue :) the deployment template commands are meant for ECE only. I have added the corresponding text that mentions that

@karencfv Ok, got it. But how will a user know what template type can be used in the case of ESS Cloud?

@111andre111 For the time being they will be directed to our documentation to find this information. Both here -> https://www.elastic.co/guide/en/cloud/current/ec-getting-started-templates.html and here -> https://www.elastic.co/guide/en/cloud/current/ec-regions-templates-instances.html (this link doesn't work yet)


Improve accessibility for the global navbar

Changes include

use <nav> instead of <div> as top-level element
convert dropdown <div>s to <button role="menu"> (the dropdowns behave like a button until opened, so should also offer the functionality of a button) (not anymore, see https://github.com/go-gitea/gitea/pull/23125#discussion_r1117281548)
improve/delete some comments in the template

The appearance did not change.

The dropdowns can now be opened without the mouse, their items cannot be focused however.
This should probably be fixed in a future PR as well.

Actually, before this PR, all dropdowns can be opened by keyboard without mouse, and their items work with keyboard.
I have tested with keyboard and screen reader before when I was writing the aria.js.

Codecov Report

Merging #23125 (e9894ce) into main (e77528b) will increase coverage by 47.41%.
The diff coverage is n/a.

:exclamation: Current head e9894ce differs from pull request most recent head 2f90bb3. Consider uploading reports for the commit 2f90bb3 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff            @@
##           main   #23125       +/-   ##
=========================================
+ Coverage      0   47.41%   +47.41%
=========================================
  Files         0     1140     +1140
  Lines         0   150889   +150889
=========================================
+ Hits          0    71538    +71538
- Misses        0    70876    +70876
- Partials      0     8475     +8475

Impacted Files
Coverage Δ

models/webhook/webhook_system.go
26.00% <0.00%> (ø)

routers/web/auth/password.go
2.71% <0.00%> (ø)

services/auth/source/smtp/source.go
14.28% <0.00%> (ø)

models/db/iterate.go
71.42% <0.00%> (ø)

modules/git/repo_commit.go
55.52% <0.00%> (ø)

modules/indexer/issues/db.go
48.00% <0.00%> (ø)

modules/private/key.go
46.87% <0.00%> (ø)

routers/web/base.go
16.52% <0.00%> (ø)

modules/git/repo_commit_nogogit.go
56.25% <0.00%> (ø)

models/repo/update.go
30.00% <0.00%> (ø)

... and 1130 more

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

Actually, before this PR, all dropdowns can be opened by keyboard without mouse, and their items work with keyboard.
I have tested with keyboard and screen reader before when I was writing the aria.js.

Yup, at the time my problem was simply that I didn't know how to navigate a menu, and expected it to work with an additional tab press.
As I've now learned, it is up/down, which still works.


Issue 693 fix broken ci

The CI is broken due to various issues, in this request all issues with CI are solved.
Unfortunately I had to cherry-pick the commit https://github.com/denoland/saaskit/pull/692/commits/6c6399bd1b6a8fec351725be1700013d37df17c3 as it was blockage.
The commands
deno task build
deno fmt --check && deno lint && deno task check:license --check && deno task check:types && deno task test
work well on Deno 1.46.3 as well as on 2.0.2

PTAL @kt3k


Add health endpoint for usage with kubernetes

Does GET /health return 200, 'OK' work or do you need more data or a particular format?

Yes that is exactly what I need.

Releasing v1.4.0 right now


[FLINK-25742][akka] Remove the serialization of rpc invocation at Fli…

…nk side.

What is the purpose of the change
(For example: This pull request makes task deployment go through the blob server, rather than through RPC. That way we avoid re-transferring them on each deployment (during recovery).)
Brief change log
(for example:)

The TaskInfo is stored in the blob store on job creation time as a persistent artifact
Deployments RPC transmits only the blob storage reference
TaskManagers retrieve the TaskInfo from the blob cache

Verifying this change
Please make sure both new and modified tests in this PR follows the conventions defined in our code quality guide: https://flink.apache.org/contributing/code-style-and-quality-common.html#testing
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (100MB)
Extended integration test for recovery after master (JobManager) failure
Added test that validates that TaskInfo is transferred only once across recoveries
Manually verified the change by running a 4 node cluser with 2 JobManagers and 4 TaskManagers, a stateful streaming program, and killing one JobManager and two TaskManagers during the execution, verifying that recovery happens correctly.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Sorry for nitpicking. Could we please pay more attention to PRs description? We should serve as an example if we require contributors to fill it in.

@dawidwys I'll fill that.

Thanks @KarmaGYZ ! I know from personal experience it might be daunting at times :sweat_smile:

@zentol This PR is an improvement of the job execution for Flink OLAP. In our internal benchmark, the serialization of the RemoteRpcInvocation at the Flink side will increase the end-to-end latency of a 1000*1000 WordCount for about 100ms.

Without this serialization

But things are serialized anyway because of Akka; it just happens later. So where is this performance gain coming from?

@zentol Now, the RemoteRpcInvocation are serialized twice, first in Flink and second in Akka. This PR removes the first serialization in Flink.

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure


Make sure to enable the sites framework

Without it, the download links don't work

I'm having trouble re-creating this error. Can you explain specifically what you're doing?
EpisodeDownloadView uses Django's get_current_site and add_domain functions to help determine the fully qualified URL for the download. Ultimately, it falls back on the RequestSite class, which works with or without the Sites app because it parses the request object.
The URL is supposed to have a "download/" appended to it, and an auto-download response is returned. It looks something like this. Note the URL at the bottom.

I should have taken better notes. I don't remember exactly what happened, but I was testing it with a very vanilla Django install, the download failed, and Googling the error message suggested you had to enable the sites framework. I figured that was something you just did automatically and so hadn't thought to include it in the instructions.
I'll see if I can recreate and get back to you.

My test demo, which is local and not online (my bad!), does not have the Sites framework installed and it seems to work just fine. I'm going to close this pull for now, but if you can further tell me how I can reproduce this error, I'm happy to reopen.


[Fuchsia] Fix root SSL cert path

Fuchsia stores the root SSL cert in a different location than Linux.
This commit corrects this, so that SSL connections work.

@veblush

Closing and reopening the PR to wake up tests.


Error: unsupported release of Debian 8.2

I'm getting this
/usr/src/app/node_modules/mongodb-download/index.js:127
throw new Error("unsupported release of Debian " + os.release);
^
Error: unsupported release of Debian 8.2

trying to install mongoose

@franky-continu I've checked 7.1 binaries on 8.2 and they are starting, and seem to work fine, so i've updated download to pull 7.1 release

Fantastic will check them out as soon as I get in, thanks. Will inform here any progress


[CLOVER] 5MSS OM | [COPE] LC Buffs

Description
Adds 5MSS as a Clover OM (allows for the usage of five moves per mon instead of four).
Gives Sphare and Snurl some learnset buffs.


We might want to configure Nginx to cache static files

spinning off from #978, a lot of the traffic that nginx is forwarding to gunicorn is requests for static files like the logos in the footer. If we configured nginx to cache such static files, we could reduce much of the load currently placed on gunicorn/django.

@alastair pointed out that nginx is already serving these static files - had I looked at the Gunicorn logs, I would have seen that the requests for static files were/are only making it as far as nginx, and not as far as Gunicorn.


Nix build (legacy Plutus)

This PR:

allows the library and executables to be built with nix-build,
adds the build commands to the makefile,
creates a provisioned nix-shell from the same files that the nix-build uses.

Note:

the Plutus repository that is built, is the old version, which was pinned with the repository.

To do:

[ ] Update to latest Plutus Repo

Opened #86 as a continuation task.


Vector2d unification

Description
Rename and remake the Position class to be more general.
The vector_math vector2 was built around factories which made it hard to extend it, so box2d interaction will still need to convert between the vector formats. I will look into converting box2d to use this vector instead.
Fixes #373
Type of change
Please delete options that are not relevant.

[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

Checklist:
If something is unclear, please submit the PR anyways and ask about what you thought was unclear.

[x] This branch is based on develop
[x] This PR is targeted to merge into develop (not master)
[x] I have added an entry under [next] in CHANGELOG.md
[x] I have formatted my code with flutter format
[x] I have made corresponding changes to the documentation
[x] The continuous integration (CI) is passing

I was wondering what the reasoning is by implementing your own Vector2d class instead of using (and exporting) the Vector2 class from the vector_math library?
Having done a quick look I couldn't find any method on this implementation that isnt possible to do with the existing one from vector_math, and the only reason I found was in the description of this PR to be able to extend. But I personally never saw any usecase of a engine user doing so?
The reason why I am asking this is because it has been bugging me that I have to use both Position Offset and Vector2 in my game. And with this change I will still need to use Vector2d Offset and Vector2. For context, I am using Vector2 for Matrix4 transformations on the canvas.
So unless I missed something, I think switching to Vector2 of the vector_math library would be an better option from a engine user experience. And that might fix the problem @spydon mentioned about Box2D. Because Box2D already uses the vector_math vectors I believe

I was wondering what the reasoning is by implementing your own Vector2d class instead of using (and exporting) the Vector2 class from the vector_math library?
Having done a quick look I couldn't find any method on this implementation that isnt possible to do with the existing one from vector_math, and the only reason I found was in the description of this PR to be able to extend. But I personally never saw any usecase of a engine user doing so?
The reason why I am asking this is because it has been bugging me that I have to use both Position Offset and Vector2 in my game. And with this change I will still need to use Vector2d Offset and Vector2. For context, I am using Vector2 for Matrix4 transformations on the canvas.
So unless I missed something, I think switching to Vector2 of the vector_math library would be an better option from a engine user experience. And that might fix the problem @spydon mentioned about Box2D. Because Box2D already uses the vector_math vectors I believe

One important thing to have our own class, is to server as a bridge between the many other classes existent on the Flutter universe, if we forget about Flame, there is already Offset, Size and Vector2d from vector_math.

One important thing to have our own class, is to server as a bridge between the many other classes existent on the Flutter universe, if we forget about Flame, there is already Offset, Size and Vector2d from vector_math.

Well that is exactly what I mean, by doing so you are adding to the pile, yea sure you can add from and to methods but if there is already a vector class that does pretty much what your own class does. Then why would you want to make your own? With the same functionality? But not "interchangeable" with the other one?
As you said there is Offset Size(tho in Flutter this tends to not be used for positioning) and Vector2 from vector_math, but now you want to add another one? That cant be used anywhere where u have to use either Offset or Vector2?
Not saying that is it bad, I just try to find the reasoning behind it. But just adding a Vector class for the sake of having your own while there is already one out there with the same abilities strikes me as odd. The only reason I can think of is the from and to helper methods like toOffset or fromOffset. But the toOffset can be added using extensions and you can always just make a fromOffset helper method like so:
Vector2 fromOffset(Offset offset) => Vector2(offset.dx, offset.dy);

Yep, we are adding one to the pile, but the thing is, the pile is already there, it is common to have to translate an Offset to other things, and so on, if we don't have our own class, that will have to be manual.
We have thought on using extensions, but we have given up on that ideia, but I can't remember why right now, @luanpotter @spydon @renancaraujo do you remember why?

We have to keep our API protected from changes over the APIs external libs. But in this case, we have a very basic type and the lib in question is a very reliable one with a 100% popularity over on pub.
By using Vector2 we have:
Pros:

We don't reinvent the wheel
We integrate more with the dart ecosystem of packages rather than living like a strange one.

Cons:

We may still have to provide an auxiliary element (subclass or a class extension) trough our API in order to use and provide customizations (from and to methods).

I'm all for it but be aware, this is only possible now because we are in a pre v1 era. After that changes like this one won't be that easy to pull off.

@wolfenrain After this long discussion, we decided to merge this, as this is a rename of an existent thing, and already improves some stuff. We will open this discussion again before we release the v1.
@spydon the only thing missing on this PR, is to solve the conflicts and rebase it to the v1 branch, could you take a look on this so we can merge this?

@erickzanardo Fine by me :-). If there is any need for more discussing, just ping me.

@wolfenrain I tried to extend the vector_math's vector2 class, but it is a bit problematic since it is using factories that gives back its own class for many methods. You could have a look if you can find a good way to do it, I think that would be the optimal way to go if it was possible.

@wolfenrain I tried to extend the vector_math's vector2 class, but it is a bit problematic since it is using factories that gives back its own class for many methods. You could have a look if you can find a good way to do it, I think that would be the optimal way to go if it was possible.

In what scenario is it needed to be able to extend it? As far as I can see it already provides the required methods. I think @renancaraujo solution with simple from and to functions will do the trick. Unless I am missing something ofcourse.

@wolfenrain I tried to extend the vector_math's vector2 class, but it is a bit problematic since it is using factories that gives back its own class for many methods. You could have a look if you can find a good way to do it, I think that would be the optimal way to go if it was possible.

In what scenario is it needed to be able to extend it? As far as I can see it already provides the required methods. I think @renancaraujo solution with simple from and to functions will do the trick. Unless I am missing something ofcourse.

Yeah, that was my initial idea. But I think @erickzanardo and @luanpotter wants to be able to extend it if needed without having to have another class (for example adding angles as doubles doesn't exist).

Just for reference, closed in favor of https://github.com/flame-engine/flame/pull/459


ARROW-1853: [Plasma] Fix off-by-one error in retry processing

When a user construct a plasma client that should not perform a single retry, by passing num_retries = 0, nothing happens due to an off-by-one error in the retry processing. This commit fixes the issue and polishes the error output.

+1 LGTM


Error on raspberry pi

[anirip] Logging into Crunchyroll...
[anirip] Successfully logged in!
[anirip] Scraping show metadata...
[anirip] Error: There was an error retrieving show page - Get Haganai: unsupported protocol scheme ""
I traced this through the files to
main.go line 99: show.Scrape(client, showURL)
--> crunchyroll/show.go line 22 client.Get(showURL, nil)
where client = common.NewHTTPClient() and showURL = Haganai

whoops, I use a custom script for my windows pc and forgot I had to enter the full URL...


Make lock 2x faster by loading repo state once

If no ~rt argument is passed to OpamSwitchState.with_, it will load the repository state itself. We already load this state in check_repo_config, so this was causing it to be loaded twice.
Instead, we can load it manually and pass it as argument and it will use it.
Since that function dominates lock time, this makes lock twice as fast (on opam-monorepo itself: 12.4s down to 6.7s).

Hmm what happened to the changelog check btw?

Can you add a changelog entry for this?

Out of curiosity, do you know if the repo_state can be extracted from the switch state directly by any chance? I.e. if we can obtain it without calling OpamRepositoryState.with_.

I haven't found a way to do that as a shortcut, but in terms of performance it would be the same (I agree that keeping the implicit loading would read a bit better).
OTOH I think that we can expose a way to list the repositories without loading a switch state, but that state is used later to call the solver.

OTOH I think that we can expose a way to list the repositories without loading a switch state, but that state is used later to call the solver.

I think that as you're suggesting, I don't think there's any value in that for us. Furthermore, I assume a switch state is needed because we do not want a bare repository list but the specific list of repos enabled for the current switch so to me, it make sense that we're getting it from the SwitchState.
My impression is that those states contain a lot more information than we need most of the time and that lazily loading things and exposing the proper functions to access the proper subset of information without triggering a full load is what we'd need the most from opam libraries.


Broadlink media_player platform

Description:
Add a media player platform for use with an broadlink IR transmitter.
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#5791
Example entry for configuration.yaml (if applicable):
media_player:
  - platform: broadlink
    host: <IP_ADDRESS>
    mac: '11:22:33:44:55:66'
    name: 'Samsung Living Room TV'

    command_on    : 'JgBGAJKVETkRORA6ERQRFBEUERQRFBE5ETkQOhAVEBUQFREUEBUQOhEUERQRORE5EBURFBA6EBUQOhE5EBUQFRA6EDoRFBEADQUAAA=='
    command_off   : 'JgBGAJOVEDoQOhA6DxYPFhAVEBUQFRA6ETkROREUERQRFBEUEBUQFREUERQRORE5EBUQFRE5ETkRORE5ERUQFRA6DzsPFhAADQUAAA=='

    volume_up     : 'JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISNxI3EjcSEhISEhISEhISEhISEhISEjcSNxI3EjcSNxIABfgNBQ=='
    volume_down   : 'JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISNxI3EhISNxISEhISEhISEhISEhI3EhISNxI3EjcSNxIABfgNBQ=='

    previous_track: 'JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISEhISEhISEhI3EhISEhISEjcSNxI3EjcSEhI3EjcSNxIABfgNBQ=='
    next_track    : 'JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISEhI3EhISEhI3EhISEhISEjcSEhI3EjcSEhI3EjcSNxIABfgNBQ=='

    digits:
      '0'     : 'JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISEhISEhISEhISEjcSNxISEjcSNxI3EjcSNxISEhISNxIABfgNBQ=='
      '1'     : 'JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISEhISEhISEhISEjcSNxISEjcSNxI3EjcSNxISEhISNxIABfgNBQ=='
      '2'     : 'JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISEhISEhISEhISEjcSNxISEjcSNxI3EjcSNxISEhISNxIABfgNBQ=='
      '3'     : 'JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISEhISEhISEhISEjcSNxISEjcSNxI3EjcSNxISEhISNxIABfgNBQ=='
      '4'     : 'JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISEhISEhISEhISEjcSNxISEjcSNxI3EjcSNxISEhISNxIABfgNBQ=='
      '5'     : 'JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISEhISEhISEhISEjcSNxISEjcSNxI3EjcSNxISEhISNxIABfgNBQ=='
      '6'     : 'JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISEhISEhISEhISEjcSNxISEjcSNxI3EjcSNxISEhISNxIABfgNBQ=='
      '7'     : 'JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISEhISEhISEhISEjcSNxISEjcSNxI3EjcSNxISEhISNxIABfgNBQ=='
      '8'     : 'JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISEhISEhISEhISEjcSNxISEjcSNxI3EjcSNxISEhISNxIABfgNBQ=='
      '9'     : 'JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISEhISEhISEhISEjcSNxISEjcSNxI3EjcSNxISEhISNxIABfgNBQ=='

    sources:
      hdmi  : 'JgBGAJGWETkRORI4ERQRFBAVERQRFBE5ETkROREUEBURFBEUERQQOxA6EBUPOw8WDxYQFRA6DxYQFRA6EBUPOxA6ETkRFBEADQUAAA=='
      dtv   : 'JgBGAJSTEjgSOBI4EhMSExITEhMSExI4EjgSNxMTEhQRFBEUERQRORE4EhQSExITEhMSOBITEhMSExI4EjgSOBI4EhMSOBIADQUAAA=='

Checklist:

[X] The code change is tested and works locally.
[X] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[X] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[X] New dependencies have been added to the REQUIREMENTS variable (example).
[X] New dependencies are only imported inside functions that use them (example).
[X] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[X] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Docs update not performed yet. But i thought i might get some other eyes on it at this point.

Any relation to https://github.com/vpnmaster/homeassistant-custom-components/blob/master/custom_components/media_player/broadlink.py ?

It has no relationship with https://github.com/vpnmaster/homeassistant-custom-components/blob/master/custom_components/media_player/broadlink.py
@Danielhiversen never tried it, but seems it might have been able to work. Maybe except for direct channel change. Configuration is much smaller with this too.

Run flake8 locally and fix the errors

I don't think we should merge this. As @Danielhiversen said, we already have the universal media player for this purpose.

Although one could implement this functionality with the universal media player, the configuration burden associated with such solution is quite high for such a common use case.
The custom_component I referred previously is quite popular for this use case, exactly because of this issue.
I'm therefore in favour.

If you build a script that takes the command as a variable, there is hardly any boiler plate. I agree that this is just a variation of the universal media player.

According to the docs, the universal media player requires a list of children. There is none in this case.
That said, is seems the source code does not require it. I will try creating a identical config as above using the universal player just to see how big difference in config it ends up being.

Got part way there, but not sure how to get further
The broadlink switch to get the service:
switch:
  - platform: broadlink
    host: <IP_ADDRESS>
    mac: 'XXXXXXX'
    type: 'rm2'

The standard commands works okey:
  - platform: universal
    name: 'Living Room TV (Universal)'
    commands:
      turn_on:
        service: 'switch.broadlink_send_packet_192_168_16_229'
        data:
          packet: 'JgBGAJKVETkRORA6ERQRFBEUERQRFBE5ETkQOhAVEBUQFREUEBUQOhEUERQRORE5EBURFBA6EBUQOhE5EBUQFRA6EDoRFBEADQUAAA=='

      turn_off:
        service: 'switch.broadlink_send_packe_192_168_16_229'
        data:
          packet: 'JgBGAJOVEDoQOhA6DxYPFhAVEBUQFRA6ETkROREUERQRFBEUEBUQFREUERQRORE5EBUQFRE5ETkRORE5ERUQFRA6DzsPFhAADQUAAA=='

      volume_up:
        service: 'switch.broadlink_send_packet_192_168_16_229'
        data:
          packet: 'JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISNxI3EjcSEhISEhISEhISEhISEhISEjcSNxI3EjcSNxIABfgNBQ=='

      volume_down:
        service: 'switch.broadlink_send_packet_192_168_16_229'
        data:
          packet: 'JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISNxI3EhISNxISEhISEhISEhISEhI3EhISNxI3EjcSNxIABfgNBQ=='

      media_previous_track:
        service: 'switch.broadlink_send_packet_192_168_16_229'
        data:
          packet: 'JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISEhISEhISEhI3EhISEhISEjcSNxI3EjcSEhI3EjcSNxIABfgNBQ=='

      media_next_track:
        service: 'switch.broadlink_send_packet_192_168_16_229'
        data:
          packet: 'JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISEhI3EhISEhI3EhISEhISEjcSEhI3EjcSEhI3EjcSNxIABfgNBQ=='

Even the select source can be done, but a bit messy
    select_source:
      service: 'switch.broadlink_send_packet_192_168_16_229'
      data_template:
        packet: >
            {% if value_json.source == 'hdmi' %}
              JgBGAJGWETkRORI4ERQRFBAVERQRFBE5ETkROREUEBURFBEUERQQOxA6EBUPOw8WDxYQFRA6DxYQFRA6EBUPOxA6ETkRFBEADQUAAA==
            {% elif value_json.source == 'dtv' %}
              JgBGAJSTEjgSOBI4EhMSExITEhMSExI4EjgSNxMTEhQRFBEUERQRORE4EhQSExITEhMSOBITEhMSExI4EjgSOBI4EhMSOBIADQUAAA==
            {% else %}
              {{ 0/0 }}
            {% endif %}

How do i set a toggling on off state? Do i need a dummy entity to link the state to?
attributes:
  state: ???? dummy entity ????

Same here, i need a constant list of sources and state, so a dummy input_select entity?
attributes:
  source: ????
  source_list: ???? constant list of sources ????

Then the problem of sending a sequence of digits to change to channel 55 for example. I'm not sure yet of the below works. It lacks a delay, also not sure if the templating here is actually generating a json list of the packet, or a single long string.
      play_media:
        service: 'switch.broadlink_send_packet_192_168_16_229'
        data_template:
          packet: >
            {% for digit in value_json.media_id %}
              {% if digit == '1' %}
                - '1JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISEhISEhISEhISEjcSNxISEjcSNxI3EjcSNxISEhISNxIABfgNBQ=='
              {% elif digit == '2' %}
                - '2JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISEhISEhISEhISEjcSNxISEjcSNxI3EjcSNxISEhISNxIABfgNBQ=='
              {% elif digit == '3' %}
                - '2JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISEhISEhISEhISEjcSNxISEjcSNxI3EjcSNxISEhISNxIABfgNBQ=='
              {% elif digit == '4' %}
                - '2JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISEhISEhISEhISEjcSNxISEjcSNxI3EjcSNxISEhISNxIABfgNBQ=='
              {% elif digit == '5' %}
                - '2JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISEhISEhISEhISEjcSNxISEjcSNxI3EjcSNxISEhISNxIABfgNBQ=='
              {% elif digit == '6' %}
                - '2JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISEhISEhISEhISEjcSNxISEjcSNxI3EjcSNxISEhISNxIABfgNBQ=='
              {% elif digit == '7' %}
                - '2JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISEhISEhISEhISEjcSNxISEjcSNxI3EjcSNxISEhISNxIABfgNBQ=='
              {% elif digit == '8' %}
                - '2JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISEhISEhISEhISEjcSNxISEjcSNxI3EjcSNxISEhISNxIABfgNBQ=='
              {% elif digit == '9' %}
                - '2JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISEhISEhISEhISEjcSNxISEjcSNxI3EjcSNxISEhISNxIABfgNBQ=='
              {% elif digit == '0' %}
                - '2JgBGAJOTEjcSNxI3EhISEhISEhISEhI3EjcSNxISEhISEhISEhISEhISEhISEhISEjcSNxISEjcSNxI3EjcSNxISEhISNxIABfgNBQ=='
              - 'SOME_TYPE_OF_DELAY_IR_CODE'
              {% endif %}
            {% endfor %}

So summary on issues

Broadlink annoying ip address in the service name instead of some controllable entity name
Dummy entity needed for holding on/off state to toggle (perhaps can re-use the switch state of the broadlink platform)
Dummy entity needed for source list
Delays in commands between keypresses. May be possible to use a dummy IR code for that
Very complicated configuration to get a single IR remote running.

At the moment my attempt to get this working with universal player is more than thre times as many lines of configuration, yet still not working with state changes. I would assume i would need some scripts for turn_on/off/select_source that update the dummy inputs. That would add several more lines and complexity.
Imho. This is very messy, and very hard for a normal user to set up.

I tried to setup with media player universal and really, it creates a mess. Adding this component won't hit anyone bad as it is a very common used devices on automation scene.

Adding this component won't hit anyone bad

Easy to say when you don't have to maintain it 😉

I suggest we keep this around as a custom component.


The shadows falling on the terrain at different heights are out of alignment.

It will appear as follows.

If there is a way to solve this problem, I would appreciate it if you could let me know.
Also, the quality of the shadows seems to be different from the sample here, any idea as to the cause of this?

I will try to have a look, but you may need to increase the size of the shadow map tetxure and also check the bias.

If you can, plese share with me your testing repo to have a look. I don't have too much spare time these days and it will help me to diagnose the issue.

If you can, plese share with me your testing repo to have a look. I don't have too much spare time these days and it will help me to diagnose the issue.
I am very sorry, I found the cause of the misalignment.
I had the illusion that the shadows were out of alignment because the entity was floating against the terrain.


Align some timeout loops to be test-after-loop

Apparently, when CPU load is high, test-before-loop can cause timeouts without giving the loop at least one chance to complete successfully.

Since we have a single-check core_halted in there, we could almost replace all wait_for_core_halted impls with a default implementation on CoreInterface, except the archs use a different error. I think finding a way around this is worth it.


Add documentation for Panel widget

Working on this now

Awesome! 👍

Hello @patrickfreilinger, just checking in to see how you are going with preparing a PR for this issue?

@FrankRay78 A PR has been merged for this issue. See here: https://github.com/spectreconsole/spectre.console/pull/1016


Revert "chore: disable ladle until we run node20"

Reverts vitejs/vite-ecosystem-ci#245
reinstate ladle, they reenabled node18 on their repo https://github.com/tajo/ladle/pull/497

That was quick!


tremor-rs: 0.10.1 -> 0.11.0

Motivation for this change
Things done

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

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

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

@SuperSandro2000 I think this can be merged now. I have to create a wrapper script to expose the TREMOR_PATH variable which I've added as a TODO but I suppose I can do that after @stephank's PR for Darwin support. wdyt?

@SuperSandro2000 I think this can be merged now. I have to create a wrapper script to expose the TREMOR_PATH variable which I've added as a TODO but I suppose I can do that after @stephank's PR for Darwin support. wdyt?

Was this broken before this update?

Was this broken before this update?

Yeah I didn't know the TREMOR_PATH variable was supposed to be exported when I made the init PR

Do we need this file?
.r--r--r-- root root 234 MB Thu Jan  1 01:00:01 1970  results/tremor-rs/lib/libtremor_script.a

Do we need this file?
.r--r--r-- root root 234 MB Thu Jan  1 01:00:01 1970  results/tremor-rs/lib/libtremor_script.a

Yeah that can be removed. I'll update the PR

Thanks for having the patience.

Thanks @SuperSandro2000 for all the help. btw are your sure about the cargoSha256 suggested change? All the tests that I did was done with the old one because the new one was failing

Thanks @SuperSandro2000 for all the help. btw are your sure about the cargoSha256 suggested change? All the tests that I did was done with the old one because the new one was failing. But apparently the CI passes

Do you already have the rust 1.51 update in your branch? That changed some cargoSha hashes.


Migrate to support Godot 4

Godot 4 is fast approaching a stable version and I would love to see this plugin support the new code conventions and features of the lastest Godot version.
Much appreciated.

I guess I will switch over every repository when Godot hits stable 4.x.
Until that time, I have tried many times to use Godot 4 but every time it comes up failing my projects.
When I switch over my current projects to the next version everything will be updated.

While I do not yet consider 4.x game-jam ready.
This branch contains 4.x compatible code and will become the default in the near future.


ForType Settings

Hello. I have some missunderstanding. I have example code
    public class SourceClass
    {
        public string A { get; set; }
        public string B { get; set; }
        public string C { get; set; }
    }
    public class DestClass
    {
        public string A { get; set; }
        public string B { get; set; }
        public string C { get; set; }
    }
    private void MappingTest()
    {
        var sourceItem = new SourceClass()
        {
            A = "A source",
            B = "B source",
            C = "C source",
        };

        TypeAdapterConfig<SourceClass, DestClass>.NewConfig().Map(dest => dest.A,
          src => string.Format("{0} {1}", src.A, "Add something To A"));

        // This mapping gives:
        // firstResult.A =  "A source Add something To A"
        // firstResult.B = "B"
        // firstResult.C = "C"
        var firstResult = TypeAdapter.Adapt<DestClass>(sourceItem);

        // This config should extend previous config
        TypeAdapterConfig<SourceClass, DestClass>.ForType().Map(dest => dest.C,
            src => string.Format("{0} {1}", src.C, "Add something To C"));

        // This mapping also gives:
        // secondResult.A = "A source Add something To A"
        // secondResult.B = "B"
        // secondResult.C = "C"
        // But It should give
        // secondResult.A = "A source Add something To A"
        // secondResult.B = "B"
        // secondResult.C = "C Add something To C"
        var secondResult = TypeAdapter.Adapt<DestClass>(sourceItem);

    }

As you see, second ForType config is not applied. I resolved this by calling on second (ForType) config Compile() method. Is it correct?

TypeAdapterConfig should be called at application startup. Once, we call TypeAdapter.Adapt, execution logic will be compiled. If we change the config after compilation, there will be no effect.
To solve this issue, you might create new TypeAdapterConfig() rather than using static method and pass config to adapt method. Then you can have different config per mapping.
Thank you for reporting issue, I will add error message, when changing config after compilation.

Ok. But how can I create new TypeAdapterConfig() based on current settings (on global config)?

@Stalso This version still cannot create config based on another setting right now. There is also another ticket (#41) mention this. It is in TODO list.

Thank you. Maybe then you will give me advice. I want to pass some papameters, when mapping entities. For example, I want to map some model in respect of current user in mvc controller,  and I should pass username to mapping. How can I implement this?  Maybe some anther tool will help me?

I also have plan for this. Other tool that can do this is AutoMapper. It allows to pass run-time value to mapping.
For current version of Mapster, you might need to map manually after mapping.
var dest = src.Adapt<DestClass>();
dest.UserId = this.User.Identity.Name;


Check when we call get_highest_block_header whether we mean complete block or not

Ticket closed accidentally.

Analysis of get_highest_block_header and get_highest_block calls:
storage::get_highest_block_header called only through effect::get_highest_block_header_from_storage:

deploy_acceptor::accept when accepting a deploy (probably most interesting call site for this issue, but seems fine as is)
rpc_server for QueryBalance request

storage::get_highest_block called through:

effect::get_highest_block_from_storage, which is called from:

rest_server for Status request
rpc_server for GetStatus request

storage::get_blocks_while -> storage::read_blocks_since -> main_reactor::control::initialize_next_component when initializing the deploy buffer with the blocks more recent than the last known switch block

After checking in with @Fraser999 , the get_highest_block_header were not correct.
In both the deploy_acceptor and query-balance cases, we want to do the relevant checks with the latest available global state, which means getting the highest complete block header from storage. Getting the highest block header might not have the necessary global state and an early failure is not the desired behavior in either case.

Fixed in #3350


Following documentation for newest PufferPanel does not work properly

I installed PufferPanel 0.8.3 via the in the documentation provided installer script. It went through just fine but when I open the page in my webbrowser (Chrome / Firefox) I get this:

The nginx config I use:
server {
  listen 80;
  server_name panel.example.com;
  # enforce https
  return 301 https://$server_name$request_uri;
 }

server {
    listen 443 ssl;
    server_name panel.example.com;

     # strenghen ssl security
  ssl_certificate /etc/nginx/SOMESSL;
  ssl_certificate_key /etc/nginx/SOMESSL;
  ssl_protocols TLSv1 TLSv1.1 TLSv1.2;
  ssl_prefer_server_ciphers on;
  ssl_session_cache shared:SSL:10m;
  ssl_ciphers "EECDH+AESGCM:EDH+AESGCM:ECDHE-RSA-AES128-GCM-SHA256:AES256+EECDH:DHE-RSA-AES128-GCM-SHA256:AES256+EDH:ECDHE-RSA-AES256-GCM-SHA384:DHE-RSA-AES256-GCM-SHA384:ECDHE-RSA-AES256-SHA384:ECDHE-RSA-AES128-SHA256:ECDHE-RSA-AES256-SHA:ECDHE-RSA-AES128-SHA:DHE-RSA-AES256-SHA256:DHE-RSA-AES128-SHA256:DHE-RSA-AES256-SHA:DHE-RSA-AES128-SHA:ECDHE-RSA-DES-CBC3-SHA:EDH-RSA-DES-CBC3-SHA:AES256-GCM-SHA384:AES128-GCM-SHA256:AES256-SHA256:AES128-SHA256:AES256-SHA:AES128-SHA:DES-CBC3-SHA:HIGH:!aNULL:!eNULL:!EXPORT:!DES:!MD5:!PSK:!RC4";
  ssl_dhparam /etc/ssl/certs/dhparam.pem;

  # Add headers to serve security related headers
  add_header Strict-Transport-Security "max-age=15768000; preload;";
  add_header X-Content-Type-Options nosniff;
  add_header X-Frame-Options "SAMEORIGIN";
  add_header X-XSS-Protection "1; mode=block";
  add_header X-Robots-Tag none;

    root /var/www/panel.example.com;
    index index.php;

    client_max_body_size 20m;
    client_body_timeout 120s;

    location / {
        try_files /public/router.php =404;
        fastcgi_split_path_info ^(.+?\.php)(/.*)$;
        fastcgi_pass unix:/var/run/php5-fpm.sock;
        fastcgi_index router.php;
        include /etc/nginx/fastcgi_params;
        include fastcgi_params;
        fastcgi_param SCRIPT_FILENAME $document_root$fastcgi_script_name;
    }

    location /assets {
        try_files /app/$uri =404;
    }

}

This error only happens when installing the latest panel version. I did test this on two physical separate servers with the same error.

Are you accessing your site with panel.example.com, because that is what you told nginx to listen for.
Once that is fixed, if it still doesn't work it is because the assets URL got entered wrong, go into the Admin CP and go to URL Settings and submit the form, it should correct them automatically for you.

Whatever, changing https --> 0 to https --> 1 in acp_settings did the trick. I might add a pull request to the script with additional question if server will be ssl on default. Thanks!


mtd.c: W25 flash memory with configurable partition size and type

Support for mtd attached memories to have partitions of defined size instead of equal splits.  Each partition can be a character access block or FAT formatted.  This enables a fixed flash memory to emulate a removable sdcard.
Basic functional tests on ramtron based hardware have been done.
Winbond W25Q64 has been tested on alternative hardware.

Closed - to be reopened against a branch on fork instead of master on fork


Auto install prereqs on launcher start to game-files/launcher/prereqs

closes #433
Still need to work out when/if to run these, but that will be a separate issue

Superseded by https://github.com/Wildlander-mod/Launcher/pull/566


A single failing (Telos) node should not break the app

What
If a node fails to respond to requests the app will currently not work. The fix should be that the app should try X number of other nodes before telling the user.

Closing as a legacy issue.


Some Settings In External Configuration Overridden By Inline Defaults

The Problem
I discovered that the warn-only parameter is not settable in an external configuration file by first seeing it get ignored when set only there, then seeing it honored when it was set inline. I believe that this is the source of the issue:
The default value for warn-only is set here:
https://github.com/actions/dependency-review-action/blob/40eb2b8b00a215a5a327a2de3fdf3d6ab861c1fb/action.yml#L64-L67
This means that this parameter, along with every other parameter that has a default value defined in action.yml, will have an inline setting when the code is merging configurations. However, the way that they are merged is problematic:
https://github.com/actions/dependency-review-action/blob/40eb2b8b00a215a5a327a2de3fdf3d6ab861c1fb/src/config.ts#L19-L22
As documented in MDN, whenever two objects are merged using the spread operator, fields in a later object will override fields from an earlier object with the same key. This makes warn-only and other settings with default values un-configurable from an external file. This is a problem if you want to distribute one configuration file across PR checks in many repositories, including some owned by others that are subscribing to your settings.
Suggested Fixes

Flip the order in which the settings are merged with the spread operator so that parameters set in the external configuration file override inline settings.
Add another boolean parameter that dictates if the inline settings override the external settings, with a default value of false in order to fix the bug.
Simply remove the default values from all of the parameters except one. They are all denoted as not required in action.yml:
https://github.com/actions/dependency-review-action/blob/40eb2b8b00a215a5a327a2de3fdf3d6ab861c1fb/action.yml#L7-L10
https://github.com/actions/dependency-review-action/blob/40eb2b8b00a215a5a327a2de3fdf3d6ab861c1fb/action.yml#L56-L67
...and they are all pulled into the action using functions that denote they are optional:
https://github.com/actions/dependency-review-action/blob/40eb2b8b00a215a5a327a2de3fdf3d6ab861c1fb/src/config.ts#L44-L50
The one exception to this is repo-token:
https://github.com/actions/dependency-review-action/blob/40eb2b8b00a215a5a327a2de3fdf3d6ab861c1fb/src/dependency-graph.ts#L13
...however, tokens shouldn't be hardcoded into config files anyway, so that's not an issue like the others.
Move the setting of each default value into the underlying code after checking to see if the parameter in question is not set in the inline configuration or an external settings file. This is the only suggested solution that isn't a breaking change, but it also makes the code more messy, since there are now three places that parameters could be set.

@AlexWilson-GIS thank you for the very detailed report. Can you test the Action with the branch remove-warn-default and see if it fixes the issue?

    uses: actions/dependency-review-action@remove-warn-default

Pull request up here: https://github.com/actions/dependency-review-action/pull/722

Unfortunately, it appears to still be failing.

@AlexWilson-GIS thanks for the update, we'll try to go through it this week and see what's up. If possible, please leave the reproduction repo up!

@AlexWilson-GIS I think I understand what the problem is now: If you take a look at the raw log for your PR, the error you are running into is Error: Dependency review detected denied packages.. The denylist contains the faulty log4j package, so it'll fail regardless of warn_only or fail_on_severity.
I created this PR with the same pom.xml you are using, but without providing a denylist for log4j. You can see it has been marked as a successful run. If you dig into the logs however, you can see the vulnerable package:
Vulnerabilities
  pom.xml »<EMAIL_ADDRESS>– Improper Input Validation and Injection in Apache Log4j2 (moderate severity)
    ↪ https://github.com/advisories/GHSA-8489-44mv-ggj8
  Warning: Dependency review detected vulnerable packages.

These are two different issues:

There is a problem in the latest release and in main with the handling of external config files (and other options). This PR provides a fix for that.
warn_only does not work in conjunction with deny_list. This is not a bug, but we might want to reconsider this interaction. The reasons for this behavior are historical, not technical.

I'm marking this as closed and will be merging the PR/doing a new release in the morning. If you feel we should change the behavior of warn_only to take deny_list into account (understandable!) please open a new issue (cc @jonjanego).

Sounds good, thanks. I will open a new issue to request that.


Give focus to search text field when new node menu appears

Usually when the menu is triggered, I try to type right away assuming the search text box has focus.  It doesn't though.  So then I have to do a second click.  Would be nice to optimize this workflow by focusing the text box right away if the menu opens.

never mind. it seems to behave that way.  I am confused. But anyhow this issue is incorrect.


Move crate-level lint config to Cargo.toml lints table

          This can be configured per each crate. ([See announcement](https://blog.rust-lang.org/2023/11/16/Rust-1.74.0.html#lint-configuration-through-cargo)) If other crates don't use the `[lints]` table, then perhaps they need to be changed as well.

Originally posted by @BD103 in https://github.com/bevyengine/bevy/pull/11341#discussion_r1451801529

Seems currently impossible: https://github.com/bevyengine/bevy/pull/11341#discussion_r1451803090


Implement reconnecting if devices are missing on service start

I've moved initial connection logic from the _create_device_link() method into _connect_single_link so failures may be handled by already existing retry logic.

Great stuff. Had a quick glance and I really like it! Had one or two questions, but maybe they'll be answered by a fresh pair of eyes tomorrow.
Thank you @ig-sinicyn

Hey @ig-sinicyn
Just to give you a heads-up: I'm holding this merge back until #46 is solved. Would rather not introduce another unknown variable to this.
Hope everything's still working nicely for you? Anything you miss or would like to see implemented?


Django project in subversion, raven does not work

Project: Django
OS: Windows 10
code manage: subversion
in setting.py:
import raven
RAVEN_CONFIG = {
    'dsn'<EMAIL_ADDRESS>    # If you are using git, you can also automatically configure the
    # release based on the git info.
    'release': raven.fetch_git_sha(BASE_DIR),
}

When I have errors:
Traceback (most recent call last):
  File "E:/code/PycharmProjects/pss_patent/manage.py", line 15, in <module>
    execute_from_command_line(sys.argv)
  File "E:\code\PycharmProjects\pss_patent\venv\lib\site-packages\django\core\management\__init__.py", line 381, in execute_from_command_line
    utility.execute()
  File "E:\code\PycharmProjects\pss_patent\venv\lib\site-packages\django\core\management\__init__.py", line 325, in execute
    settings.INSTALLED_APPS
  File "E:\code\PycharmProjects\pss_patent\venv\lib\site-packages\django\conf\__init__.py", line 57, in __getattr__
    self._setup(name)
  File "E:\code\PycharmProjects\pss_patent\venv\lib\site-packages\django\conf\__init__.py", line 44, in _setup
    self._wrapped = Settings(settings_module)
  File "E:\code\PycharmProjects\pss_patent\venv\lib\site-packages\django\conf\__init__.py", line 107, in __init__
    mod = importlib.import_module(self.SETTINGS_MODULE)
  File "E:\code\PycharmProjects\pss_patent\venv\lib\importlib\__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 955, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 665, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "E:\code\PycharmProjects\pss_patent\pss_patent\settings.py", line 23, in <module>
    'release': raven.fetch_git_sha(BASE_DIR),
  File "E:\code\PycharmProjects\pss_patent\venv\lib\site-packages\raven\versioning.py", line 19, in fetch_git_sha
    'Cannot identify HEAD for git repository at %s' % (path,))
raven.exceptions.InvalidGitRepository: Cannot identify HEAD for git repository at E:\code\PycharmProjects\pss_patent

I had lookup a lot of informations, All of body said: I needed git, but My project is in subversion, How can I resolve it???

Then set the release explicitly

Then set the release explicitly

How can I set it? I cannot find the subversion configuration information in raven's documents.


Fix column type for active/standby routes

Checklist

[x] Affected Issues have been mentioned in the Closing issues section
[ ] Documentation has been written/updated
[ ] PR title is ready for changelog and subsystem label(s) applied

The column type was incorrectly defined as varchar(100) when it should be text.
I think there may be another step required to actually modify this though on existing clusters (may need to update this PR with a alter table modify command)?

Closing issues
closes #2171

@shreddedbacon
correct we need an actual alter table command that runs if the type is wrong, there is an example here:
https://github.com/amazeeio/lagoon/blob/605a1986dce090d2eaafe9866481853f5ace9537/services/api-db/docker-entrypoint-initdb.d/01-migrations.sql#L282-L300

@shreddedbacon merged this in, can you create a follow up ticket for creating the alter table?


翻译issue127

翻译的文档的 Issue 编号是：https://github.com/servicemesher/trans/issues/127

all of problems has been fixed.

Done


/dev/null on Windows

os.DevNull is the platform independant way of using /dev/null, but I'm not sure if /dev/null/foo will work as expected here and here on Windows.

At this point in time, both the bosh-cli and development are not fully supported on Windows. Going to close this issue. Please feel free to create a new issue if you would like to discuss the larger idea of support on Windows.

@jmcarp I've already created a larger detail of Windows support here. It is actually very close to being possible, but there are hurdles that only someone inside the bosh team can overcome as currently the development is moving in a different direction, away from cross platform support, e.g. bosh releases for Windows working differently to Linux.
To get create-env to work it requires finding what is opening Linux files/streams as though the were Windows files/streams. I only see the point at which the receiver, e.g. bosh-docker-cpi-release reading one of the certificate passed as a parameter, can't process the data because the new line characters have been read as though they are just a space character.


Enhancement: have separate "neovimPath" config for different OS.

For people that sync their settings across different OS, it would be nice if we can support different neovimPath for different OS, just like the how vscode has different terminal settings:

terminal.integrated.shell.windows
terminal.integrated.shell.osx
terminal.integrated.shell.linux

It would be nice if we can have separate:

vscode-neovim.neovimPath.windows
vscode-neovim.neovimPath.osx
vscode-neovim.neovimPath.linux

Same would be nice for neovimInitPath.
Right now, whenever I sync settings, I break vscode neovim on different OS.
Moreover, useWSL should only have effects on windows.  Currently, turning it on in OSX breaks vscode-neovim.
Thank you very much!

Actually, found https://github.com/asvetliakov/vscode-neovim/pull/157
Closing this issue

I'm not sure if it's released yet though. In the meantime, you can check out the new built-in settings sync on VS Code Insiders. Make sure to set the ”sync.ignoreSettings" (or something like that) to "vscode-neovim.neovimPath" and "vscode-neovim.useWSL"


[NEMO-414] Command-line specified runtime data plane configurations not applied

JIRA: NEMO-414: Command-line specified runtime data plane configurations not applied
Major changes:

User-specified data plane configuration now applies to parameters.

Minor changes to note:

The number of serialization threads for scheduling(schedule_ser_thread) now can be set through the command-line argument.

@hy00nc Thanks for the PR! How about creating a wrapper class that contains the executor variables?
e.g.
final class ExecutorConf {

@Inject
private ExecutorConf(Parameter(JobConf.GlusterVolumeDirectory.class) final String glusterDirectory, ...) {
...
}

getExecutorConfiguration()

}

Thanks for the comment! I created a wrapper class named DataPlaneConf.
To avoid confusion, the existing executor configurations except for the data plane configurations remain as they were before. It is very tricky to clearly separate the executor configurations inside NemoDriver.java :(


Fix for pcs command change! :)

Small fix for this issue, allows newer versions of pcs:
https://github.com/Zorlin/ansible-role-pacemaker/issues/2

cheers!


Add spiceblow-database extension

Description
A Raycast database extension to manage Postgres or Mysql databases inside Raycast, you can think of it like TablePlus but built into Raycast.
About security
The extension can generate where clauses based on natural text, this feature depends on a backend service, this server does not have access to the database connection string, only the database schema is sent to the service to be able to generate the SQL query.
All the data returned by the server cannot update or delete any data in the database, all the queries returned by the server are run in a read only session to prevent any destructive update to the database.
Any update, insertion and deletion to the database is presented to the user before running, showing the full SQL query, the user has to explicitly agree to run the query.
About Google login
This extension uses a google token to identify the user, this token is used to connect a license to the user after the payment via Stripe via webhooks.
To test the payment I prepared a Coupon code to make the service free during testing: LC51UYGZ
Demo
Here is a quick demo of what it can do:
https://github.com/user-attachments/assets/02e7683a-5acb-4906-bd71-5199a01e126d

Do you need any help with the review?

No, we just came back from a offsite last week, let me have a look now 🙂

It’s the API backend, It is used to generate the sql query and sql filter based on user natural text using an LLM, I did not create a landing page yet so it just redirects to Raycast now.
Login is required to later connect the user payment, it also prevents possible abuse in the backend, which uses OpenAI API to generate the sql. This is a paid extension, after a free requests limit the user is prompted to buy a one time purchase license which will be connected to the user Google email in the Spiceblow database.

Hi,

Drafted values are not inserted when opening a draft

I fixed the draft values, thank you

Use the term SQL instead of Sql so it's consistent

Done

Can a lot go wrong with this extension, do you think? It feels a bit insecure to just have it like that (since you have access without any passwords)

Some popular apps like TablePlus already exist where there is no password to access the app, Raycast already stores LocalStorage data in encrypted storage so the connection string is safe.

Maybe we should show a toast and poptoRoot

What do you mean? In what part of the extension?

What do you mean? In what part of the extension?

Sounds like some of my text was not included 🫠 When deleting a database, otherwise you might end up in a view where a database is deleted

It's not possible to delete databases when you reaching the limit, maybe it's possible to delete from this view too?

Sounds like some of my text was not included 🫠 When deleting a database, otherwise you might end up in a view where a database is deleted

Ok, I will add a pop after you delete a database, thank you for telling me
You can use code LC51UYGZ to buy the license for free and see everything works if you want to test database deletion again.

It's not possible to delete databases when you reaching the limit, maybe it's possible to delete from this view too?

If the use case is to delete the data without buying a license you can use the "clear extension data" action in Raycast.

OK, I will add a pop-up after you delete a database. Thank you for telling me

Awesome 🔥

If the use case is to delete the data without buying a license, you can use the "clear extension data" action in Raycast.

Maybe we could add an action to push to the overview, where it's possible to delete, or is that too much? It should be intuitive

Maybe we could add an action to push to the overview, where it's possible to delete, or is that too much? It should be intuitive

I don't think it is useful, to use the extension you have to buy the license anyway, it would just add noise in my opinion.

The first thing an user would do to delete the data is to uninstall the extension, which will clear the extension storage, right?

It would yes, let's stick with the current flow - I'll re-test it

Thank you!


Set deletion_protection=false in order to destroy resource "google_bigquery_table"

Hi Bruno,
In your file modules/stream2bq/main.tf, for the resources  "google_bigquery_table" "assets" "compliance_status" and "violations" you're setting the deletion_protection parameter to true.
In order to be able to destroy those resources through terraform, it needs to be set to false.
I understand that the documentation recommend to not set this field (or set it to true) until you're ready to destroy, but would it be possible to change the value to false ? I tried to override the value when deploying RAM but without success.
Best regards,

Hi Yann,
Thanks for contributing.
Instead of changing the deletion_protection setting from true to false it is now exposed from the module realtime-asset-monitor as variable bq_tables_deletion_protection with a default value to true that you can change to false
to be released in v0.7.1


fix: style 引入失败

👀 PR includes
🐛 Bugfix

[x] Solve the issue and close #0

📝 Description
s2-react 的 tooltip css 退回引入 dist 的方式
🖼 Screenshot

Before
After

❌
✅

🔗 Related issue link

🔍 Self Check before Merge

[ ] Add or update relevant Docs.
[ ] Add or update relevant Demos.
[ ] Add or update relevant TypeScript definitions.

:tada: This PR is included in version @antv/s2-react-v1.17.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket: