Why write these components with Ts and SCSS instead of Plain Js and CSS in Developing without a Build style ?

** PLEASE READ THIS BEFORE FILING AN ISSUE **
I'm submitting a:

[ ] bug report
[ ] feature request

What OS are you using?

What browser(s) is this bug affecting:

Current behavior:

Expected behavior:

Steps to reproduce:

Related code:
// insert any relevant code here or add a include a codepen

Other information:

We use TypeScript because TypeScript adds types to JavaScript.
Types help make code less error-prone, because they allow the compiler to detect many common errors, such as passing the wrong kind of object to a function, or forgetting to check that an object could be undefined before accessing it. When a project gets to a certain size and complexity, and is being written by many different people, then these kinds of checks become extremely valuable.
Types are also helpful for reading and writing code, because they allow your editor (e.g. VSCode) to provide additional information about the code (e.g. hover your mouse on a variable to see what type it is, or click on a function to see where that function is called), and to suggest auto-completions (e.g. if you type input.ch, then your editor can suggest that you might want to complete to input.checked).
Note that you can view the published JavaScript generated from our TypeScript code using unpkg (for example https://unpkg.com/browse/@material/mwc-button/mwc-button-base.js). The TypeScript and generated JavaScript will look very similar, except that the JavaScript doesn't have type annotations, and decorators (which we make use of, another feature we like about TypeScript) are converted into a plain JavaScript equivalent.
We primarily use SASS because this library is based on the MDC Web library, so we inherited that system. The MDC Web library uses SASS because SASS adds a lot of useful features to CSS, such as build-time variables and functions. For large, complex code-bases of CSS, these features can be very helpful for productivity.
Whether we continue to use SASS for the Material Web Components in the longer-term remains to be seen. One advantage of Custom Elements is that, because your styles are scoped to individual components, it is usually possible to write much simpler CSS than used to be required for applications with traditional globally-scoped styles. That might mean tools like SASS are of slightly less value.
Hope that answers your question, closing this issue.

Yep, thanks


Saving and restoring Stream state for Online recognition

Would it be possible to serialize the state of a Stream and then recreate the stream from the serialized data so that the recognition can be continued using a similarly configured Online Recognizer on say, another machine?
I see that Streams have GetStates(), and SetStates(). Is that all we should be concerned with to recreate another stream with the exact same state?

OnlineStream is stateful and contains states of the feature extractor and the neural network model.
Could you describe why you need such a function to save the states of a stream on one machine and load them on another machine?

The feature would enable the possibility of easily (and cheaply) scale streaming speech recognition.
You could have a serverless function (AWS Lambda, Azure Function) process a chunk of data, save the state of the stream somewhere (Redis/memcached/network disk, etc) and then have another serverless function fetch the previous state and process the next chunk of data.
This would effectively decouple recognition of a stream from a particular machine.

So do you need this only for Python APIs?

I think that will suffice for most cases, unless someone is writing the backend in a different language, maybe Go.


Improve boolean validator

Description:
Improved the boolean validator to make it more robust. Picked out the important bits of #23293
No longer allows non-boolean, non-number, non-string, non None types through (invalid value). Strips out surrounding white space. It seemed odd that "  true" would be an invalid value as would "dog", but [56] would not and would validate to True.
The core of the validator is unchanged in that strings are either explicitly true or false, or invalid, it's just more robust in the face of non-string types and whitespace.
It may be worth considering 'open' to be true and 'closed' to be false as well.
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

"  true" simply isn't a valid boolean in YAML 1.1, for YAML 1.2 it becomes even stricter.
In YAML 1.1: y|Y|yes|Yes|YES|n|N|no|No|NO|true|True|TRUE|false|False|FALSE|on|On|ON|off|Off|OFF
In YAML 1.2: true|false
I would like to suggest not making it less strict on that part. So I personally do not agree on the .strip() part.
PS: Recently we made all documentation YAML 1.2 compatible.

@frenck Yes, YAML 1.2 no longer auto-converts certain text to booleans. But the reason that was done was very simple: tons of people were trying to use the given strings as text (strings), but any valid YAML parser would happily auto-convert that to a boolean, removing the original text.
However, here we know that the given option is a boolean, so why not make it aceept a bunch of types that clearly indicate the user's intent.
The only reason not to that i can think of is that having multiple possible values for booleans is potentially confusing. So for example users might be confused whether all of the above have the same semantics (which they do).
But on the other hand, it would also be a huge breaking change - and i think there's a lot of value for accepting data when the user's intent is very clear.

I would rather educate people than handle every possible "wrong" use-case in code just to be friendly. IMHO we should stick to spec to prevent falling into edge cases like this and giving the user the false sense of doing it right.

While the package is config validation, it also validates other things too like arguments to service requests, so we're not strictly talking yaml here.
I don't really feel that there's all that much to be gained by not accepting "  true" since like @OttoWinter said, the user's intent is pretty clear from that, particularly in the context that we're actually validating a boolean 'thing'. But... I don't think it matters all that much in the grand scheme and could be persuaded.
Really the main thing I didn't like about the old validator was that it validated non-empty complex types like arrays as True and that's gotta be a bad thing.

I think that strip is fine, because YAML will also automatically strip non-quoted strings:


Build and publish dev InstaScale image for every commit

Issue link

Fixes https://github.com/project-codeflare/instascale/issues/123
What changes have been made

Every commit affecting InstaScale code will trigger a workflow building InstaScale image and publishing it on quay.
Verification steps

Push the change from PR to main branch of your fork - workflow s triggered and tries to upload built InstaScale image on Quay.
Checks

[x] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[x] Manual tests
[ ] Testing is not required for this change

/lgtm

Had to reopen the PR to refresh the OpenShift CI info about merging options.

/lgtm

/approve


Incorrect Result

Hello,
I used my own data to get the ply but the result doesn't look correct. What will be the possible problems? Is it related to my camera parameters?

How did you compute your pose matrices?

I used colmap and then ran colmap2mvsnet.ply, which is from mvsnet and will read camera.bin to generate cams intrinsic and extrinsic.

Hi! Sorry to bother, but did you figure out the reason? I am having the same issue here...


Color problem

Below is a video demonstrating the problem. When the kettle is in a heating state, it turns orange. When the kettle is off, it should return to its silvery color. After refreshing the browser page, the color of the model is correct. I think this is a bug.
https://youtu.be/ofSqNL5Z8Jw

I found the reason, it is necessary to register the off state in the configuration. You can add a function to return the object color to its original color if state is not true. I thought that if I add one condition and when it is false, the color is returned, and not saved as in my video.

Well spotted, I'm going to load a release where the color is reset to the initial one when the state is not found (sort of 'else').

that's what I meant.  Great, I'm waiting for a new release.  In general, as a new feature, I would like to open the HA entity when I click on the object, now you can only call the service

Well spotted, I'm going to load a release where the color is reset to the initial one when the state is not found (sort of 'else').

Loaded fix in the release 0.9.17, please test and close if ok with the resolution


first cut at kafka spout example in Go

To Do: add storage manifests for GCP and Azure
Otherwise, it works fine as a demonstration of integration and a copypasta example for users.

@pappasilenus can you rebase your branch to master to make sure the PR passes the netlify checks?

@pappasilenus can you rebase your branch to master to make sure the PR passes the netlify checks?

I think I've done this, but it's still failing.  May need you to pair with me to see if I'm doing it wrong

Closed in favor of rebased version


Support parallel builds with MSVC

Jom is an nmake clone that allows parallel builds.  To support that,
we need to make sure updates of the PDB files are serialized, or there
will be errors.
Fixes #3272

I'll make a separate PR for 1.1.0

A note: the added option is only supported from VS 2013 and up.  I'm unsure how far back we claim to support CL, so it's possible we need to make a more elaborate check of its presence.

It's only natural to make least assumptions and instead empower user in choice of customizations. And in this case user is perfectly capable of passing additional -FS option and config time. As reminder VC configs are known to be "agnostic" in respect to choice between Microsoft and Intel compilers, so it's not sufficient to justify with "this is supported for VS XYZ and later" :-)

This seems like an unnecessary enhancement, the work-around seems sufficient.

Ok, I hear you.  I suspect, though, that as jom gets increased popularity (I'm hearing about it more and more often), a FAQ entry on the subject might be appropriate.
Closing this.

It's only natural to make least assumptions and instead empower user in choice of customizations.

Out of curiosity, whether PR https://github.com/openssl/openssl/commit/c32cbe97873bf919dc4d1bc332e4758ee6caea78 does not contradict this rule?
And for what it was done to MSVC builds only.


Alternative implementations

Does anyone know if this project has been written in a different environment such as Node, Go, or Java? or have stumbled upon a similar project to this one?
Been really interested in fingerprinting and processing audio, would like to get more resources on this topic. Any pointers are appreciated 😃.

I don't think you're gonna find a project as simple and awesome as this one... And sadly Shazam and SoundHound don't open source their technology...
I'm also curious for more because i don't think this tells you at what point in the original the snippet started, does it?

here is a similar implementation. I haven't checked the details of the algorithm though
https://github.com/JorenSix/Olaf


type "rfdlsfdmgjdflk" return Cannot read property 'lat' of null

When I type  "gdfg fdg ...G. DF GDFG FD", I have an real error.

Cannot read property 'lat' of null

Excepted :
STATUS : ZERO_RESULT

Solution :
if result[0] exist define lat : results[0].geometry.location.lat(
else lat : 0
Something like that.

Inside this function :
    geocoder.geocode({ address: address }, function (results, status) {
       if (status !== OK) {
         callback({ status: status }, null, results);
         return;
      }
      var latLng = {
        lat: results[0].geometry.location.lat(),
        lng: results[0].geometry.location.lng()
      };

      callback(null, latLng, results);
   });

We could reference this question/ request with #38?

You can actually also reference #20. Code splitting would be a good idea, especially for error handling.

Closing this issue


How to release and access to the microphone?

Briefly summarize your issue:
How to release and then access to the microphone? I'm running the Alexa on a AGL(Automotive Grade Linux) develop-board.
Other software may use microphone, however the Alexa always occupies the microphone and never realeases the microphone.
The question is:  Is there any function will let Alexa release the microphone? if yes, could you tell me where it is?if not, how can I  let Alexa release the microphone?
I'll appreciate it if you could help me.
Thanks.
What is the expected behavior?
Release microphone --> access to microphone again.
Tell us about your environment:

What version of the AVS Device SDK are you using?
version: 1.9

Tell us what hardware you're using:

[ ] Desktop / Laptop
[ ] Raspberry Pi
[ ✓ ] Other - tell us more: AGL board

Tell us about your OS (Type & version):

[ ✓ ] Linux
[ ] MacOS
[ ] Raspbian Stretch
[ ] Raspbian Jessy
[ ] Other - tell us more:

Hi @PattenKuo, we may provide this functionality already in the SampleApp per the mute functionality, which invokes the sampleApp::PortAudioMicrophoneWrapper::stopStreamingMicrophoneData method.  This invokes Pa_StopStream on PortAudio, which may be sufficient to release the microphone.  If that does not have the desired result, we would suggest digging deeper into the PortAudio interface to identify what further needs to be done.
Let us know if you need any further guidance on how to integrate that into the SampleApp.


fix operator bundle  build

Automate operator bundle build
Problem:
Operator bundle build not working

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

Test name
Status

affinity-8-0
passed

auto-tuning-8-0
passed

cross-site-8-0
passed

demand-backup-cloud-8-0
failed

demand-backup-encrypted-with-tls-8-0
passed

demand-backup-8-0
passed

haproxy-5-7
passed

haproxy-8-0
passed

init-deploy-5-7
passed

init-deploy-8-0
passed

limits-8-0
passed

monitoring-2-0-8-0
passed

one-pod-5-7
passed

one-pod-8-0
passed

pitr-8-0
passed

proxy-protocol-8-0
passed

proxysql-sidecar-res-limits-8-0
passed

recreate-8-0
passed

restore-to-encrypted-cluster-8-0
passed

scaling-proxysql-8-0
passed

scaling-8-0
passed

scheduled-backup-5-7
passed

scheduled-backup-8-0
failed

security-context-8-0
passed

smart-update1-8-0
passed

smart-update2-8-0
passed

storage-8-0
passed

tls-issue-cert-manager-ref-8-0
passed

tls-issue-cert-manager-8-0
passed

tls-issue-self-8-0
passed

upgrade-consistency-8-0
passed

upgrade-haproxy-8-0
passed

upgrade-proxysql-8-0
passed

users-5-7
failed

users-8-0
failed

validation-hook-8-0
passed

We run 36 out of 36

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1361/commits/fbfa6db05700db1ca85821cbf2121ee017260369
image: perconalab/percona-xtradb-cluster-operator:PR-1361-fbfa6db0

Test name
Status

affinity-8-0
passed

auto-tuning-8-0
passed

cross-site-8-0
passed

demand-backup-cloud-8-0
failed

demand-backup-encrypted-with-tls-8-0
passed

demand-backup-8-0
passed

haproxy-5-7
failed

haproxy-8-0
passed

init-deploy-5-7
passed

init-deploy-8-0
passed

limits-8-0
passed

monitoring-2-0-8-0
failed

one-pod-5-7
passed

one-pod-8-0
passed

pitr-8-0
passed

proxy-protocol-8-0
passed

proxysql-sidecar-res-limits-8-0
passed

recreate-8-0
failed

restore-to-encrypted-cluster-8-0
passed

scaling-proxysql-8-0
passed

scaling-8-0
passed

scheduled-backup-5-7
passed

scheduled-backup-8-0
passed

security-context-8-0
passed

smart-update1-8-0
passed

smart-update2-8-0
passed

storage-8-0
passed

tls-issue-cert-manager-ref-8-0
passed

tls-issue-cert-manager-8-0
passed

tls-issue-self-8-0
passed

upgrade-consistency-8-0
passed

upgrade-haproxy-8-0
passed

upgrade-proxysql-8-0
passed

users-5-7
passed

users-8-0
passed

validation-hook-8-0
passed

We run 36 out of 36

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1361/commits/dcc3416c256de2a75959da6d047cba94007fa82d
image: perconalab/percona-xtradb-cluster-operator:PR-1361-dcc3416c

Test name
Status

affinity-8-0
passed

auto-tuning-8-0
passed

cross-site-8-0
passed

demand-backup-cloud-8-0
passed

demand-backup-encrypted-with-tls-8-0
passed

demand-backup-8-0
passed

haproxy-5-7
passed

haproxy-8-0
passed

init-deploy-5-7
passed

init-deploy-8-0
passed

limits-8-0
passed

monitoring-2-0-8-0
passed

one-pod-5-7
passed

one-pod-8-0
passed

pitr-8-0
passed

pitr-gap-errors-8-0
failure

proxy-protocol-8-0
passed

proxysql-sidecar-res-limits-8-0
passed

recreate-8-0
passed

restore-to-encrypted-cluster-8-0
passed

scaling-proxysql-8-0
passed

scaling-8-0
passed

scheduled-backup-5-7
passed

scheduled-backup-8-0
passed

security-context-8-0
passed

smart-update1-8-0
passed

smart-update2-8-0
passed

storage-8-0
passed

tls-issue-cert-manager-ref-8-0
passed

tls-issue-cert-manager-8-0
passed

tls-issue-self-8-0
passed

upgrade-consistency-8-0
passed

upgrade-haproxy-8-0
passed

upgrade-proxysql-8-0
passed

users-5-7
passed

users-8-0
passed

validation-hook-8-0
passed

We run 37 out of 37

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1361/commits/ba2feb2d700089ef5193ff969071981ab0409801
image: perconalab/percona-xtradb-cluster-operator:PR-1361-ba2feb2d

Test name
Status

affinity-8-0
passed

auto-tuning-8-0
passed

cross-site-8-0
passed

demand-backup-cloud-8-0
passed

demand-backup-encrypted-with-tls-8-0
passed

demand-backup-8-0
passed

haproxy-5-7
passed

haproxy-8-0
passed

init-deploy-5-7
passed

init-deploy-8-0
passed

limits-8-0
passed

monitoring-2-0-8-0
passed

one-pod-5-7
passed

one-pod-8-0
passed

pitr-8-0
passed

pitr-gap-errors-8-0
passed

proxy-protocol-8-0
passed

proxysql-sidecar-res-limits-8-0
passed

recreate-8-0
passed

restore-to-encrypted-cluster-8-0
passed

scaling-proxysql-8-0
passed

scaling-8-0
passed

scheduled-backup-5-7
passed

scheduled-backup-8-0
passed

security-context-8-0
passed

smart-update1-8-0
failure

smart-update2-8-0
passed

storage-8-0
passed

tls-issue-cert-manager-ref-8-0
passed

tls-issue-cert-manager-8-0
passed

tls-issue-self-8-0
passed

upgrade-consistency-8-0
passed

upgrade-haproxy-8-0
passed

upgrade-proxysql-8-0
passed

users-5-7
passed

users-8-0
failure

validation-hook-8-0
passed

We run 37 out of 37

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1361/commits/782b91361956692bbd9e2a536b894f480e6b6577
image: perconalab/percona-xtradb-cluster-operator:PR-1361-782b9136

@nmarukovich could you please test the command to build bundle and maybe ask Everest team to check it out if it complies to their requirements?

@nmarukovich ping

@dkhachyan we'll check this next week and merge.

Test name
Status

affinity-8-0
passed

auto-tuning-8-0
passed

cross-site-8-0
passed

demand-backup-cloud-8-0
passed

demand-backup-encrypted-with-tls-8-0
passed

demand-backup-8-0
passed

haproxy-5-7
passed

haproxy-8-0
passed

init-deploy-5-7
passed

init-deploy-8-0
passed

limits-8-0
passed

monitoring-2-0-8-0
passed

one-pod-5-7
passed

one-pod-8-0
passed

pitr-8-0
passed

pitr-gap-errors-8-0
passed

proxy-protocol-8-0
passed

proxysql-sidecar-res-limits-8-0
passed

recreate-8-0
passed

restore-to-encrypted-cluster-8-0
passed

scaling-proxysql-8-0
passed

scaling-8-0
passed

scheduled-backup-5-7
passed

scheduled-backup-8-0
passed

security-context-8-0
passed

smart-update1-8-0
passed

smart-update2-8-0
passed

storage-8-0
passed

tls-issue-cert-manager-ref-8-0
passed

tls-issue-cert-manager-8-0
passed

tls-issue-self-8-0
passed

upgrade-consistency-8-0
passed

upgrade-haproxy-8-0
passed

upgrade-proxysql-8-0
passed

users-5-7
passed

users-8-0
passed

validation-hook-8-0
passed

We run 37 out of 37

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1361/commits/f0211514697532a2f8fb7dceb056586163a0e114
image: perconalab/percona-xtradb-cluster-operator:PR-1361-f0211514


Suggestion: Make Parent Contract name in SceneContext optional

If I'm loading in multiple child scenes and inheriting bindings from a parent, it is mandatory that each scene can still load its bindings without loading in the parent. Currently I just made the _parentContractName string public and changing it via script if the Root scene is loaded, but this should not be a mandatory step.

So you're suggesting the following:
Given a scene Root and a scene Child with the Scene Parent of Child set to the Root.  When scene Child is loaded:

If the Root scene is loaded, then inherit bindings from that
If the Root scene has not been loaded, continue anyway without a scene parent.

Is that right?  If so, I disagree.  I think in most cases the user will have intended to have the parent scene loaded so it's better to throw an exception.


Add mental model conceptual guide to Migrate docs

Add some conceptual information/diagrams giving a high level overview of different Migrate commands
See @ruheni's PR: #3749

PR https://github.com/prisma/docs/pull/4214

Merged in https://github.com/prisma/docs/pull/4214

https://www.prisma.io/docs/concepts/components/prisma-migrate/mental-model


Add Polygonal Boundaries at EDDT

This PR also includes some tweaking to the arrival traffic.

@Fechulo In replacing all wave arrivals with surge arrivals (to stay consistent with previous traffic flows, I essentially undid some of the changes to the arrival traffic you made here via #493. Sorry, and you may want to play with it briefly, and adjust if necessary. It should hopefully be a pretty close translation to what it was originally.

I was, recently playing at EDDM, where the traffic is very manageable and not stressing to control. At that point I realized that in most of the airports I've made, the traffic is very difficult and and it certainly stresses me a bit. So I decided that for future airports I would try to lower down the amount of waves and amount of traffic in general. That's why I did those changes to EDDT. Don't worry about, I actually wanted to play around with the new flows, to see if I wanted to change something.


Fix last.fm scrobbler

https://github.com/tomahawk-player/tomahawk/issues/435
https://getsatisfaction.com/lastfm/topics/authentication-of-3rd-party-apps-should-not-be-marked-off-as-plex-is-still-erroring-out-which-i-assume-means

Worked on new account with application home page https://github.com/tb0hdan/voiceplay.
Added fallback to key/secret without scrobbling.


Feat: Operator Dash - Add version card to instance health tab

It'd be nice (and easy) to add a card to the main Operator dash with the versions of various pieces of the Aspects systems, such as:

Aspects plugin (hard coded at init)
ClickHouse (selected from the db)
event-routing-backends (selected from the most recent event?)
Vector (hard coded at init?)

Maybe Tutor version too, if it exists?


Prisma CLI generates the wrong endpoint and doesn't allow users to deploy!

Describe the bug
When using the Prisma CLI to init a project and generate the yml file, the endpoint is missing information. When the user then runs prisma deploy, an error is thrown:
TypeError: Cannot read property 'split' of undefined
at getWorkspaceFromPrivateOrigin (/Users/ferdinandismael/.nvm/versions/node/v10.15.0/lib/node_modules/prisma/node_modules/prisma-yml/src/utils/parseEndpoint.ts:20:79)
at Object.parseEndpoint (/Users/ferdinandismael/.nvm/versions/node/v10.15.0/lib/node_modules/prisma/node_modules/prisma-yml/src/utils/parseEndpoint.ts:46:21)
at PrismaDefinitionClass.get [as cluster] (/Users/ferdinandismael/.nvm/versions/node/v10.15.0/lib/node_modules/prisma/node_modules/prisma-yml/src/PrismaDefinition.ts:121:13)
at PrismaDefinitionClass.getClusterName (/Users/ferdinandismael/.nvm/versions/node/v10.15.0/lib/node_modules/prisma/node_modules/prisma-yml/src/PrismaDefinition.ts:256:17)
at PrismaDefinitionClass.validate (/Users/ferdinandismael/.nvm/versions/node/v10.15.0/lib/node_modules/prisma/node_modules/prisma-yml/src/PrismaDefinition.ts:127:30)
at PrismaDefinitionClass. (/Users/ferdinandismael/.nvm/versions/node/v10.15.0/lib/node_modules/prisma/node_modules/prisma-yml/src/PrismaDefinition.ts:63:12)
at step (/Users/ferdinandismael/.nvm/versions/node/v10.15.0/lib/node_modules/prisma/node_modules/prisma-yml/dist/PrismaDefinition.js:32:23)
at Object.next (/Users/ferdinandismael/.nvm/versions/node/v10.15.0/lib/node_modules/prisma/node_modules/prisma-yml/dist/PrismaDefinition.js:13:53)
at fulfilled (/Users/ferdinandismael/.nvm/versions/node/v10.15.0/lib/node_modules/prisma/node_modules/prisma-yml/dist/PrismaDefinition.js:4:58)
Exiting with code: 1
To Reproduce
Steps to reproduce the behavior:

npm install -g prisma
prisma init

Important that no project name is specified. Use the CLI wizard to generate the files.
Select Demo Server
Choose a region (This could be the first place for an error? Us1 appears to be the endpoint, but the description includes us2. In the file the blows up, parseEndpoint.ts it looks for a public end point with either eu1 or us2 in it. The generated end point from here is us1. Not sure what is intended)
Don't create client

prisma deploy
Error is thrown. 😿

Expected behavior
Expected that I could deploy & create my service.
Resolution steps
I was able to fix the problem by manually adding a workplace slug to my endpoint ->
prisma init generates an end point such as: "https://us1.prisma.sh/josh-tes/dev"
It looks like the code is expecting a workplace slug. So I added it... ""https://us1.prisma.sh/josh-a1b2c3/josh-sick-fits/dev". I can then deploy no problem. This solves the problem. However, I'm not sure what the intended behaviour is, and whether I should need the slug...
Versions (please complete the following information):

prisma CLI: [e.g. prisma/1.28.0 node-v11.5]
OS:  OS X High Sierra 10.13.6

Had the same issues today. Josh's solve worked for me as well. Just update the URL and all is well.
prisma -v outputs
prisma/1.28.0 (linux-x64) node-v8.2.1

Thanks for reporting! We released a patch and just fixed it. Let me know if you're still facing the issue.


Support for robotics tasks in control_env + MPPI Bug fix

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Motivation and Context / Related issue
There was a small bug in the MPPI implementation concerning the noise. For future planners, it might be also of interest to have harder environments, so I added a few lines to support the robotics tasks
Checklist

[x] The documentation is up-to-date with the changes I made.
[x] I have read the CONTRIBUTING document and completed the CLA (see CONTRIBUTING).
[x] All tests passed, and additional code has been covered with new tests.

Hi @freiberg-roman. Any comments on the above question?

HI @freiberg-roman, should I wait for your fix before merging this branch? Is there any other test I can do in the mean time?


Visual Studio cmake unable to find dependencies installed using vcpkg

I have a problem building my project using Visual Studio cmake after installing packages with vcpkg.
I've installed dependencies like this:

set VCPKG_DEFAULT_TRIPLET=x64-windows

vcpkg install cpprestsdk cpprestsdk --triplet x64-windows

vcpkg install libmysql libmysql --triplet x64-windows

And the list of installed packages:
vcpkg list
boost-algorithm:x64-windows                        1.67.0           Boost algorithm module
boost-any:x64-windows                              1.67.0           Boost any module
boost-array:x64-windows                            1.67.0           Boost array module
boost-asio:x64-windows                             1.67.0-1         Boost asio module
boost-assert:x64-windows                           1.67.0           Boost assert module
boost-atomic:x64-windows                           1.67.0           Boost atomic module
boost-bimap:x64-windows                            1.67.0           Boost bimap module
boost-bind:x64-windows                             1.67.0           Boost bind module
boost-build:x64-windows                            1.67.0           Boost.Build
boost-chrono:x64-windows                           1.67.0           Boost chrono module
boost-compatibility:x64-windows                    1.67.0           Boost compatibility module
boost-concept-check:x64-windows                    1.67.0           Boost concept_check module
boost-config:x64-windows                           1.67.0           Boost config module
boost-container-hash:x64-windows                   1.67.0           Boost container_hash module
boost-container:x64-windows                        1.67.0           Boost container module
boost-context:x64-windows                          1.67.0           Boost context module
boost-conversion:x64-windows                       1.67.0           Boost conversion module
boost-core:x64-windows                             1.67.0           Boost core module
boost-coroutine:x64-windows                        1.67.0           Boost coroutine module
boost-date-time:x64-windows                        1.67.0           Boost date_time module
boost-detail:x64-windows                           1.67.0           Boost detail module
boost-endian:x64-windows                           1.67.0           Boost endian module
boost-exception:x64-windows                        1.67.0           Boost exception module
boost-filesystem:x64-windows                       1.67.0           Boost filesystem module
boost-foreach:x64-windows                          1.67.0           Boost foreach module
boost-format:x64-windows                           1.67.0           Boost format module
boost-function-types:x64-windows                   1.67.0           Boost function_types module
boost-function:x64-windows                         1.67.0           Boost function module
boost-functional:x64-windows                       1.67.0           Boost functional module
boost-fusion:x64-windows                           1.67.0           Boost fusion module
boost-geometry:x64-windows                         1.67.0           Boost geometry module
boost-graph:x64-windows                            1.67.0           Boost graph module
boost-integer:x64-windows                          1.67.0           Boost integer module
boost-intrusive:x64-windows                        1.67.0           Boost intrusive module
boost-io:x64-windows                               1.67.0           Boost io module
boost-iostreams:x64-windows                        1.67.0-1         Boost iostreams module
boost-iterator:x64-windows                         1.67.0           Boost iterator module
boost-lambda:x64-windows                           1.67.0           Boost lambda module
boost-lexical-cast:x64-windows                     1.67.0           Boost lexical_cast module
boost-locale:x64-windows                           1.67.0           Boost locale module
boost-math:x64-windows                             1.67.0           Boost math module
boost-modular-build-helper:x64-windows             2018-05-14
boost-move:x64-windows                             1.67.0           Boost move module
boost-mpl:x64-windows                              1.67.0           Boost mpl module
boost-multi-index:x64-windows                      1.67.0           Boost multi_index module
boost-multiprecision:x64-windows                   1.67.0           Boost multiprecision module
boost-numeric-conversion:x64-windows               1.67.0           Boost numeric_conversion module
boost-optional:x64-windows                         1.67.0           Boost optional module
boost-parameter:x64-windows                        1.67.0           Boost parameter module
boost-phoenix:x64-windows                          1.67.0           Boost phoenix module
boost-polygon:x64-windows                          1.67.0           Boost polygon module
boost-pool:x64-windows                             1.67.0           Boost pool module
boost-predef:x64-windows                           1.67.0           Boost predef module
boost-preprocessor:x64-windows                     1.67.0           Boost preprocessor module
boost-property-map:x64-windows                     1.67.0           Boost property_map module
boost-property-tree:x64-windows                    1.67.0           Boost property_tree module
boost-proto:x64-windows                            1.67.0           Boost proto module
boost-qvm:x64-windows                              1.67.0           Boost qvm module
boost-random:x64-windows                           1.67.0           Boost random module
boost-range:x64-windows                            1.67.0           Boost range module
boost-ratio:x64-windows                            1.67.0           Boost ratio module
boost-rational:x64-windows                         1.67.0           Boost rational module
boost-regex:x64-windows                            1.67.0           Boost regex module
boost-serialization:x64-windows                    1.67.0           Boost serialization module
boost-smart-ptr:x64-windows                        1.67.0           Boost smart_ptr module
boost-spirit:x64-windows                           1.67.0           Boost spirit module
boost-static-assert:x64-windows                    1.67.0           Boost static_assert module
boost-system:x64-windows                           1.67.0           Boost system module
boost-test:x64-windows                             1.67.0-2         Boost test module
boost-thread:x64-windows                           1.67.0           Boost thread module
boost-throw-exception:x64-windows                  1.67.0           Boost throw_exception module
boost-timer:x64-windows                            1.67.0           Boost timer module
boost-tokenizer:x64-windows                        1.67.0           Boost tokenizer module
boost-tti:x64-windows                              1.67.0           Boost tti module
boost-tuple:x64-windows                            1.67.0           Boost tuple module
boost-type-index:x64-windows                       1.67.0           Boost type_index module
boost-type-traits:x64-windows                      1.67.0           Boost type_traits module
boost-typeof:x64-windows                           1.67.0           Boost typeof module
boost-unordered:x64-windows                        1.67.0           Boost unordered module
boost-utility:x64-windows                          1.67.0           Boost utility module
boost-variant:x64-windows                          1.67.0           Boost variant module
boost-vcpkg-helpers:x64-windows                    4                a set of vcpkg-internal scripts used to modulari...
boost-winapi:x64-windows                           1.67.0           Boost winapi module
boost-xpressive:x64-windows                        1.67.0           Boost xpressive module
bzip2:x64-windows                                  1.0.6-2          High-quality data compressor.
cpprestsdk:x64-windows                             2.10.3-1         C++11 JSON, REST, and OAuth library The C++ REST...
icu:x64-windows                                    61.1-1           Mature and widely used Unicode and localization ...
libevent:x64-windows                               2.1.8-3          An event notification library
liblzma:x64-windows                                5.2.3-2          Compression library with an API similar to that ...
libmysql:x64-windows                               8.0.4-2          A MySQL client library for C development.
lz4:x64-windows                                    1.8.2            Lossless compression algorithm, providing compre...
openssl-windows:x64-windows                        1.0.2o           OpenSSL is an open source project that provides ...
openssl:x64-windows                                0                OpenSSL is an open source project that provides ...
websocketpp:x64-windows                            0.7.0-1          Library that implements RFC6455 The WebSocket Pr...
zlib:x64-windows                                   1.2.11-3         A compression library

But when I tried to cmake the project using Visual Studio 17 I receive errors like:
CMake Error at C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/Common7/IDE/CommonExtensions/Microsoft/CMake/CMake/share/cmake-3.11/Modules/FindBoost.cmake:1903 (message):
  Unable to find the requested Boost libraries.
  Unable to find the Boost header files.  Please set BOOST_ROOT to the root
  directory containing Boost or BOOST_INCLUDEDIR to the directory containing
  Boost's headers.
Call Stack (most recent call first):
CMakeLists.txt:53 (find_package)
CMake Error at C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/Common7/IDE/CommonExtensions/Microsoft/CMake/CMake/share/cmake-3.11/Modules/FindPackageHandleStandardArgs.cmake:137 (message):
  Could NOT find OpenSSL, try to set the path to OpenSSL root folder in the
  system variable OPENSSL_ROOT_DIR (missing: OPENSSL_CRYPTO_LIBRARY
  OPENSSL_INCLUDE_DIR) (Required is at least version "1.0.0")

I've tried to add OPENSSL_ROOT_DIR environment variable but received the same issue with zlib:
CMake Error at C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/Common7/IDE/CommonExtensions/Microsoft/CMake/CMake/share/cmake-3.11/Modules/FindPackageHandleStandardArgs.cmake:137 (message):
  Could NOT find ZLIB (missing: ZLIB_LIBRARY ZLIB_INCLUDE_DIR)

And for some builds I still receive error about OpenSSL...
CMakeLists.txt part:
find_library(CPPREST_LIB NAMES cpprest_2_9d cpprest_2_9 cpprestd cpprest)
find_package(Boost REQUIRED COMPONENTS random system filesystem regex locale)
find_package(OpenSSL REQUIRED)
find_package(ZLIB REQUIRED)

When tried to install openssl additionally I receive further output.
>vcpkg install openssl
The following packages are already installed:
    openssl[core]:x64-windows
Starting package 1/1: openssl:x64-windows
Package openssl:x64-windows is already installed
Elapsed time for package openssl:x64-windows: 1.988 ms

Total elapsed time: 2.98 ms

The package openssl is compatible with built-in CMake targets:

    find_package(OpenSSL REQUIRED)
    target_link_libraries(main PRIVATE OpenSSL::SSL OpenSSL::Crypto)

The same with other packages. So it should work...
I've removed the dependencies using
vcpkg remove cpprestsdk --recurse

and installed again - same thing.
I've done
vcpkg integrate install

And tried to add -DCMAKE_TOOLCHAIN_FILE=D:/repo/vcpkg/scripts/buildsystems/vcpkg.cmake running cmake command from cmd:
"C:\PROGRAM FILES (X86)\MICROSOFT VISUAL STUDIO\2017\COMMUNITY\COMMON7\IDE\COMMONEXTENSIONS\MICROSOFT\CMAKE\CMake\bin\cmake.exe"  -G "Ninja" -DCMAKE_INSTALL_PREFIX:PATH="C:\Users\user\CMakeBuilds\6ec40ad3-aec5-e439-9463-46e3ff20b362\install\x64-Debug"  -DCMAKE_CXX_COMPILER="C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.14.26428/bin/HostX64/x64/cl.exe"  -DCMAKE_C_COMPILER="C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.14.26428/bin/HostX64/x64/cl.exe"  -DCMAKE_BUILD_TYPE="Debug" -DCMAKE_MAKE_PROGRAM="C:\PROGRAM FILES (X86)\MICROSOFT VISUAL STUDIO\2017\COMMUNITY\COMMON7\IDE\COMMONEXTENSIONS\MICROSOFT\CMAKE\Ninja\ninja.exe" "D:\repo\project" -DCMAKE_TOOLCHAIN_FILE=D:/repo/vcpkg/scripts/buildsystems/vcpkg.cmake

And received an errors:
-- VERSION: 2.0.0(1)
-- The C compiler identification is unknown
-- The CXX compiler identification is unknown
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.14.26428/bin/HostX64/x64/cl.exe
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.14.26428/bin/HostX64/x64/cl.exe -- broken
CMake Error at C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/Common7/IDE/CommonExtensions/Microsoft/CMake/CMake/share/cmake-3.11/Modules/CMakeTestCCompiler.cmake:52 (message):
  The C compiler

    "C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.14.26428/bin/HostX64/x64/cl.exe"

  is not able to compile a simple test program.

  It fails with the following output:

    Change Dir: C:/Windows/System32/CMakeFiles/CMakeTmp

    Run Build Command:"C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/Common7/IDE/CommonExtensions/Microsoft/CMake/Ninja/ninja.exe" "cmTC_e1a7a"
    ninja: error: loading 'build.ninja': The system cannot find the path specified.

CMakeError.log:
Compiling the CXX compiler identification source file "CMakeCXXCompilerId.cpp" failed.
Compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.14.26428/bin/HostX64/x64/cl.exe
Build flags:
Id flags: --c++

The output was:
2
Microsoft (R) C/C++ Optimizing Compiler Version 19.14.26433 for x64
Copyright (C) Microsoft Corporation.  All rights reserved.

cl : Command line warning D9002 : ignoring unknown option '--c++'
CMakeCXXCompilerId.cpp
c1xx: fatal error C1083: Cannot open source file: 'CMakeCXXCompilerId.cpp': No such file or directory

Compiling the CXX compiler identification source file "CMakeCXXCompilerId.cpp" failed.
Compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.14.26428/bin/HostX64/x64/cl.exe
Build flags:
Id flags: --ec++

The output was:
2
Microsoft (R) C/C++ Optimizing Compiler Version 19.14.26433 for x64
Copyright (C) Microsoft Corporation.  All rights reserved.

cl : Command line warning D9002 : ignoring unknown option '--ec++'
CMakeCXXCompilerId.cpp
c1xx: fatal error C1083: Cannot open source file: 'CMakeCXXCompilerId.cpp': No such file or directory

Checking whether the CXX compiler is IAR using "" did not match "IAR .+ Compiler":
Microsoft (R) C/C++ Optimizing Compiler Version 19.14.26433 for x64
Copyright (C) Microsoft Corporation.  All rights reserved.

usage: cl [ option... ] filename... [ /link linkoption... ]
Checking whether the CXX compiler is IAR using "" did not match "IAR .+ Compiler":
Microsoft (R) C/C++ Optimizing Compiler Version 19.14.26433 for x64
Copyright (C) Microsoft Corporation.  All rights reserved.

usage: cl [ option... ] filename... [ /link linkoption... ]
Determining if the C compiler works failed with the following output:
Change Dir: C:/Windows/System32/CMakeFiles/CMakeTmp

Run Build Command:"C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/Common7/IDE/CommonExtensions/Microsoft/CMake/Ninja/ninja.exe" "cmTC_e1a7a"
ninja: error: loading 'build.ninja': The system cannot find the path specified.

Documentation and promo video says it should work almost from the box.
https://docs.microsoft.com/en-us/cpp/vcpkg
https://www.youtube.com/watch?v=zo6nqFVBd5o
Please advise - I'm out of ideas.
Thanks in advance.

@gera710, does this still be a problem for you?

@gera710, boost has been upgrade to 1.76 in vcpkg, please try latest source and open a new issue if still be a problem for you.


:lock: security allow actuator probes

Description
Permit actuator probe endpoints for deployment health checks.
Required for #23

LGTM and works locally. But can you write tests for it like we did here

LGTM and works locally. But can you write tests for it like we did here

Unfortunately, this was not straight forward to implement because of context problems, so I have extracted it to #37


Adding job notification through named pipe

Named pipe are OK for one way communication, which appears to be the usage here. On Unix named pipes are emulated, so it may cause issues.

looks great!!!!


[Typo] Fix for o365_calendars_convetted

I hope this is only wrong in the documentation.

Thank you. This was only on the documentation.


Remove sender upload page & from menu

Problem statement
When a sender logs in they still see Upload in the menu and the page with this on it:

This is no longer needed as this moved to SimpleReport a while ago.
What you need to know
This is the page you see when logged in
Acceptance criteria

[ ] Page is removed and no longer in the menu when a sender is logged in

To do

[ ] ...

This has been removed.


Re-evaluate whether table headers should be included when partitioning Microsoft Excel Documents

Someone posted in the LangChain discord because they want to access table headers when using UnstructuredExcelLoader from LangChain's Python sdk. It looks like we intentionally omit table headers: https://github.com/Unstructured-IO/unstructured/blob/79a1eb86837ccfd84eb265985d6981d9ef9fe965/unstructured/partition/xlsx.py#L62
We should investigate the intent behind omitting the headers and decide whether it's a choice we'd like to uphold.

IMO, for the text_to_html metadata field in a element, we should include headers by default, which will presumably will be in <th>tags. It probably makes sense to include in text as well. But, this also should be configurable through partition() kwarg.

It looks like that was decided by Matt during his first pass at creating the partition_xlsx function, so I think it is safe to update.
Original pull request: https://github.com/Unstructured-IO/unstructured/pull/594
Original issue: https://github.com/Unstructured-IO/unstructured/issues/587
I'll get to work on incorporating headers.


e2e test support: Store test pod logs and events

Issue link

Fixes https://github.com/project-codeflare/codeflare-operator/issues/290
What changes have been made

Store logs of all containers in test namespace. Also store all namespace events.
Verification steps

See PR check logs.
Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[ ] Testing is not required for this change

/lgtm

/lgtm

/approve


merge from template-arlac77-github

.github/workflows/update_package_lock.yml

chore: (on.schedule)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.778%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
413

Relevant Lines:
436

💛  - Coveralls


child_veg_access

Value of children having more access to safe vegetables (million VND/year)
Do we need a Proxy ???

child_veg_health_care_savings
healthier kids might contribute to  might have lower healthcare costs associated with improved child health due to increased vegetable consumption. This could include factors like reduced instances of certain illnesses or conditions.
child_veg_school_performance_value
healthier kids might contribute to  might be better at school: Improved nutrition has been linked to better cognitive function and academic performance. We could explore the potential increase in educational outcomes and use that as a proxy for value.
child_veg_community_engagement_value
healthier kids may be more productive in various aspects of life, from work to community engagement. They might contribute to the overall well-being of the community. i.e. social benefits, community cohesion, or cultural aspects ...


Single-quoted strings generate incorrect method names when using regex snippets

If I have a feature that looks like:
Feature: Test

Scenario: Testing Replacements
  When I run 'foo'
  When I run 'bar'
  When I run "baz"
  When I run "qux"

behat output looks like:
$ behat --snippets-type=regex
Feature: Test

  Scenario: Testing Replacements # features/test.feature:3
    When I run 'foo'
    When I run 'bar'
    When I run "baz"
    When I run "qux"

1 scenario (1 undefined)
4 steps (4 undefined)
0m0.01s (7.85Mb)

 >> default suite has undefined steps. Please choose the context to generate snippets:

  [0] None
  [1] FeatureContext
 >

--- FeatureContext has missing steps. Define them with these snippets:

    /**
     * @When /^I run '([^']*)'$/
     */
    public function iRunFoo($arg1)
    {
        throw new PendingException();
    }

    /**
     * @When /^I run '([^']*)'$/
     */
    public function iRunBar($arg1)
    {
        throw new PendingException();
    }

    /**
     * @When /^I run "([^"]*)"$/
     */
    public function iRun($arg1)
    {
        throw new PendingException();
    }

$

This should only generate a single iRun method rather than generating iRunFoo and iRunBar
Turnip snippets do the correct thing:
$ behat --snippets-type=turnip
Feature: Test

  Scenario: Testing Replacements # features/test.feature:3
    When I run 'foo'
    When I run 'bar'
    When I run "baz"
    When I run "qux"

1 scenario (1 undefined)
4 steps (4 undefined)
0m0.01s (7.85Mb)

 >> default suite has undefined steps. Please choose the context to generate snippets:

  [0] None
  [1] FeatureContext
 >

--- FeatureContext has missing steps. Define them with these snippets:

    /**
     * @When I run :arg1
     */
    public function iRun($arg1)
    {
        throw new PendingException();
    }

$

Note: if you use --snippets-type=turnip then it works fine, you just get:
--- FeatureContext has missing steps. Define them with these snippets:

    /**
     * @When I run :arg1
     */
    public function iRun($arg1): void
    {
        throw new PendingException();
    }


Fixes #80 reduced the border radius and fixed button hover

Closes #80
🛠 Fixes Issue : "#80 "
👨💻 Changes proposed:

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.
[x] I am a participant of Hacktoberfest 2022.

📄 Note to reviewers
too much rounded border does not look good with text area, so reduced border radius of inputs
also now button text color stays white when hovered

@Aryan1982 This issue was not assigned to you, so I'm closing it.
If you're having trouble finding issues to work on, please visit the Readme.md file of this repository; you'll find many ideas there. Then, as a first step before starting work on a particular issue, kindly raise a new issue. ✨


minimum IAM role needed to select a subscription in App Centre org

what is the minimum IAM Role needed to view and select a subscription for billing against an app in app centre?
this should be outlined here as shops will have a governance element to consider for this.

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

ID: 1143ee54-3dad-1bbb-1b43-30dd4d6c3e6d
Version Independent ID: c27a4b60-76ce-033a-0da1-87e5f9d14add
Content: Azure Subscriptions in App Center - Visual Studio App Center
Content Source: docs/general/azure-subscriptions.md
Service: vs-appcenter
GitHub Login: @jwhiteDev
Microsoft Alias: jawh

so will this get any traction?

Hi @bakerm00 sorry for the delay in this response. You need to be a subscription owner to associate it with an App Center organization, as mentioned here in the doc: https://docs.microsoft.com/en-us/appcenter/general/azure-subscriptions#connecting-to-an-existing-azure-subscription


array_cum_sum may have bug for const array column

Steps to reproduce the behavior (Required)
https://github.com/StarRocks/starrocks/blob/57d68dd6d869900a2e47551eded85b5d9cb108fe/be/src/exprs/vectorized/array_functions.cpp#L346-L350
if the const array column has N (N>1) values, but only return the only 1 array values of cum sum.
Expected behavior (Required)
Real behavior (Required)
StarRocks version (Required)

You can get the StarRocks version by executing SQL select current_version()
latest main

resize in expressionContext, so L:350 is also OK without setting the size of the const column.


Implement tests for pipeline.query and pipeline.apply

Also remove _test() from pipeline_test.py

Done


GH-56 movie rename endpoint

Setup movie rename endpoint. Movies completed additions to the Plex folder.

Resolves #56

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

87 of 90 (96.67%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.5%) to 91.003%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

addition/views.py
54
55
98.18%

addition/clients.py
23
25
92.0%

Totals

Change from base Build<PHONE_NUMBER>:
1.5%

Covered Lines:
354

Relevant Lines:
389

💛  - Coveralls


Split vote module

Description

Split vote status from vote module.
Checklist

[ ] Targeted PR against correct branch.
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Wrote unit tests.
[ ] Re-reviewed Files changed in the Github PR explorer.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@0cab80e). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head 271b8c6 differs from pull request most recent head 3b22008. Consider uploading reports for the commit 3b22008 to get more accurate results

@@            Coverage Diff            @@
##             master     #228   +/-   ##
=========================================
  Coverage          ?   73.81%
=========================================
  Files             ?       54
  Lines             ?     2032
  Branches          ?        0
=========================================
  Hits              ?     1500
  Misses            ?      456
  Partials          ?       76

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 0cab80e...3b22008. Read the comment docs.


error reading rebecca-martin.md

Error: YAML Exception reading /Users/shjzhang/github/UNLV-SPFG.github.io/_team/rebecca-martin.md: (<unknown>): found unexpected end of stream while scanning a quoted scalar at line 11 column 11

There is a missing double quotation mark that should go at the end of the line below
<li> Planetary and solar system dynamics"

that I missed in review.

Good catch, @hbaehr. This is actually related to what I just mentioned to @DavidRRice in #21.

Resolved.


Crashes in debug mode

Really cool exploration! However, i'd like to bring attention to debug mode where the app crashes
If I enable -com.apple.CoreData.ConcurrencyDebug 1 and -com.apple.CoreData.SQLDebug 1 the app crashes which suggests there may be issues with concurrency handling.

How about removing fetch() and changing models to a get throws, e.g.
    var models: [T] {
        get throws {

And I have more ideas for making FetchedResultsController more Swifty if you would like to collaborate?


Better Docker instructions

I want to ditch docker-compose and create a bash script + Dockerfile instead. Documentation needs to be updated accordingly.
Why? docker-compose is meant for building multiple Dockerfiles and running them as services, while I'm currently using it for the mounts. Probably something with --mount type=bind,src={SRC_FOLDER},target=/src would work as well. {SRC_FOLDER} also needs to have an absolute path afaik, so there's some stuff to be tried first.
This doc could help: https://nodejs.org/en/docs/guides/nodejs-docker-webapp/
Time required: 1 hour +/- 75%

Fixed in db6d6ad. New guides added as well. Also solves related issue #50.


It does not work on my machine.

I verify everything but still does not work on my machine

Did you try    ?

I tried ctrl + e first before reporting here
On Feb 21, 2018 10:40 PM, "Timothy Lopez"<EMAIL_ADDRESS>wrote:

Did you try ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kelseyhightower/nocode/issues/1949#issuecomment-367347002,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAK0cahvZsO1zSIzrwoidmdbP3bTioTfks5tXCrggaJpZM4SNkFd
.

I work well.

I am having the same issue, can devs please respond?

Works for me.

I tried again by upgrading the OS of my keyboard and mouse, and moving
12pixels to the left and pressing ctrl + e, actually starts the app.
How can i file this bug report and resolution steps?
On Feb 23, 2018 12:13 AM, "Anton Smyk"<EMAIL_ADDRESS>wrote:

Works for me.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kelseyhightower/nocode/issues/1949#issuecomment-367732121,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAK0ce0C9Q9BZ_r6gJ-394aD3qVJL9Taks5tXZItgaJpZM4SNkFd
.

You need to remove your operating system entirely first. Some users reported that wiping out the BIOS helped.


Setting - About - Extra space between components in about page @mdneyazahmad

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open app
Go to Settings
Click About

Expected Result:
There should not be more spacing
Actual Result:
Extra space between components in about page
Workaround:
Unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
Email or phone of affected tester (no customers): any
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
Issue reported by: @mdneyazahmad
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1661256360362129
View all open jobs on GitHub

Proposal
We need to remove styles.mb6 https://github.com/Expensify/App/blob/d037a999e5e20993c6ee3537be338e215654a751/src/pages/settings/AboutPage/AboutPage.js#L79

I am 99.9% sure this is externally fixable, but let me apply the engineer label to make sure.

PS I just reproduced this issue and it's certainly valid.

This is a great external issue and thanks for the early proposal, @Puneet-here.
@Expensify/design Could you please weigh-in here on what the expected margin should be?

I think this is fine as-is and it's been this way ever since we built this page. So I would vote to just close this one out.

That's fair, closing for now!
Note: I know we are all aware of this by now, but saying it juuuuust in case :) - if we feel this should be changed or improved even slightly, then it's worth the effort of paying the $250 for this lil fix.

Fair point. But I think this is another example of where it would be really helpful to float this into the open source room first for visibility. I understand that's not currently part of Applause's process but I think it should be added.

When you say float this into the room first, do you mean like have them post in the channel before they create the GH? Or something elsE?


fix SRU call return type

Because many other cells , such as BasicLSTMCell 、 LSTMCell 、NASCell 、 LayerNormBasicLSTMCell , return h and stateTuple with c and h when call . I think sru should return the same type as lstm .

It's a bit off-topic but I've found that doing U = math_ops.matmul(inputs, self._kernel) inside the  def call(self, inputs, state): is not helpful to achieve the 10x speed up mentioned in the paper. I believe the matrix multiply should happen outside of the call.

I don't think it's clearer to return the output h as part of the recurrent state as the essential trick of the SRU is that only c takes part in the recurrence. This pr makes it look like h is used for the next timestep.

@marcbelmont achieve the 10x speed up need cuda code implement SRU , not python code . see more in https://github.com/taolei87/sru .

@carlthome I agree with you as only c takes part in the recurrence . So you advice is no need for change thr origin code ? I just think it should be compatible with lstm , so  that user only change lstm to sru with no other experiment code change . Maybe I should close this pr .

@ekelsen is this PR good now?

IMHO this should not be merged because there's no clear benefit to the breaking change and it's also less mathematically correct to pass around h in the state when it's not actually part of the recurrence.
There are many other RNN cells that also doesn't return a state tuple (GRU for example) so you'd still have to check what state actually is anyway (or use tf.contrib.framework.nest).

@carlthome  OK ， I close this pr .


SPane error on scroll

I need to wrap some content into something that scroll, I'm trying SPane with:
import QtQuick.Controls 1.2

import "StratifyQML"

ApplicationWindow {
    id: app
    visible: true
    property string appTitle: "Test"

    minimumWidth: 500
    minimumHeight: 200

    SCustomize {
        id: theme
        gray_base: "#303F9F"
    }

    SPane {
        data: SColumn {
            SRow {
                SBaseRectangleText {
                    text: "Row 1"
                }
            }
            SRow {
                SBaseRectangleText {
                    text: "Row 2"
                }
            }
            SRow {
                SBaseRectangleText {
                    text: "Row 3"
                }
            }
            SRow {
                SBaseRectangleText {
                    text: "Row 4"
                }
            }
            SRow {
                SBaseRectangleText {
                    text: "Row 5"
                }
            }
            SRow {
                SBaseRectangleText {
                    text: "Row 6"
                }
            }
            SRow {
                SBaseRectangleText {
                    text: "Row 7"
                }
            }
        }
    }
}

When I load that QML:
qrc:/qml/StratifyQML/SPane.qml:27:5: QML Flickable: Binding loop detected for property "contentHeight"

loop in console a few hundred times and:
qrc:/qml/main.qml:18:5: QML SPane: Binding loop detected for property "implicitHeight"

I see the problem. It is in SPane
    implicitHeight: fillHeight ? ((parent != null) ? parent.height : 0) : contents.childrenRect.height + paddingVertical*2;

if you add:
style: "fill"

to SPane that will set the fillHeight variable to true and the SPane should take the height of the parent.
Alternatively, you can just override the implicitHeight value with a fixed variable like
implicitHeight: 200

You can play around with setting height and implicitHeight.
I will add this as a bug so that it is clear SPane needs to be the size of the parent or a fixed size.

@tyler-gilbert these kinds of details aren't documented in .md or in source code as comment in file such as SPane.qml
If I wish to give back "documentation" on the usage, like this case, where it should be? comment or .md?

@bclermont I would say let's put it in the qml file.  Eventually, I would like to have the doc's generated using http://doc.qt.io/qt-5/qdoc-categories.html


Revive VoronoiDelaunay.jl instead?

Any chance this effort could be directed to reviving VoronoiDelaunay.jl? I think VoronoiDelaunay.jl had state-of-the-art performance across all languages, and it would be nice to get some people there maintaining it instead of starting a new package from scratch.
Also, as far as I remember, computing the Delaunay tessellation to extract the Voronoi diagrams is more efficient than trying to compute the Voronoi diagrams directly from the set of points.

Thank you for clarifying it, I wasn't aware of this discrete variant. Closing the issue.


feat: multiline

close #2

:tada: This PR is included in version 1.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Decommissioning supported Maps SDKs

VERSIONS

Xamarin.Forms.GoogleMaps - 3.3.0
Xamarin.Forms - 5.x.xxxx

PLATFORMS

[ ] Android
[x] iOS
[ ] UWP

Currently using Xamarin Forms Google Maps API, However, 3.3.0 version gets warning in iOS to use latest Google maps SDK. Need support to avoid this issue in future

This is quite concerning.
Let me tell you in advance: This project is dead.
@amay077 has been tagged several times by several threads including my own thread asking if the project is dead #772
This project is in need of a fork by someone else if it wants to continue existing.

@jari-kuipers @valliappan16 here you are mates, i updated the library versions and made a "pirate" nuget. Try it out on your spare time
Xamarin.Forms.GoogleMaps.4.0.0-BETA2.nupkg.zip

@jari-kuipers @themronion @valliappan16 are you sure @amay077 is not supporting this package anymore?
Do you have a suggestion for another alternative? Our project is heavily depends on this library :/

@AlonRom 99% sure. Try my nuget and let me know if it works fine

@themronion I don't have a specific issue to test now but we surely need future support.

@AlonRom I am 100% sure. I even tried emailing @amay077 's personal email address on February 1st and I still haven't gotten any response.
There are no alternatives and other than forking this repo and updating it or trying @themronion 's package, there is nothing you can do.
I suspect that if @amay077 does return, he will only do so to deprecate this package. Maybe you can get him to answer if we all send him emails (if his personal email is still in use).
If you wanna try it for yourself, go ahead and try. His personal email is<EMAIL_ADDRESS>But to be frank with all of you, you shouldn't be using Xamarin in the first place. All of you should have migrated long ago, which is what we did and we never regretted it. Xamarin will soon be end of life and MAUI has not and will definitely not pick up enough speed to be able to completely replace it by then.
If you "absolutely can't move" which is of course always the case for all of you, then I wish you the best of luck in EOL hell...

@jari-kuipers that's very unfortunate. can I ask to which technology did you migrate to and do have there all these package capabilities?

@AlonRom I'd be happy to answer that since I did extensive research on this topic for my own company when we had to make the decision to stay with Xamarin or move on to something else.
The best platforms to migrate to are of course the native platforms. You'll have all the support you'll need to upkeep your long-living app for many many years. With Jetpack Compose for Android and SwiftUI for iOS the platforms have evolved thusly that the development speed for them is on par with Xamarin (I would say).
Lots of companies don't want to invest in this though because of "development and learning costs", often times not realizing the time and money that goes into the upkeep of half-baked apps.
Also the "benefit" of using C# for everything is not that great. The frameworks and runtimes are different enough to make for quite a learning curve either way. Sure you'll be able to whip up something quickly in Xamarin if you already know C#. But C# and .NET as a whole weren't designed for these types of applications, and you must have noticed the quirks that come with it unless your apps are really simple.
If you really need to have a cross platform app I would highly suggest Flutter for now. It'll have a much longer lifespan than the other cross platform solutions and the development and learning speed is unrivaled by a mile. React-native also wouldn't be the worst choice since it has very mature community support. It's especially useful if you already have people with web dev experience around.
If all of you have medium to large apps that need to live for an extended amount of time with minimal upkeep, then I would seriously look into moving to native applications. You'll also be able to produce better quality apps that way. The investment will be worth it.

@jari-kuipers Thanks for the comprehensive answer, I thought a lot about moving to NATIVE, but the idea of rewriting the logic twice sounds awful to me, it's something that needs to move from the world in my opinion

@AlonRom does The official flutter Google maps package not satisfy your needs?
There's also an upcoming framework that you might want to look out for called Kotlin Multiplatform that keeps both iOS and Android in their worth while using shared logic written in Kotlin.

@jari-kuipers doesn't look like it, we support stuff like clustered icons, custom icons, edit shapes with validations, drag & drop pins, and more. the Flutter project looks very light at this point.

Just thought I would mention here that there is a fork: https://github.com/INHack20/Xamarin.Forms.GoogleMaps that seems to be maintained :)

@john-dalsgaard but there hasn't been a commit since 2021 right?

Right - but it is, however, newer than this.... Predominantly updates to packages so that version may avoid the decomissioning issues. I'm fighting other errors where the app has started crashing when I bind a variable from my view model to InitialCameraUpdate. If I leave that out it runs steady... So that is a bigger concern at the moment :-(

Hi @valliappan16 ,
I deployed v5.0.0 to nuget.
Could you try it?
https://github.com/amay077/Xamarin.Forms.GoogleMaps/issues/783#issuecomment-1232771417


Improve docs string for dialog events

Is your feature request related to a problem? Please describe
The current docs string for the ui.dialog events parameter is events – The events to capture on this dialog. However, there is a specific list of events that can be captured but this isn't documented making it hard for the user to know how to use this feature.
Describe the solution you'd like
Change the description to events – The events to capture on this dialog. One of 'dismissed'.
to match other descriptions like ui.zone(direction):
"direction – Layout direction. One of 'row', 'column'. See enum h2o_wave.ui.ZoneDirection."
Notes

Sidepanel and notification bar might also be related?

Apply the same for side panel as well.


some times had .env making error

Today I Pushed in github
some times have problem of not making .env file
07:04:03.638  Successfully built c8143ac15320
07:04:03.640  Successfully tagged ckzrgvqm539530nrxuwnveff1:77fd117
07:04:03.934  Failed to deploy!
07:04:03.934  Reason: Error: ENOENT: no such file or directory, open '/tmp/build-sources/dondonjang/bain_nest/cl06ctzvi000f0op7e3uyefca/.env'

Not sure why, but I added a small check, so it won't happen. It will be fixed in the next version!


Move HOMEBREW_SIMULATE_MACOS_ON_LINUX handling to SimulateSystem

Follow-up to https://github.com/Homebrew/brew/pull/13599
This PR moves the handling of HOMEBREW_SIMULATE_MACOS_ON_LINUX into SimulateSystem. Since the env var is accomplishing the same thing as simulating using the SimulateSystem methods, it makes sense to handle it in the same place instead of needing to have separate conditionals for the two possible ways of simulating. When HOMEBREW_SIMULATE_MACOS_ON_LINUX is set, we assume that we're simulating a generic macOS version, so Homebrew::SimulateSystem.os returns :macos. I've also set it up so that simulations set up using SimulateSystem methods take precedence over the environment variable, but Homebrew::SimulateSystem.clear will not clear the simulation since the env var is still set.
In this PR, if the env var is set while actually on macOS, it is essentially a no-op. Homebrew::SimulateSystem.os will be nil and Homebrew::SimulateSystem.current_os will return the symbol for whatever version of macOS is actually being used. Should setting this variable on macOS use the generic :macos instead? I don't think I see much value in this, but it's a possibility.

Review period will end on 2022-07-29 at 19:10:55 UTC.

Review period ended.


as per 8962199

closing this pull request as per Dan's comment that it is out of our team's scope to handle PowerShell documentation. https://office.visualstudio.com/MAX/_workitems/edit/8962199


How to run and test it locally on Ubuntu?

Hi, I want to clone it on my local computer with WSL.
Can you give me detailled steps for installation?
Thanks!

Hi, the system is adapted to AWS F1 instance; it requires the "FPGA shell" of AWS F1 and it cannot be run locally.


oneOf doesn't fail when nested in a list?

Schema = jiffy:decode(F, [return_maps]).
#{<<"additionalProperties">> => false,
  <<"minProperties">> => 1,
  <<"patternProperties">> =>
      #{<<"^[a-zA-Z]{1}[a-zA-Z0-9_]{2,}$">> =>
            #{<<"oneOf">> =>
                  [#{<<"type">> => <<"number">>},
                   #{<<"type">> => <<"string">>},
                   #{<<"type">> => <<"boolean">>},
                   #{<<"type">> => <<"null">>},
                   #{<<"items">> =>
                         #{<<"oneOf">> =>
                               [#{<<"type">> => <<"number">>},
                                #{<<"type">> => <<"string">>},
                                #{<<"type">> => <<"boolean">>}]},
                     <<"type">> => <<"array">>}]}}}
27> Body1 = #{ <<"key1">> => 9, <<"albatross">> => <<"value1">>, <<"list">> => [true, 1, [<<"something">>, null]]}.
#{<<"albatross">> => <<"value1">>,<<"key1">> => 9,
  <<"list">> => [true,1,[<<"something">>,null]]}

Expected result: a list value with a list inside of it should fail oneOf validation as defined above.
Actual result:
28> jesse:validate_with_schema(Schema, Body1).
{ok,#{<<"albatross">> => <<"value1">>,<<"key1">> => 9,
      <<"list">> => [true,1,[<<"something">>,null]]}}

I don't understand why?

It's because the default validator seems to be schema-03. When I explicitly turn on schema-04 it works.

Yeah, looks strange. Could this example be further reduced to minimal one
to reproduce?
пн, 15 июл. 2019 г., 22:31 Mark Allen<EMAIL_ADDRESS>
Schema = jiffy:decode(F, [return_maps]).
#{<<"additionalProperties">> => false,
<<"minProperties">> => 1,
<<"patternProperties">> =>
#{<<"^[a-zA-Z]{1}[a-zA-Z0-9_]{2,}$">> =>
#{<<"oneOf">> =>
[#{<<"type">> => <<"number">>},
#{<<"type">> => <<"string">>},
#{<<"type">> => <<"boolean">>},
#{<<"type">> => <<"null">>},
#{<<"items">> =>
#{<<"oneOf">> =>
[#{<<"type">> => <<"number">>},
#{<<"type">> => <<"string">>},
#{<<"type">> => <<"boolean">>}]},
<<"type">> => <<"array">>}]}}}
27> Body1 = #{ <<"key1">> => 9, <<"albatross">> => <<"value1">>, <<"list">> => [true, 1, [<<"something">>, null]]}.
#{<<"albatross">> => <<"value1">>,<<"key1">> => 9,
<<"list">> => [true,1,[<<"something">>,null]]}
Expected result: a list value with a list inside of it should fail oneOf
validation as defined above.
Actual result:
28> jesse:validate_with_schema(Schema, Body1).
{ok,#{<<"albatross">> => <<"value1">>,<<"key1">> => 9,
<<"list">> => [true,1,[<<"something">>,null]]}}
I don't understand why?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/for-GET/jesse/issues/82?email_source=notifications&email_token=AADHA7UWQE7Q32AZS2R2R2DP7TNB7A5CNFSM4ID2QU7KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G7KAQUA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADHA7VDUWUJSKZBKNTLPMLP7TNB7ANCNFSM4ID2QU7A
.

Works as expected afaik. I'm cannot check myself right now but v3 doesn't have a definition for oneOf, allOf, etc Might even be that v3 and earlier expect singular "item", not plural "items". So at least 1, but maybe even 2 rules are ignored. It didn't occur to me until now but maybe it's useful to return the schema paths that are ignored.

Ah, sorry, true.

It didn't occur to me until now but maybe it's useful to return the schema paths that are ignored.

Sounds useful. Can actually be a separate function to just feed it with schema and it will return a list of rules it doesn't know? Because the list of rules that are ignored doesn't depend on data, only schema.

I figured out my issue :) Thanks


Build breaks because of missing icons

Problem:
When using the footer component with icons to social media platforms, there is little documentation from where the icons are used. Build breaks immediately.
Solution:
Please integrate in the docs a description that icons are imported from Bootstrap
import { ArrowRight } from 'react-bootstrap-icons' <Footer.Icon href="#" icon={ArrowRight}/>

We normally use this package.
https://react-icons.github.io/react-icons/


Slightly more compatible Highlander fingerprints

As discussed with Daehyuk, Arne, and HskrOP on Discord in #help...
https://discordapp.com/channels/469524606043160576/524592892627517450/576451670268051480
PR change is the same as seen in @arne182's fork...
https://github.com/arne182/openpilot/blob/release2/selfdrive/car/toyota/values.py
Additional modification(s) may be required later, but this is apparently a step in the right direction.

I thought I clearly said that comma wants to keep trims seperate.

...and I agreed with that, yet the groups seemed to agree there's a compatibility issue with the current values.  Feel free to propose change(s).

If you guys can give us the dongleID of users who had fingerprint issue, we'll handle the rest. How is that?
I'm really tempted to merge this PR and https://github.com/commaai/openpilot/pull/631, but then it will be hard to keep adding new models in the future.

Sent IDs to you via Discord.


Ticket 998 document DH keysize

Document how stronger DH keys can be generated when self-signed certificates are generated. Also fixes an unrelated typo.

Doh.


Phashing large files eats a lot of memory

Recently found myself running out of RAM during phashing. And I'm guessing it's because during phashing, the entire binary content of each video file is loaded into memory before processing.. Since video files can easily take up several GB each, and since many files are processed in parallel, it's surprisingly easy to run out of memory. Doesn't help that queries are sorted by filesize, ensuring that the largest files in a query will be processed at the same time.
For now, I can avoid this issue by setting a lower job count, but it's not very user-friendly to require users to manually estimate how many jobs they can run based on the filesizes in their collection and available memory. And it's not inconceivable for someone to have video files that are larger than their total available memory, which would make those files impossible to process.
I can think of two obvious solutions, but both have drawbacks:

Pass the filepath to pyav so it can read the file directly from disk instead of loading it into memory first. Simple and straightforward, except that there's currently no way to determine the filepath of a file in hydrus. And there might be people running hydrus on a remote device.
Stream the file content instead of fetching it all at once. Theoretically this could just be a matter of using video_response.raw instead of video_response.content, but I've messed with this in the past, and I recall that it's not always as straightforward as it should be. Something about some video formats not presenting data in the same order pyav wants to read it, I believe.

Adding a dynamic thread count that adjusts based on the available memory to file size ratio is feasible.
Streaming files is out of the question since it would probably be unstable, slow, and complicated. Media containers are tricky and I don't want to introduce any possible parsing issues with that.


mWeb/Safari - Unable to upload a photo to Profile

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to URL https://staging.new.expensify.com/
Login with any account
Navigate to settings by clicking on the user avatar over LHN
Click on Profile
Click on the avatar edit icon
Click on upload photo and upload any image

Expected Result:
Able to upload a photo to Profile
Actual Result:
Unable to upload a photo to Profile
Workaround:
Unkown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[x] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/221932717-f4aeb6d1-1943-4be0-a434-52076920277e.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

Triggered auto assignment to @tylerkaraszewski (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

This is a dupe of https://github.com/Expensify/App/issues/15559, should be fixed once that is fixed! @kbecciv

⚠ Looks like this issue was linked to a Deploy Blocker here
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.
If a regression has occurred and you are the assigned CM follow the instructions here.
If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.

@mountiny - should we close this one then?

Yes.


Create eleventy-plugin-ids.json

Simple plugin for generating/adding ids for headers and other elements as a possible solution for this issue.

Foo Bar

becomes

Foo Bar

Thank you!


[How-To] Using the dump-server

Because the Symfony var-dump-server only checks for tcp://localhost:9912, running dump-server in the workspace or in the host machine won't work, since they are on a different IP address to the dumping client.
The solution is to run the dump-server in the php-fpm container.
docker exec -it laradock_php-fpm_1 /var/www/artisan dump-server

You should change the container name to your respective container.

Work like a champ:
$sudo docker-compose exec  php-fpm bash $cd path_to_code_base $php artisan dump-server

Yikes. Don't do sudo docker-compose. It will mess up your permissions big time. you should add your own user to the docker group on your system.

Reply to this email directly or view it on GitHub:
https://github.com/laradock/laradock/issues/1873#issuecomment-509564166

--
Yours Sincerely,
MegaXLR

docker-compose exec --workdir /var/www/project  php-fpm  php artisan dump-server

Not an issue anymore, close this


In the puzzle game UI is not organised.

Describe the bug
A clear and concise description of what the bug is.
The restart button should be in middle.

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

@Sulagna-Dutta-Roy please assign this to me i am GSSOC'24 contributor with 2+ years of experience in development


Clarify that additional pkg needs to be installed

As a beginner working through this page of the docs I spent a bit of time struggling to make the @serverless-stack/node/config import work, not realizing that I needed to manually install an additional npm package.
Other users may assume, like me, that create-sst would install all needed packages automatically. I think it would be useful to add a bit of clarification about that point on this page to remove friction from the learning process.
I've made changes I think would be best, but the exact wording and positioning are of course up for debate.

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


Check if another file has been uploaded with the same blake2 digest

This tries to resolve  #2490
I'm not sure if the uniqueness of the blake2 digest should be validated on file uploading or before, or if hijacking on the same "File already exists" message is helpful enough.
Also, I'm getting a weird error on my test, making it fail. Something about the request object missing a 'user' attribute.
Thanks for your feedback y'all.
🐍

The error is because your test is getting to this line:
            request.db.add(
                JournalEntry(
                    name=release.project.name,
                    version=release.version,
                    action="add {python_version} file {filename}".format(
                        python_version=file_.python_version,
                        filename=file_.filename,
                    ),
>                   submitted_by=request.user,
                    submitted_from=request.remote_addr,
                ),
            )

Which means it's trying to add the file being uploaded, so it seems like either something is wrong with your test, or with the new logic you added.

Hey @alanbato, need any help or more direction here?

I think @alanbato is having computer problems.

Yes I am 😢
If anyone wants to continue and build on this PR, please do so. I'll ping this thread when my computer gets better 😓

Thanks for all your work on this @alanbato which @yeraydiazdiaz was able to build on. Hope your computer heals soon!

@brainwane I'm glad it was of help :) And my computer lives to see another, so I'm ready to start contributing again! :sunglasses:

@alanbato I'd love your help with #3151.


Integrate OSV.dev as advisor

See individual commits.
A first iteration for #3599 with a couple of things left as TODO.

@oheger-bosch I'd appreciate if you could also review this.


Fixes for release-utils

Main issues here were uploadZip would never fulfill it's returned promise, and the emoji filtering didn't work.

and the emoji filtering didn't work.

What did not work?

From the commit:
The emoji *must* be at index=2 as each line looks like:
    "* <commit message>"
We use index=2 to make test whether emoji is first character(s) in the
commit message

Basically we were incorrectly filtering and not getting the correct commits

Oh ok.
Ok cool if it works, then let's merge 👍 💅🏼


Fragment Sorting no longer happening on v3.0

As far as I can tell the sort code no longer sorts an analysis tree when starting from a fragment. I can make it sort through to an analysis tree from a midas file. Not sure of the origin of the problem at this time.

I believe the issue is in SetUpPipeLine in TGRSIint around line 560.
The code just stops sorting here:
if(!write_fragment_histograms &&
!write_fragment_tree &&
!write_analysis_histograms &&
!write_analysis_tree) {
   return;
}

I'm not sure why write_analysis_tree is not being set yet tough.

It goes back further than the above. The code doesn't even know that it has a fragment tree in this scenario....

The issue is opt-SortRoot being in the logic statement. This is supposed to tell the code to sort the fragment into histos and is not required for building analysis tree. Commenting this fixed it. Now I'm working on a problem with the denominator (or number of fragment entries) of this sort.

I think this is all fine now. I'll push fix now.

You just need to use the -s flag. Nothing needed to be changed, just using -as sorts a fragment file into an analysis file.

No this is the problem. -a doesn’t do fragment->analysis. It was requiring -sa to do the job the way it was written which doesn’t seem proper.

Is there a reason we would want to have both there? I don't always want to build -s type histograms when I'm just trying to sort from fragment to analysis.

As I said, the functionality was changed (w/o documenting it) from requiring -a to requiring -as (or -sa or whatever order you want to write it in). See also #751, where I described this exact issue.

Sorry #751 kind of turned into a TChannel conversation so I forgot that this and the RunInfo had been brought up. I think it's better that we have the separate and specific issues. After you initially brought it up, it never really got mentioned again. Do you think there is any reason to not keep it as -a?

No, in fact I would much prefer that. I just assumed everybody else wanted it that way because no-one said anything in #751. But I guess everybody else got distracted as well ;-)


Please, clarify Stream.asBroadcastStream() behavior

The description for method dart:async:Stream.asBroadcastStream() reads:

Stream<T> asBroadcastStream({
void onListen(
StreamSubscription<T> subscription
),
void onCancel(
StreamSubscription<T> subscription
)
})
Returns a multi-subscription stream that produces the same events as this.
The returned stream will subscribe to this stream when its first subscriber is added, and will stay subscribed until this stream ends, or a callback cancels the subscription.
If onListen is provided, it is called with a subscription-like object that represents the underlying subscription to this stream. It is possible to pause, resume or cancel the subscription during the call to onListen. It is not possible to change the event handlers, including using StreamSubscription.asFuture.
If onCancel is provided, it is called in a similar way to onListen when the returned stream stops having listener. If it later gets a new listener, the onListen function is called again.
Use the callbacks, for example, for pausing the underlying subscription while having no subscribers to prevent losing events, or canceling the subscription when there are no listeners.

As follows from description, it is possible to unsubscribe the returned stream from the original stream via onListen or onCancel callbacks. Please, clarify the behavior of returned stream after it is unsubscribed. For example, if method listen is called, will it throw error or just send done event immediately or something else?
The following sample code demonstrates the current behavior:
import "dart:async";

main() {
  Stream s = new Stream.fromIterable([1,2,3]);
  Stream b = s.asBroadcastStream(onListen:(ss) {ss.cancel();});

  b.listen(print, onDone:() { print("done1");});
  b.listen(print, onDone:() { print("done2");});
  b.listen(print, onDone:() { print("done3");});
}

Output:

done2
done3

Please, note that done1 is never printed, which is quite strange.

As follows from description, it is possible to unsubscribe the returned stream from the original stream via onListen or onCancel callbacks.

It's possible to cancel the subscription on the original stream. When that happens, no further events will occur on that subscription, and hence no further events will be emitted to subscriptions on the returned stream.
So, when you cancel, there will be no further events emitted on the corresponding subscription.
What happens in the example is that the second and third listen are listening on a stream that knows it's done (because its internal state knows that the original stream was cancelled), but apparently it cannot tell the difference between being cancelled and having sent a done event, so it does what an already completed broadcast stream does - it sends a done event when you try to listen to it.
We should be consistent and not send any done events.
The usage here is not intended - cancelling the original in response to a listen is not really useful, so I have no problem with just stopping events without a "done" when it happens. The functionality is there so you can cancel or pause in response to not having any listeners. The onListen might not even need the argument, but it's there for consistency (and maybe so you can pause if you really want to).

I think that the first listener should receive done event.
Let's rewrite the sample code for ease of reference:
import "dart:async";

main() {
  Stream s = new Stream.fromIterable([1,2,3]);
  Stream b = s.asBroadcastStream(onListen:(ss0) {ss0.cancel();});

  StreamSubscription ss1 = b.listen(print, onDone:() { print("done1");});
  StreamSubscription ss2 =  b.listen(print, onDone:() { print("done2");});
  StreamSubscription ss3 =  b.listen(print, onDone:() { print("done3");});
}

So we have two different stream subsciptions: ss0 (between returned stream and original stream) and ss1 (some external listener). When ss0 is cancelled no further events are sent from the original stream. This is correct and expected behaviour. However ss1 have not been canceled and waits for events. ss1 has no relation to ss0 and has no way to 'understand' that there will be no more events.
So I think it is a duty of the returned stream to inform all listeners about the situation by sending done event.

As I said, it's not really important. In practice, you should never call cancel in response to onListen.
That said, it's probably safer to send the done then. It makes no difference when source.cancel is called when there is no listeners, which is the case we are actually supporting, but it is more consistent and easily explainable. It's simpler than disallowing cancel during onListen.

I think the current behavior will stand, and the documentation should be updated to say that a cancel while there are listeners will leave those listeners without a done event.


Add support for node-local-dns in dual-stack cluster.

node-local-dns accepts only one ip-family as localip. With this PR, we only use addresses of the primary ipfamily.
Queries to address of the secondary ipfamily of the service would not be cached.
How to categorize this PR?

/area networking
/kind enhancement
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

Add support for `node-local-dns` in dual-stack cluster.

/cc @ScheererJ @DockToFuture

/assign

/lgtm
/approve
/retest


Ms2 decon search

I totally agree! The code that is there is pretty preliminary, I just wanted to get it out in the open in a pull request. I think it would slightly slow down bottom-up searches, but it would be more accurate (fewer false-positives as Zach mentioned, and of course more correct mass fragments). We should reevaluate the "top 200" ion strategy as well, since deconvoluting with only the 200 most intense peaks in a scan may give some deconvolution errors. I'm not super familiar with the localization and scoring code either; would this need to be rewritten to use the new MS2ScanWithSpecificMass, assuming it has a list of fragment masses attached to it? i.e., do these methods re-read the scan and thus have to be rewritten to use the deconvoluted masses instead?

All these methods (scoring and localization) could easily switch over to using a sorted list of masses instead of a list of m/z

And I am open to reevaluating the top 200 strategy too. The idea is to remove noise peaks. Maybe there is a better way of doing this.

I still think we should remove all peaks in a mass bin. If we're only looking at b and y ions, we shouldn't expect more than 2 correct peaks every 57 Da (at a minimum).

What about co-isolation?
In any case, this is a different issue! Let's tackle the less controversial one first.


Microsoft.SignalRService/signalR

Describe the noise
Resource type (i.e. Microsoft.Storage/storageAccounts)
Microsoft.SignalRService/signalR
apiVersion (i.e. 2019-04-01)
2018-10-01]
Client (PowerShell, Azure CLI, or API)
PowerShell
Relevant ARM Template code (we only need the resource object for the above resourceType and apiVersion, but if it's easier you can include the entire template
Expected response (i.e. "I expected no noise since the template has not been modified since the resources were deployed)
No changes to resource
Current (noisy) response (either include a screenshot of the what-if output, or copy/paste the text)
 ~ Microsoft.SignalRService/signalR/sentiatest01 [2018-10-01]
    - properties.hostNamePrefix: "sentiatest01"
    - sku.size:                  "F1"
[
      - 2:

          flag:  "EnableMessagingLogs"
          value: "False"

      ]

Additional context
Add any other context about the problem here.

[w31]

This should be resolved, let us know if you still see this noise


Using react-reactonlickoutside with React.forwardRef results in a runtime error

Using the following component that uses React.forwardRef:
const TestComponent = onClickOutside(
    React.forwardRef((props, ref) => <div {...props} ref={ref} />)
);

Leads to the following error:
Uncaught TypeError: Cannot read property 'isReactComponent' of undefined
    at onClickOutside.render (react-onclickoutside.es.js:329)
    at finishClassComponent (react-dom.development.js:13727)
    at updateClassComponent (react-dom.development.js:13690)
    at beginWork (react-dom.development.js:14489)
    at performUnitOfWork (react-dom.development.js:17014)
    at workLoop (react-dom.development.js:17054)
    at HTMLUnknownElement.callCallback (react-dom.development.js:149)
    at Object.invokeGuardedCallbackDev (react-dom.development.js:199)
    at invokeGuardedCallback (react-dom.development.js:256)
    at replayUnitOfWork (react-dom.development.js:16366)

As styled-components uses React.forwardRef, react-onclickoutside cannot be directly used with styled components.
Related to: https://github.com/Pomax/react-onclickoutside/pull/293

I am very glad they finally added this as a thing, because it solves a problem that is inherent to HOC implementations. That said, this is absolutely worth updating the code for, and then releasing it as a new major version that does away with the whole instance forwarding code this HOC has had to use for years now.
As per https://reactjs.org/docs/forwarding-refs.html#note-for-component-library-maintainers - let's use this! And mark it as breaking change.
/cc @Andarist

Sure - doing a major release for this is cool 👍

Any update on that?

Any updates on this fix?

@Andarist your pull request is broken

@Andarist could you please update your PR

are there any workaround no to use with styled-components?

#326 should be able to work with forwardRef without a breaking change

I've not had any free time to look at this before today, but I've left some questions on that PR - there's some odd things going on in it and I'd like to make sure those are all intentional.


Add a missing step to install in Openshift

Signed-off-by: Eloy Coto<EMAIL_ADDRESS>

This should not be required, mvnw* scripts were added in 681f44672ae615d6fc13ccc788934863aac8596f