Add Wait time for KIND cluster to become ready

What this PR does / why we need it:
First Testcase Fails randomly in CI at times.  So wait has been added for control plane components to come up.

/ok-to-test

/lgtm
/approve


primitives: Consider removing serde stuff

There is some debate about what benefit/costs serde's derive feature gives us. Before releasing primitives we should re-visit the topic and make a decision.
Original discussion is in https://github.com/rust-bitcoin/rust-bitcoin/pull/2991
Note especially: https://github.com/rust-bitcoin/rust-bitcoin/pull/2991#issuecomment-2226025957

To be clear:

Keep obvious impls
Fix or remove non-obvious impls
Change the remaining derives to manual impls (may use decl macros)

We should also come up with a game plan for what happens if we remove the "non-obvious impls" like those on TxOut or Transaction which are ad-hoc and defined as "whatever serde derive does with the struct as we defined it back in 2015", and then people show up saying that they have a critical dependency on this.
I think we can create a bitcoin-legacy-serde crate that re-implements these things, provides a module, and then these people can add #[serde(with = "bitcoin_legacy_serde::transaction")] etc to their existing stuff, with very little in terms of breakage.
Maybe we should do this on day one, and put this in the CHANGELOG, and tell people that if they do use the legacy thing, to maybe file an issue so we can be aware of whatever they're doing.

I think we can create a bitcoin-legacy-serde crate that re-implements these things, provides a module, and then these people can add #[serde(with = "bitcoin_legacy_serde::transaction")] etc to their existing stuff, with very little in terms of breakage.

I like this. `#[serde(remote = "...")] should do the job.

Maybe we should do this on day one

Yes, I we shouldn't release primitives with things we know we want to remove.

I had never heard of serde(remote) but this is the second time you mentioned it so I looked it up: https://serde.rs/remote-derive.html
Very nifty.

One somewhat-annoying thing is that bitcoin-legacy-serde will need to depend on primitives, and we won't be able to release it until primitives is released. So we will need to be a bit quick on the draw to do both releases at the same time

So we will need to be a bit quick on the draw to do both releases at the same time

Shouldn't be a too big of a deal to run cargo publish twice.


⚠ Bofrost.fr has degraded performance

In 5394b70, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1124 ms

Resolved: Bofrost.fr performance has improved in d27aea2 after 6 minutes.


Light property not working with FilePlayer

Using FilePlayer the light image url does not appear, but using ReactPlayer yes

Stumbled upn the same Error today using local files as url prop.
      <ReactPlayer
        className="react-player-custom"
        width={'100%'}
        height={'100%'}
        controls
        loop={false}
        url={VIDEO_DIR + video.name + VIDEO_EXT}
        light
      />

leads to Uncaught (in promise) SyntaxError: Unexpected token U in JSON at position 56     at C:\Users\proxo\Documents\Code\hochlicht\node_modules\react-player\lib\Preview.js:156
noembed response looks like this:
  "error": "no matching providers found",
  "url": "C:Usersproxo/Documents/Overwatch/videos/overwatch/afe.mp4"
}

@pcegarra The logic for light mode lives in the top level ReactPlayer component, and not in the individual imported players. You’ll have to use ReactPlayer if you want to use light mode.
@oxorp That makes sense. If you're playing a local file and using light mode, you need to pass in a poster image URL to the light prop, otherwise noembed obviously won't know anything about your local file URL.


Problems on Windows - Help! No such file or directory 2: ./node_modules/pre-commit/hook : integer expression expected: [: 127 : numeric argument required 5:

I've been pulling my hair out trying to make sense of this for someone I'm working with who is having this problem. It's breaking their workflow.
No such file or directory 2: ./node_modules/pre-commit/hook
: integer expression expected: [: 127
: numeric argument required 5:

I've tried Windows machines near me, and VMs and don't have the problem.
This happens in cmd, powershell, MinGW, WebStorm.
Any ideas?

Same problem with babun on Windows 7, I got this when I try to commit :
: No such file or directory     .git/hooks/pre-commit: line 2: ./node_modules/pre-commit/hook
: integer expression expected   .git/hooks/pre-commit: line 4: [: 127
: numeric argument required     .git/hooks/pre-commit: line 5: exit: 0

Hope we'll find a solution

Same here on Windows 10 in WSL Bash

In node_modules/pre-commit/index.js change line
this.json = require(path.join(this.root, 'package.json'));

to
this.json = require(path.join(this.root, 'package.json').replace(/\\([a-zA-Z])\\/, '$1:\\'));

This will change \c\Users\rofrol\project1\package.json to c:\Users\rofrol\project1\package.json.


Doc suggestion - Docker run

Hi, great project! I just wanted to share a quick-and-dirty docker run one liner if anyone feel a docker container is useful there as I did.
You may close this issue without changing anything if you want.
I didn't know where else to share this.
Cheers guys
# Access the working directory that you have the markdown files
# change what you require on the command below, then run it
docker run -it --name python-grip --rm \
    -p 6419:6419 \
    --env FILE=README.md \
    --env DEBUG=True \
    --env DEBUG_GRIP=True \
    --env HOST=<IP_ADDRESS> \
    -v "$(pwd)":/workspace \
    python bash -c "pip install grip && mkdir ~/.grip/ && bash -c \"echo -e \\\"DEBUG=\$DEBUG\nDEBUG_GRIP=\$DEBUG_GRIP\nHOST='\$HOST'\\\" >> ~/.grip/settings.py \" && cd workspace/ && grip \$FILE"

# access the page at localhost:6419 on your browser

Thank you @jfftonsic


reset the wrapper.startX margin when wrapping into a new page

fixes #435

just realized I also commited white space trimming.
I can omit those lines if you'd like.

One problem with this is that if you set the x position to something other than the page margin on the previous page, you might want that position to be retained on the next page rather than jumping back to the margin. Maybe only do this if it was at the margin on the previous page?

Makes sense. I can have a go, but I'm not too familiar with the code, any hint on how I could achieve that?

I'd save the old margin before adding the new page, then test if startX is equal to that margin. If so, then adjust for the new margin as you've done already.


sqlite-web: 0.3.6 -> 0.6.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for sqlite-web is: Web-based SQLite database browser
meta.homepage for sqlite-web is: https://github.com/coleifer/sqlite-web
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.6.1 with grep in /nix/store/nz12k7rg6wqywq5015xchdcm4n6jznd1-sqlite-web-0.6.1
found 0.6.1 in filename of file in /nix/store/nz12k7rg6wqywq5015xchdcm4n6jznd1-sqlite-web-0.6.1

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
sqlite-web

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/nz12k7rg6wqywq5015xchdcm4n6jznd1-sqlite-web-0.6.1 \
  --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 sqlite-web https://github.com/r-ryantm/nixpkgs/archive/c676f21e4c64e7b1ebe9423dc24f0328f864a348.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/c676f21e4c64e7b1ebe9423dc24f0328f864a348#sqlite-web

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/nz12k7rg6wqywq5015xchdcm4n6jznd1-sqlite-web-0.6.1
ls -la /nix/store/nz12k7rg6wqywq5015xchdcm4n6jznd1-sqlite-web-0.6.1/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

1 package built:

sqlite-web

Maintainer pings
cc @costrouc for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for sqlite-web is: Web-based SQLite database browser
meta.homepage for sqlite-web is: https://github.com/coleifer/sqlite-web
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.6.2 with grep in /nix/store/6phs4z9diar4i89n94w6akg2lj8varl7-sqlite-web-0.6.2
found 0.6.2 in filename of file in /nix/store/6phs4z9diar4i89n94w6akg2lj8varl7-sqlite-web-0.6.2

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
sqlite-web

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/6phs4z9diar4i89n94w6akg2lj8varl7-sqlite-web-0.6.2 \
  --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 sqlite-web https://github.com/r-ryantm/nixpkgs/archive/3590918e9dee826e0592713859317dbbaf3fc7aa.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/3590918e9dee826e0592713859317dbbaf3fc7aa#sqlite-web

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/6phs4z9diar4i89n94w6akg2lj8varl7-sqlite-web-0.6.2
ls -la /nix/store/6phs4z9diar4i89n94w6akg2lj8varl7-sqlite-web-0.6.2/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

1 package built:

sqlite-web

Maintainer pings
cc @costrouc for testing.


Better (more structured) documentation and layout

In #63 it was pointed out that finding one's way around in the library is rather hard. The reason probably is that some parts of the API simply grew over time and the docs were added, while the general vision was not described.
One needs to understand:

Some calls are low level and very much manual (register vs unregister, SigID being just dumb ID, not a handle/token and doesn't unregister on drop).
What APIs would one use for what.
How signal removal is actually meant to happen.

This definitely calls for better documentation, but maybe even for some kind of re-structuring of API calls? The register in root, for example, is a legacy of when it lived inside the crate and wasn't a re-export from signal-hook-register and it might make sense to „hide“ it somewhere a bit ‒ after all, users of signal-hook use it because of the higher level APIs (otherwise they'd just use signal-hook-registry) and are unlikely to touch these.

Just to add to this, I personally find the separation between root module and cleanup a bit unclear. Cleanup has methods for registring and unregistring signals, just like root. So when should which one be used?
Things like signal_hook::cleanup::register for example take a bit to understand (you register a signal to remove other signals? Which of the cleanup methods is used here?).
Most of the things are well documented, so you can get a picture of the library by reading through that. But ideally one shouldn't have to read and compare all functions to understand how the API is supposed to be used and composed.
The cleanup module itself also is a bit confusing. It has the functions cleanup_signal, register and unregister_signal for example. Of which all have the purpose of being used during "cleanup" (whatever that might mean? Application shutdown, or cleaning up signals?), but they all are a bit different in how exactly they do cleanup (instruct cleanup vs actually cleaning up). Then there's unregister in root, which does the exact same thing as some methods in cleanup and is also definitely used for cleaning up signals, but doesn't sit in the cleanup module.
Generally I'm not sure the cleanup module sheds any more light on the API. It seems more like an artificial separation that doesn't have a clean separation.

I think this got improved in the just released 0.3 ‒ hopefully the modules make some more sense, low level stuff like register is now in a submodule, not in the root and there's an extensive example just at the root of the documentation. I'll close this as I don't have any more specific ideas.


Fix Performance/RedundantStringChars cop error in case of implicit receiver

echo 'chars.size' | be rubocop --only Performance/RedundantStringChars --stdin bug.rb -d

Inspecting 1 file
Scanning bug.rb
An error occurred while Performance/RedundantStringChars cop was inspecting bug.rb:1:0.
undefined method `begin_pos' for nil
lib/rubocop/cop/performance/redundant_string_chars.rb:76:in `correction_range'
lib/rubocop/cop/performance/redundant_string_chars.rb:62:in `block in on_send'

Before submitting the PR make sure the following are checked:

[X] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[X] Wrote good commit messages.
[X] Commit message starts with [Fix #issue-number] (if the related issue exists).
[X] Feature branch is up-to-date with master (if not - rebase it).
[X] Squashed related commits together.
[X] Added tests.
[X] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[X] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Thanks!


DAS message tracing uses wrong tenant configurations when sending events related to responses

Description:
Publish an api in the tenant mode
Enable Message Tracing for the Tenant User
Try Invoking the api and explore the tracer.
Can Identify that, only for the inflow the message tracing has traced and for the out flow we cannot see the message tracing

By @nadeeshaan from https://wso2.org/jira/browse/APIMANAGER-4560

Invalid due to feature not available in latest release.


Fixed TLS configuration, Obvious fix

Signed-off-by: Mario Haustein<EMAIL_ADDRESS>Description
This commit disables support for SSLv2 and SSLv3. There are four configuration options in Postfix to configure the supported SSL protocols. The author of 315c8c658c190b26eed9e1136cd13748d93f45d7 added four configuration statements, but effectively changed only two options. Therefore I assume it was a typo and added the missing two options.
Check List

[x] All tests pass. See TESTING.md for details.
[ ] New functionality includes testing. (not applicable)
[ ] New functionality has been documented in the README if applicable. (not applicable)

          1 Warning

:warning:
This Pull Request is probably missing tests.

  Generated by :no_entry_sign: Danger

Released as: 6.0.3


Clarification for "Working with LINQ" tutorial

Summary
User "regal60740" (can't @ them for some reason) made some comments in the recently closed issue #7829 that I felt were pretty valid. I added a brief explanation on where the logging extension method goes and made a minor alignment change to some of the code using the LogQuery method to keep it consistent.
I'll also have to make a PR in dotnet/samples to fix the other issue they raised which highlighted some inconsistency between the markdown code snippet and the embedded one in the Optimization section
Edit:
That pull request is available here: https://github.com/dotnet/samples/pull/469

It's quite alright, @rpetrusha. I'm happy to help in the first place.


Long names without spaces spill out of tooltips

In both the left and right sidebars, long user names without a space spill out of tooltips.

They need to be line-wrapped, like the user card tooltip.

@zulipbot claim

@zulipbot claim

Yes Working on it , an error occurred due to which was not able to give updates


Move GitAuth to EE

Part of https://app.zenhub.com/workspaces/team-pesto-6127680175876a0014fc5657/issues/weaveworks/weave-gitops-enterprise/1879

What changed?
We need to make some changes to Git Auth - specifically let the user choose the url repo. This targets components present here so we're taking this opportunity to move Git Auth to EE since it's not used here.

remove applications related content from AppContext
remove Git Auth components and context from ui
updated exports for EE
removed applications from api

@weaveworks/weavegitops-core-maintainers Hey team! When you have a minute can you have a look at this PR please see if I've missed anything? We've already added the relevant api endpoint and components to EE and merged to main. Thanks Also tagging Simon @foot

@weaveworks/weavegitops-core-maintainers  Hello team, sorry this fell off my radar. Coming back to it now. Thank you for the UI approval Olga, can someone have a look at the BE files also please? Thanks!

@AlinaGoaga the backend looks fine to me! Tested the app locally, everything is working as expected. 🎉
Your PR can be merged as is, but I will create a PR with Snyk security fixes in a couple hours, when I am back. So, we can merge it after that one is merged to be on the safe side.


⚠ Bofrost.be has degraded performance

In 033f3c0, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1486 ms

Resolved: Bofrost.be performance has improved in 005e806 after 13 minutes.


Test fails with "mkdir:<PHONE_NUMBER>ABCDEF[...]: Operation not permitted

Hello,
testing bfs 2.0 fails for me on macOS 10.13.6.
./tests.sh --bfs="/opt/local/var/macports/build/_Users_rschmidt_macports_macports-ports-svn-trunk_sysutils_bfs/bfs/work/tavianator-bfs-4d98dc6/bfs -S bfs"
mkdir:<PHONE_NUMBER>ABCDEF0123456789ABCDEF0123456789ABCDEF0123456789ABCDEF0123456789ABCDEF0123456789ABCDEF0123456789ABCDEF0123456789ABCDEF0123456789ABCDEF0123456789ABCDEF0123456789ABCDEF0123456789ABCDEF0123456789ABCDEF0123456789ABCDEF0123456789ABCDEF0123456789ABCDE: Operation not permitted

Strange.  I don't have a Mac to test on locally, but the Travis tests pass on macOS 10.13.6: https://travis-ci.org/github/tavianator/bfs/jobs/735782146
Can you run these commands for me, from a git clone of bfs, to see where it's failing?
$ make all
$ bash -c "set -x; . ./tests.sh --verbose"

With some further experimentation, I see that the tests do complete when I run them "normally" as you suggested.
Originally, I was running the tests within MacPorts, where they failed. I haven't yet figured out which difference about that environment is affecting the tests here.
MacPorts sets a different (longer) value for TMPDIR, but I tried some other values for TMPDIR myself outside of MacPorts and couldn't reproduce the failure.
Here's a log of the failure within MacPorts when I modify it to run the tests with --verbose and set -x:
https://gist.github.com/ryandesign/721180045372c29b91c73d038b67badd

Does it work if you set TMPDIR=/opt/local/var/macports/build/_Users_rschmidt_macports_macports-ports-svn-trunk_sysutils_bfs/bfs/work/.tmp but run it outside of macports?
If so, it must be some other environmental difference.  Maybe a different bash, mkdir, environment variables, etc.?

Does macports use any kind of sandbox while building or running tests?  I've had issues with Homebrew for that reason.
I see macports has a "trace" feature.  Is it active?  It looks like it might not support paths longer than PATH_MAX, which is what that test is trying to make.

I'm labeling this downstream/wontfix, since it appears to be a macports limitation that's causing the issue.  If I'm wrong, please let me know and I'll reopen this.  Thanks!


Auto boxing inside foreach DenseVector[Double] when call norm function

When we call norm function for DenseVector we always have boxing/unboxing stuff.
I am not pro in scala but looks like it is because of foreach use generic approach:
override def foreach[@spec(Unit) U](fn: (V) => U)
Also the second problem would be:
val nn = f.sNorm(v); sum += nn * nn
looks like here it will be also convert to java.lang.Double
May be it is possible to simple rewrite it as normal for-loop and don't use implicit resolving sNorm function.
I spent a bit of time but not sure how to do it correctly and in generic way, and this is why I decide to  write here may be you can help.
Thanks in advance

Thanks a lot, for a quick fix. I have a look into changes, looks cool.
But I have 2  question:

DenseVectorOps also has some on method, which mostly calls and make a bigger problem:
  implicit def canNorm[T: Field]: norm.Impl[DenseVector[T], Double] = {     val f = implicitly[Field[T]]     new norm.Impl[DenseVector[T], Double] {       override def apply(v: DenseVector[T]): Double = {         import v._         var sum = 0.0         foreach(v => { val nn = f.sNorm(v); sum += nn * nn })         math.sqrt(sum)       }     }   }
2)
Changes from  v.foreach -> to simple cforRange(0 until v.length) can be a bug, because foreach used
stride, offset, what do you think?
 override def foreach[@spec(Unit) U](fn: (V) => U): Unit = {     if (stride == 1) { // ABCE stuff       cforRange(offset until (offset + length)) { j =>         fn(data(j))       }     } else {       var i = offset       cforRange(0 until length) { j =>         fn(data(i))         i += stride       }     }   }

(1)'s a good point and I'll have to fix.
(2) it's only an issue if you go through the data array. If you use apply on DenseVector (which I do in norm) it's handled there. It's a bit slower to not hit data directly, but it's more of a pain to write.


Patch/palindrome

@Vishal-Singh-1996 Please make separate commits for each algorithm and link them to the issues.


Well, this just doesn't work

Combat Plus does not work, it the table turns into a barrel with like carrot sticks and like it doesn't work...

Enable the resourcepack on your client.


Disable-deri-bsc

why are we disabling?

Deri adapter is NFT based and has no way of fetching user positions in the contract. It needs to be done by fetching all the user events, which is impossible in BSC.


Fix installation when using composer 1.7+

This avoid the following error while installing the demo application with composer 1.7.
Declaration of Symfony\Flex\ParallelDownloader::getRemoteContents($originUrl, $fileUrl, $context) should be compatible with Composer\Util\RemoteFilesystem::getRemoteContents
  ($originUrl, $fileUrl, $context, ?array &$responseHeaders = NULL)

Travis error is unrelated (security checker failure).

Thank you Fabien.


fixed intermitent failures for delayed alert fire

@marinakog, how should the conflict be resolved?

resolved merge

I was reviewing the code, had some comments. I gave my comments to Marina.


Move main to OpenSearch 3.0.0-SNAPSHOT

Signed-off-by: Andriy Redko<EMAIL_ADDRESS>Description
Move main to OpenSearch 3.0.0-SNAPSHOT
Issues Resolved
Follow up on https://github.com/opensearch-project/job-scheduler/pull/208#issuecomment-1233427519
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[X] Commits are signed per the DCO using --signoff

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

@dblock moving to 3.0.0-SNAPSHOT, thanks!

The BCW test is failing on https://github.com/opensearch-project/OpenSearch/issues/3615

I made a bit of progress in https://github.com/opensearch-project/job-scheduler/pull/242, but check out https://github.com/opensearch-project/OpenSearch/issues/3615#issuecomment-1247100028. I think we need 2.4.0 for bcw to work which doesn't seem right, should we relax that limitation and allow 3.0 to join any 2.x cluster?

Done in https://github.com/opensearch-project/job-scheduler/pull/242


Last review pass on the reference

The reference page might need a final look to confirm it reflects the current state of the extension! I particularly need to add the style options.

Have added style options to the reference in #118 118 (plus fixed a few other small typos of mine). I also expanded the EJS template you wrote (nice work on that, btw — great use of listings!) to accept a single string for possible-values, since most of the style options take a class of possible values rather than any single set. Happy for you to make any other commits necessary there on the reference or other docs!

This is another case, like the cr-section in the YAML metadata, where my initial conceptualization maybe isn't the most useful. The original plan was to document the YAML metadata and then separately document the options that you can pass to an individual cr-section, an individual sticky and an individual trigger. Under that logic, all of the cr-style stuff would get duplicated, which doesn't sound useful. I'm inclined now to put all that that under the document metadata; I think it's clear enough to write in the preamble to the refs (and also in the guide) that those options can get passed to a section using inline styling.
So that removes the need for a cr-section heading. The sticky heading seems nearly vacuous (at least for now), so that could probably be nixed. That leaves the trigger options, which might be clearer as "focus effects". One question: where to put the progress bar and the spacer? We could have an "other" section?

I think an other section probably makes sense for those!

And I agree re. just having all the document metadata together and mentioning the attributes in the preamble :)

Great. I'll go ahead and do some changing around right now.

Btw, the thought with those ejs templates: they're a bit of overkill for not, but they open up the idea of writing a project extension that is a pkgdown / roxygen2 analog. Essentially just pre-render scripts that reads the necessary information (from... the lua filter?) and writes the yaml contents for the ejs.

I'm part way through implementing this and running out of gas. My proposal: let's go ahead and merge this and then tomorrow morn (I have a big chunk of time to work on this) I'll continue work on the reorg. Likely it'll get integrated before the post goes live but if not, doesn't seem like a serious problem.
So if we merge this... should we PR into dev and make the release? (don't think it's a problem that the docs revisions come in after 1.0)

No, I don't think it's a problem if the docs are tweaked after 1.0 either 🙂
I reckon we merge this into dev, merge dev into main and the create the release from main, just in case there's any unexpected merge problem going into the latter. Not too fussed either way, though!

Sounds like a plan!


add test about permission deny

Add test about permission deny
[Description of the issue]
 #[test]
    fn test_contract_address_from_permission_denied() {
        let keypair = KeyPair::gen_keypair();
        let privkey = keypair.privkey();

        let mut executor =
            helpers::init_executor(vec![("SysConfig.checkCreateContractPermission", "true")]);

        let chain = helpers::init_chain();
        let data = helpers::generate_contract();
        let block = helpers::create_block(&executor, Address::from(0), &data, (0, 1), &privkey);
        let inchain = chain.clone();
        let txs = block.body().transactions().clone();
        let hash = txs[0].hash();
        let h = executor.get_current_height() + 1;

        let closed_block = executor.into_fsm(block.clone());
        let executed_result = executor.grow(closed_block);
        inchain.set_block_body(h, &block);
        inchain.set_db_result(&executed_result, &block);

        let receipt = chain
            .localized_receipt(hash)
            .expect("failed to get localized_receipt");
        assert_eq!(receipt.contract_address, None);
        assert_eq!(receipt.error, Some(ReceiptError::NoContractPermission));
    }

Add this test case in ci check.

#638


.git suffix

There's an extra wrinkle in the command/url-to-subdir resolution: git-cloning an URL ending in .git creates a folder with .git removed. plug.kak solves this by always removing the .git suffix.
We should probably do this too; and probably not just for bundle URL.git, but also for bundle 'command ... /*.git', because command could be git clone. If the user really needs a .git subdir in bundle/plugins, they can append  # ./subdir.git to the command.

So, essentially what you're suggesting is basically to strip ".git" from the end of the repo/command whenever bundle is called and then store the result in bundle_plugins? If so, then that raises an issue with the primary motivation behind the creation of kak-bundle - speed.
To my knowledge, you can't do this with just kakscript (please feel free to correct me if I'm wrong), so you'd have to spawn a shell to perform this logic. However, not having to spawn a shell within the bundle command is the main reason kak-bundle is faster than plug.kak. Losing that advantage would be to give up the primary reason this plugin manager exists in the first place.
In short, although I agree that stripping the *.git suffix would help remove inconsistencies between the subdirs stored in bundle_path and the values listed in bundle_plugins, I don't think this is achievable without removing kak-bundle's only real advantage over plug.kak. As a result, unless there's a way to do this in pure kakscript that I don't know of, I'm unfortunately going to have to close this as a wontfix.

Don't forget we're on the same page about zero shell calls — have you looked at kak-bundle-plug? It's written in pure kakscript, because I figured out how to loop, catch a specific error, and streq / strne in kakscript.
However, this is irrelevant here. The only code that maps plugin URL/cmd to subdirs is shell code (correct me if I'm wrong). So the stripping would only happen in shell code that is... already shell code, not kakscript.
As to the more general question of whether it is possible to do string processing in kakscript: yes, it is possible. See sel-editor. It's tricky, but it works.
Again, this is irrelevant here. You already don't have kakscript code that maps from URLs/cmds to subdirs.

You make some good points. I thought that you were referring to stripping the suffix before storing the plugin name in bundle_plugins. Fair enough; I'll reopen.

OK, I just pushed a change that hopefully fixes this. It was actually fairly simple and seems to work on my end, but feel free to reopen if the issue isn't solved.


[XLA] Extra VLOG for PTXAS calls

This PR does a few changes related to PTXAS:

print the PTXAS command line used via VLOG
It will also output the verbose output of PTXAS as already asked on line 210, but not displayed.
Add the TF_EXTRA_PTXAS_OPTIONS environment variable that allows to pass extra parameter to PTXAS.

I tag this XLA as to my knowledge this code is only used by XLA.

@nouiz  Can you please check @timshen91's comments and keep us posted. Thanks!

Will do. But it could take a few days.

Updated per instruction. I tested manually with run_hlo_modules and it behave as expected.
Note, I tested with replay_computation and now the XLA_FLAGS aren't passed correctly for it. So it doesn't behave correctly. But this isn't related to this PR.

run_hlo_module seems to replace replay_computation. As replay_computation is partially broken, why not just remove it and point to the new tools?


sync up to v1.4.0 release

sync up to the final 1.4.0 release

/cc @Tomcli
verified this on IKS. works properly.


Disable run cell when kernel discovery is in progress or force user to select a kernel

Open a notebook
the progress indicator is displayed on the kernel status
however we can still hit the run button and this causes issues
its impossible to tell what kernel was used to run the cell

This is even worse when you have other extensions that contribute to remote jupyter connections,
basically i have no idea what kernel is used when i click the run button
Suggestions:

disable run (not the best option, as some kernles from some extensions are ready to go even if jupyter extension isn't)
ensure user is forced to pick a kernel , if the user does pick a kernel,. then we should not display the progress message at this stage, instead we should display the kernel selected, After all, the user is now running a cell against a kernel (e.g. .net, julia or nodejs notebook kernel, which is very quick to load)

I believe this is fairly crucial, as i ended up in a broken state due to this. (wrong kernels running the cells)

This was already resolved by showing the active kernel name.


feat: run sealos without root privileges

This PR contains the following changes:

refactor/ (migrate) most of the subcommands relies on buildah, like build/images/rm ...
refactor imageService/clusterService/registryService
remove pkg/image module
support running sealos without root privileges, to support this feature, fuse-overlayfs lib is required

https://github.com/labring/sealos/issues/1756  This problem needs to be tested, which depends on the binary buildah.


support/datastore: Add CRC32C validation when exporting and consuming tx meta files

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
services/friendbot, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.
[ ] I've updated any docs (developer docs, .md
files, etc... affected by this change). Take a look in the docs folder for a given service,
like this one.

Release planning

[ ] I've updated the relevant CHANGELOG (here for Horizon) if
needed with deprecations, added features, breaking changes, and DB schema changes.
[ ] I've decided if this PR requires a new major/minor version according to
semver, or if it's mainly a patch change. The PR is targeted at the next
release branch if it's not a patch change.

What
Add CRC32C validation validation when uploading and downloading tx meta files.
https://stellarorg.atlassian.net/browse/HUBBLE-271
https://stellarorg.atlassian.net/browse/HUBBLE-278
Why
From https://cloud.google.com/storage/docs/hashes-etags

There are a variety of ways that data can be corrupted while uploading to or downloading from the Cloud:

Memory errors on client or server computers, or routers along the path
Software bugs (e.g., in a library that customers use)
Changes to the source file when an upload occurs over an extended period of time

Cloud Storage supports two types of hashes you can use to check the integrity of your data: CRC32C and MD5. CRC32C is the recommended validation method for performing integrity checks.

Known limitations
Unfortunately, the CRC32 hash must be computed at the beginning of the HTTP request which means we must write out the payload to a temporary buffer instead of computing the hash on the fly.

@tamirms , I made small edit on description, I think the intention was to reference HUBBLE-278  for crc check.

@sreuland my bad, thanks for correcting the mistake!


please push update 0.17.36 to dockerhub

Please push the recent update to dockerhub or fix the update mechanism

See: https://github.com/dtandersen/docker_factorio_server/issues/229#issuecomment-488607081
When we finish the move fully the images will be updated automatically. Until then you need some patience.

0.17.36 is pushed manually


Single Column issues

I am trying to write an Excel file, but sometimes the data will contain a single column. In such scenarios, how should I proceed?
let data = "Corridas Realizadas\r\n21239\r\n";
var wb = XLSX.read(result.data, { type: "string" });
XLSX.writeFile(wb, "test.xlsx");

The code above yields the following result:

I have tried adding a sep=, to the beginning of the text, but didn't work.
Thanks in advance.

Thanks for reporting!
The overarching problem is that without a delimiter, the default is to interpret as lotus formatted text.  See https://docs.sheetjs.com/#guessing-file-type for an explanation of how plaintext formats are deduced.  Clearly that needs to be revisited.
The separator check is specifically looking for "\n" (char code 10) in the sixth character, which is clearly incorrect.  After https://github.com/SheetJS/js-xlsx/blob/master/bits/40_harb.js#L774 there should be something like:
else if(str.slice(0,4) == "sep=" && str.charCodeAt(5) == 13 && str.charCodeAt(6) == 10) { sep = str.charAt(4); str = str.slice(7); }

To disable the PRN parsing entirely, https://github.com/SheetJS/js-xlsx/blob/master/bits/40_harb.js#L848 should be changed to call dsv_to_sheet_str

Thank you! I have added data = data.split('\r\n').join(',\n'); as an intermediate step and it worked!


Add Bernhard Breytenbach's Profile

Added my own profile to the list!

Thanks @Xethron
We now have a 66.6% organiser contribution :)


Feature: Comparison between yesterday and today's data

Is it possible to show the new death count beside each state just like the new cases has been denoted using red up arrow.
And also is it possible to have a separate tab to view the yesterdays’s data to compare with today’s data.

The state deltas you're talking about has already been issued a little earlier, you'll have the status of that answered here at #316
Regarding the comparison of yesterday vs today's data, I think a timeline scrub would be a nice feature to compare data across different dates. At the moment you can do the comparison in the time series graphs below across al dates, but I'm assuming you'd like to see data changes in the table and the maps too.
This would be a nice feature.
Here's how I see it being done, the data is fetched in the home component and distributed to the map, the table and the charts. Manipulating/Slicing the data would enable us to limit what's being supplied to the other components and achieve this timeline effect.
As a start, a timeline axis can be placed at the bottom and on slicing/scrubbing can manipulate the data above for ranges of dates, or on a specific date.

@shayanpaul Daily delta of number of deceased and recovered cannot the calculated from the raw_data directly as of now. This has to be calculated in a special way at the data source level, @mohammedshafeeqet is working on this.
After this is available through the API, we can show the delta numbers for each state.

At the data source level we can put a filter of time on data (Statewise) . Data which comes between 12:00:00AM to next day 12:00:00AM can be considered that day data and that way we can calculate the delta of that data for each state.

There could be many ways through which we can collect the data:-

Make a simple google sheet form for the hospital admin staff to maintain the record. And then we can collate those results.
Maintain day wise data for each state.

Just chiming in to say that 1. and 2. (specifically) have been added. You can check the individual states' pages to compare between the last five days :)


Imp test that treasury withdrawal cannot put treasury into negative

Submitting a few TreasuryWithdrawal proposals that are by itself within the limit could put the treasury into negative balance if we didn't have appopriate checks.
We need to write a couple of unit tests that confirms this check:

Two treasury withdrawals enacted in the same epoch that by themselves are smaller than treasury amount would together put treasury into negative result. In this case only one treasury withdrawal should be enacted, until treasury amount is sufficiently large (eg topped off with a donation)
Multiple treasury withdrawals that individually are small enough, but together so large that would overflow maxBound :: Word64, still result in enactment only as many treasury withdrawals as the actual treasury balance allows

Resolved in #4189, #4201 and #4248


Allow specifying the Rust toolchain to use

In a recent zerocopy PR, we had to stop testing our nightly-only features under Kani. The reason is that those features were recently updated to work with a more recent version of the nightly Rust toolchain, and so are now incompatible with older nightlies. Since the Kani GH action selects its own toolchain, it makes that action incompatible with our nightly-only features. At the moment, we don't use Kani to test any of those features, so we simply disabled those features when using Kani. In the future, though, this could become a problem for us if we want to use Kani to test nightly-only features.
It would be great if the Kani GH action allowed specifying which Rust toolchain to use.
Here's the code in question from our Kani GH action invocation in CI:
          # Use `--features __internal_use_only_features_that_work_on_stable`
          # because the Kani GitHub Action uses its own pinned nightly
          # toolchain. Sometimes, we make changes to our nightly features for
          # more recent toolchains, and so our nightly features become
          # incompatible with the toolchain that Kani uses. By only testing
          # stable features, we ensure that this doesn't cause problems in CI.
          args: "--package zerocopy --features __internal_use_only_features_that_work_on_stable --output-format=terse --randomize-layout --memory-safety-checks --overflow-checks --undefined-function-checks --unwinding-checks"

Okay, thanks for the context! Not a big deal for us to work around it as we're currently doing. Feel free to close if you'd prefer not to leave this open :)


Add support for XML

What output do you want from xml? It's not a programming language

XML is a markup language and therefore can not be executed.


fixup! Fix empty AWS Profile issueg

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm


Extract Writer and Code Profiles into its own package

Both gossi/php-code-generator and gossi/php-code-formatter use the writer to output code. This shared code + the code profiles from formatter should go into their own package and be used respectively.
Ref #23

Done, see #54


Added docker file

What was wrong?

Ability to run multiple trinity instances from just one VM.
Gives Windows users very quick jumpstart
Ensure that trinity can run on docker
Paves way for automated testing and development frameworks

How was it fixed?
Added and tested the docker file.
Cute Animal Picture

When I run docker build . I run into the following exception.
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/virtualenv.py", line 352, in copyfile
    os.symlink(srcpath, dest)
FileExistsError: [Errno 17] File exists: '/usr/local/lib/python3.6/config-3.6m-x86_64-linux-gnu' -> '/usr/src/app/venv/lib/python3.6/config-3.6m-x86_64-linux-gnu'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/virtualenv.py", line 2343, in <module>
    main()
  File "/usr/local/lib/python3.6/site-packages/virtualenv.py", line 712, in main
    symlink=options.symlink)
  File "/usr/local/lib/python3.6/site-packages/virtualenv.py", line 927, in create_environment
    site_packages=site_packages, clear=clear, symlink=symlink))
  File "/usr/local/lib/python3.6/site-packages/virtualenv.py", line 1132, in install_python
    copyfile(join(stdlib_dir, fn), join(lib_dir, fn), symlink)
  File "/usr/local/lib/python3.6/site-packages/virtualenv.py", line 355, in copyfile
    copyfileordir(src, dest, symlink)
  File "/usr/local/lib/python3.6/site-packages/virtualenv.py", line 330, in copyfileordir
    shutil.copytree(src, dest, symlink)
  File "/usr/local/lib/python3.6/shutil.py", line 315, in copytree
    os.makedirs(dst)
  File "/usr/local/lib/python3.6/os.py", line 220, in makedirs
    mkdir(name, mode)
FileExistsError: [Errno 17] File exists: '/usr/src/app/venv/lib/python3.6/config-3.6m-x86_64-linux-gnu'

Hello Chistopher,    I think the issue is: You must have setup virtualenv on ur directory.I want to tell you that docker is a container frameworks which tries to run the said application in the directory.Much like chroot its just an container with isolated priveledges.
To run it successfully de-activate ur virtualenv and remove the venv directory from the base directory probablyinside py-evm.
On Monday, July 23, 2018, 4:31:05 PM GMT+5:30, Christoph Burgdorf<EMAIL_ADDRESS>wrote:

When I run docker build . I run into the following exception.
Traceback (most recent call last):
File "/usr/local/lib/python3.6/site-packages/virtualenv.py", line 352, in copyfile
os.symlink(srcpath, dest)
FileExistsError: [Errno 17] File exists: '/usr/local/lib/python3.6/config-3.6m-x86_64-linux-gnu' -> '/usr/src/app/venv/lib/python3.6/config-3.6m-x86_64-linux-gnu'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.6/site-packages/virtualenv.py", line 2343, in
main()
File "/usr/local/lib/python3.6/site-packages/virtualenv.py", line 712, in main
symlink=options.symlink)
File "/usr/local/lib/python3.6/site-packages/virtualenv.py", line 927, in create_environment
site_packages=site_packages, clear=clear, symlink=symlink))
File "/usr/local/lib/python3.6/site-packages/virtualenv.py", line 1132, in install_python
copyfile(join(stdlib_dir, fn), join(lib_dir, fn), symlink)
File "/usr/local/lib/python3.6/site-packages/virtualenv.py", line 355, in copyfile
copyfileordir(src, dest, symlink)
File "/usr/local/lib/python3.6/site-packages/virtualenv.py", line 330, in copyfileordir
shutil.copytree(src, dest, symlink)
File "/usr/local/lib/python3.6/shutil.py", line 315, in copytree
os.makedirs(dst)
File "/usr/local/lib/python3.6/os.py", line 220, in makedirs
mkdir(name, mode)
FileExistsError: [Errno 17] File exists: '/usr/src/app/venv/lib/python3.6/config-3.6m-x86_64-linux-gnu'
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

To run it successfully de-activate ur virtualenv and remove the venv directory from the base directory probablyinside py-evm

Thanks, that does the trick!
However, I'm running into another problem now that I'm trying to create a named image from that docker file. If I run docker build -t some-name ., it's processing just fine and I can see that trinity is starting up at some point. However, once I go and run something like docker image ls I don't see the created image.
Am I doing it wrong? Basically, I'm trying to figure out our process to create images that we can directly push to https://hub.docker.com.
Sorry if these questions turn out as docker newbie questions, I really appreciate your effort to help us getting started with docker support.

Docker seemed to have changed the way their tags work.
Your image got created checkout the relationship between your running container and the image:
docker image ls
docker container ps
There will be atleast one image id that matched to your runnning container.
You can now push the image following these steps:

Create a tag for the image that you want to push:
Eg:
docker image tag af0b2985d9a8 friendlyhello
docker image tag

login to the docker repo:
docker login

Create tag for your locally (tested) image in the docker repo:
eg: docker tag friendlyhello divyanks:py-evm
docker tag local_tag

Push the locally tagged repo
Eg:
docker push divyanks:py-evm
docker push <tag name created in step 3>

Let me know how it goes.

@divyanks I can confirm that naming an image afterwards (based on the id that shows up in docker ps) works. However, the problem is that it's hard to fully automate this step as part of our release process if the naming/tagging doesn't work that way.
The good news is: Based on this SO answer I figured out what's wrong. We have to drop the final RUN trinity or otherwise the last step (which is naming the image) won't get executed. I think we can instead simply print out something that tells the user to either type trinity or trinity --help from there.
I also noticed that if you run docker run -it <image-name> it will start into a Python REPL which is probably not what we want.
I also looked at the dockerfile that geth uses and noticed they expose a couple of ports.
https://github.com/ethereum/go-ethereum/blob/93c0f1715d1ce6e590811a659d20f0e80277ba6a/Dockerfile#L15-L16

@divyanks I pushed some changes directly into this PR. We'll squash it all into one commit of yours in the end.

@divyanks I exposed the needed ports and made trinity the entrypoint. Squashed everything in one commit. I'm currently working with the devops at the foundation to get this up at the official ethereum accoun at https://hub.docker.com/u/ethereum/
A couple more things we should do:

Add a CI job that checks if docker images can be built and run without errors
Update our release scripts to automatically cut and push new images
Update our documentation to show how to run trinity from:
a.) latest master using the dockerfile
b.) some image from https://hub.docker.com/u/ethereum/

Please feel free to help with any of these if you like to. We may leave this PR open land the docker support with all these things tackled at once.
@pipermerriam should make release-trinity cut a new docker image and push it to the docker hub right away or would you rather have that being a separate task (hint: will take several minutes)?

Good news. I think I have write access to the ethereum docker hub now (Gotta try it out in a minute). I started a new PR #1180 that will include the commit from this PR but adds docs and potentially a release script as well. Closing here.

@cburgdorf if possible, I think I'd like to keep it separate from make release-trinity and make it something like make release-trinity-dockerhub until we're confident that it runs without regular hiccups/errors/etc.  As it stands, make release-trinity is not idempotent so failures part-way-through are annoying because you have to fall back to manual things.

@pipermerriam I agree. I'm still blocked to move this forward as I still seem to be unable to create that dockerhub repo. I escalated it with the devops team again.


Proxy: Add Parent Proxy feature

Closes #2068

[x] 1. I'm unsure about the required fields in model, should they be better set to yes?
[x] 2. The templating creates a good config (in my point of view), but it's not sent via Parent, perhaps you have an idea why?

Example:
root@OPNsense:~/core # cat /usr/local/etc/squid/pre-auth/parentproxy.conf
cache_peer <IP_ADDRESS> parent 8080 0 no-query default
acl ExcludePPDomains dstdomain test.de bla.de
acl ExcludePPIPs dst <IP_ADDRESS> <IP_ADDRESS>
cache_peer_access <IP_ADDRESS> deny ExcludePPDomains
cache_peer_access <IP_ADDRESS> deny ExcludePPIPs
cache_peer_access <IP_ADDRESS> allow all
never_direct allow ExcludePPDomains
never_direct allow ExcludePPIPs
never_direct deny all

Reference:
https://forum.opnsense.org/index.php?topic=6763.msg29212#msg29212

If you need a parent proxy for testing just ping me :)


Add sql server support

Overview
Add support for SQL Server
Update type - breaking / non-breaking

[ ] Minor bug fix
[ ] Documentation improvements
[ ] Quality of Life improvements
[x] New features (non-breaking change)
[ ] New features (breaking change)
[ ] Other (non-breaking change)
[ ] Other (breaking change)
[ ] Release preparation

What does this solve?

Outstanding questions

What databases have you tested with?

[ ] Snowflake
[ ] Google BigQuery
[ ] Databricks
[ ] Spark
[ ] N/A
[x] SQL Server (2022)

I've also added CI test support. Unsure how to test that. Any help is welcome.

I've fixed the errors that came up during the checks. Linting errors should also be fixed except for the errors saying base or dummy_cte is not used and the parsing errors after the "as alias" column.


"Class 'App\User' not found" Error During Installation

Laravel Version: 5.5.9
Voyager Version: 1.0.0
PHP Version: 7
Database Driver & Version:

Description:
I receive the following error when I try to install Voyager:

I am attempting to install Voyager into an existing project with a custom namespace. As far as I can tell the project has been properly updated to use the custom namespace.
Steps To Reproduce:
Run 'composer require tcg/voyager'

Voyager is built with the expectation that it will be installed on a fresh install of Laravel (or at least one with minimal changes from the defaults).  Doing anything else may require some changes to the process.
In your case, you've moved the User model from \App to (I presume) \App\Models.  You will need to publish the voyager config file (run artisan vendor:publish then select VoyagerServiceProvider), change the user model config to the appropriate namespace, then run artisan voyager:install again.


Fix download links

Fix download links broken in 49d939.  The URL pattern I
was following from clutch was not correct.  This version
using closer.cgi actually works as intended...

@mrutkows -- ping


Mapping of CASA's coordinate frame to astropy's coordinate frame.

https://casaguides.nrao.edu/index.php/Coordinate_Frames .
This link contains CASA coordinate frames . We need to map them into astropy's coordinate frames.
Apart from 'j2000',  'icrs',  'galactic',  'supergal',   'ecliptic', 'b1950', I am not able to identify others.

We can safely ignore the other ones for now.  They're not very likely frames to use when defining regions, and I don't know yet whether astropy is capable of supporting transformations between several of them.  We should raise NotImplementedErrors when encountering the various terrestrial frames.


Proposition for a 'prefix' command

Add the possibility to add a prefix to a route.
prefix '/pref';
get '/abc' => sub { 'abc' }; # Will match '/pref/abc'
noprefix;
prefix '/b', {
  get '/abc' => sub { 'abc' }; # Will match '/b/abc'
}

Does not work with Route object (should it? ).

Thanks for the pull request. I am not having much time currently. Do you have an Idea how to apply this to the OO approach as well?

any downside to using a scope-based approach?
sub prefix(Str $name, $fn) {
  @*bailador-prefix.push: $name;
  $fn();
  @*bailador-prefix.pop;
}

prefix "/name", {
  get '/foo' ...;
};
```

I think a scope-based variable does not work because you could have more then one instance of a Bailador::App. I couldn't imagine a good place to declare this variable so it would work for more then one App.
On the other side I like the idea of having an array, it might make things simpler if you have nested routes (OO approachI) or nested prefixes.

I think the object has to be cloned with a new path attribute, but his path already contains "Start and End of string" checks ('^' and '$'). The regex becomes something like
/ ^ 'prefix' ^ 'path' $ $/

I suggest '^' and '$' modifiers to be added at the run or the match step. It does not prevent a regex already created by the end user.

Right!
In Addition I think the way nested routes work is quite different (compared to perl5 web frameworks). Maybe hey should work in a "prefix way". Maybe this Line https://github.com/ufobat/Bailador/blob/master/lib/Bailador/Route.pm#L24 should do a
$match.postmatch

something like this? https://github.com/ufobat/Bailador/commit/a248ac254c473ed1023d38729f44d9b4c0a488a1

Thans for all the discussion
jsimonet++


After update: Failed to activate the jshint package

Hi,
I am using Atom 1.0.2 and JsHint 1.3.9.
When I open atom, I immediately have an error message in a red pop up.
Here is related stack trace:
[Enter steps to reproduce below:]

Open Atom

Atom Version: 1.0.2-9432598
System: Ubuntu 15.04
Thrown From: jshint package, v1.3.9
Stack Trace
Failed to activate the jshint package
At Arguments to path.join must be strings

TypeError: Arguments to path.join must be strings
    at Object.posix.join (path.js:449:13)
    at findFile (/home/kevin/.atom/packages/jshint/load-config.js:57:66)
    at loadNpmConfig (/home/kevin/.atom/packages/jshint/load-config.js:114:12)
    at loadConfig (/home/kevin/.atom/packages/jshint/load-config.js:138:16)
    at lint (/home/kevin/.atom/packages/jshint/index.js:218:34)
    at registerEvents (/home/kevin/.atom/packages/jshint/index.js:271:2)
    at Object.plugin.activate (/home/kevin/.atom/packages/jshint/index.js:318:2)
    at Package.module.exports.Package.activateNow (/home/kevin/.local/share/atom/resources/app.asar/src/package.js:240:19)
    at /home/kevin/.local/share/atom/resources/app.asar/src/package.js:221:30
    at Package.module.exports.Package.measure (/home/kevin/.local/share/atom/resources/app.asar/src/package.js:165:15)

Commands

Config
{
  "core": {
    "disabledPackages": [
      "jsformat"
    ]
  },
  "jshint": {}
}

Installed Packages
# User
jshint, v1.3.9
linter-pep8, v1.0.0
minimap, v4.11.2
pigments, v0.8.4
project-manager, v1.15.11

# Dev
No dev packages

Thanks

Help :sob:

Open Atom.

Atom Version: 1.0.2
System: "Fedora release
Thrown From: jshint package, v1.3.10
Stack Trace
Failed to activate the jshint package
At ENOENT: no such file or directory, uv_chdir

Error: ENOENT: no such file or directory, uv_chdir
    at Error (native)
    at process.chdir (/usr/share/atom/resources/app.asar/node_modules/babel-core/node_modules/regenerator/node_modules/commoner/node_modules/graceful-fs/polyfills.js:14:9)
    at lint (/home/jctherond/.atom/packages/jshint/index.js:208:10)
    at registerEvents (/home/jctherond/.atom/packages/jshint/index.js:271:2)
    at Object.plugin.activate (/home/jctherond/.atom/packages/jshint/index.js:318:2)
    at Package.module.exports.Package.activateNow (/usr/share/atom/resources/app.asar/src/package.js:240:19)
    at /usr/share/atom/resources/app.asar/src/package.js:221:30
    at Package.module.exports.Package.measure (/usr/share/atom/resources/app.asar/src/package.js:165:15)
    at Package.module.exports.Package.activate (/usr/share/atom/resources/app.asar/src/package.js:213:14)
    at PackageManager.module.exports.PackageManager.activatePackage (/usr/share/atom/resources/app.asar/src/package-manager.js:452:21)
    at /usr/share/atom/resources/app.asar/src/package-manager.js:436:29
    at Config.module.exports.Config.transact (/usr/share/atom/resources/app.asar/src/config.js:311:16)
    at PackageManager.module.exports.PackageManager.activatePackages (/usr/share/atom/resources/app.asar/src/package-manager.js:431:19)
    at PackageManager.module.exports.PackageManager.activate (/usr/share/atom/resources/app.asar/src/package-manager.js:412:46)
    at Atom.module.exports.Atom.startEditorWindow (/usr/share/atom/resources/app.asar/src/atom.js:645:21)
    at Object.<anonymous> (/usr/share/atom/resources/app.asar/src/window-bootstrap.js:12:8)

Commands

Config
{
  "core": {
    "themes": [
      "seti-ui",
      "seti-syntax"
    ],
    "disabledPackages": [
      "pairing-chat",
      "backbonejs-snippets",
      "pairing-chat",
      "backbonejs-snippets",
      "pairing-chat",
      "node-resolver"
    ],
    "projectHome": "~/project"
  }
}

Installed Packages
# User
Stylus, v1.0.0
atom-alignment, v0.12.1
atom-backbone, v0.6.1
autocomplete-modules, v1.0.0
autocomplete-paths, v1.0.2
color-picker, v2.0.11
docblockr, v0.7.3
file-type-icons, v0.7.0
fold-functions, v0.2.1
gulp-manager, v0.2.0
gulp-snippets, v0.6.1
jshint, v1.3.10
linter, v1.2.4
minimap, v4.12.1
motepair, v0.20.0
project-manager, v1.15.11
project-view, v0.1.2
remote-edit, v1.8.14
save-session, v0.15.7
seti-syntax, v0.4.0
seti-ui, v0.7.1
svn, v0.0.7
tree-view-open-files, v0.2.4

# Dev
No dev packages

[Enter steps to reproduce below:]

...
...

Atom Version: 1.0.2
System: Ubuntu 15.04
Thrown From: jshint package, v1.3.10
Stack Trace
Failed to activate the jshint package
At ENOENT: no such file or directory, uv_chdir

Error: ENOENT: no such file or directory, uv_chdir
    at Error (native)
    at lint (/home/damon/.atom/packages/jshint/index.js:208:10)
    at registerEvents (/home/damon/.atom/packages/jshint/index.js:271:2)
    at Object.plugin.activate (/home/damon/.atom/packages/jshint/index.js:318:2)
    at Package.module.exports.Package.activateNow (/usr/share/atom/resources/app.asar/src/package.js:240:19)
    at /usr/share/atom/resources/app.asar/src/package.js:221:30
    at Package.module.exports.Package.measure (/usr/share/atom/resources/app.asar/src/package.js:165:15)
    at Package.module.exports.Package.activate (/usr/share/atom/resources/app.asar/src/package.js:213:14)
    at PackageManager.module.exports.PackageManager.activatePackage (/usr/share/atom/resources/app.asar/src/package-manager.js:452:21)
    at /usr/share/atom/resources/app.asar/src/package-manager.js:436:29

Commands

Config
{
  "core": {
    "destroyEmptyPanes": false,
    "disabledPackages": [
      "autoclose-html"
    ],
    "ignoredNames": [
      "dist"
    ]
  },
  "jshint": {
    "supportLintingJsx": true
  }
}

Installed Packages
# User
file-icons, v1.5.8
highlight-selected, v0.10.1
jshint, v1.3.10
project-manager, v1.15.11

# Dev
No dev packages

Similar to the above scenarios happens to me.


Provide a sample Open from Exsiting Code option

[Apply a sample code environment that benefits from the Open from Existing Code option. One example is to create a OpenCV project from another existing code.]

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

ID: 3925d061-52dd-b10e-50f8-a408292d7278
Version Independent ID: 99a1184b-b28e-4d89-e4ff-b57240494ff0
Content: How to: Create a C++ Project from Existing Code
Content Source: docs/ide/how-to-create-a-cpp-project-from-existing-code.md
Product: visual-cpp
GitHub Login: @mikeblome
Microsoft Alias: mblome

@jecalderon I agree we could use an example here. I have added it to our backlog.

Closing because the issue is tracked on our backlog and will be prioritized against other work.


How to build package for mulitple versions of Visual Studio / multiple toolsets?

Is it possible to build e.g. boost for multiple toolsets (140 and 141) with vcpkg?
(If 140 and 141 is a bad example because they are compatible, think of 141 and 150)
Perhaps the triplet could be expanded to include toolset as well (with empty toolset => latest being the default)?

The merge request #1098 adds support for pretty much exactly what you want as a side effect to supporting Fortran.
You can specify the toolset version via VCPKG_PLATFORM_TOOLSET in the triplet. If it is empty, vcpkg will do at it does now and just takes the latest VS version it can find.

Wow, that's great!
Could you clarify with an example of compiling a package foo for both vs2015 and vs2017 (when the feature is merged)?

I would suggest to create a triplet (e.g. x64-windows-v140) with the content of x64-windows and additionally the line
VCPKG_PLATFORM_TOOLSET=v140

and a second one (e.g. x64-windows-v141) with
VCPKG_PLATFORM_TOOLSET=v141

Then you can use
vcpkg install boost:x64-windows-v140 boost:x64-windows-v141

to build boost with both, VS2015 and VS2017.
Some additional remarks regarding the pull request: Since the PR is work in progress and tries to solve a more complicated problem (Fortran support) I am not sure when it will be ready to be merged. Maybe it makes sense to extract the changes that allow setting a toolset version explicitly into an other PR which can be merged without that much effect on other parts. @ras0219-msft what do you think?

I totally agree with splitting this into two separate PRs! It has been a bit since I looked at the code changes, but I recall the toolset option support being pretty much good to go.
The triplet suggestion is also exactly as I would recommend handling the toolset difference (though the exact syntax would be set(VCPKG_PLATFORM_TOOLSET v140)).

We'll get an explosion of triplets sooner or later ;). Each feature that's added to them doubles the number of potential "triplets".

Yep, there's an explosion of possible triplets. This is just the wonderful/terrible nature of the beast that is C++!
However, that doesn't mean we must add all of them by default! It's important to be capable of handling these scenarios, but it's also important to clearly indicate where the path ends and the forest begins!
🌲 🌲 🌲
For this specific case, we'll continue with what we do today: pick the latest of VS2015 or VS2017. In some future where there's another abi break, we'll need to add a new set of triplets for that.

простите, а вынести в командную строку можно данный параметр?
или это возможно только вносить через скрипт cmake

Sorry, but the command line can be put in this option?
or is it only possible to make cmake via script

C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V140\Platforms\Win32\PlatformToolsets>dir
LLVM-vs2014
LLVM-vs2014_xp
v140
v140_clang_3_7
v140_clang_c2
v140_xp
WindowsApplicationForDrivers10.0
WindowsKernelModeDriver10.0
WindowsUserModeDriver10.0

простите, а вынести в командную строку можно данный параметр?
или это возможно только вносить через скрипт cmake

@Voskrese I'm not totally sure what you mean by that, but many of those toolsets are slightly different MSBuild wrappers around a small set of true compiler versions (which may have different default compiler macros/flags).
We've recently added support for specifying compiler flags in the triplet file via https://github.com/Microsoft/vcpkg/commit/0c70ca21940f48e4129b677af544b52379b3c232, which may help here.
As for using clang, we don't currently support it though we'd be open to PR's that make it work! The trouble would be figuring out how to pipe it through into all the various library buildsystems. Most might react to a CXX environment variable.

ras0219-msft, thanks for the reply, I have already realized) only version 14 and 15 are fully compatible
do not judge strictly, due to ignorance of many things)


Unable to download VC with the UIN which is created in Arabic language.

Describe the bug
Unable to download VC with the UIN which is created in Arabic language.
To Reproduce
prerequisite : have a UIN created with English and Arabic.

Open app and head to home page
Click on add ID
Enter a valid UIN and click on generate VID
Enter the OTP

Current behavior
Unable to download VC with the UIN which is created in Arabic language. Displayed "Template exceptionServiceError"
Expected behavior
We should be able to download VC even with the UIN that is created in Arabic language.
Screenshots

Smartphone (please complete the following information):

Device: iPhone7 & iPhone XR
OS: 15.4.1
Inji app version: 0.4.0 (16) in test flight
MOSIP Version: 1.2.1
MOSIP server: qa-121.mosip.net

Additional context
Issue in Android is registered https://github.com/mosip/inji/issues/196

@Anushree09-N is the VC generation happening from the API?

@rakhimosip The issue is from API

Non-RTL language VC download is working

@Anushree09-N , from the ID Repo the preferredLang seems to be not having the languageCode (ara) but the language name itself like (Arabic). It needs correction from registration client to send the language code only in preferred language attribute instead of language name.

@MonobikashDas , thanks, I got it. We already started working on fixing this. Will update you once done.

verified as fixed in the build with commit id in the below screenshot.


docs: Add upgrading doc

Add a document explaining how to upgrade a system:

Running Clear Containers.
Already installed with Kata Containers binaries.

Fixes #78.
Signed-off-by: James O. D. Hunt<EMAIL_ADDRESS>

@egernst, @jcvenegas, @klynnrif - ptal.
@bergwolf, @gnawux, @laijs - As the summary shows, this is (mostly) Clear Containers specific. I am happy to add runv details if you can supply me with details ;) Alternatively, do you want to add those details in a follow-up PR?

There is simply no such thing as upgrading from runv. kata is a fresh installation and it can even run together with runv without affecting each other.

Hi @bergwolf - I thought that would be the case. Are you happy for this doc to land and then potentially for someone to raise a follow-on PR to update this doc explaining the runv approach?

@jodh-intel yes, that's fine with me. I'll raise a PR to add the runv approach after this is landed.

Great - thanks @bergwolf.

@jodh-intel  looks good - I'll be giving a try today to make sure it works in my clear-containers dev machine.

Assuming someone (@jcvenegas ) tests/verifies, the text and plan looks great to me.

Thanks @klynnrif - branch updated.


devbox: Add delve

TODO:

[ ] Fix it, it does not work with current NIX_HARDENING_ENABLE=fortify fortify3 stackprotector pic strictoverflow format relro bindnow, must remove fortify

Delve is not working. Relevant discussion: https://discourse.nixos.org/t/golang-delve-debugger-with-nix-flake-issue/22740

I don't think that this is possible with Devbox so I'm probably going to have to adopt flake early on.


Fix unified header related issues:

included OpenSLES.h BEFORE OpenSLES_Android.h
OpenSLES.h inclusion is removed from OpenSLES_Abndroid.h
added stdlib.h/cstdlib concerning malloc for sample
endless tunnel
native-activity
teapots derivative samples

Related info:
https://android.googlesource.com/platform/ndk.git/+/ndk-r14-release/docs/UnifiedHeaders.md
@hak @bbilodeau @qchong @DanAlbert please help take a look, thanks!

LGTM


Plasmid reads

I would like to contribute to this effort, but I want to make sure that my methods are sound. I would love feedback and insight.
I think I can create a toy dataset for some plasmids containing AMR genes.
Here's my current plan:

Assemble nanopore reads into genome with flye (or do hybrid assembly with unicycler) to create a closed genome

I'd be using existing assemblies, which are mostly Citrobacter and Acinetobacter, but there are some other organisms I could look into if needed

Use minimap2 to map nanopore reads to the assembled genome
Separate nanopore reads by plasmid
Ensure that nanopore read subset re-assembles into a similar plasmid using flye (and perhaps other assemblers like raven?)
Ensure that the nanopore fastq.gz files are "small enough" for github
Use minimap2 to map illumina reads to the assembled genome
Separate illumina reads by plasmid
Ensure that the nanopore + illumina read subset still assembles with unicycler
Ensure that the illumina fastq.gz files are "small enough" for github
Add resultant files to this repo via a PR

This is interesting, thank you!  I think any contribution would be appreciated. I haven't updated the spec yet where we would host the datasets yet but I will brainstorm more. For now, I think a dataset with accessions and perhaps AMR results would be the most helpful. Let me know!

Don't thank me just yet.
I've attached a file that may be helpful to you.
There are six columns in this file that designate

Organism: predicted organism (many of which have changed since submission)
ID : ARLN ID of the isolate in case googling is needed
-Illumina SRA: SRA accession of paired-end Illumina reads
nanopore SRA: SRA accession of nanopore reads
Accessions: NCBI genomes accessions of chromosome and plasmids

These are listed from initial accession to last accession (for example: CP118189-CP118194 actually means CP118189, CP118190, CP118191, CP118192, CP118193, and CP118194)

AMR: potential AMR gene located in sequence

There are some caveats to this file. This file may contain assemblies or SRA accessions that are not, yet, publicly available. Also, some of these isolates may have their AMR gene on their chromosome as opposed to a plasmid. I wanted to vet these problems first, but I do not think that I'll have the time for that for awhile.
I may come back and edit filter this information in the future, but it's here if it will start being useful.
LR Seq of ARLN.csv

Hi @erinyoung,
Just came across this issue while I was looking for some more benchmarking datasets for my tool Plassembler which implements a good chunk of what you outline :) It doesn't go to the individual plasmid level though.
It's still a work in progress for now, but just thought I would share. I'm going to implement a "--keep fastqs" flag now I think based on your comments so thanks for that as others may find it useful!
https://github.com/gbouras13/plassembler
George


Update README.md

Update README.md code blocks style and adding Kotlin library setup example.

 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.


API endpoint for Posting a question

What does this PR do?
A new feature (endpoint for posting a question).
Description of Task to be completed?

Writing endpoint using the MVC structure
Verification of token

How should this be manually tested?

Clone the repo using git clone https://github.com/fejiroofficial/stackOverflow-lite.git.
run git checkout ft-post-question-160037684
run npm install and npm start.
npm run migrate

To test for error

npm test
-using postman

Pull Request Test Coverage Report for Build 86

12 of 40 (30.0%)  changed or added relevant lines in 6 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.4%) to 31.871%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

models/questions.js
0
1
0.0%

controllers/user/signup.js
0
2
0.0%

controllers/postQuestions.js
1
11
9.09%

middlewares/verifyToken.js
8
23
34.78%

Files with Coverage Reduction
New Missed Lines
%

controllers/getSingleQuestion.js
3
22.22%

Totals

Change from base Build 84:
-0.4%

Covered Lines:
103

Relevant Lines:
244

💛  - Coveralls


Hsl regex improvements

Many thanks for creating this great library Lokesh :)
I've created a test class for the Hsl colorspace model, and some tests failed due to bugs in the Hsl::regex() method. You can see the failures here.
The hsl regex should allow for whitespace around the elements, because that is valid CSS:
hsl(   120,   45%,   90%)
hsl(   120   ,   45%   ,   90%   )
hsl(120   ,45%   ,90%   )

Also, the regex should not allow invalid values for the elements. Currently, the regex accepts this invalid string:
hsl(56,invalid,invalid)

I've improved Hsl::regex() and fixed these bugs.

@michaelhogg Thanks for you valuable contribution! I am gonna merge this in my more advanced upcoming sodium library. :)


Remove extra angle bracket from polymorphic primitive basic example

Removes an extra angle bracket from the polymorphic primitive basic example, which causes a typescript error when copy-pasted directly into an editor.
This pull request:

[ ] Fixes a bug
[ ] Adds additional features/functionality
[x] Updates documentation or example code
[ ] Other

Thanks! :D


support live diff preview at the level of model abstraction

Feature
provide live diff preview, so that users will see what changes will be made before actually applying. And the preview should be displayed at the level of model abstraction to avoid the cognitive burden to be aware of infra-oriented details.
Design
The live diff preview can be triggered by 2 entries:

Command Palette -> select Kusion: Preview Live Diff and Apply
Right-click on a stack -> click Preview Live Diff and Apply

users can:

see side-to-side diff view with different values highlighted(green for added, red for deleted)
select or fill in the dynamic parameters for local configurations to digest
sync runtime configurations to local files by clicking the sync button on the diff lines
check the diffs, component by component
navigate through diff blocks up and down with current diff blocks highlighted

Due to configuration drift, only displaying the stack configuration diff through the kusion state file or the git state cannot reflect the real situation. Therefore, additional means are needed to configure the mapping of the stack configuration and the cluster's real k8s resources. This configuration can usually be obtained in YAML format or directly written in KCL code through the KCL API. The following describes the approximate format written in KCL configuration.
schema Resource:
     @info(mapping_to = "spec.template.spec.containers[0].resources.requests.cpu")
     cpu: int

schema Server:
     @info(mapping_to = "spec.template.spec.containers[0].resources.requests")
     resource: Resource

Reference:

https://docx.guku.io/docs/diff

support live diff preview at raw k8s level: #50 #52


[Auth] Front End Auth Verification

Make sure to validate user cookies as valid when accessed in the front end

Moved to Jira


Added sns events and event publisher classes

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


WIP for support selecting tag keys

A potential solution for #4630. Allows the @ sigil to be used to refer to a tag specifically.

@jwilder @mark-rushakoff a prototype if we decide on an alternative to #6410. This allows the following query to work:
SELECT value, host, @host FROM cpu

It will select both the field and the tag. The awkward part is that the columns conflict with each other and have to be implicitly renamed, but this isn't something we should encounter too often. This change brings the query language in line with treating tags and fields as fundamentally different. If we were at an earlier time (and maybe for 2.0), I would propose that we don't allow host to ever refer to a field and require @host to refer to a tag. We're not, so both syntaxes will have to be supported with @host referring specifically to a tag.
A possible benefit is we can detect a common error where people try to select only a tag. We can't currently detect that error since the query engine isn't sure if only tags are being referenced except when creating the cursors.

There's still the issue of how to select all the fields. The problem is that SELECT * will also select the tags.

@pauldix should the wildcard only select fields or select fields and tags? The implementation in this PR will select both so this:
SELECT * FROM cpu

will expand to
SELECT host, @host, value FROM cpu

@jsternberg took my discussion over to the big issue you opened :)


Update resource-binding.md "Cost" documentation

I believe an amendment to the "Cost" section of the resource-binding would be helpful. While AWS SSM is free to use, decrypting the secrets in AWS KMS is not free forever, the free tier has 20,000 API interactions per month and costs $0.03 per 10,000 subsequent API calls. I think this might be hit a lot sooner than many might think, I have been working on a Remix app and hit 85% of this limit with 3 secrets when developing this (lots of reloads with file saves etc). The Remix construct also does not have the api for Config.bind in order to cache subsequent requests with a warm lambda. This limit can probably be reduced but I believe other will incur costs in production that might otherwise not be anticipated.

Thanks for this! Edited and added to the new docs.


PINPAD VALIDATOR

About PinPad validator

I have created a pin-pad validator, if u enter wrong pin  it will show "INCORRECT", else it will show "SUCCESS" .
The correct pin is in the files section (js file) :)

Closes: #497
Describe the changes you've made
there are three files, one for HTML where all the base elements are present, one for CSS to make the project eye-catching, and a JS file to make the elements functioning.
Describe if there is any unusual behavior of your code
NA
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[x] New and existing unit tests pass locally with my changes.

Screenshots

Main page

If u enter an incorrect pin

If u enter correct pin

@ridsuteri  @urvashi-code1255  @991rajat , please guide me if any changes required.

@ridsuteri  sir, I have updated the script.js file and added gif please verify it.

Thank you sir :)


Migrate code analysis tools

Looks like what we were using is deprecated

have another issue open for this #8375


LICENSE information

Hi @royerloic, we are in the process of making conda-packages for napari plugins and dependencies.
I had a question regarding this package. The LICENSE file says it is MIT, but the setup.py metadata says it is BSD.
Also the LICENSE file is not currently being bundled with the tarballs.
I can open a PR to update the packaging once the licensing is clarified.
Thanks :)

@royerloic @AhmetCanSolak @JoOkuma any chance you could address this so we can complete the updating of the conda-forge recipe, thanks so much!!!

@sofroniewn @goanpeca thanks for pointing this out.


Ledger Cosmos App hangs in address confirmation screen

Linked ticket:  https://github.com/cosmos/ledger-cosmos-js/issues/9
@faboweb

If I trigger the address confirmation and the Ledger goes into screensaver mode. After it wakes up again, it is not responsive.

There has been significant changes in v1.5.1 and this issue does not seem to be reproducible in this new release. I will keep the bug open so we can review the fix once this new version is published.

Looking forward to it.
Am So., 19. Mai 2019 um 21:37 Uhr schrieb Juan Leni <
<EMAIL_ADDRESS>
There has been significant changes in v1.5.1 and this issue does not seem
to be reproducible in this new release. I will keep the bug open so we can
review the fix once this new version is published.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cosmos/ledger-cosmos/issues/138?email_source=notifications&email_token=ABMY5WPJELL5YJA2ORZUSM3PWGUAZA5CNFSM4HK7D4IKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVXJBTA#issuecomment-493785292,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABMY5WL2DZPMLBAXXMF6PDTPWGUAZANCNFSM4HK7D4IA
.

--
Greets Fabo

Significant changes went into:
https://github.com/LedgerHQ/ledger-app-cosmos/pull/8
https://github.com/LedgerHQ/ledger-app-cosmos/pull/9
Please reopen with more information if you can reproduce this issue again.


Deploy to pypi only once when using build matrix

Makes use of TravisCI build stages. Note that this feature is still
in beta.

Codecov Report

Merging #269 into dev will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##             dev    #269   +/-   ##
=====================================
  Coverage   39.1%   39.1%
=====================================
  Files         27      27
  Lines       2056    2056
  Branches     226     226
=====================================
  Hits         804     804
  Misses      1228    1228
  Partials      24      24

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 8658639...0a26dc7. Read the comment docs.

Thanks! This confused me for a while yesterday. I thought the error was incorrect versioning

Sure, no problem.
Yeah, Travis features and syntax is a bit confusing and hard to test before commiting.
Hope it works 😉


Modifiers that still need to be added

Section 7.3 of Handling Tool Manual
-Corner Region Termination (CRy)
-Corner Distance (CDy)
-Approach Linear Distance (AP_LDx)
-Retract Linear Distance (RT_LDx)
-Process Speed (PSPDx)
-Continuous Rotation Velocity (CTVx)
-Simultaneous Extended Axis Velocity (EVx%)
-Independent  Extended Axis Velocity (Ind.EVx%)
-Remote TCP motion (RTCP)
-Search Motion Option (Search [ ])
-Distance Before (DB xmm, prog)
-'POINT LOGIC' for use in TA,TB,DB statements

AP_LD, RT_LD, CD, Ind.EV, EV, RTCP, FPLIN handled in cce3328

CR handled in 717bdf3

PSPD handled in e05ef00

ctv handled in b716ec5

still need
-Search Motion Option (Search [ ])
-Distance Before (DB xmm, prog)
-'POINT LOGIC' for use in TA,TB,DB statements

Add Independent and Simultaneous motion group instructions
Independent motion
linear_move.to(p1).at(100, 'mm/s').term(100).independent(group(1))
joint_move.to(p1).at(100, '%').term(50).independent(group(3))

Independent GP
GP1 L P[1] 50mm/s CNT100
GP3 J P[1] 100% CNT50

simultaneous motion
linear_move.to(p1).at(90, 'mm/s').term(100).simultaneous(group(1))
joint_move.to(p1).at(100, '%').term(50).simultaneous(group(3))

Simultaneous GP
GP1 L P[1] 90mm/s CNT100
GP3 J P[1] 100% CNT50

Distance Before added in 2243f29


Mount unmount PR changes

Codecov Report

Merging #145 into master will decrease coverage by 5.4%.
The diff coverage is 41.25%.

@@            Coverage Diff             @@
##           master     #145      +/-   ##
==========================================
- Coverage   53.96%   48.55%   -5.41%
==========================================
  Files           4        5       +1
  Lines         391      451      +60
==========================================
+ Hits          211      219       +8
- Misses        164      207      +43
- Partials       16       25       +9

Impacted Files
Coverage Δ

dom.go
47.74% <41.25%> (-4.54%)
:arrow_down:

storeutil/storeutil.go
0% <0%> (ø)

Continue to review full report at Codecov.

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

Merged via CLI https://github.com/gopherjs/vecty/commit/55b9fb86a2c62898770f8f3a40fa57c5aeb9d53d and https://github.com/gopherjs/vecty/commit/1c8ef16bdb3029a2f8a97ddbd71feac443bd5aa0


[Need Help] I want build a docker-compose for run with locust old version 0.14.6

This is My code:
version: '3'
services:
locust-master:
image: locustio/locust:0.14.6
volumes:
- ./locust:/mnt/locust
ports:
- "8089:8089"
locust-worker:
image: locustio/locust:0.14.6
command: "--master-host=locust-master"
volumes:
- ./locust:/mnt/locust
It can't start please help me review it

Sorry, we use Gihub issues only for actual issues with Locust, and it sounds like you're issue is more related to docker compose. If you believe there's an actual issue with Locust you need to provide much more information.


Implemented diagnostic for SA1118: ParameterMustNotSpanMultipleLines

Fixes #52.

This rule should be enabled by default.

You might want to add a test for attributes


Allow to configure TLS versions and ciphers

This patch uses 4 new envs:

DRONE_TLS_MIN_VERSION minimum TLS version that is acceptable
DRONE_TLS_MAX_VERSION the maximum TLS version that is acceptable
DRONE_TLS_CURVES list of elliptic curves that will be used in an ECDHE handshake
DRONE_TLS_CIPHER_SUITES list of supported cipher suites

For example:
DRONE_TLS_MIN_VERSION="TLS12"
DRONE_TLS_CURVES="CurveP256,X25519"
DRONE_TLS_CIPHER_SUITES="ECDHE_ECDSA_WITH_AES_256_GCM_SHA384,ECDHE_RSA_WITH_AES_256_GCM_SHA384,ECDHE_ECDSA_WITH_CHACHA20_POLY1305,ECDHE_RSA_WITH_CHACHA20_POLY1305,ECDHE_ECDSA_WITH_AES_128_GCM_SHA256,ECDHE_RSA_WITH_AES_128_GCM_SHA256"

 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.

This PR has been automatically closed due to inactivity.


[Sprint] Prettier Multi-GPU CLI

Description

Run multi-gpu options without starting with torch.distributed.run or torchrun.
Equalize with netspresso-trainer or netspressor-trainer-fx and add --num-gpus options to select how many gpu(s) are used.

Checklists

[x] I would create a corresponding branch for this issue from the designated (mostly master) branch.
[x] This issue only contains a preceding agreement between project owners.

Found some ways for DDP training only with --num-gpu, but it makes code messy

should import torch.distributed.run module
Dual arg parser (one for ddp training, the other for our training script)
Should run DDP processes as python subprocess

Other trainer code examples just provide a training script for DDP training without using setuptools' entry_point


How to install movingpandas?

Hi,
I am still new with python and am a bit confused.  The trajectory.py code has a line
import movingpandas as mp
how exactly do I install movingpandas? I have downloaded a clone of the files and can't seem to install it.
Thanks for any help!

There is no packaged release of MovingPandas yet, so it cannot be installed in the same sense as a proper Python page.
trajectory.py does not contain import movingpandas as mp. Do you mean one of the ipynb notebooks? (They do contain this line.)
The easiest way to use MovingPandas right now is through the MyBinder notebook at: https://mybinder.org/v2/gh/anitagraser/movingpandas/master
Alternatively, I recommend installing Anaconda. Then you can use the environment.yml config file to tell conda how to configure a Python environment that is compatible with MovingPandas. This way you can run the included ipynb notebooks locally on your machine or start using MovingPandas in your own projects.

Hi! Thanks for the reply. If i opt to go with the creation of an environment through anaconda, what exactly do I need to do?
I have the following steps in mind:
1.) Install anaconda,
2.) copy the environments.yml file to the env directory of conda
3.) Then execute the following code in my terminal
conda env create -f environment.yml
But, do I still need to add anything to the environment.yml file downloaded from github? What other steps am I missing?
I want to create a similar plot you have in the mybinder notebook at Out[21] but with my own point data.

The following steps should do it:

Install Anaconda
Clone the movingpandas repository
In Anaconda Navigator | Environments | Import select the movingpandas environment.yml from the cloned directory:

Wait until the environment is ready, then change to the Home tab and install Jupyter notebooks into the movingpandas environment
Launch Jupyter notebooks and navigate to the cloned directory to execute them
Now you can adjust the code to use your own data

Hi,
Thanks for the tips.
I did everything from 1 to 6, but whenever I try to execute the import statements in the notebook, I get this error. I tried reading it but cannot seem to understand why it occurs
I ran these codes first in the notebook:
import urllib import os import pandas as pd import movingpandas as mp import contextily as ctx
Then this is the error message:
`---------------------------------------------------------------------------
OSError                                   Traceback (most recent call last)
 in
2 import os
3 import pandas as pd
----> 4 import movingpandas as mp
5 import contextily as ctx
~/Downloads/movingpandas-master/movingpandas/init.py in
3 """
4
----> 5 from .trajectory import *
6
7 version = '0.99.0.dev'
~/Downloads/movingpandas-master/movingpandas/trajectory.py in
4 import sys
5
----> 6 import contextily as ctx
7
8 from shapely.affinity import translate
~/anaconda3/envs/movingpandas_/lib/python3.7/site-packages/contextily/init.py in
4
5 from . import tile_providers as sources
----> 6 from .place import Place, plot_map
7 from .tile import *
8 from .plotting import add_basemap, add_attribution
~/anaconda3/envs/movingpandas_/lib/python3.7/site-packages/contextily/place.py in
4 import matplotlib.pyplot as plt
5 from warnings import warn
----> 6 from .tile import howmany, bounds2raster, bounds2img, _sm2ll, _calculate_zoom
7 from .plotting import ATTRIBUTION, INTERPOLATION, ZOOM, add_attribution
8
~/anaconda3/envs/movingpandas_/lib/python3.7/site-packages/contextily/tile.py in
10 import rasterio as rio
11 from PIL import Image
---> 12 from cartopy.io.img_tiles import _merge_tiles as merge_tiles
13 from rasterio.transform import from_origin
14 from . import tile_providers as sources
~/anaconda3/envs/movingpandas_/lib/python3.7/site-packages/cartopy/init.py in
105 # Commonly used sub-modules. Imported here to provide end-user
106 # convenience.
--> 107 import cartopy.crs
108 import cartopy.feature  # noqa: F401  (flake8 = unused import)
~/anaconda3/envs/movingpandas_/lib/python3.7/site-packages/cartopy/crs.py in
30
31 import numpy as np
---> 32 import shapely.geometry as sgeom
33 from shapely.prepared import prep
34 import six
~/anaconda3/envs/movingpandas_/lib/python3.7/site-packages/shapely/geometry/init.py in
2 """
3
----> 4 from .base import CAP_STYLE, JOIN_STYLE
5 from .geo import box, shape, asShape, mapping
6 from .point import Point, asPoint
~/anaconda3/envs/movingpandas_/lib/python3.7/site-packages/shapely/geometry/base.py in
15
16 from shapely.affinity import affine_transform
---> 17 from shapely.coords import CoordinateSequence
18 from shapely.errors import WKBReadingError, WKTReadingError
19 from shapely.ftools import wraps
~/anaconda3/envs/movingpandas_/lib/python3.7/site-packages/shapely/coords.py in
6 from ctypes import byref, c_double, c_uint
7
----> 8 from shapely.geos import lgeos
9 from shapely.topology import Validating
10
~/anaconda3/envs/movingpandas_/lib/python3.7/site-packages/shapely/geos.py in
111         _lgeos = load_dll('geos_c', fallbacks=alt_paths)
112
--> 113     free = load_dll('c').free
114     free.argtypes = [c_void_p]
115     free.restype = None
~/anaconda3/envs/movingpandas_/lib/python3.7/site-packages/shapely/geos.py in load_dll(libname, fallbacks, mode)
54         raise OSError(
55             "Could not find lib {0} or load any of its variants {1}.".format(
---> 56                 libname, fallbacks or []))
57
58 _lgeos = None
OSError: Could not find lib c or load any of its variants [].`
I am not sure what is happening. I have done all the steps including the install Jupyter from the Home tab to my movingpandas environment. Any ideas on what could be happening?

Which operating system are you using?
http://anitagraser.com
On Thu, Aug 1, 2019, 01:35 jakeeegarcia-alb<EMAIL_ADDRESS>wrote:

Hi,
Thanks for the tips.
I did everything from 1 to 6, but whenever I try to execute the import
statements in the notebook, I get this error. I tried reading it but cannot
seem to understand why it occurs
I ran these codes first in the notebook:
import urllib import os import pandas as pd import movingpandas as mp
import contextily as ctx
Then this is the error message:
`---------------------------------------------------------------------------
OSError Traceback (most recent call last)
in
2 import os
3 import pandas as pd
----> 4 import movingpandas as mp
5 import contextily as ctx
~/Downloads/movingpandas-master/movingpandas/init.py in
3 """
4
----> 5 from .trajectory import *
6
7 version = '0.99.0.dev'
~/Downloads/movingpandas-master/movingpandas/trajectory.py in
4 import sys
5
----> 6 import contextily as ctx
7
8 from shapely.affinity import translate
~/anaconda3/envs/movingpandas_/lib/python3.7/site-packages/contextily/
init.py in
4
5 from . import tile_providers as sources
----> 6 from .place import Place, plot_map
7 from .tile import *
8 from .plotting import add_basemap, add_attribution
~/anaconda3/envs/movingpandas_/lib/python3.7/site-packages/contextily/place.py
in
4 import matplotlib.pyplot as plt
5 from warnings import warn
----> 6 from .tile import howmany, bounds2raster, bounds2img, _sm2ll,
_calculate_zoom
7 from .plotting import ATTRIBUTION, INTERPOLATION, ZOOM, add_attribution
8
~/anaconda3/envs/movingpandas_/lib/python3.7/site-packages/contextily/tile.py
in
10 import rasterio as rio
11 from PIL import Image
---> 12 from cartopy.io.img_tiles import _merge_tiles as merge_tiles
13 from rasterio.transform import from_origin
14 from . import tile_providers as sources
~/anaconda3/envs/movingpandas_/lib/python3.7/site-packages/cartopy/init.py
in
105 # Commonly used sub-modules. Imported here to provide end-user
106 # convenience.
--> 107 import cartopy.crs
108 import cartopy.feature # noqa: F401 (flake8 = unused import)
~/anaconda3/envs/movingpandas_/lib/python3.7/site-packages/cartopy/crs.py
in
30
31 import numpy as np
---> 32 import shapely.geometry as sgeom
33 from shapely.prepared import prep
34 import six
~/anaconda3/envs/movingpandas_/lib/python3.7/site-packages/shapely/geometry/
init.py in
2 """
3
----> 4 from .base import CAP_STYLE, JOIN_STYLE
5 from .geo import box, shape, asShape, mapping
6 from .point import Point, asPoint
~/anaconda3/envs/movingpandas_/lib/python3.7/site-packages/shapely/geometry/base.py
in
15
16 from shapely.affinity import affine_transform
---> 17 from shapely.coords import CoordinateSequence
18 from shapely.errors import WKBReadingError, WKTReadingError
19 from shapely.ftools import wraps
~/anaconda3/envs/movingpandas_/lib/python3.7/site-packages/shapely/coords.py
in
6 from ctypes import byref, c_double, c_uint
7
----> 8 from shapely.geos import lgeos
9 from shapely.topology import Validating
10
~/anaconda3/envs/movingpandas_/lib/python3.7/site-packages/shapely/geos.py
in
111 _lgeos = load_dll('geos_c', fallbacks=alt_paths)
112
--> 113 free = load_dll('c').free
114 free.argtypes = [c_void_p]
115 free.restype = None
~/anaconda3/envs/movingpandas_/lib/python3.7/site-packages/shapely/geos.py
in load_dll(libname, fallbacks, mode)
54 raise OSError(
55 "Could not find lib {0} or load any of its variants {1}.".format(
---> 56 libname, fallbacks or []))
57
58 _lgeos = None
OSError: Could not find lib c or load any of its variants [].`
I am not sure what is happening. I have done all the steps including the
install Jupyter from the Home tab to my movingpandas environment. Any ideas
on what could be happening?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/anitagraser/movingpandas/issues/27?email_source=notifications&email_token=AAEQEMOTVP25KOQ5GXXC5STQCIOTJA5CNFSM4IHMVGGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3I3Q5A#issuecomment-517060724,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEQEMIVQFGQS2U5Y63RWFLQCIOTJANCNFSM4IHMVGGA
.