[Event Change Request] Codeunit 90 Purch.-Post - procedure PostCombineSalesOrderShipment- Event OnPostCombineSalesOrderShipmentOnAfterUpdateSalesOrderLine

We would like the Parameter/-s SalesOrderLine added to the following event.
Original:
Codeunit 90 Purch.-Post - procedure PostCombineSalesOrderShipment- Event OnPostCombineSalesOrderShipmentOnAfterUpdateSalesOrderLine

OnPostCombineSalesOrderShipmentOnAfterUpdateSalesOrderLine(SalesShptHeader, SalesOrderHeader, SalesOrderLine);

[IntegrationEvent(TRUE, false)]

    [IntegrationEvent(false, false)]
    local procedure OnPostCombineSalesOrderShipmentOnAfterUpdateSalesOrderLine(SalesShptHeader: Record "Sales Shipment Header"; SalesOrderHeader: Record "Sales Header"; var SalesOrderLine: Record "Sales Line)
    begin
    end;

Change:

OnPostCombineSalesOrderShipmentOnAfterUpdateSalesOrderLine(SalesShptHeader, SalesOrderHeader, SalesOrderLine, SalesShptLine);

[IntegrationEvent(TRUE, false)]

    [IntegrationEvent(false, false)]
    local procedure OnPostCombineSalesOrderShipmentOnAfterUpdateSalesOrderLine(SalesShptHeader: Record "Sales Shipment Header"; SalesOrderHeader: Record "Sales Header"; var SalesOrderLine: Record "Sales Line"; SalesShptLine : Record "Sales Shipment Line")
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 13321.


[FEATURE] JavaScript Capability's To Plugin Command

Currently the plugin command returns a plugin example in typescript but Sern may get users that make a bot in JavaScript. They might want JavaScript examples. I think it could be good to have a choice option on the slash command that allows users to specify whether they want TypeScript or JavaScript. JavaScript examples could be easily gathered, as plugins are now automatically converted to JavaScript in GitHub.

The examples that are returned are valid JavaScript
There are no types in the example, so its pure JavaScript.
Even if it did, JavaScript plugins won't have any different JSdoc comments
So basically any example that is written in the typescript file, will be copied to javascript file
But all of the plugins have valid javascript examples, there is no point adding option for language in /plugin command.


add Telia company to landscape issue #191

Build failed because of:
No cached entry, and can not fetch: teliasonera. Can't resolve stock ticker TELIA; please manually add a "stock_ticker" key to landscape.yml or set to null

Build failed because of:
No cached entry, and can not fetch: teliasonera. Can't resolve stock ticker STO:TELIA; please manually add a "stock_ticker" key to landscape.yml or set to null

had to change stock_ticker to null...


Update techmagic repositories to use ros2-gbp release repo.

These repositories have been mirrored into the ros2-gbp organization and the ros2-gbp copies should be used for releases going forward.
@Kansea please give a 👍🏼 or Approving review before the ROS team merges this PR.

👍🏼


Test-case label is added to JIRA along with other labels specified in trello

What
Using the tool to transfer cards with labels on them the label test-case is also added to each task in JIRA. This requires to be removed manually for tasks that are not test-case

There is a different items saying this should be configurable somehow - #2. I believe would could add a label flag in next version and change behavior of current invocation?
E.g. to preserve current behavior, one would need to add
--add-jira-label test-case

This has been fixed, in version 0.5.0 you can use --add-label parameter to specify comma separated list of labels that will be added to jira tickets. There is not any default label defined anymore.

@trepel @kpiwko thanks for the change. :D


Create script

Create script Inspired by reddit user
https://www.reddit.com/r/elementaryos/comments/3ivnb7/how_to_change_terminals_colors/cuklw6i

504892619d0ba2025a1a930fdffee88c257f577f


Partition values issue

Hi guys,
I am very interested in the Finra Herd project, and want to check it out, but after I followed the tutorial steps, the webapp told me the partition values are not correct, could someone help?
Also I just want to check out the Herd demo with some sample data, do you have an existing demo that people can play with?

Thanks a lot!

Hello Cheng, nice to meet you. I work at FINRA as the Product Owner of the
team that developed Herd and released it to the OSS community. I’m glad
you’re interested in Herd and took the time to get through the demo install
and tutorial.
Herd was developed as API-first and our primary use case at this time is
programmatic access by technical teams. The page you’re using was part of
our first foray into UI – the error you are seeing is caused by that page
making a [bad] assumption about how data is registered. I can provide a
curl script that adds data to the registrations you created in the quick
start tutorial – but the really good news is that we’re well into the
development of a UI that has much more comprehensive coverage of our
metadata model.
In my experience the best thing for us to do is arrange for a video or
telephone call to learn more about your interest and share more about why
we created Herd and what makes it click. I can also update you on plans for
our UI release and how to best demo Herd and create sample data.
I’m available to talk soon, have some openings mid-day Wed and Thurs and a
lot of Friday afternoon (all time US Eastern). Also free in evenings. Let
me know a few times that work for you and we can talk for 30 minutes or so
and take it from there.
Thanks

Nate Weisz

On Sep 12, 2016 9:53 PM, "Cheng Chen"<EMAIL_ADDRESS>wrote:

Hi guys,
I am very interested in the Finra Herd project, and wanted to check it
out, but when I followed the tutorial steps, it told me the partition
values are not correct, can someone help?
Also I just want to check out the Herd demo with some sample data, do you
have an existing sample that people can play with?
[image: screenshot 2016-09-13 09 49 45]
https://cloud.githubusercontent.com/assets/682886/18458868/84370000-7997-11e6-8b9f-2e0c957e4e66.png
Thanks a lot!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/FINRAOS/herd/issues/170, or mute the thread
https://github.com/notifications/unsubscribe-auth/ADFd9Yn1xsd5eXLBhq_EY1mXv52_8UCUks5qpgIJgaJpZM4J7Nqz
.

@nateiam thanks so much for the detailed response, I'd love to chat with you and learn more about Herd. We are trying to see if we can use Herd in our project as a metadata/schema management tool. Lets try to talk Mid-day Thursday, can I have your skype username(or your preferred communication tool)?

Looking forward to talking mid-day Thursday. I'm not on Skype much but am
always on Google Hangout chat as<EMAIL_ADDRESS>and also you can SMS me
at<PHONE_NUMBER> to make arrangements. Let me know a few times that work and
we can choose the right media to communicate :-)
On Tue, Sep 13, 2016 at 10:11 PM, Cheng Chen<EMAIL_ADDRESS>wrote:

@nateiam https://github.com/nateiam thanks so much for the detailed
response, I'd love to chat with you and learn more about Herd. We are
trying to see if we can use Herd in our project as a metadata/schema
management tool. Lets try to talk Mid-day Thursday, can I have your skype
username(or your preferred communication tool)?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/FINRAOS/herd/issues/170#issuecomment-246884863, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADFd9Un33p5Qbzwtamb6IjGFmiuFLvslks5qp1e2gaJpZM4J7Nqz
.

added you on google hangout! Looking forward to chat!


Updated compiling

Some link changes
Added dependency for extension-webm

extension-webm will be deleted soon from the repo but thanks for the pr anyways.


Enable-WriteCore-feature-for-storagepool

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.StoragePool


Blazor Initialize culture function

which method replaces Initializeculture, is that I need to apply translations and I see that the OnInit is not working (recommendations are accepted in any case),
blazor server side

Thanks for contacting us, @MarvynHarry.
We'll be writing documentation covering this for the upcoming Preview8 release.


Avoid requesting JARs when POM for a plugin is missing

This deactivates the legacy Maven fallback of just trying
to find the JAR when no POM file is present. We don't need
this fallback for Gradle plugins, as all of them should be
published with a POM file. This avoids unnecessary network
traffic.
@bigdaz assigning you since you just proposed this in the
discussion on bintray timeouts often failing on these.

Nevermind, I think I misunderstood your point. I now see that we look for a JAR even when we find a POM with pom packaging. Why do we do that?


expose remap index api

expose index remap options in compact_files

There are no index remapper options today right?  Why did we need to expose it?

There are no index remapper options today right? Why did we need to expose it?

we use the rust version directly in our internal repo. This isn't exposed to JS but we do use it in our internal code


Added ProtonVPN to VPN services

ProtonVPN is among the few no-log policy VPN providers, with features like Secure core servers, Perfect forward secrecy, etc. ProtonMail users can log-in with their proton account.
Plus, like Mullvad, users can pay for plans with Bitcoin.

Thank you sir

Thank you sir


cannot look up '' in pure evaluation mode

[cody@nixos:~]$ nix flake new -t "github:nrdxp/nixflk" flk # 2bdead324bbf15685db291cdd15c961d9d0d51a9
[cody@nixos:~]$ cd flk/
direnv: loading ~/flk/.envrc
direnv: using flake
direnv: export +AR +AS +CC +CONFIG_SHELL +CXX +HOST_PATH +IN_NIX_SHELL +LD +NIX_BINTOOLS +NIX_BINTOOLS_WRAPPER_TARGET_HOST_x86_64_unknown_linux_gnu +NIX_BUILD_CORES +NIX_BUILD_TOP +NIX_CC +NIX_CC_WRAPPER_TARGET_HOST_x86_64_unknown_linux_gnu +NIX_CFLAGS_COMPILE +NIX_ENFORCE_NO_NATIVE +NIX_HARDENING_ENABLE +NIX_INDENT_MAKE +NIX_LDFLAGS +NIX_STORE +NM +OBJCOPY +OBJDUMP +RANLIB +READELF +SIZE +SOURCE_DATE_EPOCH +STRINGS +STRIP +TEMP +TEMPDIR +TMP +TMPDIR +buildInputs +builder +configureFlags +depsBuildBuild +depsBuildBuildPropagated +depsBuildTarget +depsBuildTargetPropagated +depsHostHost +depsHostHostPropagated +depsTargetTarget +depsTargetTargetPropagated +doCheck +doInstallCheck +dontAddDisableDepTrack +name +nativeBuildInputs +nobuildPhase +out +outputs +patches +phases +propagatedBuildInputs +propagatedNativeBuildInputs +shell +shellHook +stdenv +strictDeps +system ~PATH ~XDG_DATA_DIRS

[cody@nixos:~/flk]$ rebuild NixOS test
[sudo] password for cody:
building the system configuration...
error: --- ThrownError ------------------------------------------------------------------- nix
at: (217:11) in file: /nix/store/wkjaivy3f1viwbj574n7p30wvd6llssi-source/modules/modules.nix

   216|         else
   217|           <nixpkgs>);
      |           ^
   218|       _module.args.pkgs = lib.mkDefault pkgs;

cannot look up '<nixpkgs>' in pure evaluation mode (use '--impure' to override)
(use '--show-trace' to show detailed location information)

Shouldn't everyone that uses this flake be getting this error? It seems like it's only me but that defeats the purpose of flakes having more hermetic builds right?

Maybe my config.home.stateVersion is messed up?
That way it tries to load nixpkgs is from:
      _module.args.pkgsPath = lib.mkDefault (
        if versionAtLeast config.home.stateVersion "20.09" then
          pkgs.path
        else
          <nixpkgs>);

You're not alone, I've spent a lot of time fighting this this morning...

As mentioned in the referenced home-manager issue 20.09 is the first home-manager state version to officially support flakes. The first, probably best option, is to change your state version to 20.09 (making sure to update anything you need to based on the 20.09 release notes and older releases if you're updating your state version from something before 20.03).
You can also run your flake command with --impure but then home-manager will use whatever nixpkgs is available globally on your system for it's pkgs.

I have resolved this in my config here. However, I this is not an ideal solution as it must be applied on a per user basis. Does anyone no of a simple way to defined a default value for this for any give user?

I have resolved this in my config here. However, this is not an ideal solution as it must be applied on a per user basis. Unfortunately, updating system.stateVersion doesn't help and it must be set per user.
Does anyone know of a simple way to define a default value for this for any give user?

I think it's alright to add the newer state version to the NixOS demo user most people probably copy to create their user.
As far as overriding the home manager module default state version... I'm not sure. override and mkForce come to mind though.

After quite some time, this solved this issue for me:
  home-manager = {
    useGlobalPkgs = true;
  };

I've set it in my user config for now, as that's where my home-manager config is, but it could also be set on a system level.

After quite some time, this solved this issue for me:
  home-manager = {
    useGlobalPkgs = true;
  };

I've set it in my user config for now, as that's where my home-manager config is, but it could also be set on a system level.

The source for useGlobalPkgs says:
        using the system configuration's <literal>pkgs</literal>
        argument in Home Manager. This disables the Home Manager
        options <option>nixpkgs.*</option>

Is that desirable? I'm having a hard time parsing those docs right now.

If I am understand correctly, this sidesteps the problem of import nixpkgs entirely by simply using the pkgs attribute defined by the nixos system configuration. Since we don't make any serious changed to nixpkgs beside importing the defined overlays, I don't think we lose anything, and this is probably the simplest solution.
I'll open a pr that closes this issue in a bit.

If I am understand correctly, this sidesteps the problem of importing nixpkgs entirely by simply using the pkgs attribute defined by the nixos system configuration.

But then you aren't using the nixpkgs defined by flakes. I'm guessing most users want their system based on flake inputs and not the nixpkgs channel their system previously had.

After following the code, I don't think thats what it's doing. I believe it is pulling in the pkgs defined here, unless I am mistaken. I'll leave the PR unmerged until everyone is confident.

After following the code, I don't think thats what it's doing. I believe it is pulling in the pkgs defined here, unless I am mistaken. I'll leave the PR unmerged until everyone is confident.

Maybe you are right... couldn't we test which is the case by adding something to pkgs in flake.nix and trying to use it from the home manager module?
Maybe I can get the time to try this later or a different test if someone suggests it.

Okay, so after reviewing the code one more time, and after viewing home-managers own README.md's section on flakes, it states that both:
home-manager.useGlobalPkgs = true;
home-manager.useUserPackages = true;

should be set when using flakes. It would be impossible for home-manager to use the previous generations nixpkgs, as that would also require it to pull something externally. I see that, when useGlobalPkgs is set, the nixpkgs module is skipped in home-manager and the NixOS pkgs attribute is used instead. This is exactly what we want, and adding both these lines has resolved the issue in all cases on my machine.
I am gonna go ahead and merge the PR. I feel it is safe at this point, and I don't want to frustrate new users any more than we need to.

Okay, so after reviewing the code one more time, and after viewing home-managers own README.md's section on flakes, it states that both:
home-manager.useGlobalPkgs = true;
home-manager.useUserPackages = true;

should be set when using flakes. It would be impossible for home-manager to use the previous generations nixpkgs, as that would also require it to pull something externally. I see that, when useGlobalPkgs is set, the nixpkgs module is skipped in home-manager and the NixOS pkgs attribute is used instead. This is exactly what we want, and adding both these lines has resolved the issue in all cases on my machine.
I am gonna go ahead and merge the PR. I feel it is safe at this point, and I don't want to frustrate new users any more than we need to.

Totally reasonable. Thanks for looking again, I just didn't want this template to get bad PR for subverting flakes or breaking any guarantees.
As soon as I figure out how to migrate to flakes I plan on writing some documentation I hope will help others do the same more smoothly if I can squeeze out the time


missing blank lines should cause an error, not a warning

i'm closing this for now until we can do a big merge


% over @define-placeholder

Any particular reason you used @define-placeholder over % (which is in the proposal)?

Yeah: the reason is that I wanted to use an existing flexible language construct, the at-rule, instead of trying to introduce a new one for just this purpose.
I'm not sure how far along that proposal is, how plausible it is that it will continue and eventually be adopted. Unless it's important to you or others I'd hesitate to try to introduce % at least until that is more likely to become a reality. Is it important?
I'll leave this open to see if other people have opinions on the issue.

Not really important to me, and I suspected that was the reason. I was just
curious! I agree the spec syntax is likely to change before then so it's
not really a concern of mine at this point - I just try to keep things as
"future-friendly" as possible. I have no real qualms with the current
implementation.
On Mon, Apr 13, 2015 at 11:21 David Clark<EMAIL_ADDRESS>wrote:

Yeah: the reason is that I wanted to use an existing flexible language
construct, the at-rule, instead of trying to introduce a new one for just
this purpose.
I'm not sure how far along that proposal is, how plausible it is that it
will continue and eventually be adopted. Unless it's important to you or
others I'd hesitate to try to introduce % at least until that is more
likely to become a reality. Is it important?
I'll leave this open to see if other people have opinions on the issue.
—
Reply to this email directly or view it on GitHub
https://github.com/davidtheclark/postcss-simple-extend/issues/3#issuecomment-92399649
.

I'd make that cleare in the README because in the first lines it's stated

The functionality should mirror Sass's @extend with % placeholders

Thus maybe later on, when explaining the syntax, it could be useful to explicitly state that % it's not supported

@mattecapu : Done at https://github.com/davidtheclark/postcss-simple-extend/blob/master/README.md#usage

@davidtheclark great!


make query property configurable

It would be nice to be able to modify the query property without having to monkey-patch.  Here's an example of how I'm going about it now: http://paste.pocoo.org/show/265847/

Killing this due to age.


Feature request: Trusted Device

Is your feature request related to a problem? Please describe.
A list for devices that will allow Quick Save
Describe the solution you'd like
A setting in a favorite device which will allow you to automatically accept file transfers from this device (Quick Save)
Describe alternatives you've considered
None
Additional context
No response

Yes, such implementation makes sense, especially considering that now there's an option to add devices to the "Favorites" category. It would be something similar to how it's done in Google's Nearby Share. Though in that case, it relates to displaying the device on the network, whereas in our case, it's about Quick Save.
Then, instead of a toggle switch in the settings, we would have a dropdown list for Quick Save with the following options:

Disabled;
Enabled only for Favorite devices;
Enabled for all.

In combination with a excellent feature request #860, the "Favorite" function, in my opinion, would make much more sense, as currently, it seems not very useful.


Network Zone not imported

Describe the bug
I use terraform-provider-dynatrace export dynatrace_network_zones which is supposed to downloads all existing network zone configuration
To Reproduce
Steps to reproduce the behavior:

launch terraform-provider-dynatrace export dynatrace_network_zones
Check downloaded network zone
I just have: resource "dynatrace_network_zones" "NetworkZones" {
enabled = true
}

Expected behavior
List of all the existing network zone.
Desktop (please complete the following information):

OS: windows

Additional context
Add any other context about the problem here.

Hi @MaudVandevelde,
Thank you for your patience. Release 1.16.0 includes the ability to configure network zones via 'dynatrace_network_zone. This resource is based off of environment API v2 and will have additional functionality beyond the previous dynatrace_network_zones` which only included the ability to enable/disable.
Thanks!
Kodai


Issue 2359 download all race condition

Created a new function to check collection status. Added ajax calls to check collection status before allowing user to download all submissions.

It looks like the locale strings didn't get committed.
Collection Incomplete Warning

@alexvail for the Portuguese translation, we've just been using Google translate for now, so please do that and add it in. Also, if you could delete the two blank lines that Hound mentioned, that would be great.

Hi @alexvail, sorry I didn't get a chance to look at this until now. Please look at some of the style comments Hound and I have left here.

Thanks, @alexvail!


Getting Started with Tensor flor - scalar.ndim

NameError                                 Traceback (most recent call last)
 in <cell line: 1>()
----> 1 scalar.ndim
NameError: name 'scalar' is not defined

could it be that the 'scalar' variable is not declared yet in your working notebook?


Issue with "ExploitProtections" not preventing a certain "exploit"

Detailed description of the issue:
Exploit Protection in generalConfig.yml only protects from hoppers and not from droppers. This means players can shoot items from a dropper directly into the furnace and brewing stand container and they will still get paid for it.

CONFIG SECTION (DELETE IF NOT RELEVANT):
ExploitProtections:
  Smelt:
    PreventHopperFillUps: true
  Brew:
    PreventBrewingStandFillUps: true

Jobs version:
<IP_ADDRESS>
Server Type (Spigot/Paper/etc):
Custom Pufferfish fork
Server Version (using /ver):
1.19.3.

Interesting bypass, and I think it's something that would be nice to have addressed. I will give zrips a poke about this.

@SirKillian
Could you elaborate more on this one. As of my knowledge and some testing, you can only fill up base ingredients with dropper into brewing stand and furnace, like only fuel for there furnace will be accepted which on itself removes block ownership. Can you give screenshot of exact setup (as simple as it is) in your Custom Pufferfish fork which is most likely the cause wich derps out default behavior and might need some more special support for it.

@SirKillian Could you elaborate more on this one. As of my knowledge and some testing, you can only fill up base ingredients with dropper into brewing stand and furnace, like only fuel for there furnace will be accepted which on itself removes block ownership. Can you give screenshot of exact setup (as simple as it is) in your Custom Pufferfish fork which is most likely the cause wich derps out default behavior and might need some more special support for it.

Sorry for the delay, here is a video on how it happens. I'm still getting paid after the hopper feeds the magma cream into the brewing stand; https://youtu.be/kyVNHG28y4I

Bump. This is still an issue. Using a setup like this

putting ingredients into the top, and blaze powder and water bottles into the dropper. It will brew potions when powered and ignore that the brewing stand is disabled. Big issue for sure.


License?

I'm very much interested in using this project and hopefully also contributing to it, but I don't see a license. Would you be so kind as to add one?

Hi sorry for the late reply, I've just added the MIT license. Feel free to let me know if you've any comments.


Return full object when valueProp is empty

 (2) Update MultiSelect and Select to use full object

 (2) Update MultiSelect and Select to use full object

Closing as un-needed.


Adjust LISTEN_PID for reexec in varlink mode

Because the varlink server honors the socket activation protocol,
LISTEN_PID has to be adjusted with the new PID.
https://varlink.org/FAQ.html#how-does-socket-activation-work
Signed-off-by: Harald Hoyer<EMAIL_ADDRESS>

bot, add author to whitelist

LGTM

/lgtm

/approved

/approve


trouble with semicolon in rss url

Hi
For some reason I am member of a site that uses ; in the query string, which results in qbittorrent controller truncated everything from that semi colon
site.con/rss?download;7;u=2305860;tp=0
When I add this as an rss feed I get an error that it couldn't download the feed and the url is changed to
site.com/rss?download
It probably works if I url encode the semi colons myself, but haven't tried it yet.

Nope even if I manually url encode both ; and = it doesn't work

I included the issue in the next release, so it will fixed as soon as possible.

I found the problem. RSS feed and its properties are stored one per line using semicolon as delimiter, therefore if you add an URL with semicolons you will be breaking the format.
As a possible solution I will try to encode the URL before store the RSS feed.

That sounds great, looking forward to test it :)
On Oct 26, 2015 11:35 PM, "Luis M. Gallardo D."<EMAIL_ADDRESS>wrote:

I found the problem. RSS feed and its properties are stored one per line
using semicolon as delimiter, therefore if you add an URL with semicolons
you will be breaking the format.
As a possible solution I will try to encode the URL before store the RSS
feed.
—
Reply to this email directly or view it on GitHub
https://github.com/lgallard/qBittorrent-Controller/issues/44#issuecomment-151304377
.

Fixed in testing. It will be available in the next upcoming release.


listVirtualMachinesMetrics doesn't filter VMs by Host ID

listVirtualMachinesMetrics with listall=true and hostid param returns all the VMs and doesn't return VMs by provided host ID. However, listVirtualMachines works.
Found via UI in env with more than 1 host. As root admin go to Infra -> hosts -> select a host -> see instances on this host.
ISSUE TYPE

Bug Report

COMPONENT NAME

UI

CLOUDSTACK VERSION

<IP_ADDRESS>

cc @rahulbcn27

@rohityadavcloud is this same as https://github.com/apache/cloudstack/issues/6756?

Not entirely sure @shwstppr - this is about the list API response; list API not filtering based on provided API param

Closing as addressed by @shwstppr in his PR https://github.com/apache/cloudstack/pull/6851

@rohityadavcloud #6851 is not merged yet. should this be closed?


#163177443 otpl events handler

This PR closes #163177443, by:

Adding listener to onetimepaymentlinks contract events
adding a feed index from hashedcode to txid
Updating feed txes with withdrawn status or cancelled if the original sender withdrawn it or cancelled it
Fixing some styles
...

Extra:

Any extra information #1
Any extra information #2
...

@dariominones what do you mean "no way of withdrawing" ? no way to cancel the link you sent to someone? there's a story for that.

@dariominones ok now it won't add a TX to the feed when you self withdraw a tx
i've also added an update to the index, so if a TX is withdrawn a few days later its day index needs to be updated
also I've encountered this strange thing (commit add: STRANGE), i've noticed that we JSON.stringify the day index content as it is an array but when we read the day index we don't need to JSON.parse it. it works also in the unit tests. in the browser if i try to get the day index manually it is returned as STRING!

@dariominones waiting your re-review


[bilibili] Unable to download JSON metadata: HTTP Error 502: Bad Gateway (caused by HTTPError())

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2021.12.17 2023.08.07, 2024.02.03
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

youtube-dl --verbose https://www.bilibili.com/video/BV1Ac411v7a8
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://www.bilibili.com/video/BV1Ac411v7a8']
[debug] Encodings: locale cp936, fs mbcs, out cp936, pref cp936
[debug] youtube-dl version 2024.02.03 [4416f82c8] (single file build)
[debug] ** This version was built from the latest master code at https://github.com/ytdl-org/youtube-dl.
[debug] ** For support, visit the main site.
[debug] Python 3.4.4 (CPython AMD64 32bit) - Windows-10-10.0.22000 - OpenSSL 1.0.2d 9 Jul 2015
[debug] exe versions: ffmpeg 2022-12-19-git-48d5aecfc4-full_build-www.gyan.dev, ffprobe 2022-12-19-git-48d5aecfc4-full_build-www.gyan.dev
[debug] Proxy map: {}
[BiliBili] 1Ac411v7a8: Downloading webpage
[BiliBili] 1Ac411v7a8: Downloading video info page
WARNING: Unable to download JSON metadata: HTTP Error 502: Bad Gateway
[BiliBili] 1Ac411v7a8: Downloading video info page
ERROR: Unable to download JSON metadata: HTTP Error 502: Bad Gateway (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "D:\a\ytdl-nightly\ytdl-nightly\youtube_dl\extractor\common.py", line 678, in _request_webpage
  File "D:\a\ytdl-nightly\ytdl-nightly\youtube_dl\YoutubeDL.py", line 2465, in urlopen
  File "C:\hostedtoolcache\windows\Python\3.4.4\x86\lib\urllib\request.py", line 470, in open
  File "C:\hostedtoolcache\windows\Python\3.4.4\x86\lib\urllib\request.py", line 580, in http_response
  File "C:\hostedtoolcache\windows\Python\3.4.4\x86\lib\urllib\request.py", line 502, in error
  File "C:\hostedtoolcache\windows\Python\3.4.4\x86\lib\urllib\request.py", line 442, in _call_chain
  File "C:\hostedtoolcache\windows\Python\3.4.4\x86\lib\urllib\request.py", line 685, in http_error_302
  File "C:\hostedtoolcache\windows\Python\3.4.4\x86\lib\urllib\request.py", line 470, in open
  File "C:\hostedtoolcache\windows\Python\3.4.4\x86\lib\urllib\request.py", line 580, in http_response
  File "C:\hostedtoolcache\windows\Python\3.4.4\x86\lib\urllib\request.py", line 508, in error
  File "C:\hostedtoolcache\windows\Python\3.4.4\x86\lib\urllib\request.py", line 442, in _call_chain
  File "C:\hostedtoolcache\windows\Python\3.4.4\x86\lib\urllib\request.py", line 588, in http_error_default

Description

The error is the same with version 2023.08.07, 2024.02.03, both without and with a proxy. I tried several videos but all with the same error.

Working in yt-dlp 2023.06.22 but 502 or 429 for me with yt-dl masater. Back-port needed.

Working in yt-dlp 2023.06.22 but 502 or 429 for me with yt-dl masater. Back-port needed.

Thanks, dirkf.  yt-dlp works. :)

The existing yt-dl extractor is completely obsolete. After some work:
$ python -m youtube_dl -vF 'https://www.bilibili.com/video/BV1Ac411v7a8'
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-vF', u'https://www.bilibili.com/video/BV1Ac411v7a8']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Git HEAD: 4416f82c8
[debug] Python 2.7.15 (CPython i686 32bit) - Linux-6.1.0-17-686-pae-i686-with-debian-12.5 - OpenSSL 1.1.1a  20 Nov 2018 - glibc 2.1.3
[debug] exe versions: ffmpeg 5.1.4-0, ffprobe 5.1.4-0
[debug] Proxy map: {}
[BiliBili] 1Ac411v7a8: Downloading webpage
[BiliBili] BV1Ac411v7a8: Extracting videos in anthology
[BiliBili] 284254516: Extracting chapters
[info] Available formats for BV1Ac411v7a8:
format code  extension  resolution note
30216        m4a        audio only   33k , mp4a.40.5
30232        m4a        audio only   67k , mp4a.40.2
100022       mp4        640x360      15k , av01.0.08m.<IP_ADDRESS>.01.01.0, 30.019fps, video only
30011        mp4        640x360      25k , hev1.1.6.l120.90, 30.303fps, video only
30016        mp4        640x360      39k , avc1.64001e, 30.303fps, video only
100023       mp4        854x480      19k , av01.0.08m.<IP_ADDRESS>.01.01.0, 30.019fps, video only
30033        mp4        854x480      32k , hev1.1.6.l120.90, 30.303fps, video only
30032        mp4        854x480      50k , avc1.64001f, 30.303fps, video only (best)
$

Is this just some sort of text presentation? I'm just seeing a logo at top right and various text bottom centre (with yt-dlp too).

The same.

This test URL https://www.bilibili.com/video/BV1jL41167ZG/ from the yt-dlp extractor is described as "supporter-only" but not detected as such, by either the original extractor or my back-port.
Is the short looping video that I see telling me that? I don't see any relevant text in a G-translated page or in the metadata.

Yes, the video in (https://www.bilibili.com/video/BV1jL41167ZG/) is indicating itself as supporter-only by saying "该视频为「高级充电回馈」专属视频 开通「18元档包月充电」即可观看" (￥18 / month)

Is this a standard video (that might be identified by its size, say)?
Otherwise, as I commented, it's not obvious how to detect "supporter-only" videos, which would be useful to do.

I'm not sure... Such kind of videos is very rare in bilibili - kinda surprised to see it being picked as a test URL, honestly.

This is just one of many test URLs. The point of tests is to use examples that cover all the various cases that users may encounter.
Perhaps this metadata item is_upower_exclusive indicates the status. Unfortunately, there isn't any relevant text in the page, or at least not in the static HTML that yt-dl sees. Another characteristic is that the video duration is ~10s while the full video should be ~695s, but that isn't very specific. More investigation needed.

Okay, thanks for detailed explanation :)

Indeed that is the only test video that has is_upower_exclusive_: true, so I'm assuming that it means "supporter-only".
A WIP extractor based on yt-dlp's (with the new PR https://github.com/yt-dlp/yt-dlp/pull/9117) looks good but is prone to hit a captcha page especially with Python 2.


Use informer.NewInformer where appropriate

Instead of manually creating a store using cache.NewStore and passing it to informer.NewInformerWithStore, use informer.NewInformer which does that internally.

/test

Runtime tests failed on #22078 (https://jenkins.cilium.io/job/Cilium-PR-Runtime-net-next/3897/) all other required tests passed and reviews are in. Marking as ready to merge.


Adds manually ran simulations

:tada: This PR is included in version 2.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


TriggerAim / MagnetTrigger

I know, it is ugly code, but it works and maybe someone of you can optimize it. Or someone can use it.
I think it is a good feature for gamers that use aimbot with aimlock and hold key on mouse 1.
When TriggerAim is active with mouse 5 for example, trigger will activate aimbot and make single shots while aimlocking.
I love it ;)
I don't know why the pull request have some code pulled that was not changed...Sorry for that, im learning how to use github

so do aimbot & trigaim share same cfg? or its individual

they share the same, when you watch the changes, it's just a "keymapping"... quick and dirty ;)

Better one here #751

THX @thomas0aotian will implement your way soon.

Alright bud and u sneakly implement standalone rcs u nerd. :smile: about to try it tmr, better be working:kissing_heart:


Awards info

Changes made in turing cup page are attached above

Remove the empty space between two components it should be continuous

Nice work it looks good


Feature Request: Drag and Drop Upload/Download

That's it. I would like to drag some file into the tree view and automatically upload it to the server.
By the way, awesome plugin man, I'd like to see it someday in the official package

Duplicate of #25 Implement drag & drop

Thank !


Create a nice page for Hapao

On GitHub

Readme file has been updated, do you think we can close this issue?


My Projects Page - Display

Web Path
Home Page -> John Doe -> My Projects

Requirement
Display the page as shown in wireframe
More detailed requirement will be provided when this task is assigned

Wireframe

Current Page

This issue was moved to Code4SocialGood/c4sg-web#60


GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run SQL PostCommit

Run SQL PostCommit


Update for RubyMine Versions 2023.3.2+

Unfortunately this gem is no longer compatible. It is so much better than RubyMine's endpoint tool that it is silly. Please update if possible!

agreed, sad not to have this amazing plugin!!

Please someone update this tool, its so good.


Rendering Parameters templates don't inherit from Standard Rendering Parameters

Description
Rendering Parameters templates created during app deployment don't inherit from the 'Standard Rendering Parameters' template. This affects updating presentation details of a rendering for other fields, such as Datasource and cache parameters.
Expected behavior
Rendering Parameters templates created during app deployment should include the 'Standard Rendering Parameters' as a base template
Steps To Reproduce

Add params to  a component
Deploy the app
View Presentation details for an item using the component and click to edit the component
Notice the only section available is 'data' and the default sitecore sections are missing

Possible Fix

Your Environment

Sitecore Version: 9.3 Initial
JSS Version:13.2.0
Browser Name and version: Chrome 84.0.4147.89
Operating System and version (desktop or mobile): Windows 10
Link to your project (if available):

Screenshots

This was discussed a little on Slack, adding some details from that conversation:

ExpandRenderingParametersTemplates does not set inherits on the rendering parameters template during import. should be able to inherit from ExpandRenderingParametersTemplates and override what you need there to set it.
CreateTemplateDefFromRenderingParametersDefs is the method you would override to set this.

Did this address the issue @slate1325?

This was discussed a little on Slack, adding some details from that conversation:

ExpandRenderingParametersTemplates does not set inherits on the rendering parameters template during import. should be able to inherit from ExpandRenderingParametersTemplates and override what you need there to set it.
CreateTemplateDefFromRenderingParametersDefs is the method you would override to set this.

Did this address the issue @slate1325?

@erzr , working on the fix now. Will keep you posted.
Thanks!

Created a github repo that addresses the issue: https://github.com/slate1325/JSS-Import-Rendering-Parameters-Template

Just a heads up, a backlog item has been added to address this issue in a future release. Thanks for reporting the issue.

Looks like this is still an issue? I'm new to jss so just wondering what is the impact of overriding generated rendering parameters to inherit from Standard Rendering Parameters template?

@markgibbons25 Just confirmed, this is still an open issue (I've poked to get prioritized). Using the workaround in the patch linked above to inherit the Standard Rendering Parameters template should have no detrimental impact, and in fact is what the proper fix will likely look like.

This would be fixed in the next release (20.0.0)


podman login with different mandatory parameter than docker login

/kind bug
Description
Ran into this problem, when I was checking if scripts developed for docker were running with podman.
podman supposed to be compatible to docker in terms of parameters. However, there are still breaking differences:
docker --version && docker login --help

Docker version 1.13.1, build 07f3374/1.13.1

Usage:  docker login [OPTIONS] [SERVER]

Log in to a Docker registry

Options:
      --help              Print usage
  -p, --password string   Password
  -u, --username string   Username

whereas:
podman --version && podman login --help

podman version 1.8.0

Login to a container registry

Description:
  Login to a container registry on a specified server.

Usage:
  podman login [flags] REGISTRY

Examples:
  podman login -u testuser -p testpassword localhost:5000
  podman login -u testuser -p testpassword localhost:5000
  podman login --authfile authdir/myauths.json quay.io

Flags:
      --authfile string   Path of the authentication file. Use REGISTRY_AUTH_FILE environment variable to override
      --cert-dir string   Pathname of a directory containing TLS certificates and keys used to connect to the registry
      --get-login         Return the current login user for the registry (default true)
  -p, --password string   Password for registry
      --password-stdin    Take the password from stdin
      --tls-verify        Require HTTPS and verify certificates when contacting registries (default true)
  -u, --username string   Username for registry

The issue the mandatory parameter (REGISTRY) for the podman command. This breaks any compatibility.
Describe the results you expected:
No additional mandatory parameter for parameter inherited from docker.

Well Docker is only supporting one registry Docker.io.  We could do similar and only support the first registry in registries.conf.  I am not crazy about hard coding docker.io into podman login.

This is exactly what we do. We change the registries.conf to point to an internal registry. This is where we push then our images with the scripts.

Ok, we can default to the first registry in registries.conf if it does not exist.
Would you be interested in opening a PR for this?

As soon as #5233 gets merged this issue will close.


Page breaks are misaligned on previewing multiple pages

Preflight Checklist

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

You must agree to search and the code of conduct. You must fill in this entire template. If you delete part/all or miss parts out your issue will be closed.
If you are technical, you should reporting bugs along the lines of https://marker.io/blog/how-to-write-bug-report. If you are not technical, we will make allowances, please try to make an effort to understand the process.
Describe the bug
When I try to preview multiple pages, page breaks are misaligned between edit screen and preview.
To Reproduce
Steps to reproduce the behavior:

Make a new file with A3/landscape
Place any shape at (0, 0) *upper left of the first page
Place another shape at (0, 23380) *upper left of the page no. 20
Press Ctrl + P to show print dialog
Click "Preview" button with default print configuration
See error; the second shape spans page no. 19 and no. 20

Expected behavior
The second shape is placed at the upper left of the page no. 20
Screenshots

draw.io version (In the Help->About menu of the draw.io editor):

draw.io version 24.4.13

Desktop (please complete the following information):

OS: Windows 10
Browser: Google Chrome
Browser Version: 125.0.6422.112

I tested the problem in incognito/private mode with all browser extensions switched off, write "yes" below:

yes

Additional context
The page number I specified above is not important. This behavior occurs with any numbers (e.g., 10, 20, 50, 100) of pages.

24.7.4

24.7.4
I confirmed the expected behavior. thank you so much!


[Feature]: Add triggerElement prop to the DialogRoot

Describe the feature
Currently, if a dialog is controlled by modelValue and does not use the DialogTrigger, focus cannot be handled properly (move focus back to the trigger after closing).
I suggest adding a triggerElement prop to the DialogRoot so that the user can specify which trigger was used to open the dialog.
Additional information

[X] I intend to submit a PR for this feature.
[x] I have already implemented and/or tested this feature.

Thanks for the issue and PR @enkot . However I have different solution in mind, similar to Headless UI.. perhaps internally we can track the previous focus of the target, and focus on it again when the dialog close.
This way, user might need to not set additional prop, and have a more consistent behaviour. Imagine 2 buttons could trigger the same dialog.

I agree, it will be a better solution.


BUG: Correctly handle broadcasted ydata in curve_fit pcov computation.

When pcov is passed a scalar y that's intended to be broadcasted,
the fitted value is correctly computed, but the computation of the
covariance matrix fails because y.size is not the output dimension.
Fix this by using the correct output dimension.
Closes #7018.

LGTM, merged. Thanks @anntzer


feat(cuda-add-ci): Create file 'windows-cuda.yml'

This commit contains a simple script file which installs CUDA & checks nvcc on runners' environments

You should check CodeFactor issues. I'll fix it soon.

I removed the redundant whitespaces in the file and commited again! Thanks.


How to use pytorch-transformers for transfer learning?

❓ Questions & Help
pretrained_weights='bert-base-uncased'
tokenizer = tokenizer_class.from_pretrained(pretrained_weights)
model = model_class.from_pretrained(pretrained_weights)

features = []
input_ids = torch.tensor([self.tokenizer.encode(phrase, add_special_tokens=True)])
with torch.no_grad():
    # Models outputs are tuples
    outputs = self.model(input_ids)
    last_hidden_states = outputs[0]
    avg_pool_hidden_states = np.average(last_hidden_states[0], axis=0))
    return avg_pool_hidden_states

I am working on Sentence Similarity problem, given sentence S, find similar sentences W. So I want to encode sentence S and find the closest top-K sentences from W.
I have read the documentations, but I want to confirm few things how do I do. the following:

How do I get the second to last layer?
Am I average pooling the last hidden states correctly?

Hey, so not exactly a direct answer to your question, but bert outright doesn't do amazing on sentence similarity. This repo here should help with your question and their paper does a great job at explaining how their method works https://github.com/UKPLab/sentence-transformers. I think you will find better results with that. Hope it helps.


Discrepancy Between Multi-Ligand Docking Score and Summed Individual Scores in AutoVina

I am using AutoVina for multi-ligand docking and encountered an issue related to the binding energy scores. A fter performing a multi-ligand docking run, I extracted each ligand from the complex and calculated their binding scores individually using the "score only" mode. I observed that the sum of  these individual scores is consistently higher than the binding energy score of the original multi-ligand docked complex.
I calculated multi-ligand docking score by these code:
`
rep=rep
lig=[A,B]
v = Vina()
v.set_receptor(rigid_pdbqt_filename=rep)
v.set_ligand_from_file(lig)
v.compute_vina_maps(center=center, box_size=box_size)
v.dock(min_rmsd=0.5)
v.dock(exhaustiveness=40)
v.write_poses(pdbqt_filename=out_dir+"%s_dockingPose.pdbqt"%("_".join(lig_name)))
print("file : %s_dockingPose.pdbqt is saved."%("and".join(lig_name)))
`
I calculated ligand docking score by these code:
`
docking ligand1
v = Vina(no_refine=True)
v.set_receptor(rep)
v.set_ligand_from_file(lig[0])
v.compute_vina_maps(center=center, box_size=box_size)
score = v.score()
print(f"Binding affinity:{lig_name} {sum(score)} kcal/mol")
docking ligand2
v = Vina(no_refine=True)
v.set_receptor(rep)
v.set_ligand_from_file(lig[1])
v.compute_vina_maps(center=center, box_size=box_size)
score = v.score()
print(f"Binding affinity:{lig_name}_SAM: {sum(score)} kcal/mol")
`
I use pymol to get the center and box size of the ligand.
Result:
The multi-ligand docking score:(A+B)
REMARK VINA RESULT:   -11.001      0.000      0.000
and each ligand docking score is follows:
Binding affinity:A: -19.916 kcal/mol
Binding affinity:B: -8.59 kcal/mol
Objective:
I aim to understand the contribution of each ligand  to the overall binding affinity in a multi-ligand context. However, the summed individual binding scores do not match the binding score of the multi-ligand complex, leading to confusion about how to interpret the results.
Questions:

Proper Methodology: What is the recommended approach to calculate and interpret the contribution of each ligand in a multi-ligand docking scenario using AutoVina?
Scoring and Energy Calculation: Are there specific settings or methodologies in AutoVina that should be used to achieve accurate individual ligand contributions from multi-ligand docking results?
Common Pitfalls : Are there known issues or adjustments to be aware of when comparing individual ligand scores to the multi-ligand docking score?

AutoVina version:

Hi @Squirrelity
This is because of how the number of torsions is counted for a multi-fragment ligand. I encourage you to check the following:
In your dual-ligand output, you can retrieve the number of torsions by the following values printed after REMARK:
total torsion numbers = (INTER / VINA RESULT - 1)/0.0585
From singly scored output, you can retrieve the number of torsions by the following values printed after REMARK:
individual torsion numbers = (score[1] / score[0] - 1)/0.0585
The individual torsion numbers should add up to the total torsion number. As you can see, the multi-ligand poses are penalized more (scaled down) because of the degrees of freedom.
I'm not very sure about the best scoring function for docking multiple ligands. But one common pitfall i can think of is that the Vina scoring function penalizes ligands with higher degrees of freedom by a scaling factor. Therefore, I would avoid comparing ligands that are drastically different in terms of size and/or number of free torsions. The scoring function might be more useful (and fair) when comparing ligands or multi-ligand systems that are similar in size and number of free torsions.
Please let us know what you think and if you find any further issues with that!

Hi @rwxayheee
Thank you for the detailed explanation regarding the discrepancy due to calculations in multi-ligand docking.
I have followed your instauctions to calculate the individual torsion numbers and the total torsion number. Here are my results:
Individual Torsion Numbers:
For ligand A: 6.996327125672311 kcal/mol
For ligand B: 6.996979628558575 kcal/mol
Calculation Details for Total Torsion Numbers from Multi-Ligand Output:
MODEL 1 REMARK VINA RESULT:   -11.001      0.000      0.000 REMARK INTER + INTRA: -22.274 REMARK INTER:         -20.004 REMARK INTRA:         -2.270 REMARK UNBOUND:       -2.270
Using the formula (INTER / VINA RESULT - 1) / 0.0585 ,I computed the total torsion number to be 28.49335204972235, which does not match the sum of the individual torsion numbers.
This discrepancy between the total torsion number and the sum of individual torsion numbers is causing confusion. Could you please clarify whiy the total number from the multi-ligand output does not math the sum of the individual torsion number?

Hi @Squirrelity
Thanks for the information. Could you please share an example of your docking commands, and receptor & ligands files? I can take a closer look at them

Hi @rwxayheee
My ligand is v22 from 7O2I（pdb）and sam from 5K7U(pdb)，my receptor is 5K7U.
I used python api to dock. My docking commands  are as follows:
`lig_list = [[v22],[v22,sam]]
for lig in lig_list:
rep = 5K7U
lig_name = [i.split("/")[-1].split(".")[0] for i in lig]
v = Vina()
v.set_receptor(rigid_pdbqt_filename=rep)
v.set_ligand_from_file(lig)
v.compute_vina_maps(center=center, box_size=box_size)
v.dock(min_rmsd=0.5)
v.dock(exhaustiveness=40)
v.write_poses(pdbqt_filename=out_dir+"%s_dockingPose.pdbqt"%("_".join(lig_name)))
print("file : %s_dockingPose.pdbqt is saved."%("_and_".join(lig_name)))`

I found the "INTER" and "Vina result" from dockingPose.pdbqt
and I calculate individual torsion number by using this code:
   v = Vina(no_refine=True)     v.set_receptor(METTL3_METTL14_5K7U)     v.set_ligand_from_file(lig)     v.compute_vina_maps(center=center, box_size=box_size)     score = v.score()     print(f"Binding affinity:{lig_name}_SAM: {sum(score)} kcal/mol")         individual_torsion_numbers = (score[1] / score[0] - 1)/0.0585     print(f"individual_torsion_numbers:{individual_torsion_numbers} kcal/mol")

Hi @Squirrelity
I will set up a similar calculation in a moment. After taking a quick look at the PDB structures, one thing I noticed in your original post is that -
When you were doing "docking ligand1" and "docking ligand1", you didn't v.dock(). Was that intentional? If you didn't  ask Vina to dock, the scores will be from single-point evaluation. The original positions of the ligands in their inputs would not change, and the scores likely won't add up to a co-docking score, especially if the original positions overlap
I will write another reply with my docking results shortly

Hi @Squirrelity
With just v.dock(), the torsions should add up in these single point calculations. Here's what I did & what I got:
5K7U_A.pdbqt.txt
V22.pdbqt.txt
SAM.pdbqt.txt
from vina import Vina

v = Vina(sf_name='vina')

v.set_receptor('5K7U_A.pdbqt')

lig_V22 = 'V22.pdbqt'
lig_SAM = 'SAM.pdbqt'

bcenter = [-43.014, 34.412, 25.684]
bsize = [40, 40, 40]

v.compute_vina_maps(center=bcenter, box_size=bsize)

v.score() for V22; Ntor = 7
Input
v.set_ligand_from_file(lig_V22)

score = v.score()
print(score)
print((score[1]/score[0]-1)/0.0585)

Output
[ 32.668  46.037   0.      0.      0.     -0.651 -13.369  -0.651]
6.995528178337043

v.score() for SAM; Ntor = 9
Input
v.set_ligand_from_file(lig_SAM)

score = v.score()
print(score)
print((score[1]/score[0]-1)/0.0585)

Output
[-4.807 -7.337  0.     0.     0.    -0.634  2.529 -0.634]
8.996851102114258
v.score() for V22 and SAM; Ntor = 16
Input
v.set_ligand_from_file([lig_V22, lig_SAM])

score = v.score()
print(score)
print(((score[1]+score[2])/score[0]-1)/0.0585)

Output
[ 220.713   38.7    388.458    0.       0.      -1.285 -206.446   -1.285]
15.988973730475136
``
Here Vina writes the interactions between ligands in `flex_inter` (which is `score[2]`). I didn't know about this... So to calculate the Number of torsions (Ntor), we should do:
`((score[1]+score[2])/score[0]-1)/0.0585`?
I'm not entirely sure when `other_inter` (`score[3]`) could get a nonzero value. But for multi-ligand docking, I will do a bit more checking on how docking outputs are written. Thanks again for bringing this issue up


[Cosmos] Full Text Search (preview)

We can use this to track Full Text Search

Is your feature request related to a problem? Please describe.
More details to follow.
Describe the solution you'd like
More details to follow.
Describe alternatives you've considered
More details to follow.

Done with https://github.com/Azure/azure-sdk-for-python/pull/38275.


Topics are not available

Describe the bug
Topics are not available
Set up
https://www.kafka-ui.provectus.io/ui/clusters/local/all-topics?perPage=25
Screenshots

resolved


Can't run netlify dev on wsl2.

Describe the bug
I'm unable to run any commands with netlify on wsl2, getting an error saying it is unable to spawn powershell.exe.
To Reproduce
Steps to reproduce the behavior:

netlify dev
or .node_modules/.bin/netlify dev

Configuration

node v12 installed via nvm on wsl2.
have tried with node v14 also
have tried local and global installs of netlify-cli

current /etc/wsl.conf:
[network]

[automount]
enabled=true

[user]
default=scarter1

# Mount root to /drive/ instead of /mnt/drive for docker compatibility.
root=/

# Unsets (unmasks) -wx bits for group/other so perms don't show up as 777 on WSL.
options="metadata,umask=22,fmask=11"

I've tried adding

[interop]
enabled=false
appendWindowsPath=false

but that didn't work.
npx envinfo --system --binaries --npmPackages netlify-cli --npmGlobalPackages netlify-cli

  System:
    OS: Linux 4.19 Ubuntu 20.04.2 LTS (Focal Fossa)
    CPU: (6) x64 Intel(R) Core(TM) i5-8500 CPU @ 3.00GHz
    Memory: 1.49 GB / 4.82 GB
    Container: Yes
    Shell: 5.0.17 - /bin/bash
  Binaries:
    Node: 12.14.1 - ~/.nvm/versions/node/v12.14.1/bin/node
    Yarn: 1.22.10 - ~/Development/react-conf-videos/node_modules/.bin/yarn
    npm: 6.13.4 - ~/.nvm/versions/node/v12.14.1/bin/npm
  npmPackages:
    netlify-cli: ^3.39.4 => 3.39.4
  npmGlobalPackages:
    netlify-cli: 3.39.4

Expected behavior
It runs the development server
CLI Output
◈ Netlify Dev ◈
◈ Ignored general context env var: LANG (defined in process)
◈ No app server detected and no "command" specified
◈ Using current working directory
◈ Unable to determine public folder to serve files from
◈ Setup a netlify.toml file with a [dev] section to specify your dev server settings.
◈ See docs at: https://cli.netlify.com/netlify-dev#project-detection
◈ Running static server from "react-conf-videos"
◈ Functions server is listening on 33877

◈ Server listening to 3999

/home/scarter1/Development/demoProject/node_modules/netlify-redirector/lib/redirects.js:116
      throw ex;
      ^

Error: spawn /c/Windows/System32/WindowsPowerShell/v1.0/powershell.exe ENOENT
    at Process.ChildProcess._handle.onexit (internal/child_process.js:264:19)
    at onErrorNT (internal/child_process.js:456:16)
    at processTicksAndRejections (internal/process/task_queues.js:81:21)
Emitted 'error' event on ChildProcess instance at:
    at Process.ChildProcess._handle.onexit (internal/child_process.js:270:12)
    at onErrorNT (internal/child_process.js:456:16)
    at processTicksAndRejections (internal/process/task_queues.js:81:21) {
  errno: 'ENOENT',
  code: 'ENOENT',
  syscall: 'spawn /c/Windows/System32/WindowsPowerShell/v1.0/powershell.exe',
  path: '/c/Windows/System32/WindowsPowerShell/v1.0/powershell.exe',
  spawnargs: [
    '-NoProfile',
    '-NonInteractive',
    '–ExecutionPolicy',
    'Bypass',
    '-EncodedCommand',
    'UwB0AGEAcgB0ACAAIgBoAHQAdABwADoALwAvAGwAbwBjAGEAbABoAG8AcwB0ADoAOAA4ADgAOAAiAA=='
  ]
}

Was able to do a temp fix by doing export BROWSER=none, from https://github.com/facebook/create-react-app/issues/7251#issuecomment-827602712.


[rtl] Replace always_ff with always @(posedge .. in FPGA regfile

This resolves lowRISC/Ibex#1485.
Note: I've tested that these changes don't impact the Vivado's capability to infer LUTRAM for the FPGA regfile.

Thanks @rswarbrick !


Calculate identity deposit

closes #499

Submission checklist:
Layout

[ ] Change inspected in the desktop web ui
[ ] Change inspected in the mobile web ui

Compatibility

[ ] Functionality of change validated with a connected account with multisig
[ ] Applicable elements hidden / disabled for watched multisigs / pure
[ ] Looks good for solo multisig
[ ] Looks good for multisig with proxy

wrong branch https://github.com/ChainSafe/Multix/pull/502


use previously calculated stored_size

Problem
Trying to remove mmap on append vecs to relieve memory pressure. Also working towards using hot storage format.
Summary of Changes
There already a stored_size on an account while doing ancient packing. Use that instead of recalculating it. This is especially important because the stored size may be different when we have mixed storage types.
Fixes #

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

Project coverage is 81.9%. Comparing base (9706a64) to head (4e65f5c).
Report is 9 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #757   +/-   ##
=======================================
  Coverage    81.9%    81.9%
=======================================
  Files         851      851
  Lines      231005   231070   +65
=======================================
+ Hits       189319   189388   +69
+ Misses      41686    41682    -4


ANSI-721 - ZK Digest Auth Mode fix + variable rename

Description

Fixes bug that actually enforces digest auth for server to server auth.
Creates two new variables zookeeper_quorum_authentication_type and zookeeper_client_authentication_type - before it was unclear what auth was applied to client vs quorum connections

ZK Docs: https://cwiki.apache.org/confluence/display/ZOOKEEPER/Server-Server+mutual+authentication
Fixes # (issue)
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Reran zk-kerberos and zk-digest scenarios
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Any dependent changes have been merged and published in downstream modules
[ ] Any variable changes have been validated to be backwards compatible

zookeeper_health_check_command: "{% if zookeeper_sasl_protocol in ['kerberos', 'digest']

this should use zookeeper_client_auth_type

Double check zookeeper health check command with digest auth type

@xnulinu I did some testing around the health checks and when kerberos client auth is enabled the health check command ran without a jaas file...
usr/bin/kafka-run-class org.apache.zookeeper.client.FourLetterWordMain zookeeper1 2181 srvr
Zookeeper version: 3.5.8-f439ca583e70862c3068a1f2a7d4d068eec33315, built on 05/04/2020 15:53 GMT
Latency min/avg/max: 0/0/2
Received: 49
Sent: 48
Connections: 4
Outstanding: 0
Zxid: 0x144
Mode: standalone
Node count: 268

I also confirmed that secureClientPort must be used w TLS connections. It seems like the jaas file is not necessary in the health check commands


Improve on Starling drawing functions

Primitive drawing functions have taken a big hit with the move to Starling, since they're basically down to running Bresenham and setpixeling everything. See what you can do about improving these!

Reimplemented this starling extension in haxe, which fixes the problem! Primitives are back! https://github.com/Fovea/starling-shapes


Pages cloud not resolve node core module perf_hooks

yarn run v1.22.17
$ cross-env NODE_ENV=development run-p dev:*
$ remix watch
$ npm run build:css -- --w
$ wrangler pages dev ./public --watch ./build

> build:css
> postcss styles/**/*.css --base styles --dir app/styles "--w"

Watching Remix app in development mode...
💿 Built in 152ms

warn - The `purge`/`content` options have changed in Tailwind CSS v3.0.
warn - Update your configuration file to eliminate this warning.
Compiling worker to "/var/folders/td/zr87rwn546n4q0y3zddn95zw0000gn/T/functionsWorker.js"
✘ [ERROR] Could not resolve "perf_hooks"

    build/index.js:86:28:
      86 │ import { performance } from "perf_hooks";
         ╵                             ~~~~~~~~~~~~

  The package "perf_hooks" wasn't found on the file system but is built into node. Are you trying to bundle for node? You can use "platform: 'node'" to do that, which will remove this error.

/Users/v0/Sites/poc/my-remix-app/node_modules/@cloudflare/pages-functions-compiler/node_modules/esbuild/lib/main.js:1493
  let error = new Error(`${text}${summary}`);
              ^

Error: Build failed with 1 error:
build/index.js:86:28: ERROR: Could not resolve "perf_hooks"
    at failureErrorWithLog (/Users/v0/Sites/poc/my-remix-app/node_modules/@cloudflare/pages-functions-compiler/node_modules/esbuild/lib/main.js:1493:15)
    at /Users/v0/Sites/poc/my-remix-app/node_modules/@cloudflare/pages-functions-compiler/node_modules/esbuild/lib/main.js:1151:28
    at runOnEndCallbacks (/Users/v0/Sites/poc/my-remix-app/node_modules/@cloudflare/pages-functions-compiler/node_modules/esbuild/lib/main.js:1069:65)
    at buildResponseToResult (/Users/v0/Sites/poc/my-remix-app/node_modules/@cloudflare/pages-functions-compiler/node_modules/esbuild/lib/main.js:1149:7)
    at /Users/v0/Sites/poc/my-remix-app/node_modules/@cloudflare/pages-functions-compiler/node_modules/esbuild/lib/main.js:1258:14
    at /Users/v0/Sites/poc/my-remix-app/node_modules/@cloudflare/pages-functions-compiler/node_modules/esbuild/lib/main.js:629:9
    at handleIncomingPacket (/Users/v0/Sites/poc/my-remix-app/node_modules/@cloudflare/pages-functions-compiler/node_modules/esbuild/lib/main.js:726:9)
    at Socket.readFromStdout (/Users/v0/Sites/poc/my-remix-app/node_modules/@cloudflare/pages-functions-compiler/node_modules/esbuild/lib/main.js:596:7)
    at Socket.emit (node:events:390:28)
    at addChunk (node:internal/streams/readable:324:12) {
  errors: [
    {
      detail: undefined,
      location: {
        column: 28,
        file: 'build/index.js',
        length: 12,
        line: 86,
        lineText: 'import { performance } from "perf_hooks";',
        namespace: '',
        suggestion: ''
      },
      notes: [
        {
          location: null,
          text: `The package "perf_hooks" wasn't found on the file system but is built into node. Are you trying to bundle for node? You can use "platform: 'node'" to do that, which will remove this error.`
        }
      ],
      pluginName: '',
      text: 'Could not resolve "perf_hooks"'
    }
  ],
  warnings: []
}

Node.js v17.2.0
wrangler pages dev [directory] [-- command]

🧑💻 Develop your full-stack Pages application locally

Positionals:
  directory  The directory of static assets to serve  [string]
  command    The proxy command to run  [string]

Flags:
  -c, --config   Path to .toml configuration file  [string]
  -h, --help     Show help  [boolean]
  -v, --version  Show version number  [boolean]

Options:
  -l, --local        Run on my machine  [boolean] [default: true]
      --port         The port to listen on (serve from)  [number] [default: 8788]
      --proxy        The port to proxy (where the static assets are served)  [number]
      --script-path  The location of the single Worker script if not using functions  [string] [default: "_worker.js"]
  -b, --binding      Bind variable/secret (KEY=VALUE)  [array]
  -k, --kv           KV namespace to bind  [array]
  -o, --do           Durable Object to bind (NAME=CLASS)  [array]

BuildError [1]: Build failed with exit code 1
    at ChildProcess.<anonymous> (/Users/v0/Sites/poc/my-remix-app/node_modules/@miniflare/core/src/plugins/build.ts:73:25)
    at ChildProcess.emit (node:events:390:28)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:290:12) {
  code: 1,
  cause: undefined
}
/Users/v0/Sites/poc/my-remix-app/node_modules/wrangler/wrangler-dist/cli.js:27072
            throw ex;
            ^

BuildError [1]: Build failed with exit code 1
    at ChildProcess.<anonymous> (/Users/v0/Sites/poc/my-remix-app/node_modules/@miniflare/core/src/plugins/build.ts:73:25)
    at ChildProcess.emit (node:events:390:28)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:290:12) {
  code: 1,
  cause: undefined
}

Node.js v17.2.0
error Command failed with exit code 7.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
ERROR: "dev:wrangler" exited with 7.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

besides. process is not defined too.
warn - The `purge`/`content` options have changed in Tailwind CSS v3.0.
warn - Update your configuration file to eliminate this warning.
Compiling worker to "/var/folders/td/zr87rwn546n4q0y3zddn95zw0000gn/T/functionsWorker.js"
Finished in 55ms.
wrangler pages dev [directory] [-- command]

🧑💻 Develop your full-stack Pages application locally

Positionals:
  directory  The directory of static assets to serve  [string]
  command    The proxy command to run  [string]

Flags:
  -c, --config   Path to .toml configuration file  [string]
  -h, --help     Show help  [boolean]
  -v, --version  Show version number  [boolean]

Options:
  -l, --local        Run on my machine  [boolean] [default: true]
      --port         The port to listen on (serve from)  [number] [default: 8788]
      --proxy        The port to proxy (where the static assets are served)  [number]
      --script-path  The location of the single Worker script if not using functions  [string] [default: "_worker.js"]
  -b, --binding      Bind variable/secret (KEY=VALUE)  [array]
  -k, --kv           KV namespace to bind  [array]
  -o, --do           Durable Object to bind (NAME=CLASS)  [array]

ReferenceError: process is not defined
    at env (/var/folders/td/zr87rwn546n4q0y3zddn95zw0000gn/T/functionsWorker.js:19498:17)
    at isProduction (/var/folders/td/zr87rwn546n4q0y3zddn95zw0000gn/T/functionsWorker.js:19504:10)
    at build/index.js (/var/folders/td/zr87rwn546n4q0y3zddn95zw0000gn/T/functionsWorker.js:19616:17)
    at __init (/var/folders/td/zr87rwn546n4q0y3zddn95zw0000gn/T/functionsWorker.js:9:50)
    at functions/[[path]].js (/var/folders/td/zr87rwn546n4q0y3zddn95zw0000gn/T/functionsWorker.js:19774:5)
    at __init (/var/folders/td/zr87rwn546n4q0y3zddn95zw0000gn/T/functionsWorker.js:9:50)
    at ../../../../../var/folders/td/zr87rwn546n4q0y3zddn95zw0000gn/T/plinko-ZWdvUa/routes.mjs (/var/folders/td/zr87rwn546n4q0y3zddn95zw0000gn/T/functionsWorker.js:19785:5)
    at __init (/var/folders/td/zr87rwn546n4q0y3zddn95zw0000gn/T/functionsWorker.js:9:50)
    at /var/folders/td/zr87rwn546n4q0y3zddn95zw0000gn/T/functionsWorker.js:19798:1
    at SourceTextModule.evaluate (node:internal/vm/module:224:23)
[pages:err] Unhandled Promise Rejection: ReferenceError: process is not defined
    at env (/var/folders/td/zr87rwn546n4q0y3zddn95zw0000gn/T/functionsWorker.js:19498:17)
    at isProduction (/var/folders/td/zr87rwn546n4q0y3zddn95zw0000gn/T/functionsWorker.js:19504:10)
    at build/index.js (/var/folders/td/zr87rwn546n4q0y3zddn95zw0000gn/T/functionsWorker.js:19616:17)
    at __init (/var/folders/td/zr87rwn546n4q0y3zddn95zw0000gn/T/functionsWorker.js:9:50)
    at functions/[[path]].js (/var/folders/td/zr87rwn546n4q0y3zddn95zw0000gn/T/functionsWorker.js:19774:5)
    at __init (/var/folders/td/zr87rwn546n4q0y3zddn95zw0000gn/T/functionsWorker.js:9:50)
    at ../../../../../var/folders/td/zr87rwn546n4q0y3zddn95zw0000gn/T/plinko-ZWdvUa/routes.mjs (/var/folders/td/zr87rwn546n4q0y3zddn95zw0000gn/T/functionsWorker.js:19785:5)
    at __init (/var/folders/td/zr87rwn546n4q0y3zddn95zw0000gn/T/functionsWorker.js:9:50)
    at /var/folders/td/zr87rwn546n4q0y3zddn95zw0000gn/T/functionsWorker.js:19798:1
    at SourceTextModule.evaluate (node:internal/vm/module:224:23)
💿 File changed: app/root.tsx
💿 File changed: app/services/theme.server.ts
💿 Rebuilding...

Closing as dupe of #1074.


Make FlatState a Mapping instead of a dict

Add nnx.traversals.{flatten_mapping, unflatten_dict}

These are modified from traverse_util.{flatten_dict, unflatten_dict}
They are annotated so that any future changes to the function (e.g., changing it back to work with dicts only) triggers a type error.

Minor tweaks to imports:

import from collections.abc instead of typing since the latter imports are deprecated.
Import from flax.typing instead of flax.core.scope since the latter has been moved.
Add from __future__ import annotations so that the annotations work on Python 3.9.

Minor tweaks to code:

Annotate some private functions to make them easier to understand.
When printing a type, print its qualname since that's a bit easier to read (str instead of <class 'str'>).

Fixes #3879

@cgarciae (Done, as you requested.)

Codecov Report
Attention: Patch coverage is 0% with 65 lines in your changes are missing coverage. Please review.

Project coverage is 0.00%. Comparing base (2c7d7cd) to head (6776c47).
Report is 42 commits behind head on main.

Files
Patch %
Lines

flax/experimental/nnx/nnx/traversals.py
0.00%
58 Missing :warning:

flax/experimental/nnx/nnx/state.py
0.00%
5 Missing :warning:

flax/experimental/nnx/__init__.py
0.00%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #3928       +/-   ##
==========================================
- Coverage   60.43%   0.00%   -60.44%
==========================================
  Files         105     101        -4
  Lines       13263   13051      -212
==========================================
- Hits         8015       0     -8015
- Misses       5248   13051     +7803

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

@NeilGirdhar looks great! Thanks for doing this.
Approved but left a comment.

Seems there are some git conflicts.

Rebased to main.


hooks for starting web server only to get auth/cert

Assume you want to create certificates for some other service, like jabber server. web server is turned off by default there and needs to be started only for letsencrypt auth.
So some hooks for that would be nice. Right now I'm doing it in a bit hacky way:
case "$1" in
deploy_cert)
[...]
        cat "$FULLCHAINCERT" "$PRIVKEY" > /etc/openssl/jabber.example.com.pem
        /sbin/service ejabberd reload
       /sbin/service lighttpd stop
        ;;
clean_challenge)
        /sbin/service lighttpd stop
        ;;
deploy_challenge)
        /sbin/service lighttpd start
        ;;

clean_challenge is not always called AFAIK and deploy_cert is called even if certificate did not change, so these hooks are not best for such job.

I'm making preparations to do something very similar, but with nginx instead.
According to the documentation in docs/examples/hook.sh, or at least what I make out of it, the clean_challenge hook only gets called when there is something to clean. This can explain what you are experiencing when you say that it is not always called.
For deploy_cert my understanding is that it only gets called when a new certificate has been produced/retrieved. If deploy_cert always gets called for you, then that sounds like either a bug inside the letsencrypt.sh script itself (or an error in the documentation), or something is wrong in your setup, although the snippet you pasted looks good.
For me, in another setup that I have (also an nginx webserver but one that is always running and needs certificates for itself) the deploy_cert hook only gets called as described in the documentation (when a new certificate has been produced/retrieved).
Anyhow, as I said, I'm making preparations (writing a hook script) to do something very similar, and as it is not ready yet I have yet to test it out against the LE staging server. What I'm going to do is start the nginx server in deploy_challenge, and then in deploy_cert I'm stopping the nginx server and copying the new certificates to where they need to be. Provided the documentation is correct (and my understanding of it is also correct) I should have no problems with this, and also not need any of the other hooks, only the two hooks I mentioned.

I.e. there is need in two hooks:

Before any update operation (to start webserver)
After any update operation (to stop webserver)
Last hook can be used for webserver reload in common HTTPS tasks (as requested in #316).

It would be great to have one hook running directly after starting dehydrated to do some initialization (e.g. starting the web server or enabling firewall rules on systems which do not allow incoming connections on port 80 or outgoing connections on port 80 and 443). Additionally one hook should be run just before exiting dehydrated to clean everything up (regardless of new certificates being deployed or an error occured).

So far, I haven't seen any use case outlined here where the existing hooks would not suffice.

deploy_challenge works for starting a web daemon and/or altering firewall rules.
clean_challenge works for killing the web daemon and/or re-altering firewall rules.
deploy_cert works for writing out the cert (in whatever format is required) and reloading whatever daemon needs to know about the new cert.

If you expect to be doing a lot of challenges and don't want to be continually starting and stopping the daemon, it seems trivial to write a script which does the needful before and after, and simply calls dehydrated in the middle.

I agree that deploy_cert is sufficient for restarting the daemons. However setting up firewall rules in deploy_ and clean_challenge is not sufficient for systems which normally do not allow outgoing traffic on port 80 and 443.
Having a generic initialization and cleanup hook would solve this problem and might be helpful in other use cases.
Of course using a wrapper script calling dehydrated in the middle is possible, but this feels like a workaround for missing hooks.

The same hooks are requested in #331

I don't know what exactly you want, the deploy_challenge and clean_challenge hooks would be the ideal place for starting and stopping a web-server as they are really only called when there is work to do and they are always called in that case, only exception is on errors in the communication with Let's Encrypt, and there is a different hook for that (see request_failure).
What you could also do to maybe spare some start/stop cycles is to just start the server in the deploy_challenge hook if it's not running yet, and just always stop it after dehydrated did its job...
And just to make this clear: For me using a wrapper script is not a workaround, it's more or less the intended usage of this tool, that is one of the reasons why there are so many options on this script, hooks should only be for things that can't easily be solved in a different (clean) way.


[VS Code] Joining collaboration session: The specified value isn’t a valid Live Share URL. Please check the link provided by the host and try again.

%0A%0A%23%23 Error:%0AJoining collaboration session: The specified value isn’t a valid Live Share URL. Please check the link provided by the host and try again.%0A%0A%23%23 Steps to Reproduce:%0A1.%0A2.%0A%0A||Version Data|%0A|-:|:-|%0A|**extensionName**|VSLS|%0A|**extensionVersion**|1.0.337|%0A|**protocolVersion**|2.2|%0A|**applicationName**|VSCode|%0A|**applicationVersion**|1.35.0|%0A|**platformName**|MacOS|%0A|**platformVersion**|18.6.0|

Hi @thoonly, thanks for the feedback! Do you remember what was the link you've used? What channel/app did you use to transfer it over the network?
Thanks!

Sorry i just copy invalid URL , it's a Slack problem when i use copy link

@thoonly, no worries, thanks for letting us know!


Add support for appium to local Grid implementation

This enhancement is currently in progress in pull requests #121


trying to understand  multibox

can you please explaned the multibox function,
i dont understand why we have
vgg_source = [24, -2],
what happens when v=-1
def multibox(vgg, extra_layers, cfg, num_classes):
    loc_layers = []
    conf_layers = []
    vgg_source = [24, -2]
    for k, v in enumerate(vgg_source):
        loc_layers += [nn.Conv2d(vgg[v].out_channels,
                                 cfg[k] * 4, kernel_size=3, padding=1)]
        conf_layers += [nn.Conv2d(vgg[v].out_channels,
                                  cfg[k] * num_classes, kernel_size=3, padding=1)]
    for k, v in enumerate(extra_layers[1::2], 2):
        loc_layers += [nn.Conv2d(v.out_channels, cfg[k]
                                 * 4, kernel_size=3, padding=1)]
        conf_layers += [nn.Conv2d(v.out_channels, cfg[k]
                                  * num_classes, kernel_size=3, padding=1)]
return vgg, extra_layers, (loc_layers, conf_layers)

Because we adopt the feature map before the relu layer by default for detection


Monitoring for metadata expiration?

Question
Do we have monitors/pagers in place to make sure that root/timestamp/snapshot/targets metadata are renewed before actually expiring in production? Otherwise, we risk accidentally running into the old "forgot to renew TLS cert in prod..."

We should! Even a slack app alert would help. WDYT?

Perhaps a GitHub Action which parses metadata, checks expiry, and files an Issue when the expiration is close?

I LOVE THAT JOSHUA! Assigning, love it love it.


Hotfix/issue259 refreshtoken not reused

Summary

Fix https://github.com/interTwin-eu/interLink/issues/259 and some debug log added

Related issue :
#259

I'll run integration tests locally, since we are experiencing false negative through gh actions. Once checked I'll merge this

For info, this is the new debug info, plus the response info in case of error, because it is a good info to know it fails because of expiration:
INFO:__main__:Verbose mode: setting log to debug.
INFO:__main__:No cache for refresh token, starting from ENV value
INFO:__main__:Current refresh token: REDACTED
DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): github.com:443
DEBUG:urllib3.connectionpool:https://github.com:443 "POST /login/oauth/access_token HTTP/11" 200 None
DEBUG:__main__:{'error': 'bad_refresh_token', 'error_description': 'The refresh token passed is incorrect or expired.', 'error_uri': 'https://docs.github.com/apps/managing-oauth-apps/troubleshooting-oauth-app-access-token-request-errors/#bad-verification-code'}
DEBUG:__main__:iam_client_id REDACTED iam_client_secret REDACTED response {'error': 'bad_refresh_token', 'error_description': 'The refresh token passed is incorrect or expired.', 'error_uri': 'https://docs.github.com/apps/managing-oauth-apps/troubleshooting-oauth-app-access-token-request-errors/#bad-verification-code'}
WARNING:__main__:ERROR oidc get token: 'access_token'
Traceback (most recent call last):
  File "/opt/refresh.py", line 124, in <module>
    token = response['access_token']
KeyError: 'access_token'
WARNING:__main__:Response if available: {'error': 'bad_refresh_token', 'error_description': 'The refresh token passed is incorrect or expired.', 'error_uri': 'https://docs.github.com/apps/managing-oauth-apps/troubleshooting-oauth-app-access-token-request-errors/#bad-verification-code'}

I confirm, this is good to go


columns in select result set is in different order on different environment

i'm trying to generate formatted string from query result. however query result is not same on my local and remote environment. in my condition, i cannot use column names so order of the columns is important.
select query
$this->db->select('activity_elevation',['elevation','time'],[ 'activity_id'=>$activityId]);
result on local
array ( 'time' => '3', 'elevation' => '98.21', )
result on remote
array ( 'elevation' => '98.21', 'time' => '3', )

And the remote and local database has the same exact table layout I assume?

there is only remote database. i connect remote database from my local


Add optional message parameter to 'Wait For Elements State'

I really treasure the fact that SL keywords like Element should be visible have an optional message parameter to override the default error message coming from the keyword itself.
This helps non-tech people to understand better what really happened.
(Pseudo) example:
ReportCheck 5AAFS                                                     | PASS |
------------------------------------------------------------------------------
ReportCheck 5BB11                                                     | FAIL |
Property innerText `-` should match `\d+(\.\d+)?\s.C`
------------------------------------------------------------------------------

vs.
ReportCheck 5AAFS                                                     | PASS |
------------------------------------------------------------------------------
ReportCheck 5BB11                                                     | FAIL |
Error: The report table could not be found. See KB-300-11 for a possible solution.
------------------------------------------------------------------------------

It would be great if the Wait For Elements State keyword in BL would also have an optional message parameter.
Thanks :-)
Simon

I agree, our error messages could be improved. Also in other ways than this. But in any case, providing way to provide custom error message is is useful.

A suggestion: The custom message could be a pattern also. It could contain ${selector}, ${state} and ${timeout}..

The pattern approach could be a good default which makes the default message more informative for tech people.
It then could be overridden with a custom message to make it lot easier to localize problems.
Or even more, make the default pattern accessible by a placeholder variable within the custom message ?

Also see #595


[fb-package] Report initial set of contingency tables

Description
The set of expanded aggregates doesn't include all tables originally issued -- see the first ~160 lines; aggs that have been commented out are already covered in the new set. We want to continue reporting these, so add them back in, plus some supporting features.
Changelog

Report contingency tables from initial data issue not already reported.
Switch numeric compute function to one that reports standard errors.
Create function to create aggregates from all combinations of input groups, regions, and indicators.

@davidkretch I can't seem to officially request a review from you, but I'd appreciate if you could take a look at this and leave any comments/suggestions.

No problem! Will get to it later today.

Looks good to me and works for me too. I like the new create_aggs_product. I don't have any other comments. 👍

Thanks @davidkretch!


get count from relatedQuery |  question

Is there any simpler option how to get count number from relatedQuery?
Currently I am doing this
    const followersCount = await user
      .$relatedQuery('followers')
      .count('*')
      .pluck('count')
      .first();

I have tried next code but it is not working without '*'. With '*' it will returns array [{ count: '22' }].
const followersCount = await user
      .$relatedQuery('followers')
      .count();

Unfortunately there's no easier way to do that at the moment. count('whatEver') simply adds a SELECT COUNT(whatEver) as count to the query.

I saw one solution for this case in another library. They have function scalar() in query builder. It will take first item from array and first item in the object.
You are able to change next code:
const followersCount = await user
      .$relatedQuery('followers')
      .count('*')
      .pluck('count')
      .first();

with
const followersCount = await user
      .$relatedQuery('followers')
      .count('*')
      .scalar();

I am not sure if you are intristed. Maybe it can be nice addon to current API

@koskimas @seeden That scalar method seems to be very useful!

@andrepcg  yes it is. The strange thing is that you need to write different code for static and instance method.
static example:
await User
    .query()
    .count('*')
    .pluck('count')
    .first()

instance example (without first):
const res = await this
      .$query()
      .where((builder) => {
        builder
          .where('value', '>', value)
          .where({ type });
      })
      .count('*');

 return res.count;


Missing "New" button in the left side menu on the Home page

It's visible without refined-github, but gone after enabling the extension - I'm not sure if this is deliberate.
No plugin:

Yes plugin:

Yes this is deliberate to clean up the interface. There’s a button in the top right next to the notification button

Ah, true. Thanks for the clarification :)


Allow client to parameterize marionette format

The marionette client does not accept an input parameter to specify the marionette format. We should add a few tests in marionette.cli_tests that tests marionette under a few different formats.

Required substantial amount of underlying working: e.g., added twisted as a framework from the client/server logic, and fixed a number of bugs such that the marionette.cli_tests pass.


typo in code of pyglet window example?

Description
I haven't run it, but on inspection of the pyglet window example, at this line:
https://github.com/cprogrammer1994/ModernGL/blob/master/examples/window/pyglet/window.py#L37
I think the assignment of config.minor_version contain a bug:
config.major_version = self.gl_version[0]
config.minor_version = self.gl_version[0] # should be 1, not 0

Possibly undetected because the default glversion is (3, 3)

Wow. Thank you for pointing this out. Fixing right now.

Fixed. Merged. Close 😄

Thank you for the bug report, and Thank you for the fix :)

This inspired #285 causing a lot of other fixes as well.


Allow to turn off stacktrace bookkeeping

Hi,
The stack = traceback.extract_stack()[:-1] instruction makes inserts too slow for my use case.
This PR adds a flag to disable this behavior if needed.
According to some basic testing, it speeds up inserts by ~20x:
Test code:
from sqlitedict import SqliteDict

def insert(outer_stack):
    d = SqliteDict(outer_stack=outer_stack)
    for i in range(10000):
        d["key_{}".format(i)] = "value_{}".format(i)
    d.close()

%%timeit
insert(outer_stack=True)

=> 2.68 s ± 42.8 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)
%%timeit
insert(outer_stack=False)

=> 121 ms ± 3.04 ms per loop (mean ± std. dev. of 7 runs, 10 loops each)

makes inserts too slow for my use case

Can you post some benchmarks? before / after your PR.
Intuitively, I'd expect debugging features that significantly slow down normal execution to be an opt-in, not opt-out. But if the difference is "slight", it's OK as opt-out for power-users. So the benchmark numbers matter. CC @jquast .

@mpenkov Thanks for the review - Amended the commit based on your comments, PTAL

@piskvorky Although I cannot share my code (not open source), I observed a similar improvement as shown above (about ~23x faster after than before, although I didn't compute the variance in the real use case, as opposed to above).
FWIW I am evaluating this map compared to a leveldb index and sqlitedict is still ~4x slower than leveldb in my use case.

Yeah, that wouldn't surprise me. Sqlitedict is aiming to be a simple-to-use wrapper for SQLite, speed is not one of our primary objectives.

@piskvorky LMK if you think I should make this the default behavior

@yonromai I'm not very familiar with this part of the code. Can you summarize the pros/cons?
What would we (the users, the developers) gain / lose by making it the default?

Pro: Inserts are 23x faster.
Con: If an error occurs, the stack trace will mostly get swallowed, and 'No outer stack to display. Enable it using outer_stack=True' will be displayed in place of the stacktrace.

I'm not sure. Leaning slightly toward more intelligible messages… but damn, 23x is a lot! @mpenkov WDYT?

'No outer stack to display. Enable it using outer_stack=True' will be displayed

That message is too vague: where should I (as a user) enter this outer_stack=True? Can you please make it more specific and actionable.

@yonromai why did you close this  PR?

@mpenkov I closed the PR because it's been stale and I'm trying to keep my github.com/pulls clean since I use it a lot.
I kept the fork alive in case you want to cherry pick the changes.

Are you able to complete the PR? You received feedback from @piskvorky. From what I can see:

Document use case (most importantly, source of exceptions)
Improve error message

@mpenkov Sorry but I don't have much more bandwidth to dedicate to this (we decided to not use sqlitedict).

OK, that's fair enough. Thanks for the changes in your PR. We'll take it from here.

Has there been any progress on this PR? traceback.extract_stack() stat(2)'s all Python source files in the traceback--making more syscalls than sqlite itself. No wonder this is so slow.

@imachug No. Are you interested in fixing this?

I thought this PR is more or less ready to merge, except maybe

That message is too vague: where should I (as a user) enter this outer_stack=True? Can you please make it more specific and actionable.

Do you have any other issues with this?

No, TBH, I think that's the only thing left. Let me go clean up the error message and we can merge this.


iOS: add support for key commands

iOS 8 & 9 can access functions and navigate via key commands using an attached hardware keyboard, and iOS 9 can display a bezel to list those shortcuts when the Cmd key is held down.
That would be very useful to support.  Imagine using arrow keys to navigate around the feeds and stories, Cmd-F to go to the Find field, Cmd-N to add a new feed, and more.

Yep, @nriley brought this up yesterday. Let him work on it, he's happy to contribute.

Making some progress.  The biggest issues I've had thus far are:
(a) somehow the view controller isn't getting keyboard focus and I have to tap on the story before I can use the keyboard commands (right now I have it attached to StoryPageControl).
(b) I'm not sure if I'm triggering all of the read/unread machinery correctly as it's distributed among a
number of classes and is pretty hard to follow.  I can either try to dig some more myself, or I can put my code up for review and see if you have any ideas about how I can rewrite it so it's clearer what's going on.

I fixed the keyboard focus issue (had forgotten about the existence of UIResponder).  There may be some places where the responder isn't getting set correctly now, but at least this is a start.

I realize it seems the terminology is inconsistent between the iOS and Web app.  iOS calls the RSS-feed-text "Story" (in both code and the UI) whereas the Web calls it "Feed"; there is no version of the Web's "Story" on iOS.  Confusing but if nobody's complained, maybe nobody cares? :-)

Anyway, think this is good enough for a first shot; please report bugs!

Things seem pretty reliable now and I can do most of my newsreading entirely from the keyboard.  (Note that I read almost exclusively from "All Stories").  I've seen some visual artifacts and multiple copies of the same story, but I don't think my code is responsible, although I still don't entirely understand the changes I needed to make to unread handling.  There definitely seem to be some races where the unread indicator in the story list and in the header are inconsistent, but again…

I'm now covering essentially anything I'd ever do with NewsBlur, so I'm going to stop now :-) If nothing else it makes navigating in the simulator a lot faster! I can squash my commits into a single one if you'd prefer.

Now committed to the 5.1 branch.  I'm not aware of any remaining issues, but please feel free to report.


feat(stepper): add input for controlling the animation duration

Adds a new input called animationDuration that allows users to control the animation duration of a stepper, similarly to other components.
Fixes #17130.

I think this PR is duplicated from one I sent before (#17131). Maybe you can merge the one I mentioned and adding the examples as is the only difference I find. What do you think?

We probably ended up working on it at the same time. Aside from the example, these changes are mostly just boilerplate. I'm not sure it makes sense to be merging it in two stages.

I just reviewed #17131, that one doesn't have examples and this one doesn't have tests. @ivandevp or @crisbeto, does one of you want to make a PR that includes both?

I don't mind adding tests, but there isn't that much there to test. We're just passing an input along to @angular/animations.

The tests I added was to confirm the default values for the input is working the same as before this new input.

Added a test for the logic that adds units if none are set. I don't think it makes sense to be testing default values.

Is there any plan to merge this soon?

I'm getting errors on a few tests that looks like this:
@stepTransition has failed due to:
- Please provide a value for the animation param animationDuration
- The provided timing value " cubic-bezier(0.35, 0, 0.25, 1)" is invalid.

It appears the duration is misisng, but looking at the code in this PR, I don't know how that would happen. Will need to investigate further


Quick fixes to the plugin process docs

When we were going through the plugin process (Christopher, Lev and I) we thought these might be some easy fixes to make the process a bit clearer for external folks.
Completely open to suggestions/rewording/improvements.

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

@cpoile This one might be ready to merge?

@jasonblais Does it look good now?


PyPi repo has old version

Old version when users pip install from Pypi
Would be useful if the version was bumped to 0.8.1 to upload the previous two merges to handle USBank codes, and an empty line in the BAI file. Looks like neither of the previous merges updated the version, and the Pypi code is out of date.

Have released 0.8.1 with the latest additional codes.