Stdin does not work with quoting on $input

There is either something wrong with the handling of quotes when receiving input from json -> jq and parsing it to a quoted variable to bash.
Proof of concept to replicate below.
Created addon for test/play/verification: https://github.com/endor-force/stdin-addon
Service call: {"addon":"local_stdintest","input":"l /"}
#!/bin/bash
set -e

# Listen for input to tdtool
echo "[Info] Starting event listener"
while read -r input; do
    # removing JSON stuff
    input="$(echo "$input" | jq --join-output '.')"
    echo "[Info] Read alias: $input"

    if msg="$(ls -"$input")"; then
        echo "[Info] TellStick Command success -> $msg"
    fi
done

Sending data without spaces works.
But if you send l / (to run ls -l /) it will fail.

that will work?

no.. it did not

now I know what do you want. You need parse it into array by spliting with ' ' and parse the array to args like:

Maybe it will be easy if you modify the input as json like:
{
  "type": "learn",
  "id": "id"
}

So you can parse the input data like this schema. Maybe you can use other types for different functions.

Tried this.. Same result.. fail.
input_arr=()
for word in $input; do
  input_arr+=("$word")
done

echo "[Info] Read array: "${input[@]}""

if msg="$(ls -"${input[@]}")"; then
  echo "[Info] Command success -> $msg"
fi

Verified theory with tellstick add-on locally and it works.

Forget it.. with * it is back to square one .. it does not work in reality.

Thank you! 🥇 👍

Works like a charm.
Updated documentation: https://github.com/home-assistant/home-assistant.github.io/pull/4156


Use send to monkey patch to support Ruby < 2.2.0

Fixes https://github.com/jekyll/jekyll-redirect-from/issues/135.
Prior to Ruby 2.2.0, the include class method is private. Starting with Ruby 2.2.0 it's public.
This PR ensures our method of "cleaner" monkey patching is compatible with Ruby >= 2.0.0 and < 2.2.0.
/cc @michaelcurry

@jekyllbot: merge +minor.


how to map a rust enum to an sql varchar type in postgres?

e.g. I have a simple enum in rust
#[derive(Debug, Serialize, Deserialize, Clone, PartialEq)]
pub enum TradeType {
  Buy,
  Sell,
}

pub struct Trade {
  pub trade_type: TradeType
}

What can I do to map this struct to a table where trade_type is defined as VARCHAR
Currently I get compilation errors e.g. (expected enum '...', found struct std::string::String) when using macros like sqlx::query_as!

This seems more suited for Discussions instead of an issue. And I'm pretty sure you should find an answer in the docs or existing issues / discussions.

@jplatte yeah thank you for the hint. Of course I tried to find an answer (stackoverflow, docs etc.) but I was not successfull.  But I will close this issue and use the discussions area


Add retry for clone step

Proposed Changes

Added a retry settings for clone step, allowing repeat the process three times.

Types of Changes

Drone CI

Verification

Testing

Linked Issues

User-Facing Change

Further Comments

Is this necessary? If the clone steps fails, there is likely a GitHub outage that will require a re-run of CI pipelines due to other subsequent failures.

Other than github outages, have we had sufficient problems with the clone step that make this necessary?

The repeated issues were what inspired the effort. Has happened a number of times with rke2 and k3s over the last few releases but we're overlooked chalking it up to anomalous behavior. This is also part of the effort for more automation and to remove the need for unnecessary manual intervention when a restart is all that's needed.


Add support for passing inputs to ViewGroups

Today ViewGroup is missing a lot of functionality. One main use case is adding support for passing inputs when calling (navigate-view-group)[https://docs.ensembleui.com/actions/navigate-view-group]
Within the ViewGroup, you should be able to use expressions to dynamically assign properties/evaluate inputs.
suggested yaml -
navigateViewGroup:
  viewIndex: 1
  inputs:
    foo: bar

cc @amin-nas


HELM-129: Use the Measurements API in relaxed mode

JIRA: https://issues.opennms.org/browse/HELM-129
Here we set relaxed=true by default in the queries to the Measurements API. This allows queries to be processes that contain series which are not present.
I opted to always keep this enabled as opposed to adding another flag to enable/disable it.

LGTM, I'll test when you resolve confilcts with Dustin's merge.

WFM


Firestore Client Doesn't Work in Python 3.3

Firestore library doesn't work on Python 3.3. Here's a sequence of steps to reproduce:
$ virtualenv sandbox
$ cd sandbox
$ source bin/activate
$ pip install google-cloud-firestore --upgrade
$ python --version
Python 3.3.0

$ python -c 'from google.cloud import firestore'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/usr/local/google/home/hkj/Projects/firebase-admin-python/public/sandbox/lib/python3.3/site-packages/google/cloud/firestore.py", line 18, in <module>
    from google.cloud.firestore_v1beta1 import __version__
  File "/usr/local/google/home/hkj/Projects/firebase-admin-python/public/sandbox/lib/python3.3/site-packages/google/cloud/firestore_v1beta1/__init__.py", line 20, in <module>
    from google.cloud.firestore_v1beta1 import types
ImportError: cannot import name types

The issue seems to be grpc-related. The dependency takes an unusually long time to install.
Here's a link to a Travis build of the Firebase Admin SDK which depends on Firestore. Note how things fail only in the Python 3.3 environment.

Officially, we only support Python 3.4 and up (and 2.7), in keeping with the policy if most other large open source projects. That said, we will take a look at this and fix it if it is straightforward. :-)

This is most likely down to pip / setuptools bugs WRT namespace packages. @hiranya911, can you please try adding the following:
$ pip install --upgrade setuptools pip

Before installing google-cloud-firestore?

@tseaver Same error as before. Nothing was updated when I tried to upgrade setuptools and pip. My versions are:

pip 9.0.1
setuptools 36.5.0

Given that this also occurs in a Travis CI environment, I doubt that it is due to outdated tools.

Given that this also occurs in a Travis CI environment, I doubt that it is due to outdated tools.

Travis is not notorious for having up-to-date environments, e.g. their latest Ubuntu image is the previous LTS from April 2014.

Hi @hiranya911,
I looked into this, and the ImportError is actually coming from grpc, which is issuing import enum.
enum was added to the standard library in Python 3.4, but there is a backport on PyPI for previous versions (pip install enum34). If you install enum34 from pip when using Python 3.3, things might work.
What I mean by "things might work" -- doing so will get you past that particular ImportError. I do not guarantee everything will work past that, since we do not officially support or test on Python 3.3. (Your chances are decent that everything will work fine, though.)

@hiranya911 I also filed grpc/grpc#12976 that will fix this on their side.
Again, no promises of Python 3.3 support, but this seems like sufficiently low hanging fruit.


Partial convert mxGraph to es6 based on a provided codemod

Based on the codemod provided by @adrianhdezm : https://github.com/adrianhdezm/mxgraph-codemod
regenerate a base es6 version of mxGraph from this repository sources and test it.
To run the codemod :
launch node on the index.js with the clone of this mxGraph repo in javascript dir in parameter :
e.g. : node index.js /home/user/mxgraph-clone/javascript
I'm not yet sure if it is the correct procedure, @adrianhdezm could you confirm?

you could exec as below:
node index.js /home/user/mxgraph-clone/javascript/src

Please confirm the dir. you know, only dir 'src' should be transpiled.
And this is the start.

Thanks @liyunhechn , indeed it is working well launching it directly on src.
I'll now check every classes generated for the missing part and add the imports

Hi @coclav, no sure what is the flow here, but could I have access to the repo to push the work on this issue into a dedicated branch?

I can do this for you @CommunityMan - what  branch name is best?

IMHO we can close this issue. We already have a conversion done with #16

@tbouffard I agree - do you want to go ahead and close this issue #5

So closing as @jgadsden agreed 😸


Checks before starting TiKV

Before starting TiKV with existing RocksDB, we should do some checks, including:

Is the db created by a newer version of TiKV?
Is the dynamic linked RocksDB library supported?
Does the db need to be modified or cleaned up before providing service?

@siddontang I am interested in learning more about TiKV, can you assign this to me?

Thanks @Ishiihara
But we have not figured out a way to handle this issue yet. :sob:
Maybe we can check dynamic linked RocksDB version first,  we can get the minimum version from rust-rocksdb, then check with current RocksDB.
/cc @BusyJay

This is closable since we use our own RocksDB now.


Prevent deletion of snapshot backing index

Description
The changes prevent deletion of snapshots that are backing searchable snapshot indexes
Issues Resolved
#4967
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Can you add some tests of the new behavior?

Codecov Report

Merging #5069 (081a207) into main (933e8c3) will increase coverage by 0.03%.
The diff coverage is 86.95%.

:exclamation: Current head 081a207 differs from pull request most recent head 8434fbd. Consider uploading reports for the commit 8434fbd to get more accurate results

@@             Coverage Diff              @@
##               main    #5069      +/-   ##
============================================
+ Coverage     71.07%   71.11%   +0.03%
- Complexity    58135    58153      +18
============================================
  Files          4704     4705       +1
  Lines        277266   277289      +23
  Branches      40147    40151       +4
============================================
+ Hits         197080   197194     +114
+ Misses        64001    63923      -78
+ Partials      16185    16172      -13

Impacted Files
Coverage Δ

...pensearch/snapshots/SnapshotDeletionException.java
40.00% <40.00%> (ø)

.../main/java/org/opensearch/OpenSearchException.java
92.35% <100.00%> (-0.20%)
:arrow_down:

...n/java/org/opensearch/snapshots/SnapshotUtils.java
87.50% <100.00%> (+4.16%)
:arrow_up:

...ava/org/opensearch/snapshots/SnapshotsService.java
50.43% <100.00%> (+0.43%)
:arrow_up:

...g/opensearch/index/analysis/CharFilterFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...search/indices/recovery/RecoveryTargetHandler.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rch/test/junit/listeners/ReproduceInfoPrinter.java
9.52% <0.00%> (-66.67%)
:arrow_down:

...a/org/opensearch/tasks/TaskCancelledException.java
50.00% <0.00%> (-50.00%)
:arrow_down:

...cluster/coordination/PendingClusterStateStats.java
20.00% <0.00%> (-48.00%)
:arrow_down:

...opensearch/persistent/PersistentTasksExecutor.java
22.22% <0.00%> (-44.45%)
:arrow_down:

... and 469 more

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

Is this a breaking change for 2.x? Was the user experience broken when these snapshots were deleted in 2.4.0?
@Vishalks this will need a manual backport

Was the user experience broken when these snapshots were deleted in 2.4.0?

Yes, it was broken. Searchable snapshots are still behind a feature flag so we have some latitude for changing behavior here.


c-tagging shape SFs

Implementation of the c-tagging scale factors.

Rebased to main branch.
This PR is ready for review, please take  look @valsdav .
It introduces c-tagging SFs and norm calibration corrections.


Sort Block Producers

Block producers can be sorted by

Votes
Alphabetical
Uptime
Node Types
etc

see example sorting options
https://glass.cypherglass.com/map/main/all


Handle deletes

Add isDelete parameter to endpoints that is then transferred to the CHSDelta AVRO object

Not required.


Add Metrics to core project

This PR adds various metrics to Silk. This PR is based on the redis branch, so that PR must be merged first.
When this is merged, a new subdomain will be assigned on the silkbot.cc domain, likely stats.silkbot.cc as to not interfere with the dashboard, which will likely be assigned to dash.silkbot.cc, though silkbot.cc/dash[board] is also acceptable, though introduces ambiguity.

As a status update, this may be postponed until we migrate over to k3s.


Anyway to print without it always being on a new line?

It looks like the source code uses fmt.Println for everything, but what if I want things to be on the same line just in a different print statement.   For example, if I wanted to use two colors on the same line.
How would I do this? Or do I have to edit the source file?
Thanks!

Check out this section of the readme.
https://github.com/fatih/color/blob/master/README.md#insert-into-noncolor-strings-sprintfunc

It seems this fails on Win10. I'm building on Mac with [build] constraints to compile for Windows/Linux/Darwin. With the followings
Main_win.go
yellow = color.New(color.FgYellow).SprintFunc()
Main.go
var yellow func(a ...interface{}) string
fmt.Printf(" %s \n", yellow("test"))
Stdout > test

Hi,
Unfortunately, I'm archiving this project and will no longer maintaining it. For more information, please read my blog post: Taking an indefinite sabbatical from my projects.
Thanks for the feedback and contribution.


Use ppx_irmin internally

Last in the batch of PPX updates for now :) This one uses the new features in ppx_irmin inside the irmin repository itself.
Will need to watch Travis closely to see if opam is able to handle the new constraints.
Depends on https://github.com/mirage/irmin/pull/1087.

Now rebased, which makes the negative diff nicely obvious :slightly_smiling_face:


CI failure with 1.2.0 as base compiler

I have created a branch to prepare master for 1.3-dev (https://github.com/crystal-lang/crystal/tree/ci/1.3-dev) including updates to 1.2.0 as base compiler for CI.
But there are errors in CI: https://app.circleci.com/pipelines/github/crystal-lang/crystal/7044/workflows/0dffd249-0e5a-44dd-a6f1-38476f2022b8
Looks like LLVM runs out of memory. It's inconsistent (which is not unlikely for such a failure). Do we need more memory on our CircleCI runners?

For all I know, 1.2.0 should maybe even use less memory due to #11167 and #11188.


Using a multiple instance crashes the game

If you do FLMusicLib.new() twice and try to use the other one, or play more then one sound, it crashes the game

Multiple instances should work in version 0.5.1 #1


[Feature Request] Preview Selected Item (CRTL+SPACE) -> Ability to Open File from the Modal

Hello @qawatake !!!!!!
This FR is related to the behavior inside the Preview View after hitting CRTL+SPACE
Would be amazing to be able to open the file (using the command CRTL+ENTER) while the user is located in the Preview View (CRTL+SPACE)
After using the preview view, the user can exit (ESC) or open the note (CRTL+ENTER) to start working :)

Thanks for reading this :)

implemented in #7

implemented in #7

Hello @qawatake !!!!
Working perfect on version 2.0.0 on Obsidian 0.13.19
Thank you very much !!!!!


Basic 'Hello World' Port/Channel example causes abort/memory leak

I tried to run the example code from the top of lib/comm.rs (here, for your convenience: http://pastebin.com/kwhhMK3Z)
Output of rustc --version:
rustc prerelease (06d14f3 2011-11-07 15:46:00 -0800)
host: i686-unknown-linux-gnu
OS: Ubuntu 11.10, amd64
Error Message:
rust: fatal, 'leaked memory in rust main loop (1 objects)' failed, ./src/rt/memory_region.cpp:131 1 objects
Aborted
FWIW, I tried taking out the argv, that was not the cause. Its my first time using rust, so its possible I did something wrong, but I figured I'd be cautious and open a bug.

This is cute. By default log is turned off, and doesn't evaluate its arguments, so recv isn't even being called. Combine that with the fact that we don't clean up our queues when data gets left in them and the "Hello World" string gets leaked. I bet if you changed 'log' to 'log_err' it would work as expected.
I think we should fix the example to be less error prone, and create another issue for properly cleaning up the port queues when they die.

Partially for my own edification, I just spent a few minutes poking around in the rt/ folder trying to get a handle on this bug/how ports & channels are structured internally. When the string is sent from the channel, it's buffered by the channel until one if its ports calls recv? And if there are >1 ports bound to one channel, only one port will ever recv "Hello World", or is it broadcast to everyone?
If my understanding is correct, this bug would get fixed by modifying the destructor in rt/rust_chan.cpp?

also how do I enable log, for future reference? My understanding is that its somewhat of a language builtin?

There's a section in the usage FAQ about enabling logging: https://github.com/graydon/rust/wiki/Usage-FAQ and there is some additional information in the language reference documentation.
There is only ever one port associated with a given channel, but there may be any number of channels associated with a port. It looks like messages are queued in individual ports. The fix here is that the rust_chan destructor needs to run the drop glue of the channel's element type on every element in the queue.
This presents some problems in that the channel doesn't currently know the type descriptor of the type it contains, and because channels can be sent across tasks, but type descriptors can't. Looks like a bit of a mess - probably we can just pull the drop glue function out of the type descriptor and store it with the channel instead of the entire type descriptor. But now that I think of it, we aren't going to be able to run the drop glue from C++ code once we have split stacks.
Maybe instead of drop glue we should just walk the shape of the type, like the cycle collector.
As a stopgop, we could just call kernel->free() on every element when we know that they are pointers. That would at least fix this problem with strings, vectors and unique boxes. It would not fix the problem with structural types that contain pointers though.

Both this and #1038 require tydescs that can move between tasks

This turned out to be easier than expected because all the queuing actually happens on the port side, so it was just a matter of draining the queue (in rust code) when the port dies.

Reopening. There are still two cases that aren't covered:

Ports are refcounted, so task A can take a ref, task B can run the Rust-side port destructor, then task A can send to the port and drop the ref.

I don't know what happens when you try to send after the port doesn't exist


Information disclosure

Hello @msaw328
I've noticed that in gitignore and git config files you have referenced another file called 'gitconfig.identity' which presumably contains sensitive data you wish not to be exposed publicly on github. However, even exposing the name of the file (along with it's relative path ) could result in the potential attacker being able to identify such file existing on your filesystem and obtaining sensitive information like login or credentials.
To get around this I strongly encourage you to use following gitignore syntax:
# ignore everything
/*

# except for:
!.gitignore
!git/
git/*
!git/config

After thorough consideration i have realized that this issue does not make any sense.  Closing.


How to get position in every 50ms?

onPositionChanged returns position in  every  200ms. So how can i customize this? Like how can i get position in every 50ms or any other custom value?

This is now possible in v6.0.0. By default it updates on every frame now. You can customize this with PositionUpdater though.


Remove engine.Table from docker search and fix missing field

And registry/SearchResults was missing the "is_automated" field.
I added it back in.
Pulled this 'table' removal one from the others because it fixed
a bug too
Signed-off-by: Doug Davis<EMAIL_ADDRESS>

LGTM

ping @dmcgowan @dmp42 for registry

oooo cool LGTM

LGTM


Use java.util.Base64 for encoding and decoding

Changes
These changes enable us to use standard Java Base64 encoding/decoding available since JDK8. This allows us to remove a third-party dependency, is more idiomatic Java, and may offer performance improvements over the Apache library.

Use java.util.Base64 for Base64 encoding and decoding
Throw appropriate exception when attempting to decode invalid Base64 token
Remove commons-codec dependency

NOTE: These changes are targeted at v4 of this library.

References

java.util.Base64 JavaDoc
JavaWorld article on java.util.Base64 encoding and decoding

Testing
Tests updated to use java.util.Base64 encoding and decoding.
Checklist

[X] I have read the Auth0 general contribution guidelines
[X] I have read the Auth0 Code of Conduct
[X] All existing and new tests complete without errors

@jimmyjames Do you have a timeline for when you expect to release v4?

Has this been included in any release yet?

We've had to pause on v4 work, but there is good news. We are going to bump the min Java version to 8 in an upcoming minor release, prior to the end of the year (see #457 for more info). Once that is complete, this (and other changes that require Java 8) can be included in the current release stream. Given that, makes sense to open a new issue to apply this change on master once that Java 8 change is made, so created that in #461


docs: Fix typo in plans.html

This was not found by a typechecker, I just stumbled across it :/
It would be really great to have a reliable tool for these sorts of issues, especially on these critical pages.

Yeah, I guess once we have spellchecker tooling in place, there'll need to be a few rounds of tasks of making the spellchecker run on new parts of the codebase (e.g. here, we want it to run on the HTML templates).


RFC: Change operator method lookup to be less magical

 Issue by nikomatsakis
Wednesday Feb 13, 2013 at 21:40 GMT
For earlier discussion, see https://github.com/rust-lang/rust/issues/4920
This issue was labelled with: A-traits, A-typesystem, B-RFC, I-cleanup, P-backcompat-lang in the Rust repository

Right now, operator "methods" proceed according to the same logic as any other method.  But this leads to surprising results like this: https://gist.github.com/bstrie/4948410
Also, operators are already handled somewhat specially because we auto-ref their arguments.  This means that you can write val1-of-my-linear-type == val2-of-my-linear-type and it doesn't move or do anything creepy (to call eq explicitly, in contrast, you'd write val1-of-my-linear-type.eq(&val2-of-my-linear-type)).
I think method lookup for operators should never apply any automatic transformations to the receiver or arguments, other than the current auto-ref behavior (which means that a OP b is kind of implicitly (&a).OP(&b))
@pcwalton I expect you in particular might have an opinion about this.

This has largely been covered by other RFCs and/or implemented. Closing.


x/tools/go/analysis/passes/lostcancel: add new context functions and make it configurable

Go version
go version go1.23.2 linux/amd64
Output of go env in your module/workspace:
-

What did you do?
run nogo
What did you see happen?
Analyzer lostcancel does not check if cancel returned by new context functions (WithCancelCause, WithDeadlineCause, WithTimeoutCause) is called. It is also not possible to add new import paths to check to the analyzer,
What did you expect to see?
Analyzer lostcancel checks for these new functions too. It also has configuration flag to specify additional import paths for context-like packages. You may also add another flag to specify more functions to check but this is more tricky in general, as cancel function may not be returned as a second value as for context functions.

Related Issues and Documentation

cmd/vet: -lostcancel: check for discarded result of context.WithCancel
proposal: annotation to generalize `lostcancel` for other kinds of cleanup function #65682

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

CC @golang/tools-team

Thanks for the report. Adding the three new context functions is trivial; I'll do that presently.
Making it configurable is more work, and in our experience no-one ever configures analyzers, so they need smart defaults or heuristics to infer the relevant patterns. A good heuristic here might be "returns a context.CancelFunc". A generalization of this might be "returns any func value named cancel, release, or stop" or "returns a func of a named type whose type name includes those words", but both of those heuristics would need an empirical evaluation.

In my case I need to check my multicontext type, which has the same With* functions as context, so ability to add new import path would be enough for me.
We already use configuration option provided by printf analyzer to check if our logging printf-like functions are used correctly. I would like to do something similar with lostcancel.

Interestingly, the printf analyzer is able to infer whether a function is printf-like based on whether it delegates (perhaps indirectly) to fmt.Sprintf; this is an example of how it is simpler and more robust to get the tool to recognize the pattern than to use an allowlist. (I wonder why your logging printf-like functions need the configuration option. Is it because dynamic calls are involved, so the analyzer can't observe the delegation? See #58340.)

Yes, we have logger interface like in linked issue.

Thoughts:

My intuition is that we should prefer to either infer the delegation or require some kind of source annotation within the With* functions. I think this matches what @adonovan said.
allowlists end up kinda hacky. They can quickly fix specific problem for motivated users (like those that post issues to the go tracker). But they scale kinda poorly as all users need to know to reach for them and remember to set the desired knobs, and they constrain the API going forward.
The different configuration solutions are better matches for a proposal IMO. I kinda think this issue should be closed (the new functions were added) and a proposal should be filed for configuration.
How frequently do we think users need configuration for lostcancel?


proper dark mode

you mean the overall stroke color of the points?

all strokes:

or should it just of the selected one?

with selection

Just on selection.


got a 502 when state has tasks

my asynqmon version is 0.20, and asynq version is 0.18, when a state have tasks, it will show 502 error page, but have no tasks is ok

the panic log is below

Hi, looks like you have an invalid uuid on Redis (or invalid protobuf message).
Did you follow the migration guide?
https://github.com/hibiken/asynq/wiki/Version-Upgrade-Guide-(from-0.17-to-0.18)

sorry, that's my fault, one server has not update successfully, so it send the old data, thank you for your answer


When scrolled down, after the refresh the scroll position is reset

When I edit the LilyPond text file and save, the PDF gets regenerated but the scroll position isn't preserved.

Can repro--thanks for this.
I'm looking into overhauling the entire PDF rendering thing--I've been following another extension that has inherent problems like these (bad choice), but I found another extension that does things more nicely and does fix this. Bear with me for a moment with this--I'll need some time to work on it

Fixed in 0.1.9 #23 :)


⚠ 366.ru has degraded performance

In 0ab2241, 366.ru (https://366.ru) experienced degraded performance:

HTTP code: 200
Response time: 1496 ms

Resolved: 366.ru performance has improved in c9879cb.


Fix#10584: Add Data Model as an entity

Part of #10584
Describe your changes :
Add models and endpoints for Data Model as an entity.
Pending:

Example data
Improve testing
Add/Remove data model to a dashboard

Type of change :

[x] New feature

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers
Backend: @open-metadata/backend

@nahuelverdugo: Could this also be implemented for Power BI Data Models?

Done :)
https://github.com/open-metadata/OpenMetadata/issues/10689


Fix order by price indexation without tax rate (default country fix)

Questions
Answers

Description?
When you configure default sort by for category page with value sort by price and products in category don't have a tax configured (without tax) or an option use taxes is disabled in the module and you have more than one country configured for current shop, so in this case you will not see products in category. It happens only when you access FO using customer that country is different from the default country. The module tries to make joins by current country id, but price index use only default country for each shop/product (the line is not present in DB, because we made indexes only for default country). To fix this problem i propose to fetch activated countries for each shop and add them all to the table of price indexes (not only default country).

Type?
bug fix

BC breaks?
no

Deprecations?
no

Fixed ticket?
Fixes #157.

This change is

@denys202 thank you for your contribution.
The problem is not solved : https://drive.google.com/file/d/1BEPXNM5uXES6FS_rtypa-DYjdxUu-WOf/view
Could you check the video and tell me if there is another step ?

Hello, @sarahdib , yes, to reproduce my issue you should apply this configuration or don't apply taxes for some products (or there is one more reason, i will describe it later):

After that, it will be good to rebuild index of prices and then you will be able to filter products with my fix.
But on your video, I see that there is no filtering by price. I tried to reproduce your issue, by debugging the code and i did it only once: when the product's tax rule group was not associated with selected country. So, i just added my country in tax rule group and products appeared while filtering.
Do not hesitate to write to me if you have any other questions.

@denys202 thank you for the information it's OK now :)


DFPlayer Behavior on LSOS 1.5

I am having troubles with my DFPlayer, the volume is correctly set, but it keeps looping on "Soundfont" sound and doesn't play anything.
I have tried with the old library, and it is sometimes working.
What can I do to fix this ?

try the modes with the usual LEDs on newpixel it happens

I kinda figured it out, why my arduino nano is loaded with too much code (90% of the space), the whole system behaves weirdly...
I think I'll rewrite some features when I have to time so that they are cleaner and lighter

I'm sorry to re-open this issue but I'm really struggling getting this feature to work.
Can anyone help me ?

It was an hardware issue, switched to another dfplayer and it works fine


not able to run ALLCatchR

Hello,
I am trying to run the software using my RNA-seq results. The test included in the package runs ok.
In my sample, I have the counts in tpm, and I'm doing a test with only two samples. The input file looks like this:
"target_id"	"sample_HLP011"	"sample_HLP009"
"ENST00000415118.1"	0	0
"ENST00000434970.2"	0	0
"ENST00000448914.1"	0	0
"ENST00000632684.1"	0	0
"ENST00000631435.1"	0	0
But using the following code , I get an error:
allcatch(Counts.file = "combined_counts.txt", ID_class = "ensemble_ID", sep = "\t")
counts loaded...
Error in apply(Counts.norm, 2, scale) :
dim(X) must have a positive length
I would appreciate any suggestions,
Thank you,
Maria Sol

Hello Maria Sol,
the problem is the ".1"/".2" in the Ensembl IDs. ALLCatchR needs them without the extension.
Best
Thomas

Hello Thomas,
thank you for your reply. I removed the version number from the Ensembl IDs, but I still get the same error. The input looks like this>
value           sample_HLP011 sample_HLP009

1 ENST00000542354          1.53         0.737
2 ENST00000390423          1.48         1.42
3 ENST00000390436          2.93         0.843
4 ENST00000390441          1.01         0.731
5 ENST00000390341          1.73         3.51
Thank you,
Sol

Hello Sol,
the reason is that you provide Enseml transcript IDs, not gene IDs. They start with ENSG. I did not recognize this in your first message. Sorry for that. Do you have the original raw counts for the data? ALLCatchR was not thoroughly tested on tpms.

Thank you, I was able to run it now!


Fix dialogs crashing on X11 when Title is set to null

What does the pull request do?
This PR fixes an issue where setting the Title property to null on X11 crashes the application with an ArgumentNullException in the X11Window.SetTitle method. This mainly caused problems with the OpenFileDialog not functioning properly when managed system dialogs was enabled.
Related issues:
#4001
#5570
What is the current behavior?
When directly setting the Window.Title property to null, an ArgumentNullException is thrown, due to the lack of a null check before calling Encoding.GetBytes in X11Window.SetTitle.
What is the updated/expected behavior with this PR?
null values are now allowed for the Title property.
How was the solution implemented (if it's not obvious)?
When setting null or the empty string, instead of calling Encoding.GetBytes and XChangeProperty, the _NET_WM_NAME X property is deleted from the window handle.
Breaking changes
It may be a potential relaxation of the Window contract, as it would accept null to be set to the Title property.
Fixed issues
Fixes #4001
Fixes #5570

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


Upload of large files blocked on detect-learn

I am trying to upload some large files to my web foto gallery. All configuration is default, when I try to define specific rules (with higher upload limit), then it also fails. What I am getting is 413 errors
I thought that detect-learn shouldn't block anything. Is my assumption wrong ?
When I baypass openappsec on my load-balancer and send the traffic directly to backend server, the it all starts to work again.
Example log:
{"eventTime": "2023-10-14T13:40:21.275","eventName": "Web Request","eventSeverity": "Info","eventPriority": "Low","eventType": "Event Driven","eventLevel": "Log","eventLogLevel": "info","eventAudience": "Security","eventAudienceTeam": "","eventFrequency": 0,"eventTags": ["Threat Prevention","Web Application & API Protection"],"eventSource": {"agentId": "495390e2-2692-4331-9ffc-b7d4908e9315","eventTraceId": "","eventSpanId": "","issuingEngineVersion": "1.1.0-open-source","serviceName": "HTTP Transaction Handler","serviceId": "2","assetId": "Any","assetName": "Any"},"eventData": {"logIndex": 75,"eventReferenceId": "bc9be452-35b1-4c89-9ff1-dc6dd03e48c0","assetId": "Any","assetName": "Any","sourceIP": "<IP_ADDRESS>","httpSourceId": "<IP_ADDRESS>","sourcePort": 47764,"httpHostName": "example.com","httpMethod": "GET","ruleId": "Any","securityAction": "Detect","waapOverride": "None","practiceType": "Threat Prevention","practiceSubType": "Web Application","ruleName": "Any","practiceId": "36dec53c-3703-4357-8505-7993b6c390fd","practiceName": "local_policy/webapp-default-practice","waapIncidentType": "","matchedSample": "","matchedLocation": "","matchedParameter": "","waapFoundIndicators": "","matchedIndicators": "","learnedIndicators": "","waapUserReputationScore": 444,"waapUserReputation": "Normal","waapUriFalsePositiveScore": 0,"waapKeywordsScore": 0,"waapFinalScore": 0,"waapCalculatedThreatLevel": 0}}
{"eventTime": "2023-10-14T13:44:58.865","eventName": "Web Request","eventSeverity": "Info","eventPriority": "Low","eventType": "Event Driven","eventLevel": "Log","eventLogLevel": "info","eventAudience": "Security","eventAudienceTeam": "","eventFrequency": 0,"eventTags": ["Threat Prevention","Web Application & API Protection"],"eventSource": {"agentId": "495390e2-2692-4331-9ffc-b7d4908e9315","eventTraceId": "","eventSpanId": "","issuingEngineVersion": "1.1.0-open-source","serviceName": "HTTP Transaction Handler","serviceId": "2","assetId": "Any","assetName": "Any"},"eventData": {"logIndex": 84,"eventReferenceId": "ff9f3267-4732-4aea-83c7-aa830e69cdd0","assetId": "Any","assetName": "Any","sourceIP": "<IP_ADDRESS>","httpSourceId": "<IP_ADDRESS>","sourcePort": 39264,"httpHostName": "example.com","httpMethod": "GET","ruleId": "Any","securityAction": "Detect","waapOverride": "None","practiceType": "Threat Prevention","practiceSubType": "Web Application","ruleName": "Any","practiceId": "36dec53c-3703-4357-8505-7993b6c390fd","practiceName": "local_policy/webapp-default-practice","waapIncidentType": "","matchedSample": "","matchedLocation": "","matchedParameter": "","waapFoundIndicators": "","matchedIndicators": "","learnedIndicators": "","waapUserReputationScore": 444,"waapUserReputation": "Normal","waapUriFalsePositiveScore": 0,"waapKeywordsScore": 0,"waapFinalScore": 0,"waapCalculatedThreatLevel": 0}}

Hi,
In case the images are larger then the size limitations they will be dropped (hence the 413 response), you can adjust this using the Web UI for managed mode:

or using the policy file for declarative configuration.
Best,
Oriane
open-appsec team

That's exactly what I did ! I have increased the Body size 1000 fold.
Unfortunately the result is still the same.
Any tips how to solve it ?

Hi @SpiderD555,
Thanks for the update and the feedback! We created a new troubleshooting guide for this scenario based on your recommendation.
Best,
Oriane
open-appsec


Migrate details pages to use PatternFly composable table

Migrating the remaining details pages to use the newer PatternFly composable version. The legacy table will be deprecated for the upcoming PatternFly breaking change release.
https://issues.redhat.com/browse/COST-2349

Codecov Report

Merging #2661 (2ad6ab3) into main (46b6567) will not change coverage.
The diff coverage is n/a.

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

@@           Coverage Diff            @@
##             main    #2661    +/-   ##
========================================
  Coverage   65.70%   65.70%
========================================
  Files         451      451
  Lines        8918     8918
  Branches     2081     1937   -144
========================================
  Hits         5860     5860
  Misses       3055     3055
  Partials        3        3

Impacted Files
Coverage Δ

src/locales/messages.ts
100.00% <ø> (ø)

Continue to review full report at Codecov.

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


Send email only when status of service is changed

To prevent huge email spam when service contains many instances,
send notification only when status of service is changed.

:+1:


Fix more button shown after logs deleted

Description
If a more button was shown and all wallet logs are deleted, the button was still shown.
Checklist
Are your changes backwards compatible? Please answer below:
yes

On a scale of 1-10 how well and how have you QA'd this change and any features it might affect? Please answer below:
5. I deleted wallet logs and more button disappeared.
For frontend changes: Tested on mobile? Please answer below:
n/a
Did you introduce any new environment variables? If so, call them out explicitly here:
no

The issue I had before was that deleteLogs from useWalletLogger is in a context where setHasMore is not available.
But I replaced the side effect by wrapping setLogs now since deleteLogs calls setLogs where I then have access to setHasMore.


Does not like matrix for runs-on

I have quite a few jobs with a matrix that defines the runs-on:

...

  process-managers:
    needs: start-time

    strategy:
      fail-fast: false
      matrix:
        engine: ['napi', 'binary']
        feature: [pm2]
        os: [ubuntu-latest, windows-latest, macos-latest]
    runs-on: ${{ matrix.os }}

...

The ${{ matrix.os }} is marked as an error:

But this works just fine in practice.
Via https://github.com/prisma/e2e-tests/blob/dev/.github/workflows/test.yaml

Duplicate of https://github.com/cschleiden/vscode-github-actions/issues/92


Could not unserialize key data

Any idea what the following error means? I have my private key install locally and is accessible
2016-02-15 11:18:01 [startup]
Traceback (most recent call last):
  File "letsencrypt-aws.py", line 486, in <module>
    cli()
  File "/Library/Python/2.7/site-packages/click/core.py", line 716, in __call__
    return self.main(*args, **kwargs)
  File "/Library/Python/2.7/site-packages/click/core.py", line 696, in main
    rv = self.invoke(ctx)
  File "/Library/Python/2.7/site-packages/click/core.py", line 1060, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Library/Python/2.7/site-packages/click/core.py", line 889, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Library/Python/2.7/site-packages/click/core.py", line 534, in invoke
    return callback(*args, **kwargs)
  File "letsencrypt-aws.py", line 432, in update_certificates
    s3_client, acme_directory_url, acme_account_key
  File "letsencrypt-aws.py", line 378, in setup_acme_client
    key, password=None, backend=default_backend()
  File "/Library/Python/2.7/site-packages/cryptography/hazmat/primitives/serialization.py", line 20, in load_pem_private_key
    return backend.load_pem_private_key(data, password)
  File "/Library/Python/2.7/site-packages/cryptography/hazmat/backends/multibackend.py", line 282, in load_pem_private_key
    return b.load_pem_private_key(data, password)
  File "/Library/Python/2.7/site-packages/cryptography/hazmat/backends/openssl/backend.py", line 1606, in load_pem_private_key
    password,
  File "/Library/Python/2.7/site-packages/cryptography/hazmat/backends/openssl/backend.py", line 1802, in _load_key
    self._handle_key_loading_error()
  File "/Library/Python/2.7/site-packages/cryptography/hazmat/backends/openssl/backend.py", line 1874, in _handle_key_loading_error
    raise ValueError("Could not unserialize key data.")
ValueError: Could not unserialize key data.

What format is your private key in?
On Mon, Feb 15, 2016 at 6:23 AM, Marcus Williams<EMAIL_ADDRESS>wrote:

Any idea what the following error means? I have my private key install
locally and is accessible
2016-02-15 11:18:01 [startup]
Traceback (most recent call last):
File "letsencrypt-aws.py", line 486, in
cli()
File "/Library/Python/2.7/site-packages/click/core.py", line 716, in call
return self.main(*args, **kwargs)
File "/Library/Python/2.7/site-packages/click/core.py", line 696, in main
rv = self.invoke(ctx)
File "/Library/Python/2.7/site-packages/click/core.py", line 1060, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/Library/Python/2.7/site-packages/click/core.py", line 889, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/Library/Python/2.7/site-packages/click/core.py", line 534, in invoke
return callback(*args, **kwargs)
File "letsencrypt-aws.py", line 432, in update_certificates
s3_client, acme_directory_url, acme_account_key
File "letsencrypt-aws.py", line 378, in setup_acme_client
key, password=None, backend=default_backend()
File "/Library/Python/2.7/site-packages/cryptography/hazmat/primitives/serialization.py", line 20, in load_pem_private_key
return backend.load_pem_private_key(data, password)
File "/Library/Python/2.7/site-packages/cryptography/hazmat/backends/multibackend.py", line 282, in load_pem_private_key
return b.load_pem_private_key(data, password)
File "/Library/Python/2.7/site-packages/cryptography/hazmat/backends/openssl/backend.py", line 1606, in load_pem_private_key
password,
File "/Library/Python/2.7/site-packages/cryptography/hazmat/backends/openssl/backend.py", line 1802, in _load_key
self._handle_key_loading_error()
File "/Library/Python/2.7/site-packages/cryptography/hazmat/backends/openssl/backend.py", line 1874, in _handle_key_loading_error
raise ValueError("Could not unserialize key data.")
ValueError: Could not unserialize key data.
—
Reply to this email directly or view it on GitHub
https://github.com/alex/letsencrypt-aws/issues/26.

--
"I disapprove of what you say, but I will defend to the death your right to
say it." -- Evelyn Beatrice Hall (summarizing Voltaire)
"The people's good is the highest law." -- Cicero
GPG Key fingerprint: 125F 5C67 DFE9 4084

After running python letsencrypt-aws.py register<EMAIL_ADDRESS>I pasted into a file and save as key.pem

@dottodot
Make sure your key has this format:
-----BEGIN RSA PRIVATE KEY-----
# RSA key
-----END RSA PRIVATE KEY-----


Missing update of Kiosk mode with assigned access for new stable EDGE versions

Hello,
there is already the stable version 88 Build 88.0.705.56 out and the documentation still refers to "the Microsoft Edge Dev Channel, version 87.0.644.4" for assigned access. Could this please be updated?
Regards,
Manfred Pohlemann

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

ID: 229e4d46-19a6-c042-8ce1-844c44c00112
Version Independent ID: 8ab07629-42a6-e7a1-86a9-f8103825ebc4
Content: Configure Microsoft Edge kiosk mode
Content Source: edgeenterprise/microsoft-edge-configure-kiosk-mode.md
Product: microsoft-edge
GitHub Login: @Aguta
Microsoft Alias: aguta

@ManfredPohlemann Hello!  Thanks for the feedback.  I believe the documentation has been updated.
https://docs.microsoft.com/en-us/deployedge/microsoft-edge-configure-kiosk-mode#single-app-kiosk
"Microsoft Edge kiosk mode with assigned access single app is currently available for testing with the latest Windows 10 Insider Preview Build, version 20215 or higher, and with the Microsoft Edge Beta Channel, version 89 or higher."
Thanks!


Prepare for release 0.4.0.

The CI seems borked for this one after all the (connectivity) issues on GH last night. Creating a new one to see if that works, closing this one.


tplink component device name support gone

Home Assistant release with the issue:

0.89.2
Last working Home Assistant release (if known):
0.88.0
Operating environment (Hass.io/Docker/Windows/etc.):

Python venv
Component/platform:

tplink
Description of problem:
this config no long works
- platform: tplink
  host: <IP_ADDRESS>
  name: "Back Room"
  scan_interval: 60

only host: is valid entry.  Should allow alias or friendly_name or better yet, stick with name as before.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

Traceback (if applicable):

Additional information:

I have 10 devices... extremely inconvenient. Why was this removed? Can it not be put back in?

All devices can be set up in Configuration -> Interfaces, it will take just a couple of minutes to rename them all ... and you don't have to restart.

All devices can be set up in Configuration -> Interfaces, it will take just a couple of minutes to rename them all ... and you don't have to restart.

Doesn't answer any of my questions... although i like the idea that they can be renamed in the UI (which I believe has existed for some time) without restart... this is only good for setting up one at a time.
installers or people with existing large number of devices should still have a way to quickly setup in config files.
Another benefit of having them named in config files is quicker to code automations, as my IDE picks up the names for auto complete. Its also a great place to reference when coding (yes, UI can as well, but have to constantly change the filter).

I was not trying to answer your questions but to make you see that it is not so bad.
Your point of view is somewhat addressed in https://github.com/home-assistant/architecture/issues/143#issuecomment-461929615
In short, yes it can be put back but someone has to volunteer to do the work.

@amelchio thanks for that link... I would be happy to try and implement this; would you be able to point me at some more posts that would help me get started? I'm not new to python development, but have no idea where configuration loading would take place, and how it reads/looks for the name or friendly_name directive.  likely a trivial task for someone that has done it before...
My "point of view" has nothing to do with complaining about people volunteering their time to improve HA.  Note that sometimes HA people that have volunteered their time shouldn't really be saying...   Both extremes don't help HA.  I assume that HA wants peoples feedback as I have given.
I'm just trying to figure out why something that worked well was removed before its replacement was ready (i get why auto-config/components is becoming the norm, its great for people less techie.)... but i think it hurts HA's credibility/adoption rate when features are removed, particularly without full explanations as to why, and how, in the opinion of HA devs, it will be better for HA in the long run.

Great. To get help with development, it is best to get on the #devs_backend channel on Discord.
(Sorry, I am not joining the discussion on "why")

@marc-gist see this discussion on the forum for suggestion from @rytilahti. I don't know python, but comparing the new code with the old switch/tplink code, I think this is missing from init.py:
from homeassistant.const import (CONF_HOST, CONF_NAME)

...
DEFAULT_NAME = 'TP-Link Switch'

TPLINK_HOST_SCHEMA = vol.Schema({
    vol.Required(CONF_HOST): cv.string,
    vol.Optional(CONF_NAME, default=DEFAULT_NAME): cv.string
})

But I have no clue where to go from there or how to check if the name is still DEFAULT.

@Enzo-Matrix  thanks, I "fixed" it and did a PR, but the powers that be have closed my PR as they don't deem allow name config in the files is ok... hoping more "powers" look at it and can allow it.
a little frustrated that it currently appears its not even a discussion 😦

I agree, the names should have been left in.  For people with loads of TP-Link switches that are already setup and working fine, this presents issues.  Don't see the reason why the names feature was removed as I'm not aware of it hurting anything.

This was elaborated in chat. The visible name and entity_id are now handled by the Entity Registry. That is available in the frontend UI where both can be changed.
Support for renames can in principle be added to the YAML config but it will have to be in a way that handles the entire Entity Registry so each integration will not have to add code for that.

Makes perfect sense that it be a core feature... however, until someone makes it a core feature, I would still like to see my PR allowed as a temporary "feature".

Even with your PR, the entity_id will be frozen by the Entity Registry after the first setup.
So it is really only about the friendly name and this can already be set in YAML by using customize, https://www.home-assistant.io/docs/configuration/customizing-devices/#friendly_name

@amelchio  i understand that and the reasons to not do it this way normally... but until someone does it in the core, why go backwards? my method although can't change the entity_id once its set in the Registry, it will set the entity_id on upgrade from one version to another, or initial setup (which is typically what the YAML file does - or at least how I've used it.  I've never used the YAML to override something setup via auto config, unless there is an issue... i.e. auto config isn't working... so again, my  Temporary method would work well until a better solution is in place).

In my case, I do NOT have the config option in UI. All I am able to change in the UI are users and lovelace (for lovelace I change the .storage/lovelace file anway), so having an option to change entity names via yaml files is a valuable feature.
@marc-gist can you post the link to your fix? At least I can run a patch with your fix when I build hass.

As I mentioned earlier, I personally have nothing against for this configuration entry, but I see that there is simply no point in that (given that fact that it'll be set to stone after the initial naming), as mentioned already.
Regarding to why it was removed (and this my personal opinion) was that I saw no use for it when the ability to rename them via other means thanks to unique ID was introduced. I wanted to keep the code clean to get it merged at some point and get some people to test it; this wasn't a brand new PR, the whole conversion process took almost half a year to get reviewed and merged, and at no point no one apparently tested it.... For how long should we (as developers) be keep backward-compatibility on such cases when half a year is not enough?
@Enzo-Matrix the PR is here: https://github.com/home-assistant/home-assistant/pull/22122 - and as I said, I don't personally have anything against that, it is just that it does not apparently make sense to have when the ID is bound during the initial initialization of the component...

@rytilahti thanks for the great and straight forward code.  as for "no one testing for 1/2 year", well, not sure how many people pull from the dev branch :) ... the "tough" part of HA is how big it is, which is great, but then things like this are going to happen if backward compatibility don't happen.
Also, perhaps depreciation warning should happen in the UI, as I never saw the warning for TP-Link platform being removed and going to component, or i would have tested before an update, or seeing the "breaking change" in the change log.  way too many things flash by on HA startup logs.

@Enzo-Matrix thankfully for me when I moved without my patch... most of my TP Link devices have names that matched the HA name, so entity_id came back for all but a few.  Before I updated I copied the entity_id.  was a pain to have to force them back via the UI, but now the are in the registry and should, in theory, never have this issue again :)

Aww, just noticed the core.device_registry and core.entity_registry in the .storage directory. Guess I will edit these files to get the names back.

Any of you guys know how to get the TP-Link HS300 (powerstrip) available on homeassistant on the front end?  I'm a noob to HA, and appreciate your work.  It would complete the TP link library.

I hope everyone realizes that the tplink configuration can now be completely removed from configuration.yaml and the integration can then be autoconfigured by pressing a button in the UI. From there, name and entity_id can be changed if you want.
This is a huge improvement is usability:

static IP addresses are no longer needed
you don't have to read the documentation
your entity_id won't suddenly change and break your automations
entities can be placed in Areas

To get these benefits (and more), yes – if you previously renamed your entities in YAML, you must now do that in the UI. This is a one-time operation that takes no more than 20 seconds per entity.
I understand that this is an inconvenience right now but saying that this change is a step backwards is, frankly, a lack of respect for the huge work @rytilahti has put in over the course of six months, to make things better for everyone in the long term.

I'm closing this issue since it's not a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum. Thanks!


New version: GPUArrays v0.7.1

Registering package: GPUArrays
Repository: https://github.com/JuliaGPU/GPUArrays.jl
Created by: @maleadt
Version: v0.7.1
Commit: 83a5809e292e8e01d574508cbb20329e31dbe6a9
Reviewed by: @maleadt
Reference: https://github.com/JuliaGPU/GPUArrays.jl/commit/83a5809e292e8e01d574508cbb20329e31dbe6a9#commitcomment-33746592
Release notes:

make isapprox consistent with Base implementation
support for generating random complex numbers

Seems like TagBot is unable to push the tag it created to this repo, @maleadt.

A retry for another repo worked, maybe this will: TagBot tag

Hmm, maybe just GitHub flakiness.

error: RPC failed; HTTP 401 curl 22 The requested URL returned error: 401
fatal: the remote end hung up unexpectedly
fatal: the remote end hung up unexpectedly


Move test PRs to a different branch or tag

It's too noisy in the PR list right now.

All existing test PRs are labeled under example.


build: refactor test run script / targets

#659  👈
main

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

Join @davidlj95 and the rest of your teammates on  Graphite


as_root(): accept all KISS_SU values by default

This change ensures that niche tools that are not packaged in repositories
can be used without the need of it being implemented inside the package
manager itself. We don't support these tools, however most of them follow
a similar syntax as with 'sudo', and they can be run.
See: https://github.com/kiss-community/kiss/issues/12#issuecomment-799055508

Yeah this is what I was thinking (you don't have to implement every change yourself!)
This might require some slight documentation tweaking before it lands, and of course we'll have to modify this if some obscure sudo variant decides to be special.

No worries, I am just trying to help out as much as I can! I think this can be merged for now and the wiki documentation could be updated before a new release is made.


Update get clusters endpoint

Linked issue
Resolves: #xxxxx
Add 'currentPage' to getClustersPaginated endpoint
What kind of change does this PR introduce?

[ X] Bug fix
[ ] New feature
[ ] Refactor
[ ] Docs update
[ ] CI update

What is the current behavior?
Describe the state of the application before this PR. Illustrations appreciated (videos, gifs, screenshots).
currentPage field should be available for FE to know on which page it exists
What is the new behavior?
Describe the state of the application after this PR. Illustrations appreciated (videos, gifs, screenshots).
currentPage field is now added to KwClustersModelResponse
Other information
Additional changes, explanations of the approach taken, unresolved issues, necessary follow ups, etc.
Requirements (all must be checked before review)

[ ] The pull request title follows our guidelines
[ ] Tests for the changes have been added (if relevant)
[ ] The latest changes from the main branch have been pulled
[ ] pnpm lint has been run successfully

@programmiri we could fix ts issue ?


Unabale to change Settings

Please verify that this bug has NOT been raised before.

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

Describe the bug*
Its not possible to change any Settings with new UI.

For example, when I click on "my Compamy Name" to change this, only a black page appears. (or white, depending on the theme color).
2 errors appear in the Dev Console:

Steps to Reproduce

Use new Platform UI
go to Settings
try to change a Value

Expected behaviour
I expected the modal Popup to change the settings.
Deployment Method

[X] Docker
[ ] Package
[ ] Bare metal
[ ] Other - added info in Steps to Reproduce

Version Information
InvenTree Version
0.15.4
Commit Hash
fa1a9da
Commit Date
2024-06-16
Commit Branch
API Version
196
Python Version
3.11.9
Django Version
4.2.12
Please verify if you can reproduce this bug on the demo site.

[ ] I can reproduce this bug on the demo site.

Relevant log output
No response

@Tyroonis what does the actual error message say? You can click through to the "console" window in the developer tab to see the full message

@Tyroonis what does the actual error message say? You can click through to the "console" window in the developer tab to see the full message

Thats the full Message!
When I click on the message, it is only highlighted. Nothing more.

@Tyroonis can you try deleting all cookies/data for your instance in the browser and trying to reproduce again? This looks like it could be a caching problem.

@Tyroonis closing due to inactivity. Please ping us if you want to re-open


Return nice-looking HTML on home page

Right now it's really ugly.

Resolved in above-referenced commit.


Adding -ManagementServicesPort parameter causes container to become Unhealthy

When using 2 or more containers on one host and use them remotely we have to map the internal ports.
I can map client port 7046 to an external port 8000, but this still doesn't work when I try to connect an RTC client to a remote container. So thanksfully you added parameter -ClientServicesPort (and -ManagementServicesPort) to New-NavContainer.
Now, when using -ClientServicesPort 8000 and map this port with -AdditionalParameters "-p 8000:8000", I'm able to connect a remote RTC client perfecty. But for the Development environment to work, to be able to save a table and to use the debugger, I have to do the same for the -ManagementServicesPort. So when adding -ManagementServicesPort 8001 and -AdditionalParameters "-p 8001:8001", I'm perfectly able to debug and save a table.
But here's my issue. Everythings seems to work just fine, but when adding the -ManagementServicesPort and -AdditionalParameters "-p 8001:8001" the created container goes to status = Unhealthy.
Can you tell my if this is an issue with the -ManagementServicesPort parameter and the portmapping, because this will raise a red flag with my administrator about the container status. I've added the inspect-file, to show you the container in question.
inspect.txt

Same issue overhere. If you enter a powershell command and try to set the ManagementServicesPort you also get an error. It tries to add a firewall rule for the new port and that fails. Looks like it cannot find the firewall service. I don't know if this could be the problem for the container to go unhealthy. Just adding this info hoping to get this issue resolved quickly.

Any idea when this issue will be investigated?
I am waiting for a solution or work-around.
More details about the configuration I am using:

VM DockerServer with Windows Server 2016 (4 CPU, 16 GB RAM) on HyperV host with Windows Server 2012 R2 (HP DL360 Gen9, 2 CPU, 384GB RAM)
All Windows security updates installed on HyperV host and DockerServer

The inspect file of the container used: acket-test.inspect.txt
Need more information? Let me know!

Duplicate of this: https://github.com/Microsoft/nav-docker/issues/262
Which is fixed in the next version


Support CAST from Decimal datatype to String

Which issue does this PR close?

Closes #3991.
Rationale for this change
Support cast Decimal datatype to string

What changes are included in this PR?

Are there any user-facing changes?
No

@tustvold not sure why the test fails on
Test casting Decimal128(38, 0) --> LargeList(Field { name: "item", data_type: Utf8, nullable: false, dict_id: 0, dict_is_ordered: false, metadata: {} })
thread 'test_can_cast_types' panicked at 'called `Result::unwrap()` on an `Err` value: InvalidArgumentError("non-nullable child of type Utf8 contains nulls not present in parent LargeList(Field { name: \"item\", data_type: Utf8, nullable: false, dict_id: 0, dict_is_ordered: false, metadata: {} })")', arrow-data/src/data/mod.rs:1795:30

Perhaps related to #3983

My guess is the cast is failing and inserting a null, but the field doesn't permit nulls. I'm not sure what the cast kernel should do in such a case tbh, but in the immediate term I'd suggest finding a way to avoid the decimal cast from failing

My guess is the cast is failing and inserting a null, but the field doesn't permit nulls. I'm not sure what the cast kernel should do in such a case tbh, but in the immediate term I'd suggest finding a way to avoid the decimal cast from failing

@tustvold I have removed None from decimal array as other tests casts with not nullable values


remove : in the doc header

https://zebrunner.github.io/carina/migration/#project-side-dependency-requirements has a colon symbol et the end:
Project side dependency requirements:

let's remove it because all the rest doesn't have it.

@okamara wait until PR https://github.com/zebrunner/carina/pull/2182 is merged

verified


Improve how pytype fixer works

Improve how module paths are handled as they weren't working before. Use the output of pytype as an image for merge-pyi so we don't have to modify the repository source in the volume. Run all merge operations in one docker command instead of N.

Codecov Report

Merging #265 into master will decrease coverage by 0.06%.
The diff coverage is 89.47%.

@@            Coverage Diff             @@
##           master     #265      +/-   ##
==========================================
- Coverage   91.09%   91.03%   -0.07%
==========================================
  Files          46       46
  Lines        2719     2732      +13
==========================================
+ Hits         2477     2487      +10
- Misses        242      245       +3

Impacted Files
Coverage Δ

lintreview/tools/pytype.py
94.8% <89.47%> (-2.07%)
:arrow_down:

lintreview/docker.py
97.32% <0%> (-0.9%)
:arrow_down:

Continue to review full report at Codecov.

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


classList.add() is not working with similar strings

If the element has e.g: 'btn-active' class you can't add 'btn' because the indexOf already returns that it found the string 'btn' as part of 'btn-active'.

@remy , may I submit a pull request to fix it?


[FLINK-18335] [tests] Add more debuging logs for NotifyCheckpointAbortedITCase

What is the purpose of the change
Someone reported that NotifyCheckpointAbortedITCase is not stable in FLINK-18335. However, I cannot reproduce this problem locally and on my private CI.
Thus, as discussed in the ticket, we plan to increase the timeout and add more debuging logs.
Brief change log

Add more debuging logs for NotifyCheckpointAbortedITCase

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
This change added tests and can be verified as follows:
Does this pull request potentially affect one of the following parts:

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

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

Thanks a lot. Merging!


The lock is NOT released as soon the tasks finishes

You can also set lockAtMostFor attribute which specifies how long the lock should be kept in case the executing node dies. This is just a fallback, under normal circumstances the lock is released as soon the tasks finishes. You have to set lockAtMostFor to a value which is much longer than normal execution time. If the task takes longer than lockAtMostFor the resulting behavior may be unpredictable (more than one process will effectively hold the lock).

This is a reference to the document. For the following sentence, if I set lockAtLeastFor attribute I found that the lock is NOT released as soon the tasks finishes. But if I don't set this attribute, I can't makes sure the scheduled tasks are executed at most once at the same time.

under normal circumstances the lock is released as soon the tasks finishes

Can you please describe what you want from me. Do you. Want to make the documentation more clear?

I mean the document and the code is not match, as mentioned from the question title.
The lock is NOT released as soon the tasks finishes

The whole purpose of the lockAtLeastFor is to keep the lock longer than for the task duration

Ok


Added South Indian Education Society (sies)

sies is a name of the college which has many extentions and sies gst (South Indian Education Society Graduate School Of Technology) provides IT courses the domain of students end with ".gst.sies.edu.in" this is the official site if our college http://www.siesgst.edu.in/

@Aman-Pinjara Thank you for the request. To represent gst.sies.edu.in domain, I edited a file path a bit, to have just this domain listed, not its parent domain. Pull request is merged.


Bump dependencies

Bumps

golang.org/x/net from 0.12.0 to 0.17.0
google.golang.org/grpc from 1.56.2 to 1.56.3

Resolves https://github.com/flownative/localbeach/security/dependabot/8

Builds fine, runs fine.


Unnecessary lines between methods

I don't want to the lines between methods. How to make it just like what VSCode shows below? It will be more clear.

my configuration
vim.opt.list = true
vim.opt.listchars:append "space:⋅"
require("indent_blankline").setup {
    enabled = true,
    space_char_blankline = " ",
    show_current_context = true,
    show_current_context_start = true,
    use_treesitter = true,
    use_treesitter_scope = true,
}

You can turn this off with show_trailing_blankline_indent = false


[Enhancement]: Print build log to stdout for remote-build

What changes would you like?
Currently, the remote-build log is lost, since it's captured to a file, and then deleted from Launchpad. Having our action print the contents of the output file to stdout would make debugging build failures easier.
Any extra information?
No response

I did a pretty workaround for this here... it's simple but we can use it
https://github.com/Sjoerd1993/Graphs/blob/main/.github/workflows/main.yml#L82

I think that should work! What do you think, @jnsgruk?

Closed in #10


GPR curation for Retinol metabolism/Riboflavin metabolism

Current behavior:

18 genes are discovered to be uncorrelated with their corresponding reactions in Retinol metabolism/Riboflavin metabolism

Gene
UniProtID
Reaction
GPR
Correlation
Manually check

ENSG00000147576
Q8IWW8
MAR06631: NAD+ + retinol --> H+ + NADH + retinal
ENSG00000025423 or ENSG00000135437 or ENSG00000147576 or ENSG00000170786 or ENSG00000172955 or ENSG00000187758 or ENSG00000196344 or ENSG00000196616 or ENSG00000197894 or ENSG00000198099 or ENSG00000248144
Pending No, the gene is not relevant to the reaction provided. Analysis and explanation: The gene ENSG00000147576 encodes a protein that catalyzes the oxidation of gamma-hydroxybutyrate to succinic semialdehyde and the reduction of 2-ketoglutarate to D-2-hydroxyglutarate. It also acts on D,L-3-hydroxyisobutyrate and L-3-hydroxybutyrate with lower activity. This gene's catalytic activities do not involve the conversion of NAD+ and retinol to retinal, NADH, and H+, which is the provided reaction. Therefore, the gene ENSG00000147576 and the provided reaction are not related.
No

ENSG00000204195
Q58HT5
MAR06635: PC-LD pool + retinol --> 1-lysolecithin pool + retinyl-ester
ENSG00000062282 or ENSG00000114113 or ENSG00000114115 or ENSG00000121207 or ENSG00000147160 or ENSG00000185000 or ENSG00000204195
Pending No, the gene is not relevant to the reaction provided. Analysis and explanation: The gene ENSG00000204195 encodes an acyltransferase that specifically catalyzes the formation of ester bonds between fatty alcohols and fatty acyl-CoAs to form wax monoesters. It shows strong preferences for particular substrates, such as decyl alcohol and saturated acyl-CoAs, as detailed in various catalytic activities involving different fatty alcohols and fatty acyl-CoAs (PubMed:15671038). The provided reaction, PC-LD pool + retinol --> 1-lysolecithin pool + retinyl-ester, involves the esterification of retinol which does not correspond to the described activities of the acyltransferase encoded by the gene. Thus, the gene's function does not align with the reaction provided.
No

ENSG00000170786
Q8N3Y7
MAR06638: 11-cis-retinal + H+ + NADH <=> 11-cis-retinol + NAD+
ENSG00000025423 or ENSG00000135437 or ENSG00000139547 or ENSG00000140522 or ENSG00000157326 or ENSG00000170786 or ENSG00000187630 or ENSG00000265203
Pending No, the gene is not relevant to the reaction provided. Analysis and explanation. The gene ENSG00000170786 encodes an enzyme with oxidoreductase activity that specifically functions on all-trans-retinol to convert it to all-trans-retinaldehyde with NAD(+) as a cofactor, as supported by multiple references (PubMed:18926804). However, the reaction provided involves the conversion of 11-cis-retinal to 11-cis-retinol with NADH as a cofactor, which is not a recognized activity of the enzyme encoded by ENSG00000170786. Additionally, the enzyme has no activity with 11-cis-retinol or 11-cis-retinaldehyde as substrates, further confirming the irrelevance.
No

ENSG00000163684
O95059
MAR06640: retinol --> anhydroretinol + H2O
ENSG00000163684 or ENSG00000244486
Pending No, the gene is not relevant to the reaction provided. Analysis and explanation: The gene ENSG00000163684 encodes a component of ribonuclease P, which is a ribonucleoprotein complex involved in generating mature tRNA molecules by cleaving their 5'-ends. This function is unrelated to the mentioned reaction, which involves the conversion of retinol to anhydroretinol and the production of water. Therefore, the gene and the reaction are not related.
No

ENSG00000155016
Q7Z449
MAR06657: H+ + NADPH + O2 + retinoate --> 18-hydroxy-all-trans-retinoate + H2O + NADP+
ENSG00000021461 or ENSG00000100197 or ENSG00000106258 or ENSG00000108242 or ENSG00000130649 or ENSG00000134716 or ENSG00000137869 or ENSG00000138061 or ENSG00000138109 or ENSG00000138115 or ENSG00000140465 or ENSG00000140505 or ENSG00000142973 or ENSG00000155016 or ENSG00000160868 or ENSG00000160870 or ENSG00000165841 or ENSG00000167600 or ENSG00000171903 or ENSG00000186160 or ENSG00000186204 or ENSG00000186377 or ENSG00000186526 or ENSG00000197408 or ENSG00000197446 or ENSG00000197838 or ENSG00000198077 or ENSG00000255974
Pending No, the gene is not relevant to the reaction provided. Analysis and explanation: The gene ENSG00000155016 encodes a cytochrome P450 monooxygenase, which is involved in the metabolism of arachidonic acid and its conjugates. Its known activities include hydroxylation of long-chain fatty acids and oxidation of the indole ring of N-arachidonoyl-serotonin. The provided reaction involves the conversion of retinoate to 18-hydroxy-all-trans-retinoate, a reaction that falls outside the specified functional parameters of this gene's enzymatic activities. Therefore, based on the gene's documented functions, it is not related to the given reaction.
No

ENSG00000137869
P11511
MAR06657: H+ + NADPH + O2 + retinoate --> 18-hydroxy-all-trans-retinoate + H2O + NADP+
ENSG00000021461 or ENSG00000100197 or ENSG00000106258 or ENSG00000108242 or ENSG00000130649 or ENSG00000134716 or ENSG00000137869 or ENSG00000138061 or ENSG00000138109 or ENSG00000138115 or ENSG00000140465 or ENSG00000140505 or ENSG00000142973 or ENSG00000155016 or ENSG00000160868 or ENSG00000160870 or ENSG00000165841 or ENSG00000167600 or ENSG00000171903 or ENSG00000186160 or ENSG00000186204 or ENSG00000186377 or ENSG00000186526 or ENSG00000197408 or ENSG00000197446 or ENSG00000197838 or ENSG00000198077 or ENSG00000255974
Pending No, the gene is not relevant to the reaction provided. Analysis and explanation: The gene ENSG00000137869 encodes for a cytochrome P450 monooxygenase that catalyzes the conversion of C19 androgens to C18 estrogens, and other related reactions involving estrogen and androgen metabolism. However, the provided reaction involves the conversion of retinoate to 18-hydroxy-all-trans-retinoate, which does not fall under the scope of reactions catalyzed by this gene as per the provided annotations. Thus, there is no direct relevance between the gene and the specified reaction.
No

ENSG00000134184
P09488
MAR06661: 13-cis-retinoate <=> retinoate
ENSG00000134184
Pending No, the gene is not relevant to the reaction provided. Analysis and explanation. The gene ENSG00000134184 is involved in catalyzing the conjugation of reduced glutathione to exogenous and endogenous hydrophobic electrophiles, including reactions involving prostaglandins A2 and J2, as well as other hepoxilin-related compounds. The reaction provided, which is the isomerization of 13-cis-retinoate to retinoate, does not involve glutathione conjugation or the substrates handled by the gene's encoded enzyme. Thus, the gene's function and catalytic activities do not align with the described reaction.
No

ENSG00000121039
Q8IZV5
MAR06670: 4-oxo-9-cis-retinoate + H+ + NADPH <=> 9-cis-4-oxo-13,14-dihydro-retinoate + NADP+
ENSG00000087076 or ENSG00000099377 or ENSG00000100867 or ENSG00000109854 or ENSG00000121039 or ENSG00000138400 or ENSG00000139988 or ENSG00000145439 or ENSG00000159692 or ENSG00000160439 or ENSG00000164039 or ENSG00000167733 or ENSG00000170426 or ENSG00000170786 or ENSG00000176387 or ENSG00000183921 or ENSG00000184860 or ENSG00000186153 or ENSG00000187134 or ENSG00000197894 or ENSG00000198074 or ENSG00000198189 or ENSG00000198610 or ENSG00000204228 or ENSG00000227471 or ENSG00000240857
Pending No, the gene is not relevant to the reaction provided. Analysis and explanation: The gene ENSG00000121039 encodes for a retinol dehydrogenase that specifically converts all-trans-retinol to all-trans-retinal using NADP+ and has no activity towards other forms of retinol. The provided reaction involves the conversion of 4-oxo-9-cis-retinoate to 9-cis-4-oxo-13,14-dihydro-retinoate, which does not involve all-trans-retinol or all-trans-retinal and also specifies different substrates and reaction products. Therefore, the protein encoded by ENSG00000121039 is not relevant to this reaction.
No

ENSG00000087076
Q9BPX1
MAR06670: 4-oxo-9-cis-retinoate + H+ + NADPH <=> 9-cis-4-oxo-13,14-dihydro-retinoate + NADP+
ENSG00000087076 or ENSG00000099377 or ENSG00000100867 or ENSG00000109854 or ENSG00000121039 or ENSG00000138400 or ENSG00000139988 or ENSG00000145439 or ENSG00000159692 or ENSG00000160439 or ENSG00000164039 or ENSG00000167733 or ENSG00000170426 or ENSG00000170786 or ENSG00000176387 or ENSG00000183921 or ENSG00000184860 or ENSG00000186153 or ENSG00000187134 or ENSG00000197894 or ENSG00000198074 or ENSG00000198189 or ENSG00000198610 or ENSG00000204228 or ENSG00000227471 or ENSG00000240857
Pending No, the gene is not relevant to the reaction provided. Analysis and explanation: The gene ENSG00000087076 encodes for an enzyme with NAD-dependent 17-beta-hydroxysteroid dehydrogenase activity, primarily converting oestradiol to oestrone and acting on 5-androstene-3-beta,17-beta-diol. The reactions catalyzed by this enzyme involve steroid hormones and NAD(NADH) as cofactors. However, the reaction provided involves 4-oxo-9-cis-retinoate and NADPH, which is not a substrate or cofactor associated with the enzyme's activity as described. Therefore, based on the gene's function and catalytic activity annotations, it is not relevant to the reaction involving 4-oxo-9-cis-retinoate and NADPH.
No

ENSG00000099377
Q9H2F3
MAR06670: 4-oxo-9-cis-retinoate + H+ + NADPH <=> 9-cis-4-oxo-13,14-dihydro-retinoate + NADP+
ENSG00000087076 or ENSG00000099377 or ENSG00000100867 or ENSG00000109854 or ENSG00000121039 or ENSG00000138400 or ENSG00000139988 or ENSG00000145439 or ENSG00000159692 or ENSG00000160439 or ENSG00000164039 or ENSG00000167733 or ENSG00000170426 or ENSG00000170786 or ENSG00000176387 or ENSG00000183921 or ENSG00000184860 or ENSG00000186153 or ENSG00000187134 or ENSG00000197894 or ENSG00000198074 or ENSG00000198189 or ENSG00000198610 or ENSG00000204228 or ENSG00000227471 or ENSG00000240857
Pending No, the gene is not relevant to the reaction provided. Analysis and explanation: The gene ENSG00000099377 encodes an enzyme that plays a crucial role in the biosynthesis of hormonal steroids, specifically involved in bile acid synthesis and the degradation of 7-alpha,25-dihydroxycholesterol. The enzyme catalyzes reactions involving 7-alpha-hydroxycholesterol and related sterols. However, the provided reaction involves 4-oxo-9-cis-retinoate and NADPH, which are not substrates or products in any of the catalytic activities listed for this gene. Therefore, the gene is not related to the provided reaction.
No

ENSG00000147576
Q8IWW8
MAR06633: NAD+ + retinol --> H+ + NADH + retinal
ENSG00000025423 or ENSG00000135437 or ENSG00000147576 or ENSG00000170786 or ENSG00000172955 or ENSG00000187758 or ENSG00000196344 or ENSG00000196616 or ENSG00000197894 or ENSG00000198099 or ENSG00000248144
Pending No, the gene is not relevant to the reaction provided. Analysis and explanation: The gene ENSG00000147576 encodes a protein that catalyzes the oxidation of gamma-hydroxybutyrate to succinic semialdehyde and the reduction of 2-ketoglutarate to D-2-hydroxyglutarate. It also acts on D,L-3-hydroxyisobutyrate and L-3-hydroxybutyrate. However, the provided reaction involves the oxidation of retinol to retinal using NAD+ as a cofactor, which is not among the reactions catalyzed by the gene's protein product. Thus, the gene and the reaction are not related.
No

ENSG00000100867
Q13268
MAR06670: 4-oxo-9-cis-retinoate + H+ + NADPH <=> 9-cis-4-oxo-13,14-dihydro-retinoate + NADP+
ENSG00000087076 or ENSG00000099377 or ENSG00000100867 or ENSG00000109854 or ENSG00000121039 or ENSG00000138400 or ENSG00000139988 or ENSG00000145439 or ENSG00000159692 or ENSG00000160439 or ENSG00000164039 or ENSG00000167733 or ENSG00000170426 or ENSG00000170786 or ENSG00000176387 or ENSG00000183921 or ENSG00000184860 or ENSG00000186153 or ENSG00000187134 or ENSG00000197894 or ENSG00000198074 or ENSG00000198189 or ENSG00000198610 or ENSG00000204228 or ENSG00000227471 or ENSG00000240857
Pending No, the gene is not relevant to the reaction provided. Analysis and explanation. The gene ENSG00000100867 encodes an NADPH-dependent oxidoreductase that catalyzes the reduction of specific dicarbonyl compounds and displays minor hydroxysteroid dehydrogenase activity toward bile acids. However, it does not show any activity with retinoids as substrates, which makes it unrelated to the reaction of converting 4-oxo-9-cis-retinoate to 9-cis-4-oxo-13,14-dihydro-retinoate. Therefore, the gene¡¯s function does not align with the reaction specified.
No

ENSG00000170786
Q8N3Y7
MAR08709: 11-cis-retinal + H+ + NADPH <=> 11-cis-retinol + NADP+
ENSG00000025423 or ENSG00000135437 or ENSG00000139547 or ENSG00000140522 or ENSG00000157326 or ENSG00000170786 or ENSG00000187630 or ENSG00000265203
Pending No, the gene is not relevant to the reaction provided. Analysis and explanation: The gene ENSG00000170786 encodes for an oxidoreductase that is active in both oxidative and reductive directions with a strong preference for NAD. It catalyzes the oxidation of all-trans-retinol to all-trans-retinaldehyde with NAD(+) as a cofactor. However, this enzyme shows no activity with 11-cis-retinol or 11-cis-retinaldehyde as substrates, irrespective of whether NAD(+)/NADH or NADP(+)/NADPH are used. The provided reaction involves 11-cis-retinal and NADPH, which are substrates for which the enzyme encoded by this gene exhibits no activity, thus making the gene irrelevant to the provided reaction.
No

ENSG00000186377
Q8N118
MAR06658: H+ + NADPH + O2 + retinoate --> 18-hydroxy-all-trans-retinoate + H2O + NADP+
ENSG00000021461 or ENSG00000100197 or ENSG00000106258 or ENSG00000108242 or ENSG00000130649 or ENSG00000134716 or ENSG00000137869 or ENSG00000138061 or ENSG00000138109 or ENSG00000138115 or ENSG00000140465 or ENSG00000140505 or ENSG00000142973 or ENSG00000155016 or ENSG00000160868 or ENSG00000160870 or ENSG00000165841 or ENSG00000167600 or ENSG00000171903 or ENSG00000186160 or ENSG00000186204 or ENSG00000186377 or ENSG00000186526 or ENSG00000197408 or ENSG00000197446 or ENSG00000197838 or ENSG00000198077 or ENSG00000255974
Pending No, the gene is not relevant to the reaction provided. Analysis and explanation: The gene ENSG00000186377 encodes a cytochrome P450 monooxygenase that catalyzes the epoxidation of the last double bond of the arachidonoyl moiety of anandamide, modulating endocannabinoid signaling. The associated reaction involves N-(5Z,8Z,11Z,14Z-eicosatetraenoyl)-ethanolamine, O2, and NADPH--hemoprotein reductase. However, the provided reaction involves retinoate, NADPH, and O2 to produce 18-hydroxy-all-trans-retinoate, which does not correlate with the gene's described function or established catalytic activity.
No

ENSG00000155893
Q8TE99
MAR06507: FMN + H2O --> Pi + riboflavin
ENSG00000014257 or ENSG00000102575 or ENSG00000142513 or ENSG00000155893 or ENSG00000162836 or ENSG00000183760
Pending No, the gene is not relevant to the reaction provided. Analysis and explanation. The gene ENSG00000155893 encodes a protein responsible for the 2-O-dephosphorylation of xylose in the glycosaminoglycan-protein linkage region of proteoglycans. It specifically regulates the maturation of glycosaminoglycan (GAG) chains by catalyzing the removal of phosphate from xylose. In contrast, the reaction provided, FMN + H2O --> Pi + riboflavin, involves the hydrolysis of FMN to yield phosphate (Pi) and riboflavin. This reaction does not pertain to the activities associated with GAG-protein linkage or xylose dephosphorylation. Hence, the gene is not relevant to the given reaction.
No

ENSG00000162836
Q9NPH0
MAR06507: FMN + H2O --> Pi + riboflavin
ENSG00000014257 or ENSG00000102575 or ENSG00000142513 or ENSG00000155893 or ENSG00000162836 or ENSG00000183760
Pending No, the gene is not relevant to the reaction provided. Analysis and explanation: The gene ENSG00000162836 encodes an enzyme that hydrolyzes lysophosphatidic acid containing a medium length fatty acid chain to the corresponding monoacylglycerol. The reactions catalyzed by this enzyme involve the conversion of lysophosphatidic acid to monoacylglycerol and phosphate. However, the provided reaction describes a different process: the conversion of FMN and water to phosphate and riboflavin. This reaction does not involve lysophosphatidic acid or monoacylglycerol and therefore does not relate to the function or catalytic activity of the gene ENSG00000162836.
No

ENSG00000102575
P13686
MAR06507: FMN + H2O --> Pi + riboflavin
ENSG00000014257 or ENSG00000102575 or ENSG00000142513 or ENSG00000155893 or ENSG00000162836 or ENSG00000183760
Pending No, the gene is not relevant to the reaction provided. Analysis and explanation: The gene ENSG00000102575 is involved in osteopontin/bone sialoprotein dephosphorylation and catalyzes reactions involving a phosphate monoester and water, yielding an alcohol and phosphate. However, the provided reaction involves the conversion of FMN to riboflavin and phosphate, which does not align with the specific dephosphorylation function of the gene.
No

ENSG00000112874
Q9BQG2
MAR20117: FAD + H2O --> AMP + FMN + 2.0 H+
ENSG00000112874
Pending No, the gene is not relevant to the reaction provided. Analysis and explanation. The provided gene, ENSG00000112874, encodes an mRNA decapping enzyme. The functional annotation indicates that this enzyme specifically removes NAD caps from mRNAs and hydrolyzes NAD(H) and NADPH molecules to various products. However, the reaction given is "FAD + H2O --> AMP + FMN + 2.0 H+", which involves the hydrolysis of FAD to FMN and AMP. The gene described does not have any known activity involving FAD hydrolysis, and thus is not relevant to the provided reaction.
No

LGTM! Please go ahead and make changes. This issue mainly involves wrongly associations with reactions with enzymes. Could be a good example for showcasing this such as ENSG00000155893 for MAR06507

Fixed by #914


381 Allow multiple contact emails for a sponsor

Must be at least one
Fixes: #381

:email: :email:


Error handlign delegate

Since we still have crashing bugs, we need to hook all delegate errors
and see if they are crashing the process

No delegates anymnore!


sat consistency checks

closed in favor of https://github.com/OpenTOSCA/opentosca-vintner/pull/244


chore: cleanup libzkp interfaces used by coordinator and update da-codec dependency

Purpose or design rationale of this PR
PR title
Your PR title must follow conventional commits (as we are doing squash merge for each PR), so it must start with one of the following types:

[x] refactor: A code change that doesn't fix a bug, or add a feature, or improves performance

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 52.56%. Comparing base (d2ff088) to head (e0903ef).

Additional details and impacted files
@@                Coverage Diff                @@
##           feat/upgrade4    #1469      +/-   ##
=================================================
- Coverage          52.65%   52.56%   -0.09%
=================================================
  Files                157      156       -1
  Lines              12531    12444      -87
=================================================
- Hits                6598     6541      -57
+ Misses              5372     5342      -30
  Partials             561      561

Flag
Coverage Δ

bridge-history-api
71.16% <ø> (ø)

common
47.21% <ø> (-2.03%)
:arrow_down:

database
42.85% <ø> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


STM32: mcu goes to sleep inadvertently when using PM.

Describe the bug
When using the (experimental) STM32L4 power management, the deepsleep enable bit (sleep in Zephyr terms) is not set back to sleep (idle in zephyr terms) when waking up, resulting in the next WFI call also going to sleep.
To Reproduce
Steps to reproduce the behavior:

use stm32l* PM
Do a long wait to go to sleep.
Do a short wait that should not go do sleep but idle.
System goes to sleep anyway

Expected behavior
System only goes to sleep when executing sys_set_power_state with the appropriate power state.
Fix
In the soc specific power.c, add  LL_LPM_EnableSleep to _sys_pm_power_state_exit_post_ops. This sets the system configuration bit to use regular sleep (idle) instead of deepsleep when executing WFI.
I found this bug while porting powermanagement to the STM32L5, but I think the behavior should be the same on the other STM32 cores.
edit: fixed some potential confusion with idle/sleep/deepsleep terminology.

@BBekker  the soc residency policy does not go to sleep for a too "short wait"
do you mean that _sys_pm_power_state_exit_post_ops() has to LL_LPM_EnableDeepSleep (like the sys_set_power_state function does) ?

The bug is that the MCU goes to sleep, even when the residency policy does not decide it has to go to sleep.
sys_set_power_state uses LL_LPM_EnableDeepSleep to set the SLEEPDEEP bit in the cortex-m33 system control register.
https://developer.arm.com/documentation/100235/0003/the-cortex-m33-peripherals/system-control-block/system-control-register?lang=en
But since that bit is not reset after wakeup, it stays active.
_sys_pm_power_state_exit_post_ops should disable that bit again to prevent the system going to deep sleep at every WFI instruction.

This is the position of the SLEEPONEXIT bit of the System Control Register (SCR) of the cortex
Clearing this bit will avoid going to sleep when returning to Thread mode
This is the LL_LPM_DisableSleepOnExit() call during _sys_pm_power_state_exit_post_ops


I am using the model in our own dataset., data_shape=(384,384,3), ValueError: Negative dimension size caused by subtracting 32 from 24 for 'average_pooling2d_1/AvgPool' (op: 'AvgPool') with input shapes: [1,24,24,2048],can you tell me what it hapen ?

but when data_shape=(512,512,3) ,have no problem

What output stride and backbone do you use?

output stride =16 @bonlime

output stride=16 ,  backbone  is xecption @bonlime

@daxiaoshu
I tried to use:
model = Deeplabv3(weights=None, input_shape=(384, 384, 3), classes=15, backbone='xception', OS=16)
on images with shape (384,384,3) and it works fine for me. Maybe problem is in some other place
This error happens to you because int(np.ceil(input_shape[0] / OS)) in pool_size is bigger than 24, which don't happen in my setting

I also have this issue, no matter how much I change the input shape or other parameters :(

NOTE FOR OTHERS: It may be the case that this error happens; By restarting the kernel, I was able to solve it.

@CalinTimbusLucian I can't help you because I can't reproduce this error. Give me the steps to get it and I will try to fix

bonlime thanks for the prompt response. I restarted the kernel in Spyder and then it worked(don't ask me why, I have absolutely no idea) 👍


[Snyk] Upgrade nestjs-pino from 3.4.0 to 3.5.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade nestjs-pino from 3.4.0 to 3.5.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 23 days ago, on 2023-09-21.

Release notes

Package name: nestjs-pino

3.5.0 - 2023-09-21 Read more

3.4.0 - 2023-08-21 Read more

    from nestjs-pino GitHub release notes

Commit messages

Package name: nestjs-pino

b4e314f update version
6c48c9f add custom err attribute key support
6ca55ca build(deps-dev): bump @ types/node from 20.6.2 to 20.6.3
5f5b112 build(deps-dev): bump @ typescript-eslint/eslint-plugin
28f1ef0 build(deps-dev): bump @ typescript-eslint/parser from 6.7.0 to 6.7.2
412c34f build(deps-dev): bump @ types/jest from 29.5.4 to 29.5.5
b052514 build(deps-dev): bump @ types/node from 20.6.0 to 20.6.2
c4bfdce build(deps-dev): bump jest from 29.6.4 to 29.7.0
ecb45d9 build(deps-dev): bump @ nestjs/testing from 10.2.4 to 10.2.5
76b3c64 build(deps-dev): bump @ typescript-eslint/eslint-plugin
c1cf599 build(deps-dev): bump @ nestjs/core from 10.2.4 to 10.2.5
6646b47 build(deps-dev): bump @ nestjs/platform-express from 10.2.4 to 10.2.5
d2b5e36 build(deps-dev): bump @ nestjs/platform-fastify from 10.2.4 to 10.2.5
984b1b2 build(deps-dev): bump @ nestjs/common from 10.2.4 to 10.2.5
a69e0c4 build(deps-dev): bump @ typescript-eslint/parser from 6.6.0 to 6.7.0
0278d6f build(deps-dev): bump eslint from 8.48.0 to 8.49.0
4915f26 build(deps-dev): bump @ types/node from 20.5.9 to 20.6.0
f4e4956 build(deps-dev): bump pino from 8.15.0 to 8.15.1
d731a31 build(deps-dev): bump @ typescript-eslint/eslint-plugin
470c925 build(deps-dev): bump pino-http from 8.4.0 to 8.5.0
7e832bb build(deps-dev): bump @ typescript-eslint/parser from 6.5.0 to 6.6.0
c7b4a57 build(deps-dev): bump @ types/node from 20.5.7 to 20.5.9
e72f8b1 build(deps-dev): bump @ nestjs/common from 10.2.3 to 10.2.4
ae7f67e build(deps-dev): bump @ nestjs/testing from 10.2.3 to 10.2.4

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

@SocketSecurity ignore<EMAIL_ADDRESS>


Don't list rebased commits; Detect a force push

If commits of a branch are rebased on top of another branch, these commits get a new commit hash when the updated branch gets force pushed, which makes the bot assume that they are new commits (and list them).
As an example scenario, there is a topic branch based off master. While the topic branch is being worked on by a single developer, the developer finds bugs on the master branch that need to be fixed to make the topic branch work. However, putting these fixes in the topic branch would mean that their introduction into master can be delayed quite a bit. So the developer decides to fix the bugs on the master branch and pull them into the topic branch via git rebase master (so that the topic branch isn't filled with empty merge commits). This will rewrite the history of all the topic commits. When the topic branch is now pushed (the force switch is needed), the bot lists the topic commits as if they were entirely new, which can create a wrong impression and leads to spam when done frequently. Furthermore, no indication is given that the branch has been force-pushed.
The bot should warn whenever existing commits are removed (any force push) and in case a commit has been relabeled with a new commit hash (every rebase), detect that it's really the same commit under a new name.

No idea if that was true back when this issue was opened, but the GitHub API now sends {"forced": true} for force pushes. Keep an eye out for {"created": true} though, as creating a new branch seems to always count as a force-push.


Additional formats won't generate

Hello!
apiVersion: trust.cert-manager.io/v1alpha1
kind: Bundle
metadata:
  name: rootca
spec:
  sources:
  - useDefaultCAs: true
  - secret:
      name: "fullchain-rootca"
      key: "fullchainRootCA.crt"
  target:
    configMap:
      key: "bundle-certs.pem"
    additionalFormats:
      jks:
        key: "bundle-certs.jks"
      pkcs12:
        key: bundle-certs.p12
        password: '123'
    namespaceSelector: {}

I provided this bundle, but in resulting configmap there is no pkcs12 (I tried without password, with blank password ('') and with test 123 pass) there is only bundle-certs.pem content.
I've tried latest v0.10.0 version of application.
Kubernetes 1.29

Hi @atillaqb! Anything interesting in the trust-manager pod log? And how does the status of your Bundle resource look like?

I found issue. By default, additional formats works only when you are using secret as a target. I think it should be reflected in official documentation.

I found issue. By default, additional formats works only when you are using secret as a target. I think it should be reflected in official documentation.

How come? If that's the case, it's a bug. But I think this is unlikely. Are you using Octant to check your resources, ref. https://kubernetes.slack.com/archives/C4NV3DWUC/p1716453848338929?thread_ts=1716409603.775449&cid=C4NV3DWUC

I am also experiencing this. Can we reopen this as a bug, instead of duplicating it into another issue?

I am also experiencing this. Can we reopen this as a bug, instead of duplicating it into another issue?

I'll be happy to reopen if this is a true bug. Can you please provide more details on what"s not working  for you?