🐛 Add failsafe for detecting directory

I think there is probably a better fix using mode constants.
Getting the isDir detection wrong has a knock-on effect on the rest of the extraction and ends up throwing an EBADF error. It might be possible to do better error handling as well, however I think this fail safe would fix the majority of cases.

uses the trailing slash as a fallback for detecting directories
based on: https://github.com/Stuk/jszip/blob/master/lib/zipEntry.js#L155
yauzl usage also shows detecting directories based on a trailing slash: https://github.com/thejoshwolfe/yauzl#usage

To provide some context, we depend on extract-zip in gscan, which is in turn used inside of Ghost. We are seeing lots of these errors both on the gscan site and on our hosted platform for Ghost. Due to the volume of errors we have a tight deadline of tomorrow morning for shipping a fix.
With that in mind, it would be truly amazing if was possible to get some feedback on this quickly. This is a polite request and not a demand 😉  I can always workaround it 👍

In case anyone needs it, I have published extract-zip + this fix as a new npm package called extract-zip-fork.

@ErisDS Hi, sorry for not responding sooner and thanks for all the detailed work you put into this PR. I am sorry you had to spend so much time debugging. I haven't much time lately to maintain some of my modules and have been trying to find active collaborators to help maintain them.
I think the only thing this PR needs is a fixture with the offending zip entry and a simple test case so we can catch regressions in the future.

Hi @maxogden,
I'm sorry the broken zips I got my hands on at the time had user data in them, so I wasn't able to commit them here. I wasn't able to reproduce the steps to create a broken zip.
When testing with broken zips, I was able to see that the mode that was produced was incorrect, and so isDir was set to false. The result of this is that the logic assumes the entry is a file, and isn't able to process it.
I solved this issue by researching the zip spec & alternative implementations. The 2 alternative implementations linked both use a similar strategy to the one applied here. We have been using this fix in production for 3+ months and we are no longer seeing the EBADF error when extracting zips. In fact, we've not had any more unzipping problems at all 🎉
I know that "I tell you it works" is shite when you're trying to maintain a repository for everyone, but I'm afraid it's the best I can do 😞


Update Cloudstack E2E to use IP Manager 🫡

Description of changes:
Update Cloudstack E2E to use ipman
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/cherry-pick release-0.17


Option to use CSS-Nano over Clean-CSS

I will be ignorant in saying that the majority of new applications are using angular/react/vue/etc + Webpack + PostCSS... CSSNano is commonly used in this config and I personally would like to use 1 minifer vs needing two dependencies. Do you have any thoughts on this?

In my own use I've seen better minification using clean-css. It's more customisable with more rules (and possibly harder to use), works well with grouping charsets and media queries, and has very active development. The big thing that makes cssnano desirable is that it's part of the PostCSS ecosystem.
Personally I choose cssnano only when I want to keep a pure PostCSS setup otherwise I choose clean-css for basically 80% of my projects.


Raise test coverage to 100%

This raises the line-by-line test coverage from 98% to 100%. I tried to keep my edits limited to only the tests, with the exception of src/js/store/configureStore.ts having a block of code that looks like it exists for debugging/development.

:shipit:


Please be more clear on this vscode extension

I believe that there are some people mistaken using this extension instead of original black formatter one.
If you intend to use the name and the code has partial integrate with the original, you should write more clear instruction on detail usage.
Raise clear purpose of what this extension does and how to use it.  Not simple make people misunderstood and install the extension.  please kindly take it into consideration.  Thanks

Thank you for your suggestion. However

original black formatter one
Unfortunately, Black is a Python package and has nothing to do with VSCode.
the code has partial integrate with the original
The code is independent actually

This package was made for myself in the first place. There is no intent to mislead you. Nor do I refrain from writing about it as it has nothing to do with other extensions. Thank you in advance for your understanding.


SQLPP_PRIMARY_KEY & SQLPP_AUTO_INCREMENT is fixed

subj

Thanks!

Coverage decreased (-15.8%) to 78.972% when pulling 93211cac5f7ada9c7406afb8b9a2dc90b7cd1204 on niXman:develop into e478695891d5cc7cb8b918d5958b70b49efcf269 on rbock:develop.


[BUG][Spec] Incorrect Schema for Search sort:

Spawning off of https://github.com/opensearch-project/opensearch-js/issues/850
Please check that ticket for details.

@nhtruong hey mate, I'm looking the fix or working around here.
Thanks!

Xin Chao :D
Did you you say you wanna work on this (if so I'll assigned the ticket to you), or you're waiting for this to be fixed?

@nhtruong I’m looking for the fix, since I didn’t familiar with the source code yet.


Electra: Rename objects with prefix ExecutionLayerXXX

Rationale:
Following https://github.com/ethereum/consensus-specs/pull/3757, which renames from DepositReceipt to DepositRequest to conform with the patterns in EIP-7685, this PR aims to have further conformity by dropping the ExecutionLayer prefix from WithdrawalRequest and ConsolidationRequest.
Changes:

Rename ExecutionLayerWithdrawalRequest->WithdrawalRequest
Rename ExecutionLayerConsolidationRequest->ConsolidationRequest
Rename process_execution_layer_withdrawal_request->process_withdrawal_request
Rename process_execution_layer_consolidation_request->process_consolidation_request
Rename variable/argument execution_layer_withdrawal_request->withdrawal_request
Rename variable/argument execution_layer_consolidation_request->consolidation_request

I like this clean-up! Hope to see more support (approvals/👍) before merging it.


Add searchable_blogentries*.json to .gitignore

This PR resolves issue https://github.com/corona-warn-app/cwa-website/issues/3037 "searchable_blogentries missing in .gitignore".
It adds
searchable_blogentries*.json
to
https://github.com/corona-warn-app/cwa-website/blob/master/src/data/.gitignore
Verification

Execute npm run build
Watch the folder /src/data in VS Code and note that the files

searchable_blogentries_de.json and
searchable_blogentries.json
are temporarily shown and are greyed out

Internal Tracking ID: EXPOSUREAPP-13674

LGTM, thank you @MikeMcC399 !


track2 python for datadog

Link : https://github.com/Azure/sdk-release-request/issues/839

/azp run automation - sdk


Add the capability to update API resources persisted in the DB when new scopes are added to the api-resource-collection.xml

Description
Due to limitations in the current implementation, whenever a new scope is added to an API resource, in order to update the scope in the DB, a duplicate API resource entry should be added to the api-resource-collection.xml with the new scopes. Due to this limitation, an entry should be added to the deployment.toml when ever a new scope was added to an API resource.
We need to improve the current implementation to allow updating API resources in such scenarios without duplicate entries.

IINM, the way we currently handle this resource registration (system resources and collection) is to prevent latency during server startup, as the system resource registration happens at server startup. We may be able to improve the UX by fetching scopes of the resources and cross check, but we need to consider the server startup time, as fetching all scopes would be resource-intensive.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1261 ms

Resolved: Bofrost.at performance has improved in 9f84f9b.


Release v0.8.4

Release Checklist

[ ] OWNERS must LGTM the release proposal.
At least two for minor or major releases. At least one for a patch release.
[ ] Verify that the changelog in this issue and the CHANGELOG folder is up-to-date

[ ] Use https://github.com/kubernetes/release/tree/master/cmd/release-notes to gather notes.
Example: release-notes --org kubernetes-sigs --repo kueue --branch release-0.3 --start-sha 4a0ebe7a3c5f2775cdf5fc7d60c23225660f8702 --end-sha a51cf138afe65677f5f5c97f8f8b1bc4887f73d2 --dependencies=false --required-author=""

[ ] For major or minor releases (v$MAJ.$MIN.0), create a new release branch.

[ ] An OWNER creates a vanilla release branch with
git branch release-$MAJ.$MIN main
[ ] An OWNER pushes the new release branch with
git push release-$MAJ.$MIN

[ ] Update the release branch:

[ ] Update RELEASE_BRANCH and RELEASE_VERSION in Makefile and run make prepare-release-branch
[ ] Update the CHANGELOG
[ ] Submit a pull request with the changes:

[ ] An OWNER creates a signed tag running
git tag -s $VERSION
and inserts the changelog into the tag description.
To perform this step, you need a PGP key registered on github.
[ ] An OWNER pushes the tag with
git push upstream $VERSION

Triggers prow to build and publish a staging container image
us-central1-docker.pkg.dev/k8s-staging-images/kueue/kueue:$VERSION

[ ] An OWNER prepares a draft release

[ ] Create the draft release poiting out to the created tag.
[ ] Write the change log into the draft release.
[ ] Run
make artifacts IMAGE_REGISTRY=registry.k8s.io/kueue GIT_TAG=$VERSION
to generate the artifacts in the artifacts folder.
[ ] Upload the files in the artifacts folder to the draft release - either
via UI or gh release --repo kubernetes-sigs/kueue upload <tag> artifacts/*.

[ ] Submit a PR against k8s.io,
updating registry.k8s.io/images/k8s-staging-kueue/images.yaml to
promote the container images
to production:
[ ] Wait for the PR to be merged and verify that the image registry.k8s.io/kueue/kueue:$VERSION is available.
[ ] Publish the draft release prepared at the GitHub releases page.
Link:
[ ] Run the openvex action to generate openvex data. The action will add the file to the release artifacts.
[ ] Update the main branch :

[ ] Update RELEASE_VERSION in Makefile and run make prepare-release-branch
[ ] Release notes in the CHANGELOG
[ ] SECURITY-INSIGHTS.yaml values by running make update-security-insights GIT_TAG=$VERSION
[ ] Submit a pull request with the changes:
[ ] Cherry-pick the pull request onto the website branch

[ ] Run the SBOM action to generate the SBOM and add it to the release.
[ ] For major or minor releases, merge the main branch into the website branch to publish the updated documentation.
[ ] Send an announcement email to<EMAIL_ADDRESS>and<EMAIL_ADDRESS>with the subject [ANNOUNCE] kueue $VERSION is released.
[ ] For a major or minor release, prepare the repo for the next version:

[ ] Create an unannotated devel tag in the
main branch, on the first commit that gets merged after the release
branch has been created (presumably the README update commit above), and, push the tag:
DEVEL=v0.$(($MAJ+1)).0-devel; git tag $DEVEL main && git push $DEVEL
This ensures that the devel builds on the main branch will have a meaningful version number.
[ ] Create a milestone for the next minor release and update prow to set it automatically for new PRs:

[ ] Create the presubmits jobs for the next patch release:

Changelog
- Change, and in some scenarios fix, the status message displayed to user when a workload doesn't fit in available capacity (#3551, @gabesaba)
- Determine borrowing more accurately, allowing preempting workloads which fit in nominal quota to schedule faster (#3551, @gabesaba)

+1

/close
I consulted with @tenzen-y we are skipping the announce since the release is followed by 0.9.1 now: https://github.com/kubernetes-sigs/kueue/issues/3521.


Materialize.updateTextFields messing up my class.active logic

Hi
Line:
https://github.com/InfomediaLtd/angular2-materialize/blame/master/src/materialize-directive.ts#L94
Consider the following line of my app
<input [(ngModel)]="costprice.name"...
 <label for="name" [class.active]="costprice.name || invalidKeys['name']">Name</label>
Issue:
class.active is not set even if costprice.name value exists.
Reason:
Before I had no issue, then I added later in the template a select with materialize="material_select"  and the issue appeared. Apparently because of that materialize= that performElementUpdates is called and so updateTextFields, at the wrong moment.
I guess somehow the flow is :
(after costprice object is resolved)

Process class.active logic
ngAfterViewInit kicks in, performElementUpdates and updateTextFields are called but as the [(ngModel)]'s value is not yet filled in the input, class.active is removed by Materialize
[(ngModel)]'s value is filled

Conclusion my class.active logic is overridden.
My solution for now was to remove the line materialize-directive.ts#L94 to perform updateTextFields myself.
But I am not happy with the solution...
Is my code wrongly designed ?
Is materialize-directive.ts#L94 legitimate ?
Any idea for a better solution ?

+1


Problem when selecting all columns from a join

When running a query alike:
SELECT * FROM
(SELECT * FROM t1)
ALL LEFT JOIN
(SELECT * FROM t2)
USING join_id
The result shows only the columns from t1, if I explicitly specify columns from t2 (e.g. SELECT *, col1, col2, col3... etc) then they are shown properly, but I would have assumed the * operator to SELECT all columns from the join, rather than from the first table. Is this normal behavior ? Or is it a bug or misuse of the query language ?

This is current behaviour, please don't rely on it.
We have intention to change it to standard behaviour as soon as possible.

Fixed in version 18.12.
https://github.com/yandex/ClickHouse/blob/master/CHANGELOG.md

In requests with JOIN, the star character expands to a list of columns in all tables, in compliance with the SQL standard. You can restore the old behavior by setting asterisk_left_columns_only to 1 on the user configuration level. Winter Zhang


Fixing bugs: fetching the right data in order from putting text input (filtering)

solved with issue #64 together on Sydney's computer -- can check there.


Fix flow output

Remove deprecated --old-output-format in favor of --from-vim. Output looks alright now.
Should fix #241.

LGTM, and although the change was made not even a month ago, there were several flow releases since then.
Thanks!


Suppression of specific CVE based on CVSS value

Our implementation of DependencyCheck uses suppression files to 'quiet' issues which are considered not-applicable or low risk, based - in part - on the CVE's CVSS.  However, the CVSS ratings are subject to change based on new information or assessment.  Our suppression files currently expire, using the 'until' option, resulting in re-analysis on a regular basis.
In addition to time-bounded suppression, we would like to see CVSS-bounded suppression.
While the CVE's CVSS is below client specified threshold, the issues is suppressed.
If CVSS changes to exceed the threshold, the CVSS-bounded suppression fails and the issue is restored to the report.
If both time-bound and CVSS-bound criteria are present, the system would use an 'OR' condition - if either is violated the suppression is no longer valid.
e.g.
CVE-2022-34305 is currently under review (https://nvd.nist.gov/vuln/detail/CVE-2022-34305).  We don't currently consider this an issue requiring action - but that may change depending on the outcome of the reanalysis.  Our current plans are to review it in November, but, if the risk escalates our plans could change.
The cvss_limit in the example would be our proxy for a risk escalation.  With this setup the dependency_check logic would be suppress until 2022-11-01 AND while cvss_limit < 7.0.
<suppress until="2022-11-01" cvss_limit="7.0">
      <notes>
      <![CDATA[
      IMCR-2162 - Temporarily suppress CVE-2022-34305 as we expect an update to Tomcat 10.x
      file name: tangerine-release-<IP_ADDRESS>-5.tar.gz: tangerine-release-<IP_ADDRESS>-5.tar: catalina-ant.jar
      ]]>
      </notes>
      <cve>CVE-2022-34305</cve>
</suppress>

A remark on the specific CVE you mention: I would assume your installed Tomcat instances would not include the examples webapplication? So that issue could be fully suppressed regardless of CVE rating (after ensuring you did not copy/paste the example code to your own authentication form).

the Form authentication example in the examples web application displayed user provided data without filtering, exposing a XSS vulnerability

See the sample suppression file: https://jeremylong.github.io/DependencyCheck/general/suppression.html (line 38). Is this what you are looking for?  It might not work as expected as the suppression rules are currently or conditions...

@jeremylong Not quite what @rseeton appears to be looking for. The case to be solved is to get a new alert (to trigger re-evaluation by the security team) when a CVE gets re-analyzed to a higher CVSS rating than before, which might influence their internal assessment. At the initial CVSS rating the CVE has been judged 'accepatable risk'.
Let's assume (for the sake of argument, and with an extreme difference in CVSS vector) an issue that according to the initial vulnerability analysis was classified as a locally exploitable issue requiring authentication. After initial publication of the CVE someone finds out that the bug that was fixed could actually have been remotely exploited without authentication, so the CVE gets re-evaluated at a new CVSS score reflecting a remote, unauthenticed attack.
Based on the initial CVSS (local, authenticated) it may have been considered an acceptable risk until next release, but after the discovery that it actually could be exploited remotely and did not require authentication suddenly the team would no longer consider it suppressable until next release.
So the request is for an ANDed condition. Suppress the vulnerability if it is 'this vulnerability' and does not exceed 'CVSS of the vulnerability at the time of our assesment'
cvssBelow would suppress any issue not exceeding the cvss threshold.

Hello all,
I've had a chance to test @jeremylong 's suggestion (apologies for the delay) and @aikebah is unfortunately correct - cvssBelow suppresses all issues below a threshold, which is greedier than what we were hoping for.
This is not a critical issue - the tool works well for us, but it would be a nice enhancement.  Thanks.


AAAA + IPv6 for https://black.now.sh/

First of all, thanks for the awesome tool for the black demo.
How hard would it be to add IPv6 to the Hostname for us people who use IPv6 only? If it costs extra and you're paying for this yourself I understand.

Maybe add a donation page? (if it does - I have no idea of aws costs)

Thanks - happy to help with any IPv6 questions,
Cooper

Ah! Didn't think about this at all...
Custom domains on now require a premium plan, not worth for this imho. Might be able to run this on Heroku for free in my account, but will run out of free dyno hours(same thing eventually happened with https://yapf.now.sh).
I could throw it up on AWS with serverless, that'll probably be next to nothing.

Yeah, I might just throw your code up on one of my VPS devices too as they are dual stacked :D Many thanks for this and treat it very very very low priority :)

@cooperlees playing around with the idea of enabling this by deploying to AWS(lambda + api gateway) with serverless. No idea how much that might eventually cost, but worth the experiment for this and other things.
https://black.jpadilla.com


COSMOS-2804: Nexus SDK should be able to infer the annotations and la…

In Nexus SDK, the update method expect that the object metadata (along with the annotations and labels) be populated on the object before an update is called.
Nexus SDK should be able to infer the annotations and labels needs it and not need the app code to populate them for every update.
This commit is to Fix the above issue.

This is duplicate PR, changes is already merged in different PR


fix: properly export type declarations

Addressed<EMAIL_ADDRESS>warnings

Thanks for being so quick!


Options for authorization URL

In the same vein as the OAuth2 client the OAuth1 client should allow for additional options when constructing the Authorization Url. As per OAuth 1 RFC 5849:

Servers MAY specify additional parameters.

A second parameter for $options that would merge with the token from temp credentials would be ideal IMHO.

This functionality exists within V2 which I’m working through documenting in my spare time :)
I’m not planning on adding any new functionality to the existing version except security patches at this stage :)
Sent from my iPhone

On 13 Dec 2020, at 7:59 am, Patrick Kusebauch<EMAIL_ADDRESS>wrote:

In the same vein as the OAuth2 client the OAuth1 client should allow for additional options when constructing the Authorization Url. As per OAuth 1 RFC 5849:
Servers MAY specify additional parameters.
A second parameter for $options that would merge with the token from temp credentials would be ideal IMHO.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


how to add eth_getLogs?

please have a look at this example:
https://github.com/matter-labs/web3swift/blob/e36e767eb2de26decb0c157aeef2d023a1175076/Sources/web3swift/Web3/Web3%2BEventParser.swift#L254
If it doesn't help, please provide more details about your case.

Close for now. Feel free to comment issue if you still need help. 👍


Update vscode cask

It appears that the cask name for Visual Studio code is visual-studio-code, not vscode.

Closing, didn't see this was a duplicate of #224


SIGSEGV with Perl global variables in plugins

Steps to reproduce
rc.vim:
vim9script

perl <<EOF
my $tick = 0;
my $last = "";
sub WordCount {
    if (@_[0] == $tick) {
        return $last;
    }
    $tick = @_[0];
    my $ctnt = join ' ', $curbuf->Get(1..$curbuf->Count());
    utf8::decode($ctnt);
    my $hanzi = 0 + $ctnt =~ s/\p{Han}/ i /g;
    my $count = () = $ctnt =~ /\b\w[\w'.]*\b/g;
    $last = "$count WD";
    if ($hanzi != 0) {
        $last = $last . ", $hanzi HZ";
    }
    return $last;
}
EOF
def QAQ(): string
    return perleval("WordCount " .. b:changedtick)
enddef
au InsertLeave * QAQ()
au TextChanged * QAQ()

Start vim with vim -Nu rc.vim rand.txt, where rand.txt is a file with a lot text. Randomly insert many parentheses in the file, move around, and save the file, and vim crashes.
At first when I tried to edit markdown files (my config is at here), this issue behaves as the source code of <SNR>7_Highlight_Matching_Pair(), starting from line 103, is replaced with random content, and if insist to edit after the first error of unknown command, vim soon crashes.
vim version: 8.2.4877

I can reproduce with this rand.txt file:
xxxxxxx {x
xx xxxxx xxx xxxxx xxxxxxx{
xxxxxxxxx
xxxxxxx
xxxxxxx
xxx xxxx xxxx xxx xxxxx xxxx
xxxxxxx xxxx xxxxxxx xxxx xxxxxx xxxxxxxx xxxxx xxxxxxx xxx xxxxxxxxxx xx xxx
xxxxxxxxxxx
}
xxxxxxxxxxxxxxxxxxxxxxx  xxxx
xxxxxxxxxxxxxxx  xxxx
xxxxxxxxx  xxxxxxxx
xxxxxxxxxxxxxxxxxxxxx  xxxx
xxxxxxxxxxxxxxx  xxxx
xxx xx xxxxxxx xxxxx{
xx xxxxxxxxx
xxxxxxxxxxxxxxxxxxx  xxxx
xx xxxxxxxx
xxxxxxxxxxxxx  xxxx
xxx xxxx xxxxxxxxxxx
}
xxx xxxxx xx xx x xxx xxxx{
xxxx xxxxxxxxxx xxxxxxx  xxxxx xxxx xx xxxxx  xxx x xxxx xxxx  xx xxxx xxxx
xxxxxxxxx xxxx
xxx xxxxxxx xxxxxxx xxxxx xxx  xxxxxxx xxxxxxx xxxxxxxx xx xxxxxxxxxxx
xxx xxxxxxxxxx xxxxxx xxxx
xxxxx xxxx xxxx xxx xxxx x xxxxxx xxxx xxxxxxx xx xx xxxxxxxxx xxx
xxx xxxxxxx xx xxx xxxxx xx xx xxxx xxx
xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
xx xxxx xxxx xxx xxxxxxxxxxx xxxx xx x xxx xxxxx
xxxx xxxxx xxxx xxxx xxxx
xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
}

And starting Vim like this:
vim -Nu NORC -S /tmp/crash.vim /tmp/rand.txt

Press o to open a new line below the first one, then insert some parens (()()()...).  Quit insert mode, and do it again.    Quit insert mode, and do it again.  Usually, it crashes after 2 or 3 times of entering and quitting insert mode to write new parens.

backtrace
ubsan log
valgrind log

The valgrind log is most useful.  It shows a double free in
call_def_function().  Most likely before this something failed, was
there any error message?
The stack shows a call to invoke_popup_filter(), but I don't see a
popup in the script.  I assume this is using a plugin?
--
hundred-and-one symptoms of being an internet addict:
145.
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Oh, really sorry, it's not the right valgrind log.  I'll post a new one later.

Here is the correct log.

And here is the correct backtrace.


Unicode support in iHaskell

Describe the bug
iHaskell notebooks launched using jupyterWith yield errors when I attempt to print Unicode.
To Reproduce

Follow instructions in the jupyterWith README to set up a repository for jupyterWith and start a JupyterLab server. It does not matter if one chooses to go through steps to add JupyterLab extensions or not. Alternatively, cd into jupyterWith/example/Haskell.

Open an existing iHaskell notebook that prints non-ASCII Unicode, or start a new notebook/console that does the same (e.g. putStrLn "©"). Trying putStrLn "©"in jupyterWith/example/Haskell/test.ipynb should suffice.

You should get
<stdout>: hPutChar: invalid argument (invalid character)

In contrast, Python notebooks launched from the same JupyterLab server have no such trouble, neither does the ghci instance installed by nix system-wide, and iHaskell set up via stack on macOS 9 months ago also has no such issue.
Expected behavior
A string representation containing appropriate Unicode should be printed as output and rendered as a Unicode character.
Environment

OS name + version:

OS: NixOS 21.05 (Okapi) x86_64
Kernel: 5.14.8

Version of the code:

jupyterWith: Whatever version of jupyterWith is pulled by following the instructions in the README right now.
iHaskell/GHC: <IP_ADDRESS>/8.10.4 is what ends up being built and launched on my machine.

Additional context
For the purpose of eliminating possible alternative explanations, I've set my preferred serif/sans/mono fonts to be DejaVu Serif/Sans/Sans Mono in Firefox and Brave (a chromium variant), and in my JupyterLab preferences. (DejaVu ___ has good unicode support.) This did not change reproducibility of the bug on my end.

I cannot reproduce locally, also on NixOS and Firefox. Is this the first time you use JupyterWith? If not, try setting the commit to be sure to get the latest version.
let
  jupyter = import (builtins.fetchGit {
    url = https://github.com/tweag/jupyterWith;
    rev = "73bdac9ca036c0303fc3a487129e23f9c4ad0bcf";
  }) {};


Fix hooks directory doesn't exists (circle CI issue)

Thanks for your PR!!

This was fixed on 0.2.1. Thank you again for your help!


Please add this theme hueman

Like Themes Hexo https://github.com/ppoffice/hexo-theme-hueman

I would like it very much if this was a wish list for theme ports, and that me and others had plenty of time to do that work, but it isn't. If a theme exists for Hugo, please open up an issue and I will add it to the list.


Gorilla EOL

The gorilla framework has been declared EOL. There was talk of a fork, but it's uncertain to me the status of that.

Is there a fork?
Maybe we switch to something else? (framework, use golang stdlib, or even a different language)

There is a fork here by someone saying they are going to maintain some of them with no new features planned. https://blog.smantic.dev/posts/gorilla/

We had a small chat about the way forward and perhaps reached some sort of consensus to do things as in Nebraska (oapi-codegen and echo).
That is:

echo, https://echo.labstack.com/ https://github.com/labstack/echo
oapi-codegen

I'll ask for lazy consensus here...
if no one has any objections for a week
this is the way
we will Go.
cc @yolossn @joaquimrocha @ashu8912 @vbatts

Looks like Gorilla has new maintainers... https://gorilla.github.io/blog/2023-07-17-project-status-update/


Print Version Number on startup

Based on Issue 1654
Built a function that will print the current ARC version by fetching the enviroment variable ARC_VERSION with a fallback incase the enviroment variable isnt set yet.
ARC_VERSION is set inside the dockerfile which gets its value from the Publish ARC workflow
Works on my local windows machine, Havent confirmed on either my linux machine or K8s cluster. I will in future contributions setup a local test enviroment

Im not totally sure how the docker build and push works as i personally use buildah but it should be the same logic, as you set the enviroment variable on the runner machine and the docker daemon is hosted on the same "container" so it should inherit the enviorment variable. If not then maybe we can pass it.

It's probably worth doing https://github.com/actions-runner-controller/actions-runner-controller/issues/1161 as part of this PR or in a follow up PR

It's probably worth doing #1161 as part of this PR or in a follow up PR

I have a quick question then. Shall i put the get version function inside github.go.  I dont feel like it belongs there really, as its not really related to github per say. But it makes sense inheritance wise, that way we can have one function that both github.go calls internally and the same function that main.go calls, so we dont have two diffrent functions / code blocks. Could you make a  "design" call on this kindly

@ViktorLindgren95 Thanks a lot for your effort! The direction looks very good. Implementation-wise, I have two comments- would you mind reviewing those comments and, if that makes sense to you, make corresponding changes, so that we can merge this before 0.26.0? Thanks in advance for your help ☺

@ViktorLindgren95 Thanks a lot for your effort! The direction looks very good. Implementation-wise, I have two comments- would you mind reviewing those comments and, if that makes sense to you, make corresponding changes, so that we can merge this before 0.26.0? Thanks in advance for your help ☺

No problem im just happy to do some coding in my free time 🥰

The default didn't seem to take when I tested (built using the Makefile make docker-buildx). Additionally, the Makefile needs adjusting to support passing in a version if desired (I still expected to see the verison as dev but instead I got version": "",). We can probably move the default value being set from the Go code to being set via an ARG in the Dockerfile with a default value if no arg is passed in. The project is designed to run from the image produced from its Dockerfile so I don't see an issue with doing that.

Im on the changes, ignore my re-request

The default didn't seem to take when I tested (built using the Makefile make docker-buildx). Additionally, the Makefile needs adjusting to support passing in a version if desired (I still expected to see the verison as dev but instead I got version": "",). We can probably move the default value being set from the Go code to being set via an ARG in the Dockerfile with a default value if no arg is passed in. The project is designed to run from the image produced from its Dockerfile so I don't see an issue with doing that.

Ok i think i know what happens. If the $VERSION variable is empty, aka not set, the version becomes empty instead of dev. I just confirmed this in my local enviroment.
Because i got the version from the github actions variable its not fetched from any local instance. If you set version tag inside your own enviroment do you see a version being printed or is it still empty?
I unfortunately dont know much about makefiles but, Would the makefile create it with the dockerfile? then try setting the version env in your enviroment and confirm for me

You're going to want to use CONTROLLER_VERSION ?= dev so Make sets the var only to dev only if it hasn't been set by the user. Then pass here that var like this:
	docker buildx build --platform ${PLATFORMS} \
		--build-arg RUNNER_VERSION=${RUNNER_VERSION} \
		--build-arg DOCKER_VERSION=${DOCKER_VERSION} \
		--build-arg CONTROLLER_VERSION=${CONTROLLER_VERSION} \
		-t "${NAME}:${VERSION}" \
		-f Dockerfile \
		. ${PUSH_ARG}

then have CONTROLLER_VERSION as an ARG in the Dockerfile with a default so it's set for those not using the Makefile ARG CONTROLLER_VERSION=dev

mmmm looks like some changes are needed:

https://github.com/actions-runner-controller/actions-runner-controller/blob/master/Makefile#L7 needs to become dev instead of latest
https://github.com/actions-runner-controller/actions-runner-controller/blob/master/Makefile#L121 a --build-arg added for passing in VERSION to the Dockerfile
https://github.com/actions-runner-controller/actions-runner-controller/blob/master/Dockerfile#L28 ARG VERSION=dev
The default removed from the Go code

mmmm looks like some changes are needed:
1. https://github.com/actions-runner-controller/actions-runner-controller/blob/master/Makefile#L7 needs to become `dev` instead of latest

2. https://github.com/actions-runner-controller/actions-runner-controller/blob/master/Makefile#L121 a --build-arg added for passing in VERSION to the Dockerfile

3. https://github.com/actions-runner-controller/actions-runner-controller/blob/master/Dockerfile#L28 `VERSION=dev` added to the ARG list with a default for those not using the Makefile

4. The default removed from the Go code (build/version.go)

EDIT @ViktorLindgren95 really appreciate your work so far, we're keen to get this in the current milestone v0.26.0 so if you do find some time to make those changes sooner rather than later we'd very much appreciate it 🙏

I will try to get a push before tommorow evening. When is 0.26 deadline?
Also thanks for the patience on your end. This is all pretty new to me, the contributing to opensource and all that ^^

I will try to get a push before tommorow evening. When is 0.26 deadline?

We don't have specific deadlines for releases as the maintainers have full time jobs. There's only this + a small refactor to do and then we are ready to release so we are hoping in the next week or 2.

Updated the N/A to NA so that it looks a bit more natural in the default User-Agent string actions-runner-controller/N/A vs actions-runner-controller/NA. I don't think NA as an abbrev for Not-Available is not so common but I think it's a bit better than breaking UA string 😁
Also fixed the failing test related to the UA string generation in a4b3019

Merged. The next release of our canary image which will be available within 10 minutes or so would show canary as the version number.
It would be great if anyone could confirm it by looking into the controller logs or seeing outgoing HTTP requests or etc.


Add Swagger / OpenAPI documentation

Hi @muety
Is the aim to build declarative comments into the Go code or to develop an independent OpenAPI specification?

Yes, I'd love to have declarative comments from which OpenAPI / Swagger docs are generated automatically.

@muety Are you open to PRs for OpenAPI 3.0 compatibility?


Add basic Ubuntu support

Only what is needed if used with vsock

Just a quick note before I forget, ubuntu still has this issue where one needs to log out/log in again after the user is added to the libvirt group. Without that, the virsh commands won't be able to access qemu:///system.

Added a line for the group change on Ubuntu !

I tried with a fresh ubuntu install and vsock. Works well.

Related to https://github.com/code-ready/crc/issues/97

I've rebased this (+ added a few more changes) on top of master at https://github.com/cfergeau/crc/tree/ubuntu
Needs testing though, I've relied on the preflight unit tests for sanity checks but did not run it.


00962791526041

jast

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

ID: 54585ff0-1a67-0fd0-3169-e0cc98d45b4b
Version Independent ID: 702fd06d-fd2f-48c6-ac57-5f8a6afe84da
Content: Welcome to Remote Desktop Services in Windows Server 2016
Content Source: WindowsServerDocs/remote/remote-desktop-services/Welcome-to-rds.md
Product: windows-server-threshold
GitHub Login: @ChristianMontoya
Microsoft Alias: chrimo

Not enough info to take action.


Improve socket API by adding read action

Purpose

Add read action for the client/caller.
Remove onClose
Rename onAccept -> onConnect

New Syntax
Listener
import ballerina/io;
import ballerina/socket;

listener socket:Listener server = new(59152);

service echoServer on server {

    resource function onConnect(socket:Caller caller) {
        io:println("Join: ", caller.remotePort);
    }

    resource function onReadReady(socket:Caller caller) {
        var result = caller->read();
        if (result is (byte[], int)) {
            var (content, length) = result;
            if (length > 0) {
                _ = caller->write(content);
                io:println("Server write");
            } else {
                io:println("Client close: ", caller.remotePort);
            }
        } else {
            io:println(result);
        }
    }

    resource function onError(socket:Caller caller, error er) {
        io:println(er.detail().message);
    }
}

Codecov Report

Merging #13738 into master will decrease coverage by 0.61%.
The diff coverage is 82.09%.

@@             Coverage Diff              @@
##             master   #13738      +/-   ##
============================================
- Coverage     65.14%   64.52%   -0.62%
  Complexity      426      426
============================================
  Files          2116     2160      +44
  Lines         94178    95424    +1246
  Branches      12492    12687     +195
============================================
+ Hits          61349    61572     +223
- Misses        27844    28860    +1016
- Partials       4985     4992       +7

Impacted Files
Coverage Δ
Complexity Δ

...alang/stdlib/socket/endpoint/tcp/client/Close.java
61.11% <0%> (+3.96%)
0 <0> (ø)
:arrow_down:

...nalang/stdlib/socket/endpoint/tcp/client/Read.java
100% <100%> (ø)
0 <0> (?)

...ng/stdlib/socket/endpoint/tcp/server/Register.java
100% <100%> (+6.66%)
0 <0> (ø)
:arrow_down:

...g/ballerinalang/bre/BLangCallableUnitCallback.java
78.78% <100%> (+19.41%)
0 <0> (ø)
:arrow_down:

...tdlib/socket/endpoint/tcp/client/InitEndpoint.java
81.48% <100%> (-3.31%)
0 <0> (ø)

...alang/stdlib/socket/tcp/TCPSocketReadCallback.java
100% <100%> (ø)
0 <0> (?)

.../ballerinalang/stdlib/socket/tcp/SocketReader.java
100% <100%> (ø)
0 <0> (?)

...nalang/stdlib/socket/tcp/ReadPendingSocketMap.java
100% <100%> (ø)
0 <0> (?)

...ballerinalang/stdlib/socket/tcp/SocketService.java
72.72% <100%> (+6.06%)
0 <0> (ø)
:arrow_down:

...inalang/stdlib/socket/tcp/ReadPendingCallback.java
100% <100%> (ø)
0 <0> (?)

... and 67 more

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 105430c...2779180. Read the comment docs.


Asset Mapper not loads css files

Not able to load css files in  asset mapper

Hey!
Can you please share some more details? What does your code look like? Do you have link tags to these files from a template? If so, what is the code in that template? This is almost definitely a mistake you made (and that's ok!) but it's not possible to assist without all the details :).
Cheers!

hey
import statement are not working when using logical path in app.js
third party imports are woking fine
also when adding path in importmap.php those files are also working fine

Can you try after removing all the comments in your JS file ? It could well be related (and a fix is merged tonight about this)

It's because of the asset mapper : https://symfony.com/doc/current/frontend/asset_mapper.html#mapping-and-referencing-assets
To solve the problem, you should do :  _**php bin/console asset-map:compile when you are in prod environment.


Ping as string on influxdb

So im creating a grafana dashboard getting values from influxdb2, while i was doing some graphs on latency related i see that the ping value from influxdb is retrieved as a string instead of a number and infludb gets the value from the speedtest-tracker too as a string, is this supposed to be this way? i`m using the latest version of speedtest-tracker, influxdb2 and grafana. The values of packet_loss and ping_jitter are in numbers, so thats right only ping is in string.

Short answer, no it shouldn't be that way.
The InfluxDB integration needs a complete rewrite, likely with breaking changes to fix authentication and data quality/formats.

Ok, i saw too that you have a milestone for prometheus too, looking forward to it, i just use influxdb for the speedtest-tracker, the rest is all with prometheus, on the ping related matter i`m using a data transformation directly on grafana to convert it to a number as a workaround.


Parse Syntax Error

Hi
i've got error like this in my console log
Error: [$parse:syntax] http://errors.angularjs.org/1.2.16/$parse/syntax?p0=LdtcfUSAAAAAOBJrsfpYCrT3abxR9FzHh8Z9HhD&p1=is%20an%20unexpected%20token&p2=2&p3=6LdtcfUSAAAAAOBJrsfpYCrT3abxR9FzHh8Z9HhD&p4=LdtcfUSAAAAAOBJrsfpYCrT3abxR9FzHh8Z9HhD
    at Error (native)
    at http://ajax.googleapis.com/ajax/libs/angularjs/1.2.16/angular.min.js:6:450
    at $a.throwError (http://ajax.googleapis.com/ajax/libs/angularjs/1.2.16/angular.min.js:165:141)
    at $a.parse (http://ajax.googleapis.com/ajax/libs/angularjs/1.2.16/angular.min.js:164:6)
    at http://ajax.googleapis.com/ajax/libs/angularjs/1.2.16/angular.min.js:96:122
    at http://ajax.googleapis.com/ajax/libs/angularjs/1.2.16/angular.min.js:52:36
    at q (http://ajax.googleapis.com/ajax/libs/angularjs/1.2.16/angular.min.js:7:386)
    at J (http://ajax.googleapis.com/ajax/libs/angularjs/1.2.16/angular.min.js:51:294)
    at f (http://ajax.googleapis.com/ajax/libs/angularjs/1.2.16/angular.min.js:46:399)
    at f (http://ajax.googleapis.com/ajax/libs/angularjs/1.2.16/angular.min.js:46:416)
 angular.js:9778

my HTML

I use it with angular 1.2.16 ..any idea?
Thanks

Sorry for commenting on closed issue, but I am getting exact same error, and not able to figure out the cause & answer provided (I am sure, we are doing something silly).
What exactly @ghprod needs to do to fix this error ?
my .html is -

and controller is -
$scope.model = {
key:"xyzkey"
}
I also tried providing single quote within key value i.e. key:"'xyzkey'"

There is no need for single quotes when you are setting the value in the model as such.
If you are using an older version, you may need to wrap model.key in curlies key="{{model.key}}"
Can you verify that model.key is within the same scope as the directive?

I removed the dependency on scope/controller for key value. and made my .html simpler

my public key is starting from number, could that be problem?
I am using latest directive version of vc-recaptcha (2.2.5), and angular is 1.4.5
Error is
Error: [$parse:syntax] http://errors.angularjs.org/1.4.5/$parse/syntax?p0=LdsjhQTAAAAAK....SAa…AAAA...MYTe1kocbPrB&p4=LdsjhQT....YTe1kocbPrB
at Error (native)
at http://localhost:8080/bower_components/angular/angular.min.js:6:416
at Object.r.throwError (http://localhost:8080/bower_components/angular/angular.min.js:209:32)
at Object.r.ast (http://localhost:8080/bower_components/angular/angular.min.js:201:296)
at Object.rd.compile (http://localhost:8080/bower_components/angular/angular.min.js:210:406)
at fc.parse (http://localhost:8080/bower_components/angular/angular.min.js:236:376)
at http://localhost:8080/bower_components/angular/angular.min.js:116:396
at http://localhost:8080/bower_components/angular/angular.min.js:73:398
at n (http://localhost:8080/bower_components/angular/angular.min.js:8:3)
at Y (http://localhost:8080/bower_components/angular/angular.min.js:73:110)(anonymous function) @ angular.js:12416(anonymous function) @ angular.js:9203(anonymous function) @ angular.js:14642m.$eval @ angular.js:15878m.$digest @ angular.js:15689m.$apply @ angular.js:15986g @ angular.js:10511L @ angular.js:10683A.onload @ angular.js:10624

Some relevant code would help. For the HTML, provide as much as possible, include at least everything within the same controller scope.
For the JS, provide the full controller if possible.
Omit any sensitive information with reasonable dummy values/text as needed.

I have login Directive, and I am using angular-recaptcha inside the .html of this directive.
Here is code
'use strict';
angular.module('amodule').directive('login', [
    function() {
    return {
        restrict: "E",
        templateUrl: "common/partials/login.html",
        controller: "loginCtrl",
        scope:{
        	hideRegister: "=hideRegister" ,
        	key: "=key"
        }
    };
}]);

Login to EhrDiary

    <form name="loginForm">
      <form-mobile-input isdmodel="user.countrycode" mobilemodel="user.mobile"></form-mobile-input>

        Password

#

          <input type="password" class="form-control" id="password" placeholder="Password" required ng-model="user.password">

          <input type="checkbox" ng-model="user.rememberMe">  Remember Me

        <button type="submit" class="btn btn-default btn-raised ehr-save-btn-lg" ng-class="{'btn-success': !loginForm.$invalid}" ng-disabled="loginForm.$invalid" ng-click="login()">Login</button>
        <button type="button" class="btn btn-raised btn-danger ehr-cancel-btn-lg" ng-click="loadRegisterPage()" ng-if="!hideRegister">Register</button>

    </form>

'use strict';

angular.module('amodule').controller('loginCtrl', ['$scope', '$location', 'loginService', 'vcRecaptchaService',
  function($scope, $location, loginService, vcRecaptchaService) {

    $scope.user = {
      countrycode: 91,
      rememberMe: false
    };

    $scope.login = function() {
      var gCaptchaResponse = vcRecaptchaService.getResponse ();

      if (!gCaptchaResponse) { //if string is empty
        alert("Please resolve the captcha and login!");
        return;
      }

      loginService.login($scope.user.countrycode, $scope.user.mobile, $scope.user.password, $scope.user.rememberMe, gCaptchaResponse).then(
        function(toGoNext) {
          $location.path(toGoNext);
        },
        function(status) {
          if (status.message) {
            alert(status.message);
          } else {
            alert('Invalid mobile number or password!');
            console.log(JSON.stringify(status));
          }
        });
    };

    $scope.loadRegisterPage = function() {
      $location.url("/register");
    };

  }
]);

Is this issue closed without being solved? What I am missing is code for a working solution and also information about whether a key can start with a digit or not.

Hi!
Same error is here.
Did you figure it out?
Mudin

Things have changed a little and now, how the key is read on the directive level needs to be a model value. You'll have to set the key in scope somewhere and reference the scope variable.

You can also pass the key in the service provider during the application config function:
myApp.config(function(vcRecaptchaServiceProvider){
  vcRecaptchaServiceProvider.setSiteKey('---- YOUR PUBLIC KEY GOES HERE ----')
});

Note: the string way on adding it directly to the directive may still work, but it is dependent on the key beginning with an alpha, non-numerical character. This is because when angular see a number, it assumes it is a number and something like 6Ld... is not a number.

Hi
It is perfectly working now using both way:

and  vcRecaptchaServiceProvider.setSiteKey('---- YOUR PUBLIC KEY GOES HERE ----')
Thank you very much

I got the same error...  Can you update README with correct tutorial?


Bash Completion Quoting Error on Windows with Git Bash

Hi:
I'm running qpdf 11.2.0 on Windows 11. The binary is installed in C:\Program Files\qpdf 11.5.0. I also use Git for Windows, and prefer to use the MSYS2 bash shell that comes with it instead of CMD.EXE or PowerShell.
My .bashrc has eval "$(qpdf --completion-bash)" in it to enable bash completion. Unfortunately the output of that command does not respect whitespaces in paths, which is very common on Windows.
$ qpdf --completion-bash
complete -o bashdefault -o default -o nospace -C C:\Program Files\qpdf 11.5.0\bin\qpdf qpdf

As a result, bash completion fails with the error
bash: C:Program: command not found

Since I also share my home directory with my WSL2 Ubuntu install, and I have qpdf installed in /usr/bin there, I can use it from WSL with the same .bashrc, but not from MSYS2.

Thanks. This should be easy enough to reproduce and fix.


Remove tags from measurements/topographies

Describe the maintenance task
Tags are used for organizing data, similar to folders in a file system. Right now, surfaces and topographies can be tagged and they show up in the same tag tree. This makes the UI complicated and confusing. Topographies always belong to a Surface and those should carry metadata in form of tags.
Describe why this task is necessary
Simplifying the code base, in particular the UI.
Additional context
We should first check if anybody on the production DB had tagged topographies. If not, remove the feature. Otherwise we can think about splitting surface and topography tags into different categories.

Tags are actually being used for topographies.


Update of Volunteer Panel UI

Update of the volunteer panel to show relevant information.

Coverage decreased (-0.08%) to 71.643% when pulling 743ff0c8e1bf29ea642dd3c86886c82cd72e9a3b on afiqlattif:volunteer-ui into 28d6087224b5910afeca1f41725307748a08f0cd on CS2103-AY1819S1-W16-2:master.

Currently, VolunteerPanel is visible only when context is switched to volunteers, does not appear when SocialCare begins running.

Currently, VolunteerPanel is visible only when context is switched to volunteers, does not appear when SocialCare begins running.

Is this fixed?

Currently, VolunteerPanel is visible only when context is switched to volunteers, does not appear when SocialCare begins running.

Is this fixed?

Nope, will be done as a separate pull request


Further unify IO and use dask for multiple files

Description
This PR fixes dask usage behaviour when reading in folders of files. It also further unifies the behaviour of the file > open dialog, the command line file IO, and the drag and drop functionality.
This PR does not:

allow lazy loading of single files e.g. multi-page tiffs.
work correctly with mixed dtypes or mixed shapes if the user doesn't use use_dask=False (because we need to assume that all dtypes are the same, or read in all files in sequence to check the dtype).
allow the user to set use_dask or stack options with File > open or drag and drop.

I haven't had time to test this but it could definitely have a first round of preliminary review to check the design.
Type of change

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

@sofroniewn what is the full traceback for the URLError?

Traceback (most recent call last):
  File "/anaconda3/lib/python3.7/urllib/request.py", line 1488, in open_local_file
    return addinfourl(open(localfile, 'rb'), headers, origurl)
IsADirectoryError: [Errno 21] Is a directory: '/Users/nicholassofroniew/Github/image-demos/data/mnist/'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/nicholassofroniew/Github/napari/napari/_qt/qt_viewer.py", line 388, in dropEvent
    self._add_files(filenames)
  File "/Users/nicholassofroniew/Github/napari/napari/_qt/qt_viewer.py", line 248, in _add_files
    image = io.magic_read(filenames)
  File "/Users/nicholassofroniew/Github/napari/napari/util/io.py", line 68, in magic_read
    image = io.imread(filename)
  File "/anaconda3/lib/python3.7/site-packages/skimage/io/_io.py", line 60, in imread
    with file_or_url_context(fname) as fname:
  File "/anaconda3/lib/python3.7/contextlib.py", line 112, in __enter__
    return next(self.gen)
  File "/anaconda3/lib/python3.7/site-packages/skimage/io/util.py", line 27, in file_or_url_context
    u = urllib.request.urlopen(resource_name)
  File "/anaconda3/lib/python3.7/urllib/request.py", line 222, in urlopen
    return opener.open(url, data, timeout)
  File "/anaconda3/lib/python3.7/urllib/request.py", line 525, in open
    response = self._open(req, data)
  File "/anaconda3/lib/python3.7/urllib/request.py", line 543, in _open
    '_open', req)
  File "/anaconda3/lib/python3.7/urllib/request.py", line 503, in _call_chain
    result = func(*args)
  File "/anaconda3/lib/python3.7/urllib/request.py", line 1451, in file_open
    return self.open_local_file(req)
  File "/anaconda3/lib/python3.7/urllib/request.py", line 1490, in open_local_file
    raise URLError(exp)
urllib.error.URLError: <urlopen error [Errno 21] Is a directory: '/Users/nicholassofroniew/Github/image-demos/data/mnist/'>

@sofroniewn this is ready for re-review! Ideally I'd like to add support for testing the _add_images functionality in the viewer, but might not have time to finish that today.

And yes, this closes #489! :tada:

@sofroniewn wanted this on master for my own work. Let me know if there are further fixes you need here!


Fix Series.nsmallest and nlargest.

The current implementations skip handling _scol.

Codecov Report

Merging #861 into master will decrease coverage by 1.56%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #861      +/-   ##
==========================================
- Coverage    94.3%   92.73%   -1.57%
==========================================
  Files          34       34
  Lines        6184     6184
==========================================
- Hits         5832     5735      -97
- Misses        352      449      +97

Impacted Files
Coverage Δ

databricks/koalas/series.py
95.33% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


feat(render): add multi state + empty placeholders

Related to https://github.com/bpmn-io/properties-panel/issues/69
npx @bpmn-io/sr "bpmn-io/dmn-js-properties-panel#update-placholders"

What's missing here are proper icons, but I think the texts are already worth moving forward.

To consider here, too: bpmn-io/bpmn-js-properties-panel#687 (comment)

@pinussilvestrus will you adjust that? I support the comment and think a simpler message would make sense

This PR is already aligned with what we have merged in the bpmn-js-properties-panel 👍


TMDB Request Caching

Implement caching of external tmdb api requests (24h).
Closes #59

im scared how easy this seems to have been to add with the gin cache plugin, hopefully it isn't secretly broken.
Log seems to show no tmdb requests going out whilst our cache is available.


default sdks to azure_core/enable_reqwest

Reported by @mario-guerra when using azure_messaging_servicebus 0.4.0. He had to explicitly set:
azure_core = { version = "0.4.0", features = ["enable_reqwest"] }

to avoid:

thread 'main' panicked at 'A request was called on the default http client NoopClient.This client does nothing but panic. Make sure to enable an httpclient that can actually perform requests. You can do this by ensuring that the reqwest feature is enabled.

The default in azure_core was changed 3 weeks ago while working on wasm in https://github.com/Azure/azure-sdk-for-rust/pull/939/files#diff-7635a0301d45857a3457ae111c5fc7a672bf63303f2693ed9fe6986271338d86R48 .
This sets the default of azure_core/enable_reqwest in:

azure_iot_hub
azure_messaging_eventgrid
azure_messaging_servicebus
azure_security_keyvault

@cataggar when will the crates affected by this change be updated?


NCP: add E2E tests for organizations

The new collective page doesn't have a good test coverage yet. We want to solve that by adding new cypress tests. We've added the paths that we want to cover in https://github.com/opencollective/opencollective-frontend/blob/master/test/cypress/integration/04-organization_v2.test.js.
To implement the missing tests, you need to:

Setup your local environment & check the best practices: https://github.com/opencollective/opencollective-frontend/blob/master/docs/e2e.md
Open 04-organization_v2.test.js, for each test with a // TODO:

Remove the .skip (it.skip('...') --> it('...'))
Implement the test (make sure it passes)
If you're not sure about what should be tested for a given test, please ask directly on this issue

Commit the result, CI will run your tests to make sure they pass.

:gift: :money_with_wings: A $50 bounty was attached to this issue. Anyone submitting a Pull Request will be rewarded with $50 when the Pull Request is reviewed, accepted and merged.
:warning: Because we want the cypress issues to be addressed by different contributors, this bounty cannot be assigned to the same person who will tackle https://github.com/opencollective/opencollective/issues/2442, https://github.com/opencollective/opencollective/issues/2450 or https://github.com/opencollective/opencollective/issues/2452
:arrow_right: More info

Hi @Betree, I can try this one :D.

@jzabala all yours!

Thanks @Betree

I thought according to the new bounty guidelines here: https://blog.opencollective.com/scaling-up-our-bounty-program/ the minimum is now $100?

Hey @Betree  what exactly should be tested according to this issue

@jzabala are you still on it or can someone else have a look?

@jzabala are you still on it or can someone else have a look?

@znarf I was a little busy with https://github.com/opencollective/opencollective/issues/2433 and after that I was going to work on this one. But if you prefer you can reassign it. No problem :D

@znarf I want to give it a try

I assigned it to you @JoyLubega. Anything you need just let us know 🙂

:trophy: This issue is now completed by @JoyLubega.
To claim the $50 bounty, you can either:

Submit an invoice to https://opencollective.com/engineering/expenses/new (please mention the issue number in the expense's title)
Or ask for an Open Collective gift card of the same value (to donate to other collectives)


Adding custom routes to the express server

I have tried adding a route to sign certain file upload requests from the react app, but thee route seems to be ignored.
Here is what I  tried:
app.use('/s3', require('react-dropzone-s3-uploader/s3router')({
    bucket: "oddjobz",
    region: 'us-west-2', //optional
    //headers: {'Access-Control-Allow-Origin': '*'}, // optional
    ACL: 'public' // this is default:private
}));

Hey, where are you adding these in the server? Because the Express server uses HMR middleware, you'll probably want to try placing these routes before those are passed into Express.

Hello @davezuko I added the rout to the app.js file under the server folder.
import express from 'express'
import historyApiFallback from 'connect-history-api-fallback'
import config from '../config'

const app = express()
const debug = require('debug')('kit:server')
const paths = config.utils_paths

app.use(historyApiFallback({
  verbose: false
}))

app.use('/s3', require('react-dropzone-s3-uploader/s3router')({
    bucket: "oddjobz",
    region: 'us-west-2', //optional
    //headers: {'Access-Control-Allow-Origin': '*'}, // optional
    ACL: 'public' // this is default:private
}));

...
export default app

I think you’ll need to put it before historyApiFallback, since that resolves every possible route. It needs to go after every specific route.
On 15 Dec 2015, at 06:46, Anthony Alaribe<EMAIL_ADDRESS>wrote:

Hello @davezuko I added the rout to the app.js file under the server folder.
import express from 'express'
import historyApiFallback from 'connect-history-api-fallback'
import config from '../config'
const app = express()
const debug = require('debug')('kit:server')
const paths = config.utils_paths
app.use(historyApiFallback({
verbose: false
}))
app.use('/s3', require('react-dropzone-s3-uploader/s3router')({
bucket: "oddjobz",
region: 'us-west-2', //optional
//headers: {'Access-Control-Allow-Origin': '*'}, // optional
ACL: 'public' // this is default:private
}));
...
export default app
—
Reply to this email directly or view it on GitHub.

Is there a way to watch/restart for nodejs server endpoints ?

Nodemon works great.

@davezuko I was just trying it out. Which task would you recommend running on dev with nodemon ?

@sarunast this is where it's a bit tricky, I've never really used nodemon while doing webpack development. So normally it would just make sense to re-run to whatever script starts the server (in this case it's npm start), but I'm not convinced on how well this would work if you're also doing webpack development, since it's going to restart the webpack compiler.

hello @mtsr I tried that but i'm still getting the error:
'Warning: Location "/s3" did not match any routes'
my app.js is now:
import express from 'express'
import historyApiFallback from 'connect-history-api-fallback'
import config from '../config'

const app = express()
const debug = require('debug')('app:server')
const paths = config.utils_paths

app.use('/s3', require('react-dropzone-s3-uploader/s3router')({
    bucket: "oddjobz",
    region: 'us-west-2', //optional
    //headers: {'Access-Control-Allow-Origin': '*'}, // optional
    ACL: 'public' // this is default:private
}));

app.use(historyApiFallback({
  verbose: false
}))

// Serve app with Webpack if HMR is enabled
if (config.compiler_enable_hmr) {
  const webpack = require('webpack')
  const webpackConfig = require('../build/webpack')
  const compiler = webpack(webpackConfig)

  app.use(require('./middleware/webpack-dev')({
    compiler,
    publicPath: webpackConfig.output.publicPath
  }))
  app.use(require('./middleware/webpack-hmr')({ compiler }))
} else {
  debug(
    'Application is being run outside of development mode. This starter kit ' +
    'does not provide any production-specific server functionality. To learn ' +
    'more about deployment strategies, check out the "deployment" section ' +
    'in the README.'
  )

  // Serving ~/dist by default. Ideally these files should be served by
  // the web server and not the app server, but this helps to demo the
  // server in production.
  app.use(express.static(paths.base(config.dir_dist)))
}

export default app

If you replace the dropzone uploader middleware with a dummy middleware, do you get the expected response?
app.use('/s3', (_, res) => res.send('Hello World'))

@davezuko Thanks for that. I got a response, And I just retried my earlier implementation, and it works. So I believe the issue was with keeping the handler before the HistoryApiFallback as @mtsr said.
Thanks. I'm closing this issue.

Awesome, I'll consider adding a note to the server so other people don't run into this.


[ADAM-528][ADAM-533] Adds new RegionJoin impl that is shuffle-based

First pass shuffle-based join impl.  Passes some rudimentary tests.
Certainly needs docs and more tests.
Fixes #528.
Fixes #533.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/518/
Test PASSed.

@tdanford @fnothaft What is necessary to move this forward?  I added scaladocs, and there are some tests, though there could be more.  One issues is I made a few changes that I thought would be relatively cosmetic, and now the build hangs with some Akka problem.  Any idea what's going on here?
2015-01-05 18:36:27 ERROR OneForOneStrategy:66 - Actor not found for: ActorSelection[Anchor(akka.tcp://sparkDriver@localhost:7077/), Path(/user/HeartbeatReceiver)]
akka.actor.ActorInitializationException: exception during creation
	at akka.actor.ActorInitializationException$.apply(Actor.scala:164)
	at akka.actor.ActorCell.create(ActorCell.scala:596)
	at akka.actor.ActorCell.invokeAll$1(ActorCell.scala:456)
	at akka.actor.ActorCell.systemInvoke(ActorCell.scala:478)
	at akka.dispatch.Mailbox.processAllSystemMessages(Mailbox.scala:263)
	at akka.dispatch.Mailbox.run(Mailbox.scala:219)
	at akka.dispatch.ForkJoinExecutorConfigurator$AkkaForkJoinTask.exec(AbstractDispatcher.scala:393)
	at scala.concurrent.forkjoin.ForkJoinTask.doExec(ForkJoinTask.java:260)
	at scala.concurrent.forkjoin.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1339)
	at scala.concurrent.forkjoin.ForkJoinPool.runWorker(ForkJoinPool.java:1979)
	at scala.concurrent.forkjoin.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:107)
Caused by: akka.actor.ActorNotFound: Actor not found for: ActorSelection[Anchor(akka.tcp://sparkDriver@localhost:7077/), Path(/user/HeartbeatReceiver)]
	at akka.actor.ActorSelection$$anonfun$resolveOne$1.apply(ActorSelection.scala:65)
	at akka.actor.ActorSelection$$anonfun$resolveOne$1.apply(ActorSelection.scala:63)
	at scala.concurrent.impl.CallbackRunnable.run(Promise.scala:32)
	at akka.dispatch.BatchingExecutor$Batch$$anonfun$run$1.processBatch$1(BatchingExecutor.scala:67)
	at akka.dispatch.BatchingExecutor$Batch$$anonfun$run$1.apply$mcV$sp(BatchingExecutor.scala:82)
	at akka.dispatch.BatchingExecutor$Batch$$anonfun$run$1.apply(BatchingExecutor.scala:59)
	at akka.dispatch.BatchingExecutor$Batch$$anonfun$run$1.apply(BatchingExecutor.scala:59)
	at scala.concurrent.BlockContext$.withBlockContext(BlockContext.scala:72)
	at akka.dispatch.BatchingExecutor$Batch.run(BatchingExecutor.scala:58)
	at akka.dispatch.ExecutionContexts$sameThreadExecutionContext$.unbatchedExecute(Future.scala:74)
	at akka.dispatch.BatchingExecutor$class.execute(BatchingExecutor.scala:110)
	at akka.dispatch.ExecutionContexts$sameThreadExecutionContext$.execute(Future.scala:73)
	at scala.concurrent.impl.CallbackRunnable.executeWithValue(Promise.scala:40)
	at scala.concurrent.impl.Promise$DefaultPromise.tryComplete(Promise.scala:248)
	at akka.pattern.PromiseActorRef.$bang(AskSupport.scala:267)
	at akka.actor.EmptyLocalActorRef.specialHandle(ActorRef.scala:508)
	at akka.actor.DeadLetterActorRef.specialHandle(ActorRef.scala:541)
	at akka.actor.DeadLetterActorRef.$bang(ActorRef.scala:531)
	at akka.remote.RemoteActorRefProvider$RemoteDeadLetterActorRef.$bang(RemoteActorRefProvider.scala:87)
	at akka.remote.EndpointWriter.postStop(Endpoint.scala:561)
	at akka.actor.Actor$class.aroundPostStop(Actor.scala:475)
	at akka.remote.EndpointActor.aroundPostStop(Endpoint.scala:415)
	at akka.actor.dungeon.FaultHandling$class.akka$actor$dungeon$FaultHandling$$finishTerminate(FaultHandling.scala:210)
	at akka.actor.dungeon.FaultHandling$class.terminate(FaultHandling.scala:172)
	at akka.actor.ActorCell.terminate(ActorCell.scala:369)
	at akka.actor.ActorCell.invokeAll$1(ActorCell.scala:462)
	... 8 more

I'm good with merging once the tests pass; I can give the test issue a look tomorrow.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/519/Build result: ABORTEDGitHub pull request #534 of commit 85b17fc7a5af463c5cb0b9395033704359f13e1b automatically merged.[EnvInject] - Loading node environment variables.Building remotely on amp-jenkins-worker-05 (centos) in workspace /home/jenkins/workspace/ADAM-prb > git rev-parse --is-inside-work-tree # timeout=10Fetching changes from the remote Git repository > git config remote.origin.url https://github.com/bigdatagenomics/adam.git # timeout=10Fetching upstream changes from https://github.com/bigdatagenomics/adam.git > git --version # timeout=10 > git fetch --tags --progress https://github.com/bigdatagenomics/adam.git +refs/pull/:refs/remotes/origin/pr/ > git rev-parse origin/pr/534/merge^{commit} # timeout=10Checking out Revision 50ed66ed1da1c9e339547056689d56b262c988bb (detached) > git config core.sparsecheckout # timeout=10 > git checkout -f 50ed66ed1da1c9e339547056689d56b262c988bb > git rev-list b409bd0004a00c2dac712b5c567963b3e013c8e3 # timeout=10Triggering ADAM-prb ? 2.2.0,centosTriggering ADAM-prb ? 1.0.4,centosTriggering ADAM-prb ? 2.3.0,centosADAM-prb ? 2.2.0,centos completed with result SUCCESSADAM-prb ? 1.0.4,centos completed with result ABORTEDADAM-prb ? 2.3.0,centos completed with result ABORTED
Test FAILed.

Jenkins, retest this please.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/520/Build result: FAILUREGitHub pull request #534 of commit 85b17fc7a5af463c5cb0b9395033704359f13e1b automatically merged.[EnvInject] - Loading node environment variables.Building remotely on amp-jenkins-worker-05 (centos) in workspace /home/jenkins/workspace/ADAM-prb > git rev-parse --is-inside-work-tree # timeout=10Fetching changes from the remote Git repository > git config remote.origin.url https://github.com/bigdatagenomics/adam.git # timeout=10Fetching upstream changes from https://github.com/bigdatagenomics/adam.git > git --version # timeout=10 > git fetch --tags --progress https://github.com/bigdatagenomics/adam.git +refs/pull/:refs/remotes/origin/pr/ > git rev-parse origin/pr/534/merge^{commit} # timeout=10Checking out Revision 50ed66ed1da1c9e339547056689d56b262c988bb (detached) > git config core.sparsecheckout # timeout=10 > git checkout -f 50ed66ed1da1c9e339547056689d56b262c988bb > git rev-list b409bd0004a00c2dac712b5c567963b3e013c8e3 # timeout=10Triggering ADAM-prb ? 2.2.0,centosTriggering ADAM-prb ? 1.0.4,centosTriggering ADAM-prb ? 2.3.0,centosADAM-prb ? 2.2.0,centos completed with result SUCCESSADAM-prb ? 1.0.4,centos completed with result SUCCESSADAM-prb ? 2.3.0,centos completed with result FAILURE
Test FAILed.

Jenkins, retest this please.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/521/
Test PASSed.

@fnothaft well there you go

@fnothaft @tdanford ok, please review, and let me know if there are some other things to do before merging :)

I am +1 to merge.

@fnothaft Is it uncouth to merge my own PR after it's +1'd?

@laserson resolved that problem for you ;). Thanks for the contribution!


Add support for stash pop

I'm terrible at cleaning up my old stashes.  It would be great if stash pop was supported as well as apply.
Often when a stash is applied, a project or solution file changes and Visual Studio immediately focuses the Solution Explorer pane. It's east to forget to delete the stash after you're no longer on the stash page.
Stash pop would also be useful for workflows that involve temporarily saving a stash.  E.g. when you stash before doing a pull or merge.

Great suggestion! I'll add it soon.

Much appreciated, thanks! 👍

I've just released a new version. Could you please get it (Git stash extension) and test new function.


Fixed the typo in diff tolerance type

I just renamed instances where tollerance is spelled as tolerance
This will probably need back-end changes for the tests to pass.

@Inovvia thanks for contribution
unfortunately the problem is on backend side as well and in DB so it involve lot's of changes to fix that (

Perhaps this can be linked to the Prisma upgrade in "backend" repo, so that when the database changes happen, this typo fixing is included? @pashidlos

@polyvisual let's tackle it separately as it require all SDK change or making it as non-braking change


Sending some metrics from triton container

Is there a way we can send some metrics from within the triton-inference-server container to the cloud, like sending the response time metrics so that the auto-scaling mechanism can work?

Triton exposes latency metrics in Prometheus format:
https://docs.nvidia.com/deeplearning/sdk/triton-inference-server-master-branch-guide/docs/metrics.html
In AWS, you can stick Triton in an Autoscaling Group (ASG) fronted by an Application Load Balancer (ALB).  Use the ALB metrics to trigger ASG scale up/down events.


Polymer CLI: Could you please support commonJS ?

Hi. We are building a new project using Polymer CLI and Lit element and we want to try https://github.com/amcharts/amcharts4 but we have some issues in development time: we cannot use es6 modules provided by them. I opened a issue (https://github.com/amcharts/amcharts4/issues/977) to their repo, and they said this is a Polymer responsibility to provide CommonJS modules support. What do you think about this ? Thank you.

This has been discussed many times, and the position of Polymer team was that they only support ES modules intentionally, as CommonJS is node-specific. Consider using other tools like webpack or rollup if you need to use CommonJS dependencies in your project.

Oh... i understand. Thank you, @web-padawan !


Using pretrained weights for training on 3 categories

Hi,
When i tried using it on pretrained weights by settings the number of classes to 3 i got an error.
Assign requires shapes of both tensors to match. lhs shape= [16] rhs shape= [64]
Should i train from scratch ?
Thanks.

Yes.

No, it should be caused by some parameters not modified. @deimsdeutsch


Add SQS primitives

fixes issue https://github.com/MonsantoCo/cloudformation-template-generator/issues/33

addresses https://github.com/MonsantoCo/cloudformation-template-generator/issues/33

Very much appreciated, thanks.


Custom hook for parsed useScopedI18n & useI18n

Hello, while #343 is currently not possible, I'm thinking of a workaround.
Currently, I'm using this parser: https://www.npmjs.com/package/html-react-parser
and wrap every entry of t or scopedT like this
parse(scopedT('specificKey', { specificKeyParam: 'paramValue' }

It's not clean and quite an unreliable approach, so I tried to make a custom hook. I want to return the same functions as useScopedI18n and useI18n, but parsed.
So far I got this and stuck with parameters of functions. What would be a proper way to declare them?
'use client';

import type {
  FlattenLocale,
  LocaleKeys,
  ParamsObject,
  Scopes,
} from 'international-types';

import { useCurrentLocale, useI18n, useScopedI18n } from<EMAIL_ADDRESS>import type en from<EMAIL_ADDRESS>
import parse from 'html-react-parser';

export type DefaultLocale = FlattenLocale<typeof en>;
type Scope = Scopes<DefaultLocale>;
type Key = LocaleKeys<DefaultLocale, Scope>;

export const useTranslation = (scope: Scopes<DefaultLocale>) => {
  const locale = useCurrentLocale();
  const unparsedScopedT = useScopedI18n(scope);
  const unparsedT = useI18n();

  const scopedT = (key: ???, params?: ???): ReturnType<typeof parse> => {
    return params
      ? parse(unparsedScopedT(key, params))
      : parse(unparsedScopedT(key));
  }

  const t = (key: ???, ???): ReturnType<typeof parse> => {
    return parse(unparsedT(...));
  }

  return {
    locale,
    scopedT,
    t,
  }
}

You can take a look at some examples here: https://github.com/QuiiBz/next-international/tree/main/packages/international-types
Let me know if you're still stuck on this, but I believe it should be pretty much the same.

Feel free to send a minimal reproduction (e.g. CodeSandbox) so I can try to help you with the types!


Dev

Here I add pytorch:

Notebooks
Scripts(Basic  & Advanced)

There is some .DS_Store file in there -> I think we should drop it.

Comments for notebook:

Let's remove the installation output -> you can use --quiet or simply log notebook without outputs
You can put installation and the information about neptune API token/project in this "Before you start" section as was done here https://github.com/neptune-ai/examples/blob/bed2bc874c7de6d887c8d004a7ab7c4de3a88937/integrations-and-supported-tools/optuna/notebooks/Neptune_Optuna_integration.ipynb
Main -> let's use something more descriptive. Basic example or Quickstart
It seems that you are using pytorch ignite? -> it is a basic pytorch example, we have a separate ignite integration
Try to divide it into descriptive sections as in Log Hyperparameters (which will have define parameters and log parameters subsections)
Drop the things that are not crucial from the training loop -> we need to show an example of using those tools together -> to complex or detailed training loop is making the supported parts harder to see
The log model and log images should be subsections of the More options sections -> see https://github.com/neptune-ai/examples/blob/bed2bc874c7de6d887c8d004a7ab7c4de3a88937/integrations-and-supported-tools/optuna/notebooks/Neptune_Optuna_integration.ipynb
make sure to showcase the supported parts, not generic logging of images -> neptune supports logging of torch tensors and this is what we should be showing


Update rc5

Major breaking change for RC5
Dropped simplebind feat

Coverage remained the same at 100.0% when pulling d0adfe84e5d16fd82a14d64eb5c3ccd7e0c2e8a2 on update-rc5 into c8aab84f0eb3309bb0dfd87fc2b8fea7883dbfdb on master.

Coverage remained the same at 100.0% when pulling 3f67515a8a369405cb67a9d12f314d10c085af57 on update-rc5 into c8aab84f0eb3309bb0dfd87fc2b8fea7883dbfdb on master.

Coverage remained the same at 100.0% when pulling 74ebd11c3977b997de49799600b02db62592f048 on update-rc5 into c8aab84f0eb3309bb0dfd87fc2b8fea7883dbfdb on master.

Coverage remained the same at 100.0% when pulling 64ff9f8d1d42e1b293d17fcf197e890093c003f2 on update-rc5 into c8aab84f0eb3309bb0dfd87fc2b8fea7883dbfdb on master.


fix(bug report): auto fill notification_email users

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


WRU-Server answers 502

Description
Requesting any resource beneath https://quicktest-result-cff4f7147260.coronawarn.app/ with a valid certificate results in a 502 error code
Expected behaviour
Depending on the operation, a successful status code, or, if the request is bad, a status code in the 4xx range
Steps to reproduce the issue

Have a valid certificate
Visit  https://quicktest-result-cff4f7147260.coronawarn.app/ with a browser of your choice

Technical details
Tried it with curl and chromium, with the wru certificate issued when accepted as partner

Does this problem still exist for you?

@keppla the problem still exists?


Can we link to the docs in the description?

Too many people aren't aware documentation exists. Can we add it to the end of the description like many other projects seem to? I know I always look there.

I think this is a great idea!

@zigomir I just added this to the 2.0 branch. Not sure which other branches might also need updates. I see that 1.0 is now ahead of dist - is it now deprecated? If so, should we update the default branch to 1.0?

@chrisvfritz not really sure what do you mean. Are you asking if dist is deprecated? In any case, I'm not sure, I don't really use or watch webpack template.
I only know we have some extra code in vue-cli just to check if dist branch exists and use it if it does. I think we should remove it since vue-cli supports branches. Sorry, I might be off topic.

Got it. I guess this is a discussion including @LinusBorg as well then, since I see the 1.0 branch has received some commits that the dist branch has not. My understanding is that the dist branch is currently the default for this template though. Is that correct @zigomir? I just want to make sure we're all on the same page. 😄  Once we are, it would probably also be a good time to delete any branches we don't want.

@chrisvfritz yes, that is correct. dist branch will be used by vue-cli by default (https://github.com/vuejs/vue-cli/blob/master/bin/vue-init#L116).

Honestly I wasn't sure weither to add the two PRs of mine into dist, so I targeted 1.0 instead.
Once we switch to the 2.0-version as the default template, (which should be right around the corner, right?) we could merge 2.0 into dist and I think we can leave 1.0 as it is.

I think I'd prefer to just remove the dist branch, since the other templates don't have one. We should probably keep it in the CLI though, in case other templates are relying on the behavior.

...which will mean that old vue-cli versions will error out, right?

@LinusBorg are you sure? This means that old vue-cli versions have hardcoded dist branch lookup?

Well you linked to the source yourself above...

Oh.. Wait. If no dist it returns the template without a branch tag, which should download the defAult branch?
Then we should be fine.

Yes, and also checkDistBranch is only for official templates so there should be no problem if we remove it.

This should be done now.


reactivate command broken over type mismatch

When running the keylime_tenant with reactivate the following stack trace occurs:
2020-05-12 09:32:34.709 - keylime.cloudverifier - ERROR - can only concatenate str (not "int") to str
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/keylime/cloud_verifier_tornado.py", line 499, in process_agent
    main_agent_operational_state = agent['operational_state']
  File "/usr/lib64/python3.7/site-packages/sqlalchemy/orm/query.py", line 3048, in __getitem__
    return list(self[item : item + 1])[0]
TypeError: can only concatenate str (not "int") to str

The reason for this is because agent object is coming in a SQLAclhemy type and not a dict (or list?. So its not just as case of changing  main_agent_operational_state = agent['operational_state'] to  main_agent_operational_state = agent.operational_state
If we do this, then the standard quote operation will fail
We might instead need to perform a type check and have a conditional to process accordingly.
Steps to recreate

Add an agent:
keylime_tenant -v <IP_ADDRESS> -t <IP_ADDRESS> -u D432FBB3-D2F1-4A97-9EF7-75BD81C00000 -f somefile.txt -c add

Reactive
keylime_tenant -v <IP_ADDRESS> -t <IP_ADDRESS> -u D432FBB3-D2F1-4A97-9EF7-75BD81C00000 -f somefile.txt -c reactivate

The above error will then throw.

@kaifeng are you interesting in taking this one on?

@lukehinds Hi o/
I am still get myself familiarized with the code, so not sure I can handle this, the bad side is my tpm environment crashed after a power shortage, and I am lacking a env to validate my change. After quickly browsing the code, it seems the agent was assumed to be a dictionary, so the change is likely a conversion at the entry of process_agent. The issue behind I think, is the lack of layer of database and serialization/deserialization upon it.

@kaifeng no worries, I can put in a fix with my auth patch. There are other means to achieve the same option is reactive
Regarding tpms, do you use the emulator?

I was using VM equipped with vtpm device, which is much like a real environment.

I am thinking we should move our gate over to using the vtpm, less set up steps.

I made a patch for this, hope that works.

I am thinking we should move our gate over to using the vtpm, less set up steps.

Yep, vtpm make things easier, I don't know if travis support virtual machines, it will cost quite a lot of resources.


PFB redirect URL is missing format=PFB query parameter

In dev I requested a PFB manifest. The re-direct URL was https://bvdp-saturn-dev.appspot.com/#import-data?url=https%3A%2F%2Fservice.dev.singlecell.gi.ucsc.edu%2Fmanifest%2Ffiles%3Fcatalog%3Ddcp2%26format%3Dterra.pfb%26filters%3D%257B%2522projectId%2522%253A%2B%257B%2522is%2522%253A%2B%255B%252260ea42e1-af49-42f5-8164-d641fdb696bc%2522%255D%257D%252C%2B%2522genusSpecies%2522%253A%2B%257B%2522is%2522%253A%2B%255B%2522Homo%2Bsapiens%2522%255D%257D%252C%2B%2522fileFormat%2522%253A%2B%257B%2522is%2522%253A%2B%255B%2522fastq.gz%2522%255D%257D%257D%26objectKey%3Dmanifests%252F920202e8-ac29-510a-a20c-a73dd3a276ed.avro
which doesn't work. Terra complains:

Yes. When redirecting to Terra they require a format=PFB param to distinguish from bdbag imports. This param should not be encoded, and should be part of the Terra URL. In other words:
https://bvdp-saturn-dev.appspot.com/#import-data?url=https%3A%2F%2Fservice.dev.singlecell.gi.ucsc.edu%2Fmanifest%2Ffiles%3Fcatalog%3Ddcp2%26format%3Dterra.pfb%26filters%3D%257B%2522projectId%2522%253A%2B%257B%2522is%2522%253A%2B%255B%252260ea42e1-af49-42f5-8164-d641fdb696bc%2522%255D%257D%252C%2B%2522genusSpecies%2522%253A%2B%257B%2522is%2522%253A%2B%255B%2522Homo%2Bsapiens%2522%255D%257D%252C%2B%2522fileFormat%2522%253A%2B%257B%2522is%2522%253A%2B%255B%2522fastq.gz%2522%255D%257D%257D%26objectKey%3Dmanifests%252F920202e8-ac29-510a-a20c-a73dd3a276ed.avro
should be changed to
https://bvdp-saturn-dev.appspot.com/#import-data?format=PFB&url=https%3A%2F%2Fservice.dev.singlecell.gi.ucsc.edu%2Fmanifest%2Ffiles%3Fcatalog%3Ddcp2%26format%3Dterra.pfb%26filters%3D%257B%2522projectId%2522%253A%2B%257B%2522is%2522%253A%2B%255B%252260ea42e1-af49-42f5-8164-d641fdb696bc%2522%255D%257D%252C%2B%2522genusSpecies%2522%253A%2B%257B%2522is%2522%253A%2B%255B%2522Homo%2Bsapiens%2522%255D%257D%252C%2B%2522fileFormat%2522%253A%2B%257B%2522is%2522%253A%2B%255B%2522fastq.gz%2522%255D%257D%257D%26objectKey%3Dmanifests%252F920202e8-ac29-510a-a20c-a73dd3a276ed.avro
To repoduce this, just do a Terra PFB export from the dev browser, from a project for example, and log into Terra dev and try and complete the import. For me it failed with the error above, until I corrected the URL.


Fixed bug with volume control when spotify is in system tray

Thanks! Will release once we got some more changes


Is CacheManager a dead project?

No updates in 2.5 years.  An update from the owner would be nice. My company has a policy of only using actively developed open-source projects. Preferably ones where the owner is able to monetize.

I see you have answered this question previously and then closed.  If you are no longer motivated to work on this project let people know that. It's your right to put in as much or as little time as you want, but you should let people know either way.  Good luck.


[Release Request] Release new versions in a fixed time interval

Recently I created an issue for feature request. It was already merged into the master branch. However it has been so long and that PR version is not released yet. Can you release the new versions on a regular basis, like 2 weeks/one month? The latest version was released in April.

The main reason there was no new release is that there hasn't been any new major feature, apart from the DeepSpeed support which needs a little bit more testing.
I'll see if I can make a new release next week.

That's good to hear, thanks a lot!


'NelsonMartell\Extensions\String' class compatible with PHP7

PHP7 collision issue:
Cannot use NelsonMartell\Extensions\String as String because 'String' is a special class name.
If problems in any application, just set alias for class:
use NelsonMartell\Extensions\String as Text;

TODO: Deprecate this class and replace it with Text.

Current available in `^v0.7@dev'.


How to do the midi generation and midi playing simultaneously?

Hello, dear google-magenta guys,
I tried to use your APIs to generate music.
First， create an estimator：
# Create Estimator.
run_config = trainer_lib.create_run_config(hparams)
estimator = trainer_lib.create_estimator(
		model_name, hparams, run_config,
		decode_hparams=decode_hparams)

Second, I have a midi input interface:
# Create input generator (so we can adjust priming and
# decode length on the fly).

def input_generator():
	global targets
	global decode_length
	while True:
		yield {
				'targets': np.array([targets], dtype=np.int32),
				'decode_length': np.array(decode_length, dtype=np.int32)
		}

# These values will be changed by subsequent cells.
targets = []
decode_length = 0

# Start the Estimator, loading from the specified checkpoint.
input_fn = decoding.make_input_fn_from_generator(input_generator())

Third, do prediction:
unconditional_samples = estimator.predict(
		input_fn, checkpoint_path=ckpt_path)

Fourth, I have an API to generate the MIDI note one by one, after the generation, it will read the MID file as playing:
	# Generate sample events.
	sample_ids = next(unconditional_samples)['outputs']

	#print('-------------2', sample_ids)

	# Decode to NoteSequence.
	midi_filename = decode(
			sample_ids,
			encoder=unconditional_encoders['targets'])

	return open(midi_filename, 'rb').read()

To be frank, my question is that is it possible to iterate the model output and  then playing, means that do midi generation with decoding and midi playing simultaneously instead of playing the midi file after the midi file is ready?
Thanks & Regards!
Jun Yan

Currently, the demo is generated the midi and then play with midi-file opened.
Is there the potential to iterate the prediction result and play at as one by one music sequence?
Thanks  & Regards!
Jun Yan

I am sorry.
As our small team found,
the operation caused much time is this:
	# Generate sample events.
	sample_ids = next(unconditional_samples)['outputs']

However, output data is essential for generation of midi  and also playing.
How to reduce the operation time instead of the operation of "next" caused too much time?
Thanks & Rehards!


[v0.8.x]: cannot import in complex webpack-based workflows

Hello,
I already have a PR for this, but I'm opening an issue for tracking purposes
After upgrading to v0.8.1, I was receiving an error related to a "missing export of .".
Since it is time consuming to provide a reproducible test case, I can point to some of the most recent issues related to the addition of exports to package.json:

https://github.com/Choices-js/Choices/issues/1184
https://github.com/Choices-js/Choices/issues/1184#issuecomment-2322861193

Ref: https://webpack.js.org/guides/package-exports/
I will submit a PR

Hello, I encountered an issue while using the "frappe-gantt" package with Vite. The error message I received was:
[ERROR] Failed to resolve entry for package "frappe-gantt". The package may have incorrect main/module/exports specified in its package.json: No known conditions for "." specifier in "frappe-gantt" package [plugin vite:dep-scan]

To resolve this, I modified the "exports" section in the package.json like this:
"exports": {
    ".": {
        "require": "./dist/frappe-gantt.umd.js",
        "import": "./dist/frappe-gantt.umd.js"
    }
}

This change seems to have solved the issue!

"import": "./dist/frappe-gantt.umd.js"

Isn't supposed to be the es module file, as in #441?

I'm not sure if this is related to this issue or not.

I'm confident it is, but I'm not a javascript developer

You're right, both options seem to work, and using the es module feels more appropriate in this context.

@frustak can you please check if "frappe-gantt": "tagliala/gantt#chore/import-directive-webpack" works out of the box for you? I'll then remove the draft status here

Yes, it works out of the box for me.

Is there a workaround for this until the PR is merged?

Did you try my fork?
https://github.com/frappe/gantt/issues/439#issuecomment-2376142487

I wasn't sure how to do that. I can't see an NPM package for it, and wasn't sure how npm.lock files etc. would work.

I wasn't sure how to do that. I can't see an NPM package for it, and wasn't sure how npm.lock files etc. would work.

Should be possible in this way:
In package.json
-  "frappe-gantt": "^0.8.0"
+ "frappe-gantt": "tagliala/gantt#chore/import-directive-webpack"

or, according to your package manager
npm add choices.js@tagliala/gantt#chore/import-directive-webpack

I don't know if there are ways in the host application package.json to override export directives.
Maybe imports?
https://nodejs.org/api/packages.html#subpath-imports

Thanks for your help. Adding a resolve section to my vite.config.ts also seemed to work:
  resolve: {
    alias: {
      'frappe-gantt': 'node_modules/frappe-gantt/dist/frappe-gantt.es.js',
    },
  },

With this in place, I can tell it to bypass the package.json and load the es directly. For the CSS, I could copy the CSS from the dist directory, and add import './frappe-gantt.css' to the app/root.tsx.
This seems to have the same effect as your fork.

Thanks for your feedback here folks!


Usage of strings.ToTitle

I'm currently throwing together some experimental documentation, and I (incorrectly) assumed that strings.ToTitle set the first character of each word to uppercase, only to discover that was achieved by (the now deprecated) strings.Title.
My new (and perhaps still incorrect) understanding is that the only difference between strings.ToTitle and trings.ToUpper is that only the former takes Unicode title case mapping into account.
If this is unintentional, it seems we can achieve the behavior I incorrectly assumed by using cases.Title with a language Tag (as well as optional caser Options). We could potentially accept the language tag as a function argument (by its name) to leave it up to the user, or just default to cases.Title(language.Und, cases.NoLower).String(<string>).

Multiple thoughts:
I would only include template functions that are actively in use in people's dotfiles, and by default I would assume that a template function is not used unless the opposite is clearly true. ToTitle is almost certainly not used in dotfiles.
In the particular case of ToTitle, I would argue that it's not possible to write a correct function in any case. Specifically:

It's not just the word boundaries bug in the original strings.Title function.
In English it's not possible to write a correct ToTitle function without full grammatical understanding of each word.
Even if you have a system with such understanding, capitalization has a subjective element.
Capitalization in non-English languages is even more varied.

Simply put, ToTitle should not exist. Let's drop it.
The underlying goal here is not compatibility with microminds/sprig. The goal is a sensible set of template functions that are useful for writing templates for configuration files and integrate well with Go's text/template.

As an addendum, as I now realize that I misunderstood your question, I'd still drop ToTitle as it's a grammatical concept, but I would keep ToUpper for ASCII characters only as conversion to all-uppercase is often useful in configuration files.


chore(main): release 0.0.3

:robot: I have created a release beep boop
0.0.3 (2024-12-01)
Miscellaneous Chores

upgrade mime version (ed7c218)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.0.3
:sunflower:


Could we expose getPushToStartToken so we can use it please, that would be amazing, thaaaaaanks

Thanks for raising an issue, I will implement it ASAP :)


minica on android device

Hi guys, minica working good, but just on pc and google chrome...
Someone was able to install the Minica CA root Certificate on Android?
Mi dev domain ip is a private lan, like i said, is working on my pc but on android still show me the SSL error.

Yes, I've installed a minica root certificate on an Android device and used it to validate an HTTPS site. What steps have you taken so far?


Angular 2.0 and Latest Thoughts

What are your current thoughts and plans on angular-formly and Angular 2.0.  I am about to go into full scale development of a dynamic forms solution with editor, etc.  From what I can tell Angular 2.0 does provide part of the functionality to describe a schema but not to have data driven form layout for truly dynamic forms.  So any thoughts or pointers?
Ray

Hi @rayhooker. Thanks for the question. I actually just recently answered this on my AMA here. Let me know if you have further questions :-)


prefix automatically created roles

I need to be able to prefix any roles that are automatically created when deploying a stack and I am having trouble finding where and how to do this?
So when the iam:CreateRole is called on something like arn:aws:iam::***:role/cdk-hnb659fds-cfn-exec-role-***-region I would like it to be arn:aws:iam::***:role/{$customPrefix}-cdk-hnb659fds-cfn-exec-role-***-region.
Is this possible with the current SST?

Hi @gurkang, the role you mentioned above is created by CDK as part of cdk bootstrap process. cdk bootstrap creates 3 roles and the role names are hardcoded.
The roles created in ur SST app are all prefixed by {$stageName}-{$appName}-, ie. dev-my-app-. Are you able to build the custom prefix using the stage and app name?

hmm, so is there any way to override those cdk bootstrapped roles? Since it's these specifically that are causing issues.

According to the CDK team, I think the only way to do this is to create and deploy your own bootstrap CloudFormation template. So you have full flexibility over the role name. And when you create Stacks in your app, pass in the name of the role:
new MyStack(app, "my-stack", {
  synthesizer: new DefaultStackSynthesizer({
    cloudFormationExecutionRole: "...",
    deployRoleArn: "...",
  }),
});

https://github.com/aws/aws-cdk/issues/5798#issuecomment-576577173

Oh btw, you can also try running cdk bootstrap --qualifier acme, ur roles will be prefixed cdk-acme-. It still starts with cdk-.

Oh. So then I would have to abandon working with SST? Seems like such a shame

Wait, there might some confusion. All of the above work works with SST.

@fwang
I think what is needed is a new parameter in this call https://github.com/serverless-stack/serverless-stack/blob/master/packages/core/src/index.ts#L179
there is a parameter to cdk bootstrap called --template.
Is there a way to get this changed?

@fwang, I think the issue is that sst bootstrap --qualifier acme doesn't work as expected. SST strips out the qualifier when it invokes cdk bootstrap itself. It would be great if sst bootstrap could pass all arguments that cdk bootstrap takes. In the mean, time you can work around this by running cdk bootstrap --qualifier acme && sst bootstrap.

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble