--x265 option seems to always use CPU encoding

With various 2K rips, when running a normal other-transcode, the encode will fail because the codec isn't supported unless I use the --x265 option.  When I do this, even if I try and specify the --nvenc it still uses cpu encoding.
Is this a known limitation with some video/formats?  I did try and search issues here before I asked :)

@octopush-it I'm sorry you're having a problem but I don't really understand what you mean. Your issue title doesn't really match your description of the issue itself.
I need more information. Please include the command line you're using and either a .log file or the relevant console spew so I can tell what's going on.
Thanks.

Sorry - thats what I get for trying to multitask.   Here is the issue as it relates to the title.  When I run into issues where I either :

Cannot use --mp4 because of multiple subtitle or audio streams
Cannot use the basic "other-transcode" because the file has been X265 encoded already

I have to provide the --x265 option to get the file to process successfully.  When I use the --x265 option, however, it never seems to use the GPU for transcoding, even if I use the --nvenc flag.  The file will process with the CPU at ~5-10fps.  For every other x264 encoded file regardless of container, other-transcode will reliably use the GPU for processing at ~300-600fps.
Here is the output of two recent log runs that failed, and a cap of the encoding output when I get it to process successful, though slowly.
other-transcode_--mp4-option_no-other-flags.log
other-transcode_--x265-option_lowFPS.log
other-transcode_no-options.log

@octopush-it Thanks for the .log files!
You can't use the --mp4 option if you include image-based subtitles because those aren't compatible with the MP4 format. However, that doesn't appear to be the problem with the .log files you've supplied.
The problem is that you're trying to transcode a 10-bit HEVC input to 8-bit H.264 output. That probably won't work and even if it did, the color mapping would likely be wrong.
And when you want HEVC output, pick a single HEVC encoder. Don't put both --x265 and then --nvenc in your options. The last one on the command line will be selected, that being NVenc in .log you included, but other things can go wrong that way.
I have no idea why Nvenc is failing for you. I've never seen that error before.
I see that you are using a development build of ffmpeg. Version N-104488-ga13646639f-20211109 is in the .log file. That's never a good idea. Try downloading a stable release. I recommend version 6.0. That should be the latest.
And if that doesn't work, I recommend you try my newer transcoding project. The four scripts it includes handle 10-bit inputs much better.
You can find it here:
https://github.com/donmelton/more-video-transcoding
It's based on HandBrakeCLI. If you decide to try it, use a stable release of HandBrake and not a nightly development build.
I hope that helps.

THanks!  Using the nvenc-hevc-transcode produces a different error each time its run, which is weird:
E:\transcode>ruby nvenc-hevc-transcode.rb "temp\x.mkv"
nvenc-hevc-transcode.rb: Search timed out SYNTAX_SUGGEST_TIMEOUT=1, run with DEBUG=1 for more infoC:/Ruby32-x64/lib/ruby/3.2.0/syntax_suggest/code_line.rb:119:in empty?' C:/Ruby32-x64/lib/ruby/3.2.0/syntax_suggest/code_line.rb:123:in not_empty?'
C:/Ruby32-x64/lib/ruby/3.2.0/syntax_suggest/code_block.rb:68:in `select'nvenc-hevc-transcode.rb:8: syntax error, unexpected '<' (SyntaxError)

^
E:\transcode>ruby nvenc-hevc-transcode.rb "temp\x.mkv"
nvenc-hevc-transcode.rb: Search timed out SYNTAX_SUGGEST_TIMEOUT=1, run with DEBUG=1 for more infoC:/Ruby32-x64/lib/ruby/3.2.0/syntax_suggest/code_block.rb:89:in join' C:/Ruby32-x64/lib/ruby/3.2.0/syntax_suggest/code_block.rb:89:in valid?'
C:/Ruby32-x64/lib/ruby/3.2.0/syntax_suggest/code_search.rb:94:in `push'nvenc-hevc-transcode.rb:8: syntax error, unexpected '<' (SyntaxError)

^
E:\transcode>ruby nvenc-hevc-transcode.rb "temp\x.mkv"
nvenc-hevc-transcode.rb: Search timed out SYNTAX_SUGGEST_TIMEOUT=1, run with DEBUG=1 for more infoC:/Ruby32-x64/lib/ruby/3.2.0/syntax_suggest/api.rb:118:in hash' C:/Ruby32-x64/lib/ruby/3.2.0/syntax_suggest/api.rb:118:in -'
C:/Ruby32-x64/lib/ruby/3.2.0/syntax_suggest/api.rb:118:in `valid_without?'nvenc-hevc-transcode.rb:8: syntax error, unexpected '<' (SyntaxError)

^

@octopush-it I'm glad you tried nvenc-hevc-transcode.rb! But I have no idea what's going on with those error messages. 🤷♀ It looks like something is wrong with your Ruby runtime?

@octopush-it How did you save the file. If you did something like wget https://github.com/donmelton/more-video-transcoding/blob/master/nvenc-hevc-transcode.rb then the GitHub web page for the script is saved, not the script itself.
If that happened, then the error message might make some sense. This is what the first ten lines would look like:
      1
      2
      3
      4
      5
      6
      7
      8 <!DOCTYPE html>
      9 <html lang="en" data-color-mode="auto" data-light-theme="light" data-dark-theme="dark" data-a11y-animated-images="system">
     10   <head>

The error says there's a syntax error at line 8, with an unexpected <. If the above is the actual content of the file, then that seems to line up remarkably well.
To fetch the script you'll need to get the RAW file. This can be done by clicking on the RAW button, and then using that link. For example, wget https://raw.githubusercontent.com/donmelton/more-video-transcoding/master/nvenc-hevc-transcode.rb will fetch the actual script.
Alternatively, you can clone the repo.
This may not be your problem, but the symptom seems to line up, so I thought it was worth a mention.

@ttyS0 Oh snap! That's gotta be the problem. 👍 Nice detective work, sir.

@octopush-it So, did you try to re-download and re-run the script using @ttyS0 's suggestion?

@ttyS0 uuuuunbelievable ... LOL - the simplest things.  Its working and I am dumb!  Thanks both!

@donmelton FYI using nvenc-hevc-transcode has resulted in a significant gain in transcoding over other-transcode with the x265 10bit file... from 10-20fps to 150fps .  Amazing!


SQM-17 Entity type expressions

This seems to be covering a lot of different things: collection functions, entity-type expressions.  Can you separate these into distinction branches?
The reason being that I think there is a better way to handle entity-type checks.  But I'd first like to isolate the changes here to the stuff specifically related to that change.  The change I am contemplating is a basic tenant of designing a parser that I learned over time... Basically, whenever possible it is best to interpret things based on a syntactical analysis versus a semantical analysis.

Hi, actually all of this related to the entity type expression feature. The
change around collections is just an extraction of the sub-elements of
jpaCollectionFunction into separate rules which I did to make the map key
and value funtions usable within entity expressions (which JPA allows
AFAICS). That specific change is insulated in the second commit, it may
help to look at that one separately.

So my idea is based on a few premises.
First, the JPA spec only allows this in the WHERE clause.  So these "entity type expressions" (the JPA term, section <IP_ADDRESS>) are only allowed as a predicate.  And specifically in conjunction with a limited set of predicate operators (=, <>, IN, NOT IN).
The above premises come directly from the JPA spec.  Another premise I have is that one of the expressions in the predicate has to be a TYPE function; all other forms really would not make sense.  Bearing in mind that an "entity type expression" can be either (a) a TYPE function expression, (b) a parameter or (c) a entity-type-literal and leaving operator out of the equation for now, we'd have the following combinations:

typeFunction & typeFunction
typeFunction & parameter
typeFunction & entityTypeLiteral
parameter & entityTypeLiteral

I argue that the last one makes no sense to allow.  We can debate this one.  But it is an important premise to the idea to recognize these expressions syntactically.  Why?  Because notice how, if we strip away the last combination, all the remaining combinations have at least one typeFunction expression.  That would allow us to set up a special branch in the predicate rule to recognize this.
Why is this such an important distinction (syntactic versus semantic)?  In a word, performance.  Specifically here: time, speed.  The approach you took (validating each token text as an entity name) is valid, and it works.  But you have to realize that every token/context we ask Antlr to evaluate as an expression will now involve a call to check whether the thing is an entity (so a series of method calls and eventually at least 2 Map lookups); for every possible thing that could evaluate as an expression.
What is better, generally speaking, is to recognize that an expression should be an entity-type-expression based on syntax.  The should be is important.  Because now we move into syntactic analysis and semantic validation.  So if during parsing we are able to recognize that this expression should be an entityTypeExpression, we can later validate that it is as we walk the parse tree.  That will be a much more efficient check, because its limited.
Hopefully that makes sense.

Any word on the requested changes?

Going to close this.  I have added a note to the 6.0 todo doc (in the ORM repo, on the 6.0 working branch) regarding testing/implementing the combination stuff discussed above.


openai.error.RateLimitError: Rate limit reached for default-global-with-image-limits in organization org-8Ch0kYQaMzstRX0szWKe1GLd on requests per min. Limit: 60 / min. Please try again in 1s. Contact<EMAIL_ADDRESS>if you continue to have issues. Please add a payment method to your account to increase your rate limit. Visit https://platform.openai.com/account/billing to add a payment method.

what should i do ? How does my program avoid exceptions that limit flow, for example, such as sleep 1s？
do you have a better suggestion？

see https://github.com/hwchase17/notion-qa/issues/7


[ERROR] failed to load locked problem! [0]

I am able to login to leetcode and work on locked problems, is there anyway to show locked problem in command line tool?
Thanks!

Duplicate of #91
Install from github mater will do the trick for now:
npm i -g skygragon/leetcode-cli

thanks @sfdye !


pathParam() in Foxx Swagger fails with Router if using number "joi.number().required()"

The documentation shows how to use string with URL parameters which works fine. It also works for me:
.pathParam('key', joi.string().required(), 'Key of the node.')
The router fails as soon as you use number instead of string.
.pathParam('key', joi.number().required(), 'Key of the node.')
My version of database is 3.7.11 and it runs in Docker.
Other functions also work fine!
T

Works with req.pathParams.key.toString().


[Mobile] Change color of CustomIcon icon

My issue is about [Mobile]
Hello,
It is not possible to change the customIcon icon since it is an IconData !
Do you think it is possible to change that for letting put an Icon instead of an IconData or add argument to set the icon colors ?
Thanks

Thank you for the report. I was thinking about that but it will require us to introduce a breaking change

This issue has been fixed in 8.4.0
if you have any questions feel free to ask at any time


Runtime error tensorflow.python.data.util.structure.type_spec_from_value

Click to expand!
Issue Type
Bug
Have you reproduced the bug with TF nightly?
No
Source
source
Tensorflow Version
2.10.0
Custom Code
Yes
OS Platform and Distribution
Ubuntu 22.04
Mobile device
No response
Python version
3.9
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
runtime error probably due to large tensor.

Standalone code to reproduce the issue
import tensorflow as tf
import numpy as np
from tensorflow.python.data.util import structure
try:
  try:
    with tf.device('/CPU'):
      arg_0_0_tensor = tf.saturate_cast(tf.constant(-104730106681921, shape=[], dtype=tf.int64,),dtype=tf.uint64)
      arg_0_0 = tf.identity(arg_0_0_tensor)
      arg_0_1 = -96
      arg_0 = [arg_0_0,arg_0_1,]
      out = structure.type_spec_from_value(arg_0,)
  except Exception as e:
    print("Error:"+str(e))
  try:
    with tf.device('/GPU:0'):
      arg_0_0 = tf.identity(arg_0_0_tensor)
      arg_0_0 = tf.cast(arg_0_0, tf.uint64)
      arg_0 = [arg_0_0,arg_0_1,]
      structure.type_spec_from_value(arg_0,)
  except Exception as e:
    print("Error:"+str(e))
except Exception as e:
  print("Error:"+str(e))

### Relevant log output

```shell
2023-01-21 11:58:01.038629: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2023-01-21 11:58:01.185340: E tensorflow/stream_executor/cuda/cuda_blas.cc:2981] Unable to register cuBLAS factory: Attempting to register factory for plugin cuBLAS when one has already been registered
2023-01-21 11:58:01.938964: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer.so.7'; dlerror: libnvinfer.so.7: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: :/lib/
2023-01-21 11:58:01.939239: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer_plugin.so.7'; dlerror: libnvinfer_plugin.so.7: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: :/lib/
2023-01-21 11:58:01.939251: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Cannot dlopen some TensorRT libraries. If you would like to use Nvidia GPU with TensorRT, please make sure the missing libraries mentioned above are installed properly.
2023-01-21 11:58:02.753091: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-21 11:58:02.794160: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libcudnn.so.8'; dlerror: libcudnn.so.8: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: :/lib/
2023-01-21 11:58:02.794182: W tensorflow/core/common_runtime/gpu/gpu_device.cc:1934] Cannot dlopen some GPU libraries. Please make sure the missing libraries mentioned above are installed properly if you would like to use GPU. Follow the guide at https://www.tensorflow.org/install/gpu for how to download and setup the required libraries for your platform.
Skipping registering GPU devices...
2023-01-21 11:58:02.794479: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
Error:<class 'tensorflow.python.framework.ops.EagerTensor'> returned a result with an error set
Error:<class 'tensorflow.python.framework.ops.EagerTensor'> returned a result with an error set

</details>

Hi, @SuryanarayanaY
Could you please look into this issue ? Thank you!

@nimashiri ,
The reported behaviour observed in TF 2.10v and TF 2.11V.However with tf-nightly there is clear Error message  as mentioned below for both CPU and GPU.
Error:Can't convert Python sequence with out-of-range integer to Tensor.
Please refer to attached gists of 2.10v , 2.11v and nightly versions.
As the issue resolved in tf-nightly and  not related security vulnerability it's unlikely to cherry pick this for the current versions.
Can we consider it as resolved? Thanks!

This is a vulnerability that got fixed. Since it was not reported on the proper channels, no credit is given.
@dmc1778 please stop posting vulns on GitHub. Please consult SECURITY.md for how to properly and ethically disclose them.


Blog App - processing data in models

Blog App - processing data in models
On this project I:

created model classes.
created associations between modules.

Thanks @danmainah the links and everything.

Hi @kidde60,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights

The Pull Request has a proper title and description. ✔
You have added a descriptive Readme file. Good Job. ✔
There is no linter error is present. ✔
Project has met all the technical aspects you needed for this project. Keep rocking! ✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Update 2

use hex() function to simplify original dec_to_hex.py

Thank you for you contribution to the repo and the community

On 3 Oct 2017, at 09:15, CuiXiaoDao<EMAIL_ADDRESS>wrote:
use hex() function to simplify original dec_to_hex.py
You can view, comment on, or merge this pull request online at:
https://github.com/geekcomputers/Python/pull/244 https://github.com/geekcomputers/Python/pull/244
Commit Summary
Merge pull request #1 from geekcomputers/master
Merge remote-tracking branch 'upstream/master'
Merge remote-tracking branch 'upstream/master'
simplify dec_to_hex.py
File Changes
M dec_to_hex.py https://github.com/geekcomputers/Python/pull/244/files#diff-0 (14)
Patch Links:
https://github.com/geekcomputers/Python/pull/244.patch https://github.com/geekcomputers/Python/pull/244.patch
https://github.com/geekcomputers/Python/pull/244.diff https://github.com/geekcomputers/Python/pull/244.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/geekcomputers/Python/pull/244, or mute the thread https://github.com/notifications/unsubscribe-auth/ABHa0lRQHYDlexj406ngKMx0EVfXiFv3ks5soe02gaJpZM4PrvgQ.


Send message when spellbot does not have permission to manage reactions

Description
Has spellbot message when it cannot adjust reactions because it lacks the proper permissions.

Definitely not finished, since I didn't implement a way for admins to silence this message
I chose to put the reactions code into their own file to make unit testing easier. If you'd prefer to consider these methods as private functions in the main test file, does that mean you want each place that calls them to be tested for this permissions message?
Can't figure out why the meta test for strings doesn't work for this. Probably something to do with it being in a separate file?
If you want to start pulling some of this code into smaller, more manageable files, is there a more programatic way to order the test files to ensure the Meta class gets invoked last, but not have to list every single test file?

(partially) Resolves #120

Checklist

[x] Add tests for these changes
[x] Test coverage is not decreased
[ ] Entire test suite passes

I think the meta test for strings is failing for 2 reasons:

It's only in the client fixture that we run monkeypatch.setattr(spellbot, "s", S_SPY) and none of the tests in tests/test_reactions.py use the client fixture.
Even if we add the client fixture to those tests it still won't work because we're actually monkey patching the s() function within the spellbot module. But that leaves the s() that's within the reactions module untouched.

So I was able to get the test to pass by adding the client fixture to the reactions tests and adding monkeypatch.setattr(spellbot.reactions, "s", S_SPY) to the client fixture. There's probably a nice way to refactor this code that it works and is more elegant.

is there a more programatic way to order the test files to ensure the Meta class gets invoked last, but not have to list every single test file?
For now I don't mind having to specify the order manually. If it gets to be annoying down the road we can probably just refactor pytest_collection_modifyitems() such that it just puts TestCodebase and TestMeta at the end and leaves everything else alone. But again, for now I don't mind is so much. 🤷♀

Definitely not finished, since I didn't implement a way for admins to silence this message

We could break that off into a separate follow-on PR after we get this one merged. In the interest of keeping PRs to a reasonable size.

Thanks for the help!

Codecov Report

Merging #143 into master will decrease coverage by 1.25%.
The diff coverage is 66.66%.

@@             Coverage Diff             @@
##            master     #143      +/-   ##
===========================================
- Coverage   100.00%   98.74%   -1.26%
===========================================
  Files           19       20       +1
  Lines          464      477      +13
  Branches        22       24       +2
===========================================
+ Hits           464      471       +7
- Misses           0        4       +4
- Partials         0        2       +2

Impacted Files
Coverage Δ

src/spellbot/reactions.py
66.66% <66.66%> (ø)

src/spellbot/data.py
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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

I believe the coverage has gone down because we're not testing the code path where the logger gets called. Wasn't sure how you wanted to do that though.

I believe the coverage has gone down because we're not testing the code path where the logger gets called. Wasn't sure how you wanted to do that though.

Here's an additional test for safe_clear_reactions() that gives us 100% coverage for that function:
    async def test_clear_reactions_when_http_error_is_thrown(self, caplog):
        message = MockDiscordMessage()
        fake_response = Mock()
        fake_response.status = 400
        with patch.object(message, "clear_reactions", wraps=message.clear_reactions) as m:
            m.side_effect = discord.errors.HTTPException(fake_response, "BAD REQUEST")
            with patch.object(message.channel, "send", wraps=message.channel.send):
                await safe_clear_reactions(cast(discord.Message, message))

                assert "warning: discord: could not clear reactions" in caplog.text
                assert "BAD REQUEST" in caplog.text

I am doing the cast(discord.Message, message) to make Pylance shut up about the type mismatch there. But the real meat of the test is that we can use the caplog fixture to make sure that the log contains the things we want. While working on this though I found a bug!
In reactions.py I have this code:
logger.exception("warning: discord: could not clear reactions", e)

But that is actually wrong. It needs to be:
logger.exception("warning: discord: could not clear reactions: %s", e)

If you want to fix up all the logger.exception() calls in this PR I would be very grateful!

Sorry I didn’t get a chance to fix up the logging yesterday. Your changes look :+1: to me.

Sorry I didn’t get a chance to fix up the logging yesterday. Your changes look 👍 to me.

I was mostly testing out https://docs.github.com/en/github/collaborating-with-issues-and-pull-requests/committing-changes-to-a-pull-request-branch-created-from-a-fork to see if it actually worked. Pretty neat.


--minify results in unclosed HTML tags

What version of Hugo are you using (hugo version)?
$ hugo version
Hugo Static Site Generator v0.61.0/extended darwin/amd64 BuildDate: unknown
## also tested with
Hugo Static Site Generator v0.64.1/extended darwin/amd64 BuildDate: unknown

Does this issue reproduce with the latest release?
Yes.
What's the issue?
When I run hugo --gc --minify, some HTML tags are not completed correctly.
Original html:

{{ .heartName }}

{{ .heartText }}

Minified html:

Head

Core academic content that is aligned to specific curricular materials and research-based practices.

In the minified HTML, div.icon-box is not closed after the  tag, and instead div.h3text is opened.
https://github.com/amygroshek/testteachinglab can be used for testing. The snippet referenced above is in index.html. There are other places where the HTML is also broken by minify, but this is the easiest to find.
The issue was hard to identify because all of the CSS files were clearly being loaded correctly. A clearer idea of how the pages are affected can be seen here:

https://5e3dabfcc8dff16b422123b5--eloquent-gates-b65172.netlify.com/
https://5e3dabfcc8dff16b422123b5--eloquent-gates-b65172.netlify.com/our-model/

I annotated the following screenshot while trying to track it down. Local server differs, of course, from deploy in that --gc and --minify flags are not used.

@amygroshek Thank you for detailed reporting
I have tested with Hugo Static Site Generator v0.64.1/extended darwin/amd64

Not reproduced with your minimal snippet.
Reproduced with your repository

This means that the issue is caused NOT by Hugo's implementation but your site's template.
Therefore, I created PR for your repository
Although you might suggest that Hugo have any warning or error of such kind of inconsistent minification results, it may be an upstream issue rather than Hugo's in my opinion.

Although you might suggest that Hugo should show any warning or error of such kind of inconsistent minification results, it may be an upstream issue rather than Hugo's in my opinion.

There is nothing Hugo can (or will) do about this. You may raise an issue on the minify library and see if he thinks he can do something about it.

Thanks @satotake and @bep.

If you are ok, could you close this, @amygroshek ?


Update README.md

Improvements to readme and other documentation. Closes #1, closes #2

Looks good to me


NoSuchMethodError in RedisStateMachine.readLong

While using lettuce version 4.2.1.Final and netty version 4.1.0.Final, I get a NoSuchMethodError in RedisStateMachine.readLong.
See the stacktrace in the gist:
https://gist.github.com/chscorpio/ce3af686589eb5f49e3dc30102e9e637
It seems this issue was already identified in https://github.com/mp911de/lettuce/issues/274 and fixed on master in https://github.com/mp911de/lettuce/commit/824618dc442c1eb7e0e3bb8078a85cdff750a126 but not included in 4.2.1.Final.
Can this fix or a variation of it be fixed for a 4.2.2.Final or subsequent release? Are there any workarounds that will allow me to continue using netty 4.1.0?
The reason this is an issue for us now is that we decided to upgrade to 4.2.1.Final from 4.0.0.Final after we encountered reconnection issues which we see resolved for 4.2.1.Final. The good news is that our QA team has verified that 4.2.1.Final has fixed the reconnection issues and now we want to update all our services to use Lettuce 4.2.1.Final. Unfortunately, some of our services also depend on grpc 0.13.2 which depends on netty 4.1.0 and hence this NoSuchMethodError is blocking us from upgrading those services.

I just realized that Lettuce 4.2.1.Final tries to deal with the differences in the RedisStateMachine constructor with the following code...
        Version nettyBufferVersion = Version.identify().get("netty-buffer");
        if(nettyBufferVersion != null) {
            useNetty41ByteBufCompatibility = nettyBufferVersion.artifactVersion().startsWith("4.1");
        } else {
            useNetty41ByteBufCompatibility = false;
        }

I'll be debugging now why useNetty41ByteBufCompatibility ends up false for our service.

Found it! The netty-buffer version isn't in our shaded jar because META-INF/io.netty.versions.properties is overwritten by netty-codec.
I need to find a way to merge all the META-INF/io.netty.versions.properties into our shaded jar so that the netty-codec and netty-buffer versions are found.

Adding the following lines to the maven-shade-plugin definition in our pom.xml worked for me...
<transformer implementation="org.apache.maven.plugins.shade.resource.AppendingTransformer">
    <resource>META-INF/io.netty.versions.properties</resource>
</transformer>

See details at https://maven.apache.org/plugins/maven-shade-plugin/examples/resource-transformers.html#AppendingTransformer

I'm not seeing this as resolved - in fact as far as i can see the fix #274 never made it into 4.2.1.Final or 4.2.2.Final.
Note screenshot from 4.2.2.Final-sources.jar: https://www.dropbox.com/s/62euk5a1oyk1ydm/Screenshot 2016-08-29 12.30.20.png?dl=0
here's the file in the 4.2.x branch (note use of ToLongProcessor)
https://github.com/mp911de/lettuce/blob/4.2.x/src/main/java/com/lambdaworks/redis/protocol/RedisStateMachine.java#L50
and the fix here - in master:
https://github.com/mp911de/lettuce/blob/master/src/main/java/com/lambdaworks/redis/protocol/RedisStateMachine.java#L50
Also, here's the io.netty.versions.properties for my final application jar:
netty-codec-http.version=4.1.4.Final
netty-codec-http.buildDate=2016-07-27 13\:37\:22 +0200
netty-codec-http.commitDate=2016-07-27 13\:16\:59 +0200
netty-codec-http.shortCommitHash=9466b32
netty-codec-http.longCommitHash=9466b32d05ecf63b7e3a16e8e432ca5def21dc1e
netty-codec-http.repoStatus=clean

my bad. i misunderstood the fix and this issue. plz ignore my comment.


International addresses

Context
Candidates who live overseas need to provide postal address in a localised format so that providers can contact them by post.
Changes proposed in this pull request

[x] We added the international_address and address_type columns to application_forms in an earlier merged PR.
[x] Add an international_addresses feature flag.
[x] Add new form to the contact details flow to prompt for the type of address (UK or international) that sits between the phone number and address forms.
[x] Change the existing address form to use a single text area for address input if the candidate specified international.
[x] Update the ContactDetailsReviewComponent to show international addresses when appropriate and have the change button link to the address type page (so that the candidate can switch from one type of address to the other as well as change the actual address).
[x] Add a country drop down to the Where do you live? page. (Following discussions about how we can fit international addresses into the existing Vendor API schema).

Guidance to review

Is the feature flag watertight?
Would it make sense to rename the feature flag international to cover other cards in the epic or keep it small?
We would run into problems if we enable the feature, candidates start adding international addresses and we switch it off again - the review page doesn't show international addresses unless the feature is on.
Not covered in this PR - changes to the Vendor API - separate PR to follow if needed
Should we clear the address_line1, address_line2 etc. if a candidate switches from a UK to international address and clear international_address if a candidate switches from international to UK?

Link to Trello card
https://trello.com/c/Wif90Hdc/1683-dev-🌐-international-addresses
Things to check

[x] This code doesn't rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] New environment variables have been added to the Azure config

Should we clear the address_line1, address_line2 etc. if a candidate switches from a UK to international address and clear international_address if a candidate switches from international to UK?

Yes, I think we should!

Should we clear the address_line1, address_line2 etc. if a candidate switches from a UK to international address and clear international_address if a candidate switches from international to UK?

Yes, I think we should!

👍 done


/replaceitem still recognized as valid 1.17 command

As described, replaceitem doesn't throw an error when used in a 1.17 setting.

Is datapack.env.cmdVersion set to 1.17 as well?

Yes.

Thanks, should be fixed now.

Actually no. The old behavior is done in order to provide quick fixes for updating replaceitem commands. I will release a new version to revert the changes.


README.md

Thanks for your commit to the repo

Your welcome . My will was to edit and develope your repo by some issues couldn't it happened. Like add a field that user can upload a txt file with 200 uuids in rach file and specific extra parameters that should be add in your web app like tcp delay . Xudp and exactly what is xudp . Is there in Iran work in our communication internet network 🛜?! And a second page that users can put a word or a specific sentence in find field and choose a file instead replace field or a word or a a file that must be split and each of it can be in one of created vl/vmess configs in previous my ask that was creates any configs with web app and generate uuid by Ubuntu . I want edit it about next week . I know this is Hugh and needs some help if I will run it in attached future my man.

Thank you for your message and your willingness to contribute to the development of my repository. I appreciate your input and ideas for enhancing the functionality of the web app. Regarding XUDP, it stands for Extended UDP, which is an API for extended UDP networking.
Currently, I'm facing some time constraints due to work commitments, but I'm open to collaboration and assistance. If there are specific areas or features in the repository that you'd like help with, please let me know, and I'll do my best to guide or contribute when time permits. Your suggestions for adding a field to upload a txt file with UUIDs, along with additional parameters, sound interesting, and I'd be happy to discuss further details.
Feel free to share more details about the specific issues you encountered so that we can work together on resolving them. Thank you again for your interest and willingness to contribute, and I look forward to collaborating with you in the near future.


Helm Release stuck in pending-install after successful installation

I found a bug that I can't seem to debug any further.
In our automated CI/CD we deploy a review app into a temporary namespace.
To do so we execute a basic helm upgrade --install --atomic --wait --create-namespace -n review-feature-42-n1wzux review-app chart
On a initial deployment we get a success response from helm and our apps work as expected.
Release "review-app" does not exist. Installing it now.

--
Output of helm version:
version.BuildInfo{Version:"v3.14.2", GitCommit:"", GitTreeState:"", GoVersion:"go1.21.10"}
Output of kubectl version:
Client Version: v1.28.4
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.28.3-r0-<IP_ADDRESS>
Cloud Provider/Platform (AKS, GKE, Minikube etc.):
Open Telecom Cloud / Huawai Cloud

The second after i pressed send, i remembered that the ServiceAccount used for helm doesn't have the update privileges on secrets. After adding it, it works.
So while this ticket can probably be closed I really wished for there to be any kind of error-message.

@krankkkk thanks for your issue and feedback. it's a good idea for having a kind of error-message. I will try to do that. thansk for you issue again.

@yxxhero from the code. this error message can be seen when you enable debug mode.

@yxxhero In my Opinion, an error message should never be hidden per default. Also it shouldn't quit with exitcode 0 if an error occurs, that prevents an important functionality of helm to function.


Added BoundedChannelFullMode overload to the bounded mailbox overloads

Description
According to the documentation, the bounded mailbox should let the user choose a strategy when the mailbox queue is full.

Dropping Tail Mailbox = BoundedChannelFullMode.DropNewest
Dropping Head Mailbox = BoundedChannelFullMode.DropOldest

However, there was only one overload where you could select the size, but not the fullMode.

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


onion website

Hi guys,
Would it possible to test onion website with vegeta ? with slow requests or fast
Cheers,

Vegeta only speaks HTTP. If you can reach your Onion website via an HTTP proxy, then yes.


Release Google.Cloud.Speech.V2 version 1.0.0-beta03

Changes in this release:
New features

Enable REST transport in C# (commit e04406f)

Created release for Google.Cloud.Speech.V2-1.0.0-beta03


Add documentation about our release process

Codecov Report

Merging #1159 (7c9212a) into master (4d7d6b6) will decrease coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1159      +/-   ##
==========================================
- Coverage   97.28%   97.22%   -0.06%
==========================================
  Files          67       66       -1
  Lines        4235     4216      -19
==========================================
- Hits         4120     4099      -21
- Misses        115      117       +2

Impacted Files
Coverage Δ

httpie/cli/argparser.py
95.54% <0.00%> (-0.81%)
: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 e2ba214...7c9212a. Read the comment docs.

Rebased on master.


docs: images not rendering on patterns landing page

Closes #1070
What I did

Added srcAbsolute to the example images shortcode to correct the image paths under /patterns

Testing Instructions

You should see images again in the DP

Looks good to me, if we have the card image we should add that but otherwise looks g2g

@eyevana Ready for you.
XD: https://xd.adobe.com/view/0f123ffb-ae98-411e-ada3-2526129e0c74-5a41/
Content doc: https://docs.google.com/document/d/1DIwKX9S707VmH2SnmZyqBLCpHb7ozZkzu0cWlLpzbZU/edit
Images: https://www.dropbox.com/sh/94lqqs0xhtusji9/AABHvx_WvuKM8JbUdmV8gu7ya?dl=0

@coreyvickery with the exception of that comment I made in the content doc, this is ready for your review :)


[DROOLS-879] update documentation for Drools Rule Template

+1, merged into master. Sorry it took so long, I completely forgot about this one.


hover on Life Course View / Copy To Clipboard buttons should have an explanatory tooltip

Both these buttons should have an explanatory tooltip saying what they do.

Added by @dc2007git with #61


libcrypto error on updating to 1.1

Bug Report
I am getting this new error after having moved to the new 1.1 version
Since it doesn't reference any of my code, I am assuming this is just broken dependencies.
rishav@rishav-sbook:/mnt/c/Users/risharan/Documents/GitHub/digglu$ crystal run src/digglu.cr
Showing last frame. Use --error-trace for full trace.

In lib/openssl_ext/src/openssl_ext/lib_crypto.cr:374:3

 374 | fun x509_digest = X509_digest(x509 : X509, type : EVP_MD, md : UInt8*, len : UInt32*) : Int32
       ^----------
Error: fun redefinition with different signature (was `fun x509_digest = X509_digest(x509 : X509, evp_md : EVP_MD, hash : ::Pointer(UInt8), len : ::Pointer(Int32)) : Int32` at /usr/share/crystal/src/openssl/lib_crypto.cr:171:3)

As this is not a stdlib issue, closing it.
Its being tracked at https://github.com/spider-gazelle/openssl_ext/issues/8


Add padding label config key

Padding bytes in the map key struct need to be considered in the config of the label. This is done by choosing a larger size for the label's value. For most cases this works fine.
This commit adds a new config key for labels that allows to specify padding bytes explicitly so they can be skipped by decoders.
Closes #369

Thanks!


Create in_Java.java

@4N1Z Hello I added Quick Sort In java, with proper documentation like what is quick sort, working, correct code with comment and also the Time and space complexity. Please check and please merge.


Add function/example for performing self-test

Would be useful, to check if the component itself isn't broken.
For reference, here's an implementation of said test: https://github.com/ljos/MFRC522/blob/d42749442fdd1ce58c74be9c88d65e5f48126a3f/MFRC522.cpp#L162

Seems OK and useful.
Can you implement it and make a pull request?


Playing Broadcast video  Issue

for Playing Broadcast video only Audio appears video comes in black

是不是网速的问题 丢帧策略开启后导致没有视频传输了，你把丢帧策略关闭一下试试


Unable to download 0.8.3

I'm unable to download (via HACS) level 0.8.3 that contains the fix for the issue discovered on the latest upgrade to HA (2024.7.0). I have not been auto notified of the Mixergy update via HACS and for some reason when I access the repository entry via HACS I only see a message to support the author on Ko-fi when I expect to see the README.md.
The results are the same whether I access HA via a Windows Firefox browser or via the HA App on Apple devices,  to view the HACS repository.
I hope this makes sense.
Kind regards

Sorry, that's out of my control I'm afraid. When I open the Integration via my HomeAssistant, it shows the 0.8.3 version.

Several users in issues #56 have reported success.
Perhaps raise an issue with HACS? Sorry I can't be more help.

Thanks for the hint. As it turns out I was able to download the fix in 0.8.3 after I updated HA core to 2024.7.1.


Empty list when pagination is turned on

Simple lists work fine. But if I turn pagination on ("paginate: true") then no data are displayed.
const { items: games } = useFind({
  model: context.root.$FeathersVuex.api.Game,
  params: reactive({
    paginate: true,
    qid: 'myGames',
    query: {
      $limit: 50,
      $skip: 0,
    },
  }),
});

I found one problem here: https://github.com/feathersjs-ecosystem/feathers-vuex/blob/master/src/useFind.ts#L106
const pagination =
  computes.paginationData[getterParams.qid || state.qid] || {}
...
const items = getItemsFromQueryInfo(
  pagination,

"paginationData" is computed entity, so it's a ref object and stores its data in "value" property, not directly. That's why "pagination" always is an empty object and then "items" is always an empty array.
computes.paginationData.value[getterParams.qid || state.qid] || {}

When I change this line, data are displayed at least.

Ah.  Good catch!  I'll patch this up.

Fixed in<EMAIL_ADDRESS>


support and title

hi bro, nice soft!
you need support in btc\monero?
improve installation instruction:
node-exec \ cluster-exec - this command not found
correct command on this moment:
run \ run-all

Thanks, I pushed a fix for this. This whole thing will eventually be a part of pynfra (https://github.com/gergelykalman/pynfra), but I did not have much time to make progress on that lately.
Partly as a result of that I accidentally committed some test creds, which I had to nuke with force push. Sorry about that, don't code before coffee.
Speaking of coffee, I set this up in case you were serious: https://www.buymeacoffee.com/gergelykalman


[Harmony] Adding get validators and get delegations #733

Moved gupadhyaya 's old PR (https://github.com/trustwallet/blockatlas/pull/733) here

Please, reopen the PR and fix the functional tests.
Issue 1: http://localhost:8420/v2/harmony/staking/validators
{
    "code": 500,
    "error": {
        "error": "unexpected end of JSON input",
        "meta": {},
        "type": "Platform Unmarshal Error"
    }
}

Issue 2:  http://localhost:8420/v2/harmony/staking/delegations/one1e4mr7tp0a76wnhv9xd0wzentdjnjnsh3fwzgfv
{
    "code": 500,
    "error": {
        "error": "RPC Call error: Unable to fetch delegations list",
        "meta": {
            "error_code": -32601,
            "error_message": "The method hmy_getDelegationsByDelegator does not exist/is not available",
            "method": "hmy_getDelegationsByDelegator"
        }
    }
}

@EnoRage Can you tell me how to run your test cases ?
The issue is that the endpoint has been changed to https://api.s0.os.hmny.io/.

@coolcottontail
in root of project
make test for Unit tests
make functional for Functional tests (you will need all endpoints for config.yml  - not neccessary for your case)
make integration for integration tests
Can you please just finish previous issues, new response with updated endpoints (your last commit in branch) is:
Endpoint:
http://localhost:8420/v2/harmony/staking/delegations/one1e4mr7tp0a76wnhv9xd0wzentdjnjnsh3fwzgfv
{
    "code": 500,
    "error": {
        "error": "json: cannot unmarshal array into Go value of type harmony.Delegations: json-rpc GetObject Unmarshal error: Unable to fetch delegations list",
        "meta": {
            "obj": {
                "result": null
            },
            "string": "[]"
        }
    }
}

endpoint:
http://localhost:8420/v2/harmony/staking/validators
{
    "code": 500,
    "error": {
        "error": "unexpected end of JSON input",
        "meta": {
            "method": "GET",
            "path": "hmy_getAllValidatorAddresses"
        },
        "type": "Platform Unmarshal Error"
    }
}

@EnoRage
I can only run make test without any issue.
make integration and make functional all failed even if I use the origin/master branch without any modification. Do I need to source any script before running tests ?
which test case will trigger this error ?  I am trying to duplicate and find out the reason...
http://localhost:8420/v2/harmony/staking/delegations/one1e4mr7tp0a76wnhv9xd0wzentdjnjnsh3fwzgfv
{
"code": 500,
"error": {
"error": "json: cannot unmarshal array into Go value of type harmony.Delegations: json-rpc GetObject Unmarshal error: Unable to fetch delegations list",
"meta": {
"obj": {
"result": null
},
"string": "[]"
}
}
}
make integration failed:
--- FAIL: Test/test_batch_with_invalid_coin (0.00s)
domains.go:117:
Error Trace:	domains.go:117
Error:      	Expected nil, but got: &errors.Error{Err:(*errors.errorString)(0xc000c3a760), Type:0x0, meta:map[string]interface {}{"coins":[]uint64{0x2c, 0x3c}, "name":"vitalik.eth"}, stack:[]string(nil)}
Test:       	Test/test_batch_with_invalid_coin
domains.go:118:
Error Trace:	domains.go:118
Error:      	Not equal:
make functional
--- PASS: TestApis//v2/staking/delegations (0.00s)
reporter.go:23:
    	Error Trace:	reporter.go:23
    	            				chain.go:21
    	            				response.go:585
    	            				response.go:151
    	            				http.go:57
    	            				http.go:92
    	            				asm_amd64.s:1357
    	Error:
    	            	expected status equal to:
    	            	 "200 OK"

    	            	but got:
    	            	 "400 Bad Request"
    	Test:       	TestApis

@EnoRage
I can only run make test without any issue.
make integration and make functional all failed even if I use the origin/master branch without any modification. Do I need to source any script before running tests ?
which test case will trigger this error ? I am trying to duplicate and find out the reason...
http://localhost:8420/v2/harmony/staking/delegations/one1e4mr7tp0a76wnhv9xd0wzentdjnjnsh3fwzgfv
{
"code": 500,
"error": {
"error": "json: cannot unmarshal array into Go value of type harmony.Delegations: json-rpc GetObject Unmarshal error: Unable to fetch delegations list",
"meta": {
"obj": {
"result": null
},
"string": "[]"
}
}
}
make integration failed:
--- FAIL: Test/test_batch_with_invalid_coin (0.00s)
domains.go:117:
Error Trace: domains.go:117
Error: Expected nil, but got: &errors.Error{Err:(*errors.errorString)(0xc000c3a760), Type:0x0, meta:map[string]interface {}{"coins":[]uint64{0x2c, 0x3c}, "name":"vitalik.eth"}, stack:[]string(nil)}
Test: Test/test_batch_with_invalid_coin
domains.go:118:
Error Trace: domains.go:118
Error: Not equal:
make functional
--- PASS: TestApis//v2/staking/delegations (0.00s)
reporter.go:23:
    	Error Trace:	reporter.go:23
    	            				chain.go:21
    	            				response.go:585
    	            				response.go:151
    	            				http.go:57
    	            				http.go:92
    	            				asm_amd64.s:1357
    	Error:
    	            	expected status equal to:
    	            	 "200 OK"

    	            	but got:
    	            	 "400 Bad Request"
    	Test:       	TestApis

Currently it is not possible to run all the tests without production endpoints that are not public. We are working to mock all of the data from them.
I reccomend just to fix issue with
http://localhost:8420/v2/harmony/staking/delegations/one1e4mr7tp0a76wnhv9xd0wzentdjnjnsh3fwzgfv
To test it locally just:
cd cmd/platform_api
ATLAS_PLATFORM=harmony go run main.go
Try GET request that was mentioned before.
You can also debug only platform_api binary. I saw, that your client implementation make a RPC request that does not have exist. See before. Also, can you please reopen the PR, because I cannot do it on my own?


getMetadataBuffer() memory leak?

1/  am not sure about this, but I noticed when using getMetadataBuffer() in my implementation, it creates a memory leak without using 'close()' on the buffer. I was searching the DEMO code  for the same sequence but did not find 'close()' there. Do you see memleaks there?
2/ In README.MD, I don't see a reference to RetrieveContentsWithProgressDialogActivity  (mentioned in 'https://developers.google.com/drive/android/files' and present in the code set). Probably by-design and a non-issue.
Sorry, for being obnoxious, I know these are minor issues. I'm trying to point out roadblocks I run into, most of them probably caused more by my lack of knowledge/ignorance.

Closing, please reopen if this is still an issue.


Fixes msgfmt error in line 196

The current file in upstream was incorrect, see:
msgfmt -c --statistics messages.po
messages.po:196: keyword "Apel" unknown
messages.po:196:5: syntax error
msgfmt: found 2 fatal errors
It looks like that a last minute fixed went into the wrong file of the file. This fixes the issue:
After the fix:
msgfmt -c messages.po  --statistics
552 translated messages.

Visibility cc: @darccio  @ivanbea

Oh cool, appreciate it

Perfect! Thank you @jordimas for the last minute check and @pfrazee for the merge!


Could not connect to "http://localhost:8000/" even when the app is running

Describe the bug
Swa cli fails to start with error message [swa] Could not connect to "http://localhost:8000/". Is the server up and running? after successful start of gatsby dev server
To Reproduce
Steps to reproduce the behavior:

Go to terminal
Type in command swa start http://localhost:8000/ --api http://localhost:7071 --run "npm start"
See error

Expected behavior
Expectation is that swa cli detects that gatsby has started
Screenshots

Hey @simonaco thank you for reporting this issue. The good news is that we've already fixed it in https://github.com/Azure/static-web-apps-cli/pull/281 (issue https://github.com/Azure/static-web-apps-cli/issues/275). However, we haven't published a release to npm yet. cc @anthonychu.
Just as an FYI, here are the steps I followed (using v 0.7.2 - currently in main branch):

install Gatsby: npm install -g gatsby-cli
create a new blog: gatsby new blog https://github.com/gatsbyjs/gatsby-starter-blog
cd blog
swa start http://localhost:8000 --run "npm start" (I am not using SWA API)
access blog at http://localhost:4280
access GraphQL console at: http://localhost:4280/___graphql

As you can see from my screenshots

Hey @simonaco,
We have an upcoming release this week and you should see it in the latest npm package soon! 😄

This is awesome! Thank you @Reshmi-Sriram 👍

Hii @simonaco we've done a new release today and previous changes have been included in this, please check once.

thank you @Reshmi-Sriram @sgollapudi77 . I just tested with a simple Gatsby app and one API endpoint and I'm still seeing issues, please check this screen recording swa-cli doesn't start

Thank you @manekinekko for debugging with me. It seems like the issue was related to how I was starting the CLI. When using this command,  swa start http://localhost:8000/ --api http://localhost:7071 --run "npm start" I need to have the functions server started.
Alternatively I can use this command to instruct the CLI to start the functions dev server: swa start http://localhost:8000/ --api api --run "npm start"


fix building on plan9

note that i'm unsure if this code really works, but it allows github.com/xenolf/lego/acme to build on 9front.

dunno either, it's also impossible for me to verify and probably hard to make a test for this...
See no reason not to merge this.

can try to run tests later, i'll send more PRs if something goes wrong. thanks!

would be appreciated! Thanks!


chore(deps): update dependency markupsafe to 2.1.1

This PR contains the following updates:

Package
Update
Change

MarkupSafe (changelog)
minor
==2.0.1 -> ==2.1.1

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

@balena-ci I self-certify!

Renovate Ignore Notification
As this PR has been closed unmerged, Renovate will now ignore this update (==2.1.1). You will still receive a PR once a newer version is released, so if you wish to permanently ignore this dependency, please add it to the ignoreDeps array of your renovate config.
If this PR was closed by mistake or you changed your mind, you can simply rename this PR and you will soon get a fresh replacement PR opened.


Update internal eval & latex to use the new paste service API

Relevant Issues

Description

Did you:

[x] Join the Python Discord Community?
[x] Read all the comments in this template?
[x] Ensure there is an issue open, or link relevant discord discussions?
[x] Read and agree to the contributing guidelines?

Hi Chris! Thanks for the contribution!
Please could you complete the checklist in the issue body before we can review and approve this PR. Thank you in advance!

Ah, the checklist, a true masterpiece of human achievement!  or "Did you sacrifice a rubber duck to the coding gods?"
Fear not, dear Joe! My contribution is like a diamond in the rough, waiting for the magical checklist wand to bless it. So I will go ahead, dive into the abyss of checkboxes, and emerge victorious! And remember, if I miss one, the entire universe might collapse. So no pressure!
(Disclaimer: The universe will not actually collapse.)

We have once again narrowly avoided a collapse of the universe itself, for our hero ChrisLovering has saved us from yet certain doom.


Smaller icons?

Feedback from Discord. Icons could be smaller on the posters.

Closed via 068982e7b2105024b851a1406244e93248c5d21f.


About the number of instances

There are only six categories, but the inference result shows that a total of 100 instances are detected, what is the reason?

We output test_topk_per_image=100 instances by default for coco evaluation. In actual use, you can try to set a classification threshold, e.g. 0.5, to further filter the output.

Does the classification threshold refer to the value of 0.5 in the picture?

Do I need to set 'NUM_OBJECT_QUERIES' and 'DETECTIONS_PER_IMAGE' to my number of categories 6 as well?

 Does the classification threshold refer to the value of 0.5 in the picture? I set it to 0.9 and it still outputs 100 instances

No, this is the threshold for mask predictions. The classification threshold is set for result.scores.

Do I need to set 'NUM_OBJECT_QUERIES' and 'DETECTIONS_PER_IMAGE' to my number of categories 6 as well?

If there are only a few instances in an image, you can try reducing "NUM_OBJECT_QUERIES" for better speed-performance trade-off. But this may require you to retrain the model.
'DETECTIONS_PER_IMAGE' generally does not need to be changed. It is used for evaluations.

 Does the classification threshold refer to the value of 0.5 in the picture? I set it to 0.9 and it still outputs 100 instances

No, this is the threshold for mask predictions. The classification threshold is set for result.scores.

Do I need to set 'NUM_OBJECT_QUERIES' and 'DETECTIONS_PER_IMAGE' to my number of categories 6 as well?

If there are only a few instances in an image, you can try reducing "NUM_OBJECT_QUERIES" for better speed-performance trade-off. But this may require you to retrain the model.
'DETECTIONS_PER_IMAGE' generally does not need to be changed. It is used for evaluations.

Thank you for your patience, under your guidance my problem has been successfully solved!😄

can you tell me how to slover this problem, I have meet this question? @ @Whoo-jl


Support RMSProp

Hi, would you please support RMSProp as learning rule?

Hi we've just merged an implementation of RMSProp, supported across all backends.  There are a couple of examples that you can try out by setting lrule_init = *rms for one or both of the layers in either examples/mlp/mnist-small.yaml or examples/mlp/mnist-tuned.yaml.

Great! Thanks a lot.


add test data for pangenome pipeline

This brings in a small test data set for the https://github.com/nf-core/pangenome pipeline.

Made a new branch pangenome and updated this PR to point against it (see docs).

Would be great if you could update the readme to describe what the test data is and where it came from.
See for example the test data for the atacseq pipeline: https://github.com/nf-core/test-datasets/tree/atacseq

I decided to draft this because of the deterministic challenges.

Let's do this @Zethson :)


I reversed the image and found the background changed, is this normal?

I reversed the image and found the background changed, is this normal?

Yes, it is normal: you have projected the real image on the set of images which can be generated by the model of faces.

Yes, it is normal: you have projected the real image on the set of images which can be generated by the model of faces.
If you want to keep parts of the original image, you can use a software like GIMP with transparency and masks.

Ok, thank you for your prompt reply, and I have another question.
1）I see that there is a landscape generation model with a resolution of 256（StyleGAN3 model trained on Landscapes HQ with 256x256 output resolution and saved as .pt file. Model taken from [Justin Pinkney]）. How to get a higher resolution model,
2) The image attribute acquisition of the landscape is similar to the face Attribute editing (smile, angry..), how to get rain, snow weather, depression, sea river, mountain  attributes for stylegan3

See: https://github.com/yuval-alaluf/stylegan3-editing#editing-real-images
For landscapes, I imagine you could try to find your own directions with InterfaceGAN:
https://github.com/yuval-alaluf/stylegan3-editing/blob/fd84c744f642660e84bd822af709cf68a35e34eb/editing/interfacegan/face_editor.py#L21-L26
Or more simply, fiddle with CLIP.

How to get a higher resolution model,

You would need to train your own generator on the Landscapes HQ dataset. To avoid training, we used the pretrained generator from Justin Pinkney.
Regarding the editing, @woctezuma is correct in that you can use either Interfacegan or StyleCLIP. Since InterFaceGAN requires supervision, we used StyleCLIP's global directions to perform edits on the landscapes domain.

For landscapes, I imagine you could try to find your own directions with InterfaceGAN first. The equivalent of:
https://github.com/yuval-alaluf/stylegan3-editing/blob/fd84c744f642660e84bd822af709cf68a35e34eb/editing/interfacegan/face_editor.py#L21-L26
See: https://github.com/yuval-alaluf/stylegan3-editing#interfacegan
Or more simply, fiddle with StyleCLIP.
See: https://github.com/yuval-alaluf/stylegan3-editing#styleclip-global-directions

Thank you for your reply
Okay, I understand, I have a little doubt, for attribute search, I should not know what attribute I am looking for in advance, what should I do?

If you want to edit landscape attributes, I would recommend using StyleCLIP. There you can simply use text to describe what edits you want to perform. We provide full functionality for performing edits with StyleCLIP in this repo.

If you want to edit landscape attributes, I would recommend using StyleCLIP. There you can simply use text to describe what edits you want to perform. We provide full functionality for performing edits with StyleCLIP in this repo.

I'm more inclined to use a few known .npy properties, just thinking about finding the relevant properties of the landscape，so  I want to realize the property editing of a few specific landscapes，not use styleclip  project ，I think attribute editing（use  .npy files） is more convenient for users,

Not really sure I follow. What do you mean by known npy properties and relevant properties? At the end of the day, you want to find directions in the latent space that alters a specific attribute (e.g., winter, desert, forest). You can use InterFaceGAN for this, but this would require you to get a classifier that outputs attribute scores for each attribute. I don't think its trivial to train a classifier to classify between winter and not winter.
With StyleCLIP, you can simply input the text you want (e.g., "winter") and we find the relevant direction for you in the latent space. The end result (i.e., a direction) is the same, but the way you get this direction is different.

Not really sure I follow. What do you mean by known npy properties and relevant properties? At the end of the day, you want to find directions in the latent space that alters a specific attribute (e.g., winter, desert, forest). You can use InterFaceGAN for this, but this would require you to get a classifier that outputs attribute scores for each attribute. I don't think its trivial to train a classifier to classify between winter and not winter. With StyleCLIP, you can simply input the text you want (e.g., "winter") and we find the relevant direction for you in the latent space. The end result (i.e., a direction) is the same, but the way you get this direction is different.

yes，I find it difficult to find the relevant attributes and need more classifier, but as a user, i think  dragging a slider is more convenient than writing descriptive words（like winter ...)

I guess using a text or a slider is a matter of preference :)
In any case, I will close this issue for now as I feel your questions have been answered. Feel free to open a new issue if needed

I guess using a text or a slider is a matter of preference :) In any case, I will close this issue for now as I feel your questions have been answered. Feel free to open a new issue if needed

ok，Thanks again for your prompt reply


Size counter from a selection

A very useful feature to compute the total size from selection.
Device, Directory and MediaFile details HTML should store the real size in byte in some attribute. Each item in list from a detail will have a checkbox (whatever its form), and each time we select a checkbox it update a "total size" widget somewhere.

Done in v0.6.2


How to convert a Spark DL model to keras model ?

I am new to SparkDL and basically following available resources. One of the video shows (https://databricks.com/blog/2017/06/06/databricks-vision-simplify-large-scale-deep-learning.html) , a Spark DL model got converted into Keras model before registering as an UDF. Check the "mllib_to_keras_model" function used by the presenter. Could not find that function anywhere ?

Keras is meant to train deep learning models while MLlib is a much more general framework that includes algorithms such as random forests, recommendation systems, etc. The function that was used contained some export logic that was specific to logistic and linear regression (which map well to keras equivalents). It has not been released because we feel it does not cover enough use cases to be useful at this stage.
Can you tell us more about your use cases?


OpenID callback flow hits unauthorized

Describe the bug
We are using AAD as our "main" source of auth - however due to MFA restrictions on users in the AAD, we have an OpenID Connect server (IdentityServer) for the purpose of test users.
Lately we have observed that the last step of the OAuth flow, the callback, will redirect to initiate the AAD login flow instead (as if the request is unauthorized).
The weird part is that this is sporadic so sometimes it is successful and the user is authorized but many times it is not.
To Reproduce
The SWA is not public but I'll try to explain the flow here (can provide a HAR dump if necessary).

GET https://REDACTED.azurestaticapps.net/.auth/login/test
GET https://REDACTED.azurestaticapps.net/.auth/login/test?post_login_redirect_uri=/.auth/complete&staticWebAppsAuthNonce=ItcUM53Zb721QqABRHVqCYcr7pKr64UIrMebXlxHP8I1OsXEaiHsiethFd5VjmupAO3P2laEg88N7BZ3%2B1kObJeYqiFno%2B7K0GYa7%2BdBW63Fk%2BX%2BAONg3Lpmo7H1nweD
FORWARDS to OpenID Server: https://app-REDACTED-us-e-1.azurewebsites.net/connect/authorize?response_type=code&client_id=e2etestclientid&redirect_uri=https%3A%2F%2FREDACTED.azurestaticapps.net%2F.auth%2Flogin%2Ftest%2Fcallback&nonce=21b3726e89de49ceb7c269224910214e_20231019073216&state=redir%3D%252F.auth%252Fcomplete&scope=openid+profile+roles+email&post_login_redirect_uri=%2F.auth%2Fcomplete&staticWebAppsAuthNonce=ItcUM53Zb721QqABRHVqCYcr7pKr64UIrMebXlxHP8I1OsXEaiHsiethFd5VjmupAO3P2laEg88N7BZ3%2B1kObJeYqiFno%2B7K0GYa7%2BdBW63Fk%2BX%2BAONg3Lpmo7H1nweD
Enter login info and submit
POST login info to https://app-REDACTED-us-e-1.azurewebsites.net/Account/Login?ReturnUrl=%2Fconnect%2Fauthorize%2Fcallback%3Fresponse_type%3Dcode%26client_id%3De2etestclientid%26redirect_uri%3Dhttps%253A%252F%252FREDACTED.azurestaticapps.net%252F.auth%252Flogin%252Ftest%252Fcallback%26nonce%3D21b3726e89de49ceb7c269224910214e_20231019073216%26state%3Dredir%253D%25252F.auth%25252Fcomplete%26scope%3Dopenid%2520profile%2520roles%2520email%26post_login_redirect_uri%3D%252F.auth%252Fcomplete%26staticWebAppsAuthNonce%3DItcUM53Zb721QqABRHVqCYcr7pKr64UIrMebXlxHP8I1OsXEaiHsiethFd5VjmupAO3P2laEg88N7BZ3%252B1kObJeYqiFno%252B7K0GYa7%252BdBW63Fk%252BX%252BAONg3Lpmo7H1nweD
GET https://app-REDACTED-us-e-1.azurewebsites.net/connect/authorize/callback?response_type=code&client_id=e2etestclientid&redirect_uri=https%3A%2F%2FREDACTED.azurestaticapps.net%2F.auth%2Flogin%2Ftest%2Fcallback&nonce=21b3726e89de49ceb7c269224910214e_20231019073216&state=redir%3D%252F.auth%252Fcomplete&scope=openid profile roles email&post_login_redirect_uri=%2F.auth%2Fcomplete&staticWebAppsAuthNonce=ItcUM53Zb721QqABRHVqCYcr7pKr64UIrMebXlxHP8I1OsXEaiHsiethFd5VjmupAO3P2laEg88N7BZ3%2B1kObJeYqiFno%2B7K0GYa7%2BdBW63Fk%2BX%2BAONg3Lpmo7H1nweD
GET https://REDACTED.azurestaticapps.net/.auth/login/test/callback?code=956C354430285E7D5E58899B379ED8EADDBCAEE9014E38DA7A27A2936B2C51F3&scope=openid profile roles email&state=redir%3D%252F.auth%252Fcomplete&session_state=DcTlyhBYxdrlzzjOUvm-bUwZUUNKhsZK4aQDxNKUWro.E54146CE19A619491F62BA3FD00A2A3F

But step # 7 sets a Location header to
/.auth/login/aad?post_login_redirect_uri=/.auth/login/test/callback?code%3d956C354430285E7D5E58899B379ED8EADDBCAEE9014E38DA7A27A2936B2C51F3%26scope%3dopenid%2520profile%2520roles%2520email%26state%3dredir%253D%25252F.auth%25252Fcomplete%26session_state%3dDcTlyhBYxdrlzzjOUvm-bUwZUUNKhsZK4aQDxNKUWro.E54146CE19A619491F62BA3FD00A2A3F
{
  "routes": [
    {
      "route": "/api/health",
      "allowedRoles": ["anonymous"]
    },
    {
      "route": "/logout",
      "redirect": "/.auth/logout"
    },
    {
      "route": "/*",
      "allowedRoles": ["access"],
      "headers": {
        "Cache-Control": "no-store"
      }
    }
  ],
  "navigationFallback": {
    "rewrite": "index.html",
    "exclude": ["*.{css,scss,js,png,gif,ico,jpg,svg}"]
  },
  "responseOverrides": {
    "401": {
      "statusCode": 302,
      "redirect": "/.auth/login/aad?post_login_redirect_uri=.referrer"
    }
  },
  "auth": {
    "rolesSource": "/api/GetRoles",
    "identityProviders": {
      "azureActiveDirectory": {
        "registration": {
          "openIdIssuer": "https://login.microsoftonline.com/REDACTED/v2.0",
          "clientIdSettingName": "AZURE_CLIENT_ID",
          "clientSecretSettingName": "AZURE_CLIENT_SECRET"
        },
      "login": {
        "loginParameters": []
        }
      },
      "customOpenIdConnectProviders": {
        "test": {
          "registration": {
            "clientIdSettingName": "OPENID_CLIENT_ID",
            "clientCredential": {
              "clientSecretSettingName": "OPENID_CLIENT_SECRET"
            },
            "openIdConnectConfiguration": {
              "wellKnownOpenIdConfiguration": "https://app-REDACTED-dev-us-e-1.azurewebsites.net/.well-known/openid-configuration"
            }
          },
          "login": {
            "nameClaimType": "http://schemas.xmlsoap.org/ws/2005/05/identity/claims/name",
            "scopes": [
              "openid",
              "profile",
              "roles",
              "email"
            ],
            "loginParameterNames": []
          }
        }
      }
    }
  }
}

Expected behavior
The user to be authorized and taken to the application

This is still an issue for us sadly. Curious if nobody else has experienced this?


Does not compile using Swift 3.0.2

Hi,
I just installed FontAwesome.swift using Cocoapod, but looks like it has not been upgraded to Swift 3.0.2 yet. Or is it?
Here is a piece of what Xcode complains about
FontAwesomeView.swift

FontAwesome.swift

Thank you

I haven't published v3.0.2 to CocoaPods yet. You can point your Podfile to this repo directly.

It has been published now.


[WIP] Add box constrained optimization dispatch in sciml_train

@ChrisRackauckas @Vaibhavdixit02

This doesn't cover univariate problems https://github.com/JuliaNLSolvers/Optim.jl/blob/master/docs/src/user/minimization.md#minimizing-a-univariate-function-on-a-bounded-interval though. @ChrisRackauckas should we add that as well? I am not sure wrt the use cases.

I don't think we have a need for it 🤷♂ , but open a low priority issue.


Add batch 2 scopes - Entra PowerShell cmdlets to increase usability

In this PR, I have added:

Scopes for the cmdlet.
Enriched description with additional information from API.
Removed command prompt.
Applied splatting.

Hi @SteveMutungi254 I just added two minor suggestions, otherwise LGTM. Thanks

Thank you, @csmulligan. Addressed them already.


Text is not find

I've two types of images to read :

And the second type (one letter only) :

But in both case, Tesseract return empty string.
Here is my code (found in docs) :
    public static void Main(string[] args)
        {
        var testImagePath = @"C:\plate\ExtractLicensePlate-0.jpg";
        if (args.Length > 0)
        {
            testImagePath = args[0];
        }

        try
        {
            using (var engine = new<EMAIL_ADDRESS>"eng", EngineMode.Default))
            {
                engine.SetVariable("tessedit_char_whitelist", "ABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789");

                using (var img = Pix.LoadFromFile(testImagePath))
                {
                    using (var page = engine.Process(img))
                    {
                        var text = page.GetText();//<-- empty :(

                    }
                }
            }
        }
        catch (Exception e)
        {
            Console.WriteLine(e.ToString());
            Console.Read();
        }
        Console.Write("Press any key to continue . . . ");
        Console.ReadKey(true);
    }

Maybe I miss an option or something else?

Hi to get better results from tesseracts you need to threshold the image and in second pic increase border padding and threshold the Image and then pass to the engine.


Add unit tests for rpc/v1/publish.go

You can find the file here. Take a look at this StackOverflow post for more info about gRPC testing in Go.

I'll assign you on this one @Smelton01

@matheusmosca looking forward to contributing


[SPARK-34798][SQL][TESTS] Fix incorrect join condition

What changes were proposed in this pull request?

join condition 'a.attr == 'c.attr check the reference of  these 2 objects which will always returns false. we need to use === instead
Why are the changes needed?

Although this join condition always false doesn't break the test but it is not what we expected. We should fix it to avoid future confusing
Does this PR introduce any user-facing change?

No
How was this patch tested?

UT

ok to test.

Can one of the admins verify this patch?

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40816/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40816/

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

Can one of the admins verify this patch?

ok to test.

Jenkins, retest this please.

Test build #136252 has started for PR 31890 at commit 7e4826b.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40834/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40834/

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

Test build #136252 has finished for PR 31890 at commit 7e4826b.

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

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

cc @cloud-fan

Merged to master, branch-3.1 and branch-3.0.


Threading action client and server

In order to handle simultaneous goals we need to modify the simple action client and server a bit

Closing because content is in folder_reorg


Float action button covers "Grand Total" hours

Layout of the list rows probably needs some work.

Happens when you have 4 different entries in the monthly overview (on my one plus one)


Update build-cran-package.sh

This is a test, please don't merge

We REALLY do not appreciate you spamming this repo with such test PRs. I'm locking this one. Please do not come back here.


More minor improvements to rope.rs

I have gotten rid of a few inefficiencies, two of which led to warnings.

The copy warnings seem to be an alias analysis bug (see #1144), which it looks like marijn is going to fix. In that light, do you still want to merge this? Pattern matching doesn't copy in general (it aliases), so converting things like leaf(y) to leaf({byte_len: x, _}) probably doesn't impact performance (though I'm not positive - marijn would have a better grasp on pattern-matching performance)

Ah, good to know. In that case, the only actual optimization is in function next. It's probably not worth merging yet. I will submit a pull request next time I have some more stuff to add to rope.

OK, thanks. Closing this one.


error in the code

hi !
thank you for this great work..
I tried to run your code on my laptop and i got this error " Traceback (most recent call last):
File "/home/trainees/NCAI_project/security-dataset/tools/train.py", line 13, in
from mmdet import version # contains object detection models
ModuleNotFoundError: No module named 'mmdet' " and couldnt know what is the cause of this error..
i hope you will reach my message
thank you

Thank you for your affirmation and praise for our work! I think the problem may occur in the compilation. Please refer to the usage section in README.md.


Data-navigator: Enabling navigable chart elements with alt text

This PR is demonstrating a prototype of in-progress functionality and is subject to change.

Description
This work is part of an active consultation on the accessibility of React Spectrum Charts. Under the hood we want to enable accessible interaction and navigation of every chart in RSC, regardless of whether canvas or svg are used. In addition, we want to provide an API that enables core RSC developers to have opinionated design control over the navigation and interaction experience of each visualization type.
We are targeting smart defaults that make it easy for downstream developers to get accessible, rich, interactive data visualizations out of the box. But we also want to enable overrides for complex or advanced (but valid) use cases.
In general, this is a PR that is intended to demonstrate a working prototype (at minimum) and at best actually contribute working code into RSC across the ecosystem.
Related Issue

#482
Motivation and Context

If a chart can be hovered, clicked: a user should be able to do this with just a keyboard.
If a chart can't easily be summarized in just a couple of sentences: a screen reader user should be able to navigate the data and hear alt text for each element.
If a chart is being used for important decisions or is in an exploratory context: a user should be able to navigate and explore the data in a meaningful, structured way without requiring the use of a mouse or eyesight.
Presently, these experiences are not possible.
How Has This Been Tested?

Screenshots (if appropriate):
Element-level navigation

Alt text at the element-level

Group-level navigation

The goal will be to use Vega's signals to actually show the focus indication, while my new Navigator component will handle actually making accessible navigation possible (even if the visualization is rendered using canvas).
Types of changes

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

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Current status

[x] = first pass, mostly working

Navigation (tested with screen reader and keyboard-only)

[x] Bar
[ ] Dodge/stack
[ ] Line
[ ] Area
[ ] Scatter
[ ] Pie
[ ] Combo

Semantics/labels

[x] Bar
[ ] Dodge/stack
[ ] Line
[ ] Area
[ ] Scatter
[ ] Pie
[ ] Combo

Interactivity (clicking/tooltips)

[ ] Bar
[ ] Dodge/stack
[ ] Line
[ ] Area
[ ] Scatter
[ ] Pie
[ ] Combo

Focus Indication w/ signals

[ ] Bar
[ ] Dodge/stack
[ ] Line
[ ] Area
[ ] Scatter
[ ] Pie
[ ] Combo

Bonus

[ ] Exit element
[ ] Help menu
[ ] Key re-mapping?
[ ] Dev-friendly API options?

While this PR isn't ready to be merged, what I hope for is some feedback on the general approach before populating the prototype's approach across all other visualization types.
Right now the navigation capabilities are pretty cool, even with a never-before accomplished navigation type:

Bar can be navigated left/right according to the left-right encoding (the "dimension" prop). In data navigator these encodings are all called "dimensions" via our dimensions structure generation api. This part isn't novel (pretty standard).
However, the encoding for "metric" can also be turned into a navigable dimension. This means that for a visualization that might be more complex or have more going on, you can actually navigate from smallest -> largest value (or vice-versa) as well. This is the up-down navigation. Pretty much no other library does this right now (you typically have to resort and redraw visualizations to change nav order), save for tools like Olli but that only applies to scatterplots.
For stacked/dodged you can also navigate across color using [ and ] keys. It may make more sense for these visualizations to use up/down and left/right for the categorical encodings and then [/] for metric-navigation.

The key bindings are set in the constants file, but obviously we can change these across charts and even allow end-users to set their own keybinds too (outside of the scope of this PR but worth thinking about!).

@frankelavsky sounds good. @c-lamoureux and I will try and provide feedback in the next couple of days. I'll try it out today and look through the approach.

@marshallpete sounds good and thanks! Note that I recommend entering the chart via keyboard. It'll show a big group indicator and left/right goes between high level groups. These are visually all the same at the high level (i use console logs to know where I'm at but a screen reader works too). But one is dimensions, one is metric, and (optionally) one is color. Enter goes down a level. If divisions exist (nested, for example) that is level 2. Otherwise, straight to the elements. Going back up is different depending on which "dimension" you want to traverse up.
I've got some design/ui ideas to help with wayfinding and keyboard commands we can think about too.
And any of this, from nav structure to keybinds to code quality can all be points of feedback. Nothing is sacred, it's a prototype after all.

For stacked/dodged you can also navigate across color using [ and ] keys. It may make more sense for these visualizations to use up/down and left/right for the categorical encodings and then [/] for metric-navigation.

I think we agree with you here; I lean towards using left / right / up / down for navigation Space/Enter for drilling in and Escape for drilling out. The fewer unfamiliar interaction patterns, like [ and ] for metric-navigation, the better. I used to work at an educational publisher, building interactive learning materials and games for K-6, and a wise mentor would provide feedback like, "That's great, now make it make sense to a 2nd grader." We need to keep in mind that its rare for charts to be keyboard or screen reader accessible, so it will be more helpful to use familiar interaction patterns, rather than introducing something exciting and new that adds to the cognitive load of data navigating.


Getting a mismatch trace_id

Hi, I was following the example from https://raw.githubusercontent.com/davidB/tracing-opentelemetry-instrumentation-sdk/main/examples/axum-otlp/src/main.rs to integrate axum + tracing + opentelemetry. However, I noticed that the trace_id is different between the one in HTTP header vs find_current_trace_id(). In the README of the example, it showed that the trace_ids were the same.
This resulted in Jaeger showing two different requests and their spans are not part of one another. Here's the screenshot

In Jaeger, it ended up showing like this:

Also as a reference, this is my Cargo.toml file:
[package]
name = "temp"
version = "0.1.0"
edition = "2021"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[dependencies]
axum = "0.6.20"
axum-tracing-opentelemetry = "0.13.1"
init-tracing-opentelemetry = { version = "0.13.1", features = [
  "otlp",
  "tracing_subscriber_ext",
] }
opentelemetry = { version = "0.20.0", features = ["rt-tokio"] }
serde_json = "1.0.107"
tokio = { version = "1.32.0", features = ["full"] }
tracing = "0.1.37"
tracing-opentelemetry-instrumentation-sdk = "0.13.1"

Can you point out what am I doing wrong? What do i need to do so that the trace_id becomes the same?
Thank you.

I'm also experiencing a very similar behavior, but I'm exporting the traces to datadog.
The effect is the same:

In this picture, the trace find_account should have a parent span and be inside of resource POST /account.lookup, but it being reported as it own resource probably because this mismatch of trace_id.

@rezmuh , I re-run the example

on 0.13.1, I can reproduce the error
on 0.14.1 (& master) and I got the expected behavior (aka same trace_id, and 1 trace with 2 span in jaeger)

But it seems the crate was not released as expected. I'll republish 0.14.1

@tsroka, which issue do you had with tracing-opentelemetry 0.21 ?

@davidB when i updated opentelemetry to 0.21 (but was still using axum-tracing 0.13.1) i did not see traceparent header in the http response, but did not investigate any further

Hi @davidB I just updated your crate to 0.41.1 and keep opentelemetry crate to 0.20.0. The problem is solved.
Thank you!


Forward ChatMessage

isn' there 1 other place to handle these for remote participants?

isn't there 1 other place to handle these for remote participants? - (*RemoteDataTrack).onData

yeah, it should be on cloud, let me check


add ability to pass already initiated SES object to transport

It would be really good to have a way to inject the ses object (already configured) into the transport. Then ses would't be a direct dependency.

Thats a good idea. I added a ses option and released the module as v1.3.0.


QUERY - How to align it in centre

First of all thanks for the awesome library.
I am creating a sample app with annotated string like this
 val text = buildAnnotatedString {
        withStyle(
            style = SpanStyle(
                fontSize = 90.sp
            )
        ){
            append(subjectText)
        }

        withStyle(SpanStyle(background = MaterialTheme.colorScheme.primaryContainer)) {
            append(tag)
        }
    }

the issue is the tag is not centre aligned.
is there any additional config to do so ? I played around the params of RoundedCornerSpanPainter but nothing works
2nd issue which I am facing it I have to give the topMargin = 45 SP manually else the background is full height

Are the spans centered if you display the text without extended-spans?


Martinvol/fee handler - WIP PR

Description
A few sentences describing the overall effects and goals of the pull request's commits.
What is the current behavior, and what is the updated/expected behavior with this PR?
Other changes
Describe any minor or "drive-by" changes here.
Tested
An explanation of how the changes were tested or an explanation as to why they don't need to be.
Related issues

Fixes #[issue number here]

Backwards compatibility
Brief explanation of why these changes are/are not backwards compatible.
Documentation
The set of community facing docs that have been added/modified because of this change

i think this was superceeded by a different pr


use ack instead of ld64 for no-bottle test

ld64 now has bottles for some old osx versions, ack is currently bottle :unneeded
(hopefully it'll stay that way?)

Consider using ag instead of ack. AFAICT, it's command-line compatible and a bit faster.

This is just for testing something available in homebrew-core that doesn't have a binary bottle. ag looks like an alias to https://github.com/Homebrew/homebrew-core/blob/master/Formula/the_silver_searcher.rb, which appears to be in C so does have a binary bottle. Compare to https://github.com/Homebrew/homebrew-core/blob/master/Formula/ack.rb which is just moving some perl files around.

Ah, ok. Cheers!

Changes Unknown when pulling 947f0a17de496e196a6c48690a3fb713d9eb1b00 on tk/ld64bottle into ** on master**.


feat: [LW-9020] custom staking delegation

Why do we need typescript setup?

My original idea was to migrate a few bits to TS, but it turns out it would slightly increase the scope of this task. At least we have it in place now.

I would vote for not committing the code which is not used. It's not a huge change, so in the future if we decide to migrate this codebase to TS or start using TS for new code, it is not a big deal to configure it.
wdyt?

@lucas-barros @pczeglik-iohk removed the typescript commit.


Unable to delete profile

Required information

Distribution: Ubuntu
Distribution version: xenial
The output of "lxc info" or if that fails:

config: {}
api_extensions:
- storage_zfs_remove_snapshots
- container_host_shutdown_timeout
- container_syscall_filtering
- auth_pki
- container_last_used_at
- etag
- patch
- usb_devices
- https_allowed_credentials
- image_compression_algorithm
- directory_manipulation
- container_cpu_time
- storage_zfs_use_refquota
- storage_lvm_mount_options
- network
- profile_usedby
- container_push
- container_exec_recording
- certificate_update
- container_exec_signal_handling
- gpu_devices
- container_image_properties
- migration_progress
- id_map
- network_firewall_filtering
- network_routes
- storage
- file_delete
- file_append
- network_dhcp_expiry
- storage_lvm_vg_rename
- storage_lvm_thinpool_rename
- network_vlan
- image_create_aliases
- container_stateless_copy
- container_only_migration
- storage_zfs_clone_copy
- unix_device_rename
- storage_lvm_use_thinpool
- storage_rsync_bwlimit
- network_vxlan_interface
- storage_btrfs_mount_options
- entity_description
- image_force_refresh
- storage_lvm_lv_resizing
- id_map_base
- file_symlinks
- container_push_target
- network_vlan_physical
- storage_images_delete
- container_edit_metadata
- container_snapshot_stateful_migration
- storage_driver_ceph
- storage_ceph_user_name
- resource_limits
- storage_volatile_initial_source
- storage_ceph_force_osd_reuse
- storage_block_filesystem_btrfs
api_status: stable
api_version: "1.0"
auth: trusted
public: false
environment:
  addresses: []
  architectures:
  - x86_64
  - i686
  certificate: |
    -----BEGIN CERTIFICATE-----

    -----END CERTIFICATE-----
  certificate_fingerprint: 4d893dd02b8c6bf2bb0ac783faa8c003baa0abac1e61e59258fa9054b7ea1319
  driver: lxc
  driver_version: 2.1.0
  kernel: Linux
  kernel_architecture: x86_64
  kernel_version: 4.4.0-93-generic
  server: lxd
  server_pid: 17046
  server_version: "2.18"
  storage: btrfs
  storage_version: "4.4"

Issue description
I have mistakenly created a profile with the special char "?" and now I´m unable to delete it.
lxc profile list
+---------+---------+
|  NAME   | USED BY |
+---------+---------+
| ?       | 0       |
+---------+---------+
| default | 1       |
+---------+---------+

How can I delete this profile?
lxc profile delete \?
error: not found
lxc profile delete ?
error: not found

lxc delete %3F should do the trick (it's a url-encoded version of ?)

Thank you for your reply. Unfortunately it didn´t work:
lxc delete %3F
error: not found

Oh, sorry, I meant lxc profile delete %3F

I´ve overlooked that too :)
Thx!
lxc profile delete %3F
Profile %3F deleted


TypeError: Member "verifyProof" not found or not visible after argument-dependent lookup in library MerkleProof

🎉 Description
Problem with using MerkleProof with latest solc:
TypeError: Member "verifyProof" not found or not visible after argument-dependent lookup in library MerkleProof

[x] 🐛 This is a bug report.
[ ] 📈 This is a feature request.

💻 Environment

OpenZeppelin 1.7.0
solc 0.4.21

📝 Details
MerkleProof public merkleProof;

function xxx() {
    ...
    bytes proof = ...;
    bytes32 merkleRoot = ...;
    bytes32 leaf = ...;
    require(merkleProof.verifyProof(proof, merkleRoot, leaf));
}

Tried different way:
using MerkleProof for bytes;

function xxx() {
    ...
    bytes proof = ...;
    bytes32 merkleRoot = ...;
    bytes32 leaf = ...;
    require(proof.verifyProof(merkleRoot, leaf));
}

Got:
Error: SimpleContractImpl contains unresolved libraries. You must deploy and link the following libraries before you can deploy a new version of SimpleContractImpl: MerkleProof

Solved by linking library instance to smart contract class:
const merkleProof = await MerkleProof.new();
SimpleContractImpl.link('MerkleProof', merkleProof.address);


Set branch name on build info.

Adds the branch name to the Build info to be pushed to Octopus Deploy.
If the ref starts with refs/heads/ then this will strip that and set the branch name in the build info. If the ref doesn't start with refs/heads/ then this will leave the branch name empty. This should cope with the case where builds are triggered on tags rather than branches.
Solves #17

addressed in fcd857a76efbd21b9919c1be3f9f3713d88d93b1, thanks for the contribution!


Adds deprecated decorate method to preserve backwards compatibility

Adds #decorate(EntitySpec<?>, ConfigBag) as an overloaded deprecated method in BrooklynEntityDecorationResolver interface, for backwards compatibility.
See https://github.com/cloudsoft/brooklyn-tosca/pull/113

one tweak then good to merge


Submit search on enter

What does this PR do and why?
Describe in detail what your merge request does and why.

Standardized Search Component
Search now works when the enter key is pressed, not debounced

Screenshots or screen recordings
Screenshots are required for UI changes, and strongly recommended for all other pull requests.

How to set up and validate locally
Numbered steps to set up and validate the change are strongly suggested.
Ensure that search still works across the project (tested through our integration tests as well)
PR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

[X] I have evaluated the PR acceptance checklist for this PR.

Just through usage on the website, I noticed a few inconsistencies. This is assuming we want the search field to still be populated with the search after hitting enter, and to have focus on the search bar. As well as after leaving the page, the filter would be removed

Projects dashboard - Search field is emptied and not focused after search
Groups dashboard - If I leave the groups dashboard page and come back, the search bar is still populated with the search but the groups are not filtered
Data Exports - Search field is emptied and not focused after search
Samples - Search and filter are still enabled after leaving and coming back to the page
Project Files - Search field is emptied and not focused after search

I also think if the user clicks on the "X" at the end of the search bar, that should return an unfiltered request, thoughts?

Samples table search is expected to be different as we store that server side, but I agree with your remaining comments.

@ChrisHuynh333 & @ericenns Those pages are updated along with tests


Adding .xcodesamplecode.plist to give proper markdown rending inside …

Same contents as my merged PR into swift-crypto and my merged PR into swift-numerics, giving proper markdown rendering inside Xcode.
As seen in Apple's ARKit demo (inside /ARKitExample.xcodeproj/ directory)
Motivation:
Reading of markdown becomes easier. Extra important in Swift packages, where the README is the first file opened by Xcode when opening said package.
Modifications:
Just adding the .xcodesamplecode.plist file (to the root).
Result:
This effectively disables editing markdown files from Xcode - or at least I have not found a way of doing so.
The "workaround" is just to open markdown files with your favourite alternative text editor, e.g. emacs, VIM, VI, Sublime Text or whatever
Before:

After:

This looks great, thanks @Sajjon!


[doc/um] Updated the python requirements run command for sw simple sy…

…stem
Updated the README file for python requirements run command.
@imphil
Signed-off-by: Rahul Raveendran<EMAIL_ADDRESS>

Nice one! This looks great! I'll merge this as soon as the checks pass.
One small detail for your next pull request: keep the first line of the commit message preferably under 50 characters, since it is used as "summary". If you have more text, you can write a longer description below that, e.g. like that:
[doc/um] Fix documentation on Python requirements

Fix the command to install Python dependencies in the simple system.

Signed-off-by: Rahul Raveendran<EMAIL_ADDRESS>
This also helps GitHub to show the summary better, i.e. ensure that there's no "..." visible at

(No need to change anything for this PR, however.)

Nice one! This looks great! I'll merge this as soon as the checks pass.
One small detail for your next pull request: keep the first line of the commit message preferably under 50 characters, since it is used as "summary". If you have more text, you can write a longer description below that, e.g. like that:
[doc/um] Fix documentation on Python requirements

Fix the command to install Python dependencies in the simple system.

Signed-off-by: Rahul Raveendran<EMAIL_ADDRESS>
This also helps GitHub to show the summary better, i.e. ensure that there's no "..." visible at

(No need to change anything for this PR, however.)

Congratulations to getting your first-ever pull request merged, and your first code contribution to Ibex!

Congratulations to getting your first-ever pull request merged, and your first code contribution to Ibex!

Thank you very much Imphil, I will keep these notes for my next pull request.
One doubt I have is that how can I add reviewers to my pull request, I added your name using @ in the description is there any other way to add ?

Thank you very much Imphil, I will keep these notes for my next pull request.
One doubt I have is that how can I add reviewers to my pull request, I added your name using @ in the description is there any other way to add ?

You can select reviewers on the top-right on the PR page (i.e. here). But don't worry about it, we'll see your PR and look at it even without it.

You can select reviewers on the top-right on the PR page (i.e. here). But don't worry about it, we'll see your PR and look at it even without it.

All right, but if you see in the image the reviewers is kind of disabled for me to add. Thats why.

All right, but if you see in the image the reviewers is kind of disabled for me to add. Thats why.

Yep, I think adding reviewers through the interface like that needs some extra permissions. I'm not sure whether we can turn that off. Otherwise, the lowRISC hardware engineers monitor these issues and PRs, so we can always point them at reviewers as needed.

Yep, I think adding reviewers through the interface like that needs some extra permissions. I'm not sure whether we can turn that off. Otherwise, the lowRISC hardware engineers monitor these issues and PRs, so we can always point them at reviewers as needed.

Okay, fine thank you

Okay, fine thank you