Orchestrator forwarder does not seem to honor PROXY env vars

Output of the info page (if this is a bug)
Since the output is large - I posted it at the end
Describe what happened:
We are behind a proxy. So I set PROXY env vars in all the containers. I set them for the cluster agent via the Helm Chart as follows:
# my-values.yaml
clusterAgent:
  env:
    - name: HTTP_PROXY
      value: redacted
    - name: HTTPS_PROXY
      value: redacted.
    - name: NO_PROXY
      value: redacted.

The the orchestratorExplorer is enabled in the helm chart by default - and while all the other calls to DD HQ are fine - the call to https://orchestrator.datadoghq.com/api/v1/orchestrator fails with:
TLS Handshake failure: x509: certificate signed by unknown authority

Which is indicative of a proxy issue. Again - all other calls to DD HQ are fine.
Is it possible the orchestrator calls are not honoring the proxy env vars?
Describe what you expected:
Forwarding to orchestrator.datadoghq.com should honor the proxy env vars and succeed.
Steps to reproduce the issue:
You need to be behind a proxy and set the proxy env vars the agent container. Then make a call to the orchestrator endpoint. See error below.
Additional environment details (Operating System, Cloud provider, etc):
Cloud Provider is AWS.
Repeated Errors in log
2021-06-26 16:37:50 UTC | CLUSTER | ERROR | (pkg/forwarder/transaction/transaction.go:108 in func4) | TLS Handshake failure: x509: certificate signed by unknown authority
2021-06-26 16:37:50 UTC | CLUSTER | ERROR | (pkg/forwarder/worker.go:183 in process) | Error while processing transaction: error while sending transaction, rescheduling it: Post "https://orchestrator.datadoghq.com/api/v1/orchestrator": x509: certificate signed by unknown authority

Cluster Agent Status
2021-06-26 16:43:18 UTC | CLUSTER | WARN | (pkg/util/log/log.go:541 in func1) | Agent configuration relax permissions constraint on the secret backend cmd, Group can read and exec
2021-06-26 16:43:18 UTC | CLUSTER | INFO | (pkg/util/log/log.go:526 in func1) | Found 'HTTP_PROXY' env var, using it for the Agent proxy settings
2021-06-26 16:43:18 UTC | CLUSTER | INFO | (pkg/util/log/log.go:526 in func1) | Found 'HTTPS_PROXY' env var, using it for the Agent proxy settings
Getting the status from the agent.
2021-06-26 16:43:18 UTC | CLUSTER | INFO | (pkg/util/log/log.go:526 in func1) | Found 'NO_PROXY' env var, using it for the Agent proxy settings
2021-06-26 16:43:18 UTC | CLUSTER | INFO | (pkg/util/log/log.go:526 in func1) | Features detected from environment: kubernetes,orchestratorExplorer
===============================
Datadog Cluster Agent (v1.13.0)
===============================

  Status date: 2021-06-26 16:43:18.894 UTC<PHONE_NUMBER>894)
  Agent start: 2021-06-26 16:37:08.291 UTC<PHONE_NUMBER>291)
  Pid: 1
  Go Version: go1.15.11
  Build arch: amd64
  Agent flavor: cluster_agent
  Check Runners: 4
  Log Level: INFO

  Paths
  =====
    Config File: /etc/datadog-agent/datadog-cluster.yaml
    conf.d: /etc/datadog-agent/conf.d

  Clocks
  ======
    System time: 2021-06-26 16:43:18.894 UTC<PHONE_NUMBER>894)

  Hostnames
  =========
    ec2-hostname: redacted
    hostname: redacted
    instance-id: redacted
    socket-fqdn: datadog-cluster-agent-67bbf98979-tbhmt
    socket-hostname: datadog-cluster-agent-67bbf98979-tbhmt
    hostname provider: aws
    unused hostname providers:
      azure: azure_hostname_style is set to 'os'
      configuration/environment: hostname is empty
      gce: unable to retrieve hostname from GCE: status code 404 trying to GET http://<IP_ADDRESS>/computeMetadata/v1/instance/hostname

  Metadata
  ========

Leader Election
===============
  Leader Election Status:  Running
  Leader Name is: datadog-cluster-agent-67bbf98979-tbhmt
  Last Acquisition of the lease: Sat, 26 Jun 2021 16:37:44 UTC
  Renewed leadership: Sat, 26 Jun 2021 16:43:15 UTC
  Number of leader transitions: 2 transitions

Custom Metrics Server
=====================
  Disabled: The external metrics provider is not enabled on the Cluster Agent

Cluster Checks Dispatching
==========================
  Status: Leader, serving requests
  Active nodes: 7
  Check Configurations: 0
    - Dispatched: 0
    - Unassigned: 0

Admission Controller
====================
  Disabled: The admission controller is not enabled on the Cluster Agent

=========
Collector
=========

  Running Checks
  ==============

    kubernetes_apiserver
    --------------------
      Instance ID: kubernetes_apiserver [OK]
      Configuration Source: file:/etc/datadog-agent/conf.d/kubernetes_apiserver.d/conf.yaml.default
      Total Runs: 25
      Metric Samples: Last Run: 0, Total: 0
      Events: Last Run: 6, Total: 6
      Service Checks: Last Run: 4, Total: 88
      Average Execution Time : 1.745s
      Last Execution Date : 2021-06-26 16:43:12 UTC<PHONE_NUMBER>000)
      Last Successful Execution Date : 2021-06-26 16:43:12 UTC<PHONE_NUMBER>000)

    kubernetes_state_core
    ---------------------
      Instance ID: kubernetes_state_core:e407ffa33ed108e2 [OK]
      Configuration Source: file:/etc/datadog-agent/conf.d/kubernetes_state_core.yaml.default
      Total Runs: 25
      Metric Samples: Last Run: 3,811, Total: 87,653
      Events: Last Run: 0, Total: 0
      Service Checks: Last Run: 28, Total: 644
      Average Execution Time : 90ms
      Last Execution Date : 2021-06-26 16:43:17 UTC<PHONE_NUMBER>000)
      Last Successful Execution Date : 2021-06-26 16:43:17 UTC<PHONE_NUMBER>000)

    orchestrator
    ------------
      Instance ID: orchestrator:d884b5186b651429 [OK]
      Configuration Source: file:/etc/datadog-agent/conf.d/orchestrator.d/conf.yaml.default
      Total Runs: 37
      Metric Samples: Last Run: 0, Total: 0
      Events: Last Run: 0, Total: 0
      Service Checks: Last Run: 0, Total: 0
      Average Execution Time : 1ms
      Last Execution Date : 2021-06-26 16:43:10 UTC<PHONE_NUMBER>000)
      Last Successful Execution Date : 2021-06-26 16:43:10 UTC<PHONE_NUMBER>000)

=========
Forwarder
=========

  Transactions
  ============
    Clusters: 2
    CronJobs: 0
    DaemonSets: 0
    Deployments: 2
    Dropped: 0
    DroppedOnInput: 0
    Jobs: 0
    Nodes: 5
    Pods: 0
    ReplicaSets: 2
    Requeued: 544
    Retried: 55
    RetryQueueSize: 13
    Services: 2

  Transaction Successes
  =====================
    Total number: 50
    Successes By Endpoint:
      check_run_v1: 24
      intake: 2
      series_v1: 24

  Transaction Errors
  ==================
    Total number: 9
    Errors By Type:
      TLSErrors: 9

==========
Endpoints
==========
  https://app.datadoghq.com - API Key ending with:
      - 49904

=====================
Orchestrator Explorer
=====================
  Collection Status: The collection is at least partially running since the cache has been populated.
  Cluster Name: redacted
  Cluster ID: redacted
  Container scrubbing: enabled

  ======================
  Orchestrator Endpoints
  ======================
  https://orchestrator.datadoghq.com - API Key ending with: 49904

  ===========
  Cache Stats
  ===========
    Elements in the cache: 123
    Pods:
      Last Run: (Hits: 0 Miss: 0) | Total: (Hits: 0 Miss: 0)
    Deployments:
      Last Run: (Hits: 29 Miss: 0) | Total: (Hits: 899 Miss: 58)
    ReplicaSets:
      Last Run: (Hits: 30 Miss: 0) | Total: (Hits: 930 Miss: 60)
    Services:
      Last Run: (Hits: 47 Miss: 0) | Total: (Hits: 1457 Miss: 94)
    Nodes:
      Last Run: (Hits: 7 Miss: 0) | Total: (Hits: 214 Miss: 17)
    Cluster:
      Last Run: (Hits: 1 Miss: 0) | Total: (Hits: 31 Miss: 2)
    Job:
      Last Run: (Hits: 0 Miss: 0) | Total: (Hits: 0 Miss: 0)
    CronJob:
      Last Run: (Hits: 0 Miss: 0) | Total: (Hits: 0 Miss: 0)
    Daemonset:
      Last Run: (Hits: <no value> Miss: <no value>) | Total: (Hits: 0 Miss: 0)

Hey @michaelajr,
I wasn't able to reproduce this locally. I am sure that the orchestrator forwarder applies proxy settings like every other forwarder does as well (same code path).
Could you share your sanitised proxy configuration? Alternatively, feel free to open a support ticket (in case of privacy) and  we will come back.


[Publish] Add publish one shot

some notification doesn't neet alltimes. So it's better this notification is one shot or not.

Otherwise, user always need to call publisher_unsubscribe to keep SubscriberAccount


Document build dependencies and dev dependencies

Add a new --kinds option to cargo doc, which allows to select what
kinds of dependencies should be documented: build dependencies, dev
dependencies, or normal dependencies. Multiple kinds of dependencies
can be specified, e.g., cargo doc --kinds=dev,normal.
When --kinds options is used, the default behaviour of documenting
normal dependencies is suppressed unless asked for explicitly with
--kinds=normal. The --kinds option also conflicts with --no-deps.
Follow up on proposal from #7077. Resolves #3475.
Open questions:

Should --kinds document normal dependencies by default, obviating the need
for normal kind of dependencies? Current answer is no.
Should --kinds interact with --no-deps flag? Current answer is no, but
presence or absence of --no-deps could control if dependencies of selected
dependencies are also documented.

r? @ehuss
(rust_highfive has picked a reviewer for you, use r? to override)

Hi! Thanks for the PR!

1. Should --deps document normal dependencies by default,

I think the default of "no" here makes sense to me.

2. Should --deps interact with --no-deps flag?

Hm, I would be inclined to just print an error if both flags are specified.  They are somewhat contradictory.  It's probably not too important either way.
A few questions and observations:

It's an interesting choice to never do transitive dependencies when using --deps.  Do you think there might be a use case where someone would want transitive + normal + dev + build?
What do you think about adding an "all" option?  I think the previous PR had something like that, and it seems like it would be a useful shortcut.  My only concern is that it could be confusing regarding the "transitive" question above.

Do you think there might be a use case where someone would want transitive + normal + dev + build?

When I was referring to the transitive dependencies I was mostly thinking about
normal transitive dependencies of those selected. Normal dependencies might be
part of public API of the crate being documented, so I think there is
relatively clear use-case for those.
I like idea of all-deps, no-deps, direct-deps from #8296. If it were to become
a thing at some point, I think that it should work together with feature here.
I added all kind, and changed the flag name to --kinds. I think this would
be less confusing with respect to the question of transitive dependencies.  We
could be even more explicit with --dep-kinds?
I also noticed that this sometimes introduces an output filename collision for
documented crates, but didn't have time to investigate yet.

I kinda like the original flag name of --deps. It has a good symmetry with --no-deps, and I think is very clear in what it does. Would you consider maybe switching it back?
As for the question of how to indicate whether or not you want transitive deps, I don't have any specific ideas. I'd be fine for deferring that decision to later. It could be something like --deps=normal,transitive or something like that.  I'm not sure there's a strong use case for that. I'm also thinking that in the eventual future if public-private dependencies is stabilized, then cargo could always just do "reachable public" dependencies, and never bother with unreachable private ones.

the same unit might occur more than once with slightly different mode

Do you have a specific example where this happens?  Just keep in mind that this is already a problem, so I'm not sure if you're seeing something new. #6313 has some links for rustdoc issues (like any time there is more than one version).

After looking at this a bit more, it seems to me it will require completely different approach and more substantial changes. Thanks for reviewing this so far. I will close this for now.


Does drogon http client support connection through proxy?

I want to send a http request to a remote url(which is not accessible directly) through proxy, can I do this using http client provided by drogon?

hmm a bit strange, fastest http framework on the planet and does not support this standard feature?


⚠ geray.serverssh.net has degraded performance

In 63580fb, geray.serverssh.net (http://geray.serverssh.net/) experienced degraded performance:

HTTP code: 200
Response time: 20891 ms

Resolved: geray.serverssh.net performance has improved in 85a7ba6.


Fix link to Anchor

Relates to https://github.com/solana-labs/governance-ui/pull/1380

👍 @plush-mesh . Cheers - brought this into my PR #1380


Add support for YARN with HTTPS endpoints

Options I see:

Introduce breaking change, follow the path of libs like "hdfs" where you provide http/https in list of endpoints as prefix.
Introduce boolean flag in constructors to switch to HTTPS in prefix in urls inside codebase itself. In this case breaking change not introduced, but the project is not conforming "general" style.

So far option 2 looks less intrusive, I'll prepare PR.


Segment Job API doesn't match Segment developer guide

Issue in ./help/segmentation/api/segment-jobs.md
The segment job API says I need to add the below header to the API call but the developer guide does not include this in the examples. and the API did not specify example values.
x-profile-instance-id
*string(header) | The ID of the Profile Instance.
https://www.adobe.io/apis/experienceplatform/home/api-reference.html#/Segment_jobs/postSegmentJob

Thank you for reporting this. Tracking update work in https://jira.corp.adobe.com/browse/PLAT-79337


Address Verify("abce") returns no error?

For invalid syntax email address, the Verify() func returns err is nil.   For example
emailverifier.NewVerifier()
ret, verifyErr := verifier.Verify("abce")
// verifyErr is nil

In code,  I think at least here will return a error, rather than nil?
	if !syntax.Valid {
		return &ret, nil
	}

I think it is more appropriate to return directly here, because the email address is checked and the result is syntax invalid, and no error occurred in the checking process.
A better way to handle this would be to check if the syntax is valid after a email verification ?

That is OK if it returns nil, but in README, it doesn't mention that. It is easy to misunderstand if err==nil, it is a valid email.
I have already changed my check as this.
ret, verifyErr := verifier.Verify(email)
if verifyErr != nil || !ret.Syntax.Valid {
// invalid
}

Yes, the README is still not clear enough.
Thanks for the suggestion, I will improve the examples for checking email addresses in the next update

Hi @GeniusWiki  We have update the examples. https://github.com/AfterShip/email-verifier/pull/11

close the ticket


Code error

There is an error in your code can you fix it?

Ok please tell me where is the error in the code.


Update lint-on-push.yml to show file name and line number of errors

ref) "lint" section in golang-ci-demo-by-github-actions

As the yaml in the "lint" section, steps in lint-on-push.yml were updated.

As the yaml in the "lint" section, steps in lint-on-push.yml were updated.


Create Apinf system model diagram

Revise Apinf Content Architecture diagram to reflect current project status and identify any missing views.
Definition of done

[ ] Revise the Apinf content architecture diagram into a system diagram or model.
[ ] Update the diagram against current system model (roles, views, databases).
[ ] Check for missing aspects (permissions, views, subscriptions, publications).
[ ] If any of the missing aspects is critical to MVP, revise sprint tasks.

Content architecture diagram has been reviewed.
Some sketches of the system diagram have been created on paper.
Following issues have been created: #354, #353, #352, #351, #350

System model diagram available in Apinf Github repository in docs/developers as an svg file.


Enhancement/2678

[ ] Breaking change? (if so, please describe the impact and migration path for existing application instances)
No breaking changes. Existing functionality has been altered. ALL existing TCs were working as expected.

What changes did you make? (Give an overview)
@Haarolean initially thought out two approaches.
First, I could add message count calculation(sum(offsetMax-offsetMin) for all partitions in the topic) for sorting within TopicsController>getComparatorForTopic. On UI I could perform the same operation on cluster topics listing page for sorting within a single page.
Second, I could add this calculation is within InternalTopic.build() method. I would be adding a new variable in this case, messageCount, which would then be used for comparator operations within TopicsController>getComparatorForTopic. And same DTO(InternalTopic) would be mapped to UI and used for sorting within a single cluster's topics-listing page.
I used first approach. First incurs the least overhead and almost no DTO changes and doesn't require any additional data to be passed within REST API calls. Second, would be slightly faster if extreme partition sizes of >10^1 are assumed, but is in complete violation of Open-Closed principle.
Is there anything you'd like reviewers to focus on?
The count calculation logic within TopicsController>getComparatorForTopic.
How Has This Been Tested? (put an "x" (case-sensitive!) next to an item)

[ ] No need to
[x] Manually (please, describe, if necessary)

Ran application using kafka-ui.yaml, built several topics with different partition sizes, and published a different number of messages on each topic. Tried to sort according to message count afterward on UI.

[x] Unit checks
[x] Integration checks
[ ] Covered by existing automation

Checklist (put an "x" (case-sensitive!) next to all the items, otherwise the build will fail)

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation (e.g. ENVIRONMENT VARIABLES)
[x] My changes generate no new warnings (e.g. Sonar is happy)
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged

Check out Contributing and Code of Conduct
A picture of a cute animal (not mandatory but encouraged)

@everrover could you please:

Fix the build (checkstyle warnings)
Fill all the checkboxes in the second block (otherwise the build will fail)?

@iliax PTAL

Unfortunately this approach won't work - getTopicsForPagination() return topics without offsets information. Currently we don't cache offsets because they frequently changing.
Current logic looks like:

getting topics view via getTopicsForPagination() without offsets info
applying filters to them
loading filtered topics with loadTopics()

Possible solutions:

cache offsets info as a part of Statistics gathering process and use it when doing pagination - this will lead to outdated results (currently ~30sec outdated). Maybe some kind of double check can help, but still, not very clear option.
honestly query offsets info for all topics when paginating - will probably generate additional load on system, decrease endpoint speed.
maybe others ??? like explicit "Update stats" button in UI

So, this is not easy thing to do :) If this is really wanted feature we should analyze it and check solution that works for us.
cc @Haarolean

While writing the TCs around the same i found the same issue. Offset information isn't available due to empty list set within service call.
public Mono<List<InternalTopic>> getTopicsForPagination(KafkaCluster cluster) {
    Statistics stats = statisticsCache.get(cluster);
    return filterExisting(cluster, stats.getTopicDescriptions().keySet())
        .map(lst -> lst.stream()
            .map(topicName ->
                InternalTopic.from(
                    stats.getTopicDescriptions().get(topicName),
                    stats.getTopicConfigs().getOrDefault(topicName, List.of()),
                    InternalPartitionsOffsets.empty(), // i'll add offset data here
                    stats.getMetrics(),
                    stats.getLogDirInfo()))
            .collect(toList())
        );
  }

Am attempting for adding offsets in the same. As of now I'm attempting to use either the function TopicsService>loadTopics or it's functional flows/components.
Although I'd like to ask if an intermediate call to private Mono<List<String>> filterExisting(KafkaCluster cluster, Collection<String> topics) method. Without it the call to fetch data with offsets is straightforward.

Have added the required changes. As I specified in my previous comment. And have tested 'em they are working.

@everrover hey!
After a long discussion we decided not to implement such a feature for a few reasons:

We do not want to display inaccurate stats there and allow sorting by these values in the first place;
The use case for sorting by this column is questionable at best. We think that the user has to sort by the topic size, not by the amount of messages.

Sorry that it turned out like this. We'll be happy to see any other contributions if you wish.
Cheers!


Startup gets stuck while "Checking the system..."

Hi, I've added Testcontainers to my Spring Boot project called springboot-ldap-testcontainers in order to support the implementation of integration tests.
However, I am facing something weird and I've decided to share it with you.
I am using:

macOS Mojave version 10.14.6;
testcontainers version 1.16.0;
docker desktop version 4.0.0.

When I run for the first time the integration tests, Testcontainers starts and shuts down successfully the OpenLDAP Docker container used during the tests. All tests run successfully.
But, when I run the tests again, the startup gets stuck while Checking the system..., as it's shown in the picture below

It never continues, so I cancel the tests Ctrl+C. But, testcontainers/ryuk continues running and the alpine container stays with the Created status.

A workaround that I've found is to disable the startup checks.
In order to get rid of the containers, I need to restart the docker desktop
How to reproduce

git clone https://github.com/ivangfr/springboot-ldap-testcontainers.git
cd springboot-ldap-testcontainers
run integration tests for the 1st time with startup checks enabledTESTCONTAINERS_CHECKS_DISABLE=false && ./mvnw clean verify --projects simple-service

Tests should pass

run integration tests for the 2nd time with startup checks enabledTESTCONTAINERS_CHECKS_DISABLE=false && ./mvnw clean verify --projects simple-service

It should get stuck while Checking the system...

run integration tests with startup checks disabledTESTCONTAINERS_CHECKS_DISABLE=true && ./mvnw clean verify --projects simple-service

Tests should pass

Thanks!

Closing issue as it's not happening anymore. Thanks!


add delete accelerator profile modal

closes: https://github.com/opendatahub-io/odh-dashboard/issues/1376
Description

Adds a delete modal to accelerator profiles page and updated some internal text to say acceleratorProfile instead of accelerator

How Has This Been Tested?

create a profile
located it on the table
action button > delete
delete should work and refresh the page automatically

Test Impact

added tests for delete flow
Request review criteria:

Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit tests & storybook for related changes)

If you have UI changes:

[x] Included any necessary screenshots or gifs if it was a UI change.
[x] Included tags to the UX team if it was a UI/UX change (find relevant UX in the SMEs section).

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

@yannnz pending a quick looks good from you for the UX. not much here though

@Gkrumbach07 Sorry I think the delete modal is missed in the latest update mockup. Here is the Content based on Andrew and Katie's suggestion: https://sketch.com/s/d8550d2f-fae4-44fd-82df-9f400563d459/a/3oxP1aD?annotation=3c3bd330-470c-48ca-bec3-4a210b68a41b
https://sketch.com/s/d8550d2f-fae4-44fd-82df-9f400563d459/a/3oxP1aD?annotation=33f860ce-c3e2-4c2d-bcff-d4f9d9dc8a07

@yannnz based on that. this is what i came up with
The deleteName accelerator profile will be deleted and will be unavailable for any
workbenches and runtimes. Existing resources will retain their settings but cannot revert to
this profile once changed.
i changed it becouse it does not remove the settings from existing resources, it switches into that "use existing settings" option

going to merge this. if there is anything that needs changing we can try to get it in before it merges into incubation on friday. otherwise we can log a separate issue


Weird RECV_MULTISHOT behavior on close

As far as I can tell, close doesn't seem to do anything at all? If the client connection is still open, I'll continue to receive messages even after the close syscall successfully returns. This is extremely surprising behavior and I couldn't find any docs mentioning this.
It seems like the only way to correctly close a client connection is to first perform an async cancellation (which errors out the recv) and then do the close.
I can see how being able to immediately close the fd would be useful, so I'm guessing this is expected behavior, but it should definitely be documented somewhere as it's pretty confusing to debug.

It is more likely documented in close(2), as this is where the issue lies, although this page was written before the concept of io_uring.
The section about "   Multithreaded processes and close()" hints at it
       On Linux (and possibly some other systems), the behavior is
       different: the blocking I/O system call holds a reference to the
       underlying open file description, and this reference keeps the
       description open until the I/O system call completes.  (See
       [open(2)](https://man7.org/linux/man-pages/man2/open.2.html) for a discussion of open file descriptions.)  Thus, the
       blocking system call in the first thread may successfully
       complete after the close() in the second thread.

Since the fd is ref counted, you may want to consider using shutdown(2) before close(2).
https://man7.org/linux/man-pages/man2/shutdown.2.html:
SYNOPSIS
       #include <sys/socket.h>

       int shutdown(int sockfd, int how);

DESCRIPTION
       The shutdown() call causes all or part of a full-duplex
       connection on the socket associated with sockfd to be shut down.
       If how is SHUT_RD, further receptions will be disallowed.  If how
       is SHUT_WR, further transmissions will be disallowed.  If how is
       SHUT_RDWR, further receptions and transmissions will be
       disallowed.


Cleanup and extract Transaction Workflow logic

This extracts the "operational" logic related to container / non-container Transactions.  Saying it another way: it means that the Transaction class no longer needs to be aware of "how" it was instantiated.

Closing this PR out because there were duplicate commits in its history.
The rebased version of this PR can be found here:  #3674


Clears upload file target after import

Importing a file and then clearing it does not allow importing the already selected file again. This fixes that issue by clearing target for "Upload File" after import is made from a file. @martiansideofthemoon

This change is

@nihal111 please make Travis pass. There is a trailing spaces error

Review status: 0 of 1 files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

angular_scripts.js, line 329 [r1] (raw file):

      $scope.uploadFile();
      // Clears target for "Upload File" after import is complete.
      $scope.fileEvent.target.value = null;

This is wrong. If you look at $scope.uploadFile(), it has a promise at the end. As promises are asynchronous, this line will be executed while the file is being ingested. Please add this inside the final then block in $scope.uploadFile().
Also fix the lint error

Comments from Reviewable

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

Comments from Reviewable

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


ATO-160: Update stub documentation

What?
Update the documentation for the stub
Why?
To make it easier to use and develop on

LGTM, feel free to resolve my comments if they aren't needed


added prod task def, changed existing task to pre-prod and updated CI-CD

think thats it on the updates


https://github.com/lukewestby/elm-http-builder/zipball/6.0.0/ is no longer available

Hello.
I'm watching the "introduction to Elm, V2." One of the resources used by the source files, "https://github.com/lukewestby/elm-http-builder/zipball/6.0.0/" is no longer available. I could change the elm.json file to reflect the newer version (7.0.0), but that may break other dependencies. Suggestions?

The releases are found here: https://github.com/lukewestby/elm-http-builder/releases/tag/6.0.0

Thank you. That sorts my issue. I'll change the link. Cheers.


not able to create new branch in Git

Issue Type: Bug
when you click the bottom left Git Branch name in Visual studio code and click " create new branch" and then if you give the new branch name and then press Enter button, " not working "- Error
VS Code version: Code 1.31.0 (7c66f58312b48ed8ca4e387ebd9ffe9605332caa, 2019-02-05T22:35:56.624Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-6500T CPU @ 2.50GHz (4 x 2496)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
7.91GB (1.49GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (9)

Extension
Author (truncated)
Version

githistory
don
0.4.6

xml
Dot
2.4.0

gitlens
eam
9.5.0

git-project-manager
fel
1.7.1

PowerShell
ms-
1.11.0

team
ms-
1.144.1

vscode-docker
Pet
0.5.2

vscode-java-dependency
vsc
0.3.0

vscode-maven
vsc
0.14.1

+1

What error do you see?

I am talking about the flow triggered from "Create branch" option under Cmd + Shift + p menu
Now I understood why it is not working. Earlier I used to create a new branch in just 2 steps and now I see that another step is introduced to select ref branch. Not sure why there is a need of selecting the additional ref branch, it makes more sense take the current branch as reference just like it does in git branch -b xx command.

Seems to be the same issue as https://github.com/Microsoft/vscode/issues/68918 ,  Create Branch workflow now has an additional step.  which might be confused by current users that it didnt work :/  maybe this could become a setting

/duplicate #68918


Typescript definition for FunctionalComponentOptions.render() does not allow to return undefined or VNode[]

Version
2.5.17
Reproduction link
https://github.com/destructive-dragon/vue-bug/blob/master/src/main.ts#L15
Steps to reproduce
Try to return multiple VNode's or undefined from the render function of a functional component while using typescript.
It works, but it requires you to wrap it in return xxx as unknown as VNode. The Typescript definition should probably be updated to allow VNode[] and null or undefined as return value in a render function.
What is expected?
Code should compile without needing to wrap it in return xxx as unknown as VNode
What is actually happening?
Typescript error because the typescript definition for render does not allow VNode[] or undefined as a return value.

Likely not a priority because of the Vue 3.0 Typescript rewrite, but it should be very easy to fix by just extending the FunctionalComponentOptions.render function's typescript definition with the return values of VNode | VNode[] | undefined

https://github.com/vuejs/vue/pull/8192


Fix build if ImageMagick is not installed

The current build fails if ImageMagick is not installed, since the return type wasn't changed to bool. This PR fixes it.
BTW, I'm packaging fastfetch for NixOS: https://github.com/NixOS/nixpkgs/pull/176819/

Oh thanks, that slipped through. Can you please bump the minor version number in CMakeLists.txt too? We shouldn't ship a version with this bug as latest.
And thanks for packaging on NixOS!

Done, bumped it.


Check light id ranges properly

In my addition of this feature I made a last minute range check for the light ID passed to the function.
As it turns out I didn't test this addition and the light ID check always fails..
Sorry for the noise :/

No problem!


How about make the ImageAnimator as the control and the Image as a View?

The problems I'm facing are:

it is hard for me to trigger the ImageOpened event when the PixelWidth and PixelHeight of an image file are analyzed.
The actions of Load and UnLoad are racing when I transfer the Image from one container to another one.

Both of these two problems need an object to hold the event and the best one is the ImageAnimator. If I can get the ImageAnimator by
var animator = new ImageAnimator(sourceUri, image);
, then I can get a non-null animator and attach event handler upon its event.
Furthermore, you can change its view by animator.SetViewer(image);
and change its SourceUri by animator.SetSourceUri(sourceUri);.
Then you can make a new UserControl which works like the MediaElement and its control is this ImageAnimator. Then the user can choose whether to use this one when they want to have some functions like play, pause and stop, etc. and the ImageOpened event can be easier to be embedded into this new UserControl. Hm... lot of work need to do. ^_^!!!
Just a suggestion. Never mind. ^_^

It's an interesting idea. I guess I could make ImageAnimator public. The constructor would have to be private though, because creating an ImageAnimator involves async operations, but the static CreateAsync methods could be exposed.

Furthermore, you can change its view by animator.SetViewer(image);

Yes, although I would probably pick a different name ; it would basically do the same thing as AnimationBehavior.SetAnimatorCoreAsync

and change its SourceUri by animator.SetSourceUri(sourceUri);.

That's a bit more involved, and provides no compelling benefit, as far as I can tell. You can just create another animator if you want to change the SourceUri.

Then you can make a new UserControl which works like the MediaElement and its control is this ImageAnimator. Then the user can choose whether to use this one when they want to have some functions like play, pause and stop, etc. and the ImageOpened event can be easier to be embedded into this new UserControl. Hm... lot of work need to do. ^_^!!!

I think it's out of scope for this project, but you can do it if you want ;)


new logo

Thank you for contributing to mirrord!
Please make sure you added a CHANGELOG entry.

bors r+


Suggestion: Better distinction between multiple hits that have different multipliers

Would be helpful for Multi-target configuration so you can tell which one you've selected.
Childe and Raiden for example:

Frzyc mentioned this in discord, restating here for record keeping
Could have info.hit.suffix for these and info.hit.numHits for Yoimiya normals (or other multi-hit)
Then display the suffix in the target selecter, and automatically populate the multiplier in the multi-target creator with numHits

This is more of an issue now that we can duplicate and move inidividual targets within multi-target since we don't know which multiplier version we're duplicating or moving.


arrays of references don't transpile

array<reference<User>> staff; produces an error panic: firemodel:: schema.firemodel:62:20: unexpected "<" (expected ">")

I was hoping to avoid this use-case for simplicity. Sounds like it'll be
required. Do you think firemodel needs arbitrary nesting, or is one level
sufficient?
array<array<reference>>
On Fri, Sep 14, 2018 at 2:40 PM Zak<EMAIL_ADDRESS>wrote:

array<reference> staff; produces an error panic: firemodel::
schema.firemodel:62:20: unexpected "<" (expected ">")
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mickeyreiss/firemodel/issues/14, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAC5LTh1gVm5ZOGAGKeIdP_MuzKI6TlVks5ubCI6gaJpZM4Wp_Py
.

for now I think one level is sufficient


NeuralNetworkGPU references value.target

I already have the fix for this, but want to track it.

Resolved with above reference.


reply_to_message property absent in group chats

When replying to a message, the message object should look something like this:
{ message_id: 35,
  from:
   { id: 37254959,
     first_name: 'Tom',
     last_name: 'Hansen',
     username: 'tomhansen' },
  chat:
   { id: 37254959,
     first_name: 'Tom',
     last_name: 'Hansen',
     username: 'tomhansen',
     type: 'private' },
  date:<PHONE_NUMBER>,
  reply_to_message:
   { message_id: 17,
     from:
      { id: 37254959,
        first_name: 'Hans',
        last_name: 'Tomsen',
        username: 'hanstomsen' },
     chat:
      { id: 37254959,
        first_name: 'Tom',
        last_name: 'Hansen',
        username: 'tomhansen',
        type: 'private' },
     date:<PHONE_NUMBER>,
     text: 'Hello.' },
  text: '/cmd1' }

The object above contains the reply_to_message property, because the message replies to another message. This works as expected when in a private chat, but not when in a group chat.
In a group chat the reply_to_message property is absent, even though the message replies to another message:
{ message_id: 35,
  from:
   { id: 37254959,
     first_name: 'Tom',
     last_name: 'Hansen',
     username: 'tomhansen' },
  chat: { id: -25291336, title: 'Group title', type: 'group' }
  date:<PHONE_NUMBER>,
  text: '/cmd1' }

Is there a reason for this?

Did you response a message from the bot?

The bot does get all the messages. The problem is that the message object does not have the reply_to_message property in group chats (but it does have it when you chat with the bot one on one).
The reply_to_message property should be there when the message is a reply to another message. This is illustrated in the image below:

In the illustration above, /translate is a reply to Dette er en test.. So the /translate message object should have the reply_to_message property, which contains the message being replied to, which is Dette er en test.. As mentioned above, this works fine when you chat with the bot one on one, like you can see in the illustration above, but it doesn't work in groups.The illustration below shows that:

As you can see, the bot didn't react this time, since it's in a group. This is the problem. The bot didn't react because the reply_to_message doesn't exist on the message object although it should. It would exist if I chatted to the bot directly, but since I chat to it in a group, it's not there for some reason.
Here's the code, to give you some context.

It seems reply_to_message is for bot messages only but it wold be good to have for every message too. Try contacting with http://telegram.me/BotSupport

For future reference I'd like to inform other users that might have the same problem. The reason for the absence of the reply_to_message property was that the bot was in privacy mode. All Telegram bots are in privacy mode by default, which limits them from receiving certain messages. More about that here. When I disabled this mode through the BotFather user, the reply_to_message property became present for all messages, which solved my problem.

That's not correct. According to Telegram docs bots should get the reply_to_message when users are replying to bot messages but that doesn't happen for me. However it does get the reply_to_message when users are replying to other messages using a bot command. All this in privacy mode.
The first case (users replying to bot messages) also doesn't include a reply_to_message not even when the bot is a group admin (which means the bot gets all the messages) nor in a private conversation between the user and the bot.


Add support of ignore_eos to mistral vllm truss

Add support of ignore_eos
Add support of stream
Add support of customizable prompts format & repo_id

@joostinyi changed to requirements and also simplified bunch of simplifications.


The pyo3 macros do not retain code spans

🐛 Bug Reports
When reporting a bug, please provide the following information. If this is not a bug report you can just discard this template.
🌍 Environment

N/A:
N/A:
N/A:
1.39 and 1.40 both reproduce this:
Reproducible on both 0.7 and 0.8

💥 Reproducing
use pyo3::prelude::*;
use pyo3::types::PyBool;

struct Foo<T>(T);

impl<T> Foo<T> {
    fn push(&self, t: T) {
        unimplemented!()
    }
}

#[pyclass]
struct PyFoo {
    inner: Foo<u64>,
}

/// Implement a push method for sinks.
macro_rules! implement {
    ($(fn push(self: &mut $self:ty, $py:pat, $val:pat) $body:expr)*) => {
        $(#[pymethods]
        impl $self {
            fn push(&mut self, $py: Python, $val: PyObject) -> PyResult<Option<()>> {
                match $body {
                    Ok(()) => Ok(Some(())),
                    Err(e) => Err(to_pyerr(e)),
                }
            }
        })*
    }
}

implement! {
    fn push(self: &mut PyFoo, py, object) {
        let v: &PyBool = object.cast_as(py)?;
        py.allow_threads(|| self.inner.push(&v.is_true()))
    }
}

will give errors as such:
error[E0425]: cannot find function `to_pyerr` in this scope

error[E0308]: mismatched types
  |
  = note: expected type `u64`
             found type `&bool`

error[E0308]: mismatched types
  |
  = note: expected type `()`
             found type `std::result::Result<_, _>`

error: aborting due to 3 previous errors

As soon as the #[pymethods] macro is removed, the error messages regain the span.

Thank you.
Currently, our proc-macro code is crafty and we should refactor it with a more carefully manner :sweat:

I've got another reproduction for this:
use pyo3::prelude::*;
use pyo3::types::{PyTuple};

#[pymodule]
fn invalid_module(_py: Python, m: &PyModule) -> PyResult<()> {
    #[pyclass(freelist = 1000)]
    struct UUID {
    }

    #[pymethods]
    impl UUID {

        fn broken_function(
            fields: Option<Py<PyTuple>>,
        ) -> Result<Self> {
            unimplemented!()
        }

        // Using &PyTuple fixes the code span problem
        // fn good_function(
        //     fields: Option<&PyTuple>,
        // ) -> Result<Self> {
        //     unimplemented!()
        // }
    }
}

For some reason the Option<Py<PyTuple>> type is problematic and breaks code spans, Option<&PyTuple> works fine and the errors display as normal.

As a follow up, I've retested both of the examples above with PyO3 0.13.1 and Rust 1.49.0, and I get error messages with correct spans for both of these examples.
Over time we've slowly improved a few of these cases, so I'm going to close this issue as fixed and we can resolve individual cases in the future if we should come across more.

As a follow up, I've retested both of the examples above with PyO3 0.13.1 and Rust 1.49.0, and I get error messages with correct spans for both of these examples.
Over time we've slowly improved a few of these cases, so I'm going to close this issue as fixed and we can resolve individual cases in the future if we should come across more.


Coming from browserify - can't register my root component?

Hey,
I'm trying to port an app over from a browserify based approach, so I expect I'll have a bunch of questions.
Right now I'm trying to get my root component registered as a starting point. I've created a directory under bundle, and placed it (and all its children - for now, though I will properly organise this code later) into its components directory.
In startup I've made a clientRegistration file, but this is clearly not being run - I've put a console.log in it and see nothing.
Where am I likely to be going wrong? In my view I have:
<%= react_component("Piper") %>

in my bundle directory I have
bundle
\
 piper
  \
   components
    \
     Piper.jsx
     {+ others}
   containers
   startup
   \
    clientRegistration.jsx

the Piper component is written in ES5 right now (that wont be a problem, right? I'm hoping to move to ES6 soon, but for now...), and contains:
var React = require("react");
Piper = React.createClass({
//...
});

module.exports = Piper;

And the clientRegistration file contains:
console.log('registering piper?');
var ReactOnRails = require('react-on-rails');
var Piper = require('../components/Piper');

ReactOnRails.register({ Piper});

The console.log never fires, and I see the following error output:
Exception in rendering!
exports.default(options=Object { e=Error: Could not find component registered with name Piper. Registered component names include [ HelloWorldApp ]. Maybe you forgot to register the component?, name="Piper", serverSide=false})app-bun... > eval (line 51)
render()app-bun... > eval (line 108)
forEach(fn=Object { type="object"}, className="js-react-on-rails-component")app-bun... > eval (line 64)
forEachComponent(fn=Object { type="object"})app-bun... > eval (line 54)
reactOnRailsPageLoaded()app-bun... > eval (line 120)
clientStartup/<()app-bun... > eval (line 159)

console.error('Exception in rendering!');

location: http://localhost:3000/assets/generated/app-bundle.self-fb38a30c1f13e994233fe0ad760828cf1e4fcce827f6dc07f50459143b4b225c.js?body=1 line 1125 > eval:69
exports.default(options=Object { type="object"})app-bun... > eval (line 60)
render()app-bun... > eval (line 108)
forEach(fn=Object { type="object"}, className="js-react-on-rails-component")app-bun... > eval (line 64)
forEachComponent(fn=Object { type="object"})app-bun... > eval (line 54)
reactOnRailsPageLoaded()app-bun... > eval (line 120)
clientStartup/<()app-bun... > eval (line 159)

console.error('location: ' + e.fileName + ':' + e.lineNumber);

message: Could not find component registered with name Piper. Registered component names include [ HelloWorldApp ]. Maybe you forgot to register the component?
exports.default(options=Object { type="object"})app-bun... > eval (line 63)
render()app-bun... > eval (line 108)
forEach(fn=Object { type="object"}, className="js-react-on-rails-component")app-bun... > eval (line 64)
forEachComponent(fn=Object { type="object"})app-bun... > eval (line 54)
reactOnRailsPageLoaded()app-bun... > eval (line 120)
clientStartup/<()app-bun... > eval (line 159)

console.error('message: ' + e.message);

Any pointers?

Actually, this is me being foolish - clearly registration only occurs on server start, I was expecting to see some attempt to register on page refresh. Restarting the server gives me error output that indicates why the component failed to register, so for now I'm good.


WIP: fix empty lookup commitment

Related to https://github.com/MinaProtocol/mina/pull/14064/

Fixing in https://github.com/o1-labs/proof-systems/pull/1241

Old, closing.


Remove backtick from dialog_properties_spec.rb

The backtick causes the spec to shell out, but the will command
doesn't exist, so instead you get "No such file or directory - will" in
STDERR.
@bdunne Please review
cc @gmcculloug

😧 This is the second time.  I need to remove that will alias from my system.


Support redis registry center

Feature Request
For English only, other languages will not accept.
Please pay attention on issues you submitted, because we maybe need more details.
If no response anymore and we cannot make decision by current information, we will close it.
Please answer these questions before submitting your issue. Thanks!
Is your feature request related to a problem?
No!
Describe the feature you would like.
current registry center only zookeeper,so could we provide other registry center like redis or etcd etc...

This issue is not necessary, I just keep it as volunteer wanted to open discuss and seek for contributor who want to do it.

Want Nacos as a registration center

Because they are unnecessary, and it seems like nobody want to do it, I just close this issue.


Add possibility to insert Payment Methods (especially credit card) without spaces

Hey! Thanks for this cool project, I just tried it out today and stumbled upon this thing which makes it unusable for my app.
However, I would really like to use it and can also contribute a PR if you would want to have thia as a feature.
Is your feature request related to a problem? Please describe.
When pasting credit card information from the TestCard App into our app it doesn't fully paste the credit card number, because our app is doing some matching and limits the length of the credit card based on the provider. Because the credit cards in your "database" contain spaces for making them more readable only half of the credit card is inserted into our text field
Describe the solution you'd like
I would like to have a checkbox in the AutofillScreen that would allow pasting card numbers without spaces in between
Describe alternatives you've considered
Of course we could copy paste the test credit card numbers and store them somehow in our app, but I like this dynamic approach much more and woul really like to use it
Additional context
🔴

Hah! Sure, I can open a PR for this, would you want this to be an option that can be selected or is it fine for you if the spaces " " always get "replaced" with ""?

It's fine if the spaces just get replaced. I can't think of a use case where someone would need the spaces. IMO it would probably be best to replace the spaces where we build the autofill results (in CreateDatasetUseCase). This will keep the spaces in the UI, making the numbers nicely readable, but should solve your issue


Use TTL from ODSP createBlob response

Description

This PR is a follow up of closed PR https://github.com/microsoft/FluidFramework/pull/14854

Trying to solve https://dev.azure.com/fluidframework/internal/_workitems/edit/1318

First thing tackled here is skipping reupload of stashed blobs that were previously successfully uploaded and have not expired (less than half TTL has passed), meaning that server should still have such blobs. This is done in the constructor addition.

Second thing is on resubmit, throw if blobs have expired and about to send a BlobAttach op. BlobManager could think it is safe to send the op to other clients but blob won't be on server anymore.

could you break apart some the changes here into more specific PRs? Feel like a lot is happening here.

@anthony-murphy @wes-carlson @dannimad
@alexvy86 and I found that the recent build failure in Build - client packages pipline is due to the test failure modified by this PR. I have attached a few examples below for reference.
Pipeline Failures

https://dev.azure.com/fluidframework/internal/_build/results?buildId=160673&view=ms.vss-test-web.build-test-results-tab&runId=3903669&resultId=102887&paneView=debug
https://dev.azure.com/fluidframework/internal/_build/results?buildId=159863&view=ms.vss-test-web.build-test-results-tab
https://dev.azure.com/fluidframework/internal/_build/results?buildId=159724&view=ms.vss-test-web.build-test-results-tab
https://dev.azure.com/fluidframework/internal/_build/results?buildId=159374&view=ms.vss-test-web.build-test-results-tab


Frozen core for NEVPT2

Hi,
Correct me if I am wrong, there is no frozen core option for NEVPT2. I can only find the frozen core option for the CCSD and MCSCF modules.
Best
Thomas

Frozen core is not available for NEVPT2.  I'll consider this feature when optimizing NEVPT2 in the future.

The NEVPT2 method prints the eight contributions to the second order energy correction, e.g.
Sr    (-1)',   E = -0.09815626499877
Si    (+1)',   E = -0.00002355594769
Sijrs (0)  ,   E = -0.02394811313138
Sijr  (+1) ,   E = -0.00056142061480
Srsi  (-1) ,   E = -0.01137603732026
Srs   (-2) ,   E = -0.30714701720375
Sij   (+2) ,   E = -0.00001148262247

and each contribution corresponds to one class of occupations in core and virtual space (I assume PySCF is following the notation of https://doi.org/10.1063/1.1515317)).
Only Sr(-1) and Srs(-2) involve excitations that leave the core electrons untouched, so I would say the sum of Sr(-1) and Srs(-2) should be the NEVPT2 energy correction in the frozen core approximation. Is that right?

That is true if you want to freeze all the inactive orbitals (aka core).
In most practical cases, however, one wants to freeze some number of non-relevant (like deep-core 1s) orbitals to make calculations faster/cheaper.


How to Run Only Model Inference On Nano and Rest on Rasp?

I have a MASK RCNN model which takes a frame, processes it to detect objects, and then displays them on the screen. When running without displaying the resulting frames on screen (without cv2.imshow()), the program takes about 31 MB RAM. With cv2.imshow(), however, it takes about 3 GB RAM to display the resulting output.
My question is, since I have an 8GB raspi and 2GB RAM Jetson Nano, can I run the model inference only on the Nano, and do the cv2.imshow() with the raspi, so that the processing can be smooth?

Hi @Hana-Ali, I doubt the program only takes 31MB RAM to process MASK-RCNN, because I believe the MASK-RCNN model itself is larger than that.  Perhaps you see the actual 3GB RAM usage during cv2.imshow() because it keeps your program running longer for a chance to see the actual usage.
You would need to transmit the image to the raspi somehow, but once received yes in theory it could work.
You may also want to disable ZRAM on your Nano and mount additional swap to give it a chance to work there:
https://github.com/dusty-nv/jetson-inference/blob/master/docs/pytorch-transfer-learning.md#mounting-swap
Another thing I do is use X11 tunneling over SSH to do cv2.imshow() on my Windows laptop.  I use MobaXterm and it has a built-in X11 server.  Its nice because it doesn't require you to explicitly transmit the frame at all, you just run cv2.imshow() like normal on your Jetson except it actually displays on your remote PC.  It works fine for smaller images (i.e. not streaming HD video - for that you want to do compressed RTP feed)

Hey @dusty-nv, thinking about it now, you do have a point, but then I’m not sure how to check how much RAM is being used up by the program with/without .imshow
As for the ZRAM, I actually haven’t tried that yet, so I will try it soon hopefully and get back with results!
Im not sure I fully understood MobaXterm and the last paragraph, but I’ll look into it - anything to make the model run faster, really.
Also, I just want to ask - since MRCNN is known to be quite heavy and slow, would you recommend I just go with some YOLO model instead, or try with ZRAM and tunneling? Afair both have very high precision, YOLO slightly lower (but also being lighter), so I’m not really sure if MRCNN is worth the hassle with a 2GB Nano.
Thank you so much!

MaskRCNN is primarily used for instance segmentation (+detection), so if you only need detection then yea I would go with YOLO or SSD-Mobilenet.

From: Hana-Ali @.>
Sent: Tuesday, July 27, 2021 4:23:05 PM
To: dusty-nv/jetson-inference @.>
Cc: Dustin Franklin @.>; Mention @.>
Subject: Re: [dusty-nv/jetson-inference] How to Run Only Model Inference On Nano and Rest on Rasp? (#1164)
Hey @dusty-nvhttps://github.com/dusty-nv, thinking about it now, you do have a point, but then I’m not sure how to check how much RAM is being used up by the program with/without .imshow
As for the ZRAM, I actually haven’t tried that yet, so I will try it soon hopefully and get back with results!
Im not sure I fully understood MobaXterm and the last paragraph, but I’ll look into it - anything to make the model run faster, really.
Also, I just want to ask - since MRCNN is known to be quite heavy and slow, would you recommend I just go with some YOLO model instead, or try with ZRAM and tunneling? Afair both have very high precision, YOLO slightly lower (but also being lighter), so I’m not really sure if MRCNN is worth the hassle with a 2GB Nano.
Thank you so much!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/dusty-nv/jetson-inference/issues/1164#issuecomment-887809367, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADVEGK3S3QTNGJKUDDCZLEDTZ4ISTANCNFSM5BC57O4A.


Using global context when creating a tracker causes crashes on API 23

Describe the bug
Initializing the sdk with a global context on Android 6 throws an exception. I have tried both with a static context and with context generator.
FATAL EXCEPTION: main                                                                                Process: com.example.snowplow_test, PID: 17105                                                                                java.lang.NoClassDefFoundError: com.snowplowanalytics.core.globalcontexts.GlobalContextPluginConfiguration$$ExternalSyntheticLambda0                                                                                at com.snowplowanalytics.core.globalcontexts.GlobalContextPluginConfiguration.getEntitiesConfiguration(GlobalContextPluginConfiguration.kt:28)                                                                                at com.snowplowanalytics.snowplow.configuration.PluginConfigurationKt.toStateMachine(PluginConfiguration.kt:62)                                                                                at com.snowplowanalytics.core.tracker.ServiceProvider$makeTracker$builder$1.invoke(ServiceProvider.kt:293)                                                                                at com.snowplowanalytics.core.tracker.ServiceProvider$makeTracker$builder$1.invoke(ServiceProvider.kt:262)                                                                                at com.snowplowanalytics.core.tracker.Tracker.<init>(Tracker.kt:356)                                                                                at com.snowplowanalytics.core.tracker.ServiceProvider.makeTracker(ServiceProvider.kt:297)                                                                                at com.snowplowanalytics.core.tracker.ServiceProvider.getOrMakeTracker(ServiceProvider.kt:187)                                                                                at com.snowplowanalytics.core.tracker.ServiceProvider.<init>(ServiceProvider.kt:87)                                                                                at com.snowplowanalytics.snowplow.Snowplow.createTracker(Snowplow.kt:237)                                                                                at com.example.snowplow_test.MainActivity.initializeSnowplow(MainActivity.kt:68)                                                                                at com.example.snowplow_test.MainActivity.onCreate(MainActivity.kt:37)
To Reproduce
To reproduce this issue, create a new Android app from template with minimum Android SDK 23.
Add snowplow library (version 5.0.0 - 5.1.0, 4.x.x has worked for us previously).
Initialize the sdk in MainActivity onCreate() with a global context sent to Snowplow.createTracker()
Pasting in from documentation here will cause the crash:
https://docs.snowplow.io/docs/collecting-data/collecting-from-own-applications/mobile-trackers/custom-tracking-using-schemas/global-context/#static-entities
Run the app on a device or emulator with Android 6
Expected behavior
Snowplow will have initialized and is ready to send events
Device information (please complete the following information):

Device: Any device or emulator with Android 6 Marshmallow
OS: Android 6
Browser
Version

Hi @frodstan,
I think this is caused by the fact that the global context implementation uses Java 8 features underneath.
In order to make this work on older Android SDKs, you will need to add Java 8 desugaring support to your app. You can follow the instructions here. Basically you need to use Android Gradle Plugin v4 and add this to your build.gradle:
android {
    compileOptions {
        // Flag to enable support for the new language APIs
        coreLibraryDesugaringEnabled true
        // Sets Java compatibility to Java 8
        sourceCompatibility JavaVersion.VERSION_1_8
        targetCompatibility JavaVersion.VERSION_1_8
    }
}

dependencies {
    coreLibraryDesugaring 'com.android.tools:desugar_jdk_libs:1.2.2'
}

You can see the build.gradle for our demo app which has min SDK version set to 21 and works with global context as an example.

Awesome, this fixed the issue.
Thank you!

Hi @matus-tomlein ,
I know that this problem is already "solved" using the proposed workaround, but Is there any other way to do it that does not involve having to add another dependency other than snowplow?
I´d prefer not having to include this third dependency


Update path to Code Sniffer rules for Magneto 2.2

The path to code sniffer rules in Magento 2.2 was changed from dev/tests/static/testsuite/Magento/Test/Php/_files/phpcs/ruleset.xml to dev/tests/static/framework/Magento/ruleset.xml. The latter one also exists in 2.0 and 2.1, but is not complete. The easiest solution for now would be to use dev/tests/static/framework/Magento/ruleset.xml in all Magento versions.

https://github.com/paliarush/magento2-vagrant-for-developers/pull/136 resolves this issue

#136 fixes the issue for new installations (or after init_project.sh -f is used).
Existing installations may still be affected by this issue. It be solved by updating to the latest version of this project and running init_project.sh -f in the root.


Remove assert_df

assert_not_none does the same thing ... albeit with different error message defaults, but that's not enough to justify keeping assert_df.

Thanks - I introduced assert_df to get around some mypy complaints, but I was clearly on the wrong track. Very happy to see it yanked out.


[localization] Add support to checks.h for localized messages

Sorry @BillyONeal, this included unrelated changes (because #303 wasn't merged before this); could you re-look at this?

Thanks Billy!


[BUG] Humble ros_bridge does spatial detection missing fields

Describe the bug
Humble implementation of the ros msg converter does not provide a score value for spatial detections
Expected behavior
The score value should be set into the ros msg
Additional context
CLICK ME - The Line in code where the fix should be added
something like this should be added in the IS_HUMBLE ifdef
opDetectionMsg.detections[i].results[0].score = inNetData->detections[i].confidence;

which results in a ros2 topic echo like this:
header:
  stamp:
    sec:<PHONE_NUMBER>
    nanosec: 427740811
  frame_id: oak_rgb_camera_optical_frame
detections:
- results:
  - class_id: '0'
    score: 8.31591796875
  bbox:
    center:
      position:
        x: 78.0
        y: 250.0
      theta: 0.0
    size_x: 200.0
    size_y: 252.0
  position:
    x: -0.19737829267978668
    y: -0.07760991901159286
    z: 0.5516734719276428
  is_tracking: false
  tracking_id: ''
---

Hello @kikass13 i have no comment of the bug I just have a question on how do I run the bridge files
I want to do SpatialDetection using the yolov4_spatial_publisher.cpp example which I am able to run with the launch file,
Now I want to convert the topic been sent from the node and I believe that's what TrackSpatialDetectionConverter.cpp is for
however I can figure how to run it, can you please explain how I can?

Any idea on how to publish the score in the meantime?

Hi, thanks for the report, the fix is available here and will be merged soon

Hello @Serafadam!
Was the fix merged to the humble branch?


< characters are gratuitously removed

This is unfortunate if, like me, you like to type ASCII hearts.
<3

My original line of thinking was to remove HTML in the cross-posting, since its functionality could not be guaranteed on Twitter. Of course, your ASCII heart is not HTML and therefore it should not be modified. It would seem the HTML-stripping code is doing too much.

Just as I thought! :)
Thanks for your hard work oin this project, by the way.

This is happening in MLStripper.py.
s = MLStripper()
s.feed(html)

I would have expected the stripper to remove HTML tags.  It's removing angle brackets, too.
@strugee My pleasure.

Let me know if this doesn't fix it. :-)


Provide context on unexpected EOFs.

https://github.com/square/okhttp/issues/1114

Looks like a few guys are asserting for EOF but you changed the exception type.
Tests in error:
  URLConnectionTest.getKeepAlive:2409 » Protocol unexpected end of stream on Con...
  URLConnectionTest.postBodyRetransmittedOnFailureRecovery:2633->assertContent:3098->assertContent:3094 » Protocol
  URLConnectionTest.sslFallbackNotUsedWhenRecycledConnectionFails:635->assertContent:3098->assertContent:3094 » Protocol
  URLConnectionTest.serverShutdownOutput:403->testServerClosesOutput:430->assertContent:3098->assertContent:3094 » Protocol
  CallTest.postBodyRetransmittedOnFailureRecovery:949 » Protocol unexpected end ...

LGTM though.


missing function and structure documentation

Some start point is done. Please check PR's
Create a channel stantard
Docs - Add documentation for sensors
Docs - Add documentation for structures

Pending two PR's:
Docs - Add documentation for sensors
Adc - Upgrades and doc


Initial crop marker size

Thank you for great package,
Add a parameter to choose initial crop marker size to be entire image width, height or default(Center)

@sharpsaravanan
Thank you for your feedback! I’ve added a new useInitialFullCrop property to address this. You can now use it to set the initial crop marker size to the entire image width, and height. Let me know if you have any other suggestions!


USER-PACE licence is incompatible with LAMMPS license

The inclusion of the USER-PACE package (and distribution of the resulting binary) is a violation of the LAMMPS GPL-2.0-only license.
If we want to respect the intent of the LAMMPS developers, we should not include this package.

It was merged in https://github.com/lammps/lammps/pull/2699 so I guess we anyway do not need to include it as separate package.

The "-D PKG_USER-PACE=ON" option in build.sh causes the PACE code to be downloaded and compiled into lammps.  The resulting binary violates the LAMMPS GPL-2.0-only license and cannot be legally distributed.
There is no fixing this unless PACE changes it's license to GPL-2.0.  Thus, at this time PACE cannot be built into LAMMPS and distributed through conda forge.

I just noticed that they changed it recently - somehow I missed that https://github.com/lammps/lammps/commit/9a8ac23663027f319c98ed4a08d4cda55faec606

Yeah, they argue that it's always been GPL-2.0-only, but there were errors in the license file.
See https://github.com/lammps/lammps/issues/2643

Does conda-forge/lammps distribution includes, for example, QUIP package? AFAIK, we have similar "license incompatiblity" situation as in LAMMPS/QUIP package, i.e. QUIP also has  non-GPL2 license.

Also, GPL2-only licensing implies strict limitation on the 3rd party libraries we could use in the development, and since the USER-PACE C++ codebase serves as a core for our ACE ecosystem codebase,  extra limitation  would significantly slow down the development process.
For example, soon, we are planning to release multispecies implementaiton of ACE (current public USER-PACE is single-specie only), and we are using there yaml-cpp library (MIT license). So, even if we rollback the GPL3 -> GPL2 license for USER-PACE now, the problem will appear again in a short time. That is why we made deliberate choice to distribute USER-PACE as downloadable on the user side library.

I'm not sure if LAMMPS support dynamically linked libraries for pair_styles. Maybe in that case LAMMPS and USER-* packages under non-GPL2 licenses could be distributed separately...

The QUIP package is GPL 2, so I think it is OK.
I think these licensing incompatibility issues are unfortunate, but there seems to be only two choices: follow the licenses or ignore them.
Because LAMMPS is released under GPL-2.0-only, it is simply not possible to use GPL-3.0-* licensed with it.  The MIT license is GPL compatible, so that shouldn't be a problem, but GPL-3.0 is.
The downloadable library is a good way to go, but it doesn't eliminate the problem if you want to create and distribute prebuilt binaries (like with conda-forge).


Document that plugins can require Electron

Hi everyone,
I tried my best on #398, this is my second PR.
If you think I should add something else, please tell me.
Thank you

Awesome, thank you!


Error when build proyect

I have cloned your github project and updated the dependencies to the latest versions, but when I run the sample I skip this error:
Error:Execution failed for task ':library:transformClassesWithRetrolambdaForDebug'.

Missing javaCompileTask for variant: debug/0 from output dir: /Users/juangra/Proyectos/Proyectos_pruebas/CustomizableCalendar-master/library/build/intermediates/transforms/retrolambda/debug/0

My pc is mac and Android studio is 3.0.
Help me please.

Hi @jgcorrales can you try the last release v0.1.4?


cf delete does not provide correct prompt

Command
cf delete myapp

What occurred
Returns "Really delete the app myapp?>" in the command prompt.

What you expected to occur
It does not say if the user should type "y" or "yes" and then hit enter.
CLI Version

CC API Endpoint Version

CF Trace

Platform & Shell Details

Any other relevant information

It never has. It's a simple yes/no question though, I have never heard of people confused before?
We are refactoring our commands and using another library for displaying the prompts. Once cf delete has been refactored, it will include [yN]. (You can already see that in refactored commands like cf delete-org.)

Closing as I believe my comment above addresses the point raised.


Open ssl fips 0 9 7 stable

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

This looks like a mistake


The Aspire project has been successfully deployed. Can new services automatically register themselves, or is there a configuration that allows for adding new services without the need to re-deploy the Aspire project?

The Aspire project has been successfully deployed. Can new services automatically register themselves, or is there a configuration that allows for adding new services without the need to re-deploy the Aspire project?

Can you clarify what "register" a new service means? Are you talking specifically about service discovery?

您能澄清一下“注册”新服务的含义吗？您是在具体谈论服务发现吗？

can add existing projects to the Aspire dashboard or add new projects to the Aspire dashboard after deployment without the need to recompile or restart the Aspire project.

Can you clarify what "register" a new service means? Are you talking specifically about service discovery?

can add existing projects to the Aspire dashboard or add new projects to the Aspire dashboard after deployment without the need to recompile or restart the Aspire project.

No, you can't.

No, you can't.

ok   but why?


Can't install OpenMEP Revit 2025.2

OpenMEP was working fine until I instal new Revit 2025 update, basically after the update installation all packages in Dynamo was uninstalled, and I installed all again and also OpenMEP. I install newest OpenMEP but Dynamo not see the OpenMP, I see the "red cross" that Dynamo is telling me that this package is missing, I try older version and the same. Restart also not help. Only I have this issue?

Hi @jacekgrab1 , thank you for your report, can give me more detail about version revit you are using and dynamo version updated?Also include the list package has installed in your computer to investigate

Dynamo version:

Revit version:

I have only these packages instaled:

Hi @jacekgrab1 , do you have any warning issue meesage from Dynamo ?

No, basically everything was ok during installation of OpenMEP but Dynamo not see this package and not give any warnings about this. But only I have that isue? But only I have that is issue? OpenMEP works fine for news Revit update for others?

It's working fine in my computer with base revit 2025, I need check with the latest update of revit 2025 version first, may be in Dynamo version have some patch update

Look from Revit update release, I saw it have changed with Dynamo Core but not sure which one has changed :https://help.autodesk.com/view/RVT/2025/ENU/?guid=RevitReleaseNotes_2025updates_2025_2_html
Now I'm so busy with my work , I will try take a look on this week.

ok, please let me know if this only my issue or more people has this, I appreciate your commitment with this

Hi @jacekgrab1 , I just quick fix for it with revit 2025.2 use dynamo 3.2, you can retest with latest version release.
Regard


Solid black border below tab when background color is non-black

Ubuntu 16.04

Also seeing this - looks like the border-bottom color might be hardcoded:

Problem line: https://github.com/zeit/hyperterm/blob/master/app/lib/components/tab.js#L85
It's not just a matter of making it transparent because .tabs has it's own border and the top-left is padding not an element meaning the .active tab probably needs to pull in the background color. https://github.com/zeit/hyperterm/blob/master/app/lib/components/tabs.js#L73

I took a look at this, and I'm having trouble passing in the backgroundColor from the props to the aphrodite styles object. I'm new to React/Redux, though, so there's a good chance I'm just doing it wrong.
When css is called, the context has changed (maybe because of the wrapper helper method?) and the tab's props are no longer available in styles. I'll take another look, and apologies if I'm missing something obvious!

For the time being, I hacked around this by adding some main window CSS:
.tab_active:before { border-color: $YOUR_BG_COLOR }


Why we need await Promise.resolve() in enqueue

(async () => {
	// This function needs to wait until the next microtask before comparing
	// `activeCount` to `concurrency`, because `activeCount` is updated asynchronously
	// when the run function is dequeued and called. The comparison in the if-statement
	// needs to happen asynchronously as well to get an up-to-date value for `activeCount`.
	await Promise.resolve();

	if (activeCount < concurrency && queue.size > 0) {
		queue.dequeue()();
	}
})();

Can you give me some concrete example? Thank you!

https://github.com/sindresorhus/p-limit/pull/28


enable dumplog threading

Test helm chart

replacing by https://github.com/corda/corda-runtime-os/pull/1901


timestamp last action

Hi,
would it be possible for Wallpanel to broadcast a value "time(stamp) of last user action" in its MQTT data, next to "state"?
I'm looking for a way to have Wallpanel return to its main page e.g. five minutes after last user interaction. If I knew when the user had left (= when he touched the screen last), I could send the "relaunch" command via MQTT.
I tried to use the "state" value which would give the URL currently displayed. But this value isn't updated "on change", but only after some time. Additionally, it wouldn't register if the user is still working with a subscreen (in contrast to "no more touches" = user has left).
Could a value like this be implemented, or am I overlooking an easier way to reach my goal?
Regards

Currently, the only user present methods you have are url change message, motion detected, and face detected.  I think this is could also be something implemented with the website you are using, if you want that page to return to the main page after a period of time, there are already javascript solutions for this.     Please follow the feature guidelines and do not combine feature requests into a single post.

Currently, the only user present methods you have are url change message, motion detected, and face detected.  I think this is could also be something implemented with the website you are using, if you want that page to return to the main page after a period of time, there are already javascript solutions for this.     Please follow the feature guidelines and do not combine feature requests into a single post.

URL change: As described, I can't use that because a) it's too slow and b) it doesn't reflect simple touch event (that don't lead to a different page.)
motion/face: I don't have the camera enabled on my input tablets.
JavaScript: Doable for sure, but very complex.
Mixing topics: What do you mean? My feature request was for the "time of last user action" value via MQTT. Everything else are just workarounds that won't work for me, which are why I made the request. What else should I do?

URL change: As described, I can't use that because a) it's too slow and b) it doesn't reflect simple touch event (that don't lead to a different page.)
motion/face: I don't have the camera enabled on my input tablets.
JavaScript: Doable for sure, but very complex.
Mixing topics: What do you mean? My feature request was for the "time of last user action" value via MQTT. Everything else are just workarounds that won't work for me, which are why I made the request. What else should I do?

Just use the feature request form for that one feature "adding timestamp" to outgoing json payloads.   But to me this is kind of edge case because in your specific case, the MQTT message is "slow" when the url changes and you have a webpage that appears to have multiple pages, so its allowing users to navigate away from your page.   The app reporting the URL change is slow would be a "bug" if that were true (or possible improvement) and adding the time stamp would be your request.

Just use the feature request form for that one feature "adding timestamp" to outgoing json payloads.   But to me this is kind of edge case because in your specific case, the MQTT message is "slow" when the url changes and you have a webpage that appears to have multiple pages, so its allowing users to navigate away from your page.   The app reporting the URL change is slow would be a "bug" if that were true (or possible improvement) and adding the time stamp would be your request.

The URL change is posted only when the app first launches and the background service runs and when the web url is changes remotely.  It is not sent out when the user changes the url using the web page loaded in the app.

The URL change is posted only when the app first launches and the background service runs and when the web url is changes remotely.  It is not sent out when the user changes the url using the web page loaded in the app.


add a feedback icon that link to the feedback form

Once click the icon it will bring user to the feedback form page

Looks good 👍


Add more spacing to oboarding view

Summary

Add more spacing to oboarding form fields
Checklist

[x] My code follows the code style of this project and passes make format.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] My change has adequate unit test coverage.

@homanp Thank you for the pull request! Overall, the code looks good. One improvement I would suggest is to remove the duplicate code for the "company" field. You can combine the two FormField components into one, like this:
<FormField
  control={form.control}
  name="company"
  render={({ field }) => (
    <FormItem className="flex-1">
      <FormLabel>Company name</FormLabel>
      <FormControl>
        <Input
          placeholder="Enter your company name"
          {...field}
        />
      </FormControl>
      <FormMessage />
    </FormItem>
  )}
/>

This will make the code cleaner and more maintainable. Keep up the good work!
--
Note: This comment was generated automatically by Superagent Code Reviewer 🥷


added a few things to INSTALL.md

Added a few things for the SNObj code to work. I don't see a clear problem in using LSST_DIR as an environment but would tend to be wary of this, since that is the kind of variable that is automatically setup by eups, and so there might be unforseen clashes.

This too has been superceded by work in #164 and elsewhere...


Feature Request: A stepped random shape for the [LFO]

It would be very nice to have a random shape in the LFO Operator. It might be possible to create this with a combination of Ops, but having it built in to the LFO would be a "nice to have". Also a damping parameter so you could have stepped as well as sloping random oscillations.

yes great idea ... i would also like a Trigger input for the lfo ..

[SampleCurve] gets pretty darn close to this feature (the response curve idea).

Good idea. Took a slightly longer than I expected because I had to find a good hash function.
Available on the dev branch and in next release:

Brilliant, thanks!


Can't exit claim via teleport when exit-claim is false

GPDebug:

Unable to reproduce this with latest build.


rename estimate to due

closes #85

@rultor merge

@rultor merge

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

@rultor merge

@h1alexbel Done! FYI, the full log is here (took me 1min)


refactor: modify output formatting of info command

Signed-off-by: Xiao An<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
Remove redundant spaces by refactoring output formatting of  pouch info subcommand.
E.g., fmt.Fprintln(os.Stdout, "Containers: ", info.Containers):
since fmt.Fprintln() insert spaces between operands automatically, there will be 2 spaces between "Containers:" and the number of containers. Besides, I think using string formatting (fmt.Fprintf) would be a little bit more intuitive than string concatenation.
Ⅱ. Does this pull request fix one issue?

No.
Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
No need.
Ⅳ. Describe how to verify it
No need.
Ⅴ. Special notes for reviews

Well, the PR remind me that if we can change pouch info behavior use template.Template like pouch inspect $id, so fmt.Println can be removed.

For the update this pr involved, LGTM


Add feature flag service

To make quickpizza more useful for demoing k6 x Tempo, we should add a feature flag service and allow fault simulation.

+1, I think quickpizza stands to become a really good demo app. I'd like to use it for the demo block that SEs are giving. So building it out to help us show it off is a good idea.

Related to #6 --> Most of those should be under a feature flag.


[Debugger Integration Tests] Enable step-over instruction related tests for recursive function contexts

Description:
Debugger integration cases that are related to the recursive function contexts are added with https://github.com/ballerina-platform/ballerina-lang/pull/35261/commits/f1ca3f19b5edef7def8f9edd84804d2c77115c69. However some of the step-over instruction related tests cases are currently disable due to the debugger side limitations and therefore should be enable once the step-over instruction support within recursive functions is properly fixed.
Describe your task(s)
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

We are closing this issue due to inactivity. If you need further assistance or have additional input, feel free to reopen it. Thank you for helping us maintain a relevant and focused issue list!


Typing lag in recent versions of Terminus

Looks like the more recent versions of Terminus have significantly more lag when typing in the console. I just installed V35 and it has noticeably less lag. It would be great if the lag could be removed with the latest version of Terminus!
Also want to say great work with Terminus, it is an awesome program & I use it everyday.

I've just tried v35 and can't notice any difference. Are you using any plugin? (clickable-links specifically)
If you can, please capture a performance trace from DevTools when typing (see https://developers.google.com/web/tools/chrome-devtools/evaluate-performance/reference#save)

Thanks for looking into this! I just poked around for an hour or so and it looks like this might be pretty complicated.
I'm not using any plugins.
 Some screenshots of dev tools and some performance traces. Looks like, on average, the keydown handlers in v47 are a couple of milliseconds longer than those in v35 but there isn't that big of a difference. I saw a bigger difference in the framerate in v35 vs v47. Here are some screenshots of the FPS counter. Take a look at the distribution of frame rates, to the right of the larger graph in the frame rate layover. It looks like was roughly around 15-20fps in v47 and roughly around 22-25fps in v35. I didn't see any indicator as to why there is this difference, however.
I'm on Windows 10 and have a NVIDIA GTX 1070 and an<EMAIL_ADDRESS>

Let me know if you have any questions!
files.zip

I'm still having this issue on the current build. v35 is noticeably more fluid when typing, and especially so if I hold down a key. Here are two performance traces, one of the v62 and one of v35 (I think I did them correctly). The delay occurs on both my surface pro and my much faster desktop, both running Win10. Both traces were taken on the laptop (I can retake on my desktop if it would help).
Also quick disclaimer, I'm not very familiar with all this yet, so I'm sorry if I'm misplacing something. I actually made this account to comment this issue a few month ago, but my account was randomly flagged so I never got to it.
traces.zip

I have this issue too. On a mac, version 4.1.2. typing is laggy. I have no extension or anything.


License

Hi! Does this have a license? I would like to integrate it into the Artroom app. If you're interested in collaborating, please reach out via email<EMAIL_ADDRESS>or via discord Artur#2071

fixed in 1c037e1dc0176706cce7de881e269a0574b786d4


ssh not possible with GCP Stemcell >= 3363

Hi,
we are currently deploying components with BOSH on GCP. When using Stemcell >=3363, we cannot ssh into any BOSH-managed VM any more:
ssh -i private-key.pem vcap@IP
Permission denied (publickey).

The corresponding public key is available with the vcap user in the GCP Metadata. bosh ssh works fine.  Using an older Stemcell or a non-Stemcell image, a direct ssh works.
Used versions: Bosh v262  Google CPI v25.7.1
Thanks for your support and best regards,
Jochen.

We've just noticed that another issue has been raised which is most likely a duplicate of this one: #1712
You can close one of the tickets, if duplication has been confirmed.

The issue is under discussion in [https://github.com/cloudfoundry-incubator/bosh-google-cpi-release/issues/199], so closing this one now.


About the train/val dataset in Rope3D

It seems you use gen_info_rope3d to  randomly divide the data set. Can I get your partition results in order to reproduce your results.

In addition, could you please give details of the specific evaluation indicators in the rope3d data set?
Does AP in the paper refer to AP3D with easy difficulty？
Does  the S of Rope is completely according to the https://github.com/liyingying0113/rope3d-dataset-tools/blob/main/eval tools/run eval short v2x ground Is the result from 4categories.py](https://github.com/liyingying0113/rope3d-dataset-tools/blob/main/eval_tools/run_eval_distance_v2x_ground_4categories.py?
I would appreciate it if you could answer these questions！

Not exactly, during the evaluation, we did not filter any GT boxes.


Amplify Mock - Failed to get credentials: Rate exceeded Failed to get credentials: Rate exceeded

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

How did you install the Amplify CLI?
curl
If applicable, what version of Node.js are you using?
v12.13.0
Amplify CLI Version
7.6.26
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No manual changes made.
Amplify Categories
api
Amplify Commands
Not applicable
Describe the bug
When running amplify mock, this error always occurs:
Failed to get credentials: Rate exceeded
Expected behavior
amplify mock should run.
Reproduction steps

run amplify mock in appropriate repo?

GraphQL schema(s)

# Put schemas below this line

Log output

# Put your logs below this line

Additional information
I usually have to delete the mock DB file manually and then run amplify mock a few times to get it running.

Hey @MeghTrivedi :wave: thanks for raising this! I have a few follow-up questions in order to better understand the context:

can you share the GraphQL Schema?
are you seeing this error in your terminal upon running amplify mock or is this in the logs at ~/.amplify/logs?
do you also experience this with amplify mock api?

Hey @MeghTrivedi would you mind sharing the logs at ~/.amplify/logs to better understand the context of the issue? Unfortunately I have not been able to reproduce this using schemas from other projects. By providing the minimal reproduction steps it enables us to dive deep into troubleshooting and debugging the issue prior to resolving the potential bug.

@josefaidt
Here is a log from today:
2022-04-29T18:04:55.025Z|info : amplify mock mock
2022-04-29T18:04:55.122Z|info : amplify-cli-core.banner-message/index.ts.fetch banner messages from https://aws-amplify.github.io/amplify-cli/banner-message.json({}
2022-04-29T18:08:58.039Z|info : amplify mock mock
2022-04-29T18:08:58.140Z|info : amplify-cli-core.banner-message/index.ts.fetch banner messages from https://aws-amplify.github.io/amplify-cli/banner-message.json({}
2022-04-29T18:13:07.134Z|info : amplify api mock
2022-04-29T18:13:07.237Z|info : amplify-cli-core.banner-message/index.ts.fetch banner messages from https://aws-amplify.github.io/amplify-cli/banner-message.json({}
2022-04-29T18:13:28.944Z|info : amplify mock mock
2022-04-29T18:13:29.040Z|info : amplify-cli-core.banner-message/index.ts.fetch banner messages from https://aws-amplify.github.io/amplify-cli/banner-message.json({}
2022-04-29T19:26:17.930Z|info : amplify mock mock
2022-04-29T19:26:18.028Z|info : amplify-cli-core.banner-message/index.ts.fetch banner messages from https://aws-amplify.github.io/amplify-cli/banner-message.json({}

Hey @MeghTrivedi thanks for posting that! While I don't see anything that stands out here, can you clarify where you are seeing the error posted in the original report? Also, what auth mechanisms are you currently using for your GraphQL API?

@josefaidt is there any chance you can comment on the issue I am having as well?

Hi @josefaidt a sample of one of our more complex auth rules is:
@auth(
    rules: [
      {
        allow: owner
        ownerField: "monitoringCenterId"
        operations: [read]
        identityClaim: "custom:monitoringCenterId"
      }
      {
        # For allowing super ops and ops to query their own MC
        allow: owner
        ownerField: "id"
        operations: [read]
        identityClaim: "custom:monitoringCenterId"
      }
      { allow: groups, mutations: [update], groupsField: "monitoringCenterId" }
      {
        allow: owner
        ownerField: "integratorId"
        operations: [update]
        identityClaim: "custom:integratorId"
      }
      {
        allow: groups
        mutations: [create]
        groups: [
          "IntegratorAdmin"
          "Admin"
          "BillingAdmin"
          "Controller"
          "Owner"
        ]
      }
      { allow: groups, operations: [read], groups: ["local-security"] }
      { allow: private, provider: iam, operations: [read] }
    ]
  )

Yes, we use @function. A sample is:
closeIncident(
    incidentId: ID!
    resolution: String!
    resolutionNote: String!
  ): LambdaResponse @function(name: "closeIncident-${env}")

Hey @localsecurity-emily thank you for clarifying! Unfortunately I am unable to reproduce the issue where describeStacks is called enough times to cause a Rate Limit exception. Have you continuously experienced this issue?

Yes, I am still continuously and consistently experiencing this issue. I don't know if it helps at all, but here is a summary of our cloud formation stacks for one of our environments:

Thank you @localsecurity-emily is this affecting only mock or also other operations such as amplify push?

Fortunately, only mock


Cannot pass all host config options from terraform output

For example ssh_proxy_command.

I'm quite sure we are still missing some options.

I wonder if it would be possible to make TF output a json that would need no messing around

I wonder if it would be possible to make TF output a json that would need no messing around

I don't know if that's possible.. TF transforms map to an array (with a map inside).

I thinkit should at least be possible to output a json like:
            "workers": [
                {
                    "address":  "<IP_ADDRESS>",
                    "private_address": "<IP_ADDRESS>",
                    "role": "worker",
                    "ssh_key_path": "~/.ssh/aws.pem",
                    "user": "ubuntu"
                },
               {
                    "address":  "<IP_ADDRESS>",
                    "private_address": "<IP_ADDRESS>",
                    "role": "worker",
                    "ssh_key_path": "~/.ssh/aws.pem",
                    "user": "ubuntu"
                }
            ]

Instead of the current combined one (which I think has problems in case the output has a mismatched count of private/public addresses:
            "workers": [
                {
                    "address": [
                        "<IP_ADDRESS>",
                        "<IP_ADDRESS>",
                    ],
                    "private_address": [
                        "<IP_ADDRESS>",
                        "<IP_ADDRESS>",
                    ],
                    "role": "worker",
                    "ssh_key_path": "~/.ssh/aws.pem",
                    "user": "ubuntu"
                }
            ]

This does not yet solve the map syntax, but the ${element(..)} looks promising for that. The syntax is super weird.

You cannot really compose that kind of output with TF, or can you?

There's some foreach thing in upcoming 0.12 it seems: https://www.hashicorp.com/blog/hashicorp-terraform-0-12-preview-for-and-for-each

dry-type calls to_hash on objects that are not hash:
irb(main):001:0> module ToHash
irb(main):002:1>   def to_hash
irb(main):003:2>     (size == 1 && first.is_a?(Hash)) ? first : self
irb(main):004:2>     end
irb(main):005:1>   end
=> :to_hash
irb(main):006:0> Array.include(ToHash)
=> Array
irb(main):007:0> Pharos::Configuration::Host.new(address: '<IP_ADDRESS>', bastion: [{ address: '<IP_ADDRESS>' }]).bastion
=> #<#<Class:0x00007ff917b56360> address="<IP_ADDRESS>" user=nil ssh_key_path=nil>

the map/array thing can also be avoided with something like:
      label = "${map("ingress", "nginx")}"

but anything with more fields, like bastion, will become quite ugly with that.

@kke how would you pass the information from terraform to pharos with local-exec?

A subcommand that would add/update the host in cluster.yml, something like:
resource "aws_instance" "ec2" {
        provisioner "local-exec" {
          command = 'pharos-cluster terraform update -c cluster.yml'
          environment {
            TF_PUBLIC_IP = "${var.public_ip}"
            TF_PRIVATE_IP = "${var.private_ip}"
            ....
          }
       }

How about addon configs etc?

You would have a cluster.yml without hosts like now and then terraform local-exec would modify that config to add hosts.

Fixed in #852


Use GCR images, and updated README accordingly. Minor changes to logg…

Use prebuilt GCR images, and updated README accordingly. Update to tweet handling related to change in tweet schema. Minor changes to logging and main script loops.
For redis version, exit pod script if there are repeated issues with the redis connection (allowing a more graceful restart if the redis master has crashed or been restarted).

I noticed there are a lot of TODOs and debug prints. I see that the code is not quoted in the docs, so I assume this is ok. I do see that we want some output for the docker logs step. Actually we should change the docs to have it be kubectl logs <pod>.
LGTM


⚠ Bofrost.de has degraded performance

In d32cd3e, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 876 ms

Resolved: Bofrost.de performance has improved in 6601fa0.


Option to provide a custom java binary path

Description of changes made
Added a settings option to provide a custom Java binary instead of simply using the default java command.
Why is merge request needed
The default java command may not be compatible with EvoSuite (e.g. if it is a too old or new version).
Other notes
Closes #63

When a I provide an incorrect java path, I get an exception. Can you change this to have a MessageBox appear that notifies the user that an EvoSuite exception occured?

Thank you for the comment - I made it show notifications instead.


This part of code on self.save_weights_only is not necessary

In tensorflow/tensorflow/python/keras/callbacks.py , this part of code on self.save_weights_only is not necessary:
    # Use name matching rather than `isinstance` to avoid circular dependencies.
    if (not self.save_weights_only and
        not model._is_graph_network and  # pylint: disable=protected-access
        model.__class__.__name__ != 'Sequential'):
      self.save_weights_only = True

Because when the argument save_weights_only is True, no matter what the model._is_graph_network is and no matter what model.class.name is 'Sequential' or not,  self.save_weights_only will always be True.
I am not sure if I am right. So, if you disagree with me, please explain it detaily.

@Ordgod,
Can you please respond to the above comment. Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Updated the cluster heatlh state to report 0,1,2

Provide a 0,1,2 metric for the 3 possible cluster health states.

Wondering why you deleted the PR? :grinning:

Not fully tested yet ! I'll create a new PR when I confirm this does what I expect !