python3Packages: add compute capabilities for CUDA 11

Motivation for this change
CUDA 11 supports capabilities 8.0 and 8.6. The change adds these
capabilities when CUDA 11 is used, enabling support for Ampere GPUs.
Tested with a small MNIST network.
Things done

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

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

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

Result of nixpkgs-review pr 113207 at 9e5af7c0 run on aarch64-linux 1

3 packages marked as broken and skipped:

python37Packages.pytorchWithCuda
python38Packages.pytorchWithCuda
python39Packages.pytorchWithCuda

I've improved @r-rmcgibbo's handling of timeouts (i.e. reporting them properly in the logs), so perhaps this will be a good test :/  x86_64 build is ongoing.

I've improved @r-rmcgibbo's handling of timeouts (i.e. reporting them properly in the logs), so perhaps this will be a good test :/ x86_64 build is ongoing.

Thanks! Knocking on wood :+1: . I guess it's not really feasible to do full builds with CUDA? They take quite a while, even on my 3700X.

I'm not sure in general what's feasible or not. Currently I have a timeout configured at 1 hours per drv and 2 hours for the whole nixpkgs-review (running on rather small hardware), but that's rather arbitrary and I'm willing to increase it.
Perhaps worse, I don't have my own binary cache configured (and the builders are ephemeral), so the fact that hydra doesn't build CUDA stuff means that my builder has to build quite a bit (currently I'm ssh'd in and it's working on nccl-2.7.8-1-cuda-10.2) even though you only changed pytorch. I know you've been working on a public binary cache for CUDA/HPC stuff (I think I remember this from discourse?), so perhaps when that's up I could plug r-rmcgibbo into that.

Result of nixpkgs-review pr 113207 at 9e5af7c0 run on x86_64-linux 1

1 package marked as broken and skipped:

python39Packages.pytorchWithCuda

2 packages failed to build:

python37Packages.pytorchWithCuda
python38Packages.pytorchWithCuda

1 suggestion:

warning: unused-argument
Unused argument: fetchpatch.
Near pkgs/development/python-modules/pytorch/default.nix:1:33:
  |
1 | { stdenv, lib, fetchFromGitHub, fetchpatch, buildPythonPackage, python,
  |                                 ^

Perhaps worse, I don't have my own binary cache configured (and the builders are ephemeral), so the fact that hydra doesn't build CUDA stuff means that my builder has to build quite a bit (currently I'm ssh'd in and it's working on nccl-2.7.8-1-cuda-10.2) even though you only changed pytorch. I know you've been working on a public binary cache for CUDA/HPC stuff (I think I remember this from discourse?), so perhaps when that's up I could plug r-rmcgibbo into that.

Can you forward that to me when you have it?

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/467


Investigate line ending issues

See https://github.com/hpi-swa-lab/BP2021RH1/pull/183#discussion_r1047128041

randomly stopped being an issue due to various unintended changes


Propagate error messages when using unwrap

Raise by @enitrat https://t.me/starknet_foundry_support/518

Adding some context as the title is not very descriptive - If a call made to a dispatcher fails, calling unwrap on it will make the test fail with Result::unwrap failed. This is not very informative and it would be better if the error raised in the entrypoint call was collected by the test runner.

Everything works fine while using a classic dispatcher. The above error was encountered when using .unwrap() on an error returned by the SafeDispatcher which should be handled explicitly.


[SignInPage] Add "Remember Me" as slot

Summary
No response
Examples
No response
Motivation
No response
Search keywords: signinpage rememberme

We fixed not being able to access the https://github.com/mui/toolpad/pull/4404 remember me checkbox state, it'll be available in the next release. Would that help your use case?


minor update in the documentation is required

in the restore database section, the statement should be as follows:
After you rebuild the resource database, all service packs, cumulative updates and hot fixes are lost, and therefore must be reapplied.
from SQL Server 2017 and beyond, Microsoft only releases cumulative updates and hot fixes for MSSQL ! No Service Packs anymore.

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

ID: 39e93d44-8942-2548-190e-616285b554c6
Version Independent ID: 4a1e67f5-097c-f4c9-544f-fd75ce0c974c
Content: Rebuild System Databases - SQL Server
Content Source: docs/relational-databases/databases/rebuild-system-databases.md
Product: sql
Technology: ****
GitHub Login: @stevestein
Microsoft Alias: sstein

@emad-almousa -- Emad, thank you for your feedback.  I am updating this document.

@stevestein -- Steve, please review the above-mentioned PR 13411.


ReactAvatar do not work with "src" from the start

Hi there! first of all, this component is great, so thank you guys for investing your time in it.
I think I may have found a bug. The following is not working for me:
<ReactAvatar src="http://example.com/image.jpg" name={....} round={true} size={100} />

By looking at the code, I noticed in componentWillReceiveProps, that you are doing this:
if (newProps.src !== this.props.src) nextState.src = newProps.src;

newProps.src is never different from this.props.src (as no there is no change in src) and so nextState.src is never set so this.state.src is always null/undefined unless there is an actual change in the src property.
I think the solution may be to initialize this.state.src to this.props.src inside the constructor. But maybe I am missing something. I have checked and rechecked things and I found out that the only places I actually  made ReactAvatar work were effectively whenever something provoked a change in the src prop. The rest of them simply didnt work.
Let me know if I can help you guys out on something to get this released as soon as possible.

This is resolved by this PR and is released as version 2.3.0.


All the End Events not firing on Android after scrolling.

not sure if anyone else is getting this problem,
I have added the jquery.hammer library into my site,  I am trying to get the drag/hold events to work,  everything works smoothly on ipad, but on Andoird tablets ( have tried on Galaxy tab1, tab2, and Xoom)  if I first open the site all the gestures seems to work fine,  but once I scroll the parent div, it just won't detect the gesture anymore until I single tap on any element on the screen.  then I can drag/hold elements just fine.
I added tracing against all the event "handler" functions and I can see that from logcat that without scrolling all the event cycle is in "start" "move" "end" pattern,  whereas if I scroll the page,  the cycle I get is only "start" "move"  the "END" events doesn't seems to fire?     but once I single tap on an element, the whole event cycle will become normal again.
can anyone give me any suggestion on what to look for to resolve this problem?
BTW,  thanks for this wonderful library......  happy coding:)

I'm using hammer 2 with backbone. also faced this issue. here is my fix:
this.$el.hammer({ domEvents: true }); // here $el is body and this line enables hammer to work on each dom el in the app
this.$el.data("hammer").get('pan').set({ enable: false }); // add this line. this fixes the issue. pan continues working in child elements, where it is added as:
$childEl.hammer().on("pan", ...);
$childEl.data("hammer").get('pan').set({ direction: Hammer.DIRECTION_ALL, threshold: threshold });
hope this helps somebody

the issue reproduces on android, desktop chrome


MacOS and ROS 2 Support

I wonder if there will be a simple app for MacOS that allows to control the robot and see the cam video? And, I wonder if you will upgrade the ROS version?
Thanks a lot

Hi, Jens,
No, we don't have MacOS applications unfortunately. There is also no schedule for ROS 2. Sorry about that. The team are working on other AI features.
Thanks much for your support!
The Moorebot Team
@.***
From: Jens
Date: 2023-10-23 21:35
To: Pilot-Labs-Dev/Scout-open-source
CC: Subscribed
Subject: [Pilot-Labs-Dev/Scout-open-source] MacOS and ROS 2 Support (Issue #4)
I wonder if there will be a simple app for MacOS that allows to control the robot and see the cam video? And, I wonder if you will upgrade the ROS version?
Thanks a lot
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

I wonder if there will be a simple app for MacOS that allows to control the robot and see the cam video? And, I wonder if you will upgrade the ROS version?
Thanks a lot

I made a small app for macOS that does what you were asking, if you still searching for it.

GINNOV

hey, I would be interested in having a look at your app, but could not see it on your page: would you share the code? thx :)

GINNOV

hey, I would be interested in having a look at your app, but could not see it on your page: would you share the code? thx :)

done, uploaded. https://github.com/GINNOV/Scout-Controller


Ensure correct FFI bindings

Some complications here: Uint32 changed from unsigned long to unsigned int a couple of SDKs ago, and SecCopyRandomByte's last argument changed from unsigned char * to void *.


Check whether required params for embedded addons are present

What happened:
Here's a snippet from my KubeOneCluster manifest:
apiVersion: kubeone.io/v1beta1
kind: KubeOneCluster
addons:
    enable: true
    path: ./addons
    addons:
        - name: cluster-autoscaler
versions:

As you can see, I missed to configure the AUTOSCALER_VERSION param.
After applying this, I saw this funny issue:
  Warning  InspectFailed  3m53s (x558 over 124m)  kubelet            Failed to apply default image tag "k8s.gcr.io/autoscaling/cluster-autoscaler:${AUTOSCALER_VERSION}": couldn't parse image reference "k8s.gcr.io/autoscaling/cluster-autoscaler:${AUTOSCALER_VERSION}": invalid reference format

What is the expected behavior:
kubeone should probably fail before deploying addons, if required parameters are missing.
How to reproduce the issue:
See above - install e.g. the embedded cluster-autoscaler addon but don't specify any params.
Anything else we need to know?
Better examples wrt enabling the embedded addons would be nice.
Information about the environment:
KubeOne version (kubeone version): 1.3

In this concrete case, we fixed this bug for the cluster-autoscaler addon yesterday with #1552. The version is now determined automatically based on the Kubernetes version, but it can also be overridden if needed (see the description of #1552 for more details). The fix will be included in the upcoming 1.3.1 patch release, but the release date is still TBD.
However, this might still be the issue for other addons, so I'm going to leave this issue open until we don't check and eventually fix other addons.

Thanks for the quick response @xmudrii! That's great news, looking forward to 1.3.1 :)

@xmudrii It's been a while and I also see other important fixes have been backported to 1.3. Do you think we could get a new release in the near future? :)

@judge-red We are waiting for #1607 which should be merged soon. I hope we will be able to cut 1.3.1 beginning of next week.

I don't even know how it can be possibly done in generic way. Go templates are absolutely happy to consume nils.
The best workaround I see is to define a "required" template function, that will ensure not-nilness of the given argument, and would fail the template render otherwise. Of course its usage should be explicit.

helm has "required" function like this:
https://github.com/helm/helm/blob/b6a04cfbd544f0bbeea449129c1497eec8d99e2b/pkg/engine/engine.go#L155-L174


enhance: add a better error message for errors originating from the Terragrunt libs

If someone runs into an error when running HCL parsing with Terragrunt, we weren't clarifying that the errors were from the Terragrunt libs. This updates that and points them to generate a plan JSON for just now.
Example:

Thanks! LGTM


Maxwell 3D: How can hidden model objects be included in a png exported by ExportModelImageToFile()?

Hello Massimo,
I would like to export a png containing all model objects for documentation purposes.
The issue:

app.modeler.model_objects contains all model objects including hidden objects
If the objects  app.modeler.model_objects are passed to the ExportModelImageToFile() function, the hidden objects are also hidden on the exported png.
Is there a way to temporarily unhide the objects, export the model image and hide the previously hidden objects again?
Here is the code I am using for the export:
param = [           "NAME:SaveImageParams",           "ShowAxis:=", "True",           "ShowGrid:=", "True",           "ShowRuler:=", "True",           "ShowRegion:=", "Default",           "Selections:=", ",".join(app.modeler.model_objects),           "FieldPlotSelections:=", "",           "FitToSelections:=", ",".join(app.modeler.model_objects),           "FitToFieldPlotSelections:=", "",           "Orientation:=", "view_png_export"       ]

m3d.modeler.oeditor.ExportModelImageToFile(file_name, 0, 0, param)
Please don't hesitate to contact me for further information. Thanks in advance :-).

Hello Michael,
you can use
m3d.plot()
without any option is simply plotting the full model (inlcuding hidden objects)
with the options:
m3d.plot(show=False, export_path="c:\temp\image.jpg")
it will not plot but save the picture directly
furthermore you can rung
plt_obj = m3d.plot(show=False)
plt_obj.show_axes = False
...
plt.plot(file_path)
in case you want to customize the plot before generating it.

Hello akaszynski,
thanks for your quick reply.
Running the command yields the following error:

Ok, I'll take a look on the link you provided, thank you.
Kind regards
Michael


Repo sync for protected CLA branch

The pull request is created from master637686185232232611 to master to fix git push error for protected CLA branch

@huypub : Thanks for your contribution! The author(s) have been notified to review your proposed change.


implement Display for Text, Line, Span

Problem
Someone on discord wanted to check the actual content of a Text, ignoring the style.
One solution is to loop over the inner lines and span manually. I'm wondering if implementing Display is worth it to get a string representation of Text?
Solution
Implement Display for Text, Line and Span at least.
Alternatives
Just let users code it themselves as the fields are all public and this might not be a wide use case.

I remember hitting the same case in the past. I think that the Display implementation or a method such as raw_text() would be nice.

Sure go ahead @Emivvvvv


【Inference】Add silu_and_mul for infer

📌 Checklist before creating the PR

[ ] I have created an issue for this PR for traceability
[x] The title follows the standard format: [doc/gemini/tensor/...]: A concise description
[ ] I have added relevant tags if possible for us to better distinguish different PRs

🚨 Issue number

Link this PR to your issue with words like fixed to automatically close the linked issue upon merge
e.g. fixed #1234, closed #1234, resolved #1234

📝 What does this PR do?

Summarize your work here.
if you have any plots/diagrams/screenshots/tables, please attach them here.

This PR do the following things:

add mp_type_traits to help mix precision calculation
add silu_and_mul kernel

Optimization  Result

TODO

add vectorize traits and related ld&st&cal support for further optimization
profiling for model improvement at last version with vec ld&st&cal
refactor the inference architecture such as namespace. file structure and so on
we should make our kernel more reusable by abstracting certain generalized components

💥 Checklist before requesting a review

[ ] I have linked my PR to an issue (instruction)
[ ] My issue clearly describes the problem/feature/proposal, with diagrams/charts/table/code if possible
[ ] I have performed a self-review of my code
[x] I have added thorough tests.
[ ] I have added docstrings for all the functions/methods I implemented

⭐ Do you enjoy contributing to Colossal-AI?

[x] 🌝 Yes, I do.
[ ] 🌚 No, I don't.

Tell us more if you don't enjoy contributing to Colossal-AI.

LGTM.


iOS Supported or not?

iOS is listed under supported operating Systems.
"What client operating systems can I use with Point-to-Site?
The following client operating systems are supported:
Windows 7 (32-bit and 64-bit)
Windows Server 2008 R2 (64-bit only)
Windows 8.1 (32-bit and 64-bit)
Windows Server 2012 (64-bit only)
Windows Server 2012 R2 (64-bit only)
Windows Server 2016 (64-bit only)
Windows Server 2019 (64-bit only)
Windows 10
Mac OS X version 10.11 or above
Linux (StrongSwan)
iOS"
Although this Articel states, that only Windows, MAC and Linux are supported
"Other than Windows and Mac, which other platforms does Azure support for P2S VPN?
Azure supports Windows, Mac and Linux for P2S VPN."

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

ID: a3113b20-0f6c-988e-444b-eff387f3ffef
Version Independent ID: ded2a816-93c7-a4b3-1ec2-c29ce86cd492
Content: About Azure Point-to-Site VPN connections
Content Source: articles/vpn-gateway/point-to-site-about.md
Service: vpn-gateway
GitHub Login: @cherylmc
Microsoft Alias: cherylmc

@MclMax1, P2S can be used to connect from Android and iOS (versions 11.0 and above). OpenVPN is the protocol used here by P2S for connection.

@MclMax1, we are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


Unsubscribe mail and resubscribe email when primary email is changed

What kind of a pull request is this?

QA / Code Review

Code Checklist

[x] The code follows dotkom code style
[x] The code passes the defined tests
[ ] I have added tests for the code I added
[ ] I have provided documentation for the code I added
[x] The code is ready to be merged

(Possible) Breaking Changes

[x] The changes are backwards compatible

Description of changes
Fixes #2068
Screenshot(s) if appropriate

Any thoughts for a one-off script/job to unsubscribe all "actually unsubscribed" people in OW4 to reconsolidate whoever have actually tried to unsubscribe but haven't been unsubscribed because they changed their address.


Node.js deployment on Azure

After deploying the bot on Azure, my html code is not found! When i test the bot it says 'Accepted" but when i run the web app via azure, i get this error: {"code":"ResourceNotFound","message":"/ does not exist"}

"/" doesn't seem like the correct path - isn't the web service url typically "/api/messages"?  In particular, the url in the web browser would be "https://yourbot.azurewebsites.net/api/messages" (though you have to POST to it, not just GET).

@gcrev93, what do you mean by "when I run the web app via azure"? are you visiting the web app with your browser? If that's the case, the error you're seeing is expected.
Most likely you have the following code fragment in your bot, where you tell node what uri it should listen to in server.post(). If you didn't change it from what it is in our samples, that url is '/api/messages'.
// Create chat bot var connector = new builder.ChatConnector({     appId: process.env.MICROSOFT_APP_ID,     appPassword: process.env.MICROSOFT_APP_PASSWORD }); var bot = new builder.UniversalBot(connector); server.post('/api/messages', connector.listen());
If you want to interact with the bot, you need to connect it to a channel. Try one of the following:

use the Web channel emulator https://docs.botframework.com/en-us/tools/bot-framework-emulator/
try the webchat control https://docs.botframework.com/en-us/support/embed-chat-control2/
connect to a channel (e.g. Skype, go to your bot dashboard, and click on the "Add to Skype" button)

Hope this helps

@gcrev93, I'm closing this issue, I think our answers should address your problem. Feel free to reopen if you still need help. Thanks a lot!

@gcrev93
In order for you to call https://yourbot.azurewebsites.net/ like this, you have to add this piece of code into your server.js or app.js file:
server.get(/.*/, restify.serveStatic({ 	'directory': '.', 	'default': 'index.html' }));
and create an index.html file in the same folder of your server.js
Let me know if it helps!

@pdtran3k6  hi, even adding the code server.get , i am still facing the same issues like resource code not found.


Swift package compile times are 10-20 times worse on Windows than on Linux

Description
Building the exact same Swift package on the same computer but using Windows takes orders of magnitude longer for both clean and incremental builds than doing it on Linux.
Reproduction
I've been trying to build this Swift package: https://github.com/migueldeicaza/SwiftGodot/. For future reference, I'm building tag 0.40.0
I reported this issue to the maintainer of that package and he insists the slowdown is purely up to the Swift compiler to fix: https://github.com/migueldeicaza/SwiftGodot/issues/299#issuecomment-1963062462
Expected behavior
The above package build on Linux takes 83 seconds for a clean build and 3 seconds for an incremental build (after making trivial changes).
In Windows 10 it takes 19 minutes for a clean build and 13 minutes for an incremental build. This makes the package all but unusable on Windows. Comparing the incremental builds (which are the most important, in my opinion), that's over 240 times slower than on Linux.
Environment
Swift version 5.9.2 (swift-5.9.2-RELEASE)
Target: x86_64-unknown-windows-msvc
Additional information
I'm up for profiling this to figure out exactly where the slowdown is but I don't know where to get started with that.

Can you give more info on your environment, ie what hardware and OS versions are you running on and where do you get your toolchains from? Also, try it with Swift 5.10, as that is the latest release.

AMD Ryzen 9 3900XT CPU, 40GB RAM
Arch Linux x86_64, kernel 6.7.9-arch1-1, installed Swift 5.10 binary via AUR
Building SwiftGodot commit db759c57

clean 76 seconds
incremental (trivial change to a log statement) 39 seconds

Will update shortly with Windows results.

I've heard of Windows Defender's real-time protection slowing down compilers before so I tried disabling it. I turned off real-time protection, cloud-delivered protection, and automatic sample submission. Maybe it helped but not a lot. The clean build time was 870 seconds, still 11x slower than Linux.

Compile times are slower on Windows (at least subjectively), and is definitely something that I am interested in seeing improved. I don't have a good understanding of what is contributing to that, but having more details might be helpful to identify a plan of attack.
There are some temporary files written to %LocalAppData% IIRC. The initial compile is going to be extremely slow as it builds the initial WinSDK module which is extremely large. Subsequent builds are slightly better but still feels slow. Do you have specific files/modules which are greatly different? If we can find a starting point and start chipping away from the largest outliers, perhaps we can improve this.

One thing that would be interesting to test if you have the motivation: try replacing the build with CMake + Ninja rather than SPM. The Browser Company noticed a lot of problems with SPM, particularly around incremental builds. With the latest CMake (3.29), it seems that most of those problems have been resolved.

I can try CMake. SwiftGodot defines macros so is this a good starting point for the CMake config?
Also @compnerd, do you mind trying to build SwiftGodot on your Windows machine? It's pretty easy, just clone and swift build. I'd be curious if it's significantly faster for you, especially for incremental builds.
I don't know how to narrow down which files/modules are causing the slowdown since even trivial changes (for example changing this print statement) take over 10 minutes to rebuild for me. Do you have any tips there?

@compnerd I switched to building https://github.com/apple/swift-syntax as a benchmark since it already has cmake set up. I was easily able to build it with Swift build (81 seconds), but could not get cmake to work.
I installed cmake 3.29.0-rc4 and ninja 1.12.0 (from commit 86997c40) and ran
mkdir build
cd build
cmake .. -G Ninja
ninja

And the Ninja build fails at
[1/47] Linking Swift static library lib\swift\host\SwiftSyntax.lib
error: unableToFind(tool: "link")

That is expected, you need to be in a developer command prompt to find the Microsoft linker. Alternatively, you can add -D CMAKE_Swift_FLAGS="-use-ld=lld" to use LLD as the linker.

I got different errors using developer command prompt but that -D option worked.
The results still aren't great with ninja building the swift-syntax package

Incremental build using ninja on Windows with a 3.8 Ghz 12 core Ryzen 9: 12.42s
Incremental build using Swift build on my dinky Mac with a dinky 2.3 Ghz 8 core Intel i9: 1.78s

@compnerd Did you remove the bug label by accident?

Out of curiosity, it might be interesting to see if limiting the number of compiler threads/processes by passing -j 4 or -j 8 would make any difference, if you have a chance.

@hjyamauchi -j 4 and -j 8 make no significant difference. All incremental builds of swift-syntax take about 12 seconds on Windows for me.
@AnthonyLatsis  I see you added the swift 5.9 label. My latest round of testing has been on Swift 5.10 and building is still at least 10x slower on Windows.

@silverhammermba  I thought it might be related to the compiler hang we see with a lot of cpu cores, but it doesn't seem to be. Thanks for checking with the -j values.

I hope I can look into this when I have a bit more time.


[.NET Migration Guide Revamp] [Phase 1] Move content of page 'Organize projects for .NET'

For details on revamp see: https://github.com/dotnet/docs/issues/23561
New TOC: https://github.com/dotnet/docs/issues/23215

[x] Organize projects for .NET (MOVE) Move this page to the new TOC location...

name: Migration guide
items:

name: Overview
href: ../core/porting/index.md
name: General Information
items:
name: Pre-Migration
items:
name: Migration
items:

name: Create a porting plan
items:

name: Overview
href: ../core/porting/create-a-porting-plan.md
name: Project structure
href: ../core/porting/project-structure.md

Completed with #24630


Cache address from ScriptPubKey

Performance Improvement. - This PR caches the address read from the ScriptPubKey when using the ScriptAddressReader. This is currently done for every HDPayment Record and is an expensive operation.

Tested this. It seemed to be slower with manual profiling (stopwatching, don't judge lol).
My GetHistory call with 10k entries and ~215 sends went from ~2.8s to ~3.5s.
I looked into this and tried to optimise... Avoid the ToLists and multiple settting on the dictionary, removed the lookup entirely and made the cache a normal Dictionary, we don't need concurrent.
New code would look like this:
private static readonly Dictionary<Script, string> ScriptPubKeyCache = new Dictionary<Script, string>();

            if (transactionData.SpendingDetails == null || this.ScriptAddressReader == null)
                return res;

            foreach (var paymentDetails in res)
            {
                if (ScriptPubKeyCache.TryGetValue(paymentDetails.DestinationScriptPubKey, out string storedAddress))
                {
                    paymentDetails.DestinationAddress = storedAddress;
                }
                else
                {
                    string address = this.ScriptAddressReader.GetAddressFromScriptPubKey(this.Network, paymentDetails.DestinationScriptPubKey);
                    ScriptPubKeyCache[paymentDetails.DestinationScriptPubKey] = address;
                    paymentDetails.DestinationAddress = address;
                }
            }

            return res;

Using the above code, the GetHistory call was marginally faster. Seemed to be ~2.6s.

But I don't think those results are significant enough to warrant putting in at this stage.

I concur with @quantumagi, if there is a worthwhile performance benefit from introducing this cache, then I would suggest utilising an LRU cache implementation with a bounded maximum size. See the MemorySizeCache / MemoryCountCache in Stratis.Bitcoin.Utilities as possible ready-made implementations.

<IP_ADDRESS> is closed, so I think we can close this. If you're still keen to get this in @ianadavies feel free to open a PR to <IP_ADDRESS>


How to add multi-value field?

I'm trying to make one of the user input form fields a drop-down options list. I can do this (add the HTML for the ) in the blade and have tried to have this modify the database data, to no avail.  Can you recommend how to do this. I want to use this tool to accelerate a project that has many data input forms for the user, but need to be able to have the user select from drop-down options for some fields.
Thanks in advance for your help.

Hi, this is a nice package but I think that the developers have stopped working on it. Maybe you can try this package: https://github.com/Muhammadinaam/speed-admin. It supports Laravel 8+.


Update buyResources.py

Regular Expressions fix

Now it shows offers, hit[4] doesn't seem to be used so i left it displaying the new resources url

thanks!


[DO NOT REVIEW] test CI trigger only [skip ci]

Fixes # .
Description
A few sentences describing the changes proposed in this pull request.
Status
Ready/Work in progress/Hold
Please ensure all the checkboxes:

[x] Codeformat tests passed locally by running ./runtests.sh --codeformat.
[X] In-line docstrings updated.
[X] Update version and changelog in metadata.json if changing an existing bundle.
[X] Please ensure the naming rules in config files meet our requirements (please refer to: CONTRIBUTING.md).
[X] Ensure versions of packages such as monai, pytorch and numpy are correct in metadata.json.
[X] Descriptions should be consistent with the content, such as eval_metrics of the provided weights and TorchScript modules.
[X] Files larger than 25MB are excluded and replaced by providing download links in large_file.yml.
[X] Avoid using path that contains personal information within config files (such as use /home/your_name/ for "bundle_root").

/build

/build

build

/build

/build

/build


The application couldn't find the answers I was looking for

In 0203.md I could run the second prompt

But the first prompt repeatedly gave errors

To double-check, did you put both text files into the same container and folder? At first glance, it appears that Resorts.txt may not have been vectorized and loaded.

yes
[cid:f444a2f7-a56f-4668-af21-da4d18986ff7]

From: Kevin Feasel @.>
Sent: Friday, January 12, 2024 6:12 PM
To: microsoft/TechExcel-Implementing-automation-practices-using-Azure-OpenAI @.>
Cc: manuchadha1979 @.>; Author @.>
Subject: Re: [microsoft/TechExcel-Implementing-automation-practices-using-Azure-OpenAI] The application couldn't find the answers I was looking for (Issue #6)
To double-check, did you put both text files into the same container and folder? At first glance, it appears that Resorts.txt may not have been vectorized and loaded.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/TechExcel-Implementing-automation-practices-using-Azure-OpenAI/issues/6#issuecomment-1889746741, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFNPJQ3XJHIR7DGMRSBGUOLYOF4KHAVCNFSM6AAAAABBYQN7V6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBZG42DMNZUGE.
You are receiving this because you authored the thread.

Some learners have experienced this and it might be related to using the 0615 preview. Either way, I've added a note that if you run into this issue, break it out into two separate queries, as the data appears to be in the dataset but the question might be a little too complicated for our model deployment to handle.


Plans for auto-scaling

In docs/conceptual-docs/appmodel-overview it was mentioned that auto-scaling is still to be added.
Can you elaborate on the plans for this please? How do you see this?
Things that come to my mind:

Custom metrics source(s): Will it have integration with Azure Monitor and Application Insights out-of-the-box?
Conceptually: Will it be similar to the Horizontal Pod Autoscaler (HPA) model in Kubernetes?

Hey folks,
We are also interested in how auto scaling will work going forward.
Will we be able to achieve behaviors (either out of the box or by manual implementation) similar to Azure Functions autoscale i.e. based on specific sources (e.g. azure queue and service bus queue lengths, event hub message counts etc?  These autoscale + consumption based billing properties are a wonderful combination but sometimes the functions programming model isn't ideal (e.g. 10 minute limit on execution time). Service Fabric Mesh seems like it could fill this void (consumption based + autoscale + more tradition programming model).
Thoughts? Will this be possible down the line?

We have been looking into how EventGrid could integrate with Service Fabric Mesh as a way of having a resource notify its usage and this scales out the corresponding Mesh service. Nothing concrete to report at this time, but given Mesh is designed for long running services, this is a way to go.

@msfussell is there any way to control scaling from code in SFM? With full SF I can dynamically create services and scale them from another service (orchestrator service) based on my custom conditions. Is there maybe a REST api coming to do that in Mesh?

We have been looking into how EventGrid could integrate with Service Fabric Mesh as a way of having a resource notify its usage and this scales out the corresponding Mesh service.

So some orchestrator service in our code would listen to these event grid events at a regular interval and issue scale up/down/out commands (like @aloneguid  described)? That sounds like a good way to go (and maybe similar to what happens in the Scale Controller in Azure Functions). I guess it could depend on which services were well integrated enough into Event Grid to publish their workloads (and how often).
Thank you for responding so quickly, this is helpful for long term planning.

Frankly, I would only use Azure Event Grid to emits that a scaling action has been taken and not use it as a source to make scaling decisions on.
Scaling decisions should be made on instance metrics which it provides nowadays, but also based on arbitrary metrics. This should initially be Azure Monitor and later provide the capability to bring your own custom metrics from for example Prometheus and the like. This would allow you to define all these rules in your application manifests
Another great capability would be to be able to autoscale via Azure Monitor Autoscale where you can define the scaling rules outside of a Service Fabric Mesh configuration and consolidate with your other autoscaling rules.
This is a model similar to Kubernetes that works very well.
Azure Event Grid is great for eventing but I would not use it as input for this case because who will emit what events? However, having events after the fact would be great allowing you to plug in to create observability similar to what I've indicated here.

@mayoatbm 	That's true and that's where the integration with Azure Monitor comes in. With arbitrary I mean metrics that are not related to the running service itself but externally such as Azure Monitor, Prometheus, etc.
The minimal scaling options for me personally are:

Scale on SFM service (instance) metrics
Scale on metrics coming from Azure Monitor (and thus implicitly from Application Insights for business metrics or custom tracking of metrics) and configured via resource declaration of SFM application
Scale automatically via Azure Monitor Autoscale initiating scaling actions (and thus implicitly from Application Insights)

Will this be covered @ChackDan 	?

@tomkerkhove
I just checked out the list of metrics captured in Azure Monitor for different services and i think you're right here. There's pretty wide integration with different Azure services and many of  the metrics include clear workload indicators.
Right now, using Azure Monitor would involve something like

Poll Azure Monitor & Maintain State on Results
Based on results over a a couple of calls (e.g. moving average of queue counts in state), issue scale commands.

Anything we can do to make the above easier helps because almost every developer will want to do it.

@mayoatbm 	I think my point was not clear, all the scenarios that I've described are managed by Azure to triggering the scaling and involves no custom logic.

Scale on SFM service (instance) metrics -> SFM triggers the scaling for you
Scale on metrics coming from Azure Monitor (and thus implicitly from Application Insights for business metrics or custom tracking of metrics) and configured via resource declaration of SFM application -> SFM runtime polls for metrics and triggers scaling action for you
Scale automatically via Azure Monitor Autoscale initiating scaling actions (and thus implicitly from Application Insights) -> Azure Monitor Autoscale will consult the metrics and initiate a scaling actions towards SFM

Those are all managed by the platform, but if you want to have custom scaling that's where the custom implementation would come in.
Bonus points if you could bring your own metrics provider, ie Prometheus, and let SFM scale based off of that for scenario 2.
@ChackDan @msfussell 	 	Are scenarios 1, 2 and/or 3 being considered?

@tomkerkhove
That does make things clearer and I agree, Having these scenarios managed by Service Fabric Mesh would be amazing. Ease of use + consumption billing + autoscale for bursty or high volume traffic + traditional programming model etc.
It would be amazing.

As per the classification @tomkerkhove put a couple of posts above:
#1 (scaling on service metrics) is already supported by SFM, but only for the built-in resource metrics (cpu and memory). We are aware of this limitation, and we are planning to remove it in the future (to get SFM auto-scaling to achieve parity with SF auto scaling).
We are considering #2 (scaling on custom/external metrics) and #3 (integration with Azure Monitor Autoscale) for future enhancements to SF and SFM auto scaling. We'll keep you in the loop with our plans.
Thanks everyone for the feedback and suggestions!

Bumping - What's the status here?

@tomkerkhove following up on this and will get you an update in the next couple of days.


StrippableSystem doafter overhaul

About the PR
Refactors Strippable DoAfter events to make them synchronous and organized.
Technical details
Strippable System & Component

Synchronous DoAfters
Made use of TimeSpan, GetStripTimeModifiers() and ByRefEvent
Reorganized checks, removed some redundant ones
Resolve pattern where useful
Added more asserts
Lots of cleanup

The DoAfters were grouped under one event to avoid copy-pasting eight separate cancel checks, asserts and function signatures.
Let me know if this is bad for performance and I'll roll them out instead.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes
TimeSpans
ThievingComponent, InventoryTemplatePrototype and ToggleableClothingSystem use TimeSpan in places where they intersect with StrippableComponent.
Changelog
N/A

Tests failed to build due to yesterday's bug.

I could update to master locally or I could see what happens.


webapp: support to retrieve creds from acr

Fix #3597

[x] The PR has modified HISTORY.rst with an appropriate description of the change (see Modifying change log).

Command Guidelines

[x] Each command and parameter has a meaningful description.
[x] Each new command has a test.

(see Authoring Command Modules)

Codecov Report

Merging #3888 into master will decrease coverage by 0.01%.
The diff coverage is 72.72%.

@@            Coverage Diff             @@
##           master    #3888      +/-   ##
==========================================
- Coverage   70.93%   70.91%   -0.02%
==========================================
  Files         428      428
  Lines       27048    27070      +22
  Branches     4119     4123       +4
==========================================
+ Hits        19187    19198      +11
- Misses       6608     6620      +12
+ Partials     1253     1252       -1

Impacted Files
Coverage Δ

...ice/azure/cli/command_modules/appservice/custom.py
73.44% <72.72%> (-0.02%)
:arrow_down:

...c/azure-cli-testsdk/azure/cli/testsdk/utilities.py
72.72% <0%> (-13.64%)
:arrow_down:

src/azure-cli-core/azure/cli/core/cloud.py
70.9% <0%> (-2.28%)
:arrow_down:

...nent/azure/cli/command_modules/component/custom.py
16.23% <0%> (ø)
:arrow_up:

...li-cloud/azure/cli/command_modules/cloud/custom.py
14.45% <0%> (ø)
:arrow_up:

...dback/azure/cli/command_modules/feedback/custom.py
31.25% <0%> (ø)
:arrow_up:

src/azure-cli-core/azure/cli/core/util.py
70.06% <0%> (ø)
:arrow_up:

...re-cli-acr/azure/cli/command_modules/acr/_utils.py
85.85% <0%> (+3.03%)
: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 4ba0a0b...6940699. Read the comment docs.

@derekbekoe, could you please take a look at this PR?


In the install script, let someone update key packages without setting up any API keys

This would be a "shortened" version of the install script that just facilitates updating everything at once, without questions that change the config.yml


Crash on TradeOfferChange

I am running 6f5a769da86a06641c59549ce56d96da06861eff
0|tf2autobot-1  | 2020-11-12 13:34:30 info: Message sent to <name> <id>/pre ❌ Ohh nooooes! The offer is no longer available. Reason: The offer has been active for a while. If the offer was just created, this is likely an issue on Steam's end. Please try again..
0|tf2autobot-1  | 2020-11-12 13:34:31 info: Message sent to <name> (<id>): /pre ✅ Success! The offer went through successfully.
0|tf2autobot-1  | 2020-11-12 13:34:31 trade: Offer # with <id> has been accepted.
0|tf2autobot-1  | 2020-11-12 13:34:31 error: TF2Autobot crashed! Please create an issue with the following log:
0|tf2autobot-1  | package.version: 1.6.7; node: v12.19.0 win32 x64}
0|tf2autobot-1  | Stack trace:
0|tf2autobot-1  | TypeError: Cannot read property 'length' of undefined
0|tf2autobot-1  |     at MyHandler.onTradeOfferChanged (C:\tf2autobot\dist\classes\MyHandler.js:1441:41)
0|tf2autobot-1  |     at C:\tf2autobot\dist\classes\Trades.js:547:30
0|tf2autobot-1  |     at tryCatcher (C:\tf2autobot\node_modules\bluebird\js\release\util.js:16:23)
0|tf2autobot-1  |     at Promise.successAdapter [as _fulfillmentHandler0] (C:\tf2autobot\node_modules\bluebird\js\release\nodeify.js:22:30)
0|tf2autobot-1  |     at Promise._settlePromise (C:\tf2autobot\node_modules\bluebird\js\release\promise.js:601:21)
0|tf2autobot-1  |     at Promise._settlePromise0 (C:\tf2autobot\node_modules\bluebird\js\release\promise.js:649:10)
0|tf2autobot-1  |     at Promise._settlePromises (C:\tf2autobot\node_modules\bluebird\js\release\promise.js:729:18)
0|tf2autobot-1  |     at _drainQueueStep (C:\tf2autobot\node_modules\bluebird\js\release\async.js:93:12)
0|tf2autobot-1  |     at _drainQueue (C:\tf2autobot\node_modules\bluebird\js\release\async.js:86:9)
0|tf2autobot-1  |     at Async._drainQueues (C:\tf2autobot\node_modules\bluebird\js\release\async.js:102:5)
0|tf2autobot-1  |     at Immediate.Async.drainQueues (C:\tf2autobot\node_modules\bluebird\js\release\async.js:15:14)
0|tf2autobot-1  |     at processImmediate (internal/timers.js:461:21)

Thanks for the report. Releasing v<IP_ADDRESS>.


Can not launching my app on real devices ,just flash on the screen.

hello,guys
While launching my app via command in logs:
/Users/jerry/Documents/android/sdk/platform-tools/adb -s GI7T7SY5WODQLF6S shell am instrument -e main_activity com.viplux.fashion.ui.SplashActivity -e server_port 8080 io.selendroid.com.viplux.fashion/io.selendroid.ServerInstrumentation
The real device just flash ,can not launching good.
And I use selendroid-test-app-0.12.0.apk  It's ok.
selendroid version:0.12.0

I'm closing this issue due to inactivity. If you still experience the issue described on the latest release of selendroid feel free to file a new issue. If you do please provide a concise reproducible test case and describe what results you are seeing and what results you expect.


Two recursive proofs barely do not fit into WASM limit in new versions of Nargo

Problem
If one wants to verify big circuits in Noir, they need to split them into smaller one to avoid WASM 2**19 limit.
There are two strategies then on how to combine them.

Linear
Binary tree
The second strategy allows to parallelise the proving, but requires two recursive proofs verifications to be able to fit into the WASM limit. It used to be the case with nargo 0.22.0, but not the case with nargo 0.27.0 because of a small perf regression on verify_proof.

Happy Case
Would be nice to be able to fit two recursive proofs into 2**19 gates
Project Impact
Nice-to-have
Impact Context
No response
Workaround
Yes
Workaround Description
Don't parallelise prooving
Additional Context
No response
Would you like to submit a PR for this Issue?
None
Support Needs
No response

Thanks for submitting the Issue 🙌 closing as duplicate of https://github.com/noir-lang/noir/issues/4409, follow the discussion there on updates and developments.


&amp, < and > are not decoded

We need to do some minimal HTML entity decoding, at least of &<>.

easy to repro by sending a slack message like https://www.github.com <--- here


Atmos perfomance

Multithreaded atmos
Data exchange: from game to atmos and from atmos to game via two data objects (like opengl doublebuff). Or transactions?


Requesting uninstall option in Makefile

After building from source for Linux, I noticed there is no uninstall option.
Could this be provided or is there already a built-in uninstall method I'm overlooking?

There's no make uninstall provided, but Malmo doesn't put anything anywhere except its own install folder, so there's no special work involved in uninstalling it - either just delete your build folder, or use the build/install_manifest.txt file - this is a list of all the files that got created as part of make install.
(You can do something like xargs rm < install_manifest.txt)
I hope that's sufficient for your needs, but if there's a compelling reason why we should add a make uninstall option, please reopen this issue and let us know, and perhaps we'll revisit this.
Cheers!


Fix windows_certificate functional tests under buildkite

Description
After merging #9427, which allowed tests that were limited to appveyor to run elsewhere, we're seeing failures of the windows_certificate functional tests. For example:
https://buildkite.com/chef-oss/chef-chef-master-verify/builds/2992#b1201dfd-b461-4ae0-a0ea-a1f5bcec237e/819-8366
1) Chef::Resource::WindowsCertificate action :create Again adding the same certificate Idempotent: Does not converge while addition
--
  | Failure/Error: expect(win_certificate).not_to be_updated_by_last_action
  | expected `<windows_certificate[ChefFunctionalTest] @name: "ChefFunctionalTest" @before: nil @params: {} @provider: nil @allowed_actions: [:nothing, :create, :acl_add, :delete, :fetch, :verify] @action: [:create] @updated: true @updated_by_last_action: true @source_line: nil @guard_interpreter: nil @default_guard_interpreter: :default @elapsed_time: 0.3965711 @store_name: "Chef-Functional-Test" @source: "C:/bkb1201dfd/spec/data/windows_certificates/test.cer">.updated_by_last_action?` to return false, got true
  | # ./spec/functional/resource/windows_certificate_spec.rb:132:in `block (4 levels) in <top (required)>'

and
8) Chef::Resource::WindowsCertificate action: verify When a certificate is present For a valid thumbprint Displays correct message
--
  | Failure/Error: expect(stdout.string.strip).to eq("Certificate is valid")
  |
  | expected: "Certificate is valid"
  | got: "Certificate not valid"
  |
  | (compared using ==)
  | # ./spec/functional/resource/windows_certificate_spec.rb:289:in `block (5 levels) in <top (required)>'

Reproduce these failures and correct or replace the tests.

These tests have been disabled here temporarily so we can continue to work on other issues and still get builds passing to current:
https://github.com/chef/chef/pull/9459
These are the individual tests that need investigation.


pages/learn: Add fan profile and power mode information

... for Jetson Orin devices running Jetpack 6
Change-type: patch

Please make sure to read the CONTRIBUTING document before opening the PR for relevant information on contributing to the documentation. Thanks!

@vipulgupta2048 @cywang117 anything else to change on this PR, can we merge it?


Add Decals example

Hello I need feedback and code review on this PR. This addresses #292 I added a new decal_from_url component per @FredrikNoren suggestion at:
https://discord.com/channels/894505972289134632/1078283561540530216/1096581219925377195
The example seems to render okay:

I suspect this can be cleaned up or migrated. I left the old decal component in the code because there were other crates that were dependent on it like the prefab. Not sure how to decouple or migrate the other crates. Please offer some guidance.

Created an issue for the rendering issue here: #352


Move change status from configuration to a proper task

The need is to change the status of files only.
With the actual method, you can't and are forced to activate at least one task on the list.
Creating a proper change status task would also allow to use the direct edit (by openMCDF) to change the status in a blazing fast way

I see your point.  One advantage of manipulating the Status in the configuration is that someone can change a file to Available, say for updating drawing views, then change it back.  To me, that is still needed.
However for the case you describe, it isn't.  Let me take that one.  I already planned on changing DM to OLE on the configuration side.  I can easily add a Status Change task that is compatible.  Sound OK?

What if we implement the status change in the edit properties task?

I'm open to it, but wonder if a separate task might be cleaner.
For one thing, Edit Properties can run in SE mode -- you can't change from a Read Only status that way.  We could check for that condition, but it sounds convoluted.
Also, as you know, two properties must be set for a status change.  We could deal with that, too,  But would then probably need to check that the user isn't already setting both on their own.
One more thing.  I kinda think a status change should run separately from other tasks.  I could easily add a flag for that.  That way I could run it in OLE mode without further checks.
Anyway, those are my thoughts at the moment.  Comments welcome!

A separate task would be clearer for sure, I agree to go for it.
One more thing to consider about status change is the user that commits the change. I'm not sure what property handles that value, will have to check

Added TaskSetDocumentStatus.  Help topic, FYI.  Minimally tested so far.  Let me know if you run into anything.
I didn't add Author, Change Date, or any other property updates, but can easily do so if needed.

Changing the status with Solid Edge creates these unnamed two properties that represent the Author and DateTime of the Status

In a new file, those two properties are not present therefore a check is needed before update\create them
Perhaps the UI could have a field where manually fill the user to use to sign the status change

I like the idea of having the user choose the property fields manually.  I'll give that a try.

Added updating of additional properties.  (RMB any textbox to select properties from your Favorites list.)
Note, if the task list already has TaskSetDocumentStatus, it will have to be removed and re-added.  There are some new Property fields, which the previously saved task will not have.


Where would you like us to post results?

I ran Breakout overnight - it's slow but looks like it is learning:
https://gym.openai.com/evaluations/eval_qw6rVeMThW8SUVoIGyEg
Note it took 9hrs 35 mins - for 5000 ‘games' (8.7 games per min)
Where would you like us to post results?

You should try letting it run for more number of games. You should be able to get better performance, I think. Feel free to add this in either the root README.md or a results folder. Thanks for sharing!


Bot Doesn't Start. Pyrogram works but not PTB

Expected Behavior
On sending /start the bot should trigger start.py and invoke start method
Current Behavior
/start doesn't return anything. But start.py is being triggered.
Possible Solution
Changing the position of contents of start.py inside __main__.py
Steps to Reproduce
N/A
Context (Environment)
N/A

@IzumiCypherX @Mynameisshekhar this issue is to be resolved. I'll look into it after 12 Dec


View page updates

updated App.vue for title change across all app pages
updates to NFTGet function in order to pass and apply a custom filter NFT function to reduce the list of NFTs returned
adjustments to NFTForm.vue to remove form submission and instead show list of NFTs by default Authority after loading
updates to globals

merging; all changes approved


Sign from a .pfx certificate

Hi, sorry for the newbie request.
I know that is possible to convert a pfx file to the pem format with PyOpenSSL, but what about override the sign with the possibility to receive a pfx file?
Nice library, a good alternative to pyxmlsec :+1:

Would be nice to have here too. It would also be nice if the sign API could simply take some existing X509/PFX Objects from pyopenssl/cryptograpy instead of requiring PEM serializations.

PRs welcome :)

With https://github.com/kislyuk/signxml/commit/bffe24544a4dede9e33282e9b14a187d024dd9f4, you can now pass a PyOpenSSL X509 object directly to the verify method.
The sign method already accepts any cert object that can be an input to PyOpenSSL's dump_certificate() (https://github.com/pyca/pyopenssl/blob/master/OpenSSL/crypto.py#L1656). You have to provide the key separately.

@danimaribeiro were you able to use this lib to sign the nfes? I'm in a point where I sign the xml and SEFAZ return "Rejeição 297 - Assinatura difere do calculado". I'm trying to use this instead o using pysped, because I cannot make pyxmlsec work with python 3...
thanks

I didn't try yet.
I'm implementing a new lib for it: https://github.com/danimaribeiro/PyTrustNFe
feel free to add your code to it, then we can test with signxml.

Heys guys @DanielFerreiraJorge and @danimaribeiro  from Brazil, do you have any example to sing a Nfe XML?  I had the same problem as @DanielFerreiraJorge  had, I'm using Python3 and I just want to sign the Nfe XML, do you have any example?

Sorry, I don't have examples with the signxml yet. I'm still using pyxmlsec.
https://github.com/danimaribeiro/PyTrustNFe/blob/master/pytrustnfe/servicos/Assinatura.py

I saw you use xmlsec, do you use Python 2? I'm using Python 3.4 and I can't install it, no way! The problem for me using these libs is that all of them use xmlsec behind the scenes, maybe it's my fault but I already tried a lot of tutorials about that and I couldn't make it works!

Yes, python 2.7, PyXmlSec is not maintaned since a long time ago.
Check this out: https://github.com/mehcode/python-xmlsec
If not working, I think you can provide a test case here, with a xml signed with pyxmlsec in python 2, and the same xml signed with signxml. Probably I won't have enough skills to fix the problem, but we can look at the code to see if we spot the problem.

Hey @danimaribeiro  the link you posted I checked this issue https://github.com/mehcode/python-xmlsec/issues/6 and this is exactly what happened to me, and the solution was the version of Cython as Nastya-Kruglikova said. Now the xmlsec was installed, I hope everything will work, thank you


Dismissal after performing a segue will undo the seque

If you perform a segue to a new ViewController while a Loaf is being shown the app will navigate back to the original ViewController upon dismissal (after the set interval).
It would be nice to be able to cancel all Loafs before performing a segue.

Unfortunately, this is a known issue ATM. I will be looking into this and hope to have a fix in the next release.

Impacting me as well... I'm calling self.dismiss(animated: true, completion: nil) and Loaf undoes that..

Any update on this?


⚠ Bofrost.at has degraded performance

In a280634, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1746 ms

Resolved: Bofrost.at performance has improved in 99a203c.


Support body data in POST requests

It is quite common to pass data via body in POST requests nowadays, having a -d flag à-la curl would be really useful :). As a remeinder, -d works in 2 ways:

direct mode -d'my custom embedded data'
file mode<EMAIL_ADDRESS>

Sure, this is something to consider. Do you expect the data be slowly served as well e.g. for slow POST, or enable this for only slow read type of attacks?
I can see it be useful for slow read, where a specific body data can trigger a special handling path, but don't see a benefit of slowly serving specific data. But who knows..

In our specific case, we want slow read on the response body. The body of the request is used such that the server doesn't answer back a 5xx.
Specifically, we want to see how timeouts configuration in our server behaves with clients that slowly slurp response body.

I can see another practical use-case here - if the server has added a WAF mitigation that looks for signs of non-legitimate request bodies and blocking those,  then this feature could be used to test around that.


SMO version 150 documentation is missing

Is your feature request related to a problem? Please describe.
I am trying to use SMO scripter to extract table DDL from source tables.  I am looking for documentation on SMO v 150, which is the minimum version required for connecting to Azure Managed instances
Describe the solution you'd like
I'd like some documentation on the ServerConnection that explains if it is possible in v150 to use Azure Active Directory Authentication to connection to Azure from SMO.
Describe alternatives you've considered
My alternative is to use .NET Sql connections (which support AAD) and try and describe objects through dictionary objects.
Additional context
Add any other context or screenshots about the feature request here.

@jeffshumphreys I've moved this issue to that repo: https://github.com/MicrosoftDocs/azure-docs/issues/33135


Error --no-duplicates arg

Hello,
I have an issue when I use the argument "--no-duplicates" in a entry or folder creation
An exemple of what I did :
$ pleasant-cli create entry --no-duplicates --data '
{
    "Username": "test",
    "Password": "password",
    "GroupId": "xxxxxxxxx"
}'

And I have this answer :
json: cannot unmarshal object into Go struct field Folder.Children.Credentials of type string
When I remove the argument, the command works
I use the last version 0.0.2 on a Linux machine
I already try to run it on other Linux machine but same problem
Thx for you help

@Midaxess thank you for opening an issue.
Your entry requires a name, else --no-duplicates will not be able to look for an entry with the same name to see if it already exists.
A valid JSON structure for this command would be:
$ pleasant-cli create entry --no-duplicates --data '
{
    "Name": "TestEntry",
    "Username": "test",
    "Password": "password",
    "GroupId": "xxxxxxxxx"
}'

Upon initial creation (assuming TestEntry does not already exist), creation will succeed. If the command is repeated, you'll receive error: duplicate entry found, skipping creation.
Please let me know if that helped!

Thanks for your answer,
I did your command :
pleasant-cli create entry --no-duplicates --data '
{
    "Name": "TestEntry",
    "Username": "test",
    "Password": "password",
    "GroupId": "xxxxxxxxx"
}'

But same issue.
When I run it when my entry doesn't exist and I always have this :
json: cannot unmarshal object into Go struct field Folder.Children.Credentials of type string
Same when I create the entry without "--no-duplicates" and then add it to check whether it exists.
I tried these commands on 2 different Linux machine.

Interesting. I am unable to reproduce this error on my system. Probably I'll need some more information:

What version of Pleasant Password Server are you using? Does that support API version v5?
Are you using the correct version of pleasant-cli for your system?
Can you try executing a pleasant-cli get folder --id <id> for the folder the entry is in? Does that work? If so, it would help to have an idea of the output of that command. Feel free to redact any sensitive information from it.

It seems that the server version is <IP_ADDRESS>
I'm on Debian 11 / 12 so I use pleasant-cli_0.0.2_linux_x86_64.tar.gz
Yes it works

I've just realised that I only get the error when I try to create an entry in a folder that contains subfolders.
For example: I just have the folder test-pleasant-cli which contains nothing. If I try to create an entry, no problem. However, if I create a folder in this one, for example sub-test-pleasant-cli, and I try to add an entry in test-pleasant-cli, I get an error.
Let me know if you still need output of my command

Thanks. I just tested that exact case - creating a subfolder in 'folder-a' called 'folder-b', and then in 'folder-a' create an entry with the subfolder being present. At that point, the error still does not occur. However, once I add an entry into 'folder-b' and then try to create an entry in parent 'folder-a', I get the same error.
I need to investigate where it's coming from but at least I can reproduce it now. I'll provide an update once I fix the root cause of the error.

@Midaxess I have just released v0.0.3 which should fix the issue you are encountering. Please test with that version and let me know if it works!

Works like a charm !
Thank you for your responsiveness.


Migrate enhanced abilities into new character envelopes

Our Mindthief retired recently in my regular group, and I successfully tested a process for baking her enhanced cards into all future Mindthief boxes. (Pull a fresh Mindthief from the template envelope, replace its ability cards with the enhanced cards, reset the Mindthief envelope, drop the altered Mindthief box into the now-empty envelope.) It would be nice for the campaign manager to be able to populate the character template envelopes on load with enhanced versions of the classes' ability cards, both for classes in active play and for inactive/retired characters.
My current workaround is to import the altered envelopes into a fresh copy of the mod as a Saved Object. This works for now, but I'm curious about whether this process could be automated in the campaign manager, given that it's already storing enhancement data for active characters.

Yeah, I was thinking about changing the enhancement loading process and get rid of the waiting (which sucks, as you already found out :-)) by simply replacing the character box instead of adding the stickers after everything is done.


[cxx-interop] Do not always treat types with user-declared copy constructor as safe

std::set defined insert method with a return type std::pair<iterator, bool>. This type is unsafe to use in Swift. However, it is currently treated as safe, and insert is not renamed to __insertUnsafe, because std::pair defines a copy constructor.
Let's not treat a user-declared copy constructor as a guarantee of safety.
rdar://109529750

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test


rc JSON parsing errors when path contains single quote

What is the problem you are having with rclone?
When passing commands to rclone as remote control (rc) and one of the parameters is a path that contains a single quote (e.g. as an apostrophe) there doesn't seem to be a way to escape it when sending the path as JSON. See below for examples
What is your rclone version (output from rclone version)
rclone v1.46

os/arch: linux/amd64
go version: go1.11.5

Which OS you are using and how many bits (eg Windows 7, 64 bit)
Linux 64bit, MacOS 64bit
Which cloud storage system are you using? (eg Google Drive)
not applicable
The command you were trying to run (eg rclone copy /tmp remote:tmp)
rclone rc operations/list --json '{"fs":"/", "remote":"big-storage/Ben's Test"}'
or
rclone rc operations/list --json '{"fs":"/", "remote":"big-storage/Ben\'s Test"}'
or any other similar command (e.g. sync/sync) which may have a path in it with single quotes. Clearly this is due to the fact BASH is already using single quotes to take in the JSON string so single quotes can't be used inside the string.
Note that using params works:
rclone rc operations/list fs="/" remote="big-storage/Ben's Test"
Produces the expected output:
{         "list": [                 {                         "IsDir": false,                         "MimeType": "application/octet-stream",                         "ModTime": "2019-02-19T18:47:39.738305018-06:00",                         "Name": "testfile",                         "Path": "big-storage/Ben's Test/testfile",                         "Size": 0                 }         ] }
Also, using double quotes to encase the JSON string and then escaping the double quotes inside the string also works:
rclone rc operations/list --json "{\"fs\":\"/\", \"remote\":\"big-storage/Ben's Test\"}"
So maybe the solution is simply to update the documentation for rc to show how to escape JSON for the types of characters that may be encountered in paths. This would likely require further testing of characters that could exist in paths to verify they can all be escaped properly.
A log from the command with the -vv flag (eg output from rclone -vv copy /tmp remote:tmp)
not applicable

This is a bash problem really, rather than an rclone problem...
The --json flag is there for testing - it isn't wildly convenient typing JSON in on the command line!
An alternative might be --jsonfile to read from a file or from stdin.

So maybe the solution is simply to update the documentation for rc to show how to escape JSON for the types of characters that may be encountered in paths. This would likely require further testing of characters that could exist in paths to verify they can all be escaped properly.

Perhaps recommending the single quotes, but stating: if you want to add a single quote ' into the --json parameter while using bash, then use '"'"' instead (that closes the string, adds a single quote in double quotes, then opens the string again).

This looks to be answered so closing out.


Add informative icons to the create  button

Closes #2645
To avoid when you create the game with the wrong parameters, which you did not notice

Hmm looking at this I am not sure how helpful the icons are as it isn't entirely intuitive to me on first glance what game setting each icon is related to. Also shouldn't there be one for enforced rating range as well?

Even the tooltips didn't help you? 🤔

It looks like we need another way to communicate it. Or just close the issue, since the issue is not common.

Or, in general, it is better to redo the design of the upper part of the window, so that you can pay attention to significant things.

Hmm I didn't even see the tooltips really and I don't know if it is intuitive that tool tips would be there

I think making it so that the visibility changing options are more obvious when enabled might be better

I think making it so that the visibility changing options are more obvious when enabled might be better

Okay.

I do not have an idea how to do it. I will open pr if an idea comes up.

I think making it so that the visibility changing options are more obvious when enabled might be better


Refactor gha workflows

Done via https://stackoverflow.com/questions/66598692/how-to-cache-paketo-io-cloud-native-buildpacks-pack-cli-builds-on-github-actions/66598693#66598693


gulpfile breaking npm install

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
npm install fails at the gulp step when installing dependencies
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem. You can use this jsfiddle to get started: https://jsfiddle.net/stopachka/m6z0xn4r/.
Tried this with npm versions 4.0.2, 3.10.10.

><EMAIL_ADDRESS>prepublish /Users/pritikanilaratna/Sites/draft-js
> npm run build

><EMAIL_ADDRESS>build /Users/pritikanilaratna/Sites/draft-js
> gulp

/Users/pritikanilaratna/Sites/draft-js/gulpfile.js:107
        ),
        ^
SyntaxError: Unexpected token )
    at Object.exports.runInThisContext (vm.js:76:16)
    at Module._compile (module.js:542:28)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)
    at Liftoff.handleArguments (/Users/pritikanilaratna/Sites/draft-js/node_modules/gulp/bin/gulp.js:116:3)
    at Liftoff.<anonymous> (/Users/pritikanilaratna/Sites/draft-js/node_modules/liftoff/index.js:198:16)

npm ERR! Darwin 15.6.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "build"
npm ERR! node v6.9.1
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>build: `gulp`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script 'gulp'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the draft-js package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     gulp
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs draft-js
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls draft-js
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/pritikanilaratna/Sites/draft-js/npm-debug.log

npm ERR! Darwin 15.6.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install"
npm ERR! node v6.9.1
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>prepublish: `npm run build`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>prepublish script 'npm run build'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the draft-js package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     npm run build
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs draft-js
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls draft-js
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/pritikanilaratna/Sites/draft-js/npm-debug.log```

**What is the expected behavior?**
`npm install` completes installing dependencies correctly

**Which versions of Draft.js, and which browser / OS are affected by this issue? Did this work in previous versions of Draft.js?**

Hey @nilaratna, it seems that this trailing comma got introduced in #1644. This won't work in Node 6 (which you're using). I can see that the Travis build is using Node 8 (which uses v8 5.8, so does support those types of trailing commas) – if you switch to Node 8 it should work fine.

@thibaudcolas Awesome, that worked. I was thrown off by the versions listed in package.json. Is there a plan to update so as to indicate that 6.x is not supported?
"devEngines": {
    "node": "6.x || 8.x || 9.x",
    "npm": "2.x || 3.x || 5.x"
  }

Great! Good question :) I have submitted a PR going in that direction (#1652), we'll see what happens next.


Fix ext authz deny all traffic if encountering error with the provider

Please provide a description of this PR:
Fix https://github.com/istio/istio/issues/46951.
I think this will loose the rbac, however it makes sense to only deny rules that are delegated to the ext authz provider.

@hanxiaop: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
ac0175b0e001daba39716ce55ff85cfaf7ce615a
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
5d424838413bed31f149ae529775cb267df001a1
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
5d424838413bed31f149ae529775cb267df001a1
link
true
/test gencheck

lint_istio
89bc3788e63dc031da9e7aa6d9e020f0b45f4d36
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
bcd1f697872861a1f44ca1b12cf2515c589e2188
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
bcd1f697872861a1f44ca1b12cf2515c589e2188
link
true
/test unit-tests

integ-pilot-istiodremote-mc_istio
bcd1f697872861a1f44ca1b12cf2515c589e2188
link
true
/test integ-pilot-istiodremote-mc

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


JKNS-397: Fix USE_JAVA_VERSION environment variable

Fixes #1712
Removed code from #1666 is added back to allow use of other Java versions.

/docs-approved
/px-approved
/qe-approved

/cherry-pick release-4.12

/cherry-pick release-4.13

/cherry-pick release-4.14

/cherry-pick release-4.15


Add the Project Creation automation for GitLab

Once #10 is complete, create an automation around Project Creation in GitLab when creating the project in Imbi. Leverage the slugs for Namespace and Project Type.

@gmr I can't link PRs to this issue but https://github.com/AWeber-Imbi/imbi/pull/56 closed it.

Closed by https://github.com/AWeber-Imbi/imbi-api/pull/73


KAFKA-13999: Add ProducerCount metrics (KIP-847)

This is the PR for the implementation of KIP-847: https://cwiki.apache.org/confluence/display/KAFKA/KIP-847%3A+Add+ProducerIdCount+metrics
Add ProducerIdCount metric at the broker level:
kafka.server:type=ReplicaManager,name=ProducerIdCount

Added unit tests below to ensure the metric reported the count correctly.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Thanks @anatasiavela and @artemlivshits -- I'll take a look at the build when it completes, but this looks pretty good so far.

Thank you for the review @divijvaidya! I've addressed your comments and made changes accordingly, so please take a look whenever you get the chance. Thanks!

Is there a reason we don't change the HashMap to a Concurrent*Map and just call size() on it? This way it becomes very clear that we expect some sort of a concurrent operation to happen on the map and we would like it to be ordered with respect to other operations and we do not have to rely on people knowing they need to interact with the producers map via additional functions?

clolov I see your point about people not knowing to use those methods -- maybe we can add some documentation.
However, I'm wondering if changing the map will affect performance for the reading and writing of the map. I don't think it makes sense to take a hit there. We will also be reading the size very frequently for the metric.

Is there a reason we don't change the HashMap to a Concurrent*Map and just call size() on it? This way it becomes very clear that we expect some sort of a concurrent operation to happen on the map and we would like it to be ordered with respect to other operations and we do not have to rely on people knowing they need to interact with the producers map via additional functions?

Using ConcurrentMaps would be ideal (we won't have to acquire a lock to perform read), but the size() operation of a concurrent map may produce stale data (case when it's being modified concurrently). The current approach guarantees that size is strongly consistent with the data stored in the map.

@anatasiavela Could you rebase the PR?

The current approach guarantees that size is strongly consistent with the data stored in the map.

That is not required for a metric though.
That said, I didn't look at the details of the map we are discussing here.

Proposal: Using concurrent map for ProducerStateManager.producers (currently a mutable.Map)
Pros

Simplifies code (prevents future bugs by accidental update to map without updating metrics) to implement KIP-847
Does not have any locking concerns since calculation of size() in a concurrent map does't require locks. From Java docs for concurrentMap:

Bear in mind that the results of aggregate status methods including size, isEmpty, and containsValue are typically useful only when a map is not undergoing concurrent updates in other threads. Otherwise the results of these methods reflect transient states that may be adequate for monitoring or estimation purposes, but not for program control.

Cons

The metrics for number of producers may be inaccurate when multiple threads are modifying the map concurrently. producers map is modified during truncation (when the replica is catching up) or when a producer is expired due to timeout or on every completed transaction. That means it is modified fairly frequently. Hence, possibility of the producerId metric displaying inaccurate (usually off-by-one since there are at max 2 threads modifying this map) information is fairly high.

My opinion
In cases when we have just one (or two) producers, this metric would be highly unreliable (not just stale) as it provides an "approximation" of size(). It is not un-common to produce data from limited set of producers (with a large number of consumers) and hence, I would incline towards sticking to current approach of keeping this metric accurate.
Having said that, I don't have a strong opinion here. We can always come back and make this metric accurate if we find that inaccuracy is causing problems while troubleshooting.

Has anyone checked that we always acquire a lock when we call methods like activeProducers and isEmpty? I wonder if this class has thread safety bugs.

Has anyone checked that we always acquire a lock when we call methods like activeProducers and isEmpty? I wonder if this class has thread safety bugs.

@ijuma I actually did take a peak when working on KIP-890 and other producer ID fun. There is a lock on every usage of active producers (except its usage in a log message).
As for isEmpty, there are two usages (one in UnifiedLog -- to rebuild the log and one in LogLoader) and neither is protected via lock.

@divijvaidya I'm also a bit confused by this comment:

In cases when we have just one (or two) producers, this metric would be highly unreliable (not just stale) as it provides an "approximation" of size(). It is not un-common to produce data from limited set of producers (with a large number of consumers) and hence, I would incline towards sticking to current approach of keeping this metric accurate.

We will keep the IDs of any producer used within the last day or so. If there are two producers running in a steady state, the metric will stay steady.

@jolshan From what you're saying, there are some thread safety bugs, but their impact is likely minor and hence why we haven't noticed them. From my perspective, we really need to document the concurrency model for this class more clearly and the class would be more robust if we made it thread safe.

@ijuma @jolshan what I wanted to convey was a situation where the approximate nature of this metrics may cause problem during troubleshooting. But I was wrong. I read the code incorrectly at one place and wrongly assumed that this map is updated frequently
Let me write it as a FAQ.
What is the situation that you are worried about?
When we have only one or two producers, the approximate nature of the metric may display the producerId to be 0 when actually it should be 1. Having one producer is a realistic production scenario since there are use cases for Kafka where folks use Kafka primarily for fan out, i.e. single producer but a large number of consumers.
Would the metric be inconsistent only during concurrent updates to producers map (and stable for rest of the time)?
Yes
What is the likelihood that the metric is captured by the gauge at the same time when the map is undergoing concurrent update?
Not likely (this is where I was wrong in my earlier comment). Note that producer IDs are not added frequently and the map is modified only during expiration or addition. Hence, the likelihood of metric display at the same time when map is being mutated is low.
Is the metric only going to be stale (reflecting a state after some time of it's occurrence) or is it going to be inaccurate (even after some time of state change it may reflect the wrong state)?
It can be inaccurate. This is because when modifications are being done on concurrent hash map, they are pointer manipulation and depending on the state the internal data structure the size may reflect an inaccurate value.
I don't have any further concerns with using @clolov's proposal to use ConcurrentHashMap, in fact, that is the better approach here.
Thanks all for clarifying my doubts and answering my questions. Appreciate it.

@divijvaidya Since this PR is strictly focusing on the metric, I think it's reasonable for that change to be done in a separate PR. what do you think?

@anatasiavela the proposal (using concurrentHashMap for producers) impacts this change. Let me try to explain why.
Currently you are using producerIdCount and updating it on every add/delete to the producers map. Alternative path was to simply call producers.size() whenever we want to compute the metric. But you didn't choose this path because producers is accessed by multiple threads at the same time and we usually guard access to it via a lock. Hence, if you wanted to call producers.size() you would have to acquire a lock and that is not optimal for a mere metric calculation. Hence, your approach made sense. BUT if you change the producers to a concurrentHashMap, you wouldn't have to acquire a lock to call producers.size() when computing for metrics. Without acquiring a lock, the risk is that another thread may be mutating the producers map at the same time. As per the discussion above in the PR, that is acceptable to us. Hence, you can simply change the producers to a concurrentHashMap and remove the code logic to update producerIdCount on every add/remove. This simplifies the code greatly.
Let me know if that makes sense.

Thanks @anatasiavela for the PR. Apologies for any repeat, just wanted to ask if you would plan to have this merged before or after PR-13040? Many thanks!

@Hangleton We can try to get this merged before https://github.com/apache/kafka/pull/13040 if that helps.

@ijuma Do you have any concerns with switching to ConcurrentHashMap here?

One concern is that the ConcurrentHashMap could take up more space. This metric was introduced to try to get this memory usage under control.

One concern is that the ConcurrentHashMap could take up more space. This metric was introduced to try to get this memory usage under control.

The hash map in question here stores the number of idempotent producers. Which I assume is in "hundreds" in a busy cluster. As per this old benchmark https://stackoverflow.com/a/11104401 (newer JVMs may have even smaller footprint) we have an incremental usage of 8 bytes per entry, so we are talking about 800 bytes more? I think this is a reasonable tradeoff compared the out-of-the-box thread safety & code simplification that ConcurrentHashMap provides here. The tradeoff becomes even more appealing since the class in question, is accessed by multiple threads.

Which I assume is in "hundreds" in a busy cluster

We had cases of OOM due to number of producer ids, so we definitely don't want to increase memory consumption (and should look for ways to reduce it).

Would it make sense to decouple the switch to a ConcurrentHashMap from adding this metric? Given that there is no consensus, we could tackle this larger change separately if we want to. The current approach seems reasonable to me. What do you think?

Given the memory usage concerns, I think it's fine to decouple the two things. That said, we need to port this change to the Java implementation of ProducerStateManager as that PR has since been merged.

Thanks @jolshan for following-up. Not a problem about the merge. I was just mentioning it so that we know either PR will have to be rebased and updating accordingly.

@anatasiavela do you mind rebasing again 😅

Sorry these files keep changing @anatasiavela -- I will try to review as soon as you complete the rebase (feel free to just merge trunk into it)

Seems like there was a build falilure related to rat I restarted the build.

Tests look unrelated. Going to merge.

@mimaison I've marked the JIRA as resolved

Thanks!


Fix typo in tex command

A \prod command in the description of space S000107 had been capitalized, and as such it wasn't being displayed properly.

Apparently the build failed for this, @StevenClontz @jamesdabbs


Add ContractCall::call_owned(self)

Motivation
I was trying to call multiple contracts concurrently by aggregating calls into a vec, so that I can join_all(calls).await them later.
Since ContractCall::call(&self) takes an immutable reference, compiler prevents future from being created because future references ContractCall, which may not live long enough.
I’ve tried Box-ing ContractCall, but it didn’t quite work.

Solution
Add another method that takes ownership: call_owned(self).
That completely solved my problem.

PR Checklist

[ ] Added Tests
[x] Added Documentation
[ ] Updated the changelog

Will do in a couple of minutes

Done

@gakonst maybe merge? :)

can't you just wrap in an async move {} block that takes ownership?

Yeah, that should work. You think that call_owned(self) is redundant?

tbh, I think that making call() consume self might be ideal?

Not necessarily?.. In case you want to estimate gas and do a transaction, after which transaction may fail and you want to retry?..

well, the user intent for .call() is to read the chain. there's no usage I can think of in which the ContractCall is not dropped after the eth_call is executed. So fn call(self) is probably more in line with user expectation than fn call(&self)

In case it fails because of the provider and user wants to retry.

ContractCall is Clone so that user could clone it before calling  call() or send() 🤔

I think much better dev ergonomics for joins is probably worth slightly worse dev ergo for retries? 🤔

Joins can be solved with async move { ... } blocks, as you said. .clone()-ing the call for retries is not ideal.

slept on it. this would be a good place to implement IntoFuture, and keep call() and send() the same as they are now. my_call.into_future() is a lot more sensible to read than async move { my_call.call() }

Same page as Prestwich, let's replace with IntoFuture.

ContractCall has three async methods:

estimate_gas
call
send

I'm not sure how IntoFuture can handle not just one, but all three of the usecases.

I'd default to call()


799 Cleanup console logs

https://github.com/SamProf/MatBlazor/issues/799
Cleanup the console logs plus some minor code cleanup as I was looking for the issue.

@SamProf ,
This could use a review/merge


Out of memory issue with ~1200 AA protein on NVIDIA A100

Hello! I am really enjoying experimenting with Boltz, but finding that I run into out of memory errors on my 40GB A100 (NVIDIA A100-SXM4-40GB) on any single proteins above about 1200 AA in length. I am using the most recent version of pip, 0.3.2 and just wondering if perhaps there were any parameter options to boltz (reducing sample steps, etc) or pytorch settings (such as reducing float sizes) which were worth trying.... or really anything else.
At first hiking up the shared memory available to my docker container worked, but now allowing 128gb, it seems like it is very likely a GPU memory issue. On a slightly smaller protein, boltz is taking 36GB of GPU memory.
Super excited to keep using this awesome tool, and great work to the team!

Relevant logs for context:
boltz-1  | Running predictions for 1 structure
boltz-1  | Processing input data.
boltz-1  | Generating MSA for /data/jobs/DATA-6/af_out_complexes/temp_MOT_HUM_RPP_B07_FAS1_FC_STD_pos1_MOT_HUM_RPP_B07_FAS1_FC_STD_pos2_MOT_HUM_RPP_B07_FAS1_FC_STD_pos3/MOT_HUM_RPP_B07_FAS1_FC_STD_pos1_MOT_HUM_RPP_B07_FAS1_FC_STD_pos2_MOT_HUM_RPP_B07_FAS1_FC_STD_pos3.fa with 1 protein entities.
boltz-1  |
boltz-1  | Predicting: |          | 0/? [00:00<?, ?it/s]
boltz-1  | Predicting:   0%|          | 0/1 [00:00<?, ?it/s]
boltz-1  | Predicting DataLoader 0:   0%|          | 0/1 [00:00<?, ?it/s]| WARNING: ran out of memory, skipping batch
boltz-1  |
boltz-1  | Predicting DataLoader 0: 100%|██████████| 1/1 [00:00<00:00,  1.07it/s]Number of failed examples: 1
boltz-1  |
boltz-1  | Predicting DataLoader 0: 100%|██████████| 1/1 [00:00<00:00,  1.07it/s]
boltz-1  |
boltz-1  | 2024-12-15 02:49:50,877 - boltz_watcher - WARNING - Command stderr:
boltz-1  |   0%|          | 0/1 [00:00<?, ?it/s]
boltz-1  | 100%|██████████| 1/1 [00:01<00:00,  1.10s/it]
boltz-1  | 100%|██████████| 1/1 [00:01<00:00,  1.10s/it]
boltz-1  | GPU available: True (cuda), used: True
boltz-1  | TPU available: False, using: 0 TPU cores
boltz-1  | HPU available: False, using: 0 HPUs

And parameter settings:
 2024-12-15 03:02:54,901 - boltz_watcher - INFO - Running command: boltz predict /data/jobs/DATA-1/af_out_complexes/temp_MOT_HUM_RPP_B07_FAS1_MON_STD_pos1_MOT_HUM_RPP_B07_FAS1_MON_STD_pos2/MOT_HUM_RPP_B07_FAS1_MON_STD_pos1_MOT_HUM_RPP_B07_FAS1_MON_STD_pos2.fa --use_msa_server --out_dir /data/jobs/DATA-1/af_out_complexes --accelerator gpu --recycling_steps 3 --sampling_steps 200 --diffusion_samples 1

Could you please make sure to run this in a new output directory and let me know if it happens again? I suspect that you may be using an MSA that was preprocessed prior to the latest version. We've run tests and are able to pass 1800AA on a 40G GPU.

Closing this for issue for now, let us know if you run into any problem using a new output directory!


Lidarr API client definition

This is one of the first step towards Music integration into Petio.

I should also mention that we follow a strict commit message structure, and that is conventional commits. I'm surprised you were able to commit at all without husky or commit-cli complaining.

I made the necessary adjustments:

Removed the commented out code.
Renamed the function.
Changed the commit messages.
Furthermore, I believe you're right about this PR not making total sense until the MusicBrainz integration. Let me convert this PR into draft.

The commit messages still do not follow conventional commits. https://www.conventionalcommits.org/en/v1.0.0/
An example would be Added Lidarr API client definition becoming feat(lidarr): added api definitions


Parse arguments from run phrase

run tests @brokered.clients.rhea.**

run tests

run tests

run tests @brokered.clients.rhea

run tests @brokered.clients.rhea.**

@lulf it's working :-) could you please review it?

How about “testcase=...”

run tests testcase=brokered.clients.rhea.**

@lulf Changed to testcase=

run tests testcase=brokered.clients.rhea.**


Adding  "days"

Added Days eg. "4d"
On construct can also specify hours per day for work day calculations.

Coverage increased (+0.4%) to 97.541% when pulling aee7da40d8fcfc231b57757851f6745aa696c26a on joMpire:Using-days-should-be-possible-too into 4b1fe107be4bf2256378bafb1b29e6b3e0545588 on kevinkhill:master.

Wow! Thank you so much for contributing this :smile:

It all looks great and I'm sorry its been sitting a while. I haven't been on here in a while.

Brillant! no problem on the wait :)


Update tests

Reorder imports for consistency
More importantly, move URLs, API keys, expected titles etc to constants at the top of each test
Minor style changes (e.g. remove braces where not required, don't name file objects after a metal :wink:)

don't name file objects after a metal

Lol. Tin is supposed to be short for "tracks in" when reading them from file. No wonder I still kept using tin (instead of tout) even when writing to file. :smiley_cat:
I'll take a better look soon at other changes but looks good to me on the surface.


Browsing lists and panels using vim navigation keys

Browsing lists and panels using vim navigation keys will be much natural for vim users.

Navigating lists with j and k
Moving to panel (projects, tasks, task-detail) with h and l

Based on a feedback on Reddit

One could also wish for the old WordStar diamond to navigate.
But …
I think it would be a better option to allow to use an external editor (via INI-file or commandline option). That way everybody could use their preferred editing environment.

@mwat56 Thanks for your recommendation.
Implementing external editor is in progress. It will open Task Detail in whatever editor is set in $EDITOR env variable, vim by default.
See #6

A little to initial plan - not gong to implement moving to right panel with l.
Because right side content depends on selection of left panel. So moving towards the right side panel should be based on selecting item with Enter.
Here is the plan I am going to implement now.
+------+----------------------+-----------------------+
|  P   |         T            |         D             |
|      |                      |                       |
|    Entr=>    ↓   ↑        Entr=>                    |
|      |      tab TAB         |                       |
|   <=Esc/h    j   k       <=Esc/h                    |
|      |                      |                       |
+------+----------------------+-----------------------+

So, what it's trying to visualize is -

Selecting an item with Enter will move you to right panel. That means -

Selecting a Project will load it's tasks and move to Tasks panel
Selecting a Task will load task detail and move to Detail panel

Use Esc or h (like vim) to move to left panel. Details to Tasks to Projects.
To navigate a list (Project list or Task list),

Use ↓ or j or Tab to go down
Use ↑ or k or Shift+Tab to go up


My code space is hanging

my codespace is hanging hence i cant comfortably use it. The outline is not appearing and i cant use and commands . Please help
Version: 1.84.2
Commit: 1a5daa3a0231a0fbba4f14db7ec463cf99d7768e
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Embedder: codespaces

Please file the issue against the github codespaces.


Rotation in iOS

Hi I have an issue with regards the rotation of the circle. I wanted it to be on the top to start with but rotation is not working on my end.
Here is my gist for it.
https://gist.github.com/renso3x/38e4b120404bd7f57fa2dc90951978fc
Here's what it looks like.

The property is called rotate not rotation.

Rather, use the standard svg transform attribute and the rotate transform definition. Such as here e.g. https://github.com/magicismight/react-native-svg-example/pull/7/commits/e5616d29fc707da68ea895d02a633bf78a4dd495#diff-b241035fb407e61e7ec1900126acf94cR23


.NET 6 update: Use gRPC in browser apps

@JamesNK  how do I update this to .NET 6 when Grpc.AspNetCore.Web is for .NET 5?

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

ID: 1c13a1c7-4683-a9a5-9ab3-e28d4506af26
Version Independent ID: ea3b00c8-8f09-c045-4641-dc3a0030d6e5
Content: Use gRPC in browser apps
Content Source: aspnetcore/grpc/browser.md
Product: aspnet-core
Technology: aspnetcore-grpc
GitHub Login: @JamesNK
Microsoft Alias: jamesnk

On a different thread more recently, James indicated that anything with 5.0 tag for gRPC is compatible with 6.0.
@JamesNK since we are at .NET 7 preview 4, it would be good if 6.0 (and eventually 7.0) versions could be indicated for the .NET gRPC related packages to cut down on confusion.