PdbReader's GetPdbProperties can throw uncaught UnauthorizedAccessException

There are cases when the pdbFile argument of GetPdbProperties points to a directory (like for example c:\symbols\kernel32.pdb) for example when _NT_SYMBOL_PATH=c:\symbols (w/o anything else).
In this case File.OpenRead(pdbFile) throws an expected but painful UnauthorizedAccessException.
There are many ways to change this, for example:

add this code to GetPdbProperties (simulate the IOException that's documented to happen)

if (Directory.Exists(pdbFile))
    throw new IOException();

add this code to GetPdbProperties (don't throw)

    if (Directory.Exists(pdbFile))
    {
        signature = Guid.Empty;
        age = 0;
        return;
    }

change ValidatePdb and add this code to it:

      catch (UnauthorizedAccessException)
      {
        return false;
      }

I like the exception to bubble to allow calling code/user figure out what's next.
I guess silent 'sorry no' could be of little use...

@mitikov - Of course, I agree when you're calling the method yourself, but there are cases where this method is called implicitly (I don't remember my use case, but I think it was around the symbol locator). In this case, the "user" cannot figure what's next.

@smourier The code is consumed by DefaultSymbolLocator.FindPdb that is sitting inside DataTarget.SymbolLocator property (with public setter) that gives a default implementation of SymbolLocator.
I did not find any code usages OOB as well, so it would be good to know a bit more about the scenario you are trying to achieve.
As for technical aspect - the method description states: Validates whether a pdb on disk matches....
Framework Design guidelines suggest:

Execution failure occurs whenever a member cannot do what it was designed to do (what the member name implies).

It seems we might need to update PdbReader.GetPdbProperties code to catch whatever bubbled and wrap into IOException per code documentation.
As a result the default SymbolLocator impl. of ValidatePDB will catch exception and report validation failed.
The cost is loosing the knowledge why PDB was not loaded (like file access issues in the scenario you've mentioned) in default scenario.
Despite a vew from outside as it looks to be quite easy to overcome the problem in custom code, I guess in reality there could be troubles on the way.
That is why knowing the scenario/use-case is much appreciated :)

Ok, I found my use case back. It's like I said in my first comment:
Environment.SetEnvironmentVariable("_NT_SYMBOL_PATH", @"c:\symbols");
using (var file = new PEFile(@"c:\windows\system32\kernel32.dll"))
{
    new DefaultSymbolLocator().FindPdb(file.PdbInfo);
}

This raises the following error, at least on my machine:
System.UnauthorizedAccessException: Access to the path 'c:\symbols\kernel32.pdb' is denied.
   at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
   at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)
   at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share)
   at Microsoft.Diagnostics.Runtime.Utilities.Pdb.PdbReader.GetPdbProperties(String pdbFile, Guid& signature, Int32& age)
   at Microsoft.Diagnostics.Runtime.Utilities.SymbolLocator.ValidatePdb(String pdbName, Guid guid, Int32 age)
   at Microsoft.Diagnostics.Runtime.Utilities.DefaultSymbolLocator.FindPdb(String pdbName, Guid pdbIndexGuid, Int32 pdbIndexAge)
   at Microsoft.Diagnostics.Runtime.Utilities.SymbolLocator.FindPdb(PdbInfo pdb)
   at PdbDump.Program.SafeMain(String[] args) in D:\SimonMourier\Tfs\PdbDump\PdbDump\PdbDump\Program.cs:line 65
   at PdbDump.Program.Main(String[] args) in D:\SimonMourier\Tfs\PdbDump\PdbDump\PdbDump\Program.cs:line 38

Interesting.  Reading through this and looking at the code, I think PdbReader.GetPdbProperties should throw.  You are asking PdbReader to give you a concrete answer on a single pdb, and it's appropriate to throw UnauthorizedException there.
However, FindPdb should be checking multiple paths and a single unauthorized location shouldn't result in aborting the entire path and throwing.  Looking at SymbolLocator.ValidateBinary it looks like we wrap everything with an untyped catch block, and I think it's appropriate to do the same in SymbolLocator.ValidatePdb.
I don't like untyped catch blocks in general, but in this narrow window when validating a binary it makes some sense.  I worry a little about applications which opt into handling corrupted state exceptions, but since I replaced PEFile with a safer PEImage, that is also less of a concern.


Unreachable code detected on iOS.

When I switch the platform to iOS in UnityEditor, unreachable code is detected (CS0162).
Assets/Scripts/gRPC/Core/Internal/NativeExtension.cs(185,13): warning CS0162: Unreachable code detected
To avoid this warning, I improved the code.

nice contrib, thanks!


Update C# port for number/metadata schema changes <=8.6.0

damn, you seem pretty keen!  I've granted you push access to this repo if you like.  Or we can change yours to the upstream repo, whatever makes sense.  To be honest I didn't actually write this thing in the first place, I just moved it to Git and updated the metadata a few times.  Is your nuget username the same?  I'll just give you access to that too.

Thanks, nuget is the same twcclegg, once I found out that updating the new metadata required significant code changes I kinda wanted it done before there was urgent need for any updates in them. It might make sense to move the main fork, perhaps should let others weigh in. The main thing if we did do that would be to get Google to point the link in their readme.

I've added you as an owner to nuget now too.  Yea, we were probably quite behind on the code changes, I have literally only updated the metadata.

Hi! Could you update a Project Site url on the nugt package page https://www.nuget.org/packages/libphonenumber-csharp? Currently, it is pointing to the old bitbucket repository which is confusing, because the repository has not been updated since 2012, but 12 new nuget packages have been published since then.


Dark and light theme options

I would like to create a button that will the theme of whole page as per requirement of the user.

Actually this option was already there in the Project, but due to alot of irregularities I removed it.
So no need of this option,
You can explore the Project, there's alot more things needs yet to be done, you can check out the ideas video on our slack channel.


Update dotenv.rb

This will overload the values preloaded in the .env .env.development to ensure that .env.development.remote has the highest precedence!

This is a 🐞 bug fix.

[ ] I've added tests (if it's a bug, feature or enhancement)
[ ] I've adjusted the documentation (if it's a feature or enhancement)
[ ] The test suite passes (run bundle exec rspec to verify this)

Summary

Context

Version Changes

So believe this will actually do the opposite 🤔  The precedence should work as it was fixed in #171 by using .reverse. More context: https://community.rubyonjets.com/t/local-and-remote-environment/75/7
Current jets behavior (desirable):
$ cat .env.development
MY_KEY=example1-in-env-development
$ cat .env.development.remote
MY_KEY=example2-in-env-development
$ JETS_ENV_REMOTE=1 jets c
Jets booting up in development mode!
>> puts ENV["MY_KEY"]
example2-in-env-development
=> nil
>> exit
$

Behavior with this change (undesirable):
$ cat .env.development
MY_KEY=example1-in-env-development
$ cat .env.development.remote
MY_KEY=example2-in-env-development
$ JETS_ENV_REMOTE=1 jets c
Jets booting up in development mode!
>> puts ENV["MY_KEY"]
example1-in-env-development
=> nil
>>


add 1.18.2 support

The plugin doesn't work in 1.18.2

Dear author, WE NEED 1.18.2 PLEASE HEEEELP.
Thanks a lot!

@vladmarica any news on the state of this issue?
thanks in advance

Done: https://www.curseforge.com/minecraft/mc-mods/better-ping-display-fabric/files/3790832


Notebook server has no response when wrapper kernel is shutting down

Notebook server has no response because the wrapper kernel sleeps within 5 seconds when it is shutting down.
Steps to reproduce

Open a jupyter notebook with wrapper kernel
Open a terminal on the notebook server
Shutdown the notebook
The terminal does not respond to key press in about 5 seconds

Fixed by https://github.com/NII-cloud-operation/Jupyter-LC_wrapper/pull/67


Logout icon is a "in" icon, not "out"

Bug report
Bunq app:

Top is Schengen zone entrance. Bottom is Schengen zone exit. The logout icon looks the same as the Schengen zone entrance icon. Hopefully this will prove that the icon used in the Bunq app to logout is incorrect.
Latest git version.

The problem is that there is no official logout icon for material so I ended up using something similar at the time. Looks like a simple but useful thing to implement though so I'll replace it sometime 👍

It looks like it's still not fixed....

I can confirm it's still the same icon, on the dashboard and settings page. The material UI icon is called "exit to app" and even is an alias for "logout" when you search for it. Source: https://material-ui.com/components/material-icons/
Maybe using the old-fashioned lock icons could be an alternative as I couldn't find those Schengen-like icons anywhere in the Material Icons set. Lock icons set:

Looking at an alternative, Font Awesome 5 (Free) does provide dedicated login/logout icons: https://fontawesome.com/icons?d=gallery&q=sign in
I'm wondering if it would be better for this project to start using an icon set that does have all the icons available, rather than keep looking for the second-best option all the time. Some icons indeed are confusing and I'm afraid it's due to the limited amount of Material icons available.


Inaccurate schema error when same exists in primary Metastore

Assume there is a schema in primary metastore: pm_mydb
In different hive metastore, there is schema named as mydb_another.
When configuring waggle dance, we have provided prefix for external metastore as pm_.
When we ran show databases, we get following result
hive> show databases;
pm_mydb
pm_mydb_another

After this when we tried to execute query against the pm_mydb, schema within primary MS, I am getting error
hive> use  pm_mydb;

ERROR: FAILED: SemanticException [Error 10072]: Database does not exist: pm_mydb
...

When I changed the prefix to pm2_ for external MS, everything works fine.
hive> show databases;
pm_mydb
pm2_mydb_another

hive> use  pm_mydb;
LIST_OF_TABLES

It looks like that waggle dance is resolving pm_mydb as external schema (mydb schema under the MS with pm_ as prefix).

This is expected behaviour. Waggle Dance uses the prefix to redirect to the external metastore. The pm_ prefix is configured to go to the federated metastore. Which is in turn tell you that pm_mydb (or mydb in the external MS) doesn't exist.
This is a weird situation I suspect because the database was created bypassing Waggle Dance.
Best to pick a prefix that doesn't clash with existing databases. (or rename your database)
If this is not an option there is the alternative to switch to MANUAL database resolution mode (README has more details). This doesn't use prefixes.

Closing this issue for now as it's expected behaviour. If we see users reporting this we could update the documentation to be clear that one shouldn't choose a prefix that matches the prefix of a primary database name.

This is expected behaviour. Waggle Dance uses the prefix to redirect to the external metastore. The pm_ prefix is configured to go to the federated metastore. Which is in turn tell you that pm_mydb (or mydb in the external MS) doesn't exist.
This is a weird situation I suspect because the database was created bypassing Waggle Dance.
Best to pick a prefix that doesn't clash with existing databases. (or rename your database)
If this is not an option there is the alternative to switch to MANUAL database resolution mode (README has more details). This doesn't use prefixes.

All the databases were created historically and when waggle-dance was configured, we chose the prefix according to best suitable meaning.
This caused us downtime on 1 of the cluster and took us sometime to figure out root cause and fix the prefix in order to make system up again.
It will be great to make it explicit in documentation as anyone can hit this issue (knowingly or unknowingly).

@ambrish29 I've had a go at adding this to the wording, it's a bit hard to explain but let me know if you think this helps: https://github.com/HotelsDotCom/waggle-dance/pull/138 (feel free to add comments on the PR if you're able to, otherwise put them here below).

@massdosage PR looks good to me.


You can run the application...

I am confused, I read the instructions for the Linux containers, did that, skipped over the Windows part, and landed on "you can use dotnet publish" and I don't understand what that has to do with the containers. What I would have wanted to see here is more information about how to configure the containers for production. Maybe I missed something.

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

ID: 9731aae1-505b-abb1-0d4b-d6a83ec735e1
Version Independent ID: 760790d5-0e81-ea6c-19ea-f226d08ddf1f
Content: Docker images overview - .NET Core
Content Source: docs/core/docker/building-net-docker-images.md
Product: dotnet-core
GitHub Login: @mairaw
Microsoft Alias: adegeo

Yes, this is very odd and feels out of place. I'll add this to the docker project. Thank you for bringing it up.

I just published an update to this article that completely rewrites it. I tried to make it generic enough for windows or linux. If you look at the article you may be redirected to the overview. That means the article hasn't been published yet, so try again later.
Let me know if this helps.
Cheers!

You know, it has just been so long since I looked at this that I can't even remember what was wrong in the first place. But the process you describe seems to fit with what we did:
https://github.com/chaim1221/Machete/blob/master/dockerize.sh
https://github.com/chaim1221/Machete/blob/master/Dockerfile
...so that's good!

I know.. sorry it takes time to get to these :) But your issues help us figure out ways to improve, so thanks :) :) :) :)
There was another issue with dotnet publish and I think that whole section of the original article was confusing. I simplified that part greatly.


gcp-pubsub-subscriber thread starts on PubSubTemplate creation; never stops

This is a continuation of #1503. There is also a similar (but unrelated, as Spring Cloud GCP creates its own SubscriberStub) issue in the client library googleapis/google-cloud-java#6570.
PubSubSubscriberTemplate constructor creates a SubscriberStub, which starts at least one thread. The SubscriberStub is never closed. For a web application, this is not noticeable, but for a terminal app that expects JVM to close normally when its work is done, the result is a hanging app.
I thought we've fixed this in #1503, but now I don't know how that solution (making schedulers spring managed beans) ever worked. Spring's bean destruction happens when on JVM shutdown hook, and JVM won't shut down while there is still a non-daemon thread running.
There are solutions of varying invasiveness to this:

Document the need to shut down the executor manually, as in this comment.
For sure at least call this.subscriberStub.close() in the destroy() method in PubSubSubscriberTemplate, so it cleans up properly for a web app.
(this one needs thought and analysis of how/when these threads are created) Work with the client library to avoid starting the threads unless .subscribe() is called. Are the threads created only for the streaming pull? Or are they also used for blocking operations?

I think the problem is that PubSubSubscriberTemplate.subscriberStub is not a Spring-managed bean. So, it's not automatically closed when the context shuts down.
The simplest (and proper) thing to do would be to call this.subscriberStub.close() in the destroy() method in PubSubSubscriberTemplate. Exactly as you're suggesting in (2).


Issue while building fuzi_q with latest picoquic

Noticed below issue due to below changes
-- Build files have been written to: /root/fuzi_q
root@ubuntu:~/fuzi_q# make
[  7%] Building C object CMakeFiles/fuzy_q_core.dir/lib/fuzzer.c.o
/root/fuzi_q/lib/fuzzer.c: In function ‘frame_header_fuzzer’:
/root/fuzi_q/lib/fuzzer.c:542:26: error: ‘picoquic_frame_type_ack_mp’ undeclared (first use in this function); did you mean ‘picoquic_frame_type_ack_ecn’?
  542 |                     case picoquic_frame_type_ack_mp:
      |                          ^~~~~~~~~~~~~~~~~~~~~~~~~~
      |                          picoquic_frame_type_ack_ecn
/root/fuzi_q/lib/fuzzer.c:542:26: note: each undeclared identifier is reported only once for each function it appears in
/root/fuzi_q/lib/fuzzer.c:543:26: error: ‘picoquic_frame_type_ack_mp_ecn’ undeclared (first use in this function); did you mean ‘picoquic_frame_type_ack_ecn’?
  543 |                     case picoquic_frame_type_ack_mp_ecn:
      |                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |                          picoquic_frame_type_ack_ecn
make[2]: *** [CMakeFiles/fuzy_q_core.dir/build.make:76: CMakeFiles/fuzy_q_core.dir/lib/fuzzer.c.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:90: CMakeFiles/fuzy_q_core.dir/all] Error 2
make: *** [Makefile:91: all] Error 2

Commit caused the issue is https://github.com/private-octopus/picoquic/commit/79c1bb18117ec4b7069462cec6a67ec17dd186d8

https://github.com/private-octopus/fuzi_q/pull/31

Commited PR
https://github.com/private-octopus/fuzi_q/pull/31


fix: Update context.py

Description

Testing

Additional context

Codecov Report

Merging #94 (f957608) into main (d5cf49e) will increase coverage by 0.25%.
The diff coverage is 40.00%.

@@            Coverage Diff             @@
##             main      #94      +/-   ##
==========================================
+ Coverage   93.26%   93.52%   +0.25%
==========================================
  Files          14       14
  Lines         490      494       +4
  Branches       75       77       +2
==========================================
+ Hits          457      462       +5
  Misses         24       24
+ Partials        9        8       -1

Flag
Coverage Δ

unittests
93.11% <40.00%> (+0.26%)
:arrow_up:

unittests-3.10
88.86% <40.00%> (+0.29%)
:arrow_up:

unittests-3.11
88.86% <40.00%> (+0.29%)
:arrow_up:

unittests-3.8
88.86% <40.00%> (+0.29%)
:arrow_up:

unittests-3.9
88.86% <40.00%> (+0.29%)
:arrow_up:

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

Impacted Files
Coverage Δ

src/gpt_review/context.py
63.63% <40.00%> (+3.63%)
:arrow_up:

... and 1 file with indirect coverage changes


spread syntax differs between swc loose mode and babel loose mode

Describe the bug
I'm migrating my code's compiler from babel to swc, and my code crashed because the generated code with loose config is kind of different from babel.
Code contains es spread syntax. While babel with loose config removes all TypeError and other stuff, swc with loose config remains the same as non-loose config. TypeError is the key reason of my code crash.
Input code
const data = {}
const temp  = [...data.demo, ...data.demo2]

Config
No response
Playground link
babel
https://babeljs.io/repl#?browsers=&build=&builtIns=false&corejs=3.21&spec=false&loose=true&code_lz=MYewdgzgLgBAJgQygmBeGBvAvgKFJWKAUwFsAHGNGAbQDp7Fla5SQAaGe5pBZ1gJgC6QA&debug=false&forceAllTransforms=false&modules=commonjs&shippedProposals=false&circleciRepo=&evaluate=false&fileSize=false&timeTravel=false&sourceType=module&lineWrap=true&presets=env%2Creact%2Cstage-2&prettier=false&targets=&version=7.21.8&externalPlugins=&assumptions={}
swc
https://play.swc.rs/?version=1.3.56&code=H4sIAAAAAAAAA0vOzysuUUhJLElUsFWoruVKBvNLUnMLFIAC0Xp6eiA5vZTU3HwdBWSeUSwALJOS2TsAAAA%3D&config=H4sIAAAAAAAAA1WPPQ7DIAxGd06BPHdth94hh7CoExHxJ0ykoih3LyFA2w2%2Fz%2FYzu5ASVlbwlHt5liJgZIqjLoSzS%2FguBEhZZBV1SHDr6cpnNKNhqui4EkgYF0p1iu%2BtHYz3TIWluFFDVjs951%2Bf8jZEYu5rh8qiWwz920QzgvWvrYbtHykHuuwP%2BDZ12VgMmqc%2BeV4ljg%2F3WmjeEwEAAA%3D%3D
Expected behavior
Using es spread syntax in loose config, swc behaves the same as babel.
Actual behavior
No response
Version
@swc/core: 1.3.56 @babel/core: 7.21.8
Additional context
No response

Hello @kdy1 , hope everything goes on well. Could you kindly update me on this issue's status? And if is it possible for me to make PR for this issue?

There's no update, and yes, of course! I'm open to a PR for fixing this


Enable debug tools in noc_nonblocking_api.h

Issue here is that dprint/noc_sanitize/assert don't work in specifically noc_nonblocking_api.h functions. This is because the debug tools need to call functions from this header in risc_context_switch(), circular dependency if included in noc_nonblocking_api.h. We should find a way around this.
Workaround for now is inlining the required code into the caller level and calling debug tools there.

Required for #7844 so bump to p1


Unable to support Chinese input

2024-03-07 11:52:52 INFO:     <IP_ADDRESS>:53560 - "GET /api/v1/chats/ HTTP/1.1" 200 OK
2024-03-07 11:52:52 http://host.docker.internal:11434
2024-03-07 11:52:52 {"model":"gemma:latest","messages":[{"role":"user","content":"hellow"},{"role":"assistant","content":""}],"options":{},"stream":true}
2024-03-07 11:52:52 INFO:     <IP_ADDRESS>:53560 - "POST /ollama/api/chat HTTP/1.1" 200 OK
2024-03-07 11:52:53 INFO:     <IP_ADDRESS>:53560 - "POST /api/v1/chats/fd65bd23-3113-4c25-b3f9-64edd1dc2e9a HTTP/1.1" 200 OK
2024-03-07 11:52:53 INFO:     <IP_ADDRESS>:53560 - "GET /api/v1/chats/ HTTP/1.1" 200 OK
2024-03-07 11:52:53 http://host.docker.internal:11434
2024-03-07 11:52:53 INFO:     <IP_ADDRESS>:53560 - "POST /ollama/api/generate HTTP/1.1" 200 OK
2024-03-07 11:52:53 INFO:     <IP_ADDRESS>:53560 - "POST /api/v1/chats/fd65bd23-3113-4c25-b3f9-64edd1dc2e9a HTTP/1.1" 200 OK
2024-03-07 11:52:53 INFO:     <IP_ADDRESS>:53560 - "GET /api/v1/chats/ HTTP/1.1" 200 OK
2024-03-07 11:52:53 INFO:     <IP_ADDRESS>:53560 - "GET /api/v1/chats/ HTTP/1.1" 200 OK
2024-03-07 11:53:07 http://host.docker.internal:11434
2024-03-07 11:53:07 {"model":"gemma:latest","messages":[{"role":"user","content":"hellow"},{"role":"assistant","content":"Hello, how are you?\n\nI'm an AI language model, here to help you with your queries. Would you like me to tell you more about myself or answer a question?"},{"role":"user","content":"介绍一下JavaScript"},{"role":"assistant","content":""}],"options":{},"stream":true}
2024-03-07 11:53:07 INFO:     <IP_ADDRESS>:54364 - "POST /ollama/api/chat HTTP/1.1" 500 Internal Server Error
2024-03-07 11:53:07 INFO:     <IP_ADDRESS>:54364 - "GET /api/v1/chats/ HTTP/1.1" 200 OK

Belongs here: https://github.com/ollama/ollama/issues

This commit removed the use of \x encoding for Unicode characters in the JSON body passed to the requests.request() function call: https://github.com/open-webui/open-webui/commit/f04d60b6d954ee4e68805240f19ea044028e4dba
Before:
b'{"model":"****","messages":[{"role":"user","content":"\xe4\xbd\xa0\xe5\xa5\xbd"},{"role":"assistant","content":""}],"options":{}}'
After this commit:
b'{"model":"****","messages":[{"role":"user","content":"你好"},{"role":"assistant","content":""}],"options":{}}'

The latest version should fix the issue, please let us know if the issue persists! Thanks!

It works well now and this thread shall be closed 👍


[opentitantool] Fix intermittent SPI issues

HyperDebug supports controlling multiple SPI busses through a single USB interface, a "select SPI bus" command is meant to be sent before any batch of data transfers.
When a separate function for exposing SPI EEPROM functionality was added, this call to select spi bus was accidentally omitted.  This meant that the working depended on the correct SPI bus already being selected "by accident", which unfortunately happened frequently enough that the bug was not immediately apparent.
This CL adds the call similarly to how it is done for non-EEPROM SPI communication.

This change affects only the hyperdebug transport which is not used in CI yet.  The CI failures are known to be a result of the change of the USR_SPI0 eeprom on the cw310 board.  Force-merging.


Improved auto mode frequency selection

I have been experimenting with using auto mode with a fairly tight carrier frequency range (max - min < 100 Hz) to randomly get low frequency separations with strong triphase effect. However, when the left and right carrier frequencies are exactly equal, the result can either cancel out, or add together to be very strong. I recommend rounding frequencies to a finer grid (0.1 Hz or 0.2 Hz) and selecting another frequency if the two chosen frequencies are exactly equal.

I'm sorry it took me such a long time to get back to you @DrArcularis . Do you think you might be able to send a pull request with this change? It sounds intriguing!

Yes, I've been busy here too, but I just coded up a change and submitted a pull request. Let me know if you'd like to discuss further.

Fixed by PR #20


message_edit: Stop stream_header_colorblock from overlapping others.

Merged, thanks @amanagr!

Merged, thanks @amanagr!


Send2UE: AnimationCurveNodeWrapper has no attribute 'add-keyframe'

Addon: Send to Unreal
Steps to Reproduce the Problem: I clicked Pipeline > Export > Send to Unreal
Error Log: See below
OS: Windows
Addon Version: Send to Unreal 2.4.3
Blender Version: Blender 4.01
Unreal Version: Unreal 5.03

Something is different in 4.01 - a lot of breaking changes with the Python API, I'm seeing it in other addons. But also this python error isn't showing up in the error log, I'm only finding it in the popup which I grabbed an image of
The relevant error is in core/io/fbs.py line 159
AnimationCurveNodeWrapper has no attribute 'add-keyframe'. Did you mean 'set_keyframes'?

Why is this still a problem 2 months later. Can we get a review and merge of the fix asap. @james-baber or @iigindesign

btw fix worked for me @universalconquistador  thanks!

Why is this still a problem 2 months later. Can we get a review and merge of the fix asap. @james-baber or @iigindesign

The lack of support here is quite disheartening. :/
@ryan-hummer Are you able to assist in merging the fix?


Use of .prop() instead of .attr() for setting radio button checked.

On older browsers, radio buttons were not set.
See http://stackoverflow.com/questions/6432246/radio-buttons-and-attrchecked-checked-does-not-work-in-ie7

Even though this is not quite true and attr works fine with IE 6/7 (the root cause of the aforementioned issue was incorrect input declaration), it is still a valuable change.
Thanks for the contribution!


try to make bulk mode work on bad operating systems

I think I left feedback in discord a while back but this works great on windows.


roboteam_ai crashes randomly

Describe the bug
While running grSim, roboteam_world, and roboteam_ai, it can happen that roboteam_ai suddenly gives a segmentation fault. Sometimes it happens after 15 minutes, sometimes after 30 minutes.
To Reproduce

Start grSim, roboteam_world. Start roboteam_ai in debugger mode.
Do nothing for a long time
Possibly observe a crash. Last time, the debugger pointed to an error in Visualizer::drawRobots. However, I couldn't pinpoint what caused the crash.

Ah i noticed this issue in the 'fix iomanager' PR. Thought it was my fault. It is fixed over there. the problem is that the interface runs in a different thread and is slow sometimes with drawing the robots on screen. The crash happens when the robot object is removed from memory while the interface is still trying to draw it.
Note: the fix there is not elegant: it increases the history size with factor 10 to keep the robots alive 10 times longer....
fixed in #802


[Feature Request] timer_stop

I've played with Time Sources, planning to switch to them, but I think FuwaFuwa is better.
I would like to know if adding a timer_stop(timer, runScript) method is possible?
I create a timer when I enter a state and destroy it when I leave this state, but I would like to just stop the timer without destroying it, so I can just start it again when I enter the state again.
Allowing to trigger the attached script or another callback when I do would be great, too.
What do you think?

I'm glad this library is still helpful! Just to clarify, what's the expected difference in behavior from timer_pause (outside of the script-running part)? Is it that it would reset progress like in timer_restart in addition to keeping the timer paused, or...? In that case it can be done by calling them sequentially, but obviously that's kinda inconvenient and could be rolled into a single function since timer reuse should be easy. Script-running on resume should be doable too.
Either way, if a timer_stop is added it'd probably be best to rename timer_unpause to timer_resume so the name makes more sense.

Oh yes it is still helpful : timer_append and timer_set_duration for the win !
My use case is that I use an irandom_range function as the time parameter and a script that append this same function with timer_appendand then destroy the timer once I leaved the state.
With GM's Time Sources, you can't do that. I need to check if the Time Source  expired, then generate a new time in a variable with irandom_range, use the painful time_source_reconfigure to apply the new time which need also to set again all the same other parameters you set the first time with time_source_create and then start the time source again with time_source_start. The only usable thing was that I could call tiler_source_reset or timer_source_stop when I'd leave the state.
So that's just what I miss in fuwafuwa.
Regarding the behaviour, yes, timer_stop would reset the timer like in timer_restart and immediately paused it like in timer_pause, trigger the callback script if you want.
You then would call timer_unpause to run it again, but yeah, it should be rename to timer_resume I guess, or use timer_restart instead ?
True that resume, restart, start, stop and pause can be confusing because that all more or less the same XD
Another thought : to be consistent with timer_stop, maybe renaming timer_create_paused to timer_create_stopped would make more sense ? Then you use timer_resume to run it (the former timer_unpause)
I also noticed that Time Sources dissociated Reset and Stop : reset work on an expired time source so you can start the time source again while Stop reset only the time but not the repetition so if the time source was already expired, it won't go back to its original state. Maybe that's something to get inspiration of for the naming convention XD

🥳

Thank you \o/


Hidden field getting focus with Tab during Add attachments

🐛 Bug Report
Unknown field getting focus while pressing Tab during Add attachments
🤔 Expected Behavior
After Select files, next visible button is 'Cancel' button and should be highlighted
😯 Current Behavior
After Select files, user has to press 'Tab' two times to get focus to cancel button

💁 Possible Solution

🔦 Context
Step 1: Click on 'Add attachment' button
Step 2: Click on Discussion header with mouse
Step 3: Now press 'Tab' on keyboard (2 times)
Step 4: After 'Select files' is highlighted, press 'Tab'

💻 Code Sample

🌍 Your Environment

react-spectrum
3.15.1

Browser
Chrome 94.0.4606.81

Operating System
MacBook Pro

🧢 Your Company/Team

🕷 Tracking Issue (optional)

Hey! thanks for the issue. Did you mean to attach a codesandbox or link to an example where we can reproduce your steps?

As there are inter dependencies I cannot reproduce this issue in codesandbox. You can check this issue here: https://stage.adminconsole.adobe.com/F1AB5BD45908E2FD0A494121@AdobeOrg/support
Steps to reproduce the issue:
Step 1: Login to Admin Console and enable FF 'temp_react_port_support_ticket_add_discussion_item'
Step 2: Open any support case
Step 3: Click on 'Add attachment' button
Step 4: Click on Discussion header with mouse
Step 5: Now press 'Tab' on keyboard (2 times)
Step 6: After 'Select files' is highlighted, press 'Tab'
Expected result: After Select files, next visible button is 'Cancel' button and should be highlighted
Actual result: After Select files, user has to press 'Tab' two times to get focus to cancel button

Please do not post any links to adobe in this issue as it is public. If you need more help, please reach out in Slack.

Is there codesandbox to demonstrate this issue with pure React Spectrum outside of an application?

Closing. We can reopen if a reproduction is shared.


hardwareBackPress is not working in Webview

I have a webview app. I want to active harwarebackpress on my webview. But my codes are not working.
 webView = {
    canGoBack: false,
    ref: null,
  }

  onAndroidBackPress = () => {
    if (this.webView.canGoBack && this.webView.ref) {
      this.webView.ref.goBack();
      return true;
    }
    return false;
  }

  componentWillMount() {
    if (Platform.OS === 'android') {
      BackHandler.addEventListener('hardwareBackPress', this.onAndroidBackPress);
    }
  }

  componentWillUnmount() {
    if (Platform.OS === 'android') {
      BackHandler.removeEventListener('hardwareBackPress');
    }
  }

<WebView
            source={{ uri: website }}
            style={Platform.OS === 'ios'?styles.webviewsios:styles.webviewsandroid}
            automaticallyAdjustContentInsets={false}
            javaScriptEnabled={true}
            domStorageEnabled={true}
            decelerationRate="normal"
            startInLoadingState={true}
            useWebKit={true}
            ref={(webView) => { this.webView.ref = webView; }}
            onNavigationStateChange={(navState) => { this.webView.canGoBack = navState.canGoBack; }}
            />

Need more details, nothing happens? It's crashing?

Works as expected here. As a note to @mhmtsrfglu: Some SPA websites mess up with the WebView history.
To narrow the problem down: Check with a breakpoint if your line this.webView.ref.goBack(); is reached at all.

Need more details, nothing happens? It's crashing?

nope. There is no error, or no crash. Even not log to console anything

I'm using webview-10.8.3, and it will trigger onNavigationStateChange when go forward in webview, but don't trigger when go backward.

I'm using webview-10.8.3, and it will trigger onNavigationStateChange when go forward in webview, but don't trigger when go backward.


Bar color hover

Bar chart color and hover options are now implemented like other plots and are available in the reference app and example code. Fixes #65.

This LGTM, but is there a reason not to use the iris data to drive the example (by using, e.g., species name)?

You cannot make a true bar chart unless you have field with a unique value for each row to use as the x value. Otherwise some bars will overlap with each other. The iris data has no such field.

I've thought about enforcing this as a field attribute ("unique" or something) and restricting the x value to that - but I suppose there might be cases where you want the overlapping bar behavior. We could also always provide an "index" field that is the column number to use as a unique field whenever one is needed. For LineUp I already needed to construct such a field.


overflow:auto breaks scrollTo?

I have added overflow:auto to the container that surrounds everything (to ensure that my menu is scrollable on touch devices) and this seams to break scrollTo, since I no longer is scrolled!
You can see the behavior here: http://kokensupport.varoystrand.se/DEV/content/
Removing overflow:auto from the main() and scrollTo works as expected.

I'm also having issues with this. I've confirmed that @BlackSkorpio is correct.

Hi sorry for the delay, scrollTo should work everywhere a scrollbar would show.
Set overflow auto to the inner element, the one you intent scrollTo to work. If it doesn't show a scrollbar, then that needs to be fixed using CSS in some way.


completion test for string union type

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[x] Y

re #1391
2. What is the scope of this PR (e.g. component or file name):

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[ ] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[x] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
55057

Relevant Lines:
77431

💛  - Coveralls


Error when building in parallel: sh: 1: cargo: not found

When running cross in multiple instances at once, the following error shows up:
sh: 1: cargo: not found

If this is too difficult to fix, there could be an improved error, or a warning.

I'm getting this error on GitHub action, I didn't run it in parallel

I'm getting this error just with a plain cross build as well.  Running NixOS.  Seemingly can't use cross because of this error.

It looks like this might be related to (or the same as) #260

If you're running NixOS, then yes it should be a duplicate of #260, but I don't believe the original error is. I'll see if I can replicate this in parallel.

This seems to be a more general Docker issue. Try the following and Docker will deadlock:
for i in {1..5}; do
    docker run -it --rm ubuntu:20.04 bash -c "ls" &
done

I can't replicate this anymore, and cross doesn't use global state and only calls cargo and docker which have good consistency mechanisms.


Stabilizing the CI process

Some steps that could be taken for (further) stabilizing the CI process:

Pin the version of the Docker base image under test
Bake/use an intermediate parent image after apt-get update (or equivalent for package manager)

Certain integration tests check the version of installed packages (e.g. Apache) and this could help avoid forcing those updates into otherwise unrelated changes. I've encountered this several times when submitting PRs. Updating the images/packages under test would be relegated to changes intended for that purpose. Logical separation of what is being contributed/tested/reviewed is always nice for maintainers since changes should be small and to-the-point.
This is just food for thought.

I had an initial attempt at this.
See:

https://github.com/aelsabbahy/goss/blob/master/integration-tests/test.sh#L19
And the scripts in the development folder.

Was based on the md5sum of the docker file.
The main issue is, given enough time the package no longer exists on the official repos.

Side note, thanks for all your contributions so far. Much appreciated!

I think I understand the mechanics of that now. I'd say that's probably "good enough". This is the unfortunate reality of integration testing in general.
In my particular case (i.e. all the wheezy apache package updates), it's at the mercy of the external apt repository. I wonder if it's worth using a different package as a guinea pig on wheezy. This is assuming that said package's version/release management is more sane and earlier releases aren't completely obliterated.

Side note, thanks for all your contributions so far. Much appreciated!

I'm a fan of the project! It's my pleasure.


Updating Node.js SDK to 2.1.0

/cc @danderson00 for visibility. Merging this in to get a build out.


Readme typo error

On the section Technologies (IDE) why IDE in parenthese?
I think  you don t need this IDE help us to:

Neteans IDE  and  this IDE help us to should  not be to separated.

The under section should be  removed. you just need only one (empty circle should be removed)
**#Client Side??
Installation section, i don t understand?

The IDE is in parenthese because i use it in some part of the project and not on all.

User don't need to install any thing. they just have to click on the link that i will give at the end of the project. this link will direct them on the software. there the will start use it.


Exposed completion commands as CLI commands solving #1329

Exposed completion commands as CLI commands solving #1329

Coverage remained the same at 100.0% when pulling 2e259b7667a131a2db809549cfa63dd977dab6c3 on waldekmastykarz:completion-cmds into f6a983c6ab5944ab3e845542f25fb5690767c97f on pnp:dev.

Merged manually


The Monitor the blockchain link leads nowhere

What

At Node Contribute - Clicking the Monitor the blockchain link will take you to https://github.com/rsksmart/rskj/wiki/Monitor-the-blockchain, which only contains a circular reference telling the user that This page was moved: https://developers.rsk.co/.

Why

If we have a tutorial on how to monitor the blockchain, then we should probably link to it here. Otherwise, this link is just confusing.

Refs

Closed by #189


ocaml_4_03: init at 4.03.0

Motivation for this change
flow needs ocaml 4.03 on Mac OS X. Copied nix expression for ocaml 4.02 and only changed the version.
Things done

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

[ ] NixOS
[x] OS X
[ ] Linux

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

I already got a waiting PR doing this (and a few other updates) waiting for a review or a merge.
Other than that, the ocamlbuild tool was part of the compiler distribution until 4.02, so if we add the 4.03 version, it should be added as a separate package (like in ce90c9027482175d7bc1ff9b51088dc008a9c02c).


Permission error to install tsd-s3cmd

Hi,
I was trying to install  tsd-s3cmd in nird.sigma2.no pip3 install https://github.com/unioslo/tsd-s3cmd/archive/v0.1.1.zip but getting permission error.
PermissionError: [Errno 13] Permission denied: '/usr/local/lib/python3.6'
Any help?

Hi Zillur
Try pip3 install --user https://github.com/unioslo/tsd-s3cmd/archive/v0.1.1.zip

Thanks. It works.


Fossa integration

What changed?
Add fossa integration

Why?
Dependency analysis required by Uber OSS

How did you test it?
Buildkite build should pass.

Potential risks
NA

Coverage increased (+0.003%) to 68.986% when pulling 1565d45e5034c411c334b728d1e447830e1029a1 on meiliang86:fossa into c23cd657e0e10ffd5b53c5f84bee6d57bf0ee525 on uber:master.


TypeError: Cannot read property 'name' of null

Version 7.20.4 seems to crash with the following error in a non-tsx file.
yarn run v1.22.4
$ prettier packages/core/src/**/*.less --check && eslint packages --ext .js,.ts,.tsx --cache
Checking formatting...
All matched files use Prettier code style!

Oops! Something went wrong! :(

ESLint: 7.5.0

TypeError: Cannot read property 'name' of null
Occurred while linting /home/runner/work/exam-engine/exam-engine/packages/cli/src/commands/create-mex.ts:23
    at Object.getStatelessComponent (/home/runner/work/exam-engine/exam-engine/node_modules/eslint-plugin-react/lib/util/Components.js:692:45)
    at Object.getParentStatelessComponent (/home/runner/work/exam-engine/exam-engine/node_modules/eslint-plugin-react/lib/util/Components.js:728:42)
    at Object.getParentComponent (/home/runner/work/exam-engine/exam-engine/node_modules/eslint-plugin-react/lib/util/Components.js:623:18)
    at Object.ArrowFunctionExpression (/home/runner/work/exam-engine/exam-engine/node_modules/eslint-plugin-react/lib/util/Components.js:916:31)
    at updatedRuleInstructions.<computed> (/home/runner/work/exam-engine/exam-engine/node_modules/eslint-plugin-react/lib/util/Components.js:970:43)
    at /home/runner/work/exam-engine/exam-engine/node_modules/eslint/lib/linter/safe-emitter.js:45:58
    at Array.forEach (<anonymous>)
    at Object.emit (/home/runner/work/exam-engine/exam-engine/node_modules/eslint/lib/linter/safe-emitter.js:45:38)
    at NodeEventGenerator.applySelector (/home/runner/work/exam-engine/exam-engine/node_modules/eslint/lib/linter/node-event-generator.js:254:26)
    at NodeEventGenerator.applySelectors (/home/runner/work/exam-engine/exam-engine/node_modules/eslint/lib/linter/node-event-generator.js:283:22)

I haven't been able to produce a reduced test case yet.

+1

I have same problem.
`javascript
// error:
export default function () {}
// success
export default function name() {}
`

Version 7.20.3 works fine.

Seems to be caused by https://github.com/yannickcr/eslint-plugin-react/pull/2696

Ah, I see someone reduced it already. An easy null-check on the node.id.name before it's used will fix this issue. Here's a full testcase:
odin@kaa:~/tmp/a$ cat default_export_without_name.js
export default function() {}
odin@kaa:~/tmp/a$ cat package.json
{
  "scripts": {
    "test": "eslint default_export_without_name.js"
  },
  "dependencies": {
    "eslint": "^7.5.0",
    "eslint-plugin-react": "^7.20.4"
  }
}
odin@kaa:~/tmp/a$ cat .eslintrc
{
    "env": { "es6": true },
    "plugins": ["react"],
    "parserOptions": { "sourceType": "module" },
    "extends": [ "plugin:react/recommended" ],
    "settings": { "react": { "version": "latest" } }
}

today ,I have same problem. +1

same problem here

Same problem :-)

Instead of commenting, just click the :+1: on the top issue instead. Better for everyone ^_^
You may also help out with the PR at #2730 if it turns out anything is needed. Though currently just waiting on review from @ljharb. :)

Thanks everyone; I'll try to get a fix out tonight.

+1
And version 7.20.3 works


docs: updating use cases for ngMocks.get

Currently ngMocks.get description could be understood to only concerns structural directives or services but it can also be used for components or common directives declarations word is too vague and can be more precise.
Consequently we may be tempted to use ngMocks.findInstance which mention components, directives and so on in the documentation. But in some cases we could have false positive tests (using ngMocks.findInstance rather than ngMocks.get), so documentation update is needed for ngMocks.get.
@satanTime let me know if the doc update looks great or if something else could be made event better.

Looks great! Thank you for contributing!

v14.3.2 has been released and contains a fix for the issue. Feel free to reopen the issue or to submit a new one if you meet any problems.


Confirmation of Git Tutorial

Hello,
I tried my best to do the homework (forked the website, did the tutorial, posted a screenshot). Please let me know if there is a problem in the visualization of the screenshot or if anything did not work out as it should have.
Best,
Erika Unterpertinger

I cannot accept this commit, since it will break our website. Check: https://tshjortile.github.io/
You need to make another fork --- do not change there anything other than adding your posts!
if you want to create your own website, you need to create a repository (tshjortile.github.io) from scratch — you can then copy all the files from our course website repository and there you will need to change _config.yml so that everything corresponds to your own details.


Don't wrap IdleInhibitorV1's destroy event data

According to wlroots' emitter for this signal, it does return the idle
inhibitor's struct wlr_surface*. However, in practice the pointed-to
data seems to be invalid, causing a crash (e.g.
https://github.com/qtile/qtile/issues/3361).
The data does not seem to be used by any compositor so until it is
needed we can probably get away without the wrapper, resolving the
problem.

Nice catch, thanks!


Some minor things

Some minor things that were clogging my todo list :)
@gligneul, can you confirm if it is OK to remove that return value from symtab:close_block?

Codecov Report

Merging #116 into master will decrease coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #116      +/-   ##
==========================================
- Coverage   92.34%   92.32%   -0.02%
==========================================
  Files          11       11
  Lines        2182     2177       -5
==========================================
- Hits         2015     2010       -5
  Misses        167      167

Impacted Files
Coverage Δ

titan-compiler/symtab.lua
86.66% <ø> (-0.84%)
:arrow_down:

titan-compiler/coder.lua
94.78% <100%> (ø)
:arrow_up:

titan-compiler/parser.lua
95.49% <100%> (-0.12%)
:arrow_down:

titan-compiler/types.lua
90.16% <100%> (ø)
:arrow_up:

titan-compiler/checker.lua
88.19% <100%> (ø)
:arrow_up:

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 5d99abb...ba3897b. Read the comment docs.


info required

where I can find information about play a stream created by:
stream = file.createReadStream([opts])
I want to play this stream directly into my player, to download files faster.

file.createReadStream() returns a node.js Readable stream. Documentation available here: https://nodejs.org/api/stream.html#stream_readable_streams
This returns the raw video data. It's not going to be easy to pipe that straight into a video player. That's why we created the file.renderTo() and file.appendTo() methods. There's a lot of work required to process the video file into a format that can be streamed into the video tag (using the MediaSource API, which has very strict requirements about the packaging of the video).
Handling seeking is also a difficult challenge – you have to translate the time that the user seeked to (in seconds) into a byte offset into the file. This requires parsing the MP4 metadata to understand the file structure.
All this magic is currently handed in the render-media and videostream packages.


Demo

In a managed Kubernetes environment. Also add a "deploy to demo" Github action.

closed by #54 .

closed by #54 .


[v6] asyncBlurFields without fields array

It would be nice to have a validateOnBlur attribute on the Fileds component instead of setting a fields array via asyncBlurFields.

That is certainly a possibility. It was kept as it is to minimize breaking changes from v5.

Because this issue saw almost no activity for more than a year, I'm closing it.


Update documentation for new repository name

After doing this we should test the action.

Done as part of #17


Placeholder for search input inside dropdown

Is there a way to add a placeholder on the search input that shows in the dropdown? I'm not convinced every user would recognise that space at the top as a search box and wondered if the UX could be improved by adding the option for an internal search placeholder there.

For now I've overcome this with a search icon

Well, typical placeholders disappear as soon as you give focus to the text field, and we do want to give focus right away to ease the experience. I can see the argument for improving the discoverability of the search feature, but if we were to show a placeholder while focus is in place, that'd also be a non-standard UI pattern, so I'm a bit hesitant towards that.
But I do like your approach of adding a search icon. What template change did you make for that? Maybe we can put that in the default templates.

What browser do you use out of interest? In Chrome the UI pattern is to retain the placeholder on focus, seen here: https://jsfiddle.net/5r0s7015/
The native browser behaviour is only to remove the placeholder once you actually start typing,

Ah, I checked, you're right. My bad then :)
Then I agree it makes more sense indeed to try to have that behavior in Selectivity as well.


[ios_platform_images] Support for macOS

Use case
Adding macOS support to ios_platform_images would greatly simplify retrieving images from an asset bundle on macOS.
Perhaps more importantly, with the advent of https://flutter.dev/go/sf-symbols likely depending on ios_platform_images, macOS support is critical for Flutter mac apps to have a native feel with the proper symbols.
Proposal
Since ios_platform_images is already a part of Flutter, it makes sense that it would expand to desktop along with the framework since it still has value in that domain.

@stuartmorgan this would involve renaming the plugin as well, ios_platform_images was a bit short sighted...

See also https://github.com/flutter/flutter/issues/22701; we should consider whether a single fully cross-platform plugin would make more sense, or whether there's no useful abstraction.


(PUP-10397) Remove facts and vars from ScriptCompiler scope

This updates the scope for a ScriptCompiler to not contain any
facts, server_facts, trusted, or pal_variables.
This work is to enable the use of the lookup function in Bolt plans
outside of apply blocks. Since the concept of facts, etc. do not make
sense in the scope of running a plan, these fields should not be
available. Removing these fields from the scope disables the use of
interpolations outside of apply blocks.
Relates to puppetlabs/bolt#1403

CLA signed by all contributors.

Actually one small comment - the commit message is inaccurate, we do still load pal_variables we just don't define any when instantiating the script_compiler on the Bolt side.

@puppetlabs/platform-core Ready for review 👍

@beechtom Bolt team can review/merge when you're ready.
Rekicked appveyor to see if it's a transient

One question, will this break older bolt versions that expect puppet to provide these facts/vars? If so, can/should the older bolt versions be constrained to require puppet < 6.15.0?

This only impacts Puppet on the Bolt controller, which ships with Bolt. Bolt uses it's own version of Puppet even if Puppet is already installed separately on the Bolt controller. So users on older version of Bolt will still use the same version of Puppet that their Bolt shipped with, and shouldn't break.


update docs about query!

Can deprecated warnings be suppressed? :(

Can deprecated warnings be suppressed? :(

 -- -A deprecated !!!


Custom Cluster Service Names

Adding an environment variable option which sets the name of the master cluster service.
This lets the same Docker image be used with multiple services (ex: a test cluster and a production cluster)

This is a great improvement, thanks @abkaplan07


Expo fail to load app

Description
After running yarn start and scan the QR code from my phone, expo fail to load the app. All I did is runnign create-react-native-app my-app and then yarn start.
Expected Behavior
The app should be loaded and run in my phone.
Observed Behavior
Console output:
Logs from serving your app will appear here. Press Ctrl+C at any time to stop.

 › Press a to open Android device or emulator.
 › Press q to display QR code.
 › Press r to restart packager, or R to restart packager and clear cache.
 › Press d to toggle development mode. (current mode: development)

12:32:48 PM: Finished building JavaScript bundle in 1605ms

My phone stuck at a screen with a blue circle keep looping.
No error showing, the console just stuck

Environment
Please run these commands in the project folder and fill in their results:

npm ls react-native-scripts<EMAIL_ADDRESS>npm ls react-native<EMAIL_ADDRESS>npm ls expo<EMAIL_ADDRESS>node -v: v6.11.1
npm -v: 3.10.10
yarn --version: 0.27.5
watchman version: 0.1.8

Also specify:

Operating system: Windows 10
Phone/emulator/simulator & version: Android 7.0

Reproducible Demo
create-react-native-app my-app
cd my-app
yarn start

Solved by open blocked port in the router....


liveserver on asp.net mvc ?

Hello, i have a problem trying to launch the liveserver when using mvc pattern. i don't know why, but with a complelete new default asp.net mvc default project, liveserver shows this immage:

Any ideas for this? im thinking maybe its routing? just a guess tho.
if i debug, and run it shows the correct site, if i set the setting to go localhost and port 5000 while debugging, the liveserver will look corret, but not work with like it usually does with the live updates to the server

Live Server can't compile your Dynamic Pages.
Use Live Server Web Extension (https://github.com/ritwickdey/live-server-web-extension) which is built top of Live Server.
Setup help : https://github.com/ritwickdey/live-server-web-extension/blob/master/docs/Setup.md

@nicho1991 If @ritwickdey 's solution doesn't helps you then reopen it. i am closing it now. Thank you for this report.


3.0.8 -> 3.0.9 breaks ICN2037BP

This is more of an FYI for someone else having issues. If you see the following issue set your clkphase to false mxConfig.clkphase = false;
true:

false:

Yeah there was a bug in versions < 3.0.9 in that the clkphase value was always false and completely ignored.

OMG, thank you so much, I was going crazy trying to figure out what broke.


Warn user if restoring a partial directory snapshot

If a backup has errors (warnings), some directories may be incomplete.
We need to inform user when restoring a directory (files not transferred just don't exists) if there was an error on the backup he wants to restore.
This issue depends on #48 to trace backup status.

Fixed in commit e1f56d3b867a027f143cbea038534e5ed4686cb1


"Failed to obtain metrics. The metrics service may not be available."

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Use Rancher to setup a Custom cluster on OpenStack
Enable monitoring

Result:
All of the monitoring pods seem to be up, but when I try to open the graphs on the cluster summary page they never load. Eventually, a red box with "Failed to obtain metrics. The metrics service may not be available. undefined" appears in the top right a couple of times. I've rebuilt the cluster a few times from scratch with the same result.
I am also unable to view either the prometheus or grafana dashboards. Clicking on the proxy links under the System App just hangs forever.
Other details that may be helpful:
None for now.
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): v2.4.4
Installation option (single install/HA): single install

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Custom on OpenStack
Machine type (cloud/VM/metal) and specifications (CPU/memory): 4 OpenStack instances with 2 vCPU and 4 GB RAM each. 1 with etc and control plane, 1 with etc, control plane and worker role, and 2 worker only nodes.
Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.2", GitCommit:"52c56ce7a8272c798dbc29846288d7cd9fbae032", GitTreeState:"clean", BuildDate:"2020-04-16T11:56:40Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.6", GitCommit:"d32e40e20d167e103faf894261614c5b45c44198", GitTreeState:"clean", BuildDate:"2020-05-20T13:08:34Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version):

Client:
 Version:           19.03.6
 API version:       1.40
 Go version:        go1.12.17
 Git commit:        369ce74a3c
 Built:             Fri Feb 28 23:45:43 2020
 OS/Arch:           linux/amd64
 Experimental:      false

Server:
 Engine:
  Version:          19.03.6
  API version:      1.40 (minimum version 1.12)
  Go version:       go1.12.17
  Git commit:       369ce74a3c
  Built:            Wed Feb 19 01:06:16 2020
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.3.3-0ubuntu1~18.04.2
  GitCommit:
 runc:
  Version:          spec: 1.0.1-dev
  GitCommit:
 docker-init:
  Version:          0.18.0
  GitCommit:

I got the same issue using canal as network driver and a custom cluster on Openstack.

I can get it to work with flannel, but neither Calico nor Canal work for me even with OpenStack port security disabled.

I solved the problem for me. It was a missing port required in Security Group, UDP/8472


Hep A FL DOH support issue: Problem CSV- not loading

As I mentioned, Tim Doyle (Florida DOH) finally sent his data. He has exported a subset of genetic links from his overall dataset, and has a very simple file with Source, Target, distance info. I tried both PROD and Angular, the file just won't load. Stuck with this processing message below. Not sure why it's parsing extra nodes. See highlighted .
Please could you treat this as high priority? He has been very patient, but this is frustrating for him. He's hoping to publish, but stuck on this piece.
I will email you guys the problem file.
Thanks!

Quick Fix on production was to first convert the file to an excel file and then either set distances dropdown field in MT to 'None' or delete/rename the origin column in the excel file.
Fix deployed to mossy. Other scenarios where uploading a csv (exported from table view) and the data has both epi and genetic links (dashed/multi links) has not been fixed. To me, this scenario seems uncommon and the fix would take more time than it is worth at the moment.


Challenge Target the same element with multiple jQuery Selectors has an issue.

Challenge Target the same element with multiple jQuery Selectors has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/50.0.2661.102 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

jQuery Playground

#left-well

#target1
#target2
#target3

#right-well

#target4
#target5
#target6

Duplicate https://github.com/FreeCodeCamp/FreeCodeCamp/issues/12394

hello

I have the same issue


fix: theme is not updated when calling paint() with another theme

Calling paint({ theme: THEME }) does not update the calendar to the given theme

In the context of React, if you are trying to dynamically adjust the theme when a user toggles dark/light mode on your page, you can do this in a non-standard-react way.
const [themeMode] = useLocalstorageState('theme', 'light');
const prevThemeMode = useRef(themeMode);

  useEffect(() => {
    if (prevThemeMode.current !== themeMode) {
      // there is no built in way to dynamically update the theme, and we don't want to destroy/repaint the entire calendar as that leads to some odd behaviors
      // @ts-ignore
      calInstance.calendarPainter.root?.attr('data-theme', themeMode);
      prevThemeMode.current = themeMode;
    }
  }, [themeMode, prevThemeMode]);


[Extensions]Add query for initialized extensions

Signed-off-by: Frank Lou<EMAIL_ADDRESS>
Description
Create a feature to have an extension query to OpenSearch to obtain a list of initialized extensions and determine whether a particular extension is initialized.
Equivalent PR on OpenSearch-sdk-java : https://github.com/opensearch-project/opensearch-sdk-java/pull/300
Issues Resolved
fixes https://github.com/opensearch-project/opensearch-sdk-java/issues/149
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

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

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

Codecov Report

Merging #5658 (1c0bc30) into main (d248643) will increase coverage by 0.03%.
The diff coverage is 44.68%.

@@             Coverage Diff              @@
##               main    #5658      +/-   ##
============================================
+ Coverage     70.93%   70.96%   +0.03%
  Complexity    58578    58578
============================================
  Files          4760     4761       +1
  Lines        279515   279558      +43
  Branches      40348    40356       +8
============================================
+ Hits         198263   198388     +125
+ Misses        65098    65018      -80
+ Partials      16154    16152       -2

Impacted Files
Coverage Δ

.../opensearch/extensions/DiscoveryExtensionNode.java
74.07% <0.00%> (-21.17%)
:arrow_down:

...a/org/opensearch/extensions/ExtensionsManager.java
52.45% <22.22%> (-1.59%)
:arrow_down:

...va/org/opensearch/extensions/ExtensionRequest.java
33.33% <36.36%> (+3.92%)
:arrow_up:

...search/extensions/ExtensionDependencyResponse.java
71.42% <71.42%> (ø)

...port/ResponseHandlerFailureTransportException.java
0.00% <0.00%> (-60.00%)
:arrow_down:

...a/org/opensearch/client/cluster/SniffModeInfo.java
0.00% <0.00%> (-52.95%)
:arrow_down:

.../java/org/opensearch/node/NodeClosedException.java
50.00% <0.00%> (-50.00%)
:arrow_down:

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

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

...regations/metrics/AbstractHyperLogLogPlusPlus.java
51.72% <0.00%> (-37.94%)
:arrow_down:

... and 496 more

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

@mloufra can you resolve conflicts with changelog.md?


[Creator Starter Kit] BUG: Documentation for image coloring cut off

Describe the bug
Trying to make icons colored by state in a detailed list.
Don't know how to make what is shown happen.
To Reproduce
Steps to reproduce the behavior:

Go to https://learn.microsoft.com/power-platform/guidance/creator-kit/detailslist#image--clickable-image
Try to use the sample code to make the colored icons

Its cut off and doesnt show how to do it.

@Jenefer-Monroe This has been fixed in the latest release.


feat: add power state to device details

fixes issue with tag editing as well
closes Show current power status on device details page #2283
PR Checklist

[ ] Unit Tests have been added for new changes
[ ] API tests have been updated if applicable
[ ] All commented code has been removed
[ ] If you've added a dependency, you've ensured license is compatible with Apache 2.0 and clearly outlined the added dependency.

What are you changing?

Anything the reviewer should know when reviewing this PR?
If the there are associated PRs in other repositories, please link them here (i.e. open-amt-cloud-toolkit/repo#365 )

:tada: This PR is included in version 3.25.0 :tada:
The release is available on:

GitHub release
v3.25.0

Your semantic-release bot :package::rocket:


Improve ockam relay list --help by mentioning it works w/ Orchestrator enrollment by default

Current behavior
When a user runs ockam relay list --help the example in the help text can be improved. This is show in this screenshot.

Desired behaviors

The help text for the command says List Relays, leave that as is (this is the short help text). Add a long help text that says: List Relays on your default node. If you pass '--to <NODE>' then it lists the Relays at the given node

Change the help for the --to argument to be: Get the list of Relays at the given node

The help text example says to run $ ockam relay list --at n2
Instead, it should say $ ockam relay list --to n2

Implementation details

Here's an example of using the long_about attribute in another file:
https://github.com/build-trust/ockam/blob/59f52a66cc84ad682d9f4ea5715057f97f8eb7a3/implementations/rust/ockam/ockam_command/src/node/create.rs#L50-L52

Please take a look at this file to get started.

Some tips for newcomers:

Create a new branch for the issue you are working on (don't push directly to your fork's develop branch)
In the PR description, include the issue that's being solved
Run cargo fmt. You can configure the editor to run it automatically every time a file is saved
Run cargo clippy to make sure the code compiles + follows rust standards/best practices
You don't need to keep your branch updated with develop until there is a merge conflict. If you still want to update it, use git rebase develop instead of git merge develop

We love helping new contributors! ❤
If you have questions or need help as you explore, please join us on Discord. If you're looking for other issues to contribute to, please checkout our good first issues.

@0scvr That's awesome, this is all yours. Please let us know if you have any questions as you explore. You can also ask questions on the contributors discord https://discord.gg/RAbjRr3kds
Also, please associate your PR and issue using this technique: https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue


chore: add struct tags and update docs

Updating README for following plugins:

[x] MySQL (identifier field isn't required for URN generation in extractors)
[x] Tableau
[x] Compass (Resolves https://github.com/odpf/meteor/issues/474)
[x] Also, adding struct tags(json and yaml) in provider configs required for marshalling into yaml

Extra yaml/json struct tags look alright but I do see a few extra changes like tableu identifier, what are these for?

LGTM


How to train from a single video

I try to record a video myself and train the model. But except image of each frame, it also needs other files. I know I can use SAM to create mask, but can you give me a more detailed instruction on how your boxes.npy, data.npy and corres.txt comes from? Thank you!

I find custom.md has a really detailed explain, Sorry to disturb.


Updated transaction application model: archive node

Part of https://github.com/MinaProtocol/mina/pull/11760.

merged into https://github.com/MinaProtocol/mina/pull/12548


Update error code

Closes #3815.

Also, should these fancy quotes be replaced with "?

Also, should these fancy quotes be replaced with "?

Yep. Fancies were likely pasted in.


Warnings with react 15 about controlled/uncontrolled inputs

When controlling an input by adding the value property react 15 throws a warning
Warning: Input elements must be either controlled or uncontrolled
(specify either the value prop, or thedefaultValue prop, but not both).
Decide between using a controlled or uncontrolled input element and remove one of these props.

 (Was actually just working on this very issue)

thank you !

@daneryl Now, input elements are controlled by default (as long as updateOn="change", which is the default anyway).
You'll still get the warning if you don't specify an initial value for each model, which is good practice:
modelReducer('user', { email: '', name: '', ...etc })

Working great with the default controlled inputs thanks. loving the library !

Why is it considered good practice to specify an initial value for each model?  I already write out all the values of the model in two places: the actual form component and the constructor.  Since the initial state is invalid, I can't construct it with the constructor.  Why is it good practice to list out all the fields in a third place, pairing them all with empty strings or other semantically null values?

@greaber V1.0 will alleviate this requirement.


UAsset Export

Hi there,
I can't find the Asset Actions>Export.... Is it exposed? If not, can you please expose it?

Thanks,
Nick

Queued :)

HI, you can use ue.export_assets([asset1, asset2, ...], directory) or the UExporter api:
https://docs.unrealengine.com/latest/INT/API/Runtime/Engine/Exporters/UExporter/
from unreal_engine.classes import SoundExporterWAV

exporter = SoundExporterWAV()

exporter.export_to_file(your_asset, filename)

Awesome, thanks! I'll try to test it in the next couple of days! 👍

Hi @rdeioris,
I've tried the following but it didn't really work.
import unreal_engine as ue
from unreal_engine.classes import StaticMeshExporterFBX

directory = r"C:\\Users\\sinokgr\\Desktop\\Exports\\"
assets = ue.get_selected_assets()

# Method 1
ue.export_assets(assets, directory)

# Method 2
exporter = StaticMeshExporterFBX()
for asset in assets:
	exporter.export_to_file(asset, directory + asset.get_name() + "_method2.fbx")

Method 1 doesn't crash or error, but doesn't do anything either.
Method 2 on the other had, reports that "FBX version number for this version of the FBX SDK is 7.5.0" in the console, and exports a 0kb file.
The file I'm trying to export, is a simple sphere mesh, I imported from 3ds Max.
btw, I cloned the project today, and built the plugin for 4.16.
Thanks,
Nick

HI, you need at least 4.17 for the export_assets function:
https://github.com/20tab/UnrealEnginePython/commit/b33e0670f1cef628b341ff2c5ded2337a507dd32

Great, I've tested it with 4.18 and works fine. Thanks @rdeioris 👍
Would it be possible, when using the UExporter api, to have a boolean or something, that will show you the export options or use the latest settings, because I tried to export a few assets and I was getting multiple dialogs. In the first method, I you get the option to "import all" which is great, but it forces you to export in subfolders. Or maybe have an option in the first method to export in the root folder without subfolders.

Hi, sorry for hijacking but is it possible to access "Migrate..." via python functionality as well ?

@scanlinevfx , I have't used the plugin for quite some time, so I'm not sure if in the newer versions you can, but back then I don't think you could. As a quick solution though, since migrate is just a copy/paste, you can use the Asset dependencies/referencers to collect all the required assets and copy them to the specified content folder.

@scanlinevfx , I have't used the plugin for quite some time, so I'm not sure if in the newer versions you can, but back then I don't think you could. As a quick solution though, since migrate is just a copy/paste, you can use the Asset dependencies/referencers to collect all the required assets and copy them to the specified content folder.

Will give it a shot, thank you @renderapp


chore(rpc): add required fields to Trace handler

prepare for trace impl

Codecov Report

Merging #1683 (4838ff5) into main (17e8ce4) will decrease coverage by 11.52%.
The diff coverage is 80.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff             @@
##             main    #1683       +/-   ##
===========================================
- Coverage   75.03%   63.52%   -11.52%
===========================================
  Files         378      378
  Lines       45463    45465        +2
===========================================
- Hits        34114    28882     -5232
- Misses      11349    16583     +5234

Flag
Coverage Δ

integration-tests
21.56% <80.00%> (-0.02%)
:arrow_down:

unit-tests
56.83% <0.00%> (-12.12%)
:arrow_down:

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

Impacted Files
Coverage Δ

crates/rpc/rpc/src/trace.rs
93.33% <66.66%> (-1.59%)
:arrow_down:

crates/rpc/rpc-builder/src/lib.rs
63.71% <100.00%> (-2.29%)
:arrow_down:

crates/primitives/src/contract.rs
0.00% <0.00%> (-100.00%)
:arrow_down:

crates/interfaces/src/p2p/priority.rs
0.00% <0.00%> (-100.00%)
:arrow_down:

crates/net/network-api/src/reputation.rs
0.00% <0.00%> (-100.00%)
:arrow_down:

crates/storage/libmdbx-rs/src/lib.rs
4.54% <0.00%> (-95.46%)
:arrow_down:

...storage/provider/src/providers/state/historical.rs
0.00% <0.00%> (-92.00%)
:arrow_down:

bin/reth/src/test_eth_chain/models.rs
11.06% <0.00%> (-69.03%)
:arrow_down:

crates/net/discv4/src/test_utils.rs
21.26% <0.00%> (-67.43%)
:arrow_down:

crates/primitives/src/proofs.rs
30.73% <0.00%> (-66.40%)
:arrow_down:

... and 105 more

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


Camera Preview shows black screen

Hi, I'm using a Huawei Mate 8 with this beyondar library, the objects on the camera shows correctly but the camera preview only shows a black screen, when I debug the app stop in the line 401 in the CameraView.java:
mCamera = Camera.open();
and shows me this error:
E/Camera-JNI: android_hardware_Camera_native_setup Error: -87
E/Camera: Camera new cameraInitNormal:-87
E/BeyondAR_camera: Exception encountered opening camera:Camera initialization failed because the max number of camera devices were already opened(-87)
I already added the permissions to the manifest:
<uses-permission android:name="android.permission.CAMERA" />
<uses-feature android:name="android.hardware.camera.autofocus" />
<uses-feature android:name="android.hardware.camera" android:required="false" />
and I don't have another apps open when I use this one

This is Android permissions - you need to request permissions during runtime too on Marshmallow and higher.
https://developer.android.com/training/permissions/requesting.html


extent wkb to ewkb.finish point,lineString,polygon scan and value

extent wkb to ewkb.

finish point,lineString,polygon scan and value

Hey @julor - this is great! Do you want to add a note in README.md under a new Contributors section with a link to your Github profile and your name? If not, I'll merge as is.

no,thx.


fix spot market warnings

rather than just ignoring a bunch of next lines for unused param, can we ignore the whole file, like for example in the FeeCollectorMock and the OracleVerifierMock especially?
also does this fix all of the warnings or just a subset of them? could just be my imagination. either way woudl be nice to clarify.

fixes all warnings in spot-market.
and i ignored the warnings for all contracts in "mocks" now.  thanks for feedback 👍🏽


Error in NewRelic on publishing to kafka in Python

Description
While publishing message to apache kafka , getting error with Newrelic Message Transaction  - Error 'MessageTransaction' object has no attribute 'destination_name'
code
producer = KafkaProducer(
            bootstrap_servers=bootstrap_servers,
            value_serializer=lambda x: json.dumps(x).encode("utf-8"),
        )
producer.send(topic, value=event_data).get()

Error Stack Trace
File "/Users/mishika/supply-metrics/app/kafka_manager/base_consumer_v2.py", line 55, in produce
    self.producer.send(topic, value=event_data, key=key).get()
  File "/Users/mishika/miniconda3/envs/myenv/lib/python3.10/site-packages/newrelic/hooks/messagebroker_kafkapython.py", line 68, in wrap_KafkaProducer_send
    return wrapped(
  File "/Users/mishika/miniconda3/envs/myenv/lib/python3.10/site-packages/kafka/producer/kafka.py", line 581, in send
    value_bytes = self._serialize(
  File "/Users/mishika/miniconda3/envs/myenv/lib/python3.10/site-packages/kafka/producer/kafka.py", line 714, in _serialize
    return f(data)
  File "/Users/mishika/miniconda3/envs/myenv/lib/python3.10/site-packages/newrelic/hooks/messagebroker_kafkapython.py", line 203, in _wrap_serializer
    topic = transaction.destination_name
AttributeError: 'MessageTransaction' object has no attribute 'destination_name'

newrelic.ini file
[newrelic]
log_level = debug
high_security = false
transaction_tracer.enabled = true
transaction_tracer.transaction_threshold = apdex_f
transaction_tracer.record_sql = obfuscated
transaction_tracer.stack_trace_threshold = 0.5
transaction_tracer.explain_enabled = true
transaction_tracer.explain_threshold = 0.5
transaction_tracer.function_trace =
error_collector.enabled = true
error_collector.ignore_errors = pycommon.exceptions.common_exceptions:DuplicateError rest_framework.exceptions:ValidationError rest_framework.exceptions:NotFound rest_framework.exceptions:ParseError
browser_monitoring.auto_instrument = true
thread_profiler.enabled = true
distributed_tracing.enabled = false
app_name =
monitor_mode = true
license_key =

Expected Behavior
Expected message should have sent to kafka topic
Steps to Reproduce

kept newrelic.ini file locally and ran kafka consumer , will be able to above stack trace
Converting  topic = transaction.destination_name to topic = getattr(transaction, "destination_name","Default") in newrelic-python-agent lib seems to fix the issue

Your Environment

Application Python version : 3.10
kafka-python==2.0.2
newrelic==9.4.0

Hi @mishikaraj --out of curiosity, how are you setting up the consumer?  My concern with the proposed solution is that there is some other issue that is being masked; destination_name should get set as the topic from ConsumerRecord (which is returned from the consumer iterator)
This is the sample app I am using, so I want to make sure I am not missing an obvious implementation:
import json
import time
from datetime import datetime
from threading import Thread

import kafka

TOPIC = "test-topic-%d" % datetime.now().timestamp()
BROKERS = ["localhost:9092"]

consumer = kafka.KafkaConsumer(
    TOPIC,
    bootstrap_servers=BROKERS,
    client_id="whatsup",
    value_deserializer=lambda v: json.loads(v.decode("utf-8")),
    auto_offset_reset="earliest",
    consumer_timeout_ms=5000,
    fetch_max_wait_ms=304999,
)
producer = kafka.KafkaProducer(
    bootstrap_servers=BROKERS, value_serializer=lambda v: json.dumps(v).encode("utf-8")
)

def consume():
    print("Starting consumer...")
    for message in consumer:
        print(f"Recieved {message.value}")
    print("Consumer finished.")

def produce():
    print("Starting producer...")
    for json_message in [
        {"foo": "bar"},
        {"baz": "bat"},
        {"user1": "Hello!"},
        {"user2": "Hola!"},
    ]:
        time.sleep(1)
        producer.send(TOPIC, value=json_message).get()
    producer.flush()
    print("Producer finished.")

def main():
    t1 = Thread(target=produce)
    t2 = Thread(target=consume)
    t1.start()
    t2.start()
    t1.join()
    t2.join()
    print("Finished.")
    consumer.close()
    producer.close()

if __name__ == "__main__":
    main()

Hii @lrafeei , For Temporary Fix I had Disabled kafka producer transaction on newrelic by keeping below in newrelic.ini file and was able to send the message to kafka

[import-hook:kafka.producer.kafka]
enabled = false

StackOverFlow Link
And for answering to your above question, Yes I have initialized my consumer in the same way
self.consumer = KafkaConsumer(
            self.topic,
            group_id=self.group_id,
            bootstrap_servers=bootstrap_servers,
            value_deserializer=lambda x: json.loads(v.decode("utf-8"))
            enable_auto_commit=False,
            auto_offset_reset="latest",
            max_poll_records=500,
            **consumer_kwargs,
        )

Please let me know if you need any other help with this and what should be the fix we can look for enabling back the kafka producer transaction on newrelic for kafka producer

Sorry about the delay, I thought I'd get an alert when you replied!  Looking at this now.


Watermark is on exported image even though documentation states it should not

Quoting the watermarking documentation:

Add image water markers for the graph. Note that, images downloaded by downloadImage or downloadFullImage will not contains the water markers.

The last part states that images downloaded to do not contain water markers.
However, this is incorrect as the code clearly includes watermark into the downloadable image:
https://github.com/antvis/G6/blob/ccce26182f2f3b3c6bdc61dc86ae3031c69cca32/packages/pc/src/graph/graph.ts#L353
This results the following exported image:

Either correct the documentation, or make it optional to include watermarks on the exported image.

Thanks for reporting, we will fix the docs as soon as possible


C4: Transition all command call-sites that use CHIPClusters.cpp over to cluster object-based method

Problem
Once #8961 is complete, transition every single call-site that used the methods in CHIPClusters over to the new, type-safe + cluster object-based method. This task can be parallelized and distributed across the various call-sites.

Completed.


Update readme

Drop a few lines about .editorconfig

Done! https://github.com/orlovmax/front-end-scaffold/commit/e0dd701093c4160436a9478f86c128db209cbad1


Move a few more type checker perf tests over to compiling with design…

…ated types.

@swift-ci Please smoke test

@swift-ci Please smoke test OS X platform


About the pretrained Pointnet++

Hi,
Thanks for releasing your excellent work! I have some questions regarding to the training process. I noticed you employed a pre-trained PointNet++ as the point encoder in your work. I'm curious about two aspects:

Could you provide details on the pre-training process for PointNet++?
How does the result differ if it is trained from scratch?

Thanks for your insights!

Thank you for your interest in our work.

PointNet++ is from 3D-VisTA: Pre-trained transformer for 3D Vision and text alignment, https://github.com/3d-vista/3D-VisTA.
We do not conduct rigorous experiments on training it from scratch. Intuitively, like 2D vision language models, using a pre-trained backbone will generally lead to better performance.

Is the U-Net backbone trained from scratch?

@ayushjain1144 Yes


chore: release main

:robot: I have created a release *beep* *boop*
@opentelemetry/resource-detector-alibaba-cloud: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

Dependencies

The following workspace dependencies were updated

devDependencies

@opentelemetry/contrib-test-utils bumped from ^0.25.0 to ^0.27.0

@opentelemetry/resource-detector-aws: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

Dependencies

The following workspace dependencies were updated

devDependencies

@opentelemetry/contrib-test-utils bumped from ^0.25.0 to ^0.27.0

@opentelemetry/resource-detector-gcp: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

Dependencies

The following workspace dependencies were updated

devDependencies

@opentelemetry/contrib-test-utils bumped from ^0.25.0 to ^0.27.0

@opentelemetry/resource-detector-github: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

@opentelemetry/browser-extension-autoinjection: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

Dependencies

The following workspace dependencies were updated

dependencies

@opentelemetry/instrumentation-document-load bumped from ^0.25.0 to ^0.26.0

@opentelemetry/host-metrics: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

@opentelemetry/id-generator-aws-xray: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

@opentelemetry/contrib-test-utils: 0.27.0
Features

instrumentation-aws-sdk: upstream aws-sdk instrumentation from ext-js (#678) (f5851e7)
support API and SDK 1.0 (#706) (096b694)

@opentelemetry/instrumentation-aws-lambda: 0.27.0
Features

support API and SDK 1.0 (#706) (096b694)

Bug Fixes

prevent invalid context propagation in lambda functions (#677) (25c0e30)

Dependencies

The following workspace dependencies were updated

dependencies

@opentelemetry/propagator-aws-xray bumped from ^0.24.0 to ^0.26.0

@opentelemetry/instrumentation-aws-sdk: 0.1.0
Features

instrumentation-aws-sdk: upstream aws-sdk instrumentation from ext-js (#678) (f5851e7)

Dependencies

The following workspace dependencies were updated

devDependencies

@opentelemetry/contrib-test-utils bumped from ^0.26.0 to ^0.27.0

@opentelemetry/instrumentation-bunyan: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

@opentelemetry/instrumentation-cassandra-driver: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

Dependencies

The following workspace dependencies were updated

devDependencies

@opentelemetry/contrib-test-utils bumped from ^0.25.0 to ^0.27.0

@opentelemetry/instrumentation-connect: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

@opentelemetry/instrumentation-dns: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

@opentelemetry/instrumentation-express: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

@opentelemetry/instrumentation-generic-pool: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

@opentelemetry/instrumentation-graphql: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

@opentelemetry/instrumentation-hapi: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

@opentelemetry/instrumentation-ioredis: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

Bug Fixes

Add support for setConfig in ioredis instrumentation (#689) (314c890)

Dependencies

The following workspace dependencies were updated

devDependencies

@opentelemetry/contrib-test-utils bumped from ^0.25.0 to ^0.27.0

@opentelemetry/instrumentation-knex: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

@opentelemetry/instrumentation-koa: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

@opentelemetry/instrumentation-memcached: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

Dependencies

The following workspace dependencies were updated

devDependencies

@opentelemetry/contrib-test-utils bumped from ^0.25.0 to ^0.27.0

@opentelemetry/instrumentation-mongodb: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

Bug Fixes

opentelemetry-instrumentation-mongodb: fix span attributes with unified topology (#663) (aeadca8)

Dependencies

The following workspace dependencies were updated

devDependencies

@opentelemetry/contrib-test-utils bumped from ^0.26.0 to ^0.27.0

@opentelemetry/instrumentation-mysql: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

Dependencies

The following workspace dependencies were updated

devDependencies

@opentelemetry/contrib-test-utils bumped from ^0.25.0 to ^0.27.0

@opentelemetry/instrumentation-mysql2: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

Dependencies

The following workspace dependencies were updated

devDependencies

@opentelemetry/contrib-test-utils bumped from ^0.25.0 to ^0.27.0

@opentelemetry/instrumentation-nestjs-core: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

@opentelemetry/instrumentation-net: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

@opentelemetry/instrumentation-pg: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

Dependencies

The following workspace dependencies were updated

devDependencies

@opentelemetry/contrib-test-utils bumped from ^0.25.0 to ^0.27.0

@opentelemetry/instrumentation-pino: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

@opentelemetry/instrumentation-redis: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

Dependencies

The following workspace dependencies were updated

devDependencies

@opentelemetry/contrib-test-utils bumped from ^0.25.0 to ^0.27.0

@opentelemetry/instrumentation-restify: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

Bug Fixes

fix restify instrumentation utils that might accept undefined types (#690) (bc11f3d)

@opentelemetry/instrumentation-router: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

@opentelemetry/instrumentation-winston: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

@opentelemetry/instrumentation-document-load: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

@opentelemetry/instrumentation-user-interaction: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

Bug Fixes

user-interaction: EventTarget is undefined in IE (#627) (5a00bed)

@opentelemetry/plugin-react-load: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

@opentelemetry/propagator-aws-xray: 0.26.0
Features

support API and SDK 1.0 (#706) (096b694)

@opentelemetry/auto-instrumentations-node: 0.25.1
Dependencies

The following workspace dependencies were updated

dependencies

@opentelemetry/instrumentation-dns bumped from ^0.25.0 to ^0.26.0
@opentelemetry/instrumentation-express bumped from ^0.25.0 to ^0.26.0
@opentelemetry/instrumentation-graphql bumped from ^0.25.0 to ^0.26.0
@opentelemetry/instrumentation-ioredis bumped from ^0.25.0 to ^0.26.0
@opentelemetry/instrumentation-koa bumped from ^0.25.0 to ^0.26.0
@opentelemetry/instrumentation-mongodb bumped from ^0.25.0 to ^0.26.0
@opentelemetry/instrumentation-mysql bumped from ^0.25.0 to ^0.26.0
@opentelemetry/instrumentation-pg bumped from ^0.25.0 to ^0.26.0
@opentelemetry/instrumentation-redis bumped from ^0.25.0 to ^0.26.0

@opentelemetry/auto-instrumentations-web: 0.25.1
Dependencies

The following workspace dependencies were updated

dependencies

@opentelemetry/instrumentation-document-load bumped from ^0.25.0 to ^0.26.0
@opentelemetry/instrumentation-user-interaction bumped from ^0.25.0 to ^0.26.0

This PR was generated with Release Please. See documentation.

Do we think the auto instrumentations packages should also be incremented to 26?

yes

https://github.com/open-telemetry/opentelemetry-js-contrib/pull/709