ENH: Feature Request - Addition of Non-parametric test based on Chebyshev's Inequality

Is your feature request related to a problem? Please describe.
Whenever I need to perform a non-parametric test using Chebyshev's Inequality, I have to write the equation myself. It would be good to have it as a part of scipy.
Describe the solution you'd like.
A function which takes in a distribution(array), a scalar value to be used for the test, the type of test(one-tailed/two-tailed), confidence and returns whether the scalar value is within the limits.
Chebyshev's inequality can also be used to define Confidence intervals which can be used to perform significance/hypothesis tests much like student's t-test. However, it is much more conservative than the student's t-test as Chebyshev's inequality doesn't assume the distribution to be normal.
Given below is the inequality,
       $P(|X-µ| ≥ kσ) ≤ 1/k^2$
here, $P$ is the probability
   $X$ is the random variable
   $µ$ is the mean or expected value of the distribution
   $σ$ is the standard deviation of the distribution
   $k$ is an integer which can be used to find the probability of a value at $k$ standard deviations
A few papers where this inequality is used are mentioned below,

https://doi.org/10.1109/ICSPCC.2013.6663961, Credibility test for blind processing results of sinusoid using Chebyshev's Inequality
https://doi.org/10.1002/for.3980080207, An examination of the accuracy of judgemental confidence intervals in time series forecasting
https://doi.org/10.1111/j.1467-9876.2004.00428.x, Chebyshev's inequality for nonparametric testing with small N and α in microarray research
https://doi.org/10.23919/FRUCT48808.2020.9087459, Stream Data Preprocessing: Outlier Detection Based on the Chebyshev Inequality with Applications

There are many more papers where you can find this inequality being used especially when the distribution is not normal or if one wants to take a conservative approach to define the confidence intervals.
if anyone thinks this will be a useful feature, do let me know I can raise a pull request.
Cheers
Describe alternatives you've considered.
Currently I have to write the code myself so I don't have any other alternative, would be better to have it as a part of scipy,
Additional context (e.g. screenshots, GIFs)
No response

Something like a nonparametric ttest_1samp? Sounds useful to me. Please propose this on the mailing list. All new functions need an email before they can be merged.

Hi @mdhaber ,
Something like Non-parametric ttest_1samp?
yes, it can be used but as suggested in the paper by Beasley et. al. However, The method is conservative and has low power and is suggested to be used in those scenarios where we want to make confident statements despite the conservative approach. Beasly et. al constructs a set of  "Chebby checkers" using the inequality.
Can p-values be computed exactly when the standard deviation is estimated from the data?
yes, having known the mean and standard deviation, the p-values can be computed.
Additionally, it can also be used to define prediction intervals for testing the quality of a forecasting model as shown by Gardner Jr, E. S.
Also, thank you for introducing me to the mailing list, I have proposed it as a new thread. can you kindly advise what is the usual process to decide if a feature is to be added, from issue raise to merge? it would be helpful for me and if I should start working on writing the code.
References:

Beasley, T. M., Page, G. P., Brand, J. P. L., Gadbury, G. L., Mountz, J. D., & Allison, D. B. (2004). Chebyshev’s inequality for nonparametric testing with small N and alpha in microarray research. Journal of the Royal Statistical Society: Series C (Applied Statistics), 53(1), 95–108. doi:10.1111/j.1467-9876.2004.00428.x
Gardner Jr, E. S. (1988). A simple method of computing prediction intervals for time series forecasts. Management Science, 34(4), 541-546. doi: https://doi.org/10.1287/mnsc.34.4.541

yes, having known the mean and standard deviation, the p-values can be computed.

Yes, I understand, but the true standard deviation is not always known; sometimes it is estimated from the sample standard deviation. This leads to a different (fatter-tailed) null distribution - for example, the t-distribution instead of the normal distribution. (This is why a t-test is different from a z-test. It has been "studentized".)
Can p-values (and confidence intervals) be computed exactly in this case?

As for the process, after you post to the mailing list, you might receive basic feedback about the idea. Typically, if there are no arguments against including it, and if a stats maintainer is willing to review it, you can open a PR. It is reviewed and revised to mesh well with the library, and when that is done, the PR is merged.

yes, having known the mean and standard deviation, the p-values can be computed.

Yes, I understand, but the true standard deviation is not always known; sometimes it is estimated from the sample standard deviation. This leads to a different (fatter-tailed) null distribution - for example, the t-distribution instead of the normal distribution. (This is why a t-test is different from a z-test. It has been "studentized".)
Can p-values (and confidence intervals) be computed exactly in this case?

Sorry for the delay. I have been dealing with datasets with high sample sizes and so did not look into this issue in before. I actually had to study a little to answer this.
Found this paper,

Saw, J. G., Yang, M. C. K., & Mo, T. C. (1984). Chebyshev Inequality with Estimated Mean and Variance. The American Statistician, 38(2), 130–132. doi:10.1080/00031305.1984.10483182

To be honest, I still do not completely understand the proof and need a little more study. However, this work is used as a basis in other work to define confidence intervals where the population mean and variance are not known. Given below are a few,

Brisaboa, N. R., & Puglisi, S. J. (Eds.). (2019). String Processing and Information Retrieval. Lecture Notes in Computer Science. doi:10.1007/978-3-030-32686-9
Wang, P., Wang, H., Hart, P., Guo, X., & Mahapatra, K. (2020). Application of Chebyshev’s Inequality in Online Anomaly Detection Driven by Streaming PMU Data. 2020 IEEE Power & Energy Society General Meeting (PESGM). https://doi.org/10.1109/pesgm41954.2020.9281553
Kabán, A. (2011). Non-parametric detection of meaningless distances in high dimensional data. Statistics and Computing, 22(2), 375–385. doi:10.1007/s11222-011-9229-0

They are using this inequality for estimated mean and variance,
       $P(|x-\bar{x}| ≥ kQ) ≤$ $1 \over N$ $[$ $(N-1) \over k^2$ $+ 1]$,
where, $Q^2=$ $(N+1) \over N$ $S^2$
    P is the probability,
    $x$ is the random variable
    $\bar{x}$ is the sample/estimated mean
    $S$ is the sample/estimated standard deviation
    $N$ is the sample size
    $k$ is a value $\ge 1$
On an initial look, the equation seems to be even more conservative considering vanilla Chebyshev's inequality and the bounds get closer to Chebyshev's inequality with the population mean and variance as N tends to infinity.
As for,

Can p-values (and confidence intervals) be computed exactly in this case?
Chebyshev's inequality doesn't assume any distribution and an exact p-value can be calculated only if the probability distribution is known. However, this allows does allow us an upper bound for the

@mdhaber are you a maintainer? I would be grateful if you could do the review for the PR.

I'm a maintainer.

Whenever I need to perform a non-parametric test using Chebyshev's Inequality, I have to write the equation myself.

It sounds like you might already have some draft code? Can I take a look at that before I commit?
Also, can you confirm that it would fit well into the interface of ttest_1samp?
That is, it might be called with arguments a (array of observations), popmean (the population mean under the null hypothesis), and (optionally) popstd), and it would return an object with a statistic, a pvalue, and confidence_interval method that returns a confidence interval at the specified confidence_level?

Hi @mdhaber, Sorry for the delay in response..
I usually use the inequality to check the performance of time-series forecasting models. Basically, I check if the forecast errors or residuals are within some bound.. it looks something like this,
.
$\bar{x}$, known mean of residuals of the sample (test set)
$σ$, known standard deviation of residuals of the sample (test set)
then the upper and lower intervals can be calculated as,
upper bound = $\bar{x}$ + $k * σ$
lower bound = $\bar{x}$ + $k * σ$
the value of $k$ is decided based upon requirements of how accurate we want the forecast to be, say K = 5
$\implies$ $1 \over k^2$ $=0.04$, i.e. the number of predictions where the forecast gives errors greater than K*σ is at most 4% or less than that..
I was hoping to add in a similar functionality which provides one with the intervals given the array/sample as input but using it similar to the ttest_1samp is also possible... however in the output, we need not return a statistic as the inequality directly gives us the confidence intervals directly.. returning a statistic may be useful if only the tables are available but we do not have them readily available on the internet, so not sure if that will be the right approach. Please advice.
As for,

Also, can you confirm that it would fit well into the interface of ttest_1samp?

I am not sure what you meant by this. I am assuming you meant whether it can be coded in a similar fashion. I think it can be done so with slight modifications..
For example,
As suggested, we can have two inputs,

a, array/samples
popmean, known population mean, say popmean = 100

Output:

k_statistic, $k$
pvalue
confidence_interval method that returns confidence interval as specified confidence_level

Null: Mean of a is the same as pop mean
Alternate: Mean of a is different
Then we can compute,
N, sample size, say N = 30
std_dev, standard deviation of sample a, say std_dev = 20
x_bar, mean of sample a, say x_bar = 130
stderr, standard error of mean as stderr = std_dev / math.srqt(N) = 3.6514837167011
The k-statistic for the sample can be calculated as,
k_stat = abs(x_bar - popmean) / stderr = 8.2158386....
As previously mentioned, I am not sure if providing this statistic value will be helpful as one may not be able to know the pvalue (the tables are not usually found on the internet).
We can provide the corresponding pvalue as pvalue = 1 / math.sqrt(k_stat) = 0.42728700639
If we set $1 \over k^2$ $= 0.05$, (this is the statistic value in the unknown distribution for a p-value of 0.05, two-tailed)
$\implies k = 4.472135954....$, we can then use this value to compare if our k_stat value is greater than or smaller than the k at 0.05 confidence level.
As you can see, since the k-statistic calculated is greater than the k-value at 0.05 we can reject the null hypothesis that the mean is the same.
But again, this seems a little too complicated to me and simply returning the confidence interval values at the set level may just be a cleaner and easy-to-understand approach, please suggest what you think will be better in this case..
The confidence intervals for Chebyshev's inequality can be calculated as,
$P(\bar{x} - k * stderr \geq popmean \geq \bar{x} + k * stderr) \leq$ $1 \over k^2$
where $k = 4.472135954....$ pvalue at 0.05
so,
lower_bound = x_bar - k * stderr = 113.67...
upper_bound = x_bar + k * stderr = 146.3299...

Note: In this case, we can use the vanilla Chebyshev's inequality as the calculations are one using standard error of the mean which is not an estimated value. The "Chebyshev's inequality using estimated variance and mean" may be applied if we are simply providing functionality where one can input an array/sample and the output is to give them the values of prediction interval at the set level
I hope this gives a better idea. Do let me know what you think..

Hi @mdhaber, should write the code and raise the pull request?

But again, this seems a little too complicated to me and simply returning the confidence interval values at

This function will be used very similarly to ttest_1samp (except this has even less strict distribution requirements), so yes, the interface should be similar to that of ttest_1samp.  Please name inputs and outputs following the example of ttest_1samp.
Yes, you're welcome to post a rough draft PR so that we can get a sense of how this will look before you spend too much time on it.
Thanks for sending out the email to the mailing list. Did you hear back from anyone?
What do you think @tupui @chrisb53 @tirthasheshpatel is this something we'd like to have? Sounds appropriate to me.

@mdhaber Haven't had any responses on the mailing list yet.. I will be making the draft PR very soon..

Posted a draft PR, tests have not been written yet.
.
I have not created a separate class/object for the output as in ttest_1samp. Using the existing class TtestResult was not feasible as the method for computing the pvalues, statistics and confidence intervals are different and changes would have to be made in the methods. So, I avoided doing that for now. Though I think that would be possible, or a similar class can be defined. I am not sure what would have more utility. Please advice..
.
Also, I was thinking of adding functionality to simply provide the prediction intervals given a sample using "Chebyshev's inequality for sample variance and mean". Should I add it as a part of this PR or separately?

If the method is less restrictive than ttest_1samp while still having a >= power than sure 👍

Thanks for proposing this feature. I would like to understand the use case better (thanks for adding the references, I need more time to check them). I would say in many situations, one can simply perform a t-test. If the normality assumption is a concern, there are already non-parametric alternatives in SciPy (mannwhitneyu, wilcoxon or a permutation test). So what is the rationale for using a crude estimate like Chebyshev's inequality?
In https://doi.org/10.1111/j.1467-9876.2004.00428.x, apparently 1000s of tests have to be run simultaneously, so maybe performance could be an argument?

Sorry for the delay in response.. had some obligations at hand..
@chrisb83 I actually had to study these alternatives a little, please do correct me if I am wrong.. I found that all have some sort of assumption about the data.. eg: Wilcoxon and permutation test(Fisher's one sample Randomization test) both assume the distribution to be symmetric, additionally, permutation tests are more compute intensive though they might not be as big of an issue these days.. Mann Whitney U test doesn't seem to have a one-sample variant..
I agree that Chebyshev's inequality is a rather crude and conservative estimate but in specific scenarios, it is desirable, as it has no assumption about the data distribution, and provides an upper and lower bound for finding a variable at certain standard deviations from the mean..
.

In https://doi.org/10.1111/j.1467-9876.2004.00428.x, apparently 1000s of tests have to be run simultaneously, so maybe performance could be an argument?

In the above paper, The 1000 tests were conducted as a simulation to evaluate the method's performance in the paper. In practice, there won't be a need to perform 1000 tests. Unless, of course, one wants to compare 1000 datasets for their own research.

@chrisb83, after reviewing the reply, would you support the addition of this feature?

@oshin94

Mann Whitney U test doesn't seem to have a one-sample variant..

There's wilcoxon, but again the distribution needs to be symmetric if you want to use it to test location.
@chrisb83 @rlucas7 would you be interested in this?

after reviewing the reply, would you support the addition of this feature?

I am not against adding it


[NEAT-232] test issue

comment

From Linear-GitHub Sync

comment

2

3

4

5

6

from lin, should work with same


Update to -observe draft 13

http://tools.ietf.org/html/draft-ietf-core-observe-13

@sjlongland I guess this issue should now be resolved as well? Or do you see any basic observe features that are still missing from the implementation? If I see it correctly there haven't been any substantial changes after draft 13.

@JKRhb It does appear that way.  I wrote that code based on the RFC-7641 spec as published which was published September 2015, vs that document which was December 2014.

@sjlongland Great, then I'll close this issue as well :)


Include full http response on exception

Summary
Includes full http response in expections
Details and comments
includes this http_response alongside error_type. alternatively we can just show the http_response.

Looks great!


feat: Braze push notification integration

Description
LEARNER-8431

Removed the standalone integration in favor of segment.io
Integrated Push Notification. For Braze Push Notifications,
if the app is in background or foreground we will see a notification
and not a popup module
Simple In-app messages are also enabled without any deep links

Integration: Segment's Braze
Notification Extras: : Extras

Test the following point as part of this PR

[ ] Test the push notification triggered from braze console.
[ ] Test the notifications and action flow against the individual user.
[ ] Test both guest and logged-in users.
[ ] Foreground notification support has been removed for notifications from braze.


[fluidsynth] Update to 2.3.3

[X] Changes comply with the maintainer guide
[X] SHA512s are updated for each updated download
[X] The "supports" clause reflects platforms that may be fixed by this new version
[X] Any fixed CI baseline entries are removed from that file.
[X] Any patches that are no longer applied are deleted from the port's directory.
[X] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[X] Only one version is added to each modified port's versions file.

Feature buildtools,sndfile tested successfully in the following triplet:

x86-windows
x64-windows
x64-windows-static
x64-linux

Feature pulseaudio tested successfully in the x64-linux triplet.

Please mark as ready when conflicts have been resolved


declaring-structures C tutorial

Populate C file with a tutorial and challenge.

I would like to work on this issue. Can this be assigned to me?


feat: additional node pool settings, single auto scaler profile and api server access

Added settings to node pools:
os_disk_type
os_disk_size_gb
orchestrator_version

Added nodepool name, so existing node pools are not renamed to "aks-"

Changed auto scaler profile configuration since Terraform only allows one auto scaler profile

Changed linux-profile-admin variable definition

Changed oms.agent variable definition

added optional api_server_access_profile element

For the following two elements:
Changed linux-profile-admin variable definition
Changed oms.agent variable definition
I am unsure it it was a good idea to change these values to variables, but i would like to discuss on that

@MDBeudekerCN: Please raise separate issues and PR's for each issue/feature.
New  default node pool properties have been split to this PR:
https://github.com/CloudNationHQ/terraform-azure-aks/pull/13
Others will follow.

Seperate PR's are created. Closing this one.


Fix errors found during latest run

fixed print_match_count() in match.py when there are ground_truth_ids

replaced hardcoded database info in match rates calculation with config specs

Looks good, merging. Also, I noticed that these pull requests are coming from BeamRecordLinkage but being pulled to chapinhall. Was that on purpose, or is that some default setting we need to look into changing?

Looks good, merging. Also, I noticed that these pull requests are coming from BeamRecordLinkage but being pulled to chapinhall. Was that on purpose, or is that some default setting we need to look into changing?

I noticed that too, I didn’t do that on purpose and it’s automatically done. We should look into it.


[geotrans] Add macro LITTLE_ENDIAN

Related issue: #6230
There are no features of this port need to test.

/azp run


How much should I set for batch size when using mps for training?

As title, I'm not sure how much should I set for batch size when using mps for training. I have 4GB of vram, and didn't know if macOS have similar things as windows' shared gpu memory so I can use the ram as well?

m1/m2 users may train RVC models.
4G vram: try batch size=1


Fixed TestFeedbackCommentFormat to genereate score results in the PR as a comment

Fixes #727

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 25.221%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2655

Relevant Lines:
10527

💛  - Coveralls


Move + and - buttons below canvas

This was a relatively quick task to improve the UI further after new elements were added. It took me about 40 minutes because I ran into an issue where the buttons needed to be within the canvas element container, but were also not functioning properly because they were getting pushed outside of this container by the canvas size. To fix this, I actually ended up removing an entire section of the original grid layout called the "stretch goal button container" and just increased the size of the canvas container to fill that space. After that, I simply made the canvas slightly smaller to reflect its previous size and placed the zoom buttons along with the save button in that space. I also labeled and styled the buttons.


Exception in trigger:state node

The trigger state node causes a exception when userDir is not specified in config.js
01/03/2018, 09:20:52 node: Door opened
msg : string[48]
"Could not find userDir to use for database store"
This exception is thrown in base-nod.js in function async getDb()  at row 72
let dbLocation = this.RED.settings.get('userDir');
A better way to read this property is
let dbLocation = this.RED.settings.userDir;
See https://github.com/node-red/node-red/issues/1543

Huge thanks for a great package. Its fantastic.

Thanks, and nice find.  I made the change and will be in the next release.


Publish image(s) to Docker Hub via GitHub Actions

@oscarotero, don't you think it's better to publish the images under lumeland/lume instead? In my opinion, it looks more aligned with the current setup that you have. Using oscarotero/lume is not wrong, because eventually you are the main developer/owner, but if you don't have any concerns around it, using lumeland/lume is, in my opinion, the best path forward.
In any case, those things can be changed always at any time, but just sharing that in case you didn't account for it.

Yes, I just created them. Based on this example: https://github.com/docker/login-action#docker-hub you have DOCKERHUB_USERNAME and DOCKERHUB_TOKEN (not password)

And about using lumeland, I've tried to create an organization but this can be done only under a paid plan and I already had a docker id.
I can create a different account for lume but I would need a different email

No problem, let's start with oscarotero/lume, and when everything is super-great, you can change the images to another namespace.


gitlab-runner: 11.6.0 -> 11.7.0

Motivation for this change
Update to latest version.
https://gitlab.com/gitlab-org/gitlab-runner/blob/master/CHANGELOG.md
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option 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"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build gitlab-runner

backported to 18.09 in 30c40858bd7fb2bd59636c94472c9466705f320b.


Flutter Benchmark Can't Run in Android12.0

When I use the master branch code to run the flutter benchmark like this
in the folder: /flutter/dev/devicelab run this cmd ../../bin/cache/dart-sdk/bin/dart bin/run.dart -t color_filter_cache_perf__e2e_summary
and this is the error:
[color_filter_cache_perf__e2e_summary] [STDOUT] stdout:
adb: failed to install app/outputs/flutter-apk/app.apk: Failure [INSTALL_PARSE_FAILED_MANIFEST_MALFORMED: Failed parse during installPackageLI: /data/app/vmdl1602685650.tmp/base.apk (at Binary XML file line #35): com.example.macrobenchmarks.MainActivity: Targeting S+ (version 31 and above) requires that an explicit value for android:exported be defined when intent filters are present]

Thanks for the report. I do not have Android 12 phone to verify this but I'll keep it open and label for further insights from the team.
/cc @blasten
A similar fix was merged : https://github.com/flutter/flutter/pull/98076

@darshankawar aha, I saw it, but another benchmark program also need this param, and you can review my pr?

Closing, as the referenced PR was landed.


terra-ui.com not accessable (or expired)

I was hoping to check out the component lib after reading your blog post. The site does not seem to exist any longer :/
Description of Issue
Navigating to http://terra-ui.com goes to a GoDaddy placeholder page.  https://www.terra-ui.com is unaccessable.
Browser and Version
Chrome (latest)

Hey @ericterpstra
, thanks for reaching out to us.
The Terra documentation site is in an early stage of development.  We are working on getting a design for the site and adding more components. Once that work as been completed, it will be accessible at terra-ui.com
If you are curious about the components we have so far, we have a prototype site deployed at https://terra-ui-staging.herokuapp.com/.  At this point, it is mostly used for testing.
Hope this helps!

We are now using the domain name terra-ui.com


Make Problem (error 2)

Hi,
after installing Opencv3.3, Openpose (work great !), I want to try PyOpenPose (thanks for your work).
I have this output when I launch "make" :
/pyopenpose/build$ make
Install prefix is set to: /usr/local
Generating with build type: Release
-- Boost version: 1.58.0
-- Found the following Boost libraries:
--   system
--   python-py27
--
-- The following OPTIONAL packages have been found:

 * Doxygen

-- The following REQUIRED packages have been found:

 * Threads
 * CUDA
 * OpenCV (required version >= 3)
 * Boost
 * PythonLibs (required version == 2.7)

-- Configuring done
-- Generating done
-- Build files have been written to: /home/hugo/pyopenpose/build
[ 25%] Building CXX object PyOpenPoseLib/CMakeFiles/PyOpenPose.dir/OpenPoseWrapper.cpp.o
/home/hugo/pyopenpose/PyOpenPoseLib/OpenPoseWrapper.cpp: In member function ‘void OpenPoseWrapper::detectFace(const cv::Mat&)’:
/home/hugo/pyopenpose/PyOpenPoseLib/OpenPoseWrapper.cpp:148:86: error: no matching function for call to ‘op::FaceDetector::detectFaces(const op::Array<float>&, float)’
     const auto faceRectsOP = membersPtr->faceDetector.detectFaces(poseKeypoints, 1.0f);
                                                                                ^
In file included from /usr/local/include/openpose/face/headers.hpp:5:0,
                 from /home/hugo/pyopenpose/PyOpenPoseLib/OpenPoseWrapper.cpp:14:
/usr/local/include/openpose/face/faceDetector.hpp:14:39: note: candidate: std::vector<op::Rectangle<float> > op::FaceDetector::detectFaces(const op::Array<float>&) const
         std::vector<Rectangle<float>> detectFaces(const Array<float>& poseKeypoints) const;
                                       ^
/usr/local/include/openpose/face/faceDetector.hpp:14:39: note:   candidate expects 1 argument, 2 provided
/home/hugo/pyopenpose/PyOpenPoseLib/OpenPoseWrapper.cpp: In member function ‘void OpenPoseWrapper::detectHands(const cv::Mat&)’:
/home/hugo/pyopenpose/PyOpenPoseLib/OpenPoseWrapper.cpp:184:86: error: no matching function for call to ‘op::HandDetector::detectHands(const op::Array<float>&, float)’
     const auto handRectsOP = membersPtr->handDetector.detectHands(poseKeypoints, 1.0f);
                                                                                ^
In file included from /usr/local/include/openpose/hand/headers.hpp:5:0,
                 from /home/hugo/pyopenpose/PyOpenPoseLib/OpenPoseWrapper.cpp:15:
/usr/local/include/openpose/hand/handDetector.hpp:17:54: note: candidate: std::vector<std::array<op::Rectangle<float>, 2ul> > op::HandDetector::detectHands(const op::Array<float>&) const
         std::vector<std::array<Rectangle<float>, 2>> detectHands(const Array<float>& poseKeypoints) const;
                                                      ^
/usr/local/include/openpose/hand/handDetector.hpp:17:54: note:   candidate expects 1 argument, 2 provided
PyOpenPoseLib/CMakeFiles/PyOpenPose.dir/build.make:62 : la recette pour la cible « PyOpenPoseLib/CMakeFiles/PyOpenPose.dir/OpenPoseWrapper.cpp.o » a échouée
make[2]: *** [PyOpenPoseLib/CMakeFiles/PyOpenPose.dir/OpenPoseWrapper.cpp.o] Erreur 1
CMakeFiles/Makefile2:117 : la recette pour la cible « PyOpenPoseLib/CMakeFiles/PyOpenPose.dir/all » a échouée
make[1]: *** [PyOpenPoseLib/CMakeFiles/PyOpenPose.dir/all] Erreur 2
Makefile:127 : la recette pour la cible « all » a échouée
make: *** [all] Erreur 2

Openpose build correctly and install without problem ...
Thanks !

Current PyOpenPose version builds with this commit.
Since the openpose api changes fast, PyOpenPose is usually a bit behind in compatibility. I will update the code to work with the latest openpose version as soon as possible, meanwhile you can try it with the above commit.

It's ok, with last commit of OpenPose (03/04/2018), in OpenPoseWrapper.cpp :

change line 148 :

const auto faceRectsOP = membersPtr->faceDetector.detectFaces(poseKeypoints, 1.0f);

to

const auto faceRectsOP = membersPtr->faceDetector.detectFaces(poseKeypoints);

change line 184 :

const auto handRectsOP = membersPtr->handDetector.detectHands(poseKeypoints, 1.0f);

to :

const auto handRectsOP = membersPtr->handDetector.detectHands(poseKeypoints);

Thanks for all !


Specify filename for PLY Export

When using ns-export to export a PLY file from a trained checkpoint, the output filename is currently hard-coded as "splat.ply", and there is no option to specify a different name. In my workflow, it would be helpful to be able to choose the filename directly from the command-line.
Could a command-line flag be added to allow users to specify the output filename for the PLY export?
Thank you for your consideration.

Another issue is that it overwrites an existing splat.ply file and you risk loosing the exported model.

I have a PR for this here: https://github.com/nerfstudio-project/nerfstudio/pull/3467

I have a PR for this here: #3467

Thanks. Does it also avoid overwriting the file name if file name is not specified?

Unfortunately not, we'd also need a separate PR to add support for setting the flag via the GUI.

These should both be pretty quick changes, if anybody has time for a PR feel free to tag me and I'm happy to review/approve!


Fix empty df

Codecov Report
All modified lines are covered by tests :white_check_mark:

Comparison is base (07e2e67) 80.18% compared to head (5bd6374) 80.18%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##           development      #51   +/-   ##
============================================
  Coverage        80.18%   80.18%
============================================
  Files                6        6
  Lines              212      212
============================================
  Hits               170      170
  Misses              42       42

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Issue with entering password for Installation process

Hi,
We are trying to automate Installation of our package and but failing to enter password when prompted for:
There is an exception when trying call driver.get("SecurityAgent)
driver.get("SecurityAgent")
File "/Library/Python/2.7/site-packages/selenium-3.9.0-py2.7.egg/selenium/webdriver/remote/webdriver.py", line 324, in get
self.execute(Command.GET, {'url': url})
File "/Library/Python/2.7/site-packages/selenium-3.9.0-py2.7.egg/selenium/webdriver/remote/webdriver.py", line 312, in execute
self.error_handler.check_response(response)
File "/Library/Python/2.7/site-packages/selenium-3.9.0-py2.7.egg/selenium/webdriver/remote/errorhandler.py", line 242, in check_response
raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.WebDriverException: Message: An unknown server-side error occurred while processing the command.
Without calling  get for  SecurityAgent, When trying to find element using XPath for password field it fails get the element with error:Exception in getting the element by xpath : Message: An element could not be located on the page using the given search parameters.
Can AppiumForMac find elements from a process i.e. the password alert is from a process "SecurityAgent" which is not an app with Dock item and Main menu bar.

What is macOS version?
It works on  10.12.6 with    :xpath:"/AXApplication[@AXTitle='SecurityAgent']/AXWindow/AXTextField[@AXValue='']"

on Mac OS 10.13.x

Looks like the same issue as pyatom/pyatom#156
Security Agent elements are not visible until the title bar is clicked.
I'll get selenium.common.exceptions.NoSuchElementException: Message: An element could not be located on the page using the given search parameters. unless I've manually clicked on the title bar.

Is there any solution for this issue ?
Or use some hotkeys event to instead manual click.


File Picker v7.2 Session and Picker UI Reachability Problem

Hi,
I have a problem with OneDrive File Picker v7.2 about showing files to user.
Problem occurs when another OneDrive tab is opened

When I refresh it, it is showing the files on picker

I think this is a session or cookie issue.
Thanks for help

Hello, the v7.2 Picker is no longer receiving updates, when possible please migrate to the v8 picker, you can ask questions for v8 picker in this issues list.


[Fix #12716] Fix false negatives for Lint/Void

Fixes #12716.
This PR fixes false negatives for Lint/Void when using parenthesized void operators.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Nice, thanks! Turns out I was majorly overcomplicating this when trying to fix myself.


jackson-datatypes tests not being run by maven/CI

No tests in the output: https://circleci.com/gh/hyperledger/quilt/166
I don't really have much experience with mvn but it seems a <packaging>pom</packaging> element seems to throw it: https://github.com/hyperledger/quilt/blob/master/jackson-datatypes/pom.xml#L11
No idea why

Nice catch, and correct diagnosis. I included a fix in PR #160 that fixes this issue.


OpenVPN / Pi-Hole - modify vpn ip addresses/pool

Hello,
I have set up a Pi with Pi-Hole and OpenVPN using pivpn.io. All is working perfectly and the extensive information I've found in forums has been very helpful. I've run into an issue. The default ip scheme in OpenVPN is <IP_ADDRESS> and I'd like to change it to <IP_ADDRESS> as this is a second Pi on my network serving the same purpose. I have modified the openVPN.conf file:

server <IP_ADDRESS> <IP_ADDRESS>
push "dhcp-option DNS <IP_ADDRESS>"

and my remote testing system connected via vpn receives a 10.8.1.x address and connects fine. It is able to resolve local dns (systems I've added to dnsmasq) but not anything outside of my network (Internet). The log file for pihole shows the following:
Dec 30 09:48:21 dnsmasq[1079]: query[A] www.raspberrypi.org from <IP_ADDRESS>
Dec 30 09:48:21 dnsmasq[1079]: forwarded www.raspberrypi.org to <IP_ADDRESS>
Dec 30 09:48:21 dnsmasq[1079]: reply www.raspberrypi.org is <IP_ADDRESS>
Dec 30 09:48:21 dnsmasq[1079]: reply www.raspberrypi.org is <IP_ADDRESS>
but the remote system errors out: This site can't be reached. www.raspberrypi.org took too long to respond. ERR_CONNECTION_TIMED_OUT
Hopefully I'm not missing something simple--I've searched quite a bit and haven't found any similar information/solutions.
Futher info:
Kernel IP routing table
Destination Gateway Genmask Flags Metric Ref Use Iface
<IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> UG 202 0 0 eth0
<IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> U 202 0 0 eth0
<IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> U 0 0 0 tun0
I am unable to ping my gateway, <IP_ADDRESS> which leads me to think that it isn't routing from <IP_ADDRESS> vpn gateway to <IP_ADDRESS> gateway, then out to the Internet.
Thank you for your help.

@henryammons sorry but the issue template is quite clear:
# PiVPN Issue Template
PLEASE READ THIS TEMPLATE CAREFULLY BEFORE OPENING AN ISSUE!
Any Issue opened that doesn't follow this template will be removed.

Closing this issue.. please edit and reopen with the information requested there otherwise we won't have all the details we need to help specially with an issue that specific as yours.


Enhancement request: Electron app?

It would be awesome if we could get an Electron version or something with Photon.css or an Apple-like CSS system for native integration.
I might do one myself based off of this, but I'd at least like to throw in a suggestion.

I'm not against the idea of making an Electron version of the app.
Unfortunately, it'll probably be a while before I get around to it, so if someone else wants to make an electron version of this, that's more than welcome by me.

I have a working PoC for MacOS ready Release 0.9.9 (p.s: it's a PoC )

I will also start working on one, mainly for Linux


"scaling" conformance class

Has there been any discussion of a "scaling" conformance class to allow a client to request a coverage at a resolution other than the coverage's native resolution?
The WCS 2.0 Scaling Extension (OGC 12-039) defines the following parameters on the GetCoverage request: scaleFactor, scaleAxes, scaleSize, and scaleExtent.  CubeWerx's test OGC API implementation at https://test.cubewerx.com/cubewerx/cubeserv/demo/ogcapi/Daraa has implemented these parameters at the /collections/{collectionId}/coverage endpoint, and they work nicely.  So it should be a simple matter of adapting OGC 12-039 lock, stock and barrel as a conformance class of "OGC API - Coverages".

@pomakis In the API EDR work, we had similar facilities, such as 'resolution' and some control over interpolation when retrieving data, but we postponed to a next version, to keep the current version simple. There were issues over how to specify some search terms consistently across different coordinate axes, such as horizontal (in metres) and vertical (in hectopascals) We decided to hide this from the end user client, as the server knows best.
I think 2D maps may not be such a great problem, but when we wrote the Met Ocean Application Profiles fro WCS, we allowed and encouraged interpolation to be forbidden, mainly for safety critical reasons. HTH.

Yes, a scaling conformance class is planned in a future part/extension (see https://github.com/opengeospatial/ogc_api_coverages/blob/master/extensions/overview.adoc)

@Schpidi @pomakis I suggest that this should be in Part 1, but in its own conformance class, as it is essential functionality which all servers which can implement this should implement. Having it in its own conformance class would still allow server that cannot or really do not wish to implement it to still conform to the other coverage conformance classes.
For example, we discussed today how this is key functionality implemented by GDAL drivers, and tools built on GDAL rely heavily on this.
Downsampling / overviews is a very common need and encouraging servers to support it means not wasting bandwidth with full resolution when what clients really want is a downsampled overview. It also means a Tiles API can easily cut out the proper coverage tiles from a Coverages API to render them.

Coverages SWG call: The SWG agreed to include scaling as an additional conformance class to part 1. There was NOTUC.
@jerstlouis agreed to work on a first version including parameters scaleFactor, scaleAxes, and scaleSize.

The scaling conformance class has been merged in, therefore I believe this issue can now be closed.

Merged scaling conformance class in #88


IWYU Output not reproducible via CLI

I have f.ex this in my output
2024-07-10 17:54:42.369 [trace] IWYU Command: include-what-you-use -Xiwyu --max_line_length=80  -Xiwyu --no_fwd_decls -Xiwyu --mapping_file=/mnt/work/psmgmt/scripts/psmgmt.imp [...includes defines and more...] -o .libs/psreservation.o ../../../lib/pscommon/psreservation.c 2>&1
2024-07-10 17:54:42.511 [trace] IWYU output:

(../../../include/psreservation.h has correct #includes/fwd-decls)

../../../lib/pscommon/psreservation.c should add these lines:
#include "stdbool.h"   // for false, bool, true

../../../lib/pscommon/psreservation.c should remove these lines:

The full include-list for ../../../lib/pscommon/psreservation.c:
#include "psreservation.h"
#include <stdlib.h>    // for NULL, free
#include "list.h"      // for __list_add
#include "pscommon.h"  // for PSC_log
#include "stdbool.h"   // for false, bool, true
---

However if I copy that command and execute it myself I see a different output
$ include-what-you-use [...same as in the log...]
(../../../include/psreservation.h has correct #includes/fwd-decls)

(../../../lib/pscommon/psreservation.c has correct #includes/fwd-decls)

I found out that the output is actually from iwyu_tool.py and not from include-what-you-use. Maybe just fix the log output so that this is clea.

When you say "I copy that command and execute it myself" you mean you run the exact command line as vscode posted in the logs?
Assuming that is what you did, then, for prosperity and record keeping:
There are still many differences that can occur.

VSCode can technically run on a different computer, I assume you would know and have checked for it.
You can have different PATH settings leading to different binaries.
A number of other environment variables influence how these tools work together. They can be present in your shell, be created by VSCode, your VSCode start scripts if any or even the build system. Any of those can influence how any of the tools involved work.
You may have multiple iwyu and/or compiler versions installed. The PATH order and other settings (see 3) can influence which tools are used at the end.
There is the iwyu.include-what-you-use setting that controls which tool to use. The trace-level logging prints exactly what will be run in a sub-process, so you can see that. In your case above it showed that the setting was still the default. But it can still be a wrapper or alias that eventually executes something else. And that can still be different from running the same on the commandline for the any of the prior points.

How to sort out the mess:

Create a wrapper script that shows the environment when called, e.g.:

#!/bin/bash
set -eux
env
which include-what-you-use
include-what-you-use "${@}"

In the extension settings point to that wrapper script using iwyu.include-what-you-use.
If that does not help you can try to do similar things with the other involved tools (e.g. compilers).
Try removing mapping files and anything else the tools are known to access.
Clear any caches you may have from building (things like ~/.cache and whatever your setup may use).

In my own setups I often find inconsistencies due to wrapper scripts I actually cannot avoid and so I have given up on running the tool in different ways. I mostly use the command line only to experiment with flags for the tools but that you can also via the extension settings which are reloaded after every change.
Hope this helps.

I updated iwyu 0.21 -> 0.22 and the problems are gone. So this issue is not on the extension side. And I don't want to invest more time in finding out what even happened yesterday. The 0.21 version seems a bit buggy. Anyway thanks for the debug recommendations


revoke code before validating redirect uri

Summary

Revoke authorization code before validating redirect uri
Linked issue(s)

fixes #82
fixes https://github.com/oauthjs/node-oauth2-server/issues/637
Involved parts of the project

Revoking authorization codes
Added tests?

NA
OAuth2 standard

NA
Reproduction

NA

@jorenvandeweyer can you also add this to 4.x so we can publish it as well?


Publish branches as npm tags

Although this does not yet add the ability to publish stable releases to npm from CI, this does publish every newly pushed branch to npm using that branch's name as the tag. This will allow us to test whether a package still works properly after it's published and built (which I'm currently somewhat unsure about, especially for the revamp).
As an example, you can now run npm install solid-panes@95-ci-publish.
Important: I created a new npm user specifically for this. I added the credentials for this account to the Inrupt password vault, but its email address is currently set to my own. Which email address would be more appropriate for this?
Note that this not prevent anyone who currently has publishing rights from publishing the way they did before.
Fixes #95, fixes #110 .

If we merge this, we should create another issue to configure the correct email address, comprising of the following tasks:

[ ] Update the email address of the npm user inrupt_ci.
[ ] Update the email address in the Travis configuration of solid-panes (it's the NPM_EMAIL environment variable.
[ ] List the updated email address in the Inrupt password vault.

@megoth Updating the email addresses only needs to be done in the Travis/npm interface - no code changes are needed. IOW, as far as I'm concerned we can just merge this :)

Email address has been updated!


Cannot return nlink == 0

This is caused by bazillion/fuse#66. Plan:

Uncomment workarounds for this in fuse tests (for example here) so that they are failing.

Fix the broken behavior in jacobsa/bazilfuse. Do it in a branch off of the upstream branch.

Make sure the tests pass.

Re-enable the corresponding assertions in jacobsa/gcsfuse, too.

Oh hooray, looks like this is fixed upstream in bazillion/fuse@58c769aaa613a6337a22c1ef7efde3558ae04eed.

Imported that fix into jacobsa/bazilfuse@1a7cafa5727b1eb5f5f82e6de7fe272d90a8c29f.


Allow content negotiation in .ResponseBody()

Would it be possible to enable content negotiation for .ResponseBody() as well?
It seems only possible now through .Map<>().WithBody()
Thanks

Ah yes, good catch - it should be in there too.


Bump dependencies

Closes several open PRs for this.

For karma-browserstack-launcher and karma-rollup-preprocessor, I think they are pinned because of Browserstack issues.
For terser it was pinned because the new version has changes that make our bundle bigger (linked to terser/terser#1317 upstream). This is probably why you needed to change values in .bundlewatch.config.json.

Ah, gotcha! Okay I'll close this out and merge some individual ones then. Thanks!


Is this project dead

Any living project would have picked this up. Closing. And i will never ever use quartz


Editing loaded shapes with leaflet draw controls

Hi, great piece of code, thanks!
When I load up a KML, I can view them on the map no problem. However the shapes are not editable with the leaflet draw controls.
Any idea how I can get this to work?
Thanks again.

It is hard to help you from here. You could probably try to setup a JSFiddle (or similar) to reproduce your problem in a minimalist fashion.
What makes you think the problem comes from leaflet.filelayer and not leaflet.draw by the way ? :)

It is an awesome piece of code.
natoelgato are you using django-leaflet and using a CreateView or something? From what I understand, django-leaflet instantiates its own new L.Control.Draw() which it populates with a featureGroup called drawnItems. You need to make sure that you add your polygon or whatever from your fileLayer to THAT particular featureGroup. The way I did it was:
`
$(window).on('map:init', function(e) {
// Get the map
map = e.originalEvent.detail.map;
// Get the featureGroup that is used by django-leaflet's drawControl
var featureGroup;
   map.on('map:loadfield', function (e) {
        featureGroup = e.field.drawnItems;
});

// Add the fileLayer
control = L.Control.fileLayerLoad({
      // Add to map after loading (default: true) ?
      addToMap: false,
     .... All other options
  });
  control.addTo(map);

  // When the data is loaded
  control.loader.on('data:loaded', function (e) {
    // Get the geojson layer
    gLayer = e.layer.getLayers()[0];

    // If it's a KML it will have altitude, so iterate through and strip them out
    newLatLngs = []
    oldLatLngs = gLayer.getLatLngs();
    $(oldLatLngs).each(function(index, obj) {
        newLatLng = {lat: obj.lat, lng: obj.lng} // Can also: new L.LatLng(obj.lat, obj.lng);
        newLatLngs.push(newLatLng);
    })

    // Set the new latlngs for the polygon
    gLayer.setLatLngs(newLatLngs);

   // Add the gLayer to the featureGroup
   featureGroup.addLayer(gLayer);
 });

});
`
But now I am stuck because when I submit it thinks the geometry field is null :( although once I use the draw buttons on the map to physical edit the polygon and then submit it, it works. Leplatrem am I approaching this the right way?

Ok, so I worked out how to do it. I hope this helps you natoelgato:

$(document).ready(function() {
    // This function is called when the map initializes (automatically via django-leaflet)
    $(window).on('map:init', function(e) {
        // Get a reference to the map
        map = e.originalEvent.detail.map;

        // Store the featureGroup (drawnItems) which django-leaflet's drawControl uses to submit to the form
        var drawnItems;
        var store;
        map.on('map:loadfield', function (e) {
            drawnItems = e.field.drawnItems;
            store = e.field.store;
        });

        // Add the filelayer to allow the uploading of files
        control = L.Control.fileLayerLoad({
            // See http://leafletjs.com/reference.html#geojson-options
            layerOptions: {
                style: { color:'red' },
                //onEachFeature: function(feature){ drawnItems.addLayer(feature); },
            },

            // Add to map after loading (default: true) ?
            addToMap: false,
        });
        control.addTo(map);

        // When the fileLayer has loaded some data (i.e. someone has loaded a file)
        control.loader.on('data:loaded', function (e) {
            // Get the geojson layer which gets added from the file layers
            gLayer = e.layer.getLayers()[0];

            // If it's a KML it will have altitude, so iterate through and strip them out
            newLatLngs = []
            oldLatLngs = gLayer.getLatLngs();
            $(oldLatLngs).each(function(index, obj) {
                newLatLng = {lat: obj.lat, lng: obj.lng} // Can also: new L.LatLng(obj.lat, obj.lng);
                newLatLngs.push(newLatLng);
            })

            // Set the new latlngs for the polygon
            gLayer.setLatLngs(newLatLngs);

            // If someone has drawn something previously we need to wipe it
            drawnItems.eachLayer(function(l) {
                map.removeLayer(l);
            })
            drawnItems.clearLayers();

            // Add the polygon to the map, not sure why this is required but django-leaflet does this
            map.addLayer(gLayer)

            // Add the polygon to the drawnItems featureGroup
            drawnItems.addLayer(gLayer);

            // Django-leaflet does this so just copy it, it seems to make the fileLayer geojson
            // get saved to the form
            store.save(drawnItems);
        });
    });
})

Hi @natoelgato ! Thank you for the feedback.
I'm cleaning and improving a bit this plugin. Is this issue still relevant ?
It it is, could you please provide a live example via a plunkr/ jsfiddle / codepen / ..., otherwise we can't help you.
If not I will close this issue.


Support SQL-VIEWs

I have a schema, which has SQL views. In the HCL export this views look like this:
table "view_customer" {
  schema  = schema.testing
  comment = "VIEW"
  column "customer_id" {
    null    = false
    type    = int
    default = 0
  }
  column "user_id" {
    null = true
    type = int
  }
}

By creating this schema, the views are created as tables.

Is it possible to ignore all views and not have to list them in --exclude
Is support for views planned / possible?

We would also love this functionality as well.

+1 :+1:
I'm in trouble for this issue. When I remove "view_customer" resource from the Atlas schema, atlas apply tries to drop the view.
Hope there were ignore property for the table resource like as follows:
table "view_customer" {
  schema = schema.testing
  ignore = true     # <===
}

+1 for me as well.
Atlas is really well thought out, and I'd love to use it to create materialised views as well :)

This is already done.
https://atlasgo.io/blog/2023/08/06/atlas-v-0-13


panic: assignment to entry in nil map

Configuring GCS + CloudDNS with nginx-ingress using https://github.com/kubernetes-incubator/external-dns/blob/master/docs/tutorials/gke.md
After first record added successfully, I am getting the below errors and external-dns goes into a crash loop.
If I manually remove in CloudDNS external-DNS records the POD recovers on the next restart and crashes again when re-adding the record.
to recover External-DNS have to clear ingress rules and remove DNS records.
external DNS runs with the following arguments:
- --source=service
- --source=ingress
- --domain-filter=dev.getantler.io # will make ExternalDNS see only the hosted zones matching provided domain, omit to process all available hosted zones
- --provider=google
- --google-project=
- --registry=txt
- --txt-owner-id=dev
- --txt-prefix=dev-
time="2018-10-03T23:42:09Z" level=info msg="All records are already up to date"
time="2018-10-03T23:43:08Z" level=info msg="Change zone: dev-main-zone"
time="2018-10-03T23:43:08Z" level=info msg="Add records: service1.dev.mydomain. CNAME [service.dev.mydomain.] 300"
time="2018-10-03T23:43:08Z" level=info msg="Add records: dev-service1.dev.mydomain. TXT ["heritage=external-dns,external-dns/owner=dev,external-dns/resource=ingress/default/nginx"] 300"
panic: assignment to entry in nil map
goroutine 1 [running]:
github.com/kubernetes-incubator/external-dns/registry.(*TXTRegistry).Records(0xc0000f8070, 0xc000000300, 0xc000951828, 0xc00007c960, 0xc000487ec0, 0xc000951828)
/go/src/github.com/kubernetes-incubator/external-dns/registry/txt.go:103 +0x49a
github.com/kubernetes-incubator/external-dns/controller.(*Controller).RunOnce(0xc00006e140, 0xc0009517f8, 0x2)
/go/src/github.com/kubernetes-incubator/external-dns/controller/controller.go:67 +0x49
github.com/kubernetes-incubator/external-dns/controller.(*Controller).Run(0xc00006e140, 0xc00007c960)
/go/src/github.com/kubernetes-incubator/external-dns/controller/controller.go:100 +0x84
main.main()
/go/src/github.com/kubernetes-incubator/external-dns/main.go:238 +0xb0e

the issues is not there on v0.5.0

On which version do you see it?

I experienced this as well and just had a quick look. It seems it was introduced in v0.5.7-1-g6285b2c, the current teapot/external-dns:latest.

yes, it is latest. sorry I thought I mentioned it

Hello!
I ran into the same issue. Could you please suggest some workaround? Maybe the tag of stable image?

It seems like there's a bug in the current version, this is fixed in PR https://github.com/kubernetes-incubator/external-dns/pull/739. We will rollout a new version this week.

@njuettner it appears this bug is still present in v0.5.8. Any idea what this might be?
panic: assignment to entry in nil map

goroutine 1 [running]:
github.com/kubernetes-incubator/external-dns/registry.(*TXTRegistry).Records(0xc0000fc0e0, 0x1, 0xc00012d500, 0x0, 0x7fb87056a000, 0xc0004b5768)
	/go/src/github.com/kubernetes-incubator/external-dns/registry/txt.go:103 +0x49a
github.com/kubernetes-incubator/external-dns/controller.(*Controller).RunOnce(0xc000472e40, 0x1d62570, 0xc0004799f0)
	/go/src/github.com/kubernetes-incubator/external-dns/controller/controller.go:67 +0x49
github.com/kubernetes-incubator/external-dns/controller.(*Controller).Run(0xc000472e40, 0xc00007e960)
	/go/src/github.com/kubernetes-incubator/external-dns/controller/controller.go:100 +0x84
main.main()
	/go/src/github.com/kubernetes-incubator/external-dns/main.go:240 +0xb0e

Also got it when upgrading to 0.5.8, 0.5.7 works

To echo the above, 0.5.7 appears to work fine, 0.5.8 sees the issue. Similarly using Nginx Ingress + Google Cloud DNS

@akozichev It seems Google is the only provider which doesn't initiate labels correctly. I've created a PR which fix it.


rkt trust: strip common url schemes from --prefix argument

Fixes #472

LGTM

Does this put us in the slightly awkward situation of ignoring the scheme in the URL? E.g. if someone puts http://example.com/bar but then discovery will try https.

Isn't that what we wanted?  It always uses https, unless you say otherwise explicitly with --inscure-allow-http.

From a security perspective yes, just seems like a slightly awkward UX.
Maybe we could just document it in the help.
On Wed, Feb 11, 2015 at 3:04 PM, Vito Caputo<EMAIL_ADDRESS>wrote:

Isn't that what we wanted? It always uses https, unless you say otherwise
explicitly with --inscure-allow-http.
—
Reply to this email directly or view it on GitHub
https://github.com/coreos/rocket/pull/514#issuecomment-73984477.

slight tangent: it'd be great if we could standardize on using --insecure on all commands that have security that can be disabled or downgraded. I realize that some commands might need multiple types of security flags, but if we could stick to one when possible that would be awesome.

@robszumski yeah, I noted that very point here

We should really error out on this instead.


add-trigger-name-to-status

Automated changes by create-pull-request GitHub action

Auto-closing pull request


CMake and QTCreator integration

Warning: I'm not familiar with cmake, my experience had been with qmake so far. My question will probably be asked again and again, as QT Creator is a popular IDE for Qt/QML applications.
I'm trying to use Qaterial with QTCreator. In this Github repo, I provide the code I use to reproduce this problem, with 1 commit per change I perform. You can very easily try yourself.
So, I start from scratch and create a Test project that use cmake. This commit just the initial output of Qt Creator.
Then, I add this commit that switch C++ and QML to Qaterial and add dependency build.
But, that don't works, build fail with
Test/main.cpp:1:10: fatal error: 'Qaterial/Qaterial.hpp' file not found
#include <Qaterial/Qaterial.hpp>
         ^~~~~~~~~~~~~~~~~~~~~~~
1 error generated.

Sure, the reason is those files are in  build-Test-Desktop_Qt_5_15_1_clang_64bit-Debug/_deps/qaterial-src/.
From QtCreator cmake integration doc:

Adding External Libraries to CMake Projects
Through external libraries, Qt Creator can support code completion and syntax highlighting as if they were part of the current project or the Qt library.
Qt Creator detects the external libraries using the find_package() macro. Some libraries come with the CMake installation. You can find those in the Modules directory of your CMake installation. For more information, see cmake-packages(7).
Syntax completion and highlighting work once your project successfully builds and links against the external library.

So, I tried find_package(Qaterial) and  output of cmake is:
Make Warning at CMakeLists.txt:38 (find_package):
  By not providing "FindQaterial.cmake" in CMAKE_MODULE_PATH this project has
  asked CMake to find a package configuration file provided by "Qaterial",
  but CMake did not find one.

  Could not find a package configuration file provided by "Qaterial" with any
  of the following names:

    QaterialConfig.cmake
    qaterial-config.cmake

  Add the installation prefix of "Qaterial" to CMAKE_PREFIX_PATH or set
  "Qaterial_DIR" to a directory containing one of the above files.  If
  "Qaterial" provides a separate development package or SDK, be sure it has
  been installed.

In Qaterial repository, there is no *config*.cmake nor Find*.cmake.
So, how can I use Qaterial with Qt Creator?

I think I found how

Fixed in https://github.com/ochagnon/qaterial-qtcreator-bug/commit/40dc0f5ff2d37dcae7580510554ce7445c57ba55

So, the binary start and can load QML, but the UI of Qt creator still complains that QML module not found (Qaterial)

Ok, so for now all your problems are not related to qt creator but to cmake (which are the one everyone is struggling with starting with cmake)
I saw your code and I Will investigate. It is strange since you added the addImportPath

@OlivierLDff thanks, this doc give a few possible fixes for the missing qml in creator itself, I'm also investigating

You should use https://github.com/OlivierLDff/QaterialGallery as a reference

I fixed QT Creator in this commit

This kind of ugly fix, because it is hard coded path. When deploying your app it will be a nightmare.
Anyway, qaterial's qml files should be embedded in qrc/Qaterial. And there should be a qrc/Qaterial/qmldir in there.

This kind of fix is ugly, because it is hard coded path. When deploying your app it will be a nightmare.

that don't solve everything, there is additional issues
module: "Qaterial.Impl.Icons" is not installed

Anyway, qaterial's qml files should be embedded in qrc/Qaterial. And there should be a qrc/Qaterial/qmldir in there.

how?

Anyway, qaterial's qml files should be embedded in qrc/Qaterial. And there should be a qrc/Qaterial/qmldir in there.

I'm looking at gallery as a reference and I don't see an obvious command that embed Qaterial into the gallery

Yes because you are missing a file in qrc:/Qaterial/Impl/Icons which is only in qrc (because generated)

I opened your example on windows
git clone https://github.com/ochagnon/qaterial-qtcreator-bug
cd qaterial-qtcreator-bug
mkdir build
cd build/
cmake ..
cmake --build . -j

And the app launched.
Did you clean your cmake build folder?

Yes because you are missing a file in qrc:/Qaterial/Impl/Icons which is only in qrc (because generated)

But, how can I add this resource with mine?
I don't see anything in this directory
There is no additional .qrc loaded in Main.cpp
I don't get any cmake build steps that combine project qml with the Qaterial dependencies :(

You shouldn't have anything to do, that is why it is strange.

For me it worked without any problem.
You should try to rm -rf * your build folder and do cmake .. again.

And the app launched.
Did you clean your cmake build folder?

yes, the application build and run, but the problem is when I try to open QtCreator designer (very important for development).
First, I had to add import QtQuick 2.12 to my Main.qml and:

This is where the missing icons error show up. This is probably because I don't use any icons at this point in my Main.qml. The very minimalistic things I tried to do don't trigger that error.
I tried with Gallery demo app, and I have an other problem.

The problem is only in QtCreator in "Design" mode

I never used design, I know qaterial should be compiled as a plugin for it to work and there is special requirements.
Qaterial isn't only qml files.
Sorry I don't have time to support that on my free time since I'm not using it but feel free to create a pull request that address the issue

The file you are missing are located in your build folder res/Icons/Generated & res/Fonts/Generated


file-manager / fit new pdf viewer

[x] I have tested my component
[x] I have updated the documentation or there is no documentation needed

:tada: This PR is included in version 1.4.0-rc.71 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: