fix(Truncate): Fix truncation markup and makes props more flexible

Overview
Fixes an issue where markup wasn't being correctly truncated and makes the API a bit easier to control.  It will also automate use of Truncate.Atom which must wrap any React component, so any markup passed to  will behave as expected.

react-truncate-markup => 5.1.0
Small changes to the Truncate API to make behavior clearer.
Alert now always truncates at 2 lines.

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: ABC-123
[x] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.10-alpha.ec6b52.0@codecademy/gamut-labs@7.13.5-alpha.ec6b52.0@codecademy/gamut@19.11.6-alpha.ec6b52.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.10-alpha.ec6b52.0@codecademy/gamut-labs@7.13.5-alpha.ec6b52.0@codecademy/gamut@19.11.6-alpha.ec6b52.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60118193101bb461512ae633--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://60118193101bb461512ae633--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.10-alpha.e8676d.0@codecademy/gamut-labs@7.14.1-alpha.e8676d.0@codecademy/gamut@19.11.6-alpha.e8676d.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.10-alpha.e8676d.0@codecademy/gamut-labs@7.14.1-alpha.e8676d.0@codecademy/gamut@19.11.6-alpha.e8676d.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.10-alpha.a9859a.0@codecademy/gamut-labs@7.14.1-alpha.a9859a.0@codecademy/gamut@19.11.6-alpha.a9859a.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.10-alpha.a9859a.0@codecademy/gamut-labs@7.14.1-alpha.a9859a.0@codecademy/gamut@19.11.6-alpha.a9859a.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60119d9857c0b6b182d0249e--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://60119d9857c0b6b182d0249e--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.10-alpha.6a9cb3.0@codecademy/gamut-labs@7.14.1-alpha.6a9cb3.0@codecademy/gamut@19.11.6-alpha.6a9cb3.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.10-alpha.6a9cb3.0@codecademy/gamut-labs@7.14.1-alpha.6a9cb3.0@codecademy/gamut@19.11.6-alpha.6a9cb3.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6011cb43e426f4ac04036d45--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://6011cb43e426f4ac04036d45--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.14-alpha.d2f68d.0@codecademy/gamut-labs@7.16.2-alpha.d2f68d.0@codecademy/gamut@20.0.1-alpha.d2f68d.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.14-alpha.d2f68d.0@codecademy/gamut-labs@7.16.2-alpha.d2f68d.0@codecademy/gamut@20.0.1-alpha.d2f68d.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60131368879709963f87439c--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://60131368879709963f87439c--gamut-preview.netlify.appDeploy Logs


By any chance, are you going to test it on CIFAR-10?

By any chance, are you going to test it on CIFAR-10?

Will do it over the weekend.


PR real time test when target branch full build

OPS Build status updates of commit e3c2fa2:
:clock10: Full build: average full build time is 8 min(s) 19 sec(s) [calculated from last 10 build(s)]

Note: This is a FULL build due to the 'pr-basebranch' branch had a configuration change, normally no action is required on your part. In the result that you find subsequent builds still performing full builds, please contact your repository admin to manually perform a full build on the 'pr-basebranch' branch. For more detailed info, please click here.

OPS Build status updates of commit e3c2fa2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

docs-ref-conceptual/testing.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Mailing list inaccessible

Hello,
The Google mailing list is inaccessible for me, giving the error:

This group either doesn't exist, or you don't have permission to access it. If you're sure this group exists, contact the owner of the group and ask them to give you access.

As an alternative to Google's data harvesting tool mailing list you might like to consider:

https://groups.io/
https://sr.ht/

🙂

Strange, seems like my Google account was temporarily locked out for some reason, thus disabling the group. It should be fixed now.
I would like to switch to a self hosted mailing list solution in the long term, however I don't have time to do that any time soon. I would prefer to do something self hosted instead of just using a different 3rd party. Suggestions and pull requests are welcomed.

I would like to switch to a self hosted mailing list solution in the
long term, however I don't have time to do that any time soon. I would
prefer to do something self hosted instead of just using a different
3rd party. Suggestions and pull requests are welcomed.

I think self-hosting email and/or a mailing list is tricky business. I
guess there are really two parts though, the mail server and the list
archive; would you self-host both parts?
One of the problems with Google Groups is that there is no (easy) way to
export the archive.

Yes. I'm sure we can figure out some way to script an export out of Google Groups. Ultimately I would like all SE discussion and project management to be hosted on our own servers.


add extraAnnotation and addLocal options

Add chart options for #980 and #944

lgtm. @deniseschannon, can you merge this please?


Remove unnecessary TLS Config

Signed-off-by: Michael Marshall<EMAIL_ADDRESS>It is incorrect to override the default for InsecureSkipVerify. There are many parts of the code base use the default value for InsecureSkipVerify without testing TLS, so I did not add any tests and I did not make it configurable. I did manually verify the correctness of the change.
Checklist

[x] When introducing a new scaler, I agree with the scaling governance policy
[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[x] Tests have been added (there isn't test coverage)
[x] A PR is opened to update our Helm chart (repo) (if applicable, ie. when deployment manifests are modified)
[x] A PR is opened to update the documentation on (repo) (if applicable)
[x] Changelog has been updated and is aligned with our changelog requirements

/run-e2e

/run-e2e

/run-e2e

/run-e2e

Could you rebase your PR please? This commit is needed to run e2e test correctly

/run-e2e

I won't be able to contribute making insecure connections configurable. If that is necessary to move forward, we can close this PR or someone can build on top of this PR.

I definately appreciate your work - I'm just checking if we need knobs for impacted customers but as mentioned above, if you are 100% sure then I'm OK.

@tomkerkhove - I am pretty confident my understanding is correct. However, would it help if I add a test that demonstrates that self signed certificates work correctly with this change? I started working on such a test tonight, but I ran into some minor config issues, so I plan to finish the test tomorrow, as long as it is helpful. Let me know, thanks.

@tomkerkhove - I added the test. I was also able to make the test fail when using incorrect hostnames or an incorrect root ca cert.

Thanks a ton @michaeljmarshall !

/run-e2e

Thank you @michaeljmarshall!


test: test: Cypress - Automated tests for Elasticsearch datasource

Description

This PR includes tests for Elastic Search datasource - with querying & usage of various fields in the query page

Type of change

New script

How Has This Been Tested?

Cypress local/CI runs

Checklist:
QA activity:

[X] Added Test Plan Approved label after reviewing all Cypress test

/ok-to-test sha=c457ac9

/ok-to-test sha=3e324cd

/ok-to-test sha=51b4774

/ok-to-test sha=2c7d071

/ok-to-test sha=446d8e4

/ok-to-test sha=5e5a78b

/ok-to-test sha=5665489

/ok-to-test sha=e8de08f

/ok-to-test sha=19281d1


[export] Add a helper that lets us export multiple entrypoints from an nn.Module

I really hope that the torch.export folks consider this a real use case to solve for, but for now it seems like it is us and the executorch folks doing gross hacks like this to support export of multiple functions from a module stack.
I went the extra mile on this a bit and went ahead to implement save/load on these stacks, which gives us the ability to do FX tracing once and then iterate on the result to compile. It should be an easier workflow (for dev) considering that Dynamo is so slow for real models.
This is all bleeding edge stuff on Torch, but I need it for LLM so will deal with the sharp edge.

Looks useful, even if it just makes it so we can load the traces instead of doing via dynamo.
What are the bleeding edge features this enables?

For things we currently have, all of the mutability bits descend from ExportedProgram. In terms we could do better with it:

It internally functionalizes and elminates the need for a bunch of related hackery in jittable.
Gives us a path for supporting non-persistent buffers.
Has a way to trace gradients and gradient updates.


not logical approach of using defaults.

If you do not define anything in the plist you expect the default value to be send, thus
in:
<key>mapping</key>
        <dict>

no key uid, thus your ldap query should use uid. but to my surprise I see apple-generateduid being send!
This i had to correct with:
<key>uid</key>
      <array>
      <string>uid</string>
</array>

A few years ago I had installed 4.2, then i had no issues. It is a bit early to deduct anything yet, but I hope your development team has not downgraded to the level of that of wordpress developers.

In the interest of having accurate expectations, I will offer a bit of history. This project was started by Apple, and almost all of the contributions have been from Apple employees. The primary target platform was macOS (at the time, "Mac OS X"), and was delivered as one of the services in Apple's Server product (initially "Mac OS X Server", then "OS X Server" when it became an app instead of an entire OS distribution, and finally "macOS Server").
CalendarServer was developed from the start as an open source project, and has taken patches over the years from the community - patches that people have needed for their own purposes. The project has never had a goal of being easy to use over the long term in a generic open source context. That is certainly a good goal, but has not been a goal of this project. For Apple's primary target (Server), the tasks surrounding upgrade and migration of CalendarServer are handled automatically by software that is not part of the CalendarServer project. The average user of CalendarServer in Server has probably never even seen the config file.
We fully recognize that upgrade and migration of any other kind of deployment of CalendarServer is a very manual task, amidst very sparse documentation. It is not for the faint of heart. I wouldn't say that CalendarServer's support for other platforms is getting worse, I would say it's always been shaky at best. That said, if you're willing to overcome these challenges, CalendarServer can and does provide a lot of features, and scale to enterprise level. Personally, I still think there is more value in CalendarServer being open source than closed source, even though it does not have the polish that many expect. I will confidently assert that filling in the gaps at the deployment / ops level is way, way less work than what was required to produce this codebase.
On the up side: a) patches are welcome! b) we have mailing lists where even non-apple-employees contribute by helping others - see https://apple.github.io/ccs-calendarserver/

I was expecting such reply actually. Although it is mainly an apple product, I still think a good approach would be to keep the application simple with eg. an expectable behaviour. And if it is easy to deploy it on other platforms it will also be easy to deploy with osx.
And you will get lots of feedback from a larger community. Opensource and open standards are the future. Things like the above issue are likely going to bite you in the future.


Diagnose ambiguous uses of #require() with an argument of type Bool?.

This PR adds new diagnostics when #require() is passed an argument of type Bool? because such an argument may be ambiguously interpreted. If the test author writes:
try #require(love?.isFleeting)

Do they mean to unwrap the optional boolean value and return it, or do they mean to test if isFleeting is true or false?
With this change, an expression such as that one that returns a value of type Bool? will produce a diagnostic of the form:

(The diagnostics are presented as fix-its in Xcode.)
Resolves rdar://105727852.
Checklist:

[x] Code and documentation should follow the style of the Style Guide.
[x] If public symbols are renamed or modified, DocC references should be updated.

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test


Fix #1260: Return avg loss from evaluate() as promised

Fixes #1260

Let's just merge this one line change.


Correct the Readme

Remove the first bulletpoint here regarding node modules as this is incorrect.
Statement from Connor:
There should never be a node_modules folder when you have cloned or downloaded a project as it should always be in the gitignore.

How is this task coming along?

Hi there, I am done with this task. I have requested review from one of the QA's


Add link to system on ship screen

At the top of the ship screen (BASEURL/ship), add a link to the system as well as waypoint. Saves a step of clicking on the location, then clicking on the system link at top of page.
Current /ship screen
Status: DOCKED

Location: X1-GT12-54011D

New /ship screen:
Status: DOCKED

Location : X1-GT12-54011D in system X1-GT12

where X1-GT12 becomes a link to BASEURL/system/X1-GT12/

Actually a GUI issue, so moving it here.

Done with commit a0506ff


Harmonize button styles in measure and draw menu

This harmonizes the button styles in the measure and draw menu.

It also applies the style to the submenu that will be used for the collapsed tool menu.

Please review @terrestris/devs.

Thanks for the review!


patch example-Instagram to learn pull request

add "imageUrl.$" definition to example-instagram schema. Change field type and seed url from String to Array.

Actually I think I might change the upload widget to support string fields, instead of changing the schema. If that doesn't work I'll merge this though :)


SpacingAroundCurly misfiring when NoEmptyClassBody autoCorrect it on

Expected Behavior
SpacingAroundCurly should only fire if there are missing spaces around curly braces.
Observed Behavior
Consistent false positive for SpacingAroundCurly when NoEmptyClassBody.autoCorrect = true and there's an issue of type NoEmptyClassBody.
Steps to Reproduce
Configuration:
autoCorrect: false

formatting:
  active: true
  autoCorrect: true
  NoEmptyClassBody:
    active: true
    autoCorrect: true
  SpacingAroundCurly:
    active: true

File:
class EmptyClass {}

Findinds:
Ruleset: formatting - 10min debt
	NoEmptyClassBody - [Element(CLASS_BODY)] at EmptyClass.kt:1:18
	SpacingAroundCurly - [PsiElement(LBRACE)] at EmptyClass.kt:1:1

To the issue will no longer happen if you set the autoCorrect from NoEmptyClassBody to false.
Your Environment

Version of detekt used: 1.0.0.RC14 (tested on master as well)
Version of Gradle used (if applicable): N/A
Operating System and version: macOS
Link to your project (if it's a public repository): N/A

Can I ask if you can replicate using ktlint directly?

I tried, but I could not replicate it in KtLint

I could reproduce this issue.
KtLint uses two runs, one for formatting and an additional for reporting the remaining errors.
We just use one and have a config flag for this issue:  excludeCorrectable: true.
Please update your config to this:
build:
  maxIssues: 0
  excludeCorrectable: true

formatting:
  active: true
  autoCorrect: true
  NoEmptyClassBody:
    active: true
    autoCorrect: true
  SpacingAroundCurly:
    active: true


The host name should not be based on already owned domain paramName

Trying to update applicationidURI in AAD app. But it is failing to update and showing following error: "Failed to update application property. Error details: The host name should not be based on already owned domain paramName"
But as per the documentation, If you get an error stating that the domain is already owned and you are the owner, follow the procedure to add custom domain to AAD.
Issue is resolved by adding custom domain to AAD. But above highlighted both statements are contradicting.

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

ID: 52221dd3-95ec-96b9-1da8-9478527c2602
Version Independent ID: 32e9c2d8-733d-0865-2ea3-727d8684a10d
Content: Single sign-on support for tabs - Teams
Content Source: msteams-platform/tabs/how-to/authentication/auth-aad-sso.md
Product: msteams
GitHub Login: @laujan
Microsoft Alias: lajanuar

Document updated with correction.


Remove example from repository

Currently the example, which was only used for internal testing, is still included in the maven publish process and is still checked into the repository.
As other methods were used in there, it must not be published and has to be removed before the project "goes life".

With the switch to a gradle multi module build, the example is now in the repository permanently, but not included in the maven publishing process.


update output format of admin bucket check api

Relates to the change proposed here: https://github.com/ceph/ceph/pull/53607/commits/32fb6a1a68398a99324b2e64ebe3bcf3a9ccf02a

cherry-picked to ceph-master

this has now been cherry-picked to all backport branches


Inovelli LZW36 - incorrect status reported when turning on fan

Checklist:

[X ] I am not using HomeAssistant. Or: a developer has told me to come here.
[ X] I am not using ZWaveJS2MQTT. Or: a developer has told me to come here.
[X ] I have checked the troubleshooting section and my problem is not described there.
[ X] I have read the changelog and my problem was not mentioned there.

Describe the bug
When I turn on the fan on my fan/light dimmer switch, Zwave is reporting the light as turned on as well even though it is not.
Device information
Which device(s) is/are affected (make/model)?  Inovelli LZW36
What are the node IDs? 7
Did you change anything?

[ ] Yes: (please describe)
[ X] No

Did this use to work before?

[ ] Don't know, this is a new device
[ ] No, it never worked anywhere
[X] Yes, in: (specify application with versions) It worked with openzwave beta

How are you using node-zwave-js

[X] zwavejs2mqtt (latest) docker image
[ ] zwavejs2mqtt (dev) docker image
[ ] zwavejs2mqtt Manual Docker build

node-zwave-js branch:
zwavejs2mqtt branch:

[ ] ioBroker.zwave2 adapter
[X] HomeAssistant version 2021.3.3
[ ] Pkg
[ ] Manual Docker build

node-zwave-js branch:
zwavejs2mqtt branch:

[ ] node-red-contrib-zwave-js

Module version (double click node):

[ ] Manually built (as described in the docs)
[ ] Other:

To Reproduce
Steps to reproduce the behavior:

Go to '...' in HA, go to devices and select "Master Bedroom"
Click on '...' Turn on "Master Bedroom (2)" this is the fan
Scroll down to '...'
See error - "Master Bedroom" indicator turns on as well. This is the light but the light is not actually on.

Additional context
Add any other context about the problem here.
Logfile:
dc781aff_files.zip
zwavejs_1.log

@AlCalzone I think this is the same issue as with Zooz:
01:25:48.055 CNTRLR   [Node 007] Mapping unsolicited report from root device to first supporting end
                      point #1

Endpoint 2 turning on is sending a root device message mapped to endpoint 1.

User confirmed via discord that adding the compat flag to disable root mapping fixes this bug. I have reported this to Inovelli for their comment.

@themitzman it looks like your lifeline group contains two associations, one for Node 1, Endpoint 0 and one for Node 1, Endpoint 1.
Please remove the one to Node 1, Endpoint 1 and operate the fan afterwards. I want to make sure that this is not just the result of an incorrect association.

The main problem here is that we're getting a multi channel encapsulated report from endpoint 2 (correct) and a multicast (!) report from the root device. The latter not only seems incorrect but it is impossible to assign it to the correct endpoint when there are multiple possible sources it could have come from.
Unless this is just the result of the incorrect association, I might have to disable mapping altogether if it is ambiguous.

Could you explain that in dummy terms? I'm still fairly new to zwavejs so
I'm notn100% clear on what you mean by lifeline.
On Thu, Mar 25, 2021, 5:44 AM AlCalzone @.***> wrote:

@themitzman https://github.com/themitzman it looks like your lifeline
group contains two associations, one for Node 1, Endpoint 0 and one for
Node 1, Endpoint 1.
Please remove the one to Node 1, Endpoint 1 and operate the fan
afterwards. I want to make sure that this is not just the result of an
incorrect association.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zwave-js/node-zwave-js/issues/2155#issuecomment-806508464,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATBJKKDW7VKQKIKVK7MFCXLTFMAYLANCNFSM4ZYNFVZA
.

Z-Wave devices use associations to report changes to other devices and the controller. There is a dedicated group for the latter (normally group 1), which is called "Lifeline". Yours contains two associations, one of which might be the cause for the wrongly addressed reports.

Ok I see. When I get to my desk for work I'll take a look and reply back. I
appreciate the guidance. Maybe this happened when I migrated from ozw to
zwavejs.
On Thu, Mar 25, 2021, 7:34 AM AlCalzone @.***> wrote:

Z-Wave devices use associations to report changes to other devices and the
controller. There is a dedicated group for the latter (normally group 1),
which is called "Lifeline". Yours contains two associations, one of which
might be the cause for the wrongly addressed reports.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zwave-js/node-zwave-js/issues/2155#issuecomment-806578621,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATBJKKFGXGBLJ3VXRW2VWN3TFMNVRANCNFSM4ZYNFVZA
.

It's likely that OZW set up the endpoint 1 target and didn't map the weird root device report to an endpoint. AFAIK OZW does not hide the unnecessary root device values by default.
Zwave-js sets up the endpoint 0 target during the device interview but now you have 2 associations. It would have cleaned up unnecessary node associations (without an endpoint), but the endpoint 1 association does not automatically get removed.

Got it. I am going to revert the compat change we did last night,
reinterview the node, then remove it from the lifeline. I'll let you know
how it goes.
On Thu, Mar 25, 2021 at 8:19 AM AlCalzone @.***> wrote:

It's likely that OZW set up the endpoint 1 target and didn't map the weird
root device report to an endpoint. AFAIK OZW does not hide the unnecessary
root device values by default.
Zwave-js sets up the endpoint 0 target during the device interview but now
you have 2 associations. It would have cleaned up unnecessary node
associations (without an endpoint), but the endpoint 1 association does not
automatically get removed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zwave-js/node-zwave-js/issues/2155#issuecomment-806635430,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATBJKKFNLT5MZS5T2ETWBBLTFMS6JANCNFSM4ZYNFVZA
.

Probably better if you first remove the unnecessary association. Not sure if that changes anything, but this way it should not leave any traces.

OK so I removed the compat flag from last night, restarted the container,
removed the second entry from the lifeline association (endpoint 1), then
reinterviewed the node.  The issue is now resolved!
So out of curiosity, do you think the migration from openzwave 1.4 was the
potential cause? Meaning, openzwave created that association when it
shouldn't have?
On Thu, Mar 25, 2021 at 8:31 AM AlCalzone @.***> wrote:

Probably better if you first remove the unnecessary association. Not sure
if that changes anything, but this way it should not leave any traces.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zwave-js/node-zwave-js/issues/2155#issuecomment-806652399,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATBJKKHHRKKP2KJFF46WNDTTFMUJNANCNFSM4ZYNFVZA
.

I think its a combination. OZW set up a weird association, which worked for OZW because it didn't do anything special with the root reports. And zwave-js doing over-eager mapping between endpoints.

Fair enough assessment. Appreciate the assistance and glad it's something
easily fixed.
On Thu, Mar 25, 2021 at 9:09 AM AlCalzone @.***> wrote:

I think its a combination. OZW set up a weird association, which worked
for OZW because it didn't do anything special with the root reports. And
zwave-js doing over-eager mapping between endpoints.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zwave-js/node-zwave-js/issues/2155#issuecomment-806712505,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATBJKKHXLF5PZW46NKH4SLTTFMYZHANCNFSM4ZYNFVZA
.


Update Rect::Set(Vector2, Vector2) function

 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.


Neuron Simultion fails to run as WebGLLayer object no longer exists in Scenery

Neuron Simultion's WebGL code needs a bit of refactoring as WebGLLayer object on which it depends, no longer exists in Scenery's master branch. @jbphet @samreid  I am Working on fixing it.

@AshrafSharf - there has been a major upgrade to Scenery that doesn't yet include WebGL layer support.  The plan for now is to continue the development of Neuron using the 0.1 branch of Scenery, which still has this support.  Apologies for not giving you a heads up on this prior to the upgrade.

Ok. No Problem. Thanks

Forgot to mention: We will also need to use the 0.1 branch of Joist as well.

I decided to reopen this issue because, while it sounds like a good idea to move forward with publishing neuron using the 0.1 branches, we will eventually want neuron to build using scenery & joist master.

Being rewritten to use WebGLNode, I don't think this issue is relevant. Closing.


Update and fix the testnode

This PR:

Sync with the upstream(not sure if this is necessary for fixing the test node. Fortunately it works and it should be better having the latest version)
Fix the nitro test node

Key places to review
config.ts
How to test this PR:
 ./test-node.bash --init ----latest-espresso-image --espresso --validate

Run the command above and wait for about 5 mins, if the validator works normally then it is good.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.4 out of 8 committers have signed the CLA.:white_check_mark: tbro:white_check_mark: ImJeremyHe:white_check_mark: sveitser:white_check_mark: nomaxg:x: gvladika:x: tsahee:x: ariskk:x: diegoximenesYou have signed the CLA already but the status is still pending? Let us recheck it.


Document how to setup/run DMD test suite on WINDOWS

Jonathan Marler (@marler8997) reported this on 2018-02-10T00:02:06Z
Transferred from https://issues.dlang.org/show_bug.cgi?id=18413
CC List

Walter Bright (@WalterBright)
Seb
kinke
Nick Treleaven (@ntrel)
Mike Franklin

Description
Running the DMD test suite on windows seems to be very rare and documentation appears to be non-existent.  The following page (https://wiki.dlang.org/Building_under_Windows) would be a good place to hold such documentation.

I think the big issue with it is how to install Make/BASH.  I've tried a few times to get these installed but failed, however, I think I may have been using a version that was too old.  I'd ask that someone who has successfully gotten the test suite to run on windows share the steps they used on the wiki page.

greensunny12 commented on 2018-03-24T15:13:49Z
BTW guys I just built the GNUmake exe via AppVeyor in case someone is looking for this:

https://issues.dlang.org/show_bug.cgi?id=18658
https://github.com/dlang/installer/pull/311
https://ci.appveyor.com/project/greenify/installer-gqmr6/build/1.0.16/artifacts

So far my steps for reproducing everything under Wine:

---
wget http://downloads.dlang.org/releases/2.x/2.079.0/dmd-2.079.0.exe
wine dmd-2.079.0.exe
---

(install dmc too)

Then export these variables:

---
export DM_HOME=C:/D
export HOST_DC=dmd
export WINEPATH="C:\DMC\dm\bin\;C:\D\dmd2\windows\bin"
---

With this I can build everything under Wine:

---
cd dmd
wine make -f win32.mak
cd ../druntime
wine make -f win32.mak
cd ../phobos
wine make -f win32.mak
---

Now for the testsuite. Here are my attempts:

---
cd dmd/test
wget https://ci.appveyor.com/api/buildjobs/twy5nix34sn0329n/artifacts/gmake-4.2.zip
unzip gmake-4.2.zip
wine make.exe
---

but that still fails:

---
...
CreateProcess(Z:\bin\bash,/bin/bash -c "echo \"Building d_do_test tool\"",...)
Putting child 001DA438 (test_results/d_do_test.exe) PID 1578448 on the chain.
Live child 001DA438 (test_results/d_do_test.exe) PID 1578448
Main thread handle = 00000040
make: *** WaitForMultipleObjects: No such file or directory.  Stop.
make: *** Waiting for unfinished jobs....
Live child 001DA438 (test_results/d_do_test.exe) PID 1578448
make: *** WaitForMultipleObjects: No such file or directory.  Stop.
---

I assume this is due to missing bash. Did either of you manage to install bash?

Last but not least, there's also:

https://github.com/braddr/at-client/blob/master/src/do_test_dmd.sh

I see cygpath there. So I maybe cygwin is needed to run DMD's testsuite?

johnnymarler (@marler8997) commented on 2018-03-24T15:23:58Z
Building DMD on linux with WINE is a good use case.  However, that has a different set of problems than building on windows directly.  Namely, how to build/install GNUmake and BASH.

slavo5150 commented on 2018-02-13T03:44:09Z
https://github.com/dlang/dmd/blob/master/appveyor.sh could provide some insight into what is required as well.

greensunny12 commented on 2018-02-13T05:18:14Z
There's also Martin's gist on how we bootstraps the Windows VirtualBoxes for the releases:

https://gist.github.com/MartinNowak/8270666/

(there are similar gists for FreeBSD and OSX)

greensunny12 commented on 2018-03-25T05:41:19Z
For reference, I found this reply on the forum<EMAIL_ADDRESS>(it boils down to using mingw-make - not very ideal either)

kinke commented on 2018-03-24T15:33:57Z
(In reply to Jonathan Marler from comment #4)
> Building DMD on linux with WINE is a good use case.  However, that has a
> different set of problems than building on windows directly.  Namely, how to
> build/install GNUmake and BASH.

GNU make comes with a Visual Studio solution; the LDC AppVeyor jobs use a prebuilt version of mine. bash is available as part of git for Windows (already installed on AppVeyor).
See https://wiki.dlang.org/Building_and_hacking_LDC_on_Windows_using_MSVC#Running_the_dmd-testsuite_tests for make download link etc.

Wrt. bash, I seem to recall that this line here was of importance (not in DMD's test makefile): https://github.com/ldc-developers/dmd-testsuite/blob/ldc/Makefile#L108

slavo5150 commented on 2018-03-24T23:45:40Z
Building and testing D in a Bash environment on Windows is not ideal. It requires too many non-Windows dependencies and results in a testing environment different from the Windows-native environment that most users will be running their compiler and their programs from during "real" development.

We need something more Windows-native.  IMO, it would be ideal if we could just write our build and test system in D, removing make and Bash altogether.  The build-system will immediately be portable to any platform D supports and will reduce the skills required to maintain the build and test system from make, Bash, D, and C/C++ to just D and C/C++, with the latter requirement being removed when DMD's backend gets converted to D.

I maintain a few large projects in C#/C++ and their build, test, and publish systems are all custom built in C#.  Due to .Net's huge library of features the code is only a few hundred lines, and is simple for anyone with C# skills to understand, which is already required to maintain the software being built anyway.  It's always been somewhat ridiculous to me that developers use extremely expressive and powerful languages to develop their software, but choose to use restrictive, archaic when building that same software.

nick (@ntrel) commented on 2022-11-06T21:25:37Z
Looks like Jonathan updated the wiki:
https://wiki.dlang.org/DMD_development#DMD_Test_Suite_-_Windows_Requirements


TypeScript 2.1 async / await throws reference error exception (es5 target)

Note: for support questions, please use one of these channels: Chat: AngularClass.slack or Twitter: @AngularClass

I'm submitting a ...
[x] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Do you want to request a feature or report a bug?
Bug

What is the current behavior?
Using async / await in current setup throws exception where __generator is not defined on  ZoneAwarePromise (polyfills.bundle.js).

What is the motivation / use case for changing the behavior?
Being able to use async / await with latest TypeScript 2.1 and es5.

Please tell us about your environment:

Angular version: 2.2.1
Browser: Chrome

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

TypeScript isn't supported by Angular 2.x. You will have to wait until Angular 4.x

Can you explain what you mean with "TypeScript isn't supported by Angular 2.x"? TypeScript is the preferred language when writing Angular 2 apps. Since the async / await is compiled to es5 JavaScript code with the use of a polyfill containing a correctly implemented Promise it should work, or at least I thought so.
Please explain why Angular 2 is a specific problem for this scenario, compared to Angular 4, since it feels like I am missing something here.
Thanks!

http://lmgtfy.com/?q=angular+with+typescript+2.1.4

Can't see any real info about that it should not work, only some specific cases. I do use TypeScript 2.1.4 together with Angular 2.2.4 and it works great. The problem is the zone promise class being used and cannot be switched out. That promise does not work in conjunction with async / await.

There are other threads around that topic as well. The general consensus is it's not supported and won't be for a while. If you choose to push past what is in Angular's package.json you are in uncharted waters.
Async/Await is a cool feature and i'm sure i'll be leveraging it in another 6 months when it has robust support. You could create an issue over at Angular but i'm all but positive they are going to tell you the same thing.


Add support persistence for JwtTokenStore

I found the mention in org.springframework.security.oauth2.provider.token.store.JwtTokenStore class :

A TokenStore implementation that just reads data from the tokens themselves.
Not really a store since it never persists anything and methods like
#getAccessToken(OAuth2Authentication) always return null.

I think it would be good idea to add ability of persistence for class JwtTokenStore.

I guess persisting JWT tokens is not very useful as these aren't strictly checked by resource servers , but on the other hand storing refresh tokens might be useful to revoke them later.
We got this requirement in our company so the plan is to extend JwtTokenStore and persist refresh tokens for later revocation.


board: reel_board: remove board.h

Remove board.h, CONFIG_GPIO_NRF5_P1_DEV_NAME
was not used and is not defined anywhere.

Codecov Report

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

@@           Coverage Diff           @@
##           master   #13116   +/-   ##
=======================================
  Coverage   48.62%   48.62%
=======================================
  Files         313      313
  Lines       46484    46484
  Branches    10728    10728
=======================================
  Hits        22604    22604
  Misses      19413    19413
  Partials     4467     4467

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 f753ae9...8ab9814. Read the comment docs.


avoid overwriting aliases which are already defined

Hello,
I have aliases defined for s, d and f. I do not want fasd to overwrite those specific aliases.
Can you please ensure that fasd does not overwrite aliases which are already aliased?
Thanks

I just figured this out myself.
You'll want to init fasd with specific arguments and then manually create the aliases you want. You can see the init command used by fasd --init auto here: https://github.com/clvv/fasd/blob/master/fasd#L67
To remove the aliases you'll want to remove posix-alias from the list. You can see the default aliases here: https://github.com/clvv/fasd/blob/master/fasd#L81
For example, this is from my .zshrc:
### jump to recently used items
eval "$(fasd --init zsh-hook zsh-ccomp zsh-ccomp-install zsh-wcomp zsh-wcomp-install)"
function fasd_cd() {
  if [ $# -le 1 ]; then
    fasd "$@"
  else
    local _fasd_ret="$(fasd -e 'printf %s' "$@")"
    [ -z "$_fasd_ret" ] && return
    [ -d "$_fasd_ret" ] && cd "$_fasd_ret" || printf %s\\n "$_fasd_ret"
  fi
}
alias z='fasd_cd -d' # cd, same functionality as j in autojump

if it helps, I did this

# save aliases that will be overwritten by fasd initialization
local my_aliases=$(alias -L s d f)

# fasd posix-alias is messing up my aliases
test -f "${_FASD_DATA}" &&
    test -f "${_FASD_SINK}" &&
    test -d "${_FASD_VIMINFO}" &&
    test -f "${_FASD_RECENTLY_USED_XBEL}" &&
    eval "$(fasd --init auto)" ||
        echo "81-fasd.zsh: fasd not initializad as it is missing initialization files"

# revert aliases overwritten by fasd
eval "${my_aliases}"
# replace j of autojump with fasd
# alias j='fasd_cd -d'     # cd, same functionality as j in autojump
alias j='fasd_cd -d -i'     # cd with interactive selection


Support for Synthetics CI trigger endpoints

Note:
Is your feature request related to a problem? Please describe.
I'd like to use the Go library to trigger and poll the results of Synthetic CI tests:

https://github.com/DataDog/documentation/blob/e14faec768723b78006d44cdf99bb335fb4286b5/content/en/synthetics/ci.md

It appears that API code is updated from a repo:

https://github.com/DataDog/datadog-api-spec/pull/451

(But I don't believe that's a public repository to open a pull-request for)
Describe the solution you'd like
Support API spec updates to generate Go code to support the following endpoints:

https://github.com/DataDog/documentation/blob/e14faec768723b78006d44cdf99bb335fb4286b5/content/en/synthetics/ci.md

Not sure the right way to contribute/open a PR for the datadog-api-spec repo:

https://github.com/supply-drop/datadog-api-client-go/commit/a825a993c4caa67b73987bf9360f3786d4c3aaaf#diff-13d8957c539d429b92c59980a6539cd2

Hi Doug. Contributions to the spec repository are kept internal at least for now. This particular endpoint is in private beta, so in different situation. The team wants additional time to be allow to make changes if necessary, and will publish the spec when ready. Thanks for your patience.

I believe this has been available for a while. Let me know if that doesn't work for you, thanks.


Bug Fix,testcases addition

dev:
lint:211
coverage:85%
redirecting:
lint:209
coverage:85%
Bug fixed in multitenancyFilter also testcases added and
Folders restructured in meaningful way

please deploy


[KeyVault] Bump API version to 2023-07-01

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.KeyVault


proof of concept

The POC is complete and published at https://jakemclelland.github.io/romanizer/
We'll move on to moving this to a dedicated github account now


the type of super() should be instance, not void

TypeScript Version:  3.8.3
Search Terms:
Code
class MyArray extends Array {
  constructor () {
    return super();
  }
}

Expected behavior: no error
Actual behavior: TS2322
Playground Link:
Related Issues:

addon:
class MyArray extends Array {
  a :1;
  constructor () {
    super().a;// better to be able to warn not init yet
    this.a = 1;
    this.a;// ok now
  }
}

Are there any plans on fixing (investigating) this any time soon?

Are there any plans on fixing (investigating) this any time soon?

any news or update?
Recently, tsc compiled super to an inline anonymous function, I wonder why

@cangSDARM
Can you post the source code which reproduces that bug with the relevant tsconfig.json configuration?
```ts repro
// source code here
```

[Workbench Repro](https://www.typescriptlang.org/dev/bug-workbench/)

@cangSDARM Can you post the source code which reproduces that bug with the relevant tsconfig.json configuration?
```ts repro
// source code here

Workbench Repro

Sry, my mistake. I did a little more digging, and this is esbuild issue :) https://github.com/evanw/esbuild/issues/3538


List events on "Calendar"

Pull events and display on "Calendar" page. May wanty to be automated. If so you can use the feedjira plugin added in 1991ef3a45ec3230effc02bdb26ece8c69b3944c

@civicdata/core Is this issue for displaying Meetup data or other event data? Are you looking for a list of events or a calendar interface? Any other thoughts/requirements?


Handle actions on buttons

Hi,
i'm trying to handle action from button in my class function. The Problem is i can't define function which will handle action, i can only define object with function where word "this" is lost. I have hard time to get this work but i'm not seeing any workaround for this can anyone help me with this?
Any help is welcome
Thanks, Matija

when you define your actions, use a lambda function, which will keep the this:
this.actions = {
    send: () => {
         // `this` will be the containing class
    }
};

@schmuli tnx for reply it works


[FEATURE REQUEST] Ability to fail early on a host inside Task Workflow system

Fail quickly/early on a host if a certain step fails.
Say we have a step at the beginning that is very important, for backup.
If this step fails then halt all further steps for the specific hosts.
Something similar to what already exists as
    - options:
        fail_if_any_host_fail_any_step: []

maybe we could have something like
halt_if_any_host_fail_any_step: ["backups"]

Added support for stop_if_fail step parameter in 0.20.0 release, feel free to test and let me know how it goes.

Yep testing that now.
On Wed, 16 Aug 2023 at 11:30, Denis @.***> wrote:

Added support for stop_if_fail step parameter in 0.20.0 release, feel
free to test and let me know how it goes.
—
Reply to this email directly, view it on GitHub
https://github.com/dmulyalin/salt-nornir/issues/35#issuecomment-1680356765,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEXFZ2SR5BSQDVFPL3ZNYTXVSOMBANCNFSM6AAAAAAZ6XTFQI
.
You are receiving this because you authored the thread.Message ID:
@.***>


Fetch vct from URL or from registry

https://www.ietf.org/archive/id/draft-ietf-oauth-sd-jwt-vc-05.html#section-6.3.1
Is defining that if vct is an https:// it should check the metadata under the well known (at least the 2nd part of the text reads like this:

i.e., by inserting /.well-known/vct after the authority part of the URL.)

Many registries are, and will be accessible via URLs, hence the metadata type is expressed via an URL; Adding or maintaining a .well-known might not fit in the existing API designs. Also note that .well-known has well-known issues with multi-tenancy. Most use cases will delegate the hosting of the information to registries.
Also

URL https:///.well-known/vct/, i.e., by inserting /.well-known/vct after the authority part of the URL.

Questions:

if schema is https, should the full URL be provided? (no ambiguity with .well-known, you can host schema on github, ...)
metadata retrieval category re-consideration:

Fetch vct from a remote source:
a) URL: HTTPS schema -> full URL that points to a schema
b) URN: domain-defined URN that MUST be understood by the wallet; The URN method defines how to map the URN to URL and retrieve the data

Fetch vct the metadata locally
a) local cache
b) Signature (signed or unsigned header); Whether or not metadata is shared in the (un)protected header is defined by the signature format, hence out of scope of this document.

2b: point to consider for the OID4VP: should there be a flag: "archival mode" or similar, that would flag that the wallet needs to provide all the referenced content in an unprotected JWS header?

#272 removes the requirement to insert .well-known (discussion in #264)


[READY] MAINT - Rename scal by dnorm

fix #247

[x] rename scal by dnorm
[x] rename tmp by theta_scaling
[x] rename norm_Xtheta and norm_Xtheta_acc

Can we fix the naming for the prox newton solver also? Indeed, we follow a similar logic yet we adopt a different naming convention.
https://github.com/mathurinm/celer/blob/4230db117a916e5158cfae85b2cd1a7249cf5475/celer/PN_logreg.pyx#L50
Also, we have two variables (norm_Xtheta, norm_Xtheta_acc) that perform the same task. We better keep only one of them

+1 to harmonize with the same logic in prox newton solver

Codecov Report

Merging #255 (16e005a) into main (4230db1) will increase coverage by 0.02%.
The diff coverage is 100.00%.

:exclamation: Current head 16e005a differs from pull request most recent head 8c1bb10. Consider uploading reports for the commit 8c1bb10 to get more accurate results

@@            Coverage Diff             @@
##             main     #255      +/-   ##
==========================================
+ Coverage   88.42%   88.44%   +0.02%
==========================================
  Files          15       15
  Lines        1132     1134       +2
  Branches      136      136
==========================================
+ Hits         1001     1003       +2
  Misses        101      101
  Partials       30       30

Flag
Coverage Δ

unittests
∅ <ø> (∅)

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

Impacted Files
Coverage Δ

celer/homotopy.py
86.89% <100.00%> (ø)

celer/tests/test_lasso.py
100.00% <0.00%> (ø)

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 4230db1...8c1bb10. Read the comment docs.


Update mollie/mollie-api-php to latest version - any reason why it is still 2.40.1 (over a year old)?

Just wondering if the mollie/mollie-api-php dependency is limited to 2.40.1 for a reason. It should at least allow patch level versions I think which would be 2.40.2.
However, if you want to use a fixed version to ensure a specific version is always used, there is still the question why 2.40.1 is still used. That version is from January 2022.

hi
yes easy answer, there was a) no need, and b) this was the time when they dropped php support for version xy which we still needed, thats why its also locked to that version, even without lock file...just feels safer....(something like that, cannot fully remember)
so theres no downside at the moment or in general
but what a coincidence...we just recently discussed, that its time for another update of this too (and now with min support of shopware 6.4.x also the php min versions might be easier :)
hope this explains it

updated in 4.2.0 :)


Update test.sh

Add the -q (quiet) flag to gsutil uploads (fix https://github.com/flutter/flutter/issues/932).

LGTM


Add option to specify custom folder

Hello.
Would you consider an option via config file or cli of passing the custom folder argument so one can utilize it with quick download?

Hi.
Yes, its a good idea. A CLI argument would be quite simple to implement.
It should be done before considering a config file.
Whould you consider making a contribution ? Thanks!


Update cluster components section

Updated the cluster components description to mention that VM data will be automatically spread out across multiple CSVs during new installations.

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

Moved to private repo. #please-close


Retrieve Button State

pressed should be true if the button is held down

I can look into a PR for this, but it's a pretty complicated change to fix this issue.

@gw1108 thank you for the offer, but yea you probably want to leave that one to the team. It's fairly complicated.

Just chuck a new event state in that's based on the button events using, when down its set the variable to true and when released change it to false. I just wrote a quick DLL to handle this  and threw it in my projects. Then it's not on the interactive team to add it, it's just a small add-on from you lot

Thanks hoggy!


Fix org.eclipse.m2e lifecycle-mapping not found

Do we really need any of that? I don't think we use the install plugin since I removed all local jars anyway (in #2367). So you can probably remove the lifecycle mappings altogether.

Dunno.  But this fixes the vastly annoying messages in IntelliJ and VSCode for developers now!  :)  As a bonus, this does show how we can use profiles quite well for future needs.  Up to you if you want to gut it all.

Yes let's remove it all and keep profiles in mind - we should definitely use them if we ever need to add IDE-specific config.


creating relation error "Getting started part II"

From @tktk8924 on May 9, 2016 3:16
I'm trying "Getting started part II" and having a problem with creating relation between CoffeeShop and Review models.
I got the following error:
? Select the model to create the relationship from: CoffeeShop
? Relation type: has many
? Choose a model to create a relationship with: Review
? Enter the property name for the relation: (reviews)

/Users/takehiro/.nodebrew/node/v5.11.1/lib/node_modules/strongloop/node_modules/rx/dist/rx.js:77
    throw e;
    ^
true

I followed the following steps:
$ git clone https://github.com/strongloop/loopback-getting-started-intermediate.git
$ cd loopback-getting-started-intermediate
$ git checkout step2
$ npm install
$ slc loopback:relation

Versions:

Node.js: v5.11.1
strongloop: 6.0.0

Thanks.
Copied from original issue: strongloop/strongloop#309

From @sam-github on May 9, 2016 14:39
cc: @superkhau

Duplicate of https://github.com/strongloop/loopback/issues/2312. Closing.


[BEAM-3202] Ensure that PipelineOptions.getOptionsId is always populated.

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

R: @staslev

SUCCESS
--none--

SUCCESS
--none--

SUCCESS
--none--

FAILURE
--none--

FAILURE
--none--

SUCCESS
--none--

@lukecwik Do you think it would be possible to have it cherry-picked for the upcoming 2.2.0 release?

If the current release candidate doesn't make it through validation, I'll try to get it cherry picked otherwise I would rather get this release out as is because of how long it has been taking.


"é" in JSON file doesn't work

It gets replaced with "é"

Lol, nevermind. Haha, just wasn't web encoded in my terminal and text viewer


DevDB - tweaks to outputs for GIS/Bytes

Incorporating @AmandaDoyle 's feedback in #21

project-level output (_export.sql, export.sh): restrict columns for export. This is mostly done in sql at the level of view creation. However, there's one column (Job_Inactv) that is not public but is used to subset the data for exports, so it's kept in the views (as to not list every column in the dataset 5 times), then dropped when csv export is done.
unit_change_summary files:

remove all geometric fields from csv export. I had a typo - in spatial.sql did not select columns with double quotes (preserving capitalization), but at export tried to exclude capitalized columns
capitalize output columns for CDTA, NTA for bytes output because that's what's expected
change logic around max year in columns to be based on version rather than date of running

alter shp_export to allow for additional options to be fed to ogr2ogr call (to set output srs and export in 2263 for all bytes outputs)

Latest action (success)

So I've come up against a slight issue running our code inside a docker image. Basically, the checkout action runs with user id 1001 while any subsequent commands in the container are run as root. This causes a mismatch in some cases of ownership in folders, which git is unhappy about when we try to use it from the command line in the container.
We use git as part of our build process because we export to folders based on branch name in digital ocean s3. See here.
This has led to this error when trying to call that function
fatal: detected dubious ownership in repository at '/__w/data-engineering/data-engineering'
To add an exception for this directory, call:

	git config --global --add safe.directory /__w/data-engineering/data-engineering

You can see the owner of folders at this point here. Note the different owner of the root workspace directory.
Two fixes to this: first is specify user as 1001 in github action
jobs:
  build:Take
    runs-on: ubuntu-22.04
    container:
      image: nycplanning/base:latest
      options: --user 1001
      env: ...

Other is to declare the working directory as a safe directory globally, and then run this in a shell script. We already have to run an "init" sort of step in the github actions, so
- name: Add s3 endpoint ...
   run: mc alias set spaces $AWS_S3_ENDPOINT $AWS_ACCESS_KEY_ID $AWS_SECRET_ACCESS_KEY --api S3v4

becomes
- name: Finish container setup ...
   run: ./bash/docker_container_setup.sh

And we don't specify userid. So slightly cleaner yml IMO. Then the script looks like
#!/bin/bash

mc config host add spaces $AWS_S3_ENDPOINT $AWS_ACCESS_KEY_ID $AWS_SECRET_ACCESS_KEY --api S3v4

if [[ -z "$CI" ]]; then
    workspace=/workspace
else
    workspace=/__w/data-engineering/data-engineering
fi
echo $workspace
git config --global --add safe.directory $workspace

For now

Fun stuff! git config --global --add safe.directory seems 👍

Re-running job after rebase fun to make sure I didn't screw anything up
https://github.com/NYCPlanning/data-engineering/actions/runs/5468754244


SFDX Cli doesn't recognise aura components nested under the aura folder

Summary
In our project we organised metadata components in the sub folders. For example Abstract Classes, Helpers are sub folders under classes folder. Base Components, Service Components are subfolder under aura folder. Everything worked like a charm before the plugin-source was integrated into the cli in version ~7.112. Starting from that version and till now (ver 7.133) if we try to deploy some aura component that is placed in the subfolder (i.e. force-app/main/default/aura/Quick Actions/Super Action  with sfdx force:source:deploy -x manifest/package.xml command we receive ... AuraDefinitionBundle was named in package.xml, but was not found in zipped directory or No source-backed components present in the package. error.
Steps To Reproduce:
Repository to reproduce: sfdx-nested-aura-issue

Clone the repository
Create a default scratch org sfdx force:org:create -f config/project-scratch-def.json --durationdays 1
Try to deploy source to scratch org sfdx force:source:deploy -x manifest/package.xml

Expected result
All Aura components are deployed even they are placed in the subfolder under aura
Actual result
No source-backed components present in the package. is thrown.
System Information
{
        "cliVersion": "sfdx-cli/7.133.0",
        "architecture": "darwin-x64",
        "nodeVersion": "node-v16.13.1",
        "pluginVersions": [
                "@muenzpraeger/sfdx-plugin 0.5.0",
                "@oclif/plugin-autocomplete 0.3.0 (core)",
                "@oclif/plugin-commands 1.3.0 (core)",
                "@oclif/plugin-help 3.3.1 (core)",
                "@oclif/plugin-not-found 1.2.6 (core)",
                "@oclif/plugin-plugins 1.10.11 (core)",
                "@oclif/plugin-update 1.5.0 (core)",
                "@oclif/plugin-warn-if-update-available 1.7.3 (core)",
                "@oclif/plugin-which 1.0.4 (core)",
                "@salesforce/lwc-dev-mobile 1.4.0",
                "@salesforce/sfdx-plugin-lwc-test 0.1.7 (core)",
                "alias 1.2.0 (core)",
                "apex 0.7.0 (core)",
                "auth 1.8.0 (core)",
                "config 1.3.14 (core)",
                "custom-metadata 1.0.12 (core)",
                "data 0.6.8 (core)",
                "generator 1.2.1 (core)",
                "info 1.2.0 (core)",
                "limits 1.3.0 (core)",
                "org 1.11.0 (core)",
                "salesforce-alm 53.7.2 (core)",
                "schema 1.1.0 (core)",
                "sfdmu 4.12.3",
                "sfdx-cli 7.133.0 (core)",
                "source 1.7.0 (core)",
                "telemetry 1.4.0 (core)",
                "templates 52.7.0 (core)",
                "trust 1.1.0 (core)",
                "user 1.7.0 (core)"
        ],
        "osVersion": "Darwin 21.3.0"
}

Hi @steals prior to that version we used a less strict version of the source deploy mechanism, that allowed developer to make things like nesting code, which is a bad code practice.
Last year we introduced a new library called SDR that is more performant and robust but also more strict about where developers place their code, you're not the first user to report this problem, unfortunately there is no way for the new library to go back being as permissive as the old one because that would break many things, we encourage developers to stay away from nesting components under any type of folder not just aura, Im sorry I can't give you any more help.


Improvements to lucid initialization api

A small improvement that would make it easier to extend a lucid class or use the very handy Utils class without the whole lucid instance.

@alessandrokonrad might wanna take a look?.

Hey thanks for the PR, I don't think it's a big deal to have to instantiate Lucid here. You can leave out the provider then you have something like this:
const lucid = await Lucid.new();
const pk = lucid.utils.generatePrivateKey();
...

I'm not convinced of changing the Utils class to taking only the network as constructor variable instead of the entire lucid instance. Maybe there will be other utils functions in the future which depend not only on the network.


train by myself datasets

I am training the bcnet network with a dataset I made myself and I wrote a script myself to register my dataset successfuly.
my json tags：（for example）
‘segmentation’ refers to the Occludee,and 'leaf_mask' refers to the Occluder

So  I made a little modification for codes according to my json tags :
in detectron/data/datasets/coco.py

in detectron/data/detection_utils.py

I could start  training but I found some exceptions：

'loss_mask_bo' and 'loss_boundary_bo'  are 0.000 always.
can you help me ?

And after the training, the visualization results are only the Occludee, not the occluded objects.

And after the training, the visualization results are only the Occludee, not the  Occluder.

can you visualize your "segmentation" and "leaf_mask" groundtruth masks?

Yes, I have visual steps when I follow the labelme operation and visualize successfuly.


Refactor isSorted boolean parameters to sorted

Minor style issue, see discussion at https://github.com/bigdatagenomics/adam/pull/1324#discussion_r94042658

I will roll this in to #1324

Closing in favor of #1324.


Does This Library support Realm database?

I want to run server requests in a separate thread and save the data in a realm database.

You can just try it out for yourself :)
if everything you're calling falls under the category "What's possible?" then it will work. Otherwise you'd have to change a few things in the code you're calling (e.g. making it a 'worklet', using a JSI only library like react-native-mmkv, etc ...)

It doesn't. Even by having simple (following code example) in the app makes the app crash. When I logged realm call out, it didn't crash.
const results = realm.objects(SomeSchemaName)
console.log(results)

The error that I get is fails withcom.myapp A/libc: Fatal signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x10 in tid 32706 (HeapHelper), pid 32608 (com.myapp)`

That's because the code you are trying to use does not have the 'worklet' directive at the top of the functions, therefore it is not supported.

From my memory, realmjs not able to communicate between threads, it doesn't compatible with this library.


a few possible additions

a few possible additions for fontello/svg-font-dump:
added DumpSVG() to make it work as a module
added -x option to save with just numeric filenames
added MakeDirectory(fp) to make the args.glyphs_dir directory tree if it isn't present
added usage examples for use as a module in node code, and from the CLI
see mainly
if(require.main!=module){/Use as a module/module.exports={DumpSVG:DumpSVG}
}else{/Use from the command line/
}
went through git hell to make this PR (as is my 1st attempt at such, hope it worked)
good work!! thanks for the svg font tools

Sorry, this project was done for internal (fontello) needs not planned to extend anyhow.


Allow timestamps in .crx file to be set deterministically

Currently the timestamps are set to the current time, which causes the .crx file to be different every time it is rebuilt.  It would be nice to have a mechanism to set the dates and permissions like grunt-contrib-compress does, or failing that, to read them from the file stats instead of from the current system time.

On closer examination, this would be kind of pointless at the moment unless you switched to something other than node-archiver as it seems it doesn't guarantee the order of the files in the ZIP file.

You mean you would like the zip file to be timestamped with the same date as the most recent file contained in the archive?

Assuming I'm understanding you correctly, no, the issue is the timestamps (one for each file) recorded in the contents of the ZIP file, not the modification time of the ZIP file itself.

Oh OK, I understand. Why would you do alter the modified time of files inside the ZIP file?

My original goal was to have the build process produce the same file, bit for bit, each time I ran it.  But now I think it's kind of moot, because I found out that if the ZIP file is built with node-archiver, the order of the files within the ZIP file can change from run to run.  You'd have to switch to a different package for building the ZIP files, which is probably too much work for this one use case.
What I ended up doing is building the ZIP files separately with jszip and feeding them to your command-line app for signing.


EmbedMolecule hangs forever

Describe the bug
The following molecule causes embedding to hang forever (more than 5 minutes on my machine), also huge delay when trying to stop with ctrl-c:
CCSCc1ccc2c(c1)c1c3c(c4c5cc(CSCC)ccc5n5c4c1n2[C@@H]1C[C@H](C(=O)OOC)[C@]5(C)O1)CNC3=O
To Reproduce
mol = Chem.AddHs(Chem.MolFromSmiles('CCSCc1ccc2c(c1)c1c3c(c4c5cc(CSCC)ccc5n5c4c1n2[C@@H]1C[C@H](C(=O)OOC)[C@]5(C)O1)CNC3=O'))
Chem.EmbedMolecule(mol)

Expected behavior
Success, failure or exception in reasonable amount of time
Configuration (please complete the following information):

RDKit version: rdkit-pypi 2<IP_ADDRESS>
OS: Ubuntu 20.04.2 LTS
Python version (if relevant): 3.8.10
Are you using conda? No
If you are not using conda: how did you install the RDKit? pip

@xtsm : I can't reproduce this, but the molecule is probably incorrect anyway.
On my machine it takes quite a while, and the conformer generation eventually fails, but it doesn't run for multiple minutes:
In [3]: mol = Chem.AddHs(Chem.MolFromSmiles('CCSCc1ccc2c(c1)c1c3c(c4c5cc(CSCC)ccc5n5c4c1n2[C@@H]1C[C@H](C(=O)OOC)[C@]5(C)O1)CNC3=O'))
In [5]: %time rdDistGeom.EmbedMolecule(mol)
CPU times: user 50.7 s, sys: 11.2 ms, total: 50.7 s
Wall time: 50.8 s
Out[5]: -1

What seems to be making this fail (and take so long) is generating conformers which match the chirality which you've specified; if I turn the use of chirality in the conformer generation process off then things run quite quickly:
In [6]: ps = rdDistGeom.ETKDGv3()

In [7]: ps.enforceChirality = False

In [8]: %time rdDistGeom.EmbedMolecule(mol,ps)
CPU times: user 167 ms, sys: 24 µs, total: 167 ms
Wall time: 166 ms
Out[8]: 0

When this situation occurs with a molecule which only has a few chiral centers it's often a sign that there's a problem with the molecule itself and, sure enough, looking at the molecule the stereochemistry is probably wrong. Here's your molecule, with atom numbers indicated:

I believe the relative chiralities of atoms 28 and 36 are incorrect; that the molecule cannot exist as specified.
Sure enough, if I flip the chirality of one of those bridgehead atoms, I end up getting a conformer quite quickly:
In [16]: mol2 = Chem.AddHs(Chem.MolFromSmiles('CCSCc1ccc2c(c1)c1c3c(c4c5cc(CSCC)ccc5n5c4c1n2[C@@H]1C[C@H](C(=O)OOC)[C@@]5(C)O1)CNC3=O'))

In [17]: %time rdDistGeom.EmbedMolecule(mol2)
CPU times: user 185 ms, sys: 2 µs, total: 185 ms
Wall time: 184 ms
Out[17]: 0

So please check your input structure.


    modified:   src/jshint.js

Coverage remained the same at 97.763% when pulling f62cf34b9be0b5c0f9ad7ea7b9e9f2343b0a54c3 on FuDesign2008:bugfix/undefined-trim into cc215bde53610e861d5a679131e7095add1bb3f8 on jshint:master.

Can you provide more insight into how this occurred?

Working backwards from the modified statement, I've identified the problem this patch addresses. Example input:
// jshint maxstatements

It's certainly invalid to declare a value-accepting option without a value, but users with JSHint editor integrations are likely to transition through this state as they type. JSHint shouldn't crash here, for sure, and this patch addressed the problem.
A few things, though:

We should include a test to demonstrate that this addresses the problem
We'll need your signature on the project's CLA. This might sound like a
hassle, but it only takes a minute or so.

We still haven't heard back from @FuDesign2008, but I'm hoping that with this more specific feedback, they might be able to finish up this patch. Otherwise, I'll submit a version of my own in a few days.

@FuDesign2008 ping, can you write a unit test and sign the CLA?


Customer feedback and no regressions between M.IM7 and 8

@jennyf19  Is there any specific work that needs to be done for this?


Possible changes to STAC schema at https://cdn.staclint.com exposes 3 Unit tests failures

Running unit tests on master as of 3/2/2020, I see 3 test failures - output attached.  These tests are failing when try to pull a non-existent file at 'https://cdn.staclint.com/master/geojson.json'.  The last known version i can find this file is at https://cdn.staclint.com/v0.7.0/geojson.json. Not sure if tests are now wrong for > version 0.7 or if validation has fundamentally changed.

Hi @hasselstromk.
Ive updated the CDN to have a copy of the geojson spec under the master route now.
I haven't updated the validator in some time so it may not be correctly validating your STAC file.


Updat value function with different action types, why?

Hi Shariq,
In your code you update the value function with actions computed by:

gumbel_softmax
onehot_from_logits

As far as I know, 1) has the gradient attached, while 2) does not.
Why did you implemented it this way?

Hi @tessavdheiden,
I believe that in one call of update you want to update actor just for one actor (namely for actor attach to agent_i). That is the reason why you send one action sample with gradient attached (representing the action of agent agent_i) and others without.


Hide "Send message as [user]..." from he input line

What should be added?
A way to hide "Send message as [user]..." from he input line
Why should it be added?
Previous versions didn't have this feature and honestly it's way too distracting. It also becomes useless when you only have one account connected.

This has been discussed to lengths in https://github.com/Chatterino/chatterino2/issues/2305.

duplicate of #2305

I find this more distracting than the chatterino implementation. You have every right to ignore it, but I don't see a reason to add more unnecessary settings:

Duplicate of #2305, see discussion there for more explanation. Also, after some time I (and others) eventually got used to this.

However after some thinking maybe we could just change the placeholder text to show us the name of the channel instead of current account, not sure if I want to touch that again though - it could cause another wave of complaints.


Can't Download Due to Character Limit

Checklist

[ ] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2<IP_ADDRESS>
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log

Z:\YouTube-DL>cd /D "Z:\YouTube-DL\"

Z:\YouTube-DL>youtube-dl https://www.youtube.com/channel/UCJCFJWc0NnkyMbbNmB32d_Q --config-location config.txt
[debug] System config: []
[debug] User config: []
[debug] Custom config: ['-o', 'L:/YouTube/Politics/%(uploader)s/%(title)s/%(upload_date)s.  %(title)s  .%(id)s, %(location)s, %(height)sp %(vcodec)s %(acodec)s YouTube %(uploader)s, Views %(view_count)s, Likes %(like_count)s Dislikes %(dislike_count)s Comments %(comment_count)s Vidrate Audrate %(abr)s FPS %(fps)s', '--verbose', '--merge-output-format', 'mkv', '--write-thumbnail', '--embed-thumbnail', '--write-info-json', '--add-metadata', '--no-playlist', 'youtube-dl', '-U', '--format', '(bestvideo[width>=1920]/bestvideo)+bestaudio/best', '--embed-subs', '--sub-lang', 'en', '--ignore-errors', '--restrict-filenames', '--write-description']
[debug] Command-line args: ['https://www.youtube.com/channel/UCJCFJWc0NnkyMbbNmB32d_Q', '--config-location', 'config.txt']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2<IP_ADDRESS>
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.16299
[debug] exe versions: ffmpeg N-65018-gad91bf8, ffprobe N-65018-gad91bf8
[debug] Proxy map: {}
youtube-dl is up-to-date (2<IP_ADDRESS>)
ERROR: 'youtube-dl' is not a valid URL. Set --default-search "ytsearch" (or run  youtube-dl "ytsearch:youtube-dl" ) to search YouTube
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpspbsbtmq\build\youtube_dl\YoutubeDL.py", line 797, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpspbsbtmq\build\youtube_dl\extractor\common.py", line 530, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpspbsbtmq\build\youtube_dl\extractor\generic.py", line 2277, in _real_extract
youtube_dl.utils.ExtractorError: 'youtube-dl' is not a valid URL. Set --default-search "ytsearch" (or run  youtube-dl "ytsearch:youtube-dl" ) to search YouTube

[youtube:channel] UCJCFJWc0NnkyMbbNmB32d_Q: Downloading channel page
[youtube:playlist] UUJCFJWc0NnkyMbbNmB32d_Q: Downloading webpage
[download] Downloading playlist: Uploads from channel with 1 video with 1000 subs
[youtube:playlist] playlist Uploads from channel with 1 video with 1000 subs: Downloading 1 videos
[download] Downloading video 1 of 1
[youtube] biS0LCX5LUk: Downloading webpage
[youtube] Downloading just video biS0LCX5LUk because of --no-playlist
[youtube] biS0LCX5LUk: Downloading MPD manifest
[info] Writing video description to: L:\YouTube\Politics\channel_with_1_video_with_1000_subs\TANKS_4_0_SUBSCRIENVEBERS_@\20190913.  TANKS_4_0_SUBSCRIENVEBERS_@  .biS0LCX5LUk, NA, 360p vp9 mp4a.40.2 YouTube channel_with_1_video_with_1000_subs, Views 2, Likes NA Dislikes NA Comments NA Vidrate Audrate 128 FPS 30.description
ERROR: Cannot write description file L:\YouTube\Politics\channel_with_1_video_with_1000_subs\TANKS_4_0_SUBSCRIENVEBERS_@\20190913.  TANKS_4_0_SUBSCRIENVEBERS_@  .biS0LCX5LUk, NA, 360p vp9 mp4a.40.2 YouTube channel_with_1_video_with_1000_subs, Views 2, Likes NA Dislikes NA Comments NA Vidrate Audrate 128 FPS 30.description
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpspbsbtmq\build\youtube_dl\YoutubeDL.py", line 1785, in process_info
FileNotFoundError: [Errno 2] No such file or directory: 'L:\\YouTube\\Politics\\channel_with_1_video_with_1000_subs\\TANKS_4_0_SUBSCRIENVEBERS_@\\20190913.  TANKS_4_0_SUBSCRIENVEBERS_@  .biS0LCX5LUk, NA, 360p vp9 mp4a.40.2 YouTube channel_with_1_video_with_1000_subs, Views 2, Likes NA Dislikes NA Comments NA Vidrate Audrate 128 FPS 30.description'

[download] Finished downloading playlist: Uploads from channel with 1 video with 1000 subs

Z:\YouTube-DL>PAUSE
Press any key to continue . . .

Description

WRITE DESCRIPTION HERE
Verbose log is above. On Windows 7 SP1 64-bit and Windows 10 64-bit this is about the longest command I can run:
-o "L:/YouTube/Politics/%(uploader)s/%(title)s/%(upload_date)s.  %(title)s  .%(id)s, %(location)s, %(height)sp %(vcodec)s %(acodec)s YouTube %(uploader)s, Views %(view_count)s, Likes %(like_count)s Dislikes %(dislike_count)s".
Here's the full path that's created (it's 239 characters):
L:\YouTube\Politics\channel_with_1_video_with_1000_subs\TANKS_4_0_SUBSCRIENVEBERS_@\20190913.  TANKS_4_0_SUBSCRIENVEBERS_@  .biS0LCX5LUk, NA, 360p vp9 mp4a.40.2 YouTube channel_with_1_video_with_1000_subs, Views 2, Likes NA Dislikes NA.mkv
Video is downloaded fine. However when I run a longer command such as the below then nothing is downloaded in Windows 7 or Windows 10. I enabled long file names in Windows 10 gpedit.msc and restarted my PC but it made no difference:
-o "L:/YouTube/Politics/%(uploader)s/%(title)s/%(upload_date)s.  %(title)s  .%(id)s, %(location)s, %(height)sp %(vcodec)s %(acodec)s YouTube %(uploader)s, Views %(view_count)s, Likes %(like_count)s Dislikes %(dislike_count)s Comments %(comment_count)s Vidrate &(vbr)s, Audrate %(abr)s, FPS %(fps)s"

Use
-o "\\?\L:\YouTube/Politics/%(uploader)s/%(title)s/%(upload_date)s. %(title)s .%(id)s, %(location)s, %(height)sp %(vcodec)s %(acodec)s YouTube %(uploader)s, Views %(view_count)s, Likes %(like_count)s Dislikes %(dislike_count)s Comments %(comment_count)s Vidrate &(vbr)s, Audrate %(abr)s, FPS %(fps)s"
Only works on Windows, each individual segment of the path must be max ~255 chars still. All the problems of deleting/playing back these files after download are your problem :3


proxy authentication from the portal to IPA

I'm not sure what this is about.

History of this user story that was created.


Feature/stringconfig trigger

Fixes #7

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

v1.5.0
GitHub release

Your semantic-release bot :package::rocket:


Blink on battery saver mode

Circular Progress blink instead of rotate when on battery saver, Android >=5.0

Seems like all application animations are disabled in battery saver mode, causing all ValueAnimators to end instantly. However, CPV just resets the animation at the end of the indeterminate animation, so it just sorta flickers and crashes my emulator. @fredericojssilva: Was flickering the only issue or did it cause other problems like crashing or battery drainage?

I don't know if there is a good solution to this. However, since battery saver mode was introduced in 5.0, I would suggest using the native progress bar when running >5.0. I believe this should be possible with the (<include>)[http://developer.android.com/training/improving-layouts/reusing-layouts.html#Include]) tag to include a layout/progress_bar.xml using a CPV, and then making a layout-v21/progress_bar.xml that uses the native ProgressBar.

@rahatarmanahmed What do you think about not running animation in battery saver mode? This flickering looks really bad.

@rongi As I said before, I still think this should be solved by not using CPV on API >= 21. Much easier to just swap in the native CPV than to deal with handling battery saver mode logic.
(That being said, if you can figure out how to do that automatically in CPV itself without breaking it for older versions, I would definitely accept a PR for that.)

@rahatarmanahmed Is it possible to have native view that looks the same as CPV on Android API >= 21? Is it possible to change color of that view? Will this native view look the same across different android devices? I'm using CPV mainly for consistency of how progress view looks across devices.

I've just looked at behaviour of standard ProgressView and it looks like they just don't show their progress if device is in a battery saving mode. It feels like even this is a better behaviour than non stopping flickering.


Dino Crisis NTSC-J anti piracy screen

tonyhax version: 1.3.2
Game name: Dino Crisis
Game code: SLPS-02180
Console model: PS1 FAT USA - SCPH-5501
Executable hash:  SLPS_021.80 - SHA1: 610FEE67C86A920B457463EBFC831F48D9535BF5
Sorry but I didn't find the executable on the original CD
Triggered at:  After the violence warning screen

Thanks for the report. Fixed with d219a71fa589559bdac511385db37a0b622edb37.


When browsing offline, cache is not accessed when clicking links only when using back/forward navigation

When clicking link:

After pressing back and then forward navigation:

This happens sometimes, sometimes the caching works.

Unable to reproduce


WIP: test L7 policy with NodePort BPF

Might fix #8971.

This change is

test-me-please

Coverage decreased (-0.01%) to 44.638% when pulling 591317cb76c195491630a93876eae4db1c4d3878 on pr/brb/l7-np-bpf into 4836b2ab7639abbecb783a9be449e350c2358636 on master.

test-me-please

test-me-please

test-me-please

test-me-please


Issues with drag and drop in firefox

When I start dragging from the sidebar in firefox it seems to swap to the Moving around mode and resets the drag, causing a stutter in the sidebar. Newest versions of firefox & ory-editor & react 16.3

Could be an issue related to react 16.3 - I'll check this and also for #535


Recipe Journal provides sparse detail, and no indication as to where an error occurred in a recipe step.

Currently, the Recipe Journal reports that a step has begun, and if an error occurs it logs the error message. It does not provide a breakdown of the location in the Step where the problem occurred, which would be useful information when investigating issues.
As an example, the CommandRecipeHandler loops through all commands in the Recipe file and executes them. If a command fails it would be valuable to know which commands succeeded, in order to more easily identify the problem item.
In order to facilitate improved logging, I propose that a string property representing the Execution ID be added to the RecipeContext model object, so that it is available to all RecipeHandlers. Also, IRecipeJournal would be passed into the constructor of each recipe handler, thereby making the WriteJournalEntry method available to the respective classes.
The proposed change would require no breaking changes. There would be a single property added to a POCO class, and some logging code in each handler. I believe that the benefit of having a trace of Features/Commands/Widgets/etc that have been successfully processed makes this a worthwhile undertaking, and our team have already found this to be of value.
I have completed the proposed changes on our implementation and would be happy to submit them to the orchard repository for review.

+1

https://github.com/OrchardCMS/Orchard/pull/5371

Pull request merged so issue closed.


Ui event expired

The element is fixedly positioned, using the bounceInRight animation, scrolling the mouse wheel, the content inside this element is not scrolling

Sorry but can you elaborate more on the question? I couldn't understand what's the actual issue.

It is good to add a sample link on codepen so that we can fix it... ;) hi @eltonmesquita good to see you here... I've been busy the past months.

I think this can be closed as there's no response or clear info about the issue.
Good to see you too @WarenGonzaga and welcome to the Busy Club Inc ;D


Initial the unit testing and add a unit test for the main class, Omise

1. Objective reason
To deliver quality software, the unit testing is one of many parts that can be achieved. It is easy to initial the unit testing at the beginning of the development.
Required pull request: #1
2. Description of change
Initial the unit testing and add a unit test for the main class, Omise. The unit testing will be run with PHPUnit.
Note:

To run unit test

$ phpunit

To run unit test with code coverage reporting

$ phpunit --coverage-html report/
From the second sample command, the report will be generated to the directory report/. It can be changed to another directory.
3. Users affected by the change
None
4. Impact of the change
-
5. Priority of change
Normal
6. Alternate solution
-

Test

@nimid got it,
anyway, could you consider about both name below,
~/D/O/omise-prestashop (feature/initial-unit-test) $ ./vendor/bin/phpunit tests
PHPUnit 4.4.5 by Sebastian Bergmann.

Configuration read from /omise-prestashop/phpunit.xml

...F.F

Time: 51 ms, Memory: 3.50MB

There were 2 failures:

1) OmiseTest::HookPayment_module_is_activated_and_enabled_display_title
Failed asserting that true matches expected 'x'.

/omise-prestashop/tests/unit/OmiseTest.php:38

2) OmiseTest::HookPayment_module_is_activated_and_enabled_display_the_payment_form
Failed asserting that true matches expected 'x'.

/omise-prestashop/tests/unit/OmiseTest.php:51

FAILURES!
Tests: 6, Assertions: 6, Failures: 2.
~/D/O/omise-prestashop (feature/initial-unit-test) $

and
~/D/O/omise-prestashop (feature/initial-unit-test) $ ./vendor/bin/phpunit tests
PHPUnit 4.4.5 by Sebastian Bergmann.

Configuration read from /omise-prestashop/phpunit.xml

...F.F

Time: 50 ms, Memory: 3.50MB

There were 2 failures:

1) OmiseTest::testHookPayment_moduleIsActivatedAndEnabled_displayTitle
Failed asserting that true matches expected 'x'.

/omise-prestashop/tests/unit/OmiseTest.php:36

2) OmiseTest::testHookPayment_moduleIsActivatedAndEnabled_displayThePaymentForm
Failed asserting that true matches expected 'x'.

/omise-prestashop/tests/unit/OmiseTest.php:46

FAILURES!
Tests: 6, Assertions: 6, Failures: 2.

I like the first one because it's kind of I'm reading just sentence. What do you think?
Btw, I'm ok with your naming style, feel free to ignore above. :)

@guzzilar For the test case name that consist of scenario and expectation, using the underscore to be the separator. It is not easy to distinguish.


Format "Airflow DAG Load Data Complete" duration

Description

The notifications are amazing! But currently show the duration in seconds like this:

We might want to:

Round
Convert to a human readable duration  (for example, "47 minutes", "1 hour", or alternatively something like "hh:mm:ss")

This is a very minor concern 😄
Relevant file
https://github.com/WordPress/openverse-catalog/blob/5188b388759a00c42785ca79f0c96949396d4d68/openverse_catalog/dags/common/loader/reporting.py#L18

We'll want to tackle #377 before this, since duration will be a float after that and will make it easier to round/manipulate.
I think it may also be worth adding a comma separation^1 to the record upsert count (e.g. f"Count: {value:,}")


feat(c): Implement FOREIGN KEY constraints

Test and Implement Foreign Keys
Discussion points

SQLite constraints are currently nameless, whereas the PostgreSQL constraints have generated names. Should it stay that way?
SQLite and PostgreSQL return their constraints in different orders. Is that by design?
Since the order of constraints can vary between DBs, the Foreign Key tests currently identify constraint columns by their number of constraints. Maybe there's a better solution to it?
c/validation/adbc_validation.cc lines [1279-1282] assume a certain order of the constraint column usages. Should we find an order-agnostic solution instead? It might be more verbose.
The tests in c/validation/adbc_validation.cc lines [1017, 1047, 1226] erroneously always pass, independent of the table name. This is already the case for the existing Primary Key test. Fixing this could be a different PR. Example:

  struct AdbcGetObjectsTable* table =
      AdbcGetObjectsDataGetTableByName(*get_objects_data, quirks()->catalog().c_str(),
                                   quirks()->db_schema().c_str(), "adbc_pkey_test");
  ASSERT_NE(table, nullptr) << "could not find adbc_pkey_test table";

This feature relies on Composite Primary Keys, which have currently no tests. I will submit those in a different PR.

Working tests
The foreign key tests fail correctly in the following ways.
Number of columns
[ RUN      ] PostgresConnectionTest.MetadataGetObjectsForeignKey
/home/ole/code/arrow-adbc/c/validation/adbc_validation.cc:1229: Failure
Expected equality of these values:
  child_table->n_table_columns
    Which is: 9
  3
[  FAILED  ] PostgresConnectionTest.MetadataGetObjectsForeignKey (103 ms)

[ RUN      ] SqliteConnectionTest.MetadataGetObjectsForeignKey
/home/ole/code/arrow-adbc/c/validation/adbc_validation.cc:1229: Failure
Expected equality of these values:
  child_table->n_table_columns
    Which is: 9
  3
[  FAILED  ] SqliteConnectionTest.MetadataGetObjectsForeignKey (0 ms)

Table names
[ RUN      ] PostgresConnectionTest.MetadataGetObjectsForeignKey
/home/ole/code/arrow-adbc/c/validation/adbc_validation.cc:1237: Failure
Expected: (child_column) != (nullptr), actual: NULL vs (nullptr)
could not find column NOT_EXISTING on adbc_fkey_child_test table
[  FAILED  ] PostgresConnectionTest.MetadataGetObjectsForeignKey (120 ms)

[ RUN      ] SqliteConnectionTest.MetadataGetObjectsForeignKey
/home/ole/code/arrow-adbc/c/validation/adbc_validation.cc:1237: Failure
Expected: (child_column) != (nullptr), actual: NULL vs (nullptr)
could not find column NOT_EXISTING on adbc_fkey_child_test table
[  FAILED  ] SqliteConnectionTest.MetadataGetObjectsForeignKey (0 ms)

Number of table constraints
[ RUN      ] PostgresConnectionTest.MetadataGetObjectsForeignKey
/home/ole/code/arrow-adbc/c/validation/adbc_validation.cc:1244: Failure
Expected equality of these values:
  child_table->n_table_constraints
    Which is: 9
  3
expected 3 constraint on adbc_fkey_child_test table, found: 9
[  FAILED  ] PostgresConnectionTest.MetadataGetObjectsForeignKey (98 ms)

[ RUN      ] SqliteConnectionTest.MetadataGetObjectsForeignKey
/home/ole/code/arrow-adbc/c/validation/adbc_validation.cc:1244: Failure
Expected equality of these values:
  child_table->n_table_constraints
    Which is: 9
  3
expected 3 constraint on adbc_fkey_child_test table, found: 9
[  FAILED  ] SqliteConnectionTest.MetadataGetObjectsForeignKey (0 ms)

Constraint Type
[ RUN      ] PostgresConnectionTest.MetadataGetObjectsForeignKey
/home/ole/code/arrow-adbc/c/validation/adbc_validation.cc:1095: Failure
Expected equality of these values:
  constraint_type
    Which is: "UNKNOWN KEY"
  key_type
    Which is: "FOREIGN KEY"
[  FAILED  ] PostgresConnectionTest.MetadataGetObjectsForeignKey (130 ms)

[ RUN      ] SqliteConnectionTest.MetadataGetObjectsForeignKey
/home/ole/code/arrow-adbc/c/validation/adbc_validation.cc:1095: Failure
Expected equality of these values:
  constraint_type
    Which is: "UNKNOWN KEY"
  key_type
    Which is: "FOREIGN KEY"
[  FAILED  ] SqliteConnectionTest.MetadataGetObjectsForeignKey (0 ms)

Constraint Column Names (less helpful, but aligned with primary key tests)
[ RUN      ] PostgresConnectionTest.MetadataGetObjectsForeignKey
/home/ole/code/arrow-adbc/c/validation/adbc_validation.cc:1106: Failure
Expected equality of these values:
  constraint_column_name
    Which is: "WRONG_NAME"
  columns[column_index]
    Which is: "id_child_col1"
[  FAILED  ] PostgresConnectionTest.MetadataGetObjectsForeignKey (97 ms)

[ RUN      ] SqliteConnectionTest.MetadataGetObjectsForeignKey
/home/ole/code/arrow-adbc/c/validation/adbc_validation.cc:1106: Failure
Expected equality of these values:
  constraint_column_name
    Which is: "WRONG_NAME"
  columns[column_index]
    Which is: "id_child_col1"
[  FAILED  ] SqliteConnectionTest.MetadataGetObjectsForeignKey (0 ms)

Target table name
[ RUN      ] PostgresConnectionTest.MetadataGetObjectsForeignKey
/home/ole/code/arrow-adbc/c/validation/adbc_validation.cc:1139: Failure
Expected equality of these values:
  constraint_column_usage_fk_table
    Which is: "WRONG_NAME"
  fk_table_name
    Which is: "adbc_fkey_parent_2_test"
[  FAILED  ] PostgresConnectionTest.MetadataGetObjectsForeignKey (103 ms)

[ RUN      ] SqliteConnectionTest.MetadataGetObjectsForeignKey
/home/ole/code/arrow-adbc/c/validation/adbc_validation.cc:1139: Failure
Expected equality of these values:
  constraint_column_usage_fk_table
    Which is: "WRONG_NAME"
  fk_table_name
    Which is: "adbc_fkey_parent_2_test"
[  FAILED  ] SqliteConnectionTest.MetadataGetObjectsForeignKey (103 ms)

Target column name
[ RUN      ] PostgresConnectionTest.MetadataGetObjectsForeignKey
/home/ole/code/arrow-adbc/c/validation/adbc_validation.cc:1146: Failure
Expected equality of these values:
  constraint_column_usage_fk_column_name
    Which is: "WRONG_NAME"
  fk_column_name
    Which is: "id_primary_col1"
[  FAILED  ] PostgresConnectionTest.MetadataGetObjectsForeignKey (99 ms)

[ RUN      ] SqliteConnectionTest.MetadataGetObjectsForeignKey
/home/ole/code/arrow-adbc/c/validation/adbc_validation.cc:1146: Failure
Expected equality of these values:
  constraint_column_usage_fk_column_name
    Which is: "WRONG_NAME"
  fk_column_name
    Which is: "id_primary_col1"
[  FAILED  ] SqliteConnectionTest.MetadataGetObjectsForeignKey (0 ms)

Skipped a mandatory constraint test
[ RUN      ] PostgresConnectionTest.MetadataGetObjectsForeignKey
/home/ole/code/arrow-adbc/c/validation/adbc_validation.cc:1293: Failure
Value of: TestedConstraints.adbc_fkey_child_test_pkey
  Actual: false
Expected: true
[  FAILED  ] PostgresConnectionTest.MetadataGetObjectsForeignKey (98 ms)

[ RUN      ] SqliteConnectionTest.MetadataGetObjectsForeignKey
/home/ole/code/arrow-adbc/c/validation/adbc_validation.cc:1293: Failure
Value of: TestedConstraints.adbc_fkey_child_test_pkey
  Actual: false
Expected: true
[  FAILED  ] SqliteConnectionTest.MetadataGetObjectsForeignKey (0 ms)

Number of constraints
[ RUN      ] PostgresConnectionTest.MetadataGetObjectsForeignKey
/home/ole/code/arrow-adbc/c/validation/adbc_validation.cc:1096: Failure
Expected equality of these values:
  constraint->n_column_names
    Which is: 9
  2
expected constraint FOREIGN KEY of adbc_fkey_child_test to be applied to 2 column(s), found: 9
[  FAILED  ] PostgresConnectionTest.MetadataGetObjectsForeignKey (102 ms)

[ RUN      ] SqliteConnectionTest.MetadataGetObjectsForeignKey
/home/ole/code/arrow-adbc/c/validation/adbc_validation.cc:1096: Failure
Expected equality of these values:
  constraint->n_column_names
    Which is: 9
  1
expected constraint PRIMARY KEY of adbc_fkey_child_test to be applied to 1 column(s), found: 9
[  FAILED  ] SqliteConnectionTest.MetadataGetObjectsForeignKey (0 ms)

Foreign Key catalog name
[ RUN      ] PostgresConnectionTest.MetadataGetObjectsForeignKey
/home/ole/code/arrow-adbc/c/validation/adbc_validation.cc:1120: Failure
Value of: constraint_column_usage_fk_catalog
Expected: (is equal to "main") or (is equal to "postgres")
  Actual: "WRONG_NAME"
[  FAILED  ] PostgresConnectionTest.MetadataGetObjectsForeignKey (99 ms)

[ RUN      ] SqliteConnectionTest.MetadataGetObjectsForeignKey
/home/ole/code/arrow-adbc/c/validation/adbc_validation.cc:1120: Failure
Value of: constraint_column_usage_fk_catalog
Expected: (is equal to "main") or (is equal to "postgres")
  Actual: "WRONG_NAME"
[  FAILED  ] SqliteConnectionTest.MetadataGetObjectsForeignKey (0 ms)

Foreign Key DB schema name
[ RUN      ] PostgresConnectionTest.MetadataGetObjectsForeignKey
/home/ole/code/arrow-adbc/c/validation/adbc_validation.cc:1131: Failure
Value of: constraint_column_usage_fk_db_schema
Expected: (is equal to "") or (is equal to "public")
  Actual: "WRONG_NAME"
[  FAILED  ] PostgresConnectionTest.MetadataGetObjectsForeignKey (97 ms)

[ RUN      ] SqliteConnectionTest.MetadataGetObjectsForeignKey
/home/ole/code/arrow-adbc/c/validation/adbc_validation.cc:1131: Failure
Value of: constraint_column_usage_fk_db_schema
Expected: (is equal to "") or (is equal to "public")
  Actual: "WRONG_NAME"
[  FAILED  ] SqliteConnectionTest.MetadataGetObjectsForeignKey (0 ms)

Changed tests from EXPECT_PRED3 to ASSERT_THAT and AnyOf. It's much nicer to read and produces better error messages. To compare:
EXPECT_PRED3:
[](auto child_catalog, auto sqlite_default_catalog, auto postgresql_default_catalog) { return child_catalog == sqlite_default_catalog || child_catalog == postgresql_default_catalog; }(child_catalog, sqlite_default_catalog, postgresql_default_catalog) evaluates to false, where
child_catalog evaluates to "something_else"
sqlite_default_catalog evaluates to "main"
postgresql_default_catalog evaluates to "postgres"

ASSERT_THAT and AnyOf:
Value of: child_catalog
Expected: (is equal to "main") or (is equal to "postgres")
  Actual: "something_else"

Using this means I have to re-do some commits for the pull request, unless the whole thing is squashed. I'll check how they do it.

I'm a bit confused, why are there no failing tests?

I rebased on top of the latest main (which is after branching off for 0.7.0).

The tests in c/validation/adbc_validation.cc lines [1017, 1047, 1226] erroneously always pass, independent of the table name. This is already the case for the existing Primary Key test. Fixing this could be a different PR. Example:

I don't quite follow this.
And, what is the difference between these two tests: "Number of table constraints" and "Number of constraints"?

Good catch. "Number of constraints" should be "Number of Foreign Key columns" instead.
The broken test passes always, even if a wrong table name is probed. It checks for a returned null pointer. But even if the table name is wrong, some pointer is returned.  Maybe I should just take it out. But I feel this should be a different PR.


Update path-params.md

Add path parameter converters (e.g. starlette urlconverters) documentation.

Thanks! I'll check it soon.

Thank you @wshayes ! Checking this I realized that paths in path operations could be improved to not require a Path(...).
I fixed/improved it at: https://github.com/tiangolo/fastapi/pull/272
Then I updated the docs here a bit, to include a source file in the docs instead of having the Python code copied. That allows the source code to be tested. That way we are sure that the code in the docs runs correctly, and helps us ensure that features are properly documented.
Now it's merged. Thanks! :tada: :rocket:


test(scorecard): scorecard test for Cryostat CR configuration changes

Welcome to Cryostat! 👋
Before contributing, make sure you have:

[x] Read the contributing guidelines
[x] Linked a relevant issue which this PR resolves
[x] Linked any other relevant issues, PR's, or documentation, if any
[x] Resolved all conflicts, if any
[x] Rebased your branch PR on top of the latest upstream main branch
[x] Attached at least one of the following labels to the PR: [chore, ci, docs, feat, fix, test]
[x] Signed all commits: git commit -S -m "YOUR_COMMIT_MESSAGE"

Fixes: #503
Description of the change:
Updates the Cryostat CR from default emptyDir storage to PVC storage and checks if the resulting redeployment is as expected.
How to manually test:
regular steps to check scorecard tests

/build_test

https://github.com/cryostatio/cryostat-operator/actions/runs/8073777424

@Mergifyio backport cryostat3


Fixed instance variable declaration and usage in InMemory

storagePrefix was declared String in in_memory_html.dart, as opposed to FutureOr<String> in in_memory_io.dart. This causes an error (described in issue #520).
Breaking changes

No breaking changes

Fixes / Enhancements

Fixed declaration and usage as described.

@Waitak can you rebase this against the latest beta, please. We had an issue with our pipeline and it has now been fixed.

I am still getting this error. The code in in_memory_html.dart seems to be the previous one before this fix? I checked the version for the graphql dependency and it is 3.0.2. Any hint about what could be wrong?
InMemoryCache({
    this.storagePrefix = '',
  }) {
    masterKey = storagePrefix ?? '' + '_graphql_cache';
  }

  final String storagePrefix;

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