Docs have you set up GATEWAY_URL

Change HELLOWORLD_URL to GATEWAT_URL per docs

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: janitha09
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: rshriram
If they are not already assigned, you can assign the PR to them by writing /assign @rshriram in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

samples/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @janitha09. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: janitha09, venilnoronha
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

samples/OWNERS [venilnoronha]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/ok-to-test

@janitha09: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
dfff082d60036f5df75a3285bce10d805f5d2d2b
link
/test e2e-simpleTests-cni

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@janitha09: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
dfff082d60036f5df75a3285bce10d805f5d2d2b
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
dfff082d60036f5df75a3285bce10d805f5d2d2b
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@janitha09: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
dfff082d60036f5df75a3285bce10d805f5d2d2b
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
dfff082d60036f5df75a3285bce10d805f5d2d2b
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
dfff082d60036f5df75a3285bce10d805f5d2d2b
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-simpleTests-cni

@janitha09: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
dfff082d60036f5df75a3285bce10d805f5d2d2b
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
dfff082d60036f5df75a3285bce10d805f5d2d2b
link
/test istio-pilot-multicluster-e2e

prow/e2e-simpleTests-cni.sh
dfff082d60036f5df75a3285bce10d805f5d2d2b
link
/test e2e-simpleTests-cni

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

/retest

@janitha09: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
dfff082d60036f5df75a3285bce10d805f5d2d2b
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
dfff082d60036f5df75a3285bce10d805f5d2d2b
link
/test istio-pilot-multicluster-e2e

prow/integ-security-k8s-presubmit-tests.sh
dfff082d60036f5df75a3285bce10d805f5d2d2b
link
/test integ-security-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

I think this needs a rebase due to some test changes

New changes are detected. LGTM label has been removed.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: janitha09, venilnoronha
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: linsun
If they are not already assigned, you can assign the PR to them by writing /assign @linsun in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS
prow/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

somehow you got like 500 commits on this with million lines changed, maybe something wrong with a rebase
/hold

@janitha09: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
dfff082d60036f5df75a3285bce10d805f5d2d2b
link
/test istio-integ-k8s-tests

prow/integ-security-k8s-presubmit-tests.sh
dfff082d60036f5df75a3285bce10d805f5d2d2b
link
/test integ-security-k8s-presubmit-tests-master

prow/integ-galley-k8s-presubmit-tests.sh
78af3e725f5ab312e098a159b2462d930a8e5ef3
link
/test integ-galley-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@janitha09: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
dfff082d60036f5df75a3285bce10d805f5d2d2b
link
/test istio-integ-k8s-tests

prow/integ-security-k8s-presubmit-tests.sh
dfff082d60036f5df75a3285bce10d805f5d2d2b
link
/test integ-security-k8s-presubmit-tests-master

prow/integ-galley-k8s-presubmit-tests.sh
78af3e725f5ab312e098a159b2462d930a8e5ef3
link
/test integ-galley-k8s-presubmit-tests-master

prow/integ-mixer-k8s-presubmit-tests.sh
78af3e725f5ab312e098a159b2462d930a8e5ef3
link
/test integ-mixer-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@janitha09: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
dfff082d60036f5df75a3285bce10d805f5d2d2b
link
/test istio-integ-k8s-tests

prow/integ-galley-k8s-presubmit-tests.sh
78af3e725f5ab312e098a159b2462d930a8e5ef3
link
/test integ-galley-k8s-presubmit-tests-master

prow/integ-mixer-k8s-presubmit-tests.sh
78af3e725f5ab312e098a159b2462d930a8e5ef3
link
/test integ-mixer-k8s-presubmit-tests-master

prow/e2e-dashboard.sh
78af3e725f5ab312e098a159b2462d930a8e5ef3
link
/test e2e-dashboard-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Quantitation tools for stable isotope labelling

Hejhej,
I have a question regarding current options in GalaxyP for protein quantitation, especially on stable isotope labels (e.g. SILAC). While there are a lot of good possibilities for peptide identification and protein inference, I do not see many options for quantitation. There is ProteinPilot and Scaffold, but its both not freeware. The integration of MaxQuant was discontinued, as @bgruening told me. I am not sure, if XPRESS and ASAPRatio from the Trans-Proteomic-Pipeline have been integrated, but I would rather not recommend them to anybody. (That's why.)
So the only possibilities I see come from the OpenMS project, e.g. FeatureFinderMultiplex and ERPairFinder. Did I get this correct or am I overlooking quantitation functions of other (free) tools?

Hello Florian,
@getiria-onsongo from the Galaxy-P team has been looking at a few
quantitation tools (PyQuant, REPORTER, etc.) to be integrated into
Galaxy-P. It would be nice to have quantitation tools integrated into
Galaxy-P.
Regards,
Pratik
Pratik Jagtap,
Research Assistant Professor, Department of Biochemistry, Molecular Biology
and Biophysics,
University of Minnesota.
Address: 7-166 MCB, 420 Washington Ave SE, Minneapolis, MN 55455
*Phone:<PHONE_NUMBER> http://612-624-0381/    Email<EMAIL_ADDRESS><EMAIL_ADDRESS> *
*Twitter: twitter.com/pratikomics http://twitter.com/pratikomics
Google Scholar: z.umn.edu/pjgs http://z.umn.edu/pjgs *
http://z.umn.edu/galaxypreferences
On Mon, Nov 7, 2016 at 2:46 PM, Florian Christoph Sigloch <
<EMAIL_ADDRESS>wrote:

Hejhej,
I have a question regarding current options in GalaxyP for protein
quantitation, especially on stable isotope labels (e.g. SILAC). While
there are a lot of good possibilities for peptide identification and
protein inference, I do not see many options for quantitation. There is
ProteinPilot and Scaffold, but its both not freeware. The integration
of MaxQuant was discontinued, as @bgruening
https://github.com/bgruening told me. I am not sure, if XPRESS and
ASAPRatio from the Trans-Proteomic-Pipeline have been integrated, but I
would rather not recommend them to anybody. (That's why.
https://www.ncbi.nlm.nih.gov/pubmed/?term=nilse+sigloch)
So the only possibilities I see come from the OpenMS project, e.g.
FeatureFinderMultiplex and ERPairFinder. Did I get this correct or am
I overlooking quantitation functions of other (free) tools?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/galaxyproteomics/tools-galaxyp/issues/82, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AKL0OjQnfWrM_i5HEdYwYBmJmRzXtjwDks5q744fgaJpZM4KrsSh
.

There is also dinosaur ( https://github.com/fickludd/dinosaur ), but I don’t think there is a Galaxy package for it yet.
cheers,
—
Jorrit Boekel
Proteomics systems developer
Lehtiö lab / NBIS
Scilifelab Stockholm, Sweden

On 07 Nov 2016, at 21:46, Florian Christoph Sigloch<EMAIL_ADDRESS>wrote:
Hejhej,
I have a question regarding current options in GalaxyP for protein quantitation, especially on stable isotope labels (e.g. SILAC). While there are a lot of good possibilities for peptide identification and protein inference, I do not see many options for quantitation. There is ProteinPilot and Scaffold, but its both not freeware. The integration of MaxQuant was discontinued, as @bgruening told me. I am not sure, if XPRESS and ASAPRatio from the Trans-Proteomic-Pipeline have been integrated, but I would rather not recommend them to anybody. (That's why.)
So the only possibilities I see come from the OpenMS project, e.g. FeatureFinderMultiplex and ERPairFinder. Did I get this correct or am I overlooking quantitation functions of other (free) tools?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

What instrument vendor are you acquiring data on??
Sipros & ProRata are good combination for ID & quantification: http://sipros.omicsbio.org/
Publication: https://www.ncbi.nlm.nih.gov/pubmed/23793753
ProRata has a great deal of flexibility b/c you can specify elemental compositions between the search ID tool (Sipros) & the quantification configuration.  I used this software years ago for 15N labeling of microbes: http://pubs.acs.org/doi/abs/10.1021/pr801088f
As these tools are under development, I am hoping to add them to the Galaxy suite.
Hope this helps!

@jhervey4 awesome, please let us know if you need any kind of support for getting these tools into Galaxy and join us in https://gitter.im/GalaxyProteomics/Lobby

@jhervey4 can you join us next Thursday and Friday for a Online Contribution fest? We could work on these tools.

Hejhej, I started making a list of tools that can handle PSM / peptide / protein quants. You are invited to help me complete it.
Based on this list, we can discuss which tool would be the tool-of-choice? For my tutorials, I will start with OpenMS tools for now, as they are already available in Galaxy and can handle anything up to protein quantitation. There are many different quantitation tools in the OpenMS, though, and I will have to work out which one is best for which type of data. Does anyone have experience with these tools?


Remove Conda instructions

Closes #56.
@philippemiron @selipot if there's a good reason to keep it, please close this PR and open a new one that clarifies the instructions in the README.

I think a conda package would be great. The repo is already there in the Clouddrift organization, I can create it when I comeback from holidays (now that Awkward v2 is fully release).
Once it's done, we can add back those instructions.

Going to merge from now and @philippemiron can eventually open a new PR


NTR tRNA C3-cytosine methylation

Needed to correct annotation made using C5 process for a C3 activity!!!!
PMID:28655767
def: The process whereby a cytosine in a tRNA is methylated at position 3 of the cytosine.
is_a tRNA methylation GO:0030488

+id: GO:0106217
+name: tRNA C3-cytosine methylation


language server does not start

this error occurs as soon as i switch the demo file to type vue
[Error - 10:32:36 PM] Server initialization failed.
Message: Request initialize failed with message: Cannot read property 'substr' of null
Code: -32603

Please provide vetur and vscode version and complete error log.

vscode 1.112
vetur 0.6.3
this is all from the output window:
[Error - 10:32:36 PM] Server initialization failed.
Message: Request initialize failed with message: Cannot read property 'substr' of null
Code: -32603

Do you have any jsconfig or tsconfig files in your workspace?

not really, just opening a single vue file without a complete project.

I can repro it thanks.

The problem is workspacePath can be null when people open single file without workspace in VSCode in here:
https://github.com/octref/vetur/blob/master/server/src/modes/javascriptMode.ts#L55-L66

@mjbvz Curious if you mind sharing VSCode's strategy for consuming TS's LS for single files? Seems the whole TS LS is built based on the assumption of a workspacePath.

We support the single file case and don't do very much special on our side. When we first tell TypeScript Server about a file, it tries to walk up the file's path to find a tsconfig or jsconfig that owns the file. Behind the scenes, I believe it then loads the entire project. This is why workspace symbol search will return results outside of the single file that is actually open
We really don't use workspace.rootPath in very many places

I'm also having this issue each time I open a .vue file -- is there any fix/workaround?
I don't mind clicking the close icon on the console that pops up but it'd be nice to not have to :)

Updated to the latest VSCode and vetur and still get:
vetur initialized
[Error - 12:17:22 pm] Server initialization failed.
  Message: Request initialize failed with message: Cannot read property 'substr' of null
  Code: -32603 ```

I have this issue regularly inside the <script> block with es6 import statements:

Looks like my specific issue was a duplicate of #215 — which was solved with the release of v0.6.11


Ignore values.schema.json ?

The recently added values.schema.json makes integration into an a wider, existing infrastructure process more difficult. I need to have global parameters passed through as well as, for example, storageClass for the postgres persistence. So I have to maintain a local version of prefect-helm charts and delete values.schema.json.

I am also having an issue where I can't include the prefect charts as a subchart with the introduction of the values.schema.json.
Error: INSTALLATION FAILED: values don't meet the specifications of the schema(s) in the following chart(s):
prefect-server:
- (root): Additional property global is not allowed

You can reproduce this by creating an empty chart that tries to deploy the prefect-server chart as a subchart with default values. Here is an example Charts.yaml:
apiVersion: v2
name: example-parent-chart
description: A Helm chart for Kubernetes

type: application

version: 0.1.0

appVersion: "1.0.0"

dependencies:
  - name: prefect-server
    version: "2023.10.17"
    repository: https://prefecthq.github.io/prefect-helm

I do not see the error using the 2023.9.29 release prior to the values.schema.json addition.


Keep getting an error for localhost

Can I use this for my localhost?
I am getting this error.

{"code": 400, "error_type": "OAuthException", "error_message": "You must include a valid client_id, response_type, and redirect_uri parameters"}

Within your Instagram client config on their website you need to include the full url of your localhost environment.


Update tagger when the original select is changed

There is a script of mine which updates the original <select>, but I can find a way to update the tagger interface to reflect it.
I have tried something like (in the examples page)
$("select#sel3").val("item0").change()

I have created a pull request.
#51

Thanks for raising this.
I have merged your changes in but I will keep this issue open until we add support for multi-selects and update the examples page.


[Issue #11351] Parallel Precise Publish Rate Limiting Fix

Master Issue: https://github.com/apache/pulsar/issues/11351
Motivation
Hello, as far as I'm concerned it is well known that precise publish rate limiting does not function well. I believe my PR fixes problem number 3 stated in the issue above.
@danielsinai:
"3. Rate limit function passed only to the msg/s rate limiter (and that's in order to avoid calling it twice)"
It was passed to message rate limiter only due to the fact that there was no implementation of a way to throttle the connection whenever only one of the limiters was exceeded.
This PR will allow both message rate & byte rate to co-exist, limit and enable socket reading only when necessary.
Modifications

tryAcquire function in PublishRateLimiterDisable will return true. If publish rate was null, this function would get called and return false, thus throttling the client for no reason. If the publish rate is null, it means it was not set by anyone so there's no reason to throttle any connection.

public boolean tryAcquire(int numbers, long bytes) {
        return true;
}

RateLimiter permits and acquiredPermits were changed to volatile.

 private volatile long permits;
 private volatile long acquiredPermits;

in order to allow reading access from multiple threads at the same time.
also the removal of synchronized keyword from getAvailablePermits() function.
public long getAvailablePermits() {
        return Math.max(0, this.permits - this.acquiredPermits);
}

This is required, since a thread dead lock will happen if not.

Created ~a HashMap to manage the byte and message rate limiters, and~ a function releaseThrottle() to handle the auto read enable.
If one of the rate limiters has no available permits we will not re-enable the auto read from the socket.

Verifying this change
This change is already covered by existing tests, such as PrecisRateLimiterTest.
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): no
The public API: no
The schema: no
The default values of configurations: no
The wire protocol: no
The rest endpoints: no
The admin cli options: no
Anything that affects deployment: no

Documentation
For contributor
For this PR, do we need to update docs?
No, this PR fixes bugs of existing documented features.
Important Additional Information
This PR fixes some core issues with precise publish rate limiting but is depdenent on another PR #11446 , I would highly prefer @danielsinai PR to be merged first before this one since it fixes core issues regarding publish rate limiting and in order to prevent unnecessary disfunctionallities.
@lhotari also has a PR in the works fixing other issues related to the same topic #10384.

/pulsarbot run-failure-checks

It seems that every time I run it, other tests fail. I believe my changes didn't hurt anything since the tests that are failing are not related to my changes.

It seems that every time I run it, other tests fail. I believe my changes didn't hurt anything since the tests that are failing are not related to my changes.

Yes, there are quite a few flaky tests. Please report the flaky tests as GitHub issues up unless they have already been reported.

Error:  Tests run: 69, Failures: 1, Errors: 0, Skipped: 51, Time elapsed: 118.967 s <<< FAILURE! - in org.apache.pulsar.broker.service.persistent.PersistentTopicStreamingDispatcherE2ETest
  Error:  testMessageExpiryWithTopicMessageTTL(org.apache.pulsar.broker.service.persistent.PersistentTopicStreamingDispatcherE2ETest)  Time elapsed: 21.188 s  <<< FAILURE!
  java.lang.AssertionError: expected [500] but found [409]
  	at org.testng.Assert.fail(Assert.java:99)
  	at org.testng.Assert.failNotEquals(Assert.java:1037)
  	at org.testng.Assert.assertEqualsImpl(Assert.java:140)
  	at org.testng.Assert.assertEquals(Assert.java:122)
  	at org.testng.Assert.assertEquals(Assert.java:907)
  	at org.testng.Assert.assertEquals(Assert.java:917)
  	at org.apache.pulsar.broker.service.PersistentTopicE2ETest.testMessageExpiryWithTopicMessageTTL(PersistentTopicE2ETest.java:1008)
  	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
  	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
  	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
  	at java.base/java.lang.reflect.Method.invoke(Method.java:566)
  	at org.testng.internal.MethodInvocationHelper.invokeMethod(MethodInvocationHelper.java:132)
  	at org.testng.internal.InvokeMethodRunnable.runOne(InvokeMethodRunnable.java:45)
  	at org.testng.internal.InvokeMethodRunnable.call(InvokeMethodRunnable.java:73)
  	at org.testng.internal.InvokeMethodRunnable.call(InvokeMethodRunnable.java:11)
  	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
  	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
  	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
  	at java.base/java.lang.Thread.run(Thread.java:829)

Is there anything else need to be done, does this test prevent this PR from being merged even though it's unrelated to my changes?


[graphicsmagick] Add delegates.mgk

Describe the pull request

What does your PR fix? Fixes #15239

delegates.mgk is required when using graphicsmagick to read a file.
Note: No feature needs to test.

@ghassanpl
Can you please test if this PR solves your issue?


Tidy up R docs header

Towards #1244, but keep it open

We don't have total control over the header in the rdocs, and we don't have as much space (the header here is limited to content width, unlike the sphinx header)

Add logo with link
Remove changelog
Change title to "R Documentation" to reflect the scope of these pages

Was:

Now:

Changed the order of _pkgdown.yml during merge: will require re-review.


menu card resizing

clicking new cards results in weird resizing. animation isn't visually fluid. previous cards expand even when they're not the current selection. currently expanded card has clunky contraction.

the only thing i can think of rn is making the animation timing a lot more ease-in (may have to use a bezier curve to truly fine tune that)

jotting this as an idea for useSectionHeight
append scroll changes to a ledger of active section possibilities
these should be culled such that the consecutive events with the same section key get removed except for the newest
as long as this ledger has more than 1 entry, keep a setInterval going that
every few milliseconds checks the ledger
it takes the most reasonable activekey and takes that
if that happens to be an older one, the ledger is kept intact but the newest entry gets an updated time
if the setInterval takes the newest entry because enough time has elapsed from it and the older entries
the older entries are deleted
breaking the setInterval until new entries come in


Is doc in FAQ "Different kinds of JAX values" wrong ?

Hi,
I am learning/reading the doc now,  and have a question in FAQ part
In "https://github.com/google/jax/blame/master/docs/faq.rst",  line 235
  def func(x):
    print(x)
    return np.cos(x)

  res = jax.jit(func)(0.)

The doc says

This is why np.cos works in the example above.

But the example doenst work in my local env, and it doen't make sense for me now. the error message is
jax._src.traceback_util.UnfilteredStackTrace: jax._src.errors.TracerArrayConversionError: The numpy.ndarray conversion method __array__() was called on the JAX Tracer object Traced<ShapedArray(float32[], weak_type=True)>with<DynamicJaxprTrace(level=0/1)>
While tracing the function func at /jax/test.py:174 for jit, this concrete value was not available in Python because it depends on the value of the argument 'x'.
See https://jax.readthedocs.io/en/latest/errors.html#jax.errors.TracerArrayConversionError

As far as I know, x is a traced object during the first run, and it only holds dtype and shape info, it may be right to do np.cos(x.shape), but not np.cos(x)
Is there something wrong?
I am working in master branch, and commit id is : 7a3a160b26787898fbf192025547430996bc2b59

Good catch – I think this should be jnp.cos rather than np.cos – it was probably missed last year when we changed the docs to use import jax.numpy as jnp rather than import jax.numpy as np.
I'll submit a fix.


Update Composer dependencies (2021-03-24-00-04)

Visual regression test passed!
View the visual regression test report


Incremental compilation inside docker container broken since SBT 1.1.0

[x] I've read the CONTRIBUTING guidelines

steps

Create a new scala project with sbt new scala/scala-seed.g8 (make sure SBT version is at least 1.1.0)
Add a Dockerfile to the root of the generated project with the following content

FROM hseeberger/scala-sbt:8u171_2.12.6_1.1.5

WORKDIR /sbt-bug

ADD build.sbt /sbt-bug/build.sbt
ADD src /sbt-bug/src
ADD project/build.properties /sbt-bug/project/build.properties
ADD project/Dependencies.scala /sbt-bug/project/Dependencies.scala

RUN sbt compile test:compile

Run docker build -t sbtbug . inside the project directory
Run docker run -i -t sbtbug sbt test to enter the container

problem
Running the tests recompiles everything because SBT invalidates the class files in the target folder, which were compiled during the build of the docker image.
expectation
SBT should use the class files in the target directory to run the tests and it should not recompile everything
notes
SBT version: >= 1.1.0
This happens since version 1.1.0. With 1.0.4 and below it works as expected. While debugging the issue I observed that putting compile and test both in the Dockerfile, as two RUN commands, also reproduces the issue. This leads me to believe that this issue is caused by some strange interaction between SBT and the layered file system in docker.

I just want to add, I was able to reproduce through compression / decompression. Don't need containers.
On my project (let's call it scala-project):

compile scala-project. then compile again - it will use incremental compilation, and no recompilation is actually going to happen
compress it (on macos 10.13.5 zip functionality if that matterrs)
delete the original scala-project folder
decompress it into the scala-project folder - preserving the name and absolute path
run compile. expected - it should use the already compiled classes, and not compile from scratch. actual - it compiles from scratch.

This happens on 1.1.6, but when I used sbt 1.0.4 - the incremental compilation worked as expected.

I've run into the same issue; turns out that something Docker does to the container filesystem (probably compression, that would make sense) truncates file mtimes to second resolution (instead of millisecond).  If you're on Java 8 passing -Dsbt.io.jdktimestamps=true to SBT is a workaround, but that won't work for JDK 10 at least.

As a workaround on newer JDKs, you can do something like this:
  Seq(Compile, Test).map { c =>
    def needsTimestampTruncation: Boolean = somehowDetectDocker

    (c / previousCompile) := {
      val realPreviousResult = (c / previousCompile).value
      if (!needsTimestampTruncation) {
        realPreviousResult
      } else {
        import java.util.concurrent.TimeUnit._
        import sbt.internal.inc.{Analysis, LastModified, Stamps}
        import scala.collection.JavaConverters._
        import xsbti.compile.CompileAnalysis
        import xsbti.compile.analysis._

        def truncateMtime(s: Stamp): Stamp = s match {
          case mtime: LastModified =>
            val truncated = SECONDS.toMillis(MILLISECONDS.toSeconds(mtime.value))
            new LastModified(truncated)
          case other => other
        }

        def truncate(stamps: Stamps): Stamps = {
          Stamps(
            stamps.getAllProductStamps.asScala.toMap.mapValues(truncateMtime),
            stamps.getAllSourceStamps.asScala.toMap.mapValues(truncateMtime),
            stamps.getAllBinaryStamps.asScala.toMap.mapValues(truncateMtime)
          )
        }

        realPreviousResult.withAnalysis(realPreviousResult.analysis().asScala.map { case a: Analysis =>
          a.copy(stamps = truncate(a.stamps)): CompileAnalysis
        }.asJava)
      }
    }
  }

For a suitable definition of somehowDetectDocker (we use a special system property set by the thing that triggers the build)

Good catch @espenhw. Note that what you're doing should not be done by default to all the stamps, as different incremental compilations that happen on the same second would not recompile incrementally correctly.

We added millisecond-precision in https://github.com/sbt/io/pull/92 because we have observed incremental compilation issues under testing environment where changes are made in quick succession.
If Docker and/or ZIP file is losing precision, we might want to allow users to somehow round off the timestamp as @espenhw has done.
ZIP
From Wikipedia

The FAT filesystem of DOS has a timestamp resolution of only two seconds; ZIP file records mimic this. As a result, the built-in timestamp resolution of files in a ZIP archive is only two seconds, though extra fields can be used to store more precise timestamps.

/cc @cunei @eatkins

I believe the docker issue is https://github.com/osxfuse/osxfuse/issues/346

Docker Desktop for Mac: File system sharing (osxfs) says:

osxfs does not use OSXFUSE. osxfs does not run under, inside, or between macOS userspace processes and the macOS kernel.

@eed3si9n, I see there's a help wanted tag in this issue, but there's no clear course of action on how to solve this.
Would a PR to https://github.com/sbt/io/ that forces second-level resolution when in Docker be the correct solution?

@edrevo I think doing some research to find out the root cause would be helpful.
Also providing options via sbt setting or JVM property to force second-level resolution might be a good workaround if in fact we have regressed in sbt 1.1.0.

Could this be a cause of recompiles on circleci? https://discuss.circleci.com/t/sbt-recompiles-everything-between-workflow-steps/29162/5

@Daenyth - yep - this is the cause of the recompilation on circle on versions 1.1.0 and above. At least this is what I'm seeing on my builds. And for some reason, the workaround mentioned here (https://github.com/sbt/sbt/issues/4168#issuecomment-417655678) did not work reliably for me on circle. For a while, we used the older sbt version on circle (1.0.4), and newer ones on our dev env as another workaround.

I can confirm that the workaround also doesn't work for me on circle

Just to share another use case, I have a similar issue with millisecond resolution timestamps, but the issue is mainly that I'm shipping compiled artifacts between build agents (in my case, with TeamCity, but the problem is generic). Since tar files only have second resolution, the same issue happens.
@espenhw thanks for the workaround! For some reason I couldn't get Dsbt.io.jdktimestamps=true to work even on JDK 8, but your code-based workaround is great. I've been struggling to figure out the syntax and best way to reuse this across projects in my proper / main multi-project build as well as my meta build.
My issues are probably just due to my unfamiliarity with sbt biting me, but any tips would be really helpful! I could post on SO, but this might be helpful to others coming across this too.
Issue with multi-project build
What's the best way to reuse the code across projects? I came up something that seems decent by moving the code to project/Utils.scala, but it can't share code with the meta build (see below). I wasn't sure if I could define this more succinctly too.
In build.sbt:
import Utils._
lazy val commonSettings = Seq(
  // Can I do these both at once?
  previousCompile in Compile ~= previousCompileWithTruncatedTimestamps,
  previousCompile in Test ~= previousCompileWithTruncatedTimestamps
)

Issue with reuse in meta build
I have project/Dependencies.scala, so it seems like I need to define this config within project/build.sbt for project/*.scala files to use the second-precision. However, then I have the same code within the main build.sbt. I could extract the code into project/Utils.scala, but then I can't use it from project/build.sbt I think, and if I move it into a deeper project/project/Utils.scala, then I think that file wouldn't be using the rules...

The workaround doesn't work on Bitbucket as well :(

Using sbt-aether-deploy plugin works as a workaround on BitBucket
On Fri, 20 Dec 2019, 14:21 Andrius Versockas<EMAIL_ADDRESS>wrote:

The workaround doesn't work on Bitbucket as well :(
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sbt/sbt/issues/4168?email_source=notifications&email_token=AAOVDVKSUFMPLLSPWRMLVMDQZSTCNA5CNFSM4FAVVDAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHMVOSA#issuecomment-567891784,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOVDVIMPTSNFR43KGNQ4CDQZSTCNANCNFSM4FAVVDAA
.

After downgrading sbt to 1.0.4 and some custom file management I managed to make it work.
If anyone interested in a working prototype -  here it is: https://bitbucket.org/myDisconnect/sbt-ci-test/.
Thank you for the suggestion @varunkvv  :)

Another quick, shell-based workaround is to manually tar with the POSIX format (-H posix), which stores timestamps in nanoseconds:
RUN sbt compile && \
    find -name target -exec tar -zcf targets.tar.gz -H posix {} + && \
    find -name target -exec rm -r {} +
RUN tar -xf targets.tar.gz && \
    sbt compile # <- no recompilation!

It seems to work well with Dockerfile, and though I haven't tested, in theory it should also work with "list of commands" in yaml configuration files.

If you like @matil019 's workaround and have lots of leaf dockerfiles with few base image, I wrote a simple sbt wrapper which you can replace sbt inside the base image with. Assuming there's a sbt compile step in the leaf dockerfiles, we can compress the target files on sbt compile and decompress the gzip file on other steps. This way we can contain the workaround code within the base image without polluting the leaf dockerfiles.
#!/bin/bash
# workaround for https://github.com/sbt/sbt/issues/4168
# `sbt-org` being the original sbt binary

if [[ "$1" == "compile" ]]; then
    echo "[sbt cache invalidation fix] will compress the compiled target dirs into targets.tar.gz"
    sbt-org $@ && \
    find -name target -exec tar -zcf targets.tar.gz -H posix {} + && \
    find -name target -exec rm -r {} +
else
  tar -xf targets.tar.gz 2>/dev/null && \
    echo "[sbt cache invalidation fix] found and unpacked targets.tar.gz"
  sbt-org $@
fi

image build example:
> sbt compile (docker step)
[sbt cache invalidation fix] will compress the compiled target dirs into targets.tar.gz
[info] Loading project definition from /app/project
[info] Loading settings for project root from build.sbt ...
[info] Set current project to testt (in build file:/app/)
[info] Executing in batch mode. For better performance use sbt's shell
[info] Compiling 1 Scala source to /app/target/scala-2.13/classes ...
[success] Total time: 4 s, completed Mar 20, 2020 7:18:52 AM
> sbt test (docker step)
[sbt cache invalidation fix] found and unpacked targets.tar.gz
[info] Loading project definition from /app/project
[info] Loading settings for project root from build.sbt ...
[info] Set current project to testt (in build file:/app/)
[info] Compiling 1 Scala source to /app/target/scala-2.13/test-classes ...
[info] HelloSpec:
[info] The Hello object
[info] - should say hello
[info] Run completed in 434 milliseconds.
[info] Total number of tests run: 1
[info] Suites: completed 1, aborted 0
[info] Tests: succeeded 1, failed 0, canceled 0, ignored 0, pending 0
[info] All tests passed.
[success] Total time: 6 s, completed Mar 20, 2020 7:19:10 AM

@yhpark If it isn't sensitive or too much trouble, could you publish your Dockerfile?

This should be fixed now since I've switched to using time-wrapped content hashing in https://github.com/sbt/zinc/pull/712 and https://github.com/sbt/sbt/pull/5417.

Some updates:
when I run sbt Test/compile several times for example with cache saving / restore in between, the incremental compilation runs well.
But if I run sbt Docker/stage after a cache restore, then all classes are compiled again.
The next sbt Docker/stage after a cache restore triggers the whole compilation again.
Something to check on https://www.scala-sbt.org/sbt-native-packager/?

Update as I found the root cause, and it might be useful for others.
I run sbt -Dfatal-warnings=true Test/compile, where fatal-warnings is used to tell scalac to treat warnings as errors.
After that, sbt Test/compile will compile everything again, as the flags are different.
So I "just" have to make sure that the sbt is being launched with the same parameters on all steps.


Dev only - Console warning on Android

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

Action Performed:

Run the app

Expected Result:
No warning should display
Actual Result:
Console warning on Android is displayed
Workaround:
unknown
Platform:

Where is this issue occurring?

Android

Version Number: 1.1.79-0
Reproducible in staging?: N
Reproducible in production?: N
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

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

It's low hanging fruit to clean up, so makes sense to open up to External contributors.

First, one can't be solved ATM due to a third-party package and the second one is resolved.

Ah good call, thanks for adding some color there! Closing.


PR #98 merged but missing files

I see that PR #98 has been merged but I don't see neither docker files, nor README.md lines I added...

@sblendorio yes, we removed some updates temporarily due to incorrect work in our test environment. We will return it back soon.


libbindgen: error: expected type, found keyword type

rustc 1.15.0-nightly (daf8c1dfc 2016-12-05)
libbindgen = "0.1.2"
clang 3.9.0-3
opencv 3.1.0-6
Arch Linux 64-bit
wrapper.hpp
#include <opencv2/core/core.hpp>

build.rs
extern crate libbindgen;

use std::env;
use std::path::PathBuf;

fn main() {
    println!("cargo:rustc-link-lib=opencv_core");

    let bindings = libbindgen::Builder::default()
        .header("wrapper.hpp")
        .generate()
        .expect("Unable to generate bindings");

    let out_path = PathBuf::from(env::var("OUT_DIR").unwrap());
    bindings.write_to_file(out_path.join("bindings.rs"))
        .expect("Couldn't write bindings!");
}

$ cargo build
    Blocking waiting for file lock on build directory
   Compiling opencv v0.1.0 (file:///home/bruno/dev/rust/opencv)
error: expected type, found keyword `type`
    --> /home/bruno/dev/rust/opencv/target/debug/build/opencv-b0a1388c401941c1/out/bindings.rs:3367:46
     |
3367 |                                           -> type-parameter-0-1>,
     |                                              ^^^^

error: Could not compile `opencv`.

Can you paste or upload the bindings.rs file?

That usually means that there's some template magic we can't support in rust. We can try to fix this case if it's not too painful, but otherwise the alternatives are to use the whitelisting/blacklisting APIs to deal with that.

Oh, and thanks for the reports, they're really helpful :)

bindings.zip

Yeah, so it's trying to generate the whole STL, that is something that
we're not extremely bad at, but still not good enough, you may see we
don't generate correct rust code for std::pointer_to_unary_function, for
example:
#[repr(C)]
#[derive(Debug, Copy, Clone)]
pub struct std_pointer_to_unary_function<_Arg, _Result> {
    pub _M_ptr: ::std::option::Option<unsafe extern "C" fn(arg1: _Arg)
                                          -> type-parameter-0-1>,
    pub _phantom_1: ::std::marker::PhantomData<_Result>,
}

We can try to fix those particular issues, but for C++ libraries it's
usually saner to just whitelist what you want. In this case, doing
something like:
.whitelist_type("Cv.*")
.whitelist_function("cv.*)

should do the trick :)

It worked for type-parameter-0-1. :+1: Thanks.
Now I have two other cases:
$ cargo build
    Blocking waiting for file lock on build directory
   Compiling opencv v0.1.0 (file:///home/bbigras/dev/rust/rs-opencv)
error: expected identifier, found keyword `type`
    --> /home/bbigras/dev/rust/rs-opencv/target/debug/build/opencv-2abe0d5d07f86d1d/out/bindings.rs:1226:19
     |
1226 |     pub unsafe fn type(&self) -> ::std::os::raw::c_int {
     |                   ^^^^

error: expected one of `(` or `<`, found `~`
    --> /home/bbigras/dev/rust/rs-opencv/target/debug/build/opencv-2abe0d5d07f86d1d/out/bindings.rs:1288:15
     |
1288 |     pub fn cv_~String();
     |               ^

error: Could not compile `opencv`.

bindings.zip

Nice, the rest are fallout from recent changes. Should ve fixed with #344.

We didn't have a test for inline destructors, and it bited us :P

Also, I didn't notice that the opencv library used namespaces, bindgen has support for those, in case of name conflicts.
We plan to enable them by default sometime soon, and since both SpiderMonkey and Gecko are using bindgen with that feature enabled, I'm pretty confident they'd be solid enough for OpenCv.

Also, I didn't notice that the opencv library used namespaces, bindgen has support for those, in case of name conflicts.

Do you mean enable_cxx_namespaces()? Can I use it instead of the whitelist options?
With your 'fix-dtors' branch (and still using .whitelisted_type("Cv.*").whitelisted_function("cv.*")) I get:
error[E0428]: a type named `std_char_traits` has already been defined in this module
   --> /home/bbigras/dev/rust/rs-opencv/target/debug/build/opencv-2abe0d5d07f86d1d/out/bindings.rs:185:1
    |
174 |   pub struct std_char_traits<_CharT> {
    |  _- starting here...
175 | |     pub _address: u8,
176 | |     pub _phantom_0: ::std::marker::PhantomData<_CharT>,
177 | | }
    | |_- ...ending here: previous definition of `std_char_traits` here
...
185 |   pub struct std_char_traits<_CharT> {
    |  _^ starting here...
186 | |     pub _address: u8,
187 | |     pub _phantom_0: ::std::marker::PhantomData<_CharT>,
188 | | }
    | |_^ ...ending here: already defined

error: unions are unstable and possibly buggy (see issue #32836)
  --> /home/bbigras/dev/rust/rs-opencv/target/debug/build/opencv-2abe0d5d07f86d1d/out/bindings.rs:67:1
   |
67 |   pub union std_namespace_basic_string__bindgen_ty_2<_CharT, _Traits, _Alloc> {
   |  _^ starting here...
68 | |     pub _M_local_buf: *mut _CharT,
69 | |     pub _M_allocated_capacity: std_namespace_basic_string_size_type<_Alloc>,
70 | |     pub _phantom_1: ::std::marker::PhantomData<_Traits>,
71 | | }
   | |_^ ...ending here
   |
   = help: add #![feature(untagged_unions)] to the crate attributes to enable

error: unions are unstable and possibly buggy (see issue #32836)
   --> /home/bbigras/dev/rust/rs-opencv/target/debug/build/opencv-2abe0d5d07f86d1d/out/bindings.rs:482:1
    |
482 |   pub union Cv32suf {
    |  _^ starting here...
483 | |     pub i: ::std::os::raw::c_int,
484 | |     pub u: ::std::os::raw::c_uint,
485 | |     pub f: f32,
486 | | }
    | |_^ ...ending here
    |
    = help: add #![feature(untagged_unions)] to the crate attributes to enable

error: unions are unstable and possibly buggy (see issue #32836)
   --> /home/bbigras/dev/rust/rs-opencv/target/debug/build/opencv-2abe0d5d07f86d1d/out/bindings.rs:497:1
    |
497 |   pub union Cv64suf {
    |  _^ starting here...
498 | |     pub i: int64,
499 | |     pub u: uint64,
500 | |     pub f: f64,
501 | | }
    | |_^ ...ending here
    |
    = help: add #![feature(untagged_unions)] to the crate attributes to enable

error: unions are unstable and possibly buggy (see issue #32836)
   --> /home/bbigras/dev/rust/rs-opencv/target/debug/build/opencv-2abe0d5d07f86d1d/out/bindings.rs:560:1
    |
560 |   pub union _bindgen_ty_16__bindgen_ty_1 {
    |  _^ starting here...
561 | |     pub __wch: ::std::os::raw::c_uint,
562 | |     pub __wchb: [::std::os::raw::c_char; 4usize],
563 | | }
    | |_^ ...ending here
    |
    = help: add #![feature(untagged_unions)] to the crate attributes to enable

error: unions are unstable and possibly buggy (see issue #32836)
    --> /home/bbigras/dev/rust/rs-opencv/target/debug/build/opencv-2abe0d5d07f86d1d/out/bindings.rs:9037:1
     |
9037 |   pub union CvFileNode__bindgen_ty_1 {
     |  _^ starting here...
9038 | |     pub f: f64,
9039 | |     pub i: ::std::os::raw::c_int,
9040 | |     pub str: CvString,
9041 | |     pub seq: *mut CvSeq,
9042 | |     pub map: *mut CvFileNodeHash,
9043 | | }
     | |_^ ...ending here
     |
     = help: add #![feature(untagged_unions)] to the crate attributes to enable

error: unions are unstable and possibly buggy (see issue #32836)
    --> /home/bbigras/dev/rust/rs-opencv/target/debug/build/opencv-2abe0d5d07f86d1d/out/bindings.rs:9177:1
     |
9177 |   pub union CvMat__bindgen_ty_1 {
     |  _^ starting here...
9178 | |     pub ptr: *mut uchar,
9179 | |     pub s: *mut ::std::os::raw::c_short,
9180 | |     pub i: *mut ::std::os::raw::c_int,
9181 | |     pub fl: *mut f32,
9182 | |     pub db: *mut f64,
9183 | | }
     | |_^ ...ending here
     |
     = help: add #![feature(untagged_unions)] to the crate attributes to enable

error: unions are unstable and possibly buggy (see issue #32836)
    --> /home/bbigras/dev/rust/rs-opencv/target/debug/build/opencv-2abe0d5d07f86d1d/out/bindings.rs:9194:1
     |
9194 |   pub union CvMat__bindgen_ty_2 {
     |  _^ starting here...
9195 | |     pub rows: ::std::os::raw::c_int,
9196 | |     pub height: ::std::os::raw::c_int,
9197 | | }
     | |_^ ...ending here
     |
     = help: add #![feature(untagged_unions)] to the crate attributes to enable

error: unions are unstable and possibly buggy (see issue #32836)
    --> /home/bbigras/dev/rust/rs-opencv/target/debug/build/opencv-2abe0d5d07f86d1d/out/bindings.rs:9208:1
     |
9208 |   pub union CvMat__bindgen_ty_3 {
     |  _^ starting here...
9209 | |     pub cols: ::std::os::raw::c_int,
9210 | |     pub width: ::std::os::raw::c_int,
9211 | | }
     | |_^ ...ending here
     |
     = help: add #![feature(untagged_unions)] to the crate attributes to enable

error: unions are unstable and possibly buggy (see issue #32836)
    --> /home/bbigras/dev/rust/rs-opencv/target/debug/build/opencv-2abe0d5d07f86d1d/out/bindings.rs:9272:1
     |
9272 |   pub union CvMatND__bindgen_ty_1 {
     |  _^ starting here...
9273 | |     pub ptr: *mut uchar,
9274 | |     pub fl: *mut f32,
9275 | |     pub db: *mut f64,
9276 | |     pub i: *mut ::std::os::raw::c_int,
9277 | |     pub s: *mut ::std::os::raw::c_short,
9278 | | }
     | |_^ ...ending here
     |
     = help: add #![feature(untagged_unions)] to the crate attributes to enable

error: aborting due to 9 previous errors

error: Could not compile `opencv`.

bindings.zip

So the untagged union thing is that we prefer to generate those if we can given we can support more nasty stuff like template parameters in unions properly. But since it's only nightly rust by default, you can override that with no_unstable_rust.
I'm really surprised about the double std::char_traits thing (I've been able to reproduce, and I'll dig into it).
That being said, I've been able to generate bindings for OpenCv with the following bits:
    let bindings = Builder::default()
        .no_unstable_rust()
        .header("wrapper.h")
        .clang_arg("-x")
        .clang_arg("c++")
        .opaque_type("std::char_traits")
        .enable_cxx_namespaces()
        .no_unstable_rust()
        .whitelisted_function("cv.*")
        .whitelisted_type("Cv.*")
        .whitelisted_type("cv::.*")
        .generate()
        .expect("Unable to generate bindings");

The whitelisting can probably get smarter, and you can cut-off parts of the API you don't want with the opaque/blacklisting APIs.

Oh, forgot to mention that the patches in https://github.com/servo/rust-bindgen/pull/346 are needed.

That being said, I've been able to generate bindings for OpenCv with the following bits

Nice, thanks!!
By the way some tests are failing when I run cargo test. Do you want me to paste them here or I guess you have installed opencv if you were able to generate bindings successfully.

Also, are the warnings normal?
warning: found non-foreign-function-safe member in struct marked #[repr(C)]: found struct without foreign-function-safe representation annotation in foreign module, consider adding a #[repr(C)] attribute to the type, #[warn(improper_ctypes)] on by default

Yes, unfortunately stuff that uses PhantomData and similar triggers that kind of warning.

That's https://github.com/rust-lang/rust/issues/34798. Regarding the type size and alignment failures, I hadn't taken a look at them. I'd need to rewrite the crate since I deleted it, but was easy enough last time, so I'll re-do it :)

So the failures are mostly expected.
On one hand, there's a failure with the Scalar type. We can't do much about it since it's defined in terms of number generics:
template<typename _Tp> class Scalar_ : public Vec<_Tp, 4>

The other ones are the std::vector type and std::string on my system. That's because we don't do associated types properly, and we end up getting a byte-sized type instead of a pointer-sized type. We can fix that with better info from libclang.
All the rest are derived from those. There's additionally the std::vector<bool> specialization, that is something that we could indeed special-case as a one-off, but otherwise it's template specialization madness we can't support on rust.
If you find one of these problems, there are multiple ways to work around the problem, depending on needs.
If you don't need to access the members, the easiest thing is marking stuff as opaque. Marking a type as opaque tries to replace that type with a proper replacement in each situation. For example:
.opaque_type("std::vector")

Would fix a bunch of tests, and replace the fields with properly-aligned arrays in rust.
Other possible workaround/fix, for when you need access to the type's field, would be using replacements. For example, I know that std::vector is represented by three words in the STL, one pointer to the initial element, other to the last element, and other to the end of the reserved storage.
I could make the wrapper.h file look as follows:
/**
 * <div rustbindgen replaces="std::vector"></div>
 */
template<typename T, typename Alloc>
class vector_replacement {
  T* start;
  T* end;
  T* storage_end;
};

#include <opencv2/core.hpp>

That will replace the definition of std::vector by the type I just wrote, that bindgen can detect correctly. This wouldn't fix the vector<bool> problem, but since we don't use it, it seems to be a nice solution if we needed access to it.
Combining those configuration options, you can ensure the layout of your types is correct, even when bindgen can't quite do the correct thing.
I'm not sure if I missed something (this should probably be documented in a more visible place). @fitzgen may correct me if I'm wrong.

I'm not sure if I missed something (this should probably be documented in a more visible place). @fitzgen may correct me if I'm wrong.

Yeah, I think we need a users' guide. I've been kind of meaning to write one, but (a) I have higher priorities (sorta lame excuse) and (b) we've been overhauling a lot of this stuff frequently (eg "_" => "::" in replacements and whitelisting etc).
I think maybe if we (I? you? :P) make a skeleton of a guide, people can start helping to contribute. But I do think we need the skeleton first to act as a framework for others who want to jump in and help.

I think we can close this issue, as it seems we have a users' guide now and there doesn't seem to be much left here that is actionable.


Scroll bar customization

Enhancement: Cutomised scroll bar
solves #13

Hey @Bhavya1403 !
Thanks for merging :)
Can you please change the label to hacktoberfest-accepted with green colored background


SFI:ROPC metadata

I think we should standardize on ROPC hits that have been fixed.
Fixes  #33727

Internal previews

📄 File
🔗 Preview link

aspnetcore/security/authorization/secure-data.md
Create an ASP.NET Core web app with user data protected by authorization

@tdykstra see 7dc93a3279e8517baa89dfb414533d61bf8791df

~@tdykstra see 7dc93a3~

Never mind, it used to list the minimum requirements which wasn't strong, so I reverted.


Allow to customize git author/committer name+email

Description
This PR adds additional non-required inputs that allow to customize the name and email used for the commit author and committer.
Checklist

[ ] Tested functionality against a test repository (see "How to test changes")
[ ] Added or updated relevant documentation (leave unchecked if not applicable)

Dunno if it counts, but I tested this in:

https://github.com/armanbilge/fs2-dom/pull/24

Using the configuration in:
https://github.com/armanbilge/nimble-agar-steward/blob/85fc256bd85befc57f627bd0852ff01c9d5b926d/.github/workflows/flake.yml

I'm terribly sorry for letting this slip through the cracks; I dunno how I missed it!
Looks good to me -- I'll test it in my test repo and then approve + merge if I don't uncover any issues (I doubt I will, but just to be sure)!

No problem at all. Thank you so much! 😃

Just released as https://github.com/DeterminateSystems/update-flake-lock/releases/tag/v15 (forgot to do that earlier, oops).


Add navigation links to wiki

As Github does not support PR's for wiki's I open this issue.
Would propose adding a navigation link list to the wiki as seen here:
https://github.com/Christofon/MetaGame/wiki

Discourse https://forum.metagame.wtf/
Original Medium https://medium.com/metagame-perspective


[FLOC 3363] add context to Enabling agent service doc

Fixes 3363
Adds an explanation of why you need to enable the agent service.

Thanks.  One comment inline with some extra info.  Merge when you're happy and the doc builder is green.


[request] Wiki Editing

Would be super cool to edit GitHub wikies from inside nvim. If a given repo has a wiki, some :GHWiki or some such could open a wiki editor

Hey! Im open too it, but is way low on the list. I think the next major feature of this plugin will be properly handling Github Notifications.

FWIW, this goal:

Would be super cool to edit GitHub wikies from inside nvim.

is definitely possible already because GitHub wikis are just git repositories. So you can clone them normally (e.g. git clone git@github.com:ldelossa/gh.nvim.wiki.git or gh repo clone ldelossa/gh.nvim.wiki), make edits in neovim, and push changes as regular git commits (assuming you have write permissions, of course).
But if you are asking for a :GHWiki command that would let you edit the wiki without cloning it, that would be move involved.


Support Edge Canary for Mac

Edge Canary for Mac is [also] detected as Chrome https://github.com/biggora/express-useragent/issues/113
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/77.0.3834.0 Safari/537.36 Edg/<IP_ADDRESS>

I added this to #113, but not sure if it should go here too...
I'm still getting Edge showing up as Chrome on MacOS with this UA: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_13_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/81.0.4044.129 Safari/537.36 Edg/81.0.416.68
I believe this PR is missing a regex change on this line as well --> https://github.com/biggora/express-useragent/blob/master/lib/express-useragent.js#L103
AFAICT, that line ought to also have edg as part of the match pattern, same as what was added to _Versions in the PR.


 If the navgiation is e.g. 48dp, your icons are 20+dp, they get aligned a bit to the bottom rather than shifted to the top so it can be vertically aligned. Can we have the option to shift the offset of the menu items? OR even better: to let BottomNavigationViewEx do that behind the scenes automatically for us?

Sorry. There is no way to control shift offest. You can cancel shift mode. Then setting padding yourself.


Cargo: run always open a new task

Version of VSCode: 1.14.1
Version of the extension: 0.4.2
OS: Archlinux
Description:
When using Cargo: run, it always open a new terminal(like 1: Cargo Task) in Terminal panel, I want to keep only 1 task.

Happy to see this was fixed, but when would it being released?

For future readers, @Yuyz0112 is indicating this document.

I'm still seeing this behaviour and I just installed extension yesterday.


mitogen_via + transport=smart throws KeyError: u'smart'

Hello,
I'm getting following error while executing Ansible with mitogen:
$ ansible -m ping 'lindevtestv02.lindev.test' --ask-vault-pass -vvv
ansible 2.7.5
  config file = /home/burdalib/ansible-mitogen/ansible.cfg
  configured module search path = [u'/home/burdalib/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /home/burdalib/ansible-mitogen/lib/python2.7/site-packages/ansible
  executable location = /home/burdalib/ansible-mitogen/bin/ansible
  python version = 2.7.5 (default, Sep 12 2018, 05:31:16) [GCC 4.8.5 20150623 (Red Hat 4.8.5-36)]
Using /home/burdalib/ansible-mitogen/ansible.cfg as config file
Vault password:
/home/burdalib/ansible-mitogen/hosts did not meet host_list requirements, check plugin documentation if this is unexpected
/home/burdalib/ansible-mitogen/hosts did not meet script requirements, check plugin documentation if this is unexpected
Parsed /home/burdalib/ansible-mitogen/hosts inventory source with yaml plugin
[pid 23865] 10:38:53.910936 D mitogen: mitogen.service.Pool(0x7fbc373ea450, size=32, th='MainThread'): initialized
[pid 23865] 10:38:53.912781 D ansible_mitogen.process: Service pool configured: size=32
META: ran handlers
Tuesday 19 February 2019  10:38:53 +0200 (0:00:00.258)       0:00:00.258 ******
[pid 23900] 10:38:53.997113 D mitogen: unix.connect(path='/tmp/mitogen_unix_suc5XS.sock')
[pid 23900] 10:38:53.998268 D mitogen: unix.connect(): local ID is 1, remote is 0
[pid 23865] 10:38:53.998221 D mitogen: mitogen.unix.Listener('/tmp/mitogen_unix_suc5XS.sock'): accepted mitogen.core.Stream('unix_client.23900')
[pid 23900] 10:38:54.004739 D mitogen: mitogen.core.Stream('unix_listener.23865').on_disconnect()
[pid 23900] 10:38:54.005015 D mitogen: Waker(Broker(0x7fbc36b9b150) rfd=9, wfd=10).on_disconnect()
[pid 23865] 10:38:54.005028 D mitogen: mitogen.core.Stream('unix_client.23900').on_disconnect()
[pid 23900] 10:38:54.005362 D mitogen: Router(Broker(0x7fbc36b9b150)): stats: 0 module requests in 0 ms, 0 sent (0 ms minify time), 0 negative responses. Sent 0.0 kb total, 0.0 kb avg.
The full traceback is:
Traceback (most recent call last):
  File "/home/burdalib/ansible-mitogen/lib/python2.7/site-packages/ansible/executor/task_executor.py", line 140, in run
    res = self._execute()
  File "/home/burdalib/ansible-mitogen/lib/python2.7/site-packages/ansible/executor/task_executor.py", line 612, in _execute
    result = self._handler.run(task_vars=variables)
  File "/home/burdalib/ansible-mitogen/lib/python2.7/site-packages/ansible_mitogen/mixins.py", line 115, in run
    return super(ActionModuleMixin, self).run(tmp, task_vars)
  File "/home/burdalib/ansible-mitogen/lib/python2.7/site-packages/ansible/plugins/action/normal.py", line 46, in run
    result = merge_hash(result, self._execute_module(task_vars=task_vars, wrap_async=wrap_async))
  File "/home/burdalib/ansible-mitogen/lib/python2.7/site-packages/ansible_mitogen/mixins.py", line 331, in _execute_module
    self._temp_file_gibberish(module_args, wrap_async)
  File "/home/burdalib/ansible-mitogen/lib/python2.7/site-packages/ansible_mitogen/mixins.py", line 310, in _temp_file_gibberish
    self._connection.get_good_temp_dir()
  File "/home/burdalib/ansible-mitogen/lib/python2.7/site-packages/ansible_mitogen/connection.py", line 702, in get_good_temp_dir
    self._connect()
  File "/home/burdalib/ansible-mitogen/lib/python2.7/site-packages/ansible_mitogen/connection.py", line 749, in _connect
    stack = self._build_stack()
  File "/home/burdalib/ansible-mitogen/lib/python2.7/site-packages/ansible_mitogen/connection.py", line 657, in _build_stack
    inventory_name=self.inventory_hostname,
  File "/home/burdalib/ansible-mitogen/lib/python2.7/site-packages/ansible_mitogen/connection.py", line 623, in _stack_from_spec
    seen_names=seen_names + (spec.inventory_name(),),
  File "/home/burdalib/ansible-mitogen/lib/python2.7/site-packages/ansible_mitogen/connection.py", line 626, in _stack_from_spec
    stack += (CONNECTION_METHOD[spec.transport()](spec),)
KeyError: u'smart'

lindevtestv02.lindev.test | FAILED! => {
    "msg": "Unexpected failure during module execution.",
    "stdout": ""
}
Tuesday 19 February 2019  10:38:54 +0200 (0:00:00.045)       0:00:00.304 ******
===============================================================================
ping ------------------------------------------------------------------------------------------------------------------------------------------------ 0.05s
 ----------------------------------------------------------------------------------------------------------------------------------------------------------
[pid 23865] 10:38:54.011405 D mitogen: Waker(Broker(0x7fbc37428cd0) rfd=6, wfd=8).on_disconnect()
[pid 23865] 10:38:54.011757 D mitogen: Router(Broker(0x7fbc37428cd0)): stats: 0 module requests in 0 ms, 0 sent (0 ms minify time), 0 negative responses. Sent 0.0 kb total, 0.0 kb avg.

$ ansible-config dump --only-changed
ANSIBLE_NOCOWS(/home/burdalib/ansible-mitogen/ansible.cfg) = True
ANSIBLE_SSH_ARGS(/home/burdalib/ansible-mitogen/ansible.cfg) = -C -o ControlMaster=auto -o ControlPersist=60s -o StrictHostKeyChecking=no -o UserKnownHostsF
DEFAULT_CALLBACK_WHITELIST(/home/burdalib/ansible-mitogen/ansible.cfg) = [u'profile_tasks']
DEFAULT_HOST_LIST(/home/burdalib/ansible-mitogen/ansible.cfg) = [u'/home/burdalib/ansible-mitogen/hosts']
DEFAULT_LOAD_CALLBACK_PLUGINS(/home/burdalib/ansible-mitogen/ansible.cfg) = True
DEFAULT_NO_TARGET_SYSLOG(/home/burdalib/ansible-mitogen/ansible.cfg) = True
DEFAULT_ROLES_PATH(/home/burdalib/ansible-mitogen/ansible.cfg) = [u'/home/burdalib/ansible-mitogen/roles', u'/etc/ansible/roles']
DEFAULT_STRATEGY(/home/burdalib/ansible-mitogen/ansible.cfg) = mitogen_linear
DEFAULT_STRATEGY_PLUGIN_PATH(/home/burdalib/ansible-mitogen/ansible.cfg) = [u'/home/burdalib/ansible-mitogen/lib/python2.7/site-packages/ansible_mitogen/plu
DEFAULT_TIMEOUT(/home/burdalib/ansible-mitogen/ansible.cfg) = 60
HOST_KEY_CHECKING(/home/burdalib/ansible-mitogen/ansible.cfg) = False
PERSISTENT_CONNECT_TIMEOUT(/home/burdalib/ansible-mitogen/ansible.cfg) = 30

$ ansible --version
ansible 2.7.5
  config file = /home/burdalib/ansible-mitogen/ansible.cfg
  configured module search path = [u'/home/burdalib/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /home/burdalib/ansible-mitogen/lib/python2.7/site-packages/ansible
  executable location = /home/burdalib/ansible-mitogen/bin/ansible
  python version = 2.7.5 (default, Sep 12 2018, 05:31:16) [GCC 4.8.5 20150623 (Red Hat 4.8.5-36)]

$ pip show mitogen
Name: mitogen
Version: 0.2.5
Summary: Library for writing distributed self-replicating programs.
Home-page: https://github.com/dw/mitogen/
Author: David Wilson
Author-email: UNKNOWN
License: New BSD
Location: /home/burdalib/ansible-mitogen/lib/python2.7/site-packages

After some tests I found out that this error happens only when:

ansible_transport is set to smart (default)
mitogen_via is set in inventory

When I tried to set ansible_transport = ssh, ping worked.
Also, when I set ansible_ssh_common_args = ' -o ProxyJump ...' instead of mitogen_via, it also worked.
Please let me know if you need more information.

This is now on the master branch and will make it into the next release. To be updated when a new release is made, subscribe to https://www.freelists.org/list/mitogen-announce
Thanks for reporting this!

I can confirm v2.5 -> master fixed this.

Hi Florient! IIRC you were having troubles with mitogen_via= / connection delegation. How are things on recent releases? Did >0.2.4 solve everything?


Make RawKernel and RawModule aware of CUDA context

Close #3193.
In #3193, I found the case from @mnicely hit this subtle bug: If a RawKernel/RawModule is compiled for one device, it cannot be reused on another device. This PR fixes that by detecting current device and recompiling when necessary. (See the added tests for how to reproduce the bug --- the current master branch, or v7, would fail in these tests.)
Usually, this issue is circumvented by using the cupy.util.memoize(for_each_device=True) decorator. However, because we can't use that for cdef classes afaik, the internal caching with device ID being part of the key is necessary.
I also took this chance to enforce some typing that we didn't do before moving to PY3-only. If it's considered unnecessary, let me know and I'll revert. I don't think this would cause any backward incompatibility though.
May be in conflict with #2966? cc: @emcastillo

I am wondering it we could just reload the existing cubin to another context and avoid recompiling...

Sorry for thinking out loud 😅

I am wondering it we could just reload the existing cubin to another context and avoid recompiling...

Ah, I suppose this is what's done internally in cupy/cuda/compiler.py, since the code is the same. The only price to pay for switching context is one call to cuModuleLoad for reloading.

@asi1024 I think this bug is related our discussion starting https://github.com/cupy/cupy/issues/3144#issuecomment-597464683.

I discussed offline with @asi1024 and we believe it is better to get the current device in the constructor and raise an exception if the kernel is called from other device.

OK fine with me. Whatever behavior should be clearly documented.
Please feel free to close this PR when the accepted solution is merged.

Also, another alternative is to have a RawKernel and RawModule factory method in python where you can use the cupy.util.memoize(for_each_device=True) decorator

Just for the record: the memoization approach taken in this PR is inspired by that of ufuncs', see https://github.com/cupy/cupy/blob/766dfd04703da1d9a971d1124951c24b91bfbc33/cupy/core/_kernel.pyx#L1060-L1072


[usb] Wire up transfers with libusb-to-JS

Finish the migration of the LibusbJsProxy code from the old
ChromeUsbApiBridge by letting the transfer requests be sent through the
new libusb-to-JS proxy.

PTAL - I haven't finished the manual testing, but I'm excited to see it's already capable of some basic operations :)


Report redundant explicit imports

Fixes #3699
Check for redundant explicit imports according to the kotlin docs.

Introduce new class in Style rules
Register this class
Add tests

What's the status of this PR? What's left to do in order to merge this into detekt?
I think this rule is really great. I can't wait to use it in my projects.

Currently I didn't have much time to return to this.
As it was commented by maintainers needs better consideration of corner cases here. I may try to return to this in nearest weeks or it can be taken by someone else with more time


Add NYU as associate member to member page

Add NYU as associate member to member page

@danlopez00 - is there a POC from NYU to ask for logo vectors/assets? Or do you have that already?

email sent to NYU to wrap up paperwork


Avoid creating a new shell instance when using wt

This PR creates shell functions that invoke wt and cd into the given directory. This partially solves #11.
What changed:

When running wt and giving it a valid worktree name, it echo's out changedir:<path>.
Adds the init sub-command for generating the shell code of the function.
The installation now requires the addition of a line to the shell config that invokes wt and runs the generated code. This is reflected in the readme.

I believe some further changes would be nice, but might be outside the scope of this PR:

Add a -- prefix for the sub-commands, to avoid collisions with worktree names.
The new init operation could also generate the completion scripts to reduce installation complexity. (This could be implemented in this PR)
Shell completions could include the sub-commands.

Some notes:

This introduces breaking changes, because the installation changed.
Instead of the shell function invoking wt, the whole script could be inside the shell function, thus not needing the ugly echoing.
The error messages are not the best...
Maybe the whole thing should be refactored :)

Hello.
I'm looking into reworking this a bit and I have a good enough solution with minimal setup needed from the end user.
Doing . wt instead of wt  runs the script in the current shell rather than creating a new subshell and running it there. This solves the issue of nested shells and not changing the directory it you avoid creating a new one in the script.
All of this would only need alias wt=". wt" from the end user to make it work.
I'll be working on this today, and comparing it with your work, but I'd like to keep this as simple as possible. I'd like to package this so I'd like the setup needed to be minimal.

Well I'll be packaging your fork as simple modifications probably won't do this well enough for publishing. Thanks for your work!

Awesome stuff! Also, thank you! As you saw, I have developed my fork a bit. Feel free to open a PR there!

My changes aren't something you'd want in your package.
I ripped out the update functionality as wt is read only in nix and you don't have the necessary permissions to overwrite it in others. I reduced the functionality to "update available".
I also ripped out the auto update stuff as since you're getting the update through a package manager, you don't want to see this for 3 days until the update is available.
The last modification I did, which I may open a PR for, is that the list option was removed and there's names instead. I'd go back to list as that's what git worktree list so I find it more intuitive both for me and new users.

I have also packaged it for nix :D (sort of) in my config. It also removes the update functionality with a patch. I think it might be best to not handle updates and have it distributed via a package manager.
I also created a names subcommand and removed list, since it felt redundant. Here ced661e, and here 7423c83.
I think the . wt might be a better solution to not spawning a subshell. Does it work in shells other than bash?

I also created a names subcommand and removed list, since it felt redundant.

I saw that but IMHO the better thing would be to rename it, rather than create a new one and then delete the old one. The full paths don't really have a use case and if you need them just do git worktree list. As this is a wrapper I'd use this to simplify the out put of git worktree list (as names does) but keep the name.
If you're interested here's my PR to nixpkgs.

Fair enough, maybe we should change names back to list.


Expose GetPlotRect in public interface

Fixes #42

Hi @anderspitman
It's perfect however there are two failures in checks

The git commits need to be signed
Golangci-lint: put a dot at the end of the comment of function description

I'll try and get this fixed tomorrow.

@navidys should be good to go

@anderspitman thanks

LGTM


docs 44# adds description for new fields

Follows up https://github.com/infinispan/infinispan-helm-charts/pull/46
#44 adds description for two new fields podLabels and serviceLabels in the docs

@dvagnero Done. Thanks :slightly_smiling_face:


net_splitpacket_maxrate

Why is "net_splitpacket_maxrate" CVAR in your network config set to "50000"? The client-side value will always be restricted to 30k, unless this gets resolved

You are absolutely right. There is no point setting it to 50000 because it will still cap at 30000.
Thank you for pointing this out!
I'll make the change in a bit!


Engge> Engge2 update

Project Link
https://github.com/scemino/engge2
Project License
N/A
Steam App Ids
https://store.steampowered.com/app/569860/Thimbleweed_Park/
Additional Context
Engge has been replaced by Engge2 which is newer.
it would be nice if you added the engine and link to this issue to the wiki: https://github.com/dreamer/luxtorpeda/wiki/Game-engines

Do you know what the license for this engine is? I sadly don't feel comfortable with supporting unless we figure that out.
Note that the dreamer/luxtorpeda repo is old and not maintained, this fork is the one that is up to date.


feat: add visual tests (WIP)

@NickVlasov
Since these tests are running in travis-ci, I added the eyes server url to the env variables.
But we also need to add our eyes' secret key, no?

@jonathanadler I'm not sure that we will use match-screenshot cause, as you mentioned, it requires to add apllitools server url and key for testing which shouldn't be available at public
I think that we will use https://github.com/americanexpress/jest-image-snapshot cause it doesn't require these "secret" configuration knowledge and everybody can contribute more easily.

@NickVlasov
Ok, I understand.
So I can revert my commit.
Just note that on Tuesday Applitools' old server will be deprecated, so tests in this pr won't work

@jonathanadler Thanks. Right now this PR has only configuration setup and no tests. It includes match-screenshot, but it will be changed for another screenshot comparison library

based on https://github.com/wix/playable/pull/150

detect why tests are not quit in travis

add npm run test:visual to .travis.yml when everything will work


Wrong balance on sapling - unstable connection

Bug in the balance after synchronisation #560
I have been running several tests on ZingoPC and Zingo Mobile, one of the main bugs is the wrong balance in Sapling when the internet connection is unstable.
The bug is still present in ZingoPC and Zingo Mobile running on the latest version of zingolib.
Some images:

The correct balance in the wallet is 0.0003 ZEC in Orchard.
Previously, when the internet connection was lost and came back, the synchronisation would not start, and you had to close and open zingopc again. Now when the internet comes back, the synchronisation starts again and the progress is shown by the percentage, but there is still an bug in the balance when the synchronisation is finished.
Linux Mint 21.2
Zingo PC 1.0.3 / Zingo Mobile 1.2.5 (126)
zingolib mob-prerelease-1.3.0-63-gd3233715

@Edicksonjga Is this still active? This is a regression from release?

@Edicksonjga Is this still active? This is a regression from release?

Yes, the error is still there and no matter how many times I try it in Production, it doesn't happen. @AloeareV

this is related with: https://github.com/zingolabs/zingo-mobile/issues/533

this is related with: https://github.com/zingolabs/zingo-mobile/issues/533

Yes, and is being followed up on:
https://github.com/zingolabs/zingolib/issues/560

ok overview/update:
as a result of further testing by @Edicksonjga, this bug has seemingly been narrowed down to a single transaction in a much smaller test wallet. the bug appears due to regular network interruptions while syncing and causes a 0.0001 fee from a send-to-self transaction to change to a receive of 0.0001 zec, therefore resulting in a 0.0002 zec excess in the final balance.
test wallet seed: drive celery ...
max known birthday to reproduce bug: 2295000
expected final balance: 0.000 zec
balance when bug is hit: 0.0002 zec
bugged transaction: d8ca1781adda081cd8a095cb3c24a29efc2542aa751ec5812f6ddaf50af7afd1

@Edicksonjga could you verify this is correct and maybe add you own report of your findings with as much detail as possible?

@Edicksonjga could you verify this is correct and maybe add you own report of your findings with as much detail as possible?

Thank you very much @Oscar-Pepper, yes, the summary you provide is correct.
I have been testing with the version that @fluidvanadium passed me and with the wallet you mention, synchronizing from a recent block height (November 2023) and obtaining in the synchronization the last 22 transactions.
At the end of synchronization the balance should be zero, but due to the error caused by interrupting the synchronization, the balance is 0.0002 ZEC in Sapling and the only transaction that changes is the one you mention that instead of being an Send-to-Self is reflected as a Received transaction.

i also discovered a possible bug that may be related while comparing zingo-cli summaries command with a emulated mobile app:
the second transcation after the bugged tx (d8ca178) is the first tx in the wallet to show the send-to-self value transfer in zingo-cli. all previous send-to-selfs (at least as shown as send-to-self in the mobile app) only show the fee value transfer, including the tx displaying the balance bug. after this first send-to-self value transfer, it seems like some send-to-selfs are missing the send-to-self value transfer (only showing the fee) and some send-to-selfs show both a send-to-self and fee value transfer.
wallet: drive celery
bday: 2295000
first transaction with send-to-self value transfer: a1ed0c0e0742613732d761c91cf6492362f132b0884a06b53af56be1176dbecf

Since my first time using summaries I notices that, sometimes I got only 1 tx with the same txid with fee, missing the other tx with sendtoself... In fact in the zingo-mobile code when happend that the App consider this tx as sendtoself directly.

oh ok good to know you were aware

i also discovered a possible bug that may be related while comparing zingo-cli summaries command with a emulated mobile app:
the second transcation after the bugged tx (d8ca178) is the first tx in the wallet to show the send-to-self value transfer in zingo-cli. all previous send-to-selfs (at least as shown as send-to-self in the mobile app) only show the fee value transfer, including the tx displaying the balance bug. after this first send-to-self value transfer, it seems like some send-to-selfs are missing the send-to-self value transfer (only showing the fee) and some send-to-selfs show both a send-to-self and fee value transfer.
wallet: drive celery bday: 2295000 first transaction with send-to-self value transfer: a1ed0c0e0742613732d761c91cf6492362f132b0884a06b53af56be1176dbecf

on further investigation, the send-to-self value transfer only are created when there is outgoing metadata. sorry for the noise but this is unlikely to help this bug hunt

I'll close that report, I hope I never open it again 😁😁. @juanky201271 @zancas @fluidvanadium @Oscar-Pepper @AloeareV ...


Cannot paste according to universal cllipboard between Macos and iPados

Issue Type: Bug
I copy some text from my iPad, but it cannot be pasted as text in vscode editor. It works well on other software, even included this issue reporter editor. When I enter command+V, it will be pasted as a filename, ex. 1f19fe3c07ded61345e32403b9328fe608185ed2.rtfd.
VS Code version: Code 1.68.1 (30d9c6cd9483b2cc586687151bcbcd635f373630, 2022-06-14T12:48:37.209Z)
OS version: Darwin arm64 21.6.0
Restricted Mode: No

System Info

Item
Value

CPUs
Apple M1 (8 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
3, 2, 2

Memory (System)
8.00GB (0.10GB free)

Process Argv
--disable-extensions --crash-reporter-id 3b3c379f-df66-4361-8b00-8fc2b9436051

Screen Reader
no

VM
0%

Extensions disabled
A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411cf:30557515
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vscaac:30438847
vsclangdf:30486550
c4g48928:30535728
dsvsc012:30540252
azure-dev_surveyone:30548225
i497e931:30553904
pyindex848cf:30577861
40g7c324:30573242

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.71.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

I have updated to the latest stable version just now, but the bug still exists.

@silver-ymz does the paste work in other parts of the UI like in Search (cmd+shift+F) or the command palette (cmd+shift+p)? Can you tell me where you copied the text from on your iPad (From a website? From Notes?)

The paste works well on Search interface (cmd+shift+F), the command palette (cmd+shift+p) and terminal, but the problem still exists on Source Control and Extension interface.
It seems that it only happens when I copy the text from Safari. I test Books, GoodNotes, Github and Gmail App, and it works well.

I guess  RTFD files make the problem.

@silver-ymz do you have a specific website you're copying from for completeness of the repro steps?

This website (github.com) makes the bug. I can't copy the title of this issue.

https://user-images.githubusercontent.com/78400701/199872858-f72e2acb-db60-4df0-8691-7f819efbb640.mp4

Hey @TylerLeonhardt @hediet, this issue might need further attention.
@silver-ymz, you can help us out by closing this issue if the problem no longer exists, or adding more information.

Same issue

Same issue. I replicated it by copying text from next.js website open on Safari in iPad , to VS Code open on MacOS
VS Code Version: 1.75.1 (Universal) Commit: 441438abd1ac652551dbe4d408dfcec8a499b8bf
iPadOS 16.2
MacOS Ventura 13.1

Still an issue. In my case, it appears that is happens with Safari, Chrome and Firefox. When I tested with my reddit app (Apollo) it works fine.
VSCode:
Version: 1.77.0 (Universal)
Commit: 7f329fe6c66b0f86ae1574c2911b681ad5a45d63
iPadOS: 16.3.1
MacOS: Ventura 13.3.1

Hey @TylerLeonhardt @hediet, this issue might need further attention.
@silver-ymz, you can help us out by closing this issue if the problem no longer exists, or adding more information.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!

Still an issue. copy text from Safari on iphone , and paste it into VS Code on MacOS
vscode
ver: 1.83.0
commit: e7e037083ff4455cf320e344325dacb480062c3c
MacOS: Sonoma 14.0

The issue still persists. We can easily reproduce it by copying text from Fireox on iPhone, and pasting it into Visual Studio Code on macOS.
Version: 1.84.1
Commit: 2b35e1e6d88f1ce073683991d1eff5284a32690f
macOS: Sonoma 14.1

The issue still exists, and easily reproducible by copying any text from webkit based app. Safari, Firefox, Chrome.
Version: Version 1.89.1
macOS: Sonoma 14.4.1

use Paste Plain Text, on the appstore


Enhancement#692

Changed endScrollTargetLocation to a public internal(set) var.

probably some insight of what is trying to be accomplished here my help.


Use relative path for measurement icon

Use relative path for measurement icon
Change  measurement style
Fix a not existing function call in clear measurement function
Display a blue circle on the last position during drawing a FireBoundary

This change is

Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


katacoda is no longer available

Katacoda is no longer available, move to alternatives like killercoda or others (https://itnext.io/katacoda-to-killercoda-migration-guide-d21961fc0c9b)

The initial two (101 and CDI) labs have been moved over to killercoda, plumbing on the website is still needed.


Great resource!

I've been playing some Base + Prosperity sets, and they were fantastic to play, thanks for publishing this! 🙂

Thanks!


Update Documentation for New-AzHighAvailabilityVirtualNetworkGatewayConnections.ps1

PR Summary

PR Checklist

[x] Documentation use only: This PR contains non-production samples used in Microsoft's official Azure PowerShell documentation.
[x] Azure PowerShell use only: The samples in this PR contain commands from the Az PowerShell module.
[x] Sample scripts only: This PR does not contain PowerShell modules, binaries, tools, images, zip or tar.gz files, or other files that are not samples.
[x] Validated samples: I have validated these samples using the latest version of the Az PowerShell module.
[x] Ongoing support: I agree to provide ongoing support for these samples and will promptly respond to issues and PRs.

@nimaller Please leave a comment with #sign-off in the message when you're ready for this PR to be merged.

#sign-off


Add README for tgpu-cli.

closes #435

I know its still a draft, but could we make it a guide instead of a README? We're going to have to document it there anyway, so maybe it would be easier to keep it in one place so the information does not diverge.

Okey👍 Should I scrap the README or do it as well as a guide?

@reczkok I'd move the contents of the README into a guide, and dive deeper into each use-case. Having this info in two places at once will be hard to keep in sync.

@reczkok Ready for review? 👀

@iwoplaza yes, you can also take a look at the bind group PR


Fix a small variable name typo

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

Thanks @emmanuelguerin


Working on an example inspired by tangle (Bret Victor)

Should we try to merge that into devel, even if not perfect yet?

Closing this as it is now based on develop


Add "Copy to clipboard" option to download matrix dialog

If I click on the icon to download a project matrix file I can select the output format but there's no option to "Copy to clipboard" which I do get if I regenerate the matrix by selecting "Request Expression Matrix" dialog.


The demo site is just a default deployment set up with the in-memory data provider. That allows people to do whatever they want since it's just going to reset when the app restarts. Besides, there isn't much by default.

          The demo site is just a default deployment set up with the in-memory data provider. That allows people to do whatever they want since it's just going to reset when the app restarts. Besides, there isn't much by default.

Anyone can register an account to see what you get as a starting point from a fresh clone.
You can also log in as<EMAIL_ADDRESS>using password A2m!nPassword.
Originally posted by @EdKaim in https://github.com/SharpLogic/LightNap/issues/1#issuecomment-2477516837

Anyone can register an account to see what you get as a starting point from a fresh clone.
You can also log in as<EMAIL_ADDRESS>using password A2m!nPassword.
Originally posted by @EdKaim in https://github.com/SharpLogic/LightNap/issues/1#issuecomment-2477516837


Is ssl_hello_world_app.erl valid?I can't even compile

I got the following error message:

src/ssl_hello_world_app.erl:25: syntax error before: ','

line 25 is:
#{env, [{dispatch, Dispatch}]}

is this a valid map?

It's not. Master is undergoing large changes and the examples and docs are probably incorrect.

It has been fixed a while ago. Some other examples are still broken though. Closing, thanks!


CocosSharp crashing on iOS 9

See forum post https://forums.xamarin.com/discussion/50824/cocossharp-crashing-on-ios-9

Should be fixed in repo.  Waiting for validation.

New NuGet package <IP_ADDRESS> is now available to fix iOS9 crashes and CCTextField for iOS9 not working


Correlations v3

Correlations  module in v3

Same here - as with tracers_v3 and bg_bolt_cl_v3, this contains all the commit history from those, even though it is based on v3_wip. We should clean it up and only branch out from v3_wip, otherwise there is no point in having separate PRs for all these changes. That's the reason we are ditching the star branch (which passes the tests and contains almost all of the changes we are going to make).

Why isn't coveralls reporting?

I see the report.


BUG:

Describe the bug
LoadCombination(...) Yields: "AttributeError: 'NoneType' object has no attribute 'factor'
in
...\Rfem\LoadCasesAndCombinations\loadCombinations.py
at mlvlp.row.factor = combination_items[i][0]
.
Can't get it to work, even with the default arguments, ie. "LoadCombination()"
To Reproduce

Create 1 or more LoadCases
Run LoadCombination(...) or LoadCombination()
See error

Expected behavior
A loadcombination with default parameters according to
no: int = 1,
analysis_type = AnalysisType.ANALYSIS_TYPE_STATIC,
design_situation: int = 1,
name: str = '',
static_analysis_settings: int = 1,
consider_imperfection: bool = False,
consider_initial_state: bool = False,
structure_modification: bool = False,
to_solve: bool = True,
combination_items = [[1.5, 1, 0, False]],
comment: str = '',
params: dict = None,
model = Model):
Seems to be an issue with the combination_items argument.

Found a workaround, which is just adding all the loads into a single loadcase instead.

Hi @BjornSchweder,
thank you for the reaction. I checked it via our test (.\UnitTests\test_loadCombination.py) and I don't see any issue in the latest version of RFEM. Could you provide better description? Version of the RFEM, version of the Client and even relevant part of your script.

Hi,
I used RFEM pkg from pypi.org (1.15.0) and RFEM v6.02.0047.244 and some newer versions and no issue. Certainly with 'factor'. You can reinstall the package to make sure you haven't introduced any changes. Inadvertently using a different or tampered version of the client is a common problem.

# from RFEM.enums import NodalSupportType, LoadDirectionType, MemberLoadDistribution, MemberLoadDirection
from RFEM.enums import *
from RFEM.initModel import *
from RFEM.BasicObjects.material import Material
from RFEM.BasicObjects.section import Section
from RFEM.BasicObjects.node import Node
from RFEM.BasicObjects.line import Line
from RFEM.BasicObjects.member import Member
from RFEM.TypesForNodes.nodalSupport import *
from RFEM.LoadCasesAndCombinations.staticAnalysisSettings import StaticAnalysisSettings
from RFEM.LoadCasesAndCombinations.stabilityAnalysisSettings import StabilityAnalysisSettings
from RFEM.LoadCasesAndCombinations.loadCase import LoadCase
from RFEM.LoadCasesAndCombinations.loadCombination import LoadCombination
from RFEM.Loads.memberLoad import *
from RFEM.Calculate.meshSettings import GetModelInfo
from RFEM.TypesForNodes.nodalSupport import setNodalSupportConditions
from RFEM.Results.resultTables import ResultTables
from RFEM.Reports.html import *
from RFEM.Reports.printoutReport import *
from RFEM.ImportExport.exports import *
from RFEM.TypesForMembers.memberHinge import *

import numpy as np
import pandas as pd
import re

Model(new_model=True, model_name = 'Referensmodell_struktur_python')
Model.clientModel.service.begin_modification()

SetAddonStatus(Model.clientModel, AddOn.structure_stability_active)

#Geometrically Linear Static analysis
StaticAnalysisSettings.GeometricallyLinear(no=1, name = 'Fully Linear')

#Linear Buckling Modes
StabilityAnalysisSettings.EigenvalueMethod(1,'Eigenvector Linear' , 10, True, False, None, None,
StabilityAnalysisSettingsEigenvalueMethod.EIGENVALUE_METHOD_LANCZOS,
StabilityAnalysisSettingsMatrixType.MATRIX_TYPE_STANDARD)

#Create load case and combinations.
LoadCase.StaticAnalysis(no=1,
name='Lineload + Prestress',
to_solve = True,
analysis_settings_no = 1,
self_weight = [False],
params = {'calculate_critical_load': True, 'stability_analysis_settings': 1})

LoadCombination()
Model.clientModel.service.finish_modification()

Alright, I'll look into that. Thanks for the help!


Is there more detailed training setup descriptions for the two pre-trained HiFi-GAN models?

I compute the md5 sums of the checkpoint files in this repo and in HiFi-Gan offical repo, and find that their md5 sums are the same!


Open dinamically modal inside modal

Hello,
I'm using the vanilla-modal inside a angular2 project.
When I use the rel attribute, like:
Login
and inside this modalLogin, I call another modal using the same way:
Signup
But, I'm trying to create a directive to make this auto to open. So, on the click I call a method that does this:
Signup
const modal = new VanillaModal({
      open: this.modalToOpen
});

modal.open('#'+this.modalToOpen);

The modal insert the content in the same modal as opened and when I close and open the second modal, the content is blank because It's inserted on the first.
I try to use destroy or force to close before open the new one with no success.
Do I'm making a mistake? Is it possible to do on this way?

Maybe it's not your case, if you are using styles from demo, you could try to delete them. I had the same problem than you and now it's ok

Unfortunately It's not my case. I'm not using the styles from demo.
I think I'm making a mistake with js instance, but I don't know how I can resolve this.

Hi @leeeandroo,
In order to open a second modal with a fresh set of parameters inside an existing modal, you'll need to instantiate VanillaModal twice.
A new instance of VanillaModal needs to exist for every modal you would like to have open at any one time on the page. In other words, if you'd like to overlay three modals above each other, you'll need to have three VanillaModal instances.
I'm not sure how that will tie in with your Angular 2 component... good luck!


Run the "presubmit" actions on the stable branch

So we can get build results from it.  Should probably rename / refactor presubmit into continuous and presubmit only sometime soon.

Yep. Same way I've normally done it, cut a branch from main to be "stable" for a specific release, so we can continue making commits to main while testing that release


Release v0.2.0

Migrated from GitLab: https://gitlab.com/meltano/sdk/-/issues/132
Originally created by @aaronsteers on 2021-05-20 20:19:59

SDK Release Checklist

[0] Indicate the vX.Y.Z version to be released here in the issue title and here: v0.2.0

SDK Release Flow

[x] Manual steps:

[x] Open this Issue
[x] Create a corresponding MR on a branch named <issue>-release-vX-Y-Z (the default name if your issue is titled correctly)
[x] The release-vX.Y.Z MR is ready when:

[x] Changelog includes all meaningful user-facing updates since the last release

[x] Compare against main branch commit history

[x] Version is bumped:

[x] pyproject.toml
[x] cookiecutter/tap-template/pyproject.toml

[x] Changelog is flushed with the appropriate version number
[x] Changes above are committed as changelog and version bump
[x] Open the Changelog in preview mode, mouse over each link and ensure tooltip descriptions match the resolved issue. Check contributor profile links to make sure they are correct.

[x] Check this box when the CI pipeline status is green :white_check_mark:
[x] Merge to main with the merge commit message Release vX.Y.Z

[ ] Release steps:

[x] Manual:

[x] Cut a tag from main named vX.Y.Z with Message=Release vX.Y.Z

Note: tag name must exactly match poetry version text

[x] Automated CD pipeline:

In response to new tag creation, these steps are performed automatically in Gitlab pipelines:

Abort if tag vX.Y.Z does not match output from poetry version --short
Publish to PyPi

[x] Check this box when confirmed

Create a Gitlab 'Release' from the specified tag

SDK Post-Release Flow

[x] Post-release announcement steps:

[x] Post announcement to Meltano slack: #announcements, cross-post (share) to #sdk
[x] Copy-paste with minor contextual edits to Singer slack (with link to original Meltano slack)
[x] Blog post
[x] Tweet the blog post

View 25 previous comments from the original issue on GitLab