Automated cherry pick of #3663: fix: aws 安全组desc为必传字段

Cherry pick of #3663 on release/2.13.
#3663: fix: aws 安全组desc为必传字段

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.13@a77688f). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.13   #3664   +/-   ##
==============================================
  Coverage                ?   6.95%
==============================================
  Files                   ?     500
  Lines                   ?   83886
  Branches                ?       0
==============================================
  Hits                    ?    5838
  Misses                  ?   77552
  Partials                ?     496

Flag
Coverage Δ

#aFlag
6.95% <0%> (?)

Impacted Files
Coverage Δ

pkg/multicloud/aws/region.go
0% <0%> (ø)

pkg/multicloud/aws/vpc.go
0% <0%> (ø)

pkg/multicloud/aws/securitygroup.go
0% <0%> (ø)

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 a77688f...1c5afe7. Read the comment docs.

/lgtm

/lgtm

/approve


[Feature Request]Show all/running builds in same job

Hi all,
I love this plugin so much!
But when I running the same job twice, it will only show the latest build.
Wonder if there is a option that show all/running builds in one job.
Thank you so much!

Love the new design in 1.12!
Will close this issue

Great stuff, I'm glad you like it!


Consultation for the bugs in Human-machine interaction interfaces

I wonder to  know when  to fix the bugs in Human-machine interaction interfaces since the release of  latest version 0.1.2? Because we want to conduct qualitative analysis urgently.

We have so far only implemented the human-machine interaction interface on the CRS model TGReDial.

Hi,
We have so far only implemented the human-machine interaction interface on the CRS model TGReDial. If you need to conduct qualitative analysis, maybe you can complete your own interface as if which was already implemented in TGReDial System.
https://github.com/RUCAIBox/CRSLab/blob/a20d28296e36699e15629d4968c09d85ed0bda25/crslab/system/tgredial.py#L293


error when installing on linux mint 22

this is my output , it says i have less than 3 gb of ram , i have 32 , amd ryzen 9 7950x, amd radeon rx 7800xt
ic annot paste all the outpu , here is the link to my output https://katb.in/owoyohatayi

I read your log file. :+1:

Can you check with this command if awk is installed on your system, please?
awk --version

And can you run this little code: gpu_disk-test.sh.zip

The issue your old wine version exists also here: https://forum.winehq.org/viewtopic.php?t=38703

exact problem here.
from @SuperPi911 log:
./autodesk_fusion_installer_x86-64.sh: line 925: /home/rares/.autodesk_fusion/bin/autodesk_fusion_launcher.sh: No such file or directory
there is no file in ~/.autodesk_fusion/bin/ folder
i manually downloaded the launcher using:
$ cd ~/.autodesk_fusion/bin/
$ wget https://raw.githubusercontent.com/cryinkfly/Autodesk-Fusion-360-for-Linux/main/files/setup/data/autodesk_fusion_launcher.sh
$ chmod +x autodesk_fusion_launcher.sh
and start manually inside bin folder ./autodesk_fusion_launcher.sh

@NiklasVoigt But Autodesk Fusion works?

@NiklasVoigt But Autodesk Fusion works?

Only with this fix, as i said, the bin folder is empty

Okay! :+1: THX ... i check my script!

I have found my issue. For example (old):
curl -L https://raw.githubusercontent.com/cryinkfly/Autodesk-Fusion-360-for-Linux/main/files/setup/data/autodesk_fusion_launcher.sh -o "$SELECTED_DIRECTORY/bin"

... and here the changed code:
curl -L https://raw.githubusercontent.com/cryinkfly/Autodesk-Fusion-360-for-Linux/main/files/setup/data/autodesk_fusion_launcher.sh -o "$SELECTED_DIRECTORY/bin/autodesk_fusion_launcher.sh"

@NiklasVoigt Bug is solved in my installation script.

@NiklasVoigt Bug is solved in my installation script.
And do you also have problems with RAM detection, because I'm running Linux Mint 22 (KVM)?

No problems, im on manjaro

We mae some progress , fusion opens now ,but no browser is opening to sign in , my output https://katb.in/jiwowujisob

Which web browser is installed?

Which web browser is installed?

On my PC Firefox and chrome, on wine nothing

This desktop actually takes care of that file:
    cat >> $HOME/.local/share/applications/adskidmgr-opener.desktop << EOL [Desktop Entry] Type=Application Name=adskidmgr Scheme Handler Exec=sh -c 'env WINEPREFIX="$SELECTED_DIRECTORY/wineprefixes/default" wine "$(find $SELECTED_DIRECTORY/wineprefixes/default/ -name "AdskIdentityManager.exe" | head -1 | xargs -I '{}' echo {})" "%u"' StartupNotify=false MimeType=x-scheme-handler/adskidmgr; EOL
... and wine automatically accesses your standard web browser from Linux.

This desktop actually takes care of that file:
    cat >> $HOME/.local/share/applications/adskidmgr-opener.desktop << EOL [Desktop Entry] Type=Application Name=adskidmgr Scheme Handler Exec=sh -c 'env WINEPREFIX="$SELECTED_DIRECTORY/wineprefixes/default" wine "$(find $SELECTED_DIRECTORY/wineprefixes/default/ -name "AdskIdentityManager.exe" | head -1 | xargs -I '{}' echo {})" "%u"' StartupNotify=false MimeType=x-scheme-handler/adskidmgr; EOL
... and wine automatically accesses your standard web browser from Linux.

But it didn't open any browser to log in

Have you set your default web browser in your system?
For example on XFCE:

Yes , chrome , i tried setting firefox , but still nothing , browsers closed , not opening , browsers open no login tab appearing

In any case, the problem is not with the script, but rather something is set incorrectly in your system, which unfortunately I cannot understand.

sidenote: i used chrome today without problems. one time on manjaro and one time on arch

I have fixed the installer script! In order for the login to work under Linux Mint, xdg-utils had to be installed.

I wish you a nice evening. :100:


[EN] Feedback for: /web/tools/chrome-devtools/?hl=en

The "Discover the Panels" section gets cut off on mobile devices.

cc @pbakaus

Thanks for the feedback, we've got an update coming next week that should fix this.


Support for gettext/gettext v5

Feature Request

[x] Yes, I reviewed the contribution guidelines.

Describe your use case and the problem you are facing
For adding new file formats to scan like blade (that WordPress base themes use),
it would be nice to know what gettext/gettext version should be used.
Currently wp i18n uses gettext/gettext v4.8, v5 introduced some changes, Extractors were removed from the base package (and the concept renamed).
Describe the solution you'd like
What gettext/gettext release should be used in the long term? Are there plans to update to 5.x?

We can‘t use v5 yet because of differing PHP version requirements. As soon as this changes we are able to upgrade. See also #217 for some context.
WP-CLI only changes its requirements when WP core does, so there is no ETA on this right now.

@swissspidy: So for attempting to add blade.php support to wp i18n, it would make sense to use the v4 branch of gettext/gettext (and Extractors)?

Yeah for instance. You could also try to fork this command to use v5 if that‘s easier

@swissspidy: I just discovered that in the gettext 4.x branch is already a readily implemented Blade extractor:
https://github.com/php-gettext/Gettext/blob/4.x/src/Extractors/Blade.php
Hence implementing a PHP-Blade Extractor for wp i18n was quite easy, PR:
https://github.com/wp-cli/i18n-command/pull/304

Closing for now since we now have Blade support and we can update to gettext v5 (and reopen this ticket) once PHP requirements allow it

Just wanted to ask if this is planned sometimes soon, since the minimal php requirement for wordpress is now 7.4. Having a hard time following these issues, but i think they will be fixed with the upgrade:

https://github.com/wp-cli/i18n-command/issues/52#issuecomment-409281487
https://github.com/timber/timber/issues/1754

@JUVOJ No. The minimum PHP version for WordPress is still 5.6 for the foreseeable future.

@swissspidy / @schlessera: Can't we find a solution regarding this problem? Something around composer's suggest? Or some environment variable, or some creative that would unlock this whole stack of problem.
Keeping wp-cli i18n away from nice features (from gettext v5+) because we are stuck by PHP 5.6 is insane.
If you relax the version restriction, PHP 5.6 user would get gettext v4 while PHP 7/8 users would get the v5. So it's "just" a question of supporting both API then. Would you accept this? It would be a bit dirty code-wise but not impossible at all.

If it's possible to support both versions in a maintainable way, that sounds like a viable middle ground. Maybe best to bring this up in the next office hours.

Since we are finally dropping support for PHP 5.6 in WordPress 6.3 could you take this into consideration again @swissspidy?
https://make.wordpress.org/core/2023/07/05/dropping-support-for-php-5/

Not yet, because gettextv5 requires PHP 7.2+ and WP still 7.0+
See also https://make.wordpress.org/cli/2019/01/15/wp-cli-php-requirements-strategy/
Again: If it's possible to support both versions in a maintainable way, that sounds like a viable middle ground

@swissspidy : Could you please have a look at the v5 diff https://github.com/wp-cli/i18n-command/pull/355/files. The change for dual v4/v5 support are obvious (but not trivial):

composer.json: ^4|^5
src/IterableCodeExtractor.php: self::$v4 ? new Translations() : Translations::create(); and alike for addExtractedComment() and so on.

Doable, but that would basically double the size of the diff.
(I'm really sad a whole community of Timber (Twig) users are waiting for this to translate Twig templates and we are all held back because of a few (very hypothetical) users of PHP 7.1 using wp-cli i18n command in their old (but shell-enabled) hosting provider.
The choice of wp-cli to blindly and inflexibly stick to WP PHP requirements is IHMO a very bad one)

It seems like a good starting point 👍  Unfortunately CI logs are no longer available, but if it can be made to work, that sounds great.


Add support for Angular 9

Getting an error Cannot find module 'angular-responsive-carousel' when installed on a Angular 9 project.
I think it's caused because of the peer deps.
"peerDependencies": {     "@angular/common": "~7.2.0",     "@angular/core": "~7.2.0"   },

@IshanRatnapala Thank. Try to install the latest version.


feat: k8s local dev setup

Local k8s dev setup, including README documentation

Tabling this until we have a container to set up that's not the app. You can't really develop in Java without JDK installed locally so we don't get anything by containerizing it unless we have something else to run with it like mysql or redis or something.


Update twilio video-android binding to v3.x.x

First of all, thank you for your amazing contribution with this Xamarin binding of Twilio.
I'm trying to do the same thing with the version 3 of twilio video for android, knowing that this version is fixing latest compatibilitiy issues due to Firefox last breaking changes.
But I'm stuck in a situation where, trying to build the binding project with the latest twilio-video.aar (build action set to LibraryProjectZip), I get error messages about the WebRTC dependency.

Hi @ksavidetove , thank you, glad to help!
Native library binding is a kind of advanced topic, so I recommend you to read https://developer.xamarin.com/guides/android/advanced_topics/binding-a-java-library/customizing-bindings/java-bindings-metadata/ first.
In your case it looks like binding tool is generating a method proxy with the wrong name (which is already taken by another generated method). I had this problem before when I was working on 2.* binding, take a look at https://github.com/dkornev/TwilioXamarinBindings/blob/master/source/Twilio.Video.Android/v2/Transforms/Metadata.xml where I set specific managedName for the same reason.
BTW, I'm planning to update Twilio Video Android SDK to 3.* the next one. But no dates yet.

@ksavidetove Thank you so much for your contributing. I'll try to instal VS 2019 and review the PV on the weekend

@dkornev Hi! Could you review and merge the PR please?

Thanks for the PR.
Unfortunatly I can't find the last version of this binding using Nuget, the latest available version is 2.6.
And I think you should warn the users that VS 2019 is mandatory to make it work, it could spare some debug time.

I could build the project on VS 2019 (Preview) for Mac without adding d8. I'll add using VS 2019 as a recommendation, but I'm not sure that it's necessary.


Design System > Elements > Enhancements in Text Editor

Design System > Elements > Enhancements in Text Editor
https://www.figma.com/design/vziFLZAgMFi8wA5SlikLh5/Raaghu-Design-System?node-id=382-4674&node-type=canvas&t=ncBHc3XyPjDjE2We-0
Docs	1]  Spinner development different than design
Default	1]  change the sequence of controls
2] Background color of controls

Implement control remove the stories.

Verified on staging that stories are removed and added points in control


ORC-1698: Upgrade commons-cli to 1.7.0

What changes were proposed in this pull request?
This PR aims to upgrade commons-cli to 1.7.0.
Why are the changes needed?
Release 1.7.0 – 2024-04-13
https://commons.apache.org/proper/commons-cli/changes-report.html#a1.7.0
The default value of getOptionValue is null. After upgrading to 1.7.0, a new method was introduced, causing compilation to fail.
this.bloomFilterColumns = opts.getOptionValue('b', null);

Error:  Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.10.1:compile (default-compile) on project orc-tools: Compilation failure
Error:  /root/orc/java/tools/src/java/org/apache/orc/tools/convert/ConvertTool.java:[202,35] reference to getOptionValue is ambiguous
Error:    both method getOptionValue(char,java.lang.String) in org.apache.commons.cli.CommandLine and method getOptionValue(char,java.util.function.Supplier<java.lang.String>) in org.apache.commons.cli.CommandLine match

How was this patch tested?
GA
Was this patch authored or co-authored using generative AI tooling?
No
Closes #1906

To @cxzl25 , you can add the following at the end of your PR description. Then, GitHub will automatically close #1906 when this (#1907) is merged.
Closes #1906

you can add the following at the end of your PR description

Thank you for your reminder. I have updated it.

Thank you. Merged to main/2.0.


fixed crash when using git time-machine in emacs

When using git time machine in Spacemacs (https://github.com/emacsmirror/git-timemachine), the elixir ls crashes on changing to a previous version of a file.
15:50:18.705 [error] GenServer ElixirLS.LanguageServer.Server terminating
** (MatchError) no match of right hand side value: ["", ""]
    (language_server 0.4.0) lib/language_server/source_file.ex:128: ElixirLS.LanguageServer.SourceFile.apply_edit/3
    (elixir 1.10.2) lib/map.ex:798: Map.update!/3
    (language_server 0.4.0) lib/language_server/source_file.ex:22: ElixirLS.LanguageServer.SourceFile.apply_content_changes/2
    (language_server 0.4.0) lib/language_server/server.ex:284: anonymous fn/4 in ElixirLS.LanguageServer.Server.handle_notification/2
    (elixir 1.10.2) lib/map.ex:837: Map.get_and_update/3
    (elixir 1.10.2) lib/map.ex:878: Map.get_and_update!/3
    (language_server 0.4.0) lib/language_server/server.ex:284: ElixirLS.LanguageServer.Server.handle_notification/2
    (language_server 0.4.0) lib/language_server/server.ex:134: ElixirLS.LanguageServer.Server.handle_cast/2
Last message: {:"$gen_cast", {:receive_packet, %{"jsonrpc" => "2.0", "method" => "textDocument/didChange", "params" => %{"contentChanges" => [%{"range" => %{"end" => %{"character" => 0, "line" => 3}, "start" => %{"character" => 0, "line" => 0}}, "rangeLength" => 50, "text" => ""}], "textDocument" => %{"uri" => "file:///Users/gseddon/git/elixir-ls/apps/language_server/lib/test_module.ex", "version" => 1}}}}}

This PR handles this edge case better. Honestly I haven't dug through the code enough to find out what assumption is being invalidated by this time-machine behaviour, so am open to feedback on how better to implement this fix.

It looks like apply_edit is called with end_line over the number of lines in the file, most likely due to race conditions. I'm not sure if this is the right pace to handle it. Should the edits from textDocument/didChange be applied at all if the file changed in between? Anyway, we'd need a unit test.

Closing in favour of https://github.com/elixir-lsp/elixir-ls/pull/411


add Dockerfile

Signed-off-by: duyanghao<EMAIL_ADDRESS>

@joyield what do you think of this pr?

@duyanghao Think you, But I don't think the dockerfile can be used by everyone


Update CI

Modernizing CI actions similar to xgcm and xmovie

Ill take care of the cloud test failure in #127 and merge this as is.


Subclass AutofireCoils to create more common names

"autofire_coils:" is confusing for users. Suggest creating what's essentially empty subclasses that have new config strings for:

pop_bumpers
slingshots

we got kickbacks and this is documented fairly well. close for now


CB-10610: Remove CDP_SDX_HBASE_CLOUD_STORAGE entitlement at GA for AWS

-From 7.2.7 Store HBase data on cloud storage for AWS at data lake even when the entitlement CDP_SDX_HBASE_CLOUD_STORAGE is disabled
-Add more test cases for this change
See detailed description in the commit message.

Ben will find out the actual cost for using S3 in CDPSDX-2808 before we decide to GA this. So hold on committing this.

@LinaAtAustin We shouldn't merge this until we have answered the cost questions in this comment and this task CDPSDX-2808 has been completed

@kmanamcheri can you reopen this pull request? Ben has tested CDPSDX-2899 and it past. The only thing left to GA the feature that stores HBase data on S3 is to remove this entitlement. Thanks!


Generate a dynamic Django SECRET_KEY

The Django SECRET_KEY setting is currently hardcoded in the template. It should be generated dynamically.

Even better: create a .envfile using django-environ.


Using accessCredentials.userPassword secret resulted in the loss of vmi IP status

What happened:
When using macvtap cni to provide network for the vm, the ip state of the vmi is synchronously changed through qemu agent，If the VM also uses accessCredentials.userPassword at this time, it will cause the IP status of the VMI to be lost when synchronizing the userpassword
What you expected to happen:
vm configured userpassword secret and macvtap network
…………
    spec:
      accessCredentials:
      - userPassword:
          propagationMethod:
            qemuGuestAgent: {}
          source:
            secret:
              secretName: 83da2c03-d274-4104-988d-c1f230a0e36f-userpassword
      domain:
        devices:
          interfaces:
          - macAddress: 52:54:00:cc:ab:52
            macvtap: {}
            model: virtio
            name: eth0
…………

Listening for the vmi status, it was found that the vmi ip disappeared after being synchronized and updated through qemu agent
[root@srv11-kvm ~]# kubectl get vmi 83da2c03-d274-4104-988d-c1f230a0e36f -w
NAME                                   AGE     PHASE     IP             NODENAME   READY
83da2c03-d274-4104-988d-c1f230a0e36f   9m53s   Running   <IP_ADDRESS>   k8s-04     True
83da2c03-d274-4104-988d-c1f230a0e36f   10m     Running                  k8s-04     True

Normal after deleting the userpassword secret and restarting the VM. I suspect that there was a conflict between the two running at the same time, as the latter's update overwrites the former's update
How to reproduce it (as minimally and precisely as possible):
Steps to reproduce the behavior.
1、vm use macvtap cni
2、vm config userPassword secret
3、create vm
4、watch vmi ip status
Environment:

KubeVirt version (use virtctl version): v1.0.0
Kubernetes version (use kubectl version): v1.26.3

FWIW macvtap does not support any sort of IPAM.
You need to use it to connect to the node's networks, by creating the macvtap interface on top of the node's interfaces. macvtap could get an IP via dhcp - if that's configured on that particular interface).
And yes, when using macvtap your only way for IP reporting is via guest agent.

@EdDev
Thank you for your reply.
Yes, I confirm that Cloudinit has correctly configured the guest IP and the IP reported on the VMI is correct.
If you try to watch VMI, you will find that the IP will cycle back and forth between its existence and disappearance.
[root@node1 ~]# kubectl get vmi d740a12f-156e-4628-aca4-a4d7d6d5e552 -w
NAME                                   AGE   PHASE     IP    NODENAME   READY
d740a12f-156e-4628-aca4-a4d7d6d5e552   18h   Running         k8s-11     True
d740a12f-156e-4628-aca4-a4d7d6d5e552   18h   Running   <IP_ADDRESS>   k8s-11     True
d740a12f-156e-4628-aca4-a4d7d6d5e552   18h   Running                    k8s-11     True
d740a12f-156e-4628-aca4-a4d7d6d5e552   18h   Running   <IP_ADDRESS>   k8s-11     True
d740a12f-156e-4628-aca4-a4d7d6d5e552   18h   Running                    k8s-11     True
d740a12f-156e-4628-aca4-a4d7d6d5e552   18h   Running   <IP_ADDRESS>   k8s-11     True
d740a12f-156e-4628-aca4-a4d7d6d5e552   18h   Running                    k8s-11     True
d740a12f-156e-4628-aca4-a4d7d6d5e552   18h   Running   <IP_ADDRESS>   k8s-11     True
d740a12f-156e-4628-aca4-a4d7d6d5e552   18h   Running                    k8s-11     True
d740a12f-156e-4628-aca4-a4d7d6d5e552   18h   Running   <IP_ADDRESS>   k8s-11     True
d740a12f-156e-4628-aca4-a4d7d6d5e552   18h   Running                    k8s-11     True
d740a12f-156e-4628-aca4-a4d7d6d5e552   18h   Running   <IP_ADDRESS>   k8s-11     True
d740a12f-156e-4628-aca4-a4d7d6d5e552   18h   Running                    k8s-11     True

This only occurs when both macvtap and userpassword secret are configured simultaneously.
If you delete the userpassword section of the VM, restart the VM, and observe again, it will be normal.
No useful entries can be seen in the virt-launcher log. I think there is a conflict between the two.

@maiqueb
Thank you for your reply.
I know that macvtap cni does not support ipam. I used cloudinit to specify a static IP address for the guest.
As you can see, the guest agent successfully reported the IP, but the IP status of the vmi has been fluctuating between existence and disappearance. This issue only exists when both macvtap and userpassword secret are configured simultaneously.
I think this is a bug
[root@node1 ~]# kubectl get vmi d740a12f-156e-4628-aca4-a4d7d6d5e552 -w
NAME                                   AGE   PHASE     IP    NODENAME   READY
d740a12f-156e-4628-aca4-a4d7d6d5e552   18h   Running         k8s-11     True
d740a12f-156e-4628-aca4-a4d7d6d5e552   18h   Running   <IP_ADDRESS>   k8s-11     True
d740a12f-156e-4628-aca4-a4d7d6d5e552   18h   Running                    k8s-11     True
d740a12f-156e-4628-aca4-a4d7d6d5e552   18h   Running   <IP_ADDRESS>   k8s-11     True
d740a12f-156e-4628-aca4-a4d7d6d5e552   18h   Running                    k8s-11     True
d740a12f-156e-4628-aca4-a4d7d6d5e552   18h   Running   <IP_ADDRESS>   k8s-11     True
d740a12f-156e-4628-aca4-a4d7d6d5e552   18h   Running                    k8s-11     True


Purge CDN last

This commit changes CDN purging of badge data to be last.
We've received two reports where an edit of project data worked,
but the badge image display has the old value.
Jason Dossett & I believe the problem is some sort of race condition
with Fastly, but we've not been able to reproduce it.
This moves CDN purges to be the last task done, which should
reduce the likelihood of triggering the race condition.
Before, in the save-after-edit case, there was nontrivial functionality
after a badge purge, making race conditions more likley.
This commit doesn't actually fix the race condition, but it should
significantly reduce the likelihood of it occurring, and that's a good thing.
Signed-off-by: David A. Wheeler<EMAIL_ADDRESS>

@jdossett - I think this reduces the likelihood of triggering the race condition, and thus is worthwhile.


Need an npm package to get around npm / graphql-js issues

Using different versions of graphql-js for generating the schema and executing queries doesn't work, because graphql-js uses instanceof to check if a schema is a graphql schema.
When installing from github, it seems that npm installs multiple copies of the apollo-server package, even when they are the same version. This leads to the issue above showing up all the time. Creating an experimental npm package should do the trick, but we'd have to do it for apollo-server and our fork of graphql-js (and express-graphql, if we want to use that as well). Hopefully a new npm version of graphql-js will solve this problem soon, because it's quite a hassle.

new version of graphql-js is out, so this isn't as much of an issue any more.


WIP Newsflash location qualification. #2218

there is a need for the Enum value of None? or it okey to use the null value.
it can lead to a situation where there is a None(the enum value) and null in the database.

there is a need for the None value in the NewsflashLocationQualification? why not using null?


Hallowed items/armor

Hallowed items/armor have worse stats than Orichalcum.

If you prioritize durability it is easy to see why you would think this.
Durability-wise Orichalcum tools are way better, and the armor has slightly more. Orichalcum also has KB resistance which Hallowed does not, because Orichalcum is designed to make for better armor.
Hallowed gear is balanced around the hallowed materials. It is not supposed to be a direct upgrade from all of them. In comparison to an Orichalcum Pickaxe, a Hallowed Pickaxe has less durability, but it is much faster and has better DPS. This goes for all Hallowed tools if you compare against Orichalcum.
If you enjoy min-maxing then keep making Orichalcum armor. Some players might not like KB resistance and can opt out by making Hallowed Armor.
If you believe that Orichalcum tools are better, then that is simply your preference.


fix: fix mock get_entry signature

Fixes failing unit test test_transfer_usd_near.
The root cause was get_entry method signature out of sync in conversion
proxy contract and in mock fpo contract.
It can be easily detected by and reported by removing the .then part
in the conversion_proxy/src/lib.rs#L107-116 . After doing that you can
use println in the unit test to print the outcome and see the reason of
the fail for the original root cause.

Another generic advice:
In your conversion proxy contract you use
        fpo_contract::get_entry(
...
        )
        .then(ext_self::rate_callback(

Which is a cross contract call, which may be relatively expensive in gas consumption.
Consider switching from pull model to push model: oracle can call method on all consumers in a cron-like manner.
fpo_contract will call all the conversion proxies every X seconds and update the near price.
and then the conversion proxy will use cached value without spending time on a cross contract call.
If you have high amount of transfer_with_reference calls and low number of oracle price updates, this might be justified.

@alexaunter  thank you so much for you help and precious advice, I could not guess that ! The rust hint to prefix provider with underscore was a big trap. I'll use #[allow(unused_variables)] instead.


document web form and command-line options

Some of the options on the web form are non-obvious, and I can't find them in the documentation http://rdf.greggkellogg.net/yard/RDF/Distiller/Application.html (probably because the search is not full-text search but only class/method names). Some of the options have an informative tooltip, but it's not always enough, and others don't have it.

expandContext: I guess this is context used for expanding? Why do we need a second context for that? ("flatten" and "compact" don't get such option, "expand" and "frame" do). Cant's see it in http://rdf.greggkellogg.net/yard/JSON/LD/Expand.html#expand-instance_method
rdfDirection: please explain this
Where are command-line options documented?

The following come from https://www.w3.org/TR/json-ld11-framing/#framing-keywords. But it'd be very useful if you describe them and link to motivating examples in the spec

explicit "Only include explicitly declared properties in output"
omitDefault "Omit missing properties from output": what else could you do with missing properties, output them as null?
requireAll "Require all properties to match": (the further mention of --no-compact-to-relative is wrong)
embed: please explain this

If this is already documented, please excuse me and point me to the doc. Cheers!

These options refer to JSON-LD specific options, defined in the JSON-LD API and JSON-LD Framing and are only available when the input format is "jsonld".
For more information see those documents and the JSON-LD Syntax.
The Distiller form elements are derived from the command line options of the rdf CLI, which can be seen with --help (after selecting an appropriate input- or output- format). In turn, these are defined in either the Format, Reader or Writer classes. (E.g., JSON::LD::Format, JSON::LD::Reader, or JSON::LD::Writer.
Examples in those files could certainly be improved, with links into examples in the specs.

expandContext is there to provide an external link to a context to use when expanding a JSON file that should be considered as JSON-LD. There is also a way to do this with a Link header.
rdfDirection is a new feature added to handle the @direction keyword added to JSON-LD. Eventually, we expect some text direction mechanism to be added to RDF. For now, this provides for two non-normative mechanisms using a datatype or a compound literal. You can read up on this in the syntax document.
explicit eliminates from the output any properties not described explicitly in the framing document.
omitDefault overrides any @default included in the frame to not include that in the output.
requireAll is used for frame matching to require that all properties described in a sub-frame be present to match (or not). (I'll fix the documentation to use --no-requireAll instead).
embed controls if referenced objects are embedded, or described elsewhere. It can be useful to have objects always be present, even if it creates duplicates, for consistency, or to force them out to the outer-most level.

Please consider helping to improve documentation in the various gems with PRs.

Immense thanks!
In particular, expandContext is very useful and I think is missing at the jsonld playground.
But I still don't understand what's the difference from context and why two of the commands need 2 contexts.
To add PRs, I think 2 UI  improvements are needed:

ability to use hyperlinks in the explanations
hover displays a transient tooltip, but the user can't click inside it.

add an (i) icon: when clicked, display a popup (so then the user can click a link inside it). Dismiss when clicked a second time

Yes, the json-ld playground doesn't expose all options, but it's lacking adequate developer support. My UI is built automatically based on options specifically defined for readers and writers.
The "expandContext" is used for expanding a document, while the "context" argument is used when compacting. It's conceivable that you need to apply a context when expanding a native JSON document, and then want to compact (or frame) it with a different context.
I don't have much time to work on it now, but again, feel free to clone and submit PRs. I think having an (i) icon to bring up more documentation would be great. Ideally, it would somehow take you to the appropriate Yard page for a given option, but more likely would need to repeat the help content in the option description with Markdown. The Markdown could be processed on the server side and sent across in the JSON that drives the UI.
For example, the Yard documentation for the "expandContext" option is buried in http://rdf.greggkellogg.net/yard/JSON/LD/API.html#class_method_details (it should be in http://rdf.greggkellogg.net/yard/JSON/LD/Reader.html#initialize-instance_method, but I need to look at the tags. Unfortunately, there's no good way to introduce fragment identifiers in the generated HTML.
This documentation could be improved to reference the definition on JSON-LD API, although that doesn't bring you to an adequate example, either. For that you'd need to see example 142 in JSON-LD, and even then this uses the equivalent Link header.


Justify why Managed keys shouldn't be manually rotated and give it more visibility

There's already a line in this doc that says to not regenerate the managed keys manually:
"Regenerate keys by using Key Vault only. Don't manually regenerate your storage account keys."
However, we're missing to specify why it's so important that the keys don't get manually rotated, doing that is a feature-breaking process and that should be stated and highlighted as a Note so the customers don't break the automation without knowing that's a breaking change.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8b568009-2a35-c627-cfa0-ff5cbfde027e
Version Independent ID: 07fc2d02-be89-1170-3b37-9e65808d5d89
Content: Manage storage account keys with Azure Key Vault and the Azure CLI
Content Source: articles/key-vault/secrets/overview-storage-keys.md
Service: key-vault
Sub-service: secrets
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

#sign-off

@shashishailaj May I have an update on this, please?

Updated #please-close


Shuffle mode

Hi
I think that having a shuffle/random photo feature is also important for a gallery app.

Can you explain this a little bit more? Is this something like a button that randomly displays a photo?

Yes, some kind of random slidesho to improve discoverability.

This has been added 👍
https://github.com/meichthys/foss_photo_libraries/commit/3dd3852ffb4ce580be1aad838ba4e9e6eeebfe54


Update s3, sns, sqs, sts to 2.15.78

Updates

software.amazon.awssdk:s3
software.amazon.awssdk:sns
software.amazon.awssdk:sqs
software.amazon.awssdk:sts

from 2.15.69 to 2.15.78.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

labels: library-update, semver-patch

Superseded by #589.


reflect: reflect.StructOf makes incorrect type with embedded interface

What version of Go are you using (go version)?

$ go version
go version go1.20.6 linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/sz/.cache/go-build"
GOENV="/home/sz/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/sz/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/sz/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/lib/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.20.6"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
GOWORK=""
CGO_CFLAGS="-O2 -g"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-O2 -g"
CGO_FFLAGS="-O2 -g"
CGO_LDFLAGS="-O2 -g"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -Wl,--no-gc-sections -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build2006396805=/tmp/go-build -gno-record-gcc-switches"

What did you do?

Consider the following code: https://go.dev/play/p/WuzpUlzKido and https://go.dev/play/p/zgAX3grM8rT
It makes a new struct type using reflect.TypeOf with a single embedded interface field. Then it makes an instance of the type and tries to invoke the method of the interface on that type.
What did you expect to see?
Output:
A.M1 invoked

What did you see instead?
It panicked.

Duplicate of #15924

Duplicate of #38783


Microsoft Rate limiting error for over 4 days now

I keep seeing the same try again in 30 minutes error on the importer page for 4 days now. The import worked well for the first few notebooks.

Hello,
Could you please install the importer file I've linked below?

Download this file: importer.zip
Unzip it
Place the main.js in YourVault/.obsidian/plugins/obsidian-importer and replace the main.js that's in already there.
Close and reopen Obsidian.

After following the steps above, please run the Importer again. After seeing the error message, could you please open the developer console and look for red error messages and send us screenshots?
Here are instructions on how to open the console: https://help.obsidian.md/Help+and+support#Capture+console+logs
Thank you!

I was getting this error before

After I swapped main.js I no longer get the rate limiting message.
What was the issue?


remap keys to move up/down a channel

I prefer to have different shortcuts for moving up/down channels.

At the moment, shortcuts are not customizable (and to be fair, they are not on most "modern" chat apps like Telegram or Slack) so this is not likely to be changed anytime soon.
Cmd + ↑ / ↓ and Ctrl + ↑ / ↓ are actually pretty standard, so there is no way we can please everyone without a full shortcut refactor, or better an API that let people write plugin for this.
Closing this for reasons explained above.


Fixes v2

-v2

removed wrong commit
set the constants as float instead of the cast

integrated, thanks


[bn] Updated content/bn/style-guide/_index.md

Signed-off-by: Shadman Rafy<EMAIL_ADDRESS>Updated content/bn/style-guide/_index.md  to Bengali
This is an intial PR as may need some improvements!

Preview: https://deploy-preview-751--cncfglossary.netlify.app/bn/style-guide/
And PTAL at https://github.com/cncf/glossary/pull/750#pullrequestreview-943796044 !


Migrations - Capture and replay v0.1.0 - Extracted Comparator and Jupyter

Description
[Describe what this change achieves]

Category (Enhancement, New feature, Bug fix, Test fix, Refactoring, Maintenance, Documentation)
Why these changes are required?
What is the old behavior before changes and new behavior after changes?

Issues Resolved
[List any issues this PR will resolve]
Is this a backport? If so, please add backport PR # and/or commits #
Testing
[Please provide details of testing done: unit testing, integration testing and manual testing]
Check List

[X] New functionality includes testing

[X] All tests pass, including unit test, integration test and doctest (Disabled unstable test)

[ ] New functionality has been documented
[ ] Commits are signed per the DCO using --signoff

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

You can also remove the kafka-broker and kafka-zookeeper copilot directories as those aren't in use since we are using MSK and didn't integrate yet

We also need to adjust the Traffic Replayer copilot manifest.yml as well to remove the netcat link


[Assistant Browser] Adapt tab bar when searching

Description
Following thread: unclickable tabs were confusing when search was on.
Initial solution was to hide tab bar. The PR goes for a similar solution with a slight difference, see gif below:
Reasons were that hiding completely tab bar had been tested (see e.g. here ) and had following issues:

layout "jump" when typing in the search bar;
people not paying attention that something was typed in the search bar, and wondering where the tab bar was / why there were agents missing

Technical: to ensure styles remain consistent, "search" display is defined as another tab
Risks
na
Deploy
front

@Duncid PMRR , at your disposal to update if not satisfactory


Configure... Doesn't properly setup the camera rig

XRTK - Mixed Reality Toolkit Bug Report
Describe the bug

using any of the preview packages, when configuring the xrtk the camera rig is not setup correctly.
To Reproduce

Install the core, sdk, and choice of platform package (I used wmr and oculus)
Click on Configure...
View Scene hierarchy
See that the Main Camera hasn't been properly moved underneath a Playspace

Expected behavior

Main Camera should be moved to a new parent object named Playspace
Actual behavior

Main Camera isn't moved, but it does have the rest of the correct components so that's a plus
Screenshots

Your Setup

Unity Version 2019.3.13f0
XRTK Version 0.2.0-preview.45
XRTK Commit SHA 533ff2e7409416982e0bd3dd43119b68eaf8c176

Seems to be fixed with the latest preview releases


⚠ Fortnite has degraded performance

In 5163229, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1685 ms

Resolved: Fortnite performance has improved in 20a5251.


Develop

Codecov Report

Merging #96 (d280a6c) into master (3e41c7a) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##           master     #96      +/-   ##
=========================================
- Coverage    0.81%   0.79%   -0.03%
=========================================
  Files          17      17
  Lines        1840    1892      +52
=========================================
  Hits           15      15
- Misses       1825    1877      +52

Impacted Files
Coverage Δ

core/analysis.go
0.00% <0.00%> (ø)

core/bindata.go
0.00% <0.00%> (ø)

core/findings.go
0.00% <ø> (ø)

core/git.go
0.00% <0.00%> (ø)

core/io.go
4.10% <0.00%> (ø)

core/localPath.go
0.00% <0.00%> (ø)

core/log.go
0.00% <0.00%> (ø)

core/session.go
0.00% <0.00%> (ø)

core/stats.go
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


chore: simplify KotlinCompile configuration

Reason: the kotlin-gradle-plugin respects JvmToolchains, see here.
In case you wanted to be explicit about it, feel free to just close this PR :)

So merge or close?

Do you an automated test?
Or once with javap -verbose -classpath build-logic/convention/build/libs/convention.jar com.autonomousapps.convention.ConventionPlugin | grep -b20 -a20 "major version"?
A thought that came to me a few days ago:
This configuration configures the bytecode version for the plugins.
As far as I understand it the convention plugins runs in the gradle jvm.
I think using toolchains it is possible to compile them to a different version than gradle is run with.
So is it even wise to use toolchains at all in this case?


Why overridding all matplotlib defaults?

https://github.com/lukelbd/proplot/blob/3f065778edce6e07b109e436b504238c7194f04f/proplot/rctools.py#L48
How about setting default configuration values to their matplotib value if it exists, at least for some of the settings?
In the example above, the default colormap is set to "fire" whereas I would like the default "viridis" from matplotlib instead.
This may apply to several other conflicting settings.
I agree that you want to make all proplot plots beautiful by default, but I'm not sure all default matplotlib settings must be changed.
A solution would be to check if a config value is set to "None" by the user, and in this case to use the matplotlib value if it exists.

Yeah this is definitely one of ProPlot's pain points. As you have guessed, I was never sure how to address this because I did want proplot to override matplotlib's settings in order to make things "nice"... but checking for user overrides and user-enabled matplotlib stylesheets is a great idea. Just have to figure out how to inject proplot's default settings in-between the steps where (1) matplotlib loads its default settings and (2) matplotlib loads the user-specific matplotlib settings.
I'll add this in a future version.

@stefraynaud I finally added stylesheet control via c8319104 and edc6f3c9. New version will come out soon.
I can't really check for None for every setting in .proplotrc because None might be valid values for some rc settings. And the way proplot's rc_configurator is currently designed, this would be really difficult anyway, because even if you try to tell proplot not to overwrite an individual setting (e.g. xtick.label.size), proplot's defaults for the "meta" settings (e.g. small, the font size for "small" text labels) could end up overwriting it anyway. When #109 is merged there will be no "default" meta settings so I could theoretically implement this.
However if you want to use all of matplotlib's default settings instead of proplot's settings, you can add the following line to your proplotrc file.
style: default

Or use any other stylesheet, like
style: fivethirtyeight

I made it so that matplotlib stylesheets are always applied on top of matplotlib's default settings, rather than proplot's default settings. Also, proplot does not support changing proplot-specific settings like abc.loc in matplotlib stylesheets, but I'll be able to do so after I start subclassing RcParams directly when #109 is merged.
Note that in your case, if you just want to change the default colormap but keep other settings, you can do so manually with:
cmap: viridis


Introduce transport and deprecate nirum.rpc.*

This patch implements #79.  Detail explanations are written up to CHANGES.rst file.
Note that this requires the compiler changes made by spoqa/nirum#149 to be merged first in order to pass the build and tests.

:100:


How do we set the HttpClient base URL if that URL is in appsettings.json file?

How do we set the HttpClient base URL if that URL is in appsettings.json file?
Tried this which works locally but fails on Azure web app so maybe it is not the correct approch.
builder.Services.AddHttpClient<SomeClass>()
.ConfigureHttpClient((sp, httpClient) =>
{
var genevaClientSettings = sp.GetRequiredService<IOptions<GenevaClientSettings>>().Value;
httpClient.BaseAddress = genevaClientSettings.GenevaBaseUrl;
})

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

ID: 881cef77-11ef-ac11-06d0-0996a1ccfa1b
Version Independent ID: 3d356334-e672-7e8b-1fef-7944ebee99eb
Content: Make HTTP requests using IHttpClientFactory in ASP.NET Core
Content Source: aspnetcore/fundamentals/http-requests.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @stevejgordon
Microsoft Alias: scaddie

Hello @xavierjohn ... I think it's in the Configuration doc, but there's a nibblet in the Configuration section of the Minimal API doc that I can cross-link quickly ...
https://docs.microsoft.com/aspnet/core/fundamentals/minimal-apis#configuration

Thanks. I will try
builder.Configuration.GetValue("{THE KEY}");
I can't use option since the app is still being configured.


Menus

-Updated iOS version to support up to 16.2
-Added an example implementation of Menus (introduced in iOS 14)

Due to this issue, this PR has been will be reverted first.
https://github.com/Jinxiansen/SwiftUI/issues/42


Inferring node to binary mapping from compile_commands.json

The compile_commands.json contains information about binary names, directories and source files.
It can be used instead of manually specifying the mapping.
Everyone who uses neovim for C++ will have compile_commands.json generated anyway.

Hey, unfortunately I don't think this is as easy without the build tool support. Compile commands do not have link commands, and that's what you would need to map from a file to an executable.
There's a ticket for integrating with build tools (likely CMake only, bazel seems to be a lot less plugin-friendly), and I'll add a hook for customization. I'll close this issue as a duplicate.


Typo "Azure Blob storage service"→"Azure Blob Storage Service"

https://docs.microsoft.com/en-us/sql/relational-databases/tutorial-use-azure-blob-storage-service-with-sql-server-2016?view=sql-server-ver15

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

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

Hi @hyoshioka0128
The updated text refers to the Azure service Azure Blob Storage. The initial capitalization in the service name Azure Blob Storage looks good and aligns with internal style guidance; thanks. The word service isn't part of the service name, so it should be lowercase.
Could you make this change (lowercase "service" where you changed text to "Azure Blob Storage Service") in this pull request and in the other pull requests that you have open for "Azure Blob Storage service"?
Many thanks.
-PR review team

Hi @hyoshioka0128
The updated text refers to the Azure service Azure Blob Storage. The initial capitalization in the service name Azure Blob Storage looks good and aligns with internal style guidance; thanks. The word service isn't part of the service name, so it should be lowercase.
Could you make this change (lowercase "service" where you changed text to "Azure Blob Storage Service") in this pull request and in the other pull requests that you have open for "Azure Blob Storage service"?
Many thanks.
-PR review team


Add interpolation strategy controls

The docs seemed to indicate that mode 1 was the default.

Changed facevarying default to 3


HIVE-16684: Bootstrap REPL DUMP shouldn't fail when table is dropped after fetching the table names.

Already merged to master.


Welcome page update: Reordering of tabs

Recommendation from @geekygirldawn

How 8Knot Works - this provides important context about 8Knot and the relationship with Augur
Plotly Figure Tools (maybe rename to ‘Using 8Knot Visualizations’)
8Knot Pages - this is more obvious, so I’d put it below the visualization instructions
Your Augur Account (maybe rename to ‘Logging into Augur’)
Adding a User Group (maybe rename to ‘Creating Project Groups’)
Note: User group is super confusing because it sounds like you are grouping users, rather than creating a group of orgs / repos under your user account.

I also recommend not having these as tabs, but as one giant page. I know that the extra scrolling isn't ideal, but I think that this is super important information that a lot of users won't notice if it's behind tabs in the middle of the page.


Bootstrap Components using JS don't work well

Taking example of Navbar dropdown, if Purifycss is provided with JS of Bootstrap, it works well. But then it includes all of the classes used in BootstrapJS which should not be the case. Is there a workaround for this?

I'm having a similar problem. When using a Bootstrap dropdown menu with purifycss, the menu doesn't work.

+1

Same issue here

+1

You have to include your js file in content

@jeremylynch, the issue is exactly about that. If you include the bootstrap.js in content, all css classes would be included, which is not desired.

Then you should only include parts of the bootstrap js that you need, eg.
bootstrap/button
bootstrap/collapse
bootstrap/dropdown


[Seg. Pointops Compile] name "pointops_cuda" conflicts with Cls. Pointops

Dear authors,
Thank you so much for your awesome work.
When I follow your source code to compile Pointops in Classification and then compile the other Pointops in Segmentation, the Pointops imported will come from Cls.'s.
This problem was found by
File './RepSurf/segmentation/modules/pointops/functions/pointops.py', line 126, in forward
    pointops_cuda.knnquery_cuda(m, nsample, xyz, new_xyz, offset, new_offset, idx, dist2)
TypeError: knnquery_cuda(): incompatible function arguments. The following argument types are supported:
    1. (arg0: int, arg1: int, arg2: int, arg3: int, arg4: at::Tensor, arg5: at::Tensor, arg6: at::Tensor, arg7: at::Tensor) -> None

Since knnquery_cuda() in Seg.'s Pointops support  (arg0: int, arg1: int, arg2: at::Tensor, arg3: at::Tensor, arg4: at::Tensor, arg5: at::Tensor, arg6: at::Tensor, arg7: at::Tensor).
Then I found setup(     name='pointops',     ext_modules=[         CUDAExtension('pointops_cuda', in  Cls's Pointops and
setup(     name='pointops_cuda',     ext_modules=[         CUDAExtension('pointops_cuda', in  Seg's Pointops. They have different python site-packages names but the same CUDAExtension name. This may cause upon problems during import pointops_cuda.
Because I am not familiar with C++, would you help me solve this problem?
(The solution for now on is to delete the other site-packages while using one)
Thanks again for your great work.

Thank you for your interest in our work.
This is the conflict of cuda extensions. A simple way to solve it is to follow our init.sh scripts to create two different conda envs for the two tasks (for cls and seg).
There is another way but it might be a bit complex - you need to compile the cuda extensions locally. By default, we use the command  python3 setup.py install to install pointops globally. To install it locally, you need to use python setup.py build_ext --inplace instead. Accordingly, you also need to import pointops extension locally by replacing Line 7-28 in pointops.py with from .. import pointops_cuda.
Hope these could help.

Thank you, your suggestions are helpful. We found that changing one CUDAExtension name and recompiling, then replacing all pointops_cuda in pointops.py with the new name will also work.


change map and sidebar from float to flexbox with 80/20 width

issue was created under a different title


How to click on "Termine anzeigen"?

How to click on "Termine anzeigen" and get a notification if there is really an appointment available?

(without docker)
I had this issue when the window size was too small. The screen resolution changed every time I logged into the machine remotely and I wondered why I got timeouts.
The cookie dialogue overlapped and so the button was invisible for puppeteer.
I just ran the browser in headless mode which solved it for me.


fix(deps): Update stack definition member versions

Update stack definition member versions

/run pipeline

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

GitHub release
v1.0.3

Your semantic-release bot :package::rocket:


Implemented a test driver for opaque signatures without storage as defined in issue #3343.

Description
Implemented a test driver for opaque signatures without storage.
The test driver supports key import, key generation, public key export and sign/verify hash. A test suite validates the functionality.
Status
READY
Requires Backporting
NO
Migrations
NO
Todos

[x] Tests
[x] Documentation

Steps to test or reproduce
Added test suite test_suite_opaque_test_driver, all.sh updated with new component test_opaque_test_driver.

Requesting review from @gilles-peskine-arm @ronald-cron-arm

For reference, @kjeir is currently on vacation and moving to a different position, meaning I will take over the work. Thanks for the early feedback. Do you want to continue driving this PR to completion (meaning I need to wait for @kjeir to give me permission on his fork), or should this one be closed and followed up with a new PR from my fork?

@stevew817 It's usually more convenient to keep the same PR in order to keep track of what's been reviewed and what comments have been resolved, but that's not a blocker. And if you rebase the code, keeping the same PR loses most of its advantage. So feel free to close this PR and open a new one.

With #3501, #3526 and #3605 PRs we have started developing two transparent test drivers and two opaque test drivers in parallel. As we go along in adding support for other driver APIs in psa_crypto.c and psa_crypto_driver_wrappers.c, it doesn't seem possible giving the time constraints to keep developing both. I see momentum in #3501 way of doing things with #3644 adding some support for ciphers. Furthermore, the testing in #3501 is the testing we definitely we need right now (exercising the test drivers and their interactions with psa_crypto.c through calls to the PSA APIs) whereas the testing in #3526 and #3605 (direct calls to the test driver APIs) is not something we need right now. For all those reasons, it would make sense to me to move the opaque test driver entry points provided by this PR to the way things are done in #3501 regarding code organization and testing. @stevew817 and @kjeir what do you think about that?

@gilles-peskine-arm @ronald-cron-arm I updated this PR given your comments and now we see "Pre Test Checks failed" and "This commit cannot be built" but I am not able to see the Details on jenkins-internal.mbed.com. How should I go about to handle these issues?
NOTE: I have not rebased yet. I should probably do so

@carlaageamundsen The previous run of the pr-merge job failed due to the merge conflict. If “PR-NNNN-merge Pre Test Checks” fails, that's the most likely reason. Now that you've rebased, Jenkins is running. Note that Travis has reported a build failure, you need to tweak something because of the change of where mbedtls_to_psa_error is defined.

@gilles-peskine-arm I removed the local copy of  mbedtls_to_psa_error() and use the one in psa_crypto.c which is now global. However there is still some error (see below) indicating issue in all.sh which I have not touched for a while. Do you have any ideas?
https://travis-ci.org/github/ARMmbed/mbedtls/jobs/725825258
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
FAILED: 1
check_generated_files: Check: freshness of generated source files: tests/scripts/check-generated-files.sh -> 1
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
The command "tests/scripts/all.sh -k 'check_*'" exited with 1.

Look through the logs for where that component failed:
^^^^check_generated_files: Check: freshness of generated source files: tests/scripts/check-generated-files.sh -> 1^^^^

And look backwards to see why it failed. Here it tells you:
'visualc/VS2010/mbedTLS.vcxproj' was either modified or deleted by 'scripts/generate_visualc_files.pl'

That's because you added a file in tests/src. But actually opaque_test_driver.c should go in tests/src/drivers and that doesn't require updating the VS project.


Set bundler environmental variables when they are provided

Hello, I'm maintainer of RubyGems and Bundler.
Bundler repo vendored turbo_tests for our test suites like https://github.com/rubygems/rubygems/tree/master/bundler/tool/turbo_tests. I hope to use release version of turbo_tests, not vendoring.
I removed bundle exec from commands builder of turbo_tests if they didn't use turbo_tests under the bundler because Bundler need to use turbo_tests without bundler environment.
How about this?
Thank you for develop this ❤

Hello, @hsbt! It's a great honor for me!
I'll test your PR manually and share my feedback.
Thanks to Discourse and RubyGems teams for the initial development of turbo_tests :hearts:


[POC] Plugins Extensions

General Information
We should investigate possibility to extend KOMM functionality via third-part libraries.

Found a way how to do it. Not the POC anymore!


helm ls shows forbidden

Rancher versions:
rancher/rancher: v2.0.8
rancher/agent or rancher/rancher-agent:
Infrastructure Stack versions:
healthcheck:
ipsec:
network-services:
scheduler:
kubernetes (if applicable):
Docker version: (docker version,docker info preferred)
17.03.2-ce
Operating system and kernel: (cat /etc/os-release, uname -r preferred)
rancherOS 1.4
Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
VirtualBox
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
singlenode rancher
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
Kubernetes 1.11.2
Steps to Reproduce:
(1) Install singlenode rancherserver v 2.0.8
(2) Create a RancherOS1.4 VM
(3) Switch RancherOS engine to docker-17.03.2-ce
(4) Add custom cluster in Rancher and copy/paste command for all roles to the RancherOS VM
(5) Wait for node to join and cluster to be active
(6) Download kubeconfig
(7) Configure kubectl context
(8) Download helm executable
(9) Check kubectl context is set to the new fresh untouched cluster
(10) Try the following commands

$ /home/me/bin/kubectl get all --all-namespaces
$ /home/me/bin/kubectl version
Client Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.2", GitCommit:"bb9ffb1654d4a729bb4cec18ff088eacc153c239", GitTreeState:"clean", BuildDate:"2018-08-07T23:17:28Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.2", GitCommit:"bb9ffb1654d4a729bb4cec18ff088eacc153c239", GitTreeState:"clean", BuildDate:"2018-08-07T23:08:19Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}
$ /home/me/bin/kubectl get all
NAME                 TYPE        CLUSTER-IP   EXTERNAL-IP   PORT(S)   AGE
service/kubernetes   ClusterIP   <IP_ADDRESS>    <none>        443/TCP   1h
$ helm ls
Error: could not find tiller
$ helm init
Creating /home/me/.helm/repository
Creating /home/me/.helm/repository/cache
Creating /home/me/.helm/repository/local
Creating /home/me/.helm/plugins
Creating /home/me/.helm/starters
Creating /home/me/.helm/cache/archive
Creating /home/me/.helm/repository/repositories.yaml
Adding stable repo with URL: https://kubernetes-charts.storage.googleapis.com
Adding local repo with URL: http://<IP_ADDRESS>:8879/charts
$HELM_HOME has been configured at /home/me/.helm.

Tiller (the Helm server-side component) has been installed into your Kubernetes Cluster.

Please note: by default, Tiller is deployed with an insecure 'allow unauthenticated users' policy.
For more information on securing your installation see: https://docs.helm.sh/using_helm/#securing-your-helm-installation
Happy Helming!
$ helm ls
Error: could not find a ready tiller pod
$ helm ls
Error: configmaps is forbidden: User "system:serviceaccount:kube-system:default" cannot list configmaps in the namespace "kube-system"
$ kubectl create clusterrolebinding fixRBAC --clusterrole=cluster-admin --serviceaccount=kube-system:default
clusterrolebinding.rbac.authorization.k8s.io/fixRBAC created
$ helm ls
$

Results:
helm ls or helm list will not work until  the command seen above "fixRBAC" is executed
** Expected **
Some documentation stating using helm outside requires a fix. Or any comment on why helm behaviour has changed in recent builds as "fixRBAC" was not required earlier (not sure exactly when it changed. There is a docs page that describes how to use helm. Not sure if https://rancher.com/docs/rancher/v2.x/en/installation/ha/helm-init/ is enough.
The matter was reported on slack so putting a placeholder here. Please close if this is not a problem.

The steps mentioned on https://rancher.com/docs/rancher/v2.x/en/installation/ha/helm-init/ are:
kubectl -n kube-system create serviceaccount tiller
kubectl create clusterrolebinding tiller \
  --clusterrole cluster-admin \
  --serviceaccount=kube-system:tiller
helm init --service-account tiller

I don't see those in the steps to reproduce, and it works on v2.0.8 with these commands.

will redo and update

$ date
Fri Sep 14 15:19:43 IST 2018
$ kubectl get all
NAME                 TYPE        CLUSTER-IP   EXTERNAL-IP   PORT(S)   AGE
service/kubernetes   ClusterIP   <IP_ADDRESS>    <none>        443/TCP   10m
$ helm ls
Error: could not find tiller
$ /home/me/bin/kubectl get all -n kube-system | grep tiller

$ helm version
Client: &version.Version{SemVer:"v2.9.1", GitCommit:"20adb27c7c5868466912eebdf6664e7390ebe710", GitTreeState:"clean"}
^C$ helm init
$HELM_HOME has been configured at /home/me/.helm.

Tiller (the Helm server-side component) has been installed into your Kubernetes Cluster.

Please note: by default, Tiller is deployed with an insecure 'allow unauthenticated users' policy.
For more information on securing your installation see: https://docs.helm.sh/using_helm/#securing-your-helm-installation
Happy Helming!
$ helm version
Client: &version.Version{SemVer:"v2.9.1", GitCommit:"20adb27c7c5868466912eebdf6664e7390ebe710", GitTreeState:"clean"}
^C$ helm ls
Error: configmaps is forbidden: User "system:serviceaccount:kube-system:default" cannot list configmaps in the namespace "kube-system"
$
serviceaccount/tiller created
$ kubectl create clusterrolebi
>
>   --serviceaccount=kube-system:ti
clusterrolebinding.rbac.authorization.k8s.io/tiller created
$ helm init --service-account tiller
$HELM_HOME has been configured at /home/me/.helm.
Warning: Tiller is already installed in the cluster.
(Use --client-only to suppress this message, or --upgrade to upgrade Tiller to the current version.)
Happy Helming!
$ helm version
Client: &version.Version{SemVer:"v2.9.1", GitCommit:"20adb27c7c5868466912eebdf6664e7390ebe710", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.9.1", GitCommit:"20adb27c7c5868466912eebdf6664e7390ebe710", GitTreeState:"clean"}
$ helm ls
Error: configmaps is forbidden: User "system:serviceaccount:kube-system:default" cannot list configmaps in the namespace "kube-system"
$ helm repo update
Hang tight while we grab the latest from your chart repositories...
...Skip local chart repository
...Successfully got an update from the "stable" chart repository
Update Complete. ⎈ Happy Helming!⎈

$ helm version
Client: &version.Version{SemVer:"v2.9.1", GitCommit:"20adb27c7c5868466912eebdf6664e7390ebe710", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.9.1", GitCommit:"20adb27c7c5868466912eebdf6664e7390ebe710", GitTreeState:"clean"}
$ helm ls
Error: configmaps is forbidden: User "system:serviceaccount:kube-system:default" cannot list configmaps in the namespace "kube-system"
$ helm repo update
Hang tight while we grab the latest from your chart repositories...
...Skip local chart repository
...Successfully got an update from the "stable" chart repository
Update Complete. ⎈ Happy Helming!⎈
$ helm ls
Error: configmaps is forbidden: User "system:serviceaccount:kube-system:default" cannot list configmaps in the namespace "kube-system"
$ helm ls
$ helm install --name mydb stable/mysql
Error: release mydb failed: namespaces "default" is forbidden: User "system:serviceaccount:kube-system:default" cannot get namespaces in the namespace "default"
$ helm ls
Error: configmaps is forbidden: User "system:serviceaccount:kube-system:default" cannot list configmaps in the namespace "kube-system"
$ helm list
Error: configmaps is forbidden: User "system:serviceaccount:kube-system:default" cannot list configmaps in the namespace "kube-system"
$

$ helm install --name mydb stable/mysql
Error: release mydb failed: namespaces "default" is forbidden: User "system:serviceaccount:kube-system:default" cannot get namespaces in the namespace "default"
$ kubectl create clusterrolebinding fixRBAC --clusterrole=cluster-admin --serviceaccount=kube-system:default
clusterrolebinding.rbac.authorization.k8s.io/fixRBAC created
$ helm install --name mydb stable/mysql
NAME:   mydb
LAST DEPLOYED: Fri Sep 14 15:28:37 2018
NAMESPACE: default
STATUS: DEPLOYED
RESOURCES:

Still not following the docs, if you init it wrong the first time (without proper RBAC), it won't work. Running helm init already shows the error: Warning: Tiller is already installed in the cluster.
Steps to fix this are to reset, and follow the docs again and make it work.
helm reset --force

Then follow the docs.
If you want to manually correct the service account, then do so by using a separate service account instead of granting cluster-admin to default.
kubectl create serviceaccount --namespace kube-system tiller
kubectl create clusterrolebinding tiller-cluster-rule --clusterrole=cluster-admin --serviceaccount=kube-system:tiller
kubectl patch deploy --namespace kube-system tiller-deploy -p '{"spec":{"template":{"spec":{"serviceAccount":"tiller"}}}}'

Thanks yes. Following the docs works, yet this morning a user tested and failed. On hind-sight the likely root cause was not force cleaning helm config.

@superseb jumping in late. The doc leaves some questions unanswered. 1) how to deal with a setup where the project user has no access to kube-system (--tiller namespace for helm init) and 2) how to grant the tiller service account in the project name space the RBAC roles it needs. Any pointers to those?


Add comments explaining why transmutes are safe

Add comments explaining why transmutes are safe so that people auditing unsafe code don't have to spend time figuring it out by themselves.

Thanks!
bors r+


Why should i need package.json, tsconfig.json and typings.json in my angularJS2 projects?

Well my question is, i can on some level understand the use of package.json file in my project. If i am right, it is used to download all the packages mentioned in the dependency property, but i am not able to understand what is the use of tsconfig.json and the typings.json file in my project? and, is all the three file depend on one another?

You should search existing issues or read existing documentation before opening support related questions, but to summarise:

package.json - specify NPM dependencies and NPM configuration
typings.json - specify Typings dependencies and Typings configuration
tsconfig.json - specify TypeScript compilation options


Image uploading fails after SUPEE-8788 patch

Uploading product images fails after the SUPEE-8788 patch
2016-10-14T11:21:13+00:00 ERR (3): Warning: pack(): Type H: illegal hex digit o  in public/app/code/community/Adyen/Payment/Model/Observer.php on line 399
2016-10-14T11:21:13+00:00 ERR (3): Warning: pack(): Type H: illegal hex digit h  in public/app/code/community/Adyen/Payment/Model/Observer.php on line 399
2016-10-14T11:21:13+00:00 ERR (3): Warning: pack(): Type H: illegal hex digit m  in public/app/code/community/Adyen/Payment/Model/Observer.php on line 399

Hi @jimmydg,
Strange, is this error occurring on multiple pages ?
Regards,
Rik

Hi @jimmydg,
Could not reproduce it myself I know the patch has updated the whole upload mechanisme but that is something outside our module.
Regards,
Rik
Adyen

IMHO that is related to a HPP HMAC Key not in Hex format.
I presume that you are working locally on an old DB, where the Adyen module is configured to work with a HMAC-SHA1 HPP key, but the installed module requires a HMAC-SHA256 key.
So you just have to edit the system configuration, in the HPP section delete (or update) the "HMAC Key for Test" and "HMAC Key for Live" fields.

Hi @jimmydg,
I close the issue for now let me know if you still have problems.
Regards,
Rik
Adyen


styled.div returns type any and throws error

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Link to live example: https://codesandbox.io/p/devbox/mui-styled-div-tz7n45?file=%2Fsrc%2FApp.tsx
Steps:

Open the codesandbox link
You'll see that the app does not work, an error is thrown

Current behavior 😯
Throws error: Uncaught TypeError: _mui_material__WEBPACK_IMPORTED_MODULE_2__.default.div is not a function
Also the type any is returned.
When using styled('div') instead of styled.div, there is no problem.
Expected behavior 🤔
No error should be thrown. Both styled('div') and styled.div must work and a reasonable type must be returned.
Context 🔦
We try to use MUI with theme scoping with styled-components v6.
Your environment 🌎

npx @mui/envinfo
  Used Browser: Chrome

  System:
    OS: macOS 12.6.1
  Binaries:
    Node: 16.20.1 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 8.19.4 - /usr/local/bin/npm
  Browsers:
    Chrome: 119.0.6045.199
    Edge: Not Found
    Safari: 15.6.1
  npmPackages:
    @mui/base:  5.0.0-beta.25
    @mui/core-downloads-tracker:  5.14.19
    @mui/material: 5.14.19 => 5.14.19
    @mui/private-theming:  5.14.19
    @mui/styled-engine:  5.14.19
    @mui/styled-engine-sc: 6.0.0-alpha.7 => 6.0.0-alpha.7
    @mui/system:  5.14.19
    @mui/types:  7.2.10
    @mui/utils:  5.14.19
    @types/react: 17.0.70 => 17.0.70
    react: 17.0.2 => 17.0.2
    react-dom: 17.0.2 => 17.0.2
    styled-components: 6.1.0 => 6.1.0
    typescript: 4.8.4 => 4.8.4

Support Key
#80425

styled.div isn't supported in @mui/material.
Duplicate of #29723, #28574.


Raw pointer map values, keySet and entries with predicate tests are missing

Added raw pointer values with predicate tests

Test FAILed.

Test FAILed.

Test FAILed.

verify

Test FAILed.

Test FAILed.

Test FAILed.

Test FAILed.

Test FAILed.

Test FAILed.

Test FAILed.

verify

Test FAILed.

Test PASSed.

Test FAILed.

Test FAILed.

verify

Test FAILed.

verify

Test FAILed.

verify

Test FAILed.

verify

Test PASSed.

Test FAILed.

verify

Test FAILed.

verify

Test FAILed.

verify

Test FAILed.

Test PASSed.


⚠ Bofrost.de has degraded performance

In 9401da1, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1682 ms

Resolved: Bofrost.de performance has improved in 80f325f after 15 minutes.


Added Default implementation for generic BuildHasher

I am not sure I understand the nature of the failure, do you happen to have any idea? These edits should not decrease the performance, I think.


fix: change offset type to Duration

Signed-off-by: Ruihang Xia<EMAIL_ADDRESS>According to the go version, the field offset is with type Duration rather than Instant:
// SubqueryExpr represents a subquery.
type SubqueryExpr struct {
	Expr  Expr
	Range time.Duration
	// OriginalOffset is the actual offset that was set in the query.
	// This never changes.
	OriginalOffset time.Duration
	// Offset is the offset used during the query execution
	// which is calculated using the original offset, at modifier time,
	// eval time, and subquery offsets in the AST tree.
	Offset     time.Duration
	Timestamp  *int64
	StartOrEnd ItemType // Set when @ is used with start() or end()
	Step       time.Duration

	EndPos Pos
}

and
// VectorSelector represents a Vector selection.
type VectorSelector struct {
	Name string
	// OriginalOffset is the actual offset that was set in the query.
	// This never changes.
	OriginalOffset time.Duration
	// Offset is the offset used during the query execution
	// which is calculated using the original offset, at modifier time,
	// eval time, and subquery offsets in the AST tree.
	Offset        time.Duration
	Timestamp     *int64
	StartOrEnd    ItemType // Set when @ is used with start() or end()
	LabelMatchers []*labels.Matcher

	// The unexpanded seriesSet populated at query preparation time.
	UnexpandedSeriesSet storage.SeriesSet
	Series              []storage.Series

	PosRange PositionRange
}

Yes, you are right. But I think it's because there is no equivalent Instant in Go. Go has to calculate the eval timestamp by Add or Sub (Sorry, but I haven't read the engine source code yet, I need to check it out). Actually I use Duration at first, then I changed to Instant.

Duration is a better choice, I mixed it with @.

Yes, you are right. But I think it's because there is no equivalent Instant in Go. Go has to calculate the eval timestamp by Add or Sub (Sorry, but I haven't read the engine source code yet, I need to check it out). Actually I use Duration at first, then I changed to Instant.

they use Time for start/end, maybe it's the equivalent in Go?
// EvalStmt holds an expression and information on the range it should
// be evaluated on.
type EvalStmt struct {
	Expr Expr // Expression to be evaluated.

	// The time boundaries for the evaluation. If Start equals End an instant
	// is evaluated.
	Start, End time.Time
	// Time between two evaluated instants for the range [Start:End].
	Interval time.Duration
	// Lookback delta to use for this evaluation.
	LookbackDelta time.Duration
}

And by the way, I think chrono's types like Duration or Instant are better suit for this usecase, which take timezone into consideration like time in Go


Download script not working

Hi there,
Thanks for releasing this codebase. Unfortunately, the donwload_videos.sh script seems not working. A file seems to be missing?
jq: error: Could not open file data/dict_sentences_per_verb_MARKERS_for_annotation_all50.json: No such file or directory
download_videos.sh: 6: download_videos.sh: cannot open : No such file
download_videos.sh: 6: download_videos.sh: mapfile: not found
download_videos.sh: 11: download_videos.sh: Syntax error: redirection unexpected

I just added the missing file.


How to display HTML report in azure pipeline tab?

Hi, I was trying to use some azure pipeline extensions to show the generated report in a tab, but I'm still unable to do it. I tested with:
HTML report viewer, but each scenario has its own file so the viewer cannot follow the  links, plus css and js are not being loaded.

Any ideas of how to properly display the index.html in a tab for azure pipelines?
Environment (please complete the following information):

multiple-cucumber-html-reporter: 2.0.0
Node.js version: 16
NPM version: 8.11
Platform name and version: debian 11
Cucucmber version: 7.3.0

A workaround I tried.
With external code, I merged all reports into a single HTML by looping through all feature/*.html files, getting the content of div.main_container and adding to the index.html file and looks good, HTML report viewer shows the complete report. I'm trying to follow the same approach with css and js files, but I'm still new with bundlers.
I wonder if a feature to generate a single HTML file can be done :thinking:

Hi @strymsg
Regarding the js/css issue, the screenshot that you shared, is that the first page that is loaded? The assets are by default loaded from an assets folder, you can also use usecdn: true in your config and then it will be loaded from a CDN
I don't know how the Azure plugin works, but my suggestion would be to check with the plugin. This module loads the data in a new url

Hi @wswebcreation , the screenshot is how the plugin shows the generated index.html file. useCDN looks good it did the trick with the css/js resources.

Regarding the feature/*.html files I used the workaround mentionend before and it is all properly displayed in the pipeline. Thanks :D

@strymsg Hi I try to use this tools too for azure but I can’t find a plugin. How do you use it on azure ?
Thanks, have a nice day.

Hi @wswebcreation , the screenshot is how the plugin shows the generated index.html file. useCDN looks good it did the trick with the css/js resources.

Regarding the feature/*.html files I used the workaround mentionend before and it is all properly displayed in the pipeline. Thanks :D

Can you please share the code to merge the feature/*html , i am stuck on same issue where i cannot follow the

@Swathi-Bhagwanji this a function I used to merge all feature files and index.html to an index-multiple.html. Hope it helps
const fs = require('fs');
const cheerio = require('cheerio');
const path = require('path');

/**
 * Merges scenario index.html and all html feature files generated by multiple-cucumber-report
 * into a single HTML file. Saves the file as index-multiple.html
 * @param reportPath Report path
 */
function mergeHtmlWithFeatureFiles(reportPath) {
  const htmlContent = fs.readFileSync(path.join(reportPath, 'index.html'), { encoding: 'utf8', flag: 'r' });
  const $ = cheerio.load(htmlContent);
  let aTags = $('a').toArray();
  aTags = aTags.filter((a) => {
    if (a.attribs && a.attribs.href) if (a.attribs.href.match(/features\//)) return a;
  });
  const filenames = [];
  for (const atag of aTags) {
    filenames.push(atag.attribs.href);
    atag.attribs.href = `#${atag.attribs.href.split('features/')[1].split('.html')[0]}`;
  }
  let featureContentsToCopy = '';
  for (const filename of filenames) {
    let htmlFeatureContent;
    try {
      htmlFeatureContent = fs.readFileSync(path.join(reportPath, filename), { encoding: 'utf-8', flag: 'r' });
    } catch (err) {
      throw new Error(`File ${filename} could not be read: ${err.message}`);
    }
    const featureContent = cheerio.load(htmlFeatureContent);
    featureContentsToCopy += `<hr><div id="${filename.split('features/')[1].split('.html')[0]}">`;
    featureContentsToCopy += featureContent('div.main_container').html();
    featureContentsToCopy += '
';
  }
  // copying the elments right after the last
 tag of the html body
  $('div.row').last().after(`<div id="features">${featureContentsToCopy}</div>`);
  try {
    fs.writeFileSync(path.join(reportPath, 'index-multiple.html'), $.html(), { enconding: 'utf-8', flag: 'w' });
  } catch (err) {
    throw new Error(`Error writing file: ${err}`);
  }
  console.log(`${$.html().length} bytes saved.`);
  return path.join(reportPath, 'index-multiple.html');
}

Thanks a lot for sharing the code :)
I still get further errors, lets see how it goes
On Thu, Jan 26, 2023 at 3:13 AM Rodrigo Garcia @.***>
wrote:

@Swathi-Bhagwanji https://github.com/Swathi-Bhagwanji this a function I
used to merge all feature files and index.html to an index-multiple.html.
Hope it helps
const fs = require('fs');const cheerio = require('cheerio');const path = require('path');
/** * Merges scenario index.html and all html feature files generated by multiple-cucumber-report * into a single HTML file. Saves the file as index-multiple.html * @param reportPath Report path */function mergeHtmlWithFeatureFiles(reportPath) {
const htmlContent = fs.readFileSync(path.join(reportPath, 'index.html'), { encoding: 'utf8', flag: 'r' });
const $ = cheerio.load(htmlContent);
let aTags = $('a').toArray();
aTags = aTags.filter((a) => {
if (a.attribs && a.attribs.href) if (a.attribs.href.match(/features//)) return a;
});
const filenames = [];
for (const atag of aTags) {
filenames.push(atag.attribs.href);
atag.attribs.href = #${atag.attribs.href.split('features/')[1].split('.html')[0]};
}
let featureContentsToCopy = '';
for (const filename of filenames) {
let htmlFeatureContent;
try {
htmlFeatureContent = fs.readFileSync(path.join(reportPath, filename), { encoding: 'utf-8', flag: 'r' });
} catch (err) {
throw new Error(File ${filename} could not be read: ${err.message});
}
const featureContent = cheerio.load(htmlFeatureContent);
featureContentsToCopy +=

;
featureContentsToCopy += featureContent('div.main_container').html();
featureContentsToCopy += '';
}
// copying the elments right after the last  tag of the html body
$('div.row').last().after(
${featureContentsToCopy}
);
try {
fs.writeFileSync(path.join(reportPath, 'index-multiple.html'), $.html(), { enconding: 'utf-8', flag: 'w' });
} catch (err) {
throw new Error(Error writing file: ${err});
}
console.log(${$.html().length} bytes saved.);
return path.join(reportPath, 'index-multiple.html');}
—
Reply to this email directly, view it on GitHub
https://github.com/WasiqB/multiple-cucumber-html-reporter/issues/216#issuecomment-1404517154,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANC62LEGRFBTC3ZCC3FR4RDWUHTXJANCNFSM6AAAAAAQYF654U
.
You are receiving this because you were mentioned.Message ID:
@.***>


Escape possible malicious chars

a case of CSV injections have become public/popular in mainstream media here. This PR sanitizes field values by prepending ' in front of possible malicious code. xlsx readers handle these values as strings
Tested with MS Excel, Numbers and LibreOffice.
ESCAPE_CHARS are taken from https://owasp.org/www-community/attacks/CSV_Injection

@FlipperPA any updates?

@willtho89 My apologies, but can you rebase from the latest on the main branch to resolve this merge conflict?

@FlipperPA done

@FlipperPA Any updates? Looking forward for a new release 😊

Thanks for the PR! This has been released in version 0.4.1: https://pypi.org/project/drf-renderer-xlsx/0.4.1/


About openpose keypoints

Hi, thanks for your wonderful work!
In the paper you mentioned that you get 2D keypoints for unlabelled datasets, are the pseudo 2D labels in openpose format?
I notice that in the code, openpose keypoints and confidence are used, I am wondering that the openpose keypoints are used for all datasets or used for unlabelled datasets.
Thanks a lot.

And I also wonder that how you generate openpose keypoints.

We use ViTPose to generate 2D keypoints. We do this for the AVA, InstaVariety and AI Challenger datasets.

We use ViTPose to generate 2D keypoints. We do this for the AVA, InstaVariety and AI Challenger datasets.

I see. So you use ViTPose to generate pseudo lables in openpose format for the above three datasets, while for other datasets you just use ground truth lables (without pseudo lables in openpose format), right?

For MPII, COCO and MPI-INF-3DHP, we use pseudo ground truth SMPL parameters generated by SPIN. For H36M, we use ground truth SMPL parameters from MoSh.


pyparsing.ParseException: Expected end of text

The newest version of a cantools (4.0.0) crashes while parsing dbc on string:
BA_ "BusType" "CAN";
Error log:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.5/dist-packages/cantools/db.py", line 547, in add_dbc
    tokens = self._grammar.parseString(dbc.read())
  File "/usr/local/lib/python3.5/dist-packages/pyparsing.py", line 1632, in parseString
    raise exc
  File "/usr/local/lib/python3.5/dist-packages/pyparsing.py", line 1622, in parseString
    loc, tokens = self._parse( instring, 0 )
  File "/usr/local/lib/python3.5/dist-packages/pyparsing.py", line 1379, in _parseNoCache
    loc,tokens = self.parseImpl( instring, preloc, doActions )
  File "/usr/local/lib/python3.5/dist-packages/pyparsing.py", line 3395, in parseImpl
    loc, exprtokens = e._parse( instring, loc, doActions )
  File "/usr/local/lib/python3.5/dist-packages/pyparsing.py", line 1383, in _parseNoCache
    loc,tokens = self.parseImpl( instring, preloc, doActions )
  File "/usr/local/lib/python3.5/dist-packages/pyparsing.py", line 3183, in parseImpl
    raise ParseException(instring, loc, self.errmsg, self)
pyparsing.ParseException: Expected end of text (at char 10209), (line:218, col:1)

File fragment:
BA_DEF_DEF_  "BusType" "CAN";
BA_ "BusType" "CAN";
BA_ "GenMsgCycleTime" BO_ 13 1;

Old, 1.0.0, works as expected.

The new version raises an exception if the parser fails to parse the whole file, while the old version silently ignored the rest of the file. Also, a lot of new syntax is accepted, ehich may have introduced a bug or two. BA_ and BA_DEF_DEF_ were introduced by pull request https://github.com/eerimoq/cantools/pull/2 recently, and not tested at all by me. No unit tests exists.
There are two options:

As a workaround, delete rows in the DBC-file that fails.
Fix the bug in cantools/db.py.

I can have a look tomorrow, but would be nice if you can make a fix and submit a pull request (with unit tests) =)

Ok, I craeted a new release, 4.0.1, which successfully parses your failed DBC-lines. Let me know if there are more problems, likely there is since the parser is not yet accepting all DBC-file syntax.

Just curious, did it solve your parsing problem?

Closing since no response was received, and that usually means it works.

Sorry, I was not able to answer your questions.
In the nearest future, I'll possibly again work with your amazing library and give you more feedback.
Best regards,
Alex


Fields Parameter does not work as expected

As documentation states, we may limit the fields returned in the response using the fields parameter like this GET /orders?fields=id. This pretty much works with orders endpoint, but returns strange results when using with customers/<id>/orders endpoint. For example:
customers/405/orders?fields=order_number,completed_at,line_items returns single cancelled order, whereas with only customers/405/orders it returns all processed user's orders without any cancelled ones.
Could you please investigate on this issue?
PS. I'm using woocommerce-api v1.0.4 and did not specify api version in the WooCommerceAPI constuctor, thus it is supposed to be v3.

Not an issue in this REST API wrapper. Maybe can be on WooCommerce. But here is not the right place for report WooCommerce issues.


Fix/column alias

Fixes https://github.com/stephencelis/SQLite.swift/issues/225 by allowing columns to be aliased. Also allows QueryClauses to be accessed publicly, though only modified internally.

closed as duplicate of #1081


Property 'getResizerProps' does not exist on type 'HeaderGroup'

Describe the bug (required)
getResizerProps() property does not work when using TypeScript with<EMAIL_ADDRESS>Steps To Reproduce (required)

    const tableInstance = useTable(
        { columns, data },
        useBlockLayout,
        useResizeColumns,
    );

    const {
        getTableProps,
        headerGroups,
    } = tableInstance;

return (
    // apply the table props
   <div {...getTableProps()} className="table">
       <div className="table-header">
           {
               headerGroups.map(headerGroup => (
                   <div {...headerGroup.getHeaderGroupProps()} className="table-header-row">
                        { headerGroup.headers.map(column => (
                            <div {...column.getHeaderProps()} className="th">
                            {column.render('Header')}
                            <div
                              {...column.getResizerProps()} //Property 'getResizerProps' does not exist on type 'HeaderGroup<object>'.  TS2339
                            />
                          </div>
                        ))}
                   </div>
               ))
           }
       </div>
   </div>
  )

Expected behavior (Recommended)
getResizerProps should exist on column

It seems the other plugins don't work either. useSortBy doesn't add any of its properties onto the column.

I have a similar issue with other plugin. I think the main problem comes from index.d.ts. They need to update it.

As per @types/react-table package it only supports up to react-table 7.0.4

As per @types/react-table package it only supports up to react-table 7.0.4

It needs this additional d.ts in your project to work with typescript https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/react-table#example-type-file

Simply add the bellow lines in a .d.ts file:
import { UseResizeColumnsColumnProps } from "react-table";

declare module "react-table" {
  export interface ColumnInstance<
    D extends Record<string, unknown> = Record<string, unknown>
  > extends UseResizeColumnsColumnProps<D> {}
}


feat [tests]: Add Monobank account pairing tests

Implemented here https://github.com/letehaha/budget-tracker-be/pull/76


It seems that the CG after each stage is not extracted

I will do it later. :)

Done with ver 1.1 (after two years :D).

Note: The CG after each stage is made by putting some elements together. The exact location of each element can be found from CompleteScreens.xml.


refactor: exploit symmetry of calculations to extract common parts

By exploiting symmetry, the functions essentially end up becoming:
def bd(xA, yA, xB, yB, interpolators=False):
    ...  # all the common functionality

def bd_PSNR(rateA, distA, rateB, distB, interpolators=False):
    rateA, distA, rateB, distB = _ensure_monotonic(rateA, distA, rateB, distB)
    rateA = np.log10(rateA)
    rateB = np.log10(rateB)
    return bd(rateA, distA, rateB, distB, interpolators=interpolators)

def bd_rate(rateA, distA, rateB, distB, interpolators=False):
    rateA, distA, rateB, distB = _ensure_monotonic(rateA, distA, rateB, distB)
    rateA = np.log10(rateA)
    rateB = np.log10(rateB)
    output = bd(distA, rateA, distB, rateB, interpolators=interpolators)

    if interpolators:
        (output, interp1, interp2) = output
    # convert to a percentage
    output = ((10 ** output) - 1) * 100
    if interpolators:
        output = (output, interp1, interp2)

    return output

(Admittedly, the bd_rate version has a bit verbose packing/unpacking of the tuple, but I kept that to avoid breaking existing code.)
This also has a bonus side-effect feature: the new bd function does not apply log10 to the input rates, so now we can use BD calculations on inputs that don't necessarily benefit from log-scaling.  (This is the reason I started looking at the code, actually.) Of course, bd_rate and bd_PSNR still apply the log10 to the rates.

TODO:

[ ] Run test suite

For future followup:

Ensure bd_akima, bd_piecewise_cubic, and bd_cubic all use similar setup and error checking code, e.g. _ensure_monotonic is inconsistent across each of these... and missing entirely for bd_cubic.
The error checking code is not sufficiently strict -- users can still pass in malformed inputs.

Based on your work, I streamlined the bjontegaard calculations even further. In the end, the only difference between the different bd calculation methods is the specific type of interpolation. This is much clearer now.
Also, as requested, bjontegaard now provides a bd_linear function that does not apply any log scaling to the set of input points.
These changes are included in #4.


Better post-quantization performance

Notes about Intriguing Properties of Quantization at Scale https://arxiv.org/pdf/2305.19268.pdf
So basically should do:

bf16 instead of fp16 training, less sensitive to quantization later
weight decay order 0.05, not too strong at 0.1 and not too weak at 0.01
layernorm at fp32
dropout order 0-0.1, not 0.4-0.8
gradient clipping order 1, not none

Related to 4-bit quant qlora paper: https://huggingface.co/blog/4bit-transformers-bitsandbytes#advanced-usage

won't do in this repo.


ARM64 Support

Does this supporting ARM64 as well? Can you confirm?

Sorry, brother, but it does not currently support arm64.

Sorry, brother, but it does not currently support arm64.

Well i managed to run this on ARM64 my dear friend. It works trust me.


Missing packages for local environment

Steps to reproduce:

pip3 install -r requirements/local.txt
python manage.py runserver

Missing packages:

django-debug-toolbar
django-extensions

Even though the packages are present in requirements/local.txt it seems to not work unless pip install <the-above-two-packages> are done. However, doing that it logs Requirements already satisfied.
@Ashesh3 also can confirm replicate behaviour

weird, I did not face such an issue 🤔

Can't be replicated. Reopen if applicable.


How Do I Uninstall and Install the Latest Version of scrcpy on Windows 11?

Hello, I am a beginner with computers. My brother installed scrcpy for me, but one day the screen-off-function suddenly stopped working.
The current version of scrcpy is 1.21.
I found out that I need to update to the latest version. However, I do not know how to uninstall and install scrcpy.
When I search, only the Linux version comes up.
I would appreciate it if you could tell me how to do this.

If there is a way to upgrade without uninstalling, please let me know that as well. I would like to use the screen off function while using scrcpy.

The package from your distribution is outdated. You must uninstall it:
sudo apt remove scrcpy

And you must install it using these steps: https://github.com/Genymobile/scrcpy/blob/master/doc/linux.md#latest-version

The package from your distribution is outdated. You must uninstall it:
sudo apt remove scrcpy

And you must install it using these steps: https://github.com/Genymobile/scrcpy/blob/master/doc/linux.md#latest-version

Thank you for your response. However, the link you provided is for Linux. I am a Windows user. Would it be okay if I still follow this method? Also, unfortunately, I am a complete beginner with computers. Should I use the sudo command in cmd?

Oh, sorry, for Windows, just delete your old folder, download the latest release and extract it: https://github.com/Genymobile/scrcpy/blob/master/doc/windows.md

Oh, sorry, for Windows, just delete your old folder, download the latest release and extract it: https://github.com/Genymobile/scrcpy/blob/master/doc/windows.md

Thank you very much. However, I'm afraid that if I uninstall scrcpy incorrectly, I won't be able to reinstall it. You suggested "sudo apt remove scrcpy," but the link "https://github.com/Genymobile/scrcpy/issues/1312" provides a different method. Which one is correct? As I mentioned, my brother installed scrcpy for me, and I simply clicked on the bat file. I don't know much about adb or scrcpy. Could you explain step by step how to do it from the beginning, including opening the cmd window?

On Windows, scrcpy is not "installed", it is just extracted to a directory. You can have several versions in different independent folders. There is nothing to "uninstall" (you can delete the directory if you want).

Could you explain step by step how to do it from the beginning, including opening the cmd window?

Ignore the version you already have.
Download the latest version: https://github.com/Genymobile/scrcpy/blob/master/doc/windows.md
Extract it
Open the folder
Follow these instructions: https://github.com/Genymobile/scrcpy/blob/master/doc/windows.md#run

The package from your distribution is outdated. You must uninstall it:
sudo apt remove scrcpy

And you must install it using these steps: https://github.com/Genymobile/scrcpy/blob/master/doc/linux.md#latest-version

The package from your distribution is outdated. You must uninstall it:
sudo apt remove scrcpy

And you must install it using these steps: https://github.com/Genymobile/scrcpy/blob/master/doc/linux.md#latest-version

The package from your distribution is outdated. You must uninstall it:
sudo apt remove scrcpy

And you must install it using these steps: https://github.com/Genymobile/scrcpy/blob/master/doc/linux.md#latest-version

Thank you for your kindness. All the problems have been solved perfectly.


cache: Fix bulk-fetch slowness for large result sets

Ready for review.

Don't merge this yet.  For at least one use case (hydrating messages) the cache is effective.  The meat of messages_for_ids (i.e. excluding some steps like dealing with message flags) always does better with memcached.  The reason for this is that we do the smart thing there--we cache the finalized Python data structure as JSON.  This means that when our cache is warm, we don't have to repeat steps such as sew_messages_and_reactions.

Ok, I am abandoning this for now.  Basically, all our bottlenecks are related to either the Django ORM or Python in general.  It makes sense to use the cache when it's not super-easy to avoid ORM/Python overhead, such as when we fetch messages from message ids.  But there are other places where we use the cache that are just stupid in the light of being able to avoid ORM overhead, and I hope to have a PR for that fairly soon.

oops, closing now