Plotting the prediction intervals

Reference Issues/PRs

fixes the issue #541
What does this implement/fix? Explain your changes.

Added the pred_int as an option to the plot_series to plot the confidence interval of the predictions.
Does your contribution introduce a new dependency? If yes, which one?
No

What should a reviewer concentrate their feedback on?
Focus on the plot_series function in the plotting.py file.

The changes on the init of the EnsembleForecaster and it predict method.
Any other comments?

PR checklist

For all contributions

[ ] I've added myself to the list of contributors.
[ ] Optionally, I've updated sktime's CODEOWNERS to receive notifications about future changes to these files.
[ ] I've added unit tests and made sure they pass locally.

For new estimators

[ ] I've added the estimator to the online documentation.
[ ] I've updated the existing example notebooks or provided a new one to showcase how my estimator works.

Hi @Ifeanyi30, I think the idea is rather to allow users to pass prediction intervals obtained from a forecaster, see for example the PR https://github.com/alan-turing-institute/sktime/pull/545

@Ifeanyi30 take a look at the tutorial notebook here: https://www.sktime.org/en/latest/examples/01_forecasting.html#Prediction-intervals - the prediction intervals already give upper and lower limits, so we only have to fill in the space between

Thanks, the information was useful and I have made changes to the PR
On Wed, Apr 7, 2021 at 9:16 AM Markus Löning @.***>
wrote:

@Ifeanyi30 https://github.com/Ifeanyi30 take a look at the tutorial
notebook here:
https://www.sktime.org/en/latest/examples/01_forecasting.html#Prediction-intervals

the prediction intervals already give upper and lower limits, so we only
have to fill in the space between

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alan-turing-institute/sktime/pull/776#issuecomment-814707203,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL45AANVF66HZ5GMWNYBUPDTHQIG3ANCNFSM42KRF2AQ
.

Hi @mloning, I have made the changes to the PR. Please review the changes.
On Wed, Apr 7, 2021, 1:35 PM ifeanyi eze @.***> wrote:

Thanks, the information was useful and I have made changes to the PR
On Wed, Apr 7, 2021 at 9:16 AM Markus Löning @.***>
wrote:

@Ifeanyi30 https://github.com/Ifeanyi30 take a look at the tutorial
notebook here:
https://www.sktime.org/en/latest/examples/01_forecasting.html#Prediction-intervals

the prediction intervals already give upper and lower limits, so we only
have to fill in the space between

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alan-turing-institute/sktime/pull/776#issuecomment-814707203,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL45AANVF66HZ5GMWNYBUPDTHQIG3ANCNFSM42KRF2AQ
.

@Ifeanyi30 would be nice if you could add this to the example notebook? :) So basically replacing the plot example there with this new functionality.

Martin Walter<EMAIL_ADDRESS>Mercedes-Benz AG on behalf of Daimler TSS GmbH.
https://github.com/Daimler/daimler-foss/blob/master/LEGAL_IMPRINT.md

Hi @mloning, I have made changes to the error message and would like you to
review the code again.
In the meeting, you emphasized on the importance of making a sensible
contribution. Well, including this PR, I have made two contributions to
sktime, is it ok to record them for my final application?
On Thu, Apr 8, 2021, 11:26 AM Markus Löning @.***>
wrote:

@.**** commented on this pull request.
In sktime/utils/plotting.py
https://github.com/alan-turing-institute/sktime/pull/776#discussion_r609548703
:

if pred_int is not None:

   # check same conditions as for earlier indices

   if not type(index) is type(pred_int.index):  # noqa

       raise TypeError("Found series with different index types.")

   elif all([x in index for x in pred_int.index]):

       ax.fill_between(

           ax.get_lines()[-1].get_xdata(),

           pred_int.lower,

           pred_int.upper,

           alpha=0.3,

           color=ax.get_lines()[-1].get_c(),

           label="prediction intervals",

       )

   else:

       raise ValueError(

           "pred_int has index values for which no other index values supplied"

Could we also improve the error message? I guess we want to say that
pred_int index does not match the index of any of the passed series.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alan-turing-institute/sktime/pull/776#pullrequestreview-631228759,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL45AAL33KBCIJ72PMZVXW3THWAEHANCNFSM42KRF2AQ
.

Hi Martin, so how do I do that?
On Thu, Apr 8, 2021, 10:09 AM Martin Walter @.***>
wrote:

@Ifeanyi30 https://github.com/Ifeanyi30 would be nice if you could add
this to the example notebook? :) So basically replacing the plot example
there with this new functionality.
Martin Walter @.***, Mercedes-Benz AG on
behalf of Daimler TSS GmbH.
https://github.com/Daimler/daimler-foss/blob/master/LEGAL_IMPRINT.md
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alan-turing-institute/sktime/pull/776#issuecomment-815593934,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL45AAMTGODOWANO6MKHM7DTHVXEPANCNFSM42KRF2AQ
.

@Ifeanyi30 regarding your application, your PRs are fine - thanks!

@Ifeanyi30 quick warning: before changing the notebook, please wait for PR https://github.com/alan-turing-institute/sktime/pull/793 to be merged, otherwise we will run into merge conflicts.

Ok
On Fri, Apr 9, 2021 at 11:46 AM Markus Löning @.***>
wrote:

@Ifeanyi30 https://github.com/Ifeanyi30 quick warning: before changing
the notebook, please wait for PR #793
https://github.com/alan-turing-institute/sktime/pull/793 to be merged,
otherwise we will run into merge conflicts.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alan-turing-institute/sktime/pull/776#issuecomment-816595727,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL45AAIVIH4R6FYHYWY3OMTTH3LHZANCNFSM42KRF2AQ
.

@Ifeanyi30 #793 is now merged!

Ok, thanks
On Fri, Apr 9, 2021, 6:00 PM Markus Löning @.***> wrote:

@Ifeanyi30 https://github.com/Ifeanyi30 #793
https://github.com/alan-turing-institute/sktime/pull/793 is now merged!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alan-turing-institute/sktime/pull/776#issuecomment-816820513,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL45AAOVRO2EIJAPOTE2LJ3TH4XBJANCNFSM42KRF2AQ
.

Hi guys, I used a windows operating system, when I commit and push the
examples/01_forevasting.ipynb file, Git automatically converts the Unix LF
format to windows' CRLF. I have tried configuring the git core.autocrlf to
false but still doesn't work and it tempers with the code quality.
What should I do?
On Fri, Apr 9, 2021, 6:15 PM ifeanyi eze @.***> wrote:

Ok, thanks
On Fri, Apr 9, 2021, 6:00 PM Markus Löning @.***>
wrote:

@Ifeanyi30 https://github.com/Ifeanyi30 #793
https://github.com/alan-turing-institute/sktime/pull/793 is now merged!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alan-turing-institute/sktime/pull/776#issuecomment-816820513,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL45AAOVRO2EIJAPOTE2LJ3TH4XBJANCNFSM42KRF2AQ
.

@aiwalter can you help with that? I'm not on Windows. git core.autocrlf seems like the right place to change that.

@Ifeanyi30 I think you can ignore this warning, I also had it sometimes. Or what is the issue?

When I push the changes the code quality test fails because of the new line
in the Unix format that is not present.
On Sat, Apr 10, 2021, 8:16 PM Martin Walter @.***>
wrote:

@Ifeanyi30 https://github.com/Ifeanyi30 I think you can ignore this
warning, I also had it sometimes. Or what is the issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alan-turing-institute/sktime/pull/776#issuecomment-817189764,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL45AAL5CY5VOJOZHOVX3ETTICPYPANCNFSM42KRF2AQ
.

@Ifeanyi30 did you install and use pre-commit?

Yes, I did.
On Sat, Apr 10, 2021, 8:53 PM Martin Walter @.***>
wrote:

@Ifeanyi30 https://github.com/Ifeanyi30 did you install and use
pre-commit?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alan-turing-institute/sktime/pull/776#issuecomment-817194051,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL45AAKNOTLVOG6OJ7JNZFDTICUBXANCNFSM42KRF2AQ
.

I suggest we merge this PR, then I keep trying to update the
examples/forecasting.ipynb file.
On Sat, Apr 10, 2021 at 9:40 PM ifeanyi eze @.***> wrote:

Yes, I did.
On Sat, Apr 10, 2021, 8:53 PM Martin Walter @.***>
wrote:

@Ifeanyi30 https://github.com/Ifeanyi30 did you install and use
pre-commit?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alan-turing-institute/sktime/pull/776#issuecomment-817194051,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL45AAKNOTLVOG6OJ7JNZFDTICUBXANCNFSM42KRF2AQ
.

I'd prefer to update the notebook first as it also is a test for the new functionality.
There seem to be a lot of discussions on the topic, see e.g. here. Could you try to solve it again and perhaps post the (error) message that you receive?

The error the message I got is attached below. It is clearly a platform
issue. I used 'git config --global core.autocrlf false' but it did not work.
On Sun, Apr 11, 2021 at 10:18 AM Markus Löning @.***>
wrote:

@Ifeanyi30 https://github.com/Ifeanyi30 I'd prefer to update the
notebook first as it also is a test for the new functionality.
There seems to be a lot of discussions on the topic, see e.g. here
https://stackoverflow.com/questions/15223290/fixing-mac-windows-new-lines-with-git-pre-commit-hook.
Could you try to solve it again and perhaps post the (error) message that
you receive?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alan-turing-institute/sktime/pull/776#issuecomment-817276298,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL45AAN43NXFDLZIHOIFICLTIFSOTANCNFSM42KRF2AQ
.

Maybe If I try resetting the sktime locally repo and git file format on my
PC then it might just work.
On Sun, Apr 11, 2021 at 6:27 PM ifeanyi eze @.***> wrote:

The error the message I got is attached below. It is clearly a platform
issue. I used 'git config --global core.autocrlf false' but it did not work.
On Sun, Apr 11, 2021 at 10:18 AM Markus Löning @.***>
wrote:

@Ifeanyi30 https://github.com/Ifeanyi30 I'd prefer to update the
notebook first as it also is a test for the new functionality.
There seems to be a lot of discussions on the topic, see e.g. here
https://stackoverflow.com/questions/15223290/fixing-mac-windows-new-lines-with-git-pre-commit-hook.
Could you try to solve it again and perhaps post the (error) message that
you receive?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alan-turing-institute/sktime/pull/776#issuecomment-817276298,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL45AAN43NXFDLZIHOIFICLTIFSOTANCNFSM42KRF2AQ
.

@Ifeanyi30 yes please see if you can solve it, otherwise post the error message so that we can help, or push the code to see why the CI checks are failing.

This is the error code
On Mon, Apr 12, 2021 at 1:50 PM Markus Löning @.***>
wrote:

@Ifeanyi30 https://github.com/Ifeanyi30 yes please see if you can solve
it, otherwise post the error message so that we can help, or push the code
to see why the CI checks are failing.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alan-turing-institute/sktime/pull/776#issuecomment-817783376,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL45AAMFGDI42W7SR73YW5LTILUAFANCNFSM42KRF2AQ
.

@Ifeanyi30 let me know if I can help with anything!

Thanks!
On Mon, Apr 12, 2021, 4:14 PM Markus Löning @.***>
wrote:

@Ifeanyi30 https://github.com/Ifeanyi30 let me know if I can help with
anything!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alan-turing-institute/sktime/pull/776#issuecomment-817895584,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL45AAJ7UNJYUVPVXTBKGPDTIME5DANCNFSM42KRF2AQ
.

This is the error message
On Mon, Apr 12, 2021 at 4:34 PM ifeanyi eze @.***> wrote:

Thanks!
On Mon, Apr 12, 2021, 4:14 PM Markus Löning @.***>
wrote:

@Ifeanyi30 https://github.com/Ifeanyi30 let me know if I can help with
anything!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alan-turing-institute/sktime/pull/776#issuecomment-817895584,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL45AAJ7UNJYUVPVXTBKGPDTIME5DANCNFSM42KRF2AQ
.

@Ifeanyi30 sorry I can't see the message

Closed in favour of #802

Yeah
On Thu, Apr 15, 2021, 2:43 PM Markus Löning @.***>
wrote:

Closed in favour of #802
https://github.com/alan-turing-institute/sktime/pull/802
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alan-turing-institute/sktime/pull/776#issuecomment-820432436,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL45AAPT4ECKAP5GFMRDQIDTI3UPJANCNFSM42KRF2AQ
.


Integrate neopolitan

Add code for event handling
Related #20

What should the web application be able to do?

Initiate neopolitan display
Exit neopolitan display
Send neopolitan display commands while it is running (related to 3)


Cannot change country code

Hi. I was working with version 0.x.x of react-native-phone-input. Everything was fine. Then I migrated to version 1.x.x. Now the selection of another country is not valid. is it related to the package update or to a bad implementation from my part. thank you.
here is attached the github link of the project I am developing with your package.
https://github.com/HamaBarhamou/TrackerClient
Thanks to you.
Translated with www.DeepL.com/Translator (free version)

Hi. I was working with version 0.x.x of react-native-phone-input. Everything was fine. Then I migrated to version 1.x.x. Now the selection of another country is not valid. is it related to the package update or to a bad implementation from my part. thank you.
here is attached the github link of the project I am developing with your package.
https://github.com/HamaBarhamou/TrackerClient
Thanks to you.
Translated with www.DeepL.com/Translator (free version)


Parsing fails with libxml2 2.13.0

Since the update of libxml 2.13.0, the parsing seems to be broken.
Bug reported to libxml2: https://gitlab.gnome.org/GNOME/libxml2/-/issues/738

Fixed in libxml2 2.13.1


docs: Fix typo

Use backticks correctly
Summary
There's two typos in the paragraph where a single quote is used instead of a backtick.

This is some old-school quoting style but apparently it's breaking the markdown. So thank you for the fix!


Gestures are intercepted by the drawer

If, say, a UISlider is added as a subview of the DrawerView, interaction with it will become mostly not working (it may start sliding a bit, but not much) and unpredictable. This is because the touch events are intercepted by the DrawerView's UIGestureHandlers and not properly passed down.

Sorry for a delay, the week was busy.
Thank you for pointing this out, and for making all those pull requests. I will review them and will write my thoughts (so far it seems like good additions) 😉


[wish] new alpha release

Make a new alpha release 18.07 and update the Homebrew/Linuxbrew formula.

Resolved in https://github.com/MediaArea/RAWcooked/pull/126 and https://github.com/MediaArea/homebrew-mediaarea/pull/32. Thank you!


Bump the eslint group with 2 updates

Patch for

#413

coverage: 90.667%. remained the same
when pulling 1940dfdcb8ee16992a936d7d0635cf26202ffd4e on eslint-update
into cf2e7496543d5f2a775108efad578babf7ab35b6 on mainline.


Uniform Standards Request: Experimental Repository

Hello!
We believe this repository is Experimental and therefore needs the following files updated:

[x] The Experimental badge
[x] The Experimental Statement in the README.md
[x] Getting Started Tutorial
[x] README.md
[x] LICENSE
[x] CI/CD
[x] OWNERS.md
[x] Developer Certificate of Origin
[x] At least one maintainer

If you feel the repository should be maintained or end of life or that you'll need assistance to create these files, please let us know by filing an issue with https://github.com/packethost/standards.
Equinix Metal maintains a number of public repositories that help customers to run various workloads on Equinix Metal. These repositories are in various states of completeness and quality, and being public, developers often find them and start using them. This creates problems:

Developers using low-quality repositories may infer that Equinix Metal generally provides a low quality experience.
Many of our repositories are put online with no formal communication with, or training for, customer success. This leads to a below average support experience when things do go wrong.
We spend a huge amount of time supporting users through various channels when with better upfront planning, documentation and testing much of this support work could be eliminated.

To that end, we propose three tiers of repositories: Private, Experimental, and Maintained.
As a resource and example of a maintained repository, we've created https://github.com/packethost/standards. This is also where you can file any requests for assistance or modification of scope.
The Goal
Our repositories should be the example from which adjacent, competing, projects look for inspiration.
Each repository should not look entirely different from other repositories in the ecosystem, having a different layout, a different testing model, or a different logging model, for example, without reason or recommendation from the subject matter experts from the community.
We should share our improvements with each ecosystem while seeking and respecting the feedback of these communities.
Whether or not strict guidelines have been provided for the project type, our repositories should ensure that the same components are offered across the board. How these components are provided may vary, based on the conventions of the project type. GitHub provides general guidance on this which they have integrated into their user experience.

@thebsdbox this repository is at experimental standard.


Avoid modification of modules.edn after kit/sync-modules

Fixes #54

Sorry, been a bit busy and just have a bit of time to get back to this now. The fix looks good to me.


Support Swift Packages

Does it support SP

Had the same issue and made it work:
https://github.com/ricobeck/Bagel
Had to fork Bagel and CocoaAsyncSocket to correct the location of the header files. As a bonus it now works on macOS.
Is this repo still maintained? I'd happy to provide a PR.

@ricobeck Yes it's still maintained :) It would be awesome if you open a PR.

Any update on this?
I currently added ricobeck his repo as a dependency.
But I would rather at the original one.

+1
@ricobeck can we have your PR please? 😃

+1


Unable to import imagegenerator 0.10.5

I'm currently using version 0.10.5, which, according to the release notes, has added the Image Generator API. However, when I try to use the library, I can't import com.google.mediapipe.tasks.vision.imagegenerator as indicated in the official documentation here. Upon inspecting the downloaded MediaPipe library, I noticed that com.google.mediapipe.tasks.vision doesn't contain imagegenerator. Interestingly, com.google.mediapipe.tasks.vision.imagegenerator.proto does have imagegenerator in core, suggesting it should also be present in com.google.mediapipe.tasks.vision, but it's missing.

The image generator is published under a different artifact. Please install com.google.mediapipe:tasks-vision-image-generator.


Automated cherry pick of #1910: fix: 避免镜像同步到私有云自定义镜像里面

Cherry pick of #1910 on release/2.11.
#1910: fix: 避免镜像同步到私有云自定义镜像里面

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11@e822db3). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             release/2.11   #1911   +/-   ##
==============================================
  Coverage                ?   7.52%
==============================================
  Files                   ?     418
  Lines                   ?   64729
  Branches                ?       0
==============================================
  Hits                    ?    4873
  Misses                  ?   59444
  Partials                ?     412

Flag
Coverage Δ

#aFlag
7.52% <ø> (?)

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 e822db3...c02bd96. Read the comment docs.

/lgtm
/approve


Include ENSO_CLOUD_PROJECT_SESSION_ID in DB audit logs

Once Cloud deploys the added ENSO_CLOUD_PROJECT_SESSION_ID environment variable, it is suggested that we include it in the audit logs.

On hold until the change is deployed in Cloud.

Deployed. You can improve on your side

Done in #10303


Django 3.0 Support

Thanks for this package and your work maintaining it.

OS version and name: Ubuntu 18.04
python version: 3.8
django version: 3.0
django-robots version: 3.1.0

I am attempting to upgrade my django project to python 3.8 and django 3.0, and when I attempt to run my test suite, I receive this error from robots/models.py on line 3.
ImportError: cannot import name 'python_2_unicode_compatible' from 'django.utils.encoding'

I believe that Django 3.0 removes this due to the drop in Python 2 support.
https://docs.djangoproject.com/en/3.0/releases/3.0/#removed-private-python-2-compatibility-apis
To achieve Django 3.0 support, I believe this will have to be removed from django-robots.

ModuleNotFoundError: No module named 'django.utils.six' too


Add unsupported method warnings

Add warning message to alert user that some action could not be effective.

implemented for set_ac_stop
commented for other method

This PR is related to #159.

Thanks @PysX

Thanks @PysX


Feat: Showing users ratings in Testimonials and also using best coding practices successfully issue 566

Pull Request Resolves [ #566 ]
Title: Showing users ratings in Testimonials and also using best coding practices successfully.
Description

In frontend, I created a separate interface in types.tsx for managing all the interface in single place.
The code is reduced in the Testimonials section also using rating I am fetching the user ratings.
I modified the Testimonial design.
The design is responsive.

Related Issues
Fixes #566
Changes Made
New feature: Showing users ratings in Testimonials and also using best coding practices.
Screenshots

I certify that I have carried out the relevant code of conduct and provided the requisite screenshot for validation by submitting this pull request.
Thank You for this contribution.

@akbatra567 give level 2 the issue was of level 2 not level 1!

@VaibhavArora314 please give level 2 and you only add the labels and level at the spot the PR is generated else it create confusion and waste the time also.
I hope you consider my request.
Thank You,
Meet Dodiya

@VaibhavArora314 please give level 2 and you only add the labels and level at the spot the PR is generated else it create confusion and waste the time also.
I hope you consider my request.
Thank You,
Meet Dodiya

There must have been some confusion from Akshish's side. Sorry for any inconveniences.


Extract calibration points from video automatically

I have to press "Add Grid" repeatedly on frames where my CharuCo grid is in the image. Can't this be done automatically?

I see your point. One possible though more computationally intensive approach is to use RANSAC. Still, everything is going to be limited by the quality of the input data, by which I mean the breadth of board positions relative to the camera and how much the board shows foreshortening.
Trying to keep things more tractable and simple, the user could autopopulate a sample of grids based on a target grid count and a threshold for how many corners have to be tracked (say 40 boards need to be selected but only if 80% of the board is in view). These boards could be selected evenly in time from across the recording. The functionality for adding a single board would then be a way to backfill this automated approach with select board views.
Does this address the concern? Do you have other thoughts on approaches to take?
Thank you for the feedback!

First Commit:
Initial autopopulation capabilities implemented; Checklist of changes still to make:

[ ]  backfill data to hit grid target
[ ] GUI controls need to be added to control target/threshold
[ ] buttons need to be disabled/enabled to block user from:

[ ] changing frame index mid-action and
[ ] calibrating more than one camera at once (which seems crashes the GUI).

@rgov ,
I'm about to merge back in this autopopulate capability:
https://youtu.be/b-NgQqZDyjA
The GUI is getting a bit cluttered at this point, but I'm holding off on trying to make things pretty until they stabilize.
I appreciate the feedback. Things that seem obvious in retrospect are hard to see without an extra set of eyes. If you have any other thoughts (or find some bug/crashing with the new behavior), please just let me know.

Thanks for the quick turnaround! I ended up writing my own stereo calibration script for my data set in which I did automatic CharuCo corner extraction from all the frames. I did note a few things:

If all the corners are colinear, there is an inscrutable assertion failure in OpenCV. I filed the bug at https://github.com/opencv/opencv/issues/24676 and in the comments have a workaround for rejecting colinear corners.

The intrinsic calibration for one of my cameras appeared to work but then always led to a very bad extrinsic calibration. I am not sure the root cause ... I ended up working around it by simply copying the other camera's intrinsics (since they ought to be very similar).
The stereoCalibrateExtended() function, if you're using it, can return the per-input error so it could possibly be used to reject frames that are outliers from the others... Or possibly you would do the calibration yourself using more sophisticated methods that work in multicamera setups.

@rgov ,
I hope that pyxy3d can still be some use for your project even after rolling your own solution. If there is some specific workflow that would be valuable, just let me know and I can reflect on how straightforward it might be to implement.
Regarding #1, does that happen if you have multiple frames going into the calibration and only one is collinear? It makes sense to me that changing the IDs but not the Corners would impact it. I'm guessing that under the hood it is looking for collinearity of the corners in a board frame-of-reference, so if you can grab an ID from one row up then it passes some internal check based on the board definition.
Regarding #2, I'm curious: were you verifying your intrinsic calibration visually? Early on when I started messing with this stuff I realized how easy it was to get a low RMSE on an intrinsic calibration, but it could still be horribly overfit to where the board samples happen to be drawn. If the stereocalibration tries to build off of that then it can get painted into a corner.
Just for context, pyxy3d uses stereocalibrate only to construct an initial estimate of where the cameras are relative to each other, so it doesn't need to be super precise. This initial estimate feeds into the bundle adjustment process where it is "tightened up" by a SciPy optimization to minimize the reprojection error across all cameras. If you're curious, this is the core of it:
https://github.com/mprib/pyxy3d/blob/62ba20308e54e8c2b23668854a3db46201947fb1/pyxy3d/calibration/capture_volume/capture_volume.py#L90C11-L90C11
Apologies if I'm explaining things that you already know!


Added option to automatically navigate the current view to the last found result

Heya :)
I've also added an option (defaults to false) to navigate to the last found result in the current active view automatically.

Good feature, I have modified it a little bit to go to the closest result rather than the last one. Thank you for the contribution.
related commit


[2.0] Add support for official ElasticSearch client

This is a BC change, so can be merged in future 2.0 version.
Changes:

Renamed legacy ElasticSearchHandler to ElasticaHandler to avoid name collision
Added new ElasticSearchHandler and ElasticSearchFormatter
Added tests
Updated upgrade guide
Updated Travis file according to https://docs.travis-ci.com/user/languages/php#Enabling-preinstalled-PHP-extensions

Related issue #901

Build is failing for nightly PHP: "php: symbol lookup error: /home/travis/.phpenv/versions/master/lib/php/extensions/no-debug-zts-20170718/mongodb.so: undefined symbol: ZEND_HASH_GET_APPLY_COUNT"
Looks like its Travis issue

Please remove the .travis.yml changes, they don't belong in this PR.

Left some comments about the Elasticsearch portions, but generally the ES bits LGTM!
Minor nit, I didn't bother commenting on them in the code since it doesn't really matter, but it's Elasticsearch not ElasticSearch or Elastic Search. :)

@polyfractal thank you very much for the valuable review! I will update code considering your notes

Any chance of getting this merged?

The question is for @Seldaek
Btw, today/tomorrow I will resolve current conflicts

@Seldaek The PR introduces a dependency on 3rd party library. In another https://github.com/Seldaek/monolog/pull/1033 you said it is "no go".
Could you please elaborate on this?

Please merge this

@akalongman Perhaps add timeout on index() to the possible options? and set a default "low" value?


Keep a log of issues encountered (i.e., errors) and past solutions

We've encountered similar or same issues that manifest in a similar/same way. Would be handy to keep a quick lookup log of issues and solutions to potentially save debugging time.

Some examples:
Missing system global variable definitions
A variable should be defined in std_defs.txt but isn't. This has happened a few times with various glibc things. A key thing to understand about ELF symbols is that a given symbol can have available backing storage in all executable objects, but at runtime, only one such storage location is chosen. This problem recently came up when lifting /bin/ls. /bin/ls uses getopt and therefore depends on the global optind, opterr, and optopt globals.
The backing storage for these three globals is in the .bss section of the /bin/ls binary, and so they are lifted as 0. At some point during dynamic linking, the first available backing storage for these variables is meant to be initialized to 1. The lifting process did not maintain the symbol names, and so a different version of the variable was initialized. The lifted bitcode operated on the lifted version, which was not initialized correctly, and so it led to the command-line arguments not being correctly parsed, and subsequent segfailts during normal operation.
Lifting non-PIE binaries and recompiling them as PIE binaries
Some compilers will emit 32-bit addresses, even in 64-bit code, because the address of some code or data will fit into 32 bits. If the lifted code is compiled as a PIE binary, then those 32-bits will not be able to hold the fully loaded address.

We now have this as the ?common errors" documents.


data-generation-execution

Hello. Thank you for providing the data generation code.
We'd like to use this code too, so could you tell us how to execute it?

We updated README.md for our code execution!
Please see below.
https://github.com/addb-swstarlab/RocksDB-data-generation/pull/8#issue-1735636800


3.4.0 packaging regression: CUDA libs packed with the main JAR

So the no-classifier JAR still seems to bundle the CUDA libs for some reason:

So it means that the official 3.4.0 JAR might not work on non-CUDA environments :(

I think the reason is this part of the earlier release workflow code:
      - uses: actions/download-artifact@v3
        with:
          name: artifacts
          path: ${{ github.workspace }}/src/main/resources/de/kherud/llama/
      - uses: actions/download-artifact@v3
        with:
          name: artifacts
          path: ${{ github.workspace }}/src/main/resources_linux_cuda/de/kherud/llama/

Both CPU and GPU artifacts used the same name, and the latter probably overwrote the former silently. Coincidentally, I just upgraded the workflow to use newer versions for the upload and download actions, which should fix the problem, I think. I'll create a second release in a second.

Version 3.4.1 seems to be correct now:

Good catch of the issue, though!


add generated cpp files & use them in setup.py if no cython installed

This PR includes the generated cpp files and fixes setup.py to work if cython is not installed. And removes the .gitignore for the generated cpp files.
fyi @kimwalisch

@kimwalisch - It looks like Pandas, a popular data anlysis library for python, has cython as an optional buildtime dependency, much like @hickford was doing here.
I see what you're talking about Cython versions. It looks like Pandas handles this by checking for a minimum version of Cython.
https://github.com/pydata/pandas/blob/master/setup.py#L21-L27
We can probably similarly be "inspired" by them (i.e. copy the relevant bits).

Here's another setup.py that uses cython optionally https://github.com/datastax/python-driver/blob/master/setup.py

add generated cpp files

Is my understand correct that we can only get rid of the cython depedency if we add the generated cpp  files to the repo?
Because from my GNU Autotools (C/C++ build system) experience one should never add the files generated in the ./configure step to the repo! Hence I think me must at all cost avoid adding the generated _primesieve.cpp and _generate.cpp files to the primesieve-python repo!?
Maybe the better solution would be to make cython a prerequisite for installing primesieve-python, is it really that important that primesieve-python does not depend on cython? Is it possible to automatically install cython if the user does pip install primesieve and he has not installed cython?

@kimwalisch  I have my fair share of autogen / configure badges of honor^h^h^h^h^h horror and I share your general aversion to including auto generated code. However, if you look at the generated files, they are aware of many build environments and IFDEF'ed all over the place.
This section of Cython says http://docs.cython.org/src/reference/compilation.html#distributing-cython-modules says:

It is strongly recommended that you distribute the generated .c files as well as your Cython sources, so that users can install your module without needing to have Cython available.

Consider the generated code as c/c++ modules that should compile on many platforms. Instead of having been typed in by people, we just had a computer type them in. But they should work on lots of platform.
That said, this package is really just providing Python bindings to functions exported from  primesieve library. Using Cython to generate those bindings is cute/clever and makes doing it really straightforward. I tend to the side that says - requiring Cython just to install this library feels a bit much and if it is possible to support people without Cython, we should.

@pstoll Awesome answer! I will merge your changes when I am back home.

As Kim says, the generated cpp don't belong in the repo. They aren't source code. So long as they are distributed in the package people can install without Cython.
See https://github.com/hickford/primesieve-python/blob/v0.1.2/setup.py#L12 for how this works


feat: create pool and create/remove liquidity position for v3

Added V3 features:

create/get v3 pool instance
create v3 liquidity position
remove v3 liquidity position

Very nice! Let me know when you want a review :)
Any idea why the CI chokes on the CLI tests?

Btw, do you think this would be easy to do at the same time? https://github.com/uniswap-python/uniswap-python/issues/254

Btw, do you think this would be easy to do at the same time? #254

Long story short:
https://stackoverflow.com/questions/71814845/how-to-calculate-uniswap-v3-pools-total-value-locked-tvl-on-chain
Due to personal reasons, I can't implement that myself at the moment. However, it seems to be not too complicated to add.

Btw, do you think this would be easy to do at the same time? #254

Long story short: https://stackoverflow.com/questions/71814845/how-to-calculate-uniswap-v3-pools-total-value-locked-tvl-on-chain Due to personal reasons, I can't implement that myself at the moment. However, it seems to be not too complicated to add.

Sure thing, seems pretty straightforward to implement

Very nice! Let me know when you want a review :)
Any idea why the CI chokes on the CLI tests?

Took a peek at the logs for the CI, seems like provider env variable may not have set

Ah yeah, I just now remember that secrets (like out PROVIDER credentials) aren't set on PRs.
Might get a second key, obfuscate it slightly, and use it for PRs.

Should have fixed it in https://github.com/uniswap-python/uniswap-python/pull/266, feel free to rebase so the tests get green :)

Now that we have all green :) Going to write tests for the methods that address #254 and should be good for a review

@ErikBjare this is now ready for review :)
wrt to #254 I've added a function get_tvl_in_pool() and have commented the logic (some weird hacks with the tickBitmap to get it working), but let me know if any questions.

Amazing job, that's impressive. The only thing I just can't get why
def get_liquidity_positions(self) -> List[int]:
instead of
def get_liquidity_positions(address: AddressLike) -> List[int]:
IMO if we have a scenario when this function has the reason to be implemented for our address, then we also have a similar/same scenario for others.

Fair point, I agree. I'll make that change quick.

@ErikBjare just take a look at this grandeur!

Merged a squashed version in https://github.com/uniswap-python/uniswap-python/commit/9edfbaa5354b2d2df70cccb06c19a4a4c33fc64d (which was messy), now trying to address some issues during rebase.

Alright, I can't manage to get this PR to get marked as "Merged" for some reason (keeps complaining about some conflict?).
Changes are merged though, fixing some lingering issues on master.
Thanks again for contributing @KeremP! :heart:

Sounds good! Thanks for the review :)

What is the difference between mint_liquidity and mint_position here? I tried to create positions using both. mint_liquidity was successful but mint_position didn't seem to create a position though the function execution was successful.


Smoke test failure: checks if 'Go to Line' works if called from the status bar

Smoke test failure
https://monacotools.visualstudio.com/Monaco/_build/results?buildId=69344&view=logs&j=a5e52b91-c83f-5429-4a68-c246fc63a4f7&t=05af1eac-c39c-5c75-1e3f-dd115ec8907d

Seems to have stabilized.


#7007: Warn that decorated methods might not be bindable.

This is the documentation-only part of #7007, and doesn't include improvements to exception messages to reinforce this message.

Now ready for review/merge. The failing PEP8 test is unrelated to my changes.

Also, it would be nice if you add something similar to the docs of WeakMethod and maybe to https://github.com/kivy/kivy/blob/master/kivy/_event.pyx#L409.

@Julian-O Any feedback on this? Thanks

Please merge this and add the additional commit to document also in WeakMethod and _event and changing the descriptions


Support tail-call operator

According to https://github.com/WebAssembly/tail-call/blob/master/proposals/tail-call/Overview.md,
we want to add support for return_call and return_call_indirect.
It is related to https://bugs.chromium.org/p/chromium/issues/detail?id=1076774

@bmeurer @EricSL @ktran PTAL :)

LGTM, thanks Z!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add credit_mutuel

preview:

base logo:

Sorry for the long delay, it should be fixed now 😅

Hi sorry again for the late fix 😅
I've edited the svg to fit the 160px size! Only thing I can't understand is when exporting to a drawable xml, it change the stroke size to 11.17px ? It may be due to the fact that I'm using a transform matrix to resize the svg 🤷♂


made purdue basketball team stat app

Description
I made an app that shows the team standings of the Purdue men's basketball team.
This is my first time using Starlark and commiting code to this repository, so I am completely open to constructive critisism.
I am using an api, and I require users of my app to include their own api_key upon using my app.

Copilot

copilot:all

I have read the CLA Document and I hereby sign the CLA

Hey @Griffinov22, and thanks for the PR!
The files you've added in this PR are not placed entirely correctly in the file structure of the repo. Apps need to go under apps/<name>/. You can look at the other directories in there to get a sense of how it all works.
Since you asked for feedback, I thought I'd also mention that the embedded graphic could be scaled down ahead of time to keep the size of the script down a bit. Ideally, you scale it down to the exact dimension you're using in the app, so it doesn't have to be rescaled at runtime, if that makes sense. To cut the size even further, you could look into stripping any EXIF data from the file, e.g. using exiftool -all= thatfile.png.

Thank you @matslina for your help. I fixed the image, and made getting the year dynamic, and corrected my app folder mistake. Do I need to make a new PR?

I made changes that have passed all the requirements up until someone merges. Do I just need to wait again, or is there more steps I need to do to prepare my PR?


Upgrade to AWS SDK v2

Upgrade to AWS SDK 2 #253

Thank you for your contribution. That's merged and polished now.


Address issue where 861 ETL fails w/o all years of data

Proposing a minor change to how balancing authority fixes are applied so that not all years of data are required to make the fixes work. This should address #828.
In the existing process, when I try do something like this:
pudl_out = PudlTabl(...)
pudl_out.etl_eia861(eia861_settings=Eia861Settings(years=sorted(range(2005, 2021)))

I get an error in these lines of code because there are values in BA_ID_NAME_FIXES.index that are not in df.index.
https://github.com/catalyst-cooperative/pudl/blob/6e71bfa1a848f67ffeb1f10a161af01db75073b8/src/pudl/transform/eia861.py#L859-L861
I played around a little bit and I think replacing that with the merge I suggest here addresses the problem and all the tests appear to pass when I run them locally.
Given the small size of the change, I didn't think any additional documentation beyond the brief comment was needed but happy to add if that would be helpful.

It looks like the same failing test across the three ci-tests. It's tests.integration.output_test.test_plant_parts_eia_filled and specifically when it goes to get fuel cost data from the EIA API and gets a Forbidden error. Since I don't think my changes affected what is exercised by this test, I wonder if this has to do with this being a PR from a fork.
I looked a little deeper and noticed that the beginning of 'Run PyTest with Tox' looks different on this PR than #1656.  (See above for error)
GLOB sdist-make: /home/runner/work/pudl/pudl/setup.py

Whereas on on #1656 it looks like this:
Run conda run -n pudl-test tox
  conda run -n pudl-test tox
  shell: /usr/bin/bash -e {0}
  env:
    CONDA_PKGS_DIR: /home/runner/conda_pkgs_dir
    API_KEY_EIA: ***
GLOB sdist-make: /home/runner/work/pudl/pudl/setup.py

The missing API_KEY_EIA would explain the failure, though I'm not sure how to fix this issue or if I even can. On the ci-notify action, this PR is missing the SLACK_WEBHOOK_URL which could be why that fails.

Hey @arengel, yes, the tests are failing on here because you don't have access to the repository github secrets as an outside contributor.
The existing tests only check whether all years can be processed together, so while this change seems like it's fine, and doesn't break the tests, we don't really know how the system will behave when only partial sets of years are processed.  In general within PUDL we've been ensuring that both all the data can be processed, and that just the most recent year of data can be processed (since we typically just process the most recent year in the integration tests). Though in the case of the EIA-861 we've only tested that all of the data can be processed.
I pulled down this branch and experimented with a few different combinations of years, and unsurprisingly most combinations failed.  Just doing 2020, or 2019-2020, or 2005-2020 all failed for reasons other than the problem fixed by this PR, sometimes in other tables. One could also restrict the list of tables that are being processed, but the BA table in particular draws BA IDs and state associations from across a large number of tables to construct the final table, so you won't get the same outputs necessarily if you don't process all of the tables.
Is there a reason why you're trying to avoid processing all of the years of data? On my machine doing all the EIA-861 tables for all the years takes ~3 minutes.

Thanks for taking a look at this @zaneselvans!  And also for taking the time to test and explain the implications of what I was proposing, I definitely have a better understanding of how PUDL works now.
To answer your question, I was looking for a way to centrally control the years that we get when pulling 861 data in a way similar to how setting start_date and end_date in PudlTabl controls the years we get for 860 or 923. So that part of it is more organization than anything else. The reason I was looking at this in first place had to do with simplifying the process of validating the way we combine a number of different 861 tables.
Given that the goal here is convenience and not performance, we don’t want to affect the processing at all. We certainly don’t want the output to change based on the set of years we pull out or the set of tables processed. It’s now clear that even if using Eia861Settings to set the years worked (as in didn’t raise errors) it’s not what we’re looking for. Instead we’ll select the years we want from the data we get from PudlTabl.
I’ll leave this PR open in case this change adds useful flexibility for PUDL but we definitely don’t need it for the Hub.

Thanks for taking a look at this @zaneselvans!  And also for taking the time to test and explain the implications of what I was proposing, I definitely have a better understanding of how PUDL works now.
To answer your question, I was looking for a way to centrally control the years that we get when pulling 861 data in a way similar to how setting start_date and end_date in PudlTabl controls the years we get for 860 or 923. So that part of it is more organization than anything else. The reason I was looking at this in first place had to do with simplifying the process of validating the way we combine a number of different 861 tables.
Given that the goal here is convenience and not performance, we don’t want to affect the processing at all. We certainly don’t want the output to change based on the set of years we pull out or the set of tables processed. It’s now clear that even if using Eia861Settings to set the years worked (as in didn’t raise errors) it’s not what we’re looking for. Instead we’ll select the years we want from the data we get from PudlTabl.
I’ll leave this PR open in case this change adds useful flexibility for PUDL but we definitely don’t need it for the Hub.

Yeah I think the right thing to do is just select the years  you want from the table after the whole thing is returned.
However, this PR still seems like an improvement -- there's no reason that the BA function should totally crash and burn just because some of the fixes are outside the range of years being processed, and I've tested that it produces exactly the same output as the current code when all years are processed, so I'm inclined to go ahead and merge it, even if it doesn't satisfy your original desire.


Broken pipe error

In Bash:
$ 'echo foo' > foo.nu

$ engine-q foo.nu
foo

$ RUST_BACKTRACE=1 engine-q foo.nu | false
thread 'main' panicked at 'failed printing to stdout: Broken pipe (os error 32)', library/std/src/io/stdio.rs:1193:9
stack backtrace:
   0: rust_begin_unwind
             at /rustc/59eed8a2aac0230a8b53e89d4e99d55912ba6b35/library/std/src/panicking.rs:517:5
   1: std::panicking::begin_panic_fmt
             at /rustc/59eed8a2aac0230a8b53e89d4e99d55912ba6b35/library/std/src/panicking.rs:460:5
   2: std::io::stdio::print_to
             at /rustc/59eed8a2aac0230a8b53e89d4e99d55912ba6b35/library/std/src/io/stdio.rs:1193:9
   3: std::io::stdio::_print
             at /rustc/59eed8a2aac0230a8b53e89d4e99d55912ba6b35/library/std/src/io/stdio.rs:1205:5
   4: engine_q::main
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

Seems like a similar issue to https://github.com/nushell/nushell/issues/4161

@kubouch I guess it's becase false didn't need input and exited before engine-q, so the pipe closed. In my opinion, there're two ways to handle it, handle SIGPIPE, or check whether file descriptor 1 is open. Which way is better, or some other ideas?

Is this still a problem @kubouch ?

I'm not on Linux at the moment, but I'll test it when I get there.

It still happens


Comment "add remove reason" added if comment removal reasons are disabled

This seems to be fixed?

I think this is fixed.


Feature Request: add option to choose language in options

The app by default chooses the language by the system language. It would be great to include a toggle in options to change it.

Latest Android version brings this as a system-wide option.
But still useful for iOS and Desktop.


feat: implement StringInsensitive

Make it possible to do a case insensitive string grab.

Fixed in https://github.com/a-h/parse/commit/e0812cc34a9eb21a76b6a4a56428e0a0f004facb


Remove node version 16 in package.json

Semantic-release/github requires node version >=18 so we shoudl remove version 16 in engines
https://github.com/janus-idp/backstage-plugins/blob/6be683865866fcfb49702db38049af455fb1c8c2/package.json#L5-7
<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version ">=18". Got "16.20.0"

Assign this one to me


Bug: It is possible to wrongly assign through a non-const ref to Unknown or IInspectable

Version
All versions
Summary
Currently all projected types inherit from winrt::Windows::Foundation::IUnknown or winrt::Windows::Foundation::IInspectable, which implies that you are able to assign a non-const lvalue reference to it when you have any non-const projected type. And since IUnknown can be assigned to anything, it allows assigning to even types other than the original.
Reproducible example
#include <winrt/Windows.Foundation.h>

int main()
{
    winrt::Windows::Foundation::Uri uri{L""};
    winrt::Windows::Foundation::IUnknown /* or IInspectable */& uriUnknown = uri;
    uriUnknown = winrt::Windows::Foundation::MemoryBuffer{ 0U }; // assigning to wrong type.
    uri.Domain(); // Undefined behaviour.
}

Godbolt repro: https://godbolt.org/z/GaEcrYjGP.
Expected behaviour
winrt::Windows::Foundation::IUnknown /* or IInspectable */& uriUnknown = uri; // error
Actual behaviour
winrt::Windows::Foundation::IUnknown /* or IInspectable */& uriUnknown = uri; // works
Additional comments
Provide a separate base class for projected types instead of using IUnknown and IInspectable, and have a conversion operator instead, just like other interfaces but there would be no need for QueryInterface.
This is a breaking change but code that relied on assigning non-const ref to IUnknown or IInspectable was probably incorrect and non-const works for most cases.

This does not warrant a breaking change. My suggestion is not to do that.

A similar issue exists for com_array. You can take a reference to array_view (due to inheritance) and assign it to any other array_view, leaking the original com_array data and, worse, calling CoTaskMemFree on the non-owning pointer set from array_view. An easy fix would be to inherit privately from array_view, disallowing binding non-const ref to array_view, and then 'using' every member function of array_view so you still get them.


Add valgrind memcheck hooks

Addresses #18
Some documentation I followed:

https://valgrind.org/docs/manual/mc-manual.html#mc-manual.mempools: documentation about hooks used to inform Valgrind about memory pools and allocation
https://valgrind.org/docs/manual/manual-core-adv.html#manual-core-adv.clientreq: documentation about where to #include valgrind headers
https://15experience.wordpress.com/2010/10/22/make-valgrind-track-your-mempool/: just some guy's blog idk

This is still a work-in-progress. I need to validate that everything is working---at first glance there appear to be some errors (perhaps false positives?) reported from existing test cases. But will be pretty epic if we can get this working.

Damnit it turns out there's a bug in Valgrind that hasn't been fixed for over a decade (!!!): https://bugs.kde.org/show_bug.cgi?id=233298
Some discussion here: https://stackoverflow.com/questions/30895018/custom-allocator-valgrind-shows-7-allocs-0-frees-no-leaks
I might just fork valgrind and apply the suggested patch

Are you saying that printf will allocate a buffer and not free it until the program terminates? Since you are calling printf here, it will allocate a buffer, and then before it has a chance to free its buffer, you will check for memory leaks, which will include printf's buffer. Is this correct?

Exactly


teuthology/suite: pass the base config to scripts

This PR is to support: https://github.com/ceph/ceph/pull/60386
Also, this PR fixes a bug: teuthology/tasks/kernel: default branch should be "distro". When the kernel dictionary is empty, the kernel task should assume distro.

Also, this PR fixes a bug: teuthology/tasks/kernel: default branch should be "distro". When the kernel dictionary is empty, the kernel task should assume distro.

Example: https://pulpito.ceph.com/pdonnell-2024-10-21_14:21:05-fs-wip-pdonnell-testing-20241019.005706-debug-wip-pdonnell-i66704-default-smithi/

https://pulpito.ceph.com/pdonnell-2024-10-23_02:34:42-fs-wip-pdonnell-testing-20241019.005706-debug-wip-pdonnell-i66704-default-smithi

@zmc good to go?


img load failure sometimes?

I used nanohttpd on my android phone, it works most of the time, but occasionally some of images could not display, even if refresh the page。
why? somebody met this？
I  guess its need to set maxProcessors issue like tomcat's server.xml? but I can't  find related configrations? Perhaps socketTimeout?  I have already set to 10 seconds when start server.
I just a android developer, be weak in backend server tech.
please help！  Thanks~

You'll have to provide a lot more detail than that if you want help on that.
As far as I know none of the users have encountered such issue before.

Thanks for your fast reply。
following picture one is right。But when I refresh(F5 or Command + R) the web page, will be occured image load failure，like the picture two and three。
I am absolutely sure the server was worked.

I dont know it's why?  please help.
Chrome Browser(latest version)
Samsung S6 android mobile(android 6.0)
nanohttpd web framework(master branch) + zepto.js

That's not what I meant. We need technical details, the web views don't matter.
Start with the following:

Log the http packets so we can see what the server is replying with.
Check that the resource requested is the correct one and exists on the server.
Run the server in debug and trace back the request to see where it fails and why.

And then we'll see what can be done next.
Note that this seems to be an issue on your side, and we're all glad to help you walk through it but you need to cooperate and do your own homework and debug tasks, the team's time is already too limited to perform our own current development tasks.

Hi @Mrzy
i'm trying to load background image or other icon from local image file but the page does not show anything,
used this code: "<img src="<image_path.jpg">
Can you share code how to show the image as your demo?
Thanks!


Add ruff to python project

Format all files according to ruff format from ARISO BioCloud/CSC
Add ruff to pipeline
Add ruff toml conf file

Don't know why Sonar is complaining. Locally (even bound to SonarCloud profile) this doesn't give an issue, so hard to reproduce. We didn't change this and we are not calling a possible None object at this point. It is also only in the piece you run locally. Considering ignoring these Sonar messages, what do you think?

I think sonar is mad about this:
specimen_data = specimen.get("attributes")  # the get operation here can return none
mas_job_record = map_to_annotation_event(specimen_data, result, str(uuid.uuid4())) # Sonar is mad here

If you want to make it happy you can check verify that specimen_data is not none, but it's just for local tests, so I'd mark these as a "false positive" in sonar and enjoy that green checkmark


VisualBoyAdvance: Revive 1.7.2, fix compiling and timing.

Nice!

FWIW we already have VBA-M (pkgs/misc/emulators/vbam), but I don't know how much they differ (I've used VBA-M before).

Ah, you're right.  Thanks!
Somehow I missed VBAM entirely.
Yes, it fixes most of VisualBoyAdvance's problems right there.
The one thing I noticed is that VBAM has moved from
SourceForge to github.  github has a 2015-12-12 version,
but SourceForge only goes up to 2015-06-10.
Just to see what it would do, I tried changing the VBAM
package to the latest version from github.
It compiled, but despite the ENABLE_WX being
the default configuration, it didn't make an executable
wxvbam
(If I figure out that, I can send another pull request.)
In any case, this pull request doesn't help.  So, I'm withdrawing it.


[Library Manager] Please add JoystickESP32.

Could you please add the JoystickESP32 library to the Library Manager?
https://github.com/Yigit-Abacioglu/JoysticESP32
It's a simple Joystick library

Hi @Yigit-Abacioglu. We have a new system for submitting libraries. The instructions are here:
https://github.com/arduino/library-registry#adding-a-library-to-library-manager
Please let me know if you have any questions.
Regards, Per


fix: upper case typo

Hey, I just made a Pull Request!
just a typo (Monitoring instead of monitoring)
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

Thanks!


Consultation about installation cost change

Hello, could I consult you what code can I use if I want to change the installation cost of all units while executing TEA? Thank you for your help.

I tried using Unit.purchase_cost =Unit.purchase_cost * 0.76, but AttributeError: can't set attribute 'purchase_cost' was reported.


Replace ReplacementManager

Having a global data structure for managing array replacement is not efficient and causes conflicts in multithreading/multiprocessing. It would be best to find a way to construct the necessary replacement dictionary from within expand_tensor from the arguments of the passed expression.

Yeah, this needs to be done ASAP. ReplacementManager was a bad idea from the start.


⚠ Bofrost.at has degraded performance

In e552032, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 3137 ms

Resolved: Bofrost.at performance has improved in f99904c.


Upgrade controller-tools to enable unservedversion

Upgrade controller-tools to v0.2.8 where kubebuilder:unservedversion
marker is landed. See https://github.com/kubernetes-sigs/controller-tools/issues/462
Tested by make test.
Part of #1035

/assign @adrianludwin
/assign @rjbez17

That was a surprisingly clean change!

Great!


Update server ip used by client to connect

What does this PR do?
Update server ip used by client to connect
Before submitting

[x] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[x] Did you make sure to update the documentation with your changes?
[Modify documentation]~Did you write any new necessary tests?~

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Closed as a cleaner patch is merges in #53


[Broken] Certain YouTube videos fail to download (HTTP Error 404: Not Found. Retrying fragment 1)

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running yt-dlp version 2021.09.25
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

$ yt-dlp -v 'https://www.youtube.com/watch?v=XuupsFx6lNk'
[debug] Command-line config: ['-v', 'https://www.youtube.com/watch?v=XuupsFx6lNk']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] yt-dlp version 2021.09.25
[debug] Git HEAD: 109f24fd6
[debug] Python version 3.9.7 (CPython 64bit) - macOS-11.6-arm64-arm-64bit
[debug] exe versions: ffmpeg 4.4, ffprobe 4.4, rtmpdump 2.4
[debug] Optional libraries: Crypto, keyring, mutagen, sqlite, websockets
[debug] Proxy map: {}
[debug] [youtube] Extracting URL: https://www.youtube.com/watch?v=XuupsFx6lNk
[youtube] XuupsFx6lNk: Downloading webpage
[youtube] XuupsFx6lNk: Downloading android player API JSON
[youtube] XuupsFx6lNk: Downloading MPD manifest
[youtube] XuupsFx6lNk: Downloading MPD manifest
[debug] Sort order given by extractor: quality, res, fps, source, codec:vp9.2, lang
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, source, vcodec:vp9.2(10), acodec, lang, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, id
[debug] Testing thumbnail 41
[debug] Unable to connect to thumbnail 41 URL "https://i.ytimg.com/vi_webp/XuupsFx6lNk/maxresdefault.webp" - HTTP Error 404: Not Found. Skipping...
[debug] Testing thumbnail 40
[debug] Unable to connect to thumbnail 40 URL "https://i.ytimg.com/vi/XuupsFx6lNk/maxresdefault.jpg" - HTTP Error 404: Not Found. Skipping...
[debug] Testing thumbnail 39
[debug] Unable to connect to thumbnail 39 URL "https://i.ytimg.com/vi_webp/XuupsFx6lNk/hq720.webp" - HTTP Error 404: Not Found. Skipping...
[debug] Testing thumbnail 38
[debug] Unable to connect to thumbnail 38 URL "https://i.ytimg.com/vi/XuupsFx6lNk/hq720.jpg" - HTTP Error 404: Not Found. Skipping...
[debug] Testing thumbnail 37
[debug] Unable to connect to thumbnail 37 URL "https://i.ytimg.com/vi_webp/XuupsFx6lNk/sddefault.webp" - HTTP Error 404: Not Found. Skipping...
[debug] Default format spec: bestvideo*+bestaudio/best
[info] XuupsFx6lNk: Downloading 1 format(s): 244+251
[debug] Invoking downloader on "https://manifest.googlevideo.com/api/manifest/dash/expire/1632750729/ei/KXhRYeTNI8uJkga8ho6QDA/ip/<IP_ADDRESS>/id/5eeba9b05c7a94d9/source/youtube/requiressl/yes/playback_host/r4---sn-o097znze.googlevideo.com/mh/nq/mm/31%2C29/mn/sn-o097znze%2Csn-n4v7snee/ms/au%2Crdu/mv/m/mvi/4/pl/20/hfr/all/as/fmp4_audio_clear%2Cwebm_audio_clear%2Cwebm2_audio_clear%2Cfmp4_sd_hd_clear%2Cwebm2_sd_hd_clear/initcwndbps/2071250/vprv/1/mt/1632728930/fvip/4/keepalive/yes/fexp/24001373%2C24007246/itag/0/sparams/expire%2Cei%2Cip%2Cid%2Csource%2Crequiressl%2Chfr%2Cas%2Cvprv%2Citag/sig/AOq0QJ8wRgIhAKuIyZoOjtrhtGGxHis7RXH3cc8rJ07g5mLn_2lq3rEvAiEAu3XHL2iRhAjNxUJQz_oDUB4DwOaDHjTCnvbuzk9bIXM%3D/lsparams/playback_host%2Cmh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps/lsig/AG3C_xAwRAIgfR-f5VbRymc1dR9Lii095noM_pwZqEnN-0QHDOV0Ld4CIDeTN3QqUJ2mL705efAXtYWkooB8RriAYGxcN3DlG8Vb"
[dashsegments] Total fragments: 4
[download] Destination: Pip luv [XuupsFx6lNk].f244.webm
[download] Got server HTTP error: HTTP Error 404: Not Found. Retrying fragment 1 (attempt 1 of 10) ...
[download] Got server HTTP error: HTTP Error 404: Not Found. Retrying fragment 1 (attempt 2 of 10) ...
[download] Got server HTTP error: HTTP Error 404: Not Found. Retrying fragment 1 (attempt 3 of 10) ...
[download] Got server HTTP error: HTTP Error 404: Not Found. Retrying fragment 1 (attempt 4 of 10) ...
[download] Got server HTTP error: HTTP Error 404: Not Found. Retrying fragment 1 (attempt 5 of 10) ...
[download] Got server HTTP error: HTTP Error 404: Not Found. Retrying fragment 1 (attempt 6 of 10) ...
[download] Got server HTTP error: HTTP Error 404: Not Found. Retrying fragment 1 (attempt 7 of 10) ...
[download] Got server HTTP error: HTTP Error 404: Not Found. Retrying fragment 1 (attempt 8 of 10) ...
[download] Got server HTTP error: HTTP Error 404: Not Found. Retrying fragment 1 (attempt 9 of 10) ...
[download] Got server HTTP error: HTTP Error 404: Not Found. Retrying fragment 1 (attempt 10 of 10) ...
ERROR: Giving up after 10 fragment retries
  File "/opt/homebrew/bin/yt-dlp", line 33, in <module>
    sys.exit(load_entry_point('yt-dlp==2021.9.25', 'console_scripts', 'yt-dlp')())
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/__init__.py", line 777, in main
    _real_main(argv)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/__init__.py", line 767, in _real_main
    retcode = ydl.download(all_urls)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 2925, in download
    res = self.extract_info(
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 1219, in extract_info
    return self.__extract_info(url, self.get_info_extractor(ie_key), download, extra_info, process)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 1227, in wrapper
    return func(self, *args, **kwargs)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 1265, in __extract_info
    return self.process_ie_result(ie_result, download, extra_info)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 1315, in process_ie_result
    ie_result = self.process_video_result(ie_result, download=download)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 2344, in process_video_result
    self.process_info(new_info)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 2812, in process_info
    partial_success, real_download = self.dl(fname, new_info)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 2489, in dl
    return fd.download(name, new_info, subtitle)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/downloader/common.py", line 408, in download
    return self.real_download(filename, info_dict), True
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/downloader/dash.py", line 63, in real_download
    return self.download_and_append_fragments(ctx, fragments_to_download, info_dict)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/downloader/fragment.py", line 477, in download_and_append_fragments
    frag_content, frag_index = download_fragment(fragment, ctx)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/downloader/fragment.py", line 440, in download_fragment
    self.report_error('Giving up after %s fragment retries' % fragment_retries)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/downloader/common.py", line 176, in report_error
    self.ydl.report_error(*args, **kargs)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 819, in report_error
    self.trouble(error_message, tb)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 775, in trouble
    tb_data = traceback.format_list(traceback.extract_stack())

ERROR: fragment 1 not found, unable to continue
  File "/opt/homebrew/bin/yt-dlp", line 33, in <module>
    sys.exit(load_entry_point('yt-dlp==2021.9.25', 'console_scripts', 'yt-dlp')())
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/__init__.py", line 777, in main
    _real_main(argv)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/__init__.py", line 767, in _real_main
    retcode = ydl.download(all_urls)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 2925, in download
    res = self.extract_info(
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 1219, in extract_info
    return self.__extract_info(url, self.get_info_extractor(ie_key), download, extra_info, process)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 1227, in wrapper
    return func(self, *args, **kwargs)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 1265, in __extract_info
    return self.process_ie_result(ie_result, download, extra_info)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 1315, in process_ie_result
    ie_result = self.process_video_result(ie_result, download=download)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 2344, in process_video_result
    self.process_info(new_info)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 2812, in process_info
    partial_success, real_download = self.dl(fname, new_info)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 2489, in dl
    return fd.download(name, new_info, subtitle)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/downloader/common.py", line 408, in download
    return self.real_download(filename, info_dict), True
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/downloader/dash.py", line 63, in real_download
    return self.download_and_append_fragments(ctx, fragments_to_download, info_dict)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/downloader/fragment.py", line 478, in download_and_append_fragments
    result = append_fragment(decrypt_fragment(fragment, frag_content), frag_index, ctx)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/downloader/fragment.py", line 451, in append_fragment
    self.report_error(
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/downloader/common.py", line 176, in report_error
    self.ydl.report_error(*args, **kargs)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 819, in report_error
    self.trouble(error_message, tb)
  File "/opt/homebrew/Cellar/yt-dlp/2021.9.25/libexec/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 775, in trouble
    tb_data = traceback.format_list(traceback.extract_stack())

[debug] Invoking downloader on "https://r4---sn-o097znze.googlevideo.com/videoplayback?expire=1632750730&ei=KnhRYb7uDIeVkgah2qCAAg&ip=<IP_ADDRESS>&id=o-ADf2_uQMgRnZMwUWvTcZLmA8bP3rElA9mMl-WHtPvkiR&itag=251&source=youtube&requiressl=yes&mh=nq&mm=31%2C29&mn=sn-o097znze%2Csn-n4v7snee&ms=au%2Crdu&mv=m&mvi=4&pl=20&initcwndbps=2137500&vprv=1&mime=audio%2Fwebm&gir=yes&clen=182135&dur=15.101&lmt=1563369864083152&mt=1632728681&fvip=4&keepalive=yes&fexp=24001373%2C24007246&c=ANDROID&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cvprv%2Cmime%2Cgir%2Cclen%2Cdur%2Clmt&sig=AOq0QJ8wRgIhANXHx7j4aA0r5VJQwUIyqWuEYv0kAMhbJhlxEwxYmq5NAiEAgNpqGv4O6TnXasWFI6BlTyvMR0vO6g_0-5yKIBFMGEI%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRAIgHKBItUABjOzyi4E5j9D37k8trbHhW2KeqqFEajTDVuwCIBttiNreQUGbOxRuxqidvpO22hOlGV2eMZLjsPhiq4u8"
[download] Destination: Pip luv [XuupsFx6lNk].f251.webm
[download] 100% of 177.87KiB in 00:00

Description

I've discovered a couple of YouTube videos that play fine in the browser, but are otherwise failing to download:

1
2

Possibly related to #502.  Main difference seems to be that downloading the videos linked above errors out within seconds, whereas the video in #502 fails extremely slowly.

Yeah this is the same as #502 afaik.

whereas the video in #502 fails extremely slowly.

From what I've seen, sometimes these particular formats fail on the first fragment, others later on or even eventually succeed.

Marking this as duplicate of #502. To the best of my knowledge, this is youtube's fault. If you try to force the format on browser, you get the same results. Nothing we can do to completely fix it. You can workaround the issue by adding a --check-formats to your command.
Feel free to continue any discussion in #502


Modified some type-strict configs and made it better.

Thank you very much! I will check the changes as soon as I find time.

Thank you very much! I will check the changes as soon as I find time.

Sure

I have checked your changes. strictNullChecks, strictFunctionTypes and strictPropertyInitialization are already set to true by strict, so explicit specification is unnecessary. I do not want to make the other changes either. Therefore, I am closing this issue. Sorry for your effort.


Update scala3-library to 3.1.3

Updates org.scala-lang:scala3-library from 3.1.2 to 3.1.3.
GitHub Release Notes - Version Diff - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang", artifactId = "scala3-library" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "org.scala-lang", artifactId = "scala3-library" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Codecov Report

Merging #99 (fcd7f71) into master (7c7e041) will not change coverage.
The diff coverage is n/a.

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

@@           Coverage Diff           @@
##           master      #99   +/-   ##
=======================================
  Coverage   29.55%   29.55%
=======================================
  Files          17       17
  Lines         247      247
  Branches        6        6
=======================================
  Hits           73       73
  Misses        174      174

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 7c7e041...f41cab8. Read the comment docs.


Feature request: On remove of provider the list of providers should be refreshed

Remove Hawkular middleware provider

Confirm

Hawkular Middleware Provider is still in list..

Refreshing page helps...

Should Angular refresh list after removing provider or should it refresh whole page?
Tested on Firefox

@miq-bot add_label providers/hawkular

This is not angular and should be handled by the normal button code in EmsCommon which then calls #deleteems for all the ems and also has code to refresh the screen.

I agree @pilhuhn this is not Angular code.
I think this might be happening for all providers, not just hawkular. The code in EmsCommon is also generic for this use case and not per provider. (I just briefly checked on my env, saw the same behavior in rhev providers)
@vprusa please also verify if this is beyond hawkular - if that's indeed the case, this issue is a general one and should not be classified as "hawkular".

@miq-bot add_label ui
@himdel  can you please also confirm the lack of auto refresh after deletion of provider is an existing behavior relevant for all providers?

@abonas it's not.. Because after deleting a provider anywhere else, the whole page gets reloaded, those parts are not angular yet.

@vprusa this feature (auto refresh) doesn't exist in miq except for 2 screens (and providers are not one of them), and it's not a specific bug to hawkular .
I suggest either changing the issue to a general missing feature request or closing the issue. (I can't modify the issue title as I didn't report it)
@miq-bot rm_label providers/hawkular

Is it just a timing thing?  The EMS deletion is put on the queue, so maybe it just needs time to be picked up?

@Fryguy isn't it a missing feature to "push" changes to UI after async task is over?

@miq-bot add_label feature

@Fryguy isn't it a missing feature to "push" changes to UI after async task is over?

Yes, but that is a general feature request not directly related to middleware or even summary screens.  As such, marking this as an enhancement . cc @dclarizio

Yes, but that is a general feature request not directly related to middleware or even summary screens. As such, marking this as an enhancement . cc @dclarizio

I agree, I also mentioned that it's missing in general above https://github.com/ManageIQ/manageiq/issues/8244#issuecomment-215054552

nothing changed and I think this is still relevant
@serenamarie125 perhaps this can be added to UX improvements? sounds like a basic capability

@miq-bot remove_label stale


feat(notification-block): implement block in React

General summary
Implement Notification Block in React

Screenshots

Check list

[x] (if has style) Add need: review-integration label
[ ] (if has textual documentation) Add need: review-doc label
[x] (if has code) Add need: review-frontend label
[x] (if has react) Check through the react dev check list
[ ] (if fix or feature) Add need: test label

Check out the contribution guide for general guidelines for submissions to the design system.

@matmkian The hover style is also implemented according to the context? (Inside Lumapps)

Seen with @matmkian. This component has to be included in the DS when some improvement of the notification Block will be designed. For now we can use the List & List Item component to migrate the notification


Update kindergarten

Fixes: https://github.com/JiriChara/vue-kindergarten/issues/17

Codecov Report

Merging #18 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #18   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           6      6
  Lines          47     47
=====================================
  Hits           47     47

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 d1c5367...9f382f0. Read the comment docs.


Add docs

This is my first attempt at writing method docs for a C-API. Let me know if I need to make changes.

Any thoughts or comments @ko1 ?

Sorry for ignoring it. I found it today. Thank you!

Thanks!


DEPRECATION WARNING: connection_config is deprecated and will be removed from Rails 6.2 (Use connection_db_config instead)

Checklist

[x] Upgrade Jets: Are you using the latest version of Jets? This allows Jets to fix issues fast. There's a jets upgrade command that makes this a simple task. There's also an Upgrading Guide: http://rubyonjets.com/docs/upgrading/
[x] Reproducibility: Are you reporting a bug others will be able to reproduce and not asking a question. If you're unsure or want to ask a question, do so on https://community.rubyonjets.com
[ ] Code sample: Have you put together a code sample to reproduce the issue and make it available?  Code samples help speed up fixes dramatically. If it's an easily reproducible issue, then code samples are not needed. If you're unsure, please include a code sample.

My Environment

Software
Version

Operating System
MacOs 11.2.1

Jets
3.0.3

Ruby
2.5.7

Expected Behaviour

run bundle exec rspec without DEPRECATION WARNING
Current Behavior
~/jets/test(master) » bundle exec rspec                                                                                jdaviderb@jorges-MacBook-Pro
DEPRECATION WARNING: connection_config is deprecated and will be removed from Rails 6.2 (Use connection_db_config instead) (called from <top (required)> at /Users/jdaviderb/jets/test/spec/spec_helper.rb:13)
DEPRECATION WARNING: DatabaseConfig#config will be removed in 6.2.0 in favor of DatabaseConfigurations#configuration_hash which returns a hash with symbol keys (called from <top (required)> at /Users/jdaviderb/jets/test/spec/spec_helper.rb:13)
DEPRECATION WARNING: spec_name is deprecated and will be removed from Rails 6.2 (please use name instead) (called from <top (required)> at /Users/jdaviderb/jets/test/spec/spec_helper.rb:13)
DEPRECATION WARNING: `environment` and `name` will be removed as parameters in 6.2.0, you may now pass an ActiveRecord::DatabaseConfigurations::DatabaseConfig as `configuration` instead. (called from <top (required)> at /Users/jdaviderb/jets/test/spec/spec_helper.rb:13)

Step-by-step reproduction instructions
jets new test
cd test
bundle exec rspec

Code Sample
NO

Solution Suggestion

Hello @tongueroo this problem is solved in this PR https://github.com/boltops-tools/jets/pull/539


Is the lack of 2020 in raw intentional?

There is still no 2020 folder with thursdays game in it in
https://github.com/guga31bb/nflfastR-data/tree/master/raw
Is that intentional and we can't bank on that, or just a new-process hiccup?

They will be saved here now:
https://github.com/guga31bb/nflfastR-raw/tree/master/raw/2020

Great! Thanks!


Streamline bloat reports, part 2 of 2

Remove the early attempt to aggregate FLASH vs RAM based on whether ELF segments are writable or not, which turned out not to be useful because several platforms mark flash segments as writable. Now that explicit section groups are in use (#33642), there is no additional value to segment aggregation even on platforms that mark them accurately.

Do we still want the individual sections in the reports, or only the FLASH and RAM summaries?


Fixing #84 base85 problem

https://github.com/Ciphey/Ciphey/issues/84

@all-contributors please add michalani for code

I approve. 👍


fix(staking,extension): skip checking DRep activity and locked stake rewards

Checklist

[x] JIRA
Fixes #1618

Testing
Please see the ticket description for ACs.

Allure Report
allure-report-publisher generated test report!

processReports: ❌ test report for e0ea34e1

passed
failed
skipped
flaky
total
result

Total
30
3
4
0
37
❌

Superseded by https://github.com/input-output-hk/lace/pull/1620


How can I access the health via an API ?

I am trying to show the health of our registered domain on our internal portal but I cannot seem to find the correct API URL for the same. Can someone please put me in right direction.
Regards,
Deepak

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

ID: d6347876-e729-1f2b-d94b-6b8e842d9379
Version Independent ID: aed5ad45-5be6-4d34-4c00-79ed913f26bd
Content: Check the health of Azure Active Directory Domain Services
Content Source: articles/active-directory-domain-services/check-health.md
Service: active-directory
Sub-service: domain-services
GitHub Login: @iainfoulds
Microsoft Alias: iainfou

@deepaksharma-507
Thanks for your feedback! We will investigate and update as appropriate.

@deepaksharma-507 Hi, unfortunately Azure AD DS doesn't have any API calls for that currently. You can look at the current Graph API available here to see if something else can work for you. I suggest creating a feature request here for API calls you desire. Please let me know if I can assist you with anything else!

Hi James, Thanks for your reply. I found this link, seems like this is what I need, but I am not able to authenticate properly. I am getting this error.

The client 'XXXX' with object id 'XXX' does not have authorization to perform action 'Microsoft.Resources/subscriptions/read' over scope '/subscriptions/XXX' or the scope is invalid. If access was recently granted, please refresh your credentials.


It is not NULL SAFETY

I have setup it in my project and getting error in console:
The library 'package:flutter_multiselect/flutter_multiselect.dart' is legacy, and should not be imported into a null safe library.
Try migrating the imported library.

Which means it does not support NULL SAFETY project. Kindly share null safety package link. Thanks a lot.

Null Safety PR: https://github.com/hiddencaliber/flutter_multiselect/pull/33

Waiting for author review and merge.


bug: repro for #4210

moc chokes compiling
let x = class () {}

Just doing some repo cleanup for better focussing on active work:
Closing PR as it seems inactive for a longer time. Please re-open if it is still relevant and after it has been updated with latest master changes.


Add TypeScript support

Is your feature request related to a problem? Please describe.
We currently use TypeScript for our React Native projects and are in the process of integrating a project with our event tracking. We've been able to get Amplitude configured but also maintain an internal mirror of events using Snowplow. We'd like to use this library to wire up our app to our internal event log.
Describe the solution you'd like
Adding type definition files (.d.ts files) to the repo to declare the signatures for the core tracker methods according to this section of the TypeScript documentation.
Describe alternatives you've considered
An alternative would be to add our own typings for snowplow in our repo, but this seems to be additive to the project in general. Another option is to create a separate typings package (ie. @types/snowplow-react-native-tracker) that users would install alongside this one if they wanted to use it in TypeScript.
Additional context
I'd be happy to contribute the .d.ts files if this is a PR the maintainers would accept.

Hi @rahulsmehta,
Thanks for raising. This would be a welcome contribution for sure!

That's great @colmsnowplow -- unfortunately we just checked and the library in its current state won't work with our app setup (Expo managed workflow) so we're going with a workaround using the nodejs snowplow tracker. Once we eject/are building the iOS/Android bundles separately, we'll revisit & definitely make this contribution.

That's a shame. I think Expo doesn't like native modules, so not a huge amount we can do with this project to make it compatible :(
Do let us know about your experiences/feedback using the Node.js tracker though! We are keen to give that one some love next year.

Will do @colmsnowplow -- we actually just got a pure JS/TypeScript implementation working that shims in some of the device platform properties etc. We're planning on open-sourcing the package in a couple weeks

`declare module '@snowplow/react-native-tracker' {
export interface TrackerProps {
// required
endpoint: string,
namespace: string,
appId: string,
// optional
method?: string,
protocol?: string,
platformContext?: boolean,
base64Encoded?: boolean,
applicationContext?: boolean,
lifecycleEvents?: boolean,
screenContext?: boolean,
sessionContext?: boolean,
foregroundTimeout?: number,
backgroundTimeout?: number,
checkInterval?: number,
installTracking?: boolean,
}
export interface SubjectData {
// optional
userId?: string,
screenWidth?: number,
screenHeight?: number,
colorDepth?: number,
timezone?: string,
language?: string,
ipAddress?: string,
useragent?: string,
networkUserId?: string,
domainUserId?: string,
viewportWidth?: number,
viewportHeight?: number,
}
export interface SelfDescribingEvent {
// required
'schema': string,
'data': Object,
}
export interface StructuredEvent {
// required
category: string,
action: string,
// optional
label?: string,
property?: string,
value?: number,
}
export interface ScreenViewEvent{
// required
screenName: string,
// optional
screenType?: string,
transitionType?: string,
}
export interface PageViewEvent {
// required
pageUrl: string,
//optional:
pageTitle?: string,
pageReferrer?: string,
}
export default class Tracker {
static initialize(props: TrackerProps): void
static setSubjectData(props: SubjectData): void;
static trackSelfDescribingEvent(): void;
static trackStructuredEvent(event: StructuredEvent, contexts?: Array): void;
static trackScreenViewEvent(event: ScreenViewEvent, contexts?: Array): void;
static trackPageViewEvent(event: PageViewEvent, contexts?: Array): void;
}
}
`
This is the .d.ts file I created

We are facing a block because the tracker cannot be integrated with Expo. Is there any workaround available?

Hey @zhconsultancyservices, unfortunately, we do not have a workaround for Expo ready. Likely the only approach to get a tracker running there would be to adopt the NodeJS tracker (see docs here) as suggested above. We plan to look into this and hope to come back an answer for you.


Date Matched Composition

With multiperiod walk-forward portfolio, we are able to grab the compositions over the multiple rebalancing periods. It would be great to have a date-matched composition i.e., portfolio weights, even if the weights repeat for multiple days, I can think of a dataframe days x assets, and then of course the NaN return dates will have zero weight for the respective asset. I have around 5 more suggestions to make, I will open a new issue for each. This is what the composition looks like currently.

maybe .portfolio_weights, .daily_weights something like that could distinguish from above.

I think it is better not to do it as a post-processing step due to countless errors that could creep up, however here is the post-processing code that I am currently working with.
import pandas as pd
import numpy as np
from datetime import timedelta

def create_daily_weights(df_select, model_composition, train_size=252, test_size=90):
    # Create a DataFrame with the same index and columns as df_select
    daily_weights = pd.DataFrame(index=df_select.index, columns=df_select.columns, dtype=float)

    # Calculate the start date for the first prediction
    start_date = df_select.index[train_size]

    # Generate rebalancing dates
    rebalancing_dates = [start_date]
    current_date = start_date
    while current_date < df_select.index[-1]:
        current_date += timedelta(days=test_size)
        if current_date <= df_select.index[-1]:
            rebalancing_dates.append(current_date)

    # Determine the number of periods to iterate over
    n_periods = min(len(rebalancing_dates), model_composition.shape[1])

    # Iterate through the rebalancing dates
    for i in range(n_periods):
        rebalance_date = rebalancing_dates[i]
        # Get weights for this period
        weights = model_composition.iloc[:, i]

        # Determine end date for this period
        if i == n_periods - 1:
            end_date = df_select.index[-1]
        else:
            end_date = rebalancing_dates[i+1] - timedelta(days=1)

        # Align the weights with df_select columns
        aligned_weights = weights.reindex(df_select.columns, fill_value=0)

        # Fill the weights for this period
        daily_weights.loc[rebalance_date:end_date] = aligned_weights.values

    # Forward fill any remaining NaN values
    daily_weights = daily_weights.ffill()

    # Ensure we only have weights for dates after the first prediction date
    daily_weights = daily_weights.loc[start_date:]

    return daily_weights

# Usage
# Assuming df_select and summary_dict["HRP"].composition are already defined
model_composition = summary_dict["HRP"].composition

## Same as CV
daily_weights = create_daily_weights(df_select, model_composition, train_size=252, test_size=90)

Yes could be useful, where do you think it could live? As a Portfolio method?

That would make the most sense yeah. Also haven't congragulated you on the package, its excellent. I am looking forward to seeing it evolve. I won't have much time to help from next week onwards due to the summer semester, but if any of my open issues are undone by September, I will take care of them.

Thank you very much. Noted, I'll start working on them.

Implementation done in dev branch here:
https://github.com/skfolio/skfolio/blob/d0d3b33f5ca3e27e45ac525c788ead119c83e92c/src/skfolio/portfolio/_portfolio.py#L670
https://github.com/skfolio/skfolio/blob/d0d3b33f5ca3e27e45ac525c788ead119c83e92c/src/skfolio/portfolio/_multi_period_portfolio.py#L560


Annotate fields and/or fragments with @live

Can't believe it has been nearly 5 years since this video https://www.youtube.com/watch?t=1090&v=ViXL0YQnioU where Lee Byron presents the idea of the @live directive.
{
  feed {
    stories {
      author { name }
      message
      likeCount @live
    }
  }
}

Really cool to follow your progress on live queries, but I've been wondering why you went with the approach of annotating the entire query vs being able to annotate the individual fields or fragments?
Are you planning, at some point, to support @live per field or fragment, or what is the rationale of annotating the whole query?
Keep up the great work!

I just noticed this issue right now, so my answer is a bit late 😅.
The problem right now is that there is not really a specification that allows sending granular partial patches to the client or re-execution partials of a operation. That’s why I choose that the InMemoryLiveQueryStore does not really interfere with anything related towards GraphQL execution. TL;DR: This was the easiest way I could implement it in my spare time without adding too much complexity.
I currently have only a few ideas on how it could work. E.g. it could be possible to hook into the process by using a subscribe handler that can be specified for a field. Another option would be to construct a adhoc Schema where the live field is added to the the Root type and a adhoc query for the given selection is built an executed. If a Schema is relay conform and has a Query.node field also only invalidated resources query subparts could be re-executed by building an adhoc query for that resource. There is still a lot of stuff to explore and I would appreciate any help.
In the App in which I am using this Rn I don’t really have the need to only have certain parts of the query defined as live as all the data should be as up2date as possible.
here some usage examples:
https://github.com/dungeon-revealer/dungeon-revealer/blob/a552f81bc7520bae900432fc2430eef627ac606f/src/splash-share-image.tsx#L13
https://github.com/dungeon-revealer/dungeon-revealer/blob/a552f81bc7520bae900432fc2430eef627ac606f/src/dm-area/token-info-aside/token-info-aside.tsx#L49

Some thoughts on how I wanna continue shaping graphql-live-query: https://dev.to/n1ru4l/graphql-live-queries-backed-by-the-relay-specification-3mlo


Test 3 Issue Linking

Description

Impact

Additional information

[ ] I'm willing to contribute to this feature

Planned issues from the component repositories
either direkt link in Feature destribtion

[ ] https://github.com/eclipse-tractusx/sig-release/issues/68
[ ] Issue Link from a componet repository  https://github.com/eclipse-tractusx/sig-release/issues/68
[ ] https://github.com/eclipse-tractusx/eclipse-tractusx.github.io/issues/416 componted info

or metinionig function as a comment in the issue itself
https://github.com/eclipse-tractusx/sig-release/issues/68#issuecomment-1748431222

related to https://github.com/eclipse-tractusx/sig-release/issues/68

Description
Impact
Additional information

[ ]  I'm willing to contribute to this feature

Planned issues from the component repositories
either direkt link in Feature destribtion

[ ]  Test 3 Issue Linking #68
[ ]  Issue Link from a componet repository  Test 3 Issue Linking #68

or metinionig function as a comment in the issue itself

Could we add more links to this feature

General Infomation
E.g. for a feature:

Everyone can create a feature.
Existing features can be edited by commiter or the creator of the feature
Feature contexts (e.g. Labels, Milstones, ...) can be changed by any contributor

Permission Rules guided by Eclipse Foundation:

Referenced at Eclipse Foundation Handbook, Chapter Access to GitHub Repositories

based on GitHub’s Repository roles for an organization

eclipse-tractusx/sig-release#68 will look like eclipse-tractusx/sig-release#68
in the same repo also possible to link directly with #68 will look like #68
from team / component repo it could be linked with eclipse-tractusx/sig-release#68 to reference the feature
Docs: