Enable gpu metrics in Prometheus with gpu plugin

With gpu plugin activated it'd be great to expose gpu metrics in grafana via prometheus as well.
This requires to run a daemonset of gpu exporter with this spec:

apiVersion: apps/v1
kind: DaemonSet
metadata:
name: pod-gpu-metrics-exporter
namespace: monitoring
spec:
selector:
matchLabels:
app: pod-gpu-metrics-exporter
template:
metadata:
labels:
app: pod-gpu-metrics-exporter
name: pod-gpu-metrics-exporter
spec:
nodeSelector:
hardware-type: NVIDIAGPU
containers:
- image: nvidia/pod-gpu-metrics-exporter:v1.0.0-alpha
name: pod-nvidia-gpu-metrics-exporter
ports:
- name: metrics
containerPort: 9400
hostPort: 9400
securityContext:
runAsNonRoot: false
runAsUser: 0
volumeMounts:
- name: pod-gpu-resources
readOnly: true
mountPath: /var/lib/kubelet/pod-resources
- name: device-metrics
readOnly: true
mountPath: /run/prometheus
- image: nvidia/dcgm-exporter:1.4.6
name: nvidia-dcgm-exporter
securityContext:
runAsNonRoot: false
runAsUser: 0
volumeMounts:
- name: device-metrics
mountPath: /run/prometheus
volumes:
- name: pod-gpu-resources
hostPath:
path: /var/snap/microk8s/common/var/lib/kubelet/pod-resources
- name: device-metrics
emptyDir:
medium: Memory

And also to update Prometheus' configmap with the following pattern:

prometheus.yml: |-
global:
scrape_interval:     15s
evaluation_interval: 15s
scrape_configs:
- job_name: pod-gpu
metrics_path: /gpu/metrics
scheme: http
static_configs:
- targets:
- :9400

We could introduce a flag in [1] eg, microk8s enable gpu:report-metrics=true with which the user could state his intentions.
If that flag is enabled we would enable the prometheus addon and apply the suggested config map.
Would that work @grebennikov ?
[1] https://github.com/ubuntu/microk8s/blob/master/microk8s-resources/actions/enable.gpu.sh


Add examples with blueprints usage.

Most production flask apps would use blueprints. Add documentation to show how to achieve tracing of such application layout.

Has anyone used this particular setup? Im working with blueprints and wondering what best practice is for setting up tracing with them using that project structure.

This can be implemented by passing the flask blueprint to the FlaskTracing.__init__ method instead of the app and trace_all_requests=True

This package is no longer being maintained, users should migrate to the opentelemetry-api package.
Closing.


Replace readme examples with actual code snippets

I also used Verify to generate the output of the snippets.

Hi and thanks a lot for the PR :)
Some questions:

Would I be right to assume that the ultimate goal of this tool is to just have all code snippets encoded as actual tests? Meaning that instead of Snippets.cs I could just reference existing test code (given that it has all the samples, of course) in readme?
Are #region blocks required for identification? Or can I also reference a snippet by method name?
Are the code snippet links in readme updated automatically by GitHub actions? Specifically I'm interested in the hrefs like this one snippet source

Overall I think MarkdownSnippets provides a nice way to ensure that code samples stay in-sync with API or behavioral changes. On the other hand, I'm not 100% convinced that the added overhead (albeit small) is worth it, given that I don't really have too many issue maintaining code samples (it's the human text part that's really hard 😅). I'll give it some more thought later. 🤔
Also, I wonder if GitHub allowed to put permalinks to code in markdown files in the same way it does in issues/PRs (where it renders that code in-place), then this could be simplified. 🤔

Would I be right to assume that the ultimate goal of this tool is to just have all code snippets encoded as actual tests?

Not always. Often tests make for good snippets, but not always. The requirements of a test (useful, fast and readable for the maintainer) can often conflict with the requirement of a snippet (readable for a consumer, concise, explains a specific api/scenario). But yes, sometimes you can use tests for snippets. You can also sometimes use implementation code as snippets, eg to explain a behavior of a scenario. Note that mdsnippets scans the whole solution directory

Are #region blocks required for identification?

No Comments are also supported. The value of regions is that is forces a beginning and end that is verified by the compiler. So you get a more useful error than the one you get from the mdsnippets engine when it detects a missing end.
If you dont like the collapsing behavior of regions you can disable it. TBH i would recommend disabling it anyway so in any code base you open they will expand.

Or can I also reference a snippet by method name?

No this is not currently supported. It is theoretically possible to do with roslyn, but the effort and likely perf impact has meant i have not gone down that route.

Are the code snippet links in readme updated automatically by GitHub actions? Specifically I'm interested in the hrefs like this one

No these are generated by mdsnippets. The addition GH action in this pr performs the same snippet+md merging that occurs when you build. the reason i also use a GH action is so that if you edit some code online or push without rebuilding tests, then the snippets will still merged into the md files

I'm not 100% convinced that the added overhead (albeit small) is worth it

Yep understood. Dont feel abliged to accept this PR. I wont be offended if you dont want to take this approach

given that I don't really have too many issue maintaining code samples (it's the human text part that's really hard 😅)

Well you did have one bug in your snippets using var iconFileStream = File.Open("new_icon.ico"); does not compile 😄

Thanks for the explanation! It all makes much more sense now.

The addition GH action in this pr performs the same snippet+md merging that occurs when you build.

Aaahh I see. So technically it's not required, it's just useful to have. Makes sense. I should probably just the readme, I'm sure most of my questions are answered there. 😅

Yep understood. Dont feel abliged to accept this PR. I wont be offended if you dont want to take this approach

I still see some definite benefits with this, so maybe I just need some time to reconcile it in my head. For now I'll just leave this PR around and see if I come back to it eventually.
Thanks a lot for showing me mdsnippets 🙂

Well you did have one bug in your snippets using var iconFileStream = File.Open("new_icon.ico"); does not compile 😄

😱 Damn, the one thing I didn't verify 😂

Decided to leave it as is for now, but thank you for the introduction to Verify and MarkdownSnippets. I may come back to it when I figure out how to better organize examples with the rest of the code (personal OCD issues 😅).


feat: submodules expose readmes and targets via jsii-reflect

For the documentation for monocdk/v2, individual submodules need to be
able to have READMEs attached to them, as well as the targets
configuration of submodules needs to be exposed so that we can use this
to determine the FQNs of types in other languages (Java/C#/...).

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID:<PHONE_NUMBER>81863fb2045eff4ec99002783bfd83
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID:<PHONE_NUMBER>81863fb2045eff4ec99002783bfd83
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: e6288983e1b7a0e720566ceee124459533df5ec4
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: e6288983e1b7a0e720566ceee124459533df5ec4
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: a4d7f124230026ff832919fa7996cd547637c7df
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: a4d7f124230026ff832919fa7996cd547637c7df
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: 5ee0b50251ab72e1705a6eb56b4e4cafaadd7296
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: 5ee0b50251ab72e1705a6eb56b4e4cafaadd7296
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: e9ce65bab60a7811d3fc0b7f715bc703ad8ef721
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: e9ce65bab60a7811d3fc0b7f715bc703ad8ef721
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: 2de20f988bef3c576c34ca85a267e2b7413bf06c
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: 2de20f988bef3c576c34ca85a267e2b7413bf06c
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: 9c683f9cac808e21904d32fe19ce5f015b4362f9
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: 9c683f9cac808e21904d32fe19ce5f015b4362f9
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Add extension mechanism for dynamically discovering RAS error states

Helps to address issue with adding any new RAS categories since existing mechanism is static array

@jandres742 @saik-intel @vaidyan1-int @wdamon-intel for review
draft for now as if this looks good, I can add back the new categories that were reverted. And also need to finish the other PR to have Sysman extension discovery mechanism

@aravindksg Please create an issue to associate with this PR.

https://github.com/oneapi-src/level-zero-spec/issues/189

@wdamon-intel please check

@jandres742  @wdamon-intel @sanilkumar0 @vaidyan1-int added new enum for RAS category to this PR since we would be using this new category via new extension RasGetStateExp. please re-review

So are we deprecating the other interface and enums? if so, where are we adding that?

So are we deprecating the other interface and enums? if so, where are we adding that?

Suggest deprecating the old interface in a future point release (v1.8 or v1.9) so that this experimental extension has the necessary time to mature.


fix(recovery): Change initialiseConnection() in XAResourceRecoveryHelper

At least on Oracle & Postgres, the connections wrapped inside the XAConnections will automatically timeout after some time.
When it happens, the recovery mechanism does not detect that the XAConnections need to be closed & recreated (as the objects are not null). This will generate warning logs at regular intervals as the recovery mechanism tries to do its thing on "dead" connections.

This commit improves this behavior by:

Attempting to properly close the "active" connection&
then always recreating a connection during the recovery process.

@danila-m   CI failed on
[INFO] Starting audit...
[ERROR] /home/tester/jenkins/workspace/narayana-tomcat/f977fc1d/tomcat-jta/src/main/java/org/jboss/narayana/tomcat/jta/internal/PoolingDataSourceFactory.java:23: Using the '.*' form of import should be avoided - javax.sql.*. [AvoidStarImport]
Audit done.

The CI job needed reconfiguration and it should be fine now.
@zhfeng , if you comment run_tests   without the underscore it will run again....

@danila-m CI failed on
[INFO] Starting audit...
[ERROR] /home/tester/jenkins/workspace/narayana-tomcat/f977fc1d/tomcat-jta/src/main/java/org/jboss/narayana/tomcat/jta/internal/PoolingDataSourceFactory.java:23: Using the '.*' form of import should be avoided - javax.sql.*. [AvoidStarImport]
Audit done.

Done

run tests

@mmadzin @jfclere Could you take it from here, please?
h2 tests passing, but those with real DBs don't, there are obsolete http instead of https URLs used etc.
Unable to collect/resolve dependency tree for a resolution due to: Failed to collect dependencies at org.postgresql:postgresql:jar:42.2.2, caused by: Server returned HTTP response code: 501 for URL: http://repo1.maven.org/maven2/org/postgresql/postgresql/42.2.2/postgresql-42.2.2.pom
```
(not related to this PR)

The project needs some attention :-)

Hello @Karm I still approve of this change. Will you either merge it or withdraw the PR.

Unable to collect/resolve dependency tree for a resolution due to:
Failed to collect dependencies at org.postgresql:postgresql:jar:42.2.2,
caused by: Server returned HTTP response code: 501 for
URL: http://repo1.maven.org/maven2/org/postgresql/postgresql/42.2.2/postgresql-42.2.2.pom

(not related to this PR)

Is there an issue for this problem?

Hello @mmusgrov,
See https://github.com/web-servers/narayana-tomcat/pull/33
I haven't been involved with the project for years now.
@mmadzin, @csutherl or @jfclere could do the merging as they still own the project.
In addition to that, you yourself and @tomjenkinson are both committers on this repo with the power to merge changes.
Cheers
K.


VS 15.5.4 this feature fails frequently and disastrously

Solution with 100 projects, view call hierarchy hangs - status bar shows "Parsing files in solution..." (why?? how often does it need parsing?)
Any call that's reasonably abstract will crash intellisense.  From that point the search never completes.  Intellisense stops working properly.  The call hierarchy search cannot be cancelled.  Visual Studio cannot be closed - it sticks on 'Unloading the selected projects...'
Only resolution is to terminate the process and reboot PC for good measure.

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

ID: 27ca4cea-be24-22ad-49d0-d34e3f800900
Version Independent ID: 8cd52773-0617-684e-8136-bd4ea03515de
Content: Viewing call hierarchy in Visual Studio - Visual Studio
Content Source: docs/ide/reference/call-hierarchy.md
Service: visual-studio-dev15
GitHub Login: @gewarren
Microsoft Alias: gewarren

@ro-jo Thanks for the feedback. Would you mind logging this under "Product feedback" instead of "Documentation feedback" at the bottom of the Call Hierarchy topic? It will get better traction from the product team that way. Also, have you tried upgrading to the latest version, 15.7.1?

Thanks, I couldn't sign in on Product feedback, or find anywhere else to report the issue.  I think here's a good place, other users who experience the problem need to know about it.
I installed 15.7.1 today.  It flickers a lot and disables the GUI during intellisense updates so prevents me from triggering a second search while it's already updating - and as such it hasn't blown up yet.
I just don't know how msft didn't notice it during testing.

@ro-jo If you are still experiencing problems with Call Hierarchy on the latest version, let me know and I'll transfer this issue to Developer Community (product feedback). For now, I'm closing this issue since it's not actionable from a docs perspective.


cf html5-list -d -u does not show app url's

Description
When I execute cf html5-list -d -u I see the following:
name   version   app-host-id   service name    destination name                     last changed   url
-      -         -             fioridemo       fioridemo_FioriDemo_html_repo_host   -              https://sap.com/DUMMY_URL
-      -         -             fioridemo       fioridemo_uaa_FioriDemo              -              https://api.authentication.eu10.hana.ondemand.com
-      -         -             cloud.service   managed-fiori-destination-html5      -              https://sap.com/DUMMY_URL
-      -         -             cloud.service   managed-fiori-destination-uaa        -              https://api.authentication.eu10.hana.ondemand.com

The line for managed-fiori-destination-html5 is the app I am interested in, but it just shows "DUMMY_URL".
If I logon to the cloud platform cockpit and navigate to my subaccount and the HTML5 Application area I can see the app and click on it and it launches just fine with this url:
https://hanatest.cpp.cfapps.eu10.hana.ondemand.com/cloudservice.demoproductsproducts-1.0.0/

Am I using the html5 plugin wrong or is it a bug that a real URL is not showing here. Please note these applications are the sap-managed html5 applications that don't use their own app runtime but instead rely on the Portal or Launchpad service.
Environment

Plugin version (e.g. 1.3.0) - How can I determine this - there is no --version option? I installed it this week so it would be the latest version.
OS (macOS/Windows) - macOS
SAP CF Landscape (e.g. Canary) - Standard paid global account for customers
IaaS provider (e.g. AWS) - AWS
Region (e.g. eu10) - eu10

CLI Output
Set environment variable DEBUG=1, run the flow and paste output here.
❯ DEBUG=1 cf html5-list -d -u
Initializing command 'html5-list'
Executing command 'html5-list': args: '[-d -u]'
Listing HTML5 applications available via destinations
Getting context (org/space/username)
Getting list of HTML5 applications available via destinations in org SAP CP CustomerSuccessTeam_hanatest / space hana_cloud_demo as<EMAIL_ADDRESS>Getting list of services
Making request to: /v2/services
Looking for 'html5-apps-repo' service
Getting service plans for 'html5-apps-repo' service (GUID: 3d7c607f-8b21-484d-8b95-a87caf18b6e4)
Making request to: /v2/service_plans?q=service_guid:3d7c607f-8b21-484d-8b95-a87caf18b6e4
Looking for app-runtime service plan
Getting service instances of 'html5-apps-repo' service app-runtime plan (&{Name:app-runtime GUID:00b3a74e-aaa3-44a0-ad4c-1aece8a0d077})
Making request to: /v2/service_instances?q=service_plan_guid%20IN%2000b3a74e-aaa3-44a0-ad4c-1aece8a0d077%3Bspace_guid:15742f31-c052-4ec5-a2c5-39d942642f36
Creating service key for bookshop_html5_repo_runtime service
Making request to: /v2/service_keys
Getting token for service key html5-key-1602729555
Making request to: https://hanatest.authentication.eu10.hana.ondemand.com/oauth/token
Access token for service key html5-key-1602729555: <redacted>
Getting list of services
Making request to: /v2/services
Looking for 'destination' service
Destination service found: &{Name:destination GUID:cdc19796-58e8-47af-87d9-e72b84c3993f}
Getting service plans for 'destination' service (GUID: cdc19796-58e8-47af-87d9-e72b84c3993f)
Making request to: /v2/service_plans?q=service_guid:cdc19796-58e8-47af-87d9-e72b84c3993f
Destination service 'lite' plan found: &{Name:lite GUID:4d02c12c-3a82-4c84-8828-937e48d87511}
Getting service instances of 'destination' service 'lite' plan (&{Name:lite GUID:4d02c12c-3a82-4c84-8828-937e48d87511})
Making request to: /v2/service_instances?q=service_plan_guid%20IN%204d02c12c-3a82-4c84-8828-937e48d87511%3Bspace_guid:15742f31-c052-4ec5-a2c5-39d942642f36
Using service instance of 'destination' service 'lite' plan: {Name:FioriDemo-destination-service GUID:02c3cfeb-e447-45ef-b46b-82289d82fd51 UpdatedAt:2020-09-25T02:16:55Z LastOperation:{Type:update State:succeeded Description: UpdatedAt:2020-09-25T02:17:23Z CreatedAt:2020-09-25T02:17:23Z}}
Creating service key for 'destination' service 'lite' plan
Making request to: /v2/service_keys
Getting token for service key html5-key-1602729565
Making request to: https://hanatest.authentication.eu10.hana.ondemand.com/oauth/token
Access token for service key html5-key-1602729565: <redacted>
Making request to: https://destination-configuration.cfapps.eu10.hana.ondemand.com/destination-configuration/v1/subaccountDestinations/
List of subaccount destinations: [{Name:BallanceData Description:Ballance data Type:HTTP URL:https://ballancebackend.cfapps.eu10.hana.ondemand.com Authentication:NoAuthentication ProxyType: TokenServiceURL: TokenServiceURLType: ClientID: ClientSecret: Properties:map[ProxyType:Internet]} {Name:ES5 Description:SAP Gateway ES5 Type:HTTP URL:https://sapes5.sapdevcenter.com Authentication:BasicAuthentication ProxyType: TokenServiceURL: TokenServiceURLType: ClientID: ClientSecret: Properties:map[HTML5.DynamicDestination:true Password:<redacted> ProxyType:Internet User:I540401 WebIDEEnabled:true WebIDESystem:Gateway WebIDEUsage:odata_abap sap-client:002 sap-platform:ABAP]} {Name:Estimator Description: Type:HTTP URL:https://cloudplatform.sap.com Authentication:NoAuthentication ProxyType: TokenServiceURL: TokenServiceURLType: ClientID: ClientSecret: Properties:map[ProxyType:Internet WebIDEEnabled:true WebIDESystem:API]} {Name:Northwind Description:Automatically generated Northwind destination Type:HTTP URL:https://services.odata.org Authentication:NoAuthentication ProxyType: TokenServiceURL: TokenServiceURLType: ClientID: ClientSecret: Properties:map[HTML5.DynamicDestination:true ProxyType:Internet]} {Name:fioridemo_FioriDemo_html_repo_host Description: Type:HTTP URL:https://sap.com/DUMMY_URL Authentication:OAuth2ClientCredentials ProxyType: TokenServiceURL:https://hanatest.authentication.eu10.hana.ondemand.com/oauth/token TokenServiceURLType: ClientID:sb-00f01838-d2e7-4248-b190-166f4b360c65!b18738|html5-apps-repo-uaa!b1685 ClientSecret:<redacted> Properties:map[ProxyType:Internet ServiceKeyName:FioriDemo_html_repo_host-key app_host_id:00f01838-d2e7-4248-b190-166f4b360c65 content_endpoint:https://html5-apps-repo-dt.cfapps.eu10.hana.ondemand.com sap.cloud.service:fioridemo uri:https://html5-apps-repo-dt.cfapps.eu10.hana.ondemand.com]} {Name:fioridemo_uaa_FioriDemo Description: Type:HTTP URL:https://api.authentication.eu10.hana.ondemand.com Authentication:OAuth2UserTokenExchange ProxyType: TokenServiceURL:https://hanatest.authentication.eu10.hana.ondemand.com/oauth/token TokenServiceURLType: ClientID:sb-fioridemo!t18738 ClientSecret:<redaceted> Properties:map[ProxyType:Internet ServiceKeyName:uaa_FioriDemo-key sap.cloud.service:fioridemo verificationkey:-----BEGIN PUBLIC KEY-----<redacted>-----END PUBLIC KEY----- xsappname:fioridemo!t18738]} {Name:managed-fiori-destination-html5 Description: Type:HTTP URL:https://sap.com/DUMMY_URL Authentication:OAuth2ClientCredentials ProxyType: TokenServiceURL:https://hanatest.authentication.eu10.hana.ondemand.com/oauth/token TokenServiceURLType: ClientID:sb-04ae20c2-fbe5-4384-bd27-ea952dec4ff4!b18738|html5-apps-repo-uaa!b1685 ClientSecret:<redacted> Properties:map[ProxyType:Internet ServiceKeyName:managed-fiori-html5-repo-host-key app_host_id:04ae20c2-fbe5-4384-bd27-ea952dec4ff4 content_endpoint:https://html5-apps-repo-dt.cfapps.eu10.hana.ondemand.com sap.cloud.service:cloud.service uri:https://html5-apps-repo-dt.cfapps.eu10.hana.ondemand.com]} {Name:managed-fiori-destination-uaa Description: Type:HTTP URL:https://api.authentication.eu10.hana.ondemand.com Authentication:OAuth2UserTokenExchange ProxyType: TokenServiceURL:https://hanatest.authentication.eu10.hana.ondemand.com/oauth/token TokenServiceURLType: ClientID:sb-managed_fiori!t18738 ClientSecret:Yd7lS/SAgtRxXSc3qPwvo29cZew= Properties:map[ProxyType:Internet ServiceKeyName:managed-fiori-uaa-key sap.cloud.service:cloud.service verificationkey:-----BEGIN PUBLIC KEY-----<redacted>-----END PUBLIC KEY----- xsappname:managed_fiori!t18738]} {Name:python-test Description: Type:HTTP URL:https://python-test-docker.cfapps.eu10.hana.ondemand.com Authentication:NoAuthentication ProxyType: TokenServiceURL: TokenServiceURLType: ClientID: ClientSecret: Properties:map[ProxyType:Internet]}]
Deleting service key html5-key-1602729565
Making request to (try 1/3): /v2/service_keys/f0330f30-06d8-42f2-a104-94f74ad01b16
Deleting service key html5-key-1602729555
Making request to (try 1/3): /v2/service_keys/5228f0b6-4aed-4a7b-99e3-bffe7b1b2395
OK

name   version   app-host-id   service name    destination name                     last changed   url
-      -         -             fioridemo       fioridemo_FioriDemo_html_repo_host   -              https://sap.com/DUMMY_URL
-      -         -             fioridemo       fioridemo_uaa_FioriDemo              -              https://api.authentication.eu10.hana.ondemand.com
-      -         -             cloud.service   managed-fiori-destination-html5      -              https://sap.com/DUMMY_URL
-      -         -             cloud.service   managed-fiori-destination-uaa        -              https://api.authentication.eu10.hana.ondemand.com
 ~/dev/html5apps/managed-html5-runtime-fiori-mta  on fiori !5 ?7 

@js1972 , thanks for reporting an issue!
You can check the version of plugin with cf plugins command that lists all commands and version of corresponding plugin that exposed each command.
The issue you experience was fixed in v1.4.1.
You can download latest version (currently v.1.4.3) of html5-plugin for your operating system from assets section of Releases.
After you download the binary of html5-plugin, you can install it with cf install-plugin -f <path_to_binary>

I was already on 1.4.1 as it turns out, but updating to 1.4.3 fixes the issue. Brilliant. thx.


.jsx Does not support the   shift + alt + f    format code

VSCode Version: Code 1.5.2 (66f37fd2a99eb9d628dd374d81d78835b410c39b, 2016-09-12T13:30:03.456Z)
OS Version: Windows_NT ia32 6.1.7601

Steps to Reproduce:
1..jsx Does not support the   shift + alt + f    format code
2.

Reading your description it seems that the issue originates from an extension and not VS Code itself. Thereby I will close this issue and kindly ask you to report this issue against the extension repository.
Most extensions are on GitHub as well so it should be easy to file it against the extension. If you open an extension on the market place, click on the Support link to find out more.

Thanks again for your feedback!


Split gdax-java into library and client

There are three separate projects within this repo.

The library which facilitates connecting to the Coinbase Pro
The spring boot application the injects the configuration for each rest api call
The order book application that loads market data into a viewable interface

Attempting to break this apart this evening into a multi project build - will publish any commits here. Might not get all the way through but certainly feasible to break it apart.

https://github.com/irufus/gdax-java/tree/%2356_Split_gdax-java_into_library_and_client branch here

It also would be nice to split feed API. For example I need only it in my project

I have a working branch here: https://github.com/irufus/gdax-java/tree/%2356_Split_gdax-java_into_library_and_client but I'm still in full time work for the time being. Take a look - it may suit your needs.

please take alook on my PR  https://github.com/irufus/gdax-java/pull/60 which also have some dependencies cleanup

reviewed - just one item I can suggest. Good to see some library bumps

I will etch out some time this weekend to review

@robevansuk Does @stokito 's changes address similar things you were fixing in that branch?

I have a huge commit but somethings broken on the live orderbook which I've been looking at for the past week or two... I haven't managed to fix it yet but am working on this branch https://github.com/irufus/gdax-java/pull/62

Once this all works, we can then look at "publish to maven" plugins for the module that contains the library code

I found the problems and am in the process of fixing them. in a nut shell, the new JSON parsing doesn't deserialise the type field and this breaks the live orderbook which depends on this information. I'll repair the deserialisation code and test it again. If all works this can be merged and more incremental changes used to update the repo.

Yes, Jackson by default don't populate the type property. The logic still works because then we'll use class info itself and we don't need the type. If you want the type to be populated then we can add a flag into the Jackson annotation. But IMHO it would be better just to remove the field for two reasons:

Performance consideration: the string field takes quite a lot of memory and it's deserialization wastes resources.
Code stability: To avoid misusing of the field e.g. in if msg.type == 'MATCH', which can't be checked by compiler and force developed to use OOP i.e. if msg instance of MessageMatch

I've added changes to populate it but in all honesty my exposure to jackson isn't that great. It works for now but I need to look at the live orderbook before I can merge my changes. Once its working I'll merge it and look at simplifying afterwards

The GUI now removed. I'll work on publishing the libs to a maven repo next so that the lib can be used more freely


Any Update for vimeo audio player not working ?

I tried other package and the vimeo audio is working fine , is there any new update regarding this issue?

@Jefriluqman See this if it can help you - https://github.com/newtaDev/pod_player/issues/188

same issue here,, any plan to fix this?

same issue here,, any plan to fix this?

As for now you can follow this method https://github.com/newtaDev/pod_player/issues/188


switch test runner to coverage

Test coverage is 64% as of this commit -- going to add additional tests to get this to at least 80%.

As of this last commit, test coverage of the codebase is at 89 percent. The key functionalities (Descriptor, SimilarityMeasure, Molecule, and MoleculeSet) are among the most thoroughly tested. Merging.


fix(assets api): import replaces dashboard

SUMMARY

For existing dashboards, modifications imported via /api/v1/assets/import/ are only appended if the modified dashboard contains a new chart. Modifications to dashboards that remove a chart are not implemented when imported through the assets API.
This change updates the assets API import method to remove all existing references to chart ids for the imported dashboard(s) before inserting the new chart ids. The change should benefit the management of Superset assets under source control.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

Remove a chart from a dashboard:

Export a dashboard and unzip assets bundle.
Remove a chart and all references to it from the exported yaml files
Import modified assets bundle as a zip by posting to /api/v1/assets/import/ (see https://superset.apache.org/docs/api/#api)
Navigate to the modified dashboard in the Superset UI and observe the deleted chart is no longer visible

Add a chart to a dashboard:

In the same dashboard, remove a chart using the UI and save the dashboard
Import the previously exported assets bundle as a zip by posting to /api/v1/assets/import/
Navigate to the modified dashboard in the Superset UI and observe the deleted chart is now reinstated

ADDITIONAL INFORMATION

[x] Has associated issue: https://github.com/apache/superset/issues/22127
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Can we add a unit test for this behavior? there should already be some for import/export you can expand on

Sure, I'll add a test 👍


v-for and a tag issue

Code:
{{ type }}

Screenshot:

The problem should be fairly obvious.

type is a reserved word for TS. We are using TS grammar for JS and this isn't easily fixable. I recommend avoid naming parameters as type.

OK, good to know.


Deploy addition of Kubernetes manifests

Signed-off-by: Dipankar Das<EMAIL_ADDRESS>
Fixes Issue

Changes proposed
Addition of Kubernetes Resources

Screenshots

Note to reviewers

Is there anything to add more?

/hold


How can I execute runs with exististing thread

`public function postMessage($user, $content, $role)
{
$this->getOrCreateThreat();
$now = Carbon::now();
//create user content
$response = $this->openAIClient->threads()->messages()->create($this->threadId, [
'role' => $role,
'content' => "$now:::\n\n {$content}",
    ]);
    foreach ($response->content as $content) {
        $message = new Message;
        $message->role = $response->role;
        $message->content = $content->text->value;
        $user->messages()->save($message);
    }

    //run thread
    $response = $this->openAIClient->threads()->?
}`

I don't find a function to run existing threads in the documentation

EDIT: Ok I find $this->openAIClient->threads()->runs()->create how can I use this?

Hi @kaemmi84
You need to provide the thread_id. See: https://github.com/openai-php/client/tree/0.8.x#threads-runs-resource-beta
Hope this helps. Otherwise, let me know.

Thanks but the question is how actuality the frontend. whithout waiting on the post endpoints. Work this with websockets?


Problem in update S95majestic. Version  dev+12721b4, 2023-12-09

No start daemon after boot camera.
Manual start.

root@openipc-hi3516cv300:~# /etc/init.d/S95majestic start
hisilicon: Loading video system has started...
Starting majestic: start-stop-daemon: option requires an argument: s
BusyBox v1.36.1 (2023-12-12 07:25:40 UTC) multi-call binary.
Usage: start-stop-daemon [OPTIONS] [-S|-K] ... [-- ARGS...]
Search for matching processes, and then
-K: stop all matching processes
-S: start a process unless a matching process is found
Process matching:
-u USERNAME|UID	Match only this user's processes
-n NAME		Match processes with NAME
in comm field in /proc/PID/stat
-x EXECUTABLE	Match processes with this command
in /proc/PID/cmdline
-p FILE		Match a process with PID from FILE
All specified conditions must match
-S only:
-x EXECUTABLE	Program to run
-a NAME		Zeroth argument
-b		Background
-N N		Change nice level
-c USER[:[GRP]]	Change user/group
-m		Write PID to pidfile specified by -p
-K only:
-s SIG		Signal to send
-t		Match only, exit with 0 if found
Other:
-o		Exit with status 0 if nothing is done
-v		Verbose
-q		Quiet
FAIL

Problem in DAEMON_ARGS="-s"

try to update from dev repo and see if it works now.


.blend import on armatures does not account for rest position

Godot version
4.0 Alpha 6
System information
Windows 10 Nvidia GTX 1060 Intel i5 4690k
Issue description
If you import an fbx file into a .blend file, save it, then import it into Godot, the animation works fine.
However, the armature/skeleton does not retain the rest position, despite the .blend file still having the original rest position.

The only way to preserve the rest is to  have the .blend saved with the rest pose set, however, this then removes the animation
Steps to reproduce

Import any animation into blender (fbx file)
Set the pose of the armature to pose position
Save the .blend file
Import the .blend file into Godot
Open the file in godot
Click show rest pose only on the skeleton.
Voila.

Minimal reproduction project
MissingRestPose.zip

FBX to gltf exports use the first frame of the first animation. We worked around it using a RESET animation track in Godot.
There may still be a bug in fbx2gltf https://github.com/V-Sekai/FBX2glTF/issues/1 related to the first frame of gltf.

Blend files once converted from the native format uses the gltf exporter. @julienduroure Did I get this description correct?

I just proposed this PR in glTF-Blender-IO to manage REST pose as default TRS pose for joints node:
https://github.com/KhronosGroup/glTF-Blender-IO/pull/1621

Merged into Blender 3.2. Closing.


Request: Slightly better usage example

Hey y'all.
The readme only has this for usage:
pip install sarif-om
import sarif_om

I'm reading through the microsoft sarif tutorials here:
https://github.com/microsoft/sarif-tutorials/blob/main/docs/3-Beyond-basics.md
And trying to figure out how I could use this class to convert the issues found in bad-eval-with-code-flow.py and output the file bad-eval-with-code-flow.sarif
It would be much easier to jump into this project if there was at least one example.
Anyways, thanks for supporting python!

Really, any documentation at all for usage would be nice.

Dropping a +1 here. If we want SARIF to have more stickiness, then we definitely have to have better tutorials and instructions on how to use it. I maintain a few open-source security projects of my own, and my company has more, even if I started a project to convert to SARIF output options tomorrow it would be very difficult.

@dindonero  All you really need to serialize is the following:
import sarif_om
from jschema_to_python.to_json import to_json

sarif_log = sarif_om.SarifLog(
    schema_uri='https://raw.githubusercontent.com/oasis-tcs/sarif-spec/master/Schemata/sarif-schema-2.1.0.json',
    ...
)

to_json(sarif_log)

jschema_to_python will automatically use the schema_property_name metadata of the fields to map to the schema names.

Thanks to @melsabagh-kw for the serialization example, is there also some simple de-serialization example showing how to read sarif files creating the related OM?

@melsabagh thanks for the input. I am trying to generate JSON from and getting following error. Would you know what might be the issue?
Traceback (most recent call last):
  File "/Users/user1/github/my-project/./my-project.py", line 159, in <module>
    main()
  File "/Users/user1/github/my-project/./my-project.py", line 151, in main
    generate_report(data=data)
  File "/Users/user1/github/my-project/./my-project.py", line 44, in generate_report
    to_json(report)
  File "/Users/user1/.local/share/virtualenvs/my-project-2_gkca5i/lib/python3.11/site-packages/jschema_to_python/to_json.py", line 20, in to_json
    return json.dumps(obj, indent=2, default=_generated_class_serializer)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/user1/.pyenv/versions/3.11.4/lib/python3.11/json/__init__.py", line 238, in dumps
    **kw).encode(obj)
          ^^^^^^^^^^^
  File "/Users/user1/.pyenv/versions/3.11.4/lib/python3.11/json/encoder.py", line 202, in encode
    chunks = list(chunks)
             ^^^^^^^^^^^^
  File "/Users/user1/.pyenv/versions/3.11.4/lib/python3.11/json/encoder.py", line 439, in _iterencode
    o = _default(o)
        ^^^^^^^^^^^
  File "/Users/user1/.local/share/virtualenvs/my-project-2_gkca5i/lib/python3.11/site-packages/jschema_to_python/to_json.py", line 26, in _generated_class_serializer
    dict = copy.deepcopy(dict)
           ^^^^^^^^^^^^^^^^^^^
  File "/Users/user1/.pyenv/versions/3.11.4/lib/python3.11/copy.py", line 161, in deepcopy
    rv = reductor(4)
         ^^^^^^^^^^^
TypeError: cannot pickle 'mappingproxy' object

@maratsal my guess is you have a custom property in your SARIF that uses a dataclass.field with a dict factory.

I think https://github.com/microsoft/sarif-python-om/pull/6 will really ease the use of this library since it provides much better type hint and a better way to serialize/deserialize SARIF report.
Hope that it can be merged soon!


Add small script for linting and testing functionality

Lol I have been looking for the K8s PM (thank you Flavio) for a few days because I wanted to contribute to some ORCL Kubernetes stuff (lot of ppl asking for help with it online) and I found this repo! Hope this small script helps some :)


Saving files limited to 100 kb.

Operation System: macOS 10.13.6
Beaker version:0.8.0
Imported new TiddlyWiki (empty.html) opens and works. In Library it Shows:
390 lines ( actually 11218 lines )
2.08 mb  ( accurate )
When edited and saved:
390 lines ( all lines after 390 missing )
100 kb ( accurate )
I tried an HTML full of lorum ipsum...
330 kb, 12000 lines, saved to 100.003 kb and 875 lines.

I can definitely confirm that the lines are getting cut, but I am not seeing a loss in file size. I created a file with ~2500 lines equaling 702.47kB. It is now down to ~339 lines but still ~702kB after save.
Will look into the issue on lines cutting off, and decreasing file size should I end up seeing that myself.

Also seeing the loss of file size. I guess it takes a moment to update that.

Fixed, thanks @DanielRJansen for reporting and @Septimus for fixing this.


Update Condition on detect_processes_used_for_system_network_configuration

Except user is NULL or unknown. It will decrease false detection

Hello,
Thank you for your contribution, since there are SPL changes in your: we would like to get them tested via our detection testing service. I create a duplicate PR https://github.com/splunk/security_content/pull/1545


Like Comments function

As a user I want to like a comment with a button (like thumbs-up) and be able to unlike it again. Similar functionality to liking a post.
Acceptance criteria:

When I press on the like button, the number increments with 1

When I press again on the button, it reduces with 1 ("unlike")

There should be a count in the database how much likes a comment has, in order to make a sort function based on most likes.

Verified with Firefox 54.0 & Safari 9.1.2 on Mac OS 10.11.6.


Add MDC entry writers to be able to convert values

Solution proposal for issue https://github.com/logfellow/logstash-logback-encoder/issues/955

Thanks for the quick turnaround on my review comments!  This will be included in the next release (soon)


(CAT-1351) - Fixing unsafe interpolation puppet-linting

Summary
Recently rolled-out puppet-lint gem checking title as well for unsafe interpolation which seems to be a issue with feature so, downgrading the gem version to avoid the unsafe interpolation issue for title.
Have reported the same to Team.
Additional Context
Add any additional context about the problem here.

[x] Unsafe interpolation issue reported by puppet-lint (https://github.com/puppetlabs/puppet-lint/pull/142)

Related Issues (if any)

N/A

Checklist

[ ] 🟢 Spec tests.
[ ] 🟢 Acceptance tests.
[ ] Manually verified. (For example puppet apply)

The title look wrong: this does not "fix" something, rather "disable" or "revert" it.  @Ramesh7, do you mind adjusting it so that when going through the changelog people better understand what is going on?


Pick pick #28860 "not is null" stats estimation fix

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

#28860


Gender Orientation needs to be expanded

Right now we only have three gender designations. Facebook has 51, for example. We should expand this. Here's a guide/list:
http://itspronouncedmetrosexual.com/2013/01/a-comprehensive-list-of-lgbtq-term-definitions/

Do we really want to give our editors a choice of 51 gender identifiers to specify essentially if a subject is male female or trans? I think it could be much more confusion than it's worth.  Even if we decided that it was worth doing.  This kind of detailed info has not been available in any of the several hundred cases I have already input.
I think this one should be on the back burner.

Facebook does it and it wouldn't take long.
On Mar 22, 2016 11:10 PM, "campgurus"<EMAIL_ADDRESS>wrote:

Do we really want to give our editors a choice of 51 gender identifiers to
specify essentially if a subject is male female or trans? I think it could
be much more confusion than it's worth. Even if we decided that it was
worth doing. This kind of detailed info has not been available in any of
the several hundred cases I have already input.
I think this one should be on the back burner.
—
You are receiving this because you were assigned.
Reply to this email directly or view it on GitHub
https://github.com/BOWiki/BOW/issues/251#issuecomment-200146644

Also, here's a case on our site
http://ebwiki.org/articles/rickie-shawatza-hall
On Tue, Mar 22, 2016 at 11:35 PM, Mark Nyon<EMAIL_ADDRESS>wrote:

Facebook does it and it wouldn't take long.
On Mar 22, 2016 11:10 PM, "campgurus"<EMAIL_ADDRESS>wrote:

Do we really want to give our editors a choice of 51 gender identifiers
to specify essentially if a subject is male female or trans? I think it
could be much more confusion than it's worth. Even if we decided that it
was worth doing. This kind of detailed info has not been available in any
of the several hundred cases I have already input.
I think this one should be on the back burner.
—
You are receiving this because you were assigned.
Reply to this email directly or view it on GitHub
https://github.com/BOWiki/BOW/issues/251#issuecomment-200146644

--
Mark<EMAIL_ADDRESS>blog.grandkru.com
github.com/trystant

And two more:
http://ebwiki.org/articles/duanna-johnson
http://ebwiki.org/articles/kayla-moore

All of these cases are covered within the 3 designations we already have.  I don't think this is necessary or a good idea.

I don't see any way this would be less than a "good" idea. The three designations remain, and other options are available. It would take less time  to add those than we've spent going back and forth about this.
In any case, it's off your plate. I'll do it. It'll be my first priority.

This is now in production.


[82,22] S3 download/upload support for xlsx,xlsm and geojson

S3 download/upload support for xlsx,xlsm and geojson
This PR closes the issues: 82 and 22.
Description
S3 download/upload support for xlsx,xlsm and geojson.
Added:

Updates to S3.py file for uploading/downloading xlsx,xlsm and geojson.
Updated setup with new package requirements.
Added tests for uploading and downloading the files.
Added dummy xlsx,xlsm and geojson files.

Checklist:

[x] I have followed Contributor Guidelines
[ ] I have updated the documentation to include any new functionality I have added or modified
[x] I have written unit tests for any new functionality I have added
[x] I have ran and passed all tests locally with >> pytest
[x] I have checked that all tests ran successfully on the Github after I pushed

Hello both! I have updated the files based on your comments :)
Things to note:

@sqr00t pandas 1.5.1 doesn't allow extras, so I went back to your initial idea!
Separated the upload/download for geodataframes into separate functions.
Noticed a few spelling errors, so fixed those!
Removed from xmlrpc.client import Boolean for S3.py as I couldn't see where it was used?

For adding geojson upload/download from dictionaries, I couldn't find a way to check typing for geojson specifically. As from my understanding of the docs, it is a json with specific formatting. So what I did was add a check to see if it has a "type" member and that member is one of the accepted types based on the specification in section 3. If anyone knows of a better way I am happy to update!


CLI should validate that project has been "init"ed

Weights and Biases version: 0.6.34
Python version: 3.6.4
Operating System: Linux

Description
Attempting to run wandb sweep without initializing the project.
Also specifying dryrun which probably shouldn't work with sweeps.
What I Did
$ WANDB_MODE=dryrun wandb sweep sweep.yaml
Creating sweep from: sweep.yaml
wandb: Network error (HTTPError), entering retry loop. See wandb/debug.log for full traceback.

The failure is because entity was not set.

Fixed in 0.7.1


Digiminer crash

digiminers are still causing crashes using mekanism 8.0.201
http://pastebin.com/tPcsykMG

Yes, this is fixed in development

ah /= no matter how many times i tell my players, they go and put on inverse mode and now the whole servers broken again

is there a way to disable the inverse option server side?

v8.0.3 is out now :)


Uncoercable attributes should not make a form explode

In rom-rails, if a form has an integer attribute:
class InputForm < ROM::Model::Form
  #...
  input do
    attribute :total, Integer
  end
  # ...
end

Assigning an uncoercable value to the attribute  InputForm.build(total: "two hundred") will raise an error.  Validations never have a chance to filter and handle this cleanly.  This is undesirable behavior for user facing input.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/9555774-uncoercable-attributes-should-not-make-a-form-explode?utm_campaign=plugin&utm_content=tracker%2F155789&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F155789&utm_medium=issues&utm_source=github).

Yeah, I had second thoughts about this myself. We added strict mode by default but it's too aggressive. This is that moment where coercion and validation are really close to each other.
I think you're right - we should turn strict mode off.

fixed in https://github.com/rom-rb/rom-rails/commit/6dd06dc4b280e991c27e1f152335bef6f0d74b7c


set path before running tox

This should fix the problem described in #615.
The default interpreter should be the one used to run Tox (https://tox.readthedocs.org/en/latest/config.html?highlight=interpreter#confval-basepython=NAME-OR-PATH) but to be sure, set PATH to contain the correct interpreter. On Windows Tox also searches there (https://github.com/msabramo/tox/blob/master/tox/interpreters.py#L121).

Current coverage is 99.27%

Merging #620 into master will not affect coverage as of c7199c9

@@            master    #620   diff @@
======================================
  Files           12      12
  Stmts          549     549
  Branches         0       0
  Methods          0       0
======================================
  Hit            545     545
  Partial          0       0
  Missed           4       4

Review entire Coverage Diff as of c7199c9

Uncovered Suggestions

+0.36% via ...okiecutter/config.py#19...20
+0.36% via ...ookiecutter/utils.py#27...28

Powered by Codecov. Updated on successful CI builds.

PS: We would remove 32 Bit Interpreters. Using 32 Bit on Travis and 64 Bit on Appveyor should be sufficient for cookiecutter I guess or even completely drop 64 Bit testing to speed tests up!

LGTM :+1:


pass-secret-service doesn't build

Describe the bug
nix-build -A pass-secret-service doesn't work anymore. The tests show errors and take forever (or are stuck).
Steps To Reproduce
Steps to reproduce the behavior:

switch to master branch in nixpkgs
nix-build -A pass-secret-service

Expected behavior
The build should finish successfully.
Screenshots
running install tests
===== Preparing gpg test keys in test/.gnupghome =====
mkdir -p -m 700 test/.gnupghome
gpg --allow-secret-key-import --import test/test_key.asc
gpg: keybox '/build/source/test/.gnupghome/pubring.kbx' created
gpg: /build/source/test/.gnupghome/trustdb.gpg: trustdb created
gpg: key CF3AF3078C2A59A7: public key "pass_secret_service test key (INSECURE TEST KEY)" imported
gpg: key CF3AF3078C2A59A7: secret key imported
gpg: Total number processed: 1
gpg:               imported: 1
gpg:       secret keys read: 1
gpg:   secret keys imported: 1
gpg --import-ownertrust test/test_ownertrust.txt
gpg: inserting ownertrust of 6
===== Preparing password store in test/.test-password-store =====
pypass init -p test/.test-password-store "8c2a59a7"
Password store initialized for 8c2a59a7.
dbus-run-session --config-file=/nix/store/6bg6qs8r26c044xgkzqyk4d0im81sa02-dbus-1.12.20/share/dbus-1/session.conf -- pytest -v test
============================= test session starts ==============================
platform linux -- Python 3.9.6, pytest-6.2.5, py-1.10.0, pluggy-1.0.0 -- /nix/store/8dxxjbiyxwkvh53q5kh6nydla2anacgi-python3-3.9.6/bin/python3.9
cachedir: .pytest_cache
rootdir: /build/source
plugins: asyncio-0.15.1
collected 23 items

test/test_collection.py::TestCollection::test_create_delete_collection ERROR [  4%]
test/test_collection.py::TestCollection::test_properties ERROR           [  8%]
test/test_collection.py::TestCollection::test_lock_unlock ERROR          [ 13%]
test/test_collection.py::TestCollection::test_persisted_item FAILED      [ 17%]
test/test_items.py::TestCollection::test_create_delete_item ERROR        [ 21%]
test/test_items.py::TestCollection::test_get_and_set_secret ERROR        [ 26%]
test/test_items.py::TestCollection::test_item_properties ERROR           [ 30%]
test/test_items.py::TestCollection::test_broken_session_path ERROR       [ 34%]
test/test_items.py::TestCollection::test_item_lookup ERROR               [ 39%]
test/test_items.py::TestCollection::test_item_in_deleted_collection ERROR [ 43%]
test/test_items.py::TestCollection::test_replace_item ERROR              [ 47%]

Additional context
Add any other context about the problem here.
Notify maintainers

/cc @jluttine
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.14.14-200.fc34.x86_64, Fedora, 34 (Workstation Edition)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.16`
 - channels(root): `""`
 - nixpkgs: `/home/cassou/Documents/projects/nix-system/nixpkgs`

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:

Ping ZHF https://github.com/NixOS/nixpkgs/issues/144627

ran checkPhase in nix develop and cancelled the check when it hung and got these
========================================================== short test summary info ==========================================================
FAILED test/test_collection.py::TestCollection::test_persisted_item - Exception: could not find .gpg-id file
ERROR test/test_collection.py::TestCollection::test_create_delete_collection - Exception: could not find .gpg-id file
ERROR test/test_collection.py::TestCollection::test_properties - Exception: could not find .gpg-id file
ERROR test/test_collection.py::TestCollection::test_lock_unlock - Exception: could not find .gpg-id file
ERROR test/test_items.py::TestCollection::test_create_delete_item - Exception: could not find .gpg-id file
ERROR test/test_items.py::TestCollection::test_get_and_set_secret - Exception: could not find .gpg-id file
ERROR test/test_items.py::TestCollection::test_item_properties - Exception: could not find .gpg-id file
ERROR test/test_items.py::TestCollection::test_broken_session_path - Exception: could not find .gpg-id file
ERROR test/test_items.py::TestCollection::test_item_lookup - Exception: could not find .gpg-id file
ERROR test/test_items.py::TestCollection::test_item_in_deleted_collection - Exception: could not find .gpg-id file
ERROR test/test_items.py::TestCollection::test_replace_item - Exception: could not find .gpg-id file
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! KeyboardInterrupt !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

I don't know how, but I think this is somewhat a critical bug that hangs nix when building any fish shell completion derivation. At this moment I just can't build packages like megasync or exodus, because these packages have some post build hooks that install fish shell completion for them. Then, when I retry the same build command, I get that exact same error reported by DamienCassou.

pass-secret-service is not a dependency to any package so it shouldn't be causing issues with other packages
You must have pass-secret-service in your config and that's why it's freezing

I'm also affected by this - and it is indeed "critical" in the sense that nixos-rebuild never completes building the system derivative anymore.
If there's a patch to be tested, I'd be happy to assist.

i'll take a look but at worst we can disable the tests
for now you can do that locally with
  nixpkgs.overlays = [
    (self: super: {
      pass-secret-service = super.pass-secret-service.overridePythonAttrs (oldAttrs: {
        doCheck = false;
      });
    })
  ];


✔Implemented Custom Vertical scrollbar in DotBox

Hey @ayush-t02
issue closes #752
I have implemented the styled vertical scrollbar in all the Scrollable Pages

Made Styling of thescrollbar to be Gaming theme with
Added scrollbar where it was not present at all - testimonials, about

Please take a look -
https://github.com/user-attachments/assets/18644873-4316-4b9e-9b6e-557e961cb4f5
Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for review.

## Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

@ayush-t02 Please review it.


支持使用密码登陆的服务器脚本部署

错误提示
Pseudo-terminal will not be allocated because stdin is not a terminal.
Warning: Permanently added '<IP_ADDRESS>' (ED25519) to the list of known hosts.

解决方案
"Pseudo-terminal will not be allocated" 可以通过添加 -t 参数强制分配伪终端来解决。
"Permanently added '<IP_ADDRESS>' to the list of known hosts" 是 SSH 安全机制的正常提示，可以忽略，或者通过 -o UserKnownHostsFile=/dev/null 禁用该提示。


Use a supported jsdoc template

Today in our JavaScript packages, we're using this template:
"jsdoc-baseline": "git+https://github.com/hegemonic/jsdoc-baseline.git",

Taking a look at the repo, it looks to be ... not maintained.  We are starting to see npm audit advisories for out of date transitive dependencies.  We need a plan to either:

maintain the template we're using or
pick one that's well supported

@jmdobry @jkwlui thinking y'all have more context than me here :)  @bcoe for FYI.

+1. jsdoc-baseline was great because it's easily customisable to cloud.google.com.
We should move to a one with better support once we move away!

Yes, hegemonic/jsdoc-baseline is what enables the Node.js reference docs to live on cloud.google.com. Once you move to your new site, use whatever template you like. I imagine hegemonic@ would be willing to publish an update for you or make you a collaborator so you can do it.

@hegemonic 👋 Looking for any contributors to https://github.com/hegemonic/jsdoc-baseline?  Or is there a different basic template we should be using?

I’m catching up on a lot of JSDoc maintenance right now, and jsdoc-baseline is next on my list. I’ll update its dependencies in the next week or so.
As requested by @bcoe, I’ll also publish the package to npm, so you don’t have to keep pulling it in directly from GitHub.

Just to be clear - I'm more than happy to help here if you're looking for maintainers!

@hegemonic :+1: very much appreciated; and like @JustinBeckwith says, please let us know if there's any way we can help.

OK, I just published version 0.1.0 of the template to npm: https://www.npmjs.com/package/jsdoc-baseline
Version 0.1.0 uses up-to-date dependencies with no vulnerabilities. It works with Node.js >=8, and it expects "jsdoc": "^3.6.2" as a peer dependency.
Thanks for the offers of assistance! I think I'm in good shape, but I'll let you know if I fall behind again and need a hand.

Thanks @hegemonic!

We're all good here now :)


Add CocoaPods support

This PR adds a podspec file so that the library can be integrated using cocoapods.

Cool, thanks!


Add url validation rule

Hi!
Add url validation rule and test related to issue #3
Please, tell me if something has to be improved.
Thanks!

Hey, @alibori! Thanks for the pull request!
@brendt, are you aware of any current reasons we shouldn't use filter_var for URL validation? Both Laravel and Symfony utilize the same regex pattern to validate URLs, but that might be because of the historical problems with filter_var, not any current ones.

So... It looks like filter_var still has some unresolved problems. For example, this issue is still a thing which returns true for a string like "https://https://www.github.com".
@alibori - Do you mind updating this rule to utilize this Regex pattern Symfony is using and credit them accordingly?

Hi @aidan-casey !
Of course, take a look now 😄
Thanks!

Thank you, @alibori!


Add Puzzle bot badge to README. And mvn ignores README changes

@Graur, Can you take a look pls?

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 3min)


Cascade config loading provokes unwanted behaviour with npm link

When a linked package from npm link is used, the postcss config is searched for into the link tree and not the local one.
Let's imagine we have this two npm repos we're working on

custom-css-library at path/to/custom-css-library, which is published on npm with the same name
some-js-app at another-path/some-js-app
We have a postcss config file at some-js-app/postcss.config.js which sould apply to any post-css processing happening with a local entrypoint (as per the docs https://github.com/postcss/postcss-loader#config-cascade)

We want to use custom-css-library in some-js-app without having to republish / reinstall every time custom-css-library changes so we do
cd path/to/custom-css library && npm link
cd another-path/some-js-app && npm install custom-css-library && npm link custom-css-library

(per the docs: https://docs.npmjs.com/cli/link.html)
After the base postcss with sass on webpack, we get
    ERROR in ./custom-css-library/main.scss (./node_modules/css-loader/dist/c
js.js!./node_modules/postcss-loader/src!./node_modules/sass-loader/dist/cjs.js!./node_modules/custom-css-library/main.scss)
    Module build failed (from ./node_modules/postcss-loader/src/index.js):
    Error: No PostCSS Config found in: some-js-app/node_modules/custom-css-library

Instead, wouldn't we expect to have some-js-app/postcss.config.js loaded ?, Maybe by appending by default the base  folder  where webpack is executed in the $PATH that the loader looks for the config file in ?

Now you can set absolute path to config (not directory), it should solve the problem, anyway if you still have a problem, please provide reproducible test repo, loading logic inside cosmiconfig package, so we can't manage it, maybe you need open the issue inside cosmiconfig, other solution is creating own utils for loading configuration, it is out of scope postcss-loader


converting from AY8910 to YM2608, the key of FM part is lowered

Converting from AY8910 to YM2608, the key of FM part is lowered.

vgm-conv converts the YM2608 clock by default, assuming 7987200 Hz.
If you have given a clock other than that, specify the clock with the '-c' option when converting.


Bigtable from Dataproc: Guava error trying to instantiate Bigtable client

Environment details
We're running a Dataproc cluster with the 1.3.43-debian9 image, Spark version 2.3.3, Scala version 2.11.8, and Java version 1.8.0_222. We've reproduced this using the google-cloud-bigtable jar at version 0.95-alpha and 0.110.0, as well as the bigtable-hbase-1.x jar at version 1.9.
Steps to reproduce

Build an assembled jar containing the google-cloud-bigtable jar; upload it to a GCP bucket.
Configure a dataproc cluster with Bigtable
ssh to the master node of a dataproc cluster.
Start spark-shell using the JAR option to include your jar on the classpath.
Run:

scala> import com.google.cloud.bigtable.data.v2.BigtableDataClient
scala> val bigTable = BigtableDataClient.create(projectId, instanceId)

Stack trace
java.lang.NoSuchMethodError: com.google.common.base.Preconditions.checkArgument(ZLjava/lang/String;J)V
  at com.google.api.gax.batching.BlockingSemaphore.checkNotNegative(BlockingSemaphore.java:40)
  at com.google.api.gax.batching.BlockingSemaphore.<init>(BlockingSemaphore.java:44)
  at com.google.api.gax.batching.FlowController.<init>(FlowController.java:168)
  at com.google.api.gax.rpc.BatchingCallSettings.<init>(BatchingCallSettings.java:70)
  at com.google.api.gax.rpc.BatchingCallSettings.<init>(BatchingCallSettings.java:46)
  at com.google.api.gax.rpc.BatchingCallSettings$Builder.build(BatchingCallSettings.java:146)
  at com.google.cloud.bigtable.data.v2.stub.EnhancedBigtableStubSettings.<init>(EnhancedBigtableStubSettings.java:
148)
  at com.google.cloud.bigtable.data.v2.stub.EnhancedBigtableStubSettings.<init>(EnhancedBigtableStubSettings.java:
81)
  at com.google.cloud.bigtable.data.v2.stub.EnhancedBigtableStubSettings$Builder.build(EnhancedBigtableStubSetting
s.java:626)
  at com.google.cloud.bigtable.data.v2.BigtableDataSettings.<init>(BigtableDataSettings.java:61)
  at com.google.cloud.bigtable.data.v2.BigtableDataSettings.<init>(BigtableDataSettings.java:57)
  at com.google.cloud.bigtable.data.v2.BigtableDataSettings$Builder.build(BigtableDataSettings.java:210)
  at com.google.cloud.bigtable.data.v2.BigtableDataClient.create(BigtableDataClient.java:128)
  at com.squarespace.databases.mumu.BigTable.getClient(BigTable.scala:34)
  ... 49 elided

External references such as API reference guides used

https://cloud.google.com/bigtable/docs/reference/libraries

Any additional information below
Happy to provide any extra information desired.

Hi @Gastove, The reason build this NoSuchMethodFound exception is guava version pulling in from spark classpath.
One workaround for that would be to shade your assembled jar with prefixing your organization name. In your pom.xml:
<plugin>
    <groupId>org.apache.maven.plugins</groupId>
    <artifactId>maven-shade-plugin</artifactId>
    <version>3.2.1</version>
    <configuration>
        <shadedArtifactAttached>true</shadedArtifactAttached>
        <relocations>
            <relocation>
                <pattern>com.google.common</pattern>
                <shadedPattern>com.shaded.google.common</shadedPattern>
            </relocation>
        </relocations>
    </configuration>
    <executions>
        <execution>
            <phase>package</phase>
            <goals>
                <goal>shade</goal>
            </goals>
        </execution>
    </executions>
</plugin>

@rahulKQL that's interesting -- I checked for that. I am, admittedly, using SBT, but I'm also using the SBT dependency tree plugin, and I verified that no earlier versions of Guava were on my classpath. I also loaded the spark-shell with my uberjar attached, and verified that I could, indeed, import the needed Precondition class, and that the checkArgument function was present.
So! I don't doubt you; Guava version conflicts are one of my oldest and least favourite kinds of transitive dependency problems. My question is: do you have any idea which dependency might be causing this, or any suggestion for how I could pin down the runtime classpath myself?

@Gastove Thanks for ack.
By simply listing the classloader shows this dependency comes from Hadoop in my case:
val cl = java.lang.ClassLoader.getSystemClassLoader;
val urls:[java.net.URL] = cl.asInstanceOf[java.net.URLClassLoader].getURLs;
urls.filter(x => x.toString.matches(".*guava.*")).foreach(println);

I am not sure if you are already doing this but, here is the link of the solution to a similar issue faced by another user.

Closing this issue due to inactivity, If still facing the issue please reopen it.

I am facing similar issues - More details in this https://github.com/googleapis/java-bigtable/issues/1789


Post http://*:8000/v1/hooks/build: EOF

Hello!
Whenever I push something to Deis, I get this strange error message
2015/04/18 04:07:33 Post http://<IP_ADDRESS>:8000/v1/hooks/build: EOF

I recently upgraded from 1.5.0 to 1.5.1. Prior to that I had the issue described in #3458, which seemed to disappear after the upgrade.
No error messages in deisctl journal builder and all of the Deis components are running according to deisctl list.
What could be causing this?

this is a post to the controller. Is there something in the controller logs?

Apr 18 04:27:41 deis-1 sh[2651]: 2015-04-18T04:27:41Z a08db6f54de7 confd[99]: ERROR template: confd_settings.py:21:91: executing "confd_settings.py" at <getv "/deis/registry...>: error calling getv: key does not exist

Times ∞

that means the registry is down. Time to follow the breadcrumbs :bread:
What's the status of the registry?

It's active and running. It is complaining in the logs, though.
Apr 18 04:27:41 deis-2 sh[23543]: 2015-04-18T04:27:41Z cb066ac10b17 confd[49]: ERROR template: create_bucket:8:32: executing "create_bucket" at <getv "/deis/store/ga...>: error calling getv: key does not exist
Apr 18 04:27:41 deis-2 sh[23543]: 2015-04-18T04:27:41Z cb066ac10b17 confd[49]: ERROR template: config.yml:103:18: executing "config.yml" at <getv "/deis/store/ga...>: error calling getv: key does not exist
Apr 18 04:27:41 deis-2 sh[23543]: 2015-04-18T04:27:41Z cb066ac10b17 confd[49]: ERROR template: config.yml:103:18: executing "config.yml" at <getv "/deis/store/ga...>: error calling getv: key does not exist
Apr 18 04:27:41 deis-2 sh[23543]: 2015-04-18T04:27:41Z cb066ac10b17 confd[49]: ERROR template: create_bucket:9:31: executing "create_bucket" at <getv "/deis/store/ga...>: error calling getv: key does not exist
Apr 18 04:27:41 deis-2 sh[23543]: 2015-04-18T04:27:41Z cb066ac10b17 confd[49]: ERROR template: config.yml:104:18: executing "config.yml" at <getv "/deis/store/ga...>: error calling getv: key does not exist
Apr 18 04:27:41 deis-2 sh[23543]: 2015-04-18T04:27:41Z cb066ac10b17 confd[49]: ERROR template: create_bucket:9:31: executing "create_bucket" at <getv "/deis/store/ga...>: error calling getv: key does not exist
Apr 18 04:27:41 deis-2 sh[23543]: 2015-04-18T04:27:41Z cb066ac10b17 confd[49]: ERROR template: create_bucket:9:31: executing "create_bucket" at <getv "/deis/store/ga...>: error calling getv: key does not exist
Apr 18 04:27:41 deis-2 sh[23543]: 2015-04-18T04:27:41Z cb066ac10b17 confd[49]: ERROR template: config.yml:104:18: executing "config.yml" at <getv "/deis/store/ga...>: error calling getv: key does not exist
Apr 18 04:27:41 deis-2 sh[23543]: 2015-04-18T04:27:41Z cb066ac10b17 confd[49]: ERROR template: config.yml:104:18: executing "config.yml" at <getv "/deis/store/ga...>: error calling getv: key does not exist
Apr 18 04:27:42 deis-2 sh[23543]: Error:  100: Key not found (/deis/registry/masterLock) [16456870

This is basically repeated over and over.

Store is also active and running. Also complaining to the logs.
Apr 18 04:27:42 deis-2 sh[20861]: Error:  100: Key not found (/deis/store/gateway/masterLock) [16456870]

Recurring theme?

Seems like some missing information. Trying to restart the registry.

GG. Now we're back to #3458

Gasp!! It worked!

weird. @onbjerg please contact us on IRC if you're seeing these issues again. Seems like you're hitting a plethora of them all at once. Sorry!

Will do :smile_cat:


Allow xr.combine_by_coords to work with point coordinates?

MCVE Code Sample

# Your code here

data_0 = xr.Dataset({'temperature': ('time', [10,20,30])}, coords={'time': [0,1,2]})
data_0.coords['trial'] = 0

data_1 = xr.Dataset({'temperature': ('time', [50,60,70])}, coords={'time': [0,1,2]})
data_1.coords['trial'] = 1

# the following fails with a ValueError:
# all_trials = xr.combine_by_coords([data_0, data_1])

# but this works:
all_trials = xr.combine_by_coords([data_0.expand_dims('trial'), data_1.expand_dims('trial')]) # works

Expected output
Both work and produce the same result.
Problem Description

I found this behavior somewhat unintuitive -- I didn't understand that I needed to call expand_dims to get combine_by_coords to work. It might make things a little easier to understand if both behaved the same way -- maybe the code that concatenates dimensions should look for full dimensions first, and point coordinates second.
This would also make code that selects dimensions with sel and then calls combine_by_coords more easy to write.
On the other hand, this could be seen as "magic", and it might also be a breaking change to combine_by_coords; I'm not sure. So I'd understand if isn't implemented.

This is a case which I can see the argument for handling, but we apparently didn't think of when implementing combine_by_coords.
What this really boils down to is "should combine_by_coords combine by 0D non-dimension coordinates as well as 1D dimension coordinates?". I think yes, that makes sense.

On the other hand, this could be seen as "magic"

This function is supposed to be "magic". But the behaviour still needs to be clearly-defined.

it might also be a breaking change to combine_by_coords; I'm not sure.

I don't think it would be breaking - the only cases which would behave differently are ones which would previously would have thrown an error.
I actually got your example to work with just a few extra lines - change the top of _infer_concat_order_from_coords to read like this:
def _infer_concat_order_from_coords(datasets):

    # All datasets have same variables because they've been grouped as such
    ds0 = datasets[0]

    concat_dim_candidates = list(ds0.dims.keys())

    # Look for 0D coordinates which can be concatenated over but aren't dims
    non_dimension_0d_coords = [coord for coord in ds0.coords
                               if coord not in ds0.dims
                               and len(ds0[coord].dims) == 0]
    for coord in non_dimension_0d_coords:
        if all(coord in ds.coords for ds in datasets):
            datasets = [ds.expand_dims(coord) for ds in datasets]
            concat_dim_candidates.append(coord)

    concat_dims = []
    tile_ids = [() for ds in datasets]

    for dim in concat_dim_candidates:
        ...   # The rest of the function is unchanged

Might you be interested in submitting a PR along these lines @kazimuth ? It would require some more robust input checks, some tests for different cases, and some minor changes to docstrings I think.

I did get one test failure TestDask.test_preprocess_mfdataset, which failed with
    def test_preprocess_mfdataset(self):
        original = Dataset({"foo": ("x", np.random.randn(10))})
        with create_tmp_file() as tmp:
            original.to_netcdf(tmp)

            def preprocess(ds):
                return ds.assign_coords(z=0)

            expected = preprocess(original)
            with open_mfdataset(
                tmp, preprocess=preprocess, combine="by_coords"
            ) as actual:
>               assert_identical(expected, actual)
E               AssertionError: Left and right Dataset objects are not identical
E               Differing dimensions:
E                   (x: 10) != (x: 10, z: 1)
E               Differing coordinates:
E               L   z        int64 0
E               R * z        (z) int64 0
E               Differing data variables:
E               L   foo      (x) float64 0.07341 -0.01756 0.6903 ... -0.08741 0.08472 -0.1905
E               R   foo      (z, x) float64 dask.array<chunksize=(1, 10), meta=np.ndarray>

xarray/tests/test_backends.py:2955: AssertionError

which needs to be looked into.

is _infer_concat_order_from_coords used in other places than combine_by_coords? I haven't really dug into xarray's source yet.

is _infer_concat_order_from_coords used in other places than combine_by_coords?

combine_by_coords can be called by open_mfdataset, but other than that no.

By the way, there is an additional problem to consider in addition to the above: ambiguity with multiple coordinates for the same dimension.
data0 = xr.Dataset({'temperature': ('time', [10,20,30])}, coords={'time': [0,1,2]})
data0.coords['trial'] = 0
data0.coords['day'] = 5

data1 = xr.Dataset({'temperature': ('time', [50,60,70])}, coords={'time': [0,1,2]})
data1.coords['trial'] = 1
data1.coord['day'] = 3

In this example then combine_by_coords won't know which coordinate to use to order along the new dimension, should it be 'trial' or 'day'? They will give differently-ordered results.
We could pass extra optional arguments to deal with this, but for a "magic" function that doesn't seem desirable. It might be better to have some logic like:

Is there a dimension coordinate for this dimension? If yes just use that.
Otherwise, can the datasets be unambiguously ordered along the dim by looking at non-dimension coords?
If yes, do that, if no, throw informative error.

That would be backwards-compatible (currently all magic ordering is being done for dimension coords), and not make any arbitrary choices.

@TomNicholas I think this is the case I ran into.
Does the old auto_combine handle this case? If so, we should get your fix in before releasing a version without _auto_combine

@dcherian it looks like the old auto_combine does handle this case, but so does the new combine_nested (which is intentional, one of the purposes of combine_nested is to make it easier to explicitly combine along new dimensions):
import xarray as xr
from xarray.core.combine import _old_auto_combine

data_0 = xr.Dataset({'temperature': ('time', [10,20,30])}, coords={'time': [0,1,2]})
data_0.coords['trial'] = 0

data_1 = xr.Dataset({'temperature': ('time', [50,60,70])}, coords={'time': [0,1,2]})
data_1.coords['trial'] = 1

all_trials = _old_auto_combine([data_0, data_1], concat_dim='trial')
print(all_trials)

<xarray.Dataset>
Dimensions:      (time: 3, trial: 2)
Coordinates:
  * time         (time) int64 0 1 2
  * trial        (trial) int64 0 1
Data variables:
    temperature  (trial, time) int64 10 20 30 50 60 70

but you can get the same result with
all_trials = xr.combine_nested([data_0, data_1], concat_dim='trial')
print(all_trials)

<xarray.Dataset>
Dimensions:      (time: 3, trial: 2)
Coordinates:
  * time         (time) int64 0 1 2
  * trial        (trial) int64 0 1
Data variables:
    temperature  (trial, time) int64 10 20 30 50 60 70

So technically this issue doesn't need to be closed before completely removing the old auto_combine, but we probably should fix this anyway.

Ok sounds good. Thanks

I opened a PR to fix this.
Also I realised that what I said about dealing with ambiguities of multiple coordinates for the same dimension doesn't make sense. In that situation there is no way to know the user wanted each of the 'trial' and 'day' coords to correspond to the same dim, and if you give them each their own dim it's a well-defined combination operation, you just end up combining along an extra dim 'day' and padding with a lot of NaNs. I don't think there's a problem with that behaviour, so I've just left it without any special treatment.
(I did make sure it handles the case which caused TestDask.test_preprocess_mfdataset to fail though, because that's to do with checking that the coordinates aren't just the same on every dataset.)

Suppose there were multiple scalar coordinates that are unique for each variable. How would combine_by_coords pick a dimension to stack along?

Suppose there were multiple scalar coordinates that are unique for each variable. How would combine_by_coords pick a dimension to stack along?

@shoyer it would expand and stack along both, filling the (many) gaps created with NaNs.
import xarray as xr

data_0 = xr.Dataset({'temperature': ('time', [10,20,30])}, coords={'time': [0,1,2]})
data_0.coords['trial'] = 0  # scalar coords
data_0.coords['day'] = 1

data_1 = xr.Dataset({'temperature': ('time', [50,60,70])}, coords={'time': [0,1,2]})
data_1.coords['trial'] = 1
data_1.coords['day'] = 0

# both scalar coords will be promoted to dims
all_trials = xr.combine_by_coords([data_0, data_1])
print(all_trials)

<xarray.Dataset>
Dimensions:      (day: 2, time: 3, trial: 2)
Coordinates:
  * time         (time) int64 0 1 2
  * trial        (trial) int64 0 1
  * day          (day) int64 0 1
Data variables:
    temperature  (day, trial, time) float64 nan nan nan 50.0 ... nan nan nan

The gaps created will be filled in with NaNs
print(all_trials['temperature'].data)

[[[nan nan nan]
  [50. 60. 70.]]

 [[10. 20. 30.]
  [nan nan nan]]]

This gap-filling isn't new though - without this PR the same thing already happens with length-1 dimension coords (since PR #3649 - see my comment there)
data_0 = xr.Dataset({'temperature': ('time', [10,20,30])}, coords={'time': [0,1,2]})
data_0.coords['trial'] = [0]   # 1D dimension coords
data_0.coords['day'] = [1]

data_1 = xr.Dataset({'temperature': ('time', [50,60,70])}, coords={'time': [0,1,2]})
data_1.coords['trial'] = [1]
data_1.coords['day'] = [0]

all_trials = xr.combine_by_coords([data_0, data_1])
print(all_trials)

<xarray.Dataset>
Dimensions:      (day: 2, time: 3, trial: 2)
Coordinates:
  * time         (time) int64 0 1 2
  * day          (day) int64 0 1
  * trial        (trial) int64 0 1
Data variables:
    temperature  (trial, day, time) float64 nan nan nan 10.0 ... nan nan nan

# gaps will again be filled in with NaNs
print(all_trials['temperature'].data)

[[[nan nan nan]
  [10. 20. 30.]]

 [[50. 60. 70.]
  [nan nan nan]]]

So all my PR is doing is promoting all scalar coordinates (those which aren't equal across all datasets) to dimension coordinates before combining.
There is a chance this could unwittingly increase the overall size of people's datasets (when they have different scalar coordinates in different datasets), but that could already happen since #3649.

I don't think it's a great idea to automatically turn scalar coords into 1d arrays. It's not uncommon to have datasets with a whole handful of scalar coordinates, which could potentially become very high dimensional due to this change.
I also don't like fallback logic that tries to do matching by coordinates with dimensions first, and then falls back to using scalars. These types of heuristics look very convenient first (and are very convenient much of the time) but then have a tendency to fail in unexpected/unpredictable ways.
The other choice for situations like this would be to encourage switching to combine_nested for use cases like this, e.g.,
>>> xr.combine_nested([data_0, data_1], concat_dim='trial')
<xarray.Dataset>
Dimensions:      (time: 3, trial: 2)
Coordinates:
  * time         (time) int64 0 1 2
  * trial        (trial) int64 0 1
Data variables:
    temperature  (trial, time) int64 10 20 30 50 60 70

We could even put a reference to combine_nested in this error raised by combine_by_coords.

I don't think it's a great idea to automatically turn scalar coords into 1d arrays.

Fair enough. Those are good reasons.

The other choice for situations like this would be to encourage switching to combine_nested for use cases like this

The fact that you can solve this use case with combine_nested does make this feature a lot less important.

We could even put a reference to combine_nested in this error raised by combine_by_coords.

The error you get is
ValueError: Could not find any dimension coordinates to use to order the datasets for concatenation

so we could try to automatically detect the coordinates the user might have wanted to combine along, or we can just add something like
ValueError: Could not find any dimension coordinates to use to order the datasets for concatenation.
To specify dimensions or coordinates to concatenate along, use the `concat_dim` argument to `xarray.combine_nested`.

Looking back at @shoyer 's comments above I think I'm just going to close this (and the corresponding PR #3982) for now.


Messaging: Replay Recommender

Messaging
The trouble with replays is that it's easy to forget to use them and it's hard to find the signal amidst the noise.
We've got some solutions to the latter now (activity scores), so here's a solution to the former: An email campaign that tells people who forget that they've got some cool new replays to watch.
Workflow
Users will enter this campaign if:

They haven't watched a recording in 3 weeks
They have ingested over 500 recordings

This ensures we are reaching people who have recordings to watch, but aren't watching them.
Once they're in the flow, we immediately trigger the following email:

If they click a link in that email, we then set replay_recommender_conversion to true on the person
The link we drive them towards is a list of activity score recordings in the last 24 hours which are greater than 30 seconds.
They then exit the campaign. They can't enter it again for another 4 weeks. This stops the emails becoming annoying.
Subscription changes
We want to give users a way to opt-out of these emails without opting out of other emails. So, I've created a new subscription type and updated the existing ones.

@raquelmsmith open to your thoughts here on what's worth adding an experiment for, etc

Nice! Love the hogbot 😍
I wonder if it's worth telling them about the "what to watch" page? Eg. "Need some other inspiration? Check out our what to watch page." Or could cycle through specific recommendations - "want another idea? how about watching users who used your signup flow?" and then we could highlight the specific template when they reach the page. (I'm okay with it if you think it's a bad idea, but since the goals are similar - help people see the variety of ways they can use replays - it might go well together)

An experiment would be good.
Control group does not get the email, prop on person (or event? I'd kind of rather do this with an event) that says they were entered into the experiment with a variant control.
Test group does get the email, prop/event saying they were entered into exp with variant test.
I'd make the control group 20% of the users and the test 80% (since most prob won't click anyway - though receiving the email at all could remind them that it exists!)

Nice! Love the hogbot 😍
I wonder if it's worth telling them about the "what to watch" page? Eg. "Need some other inspiration? Check out our what to watch page." Or could cycle through specific recommendations - "want another idea? how about watching users who used your signup flow?"

This is there are the moment, albeit buried in the footer. We can move it out, but I think we still want it minimized -- having two CTAs which are boldly put together is going to complicate the messaging, I think.

An experiment would be good.
Control group does not get the email, prop on person (or event? I'd kind of rather do this with an event)

Do we need an event to do this? I can easily configure the experiment in Customer.io and track success there, without it. I'll set it up now, but the Customer.io integration isn't two-way -- so I don't think this event will make it back to PostHog anyway.

All set up. We'll set an event and see what happens with that, but the A/B test in Customer.io should give us a decent read anyway and I can loop back with the results once it's running.

Unless any concerns, I'll set this to run tomorrow morning?

🙌 awesome! What's the event that's being sent?

🙌 awesome! What's the event that's being sent?

replay_recommender_conversion

Oh is this sending an event/updating the person in customer.io? I meant an event/prop in posthog 😄

We hit a snag with this.
One user replied saying that he didn’t use replays at all. Logged in as him and confirmed that he had no replays at all. Debugged with Annika and we can’t see any real usage from him in the last 12 weeks, no recordings on file.
We've realized this is due to the organization_usage_report segment not having a filter on it. I've now added a filter saying that that event must have come in the last 21 days, which will hone it in.
All these emails are coming directly from me at the moment, so I'll respond to any users who flag this.

30 days later, tracking a 7% conversion


Do not override "interface" in the configuration file

I'm opening this issue because:

[ ] I have found a bug
[ ] I want to request a feature
[x] I have a question
[ ] Other

I found a strange thing. When running qrcp for the first time, a config.json file is created, which holds the default software configuration. Everything looks normal except interface. interface does not hold the default like other options, but the interface I used the last time I ran qrcp. Every time I switch between ethernet and wifi I get Error: unable to find interface.
I tried setting interface to an empty string and the strange thing happened again. Whatever options are used on the command line will not overwrite the contents of the config.json file, except interface. I can only manually modify config.json every time I switch network environments.
I don't understand why qrcp store the value of interface by default, why overwrite the config file with the interface used the last time I ran qrcp. I think it should rescan available interfaces every time qrcp is run.

Hey, sorry for the huge delay! Yep, this seems to be a bug. However, I want to change the configuration file format from JSON to YAML in the next release, I will work on this too.
Thanks for reporting!

I think this should be fixed as of now.


Split opendistro-tar-install.sh into two pieces

Issue #, if available: #696
Description of changes:
Split into opendistro-onetime-setup.sh and opendistro-start.sh.
This change is to ensure that one-time activities, such as installing the demo security config, happen once. This is especially useful for Docker images where we can run the one-time script as part of the Docker build.
opendistro-tar-install.sh invokes opendistro-onetime-setup followed by opendistro-start so that existing scripts that use opendistro-tar-install will continue to work the same as they did before this change.
Test Results: Manually tested with a work-in-progress Docker image. Able to install and start a node on Amazon Linux 2.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on signing off your commits, please check here.

Can we change the scripts name to:

opendistro-setup
opendistro-run

Any particular reason? I think "onetime-setup" makes it more clear that this is intended to be run once :)

I will remove the Darwin references in linux Tar build and install when I make further changes to macOS scripts.


Don't Merge! PR real time test for target branch full build

Docs Build status updates of commit 1204053:
:clock10: Full build: the average full build time is 5 min(s) 41 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit 1204053:
:warning: Validation status: warnings

File
Status
Preview URL
Details

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

docs-ref-conceptual/testing.md

[Warning] Missing attribute: either ms.prod or ms.service is required. Use ms.prod for on-premise products, or ms.service for cloud services.
[Warning] Missing attribute: title.
[Warning] Missing attribute: either ms.prod or ms.service is required. Use ms.prod for on-premise products, or ms.service for cloud services.
[Warning] Missing attribute: title.
[Warning] Missing attribute: either ms.prod or ms.service is required. Use ms.prod for on-premise products, or ms.service for cloud services.
[Warning] Missing attribute: title.

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


PDep null pointer exception

this essentially on the latest master branch:

Reaction Set Found from Reaction Library [JP10(1) --> MA111(10)]
Reaction Set Found from Reaction Library [C4H3(54) --> C2H2(51) + C2H(41)]
RMG is only utilizing the high-pressure limit parameters for PKL reaction: C4H3(54) --> C4H2(44) + H(38)
RMG is only utilizing the high-pressure limit parameters for PKL reaction: C4H3(54) --> C4H2(44) + H(38)
RMG is only utilizing the high-pressure limit parameters for PKL reaction: C4H3(54) --> C4H2(44) + H(38)
RMG is only utilizing the high-pressure limit parameters for PKL reaction: C4H3(54) --> C4H2(44) + H(38)
Created new intra_H_migration reaction: C4H3(54) --> C4H3J(707)
Created new intra_H_migration reaction: C4H3(54) --> C4H3J(163)
ERROR: java.lang.NullPointerException
    at jing.rxn.PDepNetwork.addReactionToNetworks(PDepNetwork.java:681)
    at jing.rxnSys.ReactionModelGenerator.initializeCoreEdgeModel(ReactionModelGenerator.java:3643)
    at jing.rxnSys.ReactionModelGenerator.initializeCoreEdgeReactionModel(ReactionModelGenerator.java:3867)
    at jing.rxnSys.ReactionModelGenerator.modelGeneration(ReactionModelGenerator.java:1302)
    at RMG.main(RMG.java:96)

Exception in thread "main" java.lang.NullPointerException
        at jing.rxnSys.Logger.log(Logger.java:160)
        at jing.rxnSys.Logger.critical(Logger.java:204)
        at RMG.main(RMG.java:106)

bump!
I just ran into this issue again myself. I'm glad I decided to search the issue tracker, because the stack trace is most uninformative.


🆙 update nodejs 16 -> 20 and use @vercel/ncc

Equivalent to #6 (thanks @nilwurtz!) but also bumps more transitives; and increases the version number (we don't release tagged versions of the action, but this seems better practice when changing the Node version)
@sriv FYI - this is another repo I don't seem to have access to. (I am guessing it's repos that were perhaps moved into the org from elsewhere?)

merged, and made you an admin @chadlwilson


[Bug report] InitDynamicData() failed in Telegram Desktop x86 (Version3.2.5)（在Telegram桌面版（版本3.2.5）中出现"InitDynamicData() failed"异常）

Describe the bug
A clear and concise description of what the bug is.
Steps to reproduce
1.Open telegram
2.wait sometimes
3.BOOM
（为防翻译出错加个汉语）
1.打开Telegram
2.等上一会儿
3.程序出错
Expected behaviour
Tell us what should happen
Actual behaviour
Tell us what happens instead
Configuration

Operating system: Windows
Version of Telegram: 3.2.5
Version of the plugin: The lastest

（因为日志显示那里不知道为啥没有换行，我没写过html，也不太清楚GitHub Issue在哪上传文件，所以只能单独加个日志文件下载链接了）
Log file download link:https://onecloud.19770214.top/api/v3/file/source/243/ArLog.txt?sign=-hTNYnqCQTT2C8urhZIUKXSjOXebWsQNwbfZgHKOJ3E%3D%3A0
日志下载链接:https://onecloud.19770214.top/api/v3/file/source/243/ArLog.txt?sign=-hTNYnqCQTT2C8urhZIUKXSjOXebWsQNwbfZgHKOJ3E%3D%3A0
Logs
[2021-11-17 19:57:12.794] [Main] [info] [RealMain.cpp:32] Running. Version: "0.4.2", Platform: "x86"
[2021-11-17 19:57:12.797] [Main] [info] [IRuntime.cpp:27] [IRuntime] Telegram version: 3002005
[2021-11-17 19:57:33.920] [Main] [warning] [IUpdater.cpp:244] [Updater] Internet::HttpRequest() failed. (ByBridge)
[2021-11-17 19:57:33.920] [Main] [warning] [IUpdater.cpp:35] [Updater] GetDataByBridge() failed, try GetDataDirectly().
[2021-11-17 19:57:35.326] [Main] [info] [IUpdater.cpp:294] [Updater] Get data directly successed.
[2021-11-17 19:57:35.326] [Main] [info] [IUpdater.cpp:107] [Updater] No need to update. Local: 000004002, Latest: 000004002
[2021-11-17 19:57:35.326] [Main] [info] [IUpdater.cpp:56] [Updater] ParseResponse() successed. (Directly)
[2021-11-17 19:57:35.814] [Main] [info] [IRuntime.cpp:125] [IRuntime] InitDynamicData_MallocFree() succeeded.
[2021-11-17 19:57:35.990] [Main] [info] [IRuntime.cpp:126] [IRuntime] InitDynamicData_DestroyMessage() succeeded.
[2021-11-17 19:57:36.190] [Main] [info] [IRuntime.cpp:127] [IRuntime] InitDynamicData_EditedIndex() succeeded.
[2021-11-17 19:57:36.377] [Main] [info] [IRuntime.cpp:128] [IRuntime] InitDynamicData_SignedIndex() succeeded.
[2021-11-17 19:57:36.592] [Main] [info] [IRuntime.cpp:129] [IRuntime] InitDynamicData_ReplyIndex() succeeded.
[2021-11-17 19:57:36.838] [Main] [info] [IRuntime.cpp:130] [IRuntime] InitDynamicData_LangInstance() succeeded.
[2021-11-17 19:57:37.084] [Main] [warning] [IRuntime.cpp:1263] [IRuntime] Search toHistoryMessage index falied.
[2021-11-17 19:57:37.084] [Main] [warning] [IRuntime.cpp:131] [IRuntime] InitDynamicData_ToHistoryMessage() failed.

(The log file is in the same directory with version.dll file)

Duplicate of #88


Fix #86 and allow continue ^ in triggers

No change in order of check_syntax (still before parsing continue ^) because space is only checked in lookahead (not yet checked for syntax at the time of looking ahead)

Adding unit tests that allow continue in triggers

@kirsle  Do you have a feedback on this Pull Request?


ComicTagger will not open any cbr files.

When I try and open a cbr file, the progress bar pops up at 0% and never advances.
Run on Arch Linux.  I launch ComicTagger by typing 'python2 ./comictagger.py' from bash.

Was running the code from Google Code and not yours.  There is no issue.  Closing ticket


Train ssd from scratch get bad result?

I follow the instructions to train ssd model from scratch in ssd300_training.ipynb , but finall i get bad result.
The Related parameters i set as follows:
batch_size=15(my computer not very good);
dataset path have corresponding modification （only use VOC）;
when i have trained 60 epoch, the loss is 5.06 and val loss is 4.8. after that i use ssd300_evaluation.ipynb to get MAP, it just only 0.34.
I try many time to find problem, but failed, can you get me some advice, thank you for you attention.

The SSD300 training notebook should reproduce the original results as is. I can't really say much more than that, I've run the full training multiple times and gotten the reported results each time, so I wouldn't know what advice to give you.
The only thing you seem to have changed is the batch size, which should have at most a minor effect on the training progress.
The training and validation losses you report after 60 epochs are both where they should be, so that part looks good, but the mAP you get is inconsistent with those loss figures, so maybe double-check that the settings of the Evaluator are correct. And also just make a few predictions to get a rough qualitative evaluation of the model's performance after 60 epochs.
Another possibility that comes to mind is, maybe you're running Python 2 and that is causing issues, but that's a long shot.