Compilation against openssl 3.0.0 beta2 is failing to build unbound

Unbound when compiled against openssl beta2, it is failing to compile.
https://github.com/openssl/openssl/releases/tag/openssl-3.0.0-beta2
To reproduce
Steps to reproduce the behavior:

Use openssl 3.0.0 beta2 from https://github.com/openssl/openssl/releases/tag/openssl-3.0.0-beta2
Compile unbound

Expected behavior
The build should succeed
System:

Unbound version: 1.12.0
OS: photon

Error:
smallapp/unbound-control.c: In function 'ssl_path_err':
smallapp/unbound-control.c:490:4: warning: implicit declaration of function 'ERR_GET_FUNC'; did you mean 'ERR_GET_LIB'? [-Wimplicit-function-declaration]
490 |   (ERR_GET_FUNC(err) == SYS_F_FOPEN ||
|    ^~~~~~~~~~~~
|    ERR_GET_LIB
/bin/ld: .libs/unbound-control.o: in function ssl_path_err': /usr/src/photon/BUILD/unbound-1.12.0/smallapp/unbound-control.c:490: undefined reference to ERR_GET_FUNC'
/bin/ld: /usr/src/photon/BUILD/unbound-1.12.0/smallapp/unbound-control.c:491: undefined reference to `ERR_GET_FUNC'
collect2: error: ld returned 1 exit status
make: *** [Makefile:355: unbound-control] Error 1

As per: https://www.openssl.org/news/changelog.html
The ERR_GET_FUNC() function was removed. With the loss of meaningful function codes, this function can only cause problems for calling applications.

Thanks, it works now


feat: compress display

Description:
Feat - port of https://github.com/cssnano/cssnano/tree/master/packages/postcss-normalize-display-values
BREAKING CHANGE:
No
Related issue (if exists):
No

I'll review this after rebasing :)

@kdy1 Done :+1:

I'll merge once the new version is published


somalier (sample control + ancestry)

Description of feature
Implement the software somalier to have a quality control to detect sample swaps, as well as some output on the ancestry of the sample, very useful to interpret a clinical sample.
https://github.com/brentp/somalier
A possible alternative for ancestry would be rye:
https://github.com/healthdisparities/rye

Good idea, we're already using peddy from the same author to do ancestry, relatedness and sex check. He is suggesting switching to somalier as the more scalable solution. Will look into this for upcoming releases


Indirect all objects to a traditional IPFS filesystem

All objects are stored in their raw form in directories named for their type (.git/commits, /.git/trees, …).
Removing the header allows creating IPFS file structures using the blobs, so the root of the created repo is the contents of the HEAD commit.
An optional .git/vfs/ structure is created which gives access to all the commits as a file system.
IPNS support is added both for pulling and pushing to a key.
A breaking change is the project is referred to as ipfs-shipyard/git-remote-ipfs since IPLD is no longer used.

I discovered that since I am storing hundreds of references to the commits in the virtual filesystem (I built the tree for each commit) it makes the reported size of the repo huge.
The 20MiB repo with ~100 commits has a reported size of 5TiB.
While it is not an issue for usage, it does affect pinning services (and maybe filecoin). I had a discussion this morning on Pinata's Slack where their lead tried to convince me IPFS was somehow broken and to submit a bug to go-ipfs.
I need to address this, and while I welcome comments, I would like to delay merging until I can do a refresh of IGiS to accompany this.

Ok, so I have rewritten everything in javascript. It is about five times slower, but I decompose the git object into DAG Objects rather than maintaining a table of the entire object database. None of the SHA1 hashes are saved. Commit parents are of the same type as their children. Trees are regular IPFS filesystems.
QmZZsVWQZHC3VF6axjLPEZjbCYjddTXmNs41tkCv6yuB8U
The root of the filesystem is still the checked out branch, and the git information is stored under .git
Surprisingly enough when those structures are fed into NodeGit, it will produce the original SHA1s and object database even with signatures.
I think the Go program this pull request references is dead in the water. I want to use OrbitDB which means javascript.


Fix task context fails with non string property value

Fix task context fails with non string property value
Description
This is a regression caused by https://github.com/apache/druid/pull/14239
The above PR introduces a new method addSystemProperty with String type in the method definition for the property value. (Previously, we were using StringUtils.format which takes in Object as argument). The problem is that task.getContextValue(propName) has a return type which is a generic type parameter ContextValueType. This means the method can return a value of any type specified at the time of invocation. When the value of the context property is not a String, Java is still trying to call the addSystemProperty with String type causing the task to fail.
i.e. if we have something like
  "context": {
    "druid.indexer.fork.property.druid.processing.numThreads": 4
  }

we'll get the error:
java.lang.ClassCastException: java.lang.Integer cannot be cast to java.lang.String
	at org.apache.druid.indexing.overlord.ForkingTaskRunner$1.call(ForkingTaskRunner.java:319) [druid-indexing-service-31.0.0-SNAPSHOT.jar:31.0.0-SNAPSHOT]
	at org.apache.druid.indexing.overlord.ForkingTaskRunner$1.call(ForkingTaskRunner.java:171) [druid-indexing-service-31.0.0-SNAPSHOT.jar:31.0.0-SNAPSHOT]
	at com.google.common.util.concurrent.TrustedListenableFutureTask$TrustedFutureInterruptibleTask.runInterruptibly(TrustedListenableFutureTask.java:131) [guava-32.0.1-jre.jar:?]
	at com.google.common.util.concurrent.InterruptibleTask.run(InterruptibleTask.java:75) [guava-32.0.1-jre.jar:?]
	at com.google.common.util.concurrent.TrustedListenableFutureTask.run(TrustedListenableFutureTask.java:82) [guava-32.0.1-jre.jar:?]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149) [?:1.8.0_322]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624) [?:1.8.0_322]
	at java.lang.Thread.run(Thread.java:750) [?:1.8.0_322]

This PR fix the issue by casting the return value from task.getContextValue(propName) to Object and then use addSystemProperty with Object type for the value parameter.
This PR has:

[x] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] a release note entry in the PR description.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[x] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[x] been tested in a test Druid cluster.

@maytasm , this PR https://github.com/apache/druid/pull/16281/files also fixes the same issue.
Maybe we can just merge that existing PR instead?
It seems to have been pending because of coverage issues.

@maytasm , this PR https://github.com/apache/druid/pull/16281/files also fixes the same issue. ~Maybe we can just merge that existing PR instead?~ The original author has not responded on that PR, so we can proceed with this one.

SGTM. Closing

@kfaraz Oh.  Will update this PR to use the same approach used in https://github.com/apache/druid/pull/16281

@kfaraz Updated the PR with the approach used in #16281 and clean up some stuff. Can you please review? Thanks!


Get DiagnosticReports and Save them in MedicalRecordsStore

softbrew/phr-core#6

Retrieve DiagnosticReport iteratively (using recursive callbacks)

:+1:


[bug]: Bounding box ignores 512px render size if box size is above 512px

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Linux
GPU
cuda
VRAM
8GB
What version did you experience this issue on?
2.3.4
What happened?
Hey everyone,
I have an upscaled an inpainted image, which I want to outpaint. When I choose a big bounding box like 1024x1024, I would like it to render the content still with 512x512 because of my low GPU specs.
But if I choose 1024x1024 as bounding box, it still tries to render as 1024x1024 and the resolution area at the left is greyed out.
Is this possible?
Thanks for your help!
Best regards,
Screenshots
No response
Additional context
No response
Contact Details
No response

@hipsterusername Hey Kent, is this fixed?

Hey @erhan- - This isn't really a "bug" per se.
However, you should be able to manually control the resolution being generated at within the bounding box using the manual "Scaled Bounding Box" setting, and enforcing a 512x512 generation.

Hey @hipsterusername,
I know. I have written about it and that it doesnt work like that:
"But if I choose 1024x1024 as bounding box, it still tries to render as 1024x1024 and the resolution area at the left is greyed out.
Is this possible?" So I cant set it to 512x512 then. Has this been changed in the meantime and fixed was my question basically.
Thanks for your work!


Update requirements to TF2

Was working on @twiecki PR https://github.com/pymc-devs/pymc4/pull/90 and was getting errors that I think only will occur in TF1.
This PR is an attempt to update PyMC4 to TF2, although now I have more questions than answers.
It's my belief that we need to move to TF2, Then rebase https://github.com/pymc-devs/pymc4/pull/90, fix the errors there, then rebase https://github.com/pymc-devs/pymc4/pull/92, but I may be understanding wrong so just let met know.
Questions
For the PyMC team
Is the modification to RV.log_p correct? Due to eager execution a value is expected immediately but I don't know if this actually how the implementation should be.
@csuter
I have a lot of one off TF2 questions which I'll list below. Let me know if this is the appropriate forum, or if there's a more productive way to do this. Just pointers on where to look would be helpful
Example for questions
"""
Test TF2 and Tensorflow Probability Log values versus scipy stats
"""
import tensorflow as tf
from tensorflow_probability import distributions
from scipy import stats

# Scipy baseline
print(stats.bernoulli.logpmf(p=.5, k=1))

# TF2 Eager Execution. Returns tf.EagerTensor with "actual" values
log_p = distributions.Bernoulli(probs=.5).log_prob(1)
print(log_p)

# Non eager execution
# Returns tf.Tensor
# How do I execute this if I can't call session.run?
# https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/Tensor
graph = tf.Graph()
with graph.as_default() as g:
    log_p = distributions.Bernoulli(probs=.5).log_prob(1)
    print(log_p)

Question 1:
With eager execution how can we clear the default graph? tf.reset_default_graph() no longer works
Question 2:
Along the lines of question 1, Tf2 docs show how to create new graphs, but I'm running into another problem where these graphs aren't eager by default. Normally I'd call session.run() to evaluate the operations but tf.session isn't a thing anymore. The tf2 documentation still says to call sess.run() but I'm confused on how to do this?
https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/Tensor
Question 3:
Is there a way to get a random seed for the default graph. The documentation shows a way to set seeds for "user created" graphs but not sure how set a global random seed for tf anymore. tf.random.set_random_seed() doesn't exist in tf2

To answer your questions:

In TF2, there is no default graph. Graphs only exist inside tf.function, everything else is executed imperatively. While it is possible to create a tf.compat.v1.Graph and use it as default this is not endorsed v2 behavior.

If you get rid of these lines

graph = tf.Graph()
with graph.as_default() as g:

And just execute the code inside that scope, it will run eagerly. The eager equivalent of sess.run(t) or t.eval() is t.numpy()

Random sampling behavior is going to change in v2, but the implementation is not yet complete. Right now, to get stable behavior you can call tf.compat.v1.set_random_seed immediately before sampling. The future is this:
https://github.com/tensorflow/community/blob/master/rfcs/20181217-tf2-random-numbers.md
You can preview the behavior with this code
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/ops/stateful_random_ops.py

@brianwa84
Thanks for this information!
For question 2, how can we "namespace" graphs? For example in our tests it would be ideal if the graph was "reset" between tests.  In TF1 we did this with tf.reset_default_graph() In TF2 does this happen automatically?
I ask because of this line in the docs

Another typical usage involves the tf.Graph.as_default context manager, which overrides the current default graph for the lifetime of the context:
https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/Graph

Does this mean that this overrides the TF2 default graph which is always present? Or does TF2 just not have a global graph at all?

Yeah that reference is confusing. TF2 doesn't have a global default graph. That's why you need a tape to get gradients (tf.GradientTape). When you use Graph().as_default() it's taking you back to TF1. In TF2, in the background there is a Context object, which you hopefully never need to interact with.
I recommend watching the TF Dev summit talk on tf2. The big message is, functions not sessions. Within a @tf.function annotated python function, you are building a graph (for that particular function) and the tensors are symbolic. But you should be able to write and test the function in eager, then annotate to make it run faster. When you call that function (assuming annotated), eager executes a PartitionedCall op which references the traced-out function graph of symbolic tensors.

To get your above code working, since it is graph mode/tf1, change to
with graph.as_default() as g, tf.Session() as sess:
    log_p = distributions.Bernoulli(probs=.5).log_prob(1)
    print(sess.run(log_p))

Yeah that reference is confusing. TF2 doesn't have a global default graph. That's why you need a tape to get gradients (tf.GradientTape). When you use Graph().as_default() it's taking you back to TF1. In TF2, in the background there is a Context object, which you hopefully never need to interact with.
I recommend watching the TF Dev summit talk on tf2. The big message is, functions not sessions. Within a @tf.function annotated python function, you are building a graph (for that particular function) and the tensors are symbolic. But you should be able to write and test the function in eager, then annotate to make it run faster. When you call that function (assuming annotated), eager executes a PartitionedCall op which references the traced-out function graph of symbolic tensors.

So last question then. Why is tf.Tensor or Graph().as_default() still present in TF2? Or removal planned in the production release?

You should feel free to file some issues on TF GitHub about your confusion, maybe just reference this one. The documentation is still not completely updated. It's possible graph will stay for some reason in tf2 (serialization/export is the obvious one), but I am not aware of whether it's only for compat or there to stay.

@brianwa84 Will do and again many thanks

What is the question being asked by dist.log_prob(), ie what answer do you
expect to get?
Yes, TFP always requires an argument to log_prob.
On Sat, Mar 16, 2019, 9:49 AM Thomas Wiecki<EMAIL_ADDRESS>wrote:

@twiecki commented on this pull request.
In pymc4/random_variables/random_variable.py
https://github.com/pymc-devs/pymc4/pull/93#discussion_r266201908:

@@ -163,15 +163,15 @@ def log_prob(self):

class DiscreteRV(RandomVariable):

def log_prob(self):

def log_prob(self, value):

CC @csuter https://github.com/csuter @brianwa84
https://github.com/brianwa84
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pymc-devs/pymc4/pull/93#discussion_r266201908, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AVJZIxWSN38Pcy10sGtD2KsVovK_Grcpks5vXPZ3gaJpZM4b3hL1
.

I guess you want a sample_and_log_prob() function?
On Sat, Mar 16, 2019, 9:09 PM Brian Patton 🚀<EMAIL_ADDRESS>wrote:

What is the question being asked by dist.log_prob(), ie what answer do you
expect to get?
Yes, TFP always requires an argument to log_prob.
On Sat, Mar 16, 2019, 9:49 AM Thomas Wiecki<EMAIL_ADDRESS>wrote:

@twiecki commented on this pull request.
In pymc4/random_variables/random_variable.py
https://github.com/pymc-devs/pymc4/pull/93#discussion_r266201908:

@@ -163,15 +163,15 @@ def log_prob(self):

class DiscreteRV(RandomVariable):

def log_prob(self):

def log_prob(self, value):

CC @csuter https://github.com/csuter @brianwa84
https://github.com/brianwa84
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pymc-devs/pymc4/pull/93#discussion_r266201908, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AVJZIxWSN38Pcy10sGtD2KsVovK_Grcpks5vXPZ3gaJpZM4b3hL1
.

I guess you want a sample_and_log_prob() function?
…
On Sat, Mar 16, 2019, 9:09 PM Brian Patton 🚀 @.> wrote: What is the question being asked by dist.log_prob(), ie what answer do you expect to get? Yes, TFP always requires an argument to log_prob. On Sat, Mar 16, 2019, 9:49 AM Thomas Wiecki @.> wrote: > @.**** commented on this pull request. > ------------------------------ > > In pymc4/random_variables/random_variable.py > <#93 (comment)>: > > > @@ -163,15 +163,15 @@ def log_prob(self): > > > class DiscreteRV(RandomVariable): > - def log_prob(self): > + def log_prob(self, value): > > CC @csuter https://github.com/csuter @brianwa84 > https://github.com/brianwa84 > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <#93 (comment)>, or mute > the thread > https://github.com/notifications/unsubscribe-auth/AVJZIxWSN38Pcy10sGtD2KsVovK_Grcpks5vXPZ3gaJpZM4b3hL1 > . >

@brianwa84 its on me to more clearly explain what I think the issue may be. I'll circle back on this

What's the status of this PR?  Any updates?

I tried my code and it still runs so I think this is based on a misunderstanding. I'll try to provide an example soon.

https://github.com/pymc-devs/pymc4/pull/92 should get things running again.
The confusion here I think related to eager mode (the default in TF2). In brief, we can't evaluate logps of dists directly anymore as the value to evaluate things over is stored in the model context. Here is my fix in the tests: https://github.com/pymc-devs/pymc4/pull/92/files#diff-737badc85adb55a500b41650fe4a4e62R76

Thanks!


PD: Show step name in Step List (both default & custom naming)

behavior
Default name (step's "number", starting from one, ie stepId + 1)

Custom name
Custom Step Name can be set in MoreOptionsModal (step-name in form):

When saved, the step name is updated on the Step List:

closed in #778 I think


Update Selenium app to the latest version

This issue is a:

[ ] Bug Report
[x] Enhancement suggestion
[ ] Feature request

If the issue concerns an existing Marketplace application, please tag the maintainer of the application (found in manifest.yaml) below:
Enhancement suggestions/feature requests
Currently, we have Selenium version 3.141.59-r1 installed by Civo marketplace as a newer version 4.10.0 is available, so it needs to be updated to the newer version. We kindly request an update to the latest version of Selenium on the Civo marketplace to ensure that we have access to the most recent features and improvements.
@scndel is the maintainer for this app, so I request you to please take a look into this.

Indeed it should be updated for a long time, happy that this app is useful.
Done on my side ! #528

cc: @kunal-kushwaha it looks like this one might be close-able as well.


Blog: Docker image maintenance

The Issue
Blog: Contributor Training, Docker Image Maintenance.
With minor changes to a couple other blogs that I discovered while writing
Review at

https://20240730-blog-maintain-docke.ddev-com-front-end.pages.dev/blog/ddev-docker-image-maintenance/
https://20240730-blog-maintain-docke.ddev-com-front-end.pages.dev/blog/ddev-local-automated-testing/ (not much effort needed here, I just did minor updates)

Thanks so much @stasadev . I really appreciate you looking at these things. You have a great eye even when you don't necessarily have opinions about the content.


[MOB-549] sdk docs

We should replace javadoc style comments /** */ with swift style comments with ///.
All swift libraries are commented this way. This is also recommended here
https://google.github.io/swift/#documentation-comments

We should replace javadoc style comments /** */ with swift style comments with ///.
All swift libraries are commented this way. This is also recommended here
https://google.github.io/swift/#documentation-comments

Interesting, Apple's own documentation uses /***/, but I agree with you; I'll change it all to ///.

When you look at actual source code for Swift libraries by selecting a class and then 'Jump to definition' all comments are like this. So I think we are doing the right thing.

When you look at actual source code for Swift libraries by selecting a class and then 'Jump to definition' all comments are like this. So I think we are doing the right thing.

Yeah, definitely agree. I think the link I used might be out of date, stylistically.


[Oracle] add a new query to obtain the release

The current oracle::api::QueryMsg::ContractVersion provides the software package version only. It is primarily consumed by the feeders and dispatchers to check their compatibility with the Oracle API.
Add another QueryMsg::Release variant to return the whole Release instance of the contract. It would include the software package, the storage version, and the release label.

specified with #466


t-test in scipy.stats: combine tests?

There are four different t-tests in scipy:

ttest_1samp - Calculate the T-test for the mean of ONE group of scores.
ttest_ind - Calculate the T-test for the means of two independent samples of scores.
ttest_rel - Calculate the T-test on TWO RELATED samples of scores, a and b.
ttest_ind_from_stats - T-test for means of two independent samples from descriptive statistics.

In R, the first three cases are combined (second sample is optional, whether samples are related is determined by keyword paired):
t.test(x, y = NULL, alternative = c("two.sided", "less", "greater"), mu = 0, paired = FALSE, var.equal = FALSE, conf.level = 0.95, ...)
(https://stat.ethz.ch/R-manual/R-devel/library/stats/html/t.test.html)
I think it would be nice to have a single t-test function in scipy as well for the first three cases that then calls the relevant version of the current tests and replaces them after some time.
Any views on that?

I think an encompassing frontend can serve well here. Since the names are already a bit cryptic, forwarding the users to a common place should be easier than taming the signatures and dealing with backwards compatibility issues, should we decide to combine them. After a few releases we can reconsider these functions fate depending on the success of the frontend. By then people would probably adopted the new frontend.
Having said all that, stats residents would know better 😃

I don’t know if I’m qualified as a stats resident or not but I agree.
Sincerely,
-Lucas Roberts

On Nov 15, 2018, at 2:31 PM, Ilhan Polat<EMAIL_ADDRESS>wrote:
I think an encompassing frontend can serve well here. Since the names are already a bit cryptic, forwarding the users to a common place should be easier than taming the signatures and dealing with backwards compatibility issues, should we decide to combine them. After a few releases we can reconsider these functions fate depending on the success of the frontend. By then people would probably adopted the new frontend.
Having said all that, stats residents would know better 😃
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Somewhat related: https://mail.python.org/pipermail/scipy-user/2012-October/033441.html
I believe we discussed combining t-tests somewhere else before as well, but can't find it right now.
Combining makes sense in principle I think. The name stats.ttest is still available.

@josef-pkt 👍  excellent points I hadn't considered, especially the naming convention.

Another thought about organizing tests
Hypothesis tests named after the test distribution are focusing on the wrong distinction, and are >mainly useful for name recognition or as helper function.

e.g. z-test and t-test are the same thing and just use a different distributional assumption, but are >equivalent in large samples.

RE: compare_location => this could be a more general method that handles both. In that case for consistency, we'd want to have higher level interface functions for ANOVA too (a function to call the parametric or non-parametric versions). Perhaps that one should be called compare_2_or_more_locations  which could even call the t-tests if only 2 are given.
If this is done, should we strive for consistency in this convention across testing functions? If so, is   a plan or recommendation on which things to merge together, naming convention guidelines, etc. warranted?

Consistency is important because it requires less reading of the documentation for individual cases.
The other issue is that some function names are "famous", i.e. what the users would be searching for. In other cases the name is uninformative except for experts, e.g. I never remember what mood does exactly and  how it differs from levene and bartlett.
e.g. https://github.com/statsmodels/statsmodels/issues/5337
I think it would be useful to go through the existing functions and see whether and how easily they can be merged along the lines of handling similar use cases like comparing means or comparing variances in 2 or k samples. And also to see whether the options can be combined in good informative keyword arguments.
Again what I'm planning for and working on (on and off) in statsmodels are functions for specific use cases and then try to get as many options and properties as possible.
Besides mean/location and scale/variance/dispersion for a continuous variables, there are 1 and 2 sample versions for proportions (in [0, 1]), rates like poisson rates (in [0, inf]), correlation and association, and then multivariate analogues.
e.g. in many hypothesis test for these different cases there will be one or several z-test versions that use an asymptotic normal distribution.
(I moved a bit away from creating big classes that include everything because I often up that using the simpler functions in statsmodels or scipy.stats myself for quick use cases.)

Hypothesis tests named after the test distribution are focusing on the wrong distinction, and are mainly useful for name recognition or as helper function.

True, but combining the tests by topic like location tests has several drawbacks:

many different options / parameters (as you state for some Tests in R packages)
documentation will become very long and hard to read if one needs to distinguish many cases

I do think it would make sense to have more guidance for the user by grouping tests by topic in the documentation: link tests via "See also" and also on the overview page https://docs.scipy.org/doc/scipy-1.1.0/reference/stats.html
(In 1.2.0, the functions will already be grouped in a better way, there was a related PR.)

I think I agree with @josef-pkt here. The use cases of the various t-tests, KS tests, etc. are quite different, and I agree that it is preferable to combine functions that do similar things and have similar interfaces rather than those that share history, choice of statistic, or null distribution.
goodness_of_fit unified ks_test, anderson, and cramervonmises (and could work with essentially any other general distribution comparison statistic), but that was driven mostly out of a need to upgrade anderson; adding the other statistics took just a few extra lines of code. That said, I (personally) wouldn't push for a big effort to unify existing functions grouped by use case (like root or minimize) right now. There is a lot going on that is also valuable, and I think our attention is already spread a bit thin. I think we'll run into @chrisb83's concerns (and others) regardless of how functions are combined, and resolving them will require a fair amount of dedicated thought/work.

To follow-up
I added many statistical functions to statsmodels since the original comments here.
My pattern now is to

never mix sampling cases, eg. separate 1-sample, 2-sample and k-sample functions, separate for continuous mean, proportions, rates, variance, ...
do not mix hypothesis (except for alternative), I have equivalence tests separate from standard hypothesis test
combine available methods for one test type in one function, (1)
for now I keep nonparametric, eg. rank based, tests separate from parametric tests (also because they don't match a specific response type)
I'm not yet adding confidence interval to the hypothesis test functions. (I'm still undecided on how to combine those)
I have other function like confidence intervals, power and sample size functions separate from hypothesis test function.

(1) my oneway anova for continuous response combines methods variance assumptions (standard, Welsh, BF) and trimming (Yuen) with all combinations between the two method types.
https://www.statsmodels.org/dev/generated/statsmodels.stats.oneway.anova_oneway.html
The last one I added to work more on this pattern are tests and other statistics for poisson rates
https://www.statsmodels.org/dev/stats.html#rates
The 2-sample tests also combine test for different comparisons, rate difference, rate ratio, in proportion also odds ratio.
(I have not yet refactored the proportion test to follow this pattern. Those are still disorganized as in other packages, R, SAS)
related: trying to organize what's available in statsmodels
https://github.com/statsmodels/statsmodels/issues/8331

The use cases of the various t-tests, KS tests, etc. are quite different, and I agree that it is preferable to combine functions that do similar things and have similar interfaces rather than those that share history, choice of statistic, or null distribution.

I agree. I was used to the R approach to combine tests. I would say we can close this issue when #17767 is merged: by ordering the tests by topic, there is even less need for a combined test.


Add option for channel backup and recovery

Add option for channel backup and recovery.

It would be great to have this option in the GUI. Or at least the possibility to create / recover the onchain wallet by entering the 12 words (using lightning-hsmtool).


[RU] Added 'advanced/concurrency' lesson translation.

Thank you so much @nscyclone!
@brain-geek or @tetiana12345678, would you mind look over this one too?

@nscyclone I've added some notes here.

Thanks for the review, @brain-geek ! I've fixed mentioned mistakes.


Update Snapchat.swift

Salam,
I add Snapchat App Store ID, and I fixed a typo.

i think i will review your code on the original repo accept the PR, then ill add this PR to the original repo @Maryom


fix: removed extra space on the course tab

Removed extra space on the course tab in different scenarios like: with/without new design, with/without resume button, with/without Dates banner.

Reviewing


Enable "dot" notation for Objects

Currently when doing something like shown in the example below you will end up with an array containing only the values defined with the "dot" notation. This is due to the limitations of Arr::set() only working on arrays and the fact that the API Resource returns a stdClass rather than an array. Changing Arr::set() to data_set will allow you to use the "dot" notation on Objects as well.
In the example you could probably handle the relation in the OrderResource itself but you would miss out on the partial reload or lazy data feature in such cases.
<?php

use App\Http\Resources\AddressResource;
use App\Http\Resources\OrderResource;

return inertia('Orders/Show', [

    'order' => OrderResource::make($order),

    'order.address' => fn () => AddressResource::make($order->address),

]);

// Resulting data using Arr::set()
[
    'order' => [
        'address' => [
            // …
        ]
    ],
]

// Resulting data using data_set()
[
    'order' => [
        'id' => 1,
        'number' => 350647,
        // …
        'address' => [
            // …
        ]
    ],
]

Looks good! Let's hope data_set never gets deprecated. 😅

Released: https://github.com/inertiajs/inertia-laravel/releases/tag/v0.4.3


merge into http-client

i only defined the lenses and not the prisms myself, so maybe it's not as easy as i imagine.
can you merge it into http-client? @snoyman is nice and responsive to changes if there are no additional dependencies. you could avoid the lens dependency by unrolling the lens operators and types (at least, that's what i usually do).

Nope. https://github.com/snoyberg/http-client/pull/259


Questions about Table 6

In Table 6 of the paper, the 'Collapsed Cost Volume' experiment shows that RAFT-Stereo is faster than RAFT and has fewer parameters, but it does not provide 1px error on the FlyingThings3D dataset. Does RAFT-Stereo have an advantage over RAFT in accuracy?
Thanks!

Regarding the 'Collapsed Cost Volume' ablation experiment, collapsing the cost volume of RAFT on its own shouldn't result in improved accuracy, hence we only report the runtime improvement in this case.

thank you for your reply!


Coordinates under IFF Label on Mass Scanners and Shuttle Consoles

About the PR

Added coordinates beneath IFF labels on mass scanners and shuttle nav consoles.
Reworks the viewport clamping behavior on the labels to instead be normalised, preserving the vector direction and making them a lot more useful for navigation.
Why / Balance

Knowing the coords for different grids is very handy information especially with the removal of the Reclaimer shuttle.
Allows you to navigate to different grids with just a GPS if you have the grid coords.
Technical details

Largely just adds a secondary label to the IFF label draw, but I cleaned up some of the surrounding code, math, and comments.
Unlike the IFF labels themselves, the coord label isn't clamped to the viewport edges, so it (hopefully) shouldn't add too much visual clutter.
Media

https://github.com/user-attachments/assets/8b976e13-835b-4ebd-ae69-2eee116d805f
Requirements

[X] I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

add: Mass scanners and shuttle consoles now display coordinates beneath IFF labels

https://github.com/user-attachments/assets/c9642662-8f14-4752-8699-2742de442e05
Hey, I gave it a test and the coordinates move a little weird relative to the IFF label in some cases. Could you make both labels properly parented to each other, but not show the coords when out of range?
Otherwise this is a great QOL improvement.

IFF.coords.mp4
Hey, I gave it a test and the coordinates move a little weird relative to the IFF label in some cases. Could you make both labels properly parented to each other, but not show the coords when out of range?
Otherwise this is a great QOL improvement.

Should be very doable, thanks for catching this!


SBT should default the current project directory as the working directory of forked runs

I have a multi-project sbt file that I recently added the fork in (Compile, run) := true option on.
One of the first issues that popped up was the process wouldn't read a local file "localdir/localfiles.txt" - basically the working directory wasn't set to the project directory.  This was easily fixed by manually specifying a working directory such as baseDirectory in (Compile, run) := file(".")
I would expect this to be a default behavior, rather than needing to be explicit.  IntelliJ recently (in the last year or two?) also added a "Use project home as compile server working directory" which is the same/similar behavior I would expect.
an sbt "project subproject" "runMain foo.foo.App" will also use the top-level directory as the working directory
sbt version: 1.1.0

Thank you, this workaround helped me fix another long-lasting pain: in multi-projects, tests would run in either sbt or intellij. It works in both if your subproject/build.sbt specifies
Test/baseDirectory := (ThisBuild/Test/run/baseDirectory).value

Slightly more context: http://www.michaelpollmeier.com/2019/02/20/sbt-subprojects-tests-intellij

I'm probably working with an sbt version that still has this behavior. What is the workaround actually?

Just to exemplify a possible workaround in another way ― in the subproject:
.settings(baseDirectory := (baseDirectory in LocalRootProject).value)

Just curious whether this behavior ever changed in newer versions. I think that parity between interactive sbt sessions and using the sbt command for running tests or mains ― is a major plus. Especially if the harmony extends to IntelliJ.

@matanster  I think SBT maybe inconsistent between forked and non-forked runs, but I'm not sure if this behavior changed at all.
There is a semi-related bloop issue here: https://github.com/scalacenter/bloop/pull/1098

I don't think it would be a desirable to port the whole task/config/project axis scoping rules from sbt to bloop.

This ticket should probably be renamed/updated or something to instead just be "Make working directory consistent between forked and non-forked runs".  Bloop wants to standardize on the sub-project as the working directory, so SBT probably should standardize on that too (e.g. make the change @dwijnand suggested) which still allows for the baseDirectory override option in a build.sbt/

"Make working directory consistent regardless of fork / nofork and interactive sbt v.s. sbt as a commandline"
I guess bloop is some componentization of sbt into a language server (if this term ever picked up) and the rest of sbt, but I wouldn't really know.

What are the relationships between sbt and sbt-bloop?!

What are the relationships between sbt and sbt-bloop?! do you also have to worry about sbt-bloop compatibility here?

Bloop can read build.sbt files and create bloop-configs so that it can support most sbt projects.
The previous example/link is that sbt and bloop diverge supporting the "task/config/project axis scoping rules in sbt" for specifying the working directory.  However, bloop is more consistent, in that it uses the subproject for the working directory.
build.sbt Examples
"Native" SBT command working directory override
baseDirectory in (Test) := file("./test"),
vs
baseDirectory in (Compile, run) := file("."),
Bloop (in the build.sbt) working directory override
javaOptions += s"-Duser.dir=${baseDirectory.in(ThisBuild).value}"
Having a different test vs run is not supported by bloop (I think) /cc @jvican

The fork vs non-fork working directory discussion seems to be a duplicate of https://github.com/sbt/sbt/issues/1032

Closing, duplicate of: https://github.com/sbt/sbt/issues/1032


[Snyk] Security upgrade alpine from 3.18.0 to 3

This PR was automatically created by Snyk using the credentials of a real user.Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Changes included in this PR

monitoring-as-code/Dockerfile

We recommend upgrading to alpine:3, as this image has only 0 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Some of the most important vulnerabilities in your base image include:

Severity
Priority Score / 1000
Issue
Exploit Maturity

471
Allocation of Resources Without Limits or Throttling SNYK-ALPINE318-OPENSSL-5661572
No Known Exploit

471
Allocation of Resources Without Limits or Throttling SNYK-ALPINE318-OPENSSL-5661572
No Known Exploit

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Allocation of Resources Without Limits or Throttling

Superseded.


[Error] AttributeError: 'InteractionSerialEvaluator' object has no attribute '_end_flag'

[ ] I have marked all applicable categories:

[ ] exception-raising bug
[ ] RL algorithm bug
[x] system worker bug
[ ] system utils bug
[ ] code design/refactor
[ ] documentation request
[ ] new feature request

[x] I have visited the readme and doc
[x] I have searched through the issue tracker and pr tracker
[x] I have mentioned version numbers, operating system and environment, where applicable:import ding, torch, sys
print(ding.__version__, torch.__version__, sys.version, sys.platform)

v0.3.1 1.8.1+cpu 3.8.12 (default, Oct 12 2021, 03:01:40) [MSC v.1916 64 bit (AMD64)] win32
When running the basic example: python3 -u dizoo/classic_control/cartpole/entry/cartpole_dqn_main.py
It shows the following error.
Traceback (most recent call last):
  File "C:/ProgramData/Anaconda3/envs/PYTORCH/Lib/site-packages/dizoo/classic_control/cartpole/entry/cartpole_dqn_main.py", line 91, in <module>
    main(cartpole_dqn_config)
  File "C:/ProgramData/Anaconda3/envs/PYTORCH/Lib/site-packages/dizoo/classic_control/cartpole/entry/cartpole_dqn_main.py", line 84, in main
    evaluator = InteractionSerialEvaluator(
  File "C:\ProgramData\Anaconda3\envs\PYTORCH\lib\site-packages\ding\worker\collector\interaction_serial_evaluator.py", line 56, in __init__
    self.reset(policy, env)
  File "C:\ProgramData\Anaconda3\envs\PYTORCH\lib\site-packages\ding\worker\collector\interaction_serial_evaluator.py", line 112, in reset
    self.reset_env(_env)
  File "C:\ProgramData\Anaconda3\envs\PYTORCH\lib\site-packages\ding\worker\collector\interaction_serial_evaluator.py", line 76, in reset_env
    self._env.launch()
  File "C:\ProgramData\Anaconda3\envs\PYTORCH\lib\site-packages\ding\envs\env_manager\base_env_manager.py", line 199, in launch
[2022-05-22 16:15:02] ERROR    Env 0 reset has exceeded max retries(1)                                                                             base_env_manager.py:274
    self.reset(reset_param)
  File "C:\ProgramData\Anaconda3\envs\PYTORCH\lib\site-packages\ding\envs\env_manager\base_env_manager.py", line 242, in reset
    self._reset(env_id)
  File "C:\ProgramData\Anaconda3\envs\PYTORCH\lib\site-packages\ding\envs\env_manager\base_env_manager.py", line 281, in _reset
    raise runtime_error
  File "C:\ProgramData\Anaconda3\envs\PYTORCH\lib\site-packages\ding\envs\env_manager\base_env_manager.py", line 259, in _reset
    obs = reset_fn()
  File "C:\ProgramData\Anaconda3\envs\PYTORCH\lib\site-packages\ding\envs\env_manager\base_env_manager.py", line 251, in reset_fn
    return self._envs[env_id].reset(**self._reset_param[env_id])
  File "C:\ProgramData\Anaconda3\envs\PYTORCH\lib\site-packages\ding\envs\env\ding_env_wrapper.py", line 68, in reset
    obs = self._env.reset()
  File "C:\ProgramData\Anaconda3\envs\PYTORCH\lib\site-packages\gym\wrappers\record_video.py", line 58, in reset
    self.start_video_recorder()
  File "C:\ProgramData\Anaconda3\envs\PYTORCH\lib\site-packages\gym\wrappers\record_video.py", line 75, in start_video_recorder
    self.video_recorder.capture_frame()
  File "C:\ProgramData\Anaconda3\envs\PYTORCH\lib\site-packages\gym\wrappers\monitoring\video_recorder.py", line 155, in capture_frame
    self._encode_image_frame(frame)
  File "C:\ProgramData\Anaconda3\envs\PYTORCH\lib\site-packages\gym\wrappers\monitoring\video_recorder.py", line 213, in _encode_image_frame
    self.encoder = ImageEncoder(
  File "C:\ProgramData\Anaconda3\envs\PYTORCH\lib\site-packages\gym\wrappers\monitoring\video_recorder.py", line 337, in __init__
    raise error.DependencyNotInstalled(
RuntimeError: Env 0 reset has exceeded max retries(1), and the latest exception is: DependencyNotInstalled("Found neither the ffmpeg nor avconv executables. On OS X, you can install ffmpeg via `brew install ffmpeg`. On most Ubuntu variants, `sudo apt-get install ffmpeg` should do it. On Ubuntu 14.04, however, you'll need to install avconv with `sudo apt-get install libav-tools`.")
Exception ignored in: <function InteractionSerialEvaluator.__del__ at 0x0000017CA3252280>
Traceback (most recent call last):
  File "C:\ProgramData\Anaconda3\envs\PYTORCH\lib\site-packages\ding\worker\collector\interaction_serial_evaluator.py", line 138, in __del__
  File "C:\ProgramData\Anaconda3\envs\PYTORCH\lib\site-packages\ding\worker\collector\interaction_serial_evaluator.py", line 125, in close
AttributeError: 'InteractionSerialEvaluator' object has no attribute '_end_flag'

Process finished with exit code 1

There is already some hints in you error messages: Found neither the ffmpeg nor avconv executables.
This bug is because your setting doesn't install ffmpeg, which is used to save replay video when evaluation, and there are 2 solutions:

install ffmpeg, but it may be a lit hard in win32 system.
comment the code part related to save replay video, for example, just comment the code related to enable_save_replay

There is already some hints in you error messages: Found neither the ffmpeg nor avconv executables.
This bug is because your setting doesn't install ffmpeg, which is used to save replay video when evaluation, and there are 2 solutions:

install ffmpeg, but it may be a lit hard in win32 system.
comment the code part related to save replay video, for example, just comment the code related to enable_save_replay

I also met the above error on Windows 10 64 bit. Then I tried pip install ffmpeg, but it didn't work. After searching on the Internet, an effective resolution is to download ffmpeg here with a correct version,  unzip it and then add the 'bin' directory path to the system variable. To verify it, use ffmpeg -version.


Test order shouldn't be important

Clean up after each test.

I also think all of "test.js" should be split further into separate files, one for each module "files", "sharing", etc. The unauthenticated/unauthorized stuff can stay in single files.

blocks https://github.com/owncloud/js-owncloud-client/pull/86

bonus: use async/await in the test, would make it more readable than promises ?

After #645 and #646 this is no logner an issue

After #645 and #646 this is no logner an issue


Using tesseract and mnist dataset together

hi,
I made a license plate recognition system. I used the yolov5 model and the tesseract library to read text. I am building the project with linux operating system. but tesseract is insufficient to read license plate. I will ask you this; Can I use mnist dataset and tesseract together?
I have no clue how to do this.i need your help on this.I couldn't find any reference about it.please help me.

Please respect guidelines for posting issues: we do not provide support for 3rd party projects. Use tesseract user forum for asking questions/support.


Cannot open C++ stream

I'm passing a path to a file as an str from Rust, and I'm then trying to open it in C++ with an ifstream.
Rust:
pub mod trt_bindings {
    unsafe extern "C++" {
        // A bunch of things in here
        fn parse(parser: &UniquePtr<ONNXParserTRT>, onnx_model: &str, verbosity: i32) -> bool;
    }
}

C++:
bool parse(const std::unique_ptr<ONNXParserTRT> &parser, rust::Str onnxModelFile, int verbosity)
{
    std::cout << "onnxModelFile: " << onnxModelFile << std::endl; // Shows the correct path: /my/path/to/my/file

    std::ifstream stream(onnxModelFile.data(), std::ios::in | std::ios::binary);
    if (!stream)
    {
        // Definitely not the correct path here! It concatenates the correct path with
        // everything that has been written in std::cout or in rust println!
        // If I have a println!("hello") in my Rust main code, it will show: "Could not
        // open file /my/path/to/my/filehello
        std::cerr << "Could not open file " << std::string(onnxModelFile.data()) << std::endl;
        return false;
    }

    // Some other code here
}

What could I be missing? :worried:
This can be reproduced in the demo example in this repository by modifying the void BlobstoreClient::tag(uint64_t blobid, rust::Str tag) const function (cxx/demo/src/blobstore.cc):
// Add tag to an existing blob.
void BlobstoreClient::tag(uint64_t blobid, rust::Str tag) const {
  std::ifstream stream(tag.data(), std::ios::in | std::ios::binary);
  if (!stream)
  {
      std::cout << "Could not open file '" << std::string(tag.data()) << "'" << std::endl;
  }
  impl->blobs[blobid].tags.emplace(tag);
}

I then get the following output:
$ cargo run
   Compiling demo v0.0.0 (/home/aurelien/Projects/GitHub/cxx/demo)
    Finished dev [unoptimized + debuginfo] target(s) in 1.00s
     Running `/home/aurelien/Projects/GitHub/cxx/target/debug/demo`
blobid =<PHONE_NUMBER>040795552
Could not open file 'rusttags = invalid args/rustc/e092d0b6b43f2de967af0887873151bb1c0b18d3/library/core/src/fmt/mod.rscalled deref on a null UniquePtr<>/home/aurelien/Projects/GitHub/cxx/src/unique_ptr.rs/rustc/e092d0b6b43f2de967af0887873151bb1c0b18d3/library/core/src/alloc/layout.rs'
tags = ["rust"]

The ifstream constructor needs a null-terminated string, not a rust string.

Thanks @dtolnay! This is what I needed :smile:


Question regarding tests

Hi, I would like to help with the testing of the C to MIR translator. One of the approaches I used for dmrC was to source tests from various places and adapt them. The issue of course is sometimes the tests have their own licences. Do you care about that?
Anyway please let me know if this is of interest.

One of the approaches I used for dmrC was to source tests from various places and adapt them. The issue of course is sometimes the tests have their own licences. Do you care about that?

Thank you for the proposal.  Yes, I am interesting in C tests and I believe your experience will be useful to find such tests.
Currently, I am using some tests  from GCC testsuite.  They are small and currently very useful to find and debug C2MIR translator.  Bigger tests are not useful for me right now as some regular C11 features are not implemented yet and there are still a lot of bugs in the translator/MIR-interpreter/MIR-generator.
When I succeed with the small tests, I'll need bigger ones (probably in 1-2 months).   So I am open to any tests proposals.
The license issue can be tricky one.
It would be nice to have also tests which check C11 conformance and contains wrong C code too.   Unfortunately, such conformant tests (like perennial/plumhall) are not free.

Perhaps I should do this in a separate repo so that there is no pollution re the license.

Hi, I started this at:
https://github.com/dibyendumajumdar/mir-testing
It is on a branch called tests.
I copied across the dmrC tests which are in turn sourced from various places.
Some are small tests and some are bigger.
It should be easy for you to run the tests if you would like to: instructions are in README.
Regards

I added more tests from other sources. A key criteria is that the test must be runnable. So every test has to have a main function. The test success is determined by using a combination of return code from the test and any stdout output.

Here are some stats:
Total tests run 447
Successful tests 230
Not bad

Thank you.  I will look at this later when I resume my work on the project.

@vnmakarov I merged the latest updates. The current results are:
Total tests run 447
Successful tests 316

Pretty good improvement!
Thanks

Total tests run 447
Successful tests 337

Pretty good improvement!

Thanks for the tests.  I worked on them and they are pretty useful.
Although I found some troubles with them:

some tests (from  TESTED) check errors and are supposed to be failed
some  tests expect wrong output, gcc and clang fail on them too
some tests expect specific undefined C behaviour
some tests check GCC extensions (binary constants, case ranges, extended ternary operators etc)

But in general they are useful and I will add them to repository after their analysis.
Meanwhile I've just added about 600 GCC tests I am using myself.  They can be called by make c2mir-full-test.

Although I found some troubles with them:
* some tests (from  TESTED) check errors and are supposed to be failed

* some  tests expect wrong output, gcc and clang fail on them too

* some tests expect specific undefined C behaviour

* some tests check GCC extensions (binary constants, case ranges, extended ternary operators etc)

Yes perhaps as I have not checked all of the failures, and also many tests I just picked from other projects. The ones I use for dmrC are a smaller subset; those are included as well of course.

But in general they are useful and I will add them to repository after their analysis.
Meanwhile I've just added about 600 GCC tests I am using myself. They can be called by make c2mir-full-test.

Cool. I hope this won't cause a license issue.
Are all these tests passing?
I am very impressed by your project and am eagerly waiting to start using it.

So looks like passes those 655 gcc tests! Wonderful!

So looks like passes those 655 gcc tests! Wonderful!

They are only tests which are passing but there are a few tests from gcc I use which are not passing.  I will add them when they start passing.  It is easier this way not to introduce new regressions.

Cool. I hope this won't cause a license issue.

I am not a lawyer and can not give advices to other people but for me it is obvious that it does not violate the GPL license (and its spirit).

Hi, I have deleted the mir-testing repo because I had trouble merging changes from your repo. In any case looks like you are adding the tests to mir so no point having the testing repo.

Hi, I have deleted the mir-testing repo because I had trouble merging changes from your repo. In any case looks like you are adding the tests to mir so no point having the testing repo.

Sorry for creating a conflict.  I really appreciate your work and collecting a lot of C tests in mir-test repo.  Mine tests from gcc-torture were not enough.
I found that Andrew Chambers and especially LACC tests have much better coverage of C standard.  LACC tests revealed a lot of problems with LLVM to MIR translator.  Basically, I did not implement composite data operators yet (insert and extract value) which are generated for small structures for some ABIs. Implementing this functionality will take some time.  So currently I switched off LLVM2MIR translator testing in Makefile.
As for C2MIR compiler I've managed to fix all LACC tests.  The next big step will be achieving a bootstrap for C2MIR.

Hi, I have deleted the mir-testing repo because I had trouble merging changes from your repo. In any case looks like you are adding the tests to mir so no point having the testing repo.

Sorry for creating a conflict. I really appreciate your work and collecting a lot of C tests in mir-test repo. Mine tests from gcc-torture were not enough.

No worries I am glad that it was helpful.

I found that Andrew Chambers and especially LACC tests have much better coverage of C standard. LACC tests revealed a lot of problems with LLVM to MIR translator. Basically, I did not implement composite data operators yet (insert and extract value) which are generated for small structures for some ABIs. Implementing this functionality will take some time. So currently I switched off LLVM2MIR translator testing in Makefile.
As for C2MIR compiler I've managed to fix all LACC tests. The next big step will be achieving a bootstrap for C2MIR.

That's great news.


Friends being deleted upon dropping

Likely a race condition with the sync primitives in how friends are saved, causing one thread to use one set of data and not share the same data through the reference counter (Arc). This seems to be at random when dropping so temporary solution would be remove the manual drop function, however the best thing to do is to investigate into what is going on that would cause the split in the data set.

Note: The issue isnt a race condition as originally assumed but that at random times while saving when dropping, it stops when attempting to write to disk likely due to the runtime ending before it can write to the file. Solution would be to have it use std::fs while dropping.


chore: migrate from List.removeNth to List.eraseIdx

[x] depends on: leanprover/std4#781

bors merge


Develop

Hi @mhor,
I added support to dump MediaInfoContainer or any Type to XML.
Unit Test Added.
Regards,

Hi @mhor, you have time to see this request?
Regards,

Hi @javiertrejo,  could you:

sync your fork with this repository => https://help.github.com/articles/syncing-a-fork/
remove test.mp3
update README with new features
update pull request title

Hi @mhor,
The requested changes has been applied. Sorry for my mistake, i'm a newbie with forking 3rd party projects and contributing.
To dump to YAML it's necesary use PECL/YAML extension. This is not a common part of PHP installations but is possible to do.
Considers useful and necessary to have the option to dump to YAML?
The next week i will be applied this project in a production environment. If any bug will be detected; i will send to you a new pull request with the fixs.
Regards,

@javiertrejo I didn't know if dumping to YAML is useful. To dump yaml IMO the easiest way is the use of YAML symfony component
Thanks for pull request I will merge it!

@javiertrejo merged and released as 2.1.0


Message listener with regex /.+/ to run only if none of the above listeners match the incoming message

I am making an app which has some listeners to welcome, greet, etc and a default listener with regex /.+/ for all other messages which passes the text in the payload to the AI process running.
I want the default one to only run if the above written listeners do not match the message but it is executed along with them.
Reproducible in:
The Slack SDK version
"slack/bolt": "^3.12.2"
Node.js runtime version
v19.7.0
OS info
OS: Linux Mint 21.1 Cinnamon
Kernel: 5.15.0-67-generic
Steps to reproduce:
(Share the commands to run, source code, and project settings)
src/slack_events/messages.ts
import { App } from "@slack/bolt";
import { GenericMessageEvent } from "@slack/bolt";

export function enableMessageListeners(app: App) {
    // Handle message containing hi, hello, etc.
    app.message(/(hey|hi|hello|heya|yo)(\s<@\w+>)?$/gim, async ({context, payload, say}) => {
        console.log("HI PAYLOAD", payload)
        payload = <GenericMessageEvent>payload
        await say(`${context.matches[0].toUpperCase()}<EMAIL_ADDRESS>:wave:`)
    });

    // Handle message containing good morning, evening etc.
    app.message(/good (morning|evening|afternoon|night)(\s<@\w+>)?$/gim, async ({payload, say}) => {
        payload = <GenericMessageEvent>payload
        let prefix = payload.text!.split(" ").slice(0, 2).join(" ").toLocaleLowerCase()
        prefix = prefix[0].toUpperCase() + prefix.slice(1)
        await say(`${prefix}<EMAIL_ADDRESS>:smiley:`)
    });

    // Handle any other message and pass the text to AI process
    app.message(/.+/, async ({payload, say}) => {
        console.log("MATCH ALL PAYLOAD", payload)
        await say("Hi! I am a bot. Received your message and will be providing you the results shortly");
    });
}

src/main.ts
import { GenericMessageEvent } from "@slack/bolt";
import { app } from "./app";
import { enableAppMentions } from "./slack_events/app_mentions";
import { enableMessageListeners } from "./slack_events/message";
import { enableSlashListeners } from "./slack_events/slash";

enableAppMentions(app);
enableMessageListeners(app);
enableSlashListeners(app);

(async () => {
    await app.start(Number(process.env.PORT) || 3000);
    console.log('⚡ Bolt app is running!')
})();

Expected result:

When a message starting with hi is received only 1st listener should run.
When a message starting with good morning is received only 2nd listener should run.
Any other message should be passed to 3rd listener

Actual result:

[:x:] 1st and 3rd listener both are executed
[:x:] 2nd and 3rd listener both are executed
[:heavy_check_mark:] only 3rd is executed.

Initial building of app started with bolt-python where it worked as expected
handlers/messages.py
# Handle incoming evens
# Always acknowledge actions, commands, shortcuts, options requests & view submissions

from requests import post
from slack_bolt import Say

from .constants import EXAMPLE
from .utils import create_event, TextTokens, email_notify

rasa_url = "http://localhost:5005/webhooks/rest/webhook"

def msg_greet(payload: dict, say: Say):
    resp = post(
        rasa_url,
        json={"message": f"{payload['text']}", "sender": f"{payload['user']}"}
    ).json()
    say(resp[0]["text"])

def default(payload: dict, say: Say):
    resp = post(
        rasa_url,
        json={"message": f"{payload['text']}", "sender": f"{payload['user']}"}
    ).json()
    say(resp[0]["text"])

listeners.py
# Listen to the various events from slack and redirect them to the linked function

from re import compile, IGNORECASE, ASCII

from slack_bolt.app import App
from .handlers import events, slash

re_hi = compile(r"(hey|hi|hello|heya|yo)(\s<@\w+>)?$", IGNORECASE | ASCII)
re_good = compile(
    r"good (morning|evening|afternoon|night)(\s<@\w+>)?$", IGNORECASE | ASCII
)
re_schedule = compile(r"(schedule|set up) .*", IGNORECASE | ASCII)
re_default = compile(r".*")

def register_listeners(app: App):
    # Register all the events, interactions, slash commands, etc.
    app.message(re_hi)(events.msg_greet)
    app.message(re_good)(events.msg_greet)
    app.event("app_mention", matchers=[match_hi])(events.msg_greet)
    app.event("app_mention", matchers=[match_good])(events.msg_greet)
    app.message(re_default)((events.default))
    app.event("app_mention")(events.default)

def match_hi(event: dict):
    return re_hi.search(event["text"]) is not None

def match_good(event: dict):
    return re_good.search(event["text"]) is not None

def match_schedule(event: dict):
    return re_schedule.search(event["text"]) is not None

slack_app.py
from os import environ

from slack_bolt import App

app = App(token=environ["SLACK_TOKEN"], signing_secret=environ["SIGNING_SECRET"])

from .listeners import register_listeners
register_listeners(app)

Hi @abhie-lp, thanks for asking the question!
Indeed, bolt-python works as you expect but bolt-js is designed a bit differently. Also, we are unable to change the behavior to secure backward-compatibility for a lot of apps running across th world.
So, a simple way to deal with this would be to pass a "done" flag in context and check the flag in the following listeners. Here is a pseudo code. You can set the context property inside an executed listener, and then set onlyUndone middleware to all the listeners.
async function onlyUndone({ context, next }) {
  if (!context.done) {
    await next();
  }
}

app.message("hey", onlyUndone, async ({context}) => {
  context["done"] = true
});

I hope this helps.

It worked :exploding_head: and thank you for the quick response and help. :raised_hands:


The key of an aggregation bucket is always read as a string

We're currently running into a situation where we need to have the key of a terms aggregation (which is a double in Elasticsearch) available as a double in our code, but the value of Key in a KeyedBucket is always a string, so we now have to parse many doubles from strings to accomplish this. But when I look at the response from Elasticsearch, the key does come back as a number there, it's the following code in Nest that always reads the key as a string:
https://github.com/elastic/elasticsearch-net/blob/master/src/Nest/Aggregations/AggregateJsonConverter.cs#L538
var key = reader.ReadAsString();
reader.Read();
var property = reader.Value as string;
if (property == "from" || property == "to")
    return GetRangeBucket(reader, serializer, key);

var keyItem = new KeyedBucket {Key = key};

My proposal would be to add a KeyAsObject property to KeyedBucket that simply gets the reader.Value assigned to it. And to avoid an unnecessary string allocation, the Key property should delegate to KeyAsObject and only create a string when you use the Key property.
private string _key = null;

public string Key
{
    get
    {
        if(_key == null && KeyAsObject != null)
        {
            _key = KeyAsObject.ToString();
        }
    }
    set
    {
        _key = value;
    }
}

Please let me know your thoughts on this!

This sounds very reasonable to me although we should not call ToString() on KeyAsObject.
@gmarz thoughts?

@Mpdreamz thanks for looking into this! What is the objection to calling ToString on KeyAsObject here? That it may be null? Or that KeyAsObject.ToString() results in a different string than reader.ReadAsString()? That might be the case, considering the ToString() would use the .NET logic of converting an int or double or whatever to a string whereas ReadAsString returns the JSON token value as a string. So if the token contains the value "10.0" and .NET decides to serialize the double 10 to the string value "10" instead of "10.0" then that could be a breaking change to whomever relies on that exact value.
However, I'm not sure how else to do it, because as far as I know, ReadAsString and Read both consume the token so you can't do both. You can't both have Key and KeyAsObject. So that would require making this a setting in NEST that would do one or the other I guess. Would that be an option?

@JulianRooze you can do this:
response.Aggs.Terms<int>("my_terms");

which will return the keys as ints.
There are also no extra casts going on here, as it's read as an object now instead of string by the deserializer.


Reinstate os-locale with option to prohibit spawning

<EMAIL_ADDRESS>allows us to nuke the previous stop-gap solution. Now we can tell os-locale not to spawn any subprocesses, so we don't introduce unnecessary latency or, even worse, problems with installing spawn-sync, for downstream dependents like Atom. :sunglasses:
If the build fails due to standard problems, anything not handled by PR #247 will be subsequently fixed here.

@bcoe When you get a chance, this PR needs review. Thanks!

awesome \o/ will make the time this evening.

Great work :sunglasses:, :shipit:

Published as<EMAIL_ADDRESS>


WeaponEntityIndexOutOfRange

When parsing a Demo from 04.02 an IndexOutOfRange occured in AttributeWeapon(Int32, DemoInfo.Player), with the WeaponEntityIndex being bigger than the Array of WeaponEntitites.
I can provide the Demo in question.
http://puu.sh/fyuYT/7457f89224.png

Demo File: http://www.file-upload.net/download-10264309/003056012444641001585_1283988829.dem.html

Sorry, it was a dumb error. I should never use magic-numbers, ever.


treewide: remove obsolete kernel version checks

Motivation for this change
We don't support Linux kernels older than 4.4 in Nixpkgs.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Tangentially related, but 4.4 is projected EOL Feb, 2022 according to https://www.kernel.org/category/releases.html, so we should probably also drop that soonish. That can obviously be done in a different PR though. Just mentioning it here, because there might be more checks that could be dropped then.


⚠ Bofrost.es has degraded performance

In b1a10b1, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1475 ms

Resolved: Bofrost.es performance has improved in f4620d1 after 6 minutes.


Inference is not using GPU

Hello,
I deployed the inference pipeline in a GPU provider. However, the song generation takes too long (5 mins for a 4 min song), which is much longer that what I am reading in the threads. I found that the GPU is barely used in the generation. Probably that is the issue.
Do you know what I might be missing?
Thanks

i found out that you need to set a graph to "cuda" in task manager to see it is being used, try that maybe it rly need that long to generate

What I have noticed is that what is really taking up a lot of time is the preprocessing of songs, i.e. vocal-instruments separation, vocal-background-vocal separation and vocal denoising. Under the hood this is done using MDX-net models. I tried performing the same conversions in the UVR app manually and it is noticeably faster, so there might be room for some improvement here. I am working on it myself (but progress is slow).


Add a "back to top" arrow on post pages

Add a button that appears floating on bottom right. Clicking the button scrolls the user back to the top of the page. This should only appear on the posts page

Example page https://www.terremotocentroitalia.info/post/ (scroll down)

@MoyaSparklefarts I've added smooth scrolling to get the nice flow back to the top. It affects user scrolling too.
If you don't like it, it can removed by deleting these lines:
https://github.com/Ao-Collaboration/Ao-Collaboration.github.io/blob/217abf81f123a0706900cbb038d726dc44760413/assets/css/main.css#L19-L25

No complaints, it must be good. :) Closing

Whoops I didn't see this issue!
Can we remove the black border on the button?

Can do. Could you reopen it so I don't forget please.
On Tue, 8 Dec 2020, 14:12 Moya<EMAIL_ADDRESS>wrote:

Whoops I didn't see this issue!
Can we remove the black border on the button?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/Ao-Collaboration/Ao-Collaboration.github.io/issues/11#issuecomment-740287223,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALESSFF7BQ27XUVPREHKBLSTV4OJANCNFSM4UIFKEZA
.

I don't see a black border

Can you replicate this? What browser are you using

This is with Chrome

This is with Edge Chromium

The black border is how your browser renders that emoji. I'll change it to text and arrow so it's more consistent with the rest of the site and cross browser

Updated @MoyaSparklefarts . How's that?

Looks good now! We can close this issue I think?

Sounds good :)


Provide a cookie consent banner

Since the Keptn website makes use of cookies, a cookie consent banner should be included to inform the user and ask for consent.
Here is one example

Another example

this is good stuff let me try.

stale


[object_store,gcp] Service-principal auth using machine-identity

Is your feature request related to a problem or challenge? Please describe what you are trying to do.

I'd like to authenticate via object_store to GCP, but I don't want to have to pass around a json file of credentials.
Describe the solution you'd like

Ideally, I could use the service principal configured on my compute instance to issue requests. It can be fetched via https://cloud.google.com/compute/docs/metadata/predefined-metadata-keys#instance-metadata
As a less good, but also fine solution.. I'd like access to TokenProvider so that I can just issue the relevant API call myself.
Describe alternatives you've considered

I think I'll have to code my own CredentialProvider, but you already have a higher level abstraction I can't get access to.
Additional context

I believe this is the default behaviour if you don't configure any credential provider, is this not working for some reason?

We were using credential files for local testing, and there was an error when we went to deploy them. Looking through the docs, I couldn't see anything that mentioned getting creds from instance metadata.
Digging in the source, I finally came across https://github.com/apache/arrow-rs/blob/master/object_store/src/gcp/credential.rs#L322 but I think this needs to be a documentation improvement, not a code change.
This is where I was looking and hoping to find something useful: https://docs.rs/object_store/0.7.1/object_store/gcp/type.GcpCredentialProvider.html or https://docs.rs/object_store/0.7.1/object_store/gcp/struct.GoogleCloudStorage.html#method.credentials

but I think this needs to be a documentation improvement, not a code change.

I agree, this is not very clearly documented. I would definitely welcome a PR to improve this.

label_issue.py automatically added labels {'object-store'} from #5188


Move vavr core from Maven to Gradle build

Depends on project structure change #2438.
I did the Gradle build already on the v2.0.0 branch. We will either use the scala-plugin or ammonite for calling the Scala code generator.

Need Gradle settings for Nexus snapshot repo, similar to Maven settings:
<settings xmlns="http://maven.apache.org/SETTINGS/1.0.0"
          xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
          xsi:schemaLocation="http://maven.apache.org/SETTINGS/1.0.0
                      http://maven.apache.org/xsd/settings-1.0.0.xsd">
    <localRepository/>
    <interactiveMode/>
    <usePluginRegistry/>
    <offline/>
    <pluginGroups/>
    <servers>
        <server>
            <id>sonatype-nexus-snapshots</id>
            <username>${env.SONATYPE_USER}</username>
            <password>${env.SONATYPE_PASS}</password>
        </server>
    </servers>
    <mirrors/>
    <proxies/>
    <profiles/>
    <activeProfiles/>
</settings>

We will do a one-liner in .travis-ci.yml for current run-tests.sh script:
#!/bin/bash
set -ev
mvn test -B

# Break CI build on PRs where committed differ from generated sources
if [ -n "$(git status -su vavr/src-gen)" ]; then
  exit 1
fi

if [ -n "$(git status -su vavr-test/src-gen)" ]; then
  exit 1
fi

We should not trigger any other vavr builds anymore and save CI resources that way. Other vavr projects should not depend on snapshots of vavr (core).
We will delete trigger-build.sh:
#!/bin/bash

org="vavr-io"
repo=$1
branch=$2

if [ -z $repo ]; then

  echo "Usage: trigger-build.sh <github-repo> [<branch>]"

else

  if [ -z $branch ]; then
    branch="master"
  fi

  echo "Triggering travis-ci build of $org/$repo [$branch]..."

  body=`cat <<EOF
{ "request": {
    "branch": "$branch"
}}
EOF
`

  curl -s -X POST \
    -H "Content-Type: application/json" \
    -H "Accept: application/json" \
    -H "Travis-API-Version: 3" \
    -H "Authorization: token $TRAVIS_ACCESS_TOKEN" \
    -d "$body" \
    https://api.travis-ci.org/repo/$org%2F$repo/requests

fi


Cucumber example + readme cleanup

Added a cucumber section to the README with a better example of how to set it up. Also made the RSpec/Cucumber sections more clear and separated out the common errors into their own section.

Thanks!  I merged your changes all in as a single commit.


netty doesn't support DNS refresh

Expected behavior
We use spring-boot-starter-data-redis to connect redis with domain name，when redis server down, we switch DNS of domain name，but it doesn't work
Actual behavior
I check the code,whether io.netty.netty-resolver-dns or io.netty.netty-resolver save dns cache after successful parsing.
When will cache refresh? Maybe try for a few times or make a schedule timer to update the cache?
Netty version
4.1.65.Final
JVM version (e.g. java -version)
jdk 1.8
OS version (e.g. uname -a)
windows 10/Centos 7
Thanks for answering,have a nice day!

@normanmaurer How about adding a method to manually clear resolveCache, authoritativeDnsServerCache and cnameCache?

This is expected because a DNS query response should be cached till TTL. Lowering TTL is a good way to ensure the freshness of cached responses.

@skyguard1 it is already possible via dnsCache().clear() etc.

Can we manually specify dns cache TTL by using JVM_OPT? or, can we also use networkaddress.cache.ttl with netty dns resolver?


Possible to specify version 2.x instead of 3.x of Mailchimp API?

Is it possible to specify the version of the MC api that you want to target? Maybe in the gem initializer?

API 2 is going away at the end of this year, as I understand it. You can use an old version of the gem to target that if you need it for now though.
Thanks,
Amro
Sent from my iPhone

On Dec 19, 2016, at 1:57 PM, Seth Siegler<EMAIL_ADDRESS>wrote:
Is it possible to specify the version of the MC api that you want to target? Maybe in the gem initializer?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks @amro - I was just reading about the depreciation as you were typing that. Thanks for your work on this gem too. Great stuff.


fix: #103

With the latest improvements to the theme, I think rosewater fits perfectly.

Looks good to me!


Support declaring dependency constraint to specific commit id used in source dependencies

Expected Behavior

The following should be allowed:
dependencies {
    implementation('org.gradle.cpp-samples:utilities') {
        version {
            commitId = 'sha-1-commit-id'
        }
    }
}

We could also use the version field and if we can't find the version in the list of tag, we try to find that specific string as a commit id, then fail.
Current Behavior

Users can't add a source dependency to a specific commit id.
Context

Steps to Reproduce (for bugs)

Your Environment

Build scan URL:

How about using a ref, it would be more universal:
dependencies {
    implementation('org.gradle.cpp-samples:utilities') {
        version {
            ref = 'branch, tag, sha1'
        }
    }
}

Or support all three with different assignment (branch='...', tag='...', sha1='...').


Remove Old Notebook Editor & Interactive Window code

The new interactive window was using interactiveBase.ts IIRC. There might be some changes needed there.

The new interactive window was using interactiveBase.ts IIRC. There might be some changes needed there.

Nope, the native interactive window does not reference interactiveBase.ts

The new interactive window was using interactiveBase.ts IIRC. There might be some changes

Btw, I have tested Native Notebooks, Native Interactive Window, debugging interactive window & debugging notebooks.
hence My assuption is most of it seems to be ok.

Codecov Report

Merging #6973 (bad616e) into main (4325bcd) will increase coverage by 2%.
The diff coverage is 88%.

:exclamation: Current head bad616e differs from pull request most recent head 2c70d3c. Consider uploading reports for the commit 2c70d3c to get more accurate results

@@           Coverage Diff           @@
##            main   #6973     +/-   ##
=======================================
+ Coverage     60%     62%     +2%
=======================================
  Files        385     369     -16
  Lines      25869   23423   -2446
  Branches    3864    3486    -378
=======================================
- Hits       15539   14655    -884
+ Misses      9042    7539   -1503
+ Partials    1288    1229     -59

Impacted Files
Coverage Δ

src/client/common/application/types.ts
100% <ø> (ø)

src/client/common/constants.ts
96% <ø> (-1%)
:arrow_down:

src/client/common/serviceRegistry.ts
100% <ø> (ø)

src/client/common/utils/localize.ts
95% <ø> (-1%)
:arrow_down:

...ience/interactive-common/interactiveWindowTypes.ts
100% <ø> (ø)

...tascience/kernel-launcher/kernelDaemonPreWarmer.ts
92% <ø> (+6%)
:arrow_up:

src/client/datascience/notebook/notebookEditor.ts
41% <ø> (+<1%)
:arrow_up:

...ent/datascience/notebook/notebookEditorProvider.ts
82% <ø> (+<1%)
:arrow_up:

.../client/datascience/notebook/notebookSerializer.ts
67% <ø> (+<1%)
:arrow_up:

...ascience/notebook/outputs/rendererCommunication.ts
68% <ø> (+2%)
:arrow_up:

... and 62 more


WIP: Restrict Prometheus instance namespaces

Fixes https://github.com/coreos/prometheus-operator/issues/2186
WIP for comments and e2e tests

@brancz , it should be all green, please have a look

Depends on https://github.com/coreos/prometheus-operator/pull/1997  to support BasicAuth secrets for endpoints

WIP until single-item watches added for Secrets in SMon endpoints. Otherwise their updates are not seen right now

@redbaron with https://github.com/coreos/prometheus-operator/pull/2202 being merged, do you want to pick this up?

@redbaron just fyi: i am working actively now on this feature :-) feel free to chime in if you are still planning to work on this too.

This is effectively replaced by https://github.com/coreos/prometheus-operator/pull/2783, minus the embedded secret informer framework.

@s-urbaniak, I might have missed some details, but although it is similar it is not like for like replacement. #2783 requires clusterrolebinding to be able to watch all resources in all namespaces, it then choses to ignore some, but service account permissions are still there. With this PR operator can start and work without cluster-wide access

@redbaron indeed, you are absolutely right. #2783 is not a drop-in replacement for this PR, just a subset filtering prometheus/alertmanager custom resources.


Stop supporting python 3.8

The end of life of python 3.8 will be in October 2024 https://devguide.python.org/versions/
Maybe we could already stop supporting it?
This would facilitate the implementation of #588

in general wouldn't mind about it yeah

Solved with #632


Monorepo SBAT publish npm packages without posinstall problems

Expected Behavior
We are currently misusing posinstall script. It should be used to
install depencies or patch them, and nothing more.
To build the monorepo, we need a build npm script.
Currently we use postinstall for:

patch packages (ok)
key:decrypt (not ok)
download & generate contrat types on contractkit (not ok)
fix rn_version (ok?)
generate contract types on protocol (not ok)
utils: build & prettify (not ok)

This issue should remove the 'not ok' usage, and move it to a yarn build stage.
Current Behavior

This was done already


Test TiDB v7.4.0

What problem does this PR solve?
What is changed and how it works?
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Code changes

Has exported function/method change
Has exported variable/fields change
Has interface methods change
Has persistent data change

Side effects

Possible performance regression
Increased code complexity
Breaking backward compatibility

Related changes

Need to cherry-pick to the release branch
Need to update the documentation
Need to update the tidb-ansible repository
Need to be included in the release note

/run-all-tests tidb=release-7.4 tikv=release-7.4 pd=release-7.4


Increase default timeout to 30 seconds

I've had timeout errors on orders with timeouts under 30 seconds. The orders go through and the transaction id's are lost unless you try to go find them.

I have same problem, i'll appreciate it.

Is this going to be merged? I get a lot of timeouts atm

Hello here?

Same, I had to setup my custom timeout https://github.com/0x13a/bitcoin-trading-dca/blob/master/trading.js#L13


chore: remove .vscode

removed .vscode directory

Delete the whole .vscode folder

Okay!

Where @01Sriharsha I cannot see in the pull request?
Make sure you updated the pull request

Where @01Sriharsha I cannot see in the pull request? Make sure you updated the pull request

I have correctly made a PR! Its visible to me

@Yeasir0032  Yes, It's reflecting and on the next clone or push, It wont be shown

Okay
Let's take the step

Did you referenced the issue in the pr?


pkgs.plasma5.user-manager: init

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Thanks :)

My pleasure :smile:


Support sub-modules or sub-directories or sub-packages

What did you expect to see?
dep ensure -add github.com/Azure/azure-sdk-for-go/storage/blockblob.go
# OR
dep ensure -add github.com/Azure/azure-sdk-for-go --package=storage

I expect the above command to only install the blockblob module and all its dependencies and ignore everything it doesn't need. This is a great example of not wanting to install the entire Azure SDK and just install the blob storage piece.
What did you see instead?
Not implemented

hi, welcome!
dep treats packages as whole, indivisible units, so you cannot target individual files. this is consistent with the compiler's behavior: blockblob.go is just one file in a package whose imports must all be satisfied in order for compilation to proceed. a single file does not constitute a module, even if the authors imagine it that way.
however, it's perfectly fine to target the package itself, as the second command looks to be trying to do: the syntax for that would be dep ensure -add github.com/Azure/azure-sdk-for-go/storage.

Oh! So you're saying that targeting packages is already supported?
If so, then you can go ahead and close this issue.
Thank you

yep!
though note that the version constraint is expressed on the entire project, meaning that you can't have two different packages from the same dependency at different versions. that's by design.

@sdboyer Hi! Unfortunately I can't get this to work. I ran dep ensure -add github.com/coreos/etcd/clientv3, but it added the entire etcd project, including server and documentation etc.
$ ls -p vendor/github.com/coreos/etcd/
CHANGELOG.md                     Dockerfile-test                  V2Procfile                       clientv3/                        etcd.conf.yml.sample             logos/                           scripts/
CODE_OF_CONDUCT.md               Documentation/                   alarm/                           cmd/                             etcdctl/                         main.go                          snap/
CONTRIBUTING.md                  LICENSE                          auth/                            compactor/                       etcdmain/                        main_test.go                     store/
DCO                              MAINTAINERS                      bill-of-materials.json           contrib/                         etcdserver/                      meeting.ics                      test
Dockerfile                       NOTICE                           bill-of-materials.override.json  cover                            glide.lock                       mvcc/                            tools/
Dockerfile-functional-tester     Procfile                         build                            discovery/                       glide.yaml                       pkg/                             version/
Dockerfile-release               Procfile.v2                      build.bat                        e2e/                             hack/                            proxy/                           wal/
Dockerfile-release.arm64         README.md                        build.ps1                        embed/                           integration/                     raft/
Dockerfile-release.ppc64le       ROADMAP.md                       client/                          error/                           lease/                           rafthttp/

And Gopkg.toml now includes these lines:
[[constraint]]
  name = "github.com/coreos/etcd"
  version = "3.3.2"

Am I doing something wrong?
$ dep version
dep:
 version     : v0.4.1
 build date  : 2018-01-27
 git hash    : 37d9ea0
 go version  : go1.9.3
 go compiler : gc
 platform    : darwin/amd64


ARROW-11034: [Rust] remove rustfmt ignore list, fix format

In this PR:

Ignore list is commented out from rustfmt.toml, thus carrgo +stable fmt would not show warnings
Two files are re-formatted by nightly, stable: this may avoid formatting problem we had seen in CI (hope so).

Shell commands that I had run:
cargo +nightly-2020-11-24-x86_64 fmt
cargo +1.48.0-x86_64 fmt

Codecov Report

Merging #9013 (4e40312) into master (a4f7c4a) will decrease coverage by 0.00%.
The diff coverage is 68.42%.

@@            Coverage Diff             @@
##           master    #9013      +/-   ##
==========================================
- Coverage   82.87%   82.87%   -0.01%
==========================================
  Files         201      201
  Lines       49739    49743       +4
==========================================
+ Hits        41220    41223       +3
- Misses       8519     8520       +1

Impacted Files
Coverage Δ

rust/parquet/src/record/api.rs
97.88% <66.66%> (-0.13%)
:arrow_down:

rust/datafusion/src/scalar.rs
59.07% <68.75%> (-0.08%)
:arrow_down:

rust/parquet/src/encodings/encoding.rs
95.43% <0.00%> (+0.19%)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks @mqy !


Possible problem with StatusPages feature and error responses ?

Hi,
I'm using version 0.3.6 and everything works correctly for responses that are not in error. However, when I have an exception on my backend, no root span is created by the OpenTracingServer feature. It seems to be linked with the use of the StatusPages feature.
For ex. if I have the StatusPages feature installed with this configuration :
install(StatusPages) {
    exception<Throwable> { cause ->
        call.respond(HttpStatusCode.InternalServerError)
    }
}

and throw an exception in a get mapping :
get("test") {
    throw IllegalArgumentException()
}

-> then no span is created.
And if I uninstall the  StatusPages feature and modify my mapping to do :
get("test") {
    call.respond(HttpStatusCode.InternalServerError)
}

-> then a span is created as expected.
I think it is linked to phases but I know very little on the topic so....
Thanks

Ok, I took some time to try to understand what is going on. I started fresh from a new projet with the ktor official bootstrap, adding only the StatusPages plugin. What caught my attention is that the StatusPages is installed inside the routing { } closure and not outside. In my projet it is outside.
I then tried both configurations and the /exception endpoint logs a span only if the install(StatusPages) in inside routing { }. If outside, only the / and /error log some spans.
Working configuration :
    routing {

        install(StatusPages) {
            exception<AuthorizationException> { cause ->
                call.respond(HttpStatusCode.Forbidden)
            }
        }

        get("/") {
            call.respondText("Hello World!")
        }

        get("/error") {
            call.respond(HttpStatusCode.InternalServerError)
        }

        get("/exception") {
            throw AuthorizationException()
        }
    }

Not working one :
    install(StatusPages) {
        exception<AuthorizationException> { cause ->
            call.respond(HttpStatusCode.Forbidden)
        }
    }

    routing {

        get("/") {
            call.respondText("Hello World!")
        }

        get("/error") {
            call.respond(HttpStatusCode.InternalServerError)
        }

        get("/exception") {
            throw AuthorizationException()
        }
    }

I don't know how the internals of ktor work. Maybe it is expected to work this way...


TikTok web bad audio quality

OS/Platform
Fedora and CentOS
Installed
Flatpak
Version
128.0.6613.113
Have you tested that this is not an upstream issue or an issue with your configuration?

[X] I have tried reproducing this issue in Chrome and it could not be reproduced there
[ ] I have tried reproducing this issue in vanilla Chromium and it could not be reproduced there
[ ] I have tried reproducing this issue in ungoogled-chromium with a new and empty profile using --user-data-dir command line argument and it could not be reproduced there

Description
bad audio quality(like 64kbps on TikTok (all flags off)
How to Reproduce?
play randrom video on tiktok web
Actual behaviour
64kbps like audio
Expected behaviour
normal quality i quess? on chrome everything good
Relevant log output
No response
Additional context
No response

It would help if you showed the screenshot of the "Media" tab from devtools.

this?

this?

This one:


Current Behavior
Parses without failing.
Your Environment

software
version

Babylon
7

node

npm

Operating System

I can reproduce in http://astexplorer.net/ (which is using babylon7-7.0.0-beta.12), but I believe the problem still exists in the lastest version.
I believe the problem lies in src/tokenizer/index.js#L388

Hey @jdmota! We really appreciate you taking the time to report an issue. The collaborators
on this project attempt to help as many people as possible, but we're a limited number of volunteers,
so it's possible this won't be addressed swiftly.
If you need any help, or just have general Babel or JavaScript questions, we have a vibrant Slack
community that typically always has someone willing to help. You can sign-up here
for an invite.

We need to update https://github.com/babel/babylon/blob/master/src/tokenizer/index.js#L394-L399 to:
if (code === 42 && next === 42) {
  //...
}


More tweaks, thrice

Here are more changes of mine to come. There are some new textures this time:

A new bucket_cactus texture (.xcf file is attached in case you need to edit it; rename from this .png file: )
A texture for the revised soft-touch crystal shovel, which uses sticks instead of steel for its handle (.xcf file is attached in case you need to edit it; rename from this .png file:
); the old texture is used for a version of this shovel without soft-touch

In the meantime, there have been some major biome changes in the default game, found at https://forum.minetest.net/viewtopic.php?f=18&t=15939. You might want to consider incorporating some of these new biomes and whatever new items they use into this mod.
I would also like to suggest varying the fuel times for all the tree types introduced by this mod, as the default ones already are, so that gameplay becomes slightly more realistic.

New tweaks added, thanks again...  but, foods, crystal shovel, fishing rod and bonemeal are ethereal specific items to be enabled at all times.

Not all of my changes got committed in, but some is better than nothing. :) Anyways, you forgot about the dirt footstep changes again (a commit of mine fixed this again), and also remove the cools_lava properties from the crystal spike and crystal blocks, so that they are more consistent with the stairs and stairsplus counterparts. In the next few updates, I'll have new textures for the dirt ready.

The dirt footsteps and cool_lava were both added, make sure you are viewing MY github version of ethereal and not your forked version.

I should've went into more detail (this is what I see in your version):

The dry dirt has the grass footsteps sound, but not the other dirts other than the green dirt. Remove the footstep sound from dry dirt and place it in the function where the other dirts are being registered.
Crystal spikes and crystal blocks are not consistent with their counterparts in stairs and stairsplus, in the the stairsplus and stairs versions do not have the cools_lava property. It's up to you to decide whether you want the crystals to be able to cool lava or not.

Gotcha...  Fixed dirt sounds, crystal stairs have lava_cooling and shovel has creative checks...

Good! You're getting there, but I still notice some more things that need to change:

Dried dirt does not share the same description with its stairs and stairsplus counterparts, where they are named "Dry Dirt ..." instead. Since it's just dirt, it does not need the footstep sound.
Change the gain sound for the crystal shovel from 0.35 to 0.4 (this is minor, but I'll leave that up to you, but that's what I saw when reading the code from the default game for when digging leaves), and remove "pos = pos" in the sound function, since that causes the sound to be played at the node instead of directly for the player, and if the player digs that node and is quite far from it, the sound will be less audible.

I'm still waiting for you to fix the following things before I continue making more changes to my fork:

Dried dirt does not share the same description with its stairs and stairsplus counterparts, where they are named "Dry Dirt ..." instead. Since it's just dirt, it does not need the footstep sound.
Change the gain sound for the crystal shovel from 0.35 to 0.4 (this is minor, but I'll leave that up to you, but that's what I saw when reading the code from the default game for when digging leaves), and remove "pos = pos" in the sound function, since that causes the sound to be played at the node instead of directly for the player, and if the player digs that node and is quite far from it, the sound will be less audible.

so wait, you want stairs to be renamed to "dried dirt ..." and the removal of pos=pos in shovel sounds ?

Actually, it's up to you on what you want to do. I'm just trying to bring certain stuff in-live with the latest dev versions of the Minetest Game, but it's up to you whether you want to implement whatever I said.
But do not forget to remove the footstep sound from dry dirt, at the very least.

Okies, dry dirt sound has been fixed as has the stair names...  The pos=pos for crystal shovel will be kept though as sound shoudl come from actual block you dig.

Very good. That's all the changes I will suggest to you for now. If there are more changes of mine to come, they will come in the form of new pull requests.