fix(operators): added tips after operator description

Added tips to display after operator long description if they exist. Also removed parameter section
if no parameters exist.

@sumitarora really appreciate you helping review PRs.

Weird, looks like everything passed this time but something with selenium failed. Care to take a look @kwonoj?

Codecov Report

Merging #137 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #137   +/-   ##
=======================================
  Coverage   86.88%   86.88%
=======================================
  Files           4        4
  Lines          61       61
  Branches        7        7
=======================================
  Hits           53       53
  Misses          5        5
  Partials        3        3

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 29af945...6483d33. Read the comment docs.

It seems one time woes on travis.

Can this be reviewed / merged? It fixes a couple of display issues on the operator page that may cause confusion to others checking their operator PR's. @ladyleet @kwonoj  @sumitarora

@btroncone Please rebase with master.

@btroncone 👍  - waiting on @kwonoj or @ashwin-sureshkumar to review & merge.


How is User#digest being used?

@skhakimov @david-a-wheeler I'm unclear how https://github.com/linuxfoundation/cii-best-practices-badge/blob/master/BadgeApp/app/models/user.rb#L11-L18 is being used. Could you please provide a URL for the tutorial you were following when you implemented this? I found similar code in railstutorial.com, and I get simplifying bcrypt to speed up tests, but I don't think this implementation is correct.

I figured it out. It's used in fixtures: https://github.com/linuxfoundation/cii-best-practices-badge/blob/master/BadgeApp/test/fixtures/users.yml


Home button web fixes

Revisited application styling and addressed mobile layout.
Fix for empty image on Product detail page.
Auto-login fixes.
Updated styling for mobile app.
Prefer user's ID over e-mail address.

@laurwang Please be sure that I did not delete any of the schemas you needed during my merge. Thanks!

this contains old stuff greg.


Data processing problems

Hello, thank you very much for your work
I'm trying to use my own data to train your model, but because my MIDI data is single track, I want to know how you get the boundary_ track，melody_ boundary_tracks, theme info tracks，are they added manually？
thank you!

Hi @dedededefo
I use the midi files from POP909, in which each midi file contains 3 tracks: "melody", "bridge", and "piano".
You should run some melody extraction algorithms to attain melody and accompaniment from your single track midi.
For  other tracks (boundary_ track...), please refer to the detail description : preprocess/readme

I will close this issue now. You can reopen it if there is any further question.


Use GALAXY with versions as opposed to GXFORMAT2 for cloud instances API

Follow-up on #4163
On implementation, check affected code, for example, launch with Galaxy muli-cloud, TRS based launch-with from the Galaxy side
┆Issue is synchronized with this Jira Story
┆Issue Number: DOCK-1793
┆Sprint: Backlog
┆Issue Type: Story

CloudInstance API is supposed to be using https://github.com/dockstore/dockstore/blob/develop/dockstore-common/src/main/java/io/dockstore/common/DescriptorLanguage.java . Changing that would likely require changing quite a few things


build: cache-aware docker-xxx-build

这个 PR 实现什么功能/修复什么问题:
scripts: remove host_find_libraries.sh
build: cache-aware docker-xxx-build

是否需要 backport 到之前的 release 分支:

release/3.1

/area util
/cc @wanyaoqi @zexi @swordqiu

Codecov Report

Merging #5569 into master will not change coverage by %.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #5569   +/-   ##
=======================================
  Coverage    6.86%    6.86%
=======================================
  Files         612      612
  Lines      103327   103327
=======================================
  Hits         7090     7090
  Misses      95665    95665
  Partials      572      572

Flag
Coverage Δ

#aFlag
6.86% <ø> (ø)

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 78688a8...68861d9. Read the comment docs.

/lgtm


[RFC] Trigger red boxes for all promise rejections

This change will trigger red boxes instead of yellow boxes for all promise rejections.
At Facebook, we found that rejected promises aren't usually used for application logic and almost always represent a programming error.
Browsers, in general, also show rejected native Promises as errors.
I’ve put this up for discussion.

By analyzing the blame information on this pull request, we identified @skevy to be a potential reviewer.

Generally, I think this is fine. It would be nice to opt into (or maybe opt out of) it? I wonder if we could leave an implementation that doesn't hard error, to give people (power users I guess) the option to do global.Promise = require('PromiseWarnOnRejection'); or something.

cc @ide @sahrens @spicyj

rejected promises aren't usually used for application logic and almost always represent a programming error

Could you elaborate a bit more on this? When I think of unhandled rejections, I imagine cases where someone writes fetch('/my-logging-endpoint', someAnalyticsData) and doesn't add an error handler either because they forgot or they don't need one. Or when people create a promise in one tick of the event loop and add handlers in another.

Browsers, in general, also show rejected native Promises as errors.

With RN we're covering up the whole screen and making the app temporarily unusable for that moment, which is a lot heavier than an error message in the web console. If there are more non-fatal errors (as opposed to warnings) of this nature maybe we could add a new type of box that is colored red and has the behavior of the yellow boxes.

The most common reason this is important is for code like:
fetch(...)
  .then((resp) => {
    this.setStat({  // should be setState
      resp: resp,
    });
  });

where the callback throws and it goes into a black hole. Even if you are reasonably diligent about error handling and write like
fetch(...)
  .then((resp) => {
    this.setStat({  // should be setState
      resp: resp,
    });
  })
  .catch((e) => {
    this.setStat({  // should be setState
      error: e,
    });
  });

then because the catch block throws, the error is still swallowed silently. This is a huge problem for us because exceptions go unreported.

I think the way we handle console.error is a separate issue from handling promise rejections.
What do you think about universally moving console.error to the YellowBox system instead of full-screen RedBoxes?

@spicyj
How about if we Redbox in case of certain types of errors in promise rejection instead of all errors? For example, in your example, it's always a ReferenceError.

It's been a while since the last commit was reviewed and the labels show this pull request needs review. Based on the blame information for the files in this pull request we identified @skevy as a potential reviewer. Could you take a look please or cc someone with more context?

I think it would be better to do it for all exceptions since any exception can disrupt your code flow.

@spicyj personally in my code, there are many places where I don't care/need to handle promise rejections. With all rejections producing redbox, it'll force me to add error handlers everywhere which do nothing.

Can you give an example? I tend to think of promise rejections as akin to thrown errors, and if you wanted to silence those then an empty catch (e) {} would be a best practice.

@spicyj for example when I'm sending a post request for analytics or some kind of logging, I don't need to handle the failure

Just going to throw in my 2 cents towards treating promise rejections as a general exception. If you want to ignore an exception you would wrap it with an empty catch {}; the parallel should apply here with .catch().
This makes even more sense with async/await as await translates a rejection into an exception.

I think a failed/rejected network request is kind of a special case, but really should also be handled - you should log those failures and track them so you can monitor the health of your app/server.
The main problem is that if you do something clowny inside a promise chain, the exception shouldn't just get swallowed, it should RedBox like any other exception would.

I'd love to see this change in. Hunting down exceptions swallowed in a promise chain is not fun at all.

What's wrong with the current behavior of yellow-boxing this? A lot of folks are saying they want to be able to debug this, but you can still use the yellow box to debug.  So I think the current behavior is more in tune with how promises are expected to behave.

rejected promises aren't usually used for application logic and almost always represent a programming error
Could you elaborate a bit more on this? When I think of unhandled rejections, I imagine cases where someone writes fetch('/my-logging-endpoint', someAnalyticsData) and doesn't add an error handler either because they forgot or they don't need one. Or when people create a promise in one tick of the event loop and add handlers in another.

Is this really true though? I think the spec of fetch is that it only rejects the promise in cases of network connectivity issues or timeouts. If the server responds with a 500 etc the promise "resolves" not "rejects". I feel like network connectivity issues may be something that you don't care about, but normally I think people should.
We could also create a sensible white-list of error types that we are okay with console.warning, and then for the rest (ie, ReferenceError) we would redbox.
Thoughts?

I feel like network connectivity issues may be something that you don't care about, but normally I think people should.

Sure - but you can care about that just as easily with a yellow box. A red box makes sense if there's no further debugging value you can get from running the application. With a rejected promise, however, it's quite likely that continuing to run the application is useful. That's why I think yellowboxing is the right behavior. The question isn't "is it totally fine to ignore rejected promises", the question is whether an error like this should block further execution of the application.

I really think we should land this. You can always close a redbox and keep running the application when this happens- and it will really encourage people to catch potential problems in their app!
Plus, this does not affect production, where an unhandled promise rejection would happen in the background without crashing the app.

@ericvicenti @lacker seems like you both disagree on whether this is the right behavior. Let me know how you want to proceed.

I really think we should push this forward. I think we want all runtime errors to be "in-your-face" bad, even in rejected promises.

@ericvicenti I buy that, this is how I think of it: We want to treat sync and async errors similarly. This is consistent with Chrome and Node (in a coming release). So since we redbox on sync JS errors, it would follow we'd redbox on async JS errors too (in practice this means unhandled promise rejections since they're the mechanism for async fns).
The next thing to figure out is what to do when there is an unhandled error (regardless of whether it's sync or async). Node crashes the process, Chrome keeps it alive and prints a red stacktrace to the console. If the redbox is too disruptive, especially for large teams where someone might be in the middle of fixing an error while the rest of the team is working, we could combine the behavior of the yellowbox with the color of the redbox. That would help errors stand out from warnings, be very visible, and be less disruptive to a team.

Would it be hard to make it easily configurable? I think we want async errors to fully redbox at FB (crash in prod) because otherwise it leaves the app in a potentially undefined state, but other groups may not want that.

And if it ends up crashing all the time, then we can switch to the softer behavior as well while we fix things :)

@sahrens YellowBox allows ignoring errors. Maybe we can do similar for Redbox.

@satya164, the redbox also allows ignoring of error, doesn't it? Just hit escape or 'cancel' on the redbox and it closes.

@ericvicenti I mean ignore them so that they don't appear at all,
console.ignoredYellowBox = ['Some warning text you want to ignore'];

Have we reached any consensus? Do you want to wait until some sort of ignore configuration setting is added before landing this?

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with Canny! Check out its new home: https://react-native.canny.io/feature-requests/p/rfc-trigger-red-boxes-for-all-promise-rejections


LAD 4.0 is not writing to EventHub sink

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

ID: 13249027-1524-dd0c-9d83-15a9733a3d19
Version Independent ID: bf8a2d1a-8d9b-228a-6919-16127f315d40
Content: Azure Compute - Linux diagnostic extension 4.0 - Azure Virtual Machines
Content Source: articles/virtual-machines/extensions/diagnostics-linux.md
Service: virtual-machines
Sub-service: extensions
GitHub Login: @amjads1
Microsoft Alias: amjads

Hi, I'm following the article on Linux Diagnostic Extension 4.0 to install LAD 4.0 on a virtual machine and publish the syslog messages to an EventHub.
In the VM logs, I can see the extension is getting the following error while publishing to the EventHub:
2021-07-01T09:05:30.8460770Z: MDSCMD Error:
EH publish to https://ehns-adpoc.servicebus.windows.net/eh-syslogmsgs/messages
Expiry and access information: '&se=07/01/2031 09:02:52&skn=publishSasKey'
failed with status=PublicationFailedWithAuthError. isAsync=false

My sasURL looks like below:
https://ehns-adpoc.servicebus.windows.net/eh-syslogmsgs?sr=ehns-adpoc.servicebus.windows.net%2feh-syslogmsgs&sig=<sig_string>&se=1940685658&skn=publishSasKey

From the VM, I've curl'ed to the https://ehns-adpoc.servicebus.windows.net/eh-syslogmsgs/messages using HTTP POST method and the header Authorization: SharedAccessSignature <the above SAS token after the ? symbol> and it returned 201
I've used the following Powershell code to generate the sasUrl:
(New-AzEventHubAuthorizationRuleSASToken -AuthorizationRuleId (New-AzEventHubAuthorizationRule -ResourceGroupName $resourceGroupName -Namespace $eventHubNamespaceName -EventHub $eventHubName -Name publishSasKey -Rights @("Send")).Id -KeyType Primary -ExpiryTime $([System.DateTime]::Now.AddYears(10))).SharedAccessSignature.Replace('Primary','publishSasKey')

Thanks for the ask  . At this time we are reviewing the ask and we will reply back as soon as we can .

Can I provide any further info on this to facilitate  the investigation?

Any update on this please?

@saptarshibasu
Firstly, apologies for the delay in responding here and any inconvenience this issue may have caused.
I could not reproduce the issue at my end.
This issue needs deeper investigation. Support team will be able to check and help on this. I would recommend you to open a azure support case.
If you don't have the ability to open a technical support ticket, please let me know so that I can help with that.

@VikasPullagura-MSFT Thank you for your response. However, I'm not on a support plan.

@saptarshibasu
You can email me with Subject as "ATTN: Vikas" at AzCommunity[at]microsoft[dot]com and provide me with your SubscriptionID and link to this thread. I can then enable your subscription for one time free support. I will share further instructions over the email.
Also, once you get the issue fixed, request you to reply back here on the thread with the resolution steps for the benefit of the community.

@saptarshibasu
Thanks for reaching out over email. I have shared instructions over email.
Also, once you get the issue fixed, request you to reply back here on the thread with the resolution steps for the benefit of the community.


[LORO-288] [LORO-289] Use override linked list to store the linear cache history within a register

In the current implementation, we must go through all the history to calculate map diff if we are going backward #106
It's slow and the cache takes lots of memory. It can be more efficient with a linked list.
LORO-289

This optimization is already done by using op group


Update private-endpoint-overview.md

adding missing DPS to table

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


chore(type-definitions): until

Thanks @nissoh! I notice that skipUntil should also be named since. Would you mind including that change in this PR, too?

also noticed until in curried form will only accept endSignal type. @TylorS ?

Yep, any seems correct there.  Good catch.  This LGTM.  Any concerns, @TylorS?


fix react.py #276

fix #276

Hello,
Any idea when this will be merged?
Thank you!

Thank you @dannyoo ! I just need this to be temporarily backward compatible.
Can we do: try/except, if there's .passages use that, else do this new behavior? I know that's slightly hacky but we may need it for a while. We can make it deprecated / issue a warning.

Will merge otherwise after that!

LGTM for the PR, it'd be great to take the changes from @smwitkowski .


Getting headers before response is completed

I was using phantomJs before and phantom would fail quickly if it found out the resource was a PDF file.  I'm trying to recreate the same using the chrome remote interface but I need to get to the headers before the responseReceived is called.  I was hoping that the Network.webSocketHandshakeResponseReceived was the method I was looking for but it doesn't look like it is fired.  Do you know if this is just a versioning issue?  and if I can get it working do you know if this will help me to get the headers before the content is downloaded?

I was hoping that the Network.webSocketHandshakeResponseReceived was the method I was looking for but it doesn't look like it is fired.

I don't think the PDF is being served via WebSocket.
If you want to abort a request then you have to intercept it, take a look at this wiki recipe. There you can access the request (URL, headers, etc.) via the request field of the Network.requestIntercepted event.
Hope it helps.

Thanks, actually I have already tried this but it doesn't appear to work although I'd already followed this example.  I'm guessing it's a version issue, I'm on 59.0.3071.115, I'll try chrome beta and hopefully get some more luck, I already have the latest version of chrome-remote-interface 0.24.2

I'm guessing it's a version issue, I'm on 59.0.3071.115

The wiki clearly states: "(This requires Chrome >= 61)".

That is true :)


pip is upgraded to latest version after installing pipenv

https://github.com/heroku/heroku-buildpack-python/blob/d18f1fedd87c209d47969e5f22bf1d6767fcbb00/bin/steps/pipenv#L47
The pip is upgraded to latest version when installing pipenv. The pip version should be pinned to have stable and repeatable build environment.
Othewise, unexpected failures may happen even there is no update in codebase, such as in https://github.com/pypa/pipenv/issues/2924.

If the buildpack is upgrading pip when installing pipenv, it is likely a consequence of using the default 'upgrade strategy', which before pip 10 was set to eager, but can be set to only-if-needed (which I see you are accurately capturing in your PR)
Technically, this is a bug on our end due to a gap in one of the libraries we broke out of pipenv in the last few months -- it was failing to respect our vendored / patched pip version.  That should be fixed in the new release

After digging into this issue separately, I came to this same conclusion - 🌮 's for you all, thanks for your commits. I wish I could merge #763 directly, but unfortunately I have quite a bit of work to do still to make the tests more contributor-friendly.
The solution can be found at #769; it's being prepped for review and I'm targeting a release this week 👍

definitely hold off on upgrading pipenv until the next iteration so we can make sure the bugs are sorted out -- we had a lot of changes in the release so just want to make sure we capture everything major, but this release will bring a lot of consistency to the resolver
Thanks for your patience and sorry for the issues! I am a consumer of heroku enterprise at work myself so I have some stake in making sure this works


Fix headers not being usable in sessionId template

🚨 IMPORTANT: Please do not create a Pull Request without creating an issue first.
Any change needs to be discussed before proceeding. Failure to do so may result in the rejection of
the pull request.
Description
Makes headers usable in session id when using node's fetch.
Fixes #5102
Related #4109
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
This has only been tested locally, I was not able to write a proper test case with a fetch Request object.
However you can reproduce it with the steps from #5102.
Test Environment:

OS: MacOS
@graphql-mesh/plugin-response-cache: 0.2.7
NodeJS: 16.x

Checklist:

[x] I have followed the
CONTRIBUTING doc and the
style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests and linter rules pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Hmm we shouldn't need that actually;
https://github.com/Urigo/graphql-mesh/blob/master/packages/runtime/src/get-mesh.ts#L173
Then somehow this line doesn't work as it should be.

Maybe it's because we're not getting the "context" in the session callback but the actual request. This is from @graphql-yoga/plugin-response-cache which is calling the the session callback.

And the context only seems to be getting built after the sessionid and cache key were generated:

Should I report this bug for the yoga project instead?


Install fails with 404 for https://github.com/kubernetes-sigs/krew/releases/latest/download/.tar.gz

Operating system and version: Ubuntu 20.4
nvm debug output:

not applicable

nvm ls output:

not applicable

How did you install nvm?

curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/master/install.sh | bash
curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.39.1/install.sh | bash

What steps did you perform?
I already have the nvm installed. I ran the install command again (It is part of the bigger setup Makefile, I am possitive it was working earlier).
What happened?
I receive this output
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 15687  100 15687    0     0   239k      0 --:--:-- --:--:-- --:--:--  239k
=> nvm is already installed in /home/user/.nvm, trying to update using git
=> => Compressing and cleaning up git repository

=> nvm source string already in /home/user/.bashrc
=> bash_completion source string already in /home/user/.bashrc
=> Close and reopen your terminal to start using nvm or run the following to use it now:

export NVM_DIR="$HOME/.nvm"
[ -s "$NVM_DIR/nvm.sh" ] && \. "$NVM_DIR/nvm.sh"  # This loads nvm
[ -s "$NVM_DIR/bash_completion" ] && \. "$NVM_DIR/bash_completion"  # This loads nvm bash_completion
+ cd ''
+ OS=
+ ARCH='\(64\)\?.*/\1\2/'\'' -e '\''s/aarch64arm64/'\'')'
+ KREW=krew-_
+ curl -fsSLO https://github.com/kubernetes-sigs/krew/releases/latest/download/.tar.gz
curl: (22) The requested URL returned error: 404
make: *** [Makefile:194: /home/user/.make/init-wsl] Error 22

What did you expect to happen?
To update nvm or at least don't do anything, but not to fail.
Is there anything in any of your profile files that modifies the PATH?

No

If you are having installation issues, or getting "N/A", what does curl -I --compressed -v https://nodejs.org/dist/ print out?

curl -I --compressed -v https://nodejs.org/dist/
*   Trying <IP_ADDRESS>:443...
* TCP_NODELAY set
* Connected to nodejs.org (<IP_ADDRESS>) port 443 (#0)
* ALPN, offering h2
* ALPN, offering http/1.1
* successfully set certificate verify locations:
*   CAfile: /etc/ssl/certs/ca-certificates.crt
  CApath: /etc/ssl/certs
* TLSv1.3 (OUT), TLS handshake, Client hello (1):
* TLSv1.3 (IN), TLS handshake, Server hello (2):
* TLSv1.3 (IN), TLS handshake, Encrypted Extensions (8):
* TLSv1.3 (IN), TLS handshake, Certificate (11):
* TLSv1.3 (IN), TLS handshake, CERT verify (15):
* TLSv1.3 (IN), TLS handshake, Finished (20):
* TLSv1.3 (OUT), TLS change cipher, Change cipher spec (1):
* TLSv1.3 (OUT), TLS handshake, Finished (20):
* SSL connection using TLSv1.3 / TLS_AES_256_GCM_SHA384
* ALPN, server accepted to use h2
* Server certificate:
*  subject: CN=*.nodejs.org
*  start date: Jan 11 00:00:00 2022 GMT
*  expire date: Feb 11 23:59:59 2023 GMT
*  subjectAltName: host "nodejs.org" matched cert's "nodejs.org"
*  issuer: C=GB; ST=Greater Manchester; L=Salford; O=Sectigo Limited; CN=Sectigo RSA Domain Validation Secure Server CA
*  SSL certificate verify ok.
* Using HTTP2, server supports multi-use
* Connection state changed (HTTP/2 confirmed)
* Copying HTTP/2 data in stream buffer to connection buffer after upgrade: len=0
* Using Stream ID: 1 (easy handle 0x55899ffc8880)
> HEAD /dist/ HTTP/2
> Host: nodejs.org
> user-agent: curl/7.68.0
> accept: */*
> accept-encoding: deflate, gzip, br
>
* TLSv1.3 (IN), TLS handshake, Newsession Ticket (4):
* TLSv1.3 (IN), TLS handshake, Newsession Ticket (4):
* old SSL session ID is stale, removing
* Connection state changed (MAX_CONCURRENT_STREAMS == 256)!
< HTTP/2 200
HTTP/2 200
< date: Fri, 08 Apr 2022 11:31:07 GMT
date: Fri, 08 Apr 2022 11:31:07 GMT
< content-type: text/html
content-type: text/html
< last-modified: Fri, 08 Apr 2022 07:35:10 GMT
last-modified: Fri, 08 Apr 2022 07:35:10 GMT
< cache-control: max-age=14400
cache-control: max-age=14400
< cf-cache-status: HIT
cf-cache-status: HIT
< age: 12864
age: 12864
< expect-ct: max-age=604800, report-uri="https://report-uri.cloudflare.com/cdn-cgi/beacon/expect-ct"
expect-ct: max-age=604800, report-uri="https://report-uri.cloudflare.com/cdn-cgi/beacon/expect-ct"
< vary: Accept-Encoding
vary: Accept-Encoding
< server: cloudflare
server: cloudflare
< cf-ray: 6f8aacc3bc2927a0-PRG
cf-ray: 6f8aacc3bc2927a0-PRG
< content-encoding: br
content-encoding: br

<
* Connection #0 to host nodejs.org left intact

Don’t install nvm from master, only from a tagged release.
I assume the curl for kubernetes stuff is unrelated.


Save and Load Data to correct file name

Save and Load data (change file name) - should not be addresbook.json. Basically remove all traces of addressbook

Fix by #59.


Add GoCD chart

This adds a chart for GoCD, a CI/CD server from Thoughtworks.
I've built GoCD server and agent images that work a bit better than the official ones (namely, the official images don't log to STDOUT).

For your consideration @viglesiasce @lachie83 @mgoodness

@k8s-bot ok to test

Marking as stale. @seanknox please update within the next week.

Thanks Vic and sorry I haven't been able to devote more time to it. Closing.

@seanknox you could put this in incubator if you want to pick it up again later


DOC describe LBFGS-B logging behavior, including iterate.dat file

I found out that optimize.fmin_l_bfgs_b writes logging info to a file called iterate.dat and wished the docstring would have warned me about that, so I updated the docstring.
Please think about this "feature" before accepting the patch; I don't know any Fortran, but I'd happily learn as much as I need to tear out this behavior. Having these files strewn over my directories is irritating, but there's currently no way to get detailed logging info without generating them.

Merged in 37d1c4b894146
Yes, that logging behavior is quite ugly. The code could be modified to use a callback, however, instead.


Add syntax highlighting to examples in ockam command help.

Currently
In the help for various ockam command we're adding examples.
For example ockam

Also try:
ockam secure-channel create --help

Desired
The examples section, which is usually made of shell commands, we want to make it have colors.
Syntect seems like a good crate to use for this, but there may be others
https://crates.io/crates/syntect
The code for the above example is here https://github.com/build-trust/ockam/blob/develop/implementations/rust/ockam/ockam_command/src/lib.rs#L64-L77

We love helping new contributors!
If you have questions or need help as you work on your first Ockam contribution, please leave a comment on this discussion.
If you're looking for other issues to contribute to, checkout this discussion and labels -  https://github.com/build-trust/ockam/labels/good first issue or https://github.com/build-trust/ockam/labels/help wanted

Added in https://github.com/build-trust/ockam/pull/3364
Thank you @jlucktay 🥁


#Suggestion

Room With a View Kotlin doesn't have an option to delete saved words in room db while this function is available in Java tutorial. It would be very useful for beginners if this git repository as well as codelab include this topic.

The DAO exposes a deleteAll method


(Fix) - 772 safe zero balances

Closes #772
Description

This PR solves the problem of the ticket and other problems like if the user has selected a currency for a safe and then changes to another save, that currency will be selected also for the other safe. This allows to safe that info for each safe.

For doing that, I've refactored the currencyValues model to make it safeAddress dependant

Travis automatic deployment:
https://pr832--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr832--safereact.review.gnosisdev.com/app

Looks good to me

Travis automatic deployment:
https://pr832--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr832--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr832--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr832--safereact.review.gnosisdev.com/app

@lukasschor, are you Ok with delivering this fix with #878? Or shall we hold it until the next release?

Wow, this is a big improvement! Good job @Agupane
I'm fine including it in v2.1.0. 👍


JSON Files in VizWiz-Classification Dataset

Thank you for wonderful work!
I would like to use the training and validation datasets. Specifically, I am looking to obtain the correct labels for each image in these datasets.
While the JSON files listed on the webpage include the filenames of each image, such as "VizWiz_train_00002601.jpg", they do not appear to contain the corresponding WordNet IDs or labels for each image. Could you please provide information on how to obtain these labels?
Thank you for your assistance, and I look forward to your response.

Thank you for using our work! We did not publish the labels, but you can run the evaluation on your predictions using the EvalAI server.

Thank you for your prompt response.
I understand that the labels for the test data can be obtained through the EvalAI server. However, I am specifically interested in obtaining the correct labels for the training dataset.
Without access to the correct labels for the training data, it would be impossible to train the model effectively. Could you please confirm whether it is possible to provide the labels for the training data?

Our dataset consists only of test data and currently does not have any annotations for the training data. Thank you.

I believe that this dataset will be very useful in the field of machine learning in the future. Although the current dataset is designed for zero-shot classification tasks, I am looking forward to the release of labeled training data.
Thank you very much for your assistance.


Gas fee settings are not very intelligent, it is recommended to add low, medium, high, and recommended gas

Describe the feature request
Users may passively accept high gas fees for recommendations, and many users may not be willing to accept high gas fee transactions. If the user sets the Gas Fee arbitrarily, the transaction is likely to fail because the user does not know how much gas is needed for the current network。So customizing this option is almost useless
Describe alternatives you've considered
The platform should be more intelligent, calculating low, medium, and high gas fees for users to choose from

Additional context
This monotonous setting

hey @KorbinianK why my issue not in "https://github.com/orgs/taikoxyz/projects/9/views/25"


#508 add hasLogger method to Monolog\Registry

@Seldaek can we get this PR merged?

Yup sorry I missed this new PR, thanks!

For reference purposes.
This is a patch for issue https://github.com/Seldaek/monolog/issues/508

hi @Seldaek , when can we expect this to be released? When are you planning to cut a new tag?

I'll need some more time to review other open PRs before tagging, hopefully around this weekend?

Ok, sounds good I was wondering what your process was and wanted to make sure it did not fall through the cracks :grin:
Thanks for you quick answer!

1.13 is out!


Co-authors in template

Can anyone guide me on a way to properly add multiple authors to a post?

I don't think that your problem is a Timber issue or bug in particular but to give you an idea of a solution to your problem, use custom fields.

@shiftsave I recommend creating an object that extends TimberPost inside of which add a custom `authors

@shiftsave You will be needing the Co-Authors Plus plugin for that solution to work.

Thanks I'll try this!

Just noting that if a TimberPost had an authors attribute (e.g. {{ post.authors }}, this commit would've broken it as it references the co-authors plugin now instead of the variable.


Declarative support for PostgreSQL major online upgrades

Currently, Major upgrades of PostgreSQL can be done in an imperative way, by relying on the native logical replication capabilities and the cnpg plugin. See: https://www.gabrielebartolini.it/articles/2024/03/cloudnativepg-recipe-5-how-to-migrate-your-postgresql-database-in-kubernetes-with-~0-downtime-from-anywhere/
However, we should provide a way to support major upgrades just by changing the major field like in the following example:
apiVersion: postgresql.cnpg.io/v1
kind: Cluster
metadata:
  name: cluster-example
spec:
  instances: 3
  imageCatalogRef:
    apiGroup: postgresql.cnpg.io
    kind: ImageCatalog
    name: postgresql
    major: 16
  storage:
    size: 1Gi

To:
apiVersion: postgresql.cnpg.io/v1
kind: Cluster
metadata:
  name: cluster-example
spec:
  instances: 3
  imageCatalogRef:
    apiGroup: postgresql.cnpg.io
    kind: ImageCatalog
    name: postgresql
    major: 17
  storage:
    size: 1Gi

Alternative approaches are valid too, for example through CRDs that evoke imperative names (like upgrades) and that are useful too as they prepare the underlying PVCs for the new major.

Any updates on this?

It's on the roadmap for 1.26.0 https://github.com/orgs/cloudnative-pg/projects/1/views/1 - as is also noted by the events listed above your comment ;)


Error comparing versions

Brief description of your issue
See the discussion -

https://github.com/microsoft/winget-pkgs/discussions/192959

TL;DR
    RequireLessThan("1-rc", "1"); // Test case pases
    RequireLessThan("1.2-rc", "1.2"); // Test case passes
    RequireLessThan("1.0-rc", "1.0"); // Test case fails

I've linked my branch that I've used for testing. The only test that is currently failing is the one mentioned in the initial comment.

Test results
PS E:\winget-cli\src\x64\Debug\AppInstallerCLITests> .\AppInstallerCLITests.exe version*
Filters: version*

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
AppInstallerCLITests.exe is a Catch v2.11.0 host application.
Run with -? for options

-------------------------------------------------------------------------------
VersionCompare
-------------------------------------------------------------------------------
E:\winget-cli\src\AppInstallerCLITests\Versions.cpp(187)
...............................................................................

E:\winget-cli\src\AppInstallerCLITests\Versions.cpp(145): FAILED:
  REQUIRE( vA < vB )
with expansion:
  {?} < {?}

===============================================================================
test cases:  13 |  12 passed | 1 failed
assertions: 271 | 270 passed | 1 failed

I'm certain that each individual part is being parsed correctly
I believe that the individual part comparison is correct, but am not certain


I have imported the watson-developer-cloud and its working in the python terminal, but while importing it in spyder, it shows an error. What is the problem here? (I am using windows)

Description
What steps will reproduce the problem?

What is the expected output? What do you see instead?
Please provide any additional information below
Version and main components

Spyder Version: 3.2.6
Python Version: 3.6.4
Qt Versions: 5.6.2, PyQt5 5.6 on Windows

Dependencies
IPython >=4.0    :  6.2.1 (OK)
cython >=0.21    :  0.27.3 (OK)
jedi >=0.9.0     :  0.11.1 (OK)
nbconvert >=4.0  :  5.3.1 (OK)
numpy >=1.7      :  1.14.0 (OK)
pandas >=0.13.1  :  0.22.0 (OK)
pycodestyle >=2.3:  2.3.1 (OK)
pyflakes >=0.6.0 :  1.6.0 (OK)
pygments >=2.0   :  2.2.0 (OK)
pylint >=0.25    :  1.8.2 (OK)
qtconsole >=4.2.0:  4.3.1 (OK)
rope >=0.9.4     :  0.10.7 (OK)
sphinx >=0.6.6   :  1.6.6 (OK)
sympy >=0.7.3    :  1.1.1 (OK)

Thanks for reporting. Please first update Spyder to the latest version (via conda update spyder if you're running Anaconda/Miniconda, or pip update spyder if installed via pip), and try again. Then, open Qtconsole (via jupyter qtconsole) and see if importing works there.

Closed due to lack of response. If you can provide more information, please re-open and do so. Thanks.


Fix #120: Test and fix constructor arg conflict in FindExpressionRoot

Not sure why codecov thinks that coverage went down


Issue with iOS

Hi,
So iv been trying to debug this the last few weeks. On iOS, connecting to voice and talking works great. BUT open any new map, even the menu map, 2-7 seconds later, the app crashes.
None of this happens if i never connect to voice.
Any hints as to what might be causing this?

Adding the logs folder in the agora settings to ProjectLogsDir>voice.log also doesnt save any file on ios

@WinterPu some help please

You could build an iOS app and select [Debug - Attach to the Process] to trace the logs. I suspect that you may be triggering a garbage collection or neglecting to perform certain operations on the game's main thread. However, I'm not sure. You could give it a try.

@WinterPu I had managed to fix this in another app, but getting the smae issue in a new app. THe issue previously was that i was setting the outer for the RTCEngine and EventHandler as the player controller - which of course is destroyed once we start a new map.
Now im setting the outer as the game instance, but crash still occurs.

You could try sth like this: AgoraUERtcEngine::Get()->setAudioScenario(agora::rtc::AUDIO_SCENARIO_TYPE::AUDIO_SCENARIO_GAME_STREAMING);
Or wait for the 4.5.0

You could try sth like this: AgoraUERtcEngine::Get()->setAudioScenario(agora::rtc::AUDIO_SCENARIO_TYPE::AUDIO_SCENARIO_GAME_STREAMING);
Or wait for the 4.5.0 https://github.com/AgoraIO-Extensions/Agora-Unreal-RTC-SDK/tree/release/4.5.0

How do i do this in blueprints?
Also, should i just Use 4.5.0 now?

Yeah, we are still testing it. It is coming soon.

4.5.0 has been released.


still starting

Good morning, Since the extension is free I can't use it in VScode, it always gives me the same error.

@germanlagomarsino Do you have the persistent terminal sessions setting enabled?
https://code.visualstudio.com/docs/terminal/advanced#_persistent-sessions
I would also look in the output page for PowerShell Pro Tools for logging information.

Disabling persistent sessions solved the problem. thank you so much!

Great! I added an error and option to disable it in 2024.7.5. I think others have been running into this as well.
https://github.com/ironmansoftware/powershell-pro-tools/releases/tag/vscode-2024.7.5


Fresh install get errors with permisions logs

Describe the bug
Deploy with docker is not well end.
Errors with permissions in storage/logs /framework/sessions an views
To Reproduce
Steps to reproduce the behavior:

Go to http://localhost:8080/
See error

I need add full permisison on this folders in my Ubuntu local path
Desktop (please complete the following information):

OS: Ubuntu 18.04

Best  regards

Best if run
find storage/ -type d -exec chmod 777 {} \;


[zh-cn] sync secrets-good-practices apiserver-encryption.v1 download

#       modified:   content/zh-cn/docs/concepts/security/secrets-good-practices.md
#       modified:   content/zh-cn/docs/reference/config-api/apiserver-encryption.v1.md
#       modified:   content/zh-cn/releases/download.md

/lgtm


When using Navigator 2.0 and a RouteAware widget with RouteObserver, the widget is rebuilt after its route is popped

Steps to Reproduce

Open this DartPad
Click "Run" on the upper right
Click "Console" on the lower left to open the debug console
When the UI appears, click each of "Open Navigator 2.0 Route" and "Open Navigator 1.0 Route"
Click to pop each route
Observe the different output in the debug console

Expected results:
When using a Navigator 2.0/Router route that launches a RouteAware widget, I don't expect it to be rebuilt after it is popped. It's different behavior than using a Navigator 1.0 route.
opening Navigator 2.0 Route
didChangeDependencies
build
did pop
dispose
---------------

Actual results:
The RouteAware widget shown via the Navigator 2.0 route gets rebuilt after it is popped.
opening Navigator 2.0 Route
didChangeDependencies
build
did pop
didChangeDependencies
build
dispose
---------------

If you uncheck the "Use route observer?" checkbox, it doesn't get rebuilt when the route is popped.

Code sample

See https://dartpad.dev/?id=9c70e9ccba24e889d8a190a74671bb42

Logs
N/A, repro in DartPad

Thanks for the report. Seeing same behavior upon running the provided code sample:
flutter: opening Navigator 2.0 Route
flutter: didChangeDependencies
flutter: build
flutter: did pop
flutter: didChangeDependencies
flutter: build
flutter: dispose
flutter: ---------------
flutter: opening Navigator 1.0 Route
flutter: didChangeDependencies
flutter: build
flutter: did pop
flutter: dispose
flutter: ---------------

 stable, master flutter doctor -v
[✓] Flutter (Channel stable, 3.3.9, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 3.3.9 on channel stable at
      /Users/dhs/documents/fluttersdk/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b8f7f1f986 (12 days ago), 2022-11-23 06:43:51 +0900
    • Engine revision 8f2221fbef
    • Dart version 2.18.5
    • DevTools version 2.15.0

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.7.0-8.0.pre.5, on macOS 12.2.1 21D62 darwin-x64,
    locale en-GB)
    • Flutter version 3.7.0-8.0.pre.5 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f07db4018a (3 hours ago), 2022-12-14 03:25:49 +0200
    • Engine revision 14194c40ec
    • Dart version 3.0.0 (build 3.0.0-19.0.dev)
    • DevTools version 2.20.0
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.


add support for vcpkg-configuration.json's baseline

Changes:

the vcpkgGitCommitId input is optional, and the Git commit id can be fetched from :

vcpkg-configuration.json:
- the file is searched with provided glob expression in vcpkgConfigurationJsonGlob input.
- the baseline is searched using JSONPath $["default-registry"]["baseline"]
vcpkg.json:
- the file is searched as usual/already (glob + ignore input).
- the baseline is searched using JSONPaths (first hit wins):
-   $["builtin-baseline"]
-   $['default-registry']['baseline']
-   $['vcpkg-configuration']['default-registry']['baseline']

I'm unsure where you discovered vcpkg.json > $['default-registry']['baseline'] its associated schema doesn't mention a default-registry key in the root object. Two additional test cases for the other two vcpkg.json based baseline searches would probably be useful.
Otherwise LGTM 👍


Is frequency just 2 $Pi$ / Time?

Answered here https://github.com/ubsuny/23-Homework5G5/issues/55

Not sure.


Some .html files aren't being generated

@dontlaugh
For example:
https://docs.raku.org/syntax/given.html

Changed the subject - these files don't exist at all, it's not the same as the previous issue for, e.g. /type/Int vs. /type/Int.html

@rawleyfowler Can you point me at the link that contains a reference to 'syntax/given' ?
The secondary plugin does not generate any /syntax files at all. When I was reverse engineering Documentable to create secondaries, it seemed that all syntax data was being renamed as routines. So I need some information about the links to see which files it was intended for them to point at.

The search tends to help with things like this: https://www.google.com/search?q=site%3Adocs.raku.org+given
And luckily we even have an archive.org save: https://web.archive.org/web/20220523025237/https://docs.raku.org/syntax/given

The docs contain syntax-scoped links, an example from the control rakupod: https://github.com/Raku/doc/blob/59ffd079edc0d227ffe32ba9e12167f6fe7471b5/doc/Language/control.pod6#L661

working on the syntax/ files.
Lots more errors have been generated recently. @coke are there changes being made to content files?

@coke These links are the result of what seems to me to be an awful hack and bad documentation standards.
Basically, a secondary file is generated by Documentable as syntax/filename if and only if the header is written at =headn X<something |Syntax,file_name> Only a header with X<> markup and no other text.
Further, there is a search for things that are considered 'syntax' when analysing other headings, but these are moved to 'routines' because of the 'syntax' hack above.
For the sake of keeping existing links, I'm making secondaries consistent. However, in the long term, the documentation needs to be edited to remove X<> from headings. It was an idiom adopted before HTML 2.

syntax/ files like syntax/role syntax/class, which correspond to errors are now generated.
New content visible at new-raku.
Still some corner cases when Format codes are included in the markup.
@2colours @rawleyfowler  would you see if you can find problems.
I tried to find a link to syntax/given as described above, but could not in current new-raku

This is mostly solved by work on the secondaries plugin that got included in PR #229.

resolved


[iOS] [Fixed] - fix multiple modal not working

Summary
When I need to pop up multiple modals at the same time in react native, it doesn't work properly on iOS. It is presented on the native code of react native, iOS native cannot present multiple controllers at the same time
This modification is mainly to change the present form to addChild to solve the problem of adding multiple VC
Changelog
[iOS] [Fixed] - fix multiple modal not working
Test Plan
This modification has been verified in our project and online appstore

Please provide a reproducible test plan, preferably in the form of changes to rn-tester included in this PR.

Please provide a reproducible test plan, preferably in the form of changes to rn-tester included in this

Please provide a reproducible test plan, preferably in the form of changes to rn-tester included in this PR.

Ok, I have time to provide it


Roadmap v3

In this version, there are two main technologies we want to achieve

Collaborative Editing Support
Improved Developer Experience (DX)

HMR (Hot Module Replacement)
Redux DevTools

To support collaborative editing, the existing data structure has undergone significant changes.
It automatically supports lower version JSON data.
We are still considering whether to provide downgrade support.
Additionally, the following improvements are being considered

Enhancing the theme system with basic support for both dark and light modes.
Rewriting the SQL DDL parser.
Basic UI improvements.
Adding functionality similar to macOS's Spotlight.
Adding a diff view.

Apart from these features, we also aim to address the finer details that may have been overlooked.

Good day!

Would love to see Databricks added.

非常棒的项目，向你至敬！

Databricks support would be appreciated, as well as reverse engineering a db. Great work!


AbstractYamlDump.java:35-39: This interface should have 1 more implem...

The puzzle 38-ac09402a in src/main/java/com/amihaiemil/camel/AbstractYamlDump.java (lines 35-39) has to be resolved: This interface should have 1 more implementor class: YamlCollectionDump. This classes will encapsulate and serialize the mentioned type (i.e. Collection). The return type or method represent() should be overridden with the proper subtype.
The puzzle was created by sherif on 29-Jan-17.
Estimate: 30 minutes, role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code.

@amihaiemil please pay attention to this new issue.

@SherifWaly Do you want to do this, when you have time? :D

@SherifWaly Let's wait here, until I fix this issue :D

@SherifWaly I just closed that issue, since the problems were in branch 36, which is not merged yet :D

@amihaiemil I would like to do this task yes :D, but should I wait for branch 36 to be merged or start?

@SherifWaly it's already merged, so it's ok :)

@amihaiemil Great, I will to do this task ASAP :)

will start* :D

@SherifWaly sure, no hurry ;)

The puzzle 38-ac09402a has disappeared from the source code, that's why I closed this issue. //cc @amihaiemil


Appstudio update test-component-pac-wvfo

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-wvfo-on-pull-request has successfully validated your commit.


Down- and up-sampling on KITTI dataset

Hi,
Thanks for open-sourcing this great piece of work!
I have tried to run the code using the KITTI depth completion data as described in the instructions and have had some issues with mismatched tensors. Upon closer inspection, it appears that this arises from lines such as the following (from depthCompleNew.py):
normal2 = F.upsample(normal2, (normal2.size()[2] * 2,normal2.size()[3] * 2),mode='bilinear',align_corners=True)
The mismatch seems to occur because the upsample simply doubles the downsampled size rather than matching the input size, causing inconsistent sizes if the input tensor has a dimension with an odd value.
This is an issue for me since the KITTI data I am using sometimes contains images with an odd number of pixels in one of the dimensions. Since this has not seemed to be an issue for you or others using this dataset, I am wondering if I am supposed to preprocess the KITTI dataset or get the dataset you used from somewhere aside the main KITTI AWS server?
Thanks for the help!

You can pad the input size to even number firstly and crop the output to the original size later.


Can enzyme support rax

Is there a plan to support rax.

I've never heard of that before, but I think #742 will make that possible.


[5.4] Route getParameter() method removed undocumented change

Laravel Version: 5.4.2
PHP Version: N/A
Database Driver & Version: N/A

Description:
Illuminate\Routing\Route::getParameter() public method was removed. Use parameter() method instead.

PR created to update the docs, thanks :)
https://github.com/laravel/docs/pull/3038


[Bug]: SparkTextField: NPE Paint.setColor()

🕵 Is there an existing issue for this?

[X] I have searched the existing issues

What version of spark are you using?
1.0.0
🐛 What happened?
Crash: NPE on a setColor() on a null Paint.
Relevant logcat output
Fatal Exception: java.lang.NullPointerException
Attempt to invoke interface method 'void androidx.compose.ui.graphics.Paint.setColor-8_81llA(long)' on a null object reference
androidx.compose.foundation.text.TextFieldState.update-fnh65Uc (CoreTextField.kt:933)
androidx.compose.foundation.text.CoreTextFieldKt.CoreTextField (CoreTextField.kt:250)
androidx.compose.foundation.text.BasicTextFieldKt.BasicTextField (BasicTextField.kt:161)
com.adevinta.spark.components.textfields.SparkTextFieldKt$SparkTextField$5.invoke (SparkTextField.kt:212)
com.adevinta.spark.components.textfields.SparkTextFieldKt$SparkTextField$5.invoke (SparkTextField.kt:210)

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I failed to locate any Paint/setColor usages in SparkTextField.
Maybe this is a Compose bug?

Seems like not something that can happen by looking at the source. And since we have only one user with this issue I think we can close it as not fixable on our end


Preserve selection after aligning

Would it make sense to make valign preserve the selection of the lines it aligned once it's done? This would help when chaining multiple commands, e.g. valign and then sort-lines.

I like the general premise, but it is kind of difficult given that the selection range may or may not change based on the alignment.

Given the simplest of examples the character range is only off by 6 (Though, in the example, the current selection would not be affected).
However each line may have space prefixed, postfixed, or injected adjacent to…
I would recommend coming up with use cases/test cases if you want to continue exploring this.
FWIW: I sort lines, then valign as the cursor after sorting, leaves it in the same block—valign doesn't need selection.


Change overflow:scroll to overflow:auto to stop horizontal scrollbars

Change overflow from scroll to auto so scrollbars only appear when they're needed.

Thank you!


Memory management attributes on views passed to MDM unclear from spec

When a UIView* is set as a target, should it have a strong or weak pointer?

All targets are strongly-retained by the scheduler.


ci: avoid panic in sdk building

Using functions that can error in Container.With is a bad idea. So, what we can do is to put the actual computation into an error group (which has the convenient little side-effect that they all get computed in parallel, hopefully improving build times).
This lets us remove the panic, which was crashing the binary. Because of (I think) telemetry draining, this meant that the client job hung forever on this kind of failure.
cc @helderco I know we discussed previously :tada:

Linting failure is from https://github.com/dagger/dagger/pull/7153.


dual branch and  output score normalization

Hey!
Thanks for beeing so responsive.
I have another question again...
in the dual branch,
FC score is a fully connect that ends with a RELU and fc_weight branch ends with a sigmoid. Am I correct that it makes score q between 0 and + infinity.
Is the network supposed to output normalized scores?

Hi! It is a good question! In different perceptual scale, it has different quality score. For easily training the network, we normalize the all dataset score to 0-1. The output should between 0-1. If you want to obtain the original score, you can wrtie a reversed function of normalization.

Thanks a lot!
I am trying to use your network as a backbone, using koniq - pretrained.
I am doing a pairwise finetuning (siamese network scale) . where my data is paired images, and using ranking margin loss.
The think is I see the network output values between -5 and 5... so I was wondering how could be.

Hi, actually, from the code of the dual branch, even though we normalize the label score, it could be that the output is greater than 1.
I think this is due to fc_score branch, that ends with a RELU meaning fc_score value could be greater than 1.
if fc_score is greater than 1, then the  dual branch output value _s = torch.sum(f * w) / torch.sum(w), would also be greater than 1.
Am I correct?


Internet facing ELB and private controllers

Hello,
Sorry for maybe dummy question, but I cannot figure out from the manuals and cluster.yml whether it's possible to have a ELB and Controllers on separate EC2 instances so that ELB would internet-facing and controllers would in private networks. I assume ELB managed by kube-aws, not manually created and provided via apiEndpoints/loadBalancer/id.
Thank you!

Hi @vbogretsov you need some public subnets tagged with KuberneetsCluster=your_cluster_name

Hello @camilb!
Thank You, it works!


[Snyk] Security upgrade pug from 2.0.3 to 3.0.1

Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.

Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

package.json
package-lock.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

691/1000  Why? Recently disclosed, Has a fix available, CVSS 8.1
Remote Code Execution (RCE) SNYK-JS-PUG-1071616
Yes
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Coverage remained the same at 90.647% when pulling ef4c94976e47c12c79db8777c44ef1bc9a4b619e on snyk-fix-8b280486410c42fa6526fccb12d3388f into c2de2ffe57aa0d02c67cd2dc147446a94a5bb11c on master.


update packages

Related Issue

https://issues.redhat.com/browse/ACM-0000
Description of changes

Update packages

@jlpadilla Just updated to be ahead of CVE's, if packages get outdated.
Can you check why the syncinformers test is failing? Thanks.

Yes, the code changes were due to lint errors as we moved to go1.20

/test test-unit

/retest


[BugFix] Iceberg supports array type

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

run starrocks_fe_unittest

[FE PR Coverage Check]
:disappointed: fail : 0 / 5 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/external/iceberg/IcebergUtil.java
0
5
00.00%
[255, 256, 257, 259, 277]

[FE PR Coverage Check]
:heart_eyes: pass : 4 / 5 (80.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/external/iceberg/IcebergUtil.java
4
5
80.00%
[259]

run starrocks_admit_test

run starrocks_admit_test

[FE PR Coverage Check]
:heart_eyes: pass : 4 / 5 (80.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/external/iceberg/IcebergUtil.java
4
5
80.00%
[259]

[FE PR Coverage Check]
:heart_eyes: pass : 4 / 5 (80.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/external/iceberg/IcebergUtil.java
4
5
80.00%
[259]

[FE PR Coverage Check]
:heart_eyes: pass : 4 / 5 (80.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/external/iceberg/IcebergUtil.java
4
5
80.00%
[259]

[FE PR Coverage Check]
:heart_eyes: pass : 4 / 5 (80.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/external/iceberg/IcebergUtil.java
4
5
80.00%
[259]

[FE PR Coverage Check]
:heart_eyes: pass : 4 / 5 (80.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/external/iceberg/IcebergUtil.java
4
5
80.00%
[259]


Github action for formatting python code

We need github action either for checking formatting on the python code (or for formatting it automatically)

I use Black for both my personal and professional projects, and it's notably good. Should I proceed with integrating it? Or... do you have a personal preference?

@mmabrouk I would like to tackle this one, should i wait for somebody to assign or can i go ahead and start working on it?

Hey @Pajko97 that's awesome :) I will assign you!

@mmabrouk I am testing it out right now, the approach i chose is:

We are going to be using black latest stable version

Simplified, two most important steps are:

After running a check, it formats the files that need formatting
GH action creates a pull request with updated files that need manual
approval so we can check if everything seems fine

For this approach to work, on the org level of the repo we need to:

Go to https://github.com/organizations/YOUR_ORG/settings/actions and
Check Allow GitHub Actions to create and approve pull requests

Let me know if we can do that, and if the chosen approach is fine.

Turns out I just had to enable it in my local fork of the repo. #244 I created a PR for this.

Closed in #244

Issue turns out not to be solved, see comments in #244

Closed now (see discussion in #244)


doc: Update prerequisite: pipelines operator is no longer installed by gitops operator

What type of PR is this?

Uncomment only one  /kind line, and delete the rest.
For example, > /kind bug would simply become: /kind bug

/kind bug
/kind cleanup
/kind failing-test
/kind enhancement
/kind documentation
/kind code-refactoring

What does this PR do / why we need it:
This PR updates the documentation where it talks about day1 prerequisite.  GitOps operator no longer installs Pipelines Operator since v1.1.1
Have you updated the necessary documentation?

[x] Documentation update is required by this PR.
[x] Documentation has been updated.

Which issue(s) this PR fixes:
Fixes #?
How to test changes / Special notes to the reviewer:
Install GitOps operator on a fresh cluster and validate that Pipelines Operator is not installed automatically.   As a result, day1 prerequisite explicitly instructs users to install the Pipelines Operator from OperatorHub.

There's a small typo in this file name (extra space before the extension). Rest of the PR looks good @wtam2018

https://github.com/redhat-developer/kam/blob/0d30009bb8d2c18c3449c4b37da1a3edaa41aa55/pkg/cmd/bootstrap.go#L324

The bootstrap cmd asks users to install the GitOps operator if the pipelines operator is not found. We can now ask them to install the pipelines operator direclty

/lgtm


[question] more open source procedure of lxc runtime - runlxc

Question

Hi,
docs/architecture.md mentions runlxc.  I've read elsewhere that it is a proprietary crio for lxc implementation and that the intent is to open source it.
Does it still exist, and is the plan still to open source it?  If so what is the timeline?

Thanks for your issue. @hallyn
Yes, runlxc is a runtime to receive commands from containerd and translate them to be commands for lxc. In addition, actually when use our runlxc in production environment, we modified some minor part of legacy linux kernel. So it it a strictly general case. With the info above, we slowed down the open source procedure. What is more, we wish to see more community input for this.
Could you share more about the demands for this? I think open source is the first step, meeting your demand is what you care mostly.

I'm interested in kubernetes platforms being able to trivially replace runc with runlxc.

I'm not trying to be curt or coy, but I'm not sure what more info I could give :)  If you have specific questions please ask.  However, I'm getting ready to announce and start on github.com/lxc/crio-lxc.  I'd prefer to re-use something existing if that can be done quickly, but of course when lawyers get involved...


Type definition

Would be lovely to use this with typescript.

I have figured out how to add a definition file myself.
import { fabric } from 'fabric';

declare module 'fabric/fabric-impl' {
    interface Canvas {
        /* fabric-history */
        undo: (callback?: () => void) => void;
        redo: (callback?: () => void) => void;
        clearHistory: () => void;
    }
}

It would be great to also add the event names to the fabric EventNames type. But I havn't figured it out.


Ubuntu 14.04 atom Chinese garbled

when I open the atom editor .the Chinese garbled
below is a picture i captured ...

How could I fix this problem
My File is UTF8 encoded also without BOM. works fine with atom on windows.
So I think it's a problem associated with liunx system.

It seems your file may not  be encoded in utf8
Convent to UTF8 without BOM.

In addition, please make sure you're running Atom 1.0.12 or above.


Pop up in notification

My Setup

Operating System: Ubuntu 16.04
App Version: 2.14.3
Installation type: .deb

[X] I have tested with the latest version
[X] I can simulate the issue easily

Description
You have a pop up notification for each message you receive. Also tray Icon disappeared.
Current Behavior

Expected Behavior
No pup ups at all

same problem here notification become a modal.

Hello @tassoevan
bug still persist in version 2.14.4
Chris

@tassoevan is there any daily build for me to test? :)

@ccfiel Sure. Please look at https://github.com/RocketChat/Rocket.Chat.Electron/releases/tag/2.15.0-develop

@tassoevan I have downloaded it and now notification is working as expected


Fix crash due to mishandling of UTF-8 in progressive download.

Fixes:
Fatal Exception: java.lang.RuntimeException: Failed to create String from JSON
       at com.facebook.react.bridge.queue.NativeRunnable.run(NativeRunnable.java)
       at android.os.Handler.handleCallback(Handler.java:739)
       at android.os.Handler.dispatchMessage(Handler.java:95)
       at com.facebook.react.bridge.queue.MessageQueueThreadHandler.dispatchMessage(MessageQueueThreadHandler.java:31)
       at android.os.Looper.loop(Looper.java:234)
       at com.facebook.react.bridge.queue.MessageQueueThreadImpl$3.run(MessageQueueThreadImpl.java:193)
       at java.lang.Thread.run(Thread.java:818)

JavaScriptCore is very strict about invalid UTF symbols.
So if you pass an invalid UTF-8 string to it the string will be decoded as an empty string.
The current implementation of progressive downloading for Android blindly cuts the response in 8KB chunks.
That could cause a problem in case the last symbol in the chunk is multi-byte.
To prevent it I added a class which determines if this is the case and cut the string in the appropriate place.
A remainder is prepended to the next chunk of data.
This should fix the root cause of this issue:
https://github.com/facebook/react-native/issues/10756
Test plan:
fetch for this http://www.00ksw.org/html/1/1391/614905.html
shouldn't crash android app anymore.

continuous-integration/travis-ci/pr seems to be false-negative.
At least it looks like a bug in the CI from the logs.
I didn't change any c++/ObjC or javascript code to fail xcode tests.

I believe we've had a similar issue elsewhere in the Android codebase. It should be possible to merge/reuse your solution with that.

@javache, could you point me to another place?
I grep the code locally and it looks like NetworkModule is the only place where we split user data to chunks.
I was looking for the String creation with constructors:
https://docs.oracle.com/javase/7/docs/api/java/lang/String.html#String(byte[])
https://docs.oracle.com/javase/7/docs/api/java/lang/String.html#String(byte[], java.nio.charset.Charset)
etc.

Hmm, you're right. I think we just avoided crashing in the C++ layer when receiving invalid input.
An equivalent fix to this was made on iOS a while ago: https://github.com/facebook/react-native/commit/3ac3749ac37bf5f202eab535528f324461029366

Tell me if I could do something to push it forward.
It seems to be a pretty frequent crash.
At least at Skype, it was one of the top crashes.
To make this fool proof I fixed the code causing the crash as well + have a folly slightly patched.
But I think that should be part of the separate commit.

I'll try to have a look at the code, but in general I'm not comfortable with using our own utf8/utf16 decoding solution. I much prefer the iOS solution where we use the system decoding and then try to fall back / trim the end of the string in case it ends in an invalid character.

@javache I don't have a custom encoding, just a validation.
But I got what you mean. The validation could be done in O(1) by analyzing the last 4 symbols.
Didn't notice that initially. But anyway there is a https://docs.oracle.com/javase/7/docs/api/java/nio/charset/CharsetDecoder.html
API which allows doing what is needed with a support of multiple encodings.
I'll update the algorithm to support other encodings as well.
Thank you for feedback.

Is there anything I could do to help to move it to master?
The patch fixes the root cause of one of the most frequent react-native crashes.
The code I wrote is covered by UnitTests and supports multiple encodings.
It works the same way as iOS implementation does.

Rebased.

@hramos, did you find anyone to take a look at this PR?
It's quite a bad crash.


Only apply fallback if GL_DEPTH24_STENCIL8 is defined

Fixes https://github.com/memononen/nanovg/pull/411#issuecomment-331638190
All examples compile and run.

Thanks for the fix!

This fixes my problem as well!


Still getting overnight Springboard restarts

System Memory Reset Fix is installed (as part of unc0ver 3.1.2 on iPhone SE 12.1.1), but I am still getting Springboard restarts almost every night.
Here is the system log from the time of SB restart:
`mmaintenanced[1560] : Launched system high water mark activity
mmaintenanced[1560] : Current UserReclaimable: 70%
mmaintenanced[1560] : : 6848512
mmaintenanced[1560] : : 6258688
mmaintenanced[1560] : : 143294464
mmaintenanced[1560] : : 65454080
mmaintenanced[1560] : : 57622528
mmaintenanced[1560] : : 40009728
mmaintenanced[1560] : : 14729216
mmaintenanced[1560] : : 2752512
mmaintenanced[1560] : User reclaimable limit not crossed
mmaintenanced[1560] : Set SpringBoard memory limit to its inactive limit
kernel[0] : EXC_RESOURCE -> SpringBoard[1137] exceeded mem limit: ActiveHard 60 MB (fatal)
kernel[0] : 20727.735 memorystatus: killing_specific_process pid 1137 [SpringBoard] (per-process-limit 16) - memorystatus_available_pages: 45066`

The fix for mmaintenanced killing SpringBoard seems to be increasing the InactiveHardMemoryLimit for SpringBoard in /System/Library/LaunchDaemons/com.apple.jestsamproperties.NXX.plist. After I increased it from default 60 MB to 120 MB and rebooted, I did not experience any nightly SpringBoard restarts.


Add api to clear domain meta attribute

Signed-off-by: Ofer Levi<EMAIL_ADDRESS>

I'm not sure if I want a separate endpoint just because the java client is not able to send the empty strings to ZMS. We should try to see if we can fix the java client to handle this case correctly.


Show bid placement keypad on detail view?

Came up in discussion as a potential call to action for the bidding process.

Any further thoughts, @katarinabatina ?

Could see the potential! However, it doesn't strike me as a high priority for now

OK, gonna leave this without a milestone and we can revisit it later.


Libraries with Composer and type "path" + wildcard

Hello,
We have a (private) MonoRepository with multiples services and some "components" (shared libraries).
We install these components with the following option in composer
    "repositories": [
        {
            "type": "path",
            "url": "../../components/*/",
            "options": {
                "symlink": true
            }
        },

Apparently Dependabot cannot work because of this.
But the "path" option in Composer is the recommended way for doing this.
Is it a bug ?
Is there an alternative ?
Thanks !
Logs here: https://app.dependabot.com/accounts/Comiteo/update-logs/17088850

Closing this as it's working now

I don't think this is working now (other than via the workaround mentioned)? I'm getting the following error when using a wildcard path: The url supplied for the path (./patched-plugins/*) repository does not exist
Relevant composer config is...
"repositories": [
    ...
    {
        "options": {
            "symlink": false
        },
        "type": "path",
        "url": "./patched-plugins/*"
    }
    ...
]

The workaround isn't really viable when you have more than a handful of libraries.

Sorry, just an update on the above. I've got it working now but via another workaround (I think it's a workaround anyway).

patched-plugins/* was empty and thus you'd get the error
If I put an example sub-folder with valid composer.json in then I don't get the error.

So the path wildcard stuff works fine as long as the path isn't empty, maybe that's expecting I'm not sure 🤷

@infin8x Requesting a re-open.  Currently experiencing this issue.
The `url` supplied for the path (modules/*/*) repository does not exist

I have composer packages in the format modules/VENDOR/PACKAGE and this path repository works fine for regular composer.

I experienced the same error for a path containing a wildcard. In my case the path was having a backslash at the end, which wasn't necessary.
The example below resulted in a dependabot error, but worked for composer:
    "repositories": [
        {
            "type": "path",
            "url": "../src/packages/*/"
        },
    ],

Without a ending backslash it works for both
    "repositories": [
        {
            "type": "path",
            "url": "../src/packages/*"
        },
    ],

@TwanVermeulen do you want to file ☝ as a new issue here? Comments on old issues don't get looked at/tracked...


Add Events client method

Fixes #1381
Signed-off-by: Michael Crosby<EMAIL_ADDRESS>

Codecov Report

Merging #1423 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #1423   +/-   ##
======================================
  Coverage    40.8%   40.8%
======================================
  Files          23      23
  Lines        2924    2924
======================================
  Hits         1193    1193
  Misses       1453    1453
  Partials      278     278

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 f052817...421c607. Read the comment docs.

@mlaventure you can if you want ;)

LGTM


Error displaying the thank you message

Hi !, here's what I get after a subscription...

What is the language that you have configured in your page?
Is this only happeing for this message or also for the other messages?

French is the language for the whole website
I don't know, what are the other messages ? Afaik it does it for subscribe and unsubscribe

(Assuming you followed the Setup instructions)
A copy of the languages: section of your user/config/system.yaml would be helpful in helping you to debug.

I did follow the Setup instructions,
Here is the languages: section of my user/config/system.yaml
languages:     supported:       - fr     include_default_lang: false   translations: false   translations_fallback: true   session_store_active: true   http_accept_language: true   override_locale: false

As far as I understand these docs you need to set system.languages.translations to tru to be able to get translated stings.
Can you try to set it to true and see if it helps?

Sorry for the late delay. Is the problem solved or is there still need for help? Please let me know.


fix: hide sensitive values

Summary
utils_deep_merge_yaml data source used in main.tf of the root module outputs in clear text values that are marked sensitive. With this PR, sensitive values are passed to root module and submodule via the sensitive_values variable therefore no longer included in the deep merge.
Test platform

[x] Azure
[ ] AWS
[x] KIND

I would call the variable something like sensitive_helm_parameters instead of sensitive_values, because the documentation calls them that : https://registry.terraform.io/providers/jojand/argocd/latest/docs/resources/application#parameter
I think this would help to distinguish "helm values" which can be any structured data, from "parameters" which are just key/value pairs and are somehow "injected" in the values.

I would call the variable something like sensitive_helm_parameters instead of sensitive_values, because the documentation calls them that : https://registry.terraform.io/providers/jojand/argocd/latest/docs/resources/application#parameter
I think this would help to distinguish "helm values" which can be any structured data, from "parameters" which are just key/value pairs and are somehow "injected" in the values.

Good point.


Greedy algorithm to find minimum number of coins

Greetings sir/Mam,
I would like to contribute the code for this issue in C++/CPP.
The code is ready. I have added the code , along with proper explanation of the algorithm , test cases , time complexity and space complexity of the same, as per the guidelines.
Kindly assign it to me.
This is regarding GSSOC'21.
Regards,
Hitain Kakkar

Hi,
I am GSSOC'21 participant
My code will be in JAVA with proper comments and test cases.
@rudrakshi99 @nakul-19 @plazzy99 @tarun26091999  please assign me this issue

I'm the Participant in GSSOC'21. Please assign me this issue. I'll use Python code .

Hello! I am a GSSoC'21 Participant. I would like to work on this issue. Would be highly obliged if you could assign it to me. Thanks!

I could do it using python/c++/java. Please assign it to me under gssoc'21

Hey @raksha009, I am a participant of GSSo'C. I would love to take up this issue and solve it using python. Please assign me this issue

Hey @raksha009, I am a participant of GSSo'C. I would love to take up this issue and solve it using python. Please assign me this issue

We allot issues on first come first basis. Plz look for other issues :)

@raksha009  Sir I have created the pull request. Please check the code if it looks good please merge it sir.

Hey!! I would like to work on the issue in cpp or c @raksha009


Change index's class 'get_type()' return type

I cant see why not do that. Maybe I'm missing something.
Relevant to issue: https://github.com/Vanderhoof/PyDBML/issues/6

Project rewritten, no longer relevant


Update UG

Clarify parsing of extra parameters, invalid parameter tags

Closes #150
Closes #116
Closes #110

Codecov Report

Merging #152 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #152   +/-   ##
=========================================
  Coverage     72.57%   72.57%
  Complexity      473      473
=========================================
  Files            83       83
  Lines          1546     1546
  Branches        163      163
=========================================
  Hits           1122     1122
  Misses          355      355
  Partials         69       69

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 a11e4dc...753e4fd. Read the comment docs.


Site seems to be broke

No text, no links, just the big sidebar.
Error in browser console: Uncaught TypeError: Cannot read properties of null (reading 'querySelectorAll')

I know!
I have now decided to switch to using a proper docs Hugo theme, this one -> https://github.com/colinwilson/lotusdocs
The current theme is more like a personal blog theme, which is not ideal ;)
I will leave this issue open until I actually create a working home page for Brandt's Duployan - thanks for reporting!

Site is now using the Relearn Theme, and features a blank page.
The idea is to write an introduction, and then add some Hugo to the manual so that it can be published in a Manual section online

The manual is now online :)
Now we need an introduction and some links to the PDF and the Anki decks.
And then the site grows from here.

The site is now up at https://jacmoe.github.io/brandt/ - and I hereby close the issue ;)
Will add links to the PDF and the Anki decks, and the helper Bash script later.
Thanks for reporting!


Low* freezable buffers

A library for freezable buffers of u8.
It maintains the current frozen-up-to index (a u32) in the first 4 bytes of the buffer (the value of first 4 bytes is a u32 that is the first writable index in the buffer). Clients can call freeze to freeze up-to an input index after which the buffer cannot be mutated until that index.
The library provides two witness-able predicates: one for the contents of a slice that falls within the [4, w) range (where w is the current frozen-up-to index), and one for the value of w itself (since it also increases monotonically).
Also including some unit-tests for it in examples.

Is this easily reusable to offer arbitrarily freezable buffers that are frozen wholesale without the index encoded in the first four bytes? (Thinking about the AEAD use-case for freezable buffers where we don't care too much about encoding the length, since it's allocate-write-then-freeze-everything.)

You could use these buffers for that also, just keep writing and then freeze at the end. But it will still use the first four bytes for writing the frozen index. If you don't want to use 4 bytes, we can do it with using may be just a byte (to keep a single bit whether the buffer has been frozen or not). Not sure if we can do it without using even that.

Ah, so the interface we looked at together (https://github.com/project-everest/hacl-star/blob/protz_aead/providers/evercrypt/EverCrypt.AEAD.fsti) would still need an extra bit of space inside the ekv type in order to be implementable?

For the case of buffers that are writable until frozen, we could do without keeping any state in the buffer itself, provided extraction can erase ref of erased bool. Here is a rough sketch:
module LowStar.FreezeOnceBuffer

open FStar.HyperStack.ST

include LowStar.Monotonic.Buffer

module P = FStar.Preorder
module G = FStar.Ghost

module Seq = FStar.Seq

module HS = FStar.HyperStack
module HST = FStar.HyperStack.ST

(*
 * Rough sketch for buffers that are writable until frozen
 *
 * Using an auxiliary reference to erased bool to keep the state
 *)

let ref_preorder : P.preorder (G.erased bool) =
  fun b1 b2 ->
  b2t (G.reveal b1) ==> b2t (G.reveal b2)

type ref_t = mref (G.erased bool) ref_preorder

let ref_is_true (r:ref_t) : HST.mem_predicate =
  fun h -> G.reveal (HS.sel h r) == true

let buf_preorder (r:ref_t) (#a:Type0) : srel a =
  fun s1 s2 ->
  HST.token_p r (ref_is_true r) ==> Seq.equal s1 s2

let buf_frozen_at (r:ref_t) (#a:Type0) (snap:G.erased (Seq.seq a)) : spred a =
  fun s -> HST.token_p r (ref_is_true r) /\ Seq.equal (G.reveal snap) s

let test (r:ref_t) (a:Type0) (snap:G.erased (Seq.seq a)) =
  assert (HST.stable_on (ref_is_true r) r);
  assert (stable_on (buf_frozen_at r snap) (buf_preorder r))

type buffer (a:Type0) =
  b:(r:ref_t & mbuffer a (buf_preorder r) (buf_preorder r))
    {loc_disjoint (loc_mreference (dfst b)) (loc_buffer (dsnd b))}

let is_live (#a:Type0) (h:HS.mem) (b:buffer a) =
  HS.contains h (dfst b) /\ live h (dsnd b)

let witnessed_at (#a:Type0) (b:buffer a) (s:G.erased (Seq.seq a)) =
  witnessed (dsnd b) (buf_frozen_at (dfst b) s)

let freeze (#a:Type0) (b:buffer a) (s:G.erased (Seq.seq a))
  : ST unit
       (requires (fun h ->
         is_live h b /\
         Seq.equal (as_seq h (dsnd b)) (G.reveal s)))
       (ensures  (fun h0 _ h1 ->
         modifies (loc_mreference (dfst b)) h0 h1 /\
         b `witnessed_at` s))
  = let r = dfst b in
    r := G.hide true;
    HST.witness_p r (ref_is_true r);
    witness_p (dsnd b) (buf_frozen_at r s)

let recall_contents (#a:Type0) (b:buffer a) (s:G.erased (Seq.seq a))
  : ST unit
       (requires (fun h ->
         is_live h b /\
	 b `witnessed_at` s))
       (ensures  (fun h0 _ h1 ->
         h0 == h1 /\
	 as_seq h1 (dsnd b) == G.reveal s))
  = recall_p (dsnd b) (buf_frozen_at (dfst b) s)

cc @nikswamy

I just fixed kremlin, so now a dependent pair of a buffer and an erased reference should incur no code-generation overhead.

Cool. Though I am still a bit suspicious if this would work, proving the preorder while writing may not work. I will try writing that API too.

Thanks -- that would be useful for EverCrypt.


Yet another GAZEBO_MODEL_PATH fix

Ok, let's figure this out now.
I think setting GAZEBO_MODEL_PATH in the bashrc is the most portable thing and will work on everyone's machine.
Can you please test @Jaeyoung-Lim, so I don't break your setup once again? (Sorry...)
Also, let me know if the Readme is not clear enough about the installation
Addresses #383 #391

how can I set GAZEBO_MODEL_PATH in the bashrc ?

@lioneleeedadad did you look at the actual PR? It includes a Readme update

On my system before I didn't even have a GAZEBO_RESOURCE_PATH, so I think just without that in the launch file it should work fine. The reason we got errors is because it tried to append to a variable that didn't exist yet, and launch files are more fussy than bash.
I'd prefer for it to stay in the launch file, both to simplify setup and also to not have stuff in people's .bashrc just because they ran the planner once.

I had to remove the two lines from the launch file as well on my setup, otherwise I get an error that the PATH is not set

Does this work for you @Jaeyoung-Lim ?

@nicovanduijn This works for me. If I understand correctly, this is reverting #378 #381 ?

Essentially, yes

Can we get this merged anyway? Current master is just broken for me...Is this also the case for others? or is this just me?

lgtm


fileSize is bigger than actual

I have a image which size is 1.5mb, but the fileSize from response is much bigger

+1

Any progress on this issue @birdgg ?

I am experiencing this as well.

Fixed by set imageFileType: 'png' at options

This still appears to be an issue on iOS, my original file is 1.6mb and the response fileSize is stating it as 4.3mb (without editing any quality/height/width params).

This error is still appeared on 3.x.
I'm not sure why this issue is closed.

This error is still appeared on 3.x.
I'm not sure why this issue is closed.

I am still facing this issue

I am also facing this issue. Currently on version 4.7.0
I tested this only on Android.


Update to drone-go 1.4.0 and add build params support

@bradrydzewski can this be merged? This would be great for debugging starlark files that are triggered by the downstream plugin

Cool, that you preferred to redo my work in https://github.com/drone/drone-cli/commit/6f4c96818cf659f3f1bc44498e18ea93313d62ed over reacting to my PR. 🤷🏻♂


type safe entity ref tests

Bug description
const houseCO2Sensors = hass.refBy.label('house_co2_sensors')
        const sensor = hass.refBy.id('some_house_sensor')

        if (houseCO2Sensors.includes(sensor)) {

Argument of type 'ByIdProxy<TRawEntityIds>' is not assignable to parameter of type 'ByIdProxy<"sensor...

This test should be valid, does but types aren't transferring around as expected

@chrisdrackett

thanks, was just about to add this :D

I think this is similar if not the same bug:
const switches = hass.refBy.label('label_here')
hass.call.switch.turn_off({ entity_id: switches })

Type 'ByIdProxy<"switch.foo" | "switch.bar">[]' is not assignable to type 'PICK_ENTITY<"switch"> | PICK_ENTITY<"switch">[]'.


[MINOR][DOC] Fix ml-guide markdown typos

What changes were proposed in this pull request?
Easy fix in the markdown.
How was this patch tested?
Tested manually
Please review http://spark.apache.org/contributing.html before opening a pull request.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

can you check if we have other typo in the md file?
or similar type of typo in other md files?

@felixcheung Yes, I read the entire docs of Spark Mlib. There is no problem with other mds (I may not have seen it or other contributors solved it). This error is obvious, and it affects the link of the later issue, so i found it.
http://spark.apache.org/docs/latest/ml-guide.html#breaking-changes

I think you can add all other typos into here.

@HyukjinKwon Thanks a lot, could u help me to review these commits if u have time。

can someone help me to close this pr?  i have merged these commits to a new pr.
https://github.com/apache/spark/pull/20897

@Lemonjing you need to close the PR from github.com - we don't have access to close it

@felixcheung thanks a lot. and i closed it.


Support for GZIP'ed response on Android

I try to get a compressed response on Android o work.
It works on IOS without any additional code
like this
var httpClient = new HttpClient(new NativeMessageHandler());
but on Android it is not decompressed.
Must I do something more on Android?
(
Maybe some thing as this, nut not the same:
https://github.com/paulcbetts/ModernHttpClient/issues/35
https://github.com/paulcbetts/ModernHttpClient/issues/50
)

Can you tell me Where do you change from deflate to gzip?
I will appreciate.


inline validation with nested fields

Right now the inline validation does not show when we do something like this to validate the laravel request.
[
    'db.database' => 'required',
    'db.username' => 'required',
    'db.password' => 'required',
    'filesystem.config.host' => 'required',
    'filesystem.config.user' => 'required',
    'filesystem.config.password' => 'required',
];

Because or names should be called like this.
[
    [
        'label' => trans('labels.db_name'),
        'name' => 'db[database]',
        'type' => 'text',
    ],
    [
        'label' => trans('labels.db_username'),
        'name' => 'db[username]',
        'type' => 'text',
    ],
    [
        'label' => trans('labels.db_password'),
        'name' => 'db[password]',
        'type' => 'password_revealer',
    ],
    [
        'label' => trans('labels.ftp_host'),
        'name' => 'filesystem[config][host]',
        'type' => 'text',
    ],
    [
        'label' => trans('labels.ftp_user'),
        'name' => 'filesystem[config][user]',
        'type' => 'text',
    ],
    [
        'label' => trans('labels.ftp_password'),
        'name' => 'filesystem[config][password]',
        'type' => 'password_revealer',
    ],
];

Do they work at all?
Do they only work when the fields are named appropriately?
If they don't work at all, is there a known fix (even if you don't have a PR)?
Or what should the naming scheme be?

Inline valdiation is working. But no when we validate nested attributes.
Top level validation works fine with nested attributes, the only issue is with the inline validation.
Our nested attributes looks like this
filesystem[config][host]

Laravel validations look like this
filesystem.config.host

Changing our schema to be like laravel validation can have breaking changes.
I submited a pull request fixing it only changing a view:
https://github.com/Laravel-Backpack/CRUD/pull/987

Hi @gotrecillo , thanks a lot for the PR. Just merged it into dev.
Cheers!


Enforce category selection in product creator (#1337)

See: #1337

PR intended to be tested with API branch:
Screenshots

Pull Request Checklist

[ ] This code contains UI changes
[ ] All visible strings are translated with proper context including data-formatting
[ ] Attributes [data-test-id] are added for new elements
[ ] Changes are mentioned in the changelog
[ ] The changes are tested in different browsers and in light/dark mode

Test environment config

API_URI=https://master.staging.saleor.cloud/graphql/

Task linked: SALEOR-3606 [Simple Product] It is not possible to create simple product without category and with isPublished set as true.


Pin docker version in CI to v0.27.2

Codecov Report

Merging #275 (0edb090) into main (cf2c386) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #275   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           12        12
  Lines          425       425
=========================================
  Hits           425       425

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 cf2c386...0edb090. Read the comment docs.


Improve errors / warnings for cuda vectorize decorator

The CUDA vectorize decorator provides harsh feedback to users who provide it with unexpected arguments - an assertion failure with no further information. This commit changes the decorator in the following ways:

The nopython kwarg is now accepted, albeit with a warning that it is redundant on CUDA. The cpu and parallel targets do also accept this kwarg.
Provide a KeyError for other unsupported kwargs, consistent with the CPU and parallel targets.

The additional tests check the expected behaviour for all targets. For ths CPU target the decorator is tested with and without a signature, in order to check for normal ufuncs and dynamic ufuncs.
Minor changes:

The check_warnings context manager is made available for tests by inheriting from CheckWarningsMixin, which contains an implementation from tests/npyufunc/test_errors.py.
A small whitespace error in test_vectorize_decor.py is also corrected.

... Now I've just decided to make the PR, I've just realised the source of the issue: I'm running CUDA tests in the main numba.tests package, and it is not surprising that it doesn't work!

Now that I have the CUDA tests in the right place, things are looking better - I've removed the [WIP] tag and I believe this is ready for review.

running internal CI: numba_65

passing on Linux, not going to wait for windows workers to catch up...


enable QUIC protocol with flag

question was asked why the current implementation only works with lets encrypt. This is my response and plan to expand the current functionality.

the main reason is that drone has limited support for custom certificates (no http to https redirect, for example) which means most people probably just end up using a reverse proxy, which would not support QUIC
the lets encrypt implementation, however, is a bit more advanced. It virtually eliminates the need for a reverse proxy for ssl termination or redirect.
so I think the next step would be to improve the custom cert implementation in drone, so that it has feature parity with the lets encrypt implementation. Then we can look at integrating QUIC