Moving circular button position without touch

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
@Pfuster12  Hi Pablo, thanks for new version, this is very useful and beautiful livecharts!
Describe the solution you'd like
A clear and concise description of what you want to happen.
If the .drawTouchOverlayAlways() option is turned on, is it possible that its position is not always in the middle of the chart?
I mean, is it possible to set the position of the circular button programmatically?
Version
1.3.5
Additional context
Add any other context or screenshots about the feature request here.

@Pfuster12

@Pfuster12 Thanks man!

I've added 3 methods I think made sense for different applications:

Set the overlay initial position when the drawing operation occurs (Initial position set in the underlying view draw).
Set the overlay DataPoint position realtime after the draw op (Programmatically set the overlay from user action).
Set the overlay real pixel position realtime after the draw op (Useful for animation operations with float values).

   /**
     * Manually set the INITIAL nearest DataPoint position of the touch overlay.
     */
    @PublicApi
    fun setInitialTouchOverlayPosition(point: DataPoint): LiveChart

    /**
     * Manually set the overlay nearest DataPoint position at any time after the drawing operation.
     * IMPORTANT this must be called AFTER drawDataset() as the pathCoordinates
     * need to be extracted.
     */
    @PublicApi
    fun setTouchOverlayPosition(point: DataPoint)

    /**
     * Manually set the overlay REAL pixel position at any time after the drawing operation.
     * This is useful for animating the touch overlay.
     * IMPORTANT this must be called AFTER drawDataset() as the pathCoordinates
     * need to be extracted.
     * The given position will be mapped onto the path created if it exists.
     */
    @PublicApi
    fun setTouchOverlayRealPosition(position: Float)

It's been lightly tested so there might be bug fix releases.


user.Current may return err in some cases on Windows

panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x40 pc=0xae5ea2]

goroutine 1 [running]:
github.com/ferama/rospo/cmd.init.1()
        github.com/ferama/rospo/cmd/grabpubkey.go:15 +0x62

https://github.com/golang/go/issues/37348
I think it is better to use os.Getwd or empty string as a fallback.

You are right, thanks for reporting. The grabpubkey subcommand is not the only one that suffer from this issue. rospo needs better error handling for this scenario

Haha, in fact, this error will be triggered in sshc subcommand first. I fixed that and found not only sshc, so I opened this issue.
As a temporary fix, I adopted the following fallback strategy for home dir:

user.Current()
os.LookupEnv("HOME")
os.Getwd()
default to "/"

And the following fallback strategy for username:

user.Current()
os.LookupEnv("USER")
default to "root"

This may serve as a reference for you.
By the way, it is interesting that I encountered a similar situation on linux, but I didn't get into why. (That's why I use root as username fallback).

Would you like to contribute with a pull request? We could define a couple of functions into the utils/helpers.go file that return homeDir and username. Then we should use those function everywhere the user.Current() is actually used

I was too busy the past few days.
A pull request is created just now. Review please.


use checkout -b rather than create branch and then try to switch to it

Codecov Report

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

@@            Coverage Diff            @@
##             master     #693   +/-   ##
=========================================
  Coverage     99.49%   99.49%
  Complexity     1859     1859
=========================================
  Files           285      285
  Lines          5545     5545
  Branches        278      278
=========================================
  Hits           5517     5517
  Misses            5        5
  Partials         23       23

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 a95fe5f...57d70aa. Read the comment docs.


Windows permissions changing on certificates managed by win-acme

I keep running into odd issues where a certificate will suddenly be non-exportable, at least the process running with local-administrator privileges no longer has access to export the key.
The certificates are initially generated correctly, but at some later date the permissions change for an unknown reason.
I think it is tied to the win-acme renew task but I'm not 100% sure as of yet.
When win-acme is used to initially create a certificate it has the following permissions:
PS C:\Users\robert.biddle\Documents> $LetsEncryptCert.PrivateKey.CspKeyContainerInfo.CryptoKeySecurity.access | FL *

CryptoKeyRights   : Synchronize, GenericAll, GenericRead
AccessControlType : Allow
IdentityReference : NT AUTHORITY\SYSTEM
IsInherited       : False
InheritanceFlags  : None
PropagationFlags  : None

CryptoKeyRights   : Synchronize, GenericRead
AccessControlType : Allow
IdentityReference : NT AUTHORITY\NETWORK SERVICE
IsInherited       : False
InheritanceFlags  : None
PropagationFlags  : None

CryptoKeyRights   : Synchronize, GenericRead
AccessControlType : Allow
IdentityReference : BUILTIN\Administrators
IsInherited       : False
InheritanceFlags  : None
PropagationFlags  : None

When I run across a broken system the permissions look like this:
PS C:\Users\robert.biddle\Documents> $LetsEncryptCert.PrivateKey.CspKeyContainerInfo.CryptoKeySecurity.Access | FL *

CryptoKeyRights   : Synchronize, GenericAll, GenericRead
AccessControlType : Allow
IdentityReference : NT AUTHORITY\SYSTEM
IsInherited       : False
InheritanceFlags  : None
PropagationFlags  : None

CryptoKeyRights   : Synchronize, GenericAll, GenericRead
AccessControlType : Allow
IdentityReference : NT SERVICE\Schedule
IsInherited       : False
InheritanceFlags  : None
PropagationFlags  : None

Is it likely that the win-acme renew task is changing permissions on renewal? Any help is greatly appreciated.

Hi Rob, you can change a setting in settings.json to make private keys exportable by default.
I suspect the difference in ACL may be explained by differences in who runs the executable though. Presumable you set up the certificate as some kind of administrator, while the scheduled task runs as SYSTEM. win-acme doesn't have any opinion on the ACL by default so it just inherits what's already there, or whatever Microsoft feels appropriate by default.
Using the --acl-fullcontrol switch you can specify users or groups that you want win-acme to grant full control permissions though, perhaps that helps in your scenario. Otherwise maybe run the scheduled task under a different user account and see if that does the trick.

The initial certificate creation was run via a non-interactive process, running as a local admin.
I have the exportable setting in settings.json which looks to work.  The key is marked as exportable, just not actually exportable because of the permissions change that removed BUILTIN\Administrators at some point, I'm guessing during renewal.
I'll add the --acl-fullcontrol and see if that prevents the permissions from getting changed.

settings.json is already set to make private keys exportable.
"PrivateKeyExportable":true,
It was set like that when the cert was originally requested. The whole process (Installation & initial cert creation) is automated identically on several hundred systems, which is why it's odd that everyone once in a while this issue pops up.
The scheduled task is the one that was created automatically by win-acme during the initial cert request.
I guess if --acl-fullcontrol doesn't fix the issue then I could try modifying the existing tasks to change the user after the fact.

Hope you managed to solve the problem, please post again if you have any updates.


Release Google.Cloud.Filestore.V1 version 1.1.0

Changes in this release:

Commit ac367e2: feat: Regenerate all APIs to support self-signed JWTs
Commit a64eeae: docs(filestore): Fix various formatting issues and broken links

Created release for Google.Cloud.Filestore.V1-1.1.0


script data mismatch

Curation script for GSE25136 does not match clinical data for GSE25136. Most likely an error in the naming process.

For the Taylor et al. data (GSE21032) - using GEOquery contains mismatched information in the phenotype data. I have posted looking for help
https://community.rstudio.com/t/geoquery-getgeo-gse21032-pheno-data-mix-up/67546

Taylor et al. is now curated from the two differing GEO instances (one for GEX, one for CNA), complemented by the mutation calls pulled for cBioPortal along with additional clinical information. sampleMap-function for the mae_taylor will contain mappings of IDs between differing platforms.


[Windows] Fix precision loss in CFGetSystemUptime()

Integer division rounds result to whole seconds. This forces floating point division.
Noticed the issue because XCTest.measure depends on this value (through ProcessInfo.systemUptime) and produces unreliable results.

@swift-ci please test


WIP: Consolidate error handling, add more tests

Consolidated error handling into one method
Added a couple more tests

@Glavin001 how can we force unibeautify.beautify to fail?  As well as https://github.com/Unibeautify/unibeautify-cli/blob/0ada99c907bbca491a9fad2d1807a10b8893379a/src/commands/BeautifyCommand.ts#L96-L101

Also want to try and fix #32.

@szeck87 fixing https://github.com/Unibeautify/unibeautify-cli/issues/32 as part of this PR? Sounds good. Let me know when you need a new review 😃 .

Suddenly I can't recreate #32 using this branch.


Adds the Open Mobile Map (Map Package) sample

@philium I made the requested changes.

@saratchandrakarumuri Thanks!


Add preview for EPS files

Description of the new feature / enhancement
Add preview for EPS files.
Scenario when this would be used?
In Windows Explorer there should be a thumbnail.
There should also be a way to preview the EPS in its own window.
Supporting information
No response

Duplicate of #7737

Thanks for reporting this. This issue is a duplicate. We'll track this in #7737 and other related issues to centralize the discussion thread.

@franky920920, that was closed out due to being what we'd call an "epic". This is a single feature so great request

This would be awesome 😍
Other useful formats are *.ai, *.ps, *.idms
As far as I know, *.ai file thumbnails are already natively supported in Windows 10. However, it would also be very cool for Windows 8.

@XmlmXmlmX
When .ai files are saved with "Create PDF Compatible File" option, then you can preview them with any pdf file viewer. Including the pdf previewer included in powertoys atm. See my comments in issue #5869 how to enable that.

I support this. I have been using sagethumbs, but it doesn't work for half of the eps files, so an alternative solution is desired.


Issue #174: Order Organizations By Title

@allella I tested this locally, seems to be working now.

This is live. Thanks @bogdankharchenko


fix readme before/after

I think the before / after in readme were switched

Thanks for the contribution, but I think the current statement is expected.


Wrong height calculation

Hi
I found very strange issue. Looks like it's related to the snackbar height calculation.
https://codesandbox.io/s/v-snackbars-demo-forked-imf5y?file=/src/App.vue
How to replicate:
Add one more tag to v-combobox
Result
It displays three shackbars one above the other

I will venture to suggest that somewhere in the code this.$nextTick() is missing

The problem was due to the addition of several items at once. I've just published a fix for that.


Support TypeScript installed via JSPM

Currently TypeScript is assumed to have been installed via npm.  However, TypeScript can also be installed via jspm.
It would be great if the user could provide the location of TypeScript so that projects that use jspm for package management do not have to install TypeScript twice.

@KarlPurk I don't know too much about JSPM; can you outline the differences? Does a require("typescript") statement not resolve in the same way as it does with NPM? What do you anticipate are the required changes on our side?

I use jspm too. But why do you need this feature? tslint is a tooling on the server side, while jspm is about loading on the browser.
Do you need to use tslint on the browser to do some online linting? That would be a valid use case.
If not, it should use local/global typescript via npm.
My two cents.

Ah, yes, thanks for the info @unional. Perhaps @KarlPurk you are looking for #1001, in-browser linting?

Closing this as its old and seems irrelevant.


YARN-10576. Update Capacity Scheduler documentation about JSON-based …

…placement mapping

Description of PR
How was this patch tested?
For code changes:

[ ] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
1m  5s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
markdownlint
0m  0s

markdownlint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
33m 40s

trunk passed

+1 :green_heart:
mvnsite
0m 28s

trunk passed

+1 :green_heart:
shadedclient
50m  6s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 12s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 17s

the patch passed

+1 :green_heart:
shadedclient
16m  4s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
asflicense
0m 30s

The patch does not generate ASF License warnings.

69m 44s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3333/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3333

Optional Tests
dupname asflicense mvnsite codespell markdownlint

uname
Linux 3a4550ff8c7c 4.15.0-143-generic #147-Ubuntu SMP Wed Apr 14 16:10:11 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 75e929ec824a747ca807667f86d7935ccb7797ed

Max. process+thread count
518 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3333/1/console

versions
git=2.25.1 maven=3.6.3

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 56s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
markdownlint
0m  0s

markdownlint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
33m 48s

trunk passed

+1 :green_heart:
mvnsite
0m 24s

trunk passed

+1 :green_heart:
shadedclient
49m 56s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 12s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 17s

the patch passed

+1 :green_heart:
shadedclient
16m  4s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
asflicense
0m 29s

The patch does not generate ASF License warnings.

69m 17s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3333/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3333

Optional Tests
dupname asflicense mvnsite codespell markdownlint

uname
Linux 2cd0c72dac8f 4.15.0-143-generic #147-Ubuntu SMP Wed Apr 14 16:10:11 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 3e5c041f53870e291f85d56953c0724c824c11c4

Max. process+thread count
520 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3333/2/console

versions
git=2.25.1 maven=3.6.3

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.


Bring back intellij-idea 2016.1

After removal in https://github.com/caskroom/homebrew-versions/pull/3683
If there’s a checkbox you can’t complete for any reason, that's okay, just explain in detail why you weren’t able to do so.
After making all changes to the cask:

[ ] brew cask audit --download {{cask_file}} is error-free.
[ ] brew cask style --fix {{cask_file}} reports no offenses.
[ ] The commit message includes the cask’s name and version.

Additionally, if adding a new cask:

[ ] Named the cask according to the token reference.
[ ] brew cask install {{cask_file}} worked successfully.
[ ] brew cask uninstall {{cask_file}} worked successfully.
[ ] Checked there are no open pull requests for the same cask.
[ ] Checked the cask was not already refused in closed issues.
[ ] Checked the cask is submitted to the correct repo.

@condemil / @vitorgalvao: Is it really necessary to bring the cask back?
The only reason to bring the cask back would be for people that do not have a license for newer version. But by that logic, we would need to bring back other old jetbrains cask as well.
There are now 24 stable intellij versions alone and 112 stable versions of jetbrains products.
Maybe it would make more sense to move them to a new tap ? I Could also update the @jcbot to maintain such a tap.

There was no any other non-eap JetBrains products removed by https://github.com/caskroom/homebrew-versions/pull/3683
Fallback license holders can use latest z version from x.y.z version format, as I can see here there are only 12 x.y intellij versions released since 2012.

This is well covered in the acceptable casks policy. For at least one year, this cask has a place here.

But by that logic, we would need to bring back other old jetbrains cask as well.

No one asked for those, so no. This one was specifically asked for. This is the correct repo.
Also, this is way different from the EAP casks, as this one will presumably never be updated, hence not making it a maintainability burden.


enable DD_TAGS env var

Note: Please remember to review the contribution guidelines
if you have not yet done so.
What does this PR do?
see: https://github.com/DataDog/datadog-lambda-go/issues/35
This change enables the env var DD_TAGS
Example:
Running with DD_TAGS=key1:val1 will add the tag key1:val1 to all metrics send from this SDK.
Motivation
This env var exists on other Datadog SDK's, and enables a simple way to apply context to all metrics.
Testing Guidelines
Added 2 new tests:

ensure the metrics package applied GlobalTags to emitted metrics
ensure DD_TAGS env var is parsed into Config (required some package test uplift)

Additional Notes
Anything else we should know when reviewing?

Codecov Report

Merging #36 into master will increase coverage by 0.14%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #36      +/-   ##
==========================================
+ Coverage   88.46%   88.61%   +0.14%
==========================================
  Files          12       12
  Lines         555      562       +7
==========================================
+ Hits          491      498       +7
  Misses         48       48
  Partials       16       16

Impacted Files
Coverage Δ

ddlambda.go
75.67% <100.00%> (+1.76%)
:arrow_up:

internal/metrics/listener.go
95.94% <100.00%> (+0.11%)
:arrow_up:

Continue to review full report at Codecov.

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

@pwmcintyre Thanks for your PR! And our apologies on the late response!
Your changes looked good to us, but we have been evaluating and debating for this change internally, and eventually decided to reject this PR (at least for now) because we want to stick to our current approach of tagging data from Lambda functions (adding a tag to the Lambda function) rather than adopting a different approach to do the same thing, which will certainly cause confusions and inconsistent behaviors.
Another thought here is that we may regret and re-adopt this PR in the future, and it would be much better than adopting this PR now and regret in the future (we can't rollback the support once it's out).
That said, today you can apply tags to your custom metrics by tagging your Lambda functions in AWS and enable DdFetchLambdaTags in the Forwarder Lambda function, custom metrics forwarded by the forwarder will automatically receive tags from your Lambda functions. If you are submitting metrics synchronously (without the forwarder), we are actively working on project to apply the tags from the Lambda function to the receive metrics in Datadog backend.
Additional background on DD_TAGS, the Datadog agent needs to support this env var because there is no native way to tag a host (like how we can tag a Lambda function).
In short, at least for now, the recommended approach is tagging the Lambda function rather than using DD_TAGS. As I mentioned, the support isn't yet 100%, but we are working on it.
Does this make sense? Is there a use case I'm not seeing? We can also continue this conversation in our public slack channel too.

hi @tianchu, thanks for the update

the recommended approach is tagging the Lambda function rather than using DD_TAGS

can you please provide examples or documentation for this approach? because I don't see how that works.
ie. if the lambda is posting metrics directly to Datadog, how are the Lambda's tags applied? what mechanism is collecting / applying to the submitted metrics?
maybe @srizzling knows more?

@pwmcintyre
So you are submitting metrics directly to datadog instead of writing to logs and relayed by a forwarder lambda? If so, as I mentioned, the support on the way.

If you are submitting metrics synchronously (without the forwarder), we are actively working on project to apply the tags from the Lambda function to the receive metrics in Datadog backend.

We know the tags on your Lambda functions and which Lambda function submits these metrics, so we can automatically tag the metrics with tags from the corresponding Lambda functions.

Hi @pwmcintyre, as explained by Tian above, we're going to close this in favor of supporting tagging the Lambda functions directly. Please reach out if you would like more details or help setting up the strategy to apply Lambda tags to metrics that Tian outlined in his comment.


Fix Maven URL in the Slack notification message

What does this PR do?
Without the fix Maven URL is wrong - it is missing tag.

Codecov Report

Merging #44 (d321f42) into main (83dd535) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main      #44   +/-   ##
=========================================
  Coverage     87.62%   87.62%
  Complexity      117      117
=========================================
  Files            18       18
  Lines           323      323
  Branches         41       41
=========================================
  Hits            283      283
  Misses           25       25
  Partials         15       15


Failed authentication

I ran the project and I can't get the authentication to work (as specified in the readme file).
after being redirected to the keycloak authentication page, entering the user & password "test" & getting redirected to the /auth page I get a failure when sending the post request to the keycloak server.
The post request that keeps failing has the following information:
method: POST

url: http://localhost:8080/auth/realms/Clients/protocol/openid-connect/token
payload: grant_type=authorization_code&code=90fedb75-6615-493f-bd3b-66f6d4f2f4c7.9888d6a9-290c-40ca-8a6f-1aa71a535d6e.1adb3baa-fb7e-4f0c-ab6c-437bd3d71757&redirect_uri=http://localhost:8000/&client_id=app
headers: {'Content-Type': 'application/x-www-form-urlencoded'}

The post function returns the error:
requests.exceptions.ConnectionError: HTTPConnectionPool(host='localhost', port=8080): Max retries exceeded with url: /auth/realms/Clients/protocol/openid-connect/token (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7f2f78e79890>: Failed to establish a new connection: [Errno 111] Connection refused'))

You could try iterating through these network modes and see what sticks... that might work?
Also, I tried reproducing your error and this is what I got:
POST

url: http://localhost:8080/auth/realms/Clients/protocol/openid-connect/token
payload: grant_type=authorization_code&code=202f9eb8-ba63-4fca-80a5-7ff8b8519232.7709902b-1093-48c1-bdb4-09ea9d6f43f8.8c53b20c-9ea0-4871-a1ca-12aadcf77818&redirect_uri=http://localhost:8000/&client_id=app
headers: {'Content-Type': 'application/x-www-form-urlencoded'}

This is what the logs look like with a little modification:
app_1       | INFO: ('<IP_ADDRESS>', 47670) - "GET /login HTTP/1.1" 307
auth_1      | 02:33:01,162 WARN  [org.keycloak.services] (default task-1) KC-SERVICES0091: Request is missing scope 'openid' so it's not treated as OIDC, but just pure OAuth2 request.
app_1       | INFO: payload: grant_type=authorization_code&code=740e324a-d7d5-4946-b70b-5cd9de3d6c35.d958b24d-f259-4e3b-8d66-64d64df5ecf0.8c53b20c-9ea0-4871-a1ca-12aadcf77818&redirect_uri=http://localhost:8000/&client_id=app
app_1       | INFO: token_response.headers: {'Connection': 'keep-alive', 'Cache-Control': 'no-store', 'Pragma': 'no-cache', 'Content-Type': 'application/json', 'Content-Length': '2302', 'Date': 'Fri, 20 Dec 2019 02:33:03 GMT'}
app_1       | INFO: token_response.content: b'{"access_token":"eyJhbGciOiJSUzI1NiIsInR5cCIgOiAiSldUIiwia2lkIiA6ICJDY0xKVXhCTWxkeDFrazEzN0hscUVvSzZVaF9SOFd0Zno5LUtCdDJENlNvIn0.eyJqdGkiOiI2Y2EwMDY4ZS1hYjkzLTRjMmQtYTVkOC04N2U5NzAzZmYzZWUiLCJleHAiOjE1NzY4MDk0ODMsIm5iZiI6MCwiaWF0IjoxNTc2ODA5MTgzLCJpc3MiOiJodHRwOi8vbG9jYWxob3N0OjgwODAvYXV0aC9yZWFsbXMvQ2xpZW50cyIsImF1ZCI6ImFjY291bnQiLCJzdWIiOiJiNmYwMWMzYi1iZDE4LTQ4MDMtOWE5OS02MjQxZTc4MDI0ZTIiLCJ0eXAiOiJCZWFyZXIiLCJhenAiOiJhcHAiLCJhdXRoX3RpbWUiOjE1NzY4MDkxODMsInNlc3Npb25fc3RhdGUiOiJkOTU4YjI0ZC1mMjU5LTRlM2ItOGQ2Ni02NGQ2NGRmNWVjZjAiLCJhY3IiOiIxIiwiYWxsb3dlZC1vcmlnaW5zIjpbImh0dHA6Ly9sb2NhbGhvc3Q6ODAwMC9sb2dpbiJdLCJyZWFsbV9hY2Nlc3MiOnsicm9sZXMiOlsib2ZmbGluZV9hY2Nlc3MiLCJ1bWFfYXV0aG9yaXphdGlvbiJdfSwicmVzb3VyY2VfYWNjZXNzIjp7ImFjY291bnQiOnsicm9sZXMiOlsibWFuYWdlLWFjY291bnQiLCJtYW5hZ2UtYWNjb3VudC1saW5rcyIsInZpZXctcHJvZmlsZSJdfX0sInNjb3BlIjoiZW1haWwgcHJvZmlsZSIsImVtYWlsX3ZlcmlmaWVkIjpmYWxzZSwicHJlZmVycmVkX3VzZXJuYW1lIjoidGVzdCJ9.Xh7jvG5xFe02IVB197CyFLIkOXU6LFeaCDX_9CXeBmEjnV5i7AyMKdFxhLHajXMexP0nYy86FGzo-4IwM_TEfk6HRs8_kWwBKO2Fr-vJUwW6G1LdIiGnDkXYGgDCOm2QQZqEVTVA-KoeRxr4MrHPt89vI2VLfim-Q_tIZV22HQ_t-N2A1pKwAriU9hV-EG9Xp3fB6Vn41nQm9jKz-0IEUP4hQe5Z42U9oP1Xld_yDxeEy3--pf5xcXKAO1Dq_xZeTE9il04tNLmofvTzc3weSd3G8RQrLnf6xLLq4_TQlfBi_bhfRrLkjrqkD5ooUBM03LxOcedDWU4WzGErgtalDQ","expires_in":300,"refresh_expires_in":1800,"refresh_token":"eyJhbGciOiJIUzI1NiIsInR5cCIgOiAiSldUIiwia2lkIiA6ICJmMGI4MjBlNi0yNzc1LTQzZjktYmQwMi1kZGE0ZWY5YWUxMmQifQ.eyJqdGkiOiIzZjdkNmQwMC04YTk1LTQwMzEtYThjYy00NDNkNTA3NDg3ZTgiLCJleHAiOjE1NzY4MTA5ODMsIm5iZiI6MCwiaWF0IjoxNTc2ODA5MTgzLCJpc3MiOiJodHRwOi8vbG9jYWxob3N0OjgwODAvYXV0aC9yZWFsbXMvQ2xpZW50cyIsImF1ZCI6Imh0dHA6Ly9sb2NhbGhvc3Q6ODA4MC9hdXRoL3JlYWxtcy9DbGllbnRzIiwic3ViIjoiYjZmMDFjM2ItYmQxOC00ODAzLTlhOTktNjI0MWU3ODAyNGUyIiwidHlwIjoiUmVmcmVzaCIsImF6cCI6ImFwcCIsImF1dGhfdGltZSI6MCwic2Vzc2lvbl9zdGF0ZSI6ImQ5NThiMjRkLWYyNTktNGUzYi04ZDY2LTY0ZDY0ZGY1ZWNmMCIsInJlYWxtX2FjY2VzcyI6eyJyb2xlcyI6WyJvZmZsaW5lX2FjY2VzcyIsInVtYV9hdXRob3JpemF0aW9uIl19LCJyZXNvdXJjZV9hY2Nlc3MiOnsiYWNjb3VudCI6eyJyb2xlcyI6WyJtYW5hZ2UtYWNjb3VudCIsIm1hbmFnZS1hY2NvdW50LWxpbmtzIiwidmlldy1wcm9maWxlIl19fSwic2NvcGUiOiJlbWFpbCBwcm9maWxlIn0.J6Ojlq-cTZR1s4STvLvY7ansCir4ZqcvwVPMMbBOvwg","token_type":"bearer","not-before-policy":1565839535,"session_state":"d958b24d-f259-4e3b-8d66-64d64df5ecf0","scope":"email profile"}'
app_1       | INFO: ('<IP_ADDRESS>', 47670) - "GET /auth?session_state=d958b24d-f259-4e3b-8d66-64d64df5ecf0&code=740e324a-d7d5-4946-b70b-5cd9de3d6c35.d958b24d-f259-4e3b-8d66-64d64df5ecf0.8c53b20c-9ea0-4871-a1ca-12aadcf77818 HTTP/1.1" 307
app_1       | DEBUG: {'jti': '6ca0068e-ab93-4c2d-a5d8-87e9703ff3ee', 'exp':<PHONE_NUMBER>, 'nbf': 0, 'iat':<PHONE_NUMBER>, 'iss': 'http://localhost:8080/auth/realms/Clients', 'aud': 'account', 'sub': 'b6f01c3b-bd18-4803-9a99-6241e78024e2', 'typ': 'Bearer', 'azp': 'app', 'auth_time':<PHONE_NUMBER>, 'session_state': 'd958b24d-f259-4e3b-8d66-64d64df5ecf0', 'acr': '1', 'allowed-origins': ['http://localhost:8000/login'], 'realm_access': {'roles': ['offline_access', 'uma_authorization']}, 'resource_access': {'account': {'roles': ['manage-account', 'manage-account-links', 'view-profile']}}, 'scope': 'email profile', 'email_verified': False, 'preferred_username': 'test'}
app_1       | INFO: ('<IP_ADDRESS>', 47670) - "GET / HTTP/1.1" 200

Here is said modification:
@app.get("/auth")
async def auth(code: str) -> RedirectResponse:
    payload = (
        f"grant_type=authorization_code&code={code}"
        f"&redirect_uri={APP_BASE_URL}&client_id=app"
    )
    logger.info(f"payload: {payload}")
    headers = {"Content-Type": "application/x-www-form-urlencoded"}
    token_response = requests.request(
        "POST", TOKEN_URL, data=payload, headers=headers
    )
    logger.info(f"token_response.headers: {token_response.headers}")
    logger.info(f"token_response.content: {token_response.content}")
...

Here are a few screenshots that indicate how I set up the client, and user in Keycloak. Note that they exist in the Clients realm

Let me know if that helps!

Solved it. the problem was that the fastapi container tried to access keycloak via at the address localhost:8080, but that was not accessible via the fastapi container. solved it by changing the code to accessing http://auth:8080 and adding a link between the two services in the doker-compose file.
I also had an issue because the fastapi container was only listening to request from itself, so I set it to listen to <IP_ADDRESS>.
All of these issues probably are caused by wrong docker networking settings, but im not very proficient in docker networking...
Thanks for the help!

B.T.W have you tried using keycloak in fastapi using dependency injection? I found this very nice tutorial on the FastAPI using dependency injection & self generated jwt tokens...
Im gonna try integrating keycloak via a dependency injection, could majorly simplify the code...

B.T.W have you tried using keycloak in fastapi using dependency injection?

yup, in a separate, proprietary project. For even better solution, check out keycloak-gatekeeper. You can put this in front of the pod/container you're trying to protect.
(As for this project, I just made it to try something out).

Solved it.

closing

@hkiang01
Would you mind sharing your compose file / gatekeeper config?
I could protect the route /docs with gatekeeper but when hitting api endpoints, i have a Bad Gateway error.
@sagivoulu did you manage doing something with dependy injection? I had a look at keycloak-client middleware for starlette but haven't been able to make it run.
Thanks a lot

@HedgeShot the docker compose file is in the repo. No config was used outside of what is already provided by the jboss/keycloak image.

@HedgeShot @sagivoulu I would encourage you both to look at this blog post describing how to use Keycloak in a more systematic and independent fashion


Class name convention python

I noticed that ocr_table.py and ocr_product.py has a wrong class naming convention, It must be in CapWords but are in snake case.

Ref: https://www.datacamp.com/community/tutorials/pep8-tutorial-python-code?utm_source=adwords_ppc&utm_campaignid=1455363063&utm_adgroupid=65083631748&utm_device=c&utm_keyword=&utm_matchtype=b&utm_network=g&utm_adpostion=&utm_creative=332602034364&utm_targetid=dsa-429603003980&utm_loc_interest_ms=&utm_loc_physical_ms=1001720&gclid=EAIaIQobChMInI21_PWz6wIVko3ICh3lgQZ2EAAYASAAEgKA2fD_BwE

Nice, I will change it and I will use this Issue to make a pylint refactoring. Okay?


⚠ SceneTime has degraded performance

In 8653c87, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1009 ms

Resolved: SceneTime performance has improved in 26143c9.


All add element button not visible on low resolution screen

Linked to #36

Fixed by removing all  spacers and replacing with margin style for adding gap with FABs.


User config

Support --config and --mkconfig options.
Other options can be given alongwith --config and the options on the command line will override the ones in the config file.
When --config and --mkconfig are given together alongwith other options, the net of all options (those on command-line and from the config file) are written into the new config file given by --mkconfig.
This should partially solve #69

Thanks, I will check it on my free time. Probably in the weekend.

I didn't actually try your patch, but I commented on some lines in your commits. Unfortunately GitHub doesn't show the lines here in the issue. you have to click on the changesets above.
Anyway, you can use interactive rebase (git rebase -i) and git commit --amend create_ap to edit the commits.
Let me know when they are updated.
Thanks!

I've took care of all the points you had raised.
As for using WIFI_IFACE, INTERNET_IFACE in the config file, I couldn't think of a way to do this without executing a lot of option parsing code. For now, i'm storing the positional arguments as ARG1=val.

I initially assumed that we won't give the user the flexibility to modify the config file.
I've updated the code now to ensure that the pasing is more lenient. I have just one problem to solve and then I'll push the code. I'm trying to set $1, $2 etc. when loading configs. Currently this code is not inside a function and everything works perfectly. I want to move this code inside a function to make things look neat.
But, when executed from inside the function, the code sets the arguments of the function rather than command line arguments. I'll see if I can solve this, or I'll push the code as it is.

I couldn't figure out a way to set $1, $2 etc, from within a function. So, I was not able to move that part of the code to a function. Other than that, all other changes have been implemented.
Review and see if the changes are fine.

I've replaced touch by echo to handle the problem you had pointed out.
PS: When I update the commits and push them, the comments related to the old commits are lost from the Pull Request. Do you know of anyway to retain them?

Unfortunately no. I don't mind if they get lost, but it looks like you're talking by yourself :P
Anyway, thanks for your commits. I will test it in an hour and I will let you know.

Just tested it and I found one bug:
create_ap --mkconfig foo wlan0 eth0 test 12345678
create_ap --config foo -n wlan0 test

The second command fails, create_ap shows usage.
I will fix it by myself because I want also to see what can I do about ARG* config options.
For now I will push your user_config branch in my repo and I will do my changes there. When I merge it to master this will close automatically.
If for any reason you want to do any changes just comment here and let me know.
Thanks for your contribution! :)

This is a case I didn't foresee. Great that you would fix it by yourself.
If you need any other help, let me know.
On May 9, 2015 9:49 PM, "oblique"<EMAIL_ADDRESS>wrote:

Just tested it and I found one bug:
create_ap --mkconfig foo wlan0 eth0 test 12345678
create_ap --config foo -n wlan0 test
The second command fails, create_ap shows usage.
I will fix it by myself because I want also to see what can I do about
ARG* config options.
For now I will push your user_config branch in my repo and I will do my
changes there. When I merge it to master this will close automatically.
If for any reason you want to do any changes just comment here and let me
know.
Thanks for your contribution! :)
—
Reply to this email directly or view it on GitHub
https://github.com/oblique/create_ap/pull/77#issuecomment-100511701.

Hi, check my user_config commits.
The changes are:

I removed ARG* options, now you can use WIFI_IFACE INTERNET_IFACE SSID PASSPHRASE
I fixed an infinite loop if DAEMONIZE=1 was set in the config file
I fixed the bug that I mention above by overriding all positional arguments if at least one was passed from the command line.

If you have any comments or questions let me know. I will merge the branch if you don't reply within 3 days.
Thanks again.

Sorry for keep you waiting, I was very busy lately. Merged!


Enhancement Visualization: #4

Added bar graph to display the top 5 authors with the highest average ratings on their books.

PR submitted after acceptance was closed, sorry!

We had clearly mentioned that any PR submitted after 10pm shall not be accepted due to the freeze, sorry!

Once your PR closed, you cannot resubmit it!


Wasm support

I've got Wasm support mostly working at this repo.
I also updated to clapv4.
Since you can't spawn multiple processes in Wasm it's set up to use an async main and poll the resulting future concurrently with the gui as explained here.
The file functionality has to come from within the main function the user of the library provides since you have to read a full file to memory instead of being able to use paths directly. For example using rfd's AsyncFileDialog
Also since stdout, stderr, stdin, etc... can't be redirected without nightly I used a custom logger that writes any logs to the gui instead of stdout.
A demo which uses these changes is visible here
I don't know if there's interest in merging some of these features here or not, but I thought it was worth bringing to attention.

This is really nice, what would it take to some of the wasm support put upstream?

@mkatychev An upstream maintainer (I think this is @MichalGniadek) would have to be present and interested in merging it. I would be mostly willing to fix things as needed to make this happen, but I won't file a pull request until I see something indicating it might be merged.
You should be able to use the fork as a git dependency (https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html). I'm not opposed to fixing issues filed on the fork, though keep in mind it's a hobby.
I'm not interested in publishing the fork on crates.io, so if the issue is that you would like to publish a crate with the fork as a dependency, then yes that will rely on upstream merging since I believe crates.io has a policy of only depending on other crates on crates.io.

Thanks for the reply, I think up-streaming this addition would fare better than maintaining a fork. My question was more along the lines of what mutually exclusive compilation flags to do you foresee this would add (to for example handle wasm32 compilation targets)

Thanks for the reply, I think up-streaming this addition would fare better than maintaining a fork. My question was more along the lines of what mutually exclusive compilation flags to do you foresee this would add (to for example handle wasm32 compilation targets)

I don't know what you mean. You compile with cargo build --target wasm32-unknown-unknown. You then serve the resulting wasm in your webpage. (Or use trunk or similar to mostly automate this).

I've gone through your changes, I think I answered my own question.


wip Runnables should be functions

Feedback on the Error messages? What's your opinion on this?
Describe's second argument is not a function
describe('MyClass', new Promise(() => {}))

Before:
➜  temp mocha silly-tests.js

  0 passing (2ms)

After:
➜  temp mocha silly-tests.js
/Users/dasilvacontin/github/mochajs/mocha/lib/interfaces/common.js:119
          throw new TypeError('Suite "' + suite.fullTitle() + '" was defined but no callback was supplied (got `' + utils.canonicalize(opts.fn) + '` instead). Supply a callback or explicitly skip the suite.');
          ^

TypeError: Suite "MyClass" was defined but no callback was supplied (got `[object Promise]` instead). Supply a callback or explicitly skip the suite.
    at Object.create (/Users/dasilvacontin/github/mochajs/mocha/lib/interfaces/common.js:119:17)
    at context.describe.context.context (/Users/dasilvacontin/github/mochajs/mocha/lib/interfaces/bdd.js:38:27)
    at Object.<anonymous> (/Users/dasilvacontin/temp/silly-tests.js:8:1)
    at Module._compile (module.js:573:32)
    at Object.Module._extensions..js (module.js:582:10)
    at Module.load (module.js:490:32)
    at tryModuleLoad (module.js:449:12)
    at Function.Module._load (module.js:441:3)
    at Module.require (module.js:500:17)
    at require (internal/module.js:20:19)
    at /Users/dasilvacontin/github/mochajs/mocha/lib/mocha.js:222:27
    at Array.forEach (native)
    at Mocha.loadFiles (/Users/dasilvacontin/github/mochajs/mocha/lib/mocha.js:219:14)
    at Mocha.run (/Users/dasilvacontin/github/mochajs/mocha/lib/mocha.js:487:10)
    at Object.<anonymous> (/Users/dasilvacontin/github/mochajs/mocha/bin/_mocha:458:18)
    at Module._compile (module.js:573:32)
    at Object.Module._extensions..js (module.js:582:10)
    at Module.load (module.js:490:32)
    at tryModuleLoad (module.js:449:12)
    at Function.Module._load (module.js:441:3)
    at Module.runMain (module.js:607:10)
    at run (bootstrap_node.js:382:7)
    at startup (bootstrap_node.js:137:9)
    at bootstrap_node.js:497:3

Hooks with no callback or with a wrong type
function setup () {}
beforeEach(setup())

Before:
➜  temp mocha silly-tests.js

  0 passing (2ms)

After:
➜  temp mocha silly-tests.js
/Users/dasilvacontin/github/mochajs/mocha/lib/utils.js:815
      throw new TypeError(thing + ' definition expected a `function` as its second argument. Got ' + exports.canonicalize(fn) + ' instead.');
      ^

TypeError: beforeEach hook definition expected a `function` as its second argument. Got undefined instead.
    at Object.handleTitleAndFn (/Users/dasilvacontin/github/mochajs/mocha/lib/utils.js:815:13)
    at Suite.beforeEach (/Users/dasilvacontin/github/mochajs/mocha/lib/suite.js:265:26)
    at beforeEach (/Users/dasilvacontin/github/mochajs/mocha/lib/interfaces/common.js:57:17)
    at Object.<anonymous> (/Users/dasilvacontin/temp/silly-tests.js:1:63)
    at Module._compile (module.js:573:32)
    at Object.Module._extensions..js (module.js:582:10)
    at Module.load (module.js:490:32)
    at tryModuleLoad (module.js:449:12)
    at Function.Module._load (module.js:441:3)
    at Module.require (module.js:500:17)
    at require (internal/module.js:20:19)
    at /Users/dasilvacontin/github/mochajs/mocha/lib/mocha.js:222:27
    at Array.forEach (native)
    at Mocha.loadFiles (/Users/dasilvacontin/github/mochajs/mocha/lib/mocha.js:219:14)
    at Mocha.run (/Users/dasilvacontin/github/mochajs/mocha/lib/mocha.js:487:10)
    at Object.<anonymous> (/Users/dasilvacontin/github/mochajs/mocha/bin/_mocha:458:18)
    at Module._compile (module.js:573:32)
    at Object.Module._extensions..js (module.js:582:10)
    at Module.load (module.js:490:32)
    at tryModuleLoad (module.js:449:12)
    at Function.Module._load (module.js:441:3)
    at Module.runMain (module.js:607:10)
    at run (bootstrap_node.js:382:7)
    at startup (bootstrap_node.js:137:9)
    at bootstrap_node.js:497:3

@dasilvacontin I like this one:
beforeEach hook definition expected a `function` as its second argument. Got undefined instead.

Not sure if you accounted for this, but hooks can also have a string title as the first parameter.
I want to be careful not to throw where we weren't throwing before, unless the current behavior is unexpected.  Doing something is better than nothing.

Yep! Accounted for it. handleTitleAndFn (also accepting suggestions for method name) accepts either (string, function) or (function).

On 1 Nov 2016, at 06:58, Christopher Hiller<EMAIL_ADDRESS>wrote:
@dasilvacontin I like this one:
beforeEach hook definition expected a function as its second argument. Got undefined instead.
Not sure if you accounted for this, but hooks can also have a string title as the first parameter.
I want to be careful not to throw where we weren't throwing before, unless the current behavior is unexpected. Doing something is better than nothing.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


No mention of how to turn of the auto creation of the shadow property for missing foreign keys or alternate keys

[Enter feedback here]

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

ID: 550a3c36-7f1d-efa7-3ed8-769720168da4
Version Independent ID: 627c4301-e00d-826e-d545-3a244305dec9
Content: Shadow Properties - EF Core
Content Source: entity-framework/core/modeling/shadow-properties.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @AndriySvyryd
Microsoft Alias: avickers

Would you prefer that an exception is thrown? Then call
.ConfigureWarnings(b => b.Error(CoreEventId.ShadowPropertyCreated))


Create a starknet.js wrapper

Similar to how there are venv wrapper and docker wrapper, there should be a starknet.js wrapper and we should be moving toward relying on starknet.js instead of Starknet CLI

closed since the CLI is being removed in favor of starknet.js in https://github.com/0xSpaceShard/starknet-hardhat-plugin/pull/400


[bug] soda-scanner doesn't mark empty strings as missing

Describe the bug
It is expected that Soda SQL detects null and '<empty string>' as missing values out of the box + additionally declared missingValues. For the data set up like this:
    tableBuilder
      .createColumn(COL_MISSING2, "VARCHAR(255)")
      .scanConfiguration("missingValues", Lists.of("missing"))
      .values(
        "''",
        "'missing'"
      );

the missing count should be 2 (1 for "", and 1 for "missing"), but we get 1 since only missing is detected.
Generated SQL:
COUNT(CASE WHEN NOT (missing2 IS NULL OR missing2 IN ('missing')) THEN 1 END),

Warehouse Type: PostgreSQL

Deprecated


Migrate old volumes for the provisioner/driver update

longhorn/longhorn#878

It's better we can run this as a separate tool rather than automatically. We want to user to know that something is changing their PV/PVC definitions. Also, it's unnecessary to add more complexity in the volume controller for this.

How about creating a new API for the migration?

How about creating a new API for the migration?

Why not create a separate tool? It can be a command from the manager. Why API is better?


Ansible Role Error

I'm an old Puppet user looking to begin using Ansible, but thus far it is kicking my butt with the following error...   I'm trying to provision an Ubuntu VM but no matter what I do to the offending line of code, I still end up with similar errors on other lines.
==> drupalvm: Running provisioner: ansible...
ERROR! the role 'geerlingguy.repo-remi' was not found in /Users/markmcfate/Dev/Vagrant/drupal-vm/provisioning/roles:/Users/markmcfate/Dev/Vagrant/drupal-vm/provisioning:/etc/ansible/roles

The error appears to have been in '/Users/markmcfate/Dev/Vagrant/drupal-vm/provisioning/playbook.yml': line 22, column 7, but may
be elsewhere in the file depending on the exact syntax problem.

The offending line appears to be:

    # Essential roles.
    - { role: geerlingguy.repo-remi, when: ansible_os_family == 'RedHat' }
      ^ here

Ansible failed to complete successfully. Any error output should be
visible above. Please fix these errors and try again.

You need to run $ sudo ansible-galaxy install -r provisioning/requirements.yml --force.

Thanks, but I have.  It returns the following error...
ERROR! - please specify a user/role name, or a roles file, but not both

Are you running it the root of the project? So that provisoning/requirements.yml actually is in the CWD. Which ansible version are you using?

Yes.
ma7053:drupal-vm markmcfate$ ansible --version
ansible <IP_ADDRESS>
  config file =
  configured module search path = Default w/o overrides
ma7053:drupal-vm markmcfate$ pwd
/Users/markmcfate/Dev/Vagrant/drupal-vm
ma7053:drupal-vm markmcfate$ ls
LICENSE			config.yml		example.config.yml	mkdocs.yml
README.md		docs			example.drupal.make.yml	provisioning
Vagrantfile		drupal.make.yml		examples		tests

Hm not sure what could be wrong. Only way I can get that message is if i add an argument to the end of the command.

Thanks oxyc, that helped.  I've taken the --force argument off the end and it appears to be working.

--force works for me though. It's just happens if I add something other than a flag. But if it works it works hehe. Can I close this?

Actually I was seeing it to.  When I SSH'd in, navigated to the right directory it started building the requirements and gave an error:
- downloading role from https://github.com/geerlingguy/ansible-role-nodejs/archive/2.1.1.tar.gz
- extracting geerlingguy.nodejs to /etc/ansible/roles/geerlingguy.nodejs
- geerlingguy.nodejs was installed successfully
- downloading role 'php', owned by geerlingguy
https://galaxy.ansible.com/roles/432/versions/?page=2&page_size=50
Traceback (most recent call last):
  File "/usr/local/bin/ansible-galaxy", line 959, in <module>
    main()
  File "/usr/local/bin/ansible-galaxy", line 953, in main
    fn(args, options, parser)
  File "/usr/local/bin/ansible-galaxy", line 819, in execute_install
    if len(role_versions) > 0:
TypeError: object of type 'NoneType' has no len()

@itsgreggmarshall you need to update your ansible installation. https://github.com/geerlingguy/drupal-vm/issues/482

Ahhh


Revert "Support timing function for animation. Fix #12"

Reverts IBAnimatable/IBAnimatable#456
Sorry @phimage ,  I really like your feature and it is almost ready to go. But it is a little bit rush for merging this PR into master because it is the main feature for release 5. We would like to have at least one 👍 for each PR. And two 👍 for major PR. Can we revert it now and plan the release 5 together?
And we have a request to make a point release for release 4 for some hot-fix. It is hard to mix two releases together because we only have one main branch master. We did have a separate branch for the main release like swift3 before. But reduce to master only for easing the maintenance cost. We may treat feature/timingfunction as the develop branch for release 5.
Do you have any blocker and need to merge this branch into master to unblock? Please let us know and see how can we handle that.
I am sorry I didn't let your know the PR process and I didn't look at your PR on time. Please let me know what you think and how can we do that better.

          1 Error

:no_entry_sign:
Any changes to library code need a summary in the Changelog.

          2 Warnings

:warning:
Big PR

:warning:
Consider adding supporting documentation to this change. Documentation can be found in the docs directory.

  Generated by :no_entry_sign: Danger

No blocker
Just other features to implement into IBAnimatable, so common files to edit
When I see #472, I am wondering If my PR will be annoying to manage hotfix
I take master as a develop branch, but there is no branch for a release so that could not work


Transpile Job on macOS: sed is not portable

The transpile job is currently set like this:
babel -d lib.es5/ --presets es2015 lib/ && sed -i '1i // Generated by Babel' lib.es5/*.js lib.es5/*/*.js

However macOS machines have a different sed command: they're not using the GNU version, they're using another version.
It's not possible to use the same sed command on both Linux and macOS.
The version above only runs on Linux, on macOS it throws sed: 1: "lib.es5/error.js": extra characters at the end of l command.
As of now we're relying on contributors with Linux to transpile the code to es5.

🎉


data is being round off to 2 points

id like to show the values in this format 0.057016 in Line Chart Data part any help?

Have added optional decimalPlaces attribute to resolve this and submitted a pull request

Thanks (y)


Mdbook links seem to be broken

A few links that are relative to README.md in the root directory seem to be broken on building mdbook. I suspect it has something to do with the difference in relative paths of these directories from docs and root. Tried building mdbook from both root and docs directories. Am I missing something? Not sure how to fix it.

Thank you for noticing! Not a big deal for now since we'll probably need to update main README quite substantially soon. But I'll keep this open until we do.

Fixed by #331


Convert Html to Pdf with scripts

Hi,
I want to convert my Html page to pdf format. Html page contains javascript to hide and show the link.
Html page with link is working properly but while converting Html to Pdf that link is visible even if the hidden (style = "display:none") is applied on Linkbutton.
kindly share the idea.
soon reply helps me lot.
Thanks.

If you actually mean PDF and not PPT, check out jsPDF.
Otherwise, check out the demo under examples for working HTML-to-PPT code.


8bit TODO

Any update on getting "else if(encoding === '8bit')" going?

@scramble45
I can propose my version, because my PR was not accepted here.
https://github.com/jackkum/node-pdu


apps: Add missing cert-manager ignore labels

What this PR does / why we need it:
Since we migrated to a newer version of cert-manager, the ignore label for the webhook changed. This patch adds the new label. Without this ... I'm actually ./bin/ck8s apply (sc|wc) worked at all. :smile:
Which issue this PR fixes (use the format fixes #<issue number>(, fixes #<issue_number>, ...) to automatically close the issue when PR gets merged): fixes #
Public facing documentation PR (if applicable)
NA
Special notes for reviewer:
Checklist:

[X] Added relevant notes to WIP-CHANGELOG.md
[X] Proper commit message prefix on all commits
[ ] Updated the public facing documentation
Is this changeset backwards compatible for existing clusters? Applying:

[X] is completely transparent, will not impact the workload in any way.
[ ] requires running a migration script.
[ ] will create noticeable cluster degradation.
E.g. logs or metrics are not being collected or Kubernetes API server
will not be responding while upgrading.
[ ] requires draining and/or replacing nodes.
[ ] will change any APIs.
E.g. removes or changes any CK8S config options or Kubernetes APIs.
[ ] will break the cluster.
I.e. full cluster migration is required.

What caused the lable to change?
Also I'm not sure why it wouldn't work without this. I agree that we can add the lable, just want to understand what you mean. :slightly_smiling_face:

@viktor-f @Xartos To your question, no, we don't need the other label, but I would like to keep it for now, until we have a QA process in place that can catch such missing labels.
The exact error I was getting is:
$ ./bin/ck8s apply sc
[...]
Templating release=cert-manager, chart=jetstack/cert-manager
in /home/cklein/compliantkubernetes-apps/helmfile/01-applications.yaml: command "/usr/local/bin/helm" exited with non-zero status:

PATH:
  /usr/local/bin/helm

ARGS:
  0: helm (4 bytes)
  1: template (8 bytes)
  2: cert-manager (12 bytes)
  3: jetstack/cert-manager (21 bytes)
  4: --version (9 bytes)
  5: v1.1.0 (6 bytes)
  6: --namespace (11 bytes)
  7: cert-manager (12 bytes)
  8: --values (8 bytes)
  9: /tmp/helmfile971632085/cert-manager-cert-manager-values-84f7b998d4 (66 bytes)
  10: --validate (10 bytes)

ERROR:
  exit status 1

EXIT STATUS
  1

STDERR:
  coalesce.go:203: warning: destination for tolerations is a table. Ignoring non-table value []
  coalesce.go:163: warning: skipped value for tolerations: Not a table.
  coalesce.go:203: warning: destination for tolerations is a table. Ignoring non-table value []
  Error: rendered manifests contain a resource that already exists. Unable to continue with install: could not get information about the resource: Get "https://<IP_ADDRESS>:6443/apis/rbac.authorization.k8s.io/v1/clusterroles/cert-manager-cainjector": http2: client connection lost

COMBINED OUTPUT:
  coalesce.go:203: warning: destination for tolerations is a table. Ignoring non-table value []
  coalesce.go:163: warning: skipped value for tolerations: Not a table.
  coalesce.go:203: warning: destination for tolerations is a table. Ignoring non-table value []
  Error: rendered manifests contain a resource that already exists. Unable to continue with install: could not get information about the resource: Get "https://<IP_ADDRESS>:6443/apis/rbac.authorization.k8s.io/v1/clusterroles/cert-manager-cainjector": http2: client connection lost
exit status 1

In essence, the cert-manager Chart was installing a webhook, which was validating the cert-manager namespace but (very importantly) before the Pod handling the webhook had time to start. I would assume this to manifest rather rarely (i.e., timing needs to be bad) and only when setting up a cluster from scratch. (Hence, @viktor-f 's cleanup script is gold for QA.)

@Xartos The change seems to be buried under "Uninstall/Re-install cert-manager using the provided instructions". The exact commit is here: https://github.com/jetstack/cert-manager/commit/75a65de98a5c56fbf764ef7fffcff8817921a4b2

I don't get this. It should not be necessary to have that annotation on the namespace anymore. It was removed here.
Last time it was used was in the v0.10 installation and in v0.11 it is no longer used.
It looks like the error is that the ClusterRole cert-manager-cainjector already exists and/or there is some problem with the network/api-server when it tries to check that.

As Lennart says, labling the cert-manager namespace is not longer nessecary https://github.com/jetstack/cert-manager/pull/2135/files#diff-f732c807a49f400d275991bee55545b3d3b011ac4422fa151a9fa526cdda6914

Create https://github.com/elastisys/compliantkubernetes-apps/issues/291 to track removal of the label.

 I wonder what went wrong in my case.
I'm closing this issue for now and will reopen it if I see the error. It could be that the control-plane node was too small and I got confused by its instability.


List of sponsored accounts

I'm going to hold off on this one. Not sure in @srkirkland wants it...
Although, eventually it will be nice so sponsors can remove account access.


[chore] add to docs the default value of the html class option.

There is only one place that mention about the default value of the htmlOpenClassName.
Fixing by adding it to the docs/styles/classes.md page.
Changes proposed:

None

Upgrade Path (for changed or removed APIs):

None

Acceptance Checklist:

[x] All commits have been squashed to one.
[x] The commit message follows the guidelines in CONTRIBUTING.md.
[x] Documentation (README.md) and examples have been updated as needed.
[x] If this is a code change, a spec testing the functionality has been added.
[x] If the commit message has [changed] or [removed], there is an upgrade path above.

Coverage remained the same at 87.185% when pulling 0d00b0f08592cad2aa1f635594706e89a5652a04 on diasbruno:chore/default-value-of-html-class-option into 857a30321c593cb22f16762c93dff7f7ac4e4e28 on reactjs:master.


[Feature] Add crc ssh command to login to VM

Following are some valid reasons to have crc ssh command.

CRC is developer specific tool and we should provide an easy access to user to get into node and explore.
Sometime apiserver is not up/working and user can't use oc debug/node command to get into vm.
Sometime user doesn't have enough bandwidth to pull debug pod image and need to wait long enough.
We already have debug doc which mention how to ssh to the VM so better to make it first class citizen.
Other developer tools like minikube/minishift/k8s have ssh exposed command and user come from those tools shouldn't have to dig in the docs to perform same.

I am reopening the issue, in case of podman bundle we want to have same user experience as podman machine provides and it does have a way to ssh to machine. Also in case of and if we are also try to implement https://github.com/code-ready/crc/issues/2942 one and have both profile running simultaneously we do need to change sshPort for different profile and the way podman-machine do is random generate a ssh port and use it which is much better.

in case of podman bundle we want to have same user experience as podman machine provides

Does this mean we need to open issues for all these subcommands? https://docs.podman.io/en/latest/markdown/podman-machine.1.html

It would definitely be helpful as mentioned above in some of the comments to have a crc ssh to just debug and poke around whats there, just for investigative purposes. Right now, I do this through a privileged pod and then nsenter to get into the host but it would definitely be nice to have a ssh command though.

https://github.com/crc-org/crc/wiki/Debugging-guide has some documentation explaining how to ssh into the VM.


Unable to consume cross-project AVRO messages

Hi,
Currently it's possibile to subscribe to a topic from a different project, but it's not possible to consume messages when them are serialized with AVRO because we are unable to get the writer schema from the full name in the googclient_schemaname attribute.

Ok, I probably found a solution using the SchemaClient:
const client = pubsub.getSchemaClient()
const [schema] = await client.getSchema({
    name: 'projects/project-id/schemas/schema-id',
    view: 'FULL',
})
```


flashing in many gui examples. v0.13.1 caused a regression.

https://github.com/delaneyj/datastar/releases/tag/v0.13.1 is broken with flashing in GUI
https://github.com/delaneyj/datastar/releases/tag/v0.13.0 works
http://localhost:8080/examples/click_to_edit shows what I mean by "flashing"..

Thanks @gedw99 for the report, we didn't run the test suite prior to the hot fix.  This PR passes, sorry about that.  We need to work on a set of Github Actions that auto run prior to accepting PRs in the future

Nice work in the test suite btw!!

@rphumulock is to thank!


Bar chart starts below x-axis

Expected Behavior
Bar should start at the x-axis
Current Behavior
When series have big enough values (ex 1000000) and with yAxes bound with small enough values (from -0.05 to 0.05) rectangle flickers for duration of initial animation and starts not at the x-axis
Steps to Reproduce
https://jsfiddle.net/wgatm43e/6/
Context
I'm trying to integrate zoom plugin, and at high zoom values this issue appears
Environment

Chart.js version: 2.9.3
Browser name and version: Chrome 85

Reproduces in master: https://jsfiddle.net/efo5kvn6/

The issue seems to be with fillRect and large negative coordinates https://jsfiddle.net/3owh98f1/


[Discuss] Set ROBYN_BASE_URL environment variable when starting the application

This is an idea I was pondering over. Should we do this? This could make our tests a lot cleaner.

No. The library should not touch the env variables. Should be the other way round


Fix Consumer leader change failure

If the leader for a partition becomes a follower for that partition,
then a fetch request returns ErrNotLeaderForPartition. The Consumer
properly handled this case and refreshed metadata.
If the leader for a partition hands off the partition entirely and is no
longer a broker for that partition, it will instead return
ErrUnknownTopicOrPartition. In this case, the Consumer returned the
error and never refreshed metadata, which means that the Consumer is
stuck.
This problem can be reproduced by creating a Kafka cluster with 2 nodes.
Make a single topic with 1 partition and 1 replica. Start consuming,
then force a leader change.

Pull request seem to be stale, assigning @PragmaticCypher as the responsible developer.

Can you rebase with master and push again? I just updated travis configuration and it should build now.

@husio done


Kubernetes v1.8.4 support

What this PR does / why we need it: Adds support for Kubernetes v1.8.4
Release note:

Kubernetes v1.8.4 support

//TODO @JiangtianLi will add v1.8.4 Windows binaries.

@jackfrancis The test failure appears to be api server not reachable (dashboard, kubectl get svc). Let me run a single test.


Update version string to 1.1

After updating the firmware via Arduino on a Mac, the LCD on the BCN3DR still shows "1.0".
We tested that the LCD shows "1.1" when starting.

Thanks for the detail @jasonh-n-austin


some characters not support in jyutping

I try to use the jyutping to convert characters to jyutping, but I found some character can be convert:
for example:
txt='昆省急救服务中心嘅医护人员昆省警方。'
the output is:
[('昆', 'gwan1'), ('省', 'saang2'), ('急救', 'gap1gau3'), ('服', 'fuk6'), ('务', None), ('中心', 'zung1sam1'), ('嘅', 'ge3'), ('医', 'ai3'), ('护', None), ('人', 'jan4'), ('员', None), ('昆', 'gwan1'), ('省', 'saang2'), ('警方', 'ging2fong1'), ('。', None)]
you can see that ‘务’，‘护’，‘员’ are None

Hello! Currently, pycantonese (as of v3.4.0) supports only traditional characters. If your input contains simplified characters, you may consider piping it through a converter (such as OpenCC) before passing it to pycantonese.


Added secrets permission for rbd-provider clusterrole

Fix for issue #622

Signed now

@bruce2409 please rebase

This won't be needed once we update the image to use the new secrets stuff, thank you https://github.com/kubernetes-incubator/external-storage/issues/766


qqbot.py运行时，出现报错

运行日志如下
`当前进程pid: 2268
#################################################################
██████╗ ██╗   ██╗ ██████╗ ██████╗ ██████╗  ██████╗ ████████╗
██╔══██╗╚██╗ ██╔╝██╔════╝██╔═══██╗██╔══██╗██╔═══██╗╚══██╔══╝
██████╔╝ ╚████╔╝ ██║     ██║   ██║██████╔╝██║   ██║   ██║
██╔═══╝   ╚██╔╝  ██║     ██║▄▄ ██║██╔══██╗██║   ██║   ██║
██║        ██║   ╚██████╗╚██████╔╝██████╔╝╚██████╔╝   ██║
╚═╝        ╚═╝    ╚═════╝ ╚══▀▀═╝ ╚═════╝  ╚═════╝    ╚═╝
v<IP_ADDRESS>  BY FengLiu
#################################################################
[2023-07-12 18:52:56,531][go-cqhttp/WARNING] PyCqBot: go-cqhttp 警告 Protocol -> device lock is disable. http api may fail.
[2023-07-12 18:53:12,036][MainThread/INFO] PyCqBot: 正在连接 go-cqhttp websocket 服务
Traceback (most recent call last):
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\site-packages\websockets\legacy\client.py", line 139, in read_http_response
status_code, reason, headers = await read_response(self.reader)
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\site-packages\websockets\legacy\http.py", line 130, in read_response
raise ValueError(f"unsupported HTTP version: {d(version)}")
ValueError: unsupported HTTP version: HTTP/1.0
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "D:\Spit_chatBot\qqbot.py", line 447, in
bot.start()
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\site-packages\pycqBot\cqHttpApi.py", line 361, in start
self._websocket_start()
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\site-packages\pycqBot\cqHttpApi.py", line 401, in _websocket_start
asyncio.run(main_logic())
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\asyncio\runners.py", line 44, in run
return loop.run_until_complete(main)
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\asyncio\base_events.py", line 647, in run_until_complete
return future.result()
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\site-packages\pycqBot\cqHttpApi.py", line 375, in main_logic
async with websockets.connect(self.__host) as websocket:
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\site-packages\websockets\legacy\client.py", line 637, in aenter
return await self
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\site-packages\websockets\legacy\client.py", line 655, in await_impl_timeout
return await self.await_impl()
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\site-packages\websockets\legacy\client.py", line 662, in await_impl
await protocol.handshake(
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\site-packages\websockets\legacy\client.py", line 323, in handshake
status_code, response_headers = await self.read_http_response()
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\site-packages\websockets\legacy\client.py", line 145, in read_http_response
raise InvalidMessage("did not receive a valid HTTP response") from exc
websockets.exceptions.InvalidMessage: did not receive a valid HTTP response
PS D:\Spit_chatBot> Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.
Failed to write to log, write /dev/stdout: The pipe is being closed.`

已解决，但重新运行时出现了如下错误:
File "D:\Spit_chatBot\qqbot.py", line 431, in
load_bot_config()
File "D:\Spit_chatBot\qqbot.py", line 342, in load_bot_config
configs = json.load(f)
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\json_init_.py", line 293, in load
return loads(fp.read(),
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\json_init_.py", line 346, in loads
return _default_decoder.decode(s)
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\json\decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\json\decoder.py", line 355, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 17 column 18 (char 381)

已解决，但重新运行时出现了如下错误:
File "D:\Spit_chatBot\qqbot.py", line 431, in
load_bot_config()
File "D:\Spit_chatBot\qqbot.py", line 342, in load_bot_config
configs = json.load(f)
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\json_init_.py", line 293, in load
return loads(fp.read(),
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\json_init_.py", line 346, in loads
return _default_decoder.decode(s)
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\json\decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\json\decoder.py", line 355, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 17 column 18 (char 381)

请问后续错误是如何解决的

请问后续错误是如何解决的
配置写错了导致的

请问后续错误是如何解决的
配置写错了导致的

那看来我也写错了（笑着捂脸）


Create proof of concept for scsc

Create a program that runs this command and outputs the supply chain of the smart contract at address 0xabcdef012345678 at block number 1234.
./scsc --block 1234 --address 0xabcdef012345678

[ ] python script with cli
[ ] select of tool that collects data from the blockchain
[ ] analyse the collected data to output the result
[ ] decide the output format (e.g. dependency tree)

| Tool Name| API | Traces |

https://github.com/paradigmxyz/cryo looks similar to our tool

https://github.com/paradigmxyz/cryo looks similar to our tool
Agree, let's try it out for our tool.

Supplier info of the contracts - deployer address
Readable labels: names for contracts


IP whitelist

Hi, is it possible to turn vpn off for specific ip adresses?

This is not currently available, but is an existing feature request covered by #887.


TF400497: The backlog iteration path that you specified is no longer valid.

Hello,
how can I change the configuration / backlog iteration path?
Thx
Dominik

You can use  az boards iteration team set-backlog-iteration --id<ITERATION_IDENTIFIER> --team <YOUR_TEAM> to set backlog iteration for that team

I am hoping that above comment resolved your query. Hence, closing this issue, please feel free to reopen or raise a new one if you are facing any more issues.


Special types provided by Grape are custom ones

Users tries to use types provided by Grape as an example how to write custom types. So, it makes sense to treat them as custom.
Besides that, it fixes #1986 where a collection of a custom type wasn't coerced properly.

👍


Create & move static documentation

It would be great to move the documentation from README file to a dedicated docs
I have already enabled the github pages with build-in theme ( can be seen here), but it's very limited with no navigation capabilities.
Perhaps we can use Sphinx or smth like that?
There is an interesting read:
https://tech.michaelaltfield.net/2020/07/18/sphinx-rtd-github-pages-1/
but would require some setup like github actions to generate docs and python Sphinx
Does anyone has any knowledge or interest in how to do it? :)

I am also interested in how to do this. I tried to add it sometime ago to another project I made, but didn't manage to finish it. I believe a requirement for generating docs with sphinx or other software is you need to annotate your code like this: https://github.com/DurgNomis-drol/mytoyota/blob/1d4cc568af0daa71fa8bc9e8828152c10da42bad/mytoyota/client.py#L127 This links to how i have done it in my code, though it could probably be done a lot better 😄

I don't think we need auto-generated documentation for code.
Github wiki can do the job but it makes harder to accept PRs for it.
So simple TROUBLESHOOTING.md in repo root should be fine.


[docs] theme Material-UI components inside of a library

Duplicates

[X] I have searched the existing issues

Related page
https://mui.com/material-ui/customization/theming/
Kind of issue
Missing information
Issue description
I was looking for some guidance how to use Material-UI inside of an internal component library. Especially how to export custom components wrapping Material-UI components and theming them. I was not able to find anything on this topic in the docs.
How do I export custom components that wrap Material-UI components and theme them? My current ideas are:

wrap every exported component into a ThemeProvider (resulting in many ThemeProviders in the application using the library which sounds bad)
force the users of the library to wrap their application in a Provider exported from our library (which would include the ThemeProvider with the theme)
using only the sx property for styling the components (so not using a theme at all)
open a feature request for Material-UI to allow passing the theme directly to every component as a prop

Do I miss something or what would be the preferred option? And could this be added to the docs?
Context 🔦
No response

I now also found the createStyled() function but this will also not apply a theme to an existing MUI component.
Maybe give an example of what I want:
import { createTheme } from "@mui/material";
import Button from "@mui/material/Button";
import { yellow } from "@mui/material/colors";
import { createStyled } from "@mui/system";

const defaultTheme = createTheme({
  palette: { primary: yellow },
});

const styled = createStyled({ defaultTheme: defaultTheme });

const MyButton = styled(Button)(({ theme }) => ({
  color: theme.palette.primary.main, // why do I need to apply the theme here? why can't styled(Button) apply the theme?
}));

export const App = () => <MyButton color="primary">Hello world</MyButton>;

I want to understand what's the issue with point 2 ? This is a standard practice across several React libraries. It requires minimal changes from the consumer app as well, mostly few lines addition in a single file.
Also, if you are just customizing the theme, you just need the ThemeProvider and no other change.

Hm, maybe you are correct, I maybe just wanted to further simplify it, but in the end it is just adding one additional Provider. Thank you for clarifying this.


Personal Notification

Hi team, is there any recommended method for sending a Teams notification to a specific user without the use of a bot? Thanks!

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

ID: 4ad9f167-7d29-f933-7b47-60bd744899b1
Version Independent ID: 8097fcbd-adcc-ce49-a378-426c65af9296
Content: Send proactive messages - Teams
Content Source: msteams-platform/bots/how-to/conversations/send-proactive-messages.md
Product: msteams
GitHub Login: @clearab
Microsoft Alias: anclear

You can send notification to an user only using proactive message from a bot. There is another way to send messages to a channel using webhooks and connectors

Ah, ok thank you @Subhasish-msft - I'll close :).

@dastewa: You can also use Power Automate (formerly Flow) to send messages to a user.

Thank you for that suggestion @clearab! That's awesome, I'll be diving into that some more - cheers!


[Best Practices] Eliminate the direct usage of ErrorAndExit in CLI command

Is your feature request related to a problem? Please describe.
We recently upgraded urfave package to V2 which now supports error handling in a centralized way. For best practices, we should require every command line to return error instead of exit directly.
Proposed Solution
Deprecate the usage of ErrorAndExit and propagate error to command line instead.
Additional context
The function to remove is here https://github.com/uber/cadence/blob/605fc74ea8af939bbbaac230991404deed929991/tools/cli/utils.go#L555

Hi , I'm interested in solving this issue. Thanks


Allow --cpu=x64_arm64_windows as a compilation target on Windows

Compilation of TensorFlow on Windows currently assumes x64 as a target CPU architecture. Searching .bzl files for "x64_windows" shows the many places in TF where this assumption is made.
In principle, though, nothing should prevent compiling TensorFlow to target ARM64 on Windows (--cpu=x64_arm64_windows, see the bazel doc).
This feature request is about adding support for '--cpu=x64_arm64_windows' to produce ARM64 binaries. The goal is to be able to use TF on Windows on ARM 64 bits.

Will happily review and approve a PR

I would happily work on a PR, but unfortunately my knowledge of Bazel and TF build system is nearly zero. Passing --cpu=x64_arm64_windows causes the TF build system to download LLVM instead of using MSVC.
@mihaimaruseac : Can you hint where to start fixing/patching? The initial goal is to get the MSVC toolchain invoked, then I should be able to work on the required code changes.

TF needs to download LLVM to build as it contains a lot of code that uses LLV libraries directly (MLIR)

TF needs to download LLVM to build as it contains a lot of code that uses LLV libraries directly (MLIR)

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


Relocate DB - New feature

Hello,
It'd be much of a use to have the ability to relocate the DB to another drive.
Many thanks.

Hi!
Currently you may safely relocate the DB files, by simply moving or coping datadir on another drive:
mv /path/to/.lachesis /new/path/to/.lachesis

By default, lachesis datadir is $HOME/.lachesis
It's safe to launch lachesis on a new drive or on a completely new machine, by copying datadir files.
You may launch lachesis with new datadir, by using --datadir flag:
lachesis --datadir /new/path/to/.lachesis

Pay attention that if run a validator, you must not launch multiple instances of the same validator simultaneously - it may lead to doublesign due to race condition between those instances.
If you need to transfer your keys on a new device, then you may safely transfer files from /path/to/.lachesis/keystore on a new device


Invalid alpha validation.

Laravel Version: 9.x

Description:
https://laravel.com/docs/9.x/validation#rule-alpha
May I ask alphabetic characters. Does the meaning of this sentence include other languages, or is it just 26 English letters?
Steps To Reproduce:
Use alpha for form validation, enter Chinese, the validation rule will pass

This issue is related to this validation rule preg_match('/^[\pL\pM]+$/u', $value);

This regex is using Unicode character properties. It is many, many more characters than A-Z. If you want to restrict it to just those 26.
If you want something more restrictive, you could use the regex rule with exactly which ranges of letters you want to allow.

Thanks @joelclermont
I know this regular match is a Unicode character. But I am confused about this document. My understanding of the document description is that it will verify whether it is an English letter, not a Unicode character.

alpha
The field under validation must be entirely alphabetic characters.

Thanks @joelclermont
I know this regex match is a Unicode character. But I am confused about this document. My understanding of the document description is that it will verify whether it is an English letter, not a Unicode character.
alpha
The field under validation must be entirely alphabetic characters.


TEST: Increase test assertion timeout for ssl reload tests.

Summary of the changes / Why this improves CrateDB
Checklist

[ ] User relevant changes are recorded in CHANGES.txt
[ ] Touched code is covered by tests
[ ] Documentation has been updated if necessary
[ ] CLA is signed
[ ] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

retest this please

retest this please


pr test

@adunntest : Thanks for your contribution! The author, @meganbradley, has been notified to review your proposed change.

@adunntest : Thanks for your contribution! The author, @meganbradley, has been notified to review your proposed change.

@meganbradley : Thanks for your contribution! The author, @meganbradley, has been notified to review your proposed change.

@meganbradley : Thanks for your contribution! The author, @meganbradley, has been notified to review your proposed change.

@adunntest @adunndevster confirmed that a regular commit by me doesn't close the PR. Still trying to figure out how to create a suggestion but it looks goo so far!

@adunndevster I rand out of time to figure out Suggestions (the documented process in GitHub docs doesn't match what I see in UI) - but I'm assuming this is really fixed and removed the user story. If we discover any bugs we can revisit.


Downloading .mp4 files?

Will this project be extending the type of files it can capture or is there a way to tweak it such that it will download videos as well? E.g https://opensea.io/assets/0xf54cc94f1f2f5de012b6aa51f1e7ebdc43ef5afc/51

@zarst990 I'll try to implement this ASAP. I think I could probably extend it past just MP4 files. Maybe GIFs, etc.


feat(client): allow to convert serve value to proto value

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

GitHub release
v0.12.0
v0.12.0

Your semantic-release bot :package::rocket:


Add skew and kurtosis operations

Summary

Adds operations for skew and kurtosis.
The implementation uses the same Welford/Knuth approach as the variance operation.
https://en.wikipedia.org/wiki/Algorithms_for_calculating_variance#Higher-order_statistics
I have the operations set as non-deletable as the reversals weren't straightforward (variance is also set this way).
Why / Goal

We have existing features in our previous feature store which rely on these statistics.
Test Plan

[x] Added Unit Tests
[x] Covered by existing CI
[x] Integration tested

Checklist

[x] Documentation update

Reviewers

Solid PR!


SIGINT handling when used with PyTorch Lightning.

wandb --version && python --version && uname

Weights and Biases version: 0.9.3
Python version: 3.8.2
Operating System: Darwin

Description
I'm using PyTorch Lightning with WandB. PyTorch Lightning's training loop catches SIGINT (and others) and gracefully shuts off, allowing the script to continue its execution. This is quite useful b/c you can stop your training in the middle and still run the test loop and other tasks after. However, when I use WandB with PyTorch Lightning, it seems like WandB catches SIGINT correctly and passes it down to the script, thus stopping the execution of the script, but it also kills the script right there, so the test loop and other tasks do not get executed. Is there a workaround?

Thanks for reporting. @borisdayma could you take a look?

Hi there, Boris suggests that you try the new version of our client. @raubitsj what would be the recommended way for @yukw777 to try the new CLI?

The new version of the client is available by running the following command:
pip install --upgrade --pre wandb

with the pre-release (0.10.0rc1) I see this error:
Traceback (most recent call last):
  File "/Users/peteryu/leela-zero-pytorch/leela_zero_pytorch/train.py", line 25, in main
    trainer.fit(
  File "/Users/peteryu/pytorch-lightning/pytorch_lightning/trainer/trainer.py", line 1097, in fit
    results = self.accelerator_backend.train(model)
  File "/Users/peteryu/pytorch-lightning/pytorch_lightning/accelerator_backends/cpu_backend.py", line 41, in train
    results = self.trainer.run_pretrain_routine(model)
  File "/Users/peteryu/pytorch-lightning/pytorch_lightning/trainer/trainer.py", line 1230, in run_pretrain_routine
    self.train()
  File "/Users/peteryu/pytorch-lightning/pytorch_lightning/trainer/training_loop.py", line 359, in train
    self.on_train_start()
  File "/Users/peteryu/pytorch-lightning/pytorch_lightning/trainer/callback_hook.py", line 97, in on_train_start
    callback.on_train_start(self, self.get_model())
  File "/Users/peteryu/pytorch-lightning/pytorch_lightning/utilities/distributed.py", line 12, in wrapped_fn
    return fn(*args, **kwargs)
  File "/Users/peteryu/pytorch-lightning/pytorch_lightning/callbacks/model_checkpoint.py", line 252, in on_train_start
    ckpt_path = os.path.join(
  File "/Users/peteryu/.pyenv/versions/3.8.2/lib/python3.8/posixpath.py", line 90, in join
    genericpath._check_arg_types('join', a, *p)
  File "/Users/peteryu/.pyenv/versions/3.8.2/lib/python3.8/genericpath.py", line 152, in _check_arg_types
    raise TypeError(f'{funcname}() argument must be str, bytes, or '
TypeError: join() argument must be str, bytes, or os.PathLike object, not 'NoneType'

The issue persists on 0.9.4.

Hey @yukw777 the error looks to be in pytorch lightning itself here.  Are you specifying save_dir or log_model=True in your instance of WandbLogger?  It might also be caused by not having a name for the logger, I would also try passing name to the WandbLogger.  Looking at the code it's either a nil save_dir which will get set if log_model=True or if it's passed in, or it's a nil name which should get set to the wandb project name but may not always get set.

@vanpelt I've tried two combinations: (pl==0.8.5, wandb==0.9.4) and (pl==0.8.5, wandb==0.10.0rc1) both without setting save_dir, log_model or name (basically all the default values for WandbLogger). I only see the error I posted above with 0.10.0rc1, which suggests to me that the behavior has changed on the wandb side. I think it's b/c wandb.init() no longer generates a project name automatically if name=None on 0.10.0rc1.

I just tried again with (pl==0.8.5, wandb==0.10.0rc1) (this time I manually set the name parameter to get around the error I posted above), and the issue still persists.

Alright, upon further investigation, this is what I've found:

The SIGINT handling issue is indeed fixed in 0.10.0rc1!
run.project_name() always returns None on 0.10.0rc1. On 0.9.4 and 0.9.3, it was autogenerated if not set.

Thanks for this analysis!  We'll fix the project_name issue ASAP.

run.project_name() is fixed in 0.10.0rc2
Install update with:
pip install --upgrade --pre wandb

@raubitsj Thanks! I'll file an issue on the github.com/wandb/client-ng side. I was wondering where the RCs were being tagged haha

Hey @yukw777
In the past few months we've majorly reworked the CLI and UI for Weights & Biases. We're closing inactivate threads for the time being. Please comment to reopen. :smile:


NPOINTS for dynamic axes, to specify exact number of points

Reported by andrew wittenberg on 25 Sep 2011 19:01 UTC
For static axis definitions, we have the NPOINTS qualifer to specify an exact number of points.  It'd be nice to have the same ability for dynamic axis definitions, to prevent things like this:
atw:/home/atw> ferret
        NOAA/PMEL TMAP
        FERRET v6.71
        Linux 2.6.18-274.el5PAE 32-bit - 08/05/11
        25-Sep-11 14:11

yes? let n = 7; def ax/x=0:1/np=`n` xax; list/p=10 x[list/p=10 x[gx=0:1:`1/(n-1),p=10`](gx=xax];)
 !-> def ax/x=0:1/np=7 xax
             VARIABLE : X
                        axis XAX
             SUBSET   : 7 points (X)
 0    / 1:  0.000000000
 0.17 / 2:  0.166666672
 0.33 / 3:  0.333333343
 0.5  / 4:  0.500000000
 0.67 / 5:  0.666666687
 0.83 / 6:  0.833333313
 1    / 7:  1.000000000
 !-> list/p=10 x[            VARIABLE : X
                        axis (AX002)
             SUBSET   : 6 points (X)
 0    / 1:  0.0000000000
 0.17 / 2:  0.1666666716
 0.33 / 3:  0.3333333433
 0.5  / 4:  0.5000000000
 0.67 / 5:  0.6666666865
 0.83 / 6:  0.8333333731

I presume that the delta in the dynamic axis definition is being truncated to machine precision.  And when that truncated value is used as a regular spacing, the accumulated sum then builds up an error such that the last coordinate no longer fits inside the desired range (0:1).
I'd suggest adding an optional fourth argument to the dynamic axis definition, e.g.
xgx=start:end:delta:npoints
Then
x[gx=0:100::7]
would specify that we want 7 points from 0 to 100, inclusive.  If both DELTA and NPOINTS are given, Ferret could raise an error, or do what DEFINE AXIS does and just silently ignore DELTA and use NPOINTS.
Can dynamic axes have irregular spacing?  If so, then the above dynamic definition should work just like the static one.  If not, and if precision issues don't permit NPOINTS points to be spaced evenly over the interval, then perhaps expand the endpoint as needed until we do actually have NPOINTS points.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1885

Comment by andrew.wittenberg on 25 Sep 2011 19:37 UTC
Further refinement to afford more control -- especially if it turns out that dynamic axes have to be evenly spaced.  The user would be required to specify at least 3 of the 4 arguments, and Ferret would fill in the missing argument with a reasonable choice.
   ! the following would produce {1,2,3,4,5}
   x[  x[gx=1:5::5](gx=1:5:1]
)
   x[  x[gx=:5:1:5](gx=1::1:5]
)

   ! more examples
   x[  ! {0, 3, 6, 9}
   x[gx=0::3:4](gx=0:10:3])   ! {0, 3, 6, 9}
   x[ ! {1, 4, 7, 10}
   x[gx=0:10::4](gx=:10:3:4])  ! {0, 3.333, 6.667, 10}

   ! and if dynamic axes had to be evenly spaced:
   x[gx=0:10::7]  ! {0, 1.666, 3.333, 5.000, 6.666, 8.333, 10.00000191}

Comment by ansley.b.manke on 27 Sep 2011 17:47 UTC
Of course all the DEFINE AXIS commands suffer from the inability to specify double precision values in the commands themselves.  The start and end points of DEFINE AXIS/X=Lo:Hi, and DEFINE AXIS/FROM_DATA are other examples.
Dynamic axes are regularly spaced, but if we had the correct npoints, they wouldn't be any different from other regularly-spaced axes.  The start, end, and delta (computed in double precision) are stored in double precision and the interior coordinate values are computed not by successive addition but by double-precision multiplication.
This means that the dynamic axis x[gx=0:10::7] would behave like  DEFINE AXIS/x=0:10/npoints=7 and the coordinate values would be (from an ncdump of a variable saved on the axis),
XAXIS = 0, 1.66666666666667, 3.33333333333333, 5, 6.66666666666667,
8.33333333333333, 10

Comment by ansley.b.manke on 16 Jun 2015 19:18 UTC
With double-precision Ferret, the values in the command line are stored as double precision values.  The proposals here would be useful additions, but we can make all(?) the right definitions for regularly-spaced axes with existing syntax.

yes? list/p=15 x[gx=0:1:`1./6,p=15`]
             VARIABLE : X
                        axis (AX005)
             SUBSET   : 7 points (X)
 0    / 1:  0.00000000000000
 0.17 / 2:  0.16666666666667
 0.33 / 3:  0.33333333333333
 0.5  / 4:  0.50000000000000
 0.67 / 5:  0.66666666666667
 0.83 / 6:  0.83333333333333
 1    / 7:  1.00000000000000


Add & Fix some translation

So @akakaras help me correct and commit the translations. Please close this pull request again. Sorry. sigh