Should we rename Nancy.Owin project?

As the title suggests. Currently this project won't compile on dotnet54 due to the Owin lib not being dotnet54 compatible.  For using Nancy in dnxcore50 apps the usage will be app.UseOwin(x => x.UseNancy()); which uses the MidFunc extension in Nancy not the IAppBuilder extension in Nancy.Owin, IAppBuilder being something inside the Owin lib.
Should we rename it and/or make it clear that using the Nancy.Owin IAppBuilder extensions won't work for dnxcore50 apps?
Thoughts? @damianh

Glossary

dnxcore50 - An application that runs on CoreCLR
dotnet54 - A library that is targeting the CoreCLR

I'd wouldn't be angry if it was called Nancy.MSOwin. @grumpydev may have a
problem with breaking package name though...
On 26 February 2016 at 20:54, Jonathan Channon<EMAIL_ADDRESS>wrote:

As the title suggests. Currently this project won't compile on dotnet54
due to the Owin lib not being dotnet54 compatible. For using Nancy in
dnxcore50 apps the usage will be app.UseOwin(x => x.UseNancy()); which
uses the MidFunc extension in Nancy not the IAppBuilder extension in
Nancy.Owin, IAppBuilder being something inside the Owin lib.
Should we rename it and/or make it clear that using the Nancy.Owin
IAppBuilder extensions won't work for dnxcore50 apps?
Thoughts? @damianh https://github.com/damianh
Glossary
dnxcore50 - An application that runs on CoreCLR
dotnet54 - A library that is targeting the CoreCLR
—
Reply to this email directly or view it on GitHub
https://github.com/NancyFx/Nancy/issues/2322.

Its not MS though is it? IAppBuilder is in Owin.dll not anything MS related?
On 28 February 2016 at 17:37, Damian Hickey<EMAIL_ADDRESS>wrote:

I'd wouldn't be angry if it was called Nancy.MSOwin. @grumpydev may have a
problem with breaking package name though...
On 26 February 2016 at 20:54, Jonathan Channon<EMAIL_ADDRESS>wrote:

As the title suggests. Currently this project won't compile on dotnet54
due to the Owin lib not being dotnet54 compatible. For using Nancy in
dnxcore50 apps the usage will be app.UseOwin(x => x.UseNancy()); which
uses the MidFunc extension in Nancy not the IAppBuilder extension in
Nancy.Owin, IAppBuilder being something inside the Owin lib.
Should we rename it and/or make it clear that using the Nancy.Owin
IAppBuilder extensions won't work for dnxcore50 apps?
Thoughts? @damianh https://github.com/damianh
Glossary
dnxcore50 - An application that runs on CoreCLR
dotnet54 - A library that is targeting the CoreCLR
—
Reply to this email directly or view it on GitHub
https://github.com/NancyFx/Nancy/issues/2322.

—
Reply to this email directly or view it on GitHub
https://github.com/NancyFx/Nancy/issues/2322#issuecomment-189912537.

OWIN.dll is MS (Katana) related :smile:

I'd prefer leaving the name as is currently. The owin dll is Katana related, but not owned or maintained by MS.

If we're going to do this we should put out a new version of the current one with an obsolete attribute pointing to the new library - despite it being uploaded by "owincontrib", owin.dll is very much Microsoft, they just don't maintain it.. they just steamrollered all over the rest of the owin group then rode out of town ;)
We'd have to make it obvious though, not sure MSOwin does that, maybe something like:
Nancy.Owin.IAppBuilder
Nancy.Owin.AppBuilderExtensions
Nancy.Owin.Katana

Nancy.Microsoft.SteamRollered.Owin

I think the Katana suffix
On 2 March 2016 at 07:35, Phillip Haydon<EMAIL_ADDRESS>wrote:

Nancy.Microsoft.SteamRollered.Owin
—
Reply to this email directly or view it on GitHub
https://github.com/NancyFx/Nancy/issues/2322#issuecomment-191111935.

Nancy.Owin.Katana seems the best of the options.

The Owin and Owin.Builder seem rather basic and not linked to anything Apart from the Owin spec itself. Its the Microsoft.Owin lib that is ms flavoured, no? Is there another library that is closer to the Owin spec? I doubt it.

@MeirionHughes Where can you find IAppBuilder in the spec? :wink:


Crashing with Thread "clock_main" aborts with exception Operation not permitted in sleep()!

I am running the librespot entry point on DietPi and getting an error every time I attempt to stream to the device.  I have not been able to isolate to theentry point code or librespot yet but hoping this community has some insight as the entrypoint community is super small, <300.  Here is the full error output:
2022-11-16T17:59:03.193475000Z -> CONTINUE 1 Changed {"message":"Streaming from Spotify Connect input"}
2022-11-16T17:59:03.195545000Z liquid-stdout: [clock.main:3] Streaming loop starts in auto-sync mode
2022-11-16T17:59:03.196464000Z liquid-stdout: [clock.main:3] Delegating synchronisation to CPU clock
2022-11-16T17:59:03.197417000Z liquid-stdout: [threads:3] Thread "clock_main" aborts with exception Operation not permitted in sleep()!
2022-11-16T17:59:03.197866000Z liquid-stdout: [main:3] Shutdown started!
2022-11-16T17:59:03.198222000Z liquid-stdout: [main:3] Waiting for main threads to terminate...
2022-11-16T17:59:03.198528000Z liquid-stdout: [main:3] Main threads terminated.
2022-11-16T17:59:03.199283000Z liquid-stdout: [threads:3] Shutting down scheduler...
2022-11-16T17:59:03.200507000Z liquid-stderr: Thread 4 killed on uncaught exception Unix.Unix_error(30, "sleep", "")
2022-11-16T17:59:03.201001000Z Called from Clock.sleep in file "clock.ml", line 43, characters 6-13
2022-11-16T17:59:03.201289000Z Called from Clock.clock#run.loop in file "clock.ml", line 245, characters 38-47
2022-11-16T17:59:03.201539000Z Called from Clock.clock#run in file "clock.ml", line 269, characters 6-13
2022-11-16T17:59:03.201779000Z Called from Tutils.create.(fun).process in file "tools/tutils.ml", line 159, characters 12-15
2022-11-16T17:59:03.202009000Z Re-raised at Tutils.create.(fun).process in file "tools/tutils.ml", line 197, characters 14-52
2022-11-16T17:59:03.202237000Z Called from Thread.create.(fun) in file "thread.ml", line 41, characters 8-14

No idea but none of this output is from librespot. I think you will have to file a report with DietPi. Since you say that community is really small, and are hoping to find some answers here, I am converting this into a discussion.


[Service Bus] Detect lost connection to service bus

Is your feature request related to a problem? Please describe.
I request a way (or advise on how) to detect if the connection to the service bus is lost.
Currently, after a successful connection there is no easy way to detect that the connection is broken.
Consider this snippet:
const { ServiceBusClient } = require('@azure/service-bus')

const client = ServiceBusClient.createFromConnectionString(process.env.CONN_STR)

const [, ,topic, interval] = process.argv;

const sender = client.createTopicClient(topic).createSender()

let count = 0

setInterval(async () => {
  const timestamp = Date.now()
  const messageNumber = count++

  try {
    console.log("Send: ", messageNumber, timestamp)
    await sender.send({ body: { timestamp, messageNumber } })
    console.log("Sent: ", messageNumber, timestamp, `${(Date.now() - timestamp)/1000}s`)
  } catch(error) {
    console.log("Error: ", messageNumber, timestamp, `${(Date.now() - timestamp)/1000}s`)
    console.error(error.message)
  }
}, interval)

Run like this:
CONN_STR=<instert> node index.js some-topic 1000

Provided that the connection was successful, the topic exists and the connection string gives us sufficient rights. This will start sending a message to some-topic every second:
Send:  0<PHONE_NUMBER>379
Sent:  0<PHONE_NUMBER>379 0.403s
Send:  1<PHONE_NUMBER>379
Sent:  1<PHONE_NUMBER>379 0.018s
Send:  2<PHONE_NUMBER>382
Sent:  2<PHONE_NUMBER>382 0.018s
Send:  3<PHONE_NUMBER>383
Sent:  3<PHONE_NUMBER>383 0.018s

Then, pull your internet connection, for example by disabling the WIFI. After this, it will simply attempt to send message unsuccessfully for some time:
Send:  13<PHONE_NUMBER>392
Send:  14<PHONE_NUMBER>393
Send:  15<PHONE_NUMBER>393
Send:  16<PHONE_NUMBER>393
Send:  17<PHONE_NUMBER>393
Send:  18<PHONE_NUMBER>393

Wait for ~3 minutes, and it will start outputting timeout errors for the messages:
Error:  13<PHONE_NUMBER>392 198.007s
[connection-1] Sender "lenkan-lenkpad-heartbeat-fe08909a-216b-7b48-8dd5-d40d7985bbf2" with address "lenkan-lenkpad-heartbeat", was not able to send the message right now, due to operation timeout.

Describe the solution you'd like
I expected that the message sending would simply fail instantly or at least much quicker than it did. What is the recommended approach here? Is the operation timeout configurable?
Also, if an application that is using a ServiceBusClient is not itself initiating any messages, it would be completely oblivious to the lost connection. Because there is no way to detect this without sending a message and wait for ~3 minutes.
I really like the minimal and clean interface that @azure/service-bus is providing,  but I would like to be able to gain more insight into what happens in error cases.

Thanks for this feedback. I have routed to the appropriate team for follow-up.

@maggiepint, would you please provide any information as to why you think the service-side code would need to change to support what @lenkan is describing. It looks like a SDK-only concern.

My apologies @lenkan and @jfggdl, this issue is indeed with the client and was wrongly classified as service related.
When there is a connection issue, we try to recover the connection behind the scenes for quite a while before reporting the problem. While this is preferable on the receiving side (in case of the streaming receiver), I see that it is not ideal when sending.
The promise returned by send() method should be rejected in such cases.
We will look into this and get back with potential options.

@chradek Doing what we did in #6564 may not help with the 3 minute delay here.
My guess for the cause for the 3 min delay is the retries for the send operation itself that we have in place. There are a total of 3 attempts, with each having a timeout of 60 seconds, resulting in the 3 min delay in the error bubbling up to the user.
But, the above is true only for the send() operations that were already fired by the time the network was disconnected. The send() operations that follow the network disconnection should actually report back an error in 60 seconds as the init() would timeout and we don't have retries around this.

@richardpark-msft, @chradek,

The send() operations that follow the network disconnection should actually report back an error in 60 seconds as the init() would timeout and we don't have retries around this.

I found the reason for the above behavior. After we disconnect the network manually,

The isOpen() on the sender link returns true until rhea disconnects the connection object. This takes about a minute from the time of us removing the network connection.
- Need to check why rhea's isOpen() doesnt return false here
- Need to check why rhea takes a minute to fire the disconnect event
So, for the first minute, each new send() operation is getting queued up thinking the link is just fine. Each of these will go through the 3 retries, with 60 sec timeout on each attempt and 15 sec delay between attempts. Therefore, the final timeout error bubbles up to the user after about 200 seconds. This is what @lenkan is seeing

But, I then found another disturbing issue where the send() operations after the first minute will only have their errors bubbled up after 3 hours

When rhea fires disconnect event on the connection, we go around closing open links. So, the isOpen() on the sender link no longer returns true, resulting in subsequent send() operations trying to take a lock in order to initialize the sender link.
But, the disconnect event on the connection also results in us trying to bring up the sender link ourselves, so we take a lock for over 3 hours!! (150 retries with 60 second timeout on each attempt and 15 second delay between attempts)
This results in the send() operations fired after the first minute of network going down bubbling the error up only after 3 hours

The problem described in the previous comment on the the errors for send operations being bubbled up only after 3 hours in a worse case scenario is now fixed in the version 1.1.6 that we just released for @azure/service-bus
The original issue that was reported on having to wait for 3 minutes to have the error reported is expected behavior. We have retries set up with a total of 3 attempts, each having a timeout of 60 seconds which results in the 3 minute wait time.
In the next major version update for the @azure/service-bus library, we will allow the user to customize the retry policy where you can tweak the number of attempts, timeout value and delay between attempts.
Thanks for reporting @lenkan!


Bump wgpu version to 23.0.

wgpu v 23.0.0 is out.
entry_points are now Optional
https://github.com/gfx-rs/wgpu/releases/tag/v23.0.0

LGTM


Ruby Kickstart exercises

@lucetzer check your gem file - there's an error that's stopping the CI from passing

Thanks Nikesh. I have updated that file but still trying to figure out the Rakefile syntax error. Not sure what is wrong with it.


Configured debug type 'chrome' is not supported.

VSCode Version: 1.4.0
OS Version: fedora21

Steps to Reproduce:

clone vscode repository & ./scripts/npm.sh install --arch=x64
./scripts/code.sh --remote-debugging-port=9222 &  DEBUG from Attach to VSCode

@Luncher you have to install the chrome debug extension:  https://marketplace.visualstudio.com/items?itemName=msjsdiag.debugger-for-chrome if you want to reuse that launch config.


parse grounded operation is badly implemented

The code is
parseAtom = OperationAtom('parse', lambda s: [ValueAtom(SExprParser(str(s)[1:-1]).parse(Tokenizer()))],
                              ['String', 'Atom'], unwrap=False)

Apparently, the result of parse should not be wrapped into ValueAtom. Also, instead of a default tokenizer, the actual tokenizer from the runner might be more preferable. It would also be nice to add unit tests in test_stdlib.py

Obviously, this test expects the result of parsing to be wrapped into ValueAtom, which is just wrong, so these unit tests should be fixes together with fixing implementation of parse. As for unit tests, we can keep them in test_text_ops (no need to move them to test_stdlib). But I'd add more tests for parsing, e.g. including grounded atoms (e.g., parsing (A 2 "S")), nested expressions, maybe something additionally

fixes together with fixing implementation of parse

Yes, my bad. It is done.

Complete by #658


Updating Go and distroless images to latest while we are here.

Also removing inconsistent "ARGs" for images.
To my knowledge we never override/specify them (checked in our louhi configs):
~/Repos/gpe-configs
bwplotka@bwplotkadev:~/Repos/gpe-configs$ rgrep "IMAGE_BUILD_GO" .louhi
bwplotka@bwplotkadev:~/Repos/gpe-configs$ rgrep "IMAGE_BASE" .louhi

I will go ahead and release.


Change the readme author to right owner and service to the right service name

https://github.com/MicrosoftDocs/azure-docs-sdk-node/pull/1067/files#r574784578
What we use currently:
Hard-coded Author: Maggie
Hardcoded Service: Azure
Proposed changes:
Author: the one who run the pipeline
Service: package name

Thanks for this info! For this usecase I think what we are looking for is the idea of a primary owner... like if a nightly build fails, it's fine to send notifications to everyone listed as a codeowner, but when an issue is created for it and there are 5 owners (some of whom are just generic catch-all like Ramya and Deya) who should get assigned to the issue?
Maybe there's no great way to use CODEOWNERS for this, it's just that it's enticing because it's already there and is contiguously updated.
Do you or @weshaggard have any suggestions? I'd love to hear about them! If there isn't that's fine too - we have a separate list that I maintain for them, it's just the duplication is not ideal...
The need I'm trying to resolve is: is there a source vendors can lookup the right person to assign a nightly build failure to instead of a duplicated list? Is there a way to use CODEOWNERS for this (if we organize it so that the primary owner is listed first)?

I don't like maintaining a separate list of primary owners. I would much prefer we update our codeowners docs and just use the convention of the first one is the primary owner.

Currently, we have already written the github code owner into metadata 'author' using 'retrieve-codeowners' tool.
For ms.author, we plan to use opensource API for github<->msalias

Proposal:

Having two scripts:
generate-aadToken.ps1: The script is to generate token using necessary attributes.
github-ms-converter.ps1: The script takes in the add token + github name, returns the msAlias (userPrincipleEmail) which is calling opensource Rest API.

Have a helper class
MsNameResolver.cs: Input and output are same as script but using c# for implementation. The class serves as helper class.


Add support for creating repos from template repos

Add API for POST /repos/:template_owner/:template_repo/generate endpoint which creates a repository from a template repository.
Developer Blog: https://developer.github.com/changes/2019-07-16-repository-templates-api/
API Reference: https://developer.github.com/v3/repos/#create-repository-using-a-repository-template
Issue: https://github.com/google/go-github/issues/1235

I signed it!

Note: Creating and using repository templates is currently available for developers to preview. To access this new endpoint during the preview period, you must provide a custom media type in the Accept header:
application/vnd.github.baptiste-preview+json

And

Warning: The API may change without advance notice during the preview period. Preview features are not supported for production use. If you experience any issues, contact GitHub Support.

I will add the custom headers to the other two endpoints later today.

Alright, I manually tested all of the endpoints. It's ready for a second review. Thanks @gmlewis

Thanks @gauntface and @gmlewis. I've squashed the commits

Thank you, @shanempope, but we prefer that you don't use force-push in this repo, as it makes it more difficult for other contributors to see the change history in case they wish to make comments during the PR review period.  Additionally, we always "Squash and merge" in this repo, so it ends up being a single commit anyway, so the force-push is unnecessary.
Thank you, @gauntface.
Merging.


Add support for CFPreferences

Please can we have support for CFPreferences
Thanks.

We're working on it, stay tuned.


python310Packages.yangson: 1.4.10 -> 1.4.12

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.yangson is: "Library for working with data modelled in YANG"
meta.homepage for python310Packages.yangson is: "https://github.com/CZ-NIC/yangson"
meta.changelog for python310Packages.yangson is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
Warning: no invocation of /nix/store/jn75hdjxk8kwawrplrbx0iqka35ngckr-python3.10-yangson-1.4.12/bin/yangson had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/jn75hdjxk8kwawrplrbx0iqka35ngckr-python3.10-yangson-1.4.12/bin/.yangson-wrapped had a zero exit code or showed the expected version
0 of 2 passed binary check by having a zero exit code.
0 of 2 passed binary check by having the new version present in output.
found 1.4.12 with grep in /nix/store/jn75hdjxk8kwawrplrbx0iqka35ngckr-python3.10-yangson-1.4.12
found 1.4.12 in filename of file in /nix/store/jn75hdjxk8kwawrplrbx0iqka35ngckr-python3.10-yangson-1.4.12

Rebuild report (if merged into master) (click to expand)

30 total rebuild path(s)

15 package rebuild(s)

15 x86_64-linux rebuild(s)
15 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
ansible
ansible-lint
ansible_2_10
ansible_2_11
python310Packages.ansible-base
python310Packages.ansible-core
python310Packages.ansible-lint
python310Packages.ttp
python310Packages.yangson
python39Packages.ansible-base
python39Packages.ansible-core
python39Packages.ansible-lint
python39Packages.ttp
python39Packages.yangson
ttp

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/jn75hdjxk8kwawrplrbx0iqka35ngckr-python3.10-yangson-1.4.12 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.yangson https://github.com/r-ryantm/nixpkgs/archive/6eb51e8205cd8fc68370173578062f21b4a2d421.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/jn75hdjxk8kwawrplrbx0iqka35ngckr-python3.10-yangson-1.4.12
ls -la /nix/store/jn75hdjxk8kwawrplrbx0iqka35ngckr-python3.10-yangson-1.4.12/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

4 packages failed to build:

python310Packages.ansible-base
python310Packages.ansible-core
python310Packages.ansible-lint
python310Packages.ttp

7 packages built:

ansible (python39Packages.ansible-core)
ansible-lint (python39Packages.ansible-lint)
ansible_2_10 (python39Packages.ansible-base)
ansible_2_11
python310Packages.yangson
ttp (python39Packages.ttp)
python39Packages.yangson

Maintainer pings
cc @mweinelt for testing.

Already in #154969


Make env_id in an App optional again

What this PR does / why we need it:
This PR just removes the validations to make sure an application has the env_id field.
This got temporarily disabled at https://github.com/pipe-cd/pipe/pull/2918 as a workaround. While I'm in the middle of https://github.com/pipe-cd/pipe/issues/2915, this makes debugging in dev cluster easier.
Which issue(s) this PR fixes:
Ref https://github.com/pipe-cd/pipe/issues/2915
Does this PR introduce a user-facing change?:

NONE

/lgtm

There are some fields in the proto that are still being validated.
Should we update them in this PR too?
https://github.com/pipe-cd/pipe/search?q=env_name

Alright

This PR is just for making debugging easier so I was planning to update them later. But let me do that so that we can easily revert when something went wrong.

Made them optional as well.

in that case, should update the PR name too, since it mentioned just the env_id field 👍
/lgtm

True. I did it.

/approve


Line numbers don't correspond

I've noticed that the line numbers reported in error messages don't correspond very well to the line numbers given in the gutters of the .Rmd files. Usually they're low by, say, 10 or 20, which makes me think that it has to do with R not counting lines in the preamble.

The line numbers for errors generated in code chunks should be correct. If errors are induced from inline code, the reported line numbers typically refer to the first line of text in the block of text between code chunks in which the inline code appears (if that makes sense).

Ah, good to know. I’ll see if I can repro this. It’s been my impression that it’s further off than that for inline code, but let me check.

On Oct 17, 2018, at 3:45 PM, Sean Raleigh<EMAIL_ADDRESS>wrote:
The line numbers for errors generated in code chunks should be correct. If errors are induced from inline code, the reported line numbers typically refer to the first line of text in the block of text between code chunks in which the inline code appears (if that makes sense).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/VectorPosse/Intro_Stats/issues/12#issuecomment-430800825, or mute the thread https://github.com/notifications/unsubscribe-auth/AIeF-mp6uhkYEUIg3Wd1cj8aY49ioqgdks5ul6T3gaJpZM4Xo94V.


We now have a CLA!

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

@devium you can also sign it by going to the CLA assistant website directly :p


Duplicate Tab doesn't duplicate split panes

Steps to reproduce
Create some spit panes in a tab, then ctrl-shift-d  to duplicateTab,   new tab is a single pane tab.
Expected behavior
new tab is a duplicate of the split paned tab with the same panes
Actual behavior
single pane tab

I don't hate the idea of adding a setting to the "newTab" ShortcutAction to enable duplicating panes as well. Thanks for the suggestion!
Something like:
{ "command": { "action": "newTab", "duplicatePanes": true }, "keys": ["ctrl+shift+d"] },

I honestly wonder if duplicating a tab should simply always duplicate its panes. After all, not doing so (and terminal randomly selecting one of the panes under the tab) could be viewed as simply incorrect...

Yanking triage in any case: this is a good backlog item, but if somebody wants to work on it we can pull it ahead.

@DHowett-MSFT -  I did some short implementation of panes duplication (that almost  works, besides setting the focus in the same pane, that I simply cannot get working), but I am just wondering how useful it is when the panes are created in the initial state (e.g. default directory).

Maybe it's valuable to do even without the path preservation now, so that when #3158 / #8166 / whatever it ends up being does land, then the path duplication will just start working for duplicated tabs as well.
Personally, I'm the kind of user where I set up each profile's startingDirectory to the root of a project, and typically I stay in the root of each project, so duplicating a bunch of panes that are all in that directory already doesn't seem like a bad thing to me ☺

@zadjii-msft - I will try to push it then.. hopefully  this week / weekend.


[Suggestion] Way to intentionally share a @Share between two mixins

a @Share is unique to a mixin, which is very good for compatibility, but that behavior can occasionally be desired.
For Porting Lib, I need to wrap a parameter as early as possible, and then access it later as late as possible.
I am doing this with two mixins, one with low priority, and one with high. Since it's split, I can't use a @Share, and I'll have to use a ThreadLocal.

I think a solution would be to have an option to expand the scope to within the current mixin config, if that's possible.

I think the most general solution would be to allow you to specify a namespace, which would default to the name of your mixin (the current behaviour).

It can't default to exactly the name of your mixin, because mixin names are fairly standardized. For example, the number of MixinBlock/BlockMixin or MixinLivingEntity/LivingEntityMixin are probably in the thousands.

The fully qualified name.

Fair enough


update colmap camera models

add PINHOLE, SIMPLE_PINHOLE models for colmap

I see, so does this now work for COLMAP datasets when you specify the camera type as pinhole/simple_pinhole in the processing? Btw, what is the reason for choosing pinhole/simple_pinhole over the perspective model?

This is because colmap and various methods provided by nerfstudio are not used only for nerf provided by nerfstudio.
In order to use the colmap results/pose estimation in nerfstudio in other to apply other nerf git repos, it is necessary to utilize various camera models which uses pinhole or simple_pinhole model.

I will close this one, since you made a new PR with the requested fix. Thanks.


/var/log/fail2ban.log    has errors :   iptables v1.4.21: can't initialize iptables table `filter': Permission denied (you must be root)\nPerhaps iptables or your kernel needs to be upgraded.

Seeing this on new freshly built container ....
fail2ban is running as root yet seeing these errors

root@mail /var/log # cat  /var/log/fail2ban.log

2016-10-13 16:29:29,284 fail2ban.server         [1335]: INFO    Changed logging target to /var/log/fail2ban.log for Fail2ban v0.9.5
2016-10-13 16:29:29,286 fail2ban.database       [1335]: INFO    Connected to fail2ban persistent database '/var/lib/fail2ban/fail2ban.sqlite3'
2016-10-13 16:29:29,289 fail2ban.database       [1335]: WARNING New database created. Version '2'
2016-10-13 16:29:29,290 fail2ban.jail           [1335]: INFO    Creating new jail 'sshd'
2016-10-13 16:29:29,294 fail2ban.jail           [1335]: INFO    Jail 'sshd' uses poller
2016-10-13 16:29:29,310 fail2ban.filter         [1335]: INFO    Set jail log file encoding to ANSI_X3.4-1968
2016-10-13 16:29:29,310 fail2ban.jail           [1335]: INFO    Initiated 'polling' backend
2016-10-13 16:29:29,312 fail2ban.filter         [1335]: INFO    Added logfile = /var/log/auth.log
2016-10-13 16:29:29,313 fail2ban.filter         [1335]: INFO    Set maxRetry = 3
2016-10-13 16:29:29,313 fail2ban.filter         [1335]: INFO    Set jail log file encoding to ANSI_X3.4-1968
2016-10-13 16:29:29,314 fail2ban.actions        [1335]: INFO    Set banTime = 10800
2016-10-13 16:29:29,314 fail2ban.filter         [1335]: INFO    Set findtime = 600
2016-10-13 16:29:29,315 fail2ban.filter         [1335]: INFO    Set maxlines = 10
2016-10-13 16:29:29,381 fail2ban.server         [1335]: INFO    Jail sshd is not a JournalFilter instance
2016-10-13 16:29:29,391 fail2ban.jail           [1335]: INFO    Creating new jail 'postfix'
2016-10-13 16:29:29,392 fail2ban.jail           [1335]: INFO    Jail 'postfix' uses poller
2016-10-13 16:29:29,393 fail2ban.filter         [1335]: INFO    Set jail log file encoding to ANSI_X3.4-1968
2016-10-13 16:29:29,393 fail2ban.jail           [1335]: INFO    Initiated 'polling' backend
2016-10-13 16:29:29,394 fail2ban.filter         [1335]: INFO    Added logfile = /var/log/mail/mail.log
2016-10-13 16:29:29,395 fail2ban.filter         [1335]: INFO    Set maxRetry = 3
2016-10-13 16:29:29,395 fail2ban.filter         [1335]: INFO    Set jail log file encoding to ANSI_X3.4-1968
2016-10-13 16:29:29,396 fail2ban.actions        [1335]: INFO    Set banTime = 10800
2016-10-13 16:29:29,397 fail2ban.filter         [1335]: INFO    Set findtime = 600
2016-10-13 16:29:29,422 fail2ban.server         [1335]: INFO    Jail postfix is not a JournalFilter instance
2016-10-13 16:29:29,432 fail2ban.jail           [1335]: INFO    Creating new jail 'dovecot'
2016-10-13 16:29:29,433 fail2ban.jail           [1335]: INFO    Jail 'dovecot' uses poller
2016-10-13 16:29:29,434 fail2ban.filter         [1335]: INFO    Set jail log file encoding to ANSI_X3.4-1968
2016-10-13 16:29:29,434 fail2ban.jail           [1335]: INFO    Initiated 'polling' backend
2016-10-13 16:29:29,436 fail2ban.filter         [1335]: INFO    Added logfile = /var/log/mail/mail.log
2016-10-13 16:29:29,437 fail2ban.filter         [1335]: INFO    Set maxRetry = 3
2016-10-13 16:29:29,438 fail2ban.filter         [1335]: INFO    Set jail log file encoding to ANSI_X3.4-1968
2016-10-13 16:29:29,440 fail2ban.actions        [1335]: INFO    Set banTime = 10800
2016-10-13 16:29:29,440 fail2ban.filter         [1335]: INFO    Set findtime = 600
2016-10-13 16:29:29,466 fail2ban.jail           [1335]: INFO    Creating new jail 'postfix-sasl'
2016-10-13 16:29:29,467 fail2ban.jail           [1335]: INFO    Jail 'postfix-sasl' uses poller
2016-10-13 16:29:29,468 fail2ban.filter         [1335]: INFO    Set jail log file encoding to ANSI_X3.4-1968
2016-10-13 16:29:29,468 fail2ban.jail           [1335]: INFO    Initiated 'polling' backend
2016-10-13 16:29:29,470 fail2ban.filter         [1335]: INFO    Added logfile = /var/log/mail/mail.log
2016-10-13 16:29:29,470 fail2ban.filter         [1335]: INFO    Set maxRetry = 3
2016-10-13 16:29:29,471 fail2ban.filter         [1335]: INFO    Set jail log file encoding to ANSI_X3.4-1968
2016-10-13 16:29:29,471 fail2ban.actions        [1335]: INFO    Set banTime = 10800
2016-10-13 16:29:29,472 fail2ban.filter         [1335]: INFO    Set findtime = 600
2016-10-13 16:29:29,475 fail2ban.server         [1335]: INFO    Jail postfix-sasl is not a JournalFilter instance
2016-10-13 16:29:29,487 fail2ban.jail           [1335]: INFO    Jail 'sshd' started
2016-10-13 16:29:29,498 fail2ban.jail           [1335]: INFO    Jail 'postfix' started
2016-10-13 16:29:29,520 fail2ban.jail           [1335]: INFO    Jail 'dovecot' started
2016-10-13 16:29:29,536 fail2ban.jail           [1335]: INFO    Jail 'postfix-sasl' started
2016-10-13 16:29:29,597 fail2ban.action         [1335]: ERROR   iptables -w -N f2b-sshd
iptables -w -A f2b-sshd -j RETURN
iptables -w -I INPUT -p tcp -m multiport --dports ssh -j f2b-sshd -- stdout: ''
2016-10-13 16:29:29,598 fail2ban.action         [1335]: ERROR   iptables -w -N f2b-sshd
iptables -w -A f2b-sshd -j RETURN
iptables -w -I INPUT -p tcp -m multiport --dports ssh -j f2b-sshd -- stderr: "iptables v1.4.21: can't initialize iptables table `filter': Permission denied (you must be root)\nPerhaps iptables or your kernel needs to be upgraded.\niptables v1.4.21: can't initialize iptables table `filter': Permission denied (you must be root)\nPerhaps iptables or your kernel needs to be upgraded.\ngetsockopt failed strangely: Operation not permitted\n"
2016-10-13 16:29:29,598 fail2ban.action         [1335]: ERROR   iptables -w -N f2b-sshd
iptables -w -A f2b-sshd -j RETURN
iptables -w -I INPUT -p tcp -m multiport --dports ssh -j f2b-sshd -- returned 1
2016-10-13 16:29:29,598 fail2ban.actions        [1335]: ERROR   Failed to start jail 'sshd' action 'iptables-multiport': Error starting action
2016-10-13 16:29:29,701 fail2ban.action         [1335]: ERROR   iptables -w -N f2b-postfix
iptables -w -A f2b-postfix -j RETURN
iptables -w -I INPUT -p tcp -m multiport --dports smtp,465,submission -j f2b-postfix -- stdout: ''
2016-10-13 16:29:29,702 fail2ban.action         [1335]: ERROR   iptables -w -N f2b-postfix
iptables -w -A f2b-postfix -j RETURN
iptables -w -I INPUT -p tcp -m multiport --dports smtp,465,submission -j f2b-postfix -- stderr: "iptables v1.4.21: can't initialize iptables table `filter': Permission denied (you must be root)\nPerhaps iptables or your kernel needs to be upgraded.\niptables v1.4.21: can't initialize iptables table `filter': Permission denied (you must be root)\nPerhaps iptables or your kernel needs to be upgraded.\ngetsockopt failed strangely: Operation not permitted\n"
2016-10-13 16:29:29,702 fail2ban.action         [1335]: ERROR   iptables -w -N f2b-postfix
iptables -w -A f2b-postfix -j RETURN
iptables -w -I INPUT -p tcp -m multiport --dports smtp,465,submission -j f2b-postfix -- returned 1
2016-10-13 16:29:29,702 fail2ban.actions        [1335]: ERROR   Failed to start jail 'postfix' action 'iptables-multiport': Error starting action
2016-10-13 16:29:29,806 fail2ban.action         [1335]: ERROR   iptables -w -N f2b-dovecot
iptables -w -A f2b-dovecot -j RETURN
iptables -w -I INPUT -p tcp -m multiport --dports pop3,pop3s,imap,imaps,submission,465,sieve -j f2b-dovecot -- stdout: ''
2016-10-13 16:29:29,806 fail2ban.action         [1335]: ERROR   iptables -w -N f2b-dovecot
iptables -w -A f2b-dovecot -j RETURN
iptables -w -I INPUT -p tcp -m multiport --dports pop3,pop3s,imap,imaps,submission,465,sieve -j f2b-dovecot -- stderr: "iptables v1.4.21: can't initialize iptables table `filter': Permission denied (you must be root)\nPerhaps iptables or your kernel needs to be upgraded.\niptables v1.4.21: can't initialize iptables table `filter': Permission denied (you must be root)\nPerhaps iptables or your kernel needs to be upgraded.\ngetsockopt failed strangely: Operation not permitted\n"
2016-10-13 16:29:29,806 fail2ban.action         [1335]: ERROR   iptables -w -N f2b-dovecot
iptables -w -A f2b-dovecot -j RETURN
iptables -w -I INPUT -p tcp -m multiport --dports pop3,pop3s,imap,imaps,submission,465,sieve -j f2b-dovecot -- returned 1
2016-10-13 16:29:29,806 fail2ban.actions        [1335]: ERROR   Failed to start jail 'dovecot' action 'iptables-multiport': Error starting action
2016-10-13 16:29:29,910 fail2ban.action         [1335]: ERROR   iptables -w -N f2b-postfix-sasl
iptables -w -A f2b-postfix-sasl -j RETURN
iptables -w -I INPUT -p tcp -m multiport --dports smtp,465,submission,imap3,imaps,pop3,pop3s -j f2b-postfix-sasl -- stdout: ''
2016-10-13 16:29:29,910 fail2ban.action         [1335]: ERROR   iptables -w -N f2b-postfix-sasl
iptables -w -A f2b-postfix-sasl -j RETURN
iptables -w -I INPUT -p tcp -m multiport --dports smtp,465,submission,imap3,imaps,pop3,pop3s -j f2b-postfix-sasl -- stderr: "iptables v1.4.21: can't initialize iptables table `filter': Permission denied (you must be root)\nPerhaps iptables or your kernel needs to be upgraded.\niptables v1.4.21: can't initialize iptables table `filter': Permission denied (you must be root)\nPerhaps iptables or your kernel needs to be upgraded.\ngetsockopt failed strangely: Operation not permitted\n"
2016-10-13 16:29:29,910 fail2ban.action         [1335]: ERROR   iptables -w -N f2b-postfix-sasl
iptables -w -A f2b-postfix-sasl -j RETURN
iptables -w -I INPUT -p tcp -m multiport --dports smtp,465,submission,imap3,imaps,pop3,pop3s -j f2b-postfix-sasl -- returned 1
2016-10-13 16:29:29,910 fail2ban.actions        [1335]: ERROR   Failed to start jail 'postfix-sasl' action 'iptables-multiport': Error starting action

group of fail2ban process is adm as per
root@mail /var/log # ls -la /var/log | grep fail
-rw-r-----  1 root   adm       9367 Oct 13 16:29 fail2ban.log
-rw-r--r--  1 root   root    160032 Oct 13 16:05 faillog

here is content of
visudo
#
# This file MUST be edited with the 'visudo' command as root.
#
# Please consider adding local content in /etc/sudoers.d/ instead of
# directly modifying this file.
#
# See the man page for details on how to write a sudoers file.
#
Defaults        env_reset
Defaults        mail_badpass
Defaults        secure_path="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin"

# Host alias specification

# User alias specification

# Cmnd alias specification

# User privilege specification
root    ALL=(ALL:ALL) ALL

# Members of the admin group may gain root privileges
%admin ALL=(ALL) ALL

# Allow members of group sudo to execute any command
%sudo   ALL=(ALL:ALL) ALL

# See sudoers(5) for more information on "#include" directives:

#includedir /etc/sudoers.d

Hi @scottstensland, this Wiki may help.


rollback privacy config

Task/Issue URL:
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

#5144  👈
develop

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @malmstein and the rest of your teammates on  Graphite


Improve file format handling

What feature do you want to see added?
For the publishReport, I suggest to take inspiration from warning ng plugin.
For now the main format to support is csv, json, yaml (with specific syntax). Maybe some future files will be supported

Generic CSV (simple file format accessible)
publishReport(tools: [simpleCsv(id: 'customId', name: 'customName', pattern: '*.csv')])

json + yaml + xml (plugin internal data format)
publishReport(tools: [nativeJson(id: 'customId', name: 'customName', pattern: '.json')])
publishReport(tools: [nativeYaml(id: 'customId', name: 'customName', pattern: '.yaml')])
publishReport(tools: [nativeXml(id: 'customId', name: 'customName', pattern: '*.xml')])

In the future (with tool popularity increasing), other format might be supported (file from specific standards for metric data)
Upstream changes
No response

Thanks! Thats exactly what i am doing right now! I am now ready to take care of the merging of multiple reports..

@totocaca123 For now I have the following api:
publishReport name: "JSON Report", displayType: "dual", provider: json(pattern: "etc/*.json")
publishReport name: "XML Report", displayType: "dual", provider: xml(pattern: "etc/*.xml")
publishReport name: "YAML Report", displayType: "dual", provider: yaml(pattern: "etc/*.yaml")

the whole infrastructure is in place, now I have to merge the reports and plot the charts. And then let's see where errors still appear...


feat(design): add paginator component

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/graycoreio/daffodil/blob/master/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the new behavior?
Add the paginator component to @daffodil/design for controlling selection of pages of items.
Does this PR introduce a breaking change?
[ ] Yes
[x] No

All the spec files not related to paginator should be moved to a separate PR?

All the test files not related to paginator should be moved to a separate PR?

I removed these changes

Also for accessibility, I think paginator needs to be a  and use
 instead of s.
https://www.a11ymatters.com/pattern/pagination/

@lderrickable Also for accessibility, the page-links should use
s in an unordered list, and role="navigation" assigned via HostBinding.
Checkout this article: https://www.a11ymatters.com/pattern/pagination/

Paginator is now accessible

Also, I made the next and prev disable instead of disappearing.


XCode 12.5

Hi, i'm trying this module, in Android is working ok, but in IOS i cannot start the app. Getting this error..
'stringByReplacingPercentEscapesUsingEncoding:' is deprecated: first deprecated in iOS 9.0 - Use -stringByRemovingPercentEncoding instead, which always uses the recommended UTF-8 encoding.
How i can progress?
Thanks,
Gonzalo

did you find a solution?


Disable finish gitops setup button onclick as well and not just the s…

…tyle

the disabled prop does both things


Add test to table_DrawdownsRatio macro

Because of unit problem in BurkeRatio, PainRatio and MartinRatio, this macro cannot compare with that from R.

Done by #252


Support for the Meson-python build backend

Does anyone know if it is possible to use the Meson-python  /  Meson build system with versioneer? If no, then this is a feature request!
In the pyproject.toml, the meson-python backend is chosen by setting
[build-system]
build-backend = 'mesonpy'

As far as I can tell, you can't use versioneer as it requires the setuptools.build_meta backend.

Versioneer was pretty specifically designed around setuptools, and there's been no serious attempt that I've seen to generalize it. Of course, anybody's welcome to hack on it and submit a PR...

I kind of got versionneer to work with Meson (as shown in this PR for a different project), but I think that a few changes could perhaps be made to make this work better.
First, in the pyproject.toml file, I added a "[tool.versioneer]" section. Then, when installing versioneer in no-vendor mode you get the following output
$ versioneer install --no-vendor
 creating pyshtools/_version.py
 pyshtools/__init__.py unmodified
Traceback (most recent call last):
  File "/Users/lunokhod/miniconda3/envs/py312/bin/versioneer", line 10, in <module>
    sys.exit(main())
             ^^^^^^
  File "/Users/lunokhod/miniconda3/envs/py312/lib/python3.12/site-packages/versioneer.py", line 2001, in main
    setup_command()
  File "<string>", line 2270, in setup_command
  File "<string>", line 2235, in scan_setup_py
FileNotFoundError: [Errno 2] No such file or directory: 'setup.py'

It successfully creates a _version.py file, and the error seems to be related to the fact that we are using build-backend = 'mesonpy' instead of setuptools.build_meta.
In the main meson.build file, we defined the version as follows
project('shtools', 'c', 'fortran',
  version: run_command(find_program('python3'), '-c', 'import os; os.chdir("pyshtools"); import _version; print(_version.get_versions()["version"])').stdout().strip(),
  license: 'BSD-3-Clause',
  meson_version: '>=1.1.0',
  default_options: [
    'buildtype=release',
  ])

In pyproject.toml, we added
[project]
name = 'pyshtools'
dynamic = ['version']

And in the __init__.py file, we removed
from . import _version
__version__ = _version.get_versions()['version']

and instead used
try:
    __version__ = version('pyshtools')
except PackageNotFoundError:
    # package is not installed
    pass

Even though the project now builds, when run from a source tarball, versioneer sets the version to the uninformative '0+unknown', which is better than failing.
If you run
python -c 'import os; os.chdir("pyshtools"); import _version; print(_version.get_versions())'

you get
{'version': '0+unknown', 'full-revisionid': '9ec26db51f2f7a836b4c346e7f16328884a870f9', 'dirty': False, 'error': 'no suitable tags', 'date': None}

What is surprising to me is that the date is set to None, but when you inspect the _version.py file you see
    git_refnames = " (new_versioneer)"
    git_full = "9ec26db51f2f7a836b4c346e7f16328884a870f9"
    git_date = "2024-01-23 12:54:16 +0100"
    keywords = {"refnames": git_refnames, "full": git_full, "date": git_date}

I was also somewhat surprised that versionneer didn't deduce the version from the directory name, though I haven't dug too deep in the _version.py to see why not. Ideally, I would really like versioneer to produce something like v4.11.10.devN when run in a non-git-versioned folder, but maybe that isn't possible.

The error is just that there is no option not to look at setup.py:
https://github.com/python-versioneer/python-versioneer/blob/bdf701a688a410659a42c49cab6bd901389743d6/src/setupfunc.py#L115-L162
If you want to generalize this to another backend, maybe the approach would be to detect the backend from pyproject.toml, defaulting to setuptools, and then have a scan function per-backend. A fallback scan function could just report the backend and alert the user that we can't assure them they've set things up correctly.
Instead of the chdir, etc. You could try using runpy.
If you try from a git archive tarball, you should be able to see something better than 0+unknown. How are you making your tarball?

When I use git archive, meson dist, and downloading the source from github, the version is always 0+unknown.
Here is what gets rewritten in _version.py when using git archive:
def get_keywords() -> Dict[str, str]:
    """Get the keywords needed to look up the version information."""
    # these strings will be replaced by git during git-archive.
    # setup.py/versioneer.py will grep for the variable names, so they must
    # each be defined on a line of their own. _version.py will just call
    # get_keywords().
    git_refnames = " (HEAD -> new_versioneer, origin/new_versioneer)"
    git_full = "65ca9c0bc36940fe0ca9b2e577ef0b043ddcb25a"
    git_date = "2024-01-23 13:27:53 +0100"
    keywords = {"refnames": git_refnames, "full": git_full, "date": git_date}
    return keywords

and here is the get_versions() output:
{'version': '0+unknown', 'full-revisionid': '65ca9c0bc36940fe0ca9b2e577ef0b043ddcb25a', 'dirty': False, 'error': 'no suitable tags', 'date': None}

I note that the date is getting set correctly.

We currently have a halfway house; I've modified _version.py to have at the bottom a case to print out the versioneer-provided version by default when ran. Then in our meson.build, we have:
project(
    'pixell',
    ['c', 'fortran', 'cython'],
    version: run_command(
        [
            # Note that _version.py doesn't actually 'dynamically' set the
            # version number. We call it at build time and set it as a meson
            # property.
            'python',
            'pixell/_version.py'
        ],
        check: true,
    ).stdout().strip()
)

which freezes the version at build time, but it is still generated by versioneer. Then in pyproject.toml we write dynamic = ['version'].
This may be helpful but I can see it's not an officially supported use-case.


Commitlint miss header more 72 symbols for -m and --amend

Expected Behavior
Commint lint should return error for header more than 72 symbol
Current Behavior
Commitlint miss header more 72 symbols for -m and --amend
Affected packages

[X] cli
[ ] core
[ ] prompt
[ ] config-angular

Possible Solution

Steps to Reproduce (for bugs)

git clone https://github.com/ravecat/styled-components-toolbox.git --single-branch --branch issue/commitlint-header-more-72
yarn
Last commit more than 72 symbols

commitlint.config.js
module.exports = {
  extends: ['@commitlint/config-conventional']
}

Your Environment

Executable
Version

commitlint --version
7.2.1

git --version
2.11.0

node --version
10.12.0

Trying to reproduce this with teh current master by doing this:
git commit -m 'chore: this is a way too long message so maybe this will not work hopefully because the defaults say it should not work so here ya go'  --no-verify

After that I tried to --amend it and got this:
git commit --amend
husky > pre-commit (node v10.1.0)
 ↓ Running tasks for *.js [skipped]
   → No staged files match *.js
husky > commit-msg (node v10.1.0)
(node:9668) ExperimentalWarning: The fs.promises API is experimental

⧗   input:
chore: this is a way too long message so maybe this will not work hopefully because the defaults say it should not work so here ya go

✖   header must not be longer than 72 characters [header-max-length]
✖   found 1 problems, 0 warnings
    (Need help? -> https://github.com/marionebl/commitlint#what-is-commitlint)
husky > commit-msg hook failed (add --no-verify to bypass)

So this seems to be fine on master at least.
Adding a too long message with normal -m is prevented as well:
git commit -m 'chore: this is a way too long message so maybe this will not work hopefully because the defaults say it should not work so here ya go'
husky > pre-commit (node v10.1.0)
 ✔ Running tasks for *.js
husky > commit-msg (node v10.1.0)
(node:9820) ExperimentalWarning: The fs.promises API is experimental

⧗   input:
chore: this is a way too long message so maybe this will not work hopefully because the defaults say it should not work so here ya go

✖   header must not be longer than 72 characters [header-max-length]
✖   found 1 problems, 0 warnings
    (Need help? -> https://github.com/marionebl/commitlint#what-is-commitlint)
husky > commit-msg hook failed (add --no-verify to bypass)


buildah delete should be able to delete multiple containers

Simple change to allow users to specify multiple containers to delete.
This would support cleaning up all containers
buildah delete $( buildah list -q)

LGTM.

@rh-atomic-bot r+ 5f508b4

 (8) [Buildah] buildah CLI overhaul


i18n Remap Fonts Crashes Engine

Godot version:
3.1.0
OS/device including version:
Ubuntu 18.04.2 LTS
Issue description:
I am trying to localize my game to Japanese, but the font I use does not support it. I tried to remap the ttf file I use to one that does support Japanese (as one would for images etc.). The new ttf file appears in the window, but when I try to change the locale to Japanese the engine crashes.
Stacktrace:
ERROR: get: FATAL: Index p_index=181 out of size (size()=1)
   At: ./core/cowdata.h:151.
handle_crash: Program crashed with signal 4
Dumping the backtrace. Please include this when reporting the bug on https://github.com/godotengine/godot/issues
[1] /lib/x86_64-linux-gnu/libc.so.6(+0x3ef20) [0x7f91c2515f20] (??:0)
[2] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0x110ca13] (<artificial>:?)
[3] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0xa414e9] (??:?)
[4] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0x2174733] (??:?)
[5] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0x21e4e02] (??:?)
[6] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0x21e5e40] (??:?)
[7] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0x181dd86] (<artificial>:?)
[8] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0xae689d] (??:?)
[9] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0x2174733] (??:?)
[10] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0x21e4e02] (??:?)
[11] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0x21e5e40] (??:?)
[12] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0x17242b2] (??:?)
[13] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0x178b84b] (<artificial>:?)
[14] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0x9ef656] (??:?)
[15] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0x2174ad5] (??:?)
[16] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0x212f911] (??:?)
[17] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0x16b4648] (<artificial>:?)
[18] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0x16dbda7] (<artificial>:?)
[19] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0x16dcc84] (??:?)
[20] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0x16dcdee] (<artificial>:?)
[21] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0x9ef656] (??:?)
[22] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0x2174733] (??:?)
[23] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0x212f995] (??:?)
[24] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0x16541c3] (??:?)
[25] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0x16b3957] (??:?)
[26] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0xa7a851] (??:?)
[27] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0xa77d4d] (<artificial>:?)
[28] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0xb39c9b] (<artificial>:?)
[29] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0x7b28dc] (??:?)
[30] /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xe7) [0x7f91c24f8b97] (??:0)
[31] /home/tom/Godot/310/Godot_v3.1-stable_x11.64() [0x7bfc4e] (??:?)
-- END OF BACKTRACE --

Steps to reproduce:

Add ttf file
Go to project/project settings/Localization/Remaps
Add ttf file to Remap by locale
Change Remap's locale to any other language

This happens with png files, too.


Add suggestion to also target .NET Standard 2.0

Short addition to the .NET Standard targeting guide.
Based on .NET Standard targeting suggestions on dotnet/standard and tweet by David Fowler.

@LorenzCK

I pointed the link to the library-guidance/cross-platform-targeting.md page in #7833, which appears to be the correct document on how to target .NET Standard.

Yep, that's the document I had in mind too.

Thanks @terrajobst, fixed.

❤

Great! Thank you all 😊


Where is the test code?

Can you provide the test code?

Thanks for your interest! I'm still cleaning the codes. Will let you know as soon as I finish.

Hey @GuideWsp , the test scripts are just updated. Enjoy!


focus view

Hi,
I took  your code and load it as am iframe in my application.
My node-red version is 0.10.6.
I have a problem on windows, chrome:
On click on the editor iframe, jump my window scrolling to the top of the canvas.
it is because focus code you add on 'Allow the main view hold keyboard focus':
https://github.com/node-red/node-red/commit/fabf01371437dc4af09293820036ab222f25bda8#diff-38fe95afd24aaea22b697b31f24751b8
Why did you add the focus code?
Do you have An idea how ignor the window to auto scroll on focuse?
Thanks,

We had a similar thing reported recently here: https://groups.google.com/forum/#!topic/node-red/qbmdyjukjmk
The fix there was around the CSS of the containing page. Is that similar to what you're seeing?

This may be relevant - http://code.google.com/p/chromium/issues/detail?id=437025

I changed focusView function:
parentWindow = window.parent.window;
var x = parentWindow.scrollX, y = parentWindow.scrollY;
$("#chart svg").focus();
parentWindow.scrollTo(x, y);

I saw this happen for myself recently. Will take another look at the change you made and see how we might apply it generically.

Fixed in 0.14.0 - 7318a7b767c22846e0554357d6a7c24142665479


Different outputs are generated by convert_valid_utf8_to_latin1+fallback and convert_valid_utf8_to_latin1+rvv

I have run the benchmarks and specified the input as emoji.txt and found that Different outputs were generated by the convert_valid_utf8_to_latin1+fallback and the convert_valid_utf8_to_latin1+rvv.
The first four input elements and the first four output elements are printed , and the results are showed as below:

The return value of convert_valid_utf8_to_latin1+fallback is zero and an error message of "The output is zero which might indicate an error." are reported (showed as below), but the return value of the convert_valid_utf8_to_latin1+rvv is not zero and it will not report this messge.

The convert_valid_* should only be used on valid UTF-8 sequences. It is your responsability to ensure that the inputs are valid.
Most users should NOT use these functions. You should use the validating functions.
I am closing this issue.


feat(docs): updated README to better reflect the recent changes

Codecov Report

Merging #5 (9a8458d) into main (8f1ff86) will increase coverage by 0.35%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main       #5      +/-   ##
==========================================
+ Coverage   62.77%   63.12%   +0.35%
==========================================
  Files           5        5
  Lines         180      179       -1
==========================================
  Hits          113      113
+ Misses         56       55       -1
  Partials       11       11

Flag
Coverage Δ

unittests
63.12% <ø> (+0.35%)
:arrow_up:

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

Impacted Files
Coverage Δ

csvprocessor.go
73.77% <ø> (ø)

new.go
32.20% <ø> (+0.53%)
:arrow_up:

wrapper.go
100.00% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Switch to optional arn

Hey! Thanks for the contribution. Can you describe this PR a little better? I'm not sure what "Switch to optional arn" means.

Pulled these commits into master and added tests and published as v3.1.0 – thanks for the contribution!


feat: create chart component

adding

metrics chart
campaign metrics chart
campaign chart for sales and click metrics that will be used in shop campaign metrics

Looks good. My only concern is the build size with the new dependencies. How does it compares the size of the dist folder?

dist folder on main:
size: 534 KB
size on disk: 860 KB
dist folder on this branch:
size: 540 KB
size on disk: 872 KB


[Checkout Performance Metrics] Add anomalous tag for navigation download size

Description
Partially Solves #2396
We have added support for a anomalousNavigationDownloadSizeThreshold optional parameter that is to be passed into clientPerformanceMetrics. There is an ongoing PR that fires clientPerformanceMetrics and provides it with a threshold value
We are using anomalousNavigationDownloadSizeThreshold similar to how the anomalousNavigationDurationThreshold is implemented. We have an anomalous property that is used across all metrics within middleware.ts (this is being sent as part of the navigationTags object). Since anomalousNavigationDownloadSizeThreshold only applies to the  navigation_download_size metric (and anomalousNavigationDurationThreshold is not relevant to this specific metric), we are overriding the anomalous value for this metric based on if the total script download size is greater (anomalous is true) or less than (anomalous is false) the provided threshold value.
Tophat
I have tophatted these changes locally but would be interested in gaining more context in this area or feedback on the approach.

/shipit


Implemented delete tags functionality

Implemented functionality to be able to delete tags from a specific media item in some watchlist

Closes #57

which remind me I should also check the whether the user input is a valid tag or not. When I test it it, I all input the correct tag


[FEATURE]: Add pointers to GTFS  providers

Describe the solution you'd like
The most important step of the config is pointing to the GTFS provider. On the home page of the wiki pages you list the GTFS providers you tested, however, you do not list the providers. It would save prospect user of your integration a lot of time if you add a link to the providers you tested with.
Describe alternatives you've considered
Search the internet and hope I am using the right thing
Additional context
Many thanks for this project, it looks very promising and just what I need.

Was outstanding, done


Exclude sessions after cutoff time from starting_number

Closes #11

I've checked this on Redshift and it's working as expected


It will be good to add URLSearchParams support

To make it possible to use x-www-urlencoded data.
Thanks.

It might be out of scope for this polyfill. How do you intend to use this?

At current spec Body mixin can accept URLSearchParams to set Content-Type to be application/x-www-urlencoded.

Is there already some URLSearchParams polyfill? We could suggest that people use it in combination with this polyfill. Should we also add special handling of URLSearchParams bodies?

I found only this one, looks good to me.
I do not think need anything very special. If passed URLSearchParams set Content-Type to be application/x-www-urlencoded and use .toString() to get value.
var p = new URLSearchParams()
p.append('a', 10)
p.append('b', 'abc')
p.toString() // "a=10&b=abc"

If passed URLSearchParams set Content-Type to be application/x-www-urlencoded and use .toString() to get value.

Sounds good. Want to make a pull request for this?
People would need to include the URLSearchParams polyfill themselves.


MSTest projects with WPF dependencies get incorrect TFM

An MSTest project with dependencies on PresentationFramework or PresentationCore will incorrectly get a net5.0 TFM (instead of net5.0-windows) and won't have a <UseWpf> property.

This will be fixed once dotnet/try-convert#399 is merged and the version of try-convert used by UA is updated.


[Sample Issue]

Link to sample
https://learn.microsoft.com/en-us/rest/api/backup/backup-jobs/list?tabs=JavaScript#list-jobs-with-time-filter
Library name and version
@azure/arm-recoveryservicesbackup 11.0.0
Language of the Sample

[ ] C#/.NET
[ ] Java
[X] JavaScript/TypedScript
[ ] Python
[ ] Golang
[ ] Other - Please specify in Issue details field

Sample Issue Type

[X] Sample not working
[ ] Sample missing
[ ] Do not understand sample

Issue details
The filter example provided does not filter the data. I have tried this on my Azure environment and I always get back all JobResource records regardless of the date filter I provide.
Expected behavior
The list of JobResource records filtered by date.
Actual behavior
All records are returned.
Reproduction Steps
Run sample with dates correct for your environment.
Environment
Node 18.15.0
OSX 14.0

@kazrael2119 Could you take a look at this issue?

Hi @NavidMitchell ,could you give your code, and your screenshot of the backup Jobs on portal? I can use the filter to list jobs.

close this issue as there has been no response for a long time. please create a new issue if necessary. Thanks


ci: call release workflow from nightly workflow

Previously, the contents of nightly.yaml were copied to release.yaml to build prqlc and prql-python with the same options as in the release, but this has been changed to call release.yaml from nightly.yaml to reduce code duplication.

I tested on my fork and seems working fine.

Seems limited by this https://github.com/orgs/community/discussions/39394

Looks good!
I wonder whether we could run the same steps for both nightly and test-all, and so collapse the possible configs. But the downside of that is the build time, as we discussed on the previous PR. (Unless we think running nightly is sufficient?)
For the GitHub limitation, would commenting out the snap /rpm workflows, until the limit is raised, get us under 20?

For the GitHub limitation, would commenting out the snap / rpm workflows, until the limit is raised, get us under 20?

Very worst case, we could have pull-request-1.yaml & pull-request-2.yaml...!

(fixed by #3197)

Thanks!

Thank you!


Right-click file, missing options

In the right-click menu on a file, the option for creating a new file and new folder is missing. They appear only when right-clicking on a folder.

VSCode Version: 1.26.1
OS Version: Windows 10

Steps to Reproduce:

Have the side bar shown.
Open some folder with files.
Right-click any file.

Does this issue occur when all extensions are disabled?: Yes

This is as designed. If you would like to add that option also for files you can install some file operations extensions or write an extension yourself

Not really. The option to create a file or folder are already implemented. Not having them where it does make sense, is a bug to me.
Extensions must not become bugfixes!


ItemCheckBox merge

Possibly this can be merged back in

Merge request sent  Dec 8, '15

I made PR https://github.com/mhollweck/react-native-item-checkbox/pull/4 which is still in play


fix(scripts): use docker-compose wisely

Summary
Use var $DOCKER_COMPOSE to invoke docker-compose or docker compose.
KAG-1496
Checklist

[ ] The Pull Request has tests
[ ] There's an entry in the CHANGELOG
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

merge on green


[Help] How publish message with HEX payload using CLI?

Describe the problem you Confuse
A clear and concise description of what you are confusing about.
More detail (optional)
Add any other context or screenshots.

Hello! I have submitted some code. The link is: https://github.com/emqx/MQTTX/pull/1333
Based on your request, I add a -f parameter to the  mqttx pub  command, which will allow you to specify that the content you publish is in hexadecimal format (also supprot base64 or json).
mqttx pub -t ... -h ... -p ...  -m '68656c6c6f' -f "hex"

Hello! I have submitted some code. The link is: #1333
Based on your request, I add a -f parameter to the  mqttx pub command, which will allow you to specify that the content you publish is in hexadecimal format (also supprot base64 or json).
mqttx pub -t ... -h ... -p ...  -m '68656c6c6f' -f "hex"

Thanks for you response, But the mqttx pub not has -f parameter. It's exist in mqttx sub.

Hello! I have submitted some code. The link is: #1333
Based on your request, I add a -f parameter to the  mqttx pub command, which will allow you to specify that the content you publish is in hexadecimal format (also supprot base64 or json).
mqttx pub -t ... -h ... -p ...  -m '68656c6c6f' -f "hex"

Has the code been released?

Hello! I have submitted some code. The link is: #1333
Based on your request, I add a -f parameter to the  mqttx pub command, which will allow you to specify that the content you publish is in hexadecimal format (also supprot base64 or json).
mqttx pub -t ... -h ... -p ...  -m '68656c6c6f' -f "hex"

Has the code been released?
The code has not been released yet.
You can wait for the upcoming release (version 1.9.4, which will be available soon), or compile the code yourself from the 'cli' folder in the mqttx/main repository. Compiling the code is a simple process, and you can refer to mqttx.app  for detailed instructions.

Has the code been released?
@LucasYao93 Thank you for your feedback. Perhaps we will release version 1.9.4 next week.

Hi! Thank you for your feedback. Currently, we have added support for it in https://github.com/emqx/MQTTX/releases/tag/v1.9.4.


Bad URL after fill & submit form

Hello everyone,
I am new in using Capser.JS (1.1.0-beta3), according to the documentation, I can fill out a form and send it automatically.
But...
After sending instead of arriving on my URL + "form_comp.php" as demonstrated on the following tag :
<form name=" f1" id="f1" action=" form_comp.php " method="post" >
I arrive with the following URL:
/form.php/form_comp.php
I do not understand why Casper.JS adds the content of the tag action to the current URL ?
Have you any suggestions?

This is a very basic question about relative URLs.
From what you describe I would guess that your "my URL" from where you submit your form is .../form.php/ instead of .../form.php. If the current URL is the former, the behavior is correct and will be the same in any other browser. If it is the latter, then, in fact, this is a bug, but in the underlying engine you use (which I can hardly imagine, though).
Please ask on the mailing list first, to see if your problem is in any way related to CasperJS, before filing issues against CasperJS. Thanks.

Ok , thank you for your quick response @istr , it works.


how to access node in a trained gmodule

hello, soumith, I have some  question.If I have  a trained gmodules, how can I have access to node in the gModules? @soumith

net.modules[i] it is the  topological order of gmodle graph.
Or you can use
    local ind = 10
   local latent = nil
    for indexNode, node in pairs(net.forwardnodes) do
        if indexNode == ind then
            if node.data.module then
                latent = node.data.module.output:clone()  -- use it to get the specific module output
            end
        end
    end

Note that the net.forwardnodes has a dump node(spliting inputs into different input nodes). So net.forwardnodes[10] equals net.modules[9]

@Naruto-Sasuke thank you for your kind reply. It works. I also find nnquery https://github.com/bshillingford/nnquery also solve the problem quickly

It seems that the docs have not describe about the way to get the intermediate layer info. And some people
want to know it. Here I give more info about it.
In fact you can print out the nngraph model, and you can do it like this.
-- A simple function for printing nngraph model
function printNet(net)
    for i = 1, net:size(1) do
        print(string.format("%d: %s", i, net.modules[i]))
    end
end

It just prints out the  topological order of nngraph. So you need to tell which layer you want(It should not be tough), then you can just net.modules[i] to get the exact layer you want.
 It has been stated above.

Hi;
As suggested by Naruto, net.modules[i] seems to do the trick.
require 'nngraph'
require 'image'
----load a simple model
a= nn.Identity()()
b= nn.SpatialConvolution(3,4,3,3) (a)
c= nn.SpatialMaxPooling(2,2) (b)
model=nn.Sequential()
model:add(nn.gModule({a},{c}))
---- load data and forward pass the data
data= image.lena()
pred= model:forward(data)
----- now to see output of maxpool put 3 in get (),  2 to see convolution feature maps
res=model.modules[1]:get(3).output
---- now reshape the feature map
res= res:view(res:size(1),1,res:size(2),res:size(3))
itorch.image(res)


assign random port to listener if port=0 passed

@mrocklin @quasiben
Getting port information from UCX may take time. Refer https://github.com/openucx/ucx/issues/3534
Will the logic included in this PR suffice for now?
Example usage:
# Server
In [1]: import ucp

In [2]: ucp.init()
Out[2]: 0

In [3]: import asyncio

In [4]: loop = asyncio.get_event_loop()

In [5]: async def talk_to_client(ep, listener):
   ...:     recv_req = await ep.recv_future()
   ...:     recv_msg = ucp.get_obj_from_msg(recv_req)
   ...:     print('client connected')
   ...:     ucp.destroy_ep(ep)
   ...:

In [6]: listener = ucp.start_listener(talk_to_client, listener_port=0, is_coroutine=True)

In [7]: coro = listener.coroutine

In [8]: listener.port
Out[8]: 49038

In [9]: loop.run_until_complete(coro)
client connected

# Client
In [1]: import ucp

In [2]: import asyncio

In [3]: ucp.init()
Out[3]: 0

In [4]: loop = asyncio.get_event_loop()

In [5]: async def talk_to_client(ip, port):
    ...:     ep = ucp.get_endpoint(ip, port)
    ...:     await ep.send_obj(bytes(b"42"))
    ...:     ucp.destroy_ep(ep)
    ...:

In [6]: coro = talk_to_client('A.B.C.D'.encode(), 49038)

In [7]: loop.run_until_complete(coro)

Thanks @Akshay-Venkatesh .
I may be overly-paranoid here, but what happens if someone else grabs this port in between us closing the socket and UCX claiming that port again?  Will UCX err nicely?  If so, I wonder if we might throw this logic in a loop:
if port == 0:
    while True:
        port = get_open_port()
        try:
            listener = ucx.listen(..., port)
        except Exception:
            continue
        else:
            break

Also, unrelatedly, it might be good to start collecting your testing scripts into pytest style functions that we could use to track that this functionality persists over time.
This might be something like
# test_file.py

import pytest

@pytest.mark.asyncio
async def test_zero_port():
    async def talk_to_client(ep, listener):
        recv_req = await ep.recv_future()
        recv_msg = ucp.get_obj_from_msg(recv_req)
        ucp.destroy_ep(ep)

    listener = ucp.start_listener(talk_to_client, listener_port=0, is_coroutine=True)
    assert 0 < listener.port < 2**16

    ep = ucp.get_endpoint(ip, listener.port)

    await asyncio.gather( # make sure that things work
        ep.send_obj(b"42"),
        listener.coroutine
    )

We might then check things with
pip install pytest-asyncio  # to support async def functions in pytest

py.test tests/

I'm happy to generally set these things up to start out (it's a good way for me to get used to this codebase) but I thought I'd run the practice by you to make sure that you were ok with it.

Thanks @Akshay-Venkatesh .
I may be overly-paranoid here, but what happens if someone else grabs this port in between us closing the socket and UCX claiming that port again? Will UCX err nicely? If so, I wonder if we might throw this logic in a loop:
if port == 0:
    while True:
        port = get_open_port()
        try:
            listener = ucx.listen(..., port)
        except Exception:
            continue
        else:
            break

Thanks for the catch. I'm addressing this.

Again though, I may be overly paranoid there (too long testing concurrent systems).  My guess is that what you have here would work very well for the present, and that we could come back to this in the future if necessary.

@mrocklin thanks for the pytest-asyncio suggestion. I think both of your comments are valuable. I've taken a shot at the test. Appreciate a review!

@mrocklin are these changes good enough to go in? If so, I'll merge this to devel and close #79

Fixes #79
Note that the magic word above (Fixes) creates a link within GitHub.  So when we merge this it'll close that issue automatically.  Typically one puts this in the opening comment.

+1 from me.  Should I be merging these things?

I'll merge now. Thanks, Matt


BoardView, useful for repairing

This could be a very nice addition to this GHA: https://github.com/whitequark/kicad-boardview ... this format is commonly used by repair shops.

thank your for the suggestion, I did put it on the feature list

thank your for the suggestion, I did put it on the feature list


⚠ Samsung has degraded performance

In cc89088, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2390 ms

Resolved: Samsung performance has improved in 67fb5b0.


:arrow_up: Upgrade to styled-components 6

Putting this on hold until styled-components work in tests. See: https://github.com/styled-components/styled-components/issues/3297


Credential Type URI definition needed

In the ToIP Trust Registry V1 Specification.md a Credential Type Uri is mentioned but it is not defined anywhere. Is there an URL or resource where this term is defined?

for the v1 specification, this was specifically left undefined as there was no clear definition but a URI could be used by any community that needed it. A DID was envisioned but any valid URI was acceptable.
This remains true still.


Python function_name? behaviour

Hello everyone.
I found that method declarations aren't displayed in hydrogen.
In ipython:
In [7]: map?
Init signature: map(self, /, *args, **kwargs)
Docstring:
map(func, *iterables) --> map object

Make an iterator that computes the function using arguments from
each of the iterables.  Stops when the shortest iterable is exhausted.
Type:           type

In hydrogen:
map? (checkmark)

Thanks

@lgeiger Thanks for your help

Deprecated yet not replaced so I ended up supporting them in nteract notebook.


Box plot does not filter in dashboard

When applying filters in the dashboard the box plot does not filter, not even when force refresh.
However, the filter does apply when select Edit Dashboard.
Expected results
When applying filters in the filter box, box plot should also filter.
Actual results
When applying filters all the charts get filter, except the box plot; but if you click Edit Dashboard the box plot gets updated.
Screenshots
If applicable, add screenshots to help explain your problem.

How to reproduce the bug

Go to "World Bank's Data" dashboard
Select a region in Filter
See Box Plot does not get updated
Force Refresh in Box Plot
See Box Plot does not get updated
Edit Dashboard
Box Plot get updated

Environment
Checklist
Make sure these boxes are checked before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[X] I have reproduced the issue with at least the latest released version of superset.
[X] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
I am using Superset version installed in Preset Alpha.

I am experiencing the exact same problem, surprisingly, I just found that box plots refresh when:

zooming in/out in the web browser.
clicking on "Explore Chart" and then returning to the dashboard

I hope this helps to fix this bug soon 🙏

🏷 #bug

This might be related (or a separate issues), but when editing a chart of type 'box plot', I find that any updated in the 'Group by' field of the 'Query' section are only being reflected when I 'run query' and reload the browser page; the auto-update behaviour I see with the other viz types does not seem to work here.

Re-opening until we get clear confirmation that this is actually fixed.

I'm still having this issue, any updates?

I am experiencing the exact same problem, surprisingly, I just found that box plots refresh when:

zooming in/out in the web browser.
clicking on "Explore Chart" and then returning to the dashboard

I hope this helps to fix this bug soon 🙏

This is exactly what I'm noticing too. Spent over an hour thinking it was something weird with the data causing a problem. Did you find a fix?

OK if anyone is just dying for a quick workaround (that's ugly as hell):
index c3839f46..10d2822e 100644
---<EMAIL_ADDRESS>+++<EMAIL_ADDRESS>@@ -21,7 +21,10 @@ exports.default = transformProps;
  * specific language governing permissions and limitations
  * under the License.
  */
+var widthToggleValue = 1;
 function transformProps(chartProps) {
+  chartProps.width += widthToggleValue;
+  widthToggleValue *= -1;
   const {
     width,
     height,

This forces a resize event which causes the Box Plot to actually update on-screen. The toggling seems important (it didn't trigger with just always subtracting one). Plus, I think it might minimize the odds of runaway chart resizing.
I give up trying to find the root cause, and this will work well enough for me. I tried tracing it and can't find what's missing.

@gtg472b we have started introducing ECharts as the new official chart library as part of SIP-50, with Pie chart currently being migrated: https://github.com/apache-superset/superset-ui/pull/772. The plan is to migrate Boxplot to ECharts, too, but I haven't gotten around to doing it yet. If someone wants to take the lead on migrating Boxplot, I'd be happy to collaborate. Check the ECharts gallery for example Boxplot charts: https://echarts.apache.org/examples/en/index.html#chart-type-boxplot

Thanks for the info @villebro . I know a lot of these visualizations are being deprecated. I just needed to get that Box Plot to work for now, so it'll hold me over till something replaces it. I've made a lot of mild tweaks to several of the visualizations but it appears the project (sensibly) isn't accepting PRs on any of the deprecated ones. I started looking at building a plugin last night, but...the new Superset code deleted my custom Markup visualizations (I added the ability to reference data via Jinja as per #6771). So, if I start on a plugin, that'll be the one I work on first. If it goes well then I will try to look at these Echart ones!

Sounds good 👍  please feel free to reach out on Slack if/when you start working on it


Feature request: set Swagger defaultValue without needing to set a grape default

Right now, the grape "default" field is used to populate the swagger defaultValue field.
The expectations of these two parameters is slightly different. In Swagger, defaultValue pre-populates the field only, but in Grape, it will use the default to fill in a missing parameter.
It would be great to be able to set values like these directly with the "documentation" hash that grape lets you pass.
Will submit a PR some time next week

Makes sense. Would like to see a PR.

:+1:

+1

@jv-dan , two questions about your PR:
default and default_value are not very clear describing their behavior, since default is used by grape, how about naming the latter for example demo, sample, example, etc.?
It would be better to hint the user through swagger ui, that the displayed default value might be only a sample value, not actually the default value of the API. He should provide a specific value when requesting the API.

Good points. The grape documentation uses "example" in it's "Additional documentation"... documentation, so I'll update to that.

any updates on this?

The updates have been made. Don't know why PR #223 didn't update this thread as well...


Break out type registry into its own repository

The type registry in this repository is useful for pedagogical purposes, but it doesn't serve well as a destination for publishers. We need a standalone type registry, also owned by CNCF, as described by this comment.

Superceded by #82.


OAuth permissions request flows

At some point, Amagaki will probably require some sort of Google Sheets (or other Google APIs) access, and I want to propose from the start that ama should request the minimum amount of permissions necessary to do a particular task.
So if a user runs what is similar to grow preprocess -p my-google-sheet for example, then a readonly access token should be requested. But if at some point in the future user runs something like grow translations upload (where translations are stored in a Google Sheet), then ama should then request readwrite access. Ama will need a permissions system to handle this case.

Getting this working with the following scenarios:

oauth2 key file
autoconfigured environment from gcp project
oauth2 flow for user credentials

Amagaki supports server middleware so planning to experiment implementing this as a server middleware plugin, so other plugins that need access to Google stuff can just fetch the auth from this plugin.


Regression: servlet context path and management.endpoints.web.base-path=/

Hi,
As discussed with @marcingrzejszczak  on Gitter.
Given an application with a servlet-context path and a the base actuator path set to /
server.servlet.context-path=/app
management.endpoints.web.base-path=/

Then the export flag is set to false
INFO [appName,89c28383facaa413,89c28383facaa413,false]

See sample project here:
https://github.com/ask4gilles/sleuthzipkinissue/tree/master/demo
I think a regression was introduced by https://github.com/spring-cloud/spring-cloud-sleuth/pull/1150
in:
static Optional<Pattern> getPatternForServerProperties(ServerProperties serverProperties,
				WebEndpointProperties webEndpointProperties) {
			String contextPath = serverProperties.getServlet().getContextPath();
			if (StringUtils.hasText(contextPath)) {
				return Optional.of(Pattern.compile(
						contextPath + webEndpointProperties.getBasePath() + ".*"));
			}
	return Optional.empty();
}

In this case, the skip pattern is:
/api-docs.*|/autoconfig|/configprops|/dump|/health|/info|/metrics.*|/mappings|/trace|/swagger.*|.*\.png|.*\.css|.*\.js|.*\.html|/favicon.ico|/hystrix.stream|/application/.*|/actuator.*|/cloudfoundryapplication|/app.*

so everything is skipped because: /app.*

Proposed by @marcingrzejszczak :

Override skip-pattern:
I tried for example spring.sleuth.web.skip-pattern=/app/info but the /app.* is still added as a skip pattern.
Define own skipPatternProvider:

  @Bean
  public SkipPatternProvider skipPatternProvider() {
    return () -> Pattern.compile("app/info|app/health");
  }

It works but it's cumbersome when you have a lot of applications and endpoints.

"your management path should be app then it wouldn't skip anything else" :
Unfortunately, you can't: management.endpoints.web.base-path has to start with a / so it would be management.endpoints.web.base-path=/app
which resolve to: http://localhost:8080/app/app/info instead of http://localhost:8080/app/info ... :/

The problematic bits is that webEndpointProperties.getBasePath()  resolves to an empty string when you define management.endpoints.web.base-path=/ so you end up skipping your complete servlet-path.
IMO, a way to circumvent this would be, instead of defining a static list of possible endpoints, to retrieve all the exposed endpoints and add them to the base-path.
In this case, this would result in: /app/info, /app/health, etc where app is the servlet-context path and the endpoint name was added to an empty string.

fixed via https://github.com/spring-cloud/spring-cloud-sleuth/pull/1201#issuecomment-460665940


Extremely slow load and run

Issue Report Checklist

[x ] Searched the issues page for similar reports
[x ] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[x ] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[x] Tried basic troubleshooting (if a bug/error)

[x ] Restarted Spyder
[ x] Reset preferences with spyder --reset
[x ] Reinstalled the latest version of Anaconda
[x ] Tried the other applicable steps from the Troubleshooting Guide

[x ] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
Fresh install of Anaconda on a new computer, not used once.
On launching Spyder, the logo is displayed for around 3 minutes before the IDE loads
Next it spent 10 or so minutes attempting to connect to the kernel. I tried killing it, reloading, killing the console, killing it and running "spyder --reset" in an anaconda prompt, none made a difference. Eventually it stopped on it's own.
Then I wrote:
print("hello world")
and ran it. It took 7 seconds to execute. Subsequent runs also take 7 seconds to print "hello world."
"spyder --reset" in an anaconda prompt didn't help
"Check for updates" is disabled
I unchecked "Use a single instance" (under Preferences > General > Advanced Settings) [FYI: the Troubleshooting Guide omits that is found under the "Tools" menu.]
I uninstalled and reinstalled for all users.
I updated conda
Updated anaconda
updated spyder
Internal console:
WARNING:traitlets:kernel died: 18.000919103622437
WARNING:traitlets:kernel died: 18.000887155532837
This seems like a good IDE but so far it is basically unusable even after hours of debugging.
What steps reproduce the problem?

Open Spyder
print("hello world")

What is the expected output? What do you see instead?
Response an "hello world" in under 10 minutes, at the very lease some sort of feedback or message indicating why it takes 10 minutes from start to printing "hello world". Extremely long load time and run time.
Paste Traceback/Error Below (if applicable)

PASTE TRACEBACK HERE

Versions

Spyder version: 3.3.1
Python version: 3.6.5
Qt version: 5.9.5
PyQt version: 5.9.2
Operating System name/version: Windows 10 64-bit

3.6GHz x 4 i7-7700 32 GB RAM
Dependencies

PASTE DEPENDENCIES HERE

Sorry but this seems a problem particular to your system and not to Spyder, so we can't help you. I'd recommend a Windows re-installation. That's the only thing that occurs to me.


security risk - anyone can modify any operator manifest including the image

Kiali team just noticed someone altered the manifest for the Kiali operator (unbeknowst to the Kiali team and not authorized by the Kiali team).
This brings up a security issue - anyone unrelated to a project that owns the operator can create a PR and get it merged - this means they can potentially publish a rogue "image:" tag such that any cluster with Automatic updates will get a rogue/trojan operator installed as soon as the PR is merged. Or they could perhaps simply add bad links in the description text that links to compromised web pages that can hack the user's browser/machine.
There needs to be some oversight or authorization here to avoid anonymous people getting a PR merged such that it alters operator manifests unbeknowst to the owners of the operator.

@jmazzitelli This is a good point. As a follow-up we've implemented DCO according to our updated contribution guidelines - please re-open this issue if you feel further discussion is needed.


Enable GitHub Actions to run successfully on forks

Trigger workflow on any push or pull request.
Conditionally run Codecov Upload and Deploy only on spulec/moto repository.

This allows contributors to get CI results on their forked repo before
submitting PRs to moto.  Previously, forked repos could not run the
actions on any non-master branch and the master branch workflow
would fail (rightfully) on the steps that should only be run on the
main repo (and require tokens/secrets).

Codecov Report

Merging #3622 (46bdf10) into master (9e1fcac) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3622      +/-   ##
==========================================
+ Coverage   94.35%   94.37%   +0.01%
==========================================
  Files         420      420
  Lines       45892    45892
==========================================
+ Hits        43302    43311       +9
+ Misses       2590     2581       -9

Impacted Files
Coverage Δ

moto/batch/models.py
83.94% <0.00%> (+1.50%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #3622 (46bdf10) into master (9e1fcac) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3622      +/-   ##
==========================================
+ Coverage   94.35%   94.37%   +0.01%
==========================================
  Files         420      420
  Lines       45892    45892
==========================================
+ Hits        43302    43311       +9
+ Misses       2590     2581       -9

Impacted Files
Coverage Δ

moto/batch/models.py
83.94% <0.00%> (+1.50%)
:arrow_up:

Continue to review full report at Codecov.

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


YAML validation seems having trouble to pass syntax - !rel

this is valid syntax though,
city_10_prop_subs_id:
fields:
name: subscription_id
value: 1
model: tests.jobs.fuel_sdk_proxy_classes:PropertyProxy
legacy_city_10_props:
fields:
properties:
-
- !rel city_10_prop_subs_id
model: tests.jobs.fuel_sdk_proxy_classes:RowResultProxy

@nirvanaforu Could you please link to where such syntax is defined. I did some quick search on the spec, and I only found that commands starting with ! are language extensions. What language are you using?


Fixing long strings overflowing outside of conversations

Really small PR
Added a bit of CSS to stop long strings popping out of the conversation text area. overflow-wrap and word-wrap are both used as firefox only likes word-wrap, and everything else likes overflow-wrap apparently. lol browsers

 Bug: Comment overflow possible with long words etc.


Timeout error when navigating

Hey, I deployed to AWS and successfully got a page title but I got a timeout error when navigating. There is no issue running in a normal node env.

Navigation Timeout Exceeded: 30000ms exceeded

page.click(SEARCH_SELECTOR);
await page.waitForNavigation();

@etanxing  1. is you lambda able to access the site  2. check whether the "SEARCH_SELECTOR" exist


Bootstrap Modal popup component not working on click of angular formly element

I have a reuquirement in which i am using a multiselect of angular formly which also has logo for each row. On click of this logo i want to load modal popup component, i am following a Stackblitz Demo for calling modal-component in app.component on click of element.
The link of demo which i followed : Bootstrap modal Stackblitz Demo
My workaround demo which i am implenenting is as follows :  Workaround Demo Stackblitz
The error which i am getting on click of logo with the function openModal() is of undefined object.

How to i rectify this while working with angular formly?
Following is the snippet of the code:
multiselect formly component
@Component({     selector: 'formly-field-multiselect',    template:

<ng-template let-item pTemplate="item">

{{item.label}}

</ng-template>

,     })
app.component.ts in which modal component(calenderComponent) is called
fields: FormlyFieldConfig[] = [
{
key: "select",
type: "multiselect",
templateOptions: {
multiple: false,
placeholder: "Select Option",
options: [
 { label: "Select City", value: null },
{ label: "New York", value: { id: 1, name: "New York", code: "NY" } },
 { label: "Rome", value: { id: 2, name: "Rome", code: "RM" } },
{ label: "London", value: { id: 3, name: "London", code: "LDN" } },
{
 label: "Istanbul",
value: { id: 4, name: "Istanbul", code: "IST" }
 },
{ label: "Paris", value: { id: 5, name: "Paris", code: "PRS" } }
 ],
openModal() {
this.modalRef = this.modalService.show(CalenderComponent,  {
initialState: {
title: 'Modal title'
}
});
},
 }
}

Two things first in the context of function (openModal),the field defined in the fields object, which doesn't have the modalService service injected to the component, to overcome this, I used a lambda expression:
openModal: () => {
    this.modalRef = this.modalService.show(CalenderComponent, {
      initialState: {
        title: "Modal title"
      }
    });
  }

After I solve that problem, I faced another one issue stating entryComponents issue.
To overcome this, i had to add the CalenderComponent as an entrycomponent in app module file (app.module.ts):
entryComponents: [CalenderComponent]
Things work fine as expected.


Enroll Now card appears despite there being enrollments

Steps to Reproduce

Enroll in some courses. In my case, the courses were all part of a program; no standalones, and one had multiple dates (14.750x).
Navigate to the dashboard.
The program cards appear as expected.

Expected Behavior
Only the program cards and any card for a standalone course enrollment should be shown.
Actual Behavior
I got the Enroll Now card as well. Behavior found in Chrome latest on macOS and Firefox latest on macOS.
Related Sentry issues (3 events as of filing this): https://sentry.io/organizations/mit-office-of-digital-learning/issues/3548750468/?environment=rc&project=5864687&query=is%3Aunresolved
Screenshot or Screencast

Probably related Sentry issue: https://sentry.io/organizations/mit-office-of-digital-learning/issues/3548750468/?environment=rc&project=5864687&query=is%3Aunresolved (it at least appears when I hit the dashboard on rc)


minion: Remove docker swarm from the cluster

This patch completely removes docker swarm from the cluster.  Swarm
became obsolete in quilt when the custom scheduler was merged.
Furthermore, it presented somewhat of a security risk as it allowed
remote access to docker containers.

Looks good to me other than the one comment


Add alwarith99's file

add

created a branch, created a file and made a commit, and opened a pull request